Question: Sourcing for tests in container

Hello, Thanks for putting this and your article together. It's valuable.
My company already has a fair amount of existing tooling.  I'm trying to modify what you do to match our workflow. We can't really use the ros extension as it's too opinionated. We build our own ros distro and create custom images so we pull our own base workspace in that way.
I have a question about how you might be sourcing after a build.
Right now we populate a .env with paths to src so it will look like
PYTHONPATH=/workspaces/our_project/service`

My trouble seems to be about what to do when you need to source ../setup.bash in particular we want test discovery to work
On the cli it's simple to colcon build && source ../setup.bash but I've never figured out a way to source once the container is running and the window loaded so that VSCode tooling picks it up.

You might want to try a postCreateCommand Or adding sourcing to the dockerfile itself
ARG WORKSPACE
RUN echo "if [ -f ${WORKSPACE}/install/setup.bash ]; then source ${WORKSPACE}/install/setup.bash; fi" >> /home/ros/.bashrc

Thanks for the response.
So in your workflows if you have a project with multi ROS packages say foo and foo_msgs, where foo depends on foo_msgs if you modify foo_msgs, you are then doing a full container rebuild? That's what I'm doing now but it always felt heavy. Alas, that seems to be solution most folks have converged on.

No, the container mounts the workspace. The container can be set up to
source setup.bash for terminals, or you can grab the env variables that
setup.bash makes and add them to your container.  If you change foo_msg, it
should grab the updated files/binaries since the paths are the same.
On Thu, Jun 6, 2024 at 7:43 AM Barrett Strausser @.***>
wrote:

Thanks for the response.
So in your workflows if you have a project with multi ROS packages say foo
and foo_msgs, where foo depends on foo_msgs if you modify foo_msgs, you
are then doing a full container rebuild? That's what I'm doing now but it
always felt heavy. Alas, that seems to be solution most folks have
converged on.
—
Reply to this email directly, view it on GitHub
https://github.com/athackst/vscode_ros2_workspace/issues/76#issuecomment-2152717717,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABOQ2FIRTUZYVBPMXQQXIG3ZGBYRTAVCNFSM6AAAAABHUXINCWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSG4YTONZRG4
.
You are receiving this because you commented.Message ID:
@.***>


auto-spacing possible?

Is it possible to set the spacing between the items (and on the edges), so that it will be even? Something like the weight used on LinearLayout?
Is it  STRATEGY_FILL_SPACE?
I tried to use :
    ChipsLayoutManager chipsLayoutManager = ChipsLayoutManager.newBuilder(getActivity())
            .setChildGravity(Gravity.CENTER)
            .setScrollingEnabled(true)
            .setRowStrategy(ChipsLayoutManager.STRATEGY_FILL_SPACE).withLastRow(true)
            .setMaxViewsInRow(3)
            .build();

But it doesn't put space at the edges of the items, only between them:

Never mind, it's STRATEGY_FILL_VIEW


AzCopy doesnt fail when we provide non-existent storage account in the command

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.1.2
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy make "https://2304disks.file.core.windows.net/test23"
What problem was encountered?
In the above command, I am specifying a storage account which does not exist. AzCopy does not throw an error and seems to be stuck in the retry loop. No errors are thrown in the command prompt. I do see the following getting logged in the event log multiple times -
==> REQUEST/RESPONSE (Try=4/14.9575ms, OpTime=11.3849529s) -- REQUEST ERROR
PUT https://2304disks.file.core.windows.net/test23?restype=share&timeout=901
User-Agent: [AzCopy/10.1.2 Azure-Storage/0.4.1 (go1.10.8; Windows_NT)]
X-Ms-Client-Request-Id: [15380a58-a38c-4f0c-7775-04b2a584fbf1]
X-Ms-Version: [2018-03-28]
ERROR:
-> github.com/Azure/azure-storage-azcopy/vendor/github.com/Azure/azure-pipeline-go/pipeline.NewError, /go/src/github.com/Azure/azure-storage-azcopy/vendor/github.com/Azure/azure-pipeline-go/pipeline/error.go:154
HTTP request failed
Put https://2304disks.file.core.windows.net/test23?restype=share&timeout=901: dial tcp: lookup 2304disks.file.core.windows.net: no such host
It will be great if we surface this error in the command prompt as well as the AzCopy Logs.
This seems to be related to the ask here - https://github.com/Azure/azure-storage-azcopy/issues/86
How can we reproduce the problem in the simplest way?
As shown above
Have you found a mitigation/solution?
No

Thanks @yagohel23 for this feedback! I've logged it to improve the user experience.

Hi, I encountered the same issue.
I am using the latest version 10.4.3

Non-existent storage account are surfaced as DNS errors, which are also used for temporary network issues, so we cannot distinguish them.
The retries will ultimately all fail, and give back an error.


Hide issue/PR list on backports

@oxinabox had a great idea here: https://github.com/JuliaRegistries/TagBot/issues/181#issuecomment-890524606

@christopher-dG @oxinabox Is this PR ready to merge?

Bump @christopher-dG


California (CDPH) and LA County HIV Pilot

Problem statement
California Dept of Public Health (CDPH) is working with LA County to do HIV testing and route the results via ELR.
The plan is to route the results through SimpleReport and to CDPH (via ReportStream), which will then route only those that are intended for LA County to them.
ReportStream's goal is to help facilitate this process and route the HIV data from SimpleReport, through the UP, and to California successfully.
What you need to know

There are weekly syncs with SR, LA County, and CDPH
...

To do

[ ] ...

Closing, work completed.


Last-Modified & ETag Does it really work for all feeds?

I have been using RSS feed since 3 years now. As per my observations most of the feed provided really dont worry about ETag at all.
Couple of times i have seen there is update to feed but it is not changed Last-Modified Tag.
$this->etag = $this->getHeader($response, 'ETag');
$this->last_modified = $this->getHeader($response, 'Last-Modified');

So my question here is does it really work 100% here.

Both ETag and LastModified are checked and if at least one does not match your previous result, it will mark the response as modified, see https://github.com/fguillot/picoFeed/blob/master/lib/PicoFeed/Client/Client.php#L286
TL;DR: all given values must match to make the response not modified

Thank you Bernhard.
What if feed provider dont use ETag and Last-Modified at all. In other words it is always blank.
If it is always return blank which will match in next call, and we dont get updated feeds.
Thank you, Raj

If neither are given it's always treated as modified

Thank you thats great.
Currently i am looking at the code and so far i found it is really written in very good way.
I would love to give it try.
Currently i am using fsocketopen wrapper class to fetch the contents
http://sourceforge.net/projects/snoopy/
It is great class works perfectly fine but uses fsocketopen. I read somewhere that curl is better than socket.
So thinking to give a try to your class. but not sure whether it will do everything what snoopy does.
I will do some testing in my test environment and see and put results here.
Thank you so much for your help.

To make it even better please have look at
http://framework.zend.com/manual/1.12/en/zend.feed.reader.html
https://github.com/zendframework/zend-feed
If we can import something from there to here

@rajjanorkar Am I right that your question is answered? If so, please close the ticket!
Regarding your zend-feed links: Please specify what is missing within picofeed but implemented in zend-feed. Just pasting the links doesn't help anybody.

Thanks @mkresin
closing it down. I may list few enhancement down the line.


APP-1604: Add blue badge variant

20230410_APP-1604: Add blue badge variant
Based on Billing Figma, we need a blue badge variant to show "due" invoice status.    Added in this PR.

You forgot to add package.json?

Whoopssssss, thanks for catching!
Upping prime version to 1.9 unless anyone has any objections.


Add CMake to build the project

Hi
I adapted the project to use CMake as a development alternative. There are probably scenarios where some specific configuration is missing (for example building for Android).
I tried to make it as easy as possible to use, where CMake is responsible for downloading dependencies and detecting new .c files in the src folder.
This version has been tested on Linux (gcc 13.2.0) and WSL2 (using Ubuntu 20.2). In all cases it was possible to compile and run the base project.

@lpg2709 Personally I'm not a fan of CMake build system but I know this could be useful for some users. Thanks for the addition.


Support for metrics for a service deployed with multiple JVMs

Applications can be deployed across multiple hosts/docker containers, but have the same service name. Currently, all the metrics are reported for the same service name without the ability to see how individual instances of the deployed service perform. I'd like to see a service instance identifier field (similar to hostname) to be able to distinguish between the metrics coming from different instances. Also, UI support in Kibana is needed.

This seems relevant: https://github.com/elastic/kibana/issues/43765
@felixbarny @eyalkoren is there anything to do in the agent for this?

adding @bansriyar
I don't see any fields here capturing something like hostname/instance name
https://www.elastic.co/guide/en/apm/agent/java/current/metrics.html

@michaelhyatt there's a standard set of metadata that gets added to all of the documents, including metrics: https://www.elastic.co/guide/en/apm/server/current/metadata-api.html. Take a look at the metrics docs in demo.elastic.co.

This is related to an earlier discussion leading to the Kibana issue linked above by @axw - if you look at the referred prototype in that same issue you'd see that service instances are already top-level citizens in the metrics landing page. The question is- where these JVM identifiers come from?
There was a discussion preceding this issue (see part of it starting in https://github.com/elastic/kibana/issues/43009#issuecomment-520212246). Essentially, it would be a combination of host-environment and/or container-id, and I think there was an agreement to add a configuration option for setting an instance ID manually wherever those are insufficient.
Note that this is also relevant for other data you may want to filter by JVM, so this also applies to the drop-down filters we want to have.

is there anything to do in the agent for this?

A possible future enhancement would be to "discover" a meaningful instance unique ID, like host:port, but I don't think this is something we are going to have an open issue for at the moment.
So, basically, all we need to do in this regard is add this manually configurable instance ID, if it is decided to go for it.
Therefore, I believe we can close this issue, unless @michaelhyatt feels it is not addressed?

I can see there is host.hostname and container.id at the moment, so it needs to be supported in Kibana to have an end to end working solution. Feel free to close it, as long as the instance.id enhancement is tracked elsewhere as a low priority ticket.

I actually think the instance.id should be high priority 🙂
Anyway, it first needs to be decided and then opened as an issue in the APM repo to be implemented by all agents and server.
So I am closing this one.


Slaves with broken replication should not be shown by which-cluster-osc-replicas

The status of the slaves do not appear to be checked ahead of being suggested as candidate controllers for a online schema change (OSC).
The logs contain a message showing that orchestrator is fully aware of the issue:
Aug 23 13:57:15 db1 orchestrator[27247]: 2018-08-23 13:57:15 WARNING executeCheckAndRecoverFunction: ignoring analysisEntry that has no action plan: FirstTierSlaveFailingToConnectToMaster; key: <IP_ADDRESS>:3306

The following relates to a 2-node cluster where the slave is no longer replicating.
When requesting which-cluster-osc-replicas the slave is returned:
$ orchestrator-client -c which-cluster-osc-replicas -a cluster1
<IP_ADDRESS>:3306

Here is the live replication status:
> show slave status\G
             Slave_IO_Running: Connecting
            Slave_SQL_Running: Yes
        Seconds_Behind_Master: NULL
                Last_IO_Error: error connecting to master 'repl@<IP_ADDRESS>:3306' - retry-time: 60  retries: 153
      Slave_SQL_Running_State: Slave has read all relay log; waiting for more updates
      Last_IO_Error_Timestamp: 180823 16:15:30
1 row in set (0.00 sec)

Only healthy slaves would be expected to be suggested as candidate controllers for an OSC.

Thank you for the PR and I apologize for this late response, I missed this issue.
There is some risk in only providing "good" replicas. Isn't there purpose to returning bad replicas, too?
Say you begin a migration and one replica breaks. Do you wish to proceed with migration or stop it? I can see the argument to both sides. Can you share your thoughts

friendly ping

Just to clarify the reasoning, imagine this is an OSC to free space on a fragmented table which would save the slaves that are still healthy, whilst some had already met their fate of "No space left on device". I certainly would not want a broken slave to be presented to me for this to be used as a control slave.
The cleanest and simplest solution to the real issue (not knowing a potential control slave is broken from the command) is something like:
$ orchestrator-client -c which-broken-replicas -a cluster1
<IP_ADDRESS>:3306

This could then be used in a number of ways, including selective filtering for the control slave. It also moves the logic of continue: yes|no to the consumer, instead of orchestrator.

@cezmunsta which-broken-replicas is a good addition.
Allow me to make things simpler even more, and add a which-cluster-osc-running-replicas command?


Improve readability of 15utility.js

Edited utils.glob and utils.like to improve readability. Proved comments to help reader traverse through code.
Also rename:
und ----> undefinedOrValue
n2u ——> nanToUndefined
Thank you for the time you are putting into AlaSQL!

also: please run npx yarn test locally to make sure tests are passing.

Im getting this error - any chance you can take a look?
Running tests on<EMAIL_ADDRESS>
  Environment detected: {
    "alasqlPath": "/home/runner/work/alasql/alasql/dist",
    "isBrowser": false,
    "isCordova": false,
    "isMeteor": false,
    "isMeteorClient": false,
    "isMeteorServer": false,
    "isNode": true,
    "isWebWorker": false
}

 Exception during run: ReferenceError: n2u is not defined
    at /home/runner/work/alasql/alasql/dist/alasql.fs.js:9030:56
    at Array.forEach (<anonymous>)
    at Select.yy.Select.compileSelect1 (/home/runner/work/alasql/alasql/dist/alasql.fs.js:[8](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:9)828:15)
    at Select.compile (/home/runner/work/alasql/alasql/dist/alasql.fs.js:7414:27)
    at Function.alasql.dexec (/home/runner/work/alasql/alasql/dist/alasql.fs.js:4765:38)
    at Function.alasql.exec (/home/runner/work/alasql/alasql/dist/alasql.fs.js:4720:17)
    at alasql (/home/runner/work/alasql/alasql/dist/alasql.fs.js:137:17)
    at /home/runner/work/alasql/alasql/test/test374.js:154:21
    at Array.forEach (<anonymous>)
    at Suite.<anonymous> (/home/runner/work/alasql/alasql/test/test374.js:140:5)
    at Object.create (/home/runner/work/alasql/alasql/node_modules/mocha/lib/interfaces/common.js:148:1[9](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:10))
    at context.describe.context.context (/home/runner/work/alasql/alasql/node_modules/mocha/lib/interfaces/bdd.js:42:27)
    at Object.<anonymous> (/home/runner/work/alasql/alasql/test/test374.js:7:1)
    at Module._compile (node:internal/modules/cjs/loader:[10](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:11)92:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:[11](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:12)21:10)
    at Module.load (node:internal/modules/cjs/loader:972:32)
    at Function.Module._load (node:internal/modules/cjs/loader:8[13](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:14):14)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:201:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:154:23)
    at async Loader.import (node:internal/modules/esm/loader:177:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:437:15)
    at async formattedImport (/home/runner/work/alasql/alasql/node_modules/mocha/lib/nodejs/esm-utils.js:9:[14](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:15))
    at async Object.exports.requireOrImport (/home/runner/work/alasql/alasql/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
    at async Object.exports.loadFilesAsync (/home/runner/work/alasql/alasql/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
    at async singleRun (/home/runner/work/alasql/alasql/node_modules/mocha/lib/cli/run-helpers.js:[15](https://github.com/AlaSQL/alasql/actions/runs/9898494687/job/27345441962?pr=1878#step:7:16)8:3)
    at async Object.exports.handler (/home/runner/work/alasql/alasql/node_modules/mocha/lib/cli/run.js:370:5)
error Command failed with exit code 1.

@JoshuaPoon1 ?


Using Vercel speed insights in Astro

Hey, I installed the @vercel/speed-insights package with npm and tried to add the component to my BaseHead.astro:
---
import SpeedInsights from "@vercel/speed-insights/astro";
---

<head>
  ...

  <SpeedInsights />
</head>

But when I add it, the import gets linted and says "Cannot find module '@vercel/speed-insights/astro' or its corresponding type declarations."
This is my package.json:
{
  "name": "...",
  "type": "module",
  "private": true,
  "scripts": {
    "dev": "astro dev",
    "build": "astro build",
    "preview": "astro preview",
    "astro": "astro",
  },
  "dependencies": {
    "@astrojs/vercel": "^7.1.1",
    "@vercel/speed-insights": "^1.0.9",
    "astro": "^4.2.8",
     ...
  },
}

How do I fix it? I followed Vercel's guide exactly and there's an issue.
I tried to see if I could import SpeedInsights from "@vercel/speed-insights";
And though the linting goes away on that line, I now get linted on the component line:
<SpeedInsights />
_Component 'SpeedInsights' is not a valid component.
If this is a Svelte or Vue component, it might have a syntax error that makes it impossible to parse.ts(2604)
(alias) const SpeedInsights: {
injectSpeedInsights: typeof injectSpeedInsights;
computeRoute: typeof computeRoute;
}
import SpeedInsights_
A friend on Discord says that looking at the package, it looks like it's not packaged correctly, the types point to a file that doesn't exist.

Hi, I'm having the same issue. It currently does not work with "@vercel/speed-insights": "^1.0.9" and "astro": "^4.3.6"


Create the mssql role

Write the role
Add test playbooks

rhel-8-y/ansible-2.8 — linux-system-roles-centos7-81-6n2pv@2021-03-12 14:04:13.120085 failed with this:

I'm assuming this failure is due to "too many login attempts in a short period of time" that the wait_for is trying to address - is there some way we can disable this check in ms sql server config, rather than trying to figure out how many wait_for we need and how long the waits should be?

rhel-8-y/ansible-2.8 — linux-system-roles-centos7-81-6n2pv@2021-03-12 14:04:13.120085 failed with this:
["Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : TCP Provider: Error code 0x2749.", "Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : Client unable to establish connection because an error was encountered during handshakes before login. Common causes include client attempting to connect to an unsupported version of SQL Server, server too busy to accept new connections or a resource limitation (memory or maximum allowed connections) on the server..", "Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : TCP Provider: Error code 0x68.", "Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : Client unable to establish connection.", "Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : Client unable to establish connection due to prelogin failure.", "Sqlcmd: Error: Microsoft ODBC Driver 17 for SQL Server : A network-related or instance-specific error has occurred while establishing a connection to SQL Server. Server is not found or not accessible. Check if instance name is correct and if SQL Server is configured to allow remote connections. For more information see SQL Server Books Online.."]

I'm assuming this failure is due to "too many login attempts in a short period of time" that the wait_for is trying to address - is there some way we can disable this check in ms sql server config, rather than trying to figure out how many wait_for we need and how long the waits should be?
I tried running the same query several times one after another and MSSQL did not return any errors. I then removed wait_for modules from tests to troubleshoot, and only 2 checks failed with the same issue. I pushed a couple of unrelated commits, and now all checks have passed. I will look at it again if the checks will keep on failing. Currently, I have found no way to increase the timeout within MSSQL.

looks good - please squash into a single commit - then we can get reviews from other team members
Please ask the MS guys what to do about the Error: Microsoft ODBC Driver 17 for SQL Server : TCP Provider: Error code 0x68 error

[citest]

[citest]

[citest]

Hi @pcahyna @nhosoi, please review this PR. Let me know if you need any information. Thank you

In this pr, vars/Fedora.yml is deleted. That is, for the deployment onto the fedora hosts, rhel-8 server and tools packages are installed? I see separate repositories exist for fedora:
https://packages.microsoft.com/fedora/.

Was vars/Fedora.yml deleted because the packages are identical to the ones for rhel-8?

In this pr, vars/Fedora.yml is deleted. That is, for the deployment onto the fedora hosts, rhel-8 server and tools packages are installed? I see separate repositories exist for fedora:
https://packages.microsoft.com/fedora/.

Was vars/Fedora.yml deleted because the packages are identical to the ones for rhel-8?

The mssql packages that this role needs are in a mssql-server-2019 repository, this repo exists for RHEL 7 and 8 but not for Fedora: https://packages.microsoft.com/rhel/8/mssql-server-2019/
The role definese vars for Fedora, RHEL 8 and CentOS 8 in vars/RedHat.yml, those OSes use the same repo and it works like that. Then, the role defines different vars for RhEL 7 and CentOS 7 in RedHat_7.yml and CentOS_7.yml.
I first tried using RHEL 8 repos for Fedora just for fun, but they worked so I used them. I do not expect Microsoft to create a separate repo for Fedora.

Any further objections?  Ready to merge?

Any further objections? Ready to merge?

I spotted several minor style issues, otherwise lgtm.

[citest]

[citest]

[citest pending]


Letter of Introduction text in shop overflows

https://www.twitch.tv/videos/515582548

The string is supplied by the game, not the randomizer, so the easiest way to prevent this would be to disallow the Letter from being placed in the shop.

probably also I could just find where the string is coming from


Support MapRoulette Templating and Mustache Tags

Adds support for Templating in Challenge Instructions and Mustache Tag Replacement 🥸

Fixes #1419
Fixes #1423

Here is the templating in action!
Here is the {{osmidenifier}} mustache tag being replaced with the way id.

Now when hovering over osm identifiers it ill highlight what the challenge is talking about. Clicking on it will take you to edit mode


Upgrade Apache Commons Collections to v3.2.2

Version 3.2.1 has a CVSS 10.0 vulnerability. That is the worst kind of
vulnerability that exists. By merely existing on the classpath, this
library causes the Java serialization parser for the entire JVM process
to go from being a state machine to a turing machine. A turing machine
with an exec() function!
https://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2015-8103
https://commons.apache.org/proper/commons-collections/security-reports.html
http://foxglovesecurity.com/2015/11/06/what-do-weblogic-websphere-jboss-jenkins-opennms-and-your-application-have-in-common-this-vulnerability/

This is a TPC-E application implementation which is not integrated with the top level build.
I checked the code but I could not find any usage of commons-collections. It would make more sense to remove it altogether.

BTW, does this issue exist with commons-collections 3.1? It is being referenced in gemfirexd/tools/src/ddlutils/pom.xml

Yes, this is an unused build file. Note that the bundled DdlUtils usage of commons-collections is already using v3.2.2 (see https://github.com/SnappyDataInc/snappy-store/blob/snappy/master/gemfirexd/tools/build.gradle). The pom.xmls can be removed. Will merge this change nevertheless.


Exceptions in @BeforeScenario should mark test as failed instead of skipped.

Hi folks,
I am currently dealing with a flaky test that causes an exception in my @BeforeScenario hook. This marks the test as skipped. I'd rather have them marked as failed (Given that actual pre-condition failed this seems more consistent for us).
I cannot find a way to do that :( Any help would be highly appreciated.

As I know Behat process exists with zero status code when skipped scenarios found. Is it your main problem?
If so, try to add --strict option. It will consider any not passed step as failed and process exit code will be non-zero.

Yes, --strict is the way to solve this


CII Gold: security review

Task
Goal
Conduct a security review.
Description
As documented in CII Best Practices: Gold > Security: Other security issues (tagged as security_review):

The project MUST have performed a security review within the last 5 years. This review MUST consider the security requirements and security boundary.

Success Criteria
The security document is updated with any findings and issues are opened for any security vulnerabilities identified.

Duplicate of #21


fix: when using useOutline in different components, it can only retri…

…eve the table of contents (toc) once #408

Description

Linked Issues

Copilot Descriptions
copilot:all

https://stackblitz.com/edit/valaxy-5uhbsr?file=pages%2Fposts%2Fhello-valaxy.md
Thanks for your contribution, based on the results of the discussion in the group, we don't think this is a bug.
It also works in the smallest demo.


chore: fix TODO by removing unused variable

As far as I can tell, this variable is unused. kit.svelte.dev works without it and all the tests pass without it

Looks like this ends up being SSRNode.file, the other usage of it is here with a similarly worded TODO.

awesome. thanks for pointing that out. I've removed those usages as well


Cert Details Inaccurate

Currently when adding the public cert data, you are supposed to copy all information between the "Begin Cert" and "End Cert" brackets. While this is correct, at this time the line breaks give you an error "Invalid Certificate data format".
If you remove all line breaks in the cert this issues is resolved. You can also still use Word Wrap to copy the content and it will work properly. But line breaks do not allow the cert to function at all.

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

ID: cb0224af-2059-f88c-bb14-926ef69bd835
Version Independent ID: c8859d92-dced-ecaa-a409-7af3cc064187
Content: Use Azure Virtual WAN to create a Point-to-Site connection to Azure
Content Source: articles/virtual-wan/virtual-wan-point-to-site-portal.md
Service: virtual-wan
GitHub Login: @anzaman
Microsoft Alias: alzam

@Iwifia , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@Iwifia , Appreciate your patience. SSL certificates by default have line breaks after 64 characters. Generally, after certificate exportation we copy the newly exported certificate as you mentioned above by removing all the line-breaks in the certificate. Not removing the line breaks violates the cert data format and will be issued "Invalid Certificate data format" error.
We are closing this issue for now.If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

This data needs to be added to the document, thus why I opened the issue. Nowhere in the documentation does it say that you need to remove the linebreaks for now or you will fail at importing the cert data.

@Iwifia , We have assigned this issue to the content author for updating the document accordingly. @anzaman , Could you please provide your insights on this ?


Support subcommands

list
search
install
upgrade
remove
pm-add
pm-remove

pmo also manages package managers called by pmo by gidoichi · Pull Request #5 · gidoichi/pmo


Add Abstract and APIs sections

Contributor Agreements
Please read the contributor agreements and if you agree, please click the checkbox below.

[x] I agree to the contributor agreements.

[!TIP]
Please follow the Quick TODO list to smoothly merge your PR.

Motivation & Description of the changes

Add Abstract and APIs sections to README.md to provide better information about packages

This PR will be resolved in https://github.com/optuna/optunahub-registry/pull/130


Removes implicit to_hash conversions in favor of explicit to_h conversions

I don't think we should be defining implicit hash conversion functions.  Here's an example where we pass a dataframe to a method and Ruby implicitly converts it to a hash and populates the keyword arguments (kargs) instead of the array arguments (args).  The changes in this commit correctly put the dataframe in args.
Unfortunately, this will likely break some client code that uses to_hash, but I think it's the right direction.  I can't think of use cases where having an implicit conversion is particularly helpful.
require 'daru'

df = Daru::DataFrame.new({ a: [1,2], b: [3,4] })

def mymeth(*args, **kargs)
  puts "args: #{args}"
  puts "kargs: #{kargs}"
end

mymeth(df)

args: []
kargs: {:a=>
#<Daru::Vector:70093300385580 @name = a @metadata = {} @size = 2 >
      a
  0   1
  1   2
, :b=>
#<Daru::Vector:70093300385140 @name = b @metadata = {} @size = 2 >
      b
  0   3
  1   4
}

@mrkn could you shed some light here? Do you think it's wise to remove implicit hash conversions? Could really use your insight into this given your deep experience with Ruby.
I'm OK with API changes since daru is a very young library and users expect minor API changes.

IMHO, dataframes aren't kind of association arrays, so it is appropriate to remove implicit to_hash conversions.

Alright then.


Add macros for error and status logging.

Depends on #401.
Add the following macros for logging purpose:

SPIRV_ASSERT
SPIRV_DEBUG
SPIRV_UNIMPLEMENTED
SPIRV_UNREACHABLE

The last two is always turned on, while the first two can only
be turned on in debug build.

I'll wait for a rebase on top of https://github.com/KhronosGroup/SPIRV-Tools/pull/401 before reviewing.

@dneto0: rebased.
Sorry this one doesn't have tests. These macros will abort the program, but not through abort(), which googletest death tests depend on. As for SPIRV_DEBUG, it's only for a single build type with extra options turned on.
But I do use these macros in latter PRs. So they should be fine.

-1

-1 have to #undef snprintf again at the end of the log.h (conditionally of course)

+2
Please squash


Keeper-Contracts fails if i specify kovan as network

I changed in start_ocean.sh
# Must be set to true for the first run, change it to "false" to avoid migrating the smart contracts on each run.
export DEPLOY_CONTRACTS="false"
# Specify which ethereum client to run or connect to: kovan, ganache, or ocean_poa_net_local
export KEEPER_NETWORK_NAME="kovan"

did ./start_ocean.sh --no-pleuston --latest
this will lead to:
/keeper-contracts/scripts/keeper.sh: line 7: parity: command not found
inside the keeper-contracts docker instance.

touches https://github.com/oceanprotocol/keeper-contracts/issues/172

Also facing this issue, would like to use docker-images with Kovan.

With the latest changes you can run docker-images connected with Kovan network using the option --local-kovan-node. Example:
bash ./start-ocean.sh --local-kovan-node


Fix being able to get soft locked by dropping cat pickup in air

(should be) fixed in 9dd803f06ee553a93c33dfc4cd6d34c756ea80e5


Fails to progress due to demangle exception

Hello,
I encountered an issue running on 10.2.2 where it would progress nicely for a while then encounter a RTTI entry that it could not demangle which prevented this statement in RttiModelWrapper.java from invoking due to there not being a namespace.
		SymbolIterator symbols = table.getChildren(getNamespace().getSymbol());
I managed to modify the code in several places to essentially skip or ignore any entry that failed in this way and the plugin was able to proceed and finish analysis.   Maybe just unlucky and had one mangled string that choked the Microsoft demangler but I wanted to raise this as it seems like being able to ignore these entries would be something nice to have in the release.
By the way I know the demangler is not part of your plugin but wanted to post the bad string here as maybe someone would have some idea why.
.?$BSTHashMap@GV?$NiPointer@V?$BSPathingTraversalRefCountedContainer@V?$BSTSmallArray@V?$NiPointer@VBSPathingTraversal@@@@$03@@@@@@$$YBSTDefaultScatterTable@@@@
Thanks for the plugin and making it open source.    If you would like to see how i bypassed i can share a diff.

For what it's worth I figured out that it was tripping on the $$Y template type in the decorated string.    I managed to fork the latest ghidra build and add that type to the demangler and patched this analyzer to work with it.   All works now after doing so.
That said I'm leaving this open as may still want to be able to handle in this plugin the case when there is a decorated name that ghidra cannot demangle.


Command line option for the custom directory path

To keep the gitea directory pristine, I currently symlink the custom directory to a directory outside which contains my config and various overrides.
I think it would be useful if one could specify the path to the custom directory, for example through a -c <custom-dir> switch  (which would enable me to skip the symlink step).

Just define the GITEA_CUSTOM environment variable to set this path. We won't break this behavior for minor releases.

Thanks, I wasn't aware of that variable. Probably a good idea to add supported environment variables to --help to raise awareness.

I am currently working on documenting these lesser known features of gitea.

I would too suggest to add it at least to the --help help, as this is the command most likely used first to find out which argument needs to be set, after trying --config= and seeing it does not work 😄


[Task]: Reduce min height of the banner

Task description
As part of the feedback provided by the UI / UX team, one of the suggestions is to reduce the min height of the banner to 280px
Requirements
Reduce min height of the banner to 280px
Acceptance criteria
N/A
Creation checklist

[X] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[X] I have linked the correct milestone and/or project

Fixed in https://github.com/iotaledger/explorer/pull/205.


Not working

{
"error": {
"code": "unauthenticated",
"message": "Exception of type 'Microsoft.Vroom.Exceptions.UnauthenticatedVroomException' was thrown."
}
}

I am also seeing the same error

{ "error": { "code": "unauthenticated", "message": "Exception of type 'Microsoft.Vroom.Exceptions.UnauthenticatedVroomException' was thrown." } }

New API has been implemented.

{ "error": { "code": "unauthenticated", "message": "Exception of type 'Microsoft.Vroom.Exceptions.UnauthenticatedVroomException' was thrown." } }

New API has been implemented.

i know, any solution?


Add pre-registered service documentation

Verified that @j3tan has signed the CLA. Thanks for the pull request!

+2


Add Node School's Git, SQLZoo and Free Code Camp

@vhf Rather than having FCC posted twice on the list, I think we should have a "Full Stack web development" section. While Full Stack is a bit of a buzz word, I think it can include the following:

Free Code Camp (new, JS focused)
Odin Project (existing, Ruby/Rails focused)

I agree on the idea @onebree but I'm not sure about the section name you suggest. I'll think about it.

Hi @vhf did you have time to think about this? Please let me know if you have any questions about Free Code Camp or the other resources I recommended adding.


Make CookieOption parameters optional

I only need to set an expiration on the cookies I'm going to set in my app, but I have to specify something for every option if I'm going to set the expiration.  Or am I missing something?

Hey @digital216
No, you are not missing anything. That is a design flaw which I will fix in the next release. Thanks for pointing that out.

Fixed with the v1.2.1.


Support Tablet Screen Sizes

Another potential change we can make in the future is support for tablets that are oriented vertically. If I switch to the dimensions of an iPad Air/Mini, the page is stuck on the calendar and doesn't show the Calendar and Search tabs on the top.

Otherwise, this looks good to go!

Originally posted by @Douglas-Hong in https://github.com/icssc-projects/AntAlmanac/pull/247#pullrequestreview-885483583

Quick and simple solution for now could be to just make the min-width for the tab bar fit tablets as well

Marking as high priority because I just personally ran into this problem when opening the site across half my monitor, so this issue applies to any window with a certain width, not just tablets.


Adaeze

Okay
On Mon, 15 Aug 2022, 13:59 Olúwásetèmi, @.***> wrote:

Please I am sorry for all the trouble we have caused you but count it all
joy. Your PR is about to be merged and you will be able to see your name on
this url https://altschool-opensource-names.oluwasetemi.dev/.
To enable me to do this, READ the README again carefully, update your
code with the latest and push the update. I am here for you for the next 6
hours before I close the PR so you can initiate a fresh PR which will be
easier to merge.
—
Reply to this email directly, view it on GitHub
https://github.com/Oluwasetemi/altschool-opensource-names/pull/284#issuecomment-1214981552,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT5MS64IJIDSK7BLWD7YIX3VZI5LZANCNFSM56GBHXEA
.
You are receiving this because you authored the thread.Message ID:
@.***>


DateInput  wrapper prevents setting className, styles on outermost element

The desktop DateInput component renders with an outer <div> element that does not take the className or style props passed to the component - https://github.com/xh/hoist-react/blob/d7a09dfb28b0b893565fe666f40af55aaf6f38f9/desktop/cmp/input/DateInput.js#L391
Often classNames or styles are used for layout concerns like padding or margins, which can't be controlled here. Specifically, I might wish to use a class or style to override the padding applied to a dateInput within a toolbar - that's something that happens at the outermost layer.
I would vote for applying at least className to the outer div. If needed, could support inputClassName, but not sure we really need that.

See example of nested DOM elements:

1 anonymous (no class specific) outer div
2x BP popover spans
1 anon div (again)
1 div with all of the expected classnames (including duped up xh-input, FWIW)
1 BP input-group div
the actual text input + BP input-actions div for buttons

Reviving this as I just ran into it again.  How about having DateInput take a dedicated wrapperClassName prop?

We should confirm if this is still an issue with latest BP library


bdring port parsing error

the parser is failing with "$N0=" and "$N1="
There is also an array out of bounds for the stepsmm axis settings.

quick hacks
Grbl.cs
1756
            int i = 0;
            double stepsmm;
            do
            {
                stepsmm = GetDouble(GrblSetting.AxisSetting_XTravelResolution + i);

                GrblInfo.TravelResolution.Values[i++] = 1d / stepsmm;
            } while (!double.IsNaN(stepsmm) && i < GrblInfo.TravelResolution.Values.Length);

Process(string data)...
                string[] valuepair = data.Split('=');
                if (valuepair.Length == 2 && valuepair[1] != "")

i can do a pull request but I fear the github codebase isn't in sync with your private repo.

Yep, the github repo is out of sync. I'll check tomorrow for the issues you have found - IIRC the out of bounds issue is fixed already.
BTW the first production release will be out soon, got confirmation today that a probing option seems to be ok now - I have been holding back for that.

there is some debate on if $Nx should be reported with $$, maybe something you can coordinate with bart?
Let me know when the repo is in sync and I can see what is fixed.  Looking forward to the new release.  I've been on beta 6 for a while since 7..8 never launched without crashing.

there is some debate on if $Nx should be reported with $$, maybe something you can coordinate with bart?

They should not be reported as $Nx - is this the reason for settings parser failing? If so I have changed it to ignore non-numeric setting ids. IMO $Nx is handled outside settings for a good reason - they contain gcode (when not an empty string) that is executed for validation when set.
Out of bounds check was already added. What is the reasoning behind the empty string check? grblHAL has settings with string values and an empty string is a valid value. The added check will cause them to be ignored.

the empty string check was the quick hack to get it to start without crashing, i'll submit an issue on the $Nx in $$

this PR was accepted so this should be fixed.


fix: Flaky SSLSpec

The SSLSpec is flaky as sometimes it gets timed out. Addition of a sequential annotation to the test is fixing the timeouts.

Codecov Report

Merging #697 (371958f) into main (63ab622) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #697   +/-   ##
=======================================
  Coverage   56.33%   56.33%
=======================================
  Files          63       63
  Lines        1706     1706
  Branches       41       41
=======================================
  Hits          961      961
  Misses        745      745

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 63ab622...371958f. Read the comment docs.

This is not fixing the timeouts.


pipeline: send a notification on holidays

This helps teams be aware of a day that won't have ACH files sent off.
Fixes: https://github.com/moov-io/achgateway/issues/80

Codecov Report

Merging #90 (496911c) into master (b604633) will decrease coverage by 0.24%.
The diff coverage is 16.21%.

@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
- Coverage   41.45%   41.20%   -0.25%
==========================================
  Files          87       87
  Lines        3580     3611      +31
==========================================
+ Hits         1484     1488       +4
- Misses       1809     1834      +25
- Partials      287      289       +2

Impacted Files
Coverage Δ

internal/notify/email.go
57.30% <0.00%> (-1.32%)
:arrow_down:

internal/notify/slack.go
78.72% <0.00%> (-3.50%)
:arrow_down:

internal/pipeline/aggregate.go
24.36% <0.00%> (-3.23%)
:arrow_down:

internal/schedule/cutoff.go
91.48% <100.00%> (+0.79%)
:arrow_up:

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 b604633...496911c. Read the comment docs.


Writing Your Own Wrapper of Ramulator 2.0 for my own Simulator

I am trying to do step 4 under 'Writing Your Own Wrapper of Ramulator 2.0 for my own Simulator' in readme file, to send the memory requests from my simulator to Ramulator 2.0.
Inside frontend.h file, ("virtual bool receive_external_requests(int req_type_id, Addr_t addr, int source_id, std::function<void(Request&)> callback) { return false; }"), it returns false and hence the memory requests are not sent properly to the ramulator.
Do I have to modify any files of the ramulator code base? Or do I have to make 2 separate files of MyWrapper.h and MyWrappr.cpp?

I have an example SystemC frontend on my fork of this repo. It may be useful for your development, and feel free to reach out to me for help. It's in the Readme:
https://github.com/ankenman/ramulator2

Hi,
When you implement your own frontend that inherits from the frontend interface, your own implementation of this virtual function will override the default implementation in the interface class.

I have an example SystemC frontend on my fork of this repo. It may be useful for your development, and feel free to reach out to me for help. It's in the Readme:
https://github.com/ankenman/ramulator2

I appreciate your help. I am trying the solution you gave to me. So, here is what I do:

I installed systemC.2.3.4. I ran an example systemC code to test the installation and it works.
Then inside the ramulator codebase, I copy your example wrapper ramulator cpp file.
Then I am trying to modify the existing Cmakelist.txt file and here is where I am getting linking errors.

Is this the right way to run your example? Can you please provide me the steps to run your example ramulator wrapper file?
Thank,
Prapti


Scene 7 step is not executing when i run DAM Update Asset workflow through ACS Commons Bulk workflow manager

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: 6.3 SP2
[ ] ACS AEM Commons Version: 3.17
[ ] Reproducible on Latest? yes/no

Expected Behavior
Please describe briefly the expected behavior, i.e. when I do X, Y should happen.
Actual Behavior
Please describe briefly the actual behavior, i.e. when I do X, Z happens.
Steps to Reproduce
Starting from a vanilla instance of AEM at the above version, describe in explicit steps how to reproduce. Link to
any supporting assets, e.g. content packages, below
Links
Links to related assets, e.g. content packages containing test components

What Runner are you using?

Synthetic Workflow with Multi threading.

This is expected as Synthetic WF does not support External WF Processes (since the raison d'etre of Synthetic is to avoid use of Jobs to manage work; and External Processes indicate long-running works outside of AEM.)


multicast_dns add support for advertising/broadcasting selected name

I would like to advertise/broadcast new mDns name of my choosing from my dart program.
The example page of multicast_dns and the Readme does not say anything about it.
My program runs on Linux and Windows.
Will multicast_dns add support for advertising/broadcasting a selected name?.

Hello, are there any updates on this? Would like to add this feature to my app..


UAA URL in CF enterprise account

Where do we find the UAA endpoint URL outlined in 3.1.2 - Step 12 of SAP HANA Interactive Education (SHINE) for SAP HANA 2.0 SPS03 for SAP HANA XS Advanced Model - Document version 1.7 – 2018/09/06.
Is it same as API endpoint of the region?

Hi,
To find the UAA endpoint URL, please run the command "xs version" in the CLI. In the output you'll find url listed against "authorizationEndpoint".
Thanks and Regards,
Lakshmi

Hi Lakshmi, This is not on-premise HANA but HANA 2.0 instance in SCP CF environment.
Do you have similar commands in CF CLI or web interface to find the UAA endpoint URL?

The complete SHINE cannot be run on SCP CF. We have an adapted version of SHINE for CF environment in the shine-cf branch of this repository. Please try that.
Best Regards
Stephen

As this issue has been clarified by Stephen, we are closing the issue.
Regards,
Lakshmi


Update troubleshooting topic: UseSharedCompilation also applies to .NET Framework projects

What article on docs.github.com is affected?
Page on website: https://docs.github.com/en/free-pro-team@latest/github/finding-security-vulnerabilities-and-errors-in-your-code/troubleshooting-the-codeql-workflow#no-code-found-during-the-build
Source file: https://github.com/github/docs/blob/main/content/github/finding-security-vulnerabilities-and-errors-in-your-code/troubleshooting-the-codeql-workflow.md

What part(s) of the article would you like to see updated?

In step 5 of the "No code found during the build" section, the following statement can also apply to .NET Framework projects:

For C# projects using either dotnet build or msbuild which target .NET Core 2, you should specify /p:UseSharedCompilation=false in your workflow's run step, when you build your code. The UseSharedCompilation flag isn't necessary for .NET Core 3.0 and later.

Change this to:

For .NET Framework projects, and for C# projects using either dotnet build or msbuild that target .NET Core 2, you should specify /p:UseSharedCompilation=false in your workflow's run step, when you build your code. The UseSharedCompilation flag isn't necessary for .NET Core 3.0 and later.

Hey @hubwriter  👋
Can i have this one ?

hello do u need javascript/html/css here?


DNS timeout on logout (getaddrinfo EAI_AGAIN)

When closing the connection to salesforce, with the logout method. We are sometimes seeing the following error:
getaddrinfo EAI_AGAIN xxxx.my.salesforce.com

This indicates to me that there is not currently a DNS cache being employed to cache the DNS lookup. Adding a simple DNS cache should speed up requests, take some load of the DNS servers, and avoid these errors.
https://stackoverflow.com/a/73581100/8645145
https://www.npmjs.com/package/cacheable-lookup

@jgellin-sf @stomita Bumping because we saw this same error on connection attempts last night.

triaging old issues...
jsforce v3 uses node-fetch v2 which supports this by default:
https://github.com/node-fetch/node-fetch/tree/2.x?tab=readme-ov-file

If no agent is specified, the default agent provided by Node.js is used. Note that this changed in Node.js 19 to have keepalive true by default. If you wish to enable keepalive in an earlier version of Node.js, you can override the agent as per the following code sample.

https://nodejs.org/api/http.html#http_new_agent_options
can someone confirm if this still happens on jsforce v3?


Auto-complete not working for autoloaded global scripts

Godot version:
Godot Engine v3.0.6.stable.official
OS/device including version:
Windows 8.1 Pro & Windows 10 Home
Issue description:
Member functions not auto-completing when the variable is in autoload global.gd
Steps to reproduce:
global.gd
var array = ["zero", "one", "two"]
script.gd
print(global.array.size()) global.array autocompletes correctly but it doesn't complete the function size()
Minimal reproduction project:
minimal reproduction.zip

this issue has been fixed and should be closed

Closing per @ThakeeNathees's comment.

Still getting this behaviour.
PluginsTemplate.zip

Related to #40537.


Compatibility with Grails 3.0.3

@felansu You're filing the issue to the wrong repo. This repo is about the Heroku CLI. I believe you're looking for the Grails buildpack.

thanks man!
On Mon, Jul 20, 2015 at 10:43 PM Jingwen Owen Ou<EMAIL_ADDRESS>wrote:

@felansu https://github.com/felansu You're filing the issue to the
wrong repo. This repo is about the Heroku CLI
https://devcenter.heroku.com/articles/heroku-command. I believe you're
looking for the Grails buildpack
https://github.com/heroku/heroku-buildpack-grails.
—
Reply to this email directly or view it on GitHub
https://github.com/heroku/heroku/issues/1661#issuecomment-123126425.

Closing


Xiaomi MIIO switch: Support for different device types

Description:
The following devices are supported now:

Xiaomi Smart WiFi Socket (called Plug): mains on/off, temperature
Xiaomi Smart Power Strip: mains on/off, temperature, load_power
Xiaomi Chuang Mi Plug V1: mains on/off, usb on/off

I'm waiting for the upcoming release of python-miio by @rytilahti. This will lead to some minor changes and code clean-up.


argocd app delete is stuck with 'CleanUpAfterDeletion' enabled

when use argocd deploy the helm chart, we're not be able to delete the app when  'CleanUpAfterDeletion' enabled.
the work around is to set the the argocd finalizers to use resources-finalizer.argocd.argoproj.io/background.
Please take a look.

@mingmcb Can you share the manifests from the helm chart?

I guess this is caused by https://github.com/streamnative/pulsar-resources-operator/issues/69
For now, resource operator is using an all-in-one controller, and also the topic, namespace currently has no dependency management https://github.com/streamnative/pulsar-resources-operator/issues/130 which usually cause resource blocked during the deleting phase.

yes, it is possible. You can test it out by creating a helm charts with CleanUpAfterDeletion enabled.  A sample argocd app as shown below can be used for testing, the app will be in stuck if you try to delete. apiVersion: "argoproj.io/v1alpha1"
kind: "Application"
metadata:
finalizers:
- "resources-finalizer.argocd.argoproj.io"
name: "pulsar-resources-test"
spec:
project: applications
source:
repoURL: 'https://chartrepo'
targetRevision: x.x.x
chart: test-chart
destination:
namespace: ns-1
name: cluster-1`

