Possible to use relative paths on remote.

First of all, thanks for providing such an excellent extension.
When I insert a breakpoint, code-debug executes the following command over ssh:
7-break-insert -f "/path/to/ssh/cwd/src/file.f:15"

What needs to happen is this:
7-break-insert -f "src/file.f:15"

My configuration is this (with nonrelevant information removed)
            "name": "Launch Remote",
            "type": "gdb",
            "request": "launch",
            "target": "./executable",
            "printCalls": true,
            "cwd": "${workspaceRoot}",
            "ssh": {
                "forwardX11": false,
                "host": "host",
                "cwd": "/path/to/ssh/cwd",
            }

have you tried using "cwd": "." in the ssh section?

I need to change to the correct directory, so I put the following in bootstrap
cd /path/to/ssh/cwd
Then:
1-gdb-set target-async on
2-environment-directory "."
3-file-exec-and-symbols "executable"
4-environment-cd "."
5-exec-arguments
6-break-delete
7-break-insert -f "C:\\Program Files (x86)\\Microsoft VS Code/src/file.f:15"
Running executable
8-exec-run
-gdb-exit

so it works with a workaround, having a setting which allows you to do that would probably be better

No it does not work with a workaround. The following line is not correct:
7-break-insert -f "C:\\Program Files (x86)\\Microsoft VS Code/src/file.f:15"


Register OVN-IC APIs

Epic: https://github.com/submariner-io/enhancements/issues/186

@Jaanki Thanks for the PR , I tested it and it works fine.


⚠ ClamAV has degraded performance

In c882e59, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 220 ms

Resolved: ClamAV performance has improved in 89fb912.


How to use SD1.5 on M1 Mac? (CUDA not available Error)

At the end of the demo video you can see the usage of SD1.5.
When I try to start Lama with --model=SD1.5 I get the following errors regarding missing CUDA.
/Users/USER/miniconda3/lib/python3.9/site-packages/torch/amp/autocast_mode.py:198: UserWarning: User provided device_type of 'cuda', but CUDA is not available. Disabling   warnings.warn('User provided device_type of \'cuda\', but CUDA is not available. Disabling') /Users/USER/miniconda3/lib/python3.9/site-packages/torch/amp/autocast_mode.py:198: UserWarning: User provided device_type of 'cuda', but CUDA is not available. Disabling   warnings.warn('User provided device_type of \'cuda\', but CUDA is not available. Disabling') usage: lama-cleaner [-h] [--host HOST] [--port PORT]                     [--model {lama,ldm,zits,mat,fcf,sd1.5,cv2}]                     [--hf_access_token HF_ACCESS_TOKEN] [--sd-disable-nsfw]                     [--sd-cpu-textencoder] [--sd-run-local]                     [--device {cuda,cpu}] [--gui]                     [--gui-size GUI_SIZE GUI_SIZE] [--input INPUT] [--debug] lama-cleaner: error: argument --model: invalid choice: 'SD1.5' (choose from 'lama', 'ldm', 'zits', 'mat', 'fcf', 'sd1.5', 'cv2')
What can I do to get it working? I already downloaded both checkpoint versions 1.5 pruned and 1.5pruned emaonly from hugging face. Can someone explain to me the next steps?

For stable-diffusion model, you need to accepting the terms to access, and get an access token from here huggingface access token. Then start the server:
lama-cleaner --model=sd1.5 --device=cpu --hf_access_token=you_token

I haven't tested it on the m1 and not sure if it will work, good luck

lama-cleaner --model=sd1.5 --device=cpu --hf_access_token=you_token
Thank you! That helped to install everything! I can start a dream process but since it's only doing this with the CPU. One take takes about 4-5 minutes.
https://replicate.com/blog/run-stable-diffusion-on-m1-mac
Here they used a virtualenvironment. Do you have any ideas how to set it up for lama as well?

Install the virtualenvironment according to the guidelines in the blog, then you can install Lama Cleaner (pip3 install lama-cleaner) after Activate the virtualenv

Thanks I'll try

I'm nut sure, which blog you mean. I tried setting up the venv like this:
cd LamaCleaner
python3 -m pip install virtualenv

Sorry for the annoying log, actually you can safely ignore that CUDA warning.
After the first time you run lama-cleaner --model=sd1.5 --device=cpu --hf_access_token=you_token, you can remove --hf_access_token, and add --run-sd-local to start the server
lama-cleaner --model=sd1.5 --device=cpu --run-sd-local

Thank you. It's still giving me errors. Look at this

With that it runs but slowly.
After that I closed the terminal and started again with
lama-cleaner --model=sd1.5 --device=cpu --run-sd-local

Somehow it does not recognize --run-sd-local

Sorry for the annoying log, actually you can safely ignore that CUDA warning.
After the first time you run lama-cleaner --model=sd1.5 --device=cpu --hf_access_token=you_token, you can remove --hf_access_token, and add --run-sd-local to start the server
lama-cleaner --model=sd1.5 --device=cpu --run-sd-local

I made a typo, it should be --sd-run-local. This parameter simply allows you to not pass the huggingface token, nothing to do with running speed.

You can ignore this CUDA warning, it is normal for this warning to appear on machines without NVIDIA graphics.
The real reason the model running slow is, we should use PyTorch compiled for the M1 chip, currently Lama Cleaner not support that, I will add M1 support If I could get an M1 Mac.

SIX MONTHS LATER....
Set --device=mps run fastest stable diffusion model on m1/m2 chip, on my m2 pro the speed is 1.5~2 it/s


Remove font-awesome dependency from NoThemeReasources module.

Currently the NoThemeResources module has the line:
<stylesheet src="css/font-awesome-4.3.0.min.cache.css"/>

This forces injection of the font-awesome css.  I think it would be a good idea to remove this requirement and force the user to manually add it if they want it.
Am I missing something?  Was this left in there for a reason?  Thanks!

The NoTheme was primarily for the bootstrap css. But I can see this point.


Remove "unnecessary" imports.

In each library where an import is removed, the library uses some elements
provided by the import, BUT there is another import which provides all of the
same elements, and at least one more which the library uses.
In this change, we remove the imports which can be simply removed in favor of
the other already present imports.
See https://github.com/dart-lang/sdk/issues/44569 for more information.
List which issues are fixed by this PR. You must list at least one issue.
https://github.com/flutter/flutter/issues/74381 is improved by this issue. Not fixed.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

CI looks good :) :) :)

@srawlins friendly ping

Gold has detected about 2 new digest(s) on patchset 2.
View them at https://flutter-gold.skia.org/cl/github/85736

CI looks good :) :) :)

I perhaps need to rebase?

I perhaps need to rebase?

Yes, there were some trouble with the golden check. Rebase should resolve it.


Kiwi.HUD.Widget.MenuItem API Doc does not have a constructor.

This API docs say that it extends Kiwi.HUD.Widget.Button. I do not know if it is meant to have a constructor or not.

It should, but did not, have a constructor. What it did have was a contructor, which I can only assume is a robot in disguise waiting to ambush the Autobots.
I have eliminated the Decepticon menace. Our documentation is safe.


Proof of Concept for CI Integration tests with GCP for Airflow

Codecov Report

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

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   75.45%
=========================================
  Files             ?      199
  Lines             ?    15948
  Branches          ?        0
=========================================
  Hits              ?    12033
  Misses            ?     3915
  Partials          ?        0

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 719e0b1...518c3d0. Read the comment docs.


Add identical_to API for equal treatment of NULL values

I can merge this if you rebase. At some point will want to add some correctness tests for the Impala code, perhaps we can leave that as a TODO in a later github issue

Thanks for all your work on Ibis, @cpcloud!

@wesm rebased

I have the impala minimal docker image on my local machine so i can add a test for the correctness here and merge after that passes.

Can you rebase? Sorry... =/

Rebasing this. Moving this to 0.10, as this isn't critical and I'd like to write some more test cases.

 I'll refactor and rebase and push it up to get it merged.

Merging on pass.


ollama: mount integration under haystack_integrations.*

Checklist

[ ] Update the structure, the code, and examples on haystack-core-integrations
[ ] Update the integration tile on https://github.com/deepset-ai/haystack-integrations
[ ] Update the docs at https://docs.haystack.deepset.ai/
[ ] Update the notebooks on https://github.com/deepset-ai/haystack-cookbook (if needed)


ISTEP translation to MCRL2 leads to erroneous MCRL2 model

internal step in TorXakis leads to error in MCRL2 when processing with mcrl22lps:
$ torxakis testISTEP.txs
.....
TXS >> lpe Mod
TXS >>  LPE modeldef generated:
TXS >>  LPE_proxyModel
TXS >> lpeop mcrl2 LPE_proxyModel I
TXS >>  INS = {[[ {In} ]]}
TXS >>  OUTS = {[[ {Out} ]]}
TXS >>  ALPHABET = {[[ ]][[ {In} ]][[ {Out} ]]}
TXS >>  <>
TXS >>  Model exported to I.mcrl2!
TXS >> q
.....
$ mcrl22lps.exe I.mcrl2 I.lps
[error]   Could not find a matching declaration for action or process expression ACTION_NOT_FOUND.
testISTEP.txt

@tretmans As a bug reporter, can you check the bug is indeed removed?

@tretmans please close this issue when it is solved.


feat: allow configration of trivy-server pvc size

Description
Allows the configuration of the PVC size for the trivy-server. In our environment we had a full disk.
Checklist

[x] I've read the guidelines for contributing to this repository.

@chen-keinan I saw that the one of the build jobs failed potentially because of one of my changes. Did I missed adding something?

Great, I wasn't aware that this is possible. I've bumped the version.


Input adjustements

Using this component, I thought some <label> related improvments could be useful:

