Daily update

Automated changes by create-pull-request GitHub action

Build failed because of:
no headquarter addresses for open-earth-foundation at https://www.crunchbase.com/organization/open-earth-foundation
No cached entry, and Texas Instruments Incorporated has issues with logo: texas_instruments_incorporated.svg, Error in parsing SVG: Non-whitespace before first tag.
Line: 0
Column: 1
Char: F
No cached entry, and Sollers College has issues with twitter: https://twitter.com/SollersEdu, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Volkswagen Aktiengesellschaft has issues with twitter: https://twitter.com/volkswagen_de, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for SVA System Vertrieb Alexander GmbH: https://twitter.com/SvaGmbh
Empty twitter for Rancher Federal: https://twitter.com/RancherFederal
Empty twitter for Surge: https://twitter.com/surgetech
No cached entry, and Mellanox Technologies, Ltd has issues with twitter: https://twitter.com/mellanoxtech, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and JVC KENWOOD Corporation has issues with twitter: https://twitter.com/JVCKENWOOD_SR, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
No cached entry, and Continental Automotive Systems has issues with twitter: https://twitter.com/continental_ir, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Altran Technologies has issues with twitter: https://twitter.com/Altran, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Open Earth Foundation either has no crunchbase entry or it is invalid

Build failed because of:
No cached entry, and Texas Instruments Incorporated has issues with logo: texas_instruments_incorporated.svg, Error in parsing SVG: Non-whitespace before first tag.
Line: 0
Column: 1
Char: F
No cached entry, and Sollers College has issues with twitter: https://twitter.com/SollersEdu, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Volkswagen Aktiengesellschaft has issues with twitter: https://twitter.com/volkswagen_de, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for SVA System Vertrieb Alexander GmbH: https://twitter.com/SvaGmbh
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for Surge: https://twitter.com/surgetech
Empty twitter for Rancher Federal: https://twitter.com/RancherFederal
No cached entry, and Mellanox Technologies, Ltd has issues with twitter: https://twitter.com/mellanoxtech, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
No cached entry, and JVC KENWOOD Corporation has issues with twitter: https://twitter.com/JVCKENWOOD_SR, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
No cached entry, and Continental Automotive Systems has issues with twitter: https://twitter.com/continental_ir, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Altran Technologies has issues with twitter: https://twitter.com/Altran, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Item Texas Instruments Incorporated has no image_data


Chorus-RF-Laptimer integration in MultiGP LapSync Race Timing Desktop Software

It would be nice to have possibility to use Chorus devices in MultiGP LapSync Race Timing Desktop Software.
https://www.facebook.com/groups/FPVRaceTimer/

LapSync is not in development anymore.


Needs clarity on "consume"

It is not clear what is meant by "consume".  Today I bought two parallel jobs and ran a release pipeline.  Once finished, my count was back to zero.  However, the terminology used in the view page was "monthly" so I expected the parallel functionality to be available for an entire month.  Otherwise this will cost thousands of dollars per month to save a few minutes each release...

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

ID: 43d1ea36-caf8-c3f6-95f3-2d40ead86f5a
Version Independent ID: 46755386-7c55-bd3a-7984-30687a4e94f3
Content: Configure and pay for parallel jobs - Azure DevOps
Content Source: docs/pipelines/licensing/concurrent-jobs.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-licensing
GitHub Login: @steved0x
Microsoft Alias: jukullam

@ThinkElevenDave If you use release or YAML pipelines, then a run consumes a parallel job only when it's being actively deployed to a stage. While the release is waiting for an approval or a manual intervention, it doesn't consume a parallel job. Please see the section, How is a parallel job consumed in DevOps Services?.

Hi.  Please let me describe our situation then I'll return to your post.  I'm new to DevOps and this is not intended as a rant.
Currently we're using the free tier and have a release pipeline with around 60 steps (6 steps x 10 clients who have their own transformed version of the same app) - its a big flow chart IMHO.
Sometimes jobs start immediately.  Other times, like today, I've been waiting 14 minutes for an agent which eventually started and finished the first task.  Then I'm sat waiting for an agent again for the next step.  So in total this entire pipeline looks like it will take about 3 hours to deploy using current wait times, which is frustrating when our previous in-house ad-hoc PowerShell routine took about 15 minutes.
So I was investigating purchasing parallel jobs as a way to speed things up (based on recommendations from other forums).  However the DevOps portal where I configure this gives me no information about what a parallel job costs, nor what I actually get for it (e.g. a single parallel job / N number of jobs for a period / 1800 minutes of jobs valid for a week/month?):