In our case we are working around this issue by using argocd sync-waves to ensure deletion in a specific order


fix to create a release apk

Description
A fix to allow assign and run successfully production apk, obfuscation need to be optimized
Fixes #10
Type of change

added coil as a dependency  on app module
dynamic features are declared not instant so
added proguard rule on app module

How Has This Been Tested

[ ]  to generate an universal apk run this gradle task: packageProdReleaseBundle (I don't know why but in my case the traditional step by step release project from the IDE generate an apk that does not include all the features)

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Hi @tidoemanuele first of all thanks for your changes, I appreciate a lot of. I'm absolutly agree that we should to fix the release .apk , apparently have some issues when generate.  We also have a issue related https://github.com/VMadalin/kotlin-sample-app/issues/10 .  For some reason your changes break the build, I tried relaunch the ci build but the error persists.
We should check more in deep why these changes break the ci build. And maybe improve also the proguard workarround.

I can't understand why it fails. I just changed properties values in the manifest

Please check the related issue resolution: https://github.com/VMadalin/kotlin-sample-app/issues/10


Deployment size (shim)

Hi
Can we do anything about the deployment size, at least decrease it? It seems too weird for me to see 8MB of code for a single hello world lambda function.
Regards

Hi @iNDicat0r,
Have you updated your Docker image in the past days? We've updated the packaging script and you should have a package size of roughly 1.6MB for a simple function and complex ones should not exceed 4 or 5MB.

PS: there is also another improvement available (1.6MB -> 700KB) but it has to be supported by AWS.
https://twitter.com/fsenart/status/856479105801494528

Ok you are right, i have to update the image:) lambda supports python3.6 and i am wondering if any time soon there will be an official golang support:/


