ngen-config Update model_param spec

https://github.com/NOAA-OWP/ngen-cal/blob/3a170b81632050381f14c374b1eaeac06da0f607/python/ngen_conf/src/ngen/config/bmi_formulation.py#L45C5-L45C17
ngen now supports a model param mapping to link model params to hydrofabric feature attributes.
The config structure can be seen here in this test code.
The config schema should be updated to reflect this change and properly validate/generate configs that may have this structure.

Thanks for pinging me, @hellkite500. From a quick glance it looks like model_params can now be one either None (unspecified), Mapping[str, str] (current specified form), or a Mapping[str, Mapping[str, str]]. So as a type hint that would look like:
    model_params: Optional[Mapping[str, Union[str, Mapping[str, str]]]]

Is it valid for the inner associated value types to be either str or Mapping[str, str] in the same outer mapping type? Or are the two variant exclusive? For example is this allowed?
{
    "model_params": {
        "str_kv": "str_v",
        "map_kv": {
            "map_k": "map_v"
        }
    }
}

Or are the following two examples only allowed?
{
    "model_params": {
        "str_kv": "str_v"
    }
}

{
    "model_params": {
        "map_kv": {
            "map_k": "map_v"
        }
    }
}

It can be mixed.  Though there is a current constraint that if it is a Mapping it should be in the form of
"model_params": {
    "key": {
        "source" : "source_value",
        "from" : "from_value"
    }
}

So we should probably define a custom linking type which has strict source and from keys, and the type would become
Optional[ Mapping[ str, Union[ str, LinkType] ] ]
I'm also not convinced that str is the only type...I'm pretty sure it can be numeric or list as well, just to complicate things a bit haha.

Thanks for the clarification!

I'm also not convinced that str is the only type...I'm pretty sure it can be numeric or list as well, just to complicate things a bit haha.

If you are talking about the keys, only str's are allowed as keys in json.

I'm refering to the values, e.g.
"model_params": {
    "param1": 4.2,
    "param2": [ 4, 2 ]
}

Ah, so sequences are also supported?

I would assume sequences of numbers or str's?

I'm not sure str is actually supported at all, I'll have to check the backend NGEN code haha

https://github.com/NOAA-OWP/ngen/blob/1dcd25596596f5a2c2aef172fc53e12fecd101f9/include/realizations/catchment/Bmi_Module_Formulation.hpp#L574
Found it, not sure why "str" was here in the first place, unless there was some type conversion happening somewhere else in ngen...still looking a bit

@hellkite500, thanks for digging into that and also pulling links. Im sure either we or someone else will appreciate the paper trail in the future.
Looking at the NextGen link you sent, it looks like the appropriate type hint should be:
Numeric = Union[int, float]
MODEL_PARAMS_T = Optional[Mapping[str, Union[Numeric, Sequence[Numeric], Mapping[str, LinkType]]]]


Drop interactive cmd support for cli

Just discussed with @leoppro  about removing one of cli's current interactive modes, https://github.com/pingcap/tiflow/blob/409c9525596e8ce2c8e1ec80244202427bfbee1c/pkg/cmd/cli/cli.go#L74.
Introduced two years ago in this PR: https://github.com/pingcap/tiflow/pull/351
This mode is rarely used by users and is not documented. Also, it is difficult to use it correctly, and our implementation is rather hacky. help hints are almost useless. So I'm going to just remove it.

LGTM


Add env source

Pull request
Description
Add env onramp to read environment variables
Related

Related Issues: closes #1136
Related docs PR

Checklist

[x] The RFC, if required, has been submitted and approved
[x] Any user-facing impact of the changes is reflected in docs.tremor.rs
[x] The code is tested
[x] Use of unsafe code is reasoned about in a comment
[x] Update CHANGELOG.md appropriately, recording any changes, bug fixes, or other observable changes in behavior
[x] The performance impact of the change is measured (see below)

Performance
No hot path code changes

Coverage decreased (-0.005%) to 84.51% when pulling ffcfbb05a16ad5ec9056815a2fbba98dec4c4a9c on env-onramp into 1dab0b6f743ddaaa544e9cf16d1b26e705e391c6 on main.


Support forcing color output

Add support for the FORCE_COLOR environment variable. It is possible to run in an environment that does not look like either a TTY or a named pipe and still want ANSI escape codes emitted. The NO_COLOR environment variable takes precedence.
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

Please update the markup for XCTestScaffold.runAllTests(hostedBy:_:) too?

My thoughts are supporting only NO_COLOR is only half of the implementation of controlling terminal colours. Forcing it off when you do have a TTY is just as valid as forcing it on when you don't.
Adding another CLI flag would mean plumbing it through SwiftPM to the swift-testing binary. An environment variable lets anyone opt in all the way down the chain.
There is some debate around whether to use FORCE_COLOR or CLICOLOR_FORCE. No preference there.

Neither CLICOLOR_FORCE  nor FORCE_COLOR is widely used. If you'd like to pursue a solution here, let's use a command-line argument. Thanks!


Update PyPy versions tested against

Closes #166.

Codecov Report
Patch and project coverage have no change.

Comparison is base (f2f1727) 63.67% compared to head (71d5df3) 63.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   63.67%   63.67%
=======================================
  Files           1        1
  Lines        1481     1481
  Branches      342      342
=======================================
  Hits          943      943
  Misses        447      447
  Partials       91       91

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Great, thank you!


Added a Page Scroll Progress Bar at the Top of Your Startup Webpage

Pull Request
Title
Added a Page Scroll Progress Bar at the Top of Your Startup Webpage
Description
Feature Description
Explain feature request
I have noticed a small issue that I believe could enhance the user experience. Currently, there is no Page Scroll Progress Bar at the Top of Your Startup Webpage , which makes navigation a bit cumbersome, especially when users need to return to the top of a lengthy page.
Related Issues
Fixes #434
Changes Made

Checklist

[x] I have tested the changes locally
[x] Documentation has been updated (if necessary)
[x] Changes are backward-compatible

Screenshots (if applicable)

Additional Notes

Footer
@VaibhavArora314 @Ultimateutkarsh11 @akbatra567 #447

Looks good just dont do all the changes in index.html, create a component for this

but i haven't done all changes is some extra lines were added okay and from my i have done this and i don't so much time for do this you tell this things when you assign me the task, but you want it i can create new issue for that then i can do, for now its done! @VaibhavArora314

Looks good just dont do all the changes in index.html, create a component for this

but i haven't done all changes is some extra lines were added okay and from my i have done this and i don't so much time for do this you tell this things when you assign me the task, but you want it i can create new issue for that then i can do, for now its done! @VaibhavArora314

When working with any react project, it is obvious that you have to do most of your work by creating components and I already stated this in your previous pull request #447  for the same issue. As for the code that already exists in index.html was added by another contributor who was having some issues while implementing the same by creating a component which is not the case for you.

You had said that on my previous PR not at the time of assign? @VaibhavArora314

For now, do the changes as stated above and I won't be stating all of the changes in the issues itself. Also, it is common while contributing to open source that admins and mentors state changes on the pull request according to the work done, you cannot say no that I won't be doing these changes as you did not state in the issue.

Yes I have right to say you haven't mention at the time of assign @VaibhavArora314

Yes I have right to say you haven't mention at the time of assign @VaibhavArora314
Hey @TheNikhilRaj we appreciate the efforts you are putting in while making the pull requests. But, try to understand that after your work has been done and when you have raised the PR we will suggest changes only to improve it. Without seeing the work done by you, how is it possible to mention the specific changes? Understand that these changes can't be mentioned on the issue and are specific to the PR raised. Also, we have merged  460+ PRs till now and this is not the first time we are suggesting changes in the PR.

@Ultimateutkarsh11 @VaibhavArora314 okay i have made seperarte file of that and only i have added 8 lines in "index.html" file for it neccesary for the issue try to understad it is neccessary to add in it!


Permission Denied On Login

Im getting a permission denied error when using the command "proot-distro login ubuntu"
realpath: /linkerconfig/ld.config.txt: Permission denied

Attack screenshot.

I will make video in this topic and then send you today. Wait for it...

watch this: https://youtu.be/CMi-MkwCloY
and if you encounter further errors, please comment below.
OR
Contact us on Telegram: https://t.me/cyberkernelofficial

watch this: https://youtu.be/CMi-MkwCloY and if you encounter further errors, please comment below.
OR
Contact us on Telegram: https://t.me/cyberkernelofficial

This worked for me thank you very much

Most welcome :)

Alr everything works the only problem is that the certificate dont seem to work or have not installed properly, I've tried installing them on Android and on Termux but I still get handshake errors.

In the second part of the video; we will explore this : )

Hey, finally we uploaded video. Check it out:
https://youtu.be/o-iypVqI_P8


Fix linker asserts for vector table not located at address 0.

Force __reset_vector and __eexceptions symbols to be absolute in linker script assert.
At least arm-none-eabi-ld will interpret the symbol location as relative to the section they were defined in. This caused the asserts to fail when .vector_table is not placed at memory address 0.

Sorry for the late reply here, do you have a project which shows this fix in effect?

Not publicly :)
You can reproduce this by changing the start location of FLASH in memory.x to a non-0 value (eg. 0x1000) and building any example using the GNU linker (eg. cargo rustc --target thumbv7em-none-eabihf --example main -- -C linker=arm-none-eabi-ld).
I'm running into this problem because I want to integrate a project with a bootloader, which requires the vector table to be at a non-standard location.

I think there must be something else going on here, since many projects use FLASH starting at non-zero (e.g. any STM32 has flash origin at 0x0800_0000) and they build fine using GNU linker. Specifically your example also succeeds without error for me. What version of arm-none-eabi-ld are you using?
I've also built projects which use a bootloader and so the application has the flash origin offset by a sector or two without issue, which also seem to build fine with GNU gcc or ld.

Closing PR as this seems to be a linker bug.
I ran into this with the arm-none-eabi-ld distributed with Ubuntu 18.04:
GNU ld (2.27-9ubuntu1+9) 2.27
I don't see this issue with either 2.27.90 (6-2017-q1-update) and 2.29.51 (7-2017-q4-major) from https://developer.arm.com/open-source/gnu-toolchain/gnu-rm/downloads.


Fix typo (to test article preview)

Docs Build status updates of commit 960ba95:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/index.md
:white_check_mark:Succeeded
View (qsharp-preview)

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.


Bad code formatting