Clicking on the link takes me around in circles and back the above page. There is no information (that I can find) that explains what I can do to shorten these wait/deployment times and what the repercussions might be (especially financial, because the invoice doesn't go to me).  Perhaps my search terminology is incorrect, but I feel like there's not enough information to-hand where it's most needed.
I'll gladly take any advice you can offer on streamlining things for us - premium or otherwise!

Please reopen this as I do not feel that the clarity has been provided.


Refactor+improvements

rewrite tests to use async/await
refactor index based on @danfinlay's work
move tests to ganache-core

Fixes #4

weird travis seems to have ignored my node_js flag?

Coverage remained the same at 100.0% when pulling d35e0e9244da68bbe369c9597bbdfbb40ebb9b97 on kumavis:refactor+improvements into 4efdbc20089d8d50ffe3f392b7cc30ebe836010c on ethjs:master.

weird, seems like it got my node_js flag the second time


chore: add clang formatting to gulpfile.js

LGTM, although I feel like the clang-format could use a bit of ironing out

I am with you on that, does a weird job.


Docker manifest for arm/v7 missing

Hi, thank you for creating such an awesome project.
I'm currently working on a small project that I'm trying to deploy on Ordoid XU-4 (Raspberry PI alternative) that has linux/arm/v7 architecture. I cant pull the docker image and run it locally. Is there a possibility to add the arch to docker?
Following is the error
no matching manifest for linux/arm/v7 in the manifest list entries
I've tried adding the --platform linux/arm64 but it runs into a problem when starting the server
/openfga exec format error
I'd be happy to create a PR if it could be adopted

I was able to get it work with platform: linux/amd64

Thanks for the update @dv29! In case you encounter any more issues, we would love a contribution. Right now we are using https://goreleaser.com/ to build multi target images, so if we can easily add support for linux/arm/v7, we can consider it


CI to remind me to create tests

Please have the CI remind me to add tests.  Thank you.

whats ur address?


pull request for e2e test at 11/07/2016 15:56:08

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


⚠ Bofrost.be has degraded performance

In 27a3dc9, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 4164 ms

Resolved: Bofrost.be performance has improved in b68848b.


preprocessing will not run

Preprocessing does not run, and I get the following from otm-server
django_1    | [16:15:42] WARNING  Bad Request: /api/v1/datasets/2/preprocess        log.py:224
django_1    |            WARNING  "POST /api/v1/datasets/2/preprocess          basehttp.py:161
django_1    |                     HTTP/1.1" 400 36

This is likely due to you having previously run preprocessing, and then it failing in the middle. So the preprocessing status of the dataset is still set to Running.
There are some steps we can take to fix this, I can look into them. I'll rename this issue to better reflect the root cause.

The error I showed seems to be as you're saying: failed task prevents further attempts.
I flushed my database and did a clean start to be able to run the task again, and this time I saw from the first run why it failed:
celery_1    | [2022-08-04 14:58:21,562: INFO/MainProcess] Task optimal_transport_morphometry.core.tasks.preprocess_images[c8da8b35-4b3a-47ad-a9fd-5328202e6128] received
celery_1    | [2022-08-04 14:58:21,565: ERROR/ForkPoolWorker-1] Task optimal_transport_morphometry.core.tasks.preprocess_images[c8da8b35-4b3a-47ad-a9fd-5328202e6128] raised unexpected: ModuleNotFoundError("No module named 'ants'")
celery_1    | Traceback (most recent call last):
celery_1    |   File "/usr/local/lib/python3.8/dist-packages/celery/app/trace.py", line 451, in trace_task
celery_1    |     R = retval = fun(*args, **kwargs)
celery_1    |   File "/usr/local/lib/python3.8/dist-packages/celery/app/trace.py", line 734, in __protected_call__
celery_1    |     return self.run(*args, **kwargs)
celery_1    |   File "/opt/django-project/optimal_transport_morphometry/core/tasks.py", line 18, in preprocess_images
celery_1    |     import ants
celery_1    | ModuleNotFoundError: No module named 'ants'

I see the issue. I'll make a PR to fix it.


Improve error message when execute function is not supported from command palette

Repro steps:

Action: azureFunctions.executeFunction
Error type: Error
Error Message: No matching resources found.
Version: 0.18.1
OS: win32
Product: Visual Studio Code
Product Version: 1.38.1
Language: en

Call Stack
LocalFunctionsTreeItem.<anonymous> extension.bundle.js:1:622502
o extension.bundle.js:1:617043

@pesabonsi we don't support execute function in a few scenarios:

For C#/Java projects
If you don't have a timer trigger

I've updated the title of the issue to improve the error message


Configure Devise to use model Users from Points

The model being used from the gem (OmbuLabs::Auth::Users) is different than the one that already existed on the Points app (Users)
Check if the gem can point to existing models in the main app, so it can be coupled with rake tasks/generators for new projects.

I like this idea. I guess the gem could check if a User class already exists and use it if it does...


Add search-query-title feature

Test URLs
Screenshot

Please first open an issue to discuss the features you desired. See our contributing guide: https://github.com/refined-github/refined-github/wiki/Contributing

Not a bad idea, but to work it would need to produce much cleaner results. This is the default view and it's kinda noisy.

I'd say:

drop the defaults (sort, is:issue, is:open)
drop the classifiers like is:

So:

is:issue is:open (default)

Issues • user/repo (unchanged)

is:issue is:closed

Closed • Issues • user/repo

is:issue is:open label:bug

bug • Issues • user/repo

is:pr is:closed label:bug something

`Closed bug something • Pull Requests • user/repo

It doesn't need to be perfect to work, it just needs to help distinguish tabs, right?
If you're willing to work on this feature, we need to set up a generic transform function and related test files. For example:
function extendTitle(defaultTitle /* the initial document.title */, query /* like 'is:closed is:issue' */) {
  // logic
  return 'Closed • Issues • user/repo';
}

And the testing like:
expect(extendTitle('Issues • user/repo', 'is:closed')).toBe('Closed • Issues • user/repo')
expect(extendTitle('Pull requests • user/repo', 'is:pr is:closed label:bug something')).toBe('Closed bug something • Pull Requests • user/repo')
// etc

If you don't have time to proceed with this, it's ok, but we'll have to close this PR


Add TwitterAction by psidex

Fixes #71

Pull Request Test Coverage Report for Build 122

11 of 23 (47.83%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 60.089%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pastepwn/actions/twitteraction.py
9
21
42.86%

Totals

Change from base Build 120:
-0.2%

Covered Lines:
813

Relevant Lines:
1353

💛  - Coveralls


check variable buffer is a Buffer instance in MonitorAgentInput

in_monitor_agent plugin collects metrics by checking internal variables by instance_exec.  It implicitly asserts that @buffer is a buffer instance when get metrics of buffer_queue_length and buffer_total_queued_size. If a plugin uses the variable name for other purpose, NoMethodError happens.
Example configuration with fluent-plugin-concat which uses @buffer variable as a Hash. On this configuration, doing curl 'http://localhost:24220/api/plugins.json' causes
[error]: #0 NoMethodError in monitoring plugins key="buffer_queue_length" plugin=Fluent::Plugin::ConcatFilter error_class=NoMethodError error="undefined method `queue' for {}:Hash"

<source>
  @type monitor_agent
</source>

<filter **>
   @type concat
    key message
    multiline_start_regexp /foo.bar/
    flush_interval
    stream_identity_key container_id
    timeout_label @OUT
</filter>

Good catch! Could you add tests for such case?

Thanks. Addeda test, PTAL.

Thx!


Fix documentation format for the doc parser.

As stated in  #1619 , the documentation I updated in previous commits do not show up using make doc. Following changes are made so that they become visible.

Remove extra gaps between doc and class definition in Markdown & ECR library.
Attach doc right above Time struct.

Thanks!


Edit as Yaml of a secret fails

Version: master-head(7b948b289)
Steps:

Create a secret (opaque) providing a key/value
Edit the secret by choosing "Edit as Yaml" option . Change the key/value in data section.
Hit "Save" button

Edit fails with the error below:
{ "type": "error", "links": {}, "code": "BadRequest", "message": "Secret in version \"v1\" cannot be handled as a Secret: v1.Secret.ObjectMeta: v1.ObjectMeta.TypeMeta: Kind: Data: decode base64: illegal base64 data at input byte 4, error found in rancher/dashboard#10 byte of ...|y\":\"djFly\"},\"kind\":\"|..., bigger context ...|{\"apiVersion\":\"v1\",\"data\":{\"k1ey\":\"djFly\"},\"kind\":\"Secret\",\"metadata\":{\"creationTimestamp\":|...", "status": 400 }

Note: Edit as a Form is functional

This is expected for secret fields that are base64 encoded, https://github.com/rancher/dashboard/issues/611#issuecomment-628911302

Validated with master-build(b85ea695f)
Steps:

Create a secret (opaque) providing a key/value
Edit the secret by choosing "Edit as Yaml" option . To change the existing value/add new key/value paits, we have to base64 encode the value and then use it in the yaml.
Hit "Save" button

Editing of secret succeeds without any errors. From the list view, we can edit as form and verify that the value has been edited and stored as expected.


Blacksmith outfit not working for /smelt

After the transition to slash commands blacksmith outfit is no longer giving 10% bonus xp when using the /smelt command. It used to work for both normal smelting and blast furnace.

Fixed in #4167


Re-run CI when the GitHub Actions config changes

@davisagli Just had a though. Would we want to re-run the release step if the only thing that has changes is the CI config?

@JeffersonBledsoe I think so. Maybe the change to the CI config is being done in order to fix a problem with the release step.

@JeffersonBledsoe I like @mauritsvanrees' similar PR in #21 which also makes it possible to trigger the action manually

Superseded by #21


[Bug] Minimap gets turned off by default at VSCode launch

Describe the bug
At every new launch of VSCode, the workspace setting to show the minimap keeps being reset to "off".
To Reproduce

Turn on minimap
Close VSCode
Relaunch VSCode. After Dendron initializes, the minimap disappears

Expected behavior
Respecting the user setting to keep the minimap on
Desktop (please complete the following information):

OS: Windows 10
Version: Latest

Additional context
Minimap is useful for visually navigating in a note just as the graph view is useful for visually navigating all the notes.

Hey John,
Thanks for the update. That settings comes as a default workspace settings
but you're right, it should respect the user presets. Will be updating it
for this weeks release!
On Fri, Nov 20, 2020 at 4:02 PM John Young<EMAIL_ADDRESS>wrote:

Describe the bug
At every new launch of VSCode, the workspace setting to show the minimap
keeps being reset to "off".
To Reproduce

Turn on minimap
Close VSCode
Relaunch VSCode. After Dendron initializes, the minimap disappears

Expected behavior
Respecting the user setting to keep the minimap on
Desktop (please complete the following information):

OS: Windows 10
Version: Latest

Additional context
Minimap is useful for visually navigating in a note just as the graph
view is useful for visually navigating all the notes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/issues/366, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPIPRL7ABI64SX2EUEWLLSQ37RXANCNFSM4T5MQZOA
.

this has been updated


[WIP] APPSEC-96: Upgrade zookeeper, jetty, jersey, httpclient, mvn-artifact.

There are more to be upgraded:

commons-beanutils_commons-beanutils to 1.9.4
com.google.guava_guava to be 30.0
org.codehaus.jackson:jackson-mapper-asl  seems to have no fixed version. https://snyk.io/vuln/SNYK-JAVA-ORGCODEHAUSJACKSON-534878

If you know where they are defined, please let me know.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I have raised similar PRs here for 5.2.x:
https://github.com/confluentinc/common/pull/338
https://github.com/confluentinc/support-metrics-common/pull/57
https://github.com/confluentinc/support-metrics-client/pull/66
https://github.com/confluentinc/kafka/pull/565

Sup of https://github.com/confluentinc/kafka/pull/565


Slow IsEVMAddress Function

Context: Over time, with a bloating state, the function’s performance degrades.
Location: keeper_valset.go
Issue: The function’s iteration over EVMAddresses in the QGB evm addr to val addr prefix results in reduced performance.
Remediation: Integrate a KeySet (refer to cosmossdk.io/collections.KeySet) for faster existence verification.

no longer needed as we're switching to BlobstreamX


Start filling GLUE dataset card

The dataset card was pretty much empty.
I added the descriptions (mainly from TFDS since the script is the same), and I also added the tasks tags as well as examples for a subset of the tasks.
cc @sgugger

I replaced all the "we" and applied your suggestion

Merging this for now, we can continue improving this card in other PRs :)


Hello, could you tell me the proportion of covid-19 training set, verification set and test set? Is it random?

Hello, could you tell me the proportion of covid-19 training set, verification set and test set? Is it random?

please follow the official proportion of the QaTa-COV19 dataset.

However, the validation set is not included in the official data set of QaTa-COV19. How did you deal with it?

Please email me by your official email.

Hello, I have sent my question to your email, please pay attention to check, I hope to get your reply in time, thank you


variable naming/labeling

for best practice: while translating, all significant numbers should be assigned to variables with descriptive names and/or the lines in which they are used should be commented. Many already have this, but, for example,
if n ==18:
    data(1,13)=20;

should be something like:
pruneThreshold = 18 # description of what 'pruneThreshold' means (fake var name)
if n == pruneThreshold: # if the year is pruneThreshold, coffee plants explode
    data[1][13] =20;

closeable?


Add more functionality to dagba static scan about CVE description

Current pull request has following enchancements:

New collection at vuln_database that contains extended info about CVE (cvss_access_complexity,cvss_access_vector,cvss_authentication,cvss_availability_impact,cvss_base,cvss_confidentiality_impact,cvss_exploit,cvss_impact,cvss_integrity_impact,cvss_vector,cweid,mod_date,pub_date,summary)

The histtory of an scan will contains advance description of each CVE

You can ask to the api about CVE info ex(http://<IP_ADDRESS>:5000/v1/vuln/cveinfo/CVE-2009-2890)
[{"cveid": "CVE-2009-2890", "cvss_access_complexity": "Medium", "cvss_access_vector": "Network", "cvss_authentication": "None required", "cvss_availability_impact": "None", "cvss_base": 4.3, "cvss_confidentiality_impact": "None", "cvss_exploit": 8.6, "cvss_impact": 2.9, "cvss_integrity_impact": "Partial", "cvss_vector": ["AV:N", "AC:M", "Au:N", "C:N", "I:P", "A:N"], "cweid": "CWE-79", "mod_date": "20-08-2009", "pub_date": "20-08-2009", "summary": "Cross-site scripting (XSS) vulnerability in results.php in PHP Scripts Now Riddles allows remote attackers to inject arbitrary web script or HTML via the searchquery parameter."}]

I will review this PR for merging it ASAP because it looks like very interesting.
Be patient and thanks a lot :-)

Iḿ working on the tests to pass all of it! At soon as I finish it, i will
push the new version of code to review
Pedro Serrano Alonso
Móvil +34 636 46 51 45 <%2B34%C2%A0655%20555%20555> (230210)
<EMAIL_ADDRESS>Avenida de Monforte de Lemos 31. 28029, Madrid
Antes de imprimir este mensaje, por favor comprueba que es necesario
hacerlo.
Before you print this message please consider if it is really necessary.
2017-03-21 23:09 GMT+01:00 Elías Grande<EMAIL_ADDRESS>
I will review this PR for merging it ASAP because it looks like very
interesting.
Be patient and thanks a lot :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eliasgranderubio/dagda/pull/9#issuecomment-288235169,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOomVaR2lSDp9NFgRMGId5pktcODjNWNks5roEqxgaJpZM4MjyzP
.

Coverage decreased (-2.0%) to 58.921% when pulling 88f63d6e343f34762c9f2f0e3f1423f499798990 on i4s-pserrano:master into 2e1b55daa3e69aedbe21dcfd6dc89599f2b7da71 on eliasgranderubio:master.

new some unit test to get more coverage of the code.

Coverage increased (+0.6%) to 61.451% when pulling a302c5d23f4c5bab1343114800af2e574b5b9eac on i4s-pserrano:master into 2e1b55daa3e69aedbe21dcfd6dc89599f2b7da71 on eliasgranderubio:master.

Coverage increased (+0.6%) to 61.451% when pulling a302c5d23f4c5bab1343114800af2e574b5b9eac on i4s-pserrano:master into 2e1b55daa3e69aedbe21dcfd6dc89599f2b7da71 on eliasgranderubio:master.