ServiceMonitor Configuration Issues with Azure Log Analytics Exporter

I am experiencing difficulties with the Azure Log Analytics Exporter in a Kubernetes environment where ServiceMonitor resources are used to manage scraping configurations for Prometheus. Despite configuring separate probes with distinct workspaces, the exporter is querying metrics across all workspaces, leading to the collection of irrelevant metrics and numerous entries with a value of 0. This behaviour persists even after configuring the ServiceMonitor with specific parameters for each probe, intending to isolate the metric queries to their respective workspaces.
Steps to Reproduce:
Deploy the Azure Log Analytics Exporter in a Kubernetes cluster.
Configure multiple probes in the exporter, each with a unique workspace ID.
Set up ServiceMonitor resources for Prometheus to scrape metrics from the exporter, utilizing the probe-specific endpoints and workspace parameters.
Observe the scraped metrics in Prometheus, noting the presence of metrics from all workspaces in each probe's dataset.
Expected Behaviour:
Each probe should only fetch and return metrics relevant to its configured workspace, thereby preventing the mix-up of data across different workspaces.
Actual Behaviour:
All probes are fetching metrics from all workspaces, not just the ones they are configured to query, resulting in a large number of irrelevant metrics.
I suspect there may be a bug in the exporter’s handling of workspace-specific queries or in the ServiceMonitor configuration process that prevents the proper isolation of metrics per workspace.
Would you please help with how can figure out this issue ?
or even if the custom query doesn't have value don't set 0 value, just do not create it.