It's currently reformating the files with a different config than the one my project uses, and messing with it. For example, it's removing new lines from every code but I don't think that's prettier, prettier respects new lines.
Any way to disable formatting?
And also pass the prettier config path for it to use.
Thanks.

Thank you for you contribution to this repository.
Closing this contribution as this repository is being archived.


Using ts-jest together with ts-transformer-keys

Is it possible to use ts-jest together with custom transformers such as ts-transformer-keys? I'd like to do assertions of my data types in my tests to ensure that they are up to date with some backend data types. That's why I'd like to use ts-transformer-keys together with Jest. Is there any reasonable way to do it?

A quick look tells me that ts-transformer-keys can be imported and used like any other javascript package. If this understanding is correct then there's no reason for this to not work. It will be treated and processed like any other javascript package.
Does that answer the question?

@golddranks is talking about custom transformers, which (currently) can only be used when accessing the compiler API directly. The docs of this one illustrate this:
https://github.com/kimamula/ts-transformer-keys#how-to-use-the-custom-transformer
In case of the transpileModule function ts-jest utilizes, custom transformers have to be provided to the seconds argument transpileOptions on the key transformers, which gets forwarded to program.emit:
https://github.com/Microsoft/TypeScript/blob/ceae613e4c0ba36829a2381687883ecdc6b169c3/src/services/transpile.ts#L107
Unfortunately, this transformer requires access to the program instance, which is inaccessible from the outside when using transpileModule. Thus, supporting this feature - in its current state - is not possible without quite a large change on how modules are processed by ts-jest

I'm closing this due to inactivity - it doesn't seem like something we'd support in the near future.


Fallback when atimuth entity has state "unknown"

Hi there,
I’m using the compass card for lightnings, add-on blitzortung.org and it works flawlessly. I have one little problem though, when there is no lightning activity (95% of the time) the add-on brings back “unknown” in the azimuth entity and compass card falls back to default azimuth 0°, north. Please find cause and effect in the picture attached. I've also attached my lovelace card if it's necessary to reproduce the case.
Since “unknown” is not an integer or float value, the band filter doesn’t show any effect. As a result many times I have zero lightning in north which is a bit annoying., but not really a problem => low prio enhancement.
I'd like if there was either a filter for "unknown" or a transformation into a negative number like -361 (one more degree than a full circle).
All the best, Andi

lovelace card.txt

All degrees outside of 0-359 get recalculated to be between 0 and 359.
So having a value of -361 would get recalculated to 359 and show there.
I'll look into having a separate filter for non numeric values as this could help out with issue #778


Feature/ Editors-in-Chief console overview tab

Adds a number of useful stats and links to the EiC console.
Deployed on dev: https://dev.openreview.net/group?id=.TMLR/Editors_In_Chief

The changes look good!
Could you remove the reviewer specific tasks?
.TMLR/Reviewers/-/~Xukun_Liu1/Responsability/Acknowledgement
There will be one per reviewer and that can be a very large list.
``

@melisabok So I should not load any of the invitations that start with TMLR/Reviewers? Or are you just saying to exclude from that list the invitations that include a specific user id?

I think you should just merge it without waiting for the tests to pass. It's just changes to the webfield template, there's no breaking changes.


1 gear instead of low+

is there a way to make it shift into first instead of low+ when truck has low gears? i have separate controls for all lows

Technically, Low+ is the same as first gear. If you put on a different gearbox that doesn't have lows, it'll show that you are in first gear

ah ok i forgot L+ is same ratio as first gear and probably makes more sense to have lockable diff at first gear :) wish game had Low/High ratio switch instead of this unreasitic mess. Thanks for this awesome mod it atleast gives manual gearbox :)


Repo is too big

try this: https://rtyley.github.io/bfg-repo-cleaner/

there are too many duplicate .svg, .ipynb and .html files. the plots of our old examples are saved as .svg, the new ones are embedded (i guess as svg) in the html docs. i will try plot(bla, fmt = :png) in the docs for possibly smaller html files. with git-filter remove delete .ipynb and .svg from history. still, gh-pages will stay large. if we dont want to lose the history of the docs [and we shouldn't since many people are pinned to 0.4 at the moment because of compatibility with PowerDynamics] consider moving the docs to separate repo.

Repo size is down to 9MB. That's okay. Old docs are here: https://github.com/PIK-ICoN/NetworkDynamicsDocumentationHistory
Still it could be pruned further with filter-repo, e.g. by removing svg and ipynbs from the history. [another estimated 6MB]

git history rewritten. repo is at ~2mb now


ConditionExpression.prefixKeysIn doesnt work properly if there are more then 10 params

Im using filter for query index with AND condition, one of them eq condition, other in condtion. When you pass to "in query" more then 10 params for example, subj method doesn`t work well, looks like result.replaceAllLiterally actually replace twice for params with idx > 10
Im using scanamo-1.0.0-M12-1
Thanks

Good catch! I will publish a fix asap, thanks a lot for reporting the issue.

i would really appreciate you if you`ll publish fix version to maven repos for it :) ofcause in case of real bug here.


Feat: adds note to Lifecycle creation policy owner docs

Problem Statement
What is the problem you're trying to solve?
Related Issue
Fixes #2416
Proposed Changes
With the aim of assisting individuals who may have the same question that led to issue 2416, I have added a note to the creation policy owner documentation. This note addresses the specific case of a secret that does not have the ownerReference field set.
Checklist

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

@LucasPimentel123 Can you please sign your commit and update the branch? :) Then we can merge this. :)

@LucasPimentel123 To sign your commit when committing, add -s to your command. :)

Here is what you need to do to make DCO pass. :) https://github.com/external-secrets/external-secrets/pull/2928/checks?check_run_id=19759592444

@LucasPimentel123 Hello. Still can't merge without signing your commit. :)


Problem getting the app running locally

Report
First of all - awesome stuff!
I followed the insctruction from the docs and from the README.
On top of that I've copied the envs from server and console into .env file at the root.
Tried runnign both through:
docker-compose -f docker-compose.infra.yaml up
npx nx serve server                                                                                npx nx serve console
and just docker compose up
in both cases after login i'm facing an error when trying to get to the console:
Expected behavior
No response
Steps to reproduce the problem

bring the app up locally
signup
try to access localhost:4200

Logs (if applicable)
redacted

Pezzo version
latest git clone
How do you use Pezzo?
Docker Compose

Thanks for reporting! Based on the details provided, it seems that you mixed between the instructions of setting up Pezzo in development mode VS spinning up the working stack for usage only. Which means we should probably improve the docs.
What is your goal? Do you want to modify the code? Or just run it locally and experiment?

@arielweinberger
Just want to get it up and running so I can later learn how to deploy it.
I first tried the full docker appoach:

tried docker compose up, but that got me an error about .env.local which was not present (at the root dir level) so i created one by copying .env contents from server and console
then it worked, the services started properly
I was able to signup but after trying to log in, I got this:

[{"message":"Forbidden","locations":[{"line":2,"column":3}],"extensions":{"code":"FORBIDDEN"}}],"data":null,"status":200,"headers":{}},"request":{"query":"query GetOrg($data: OrganizationWhereUniqueInput!, $includeInvitations: Boolean = false, $includeMembers: Boolean = true) {\n organization(data: $data) {\n id\n name\n members [u/include](https://www.reddit.com/u/include/)(if: $includeMembers) {\n id\n role\n user {\n id\n name\n email\n }\n }\n invitations [u/include](https://www.reddit.com/u/include/)(if: $includeInvitations) {\n id\n email\n role\n invitedBy {\n photoUrl\n }\n }\n }\n}","variables":{"data":{"id":"clorncfeu0000n70h9749mixa"},"includeMembers":false,"includeInvitations":false}}}

after that I brough down the docker containers, tried to set it up locally

docker compose up for infra
migrations
run server & console

and i got the same error this way
Thanks for the response!

also if you help me out with that I'd be happy to contribute back by improving the readme or docs if I can :)

Cool! Let's stick to the first path of just running docker-compose up. After doing that, can you please inspect the compose stack and make sure all services are up and running (green)?
Also, can you share the logs coming from the server container in the stack?

Update: I think this is caused by an issue with signing up/in via email and password (vs SSO using Google). Will continue to investigate.

So, let's start from the start:
git clone https://github.com/pezzolabs/pezzo.git
Cloning into 'pezzo'...
remote: Enumerating objects: 8493, done.
remote: Counting objects: 100% (3772/3772), done.
remote: Compressing objects: 100% (1798/1798), done.
remote: Total 8493 (delta 2433), reused 2537 (delta 1914), pack-reused 4721
Receiving objects: 100% (8493/8493), 27.11 MiB | 24.02 MiB/s, done.
Resolving deltas: 100% (4859/4859), done.
cd pezzo
docker compose up
Failed to load /Users/grski/dev/pezzo/.env.local: open /Users/grski/dev/pezzo/.env.local: no such file or directory
cat apps/server/.env >> .env.local
cat apps/console/.env >> .env.local
cat .env
cat .env.local
PINO_PRETTIFY="true"
DATABASE_URL=postgresql://postgres:postgres@localhost:5433/pezzo
SUPERTOKENS_CONNECTION_URI="http://localhost:3567"
CONSOLE_HOST="http://localhost:4200"
KAFKA_BROKERS="localhost:9092"
OPENSEARCH_URL="http://localhost:9200"
REDIS_URL="redis://localhost:6379"NX_BASE_API_URL="http://localhost:3000"
NX_SUPERTOKENS_API_DOMAIN="http://localhost:3000"
NX_SUPERTOKENS_WEBSITE_DOMAIN="http://localhost:4200"
NX_DEBUG_MODE="true"