I've just reviewed your PR and only I've modified minor changes.
Thanks a lot for your PR :-)

I feel great to improve with someting your tool!
Great alternative open tool to analyze containers

Your PR has given me another idea for improving the information in the other collections :-)

Great! +1
One of the ideas that I wish to apply on your tool on running environments
is delete the container used to evaluate the vulnerabilities. Keep this
containers makes the space grow quickly. Something like when you run an
interactive container with --rm parameter. ex docker run -ti --rm
container:latest command
Pedro Serrano Alonso
Móvil +34 636 46 51 45 <%2B34%C2%A0655%20555%20555> (230210)
<EMAIL_ADDRESS>Avenida de Monforte de Lemos 31. 28029, Madrid
Antes de imprimir este mensaje, por favor comprueba que es necesario
hacerlo.
Before you print this message please consider if it is really necessary.
2017-03-29 9:54 GMT+02:00 Elías Grande<EMAIL_ADDRESS>
Your PR has given me another idea for improving the information in the
other collections :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eliasgranderubio/dagda/pull/9#issuecomment-290012358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOomVe068iYNhOgSCVuIJyUeZBaIjp84ks5rqg4jgaJpZM4MjyzP
.


[YouTube Downloader] File Type options not displaying

Extension
https://www.raycast.com/vimtor/youtube-downloader
Description
Not sure what I'm doing wrong or if I'm missing an install, but the export "File Type" isn't displaying any options.

Steps To Reproduce

Download the extension
Do the default Homebrew install
try using it 🤔

Current Behaviour
No response
Expected Behaviour
No response

I think it has to do with the fact that the video has not yet been fully processed by YouTube. Can you try any other video?

@darmiel, ah yea another video worked. You called it - I just pulled the most recent video on my subscriptions, it didn't work with the first few. But it works with any others!
Cheers :)


i18n(ko-KR): update 2.mdx

Description (required)

update 2.mdx
#8907

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
tutorial/5-astro-api/2.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


qtbase: Fix patch fuzz detected with devtool

Signed-off-by: Khem Raj<EMAIL_ADDRESS>

This was still needed for 5.13.2 version of the patch. This commit disappeared already, but I've recreated it just before 5.14 upgrade and then reverted it back with 5.14 upgrade - in case someone will try to backport the fix to zeus branch (I'm tempted to just push everything from master except the 5.14 upgrade to zeus)

Yeah push it to Zeus minus 5.14


Add Typescript and more

Replace axios with fetch since it's only used on the client-side where fetch is almost guaranteed
Add Typescript support
Replaced enzyme with React Testing Library

Hello bro,
I am sorry for the late response i didn't have time to check github,
i liked the idea of using Typescript and the fetch API instead of axios,
However i checked the code and it seems like this PR introduces some errors, concerning the react testing library

i tried to debug it a little bit but with no succes, can you please see why
feel free to reopen a pull request if you found the solution,
Thank you again


Erase and Menu Key bind overlap. -4.0 RC 1-

Right-Click is both a button to erase colision and open the small menu, meaning when trying to use Right-Click to erase colision, the menu opens up.
Perhaps make it so that the small menu that opens up when Right-Clicking is disabled when in the colision edit mode.

Good spot, I've updated the context menu to only appear when using the Select tool now (as some of the other tools use right click too). This will be included in v4.0.0-rc.2 hopefully within the next few days


Scheduler for publish

Scheduler using Time-driven triggers.
https://developers.google.com/apps-script/guides/triggers/installable

@JosefJezek
Good idea. Any suggestions on how I might format this?
Maybe an extra worksheet called "SCHEDULES". Then have allow a user to set a schedule for publishing. Something like that perhaps...
Good idea though! Will get on it when I have a chance.

I think you'd want to do it inline with the content you're posting. IE, for the blog sheet, you'd just have two extra columns, one called "Published" (boolean) and another called "Publish Date" or something like that.


⚠ ETF has degraded performance