maybe share your configuration and urls used for probing
please mask any uuid or other secret configuration.


Unit testing + testthat + covr

Current coverage is 57.84%

Branch #12 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.


Is division by Fun operating in-place?

Is anyone else experiencing this?
using ApproxFun
f = Fun(exp)
1./f
1./f
1./f

Sorry, some changes I'm trying to make caused this.

Turns out there was an in-place bug in ichebyshevrootstransform for fftwNumbers. Fixed here @5f2ef2bc1e416a4f393739a50fd8756699d987fe.


Format description doesn't work

This code fails with the following error:
use anyhow::*;
use time::{OffsetDateTime};

pub fn parse_timestamp(timestamp: &str) -> Result<OffsetDateTime> {
    OffsetDateTime::parse(timestamp, time::macros::format_description!("[year]-[month]-[day] [hour]:[minute]:[second].[subsecond]"))
        .context("invalid date format")
}

fn main() {
    let timestamp = "2023-04-30 08:00:55.207";
    dbg!(parse_timestamp(timestamp));
}

Error:
    Error {
        context: "invalid date format",
        source: TryFromParsed(
            InsufficientInformation,
        ),
    },

I presume you figured it out since you closed the issue, but the reason it failed is because there is no UTC offset specified.


I've collected ALL the possible markups for the configuration file, including the "unofficial" "editor.hover": false. Nothing works, it still feels the urge to tell me what a css property is and what "background-color" means.
The sheer amount of popups are getting in the way of my productivity.
It is starting to feel like an adware program in this behavior.
So. I ask again, as thousands of others before, and I know there are hundreds more of these similar posts all across the web. Maybe, do something about it?
I would LOVE. No, I NEED a way to disable ALL tooltips. I don't care if it breaks "functionality". Functionality requiring tooltips has no reason to exist in a code editor as far as I'm concerned.
Anyone has a "2019-workaround"? Thaaanks!