then docker ps:
CONTAINER ID   IMAGE                                           COMMAND                   CREATED              STATUS                        PORTS                                                                                NAMES
29d72e33f6b1   ghcr.io/pezzolabs/pezzo/console:latest          "./scripts/entrypoin…"    About a minute ago   Up About a minute             80/tcp, <IP_ADDRESS>:4200->8080/tcp, :::4200->8080/tcp                                                          pezzo-pezzo-console-1
2bca6bd8e645   ghcr.io/pezzolabs/pezzo/server:latest           "node main.js"            About a minute ago   Up About a minute (healthy)   <IP_ADDRESS>:3000->3000/tcp, :::3000->3000/tcp                                                                  pezzo-pezzo-server-1
a808ada94058   opensearchproject/opensearch-dashboards:2.5.0   "./opensearch-dashbo…"    About a minute ago   Up About a minute (healthy)   <IP_ADDRESS>:5601->5601/tcp, :::5601->5601/tcp                                                                  opensearch-dashboards
a041e688a8ef   supertokens/supertokens-postgresql:5.0          "docker-entrypoint.s…"    About a minute ago   Up About a minute (healthy)   <IP_ADDRESS>:3567->3567/tcp, :::3567->3567/tcp                                                                  pezzo-supertokens-1
a2e7cb5048c7   redis/redis-stack-server:7.2.0-v0               "/entrypoint.sh"          About a minute ago   Up About a minute (healthy)   <IP_ADDRESS>:6379->6379/tcp, :::6379->6379/tcp                                                                  redis-stack-server
b1cadfa3f546   opensearchproject/opensearch:2.5.0              "./opensearch-docker…"    About a minute ago   Up About a minute (healthy)   <IP_ADDRESS>:9200->9200/tcp, :::9200->9200/tcp, 9300/tcp, <IP_ADDRESS>:9600->9600/tcp, :::9600->9600/tcp, 9650/tcp   opensearch-node1
a07b136255fa   postgres:15-alpine3.17                          "docker-entrypoint.s…"    About a minute ago   Up About a minute (healthy)   <IP_ADDRESS>:5433->5432/tcp, :::5433->5432/tcp                                                                  pezzo-postgres-1
538db706616d   nsmithuk/local-kms:latest                       "/bin/sh -c 'echo \"S…"   About a minute ago   Up About a minute             <IP_ADDRESS>:9981->9981/tcp, :::9981->9981/tcp

meaning all is up and running.

Then I go through signup - that works. When I click the 'signup' the requests pass but all I see is:

In the console:

localhost-1699570760167.log
localhost-1699570750903.log

I started experiencing the same issue.

Any update here? Can I help in any way?

Hey folks, I'm still investigating the root cause. It's pretty recent. If you want to run Pezzo and try it out, I suggest trying v0.6.9 prior to a recent update we've made, where this is a non issue. Upgrading once fixed should be straightforward - there are no breaking changes. Sorry about that.

@arielweinberger thank you, not a problem at all!

@grski @arielweinberger Hello guys, any updates in this issues? I faced the same issue even with v0.6.9 !

@grski  @arielweinberger  Hello Folks, Do we have any update here ?

@dmockingbird551 I have tried to reproduce the issue and failed. Since it was created some things have changes. Can you try to see if it still happens to you and share new logs?
I will try to help as much as possible
@Hicham-azer fyi

@ranst91  Getting the same issue
`> add_bins_new % git clone https://github.com/pezzolabs/pezzo.git
Cloning into 'pezzo'...
remote: Enumerating objects: 9135, done.
remote: Counting objects: 100% (205/205), done.
remote: Compressing objects: 100% (149/149), done.
remote: Total 9135 (delta 67), reused 142 (delta 50), pack-reused 8930
Receiving objects: 100% (9135/9135), 29.56 MiB | 8.87 MiB/s, done.
Resolving deltas: 100% (5363/5363), done.

add_bins_new % cd pezzo

pezzo % docker compose up

WARN[0000] The "SUPERTOKENS_DATABASE_URL" variable is not set. Defaulting to a blank string.
WARN[0000] /Users/deepak/add_bins_new/pezzo/docker-compose.infra.yaml: version is obsolete
env file /Users/deepak/add_bins_new/pezzo/.env not found: stat /Users/deepak/add_bins_new/pezzo/.env: no such file or directory

pezzo %
`

This log seems to happen because of missing .env file.
This section of the readme was updated to reflect this requirement:
https://github.com/pezzolabs/pezzo?tab=readme-ov-file#set-up-the-environment-files

Can we please fix this issue ? I'm not able to use ( I see similar problems in running locally in other threads) this awesome library because of these issues.

@dmockingbird551 use the .env.example file to create a .env (and possibly also a .env.docker) files (include your actual credentials in the new files)
If you are still unable to use, the logs should tell a different story, feel free to post the new errors here

Hi sir, the bug is fixed? Im still got same error when run with docker


Add Futhark language support

Description

This PR adds support for Futhark, a language for "high-performance purely functional data-parallel array programming on the GPU"
Checklist:

[x] I am adding a new language.

[x] The extension of the new language is used in hundreds of repositories on GitHub.com.

Search results for each extension:

https://github.com/search?utf8=✓&type=Code&ref=searchresults&q=extension%3Afut+NOT+nothack

[x] I have included a real-world usage sample for all extensions added in this PR:

Sample source(s):

https://github.com/diku-dk/futhark-website/blob/master/examples/gaussian-blur.fut

Sample license(s):

ISC

[ ] I have included a syntax highlighting grammar: https://github-lightshow.herokuapp.com/
[ ] I have updated the heuristics to distinguish my language from others using the same extension.

There are no other languages using the same extension.

There is an emacs-mode that provides highlighting (and I think perhaps a gedit-grammar file or something), but Futhark doesn't currently have a TextMate syntax highlighting grammar. Is it required, and if so, any tips on how to get started? I don't have TextMate unfortunately (or a Mac)...

Please follow the instructions in CONTRIBUTING.md for adding a new language.

I see I was treating the language_id incorrectly. That has been fixed. The only other thing I can think of that's missing is the grammar. Is that required?

👋 Hey @Munksgaard, apologies for the slow response.

The only other thing I can think of that's missing is the grammar. Is that required?

No, it's not required. Grammars are only used to provide syntax highlighting, which is a nice-to-have, but entirely optional. Many languages recognised on GitHub lack grammars of their own, or use the grammar of a syntactically-similar language as a compromise.
I could write you one if you provided a definitive language specification (preferably one with an ABNF or EBNF grammar).

wave Hey @Munksgaard, apologies for the slow response.

Hi @Alhadis, no worries at all! I realize this is a busy project :-)

The only other thing I can think of that's missing is the grammar. Is that required?

No, it's not required. Grammars are only used to provide syntax highlighting, which is a nice-to-have, but entirely optional. Many languages recognised on GitHub lack grammars of their own, or use the grammar of a syntactically-similar language as a compromise.
I could write you one if you provided a definitive language specification (preferably one with an ABNF or EBNF grammar).

The closest we have is the one defined here: https://futhark.readthedocs.io/en/latest/language-reference.html
It's missing some keywords though, but we currently have the following: "true", "false", "if", "then", "else", "let", "loop", "in", "val", "for", "do", "with", "local", "open", "include", "import", "type", "entry", "module", "while", "unsafe", "assert", "match", "case"

Actually wait, I missed your earlier comment about an Emacs futhark-mode… I was going to use it as a loose reference, but noticed it's released under a permissive license. 😁 Normally these things are GPL/copyleft, which requires me to write an entirely new grammar from scratch (to eliminate any "issues" with licensing and distribution, as Linguist is permissively-licensed). But because futhark-mode is ISC-licensed, I can skip the research and just port the thing by hand. 😀
BTW, you see that Emacs Lisp file's highlighting? You're welcome. 😉

Actually wait, I missed your earlier comment about an Emacs futhark-mode… I was going to use it as a loose reference, but noticed it's released under a permissive license. grin Normally these things are GPL/copyleft, which requires me to write an entirely new grammar from scratch (to eliminate any "issues" with licensing and distribution, as Linguist is permissively-licensed). But because futhark-mode is ISC-licensed, I can skip the research and just port the thing by hand.
EDIT: Hrm, that Emacs mode has a lot of discrepancies with the official specification… I'll take what I can from both, but it seems I haven't entirely skipped the reading/research phase book

Hm, good point. We should probably see about updating that emacs mode as well at some point. Thanks for pointing it out for us!

BTW, you see that Emacs Lisp file's highlighting? You're welcome. wink

Haha, that's great :smile:

Alright, try this. It's pretty much a one-to-one translation of futhark-mode's pattern-matching, albeit with a few corrections I spotted while perusing the spec.
Let me know if that looks good to you.

That looks great @Alhadis, thank you so much!

Hm, I'm not sure how to add the grammar, doing ./script/add-grammar https://github.com/Alhadis/language-etc doesn't seem to do much, perhaps because your repository is already being used?

Ah, that merits an explanation. Normally, we add new grammars by registering a new submodule with Linguist, a process that add-grammar exists to simplify. As you guessed, this only works if a module hasn't already been added. We don't expect users will need to manually-update an existing submodule to add a grammar that was recently added upstream.
After publishing ~20 different grammar repositories, I've now taken to adding new grammars to a single monolithic collection, which is easier for me to maintain and keep track of. Unfortunately, it does make situations like these tricky....

@Munksgaard I've pushed a manual update to your branch. Should be okay now. 👍

@Munksgaard I've pushed a manual update to your branch. Should be okay now. +1

That's so nice of you, thank you for all your hard work!

@lildude I believe I've followed the instructions in CONTRIBUTING.md now. Is there anything else I need to do?

Nope. LGTM now.


[FEATURE] Allow inputting the requested parameters as program arguments

Is your feature request related to a problem? Please describe.
Currently you have to manually put in each argument as the program asks for them, this would allow inputting some or all of them as program arguments.
Describe the solution you'd like
Allow passing the various inputs it asks for as program arguments. They should be input in the same order as asked for, so link fName downloadsFolder. The program should still ask for any values not provided, or if a blank value was provided.
if downloadsFolder is y then it should try to find the default downloads folder. If it cannot find that or n was passed in then it should ask for one. If it is any other value it should be interpreted as a path. This may cause a conflict with saving to drives named y or n, I am not sure how that works exactly. If that is a problem then this could be split into two arguments, one for if a custom path should be used and a second for the custom path.
It could be worthwhile reording the arguments so that link and fName come last, which would make it easier to have a sort of 'default' downloader setup that has the downloads folder selected.
Example program arguments: https://example.com/my_file.txt "My file.txt" y which would download the file from the mentioned link and save it as My file.txt in the default downloads folder if found, asking the user for input if it cannot be found.

hey @SaptarshiSarkar12 can I take this up? Let me know

Yes @Rishav-12 . Please make sure that you will link this issue to your PR.

Hey, I'm sorry but I think I won't be able to do this since my code breaks for some reason and I'm not willing to dive too much into these errors. So I'll unassign myself, if anyone else wants to take this up feel free

Ok @Rishav-12 .
If anyone wants to work on this issue, please comment below and I shall assign you with the task.

I can work on this.

I can work on this.

Okay, assigning you with the task. Please link this issue to your PR


Configurable background opacity