a hideLabel prop: not documented on DSFR docs, but we could allow users to provide a valid label but hide it (using sr-only to make it still readable by screen readers).
removed aria-labelled attribute: as long as the label prop is mandatory and linked to the related input with htmlFor, I don't see why we should keep it.
I added the support of the type file input (https://www.systeme-de-design.gouv.fr/elements-d-interface/composants/ajout-de-fichier), wrapping it with a fr-upload-group : I was about creating a new component (to follow DSFR naming and docs), but it appears that Input and File are just the same, only the wrapper class is changing. Maybe we should create a File component that just wraps Input component?

Well, I think we should have a separate File component that wraps the Input component: at least, it could handle a max file size property, which makes a validation on the uploaded file (on onChange event).

Sorry it took me a week to review this.
I'm doing it now

Ok thank you for your contribution.
I'll merge this :)


Added fig as a keyword to savefig

In the case where the figure to be saved has not been created with matplotlib.pyplot, the matplotlib.pyplot.savefig function cannot identify the figure you are working with and hence produces a blank image. Adding fig to the api allows you to pass through the active figure, and the code changes use the methods associated with this figure to save and/or clear the image.

LGTM, thanks @pdearnshaw 👍


Correction for raw affiliation L3S and TIB

Correction needed for raw affiliation L3S and TIB
raw_affiliation_name: L3S and TIB
new_rors: 0304hq317;04aj4c181
previous_rors:
works_examples: W4393590113;W4393458274;W4393631527;W4393647884
contact: e9543653964b5ce67b4fbaaef0667646:b00cb674cadd42a031cf739342c7d078c983 @ tib.eu

Correction:
new_rors: 039t4wk02;04aj4c181

Hello @hauschke, as the issues will be consumed automatically, your comment won't be taken into consideration. If there is any error in the issue, the best would be to delete the issue and resubmit any correction into Work-magnet. Thanks !

Sorry, closing it is enough. I close it so you can reopen it if needed.


How can i stream HD video?

I can stream multiple video(below 25 MB) at time from the server. If I try to stream 70 MB HD video, i get the error "503 - Service unavailable". How can i stream large video with the help of this server.

That error is returned if we catch the OutOfMemory exception in SimpleFileWorker - basically you're running out of memory as the SimpleFileWorker is just a simple, naive implementation that tries to read the entire file into memory, then sends it down the wire.  You are running out of memory trying to read the large file.
If you want to write a streaming FileWorker, you would need to extend SimpleFileWorker with a new implementation that will read & send the data in small chunks (perhaps a single meg at a time).  N.B. there might be some other special things in HTTP for doing "real" streaming - this server is not really maintained much any more and was not designed for streaming large files, sorry!


Changing CommandBarFlyout to not take keyboard focus when opened in transient mode

CommandBarFlyout was changed to unconditionally give keyboard focus to the first focusable command when it opens.  This breaks the assumption when opening in transient mode that the flyout won't take keyboard focus in that mode.  We should only give keyboard focus when opening in standard mode.
Also added a test to make sure that this doesn't regress in the future.

/azp run

/azp run


Fixing 'error prone' constructs as reported by Codacy

Just added a missing header reported by inspect. Thanks @hkaiser!


feat(alerts): minor copy updates to create/edit alert form

Description
Some minor copy updates to the create/edit alert form based on recent Figma specs.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

feat(alerts): minor copy updates to create/edit alert form - devon

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (feat(alerts): minor copy updates to create/edit alert form - devon)

  Generated by :no_entry_sign: dangerJS against 5cf700566b7118ef60f0074cb803d107feb9b9f8


BUILD-87: secret configmap volume mounts in builds

https://issues.redhat.com/browse/BUILD-87

/retest

FYI there is auto linking of PRs to Jira items if you
/retitle BUILD-87 secret configmap volume mounts in builds

/retitle BUILD-87: secret configmap volume mounts in builds

Course I don't have permissions to do it @coreydaley :-)

/hold
Given the complexity of the feature and the time remaining before 4.8 reaches "final freeze" period, we are going to hold this until master opens for 4.9

@adambkaplan ptal

/retest

Test on PR launched cluster launch openshift/builder#245,openshift/openshift-controller-manager#183,openshift/openshift-apiserver#208,openshift/oc#843 following test cases https://url.corp.redhat.com/buildvolumes , no critical issues found.
/label qe-approved

/retest

/retest

/hold cancel

/retest

@adambkaplan @gabemontero ptal, tests are passing and it is ready for review.

Keeping the hold for docs/px approvals.
I don't think we should expand the scope of runtime-utils to add this copied code, since it pollutes that repo's purpose. I am disappointed that our PR to library-go was rejected, since other code in that repository doesn't meet their criteria.

/hold cancel


Fedora 28 - "conn.go:173:12: error: incompatible types in assignment"

Hi guys, at first command get a problem, this is what I've got
`$
go get -u github.com/emersion/neutron
github.com/emersion/neutron/backend/imap
go/src/github.com/emersion/neutron/backend/imap/conn.go:173:12: error: incompatible types in assignment
c.Updates = updates
^
`
I was looking for similar issues but with no luck.
Greetings

Yeah, this repo needs to be updated. Unfortunately I only work on open-source projects in my spare time so I won't be able to do this anytime soon. Pull requests are welcome though.

On which ubuntu version is this working?


Feature Request: Support for MMA8452Q

Perhaps I'm missing something
list_devices()
returns (29, 'Qwiic ADXL313', 'QwiicAdxl313')
Just wanna know if I'm missing something obvious here before I begin porting the Arduino lib

A package hasn't been ported yet.


Update module github.com/vbauerster/mpb/v8 to v8.8.2

This PR contains the following updates:

Package
Type
Update
Change

github.com/vbauerster/mpb/v8
require
patch
v8.8.1 -> v8.8.2

Release Notes

vbauerster/mpb (github.com/vbauerster/mpb/v8)
v8.8.2
Compare Source
Full Changelog: https://github.com/vbauerster/mpb/compare/v8.8.1...v8.8.2

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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

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

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
432

Relevant Lines:
791

💛  - Coveralls


fix broken links

Fixes broken links on the site noted in this doc
Sets up aliases/redirects for all links on twitter and all landing pages registered in analytics to date

Rebased to re-run and re-deploy the preview after URL changes: http://www.chipsalliance.org/preview/40/

@cjyabraham can you split this PR into two - one that only introduces aliases, and the other which does other stuff? hard to review otherwise. I expec the aliases stuff to go in 1:1, we just need to double check it, whereas other changes might need some discussion.

Ok, can you approve this PR and then I'll merge it and merge main back in to this branch to essentially remove the alias commits.

I've merged the aliases PR and merged that change here so now this PR only contains link fixes.


ECR DTH failed to copy image: read tcp xxxx-><IP_ADDRESS>:443: read: connection timed out

Hello,
We created an ECR DTH task to sync ECR images from us-east-1 region to cn-northwest-1 region. We have been periodically received AWS notification message emails since the task created. The email stated:
{"error":"Failed to copy image","execution":"7e7944e9-e302-0881-0ec7-6f66e027abf5_947d8293-b762-89ea-4a32-f6485e72464a","image":"<repository_name>","tag":"<tag>"}
We verified the image digest in the target region actually matches the image digest in the source region. And, we tracked down the ECS logs and noticed this error:
time="2022-09-12T20:42:44Z" level=fatal msg="trying to reuse blob sha256:bbab4ec87ac4f89eaabdf68dddbd1dd930e3ad43bded38d761b89abf9389a893 at destination: Head \"https://<aws_china_account_id>.dkr.ecr.cn-northwest-1.amazonaws.com.cn/v2/<repository_name>/blobs/sha256:bbab4ec87ac4f89eaabdf68dddbd1dd930e3ad43bded38d761b89abf9389a893\": read tcp 10.0.0.XX:52920-><IP_ADDRESS>:443: read: connection timed out"
However, not every time we saw a connection timed out error, we would receive an AWS notification message email. We don't know if the connection timed out is the root cause for the "Failed to copy image" error and are not sure how to solve this time-out issue.
Template version v1.0.3

Hi @solaamy ,
Every transfer job is designed to retry at most 3 times. If all 3 retries fail, an alarm email will be sent.
Since the data is transferred through the public internet, its transmission performance will be affected by the network environment, and sometimes a timeout error will occur.
Here is a workaround:

Go to AWS Event Bridge Rule, and adjust the time interval for ECR comparison job from the default 1 day to 1 hour. This action may solve the image transfer timeout issue by adding more retries in job level.

Due to no further updates, we are closing this issue. If you have any new questions or concerns, please create a new one.


render face disalignment with the input face

Hi, Thank you for this work. when I use the Render_block referenced the 'Deep3DFaceReconstruction'  render the recon_face and paste on the origial image face. the render face cannot overlay the original face correctly, alway with scale and offset disalignment.  the focal  use 1015 as default as your code , and the camare position also default.   can you give me some suggestion? Thank you !

To be clear, is your question related to Deep3DFaceReconstruction's results? If so:
In Deep3DFaceReconstruction, the recon_face is based on the input cropped image. Have you  ever tried to directly align the recon_face to the input? If this part is correct, you may check your calculation of the inverse of transform_params.  If possible, could you  provide your input  and result for more info?

Thank you ! I got the correct render result.


Beautify JS version out of date

I am experiencing issues in regards to Beautify extension breaking my code with syntax errors when there are optional chaining operators (?.)  https://github.com/beautify-web/js-beautify/issues/1727
This was fixed with js-beautify version 1.11

Is this project not being maintained?This is the only plugin I'm used to right now.

came here, saw same thing that this plugin is out of date. too sad, using prettier now

As a workaround I went into ~/.vscode/extensions/hookyqr.beautify-1.5.0 and installed the newest version of js-beautify there.
npm uninstall js-beautify -s
npm install js-beautify -s

Reload vs code window and the extension should use the latest version. I guess you could manually edit the package.json version if you only want to go to 1.11.
Not ideal but it works. 🤷♂


[1.2.x] on Scala 2.11 scala-parser-combinators 1.1.2 on test requires forking

Reported originally by @Philippus as https://github.com/scala/scala-parser-combinators/issues/197
steps
Philippus/bump#8
problem
Philippus/bump#8 fails with the following exception in scala 2.11:
SemVer.nextStable removes pre-release information if available: Exception raised on property evaluation.
[info] > Exception: java.lang.AbstractMethodError: nl.gn0s1s.bump.SemVerParser$.scala$util$parsing$combinator$Parsers$$lastNoSuccessVar()Lscala/util/DynamicVariable;

expectation
No failure.
workarond
Test / fork := true

notes
This is a variation of
https://github.com/sbt/sbt/issues/3405
https://github.com/scala/scala-xml/issues/195
https://github.com/sbt/zinc/pull/505

So basically sbt/zinc#505 fixed the bug for run
   def makeLoader(classpath: Seq[File], instance: ScalaInstance): ClassLoader =
-    filterByClasspath(classpath, makeLoader(classpath, instance.loader, instance))
+    filterByClasspath(classpath, makeLoader(classpath, instance.loaderLibraryOnly, instance))

but didn't for test it seems.
https://github.com/sbt/sbt/blob/v1.2.8/testing/src/main/scala/sbt/TestFramework.scala#L258
    val dual = new DualLoader(
      scalaInstance.loader,

I believe scala-xml would be affected as well on both 2.11 and 2.12 (since the scala-compiler -> scala-xml dependency wasn't removed until 2.13)

@eatkins We still have this problem because ScalaInstance loader for Scala 2.11 probably is bringing in parser combinator - http://repo1.maven.org/maven2/org/scala-lang/scala-compiler/2.11.12/scala-compiler-2.11.12.pom
I think the bottom layer needs to be a Scala Library only loader.


convert to async

Convert the project to allow asynchronous tasks

Was looking at your branch for this. What's the reasoning behind a new client construct?
Converting to async should be fairly simple since we're not using any DBs for this repo.

Hey,
I'm using it as a way to learn more about async as it's not something I've worked with yet.
The new client I'm hoping will make it work more efficiently but still quite early on, I have more code that I haven't committed yet which changes a few things from the current commit.

Since PyArr is fundamentally just a wrapper for requests, I don't think there's any need for a client construct. May just make the code more difficult to read. Just typing async in front of every function def would be the easiest way to obtain full async.
If you've ever using threading, async is a very similar concept.

Yea so my plan is to replace requests with aiohttp.
I maybe miss understanding something but in order for the async to work correctly doesn't the client connection need creating first and then all functions running through the same connection?
I believe the current way would create a new connection for each request.

An alternative is to create a client on each request, rather than a single pool that all connections funnel through. Would directly mimic what we're doing now, not require as much refactoring, and support infinitely many concurrent connections.
async with aiohttp.ClientSession(headers=headers) as session:
    pass

If this ticket stays open till the end of the year, I'll take a look at this. Perhaps we can call it our v3.0 milestone.

@marksie1988 If we convert to async, are we going to abandon sync methods?
There is no way to be both sync/async without code duplication that I'm aware of.
Also, I can put this build together in less than a day if we agree what path to take.
Side note: I've become aware someone has already forked Pyarr v2.x to be async.

After thinking about this one I was going to create a fork called aiopyarr, however as that has been done I'm not sure if it makes sense to create an async version when one already exists.
I would still be happy to do it if you felt it a worthwhile task but I'm not sure atm...

It looks like aiopyarr has changed a few things about our API, so it's not exactly a drop-in replacement. The two big tings are:

API configuration is done through objects
All API endpoints return Python objects rather than dictionaries

If we had decided to migrate this current repo to be fully async, I intended to make the transition as simple as possible.

If they intend to use the async functions as sync, then things would get visually ugly.
Something we could potentially discuss with the developer is if he'd be willing to have the aiopyarr repo exist within the totaldebug organization, that way it can become an "official async pyarr" implementation.

I did consider seeing if the aiopyarrr creator would join it up so that we can go in as one team. Saves us both having to find new endpoints and changes etc.
However I didn't think about just adding a async module to this project, that might not be a bad shout. That way we cover both with one package...

+1 for responses as TypedDict and constructors with keyworded arguments (which themselves can be stored in a dict and unpacked if desired).

I agree I would like to keep the TypedDict it has different use cases and like you say would be easier to maintain, last thing we want is to need to do urgent changes because the Servarr team aren't always the best at documenting changes to the APIs (at least not that ive seen)
I like the idea of pyarr.async, pyarr.sync and common / utils etc. this would hopefully remove unnecessary duplication, however not something I have done before so question how do you achieve using a common library for sync and async? if there are any Django examples it would be good for me to take a look.

In terms of writing out API endpoints (ex. upd_book(args, kwargs), that code specifically would need to be duplicated. One being in an async fashion and the other being sync.
There are ways to convert async code to be synchronous (and vice-versa), but none of them are performant. If we wanted to avoid code duplication at the cost of performance, the best route would be to write everything async and use asgiref.async_to_sync or asyncio.run.
All the important Django examples are still under development and haven't been released yet.

Closing this for the time, not sure if this is needed for the library at the moment. may revisit in the future.


Introduce a field to indicate the license of a data set

Until now our focus has been on data sets that are in the public domain, i.e. have been deposited within the infrastructure of INSDC. However, when thinking about the a more diversified structure of the AIRR Data Commons, data sets might come under a variety of licenses.
Taking the recommendations of RDA & CODATA - especially Principle 4: "State the rights transparently and clearly"  - into account, should we introduce a field in the AIRR Schema that indicates the license of a data set? And if yes, what would be the best level in the hierarchy for this? Sample? Repertoire?

I suggest the license of a dataset to be clearly stated at the sample
level.
On Fri, 26 Apr 2019 at 00:07 Christian Busse<EMAIL_ADDRESS>wrote:

Until now our focus has been on data sets that are in the public domain,
i.e. have been deposited within the infrastructure of INSDC. However, when
thinking about the a more diversified structure of the AIRR Data Commons
https://github.com/airr-community/common-repo-wg/blob/master/recommendations.md#airr-data-commons---a-system-of-distributed-repositories-supported-by-a-centralized-registry,
data sets might come under a variety of licenses.
Taking the recommendations of RDA & CODATA
https://www.rd-alliance.org/rda-codata-legal-interoperability-research-data-principles-and-implementation-guidelines-now

especially Principle 4: "State the rights transparently and clearly" -
into account, should we introduce a field in the AIRR Schema that indicates
the license of a data set? And if yes, what would be the best level in the
hierarchy for this? Sample? Repertoire?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/airr-community/airr-standards/issues/203, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB7DUU7JAJMMB3RZKP2VXLTPSITSRANCNFSM4HIRA6FA
.

--
Prof. Dr. Enkelejda Miho
aiNET GmbH, ETH spin-off
c/o Switzerland Innovation Park Basel Area AG
Hochbergstrasse 60C
4057 Basel
Mobile +41 76 469 91 91

@bussec I think for simplicity, the user should be able to specify a license at the study level, thus covering all data in the study. And also at the "sample" level as an override, which technically that would be at sequencing run as that's where the filenames are specified. I assume we are talking about informatic (digital data) license and not on organic material in a tube?

Yes, digital. I suggest to keep the option open on granularity. If the user
can specify the license at study level, this is fast done. I was more
focusing on the case where parts/samples of the same study might be
differently handled in terms.
On Thu, 5 Sep 2019 at 00:43 Scott Christley<EMAIL_ADDRESS>wrote:

@bussec https://github.com/bussec I think for simplicity, the user
should be able to specify a license at the study level, thus covering all
data in the study. And also at the "sample" level as an override, which
technically that would be at sequencing run as that's where the filenames
are specified. I assume we are talking about informatic (digital data)
license and not on organic material in a tube?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/airr-community/airr-standards/issues/203?email_source=notifications&email_token=AB7DUU44YR6NATIEYSPHB3LQIA2YDA5CNFSM4HIRA6FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD55HXWA#issuecomment-528120792,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7DUU3DN4SOEEOEHKVJKY3QIA2YDANCNFSM4HIRA6FA
.

--
Prof. Dr. Enkelejda Miho
aiNET GmbH, ETH spin-off
c/o Switzerland Innovation Park Basel Area AG
Hochbergstrasse 60C
4057 Basel
Mobile +41 76 469 91 91

No, that's the license for the API service. Data licenses likely need to be in the data itself.


Broken Google Ads Integration

After retrieving developer token, and test account ID, I tried to do Google Ads integration with "Test Google Ads Integration".
Tried with multiple account, still have the same issue.
The message:
Failed to connect to Google Ads. Please make sure you have provided the correct API info.
Btw, I deployed Serpbear using Docker inside Ubuntu droplet in Digitalocean.

I have no issue with connecting to GCP, since I'm using TLD in my ubuntu droplet, but stumbled in the Google Ads integration stage. Hope that @towfiqi can help assist.

I'm also running into the same issue, I'm using an FQDN for the install and redirect URL too. I wish I had come here earlier!

Hello dear friends,
I am unsuccessful in integrating SerpBear v 2.0.2 with Google Ads. The app is hosted on my personal server on a https subdomain proxied by Cloudflare.
I get the redurect_uri_mismatch error and if I refresh that page I get the invalid_grant error.
I have tried to have the Oath consent screen in test mode and as well as published.
I read and tried most of the fixes suggested on these pages.
NEXT_PUBLIC_APP_URL = Authorised redirect URIs in Google Cloud Platform, Credentials
I do not even know what to do next.
Could you please help me?
Thank you! @towfiqi
Alex

Hi , I can confirm that for local servers it seems it doesn't work, I've followed the guide here https://docs.serpbear.com/miscellaneous/integrate-google-ads until step 3, but the "Authorized redirect URIs" phase requires a TLD...
Tried with localhost, but the access was blocked by Google"device_id and device_name are required for private IP: ...".
Tried filling in my windows host file with testlocal.com and it got accepted as an authorized URI, but after filling in SerpBear's Step 1: Connect Google Cloud Project and authenticating it went through all the steps, until I got this message, seemingly from SerpBear:
"Error Saving the Google Ads Refresh Token . Details: redirect_uri_mismatch Redirected URL: https://testlocal.com:3008/api/adwords. Please Try Again!"
Don't know what else to do...

@teddyfresco I have connected the Google ads account multiple times with localhost URLs without any issues. Not sure why you are getting this message. Can you please try https://nip.io/

@teddyfresco I have connected the Google ads account multiple times with localhost URLs without any issues. Not sure why you are getting this message. Can you please try https://nip.io/

Hi towfiqi, thanks for your answer, just applied your suggestion (again with domain set on host file, and reverse proxying the local ip:port to nip.io, setting URI 1 to: https://nip.io:3008/api/adwords) but after trying authentication, this is the exact error:
Error 400: redirect_uri_mismatch
You cannot access this app because it is not Google OAuth 2.0 compliant.
If you developed the app yourself, record the redirect URI in Google Cloud Console.
Get details: redirect_uri=http://nip.io/api/adwords flowName=GeneralOAuthFlow
Tried also with serpbear.mydomain.com with a domain of mine, obviously adapting settings, same result.
I'm using caddy as a reverse proxy, could it be the culprit? It seems to be working for other purposes...

@teddyfresco Ensure your reverse proxy is setting X-Forward headers.

I am also hosting it on my Synology NAS using a reverse proxy. It is not working for me either.
I tried forwarding the X-forward headers but it's not working.
Any ideas?


backport #2292 and #2285 (child storage fix and contract using of it)

backport both changes to make contract work on v1.0

no the test is not required to pass for merging actually. But I don't know

we can wait for https://github.com/paritytech/substrate/pull/2299 as well

so #2299 is inside as well. @cheme review re-requested


Cannot update locale in upload field inside blocks

Describe the Bug
Issue: Localized Media in Blocks Fallbacks to Default Locale
Description:
When using localized content within a page collection that contains block fields, there is an issue with uploading images to a media collection in non-default locales. Specifically, within blocks (e.g., a CTA block that contains an Upload component for images), the image set for any locale other than the default one falls back to the image from the default locale (Spanish in this case).
Expected Behavior:
When setting an image in a non-default locale (e.g., English), the image for that specific locale should render, without falling back to the default locale’s image.
Actual Behavior:
1.	Setting an image in a non-default locale displays a success toast (Image updated correctly).
2.	Debugging the beforeChange or afterChange hooks shows that the correct image ID is stored for the localized field.
3.	However, the frontend renders the image from the default locale (Spanish), even though the image ID for the current locale is correct.
4.	The issue only occurs within blocks. Outside of blocks, localized media works as expected.

 This issue seems to be only inside blocks field , because otherwise it works perfectly.

Environment:
•	Localization: Enabled for blocks (localized: true)
•	Default Locale: Spanish
•	Affected Locale(s): All non-default locales
•	The issue occurs only within block fields (e.g., CTA block with Upload).

Let me know if you need further details or edits!
To Reproduce
Steps to Reproduce:
1.	Enable localized: true for the block field in a page collection.
2.	Create a block (e.g., CTA block) that includes an Upload component tied to a media collection.
3.	Upload an image for the default locale (Spanish) and verify that it renders correctly.
4.	Switch to another locale (e.g., English), upload a different image, and check the frontend rendering.
5.	Observe that the image from the default locale (Spanish) is displayed, even though the correct image ID is set for the non-default locale.

Payload Version
^2.19.3
Adapters and Plugins
@payloadcms/plugin-cloud-storage
https://github.com/user-attachments/assets/c4eced92-07b9-4d1d-bb80-a53a22ea6fd4

I see you are using Postgres and on outdated version 2.19.3. We ported a lot of Postgres fixes from 3.0 to 2.0 here https://github.com/payloadcms/payload/pull/8468 which includes fixes for this. Try the latest version https://github.com/payloadcms/payload/releases/tag/v2.30.3

Could you please clarify? We updated to the latest version of payload cms and still have the same problem. What postgres version should be used? We use a postgres adapter for payloadCMS

Since we use the caret version of PayloadCMS , it should work with all the minor versions

@payloadcms/db-postgres specifically should be on 0.8.9

Since we use the caret version of PayloadCMS , it should work with all the minor versions

I'd recommend to pin it explicitly, it's not necessary that your actual version is latest from the caret.

Yes, that solved our problem. Thanks!


Allow binding server to both ipv4/ipv6 interfaces

Starting prometheus_exporter -v -b ALL -p 8080 will include the log message:
2022-12-26 16:54:44 -0600 Starting prometheus exporter on ALL:8080
[2022-12-26 16:54:44] INFO  Listening on both <IP_ADDRESS>/:: network interfaces
[2022-12-26 16:54:44] INFO  WEBrick 1.7.0
[2022-12-26 16:54:44] INFO  ruby 2.7.2 (2020-10-01) [arm64-darwin21]
[2022-12-26 16:54:44] INFO  WEBrick::HTTPServer#start: pid=86361 port=8080

netstat:
$ netstat -an -ptcp | grep LISTEN | grep 8080
tcp4       0      0  *.8080                 *.*                    LISTEN
tcp6       0      0  *.8080                 *.*                    LISTEN

I think @xfalcox noticed a similar issue, in the original versions we supported a bind all and then it was no longer there.
I am happy with this change, but can we update the documentation as well?

Yep, i will update the docs

Yes, had the same problem when updating from .5 to the latest version of this gem in Discourse.
Over there I fixed this problem by passing * as the bind address, which in glibc means that we will bind to both IPv4 and IPv6. However that doesn't work if someone is using musl. So 👍 for this approach from me.

Should be ready to go!

arrives late Hey friends, any plans to release an updated version with this included? I was confused as to why it wasn't working only to release it's not in the 2.0.6 version that is most recent at this time. 😸

will try to post a new version tomorrow!
On Thu, Jan 12, 2023 at 4:20 AM Cory G Watson @.***>
wrote:

arrives late Hey friends, any plans to release an updated version with
this included? I was confused as to why it wasn't working only to release
it's not in the 2.0.6 version that is most recent at this time. 😸
—
Reply to this email directly, view it on GitHub
https://github.com/discourse/prometheus_exporter/pull/260#issuecomment-1379218995,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABIXNCV5LOXQBXT2JMYFTWR3TUXANCNFSM6AAAAAATJ6GVHM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Appreciated! I just set to pull from head, but lost some time to being confused why the -b ANY wasn't working for me, only to realize it wasn't released yet. 😆


[701] Pin Github actions to the major version only

Trello card
https://trello.com/c/BYSSVCNY
Context
Reduce dependabot PRs
Changes proposed in this pull request
Guidance to review

 Remove github action minor version pinning


Mbedtls 2.19.1 fails to build from source with MBEDTLS_ZLIB_SUPPORT

Description

Type: Bug

Bug
OS  : any
mbed TLS build:   2.19.1
Configuration: MBEDTLS_ZLIB_SUPPORT
Compiler and options: gcc/any
Steps to reproduce
Uncomment MBEDTLS_ZLIB_SUPPORT from the default config.h and attempt to build:
[ 19%] Building C object library/CMakeFiles/mbedtls.dir/ssl_tls.c.o
/home/jack/sw/mbedtls/library/ssl_tls.c: In function ‘mbedtls_ssl_derive_keys’:
/home/jack/sw/mbedtls/library/ssl_tls.c:1862:9: error: ‘session’ undeclared (first use in this function); did you mean ‘setsid’?
     if( session->compression == MBEDTLS_SSL_COMPRESS_DEFLATE &&
         ^~~~~~~
         setsid

Looks to have been introduced between 2.18.0 and 2.19.0 though I could not find the exact commit. I believe this should now be ssl->session_negotiate->compression

Hi @jack-fortanix Thank you for your input!
AS youc an see from the ChangeLog ZLIB support has been deprecated. So you should have received a build error, but it should have been a deprecation error, and probably not this error.
I would actually rename this bug to: "Deprecated ferature MBEDTLS_ZLIB_SUPPORT still used in some locations", or something better in grammar


Add ProxyBodyHandler for sidecar to intercept the body

For post request to do the body validation against the specification on the sidecar. The handler will duplicate the ByteBuffer and but still keep the channel to transfer the request body to the downstream backend API.

 Add ProxyBodyHandler for sidecar to intercept the body light-4j master

 move producer entities to the kafka-entity module light-kafka master


Rototrans tuto jupyter

Also, remove STL parts of Kinova arm which has nothing to do here.

I reviewed the notebook here https://app.reviewnb.com/Ipuch/bioptim-zero-to-hero/files/.
Maybe it would be better to configure on your repo to see the comments.


Fix typo

[x] Completed PR - Has all features or fixes that it was meant to and isn't half done

What is fixed
Spelling of Contributing

While you're at it fix the badge URLs. And make the credits not checkboxes.

Fix them? Whats wrong with them?

They point to the badges/shields repo instead of this one.

ah
gotcha

Merging


Deployment of cutoutcass

Deployment should also do these things on cutoutcass-0
CREATE KEYSPACE cutouts WITH replication = {'class':'SimpleStrategy', 'replication_factor' : 3 };

use cutouts;

CREATE TABLE IF NOT EXISTS cutouts (
    cutoutId      ascii,
    objectId      bigint,
    imjd          int,
    cutoutimage   blob,
    PRIMARY KEY (imjd, cutoutId)
);

Does this need to be repeated on every instance, or just one instance in the cluster?


upgrade kubernetes client-go api to apps/v1

1.Describe what this PR did
Upgrade kubernetes client-go api to apps/v1, including deployments and statefulsets.
2.Does this pull request fix one issue?
During my debug for the higher kubernetes version like v1.9.x, I found some issues. Since the client-go api apps/v1 is stable, I made the upgration.
3.Describe how you did it
Upgrade client-go api to apps/v1.
4.Describe how to verify it
/usr/local/go/bin/go test -v github.com/asobti/kube-monkey/victims/factory/deployments -run "^TestNew|TestInvalidIdentifier|TestInvalidMtbf$"
ok  	github.com/asobti/kube-monkey/victims/factory/deployments	0.088s

/usr/local/go/bin/go test -v github.com/asobti/kube-monkey/victims/factory/deployments -run "^TestEligibleDeployments|TestIsEnrolled|TestIsNotEnrolled|TestKillType|TestKillValue$"
ok  	github.com/asobti/kube-monkey/victims/factory/deployments	0.094s

/usr/local/go/bin/go test -v github.com/asobti/kube-monkey/victims/factory/statefulsets -run "^TestNew|TestInvalidIdentifier|TestInvalidMtbf$"
ok  	github.com/asobti/kube-monkey/victims/factory/statefulsets	0.092s

/usr/local/go/bin/go test -v github.com/asobti/kube-monkey/victims/factory/statefulsets -run "^TestEligibleStatefulSets|TestIsEnrolled|TestIsNotEnrolled|TestKillType|TestKillValue$"
ok  	github.com/asobti/kube-monkey/victims/factory/statefulsets	0.090s

5.Special notes for reviews
None

Already approved in #53, close when accepted.


CUDA out of memory with 8.00GB

Hello! 大佬你们好！
I recieve an error when I tried to run the dataset of yours b1.mp4  and the error is RuntimeError: CUDA out of memory. Tried to allocate 1.48 GiB (GPU 0; 8.00 GiB total capacity; 3.86 GiB already allocated; 628.69 MiB free; 5.68 GiB reserved in total by PyTorch).
Is my GPU memory insufficient? I supposed it would run with 8Gb of GPU MEM. Does it need more MEM?
Looking forward to your reply if you guys are free, thank you!

Now I've successfully run the program via output of pha, but another error ouccured like IndexError: Idx: 389 out of length: 416 , the video indeed has been ouput but the quality was not that good.

How can I avoid it?

problem solved, thank you.


no license

Hey.  Your code is cool, but you haven't published it under a license, so I can't use it.  Can you indicate a license?

Sure, done so in #4!


Questions about translation...

I'm trying to add a Chinese translation to this mod, and I'm curious about: (1.20.1 Forge)
1.Where can I find these words ↓ in game? I can't find a screen that contains them, thus unable to check my work.

2.I can't find these words ↓ in the lang json. Are they editable?

All the translation strings starting with "MineMenu" is never shown in-game. They're all used for the Minecraft text-to-speech functionality.
The gui.delete translation string, is for the "Delete" button when deleting something from the Radial Menu.
Hope that answers your question! :)

thx for that. So I can't translate lines in Q2 right?

Unfortunately not atm. no.


Lifecycle Tasks

On top of Tasks API (#1974), we can support specific task prefixes init: (blocker), startup: (non-blocker), and close: that will be automatically called by nitro during startup and server close.

@pi0 ... will this include scheduled (cron) tasks as well?

Yes but tracker for that is https://github.com/unjs/nitro/issues/1974


ExpressionChangedAfterItHasBeenCheckedError on thread message

Describe the bug
Just playing with https://dev.algorea.org/en/a/3862371645338198698;p=7528142386663912287,7523720120450464843;a=0
I get in the console:
Error: NG0100: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: '12 minutes ago'. Current value: '13 minutes ago'. Expression location: ThreadMessageComponent component. Find more at https://angular.io/errors/NG0100
Error: NG0100: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: '11 minutes ago'. Current value: '12 minutes ago'. Expression location: ThreadMessageComponent component. Find more at https://angular.io/errors/NG0100

I didn't get that for a while, I think that's fixed.


scalability: intro

What issue does this PR solve?

close #11

What is changed:
Write down the first version of Scalability.
https://github.com/pingyu/tikv-dev-guide/blob/scalability-intro/src/understanding-tikv/scalability/intro.md

Thanks for your contribution. I'm going through now.

Sorry there is nothing to comment >_<

Thanks for your review. Suggestions applied, PTAL~ @andylokandy

@pingyu Thank you!


🚀 Feature: Make Github Discovery more efficient

🔖 Need
It seems like the current implementation of Github Discovery does not work the way it says it does in some cases, and doesn't work the way it seems like it should in some.
Lets say we have a catalog section in app-config.yaml that looks like this
catalog:
  rules:
    - allow: [Component, System, API, Group, User, Resource, Location, Template]
  providers:
    github:
      backstageCatalog:
        organization: 'myOrg'
        catalogPath: '/**/*.yaml'
        filters:
          repository: 'backstage-catalog'

Problem 1 catalogPath doesnt support wildcards
The discovery ends up creating a GraphQL request.  In there, we pass in the catalogPathRef which in this case would be HEAD:**/*.yaml.  We use it to create this property in the GraphQL query
catalogInfoFile: object(expression: $catalogPathRef) {
    __typename
    ... on Blob {
        id
        text
    }
}

The problem is that the Github GraphQL API says that expression must be "A Git revision expression suitable for rev-parse."  So wildcards don't work and catalogInfoFile will always return null.  If the config's catalogPath value was catalog-info.yaml, it would work fine as expected.
Problem 2 Filters are post-query. This kills the API rate limit.
To be fair, my organization has thousands of repos, but when we say repository: backstage-catalog this query doesnt search for just the backstage-catalog repo.  It searches for every repo in the organization, and then filters the results to return ones that match the filters.  This same problem exists for all of the filters.  This can result in API rate limits being hit very quickly, even before a full scan is done.
🎉 Proposal
Proposal for "Problem 1 catalogPath doesnt support wildcards"
I haven't found a way to get the query to accept wildcards. Unless there is something I'm missing, I think the docs just need to be updated to reflect actual behavior.
Proposal for "Problem 2 Filters are post-query"
I played with Githubs GraphQL Explorer a bit, and found out we can do a search query that builds the same respones object without scanning ever single repo.
Of course there may be situations where we do want to scan ever single repo in an org, but if the person has a filter set, I think the search query will let us apply that filter to the query, greatly easing up on the API usage.  To use the repository filter, we can do
query explicitSearchQuery($catalogPathRef: String!, $cursor: String) {
  search(
    query: "repo:myOrg/backstage-catalog"
    type: REPOSITORY
    first: 100
  ) {
    nodes {
      __typename
      ... on Repository {
        name
        catalogInfoFile: object(expression: $catalogPathRef) {
          __typename
          ... on Blob {
            id
            text
          }
        }
        url
        isArchived
        defaultBranchRef {
          name
        }
        repositoryTopics(first: 100) {
          nodes {
            ... on RepositoryTopic {
              topic {
                name
              }
            }
          }
        }
      }
    }
  }
}

The big difference here is in the search query param: "repo:myOrg/backstage-catalog".  This makes the API only search for repos matching that specific pattern.  Instead of searching for thousands of repos then performing n+i searches for their properties, we do just 1.
We could also do this with other filter values like topic.include.  query: "org:myOrg topic:backstage-include".
〽 Alternatives
Ive looked, and can't find any other feasible solutions for this.  The current search for provisioning github entities is killing our rate limits, and the wildcard system doesnt seem to be working, at all, nor can I find a workaround.
❌ Risks
These changes cant be made in just the query string.  Because we need to build out the entire query string ahead of time, we will need to implement this at a higher level.  We'd also need to remove the current post query filter.
👀 Have you spent some time to check if this RFC has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you using the latest version of the GithubEntityProvider? Iirc there's been some recent development there to add support for wildcards. The upfront repository filtering should only be active if you set isCatalogInfoFilePresent to true in config, with it otherwise passing it on for discovery in the processors.
I do think it makes sense to make the filtering a bit smarter and move whatever we can to request filters rather than do all the filtering on the response. Happy to open that up for contributions if anyone wants to chip in there.
I'd also like to point out that if all you want to do is bring in entities from one or a few known repositories, then you're better off using an explicit location configured in catalog.locations rather than the provider. See https://backstage.io/docs/features/software-catalog/configuration#static-location-configuration. As long as it's supported by the integration you can use wildcards there as well.

Thank you for the thoughtful reply, Rugvip.

Are you using the latest version of the GithubEntityProvider? Iirc there's been some recent development there to add support for wildcards. The upfront repository filtering should only be active if you set isCatalogInfoFilePresent to true in config, with it otherwise passing it on for discovery in the processors.

I've looked and I don't see this as an option in the config.  Searching in the repo, it looks like that boolean is only used in GithubEntityProviderCatalogModule.ts and is conditionally set based on if the blob is found after the query. I don't see where its reading it from a config at all, but I could also be looking in the wrong place.

I'd also like to point out that if all you want to do is bring in entities from one or a few known repositories, then you're better off using an explicit location configured in catalog.locations rather than the provider. See https://backstage.io/docs/features/software-catalog/configuration#static-location-configuration. As long as it's supported by the integration you can use wildcards there as well.

Yeah, we initially tried the explicit location list, but its quickly getting out of hand and becoming hard to scale. The wildcards are kind of a secondary issue that I stumbled upon while looking for ways to make things work for our use case. We're fine requiring repo owners to create explicitly named catalog-info.yaml, api-info.yaml, etc files, if we can set the entity provider up to only scan repos with a backstage topic.

I do think it makes sense to make the filtering a bit smarter and move whatever we can to request filters rather than do all the filtering on the response. Happy to open that up for contributions if anyone wants to chip in there.

I'd be willing to put in the work for this. Ill dive into the contributing guide, but are there any objections to my initial search query proposal as a starting point?

Are you using the latest version of the GithubEntityProvider? Iirc there's been some recent development there to add support for wildcards. The upfront repository filtering should only be active if you set isCatalogInfoFilePresent to true in config, with it otherwise passing it on for discovery in the processors.

I've looked and I don't see this as an option in the config. Searching in the repo, it looks like that boolean is only used in GithubEntityProviderCatalogModule.ts and is conditionally set based on if the blob is found after the query. I don't see where its reading it from a config at all, but I could also be looking in the wrong place.

Sorry, validateLocationsExist is the name of the configuration key. isCatalogInfoFilePresent is indeed just internal as you say.

I'd be willing to put in the work for this. Ill dive into the contributing guide, but are there any objections to my initial search query proposal as a starting point?

Nice, thanks! Proposal sounds good, just as long as it has no effect on the final result set anything that we can move to the query should be a benefit.

The new GitHub Search allows queries like org:my-org path:catalog-info.yaml. Could that be used to power this in fewer requests?
The docs suggest it has a lower request limit, but supports up to 1000 results per request

@tahasi they're not deprecated, wildcards are still a thing. The refactor you're pointing to added the possibility of setting the validateLocationsExist to do an upfront check though, in which case wildcards are no longer supported. It's an optimization you can use if you don't need the wildcard discovery. More over at https://backstage.io/docs/integrations/github/discovery#configuration

I apologize, I lost track of this ticket.  I initially started on work to refactor this in keeping with current backstage patterns.  We ended up pivoting and making a very opinionated entity provider that is less configurable but performs a pretty optimized GQL query.

Thank you @Rugvip and @MarcFogleman for the responses. I'm glad that the catalogPath support for wildcard characters is not deprecated. However, it's not clear whether the GithubEntityProvider currently supports catalogPath's with wildcards. My current use of wildcards with the GithubEntityProvider results in every catalogInfoFile in the response being null.
I'm skeptical that the current GQL query can support wildcards. The response's catalogInfoFile schema appears to be an object and not an array that may hold a list of matching files.

You're right that it doesn't. We just call out like this optimistically, and then fall back to the slow path (with UrlReader as needed). So it's potential waste, but a known way that it's written at this point.

Thank you for the response, @freben.
I'm not seeing the fallback behavior that you mention.
My app-config.yaml has the following catalog config: (names changed to protect the innocent)
catalog:
  providers:
    github:
      MonoProvider:
        host: github.example.com
        organization: 'Mono'
        catalogPath: '/src/apps/**/*catalog-info.yaml'

Is there some field I need to add to the config to get the fallback behavior? Is UrlReader a plugin that I need to add?

Should this be reopened? I don't think it got properly resolved.

I think it should.

Alright

bump

Wanted to add few comments in here.
I have tried adding the filter for branch.
but i can see that in the code It is checking the catalogPath in default branch.
How can i fetch catalog-entities from other branches?

Hi! Just wanted to clarify if I didn't miss anything and if wildcard filtering is still not supported in GitHub Discovery?
I tried it with such a config, but unfortunately, it never found a catalog file.
    catalog:
      providers:
        github:
          someOrg:
            organization: 'some-org'
            catalogPath: '/**/catalog.*.yaml'
            filters:
              repository: '.*'
            schedule:
              # supports cron, ISO duration, "human duration" as used in code
              frequency: { hours: 4 }
              # supports ISO duration, "human duration" as used in code
              timeout: { minutes: 3 }

My Backstage version is 1.31.1

same. I resorted to creating an envelope/location-type file which includes the path of all catalog-info within subfolders.


Readme chet

updated queryParams in the readme. Also some minor grammar touches and a couple type-o's

Thanks.


I fixed the wget issue.

I fixed the wget issue with setup.py by adding wget to the install_requires.

yo sick kk

Just a note that I personally use urllib for this functionality which requires no dependency to be installed.

Just a note that I personally use urllib for this functionality which requires no dependency to be installed.

make a pr ill accept it!

Here are the lines: https://github.com/xloem/test_matrix_bot/blob/main/module_rwkv.py#L17L19
It again unfortunately looks like the code is untested as there is a case typo preceding it.
            if not os.path.exists(fn):
                import urllib.request
                urllib.request.urlretrieve(model_path, fn)


Exposing the exit status of pash-compiled pipelines

This PR tries to add support for returning the exit status of pash-compiled pipelines. These changes fix the issue in general but there is still a failure with --parallel_pipelines since this flag executes pipelines in the background and therefore their exit status is lost.
@mgree, @tammam1998 the problem is here: https://github.com/binpash/pash/blob/cat-redir-fail/compiler/pash_runtime.sh#L233. Is there any way to address this without losing support for parallel pipelines?

Another thought: Maybe we have a check that disables parallel-pipelines in the context of &&, ||, [ ]. This does not address the issue with expanding $? though. Also, the final dataflow region can also have the parallel-pipelines disabled.

Also set -e.

@tammam1998, @dkarnikis This change might affect the performance of for-loop benchmarks with parallel pipelines since it disables the parallel_pipelines optimization for the final command of a script, since we might need its exit code. It can be easily addressed by concluding scripts with a final command, e.g., echo done, to make sure that the for loops will execute in parallel.


Hello microverse

@ghreza-crypto looks good to me 😄 kindly copy those lines and paste them in the Pull request description, you can edit it by clicking on the 3 dots on the top right 👍


Add variants of openPin, closePin, and withPin that don't fail when already opened/closed

Closed in 0622ed5e09e7e2e703f3aa8d584a9b9e4f83c44e.
Note that, rather than adding non-failing variants, I just made all of these actions not fail when the pin is already opened/closed (in the portable DSL, that is). I think that in almost every case, the programmer would want the non-faulting version.
The Linux DSL has variants that fail or do not fail, depending on whether you want to follow the strict Linux behavior or not.


workbox-broadcast-cache-update documentation outdated?

Library Affected:
documentation / workbox-broadcast-cache-update
using SDK /4.1.1/workbox-sw.js
Browser & Platform:
all browsers
Issue or Feature Request Description:
The docs @ https://developers.google.com/web/tools/workbox/modules/workbox-broadcast-cache-update use the following syntax:
new workbox.broadcastUpdate.Plugin('api-updates')
That does not seem to work. Using the following does:
new workbox.broadcastUpdate.Plugin({channelName: 'api-updates'})

Thanks—I'll file a PR to fix this in the repo that hosts our documentation.


Example Dockerfile for Alpine seems to include an error

When I try to compile for Alpine using a modification of the example Dockerfile, I get this error:
------
 > [builder 6/6] RUN cargo build --release --target x86_64-unknown-linux-musl --bin app:
#20 1.612 error: no bin target named `app`

Here's my Dockerfile (note the only changes are basically to install some MySQL client libs)
# Using the `rust-musl-builder` as base image, instead of
# the official Rust toolchain
FROM ekidd/rust-musl-builder:1.51.0 AS chef
USER root
RUN cargo install cargo-chef
WORKDIR /app

FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM chef AS builder
COPY --from=planner /app/recipe.json recipe.json
# Notice that we are specifying the --target flag!
RUN cargo chef cook --release --target x86_64-unknown-linux-musl --recipe-path recipe.json
COPY . .
RUN apt-get update -y
RUN apt-get install -y default-libmysqlclient-dev
RUN cargo build --release --target x86_64-unknown-linux-musl --bin app

FROM alpine AS runtime
RUN apk update
RUN apk add mysql mysql-client
RUN addgroup -S myuser && adduser -S myuser -G myuser
COPY --from=builder /app/target/x86_64-unknown-linux-musl/release/app /usr/local/bin/
USER myuser
CMD ["/usr/local/bin/app"]

When I remove the --bin app flag, and change CMD to ["/usr/local/bin"], I then get
docker: Error response from daemon: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "/usr/local/bin": permission denied: unknown.
any help is appreciated!

The error doesn't seem to be related to the Dockerfile - how is your binary named in your Cargo.toml?
The error seems to suggest it's not named app.

ah, that explains the --bin error... but now I'm seeing exec user process caused: no such file or directory
I'm sorry; probably not worth your time, but this is my updated Dockerfile—you don't happen to see anything else I've misconfigured do you?
# Using the `rust-musl-builder` as base image, instead of
# the official Rust toolchain
FROM ekidd/rust-musl-builder:1.51.0 AS chef
USER root
RUN cargo install cargo-chef
WORKDIR /app

FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM chef AS builder
COPY --from=planner /app/recipe.json recipe.json
# Notice that we are specifying the --target flag!
RUN cargo chef cook --release --target x86_64-unknown-linux-musl --recipe-path recipe.json
COPY . .
RUN apt-get update -y
RUN apt-get install -y default-libmysqlclient-dev
RUN cargo build --release --target x86_64-unknown-linux-musl --bin api-service

FROM alpine AS runtime
RUN apk update
RUN apk add mysql mysql-client
RUN addgroup -S mygroup && adduser -S myuser -G mygroup
COPY --from=builder /app/target/x86_64-unknown-linux-musl/release/api-service /usr/local/bin/
USER myuser
ENV ROCKET_ADDRESS=<IP_ADDRESS>
EXPOSE 8000
CMD ["/usr/local/bin/api-service"]

Thank you for your help, still getting my sea legs with some of this stuff

At what stage are you getting that error, specifically?


WIP: sshshut down init

Once we get the local DM settled, we need to copy paste those updates to vaquero-examples/vagrant


Wrong backend URL when backend redirects to URL without trailing /

Do you want to request a feature or report a bug?

Bug

What did you do?
I setup a health check to a backend which does a redirect (I disabled following redirects afterwards as I don't need it but traefik behaviour still seems wrong, see below).
I used this config:
http:
  routers:
    airflow:
      rule: "Host(`airflow.domain`)"
      service: airflow
  services:
    airflow:
      loadBalancer:
        servers:
          - url: "http://host.domain:80/"
        healthCheck:
          path: /health

What did you expect to see?
Redirect produces a successful URL for traefik to check, health check succeeds.
What did you see instead?
It seems that traefik constructs a wrong URL after the redirect, the logs show the following error:
Nov 12 16:09:02 host cdeb4208a769[3334]: time="2020-11-12T15:09:02Z" level=warning msg="Health check still failing. Backend: \"airflow@file\" URL: \"http://host.domain:80/\" Reason: HTTP request failed: Get \"https://airflow.domainhealth\": dial tcp: lookup airflow.domainhealth on dns.ser.ver.ip:53: no such host"

It seems that the path (/health) and the resulting url of the redirect (https://airflow.domain, the Location header is without a trailing slash) were incorrectly merged so that the / was skipped.
Output of traefik version: (What version of Traefik are you using?)
v2.3.2 with the slight modification to use a lego version which handles our non-RFC compliant ACME endpoint...
see https://github.com/traefik/traefik/compare/v2.3.2...linkvt:2.3.2-optional-angle-brackets?expand=1 and
https://github.com/go-acme/lego/compare/master...linkvt:4.0.1-optional-angle-brackets

What is your environment & configuration (arguments, toml, provider, platform, ...)?
Nothing special.

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)

Thanks for your help, I will also try to debug this next week if I have time.

@linkvt Thanks for your interest in Traefik.
I tried to reproduce the issue but without any success yet. In order to go further on this, could you provide:

Your static configuration
Complete log output (in debug mode)
Complete dynamic configuration which allows a reproducible test case.

Hi @jspdown , I'm very sorry to have wasted your time but I would never have expected that somebody is able to do a http->https redirect wrong, I guess I will never trust a reputable open source project less...
I couldn't believe it at first too when creating that issue but I should have verified that manually myself before:
$ curl  http://host.domain/health -I
HTTP/1.1 301 Moved Permanently
...
Location: https://host.domainhealth
...

Thanks and sorry again, closing this ticket.


esp_wifi_restore() adds new keys to NVS (IDFGH-8446)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v4.4.2
Operating System used.
macOS
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
No response
Development Kit.
ESP32-S3-MINI-1-N8
Power Supply used.
USB
What is the expected behavior?
Calling esp_wifi_restore() is expected to restore persistent NVS settings to default values, i.e., it is expected to erase keys that were set by calling the documented functions.
What is the actual behavior?
esp_wifi_restore() adds keys to NVS, including keys that are not set by calling documented functions (e.g. "sta.pmk", "sta.phym") and keys that seem to belong to older esp-idf versions ("auto.conn").
Steps to reproduce.

Step: erase_flash
Step: set configuration
Step: call esp_wifi_restore

Compare NVS data after step 2 and 3.
Debug Logs.
No response
More Information.
No response

@NoNullptr can we have the application code which your are using to see this bug,Also can you mention the commit ID you are using.

hi @NoNullptr esp_wifi_restore() will restore persistent NVS settings to default values. "sta.pmk", "sta.phym" and "auto.conn" are setting to default values when erase_flash is calling.

hi @NoNullptr esp_wifi_restore() will restore persistent NVS settings to default values. "sta.pmk", "sta.phym" and "auto.conn" are setting to default values when erase_flash is calling.

I expect esp_wifi_restore() to remove the key-value pairs from NVS that were previously written, instead of explicitly writing defaults that are hard-coded in the codebase already. This would be more robust in case these keys or the default values change in the future.
"auto.conn" specifically seems to be an obsolete key from an older ESP-IDF (it was used in v3.3, but deprecated and then removed, see esp_wifi_set_auto_connect). I don't know if the internal WiFi code still uses this key, but in v4.4.2 only esp_wifi_restore() seems to write it – normal setup and connection doesn't.

@NoNullptr can we have the application code which your are using to see this bug,Also can you mention the commit ID you are using.

Here is a simple example that illustrates the behaviour. It does the following:

erase all net80211 NVS keys
connect as STA to WiFi
print all net80211 NVS keys
call esp_wifi_restore()
print all net80211 NVS keys again

On v4.4.2 the behavior is currently this:

After successfully connecting to WiFi, there are 18 keys on NVS.
After calling esp_wifi_restore, there are 40 keys on NVS, including keys for AP, which wasn't set or started, and keys that seem to belong to v3 ESP-IDF ("auto.conn").

When changing the configuration, only the changed parameters are stored on NVS, for example the key "sta.phybw" is only stored when esp_wifi_set_bandwidth() was called. However, esp_wifi_restore() stores all default values (and some more) on NVS, instead of erasing them.
#include <esp_event.h>
#include <esp_log.h>
#include <esp_netif.h>
#include <esp_wifi.h>
#include <freertos/FreeRTOS.h>
#include <freertos/task.h>
#include <nvs_flash.h>

const char TAG[] = "test";

wifi_config_t wifi_config_sta = {
    .sta = {
        .ssid = "",
        .password = "",
    }
};

void dump_nvs_net80211()
{
    nvs_entry_info_t info;
    nvs_iterator_t it;

    it = nvs_entry_find("nvs", "nvs.net80211", NVS_TYPE_ANY);
    while (it != NULL) {
        nvs_entry_info(it, &info);
        printf("nvs.net80211 has key \"%s\"\n", info.key);
        it = nvs_entry_next(it);
    }
}

void erase_nvs_net80211()
{
    nvs_handle_t handle;
    nvs_entry_info_t info;
    nvs_iterator_t it;

    ESP_ERROR_CHECK(nvs_open("nvs.net80211", NVS_READWRITE, &handle));
    it = nvs_entry_find("nvs", "nvs.net80211", NVS_TYPE_ANY);
    while (it != NULL) {
        nvs_entry_info(it, &info);
        ESP_ERROR_CHECK(nvs_erase_key(handle, info.key));
        it = nvs_entry_next(it);
    }
    ESP_ERROR_CHECK(nvs_commit(handle));
    nvs_close(handle);
}

void app_main(void)
{
    ESP_ERROR_CHECK(nvs_flash_init());
    erase_nvs_net80211();

    ESP_ERROR_CHECK(esp_netif_init());
    ESP_ERROR_CHECK(esp_event_loop_create_default());

    esp_netif_t *netif_sta = esp_netif_create_default_wifi_sta();
    wifi_init_config_t wifi_init_config = WIFI_INIT_CONFIG_DEFAULT();
    ESP_ERROR_CHECK(esp_wifi_init(&wifi_init_config));

    ESP_ERROR_CHECK(esp_wifi_set_mode(WIFI_MODE_STA));
    ESP_ERROR_CHECK(esp_wifi_set_config(WIFI_IF_STA, &wifi_config_sta));

    ESP_ERROR_CHECK(esp_wifi_start());
    vTaskDelay(pdMS_TO_TICKS(5000));

    dump_nvs_net80211();
    vTaskDelay(pdMS_TO_TICKS(500));

    ESP_ERROR_CHECK(esp_wifi_restore());
    vTaskDelay(pdMS_TO_TICKS(5000));

    dump_nvs_net80211();
    vTaskDelay(pdMS_TO_TICKS(500));
}

hi @NoNullptr  Yes, esp_wifi_restore() stores all default values on NVS, instead of erasing them. When changing the configuration, only the changed parameters are stored on NVS, the rest will not be changed or erased.

hi @NoNullptr Yes, esp_wifi_restore() stores all default values on NVS, instead of erasing them. When changing the configuration, only the changed parameters are stored on NVS, the rest will not be changed or erased.

I described that already in the bug report. It is reasonable to expect esp_wifi_restore() to erase the current configuration from NVS, not to store new values (even if they're the internal defaults, and in particular not values that seem to be part of an older ESP-IDF version).

`/**

@brief  Restore WiFi stack persistent settings to default values

This function will reset settings made using the following APIs:

esp_wifi_set_bandwidth,

esp_wifi_set_protocol,

esp_wifi_set_config related

esp_wifi_set_mode

@return

ESP_OK: succeed

ESP_ERR_WIFI_NOT_INIT: WiFi is not initialized by esp_wifi_init
*/
esp_err_t esp_wifi_restore(void);`
hi @NoNullptr esp_wifi_restore() just restore wifi nvs setting to default values.

I want to make prov work using wifi as multiple instance, but it doesn't seem possible to do it like this also because of the nvs.
    WiFiProvClass* pWiFiProv= nullptr;
    WiFiClass* pWiFi= nullptr;
    uint32_t cntNeworks = 0;

I listed all storages from the documentation.
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv417esp_wifi_get_modeP11wifi_mode_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv416esp_wifi_restorev
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv419esp_wifi_scan_startPK18wifi_scan_config_tb
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv424esp_wifi_scan_get_ap_numP8uint16_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv428esp_wifi_scan_get_ap_recordsP8uint16_tP16wifi_ap_record_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv415esp_wifi_get_psP14wifi_ps_type_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv421esp_wifi_get_protocol16wifi_interface_tP7uint8_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv422esp_wifi_get_bandwidth16wifi_interface_tP16wifi_bandwidth_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv420esp_wifi_get_channelP7uint8_tP18wifi_second_chan_t
flash	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv420esp_wifi_set_countryPK14wifi_country_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv416esp_wifi_get_mac16wifi_interface_tAL6E_7uint8_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv424esp_wifi_get_promiscuousPb
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv431esp_wifi_get_promiscuous_filterP25wifi_promiscuous_filter_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv436esp_wifi_get_promiscuous_ctrl_filterP25wifi_promiscuous_filter_t
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv423esp_wifi_ap_get_sta_aidAL6E_K7uint8_tP8uint16_t
flash	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv425esp_wifi_set_country_codePKcb
out	https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv420esp_wifi_get_countryP14wifi_country_t
these below doesn't mentioned that has been stored. Maybe it happens when the wifi starts...
https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv417esp_wifi_set_mode11wifi_mode_t
https://docs.espressif.com/projects/esp-idf/en/v4.4.2/esp32/api-reference/network/esp_wifi.html#_CPPv421esp_wifi_set_protocol16wifi_interface_t7uint8_t
...
They are stored in to the nvs to reboot faster and to minimize the write cycles on the memory because the are limited. Every AP or STA does that.

@kriegste You might be interested in this too.

master: ce1ab731be47fa2d509739b2ce7f0244ce836950
v4.4: 53c186af263ac76fb0a45e6f63037c43a86c3736
v4.3: 53c186af263ac76fb0a45e6f63037c43a86c3736

Close this since it is not a bug.


Added expose port for applications

I have a server that's using Apache2 as a proxy, I couldn't figure out a way to expose a port for an application, so I did this.
It could use some more checking; like when the FQDN is empty, it shouldn't allow an application to be launched when the port is in use.
Also, it should be added to the built-in services 🤷♂

Thanks for the PR!
So you intend to be able to use your proxy instead of Coolify's proxy, right? And you are just exposing the docker container's port.
Your Apache2 is configured automatically or do you configure it manually?

Yep, that's the plan. My Apache2 is configured manually; mapping subdirectories to ports.
If there's a way to make Coolify's proxy handle the current apps I have installed (even mapping to subdomains is ok), that would be great.

I'm planning to have a feature where you can add labels to containers, and Coolify will create a reverse proxy based on these labels so that you can integrate any containers.

Unfortunately the apps I've currently got aren't containerised :(

I've been playing around with different ways I might be able to get this to work with my setup.
Nothing has worked yet, but I think allowing the exposed ports for the Collify proxy to be changed could help; I could forward all subdomains to the Coolify proxy.

Exposing ports would probably also help when wanting to use something like Cloudflare Tunnels, as they always terminate at specific ports (afaik) :)


Allow following a repo releases in 2 clicks

Description
I think generally I only follow releases on repos, so I think it should be easier to do that. Currently it requires 4 clicks but maybe we can reduce it to two.
Related

#3005

Before

After

Example URLs
https://github.com/refined-github/refined-github

I would love if #3005 was done first

I looked into this, it doesn't look like neither the REST nor the GQL API offers even a way to check whether the user is subscribed to releases. Subscriptions are handled everywhere as yes | no | ignored. "Repo releases" are not a "subscribable entity" either.

rest: https://docs.github.com/en/rest/activity/watching?apiVersion=2022-11-28#set-a-repository-subscription
gql: https://docs.github.com/en/graphql/reference/mutations#updatesubscription

So on the repo root:

check the DOM
submit the specific form and authenticity_token found

On the releases page:

Fetch the repo root first
same
same


Add aria-label to some buttons for accessibility.

Thank you. I've merged it in the develop branch and also added the attribute to some of the other interactive button icons.


small improvements

Hello.
I like your program, however it lacked few small features that I felt are necessary for my personal use.
Please review my changes and add to your project the ones you like.
Sorry for my bad code, java is not my primary language. There is more intuition than real code in my commit.
I will now copy-paste my commit message here for your convenience:

Added trayicon.png and alerts.json to repository (otherwise project crashed, as they are required)
Replaced alerts.txt with alerts.json (deserialized by Jackson, model described by Alert class). More complex file structure was required for advanced configuration (multiple conditions, multiple sounds)
Matching rules (from alerts.json) is now based on regular expressions.
Added "Reload" button to System Tray. This will reload alerts.json
Added 3 default sound files (from diablo 2).
Added MessageBoxW to System32DLL. Message is shown upon reload (either success or error).
FlavourListener will no longer delete your clipboard. That was conflicting with other poe tools that require clipboard (like poe trade macro).
Alerts are now limited to one per second (to prevent deafening sounds).

P.S. I think I killed your README.md as it looks wierd after commit. But locally preview is fine. Don't know how to fix it.

Thanks! Been busy with work this week and didn't notice until now. I'll merge and perhaps patch a few things before making a release.


Compile error caused by tealr_derive

Trying to build this project causes a compile error: https://gist.github.com/JohnTheCoolingFan/3e4ae02a70dcd94d6925dbdc73e4832e
Filed an issue at tealr repo as well: https://github.com/lenscas/tealr/issues/80

are you running the newest version of rust ? rustup update if not

Yes, I am using latest stable rust 1.70.
I will try to update packages on my arch install later, mayeb that changes something...

Hmm I see, do keep us updated, I will try investigate at some point

No, updating my packages didn't fix teh issue. I'll ask for help on rust community discord.

This is probably a compiler bug, see https://github.com/rust-lang/rust/issues/111888#issuecomment-1585690896
(you would be justified in closing this issue)


[DRAW-010] On ‘Help Guide’ page, there is a missing link where text says ‘click here’

Steps to reproduce:

Launch DRAW website.
Navigate to the ‘Help Guide’ section http://test.citsci.geog.mcgill.ca/help-guide.
Observe missing link where text says ‘click here’.

Actual result:
On ‘Help Guide’ page, there is a missing link where text says ‘click here’.
Expected result:
Text which says ‘click here’ is clickable.
Notes:
Issue occurs on:
Google Chrome Version 55.0.2883.87 m, Internet Explorer 11, Firefox version 37.0.1.
Windows 7 Professional x32, Intel Cote i5, ATI Radeon HD 4550.
Repro rate: 5/5.
Screenshots attached: DRAW-010.png
Tiffany Hirsch

App content problem, not mine


As an operator I would like my target/store credentials to live in a credential manager

Now that bosh and concourse have moved to storing credentials in credential managers (credhub / vault). It would be awesome if shield could move into this direction as well.
By looking up credentials from the credential manager at runtime it would be easier to rotate the credentials used for making backups.

Can you elaborate?


wc -l adds 2 leading spaces

OS: Arch Linux
Pretty simple bug. Doing something like yay -Qq | wc -l results in __1017 (with spaces instead of underscores) instead of the expected 1017.

Pretty easy to reproduce:
$ echo "foo"|wc -l
1
$ echo "foo"|./target/debug/coreutils wc -l
 1

Yep. Any potential fix?

Sure, I will leave it as good first bug as it should be trivial to address

Oh lol, I found the problem. I'm on my phone so I can't make a pr or anything.
@sylvestre if you could quickly edit out those 2 spaces that'd be awesome.

Please don't share screenshot on github. It is preventing code search, not mentioning accessibility...

@sylvestre if you could quickly edit out those 2 spaces that'd be awesome.

I will leave this as good first bug

Please don't share screenshot on github. It is preventing code search, not mentioning accessibility...

Gotcha. It's 1 am for me and I'm on my phone about to go to bed, it was just the easiest and most immediate for me. I'll keep that in mind for the future.

It can probably wait you leave your bed tomorrow ;)

@sylvestre good morning! Created pull request #1900 .


IL2CPP not running properly (build fail)

Describe the bug
Not sure if this is by design or if this is a bug, but implementing an abstract method with the attribute [Command] and building to Android will cause an error Exception: %path% il2cpp.exe did not run properly! and therefore the build will fail.
Another error below will give details about what failed:
C:\...\Temp\StagingArea\Il2Cpp\il2cppOutput/Bulk_Assembly-CSharp_0.cpp:39348: error: undefined reference to 'Weapon_CmdUse_m4AC6A94C5xxxx913EFBEB19DA43C9ABBE1CD791D'
clang++.exe: error: linker command failed with exit code 1 (use -v to see invocation)

where Weapon_CmdUse is the method CmdUse of Weapon.cs:
[Command]
public abstract void CmdUse();

I was able to fix that by replacing it to:
[Command]
public virtual void CmdUse() {
}

Expected behavior
I suppose (if it is not by design) that the build shoud succeed without error.
Desktop (please complete the following information):

OS:  Windows 10
Build target:  Android (Oculus)
Unity version:  2018.3.6f1
Mirror branch: master

Linker is a tool which removes unneeded code, it probably thinks that that abstract method is redundant.

Yes,  I don't think you can have an abstract command.   Your wrapper solution seems reasonable to me.

@paulpauch Could weaver generate something like this?

I am sure it could.  But I don't think it is worth it.  This is a one off issue that @hologm  had with an easy workaround.  So long it gives an error message,  instead of silently fail,  we are fine.


clarification on remove -- seems not to work?

README.md says

When you use the add method to add a subview, that view will be retained by the Layout even if you remove it from the view hierarchy. If you want the Layout to forget all about the view, call remove(view) (which also calls removeFromSuperview) or forget(element_id) (which only removes it from the Layout) on the Layout.

I'm dynamically adding some buttons to a scrollview and tagging them using RMQ (since they will all have the same element_id, being all the same style).
 def add_topic_filter filter_name
    context get(:topic_filters) do
      add UIButton, :topic_filter do
        make_filter_label filter_name #this is a styling method that does some graphics stuff
        rmq(target).tag(filter_name.to_sym)
      end
    end
  end

This works great.  When it comes time to remove the filter though, nothing happens:
  def remove_topic_filter filter_name
      filter = rmq.find(filter_name.to_sym).get
      mp filter  #prints the correct UIButton instance
      remove(filter) #nothing happens
      mp self  #prints the layout, showing that the button is still in the layout
  end

I could hide it, or use rmq to remove it but that doesn't seem right. What's the correct way to remove a view from the layout and the view?
I also tried this within the context of the scroll view -- that didn't work either.

Update -- I looked in the code and found remove_child which appears to work with the caveat that it does not remove it from the layout. Seems the only way to remove from the layout would be to forget(:element_id) but that operates on all the filter buttons which is not what I want.
That said, having these guys remain in the layout does not seem to be affecting my ability to layout new ones; not sure if there would be other problems down the road.  Seems like we need a forget(view) method to handle this, along with remove(view).    I'd be happy to contribute if this makes sense.

Just added a PR #135 to address this.


[BUG] com.microsoft.azure.CloudException: The Resource 'Microsoft.Compute/disks/disk-mydisk' under resource group 'myresourcegroup' was not found.

VM is deleted, its disk and NIC are deleted.
After deletion of the VM, creation is started with the same name.
Creation of disk throws following exception, but when checked on Azure Portal, disk is actually created fine.
SDK version:  1.14
Exception or Stack Trace
Jan 03 09:22:54  Caused by: com.microsoft.azure.CloudException: The Resource 'Microsoft.Compute/disks/disk-mydisk' under resource group 'myresourcegroup' was not found.: The Resource 'Microsoft.Compute/disks/disk-mydisk' under resource group 'myresourcegroup' was not found.
Jan 03 09:22:54 at com.microsoft.azure.AzureClient.createExceptionFromResponse(AzureClient.java:740)
Jan 03 09:22:54 at com.microsoft.azure.AzureClient.access$100(AzureClient.java:33)
Jan 03 09:22:54 at com.microsoft.azure.AzureClient$24.call(AzureClient.java:716)
Jan 03 09:22:54 at com.microsoft.azure.AzureClient$24.call(AzureClient.java:713)
Jan 03 09:22:54 at rx.internal.operators.OnSubscribeMap$MapSubscriber.onNext(OnSubscribeMap.java:69)
Jan 03 09:22:54 at retrofit2.adapter.rxjava.CallArbiter.deliverResponse(CallArbiter.java:120)
Jan 03 09:22:54 at retrofit2.adapter.rxjava.CallArbiter.emitResponse(CallArbiter.java:102)
Jan 03 09:22:54 at retrofit2.adapter.rxjava.CallExecuteOnSubscribe.call(CallExecuteOnSubscribe.java:46)
Jan 03 09:22:54 at retrofit2.adapter.rxjava.CallExecuteOnSubscribe.call(CallExecuteOnSubscribe.java:24)
Jan 03 09:22:54 at rx.Observable.unsafeSubscribe(Observable.java:10327)
Jan 03 09:22:54 at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:48)
Jan 03 09:22:54 at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:33)
Jan 03 09:22:54 at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:48)
To Reproduce
Steps to reproduce the behavior:
Code Snippet
osDisk = azureInstance.disks().define(diskName).withRegion(region)
.withExistingResourceGroup(rg).withWindowsFromVhd(image)
.withSizeInGB(size).withSku(diskSkuType)
.create();
Expected behavior
Disk creation should succeed.  This issue is mostly seen during deletion and immediate re-creation sequence.
The resource not found in the group error is thrown at various stages, sometimes disk creation goes thru but VM creation fails with the similar error description. When checked from portal, VM is actually created fine .
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: Windows
IDE : [e.g. IntelliJ]
Java SDK: 1.14

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[-] Bug Description Added
[ -] Repro Steps Added
[- ] Setup information Added

@apole  I tested on version 1.29.0 and didn't reproduce this issue. The version of the SDK that you used were quite out of date. Could you help try the same code snippet on the latest SDK? Thanks.

Thank for the response. It takes few re-tries, to reproduce it.  We started seeing this in our production recently. We do want to upgrade to the latest SDK, but may not happen that quickly. So trying out how to mitigate.
In my trials, deallocating VM before delete and adding some wait (about 20 secs) seems to reduce the chances of this error happening.

The issue should be service side issue since it happened intermittently. However, the 1.14.0 version is quite out of date(around 2015-06-15).
Would suggest to upgrade the API. However, would let the service team to take a look.

I opened ticket with MS first, but they asked me to open a ticket here.  Their reasoning is,  on Azure Portal VM is created properly, so it must be SDK which is returning the error.
I am going to give this reference in that ticket now. Please report it to Service team on your side as well.

Closing this issue due to no response. I'll reopen this if we have more details from @apole on the issue.


Add githubWorkflowIncludeClean

Added a scripted test, although now I'm not sure why one of the existing ones (suppressed-scala-version) copies the expected files instead of outright running githubWorkflowCheck on the state like others do....

gah IT WORKED ON MY MACHINE

greeeeen!


0.95 met platform will be deprecated message

Home Assistant release with the issue:
0.95.0
0.95.1
Last working Home Assistant release (if known):
0.94.3
Operating environment (Hass.io/Docker/Windows/etc.):
Docker on Rasp3
Component/platform:
https://www.home-assistant.io/components/met/
Description of problem:
I have configured the platform met.
Since 0.95.x the log file shows the message that met platform via 'platform config' is deprecated:
2019-06-28 15:53:31 WARNING (MainThread) [homeassistant.components.met.weather] Loading Met.no via platform config is deprecated

The documentation does not give any hint about that.
What does that mean?
Could I change something?
File reference: homeassistant/components/met/weather.py line 36
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
weather:
  - platform: met

Additional information:
Related: https://github.com/home-assistant/home-assistant/pull/24608

This means that loading the Met configuration via configuration.yaml is deprecated; instead, the config entry (released in 0.95.0 via https://github.com/home-assistant/home-assistant/pull/24608) is the preferred method.

Dear @bachya
could you please explain what to do in more detail? How should i change my configuration?
The current documentation doesn't reflect any detail about that!

Go to Configuration >> Integrations in the UI and enable Met.no there.

@bachya - Also came across this "breaking" change on my latest upgrade.
The log message and usage does not correspond with the documentation = Misleading.
https://www.home-assistant.io/components/met
Can someone please change the documentation, thx


Reactor support?

Reactor is starting to gain some traction so would it be possible to get something like RxNullabilityPropagator to support it?

I am not familiar with Reactor, and we don't have any immediate plans to add support for it ourselves, but we would be open to someone contributing it. Based on a quick glance at that link, chances are you could add some support for it by adding a few calls to this StreamModelBuilder. If you want to contribute a PR around this feature, I'd be happy to review it and help you get it merged into NullAway.

Can I contribute to this issue?

The answer is always yes! We are happy to look at a PR on this or answer any questions required to design this.

Now that #371 has landed, adding Reactor support should be a lot easier.  In case anyone is interested 🙂