Hover setting name is editor.hover.enabled, not editor.hover
Also, there's editor.hover.delay.


HttpLink doesn't call NextLink

Describe the bug
I'm working on creating an AuthLink to fetch a token before calling HttpLink. I was using HttpLink as a model for the new link and noticed that HttpLink doesn't call NextLink. Is it supposed to call next link if a link is concatenated to it?
For completeness I got my Link working by doing the following:
class AuthLink extends Link {
  AuthLink()
      : super(
          request: (Operation operation, [NextLink forward]) {
            StreamController<FetchResult> controller;

            Future<void> onListen() async {
              try {
                var session = await UserService.getSession();
                var token = session.getToken();
                operation.setContext(<String, Map<String, String>>{
                  'headers': <String, String>{'Authorization': token}
                });
              } catch (error) {
                controller.addError(error);
              }

              await controller.addStream(forward(operation));
              await controller.close();
            }

            controller = StreamController<FetchResult>(onListen: onListen);

            return controller.stream;
          },
        );
}

Here's the corresponding code from<EMAIL_ADDRESS>StreamController<FetchResult> controller;
Future<void> onListen() async {
  Response response;

  try {
    // TODO: support multiple http methods
    response = await fetcher.post(
      uri,
      headers: httpHeaders,
      body: httpOptionsAndBody.body,
    );

    operation.setContext(<String, Response>{
      'response': response,
    });

    final FetchResult parsedResponse = _parseResponse(response);

    controller.add(parsedResponse);
  } catch (error) {
    controller.addError(error);
  }

  await controller.close();
}