Currently the opacity is a hard-coded value in stylesheet.css:
background-color: rgba(0, 0, 0, 0.8);

That's true, I will take a look into this soon! Thank you!

Now the overlay background color is fully customizable, including both the color and opacity from the same setting. Thank you for the report @awumii !!


Resolve s360 alerts for OpenAPI Hub

Kudos to @benbp for guiding me through this and solving the alerts


vbump for tsg book

This PR fixes #9565

Please remove "fixes" because we want the issue to remain open for the PR into the release branch.


ignore node_modules everywhere as default

Reasons for making this change:
Since practice is to pull node_modules relative to the box, the default config should be for global node_modules ignore to include subdirectories.
Links to documentation supporting these rule changes:
Documents relating to the original intent of node_modules, and variance of Javascript compilation across different environments.

I've been burned by changes introduced by changing the rules around node_modules so apologies for being thorough with the review. I'd also love some citations for the change we're considering here, so that I can understand that the community is behind this in some respect.

np, if no added benefit then please deny merge

absolutely that's the desired end.. maybe it was an artifact that nested node_modules dirs weren't ignored in my case. if in case you find that reproducible, please feel free to reopen.


[Bug] Gets the wrong OS type to sys.os when OS is Linux

Gets the wrong OS type to sys.os when OS is Linux

Fixed It at:https://github.com/cocos2d/cocos2d-html5/pull/2583


How to provide tags using push.RegisterAsync(string RegiId, JObject templates)

I am not able to find any sample which, shows how to provide tags while registering a device using above method. If this is possible, the please let me know. If not, then I guess I need to handle it by writing my own API. Please let me know.

When I looked into GitHub to check the source to figure out the problem, I was able to locate the correct format of templates object.
push.RegisterAsync(.., ..) method eventually calls the following controller
https://github.com/Azure/azure-mobile-apps-net-server/blob/master/src/Microsoft.Azure.Mobile.Server.Notifications/Controllers/NotificationInstallationsController.cs
This controller has two parameters. One of them is an object of the following class
https://github.com/Azure/azure-mobile-apps-net-server/blob/master/src/Microsoft.Azure.Mobile.Server.Notifications/NotificationInstallation.cs
So, to provide tags correctly, a client must provide a valid instance to NotificationInstallation. You may want use the public property named "Tags" here and provide a list of tags. However, this would still not work because of the following bug. I have created this issue within project  Microsoft.Azure.Mobile.Server.Notifications on GitHub. I am yet to hear something from the project moderator or members. So I don't know whether what I have proposed is correct or not.
https://github.com/Azure/azure-mobile-apps-net-server/issues/162
However, I needed to complete my work so I implemented my own version of NotificationInstallationsController to fix the issue. It works for me indeed. So I think that what I have proposed should be correct.
I am adding this explanation here, in case, if some one is having the same problem and looking for any solution, then this would help. If you need further information, please let me know.

This is an old problem.  The answer is to use the new "App Service Push", which has replaced the old push blade.  Use of this push blade allows you to define tags that you allow.  Alternately, you can generate a custom API with your own logic that drives the Notification Hub.  Either way will work.
See also #247 that talks about an update to the client SDK that allows you to specify tags and templates.