In 885fa14, ETF (https://markets.hankyung.com/equity-product/etf) experienced degraded performance:

HTTP code: 200
Response time: 5020 ms

Resolved: 마켓맵 performance has improved in d9213f4.


Make S3 Client Configurable

For my implementation, I need to configure the S3 client manually in both dev and production environments. Thought this might be convenient to add upstream.

Sounds good, thank you @mkeen  for contributing,
Can you please make it an optional argument ( after bucket name, prefix and onlyKeys flag ) (IE (Bucket, Prefix = "", onlyKeys = true, S3Client = s3, ContinuationToken = null, results = []) )?
If you also add the option to the readme.md document that'd be awesome, otherwise I'll do it.
Then I'll merge it and publish.

Sure, I'll make those changes here!

Did I drop the ball here? Was looking thru some old notifs and saw this was still open


About the Hardmard Product in Figure 2 (paper)

the symbol can't represent  Hardmard Product, it was Kronecker Product.

Thanks for your attention. It's out We will correct it!

Thanks for your attention.You are right. We will correct it!


The compilation database option fails to escape spaces somewhere

With libsodium (ex: https://github.com/TomMD/libsodium) you should be able to:
./autogen.sh
./configure
bear make
infer run --compilation-database compile_commands.json

And if you do that you get an error:
Capturing using compilation database...
Starting translating and linting 1 files
Usage Error: Failed to execute compilation command:
'/usr/local/lib/infer/infer/bin/../../facebook-clang-plugins/clang/install/bin/clang'
  @/tmp/cdb_clang_args.tmp.d3fd36.txt -fsyntax-only
  ++Contents of '/tmp/cdb_clang_args.tmp.d3fd36.txt':
     -c -DPACKAGE_NAME=\"libsodium\" -DPACKAGE_TARNAME=\"libsodium\" -DPACKAGE_VERSION=\"1.0.16\" -DPACKAGE_STRING=\"libsodium 1.0.16\" -DPACKAGE_BUGREPORT=\"https://github.com/jedisct1/libsodium/issues\" -DPACKAGE_URL=\"https://github.com/jedisct1/libsodium\" -DPACKAGE=\"libsodium\" -DVERSION=\"1.0.16\" -DHAVE_PTHREAD_PRIO_INHERIT=1 -DHAVE_PTHREAD=1 -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -D__EXTENSIONS__=1 -D_ALL_SOURCE=1 -D_GNU_SOURCE=1 -D_POSIX_PTHREAD_SEMANTICS=1 -D_TANDEM_SOURCE=1 -DHAVE_C_VARARRAYS=1 -DHAVE_CATCHABLE_SEGV=1 -DHAVE_CATCHABLE_ABRT=1 -DTLS=_Thread_local -DHAVE_DLFCN_H=1 -DLT_OBJDIR=\".libs/\" -DHAVE_MMINTRIN_H=1 -DHAVE_EMMINTRIN_H=1 -DHAVE_PMMINTRIN_H=1 -DHAVE_TMMINTRIN_H=1 -DHAVE_SMMINTRIN_H=1 -DHAVE_AVXINTRIN_H=1 -DHAVE_AVX2INTRIN_H=1 -DHAVE_AVX512FINTRIN_H=1 -DHAVE_WMMINTRIN_H=1 -DHAVE_RDRAND=1 -DHAVE_SYS_MMAN_H=1 -DNATIVE_LITTLE_ENDIAN=1 -DHAVE_INLINE_ASM=1 -DHAVE_AMD64_ASM=1 -DHAVE_AVX_ASM=1 -DHAVE_TI_MODE=1 -DHAVE_CPUID=1 -DASM_HIDE_SYMBOL=.hidden -DHAVE_WEAK_SYMBOLS=1 -DCPU_UNALIGNED_ACCESS=1 -DHAVE_ATOMIC_OPS=1 -DHAVE_ALLOCA_H=1 -DHAVE_ALLOCA=1 -DHAVE_MMAP=1 -DHAVE_MLOCK=1 -DHAVE_MADVISE=1 -DHAVE_MPROTECT=1 -DHAVE_NANOSLEEP=1 -DHAVE_POSIX_MEMALIGN=1 -DHAVE_GETPID=1 -DCONFIGURED=1 -I/opt/tommd/libsodium/src/libsodium -I/opt/tommd/libsodium/src/libsodium/include/sodium -I/opt/tommd/libsodium/src/libsodium/include/sodium -g -O2 -pthread -fvisibility=hidden -fPIC -fno-strict-aliasing -fno-strict-overflow -fstack-protector -fPIE -o crypto_pwhash/argon2/libsodium_la-argon2-core.o /opt/tommd/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-core.c

Error message:
clang: error: no such file or directory: '1.0.16"'

*** Infer needs a working compilation command to run.

If you sed and replace  1.0.16 with 1.0.16 then things work OK.

Thanks for the report. Which version of infer has this bug? If that's not too much to ask, could you paste the entry in compile_commands.json that infer has a problem with?

A minimized compile_commands.json that still exhibits the bug is:
[
    {
        "command": "cc -c -DPACKAGE_NAME=\\\"libsodium\\\" -DPACKAGE_TARNAME=\\\"libsodium\\\" -DPACKAGE_VERSION=\\\"1.0.16\\\" -DPACKAGE_STRING=\\\"libsodium 1.0.16\\\" -I/opt/tommd/libsodium/src/libsodium -I/opt/tommd/libsodium/src/libsodium/include/sodium -I/opt/tommd/libsodium/src/libsodium/include/sodium -g -O2 -pthread -fvisibility=hidden -fPIC -fno-strict-aliasing -fno-strict-overflow -fstack-protector -fPIC -DPIC -o crypto_pwhash/argon2/.libs/libsodium_la-argon2-core.o /opt/tommd/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-core.c",
        "directory": "/opt/tommd/libsodium/src/libsodium",
        "file": "/opt/tommd/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-core.c"
    }
]

Notice the PACKAGE_STRING cpp definition includes a space which triggers the bug.  If you remove that space then things function correctly.
For example, it should suffice to
git clone https://github.com/TomMD/libsodium.git
cd libsodium
echo <<EOF >compile_commands.json
[
    {
        "command": "cc -c -DPACKAGE_NAME=\\\"libsodium\\\" -DPACKAGE_TARNAME=\\\"libsodium\\\" -DPACKAGE_VERSION=\\\"1.0.16\\\" -DPACKAGE_STRING=\\\"libsodium 1.0.16\\\" -I/opt/tommd/libsodium/src/libsodium -I/opt/tommd/libsodium/src/libsodium/include/sodium -I/opt/tommd/libsodium/src/libsodium/include/sodium -g -O2 -pthread -fvisibility=hidden -fPIC -fno-strict-aliasing -fno-strict-overflow -fstack-protector -fPIC -DPIC -o crypto_pwhash/argon2/.libs/libsodium_la-argon2-core.o /opt/tommd/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-core.c",
        "directory": "/opt/tommd/libsodium/src/libsodium",
        "file": "/opt/tommd/libsodium/src/libsodium/crypto_pwhash/argon2/argon2-core.c"
    }
]
EOF
infer run --compilation-database compile_commands.json

N.B. you'll need to fix those paths to match your directory structure.  That's an issue I hope to address in another recent ticket.

After investigating a bit, the issue is that the arguments that end up being passed to clang (in a file) look like this:
 -c ... -DPACKAGE_STRING=\"libsodium 1.0.16\" ...

The issue is that this is too escaped. In the compilation db "spec", the example shows the same sort of argument being passed with \"single backslash escaping\" instead of \\\"triple backslash escaping\\\".
I think infer should be able to handle it with --compilation-database-escaped but unfortunately I tried and it doesn't.
That said, is it an issue with BEAR that it produces commands in this format?

Yes, this compilation database was produced with bear. Are you suggesting I
should open an issue over there?
On Wed, May 9, 2018, 3:35 AM Jules Villard<EMAIL_ADDRESS>wrote:

After investigating a bit, the issue is that the arguments that end up
being passed to clang (in a file) look like this:
-c ... -DPACKAGE_STRING="libsodium 1.0.16" ...
The issue is that this is too escaped. In the compilation db "spec"
https://clang.llvm.org/docs/JSONCompilationDatabase.html#format, the
example shows the same sort of argument being passed with "single
backslash escaping" instead of \"triple backslash escaping\".
I think infer should be able to handle it with
--compilation-database-escaped but unfortunately I tried and it doesn't.
That said, is it an issue with BEAR that it produces commands in this
format?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/infer/issues/912#issuecomment-387697127, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIaTigenHPtwVvLUaRDWlNrq8UFFgIzks5twsaBgaJpZM4T12z9
.

Nevermind about all of it, I guess my system's bear is quite old and this issue has been addressed. Sorry for the noise.


vibration when pressing "Like" (♥)

After updating the Bluesky app (Android), my smartphone vibrates every time I press "Like (♥)" on a post. This behavior has been observed not just by me.
This issue occurs only with the Bluesky app.
Bug?
Platform: Android
App version: 1.76.0

This has been fixed in #3482 and will be available in the next release in the next week or so.

should be early this coming week!

This is fixed now

Fixed in 1.77


Netbeans showing error in bootstrap.css file.

I am using bootstrap 4 in Netbeans 8.2 ;Netbeans shows error in bootstrap.css

The error is : Unexpected token SEMI found
What is it ? How to resolve it?

https://netbeans.org/bugzilla/show_bug.cgi?id=262769

Closing per latest comment.


Explore page: Add topic param to pagination (#9077)

Fix #9077.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@eb0359c). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master   #9078   +/-   ##
========================================
  Coverage          ?   41.1%
========================================
  Files             ?     548
  Lines             ?   71439
  Branches          ?       0
========================================
  Hits              ?   29366
  Misses            ?   38362
  Partials          ?    3711

Impacted Files
Coverage Δ

routers/home.go
46.68% <100%> (ø)

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 eb0359c...dc8774e. Read the comment docs.


Unable to set healthCheckNodePort for Ingress Gateway Service in Helm charts

Bug Description
When using source IP preservation via the externalTrafficPolicy: Local setting for a Service of type LoadBalancer, the service controller will allocate a port from the cluster's configured NodePort range.
In restricted environments, it's useful and often necessary to define the health check node port so that network policies can be composed to only allow specific ports rather than broad ranges.
The current Helm charts in manifests/charts/gateway and manifests/charts/gateways/istio-ingress  currently lack the ability to explicitly set the health check node port.
Version
$ istioctl version
client version: 1.15.3
control plane version: 1.15.3
data plane version: 1.15.3 (4 proxies)
$ kubectl version --short
Client Version: v1.22.15
Server Version: v1.22.15-eks-fb459a0
$ helm version --short
v3.10.1+g9f88ccb

Additional Information
No response

Closing - I'm able to solve this with https://istio.io/latest/docs/setup/additional-setup/customize-installation-helm/


Add 'wizard' mode

Running get-neighbour-transactions and the other commands repeatedly is quite frustrating.
BitBook should offer an interactive mode that guides the user to completion.
Idea: add wizard command which asks the "right" questions (and changes the prompt/context).

Ask for owned address if get-neighbour-transactions returns empty list
Automatically show updated transaction details as soon as new information has been provided
Pick highest value transaction from get-neighbour-transactions, go into transaction specific mode

exit wizard

done with transaction, pick new transaction (with highest/second-highest) value

pick a different transaction (from get-neighbour-transactions)

add description for transaction

mark inputs/outputs as owned/foreign

add descriptions for inputs/outputs

The transaction specific mode should offer shortcuts that enable the user to provide answers faster than using the commands like mark-address-as-owned and supplying the arguments via tab completion or copy&paste.

d <description> to add description for transaction
o all to mark all addresses with unknown ownership as owned
f all ... as foreign

Alternatively, go through all inputs/outputs and ask specific questions (with sensible defaults). This would be problematic for transactions with many inputs/outputs, though.

See https://github.com/C-Otto/BitBook/projects/1


New Wallet: Support Daemon APIs

Wallet supports 100% of existing daemon APIs.

duplicate of #1362


Revert "[Direct API] Update Direct API Access Config"

Reverts Shopify/cli#3258
Preparing this revert PR just in case.

Direct API has been in prod for over a week now. Closing this.


Mysql Extension Won't Install

On Mac OS X 10.9.5 when trying to install the mysql extension i get the error below.  I'm using homebrew for mysql.  On the PHPBrew troubleshooting page it says to "adjust your variant to match the options." when there are missing header files but I'm not sure what that means in this context.  The results of "locate mysqlnd.h" are below also.  Any ideas on how to resolve this?
-------- Install Output-------
===> Using xml extension meta
===> Installing mysql extension...
===> Phpizing...
===> Configuring...
===> Building...
In file included from /Users/matthewdavis/.phpbrew/build/php-5.6.1/ext/mysql/php_mysql.c:69:
./php_mysql_structs.h:39:10: fatal error: 'ext/mysqlnd/mysqlnd.h' file not found
include "ext/mysqlnd/mysqlnd.h"
     ^

1 error generated.
make: *** [php_mysql.lo] Error 1
--------Locate mysqlnd.h output--------
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/include/php/ext/mysqli/mysqli_mysqlnd.h
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/include/php/ext/mysqlnd/mysqlnd.h
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/include/php/ext/mysqlnd/php_mysqlnd.h
/usr/include/php/ext/mysqli/mysqli_mysqlnd.h
/usr/include/php/ext/mysqlnd/mysqlnd.h
/usr/include/php/ext/mysqlnd/php_mysqlnd.h
/usr/local/Cellar/php53/5.3.28/include/php/ext/mysqlnd/mysqlnd.h
/usr/local/Cellar/php53/5.3.28/include/php/ext/mysqlnd/php_mysqlnd.h
/usr/local/Cellar/php54/5.4.31/include/php/ext/mysqli/mysqli_mysqlnd.h
/usr/local/Cellar/php54/5.4.31/include/php/ext/mysqlnd/mysqlnd.h
/usr/local/Cellar/php54/5.4.31/include/php/ext/mysqlnd/php_mysqlnd.h
/usr/local/Cellar/php55/5.5.15/include/php/ext/mysqli/mysqli_mysqlnd.h
/usr/local/Cellar/php55/5.5.15/include/php/ext/mysqlnd/mysqlnd.h
/usr/local/Cellar/php55/5.5.15/include/php/ext/mysqlnd/php_mysqlnd.h

This helped me:
export PKG_CONFIG_PATH=/usr/local/opt/openssl/lib/pkgconfig

phpbrew ext install mysqlnd


Update app-service-web-php-get-started.md

Added missing quote

Hi @michha, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@michha : Thanks for your contribution to the Azure documentation! The author, @cephalin, has been notified to review your proposed change.

@michha Thanks so much for catching the error. However, there was already a merge in motion today in the private repo that includes the identical change. Please don't stop contributing in the future!
@PRmerger please close without merge.

@cephalin @michha Thanks for the contribution and the review. Closing this pull request because a pull request in the private repo adds the changes.


Server not reachable with -sudo param

Hi.
I've tried to run mongrel2 under Debian (3.16.7) with latest libzmq (github master build) and got the same problem already reported before.
Without -sudo everything works fine but when i try to run mongrel2 with -sudo i can't connect to the server anymore.
I've tried the /proc fix but it doesn't work for me.
I would be grateful for any help.
Cheers ...

Can you provide more details?


Update OWNERS

From https://git.k8s.io/community/contributors/guide/owners.md,
the approvers should be not in the reviewers section.
So this updates our OWNERS.

wait for kubeedge-bot to be added to this project

/lgtm

/lgtm

the approvers should be not in the reviewers section.

Anywhere say approvers should be not in the reviewers section? @llhuii

@fisherxu https://github.com/kubernetes/community/blob/master/contributors/guide/owners.md#maintaining-owners-files

/approve

/meow

@llhuii Looks like all the reviewers/approvers should to be kubeedge member first.

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the kubeedge org. You can then trigger verification by writing /verify-owners in a comment.

JimmyYang20

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.
pkg/OWNERS

TymonXie

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.
lib/OWNERS
pkg/OWNERS

jaypume

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.

khalid-davis

User is not a member of the org. Satisfy at least one of these conditions to make the user trusted.

@kevin-wangzefeng can you help to add these reviewers?

@kevin-wangzefeng can you help to add these reviewers?

Everyone need to open an issue to list the contributions, then community will invite them.

Ref to: https://github.com/kubeedge/community/issues/30

I have submitted a membership request, pls review it at kubeedge/community#34.
@llhuii @fisherxu

khalid-davis membership request https://github.com/kubeedge/community/issues/35

TymonXie membership request kubeedge/community#36

JimmyYang20  membership request kubeedge/community#37

/verify-owners

/verify-owners


[GHSA-x9vc-6hfv-hg8c] Npgsql vulnerable to SQL Injection via Protocol Message Size Overflow

Updates

Affected products

Comments
Release 7.0.7 has this vulnerability patched, but it still fails with a vulnerability warning
https://github.com/npgsql/npgsql/releases/tag/v7.0.7

Hi there @paul-gerste-sonarsource and @NinoFloris! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

Thank you for your submission to the GitHub Advisory Database, this has been closed due to being a duplicate of #4379.


docs/conf.py still refers to version 2.26.0

The file docs/conf.py still refers to version 2.26.0 in the 2.27.0 release tarball.

Thanks - will fix.


[Bug] The touch panel on my PS4 controller does not work

Checklist

[X] I will only use English in my report.
[X] I have used the search function for open and closed issues to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

Questions

[ ] xCloud officially supports my country/region.
[X] The bug doesn't happen when I disable Better xCloud script.
[ ] The bug didn't happen in previous Better xCloud version (name which one).

Device
Laptop
Device
Lenovo Ideapad Slim i5
Operating System
Windows 10
Android app/Browser Version
Chrome 130.0.6723.70
Better xCloud Version
5.9.5
Game list
All Games
Reproduction Steps
I know this is not a bug but I want to try to know how to solve it, I can't use my touch panel because it takes screenshots, I would like to know a way to disable the screenshots or change the button because I really don't want to use it in games like Fortnite, but no I can because it takes photos and those photos sometimes cause me a delay, it's a little uncomfortable, and I can't find any configuration that tells me to deactivate it
Screenshot/video
No response

The touchpad maps to the screenshot button on Windows. Even if you disable the screenshot feature it won't have the same function as when using on PS.

I am mapping the ps4 panel to an xbox button with DS4WINDOWS, but the
problem is that the button that I assign to it works, but it takes
screenshots and I want to disable the function to take screenshots in
xcloud, since I use a custom operating system and It disables many things,
but the problem is with xcloud better, and I want to disable it.
El mar, 19 nov 2024 a la(s) 1:06 a.m., redphx @.***)
escribió:

Closed #560 https://github.com/redphx/better-xcloud/issues/560 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/redphx/better-xcloud/issues/560#event-15343372594,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBN7HWCZ5CJ5PDRWY7DQEZ32BK2M3AVCNFSM6AAAAABR6VZPHGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGM2DGMZXGI2TSNA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I am mapping the ps4 panel to an xbox button with DS4WINDOWS, but the problem is that the button that I assign to it works, but it takes screenshots and I want to disable the function to take screenshots in xcloud, since I use a custom operating system and It disables many things, but the problem is with xcloud better, and I want to disable it.

Edit script in Tampermonnkey, replace currentGamepad.buttons[17] && currentGamepad.buttons[17].pressed with false


ERROR when try to run atomicals-electrumx with Bitcoin testnet.

ERROR:Prefetcher:ignoring unexpected exception
Traceback (most recent call last):
  File "/electrumx/electrumx/server/block_processor.py", line 128, in main_loop
    if not await self._prefetch_blocks():
  File "/electrumx/electrumx/server/block_processor.py", line 198, in _prefetch_blocks
    blocks[0] = self.coin.genesis_block(blocks[0])
  File "/electrumx/electrumx/lib/coins.py", line 166, in genesis_block
    raise CoinError(f'genesis block has hash {header_hex_hash} '
electrumx.lib.coins.CoinError: genesis block has hash<PHONE_NUMBER>33ea01ad0ee984209779baaec3ced90fa3f408719526f8d77f4943 expected<PHONE_NUMBER>19d6689c085ae165831e934ff763ae46a2a6c172b3f1b60a8ce26f

Anyone knows how to make it work?

Resolved by checking the code. Thanks.


Fix Python API's observe request

According to the docs the observe request needs an <Observe/> tag:
https://github.com/FlowM2M/AwaLWM2M/blob/master/doc/ipc.md#observe

LGTM


git-cliff --bump increments major when breaking changes, even when the major is currently zero

Is there an existing issue for this?

[X] I have searched the existing issues

Description of the bug
https://semver.org/#spec-item-4 indicates that breaking changes are fair game as long as the major is zero. I have the following in a post-commit hook:
#!/bin/bash
git cliff --bump > CHANGELOG.md
if ! git diff --quiet -- CHANGELOG.md; then
	git commit --amend --no-edit --quiet CHANGELOG.md
fi

That usually does the right thing. I now have a breaking change though, which makes git-cliff bump the major version. This is not what I want to happen.
Steps To Reproduce

Have a version 0.1.0
Make a breaking change adding a feature
Get 1.0.0

Expected behavior
I want the result to be 0.2.0, not 1.0.0
Screenshots / Logs
No response
Software information

Operating system: Chimera Linux
Rust version: Was probably built with 1.77.2
Project version: git-cliff 2.2.1

Additional context
No response

c.f. #447. I think there was another issue about 0ver / zero based versioning rules as well at some point.

This actually looks like exactly what I want. Let me try this out tomorrow, if it works like understand it, then I'll close this

See: https://git-cliff.org/docs/configuration/bump

Yep, works as expected


Fixed log_level arg, added validation error logging

Signed-off-by: Sam Curren<EMAIL_ADDRESS>

Codecov Report

Merging #73 into master will decrease coverage by 0.01%.
The diff coverage is 25%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   70.67%   70.66%   -0.02%
==========================================
  Files         176      176
  Lines        7400     7403       +3
==========================================
+ Hits         5230     5231       +1
- Misses       2170     2172       +2


Remove debugging queries from Python suite

Do you also need to exclude...
      - Metrics/Summaries/FrameworkCoverage.ql
      - /Diagnostics/Internal/.*/

They aren't included in the suite, only queries from security-extended are included which doesn't include FrameworkCoverage or internal diagnostics


ignore apple desktop service store file

Thanks, but this is in Global/OSX.gitignore

@arcresu got it, thanks for your reply!


Unmount errors during HASSOS (2.10) Shutdown

During HASSOS shutdown, several unmount errors are being recorded>
OS: HASSOS 2.10
Platform: QEMU_x64
HASSIO version: 0.87.1
During HASSOS shutdown, the following errors are being being shown (see screenshots).

I wonder if this is what is causing some corruption of HASSOS VM's if there are frequent shutdown/reboots.

Maybe unmounting fails because of corruption. Can you capture a more complete log using a serial console?

I will see what I can do.... since QNAP Virtualization Station is based around QEMU/KVM, it MIGHT be possible to do something at CLI level, but not sure. There definitely isn't a console "log" option as far as I am aware.
Based on the screen logs/error messages is seemed more like a code issue

Docker container unmoun: Invalid Argument.
Overlay2 unmount: Invalid Argument

I am still a semi-novice in some of this space though.

It is also possible to use a GUI, for example virt-manager. You can select the 'text console' you want to use. Just append console=ttyS0,115200n8 to the cmdline.txt.

No such luck.  Qnap doesn't leverage virt-manager nor expose libvirtd externally.

Should fix with 2.11?


build: add nodejs 8 for travis testing

Pull Request Test Coverage Report for Build 43

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 100.0%

Totals

Change from base Build 41:
0.0%

Covered Lines:
191

Relevant Lines:
191

💛  - Coveralls

Pull Request Test Coverage Report for Build 43

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 100.0%

Totals

Change from base Build 41:
0.0%

Covered Lines:
191

Relevant Lines:
191

💛  - Coveralls


Added continents.

I added continents.

Oh, sorry. I didn't notice those errors.
But I've fixed them now. Thanks.

Thanks.


refactor(headers): introduce safe abstractions

Context:

We were using StageDB, which is a wrapper over a mutable transaction, i.e. not really a DB, and has footguns (e.g. derefing to a tx makes it less clear to understand what is going on - my bad here when I introdcued this). This results into long-lived write transactions which we should think of as long-Mutexes, obviously bad.

Solution:

We refactor each Stage to instead take a DB: Database and instantiate safe wrappers over it which proceed to give us methods for each action we want. For Headers, there is for example HeadersReader and HeadersWriter. These abstractions are going to be consumed by anyone that wants to use the DB, whether it's a stage, or a DB client.

TODO:

Refactor these safe abstractions in storage/reth-providers

This is not entirely correct since db.update(f) opens up a transaction, calls f and then commits the transaction. Since headers, total difficulty and the stage progress are now written in 3 separate transactions, we can end up in an inconsistent state. For example, we might write 500 headers, crash and then restart the node. At this point, the checkpoint does not reflect that we wrote 500 headers, and the HeaderTD table is out of sync with the Headers table as well.
TL;DR Rule of thumb: When data is committed to the database, all state has to be 100% consistent at that point.

Closing. We decided that the StageDB abstraction is correct, in managing the lifecycle of things as-is.
We don't love:

The name StageDB: To convert to Transaction<_, DB>
StageDB has too many things: Write wrappers around the Transaction and implement them as providers.
StageDB only supports TxMut which means that we always take a write-lock when we use it.
The write transaction should be instantiated as lazily as possible to reduce the "area" that the write-lock covers
We should add support for read transactions inside of Transaction so that we don't open redundant write-locks when we don't need them.

Basically StageDB --> Transaction and its current semantics represent a Mutex where we'd like it to look more like a RWLock.


Update TLD map using updateTLDs.sh

Unfortunately, gofmt wants there to be a large amount of whitespace changes.

I'm not sure that we want to remove old TLDs given that will cause historical sites to be marked now invalid despite being valid originally. I think we probably just want to append to the existing list.

@zakird I think there's a trade-off to be weighed here. Leaving the old gTLDs in place will mean the linter doesn't flag new issuance for retired gTLDs and I think that could be equally problematic. Without capturing the date of the ICANN gTLD revocation alongside the removal in the map such that the check can be contextualized based on the certificate's issuance date I think there will need to be a decision about whether zlint prefers to avoid false-positives for historic certificates or whether zlint should prefer to avoid false-negatives for new certificates issued for revoked gTLDs. For my own purposes the latter is preferred! What do you think?

I agree with you that neither solution here is a good one. Because this has been used in Censys to effectively "grade" CAs, I'm still a hesitant to put in anything that would cause a false positive. It's easier for folks to ignore the numbers when they can point to false positives. That, for example, is why all of the Lints have effective periods. At the same time, you're absolutely correct that we want CAs following today's set.
I think the correct solution here is to similarly give TLDs effective periods as well, e.g., each TLD has a begin and end date associated with it. Thoughts?

I agree with you that neither solution here is a good one. Because this has been used in Censys to effectively "grade" CAs, I'm still a hesitant to put in anything that would cause a false positive.

Makes sense :+1:

I think the correct solution here is to similarly give TLDs effective periods as well, e.g., each TLD has a begin and end date associated with it. Thoughts?

That sounds reasonable to me but I think it might be outside of the scope of what @ezekiel signed up for with his PR before I hijacked it for this discussion :laughing: Does your team have the development time to implement this? If the answer is no that's totally fair and I'm prepared to scratch my own itch with a PR but it will take me some time.
With respect to moving this PR forward: Given that zlint was born of zmap/Censys. I don't think its reasonable to pressure you into making a decision that would result in false positives for that use-case in favour of my own. Amending the PR to leave the old gTLD's in place and only add the missing entries sounds like the right short-term decision :+1:  - On my side I'll know that I can't rely on zlint to catch mississuance for a revoked gTLD until the follow-up issue of storing effective gTLD periods is addressed.

This CSV has a list of removed TLDs (though it may need updating): https://github.com/pzb/TLDs/blob/master/removed/rmtlds.csv
This CSV has a list of gTLDs with their start dates (again, might need an update): https://github.com/awslabs/certlint/blob/master/data/newgtlds.csv

Pushed a new change which is additions-only (with a corresponding change to updateTLDs.sh which kinda helps with this operation in the future).
No more whitespace change noise, but there is a tiny bit of re-sorting noise.

I suspect the team won't have the opportunity to add this in in the near term (they're pretty underwater with a migration). If you have cycles, a PR would be tremendously appreciated. I'm going to create an issue to track this conversation after I merge this.

If you have cycles, a PR would be tremendously appreciated. I'm going to create an issue to track this conversation after I merge this.

Sounds great. I've made a note to try and slot a PR for this into my sprint next week.

If you have cycles, a PR would be tremendously appreciated. I'm going to create an issue to track this conversation after I merge this.

@zakird I think :point_up: might have been forgotten in the merge shuffle. I created
https://github.com/zmap/zlint/issues/235 with some rough notes and would appreciate a :mag: before I get down to writing code if you have a chance. Thanks!


problem with making a custom girlfriend character in character editor.

Describe your bug here.
When adding other animations for a custom girlfriend in character editor it would scroll pass other animations that don't have the same name as the ones in the xml, i typed the names correctly and copy and pasted them, it still does the same thing... i don't know why it does this, but any help is appreciated
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Neko, HashLink, or other build system
Did you edit anything in this build? If so, mention or summarize your changes.
yes, i did two custom opponents, custom scripts, aswell as towos fanmade fnf songs for placeholders.

If you're trying to use GF's original animations her DanceLEFT and DanceRIGHT animations have animation indices making only specific frames of the animation play... And remember to put a 0 at the end of the animation name to make sure it doesn't use other animations... Also keep in mind the Animation Text box is Case sensitive (Which means using upper case and lower case seriously matter) so the caps must be correct... Also a warning I have is to not leave the Animation box empty otherwise it will play every animation frame on the sprite sheet... For offsets I recommend going into the Ghost tab and making a ghost of GF's original sprites (that is if you haven't gone over them...) and then reloading the image so you can adjust them without too much shaking when animations change...


Billing Info

[I entered all my visa card details right , made sure it was correct , the only respond was "I may have entered wrong information ]

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

ID: 2d5c4c89-1778-7b70-db0e-1425315d027b
Version Independent ID: 7502f1f9-0ab9-5596-1254-1912b655b5e9
Content: Overview of Azure Cost Management + Billing
Content Source: articles/cost-management-billing/cost-management-billing-overview.md
Service: cost-management-billing
Sub-service: common
GitHub Login: @bandersmsft
Microsoft Alias: banders

@amrikinsky -  Thanks for brining this to our attention. Looks like your issue is resolved as per https://github.com/MicrosoftDocs/azure-docs/issues/66496.
At this moment, Since there is no doc update required, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Client Allocation Request for: Guazi Dynamic

Client Allocation Questions
Core Information

Name: Guazi Dynamic
Website / Social Media: guazi.io
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized: f3qaipvxrz2gxexc7mcvjsxifmscfiw7c7zfhrmq76j5ee4hcbvg3gbtpea7wgz72kkjcjmwzhm5uo2onxyocq
Notary Requested: ozhtdong

Hello, @ozhtdong, application information could be found at here. Thank you!


Make the agent retrieve the cloud instance-id from the kubelet

When the agent is started with hostNetwork: true, it is able to find the AWS/GCE instance-id
from the metadata agent (<IP_ADDRESS>).
When the agent is started with hostNetwork: false, that metadata agent should become unreachable for security reasons.
In that case, the current kubelet hostname provider tries to concatenate the node name with the cluster name.
This change makes the kubelet hostname provider first try to get the instance-id from the kubelet.

Here is what this change produces (with hostNetwork: false) before and after the change`:

That’s the same change as what we observe when we switch hostNetwork from false to true without that PR.

Today, when the datadog agent has access to the AWS metadata server (http://<IP_ADDRESS>), the EC2 hostname provider retrieves the instance-id from it.
Some datadog customers have cluster setup where PODs don’t have access to that metadata server because:
— they don’t want applicative PODs to rely on IaaS specificities and
— because some metadata server are serving secrets (The GCE one metadata.google.internal exposes the token of the service account of the VM. And this service account needs the privileges required by the kubelet to work (attaching and detaching volumes for example) and those privileges mustn’t be accessible to the PODs.
When the datadog agent POD doesn’t have access to the IaaS metadata server, it defaults to the POD name. (See the screenshot above)
What this PR does is that, on AWS, it makes the agent retrieve the EC2 instance-id from the API of the kubelet.
Here is a non-exhaustive list of drawback of the current behavior (in the case where the agent doesn’t have access to the metadata server) that this change should fix (on AWS only so far):
— the agent claims the hostname to be its POD name but it reports metrics for all the other PODs of its node.
— It’s harder to correlate metrics reported by the kubelet integration with things reported by the EC2 integration.
— when the agent is redeployed, the old time-series are discontinued and new ones are started (with the new agent POD name)
— if the host is part of the query, a graph stops and another one starts (as in the screen-shot above)
— if the host is not part of the query, for a period of time of few minutes, data are counted twice (the data coming from the new time-serie and the last point of the old time-serie)
As a example, if I graph the CPU request of a given POD (which is never touched), at the time the agent is redeployed, the graph shows a stop.
For all the above reasons, some customers are annoyed by the current behavior and would like to have the new one.
However, this is a breaking change because upgrading the version of the agent will change the hostname attached to the reported metrics.
Because this is a breaking change, there has been the idea of gating it through a feature-switch.
So, the question is : « Are there some customers that will be annoyed by forcibly having the new behavior » ?
The thing is that, at the deployment of this patch, the hostname will change. But even without this change, the hostname would also have changed (the POD name of the agent changes at every deployment).

Thanks for the summary @L3n41c!
Some additional high-level context here: overall, when the Agent changes the hostname that it reports, unless the new hostname was already a host alias of the previous hostname, a new host will be created in-app, and all the historical data will be missing in-app on the new host (and the user has to use the old hostname to see the historical data). Therefore, it can break dashboards, monitors, etc. So, when we make changes to the hostname resolution logic, we have to be very careful about how these changes impact the reported hostnames of existing deployments of the Agent.
Back to this specific PR, the deciding factor is probably what you mentioned here:

even without this change, the hostname would also have changed (the POD name of the agent changes at every deployment)

So, since the hostname reported in this case (i.e. pod name) already changes at every deployment, indeed it seems safe to introduce changes to the reported hostname.
My follow-up question to be sure we're safe is: are we absolutely sure that the changes in this PR only change the hostname when the currently-reported hostname is the POD name? Or are there  cases where this PR could potentially change a hostname that's currently stable across Agent restarts/redeployments?

Milestone moved to 6.17.0 since we would need an impact analysis to assess how many orgs and hosts will be affected by this change.

This is a breaking change that could cause big context churn.


Error: This module cannot be imported in the Vue part of your app

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.9.3
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-3etpwp?file=package.json
Describe the bug
If you have test files in your pages folder importing @nuxt/test-utils/runtime, the following error pops up when starting the project
ERROR  This module cannot be imported in the Vue part of your app. [importing @nuxt/test-utils/runtime from pages/<testname>.ts]

The error does not seem to have effect on the app startup itself (it's probably more a warning).
The error goes away when moving the test out of the pages folder.
Additional context
No response
Logs
No response

It seems that .nuxtignore can be used to avoid this.
// .nuxtignore
pages/*.nuxt.spec.ts

*.nuxt.spec.ts

Unfortunately this doesn't work on my end :(
Are there any other suggestions/fixes?

import addPlugin has same issue

The reproduction runs fine at the moment - and I think this would have been a (now-resolved) Nuxt bug: see https://github.com/nuxt/nuxt/blob/main/packages/schema/src/config/common.ts#L480.
If you still experience it, please do open a new issue on https://github.com/nuxt/nuxt/issues. 🙏


fedi bot doesn't accept invite and follow back

Cannot follow fedi bot from my account @arkd.wspanialy.eu, it doesnt accept it. Tried a couple of times, looked for something in the settings, no clue.

Your account was created around six days ago. We currently require accounts to be at least 14 days old to avoid problems with spam bots. More info here: https://fed.brid.gy/docs#troubleshooting
We could communicate this better. (See #758)
For now, please cancel the follow request and re-follow once the account is older than 14 days. It should work without issue then.


touch file to trigger travis (DO NOT MERGE)

trying to get travis to kick correctly...

@mike-kaufman,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Please upload to pip

The package is currently not available on pip:

What is your python version for this pip?

This library is for Python 3.10

Sorry, my bad.. PyLauncher was giving me the wrong version 😓


"Could not connect to server"

Trying to connect my zap desktop wallet to the zap mobile wallet. Followed the instructions and receiving the error, "Could not connect to server".
What information would be helpful to debug?

anything would be helpful :)
what instructions are you following? where are you running lnd? are you using lndconnect?

running lnd locally on Mac Mini. Yes running lndconnect and generated QR code but couldn't resize correctly so I coped and pasted after running lndconnect -j
My node is https://1ml.com/node/02b536fe1c2b50b6ab6b7074826a59dd3bbf93612594a33d4440c4f4f7095640a2

The address that populated in iOS app is correct with :10009 following. I've confirmed RPC is working because I can run lncli getinfo

have you configured lnd to use the external ip? that's always a bit complicated.
to get started you can run zapconnect -ji to use local ip. that should work if your phone is connected to the same wifi as the mac mini.

Phone is connected to same network and ran that command and gave it a try. Same error message which isn’t giving me much to go on.
I don’t know what you mean by configured to use external IP. I’m used the Node Launcher to set up LND and configured with nat = true so my node is public.

Looking at a Charles log, I'm seeing:
Error domain=NSPOSIXErrorDomainCode=61 "Connection refused" UserInfo={NSLocalizedDescription=Connection refused, NSLocalizedFailureReason=Error in connect() function}


[MNT] Replaced assert with raise ValueError messages in networks module

Reference Issues/PRs
Issue #1956

What does this implement/fix? Explain your changes.
Replaces assertions in networks module with raising ValueError messages

Does your contribution introduce a new dependency? If yes, which one?
No

Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
[x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF], [DEP] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring, deprecation or governance.

@all-contributors add @notaryanramani for code

thanks for this, looks good to me but will wait for @hadifawaz1999 or @aadya940 to take a look

@notaryanramani @TonyBagnall Apart from the comments, LGTM! 👍

@all-contributors add @notaryanramani for code


Updating word .Net

Category

[x] Content fix

What's in this Pull Request?
Updating word ".Net" to ".NET".

thx @gauravgoyal5, appreciated.


Refactor underlying code for "odo service" commands

Currently odo service supports two service backends:

Service Catalog
Operator Hub

Because of the numerous if..else blocks and short term thinking we did on this during v2 release, we've accumulated code that's:

not easy to read
difficult to debug
buggy (list of bugs)
difficult to add features without adding more if..else blocks

Besides that, we might plan to add support for other service backends future. Thus, making the code easy to add more pending features w.r.t Operator Hub integration (#2613), and easy to plug that extra backend (if we decide to) is of utmost importance.
Acceptance Criteria / Related Issues / Dependent Issues

[x] #3098
[ ] #3197
[ ] #3605
[x] #3803
[ ] #4172
[ ] #4233

/kind code-refactoring
/area service
/area service-operators

/priorit high

Scope of work for refactoring odo service create:

We will decide between using Operator Hub or Service Catalog backend by looking at the arguments passed.# below command will assume Operator Hub backend because of a forward slash
$ odo service create etcdoperator.v0.9.4/EtcdCluster

# below command will assume Service Catalog backend because there's no forward slash
$ odo service create dh-postgresql-apb --plan dev -p postgresql_user=luke -p postgresql_password=secret -p postgresql_database=my_data -p postgresql_version=9.6 -w

We need an interface that would have methods to create a service.
I will be modifying the struct ServiceCreateOptions to use this interface. https://github.com/openshift/odo/blob/5544b25ec083506393090faad5b4205126e4f675/pkg/odo/cli/service/create.go#L61-L62 Fields specific to Service Catalog and Operator Hub will be moved to their respective structs and these types will implement the interface we create for odo service create. In its simplest form, all the code from Comlpete, Validate and Run functions will be moved to methods of the same name implemented by the two structs.
As for the integration tests, the ones written currently should pass without any major modification.
As for the unit tests, more of those will be added in this refactor.

Scope of work for refactoring odo service create:

We will decide between using Operator Hub or Service Catalog backend by looking at the arguments passed.# below command will assume Operator Hub backend because of a forward slash
$ odo service create etcdoperator.v0.9.4/EtcdCluster

# below command will assume Service Catalog backend because there's no forward slash
$ odo service create dh-postgresql-apb --plan dev -p postgresql_user=luke -p postgresql_password=secret -p postgresql_database=my_data -p postgresql_version=9.6 -w

We need an interface that would have methods to create a service.
I will be modifying the struct ServiceCreateOptions to use this interface. https://github.com/openshift/odo/blob/5544b25ec083506393090faad5b4205126e4f675/pkg/odo/cli/service/create.go#L61-L62 Fields specific to Service Catalog and Operator Hub will be moved to their respective structs and these types will implement the interface we create for odo service create. In its simplest form, all the code from Comlpete, Validate and Run functions will be moved to methods of the same name implemented by the two structs.
As for the integration tests, the ones written currently should pass without any major modification.
As for the unit tests, more of those will be added in this refactor.

@dharmit I'm looking forward to be able to add a postgres db with odo for local development in CRC. It seems no service is available when using CRC, except ones added though an operator, but this makes things harder, I've been spending a few hours on adding one of the postgres operators available and making it work in odo but still didn't manage to.

@dharmit I'm looking forward to be able to add a postgres db with odo for local development in CRC. It seems no service is available when using CRC, except ones added though an operator, but this makes things harder, I've been spending a few hours on adding one of the postgres operators available and making it work in odo but still didn't manage to.

Acceptance criteria has not been met.
/reopen
/lifecycle frozen

This issue was opened as a part of "refactoring" the odo code right after v2 was released. But the time we had taken to "refactor" things didn't go well.
After that, it was decided that we would refactor the code when opening PRs for features/bugs, but won't dedicatedly work on refactoring things. If you look at #4465, it refactors a lot of code of odo service commands so that we use Go interfaces instead of if..else mess we were stuck with.
I'm removing the lifecycle frozen from this issue, but I'd urge you to close it, as we're not going to work dedicatedly on refactoring things.
/remove-lifecycle frozen
/triage unresolved


Fix #927: Prevent trying to schedule on master nodes first

Based off of what the description of the issue said and the generous aid from slack this is my current idea for the solution to this problem. I
Seeing as the base premise involved re-ordering the list of nodes I assumed the order was of little consequence aside from having masters last so the non-master nodes will be ordered differently than before.  Furthermore, I considered having two arrays that just got connected after iterating through the nodes but I was unsure if you guys would want to just keep it one array the whole time for cleanliness.
If I have done anything incorrectly or there are issues of cleanliness or efficiency just let me know.

Thanks a lot for working on this @CarterB-UT . I will look into it and let you know.

Thanks for the updates. I will get back soon.

Okay, we have tried squashing with some more changes based upon what you said. Let us know if there are any lingering issues @JayjeetAtGithub. Thanks again for the continued help.

Accidentally unsquashed the process. Will resquash.


Code inconsistencies (char literals, datatype casing, String, else begin)

      OP_EOL2:
        begin
          // \Z matches at the very and + before the final line-break (LF and CR LF)
          if regInput < fInputEnd then
          begin
            if (regInput = fInputEnd - 1) and (regInput^ = #10) then
              begin end
            else
            if (regInput = fInputEnd - 2) and (regInput^ = #13) and ((regInput + 1) ^ = #10) then
              begin end
            else
              Exit;
          end;
        end;

procedure TRegExpr.SetUseOsLineEndOnReplace(AValue: boolean);
begin
  if fUseOsLineEndOnReplace = AValue then
    Exit;
  fUseOsLineEndOnReplace := AValue;
  if fUseOsLineEndOnReplace then
    fReplaceLineEnd := sLineBreak
  else
    fReplaceLineEnd := #10;
end;

In these both parts LF and CR are expressed in decimal notation (#10 and #13). But everywhere else the hexadecimal notation is used (#$a and #$d). You should also use it here in hexadecimal.
Other inconsistencies:

Datatypes boolean, integer, byte, string are written in lowercase, but
datatypes Cardinal, Int64, UInt64, PWideChar, UnicodeString, WideString, WideChar, PAnsiChar, AnsiString, AnsiChar, PLongInt, LongInt, PtrInt, Pointer are written in correct case.
Consider also starting the lowercase ones with an upper case (Boolean, Integer, Byte).

In procedure TRegExpr.Error() you make use of datatype string, but it should be your own datatype RegExprString.

Using else in a case is by default already a block and doesn't need to have its own begin and end. Logic wise the else is part of the case, not its labels. Currently your code looks either like this (else is wrongly indented like the labels):
case * of
  *:
    ...;
  else
    ...;
end;

or it looks like this (else is correctly indented like case and end):
case * of
  *:
    ...;
else
  ...;
end;

Once you use the superfluous begin and end in the else part:
 case APtr[i] of
   '-':
     IsOn := False;
     ...
 else
   begin
     Result := False;
     Exit;
   end;
 end;

You can remove begin and end, because it is already treated like that and internally comes out as:
begin
  begin
    ...
  end;
end;

These are just recommendations to harmonize rivaling techniques used in your code - it doesn't change any functionality.

Part if these issues are caused by Embarcadero formatter app. I used it to format the entire unit.
#10 and #$a - leftovers from old times too IMO.
I don't think we need to fix this

I made fixes in my PR #332.

converted byte / integer / boolean to Byte / Integer / Boolean (but not string ID because it's usually lowercased as keyword)
indented case blocks else parts better and removed begin/end in else parts


Improve Psbt error handling: replace encoding::Error -> psbt::Error

This PR adds some changes that partially attend the issues raised in [1]. Specifically, the following changes were implemented:

psbt::Deserialize now returns psbt::Error instead of encoding::Error.

ParseFailed variant from encoding::Error is replaced by more specific variants in the psbt::Error enum. Namely:

InvalidPublicKey
InvalidXOnlyPublicKey
InvalidSignature(_)
InvalidControlBlock
InvalidLeafVersion
TaprootError(_)

This avoids the application of .into() in the deserialize functions.

Error messages are now shown as: PSBT error {} to provide more information on the details of the PSBT error.

The recursive dependency between psbt::Error and encode::Error is not fixed here but see [2] for more info.
[1] - https://github.com/rust-bitcoin/rust-bitcoin/issues/837
[2] - https://github.com/rust-bitcoin/rust-bitcoin/issues/837#issuecomment-1271726653

Regarding point 3, I believe this is bad practice. The existing write_err! code does the right thing.
The rest of the PR I'm unsure how to evaluate whether it's an improvement or not. I'd like to see the Psbt variant dropped from encoding::Error entirely. The new error variants for InvalidPublicKey etc I like though I'm not certain whether they belong in psbt::Error or encoding::Error.

Hi @apoelstra - thanks for the review.
I understand. I think, given what has been said in [1] and your comments here, I will close this PR in favour of a newer one that tries to accomplish the following:

Get rid of the Psbt variant from encoding::Error by letting Decodable/Encodable have associated error types.
Get rid of Serialize/Deserialize in PSBT and, instead, use Decodable/Encodable.
Have more specific variants instead of just ParseFailed. It's still unclear to me whether these should be in psbt::Error or encoding::Error, but I will do my best to place each variant in the enum that makes more sense.

Let me know if there's anything I might be missing here.
[1] - https://github.com/rust-bitcoin/rust-bitcoin/issues/837


Path field in retrieved data

Feature request
Summary
I liked the "url_path" field in the old url-alias plugin, it is useful in some cases.
Why is it needed?
You may need to have the path of the content you're retrieving (for links, reference...)

It’s still there. Though it’s now called url_alias and it’s a relation field. Meaning you’ll have to populate it to get it’s data.

Might have to write some documentation about this

Thaks for your quick response! Great news, thank you.
Also, i will migrate from old url_alias plugin to new strapi-webtools, is there an easy way to preserve data? I saw the migration script which renames the tables but i didn't saw the content<->alias relation table creation (in the url-alias plugin the relation is just a field in the content type table). Am I missing something?

There are also some migration scripts in place to accomodate the data movement. See https://github.com/pluginpal/strapi-webtools/tree/master/packages/core/server/admin-api/migrations
As long as you follow the migration guide you should be good

Great, thank you! 🍻

Please let me know how the migration process goes. I haven’t had much feedback on it apart from the other maintainers of this plugin.

Sure, it’s not gonna be soon but I will (it is planned tho).

Hey man, so I started the upgrade and I had some problems 😅.

When the migration script rename the 'url_paths' table it doesn't rename the column 'path' to 'url_path' consequentially you loose all the informations.
I went crazy because in the migration guide it isn't mentioned that you need to add the following code to every schema pluginOptions otherwise the migration goes well but when the plugin core is loaded it'll wipe all the data.

"webtools": {
      "enabled": true
}

On the link tables created by the script the column name is different from singularName (on table creation knex automatically fixes the name replacing '-' with '_' but not in the insert), so you lose data (without errors).

At the end of this i would suggest to use transactions so you can rollback if something goes wrong and you don't lose data.
Done all the mods the procedure went well 🍻.

I think it does say to enable webtools (or url alias) in the alpha migration
I think the knex migration is a transaction, so if anything goes wrong during the migration, it is rolled back. But strapi will delete tables not used, so you are right, if the tables are named something different strapi will delete it.
I would say I was unaware that the singular names included - maybe we should make that more clear in the migration guide?

🤦♂ I didn't look at the alpha migration (it says url alias but anyway I should have read to the end before starting).
The first attempt I made didn't go well but it also didn't rolled back, my strapi is inside a docker container tho so maybe the container crashed before it had time to roll back.
I think when you have types with spaces in the display name, strapi converts it to "-" in singular names and "" in collection names but regardless of this i think kenx does some manipulation to column names when it alters tables so maybe "-"/"" conversion is not the only thing it does.

Setting the pluginOptions of url_alias is enough since it will automatically be changed to webtools once you launch strapi. :)
If you change the singular name in the migration file to snake case is that enough? Or do you have other issues?

I didn't had any other issues, apart this

When the migration script rename the 'url_paths' table it doesn't rename the column 'path' to 'url_path' consequentially you loose all the informations.

Has the url_path field previously been named path?

In my table the column was named path 🤷♂

Ahh looks like it happened a while ago in this commit. I am not sure if there has ever been added any migration for this, but I think it is worth to add to the migration script.

@colpomatteo, I have opened a PR, do you think this covers the things you had trouble with?

Yes

I read your NOTE, just to be precise: singularName needs to be snakeCase not kebabCase (instead of CamelCase, i didn't tried camelCase names)

Thanks, yes it should be kebab-case, I have changed it now :)


Add xcstrings support

Updated version of #865. I couldn't update source branch of previous PR bacause it has references on my own apps.
Difference from #865

Rebased to version 7.5.0.
Fixed error on keys without values

I did a simple development for xcstrings support.
It is backward compatible for  for .xcstrings files converted from  .strings files. It means the same R.string codes will be generated.
However, it is not backward compatible for .xcstrings files converted from .stringsdict files, because of named arguments.
About named arguments
My implementation strips names of substitutions from generated arguments. I found using them problematic on some cases.
In my opinion, named arguments does not worth the implementation. Xcode generates substituons only if there are more than one pluralable parameter on new xcstrings files. The most of the string values will not be have any information to generate named arguments.
Problematic case 1
Some key information is lost on the xcstring convertion of stringsdict files.
Original stringsdict content.
	<key>x_users</key>
	<dict>
		<key>NSStringLocalizedFormatKey</key>
		<string>%#@users@</string>
		<key>users</key>
		<dict>
			<key>NSStringFormatSpecTypeKey</key>
			<string>NSStringPluralRuleType</string>
			<key>NSStringFormatValueTypeKey</key>
			<string>d</string>
			<key>one</key>
			<string>%d user</string>
			<key>other</key>
			<string>%d users</string>
		</dict>
	</dict>

Generated xcstrings content
{
  "x_users" : {
    "localizations" : {
      "en" : {
        "variations" : {
          "plural" : {
            "one" : {"stringUnit" : {"value" : "%d user"}},
            "other" : {"stringUnit" : {"value" : "%d users"}
}}}}}}}

R.swift creates .x_users(users: Int) for stringsdict file. However, xcstrings file does not have the argument name information any more.
If string value of the original content is  <string>Add %#@users@</string>, generated xcstrings  would keep the substitution information.
Problematic case 2
{
  "example": {
    "localizations": {
      "en": {
        "substitutions": {
          "device_iphone": {
            "argNum": 1,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg iPhone"}},
                "other": {"stringUnit": {"value": "%arg iPhones"}}
              }}},
          "device_mac": {
            "argNum": 1,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg Mac"}},
                "other": {"stringUnit": {"value": "%arg Macs"}}
              }}},
          "input_iphone": {
            "argNum": 2,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg touch"}},
                "other": {"stringUnit": {"value": "%arg touches"}}
              }}},
          "input_mac": {
            "argNum": 2,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg key"}},
                "other": {"stringUnit": {"value": "%arg keys"}}
              }}},
        "variations": {
          "device": {
            "iphone": {
              "stringUnit": {
                "value": "%#@device_iphone@ and %#@input_iphone@"
              }},
            "mac": {
              "stringUnit": {
                "value": "%#@device_mac@ and %#@input_mac@"
              }}}}}}}}

This strings value is "1 iPhone and 10 touches" on iPhones and "1 Mac and 10 keys" on Mac.
There are 2 substitutions for each device variation.
Which signature should we use?
.example(device_iphone: Int, input_iphone: Int)
.example(device_mac: Int, input_mac: Int)
.example(_ arg1: Int, _ arg2: Int)
.example(device Int, input: Int) //With some extra coding to detect shared parts of the names.
About algorithm
The alghorithm tries to convert localization of source language to single string with basic format parameters, then it uses FormatPart.formatParts(formatString:)` on this string to extract parameters.
This convertion works like that:

If localization has stringUnit, get its value, and replace the substitutions inside if.
Else if localization has variations for device or plural, convert each variation value to string with the same algorithm and select the string with most parameters. (To work correctly, all variations should have same parameters, but on some cases such as plural rule one or zero, parameter may not be used)

Substitutions replacement works like that:

Substitutions generated by Xcode contain plural variations with %arg parameter. Each substitutions has argNum and formatSpecifier variable to define real properties of this %arg parameter.
The value of a substitution is calculated by the convertion algorith above.
Then, %arg parameters on the calculated value is replaced according to argNum and formatSpecifier values.

Example 1
  "account": {
    "localizations": {
      "en": {
        "stringUnit": {"value": "Account"}
      }}}

Localization has stringUnit, get its value.
Localization does not have any substitutions. Nothing replaced.
Get paramters of "Account". []

Example 2
  "x_sets": {
    "localizations": {
      "en": {
        "variations": {
          "plural": {
            "one": {"stringUnit": {"value": "a set"}},
            "other": {"stringUnit": {"value": "%d sets"}}
          }}}}}

Localization does not have stringUnit.
Localization does not have device variations.
Localization has plural variations. Get value of each variations with parameter count.

one: Value: "a set", parameters []
other: Value: "%d sets", parameters [%d]

Select the variation with most parameters. "%d sets"
Get paramters of "%d sets". [%d]

Example 3
{
  "example": {
    "localizations": {
      "en": {
        "stringUnit": {
          "value": "%#@books@ and %#@pens@"
        },
        "substitutions": {
          "books": {
            "argNum": 1,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg book"}},
                "other": {"stringUnit": {"value": "%arg books"}}
              }}},
          "pens": {
            "argNum": 2,
            "formatSpecifier": "lld",
            "variations": {
              "plural": {
                "one": {"stringUnit": {"value": "%arg pen"}},
                "other": {"stringUnit": {"value": "%arg pens"}}
              }}}}}}}}

Localization has stringUnit.
Value is %#@books@ and %#@pens@
Localization has substitutions.
For "books" substitution
Get the value like Example 2: %arg book
Replace %arg according to argNum and formatSpecifier: %1$lld book
Replace substitution on the value: %1$lld book and %#@pens@
For "pens" substitution
Get the value like Example 2: %arg pen
Replace %arg according to argNum and formatSpecifier: %2$lld pen
Replace substitution on the value: %1$lld book and%2$lld pen
Get paramters of %1$lld book and %2$lld pen. [%lld, %lld]

Awesome !

@mac-cain13 👍

waiting for it ^^

Let's merge it

When are we gonna see that PR in action?

I am keen to use this on a new app that im starting to work on :)... Lets merge!

I think R.swift will die.
I already moved on and removed it on my project. What I did:

Colors and Images are from native XCassets
Strings: I use Swiftgen as an external tools, not as a dependency. It fixes the xcode inconsistency build error bug for SPM package.

@aiKrice
Came here because of the feeling that Swiftgen is not really updated any more and seems to die.
The inconsistency build errors all disappeared after running
 `defaults write com.apple.dt.XCBuild BuildDescriptionInMemoryCacheSize 0`

source:  https://mastodon.social/@NeoNacho/112560574952551066

Pod library version 7.6.0 has not merged the above pull request. Please work on it. Thank you!

@tomlokhorst It work!  Please merged this pull request. Thank you!

Pls merge

I pushed another commit. It fixes problems on keys extracted by Xcode automaticly.
These keys don't include value for original language. The system uses key as value on this case.
The commit does same thing on generation.


License?

Hi yeah! This repo is fab. 😸 @dauwhe pointed me here as I'm helping (a bit) with his acme-labs project.
I'm guessing from your other repos that you intend to MIT this one also, but I like to check as without a LICENSE file or other declaration...it's technically All Rights Reserved (at least most places on the 🌐).
Mind adding a license note some place or other? We'd appreciate it! :grin:
Thanks!
🎩

Added in https://github.com/jakearchibald/ebook-demo/commit/e28865919fb7c93b07753828a2ffb3dbf7e632c9
Sorry I lost track of this!