controller = StreamController<FetchResult>(onListen: onListen);

return controller.stream;

and using concat:
    var authLink = AuthLink()
        .concat(HttpLink(uri: ''));
    ValueNotifier<GraphQLClient> client = ValueNotifier(
      GraphQLClient(
        cache: InMemoryCache(),
        link: authLink,
      ),
    );

Expected behavior

NextLink executed if a link is appended to HttpLink.

Smartphone (please complete the following information):

Device: IPhone XR

Additional context
Add any other context about the problem here.

@michael-golfi Just added an official AuthLink based on your idea, it does call the link, see #171

This is awesome, thanks for including this!

HttpLink really doesn't call next link.  In my Graphql Server, authorization is performed in the resolvers, so I detect a revoked authorization token in the GraphQL errors.  I handle this in Apollo by having a link after the http call that can do this high-level logic, but in this library any link after Http Link is never called.  I copied HttpLink into my project to try to modify it, but the operation class is not capable of holding a graphql or network error result, so the HttpLink cannot call next link.  I am brand new to Dart/Flutter, so I didn't go further.
I am currently evaluating having a custom HttpLink so I can action on the HttpResult before it gets to Flutter, or having a Query and Mutation wrapper to handle this logic on every Query or Mutation invocation, but neither feel like good or maintainable options.
Are there any plans (or documentation I haven't discovered) for an ErrorLink, or RetryLink, or similar where I can take action outside of the Flutter update cycle, but after the network call?

In the meantime, I subclassed HttpLink, and I proxy the request call so I can inspect and process the result transparently to the other links.


code is not good enough for testing

Issue 1: This function seems to update secrets manager only in plaintext format not in key pair format.
Issue 2: if there is already an entry for user in key value pair in secrets manager , (let say when user is created via CF and updated to SM). those keys are not updated at all.
in both scenarios keys gets rotated and i get notification., but there is not key update on secrets manager.
i can confirm there is gap in the code and is partially running to even test for test scenarios. gap in this solution is implementation and how console or aws handles secrets in json format.

Issues 1 & 2 have been resolved with the new version 2 release.


Improve the code coverage

For now the project is badly covered by the tests. It can be greatly improved.

hi @michabs, thanks for the willingness to contribute, you can check the coverage report at https://app.codecov.io/gh/lsd-rs/lsd and find out a start point


[NEW ICONS]: Laravel Ecosystem icons

Add New Icon(s)
Icon Details

Icon ID
Icon

Breeze

Cashier

Dusk

Echo

envoyer

forge

herd

horizon

inertia

jetstream

nova

octane

pennant

pint

prompts

reverb

sail

sanctum

scout

socialite

telescope

vapor

Icon ID
Icon

laravelspark

pulse

pail

Notes

Breeze: Laravel Breeze is a lightweight starter kit that simplifies the scaffolding process for new applications, supporting Blade or Inertia for frontend rendering.
Cashier: Laravel Cashier streamlines subscription management on payment platforms like Stripe or Paddle, making it easier to handle billing and subscription-related tasks.
Dusk: Laravel Dusk automates browser testing, ensuring that your application functions correctly across different browsers and giving you confidence in your code before deployment.
Echo: Laravel Echo allows your application to listen for WebSocket events, enabling real-time communication and updates between the server and clients.
Envoyer: Laravel Envoyer facilitates seamless deployment of Laravel applications with zero downtime, ensuring a smooth transition for your users.
Forge: Laravel Forge simplifies server management, providing a user-friendly interface for tasks like server provisioning, configuration, and monitoring.
Herd: Laravel Herd offers a fast local development environment for macOS and Windows, optimizing the development workflow for Laravel projects.
Horizon: Laravel Horizon provides a visually appealing UI for monitoring and managing Redis-driven Laravel queues, aiding in queue management and optimization.
Inertia: Laravel Inertia enables developers to create modern single-page applications (SPAs) using classic server-side routing while integrating with popular frontend frameworks like React and Vue.js.
Jetstream: Laravel Jetstream is a robust starter kit that includes authentication, team management, and other common features, speeding up the development of Laravel applications.
Nova: Laravel Nova is a thoughtfully designed administration panel that simplifies the management of resources, data, and content within Laravel applications.
Octane: Laravel Octane supercharges application performance by keeping it in memory, leveraging features like JIT compilation and asynchronous processing.
Pail: Laravel Pail is a package that allows you to easily dive into your Laravel application's log files directly from the command line.
Pennant: Laravel Pennant is a simple yet powerful library for managing feature flags, allowing developers to control feature rollout and experimentation.
Pint: Laravel Pint is an opinionated PHP code style fixer designed for developers who prefer minimalistic coding standards.
Prompts: Laravel Prompts provides beautiful and user-friendly forms for command-line applications, enhancing the user experience for CLI interactions.
Pulse: Laravel Pulse offers at-a-glance insights into your application's performance and usage metrics, aiding in performance optimization and monitoring.
Reverb: Laravel Reverb delivers blazing-fast and scalable WebSockets for your application, enabling real-time communication and updates.
Sail: Laravel Sail provides a hand-crafted local development experience using Docker, streamlining the setup and management of local development environments for Laravel projects.
Sanctum: Laravel Sanctum simplifies API and mobile application authentication, providing secure authentication mechanisms without complex setup.
Scout: Laravel Scout offers lightning-fast full-text search capabilities for your application's Eloquent models, improving search functionality and user experience.
Socialite: Laravel Socialite facilitates social authentication via platforms like Facebook, Twitter, GitHub, LinkedIn, and more, making it easy for users to log in using their social accounts.
Spark: Laravel Spark is a fully-featured, drop-in billing portal that helps you launch and manage your next business by handling subscription billing, team management, and billing integrations.
Telescope: Laravel Telescope is a powerful debugging and insight tool that helps developers debug and analyze their Laravel applications, providing valuable insights into application behavior and performance.
Vapor: Laravel Vapor is a serverless deployment platform for Laravel, powered by AWS.

Closing
Breeze, Laravel Ecosystem

tandpfun/skill-icons/issues/996

Inertia

tandpfun/skill-icons/issues/526

Checklist

[x] Added icon(s) to the appropriate directory
[x] Updated documentation with the new icon(s)
[x] Verified the icon(s) display correctly
[x] If the icon has light and dark, verified the auto switch works correctly

@ItsJoshBrown it's seems that Pint and Vapour icons are broken

There's one thing about echo and dusk icon (and for sure others), the thing is that they use strokes, and strokes are pretty much hard to work with sometimes. for example, if you unscale the icon, it will be very thick and looking like a mess, and being scaled up will result with a very thin stroke that is very undistiguishable... do you think you have time to work on this ? or maybe open an issues about it and someone (or me) can work on that ?

@ItsJoshBrown There's one thing about echo and dusk icon (and for sure others), the thing is that they use strokes, and strokes are pretty much hard to work with sometimes. for example, if you unscale the icon, it will be very thick and looking like a mess, and being scaled up will result with a very thin stroke that is very undistiguishable... do you think you have time to work on this ? or maybe open an issue about it and someone (or me) can work on that ?

Oh, I see what you are talking about, some of them were giving me issues in Figma I guess now I know why :)
I'll have some free time later today I'll see if I can fix it, if not I can adjust the PR and remove the ones I can't fix and open up an issue for them.