Ok. Thanks a lot for your help!
From: Adrian Hall<EMAIL_ADDRESS>Sent: Tuesday, November 15, 2016 12:15 AM
To: Azure/azure-mobile-apps-net-client<EMAIL_ADDRESS>Cc: Mayur Bharodia<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-mobile-apps-net-client] How to provide tags using push.RegisterAsync(string RegiId, JObject templates) (#234)
This is an old problem. The answer is to use the new "App Service Push", which has replaced the old push blade. Use of this push blade allows you to define tags that you allow. Alternately, you can generate a custom API with your own logic that drives the Notification Hub. Either way will work.
See also #247https://github.com/Azure/azure-mobile-apps-net-client/issues/247 that talks about an update to the client SDK that allows you to specify tags and templates.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-mobile-apps-net-client/issues/234#issuecomment-260565190, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJ3YwfWjrxPYhYvZm0LNUaP3v95MPkQrks5q-VvegaJpZM4KMBqa.


calico role is unable to unblock openstack ports

Attempting to proceed with calico deployment in openstack i get the following error when try to unblock the neutron  ports. Could you please advise ?
failed: [mi-worker-003] => (item=mi-worker-003) => {"changed": false, "cmd": "/usr/local/bin/neutron_port_update.py \"<IP_ADDRESS>/16\" \"fa:16:3e:a6:94:7d\"", "delta": "0:00:20.117342", "end":
"2016-01-08 10:03:28.770773", "failed": true, "failed_when_result": true, "item": "mi-worker-003", "rc": 1, "start": "2016-01-08 10:03:08.653431", "stdout_lines": [], "warnings": []}
stderr: Traceback (most recent call last):
  File "/usr/local/bin/neutron_port_update.py", line 125, in <module>
    catalog = get_catalog()
  File "/usr/local/bin/neutron_port_update.py", line 56, in get_catalog
    r = requests.post(auth_url, headers=headers, data=json.dumps(payload))
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 108, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 464, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/adapters.py", line 415, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', gaierror(-2, 'Name or service not known'))
failed: [mi-worker-003] => (item=mi-worker-002) => {"changed": false, "cmd": "/usr/local/bin/neutron_port_update.py \"<IP_ADDRESS>/16\" \"fa:16:3e:1d:c1:42\"", "delta": "0:00:20.117476", "end":
"2016-01-08 10:03:49.056929", "failed": true, "failed_when_result": true, "item": "mi-worker-002", "rc": 1, "start": "2016-01-08 10:03:28.939453", "stdout_lines": [], "warnings": []}
stderr: Traceback (most recent call last):
  File "/usr/local/bin/neutron_port_update.py", line 125, in <module>
    catalog = get_catalog()
  File "/usr/local/bin/neutron_port_update.py", line 56, in get_catalog
    r = requests.post(auth_url, headers=headers, data=json.dumps(payload))
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 108, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 464, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/adapters.py", line 415, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', gaierror(-2, 'Name or service not known'))
failed: [mi-worker-003] => (item=mi-worker-001) => {"changed": false, "cmd": "/usr/local/bin/neutron_port_update.py \"<IP_ADDRESS>/16\" \"fa:16:3e:aa:6f:44\"", "delta": "0:00:20.104057", "end":
"2016-01-08 10:04:09.325214", "failed": true, "failed_when_result": true, "item": "mi-worker-001", "rc": 1, "start": "2016-01-08 10:03:49.221157", "stdout_lines": [], "warnings": []}
stderr: Traceback (most recent call last):
  File "/usr/local/bin/neutron_port_update.py", line 125, in <module>
    catalog = get_catalog()
  File "/usr/local/bin/neutron_port_update.py", line 56, in get_catalog
    r = requests.post(auth_url, headers=headers, data=json.dumps(payload))
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 108, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 464, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/adapters.py", line 415, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', gaierror(-2, 'Name or service not known'))
failed: [mi-worker-003] => (item=mi-edge-01) => {"changed": false, "cmd": "/usr/local/bin/neutron_port_update.py \"<IP_ADDRESS>/16\" \"fa:16:3e:4d:1d:3c\"", "delta": "0:00:20.108533", "end": "20
16-01-08 10:04:29.581687", "failed": true, "failed_when_result": true, "item": "mi-edge-01", "rc": 1, "start": "2016-01-08 10:04:09.473154", "stdout_lines": [], "warnings": []}
stderr: Traceback (most recent call last):
  File "/usr/local/bin/neutron_port_update.py", line 125, in <module>
    catalog = get_catalog()
  File "/usr/local/bin/neutron_port_update.py", line 56, in get_catalog
    r = requests.post(auth_url, headers=headers, data=json.dumps(payload))
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 108, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 464, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/adapters.py", line 415, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', gaierror(-2, 'Name or service not known'))
failed: [mi-worker-003] => (item=mi-edge-02) => {"changed": false, "cmd": "/usr/local/bin/neutron_port_update.py \"<IP_ADDRESS>/16\" \"fa:16:3e:b6:c6:50\"", "delta": "0:00:20.106809", "end": "20
16-01-08 10:04:49.842076", "failed": true, "failed_when_result": true, "item": "mi-edge-02", "rc": 1, "start": "2016-01-08 10:04:29.735267", "stdout_lines": [], "warnings": []}
stderr: Traceback (most recent call last):
  File "/usr/local/bin/neutron_port_update.py", line 125, in <module>
    catalog = get_catalog()
  File "/usr/local/bin/neutron_port_update.py", line 56, in get_catalog
    r = requests.post(auth_url, headers=headers, data=json.dumps(payload))
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 108, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 464, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/adapters.py", line 415, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', gaierror(-2, 'Name or service not known'))

@Zogg can you take a look at this an advise?

It looks like the problem is here, which is where the script attempts to reach the server OS_AUTH_URL. Are all your environment variables set correctly?

As @siddharthist mentioned, a shell from which ansible is run must have environment variables set for openstack:
  environment:
    OS_USERNAME: "{{ lookup('env','OS_USERNAME') }}"
    OS_PASSWORD: "{{ lookup('env','OS_PASSWORD') }}"
    OS_TENANT_NAME: "{{ lookup('env','OS_TENANT_NAME') }}"
    OS_AUTH_URL: "{{ lookup('env','OS_AUTH_URL') }}"

Trying to further debug this issue attempted to manually set the environmental vars and run the scripts. It seems that the script is able to reach the AUTH_URL however fails in a later step trying to parse a returned value. Below is the output got
#> python neutron_port_update.py <IP_ADDRESS>/16 XX:XX:XX:XX:XX:XX
Traceback (most recent call last):
  File "neutron_port_update.py", line 128, in <module>
    ports = list_ports(token, public_url)
  File "neutron_port_update.py", line 83, in list_ports
    r = requests.get(auth_url, headers=headers)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 68, in get
    return request('get', url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 450, in request
    prep = self.prepare_request(req)
  File "/usr/lib/python2.7/site-packages/requests/sessions.py", line 381, in prepare_request
    hooks=merge_hooks(request.hooks, self.hooks),
  File "/usr/lib/python2.7/site-packages/requests/models.py", line 304, in prepare
    self.prepare_url(url, params)
  File "/usr/lib/python2.7/site-packages/requests/models.py", line 357, in prepare_url
    raise InvalidURL(*e.args)
requests.exceptions.InvalidURL: Failed to parse: neutron.XXXX.XXXX.XXXX.XXXX:9696v2.0

@ipolyzos, I think it's a problem with the public_url variable. Can you post your  OS_AUTH_URL environment variable (but editing out sensitive information)?
It might be a good idea to add some logging to this script. Alternatively, we could think about starting managing port resources with Terraform. Such functionality has been recently made available: http://www.terraform.io/docs/providers/openstack/r/networking_port_v2.html

+1 if we can get this in terraform.

+1 for using terraform
@loshkovskyi  the OS_AUTH_URL value is https://keystone.XXX.XXX.XXX.net:5000/v2.0

@ipolyzos, sorry for long delay in my reply. I think I was able to reproduce this problem. In some environments Neutron public URL in Keystone catalog can contain a trailing slash which may result in such issues with the script run. You can check the Neutron public URL with:
 openstack catalog show neutron

Do you mind testing my patch https://github.com/CiscoCloud/microservices-infrastructure/commit/a612b3deccdc6fc366629b1b6770b90c4581ee08 and see if it fixes this issue for you? With this patch it doesn't matter if the URL contains the trailing slash or not.

I had this issue and tested your patch against OpenStack Mitaka.
It worked perfectly for me. In my case I didn't have a trailing '/':
ubuntu@mantl-bootstrap:~$  openstack catalog show neutron
+-----------+-------------------------------------+
| Field     | Value                               |
+-----------+-------------------------------------+
| endpoints | RegionOne                           |
|           |   internal: http://<IP_ADDRESS>:9696 |
|           | RegionOne                           |
|           |   public: http://<IP_ADDRESS>:9696   |
|           | RegionOne                           |
|           |   admin: http://<IP_ADDRESS>:9696    |
|           |                                     |
| id        | 7aafaba700a0482ead4e62c4508e7d39    |
| name      | neutron                             |
| type      | network                             |
+-----------+-------------------------------------+

But it was still the workaround!

This was closed in #1167. I'll make a new issue for tracking using terraform to unblock the ports.


update logging components from 0.42.0 to 0.43.0

How to categorize this PR?

/area logging
/kind enhancement
What this PR does / why we need it:
This PR updates the logging components from version v0.42.0 to v.043.0 and adapts the gardener for the new features:

scrapes events from the shoot default namepsace.
pack the shoot's log before being sent to the central Loki when the shoot is in the creation or deletion state.
remove the __gardener_multitenant_id__ label when logs are sent to the central Loki.
the name of the batch ID label can be changed via the fluent-bit-to-loki plugin configuration.
Also, fix a race condition in the multitenant client which could fix issue4747

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

The event-logger watches events of multiple namespaces specified by `--seed-event-namespaces` and `--shoot-event-namespaces` like comma-separated values.
The flags `--seed-event-namespace` and `--shoot-event-namespace` are dropped.

All split logs between shoot and central Loki are packed before being sent to the central Loki.

Remove the `__gardener_multitenant_id__` label in the `fluent-bit-to-loki` plugin when it is not needed.

The name of the batch ID label in the `fluent-bit-to-loki` plugin can be set via IdLabelName from the plugin configuration.

Remove race condition in the `fluent-bit-to-loki` plugin for the gardener multitenant client when the labels set is not cloned.

/assign @oliver-goetz

/approve


fix .htaccess for apache to v2.2 or 2.3 and later

mod_authz_core is authorization core, but the responsible of allow/deny access to host/ip/... is mod_authz_host or for compatibility reasons mod_access_compat, this is a reasonable aproach for a variety of installations.

ping @skilla ? did you test these ruiles on apache 2.2 and 2.4?

And does the "old" approach not work (or being deprecated)? Since these .htaccess files are just here for security, I'd favor having something that always works and is clear. So, unless the existing code doesn't work on some new Apache version, I'd be :-1: on this

The require directive comes from mod_authz_core.

mod_authz_host extends the authorization types with ip, host and local. Other authorization types may also be used but may require that additional authorization modules be loaded.

But since we only use "Require all denied", we dont need mod_authz_host.


Resolve RSpec/VerifiedDoubles todos

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

This PR resolves the RSpec/VerifiedDoubles in Library/Homebrew/test/.rubocop_todo.yml
Relates to https://github.com/Homebrew/brew/issues/14250 (apologies if this stepping on someone's toes, it's assigned but has been idle)

Wonderful, thanks so much for this @dduugg, this is great work! Hope to see more PRs from you here, all yours so far have been perfect and make me happy 😀


Add build/release branch for jammy (Ubuntu 22.04[.1] LTS)

I noticed that there's currently no support for Ubuntu 22.04 (released last April).  It appears to build correctly, so I'm assuming this is just a matter of setting up the builds somehow.  What else needs to be done to get an actual binary distribution built and available?

Voicing my support for this. With ROS1 on the path to deprecation and ROS2 supporting 22.04 and beyond only, I am being left in an impossible situation. Compatibility with Jammy should be a high priority.

I too support this! For the same ROS2 reason @bbokser-ai sited.


Better formatting of author and author_url

Currently it looks quite messy with author displayed.
We need to have better formatting for each item to show them nicely. Then we can add back author_url.
Some ideas:

Use | as separator

Since each record now contains more attributes, I think presenting the data in a tabular format would increase its readability.

Name
Category
Year
Author
Note

Angular
Framework
2016
Misko Hevery and Adam Abrons

AngularJS
Framework
2010 - 2021
Misko Hevery and Adam Abrons
Replaced by Angular

Anime.js
Library
2016
Julian Garnier


remove types field to fix webpack properties resolution

"So, if you have followed convention or used a tooling set such as npm to download @types packages, you don't need to configure typeRoots or types in your configuration as it will work out of the box with the default folder structure."
https://stackoverflow.com/a/42239406/8638301

will you make more changes ? Is it ready to review ? Looks done to me

updated package-lock

ready to merge

I merged it but type-checker throws lot of errors now.

@GrayStrider  Im gonna revert the pull request now. I deleted nodemodules and package lock and installed again. No change.

added fix #41


Avoid warning fatigue

Today we flag every application with a "issue" as yellow and a warning sign, resulting in a daunting amount of yellowness causing fatigue.
Many of the kind of warnings we give, is things that we don't have any real push behind. It's not actually important, but just nice if they fix. Examples of this is deprecated ingresses, images repositories etc.
We should make them aware of this, but change the wording and visual significance that this has - to better be able to highlight what is actually important and critical that they fix. A better word for this category might be "Todo", "Notification", "Remark". "Something to be aware of and fix when you have the spare time, because it might be deprecated harder in the future"

More examples: missing outbound rules for applications.

https://nav-it.slack.com/archives/C5KUST8N6/p1701082885188839
https://nav-it.slack.com/archives/C5KUST8N6/p1702383584277669

Today, the access policy block configures both application/token auth in addition to network policies. The former does not concern itself with outbound rules at all, and thus these warnings are misleading for applications that only need auth without performing service discovery calls.

@thokra-nav did you have any comments on this?

Yeah, mostly that I do think deprecated ingresses etc should be a proper warning and not hidden as something else. But we should also have a turn off date or something similar when we consider something as deprecated.
So I do like what someone mentioned, a best practices checklist for things we consider bad practice, want to reduce usage of, but haven't set a deadline for (so not really deprecated).

GHCR registry warning should be hidden as well imho.


SQL Prompt suggestion listboxes get maximized after installing power toys

Environment
Windows build number: 10.0.19008.1
PowerToys version: 0.11.0
PowerToy module for which you are reporting the bug (if applicable):

Steps to reproduce
Install SQL Prompt ver 9.5 and enable suggestions.
Expected behavior
Before installing power toys the SQL Prompt suggestion listboxes were of normal size.
Actual behavior
After installing power toys the SQL Prompt suggestion listboxes are maximized. I can no longer use that function in SQL Prompt. Even after uninstalling power toys and reinstalling SQL Prompt the suggestion listboxes are now maximized.
I have also tried to restore the system to a restore point prior to installing power toys but the suggestion listboxes are still maximized.
Screenshots

Hi @m466e
thank you for reporting the issue, we will investigate it.

hi @m466e, to verify you are using Red Gate's Sql Prompt, correct?  https://www.red-gate.com/products/sql-development/sql-prompt/

Hi Clint, Yes, it is.
However, this might not be a power toys error so I'm sorry for reporting it :-S
I discovered that the suggestion box had the same size as my display resolution and since the top row of a document in sql management studio is some 100 pixels from the top of the display the resizing grip of the suggestion box got hidden outside the bottom of the display.
By changing the display resolution to a lower resolution the size of the suggestion box changed and when changing back to the higher resolution the suggestion box kept the display size of the lower resolution.
I could then resize the suggestion box and now it keeps that size.
The maximizing of the suggestion box happened when I installed power toys (ok ok, I wouldn't bet the farm on it...), but now everything looks good with power toys installed (and fancy zones activated).

Never be sorry for reporting an issue. If it pops up again on a diff machine, reopen and we will look into it


[SPARK-45725][SQL][FOLLOWUP] Fix arguments of a removed SQL config

What changes were proposed in this pull request?
In the PR, I propose to fix the order of arguments of the removed SQL config park.sql.optimizer.runtimeFilter.semiJoinReduction.enabled and check the format of Spark version in the constructor of RemovedConfig.
Why are the changes needed?
To don't confuse users and prevent such kind of mistakes.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
By running the existing test suite:
$ build/sbt "test:testOnly *SQLConfSuite"

without the fix the test suite fails with the internal error:
Caused by: org.apache.spark.SparkException: [INTERNAL_ERROR] The removed SQL config spark.sql.optimizer.runtimeFilter.semiJoinReduction.enabled has the wrong Spark version: false SQLSTATE: XX000
	at org.apache.spark.SparkException$.internalError(SparkException.scala:92)
	at org.apache.spark.SparkException$.internalError(SparkException.scala:96)

Was this patch authored or co-authored using generative AI tooling?
No.

Merging to master/3.5/3.4. Thank you, @beliefer @cloud-fan for review.


pull request for e2e test at 01/02/2017 10:45:05

:white_check_mark: Validation status: passed
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.


Move discovery related code to separate component

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
35 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.7%) to 97.648%

Files with Coverage Reduction
New Missed Lines
%

custom_components/powercalc/config_flow.py
1
99.47%

custom_components/powercalc/init.py
3
95.71%

custom_components/powercalc/sensors/power.py
9
96.42%

custom_components/powercalc/sensor.py
22
92.38%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
2699

Relevant Lines:
2764

💛  - Coveralls


Added badges to README.md

Who doesn't like a good badge?

Coverage remained the same at 100.0% when pulling 9794f6d9787e30c97e96af14d1780051596a9dc2 on feature/readme_badge into 96d4ded06d4fcc3022839e4f93e8ea75efd43ac1 on develop.


”Gherța Mică” is listed in the drop-down menu

Description:
"Gherța Mică" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Gherța Mică" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Gherța Mică, SATU MARE".


Failed jobs being marked as completed.

Right now when a job fails it is being marked as completed instead which causes 404s in the UI due to the client attempting to retrieve the reports from a job that actually failed.

Not sure if this is actually bug or was a result of all the restarts creating invalid jobs.  Will keep an eye on it.


No longer working with self contained: true

I'm using the latest dev version of quarto-cli. The fontawesome extension was working fine with self-contained: true but looks to have recently stopped working:
---
title: "Self contained true"
format:
  html:
    self-contained: true
---

{{< fa thumbs-up >}}
{{< fa folder >}}
{{< fa chess-pawn >}}
{{< fa brands bluetooth >}}
{{< fa brands twitter size=2xl >}}
{{< fa brands github size=5x >}}
{{< fa battery-half size=Huge >}}
{{< fa envelope title="An envelope" >}}

Self-contained false still works fine:

This should be fixed in last prerelease. Please reopen if there is still something wrong


Mangadex: Manga "Completed" Status broken/not added

Steps to reproduce
Need to view a manga which is both publication complete and scanlation complete:

Open one of the mangadex links below
Check status of the manga

Links:

https://mangadex.org/title/c3ffcb40-f4c0-4046-9d29-f4f82cf32515
https://mangadex.org/title/881fccb1-e60c-4c82-b64c-28f709545174

Expected behavior
Above manga should show up as "Completed"
Actual behavior
Above manga shows up as "Publication Complete"
Crash logs
N/A
Tachiyomi version
Preview 412
Android version
Android 11
Device
Onclite
Other details
Neko works fine.
Parts of Dex code in Sy is still commented out.
Relevant part of Sy: https://github.com/jobobby04/TachiyomiSY/blob/892f64829b8e925d8429442629201c77c8aececc/app/src/main/java/exh/md/handlers/ApiMangaParser.kt#L101
Relevant part of Neko: https://github.com/CarlosEsco/Neko/blob/master/app/src/main/java/eu/kanade/tachiyomi/source/online/handlers/ApiMangaParser.kt#L87
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.7.0.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Addressed by https://github.com/jobobby04/TachiyomiSY/commit/0df5376545a0d1c116042346e97d8b985de2047b


Update foundations modules versions

Update folders, log sinks, service accounts versions to avoid errors with recent Terraform versions.

Tests for foundations are ok while infrastructure tests have errors, will address in a separate PR.


Show all ase alerts

PR Checklist

The ASE alerts tab currently does not show information about resolved alerts. This means that we cannot see information about, for example, an ASE restart from the previous day. We changed this to show all types of alerts from the ASE in this tab

Screenshot from before change:

Screenshot from after change:

Please can you include the alert condition in the columns shown, so the user can see if the alert is still active or is resolved? And then you can add some custom formatting to show an icon based on the alert condition, like in the main alerts blade:

Also, please apply the same change to the "all alerts" view on the overview tab.


GCP Support

Hi, When will GCP be supported? Is there a beta version?

Hello @Musseri10, thanks for your interest. Adding GCP support is in our roadmap for this year however we don't really know when we can start coding that part.

Hey @Musseri10, we have now a branch with the first integration with GCP including a bunch of checks, it is here https://github.com/prowler-cloud/prowler/pull/2125 please feel free to test it if you like.


Youtube API quota

We received a mail mid-August that I had missed:

To verify that all use of the API is in compliance with the Terms of Service (including the Developer Policies), we are conducting a periodic audit of API developers. Please complete the audit form, linked below within the next 30 days to ensure continued access to the service. For the reason for filling this form, please choose "I have been selected for a Periodic Audit".

And a couple of weeks ago:

Because we have not yet received your completed form, the API quota allocated to your projects has been adjusted as follows:
--
Project shields-io-XXXX from 200,000 to 66,600 units
Please complete the form as soon as possible. Your original quota will be restored after successful completion of the audit.

We are currently performing between 70k and 80k requests on weekdays, yet we've not yet been rate limited despite the lower quota. However, to be on the safe side as there's little indication on whether they'll further reduce our limit, I'll submit their form a little later. The last audit was barely over a year ago and was a flamboyant display of Google's bureaucracy and appalling developer experience, I'm hoping this will go more smoothly this time around.

YouTube is now asking for things like the physical address of our organisation. I had to put a bogus value, hopefully won't cause any trouble.

After we got the most recent rate limit increase, we decreased the time we cache youtube badges for. Last time we had issues with this, we set them to cache for 2 hours which massively reduced our usage.
We could always do this again
https://github.com/badges/shields/pull/9372
For a service like youtube where we are presenting number of views/subscribers/comments/likes etc which are indicative stats (this isn't build status passing/failing) and usually presented as rounded numbers we can get away with cacheing them for longer than we do at the moment without much impact.

Sigh. YouTube are claiming eight violations of their terms and conditions, even though nothing has changed neither on the Shields.io side nor in their policies since last successful audit. This is ridiculous, I'm pushing back.

Latest audit has been approved, will go ahead and close this issue.


Add docs for processing PublishingAPI messages

This adds an overview of the process where we consume messages from the PublishingAPI to track content changes.
Review Checklist

[x] Changes in scope.
[x] Added to Trello card.

 (2) Onboarding documentation - publishing API integration

@cbaines addressed your comments


Add isSame function with granularity

Hello,
I need some granularity for the isSame method so I created the missing bindings.
Also it looks like I have a refmt issue

Hey @DCKT,
Your changes look good to me!

Hello @Jimexist I think so

Hello there,
Any news about this PR ? 😄

Hey @DCKT,
Your changes look good to me!

Hey @DCKT,
Your changes look good to me!


Cache mirrors

Anyone know of any more cache mirrors?

With the shutting down of Torcache this has become critical. There are only two sources in the code from the looks of it, and both are now defunct, meaning no torrents are being cached. This has the flow on effect that reflektor.karmorra.info now doesn't work, which was crucial for my needs with showrss.info (I need full torrent files, magnets won't work).
Please replace the caches with working ones, here and on karmorra.info. iTorrents.org is apparently marketing itself as a replacement. A quick Google search also found btcache.me, torrage.info and torrasave.site.

Any news on this? I'm in the same situation as @colonelpanic-42.


feat(server): add option to save logs to a file

What problem are you trying to solve?
We have been observing several issues when using the server in the IDE:

intelliJ reports having multiple instances of the server that won't be automatically shutdown
a user in VS Code reported a case where multiple processes where present at the same time after leaving his computer idle all the night (unable to reproduce on our side)

What is your solution?
In order to better understand what's going on it would be nice to have the generated logs of the server saved into a file.
This PR adds a new option in the CLI to roll all the logs to a file. The user can decide the cadence: minutely, hourly or daily. These logs will be using JSON format for easy parsing.
These logs are saved into the /tmp folder in UNIX and will default to the TEMP env var in windows, which is usually pointing to C:\Users\<user_name>\AppData\Local\Temp. If not present, for whatever reason, then it will fallback to c:\Temp.
The idea is to leverage this option to enable a setting in the IDEs that will allow users and the dev team to enable this feature in the binary and then easily debug the static analysis server processes.
Alternatives considered
What the reviewer should know
Tested it on Windows 11 and it works, thought the compilation process didn't work out of the box due to some UNIX commands being used in the kernel build.rs file and a wrong dependency version. I managed to fix all that. I'll prepare another PR for that.
IDE-3768

I think I didn't provide a good way of flushing the messages. I'll merge this PR anyway and provide the fix in another.


Allow 'bynder_id' to be null

Projects should be able to make use of the Wagtail image model without having to specify bynder_id or any other field used for syncing with Bynder. For example, on a project I'm working on currently:

I have a Shopify integration that needs to be able to save product images as Wagtail images, allowing Wagtail's {% image %} tag to be used to render thumbnails at different sizes.
I have a Sponsor model with a logo image field, which I silently convert to a Wagtail image on save, allowing it to also be rendered at different sizes.

In order to support this properly, the bynder_id field needs to have null=True added to it, allowing the field to be completely ignored when not needed. Currently, it can only be side-stepped by generating a unique string value.

Resolved by #19


Disallow access to DOM globals in bindings

This makes it impossible to bind to e.g.
JSON.stringify(document.getElementById('hello')?.textContent)

As a side-effect this also makes it impossible to bind to localStorage. This was never intended to be supported. We will provide an alternative for its use-case.

These are not arbitrarily chosen, these are the globals defined by the ecmascript standard. We just don't support the DOM because we're not a browser.
It shouldn't be more confusing than using any other non-browser javascript runtime. Like node.js.

These are not arbitrarily chosen, these are the globals defined by the ecmascript standard. We just don't support the DOM because we're not a browser. It shouldn't be more confusing than using any other non-browser javascript runtime. Like node.js.

Ok, even URL and URLSearchParams are available in Node, right? I think the reasoning makes sense!

Yes, latest node versions. I think the most important is that whatever the types say is there is supported.


PyQt5 Windows closes with notification close

I have a simple Qt window it has a button and when you click the button it opens a explorer window for a specific location, when the notification fades out the window closes as well.
Entire Example Script
`class Example(QWidget):
def __init__(self):
    super().__init__()
    self.initUI()

@pyqtSlot()
def on_click(self):
    print('PyQt5 button click')
    subprocess.Popen(r'explorer /open,"C:\WB Resources\"')
    toaster = ToastNotifier()
    toaster.show_toast("TITLE",
                       "Python is 10 seconds awsm! and I think that this is the way to go for all information going forward",
                       icon_path="wbicon.ico",
                       duration=3)
    print('Finished?')

def on_click_cache(self):
    path = "C:/Users/XXXXX/AppData/Local/Packages/XXXXXXXXXXXX_XXXXXXXXXXXX/LocalState/cacheddata/"
    try:
        shutil.rmtree(path)
    except:
        print('No folder')

    self.btn_clearcache.setStyleSheet("background-color: green")

def initUI(self):

    QToolTip.setFont(QFont('SansSerif', 10))

    btn = QPushButton('Open WBR',self)
    btn.setToolTip('This is a QPushButton widget')
    btn.resize(150,50)
    btn.move(0, 0)
    btn.clicked.connect(self.on_click)

    self.btn_clearcache = QPushButton('Clear Cache', self)
    self.btn_clearcache.setToolTip('Clear the WB App Cache
THIS IS DESTRUCTIVE')
    self.btn_clearcache.resize(150,50)
    self.btn_clearcache.move(151, 0)
    self.btn_clearcache.setStyleSheet("QPushButton:focus:pressed { background-color: red }")
    self.btn_clearcache.clicked.connect(self.on_click_cache)

    self.setGeometry(500, 350, 900, 500)
    self.setWindowTitle('WB-TOOL')
    self.setWindowIcon(QIcon('arrow.png'))
    self.show()

if name == 'main':
app = QApplication(sys.argv)
ex = Example()
sys.exit(app.exec_())`

Hi @studiozeroseven  thank you for raising the issue.
I will look look into this and revert back within a day.

@studiozeroseven Apologies for the delay I was caught up in work.
Please confirm if you still facing the issue and need help.
I'm not able to recreate this in my Python 3.5 environment. If you need help, could you please provide the PyQt version you are using and also the imports.

Hi, I'm getting the same thing. PyQt5 is latest version from pip, on python3.
Imports look as follows:
from PyQt5 import QtGui, QtCore, QtWidgets
import sys, os, sys, shelve, time, threading, random, operator, copy, datetime, os, collections
from operator import attrgetter, itemgetter
from random import randint
from win10toast import ToastNotifier
from playsound import playsound

Hi guys,
I've sourced the issue in line 136 init.py in the library. Disabling PostQuitMessage(0) seems to fix the issue. I'm not sure of how this will affect the usage. If you have any experience in this please give your suggestion. I'll be investigating this further before pushing a change.

Heyo. Commenting out that line seemed to fix the crashing. The sleep(duration) a few lines before that hangs the program for a few seconds so I commented that out also.

Just want to add my 2 cents. I'm using PySide2 (which is basically PyQt5, so what I say should to PyQt5), and I was getting this problem as well with it crashing the app. Using multithreaded option works perfectly.
Please verify this. The issue should be fixed already.
Use threaded = True to fix the issue


Support querying indexes with _source disabled

Feature description
Looks like the current implementation of elasticsearch-spark library does not support querying indexes with _source disabled.
In particular take a look #https://github.com/elastic/elasticsearch/issues/20408 and SearchRequestBuilder.java.
If _source is disabled in the index mapping and some fields are still specified in the query, the "fields" parameter should be used instead.

There's sufficient reason to move forward on this. Using the stored_fields/fields parameters are more performance friendly for reads, but those parameters require wildcards for any fields that are not leaf level document fields. Because of this, ES-Hadoop would need to be mapping-aware when specifying the field names and wildcards in the stored_fields/fields parameters, as well as version-aware to select the correct param name.

@jbaiera  In our company we are running into this issue as well. A lot of our data live in Elasticsearch 1.7.2 cluster with _source disabled for most of the mappings. We would like to be able to query these data through Hive or run Spark jobs on top of them.
We would be interested in helping with this enhancement. Would you be willing to help me understand this part of es-hadoop, describe the proposed changes and help me evaluate the difficulty? Thanks in advance.


plbart-large window size

I believe there is something wrong with the window size (n_positions) of plbart-large. It prints the following:
>>> from transformers import PLBartTokenizer
>>> tokenizer = PLBartTokenizer.from_pretrained("uclanlp/plbart-large")
>>> tokenizer.model_max_length
1000000000000000019884624838656

I will be statically setting this to 1024.

Can you raise the issue in Huggingface transformers? I do not have knowledge about the model_max_length field.

reported issue in HuggingFace.


DNS issues after reboot

Hello . This script works for me and I have no problem.
After restart, the openvpn service runs and has no problems.
I can connect to open-vpn with the client , but it does not work and does not open any website  and does not show any errors.
OS : Ubuntu 18.4
Kernel : 4.15.18-1
what's wrong ?

I noticed that there is a problem with DNS.
I can get a ping from the IP address of different sites, but I do not have the ability to ping the domain.
what is the problem?

This problem is solved by solving the following:
edit this file :
/etc/openvpn/server.conf
Add these commands at the end of this file :
up /etc/openvpn/update-resolv-conf
down /etc/openvpn/update-resolv-conf

Run the following command:
service openvpn restart

That's strange. It usually worked fine by default. What you mentioned is used to force the VPN DNS server to prevent leaks.
You may want to look into this @Angristan.

Is /etc/openvpn/update-resolv-conf an actual file?
What DNS server were you using when you had the issue? What OS and client?

Is /etc/openvpn/update-resolv-conf an actual file?

@Angristan yes, it's a script that changes the DNS servers in resolv.conf to the ones of the VPN server and reverts them once disconnected. As explained in #149 it comes with the openvpn package.
BTW, I just noticed that @dfmabbas changed the up and down scripts on the server, but I know they should go on the client :thinking:.

What DNS server were you using when you had the issue? What OS and client?

The server he's using is Ubuntu 18.04.
Perhaps the client log without any changes is more useful?

I have the same issue after reboot

@angristan  I tried https://github.com/Nyr/openvpn-install and it works after reboot

@ayrat555 Did you try to use the system resolver (option #1 for DNS in the script)?

Yeah, I had the same issue. reinstalled 3 times and each time, after reboot it stops working. I can connect to the server but no site would open.

@iProdigy  yes, I'm using the system resolver ( I also just tried it on two other vps server with the same result- no internet after reboot).

@gonebeta Can you try pinging <IP_ADDRESS> with the VPN enabled to ensure it's not DNS?
If that's unable to connect, can you post the output of sudo ufw status?

Thanks. It says ping statistics for <IP_ADDRESS>:
packets: sent 4, received 0 (100% loss).
Even while that is happening, I'm able to login to the server and some other vps servers.  discord and skype were also working.
For good measure, I turn off my antivirus firewall ... didn't help.
I don't use ufw on the servers ( all firewall are configured in runcloud and I opened the udp port the vpn uses).
I installed Nyr version and it works fine even after reboots.

If someones still encounters the issue, please tell us what are the DNS servers used by the client (dig/nslookup)

Nice fix

Until I have useful information, yes it is.

What DNS servers are pushed by the server
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
What DNS the client is trying to use
SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)


Audit fails: descriptions are too long with full tap name

Refs: Homebrew/homebrew#47033 Homebrew/homebrew#47296

Looks like this is fixed with Homebrew/brew#70 ? I ran audit on all our nodenv formulas without error.


Release version0.33.0 of sqlparser-rs

Filing a ticket here to track the work of a release (mostly so others can follow along)
Target Release Date: Approximately 2023-04-01
Release instructions: https://github.com/sqlparser-rs/sqlparser-rs/blob/main/docs/releasing.md
Steps:

[ ] Make a PR to update changelog
[ ] run cargo publish

I am starting to work on this now

Next release tracked in https://github.com/sqlparser-rs/sqlparser-rs/issues/847

Release is on crates.io https://crates.io/crates/sqlparser/0.33.0


Problem : Raylib with Dietpi (* failed to add service – already in use?)

I have compile raylib on dietpi in native mode on my raspberry pi 3b+ by following this wiki (https://github.com/raysan5/raylib/wiki/Working-on-Raspberry-Pi). There were just few warnings while running the "make PLATFORM=PLATFORM_RPI" command in "raylib/src/" folder. Then I compile the examples by the same command (make PLATFORM=PLATFORM_RPI) in the "raylib/examples/" folder. Up to here everything was ok but when I try to run some examples, the output is always ->
"INFO: Initializing raylib 3.1-dev

failed to add service – already in use?"

I don't find any way to run an exemple without this error.
Does anybody have an idea on this?
P.S. Sorry for my bad english >.<

It is ok I have now find a solution. I had just to disable vc4-kms-v3.

@Ggcpp Oh! You find it fast! :D
Just note that you can also use raylib on Linux desktop with PLATFORM_DESKTOP.

@raysan5 Thanks for your reply and thanks for all your work on raylib, I really enjoy it ! ^o^


[cost] Allowed to set internal reference values in CoP cost

We have already discussed how to allow reading/writing internal values in cost reference functions.
However, I wanted to share with you know the specific details of this solution.
To make things easier, I have pushed only the modifications for the CoP cost.
As you can see in this PR, it is possible to get/set references without down-casting the functions in c++.
As I mentioned in our discussion, this is the main reason to this design pattern:
  using namespace crocoddyl;
  FrameCoPSupport ref(id, Eigen::Vector2d(0.1, 0.1));
  boost::shared_ptr<CostModelAbstract> = boost::make_shared<CostModelCoPPosition>(state, activation, ref);
  // Get references
  FrameCoPSupport ref_full = cost->get_reference<FrameCopSupport>();
  Eigen::Vector2d ref_box = cost->get_reference<Eigen::Vector2d>();
  FrameIndex ref_id = cost->get_reference<FrameIndex>();
  // Set references
  cost->set_reference<FrameCoPSupport>(ref_full);
  cost->set_reference<Eigen::Vector2d>(ref_box);
  cost->set_reference<FrameIndex>(ref_id);

And for the Python case, it is also possible to define the references as
  ref = FrameCoPSupport(id, np.array([0.1, 0.1])
  cost = CostModelCoPPosition(state, activation, ref)
  # Get references
  ref_full = cost.reference
  ref_box = cost.reference_box
  ref_id = cost.reference_id
  # Set references
  cost.reference = ref_full
  cost.reference_box = ref_box
  cost.reference_id = ref_id

where it is necessary to define a specific name for each reference function.
This is the only difference with respect to the agreed proposition.
Unfortunately, I can't use the same name for each case; it is a boost python limitation.
Before merging this PR, I would need to replicate this scheme for each cost function as described below.
If you have any further feedback, then this is the right moment to do it.

[x] Added set and get functions for internal references in each cost function
[x] Created the Python bindings for internal references in each cost function

I have pushed the extra bits for each cost class.

There is also an extra task. I removed the multibody/costs/control-cost.hpp header file.
It was included by @nim65s with the aim to deprecate it, but we never have it.
Note that we are already deprecating the right header file, i.e. multibody/costs/control.hpp.

Based on internal discussions, we've decided to not proceed with this design template. So I'm closing this PR.


Fix build on case sensitive systems

In browserify context, when requiring angular-schema-form in a case sensitive system (my case ubuntu):
Browserify Error: Cannot find module 'ObjectPath' from '/home/gitlab-runner/tmp/builds/project-51/backend/node_modules/angular-schema-form/dist'

Obviously all dependencies name are always in lower case. (bower and npm)

It would be really good to get this fixed. I has caused me a lot of headaches, as it was working on my machine, but failing on our build server.  Both machines were running the same ubuntu image, the only difference was my local machine had a drive synced with my host OS - Mac OS.

@LoicMahieu did this PR fix your build issue?  I'm still getting problems using ASF with browserify on ubuntu.

Hi @LoicMahieu and thanks for the PR!  I don't use browserify, how can I set up a test case that tests this? Why did you change the usage of ObjectPath name in sfPath.js, as you see from the tests it breaks, I guess it works with browserify?

Hi, sorry to hijack this thread, but the steps to reproduce are: -
> npm init
[accept all the defaults]
> npm install --save angular-schema-form
> echo "require ('angular-schema-form')" > index.js
> npm install browserify
> ./node_modules/browserify/bin/cmd.js -e index.js -o output.js

Error: Cannot find module 'ObjectPath' from '/home/timh/test/node_modules/angular-schema-form/dist'

I think the solution is to modify the minify.js gulp task, line 19 to: -
{name: 'ObjectPath', cjs: 'objectpath'},

@thardyman no problem! Thanks

@thardyman Thank you for the help! ok, so it seems you're right. Changing to {name: 'ObjectPath', cjs: 'objectpath'}, fixes the build problem, but how do I test that the build actually works? i.e. what is the minimal index.html page with a browserify build?

I pushed the fix to development, now at least.

Hmm, that's a good question! In theory browserify should have brought all the dependencies in to that output.js file we create in my steps to reproduce example above.  So all we need is to add that via a script tag to a html page and we should be away.
Then it is just a case of what is the minimum markup / code for an ASF.

Ok, I'll give it a try :)

Ok. I think I got it now. When building with browserify its objectpath thats available to us, but ObjectPath when not using browserify.

@davidlgj thanks for figuring this out and getting it into a release so quickly!


Transmit _WORKER_STARTED_INDICATOR when process pool worker is ready

Previously was relying on monitoring socket. Sometimes, a message was lost resulting in indefinite wait for all
workers to start. The new method is simpler and appears to be more robust.

Codecov Report

Merging #216 into master will increase coverage by 0.14%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master     #216      +/-   ##
==========================================
+ Coverage   85.92%   86.07%   +0.14%
==========================================
  Files          69       69
  Lines        3574     3611      +37
  Branches      528      541      +13
==========================================
+ Hits         3071     3108      +37
  Misses        408      408
  Partials       95       95

Impacted Files
Coverage Δ

petastorm/workers_pool/process_pool.py
94.15% <92.85%> (+0.54%)
:arrow_up:

petastorm/pytorch.py
96.72% <0%> (-3.28%)
:arrow_down:

petastorm/reader_impl/worker_loop.py
97.72% <0%> (+1.13%)
:arrow_up:

Continue to review full report at Codecov.

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


More extensible snapshot plugin system (custom match and diff)

Do you want to request a feature or report a bug?
Request a feature
What is the current behavior?
Currently we are only able to specify serializers for snapshot, i.e. expect.addSnapshotSerializer(serializer).
Comparison on actual/expected snapshot value is done on === equality comparison basis
https://github.com/facebook/jest/blob/master/packages/jest-snapshot/src/State.js#L123
What is the expected behavior?
Ability to specify custom compare/match function for other more advanced use cases

Could you exemplify both some use cases which are impossible/hard to do with the current API, and what you think can be done to solve them?

Ah.. I just chanced upon this very useful comment https://github.com/facebook/jest/blob/eb3ba0f31fea13fdc90bec0ba35deef6ce3bdf1e/packages/jest-snapshot/src/index.js#L101-L112
I suppose that should be good enough for me to work with! I could use the actual and expected and run my own matcher and return custom pass accordingly.
Great job, Jest team!

Great! Feel free to comment here if the API is lacking.
PR also most welcome for some docs on that :)

Sounds a bit like https://github.com/facebook/jest/issues/5847 as well 👌


Example for PostgreSQL integration with Kanister

Change Overview

In this PR:

Added Dockerfile to add required components for backup in bitnami/postgresql
Added blueprint to work with upstream helm chart
Updated README with steps to demonstrate postgres data protection with Kanister

Pull request type
Please check the type of change your PR introduces:

[ ] Work in Progress
[ ] Refactoring (no functional changes, no api changes)
[ ] Trival/Minor
[ ] Bugfix
[ ] Feature
[x] Documentation

Issues

#XXX

Test Plan

[ ] Manual
[ ] Unit test
[ ] E2E

Just a minor suggestion in README, otherwise the PR looks good to me @PrasadG193 👍  , will let @tdmanv approve it.


default intellisense engine not working

VSCode Version:
1.15.1
OS Version:
ubuntu 16.04 LTS
Steps to Reproduce:
1.install c/c++ extension
2.set C_Cpp.intelliSenseEngine as defult
3.restart vs code
4.find autocomplete doesn't work

Reproduces without extensions: Yes/No

Can you file this on the C++ extension repo, since it seems to be an issue with the extension?


Add 'recordings list' subcommand

Add capability to list and filter on recordings.

here's the ticket: https://linear.app/foxglove/issue/FG-3529/foxglove-cli-handle-wide-outputs


Update zio-sbt-ci, zio-sbt-ecosystem, ... to 0.4.0-alpha.8+6-d42e25c4-SNAPSHOT

About this PR
📦 Updates

dev.zio:zio-sbt-ci
dev.zio:zio-sbt-ecosystem
dev.zio:zio-sbt-website

from 0.4.0-alpha.6+40-c57e764c-SNAPSHOT to 0.4.0-alpha.8+6-d42e25c4-SNAPSHOT
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: sbt-plugin-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #215.


Step-by-step execution of cells

This is hotfix for issue #54.
Execution results of previous (or older) cells are displayed under the next cell as follows:
# let x = Unix.sleep 10 ; 10 ;;
# let y = 20 ;;
val x : int = 10
val y : int = 20

The above result should be
# let x = Unix.sleep 10 ; 10 ;;
val x : int = 10
# let y = 20 ;;
val y : int = 20

This problem is observed when we execute a cell before older cells are not finished.
The kernel needs to respond to an execution result message to the corresponding cell.
However the kernel replies all execution results to the last executed cell.
The new kernel waits execution of the previous cell and sends a result to the corresponding cell correctly.
@Naereen Could you check the behavior of the new version?

I'm not sure how to update, I reran the opam pin command but does see any change.

The change in code seemed alright.

@Naereen Please build and install hotfix/issue54 branch:
$ git clone -b hotfix/issue54 https://github.com/akabe/ocaml-jupyter
$ cd ocaml-jupyter
$ opam pin add -y jupyter .


[improve] [ws] PIP-290 Make WSS support E2E encryption

Motivation & Modifications
See https://github.com/apache/pulsar/pull/20923
Documentation

[ ] doc
[x] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: x

Codecov Report

Merging #20958 (1118da6) into master (63d9eaf) will increase coverage by 36.17%.
Report is 6 commits behind head on master.
The diff coverage is 67.56%.

@@              Coverage Diff              @@
##             master   #20958       +/-   ##
=============================================
+ Coverage     36.92%   73.10%   +36.17%
- Complexity    12187    32173    +19986
=============================================
  Files          1698     1877      +179
  Lines        129846   139567     +9721
  Branches      14163    15357     +1194
=============================================
+ Hits          47947   102029    +54082
+ Misses        75570    29456    -46114
- Partials       6329     8082     +1753

Flag
Coverage Δ

inttests
24.25% <6.08%> (+0.10%)
:arrow_up:

systests
25.17% <4.72%> (+0.01%)
:arrow_up:

unittests
72.37% <67.56%> (+40.29%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...a/org/apache/pulsar/websocket/ConsumerHandler.java
62.50% <ø> (+37.09%)
:arrow_up:

.../apache/pulsar/websocket/data/ConsumerMessage.java
100.00% <ø> (ø)

...he/pulsar/client/api/DummyCryptoKeyReaderImpl.java
33.33% <33.33%> (ø)

...a/org/apache/pulsar/websocket/ProducerHandler.java
60.30% <60.57%> (+29.27%)
:arrow_up:

...r/websocket/service/WSSDummyMessageCryptoImpl.java
66.66% <66.66%> (ø)

...g/apache/pulsar/client/impl/RawBatchConverter.java
93.50% <100.00%> (+19.82%)
:arrow_up:

...rg/apache/pulsar/compaction/TwoPhaseCompactor.java
74.88% <100.00%> (+4.70%)
:arrow_up:

...che/pulsar/client/impl/crypto/MessageCryptoBc.java
68.12% <100.00%> (+2.12%)
:arrow_up:

...va/org/apache/pulsar/client/impl/ConsumerImpl.java
76.91% <100.00%> (+21.04%)
:arrow_up:

...va/org/apache/pulsar/client/impl/ProducerImpl.java
82.86% <100.00%> (+19.88%)
:arrow_up:

... and 1 more

... and 1431 files with indirect coverage changes

It looks like the failed test is related to the change?

Is it possible to add an integration test for this in the future? :)

Is it possible to add an integration test for this in the future? :)

In the test, the Web Socket Proxy is a real server, so I think this is already an integration test?


⚠ Bofrost.at has degraded performance

In 45413e3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1544 ms

Resolved: Bofrost.at performance has improved in 5ebf627 after 27 minutes.


test: Identifier validation improvements

This patch adds support for the full unquoted identifier spec, including supporting the $ sign inside names and ensuring only letters and underscores can start an identifier
Test Plan

Added an unit test with 7 subtests
No changes to acceptance tests

References
The identifier spec is at https://docs.snowflake.com/en/sql-reference/identifiers-syntax.html

/ok-to-test sha=342df46

/ok-to-test sha=e842b81


Updated library for newer version of Android, fixed #5

Updated the library to work with newer versions of Android, updated Sample Activity to work with newer version of Android.
Fixed issue #5, positive and negative buttons are not customisable.
Updated README.md to reflect changes

Hi @ItsPriyesh any chance of looking at this and possibly merging it if you approve? Would love to be able to translate the buttons on the dialog.
Thank you for making this library, I have been using this in a couple of my apps for the past two years!