@LelouchFR  I believe I've got everything all fixed up. Please let me know if you see I missed anything.
I'm encountering issues with Git where every PR shows my entire commit history. Apologies for that. I tried rebasing, but it didn't work. After this PR, I'll try deleting and recreating my local and remote branches to fix the problem.

@LelouchFR  I think I have resolved the issues with them let me know if you are still seeing anything wrong with these two. I thought I had fixed these I guess I didn't after it was mentioned to be messed up I didn't realize that the masking for the pint log was messed up, and I think I had extra parts in the vapor icon that weren't being utilized.


[BEAM-1938] Update Signature of PTransformOverrideFactory

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

This enables replacements to be reobtained with the entire transform
that is being replaced.
This is required when Side Inputs are part of the input of the
PTransform Application, as PTransforms are not applied to their side
inputs.

R: @jkff

Coverage decreased (-0.04%) to 70.427% when pulling e5e6e63983a6b5523a9a92f4c5aa7d96de90b40b on tgroh:override_factory_entire_context into b2329a7eff1e2b660af1efcc0a58bc54a478bfdd on apache:master.

I'm going to merge on green and file a follow up to remove some duplication from these classes.

https://issues.apache.org/jira/browse/BEAM-1982 filed

Changes Unknown when pulling 2926965a98c73f88801204f889bfc0c04368798d on tgroh:override_factory_entire_context into ** on apache:master**.

Coverage decreased (-0.02%) to 70.445% when pulling 2926965a98c73f88801204f889bfc0c04368798d on tgroh:override_factory_entire_context into f30d5b9e9db80e35c9fae4c05d2a76655922db1c on apache:master.


Add plotly to pdf functionality

Following #359 we'd like to generate pdfs of metrics plots for report generation
Adding a pdf field to the PlotlyReportFigure datamodel class and an output_pdf would make this workflow simple.
One issue here to  to be tackled is determining the encoding of the plotly pdf. decoding the bytes with into utf-8 or ascii fails but succeeds with latin-1 but I'm not sure if that is correct. Some googling didn't yield any immediate results on this.
The decoding of the returned bytes is going to be necessary for transmitting and storing storing the pdfs as json at the API, even if we are going to need to encode again to the original format for rendering. If this proves to be a pain, we could re-instantiate plotly figures from the provided spec and generate pdfs on the fly during the greater report pdf generation process.

Turns out, PDFs can be encoded with ascii 85 https://en.wikipedia.org/wiki/Ascii85.


Protected streaming

Hello, recently i discover this disucussion in videohelp forum about donwload enrypted streams, ant it said :
"No m3u8, but mpd.
Your stream is encrypted. You will need keys to decrypt.
yt-dlp --allow-u "mpd_url"
https://forum.videohelp.com/threads/407535-Zee5-video-download
Does MediaDonwloader include by default this command " --allow-u "mpd_url" to donwload encrypted streams please ? if not, how can we add by default this command "--allow-u" please ?

You can do this using the yt-dlp Arguments feature in the new v3.7.0 update.