Improvement for k8s.io/docs/concepts/overview/

try to resolve https://github.com/kubernetes/website/issues/42644

It's obvious to me that this pull request does not include the suggested change from #42644.
Would you be willing to reivise this pull request to cover what I suggested was needed, @pegasas?

对我来说很明显，这个拉取请求不包括#42644中建议的更改。
您是否愿意重新修改此拉取请求以涵盖我建议的需求，@pegasas？

Sure. @sftim .
I will close this PR and re create an new one.
I have a few that needs to confirm with you to make sure we are on the same page.

How I recommend fixing this: add new content at the start of https://k8s.io/docs/concepts/overview/ (before “Going back in time”)
The new content should aim to answer the questions that @theking2 didn't see answers to.

I have 2 options:

Add a new paragraph to explains deeper about kubernetes.
Modify current paragraph to highlights some key points in kubernetes design

Which one do you prefer?

/priority backlog /triage accepted
If working on this, consider setting a custom linkTitle for that page as well.

What is custom linkTitle stands for?

Refer this paragraph from other paragraph in this page.
Refer some paragraph in this page to explains this question.
Others

Would you minds to give more hints about it?

I don't think this is adding much value to the existing text.
Feel free to reopen if there are different opinions.
/close


Please make better documentation

File: docs/reference/pkg/python/pulumi/_index.md
The documentation is lacking a navigation panel showing the various sections and the reference material is overcrowded, especially with the unused scrollbars wrapping code blocks

Fixed in https://github.com/pulumi/pulumi-hugo/pull/352 and supposedly merged in https://github.com/pulumi/docs/pull/6157 but currently don't see this being the case on the site.

Fixed with https://github.com/pulumi/pulumi-hugo/pull/363


small patches to Rg metainfo

@ladinesa : There are just minimal changes here. Please let me know if I can merge.

...I fixed the pycodestyle error in the pipeline now.


[WIP] Add test to verify post-install jobs succeeded

As per title

@pierDipi we probably want to change the target branch for this PR to 1.15 now that #4018 is merged

@pierDipi could you rebase this?


Extension to analyze 5D-Ricci-Flatness (Rij = 0)

An extension to view 5D-Ricci-Flatness (Rik = 0) is needed to view the negative energy of the scalar field (Kaluza-Theory),
which cancels the electrostatic field energy of a charge with mass. As the gravitational field of a mass has no field energy,
the over all energy distrubution in 4D and 5D has to be a zero-field.
(electric) ... monopole ... and the role of negative scalar field energy in vacuum solutions (2.3 eq. 16)
Next steps would be: analyzing the energies of a moving charge (has field energy E x B), comparison of 4D and 5D Reissner-Nordström solutuions.
If granted, i can write the code for it (after some time).

Integrating Kaluza theory into Warp Factory would probably take moderate rearchitecting of at least all of the solver functions.
Feel free to fork the repo though! If you let me know, I can add it as a highlighted fork in the readme.


support include marco in yaml parsing

While libfyaml itself doesn't directly support including other YAML files. We need to create a mechanism for including files.
Here's an approach that may work.
1. Manual Inclusion and Parsing:

Read the contents of the main YAML file.
Identify references to other YAML files (e.g., using a custom tag like !include).
Read the contents of the referenced files.
Parse the combined YAML data using libfyaml.

2. Preprocessing:

Use a separate preprocessing step to combine multiple YAML files into a single file before feeding it to libfyaml.
Requirement is to include either local relative files, or url downloads via curl.

3. Custom Tag Support:

Extend libfyaml to support a custom tag (e.g., !include) for including other files.
This requires implementing a custom tag handler within libfyaml that reads and parses the included file.

Example (Manual Inclusion):
#include <yaml.h>
#include <stdio.h>
#include <string.h>

// ... (error handling and other code)

FILE *fh = fopen("main.yaml", "r");
yaml_parser_t parser;
yaml_event_t event;

// Parse main file, looking for !include tags

while (yaml_parser_parse(&parser, &event)) {
    if (event.type == YAML_SCALAR_EVENT &&
        !strcmp((char *)event.data.scalar.value, "!include")) {

        // Get filename from the next event
        yaml_event_delete(&event);
        if (!yaml_parser_parse(&parser, &event)) { /* handle error */ }
        char *filename = (char *)event.data.scalar.value;

        // Open and parse included file
        FILE *included_fh = fopen(filename, "r");
        yaml_parser_t included_parser;
        yaml_parser_initialize(&included_parser);
        yaml_parser_set_input_file(&included_parser, included_fh);

        // Process events from the included file
        while (yaml_parser_parse(&included_parser, &event)) {
            // ... process events
        }

        yaml_parser_delete(&included_parser);
        fclose(included_fh);
    }
    // ... process other events
}

// ...

!include <URL>
!include <file:relativeToThisFile>

functionality to support reading from either a local file or a remote one, using the curl library ->
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <curl/curl.h>

// Function to get the size of a remote file
long get_remote_file_size(const char *url) {
    CURL *curl;
    CURLcode res;
    double filesize = 0.0;

    curl = curl_easy_init();
    if(curl) {
        curl_easy_setopt(curl, CURLOPT_URL, url);
        curl_easy_setopt(curl, CURLOPT_NOBODY, 1L);
        curl_easy_setopt(curl, CURLOPT_HEADER, 1L);
        curl_easy_setopt(curl, CURLOPT_FILETIME, 1L);

        res = curl_easy_perform(curl);
        if(res == CURLE_OK) {
            res = curl_easy_getinfo(curl, CURLINFO_CONTENT_LENGTH_DOWNLOAD, &filesize);
            if((res == CURLE_OK) && (filesize > 0.0)) {
                curl_easy_cleanup(curl);
                return (long)filesize;
            }
        }
        curl_easy_cleanup(curl);
    }
    return -1;
}

// Function to get the size of a local file
long get_local_file_size(const char *filename) {
    FILE *file = fopen(filename, "rb");
    if(file == NULL) {
        perror("Error opening file");
        return -1;
    }
    fseek(file, 0, SEEK_END);
    long filesize = ftell(file);
    fclose(file);
    return filesize;
}

// Callback function to write data into a buffer
size_t write_data(void *ptr, size_t size, size_t nmemb, void *data) {
    size_t total_size = size * nmemb;
    char **buffer = (char **)data;
    *buffer = realloc(*buffer, strlen(*buffer) + total_size + 1);
    strncat(*buffer, ptr, total_size);
    return total_size;
}

// Function to download a file using libcurl and store its content in a buffer
int download_file(const char *url, char **buffer) {
    CURL *curl;
    CURLcode res;

    *buffer = malloc(1);
    **buffer = '\0';

    curl = curl_easy_init();
    if(curl) {
        curl_easy_setopt(curl, CURLOPT_URL, url);
        curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, write_data);
        curl_easy_setopt(curl, CURLOPT_WRITEDATA, buffer);
        res = curl_easy_perform(curl);
        curl_easy_cleanup(curl);

        if(res != CURLE_OK) {
            fprintf(stderr, "curl_easy_perform() failed: %s\n", curl_easy_strerror(res));
            free(*buffer);
            return -1;
        }
        return 0;
    }
    free(*buffer);
    return -1;
}

// Function to read content from a local file into a buffer
int read_local_file(const char *filename, char **buffer) {
    long filesize = get_local_file_size(filename);
    if (filesize == -1) {
        return -1;
    }

    FILE *file = fopen(filename, "rb");
    if (file == NULL) {
        perror("Error opening file");
        return -1;
    }

    *buffer = malloc(filesize + 1);
    if (*buffer == NULL) {
        perror("Error allocating memory");
        fclose(file);
        return -1;
    }

    fread(*buffer, 1, filesize, file);
    (*buffer)[filesize] = '\0';  // Null-terminate the buffer

    fclose(file);
    return 0;
}

// Unified function to read from either a URL or a local file
int ut_read(const char *path, char **buffer) {
    // Check if the path is a URL (simple check for http:// or https://)
    if (strstr(path, "http://") || strstr(path, "https://")) {
        long filesize = get_remote_file_size(path);
        if (filesize == -1) {
            fprintf(stderr, "Failed to get remote file size.\n");
            return -1;
        }
        return download_file(path, buffer);
    } else {
        return read_local_file(path, buffer);
    }
}

int main(void) {
    const char *input = "https://example.com/file.txt"; // or "local_file.txt"
    char *buffer = NULL;

    if (ut_read(input, &buffer) == 0) {
        printf("File content:\n%s\n", buffer);
        free(buffer);  // Don't forget to free the allocated memory
    } else {
        printf("Failed to read file.\n");
    }

    return 0;
}


Do we need a contribution from the example code about pub/sub here?

I find that there is only an example code in the readme.md. wondering that if there is a contribution chance for me to add a pub/sub example to go-sdk.

Absolutely. There are simple examples in /example/client/main.go but we could always use more. Thank you!

I'm sorry for the previous mistakes, I revised them and tested them.
solved main redeclared by moving Pub. go and Sub. go to in separate folders

I use the pubsub example with natsstreaming component to test the runtime now,https://github.com/dapr/dapr/issues/1996
use the nats-streaming branch. 👍 https://github.com/abserari/go-sdk/tree/test-the-natsstreaming


Does not work

I followed the instructions on the extension page but neither ctrl+i or adding custom key binding ctrl+shift+i get it to work. I am running the following machine:

Ubuntu 18.04
VS Code 1.24.1
html tag wrapper 0.1.4

Found it works on multiple lines, just not single lines


Support Node 16

Add support for node 16 by deleting PerformanceTimer class, and make node 16 the default version for new projects.
GUS: W-12399737
Description
Over on MRT, we're working on enabling platform-level support for node 16 apps. It currently doesn't work if the app depends on pwa-kit-runtime because of an issue with PerformanceTimer, our custom wrapper around the builtin node perf_hooks API. It turns out that we don't use PerformanceTimer for anything useful (it gathers metrics that we emit to CloudWatch, but Lambda automatically emits the same metrics to CloudWatch). Since the class is marked as @private, this deletes it (which is not a breaking change) and adds explicit support for node 16.
Types of Changes

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

Remove PerformanceTimer and all its usages
Explicitly support node 16 in all package.jsons, and make 16.x the default node version for new projects
Update CircleCI workflow to include node 16 and npm 7 and 8
Update create-mobify-app-dev.js to account for breaking changes in npm 7
Fix (or make more resilient) a handful of retail-react-app tests, which were mostly failing due to race conditions that were uncovered with the switch to node 16, because the test execution environment became faster
Re-introduce changes from #865

It was released in an alpha release (pwa-kit-dev@2.4.1-alpha.0), but somehow didn't make it into the next normal release, so running npm run tail-logs in a newly-generated project returns an error.

How to Test-Drive This PR

Run npm ci at the root of this repo
Repeat the following for node 14.17.0 (npm 6), node 16.7.0 (npm 7), node 16.11.0 (npm 8), and node 16.19.1 (LTS, npm 8):

Run nvm use <node-version>, then generate a new project:

GENERATOR_PRESET=test-project node packages/pwa-kit-create-app/scripts/create-mobify-app-dev.js --outputDir
~/generated-test-project

cd to the project, then run npm start. The app should load in the browser without errors.
Make sure your Runtime Admin API key is set in ~/.mobify, then deploy your bundle to MRT: npm run push -- -s <project-slug> -t <target-slug>

In Runtime Admin, make sure the deploy succeeds, that your site works, and that your bundle is running node 16.x
For reference, I deployed my generated project bundle to https://zombie-apocalypse.mobify-storefront-staging.com

Run npm run tail-logs -- -p scaffold-pwa -e production and ensure logs appear after a few seconds.

Checklists

General

[x] Changes are covered by test cases
[x] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[x] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

@bfeister @adamraya I noticed the ticket's AC says PWA Kit default version should be set to 16. Do you know if I need to configure that anywhere?
Also, not sure if this is an issue, but I wanted to call out this warning that I see when running npm ci on a newly-generated project on this branch:
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '^14.0.0', npm: '^6.14.4 || ^7.0.0 || ^8.0.0' },
npm WARN EBADENGINE   current: { node: 'v16.17.0', npm: '8.15.0' }
npm WARN EBADENGINE }

I'm assuming that refers to template-retail-react-app, but I added v16 to that template's supported engines in package.json 🤔

@bfeister @adamraya I noticed the ticket's AC says PWA Kit default version should be set to 16. Do you know where I can configure that?

You can default generated projects to Node 16 by updating the ssrFunctionNodeVersion to 16.x on the package.json files:

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/4edfad359299e71e2992e351de5ce68bf76f4ed7/packages/pwa-kit-create-app/assets/pwa/default.js#L68

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/4edfad359299e71e2992e351de5ce68bf76f4ed7/packages/template-retail-react-app/config/default.js#L51

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/1279de555759c3812f3f7deb44964fc557fa074c/packages/template-express-minimal/package.json#L25

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/4b463a5908a6038fe857d4268b7ddff72b1b3c3f/packages/test-commerce-sdk-react/package.json#L52

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/4b463a5908a6038fe857d4268b7ddff72b1b3c3f/packages/template-typescript-minimal/package.json#L51

Other places:

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/39dddf9daa82bcd907516a73620cb69928489bf1/packages/pwa-kit-react-sdk/setup-jest.js#L34
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/b25895df603619fc7231a738f57d941b05d1abd9/packages/template-retail-react-app/config/mocks/default.js#L110

You can default generated projects to Node 16 by updating the ssrFunctionNodeVersion to 16.x on the package.json files

Thanks @adamraya - I gave that a shot but my fresh generated project still has ssrFunctionNodeVersion: '14.x' set in its default.js files, and its engines key only contains ^14.0.0. I'll keep looking around 👀
(and look into the failing build)

@raiyaj I saw a problem with the change we did to theIS_DEFAULT_NPM env variable.
Now we'll have TWO builds that will run steps that only need to happen once like: Push Bundle to MRT or Publish to NPM.
If we follow the current approach, we need a new env variable to designate the unique/main build and update the conditional to include that env variable and guarantee we run the steps only once.
pwa-kit job

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L69
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L76
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L83
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L90
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L101

generated job

https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L216
https://github.com/SalesforceCommerceCloud/pwa-kit/blob/38dbed1ede78553e9a1620a4c1233945e1c3dd14/.github/workflows/test.yml#L223


renamed folder for a valid job name

when creating job from a recipe .. I was getting error :'Invalid deployment/job name- since name shouldn't be more than 24 characters , hence changed the name to a shorter one

I noticed several other issues not included in this PR (and this PR renames some folders within the repo which isn't necessary), so I am going to close it and do another


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit e596f2a:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch 24ef958d-95af-40d1-a1f4-44cb0d8a590a has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add Kefix support

Support Kefix via require("ffux/kefir")

:+1: Would use.

:+1 What would be the steps to support Kefix?


Update assistants-api-in-a-box (new PR in response to PR #46)

This pull request includes changes primarily focused on restructuring the scenarios/Assistants directory and updating the file paths in the code. The changes also include the addition of README files for each assistant, and some minor fixes in the code and file names.
Directory and file restructuring:

scenarios/Assistants/assistants-api-in-a-box/README.md was renamed to scenarios/Assistants/api-in-a-box/README.md. The file paths in the markdown links and the location of the .env file were updated accordingly. [1] [2] [3] [4]
The notebooks in scenarios/Assistants/assistants-api-in-a-box/notebooks/ were moved to their respective directories under scenarios/Assistants/api-in-a-box/, and the file paths in the code were updated. [1] [2] [3] [4] [5] [6] [7] [8]
The .gitignore file in scenarios/Assistants/assistants-api-in-a-box/notebooks/ was removed.

Addition of README files:

README files were added for each assistant under their respective directories, providing an overview and objective of each assistant. [1] [2] [3] [4]

Minor fixes:

A typo in the title of the Math Tutor Assistant was fixed in scenarios/Assistants/api-in-a-box/math_tutor/assistant-math_tutor.ipynb.# Description

Checklist

[x] I have read the contribution guidelines
[ ] I have coordinated with the docs team<EMAIL_ADDRESS>if this PR deletes files or changes any file names or file extensions.
[ ] This notebook or file is added to the CODEOWNERS file, pointing to the author or the author's team.

To address the CI failure, please consult the Contributing Guidelines to install the dev requirements and pre-commit. Then run pre-commit run --all-files from the root of the repository.
This'll:

Format the notebooks with black


Model - TaxonName - handling plant name authorship rendering

We have discussed and partially implemented a solution for handling the "dual" authorship common in plant name use.  In general the core needed to capture two extra bits of information, 1) an authorship to render, and 2) and assertion of which authorship to render.  No new semantics are required to handle basionyms, they are identical to original combinations. There really is no need for new Protonyms in the model.  This is a citation/authorship/rendering issue.
The currently implemented experiment adds the two bits of information as follows:
a) Add a duplicate protonym of the identical spelling , this facilitates capturing 1) above, another authorship
b) Add a new plant specific basionym relationship linking the protontym to the identically spelled protonym
c) Add rendering logic to draw the authorship.
Since basionym and original combination are the same concept, it is not desirable to add the identically spelled protonym twice.
We have two possible routes forward:
The first:
a) Modify the logic behind the original combination capture to be done as a full, capital-C Combination.  A side benefit of this is that it allows us to add identifier proper to original combinations, solving a work-around for CoL exports.
b) Include a new relationship that allows Combinations to be included.  This lets us link an original combination to a Protonym.
c) Authorship (1) comes from the Citation on the relationship in b).
2 significant problems come with this approach:

We have no way to provide the equivalent to verbatim_author in a meaningful way.  The semantics of allowing this on Combinations as an attribute doesn't make sense since we draw it from a Protonym, and Citations require a full Source.  However, adding the new bit to the Combination, and adding validation to allow it only for Plant names does meet requirement 1) above.
The semantics of TaxonNameRelationships are currently very tight and well defined, they relate monomials to monomials.  Altering this to accomodate Combinations-Protonym (monomial) potentially forks our logic in many places, and dirties the model.  This may be the bigger concern.  However, implementing this in the user interface would be straightforward, and require minimal logic.

The second we discussed
a) Us a TaxonNameClassiciation as an anchor for a Citation, that let's us get authorship.
This doesn't solve the verbatim problem, but see c) below.
A third brainstorm
a) Create a new relationship in which a protonym can be linked to itself.
b) Cite that relationship, and draw authorship from the Ciation.
This may be the cleanest, however it does not solve 1) above, a slot for verbatim_basionym_author.  Perhaps this can be combined with:
c) Add a new field to TaxonName, verbatim_basionym_author
This is quite clean, but would require some interface shenaningans to make the overall process much simpler, it shouldn't be too hard to do.

After discussion with Dima we will encode/go with this approach:
== Current implementation proposal

Add 'TaxonNameClassification::CurrentAuthor'  (completely hidden in UI)
Add TaxonName#verbatim_current_author
Add Role::TaxonNameCurrentAuthor
Add Identifier::Local::IpniID (maybe global if they have a URL defined)
Update helper method to draw from new values
Update cached render to draw from new values

In interface

Expand authorship section when plant name to expose verbatim_current_author and Citation option
Citation option, when populated, creates the TaxonNameClassification

Considerations

Issues- magic behind the scene, when parent changes, we (may) need to update related records, to do this:
** Work from new combination perspective!
*** When you click change parent -> auto updates Citations/authorships/etc. on move)

This doesn't preclude us moving to a generalized Basionym/OriginalCombination model in which we move data to Combination.  It does give use the necessary semantics to encode what we need to encode to represent the Legume TSV, and a generalized simple model.
There remain multiple downstream rendering considerations for unparsed data, but those are not blocking of these basic steps.


modsec_audit.log is not populated

Modsecurity audit log is written only to stdout, even when MODSEC_AUDIT_LOG is set.
The problem seems to be a permission issue which nginx user can't write to the native "/var/log/" directory.
Workaround - set MODSEC_AUDIT_LOG to "/tmp/modsec_audit.log"
Logs are written only to stdout:
docker run -dti -p 80:80 --rm -e MODSEC_AUDIT_ENGINE=on -e MODSEC_AUDIT_LOG=/var/log/modsec_audit.log -d owasp/modsecurity-crs:3.3.5-nginx-202402070602
Workaround:
docker run -dti -p 80:80 --rm -e MODSEC_AUDIT_ENGINE=on -e MODSEC_AUDIT_LOG=/tmp/modsec_audit.log -d owasp/modsecurity-crs:3.3.5-nginx-202402070602

Thanks @udi-aharon. That issue is probably due to us now using the unprivileged nginx image.

I checked. What you're seeing is the expected behaviour for the unprivileged image. That being said, IMO you should not log to the container filesystem anyway but to a mount point. That should also take care of the permissions:
touch /tmp/host-fs-auditlog.log
docker run -dti -p 80:80 --rm -e MODSEC_AUDIT_ENGINE=on -e MODSEC_AUDIT_LOG=/var/log/modsec_audit.log -d -v /tmp/host-fs-auditlog.log:/var/log/modsec_audit.log owasp/modsecurity-crs:3.3.5-nginx-202402070602

Shall we add this to the documentation?

Not a bad idea.

I checked. What you're seeing is the expected behaviour for the unprivileged image. That being said, IMO you should not log to the container filesystem anyway but to a mount point. That should also take care of the permissions:
touch /tmp/host-fs-auditlog.log
docker run -dti -p 80:80 --rm -e MODSEC_AUDIT_ENGINE=on -e MODSEC_AUDIT_LOG=/var/log/modsec_audit.log -d -v /tmp/host-fs-auditlog.log:/var/log/modsec_audit.log owasp/modsecurity-crs:3.3.5-nginx-202402070602

good day this does not work for me, the modsec_audit.log within the container is a directory instead of a file that writes the audit logs. Please advice

I checked. What you're seeing is the expected behaviour for the unprivileged image. That being said, IMO you should not log to the container filesystem anyway but to a mount point. That should also take care of the permissions:
touch /tmp/host-fs-auditlog.log
docker run -dti -p 80:80 --rm -e MODSEC_AUDIT_ENGINE=on -e MODSEC_AUDIT_LOG=/var/log/modsec_audit.log -d -v /tmp/host-fs-auditlog.log:/var/log/modsec_audit.log owasp/modsecurity-crs:3.3.5-nginx-202402070602

This works. thank you!

PR to document volume mounts for logs: https://github.com/coreruleset/modsecurity-crs-docker/pull/225


fix: parser take enums as keys for ngx-translate #61

Part of #61

:tada: This PR is included in version 1.7.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CIP-0030 | Adjust preamble and structure w.r.t CIP-0001

No PR needs to be created for this as per #389 since it's already fully compliant with the new CIP-0001. 🤓


Could not find com.github.Raizlabs.DBFlow:dbflow-core:4.0.0-beta7

when i try to install this library on my android project i get this error:
Error:Could not find com.github.Raizlabs.DBFlow:dbflow-core:4.0.0-beta7.
Required by:
    Behandam:app:unspecified
Search in build.gradle files

my gradle:
def dbflow_version = "4.0.0-beta7"
dependencies {
    compile fileTree(include: ['*.jar'], dir: 'libs')
    ...
    annotationProcessor "com.github.Raizlabs.DBFlow:dbflow-processor:${dbflow_version}"
    compile "com.github.Raizlabs.DBFlow:dbflow-core:${dbflow_version}"
    compile "com.github.Raizlabs.DBFlow:dbflow:${dbflow_version}"
    ...
}

did you forget:

allProjects {
  repositories {
    // required to find the project's artifacts
    maven { url "https://www.jitpack.io" }
  }
}

in your top-level build.gradle.


Limited support for Symbol

I started seeing our CI builds fail after we upgraded from version 0.9.21
        ReferenceError: Can't find variable: Symbol

It looks like the more recent releases are using the Symbol primitive. At first I thought we could upgrade our headless browser to the latest version and then I wondered why bable wasn't polyfilling Symbol? The babel documentation links to a caveats section on core-js https://github.com/zloirock/core-js#caveats-when-using-symbol-polyfill concerning the limited use of the Symbol polyfill.

We can't add new primitive type, Symbol returns object.
Symbol.for and Symbol.keyFor can't be shimmed cross-realm.
By default, to hide the keys, Symbol polyfill defines setter in Object.prototype. For this reason, uncontrolled creation of symbols can cause memory leak and the in operator is not working correctly with Symbol polyfill: Symbol() in {} // => true.

I also looked and it seems that even the latest versions of Internet Explore don't have Symbol support.

We're required to maintain browser support for Internet Explore. Is it possible to rewrite without using the Symbol primitive? I'm reading back through the commit history. Looks like symbol support was was added in version 0.9.23.

@luk3thomas , per #15, use of Symbol has been removed from the LightStep code paths. I'll leave this issue open under it's confirmed this addresses the issue for you!

Hmmm, I'm still seeing a failure in the build. From what I can tell it seems be be included in the minified version of the open tracing library that is being required somewhere.

$ grep -lr 'Symbol' node_modules/lightstep-tracer
node_modules/lightstep-tracer/dist/lightstep-tracer-node-debug.js
node_modules/lightstep-tracer/dist/lightstep-tracer-node-debug.js.map
node_modules/lightstep-tracer/dist/lightstep-tracer-node.js
node_modules/lightstep-tracer/dist/lightstep-tracer.js
node_modules/lightstep-tracer/dist/lightstep-tracer.js.map
node_modules/lightstep-tracer/dist/lightstep-tracer.min.js
node_modules/lightstep-tracer/examples/browser/opentracing-browser.min.js
node_modules/lightstep-tracer/examples/browser-trivial/opentracing-browser.min.js
node_modules/lightstep-tracer/node_modules/opentracing/dist/opentracing-browser.js
node_modules/lightstep-tracer/node_modules/opentracing/dist/opentracing-browser.min.js
node_modules/lightstep-tracer/node_modules/opentracing/dist/opentracing-node-debug.js
node_modules/lightstep-tracer/node_modules/opentracing/dist/opentracing-node.js
node_modules/lightstep-tracer/test/browser/mocha.js
node_modules/lightstep-tracer/webpack.config.js

Are you seeing the same?

@luk3thomas , these reports are great - thank you for including so much detail.
What you're saying makes perfect sense. I may need to make a pull request on OpenTracing to address this aspect of it.  I admittedly was just checking that the LightStep code had been scrubbed and should have been considering that its dependencies need to be checked as well!

@luk3thomas , this should be fixed now.
I got a pull request into OpenTracing (https://github.com/opentracing/opentracing-javascript/pull/21) and then subsequently updated the LightStep bindings to the new rev (https://github.com/lightstep/lightstep-tracer-javascript/pull/18).
I'm going to optimistically close this before you confirm it :)  Let me know if it's still an issue!!

That fixed it!


[AKS] update for changes in SDK

Changes in Azure/azure-rest-api-specs#1956 to use x-ms-client-flatten implied some minor changes here in the CLI. Note that this will fail until azure-mgmt-containerservice 3.0.0 is available. (avaiable now)
Requires Azure/azure-sdk-for-python#1699

View a preview at https://prompt.ws/r/Azure/azure-cli/5086
This is an experimental preview for @microsoft.com users.
(It may take a minute or two for your instance to be ready)
Email feedback to 'azfeedback' with subject 'Prompt Feedback'.

@mboersma Looks like there are user-facing changes here?
If so can you update HISTORY.rst?

@derekbekoe the only thing a user would notice is that I bumped the example Kubernetes versions in the help text to slightly newer ones. Should that go in HISTORY.rst?

That change is fine.
I was referring to the flattening. Now the JSON output of the modified commands won't have some 'properties' fields?

Good point--I mentioned that in HISTORY.rst now.


PR testing

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request.

Status Flag 'Pull Request AutoTest' - Testing Jenkins Projects:
Pull Request Auto Testing STARTING
Build Information

Test Name: BSSW_PR_AutoTest
Build Num: 17
Status: STARTED

Jenkins Parameters

Parameter Name
Value

BSSW_SOURCE_BRANCH
pr-fail-002

BSSW_SOURCE_REPO
https://github.com/william76/betterscientificsoftware.github.io

BSSW_SOURCE_SHA
7717fe359f0b122f7e97b5e2aa77f7e947f4de59

BSSW_TARGET_BRANCH
develop

BSSW_TARGET_REPO
https://github.com/william76/betterscientificsoftware.github.io

BSSW_TARGET_SHA
571f9f42415945fd81df545f333f73880fa5ce7a

JENKINS_JOB_TYPE
Experimental

TEST_REPO_ALIAS
BSSW

Using Repos:

Repo: BSSW (william76/betterscientificsoftware.github.io)
Branch: pr-fail-002
SHA: 7717fe359f0b122f7e97b5e2aa77f7e947f4de59
Mode: TEST_REPO

Pull Request Author: william76 (William)

Status Flag 'Pull Request AutoTest' - Jenkins Testing: 1 or more Jobs FAILED
Note: Testing will normally be attempted again in approx. 4:00:00. If a change to the Pull Request source branch occurs, the testing will be attempted again.
Pull Request Auto Testing has FAILED
Failure Message(s)
File: test/test_001_fail.md
Invalid value for property 'Topics', given value 'documentation'.
>>> Failed depenency check against property/value pair 'Categories'/'['Planning', 'Reliability', 'Collaboration', 'Crosscutting', 'Performance']'
>>> Allowable values are:
>>> - 'Continuous integration testing'
>>> - 'Debugging'
>>> - 'Design'
>>> - 'Discussion forums'
>>> - 'Funding sources and programs'
>>> - 'High-performance computing (HPC)'
>>> - 'Improving productivity and sustainability'
>>> - 'Licensing'
>>> - 'Performance at leadership computing facilities (LCFs)'
>>> - 'Performance portability'
>>> - 'Projects and organizations'
>>> - 'Reproducibility'
>>> - 'Requirements'
>>> - 'Software interoperability'
>>> - 'Software publishing and citation'
>>> - 'Strategies for more effective teams'
>>> - 'Testing'

Build Information

Test Name: BSSW_PR_AutoTest
Build Num: 17
Status: FAILED

Jenkins Parameters

Parameter Name
Value

BSSW_SOURCE_BRANCH
pr-fail-002

BSSW_SOURCE_REPO
https://github.com/william76/betterscientificsoftware.github.io

BSSW_SOURCE_SHA
7717fe359f0b122f7e97b5e2aa77f7e947f4de59

BSSW_TARGET_BRANCH
develop

BSSW_TARGET_REPO
https://github.com/william76/betterscientificsoftware.github.io

BSSW_TARGET_SHA
571f9f42415945fd81df545f333f73880fa5ce7a

JENKINS_JOB_TYPE
Experimental

TEST_REPO_ALIAS
BSSW


Services appearing in search under workflows

Bug Report
Expected behaviour
Since services are in beta, I would expect them to not appear at all.
Or if we do want them there, they shouldn't be listed under workflows at least.
Actual behaviour
They appear on the search page and service is an option on the language facet. And the links are broken anyway
Steps to reproduce the behaviour

Go to https://staging.dockstore.org/search
Can find a service easily by clicking 'service' option in the Language facet
(Attach tool/workflow documents or links where applicable to help us reproduce your issue)

┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.7
┆Issue Number: DOCK-914

Services should not be inserted into elastic search at this time
(or ideally should be a different type of object ala tools)

➤ Walt Shands commented:
‘service’ no longer appears in the Language facet.


Support for non-chromium browsers

@Himesh-Kundal any updates?

extension is now working on firefox but have to test it on actual cv

create that pull request

#3 Waiting for CV portal to open and test it out.

@harshkhandeparkar can you please test these changes on firefox?

Sure. Can you please provide/link to instructions on how to install it?

I firefox, go to about:debugging, click on "this firefox", Load temporary addon. Select manifest.json

working fine on both for me

I am not sure why this is happening, maybe it is some setting I have to toggle. I'll try to check this out when I get time.


esteros, causewayman (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


Feat/update strategy

Changes proposed in this pull request:

Update prepo-vesting contract to take into consideration when vesting is paused.

Hey @ChaituVR great if you could help take a look at this! Thanks!


Error while creating backup - 'Repository' object has no attribute 'segments'

I have initialized attic with
attic init kanta.attic

and then wanted to create a backup with
sudo attic create kanta.attic::backup-$(date +%Y-%m-%d) / --do-not-cross-mountpoints
attic: /bin/bash: [Errno 95] Operation not supported: 'kanta.attic/index.tmp'
Traceback (most recent call last):
  File "/usr/bin/attic", line 3, in <module>
    main()
  File "/usr/lib/python3/dist-packages/attic/archiver.py", line 715, in main
    exit_code = archiver.run(sys.argv[1:])
  File "/usr/lib/python3/dist-packages/attic/archiver.py", line 705, in run
    return args.func(args)
  File "/usr/lib/python3/dist-packages/attic/archiver.py", line 128, in do_create
    self._process(archive, cache, args.excludes, args.exclude_caches, skip_inodes, path, restrict_dev)
  File "/usr/lib/python3/dist-packages/attic/archiver.py", line 177, in _process
    os.path.join(path, filename), restrict_dev)
  File "/usr/lib/python3/dist-packages/attic/archiver.py", line 177, in _process
    os.path.join(path, filename), restrict_dev)
  File "/usr/lib/python3/dist-packages/attic/archiver.py", line 163, in _process
    archive.process_file(path, st, cache)
  File "/usr/lib/python3/dist-packages/attic/archive.py", line 402, in process_file
    chunks.append(cache.add_chunk(self.key.id_hash(chunk), chunk, self.stats))
  File "/usr/lib/python3/dist-packages/attic/cache.py", line 180, in add_chunk
    self.repository.put(id, data, wait=False)
  File "/usr/lib/python3/dist-packages/attic/repository.py", line 360, in put
    self.segments.setdefault(segment, 0)
AttributeError: 'Repository' object has no attribute 'segments'

How to fix this?

The issue seems to be with the filesystem - I want to create the backup on a HFS+ formatted disk.

So, just to clarify, can you please give attic version, OS informations about where attic is running (OS X? Release?) / backing up to (locally mounted disk?).
Besides the traceback, there was already a error msg before the exception happened:
attic: /bin/bash: [Errno 95] Operation not supported: 'kanta.attic/index.tmp'
So, precisely what operations needed were not supported?

Note: this could be related to issue #202, same exception there.

Sorry for late response.
attic version: 0.13-1
OS: Xubuntu 14.10
backing up to: external disk connected via USB
I don't know about the specific error, will have to check again.

Maybe also try attic 0.14.

The error is just with one folder with afuse-mounted remote directories not accessible to attic.

Interesting. Can you clarify what you mean with "not accessible to attic"?

This is the only error I get from attic:
attic: /home/marting/net: [Errno 13] Permission denied: '/home/shrx/net'

The permissions are
drwx------  1 shrx shrx      0 1970/01/01 01:00:00 net

@shrx so when you ran attic, you were using another user/group than shrx/shrx?
as a side note, the time stamp on that directory looks suspicious, like a 0 timestamp + 1h tz offset.

Yes, I am running attic with sudo.

sudo root? or sudo someoneelse?

just sudo attic

hmm, just a guessing: could the error when accessing the afuse-mounted stuff maybe be related to ACL / xattr support code (like trying to read ACLs/xattrs although they are not supported there)?

I have a backup system which involves three hosts making backups to a same repository on a central server. I am hitting this bug fairly regularly. I added some debug prints and it seems the problem is in repository.prepare_txn: it immediately sets self._active_txn to True, then tries to obtain a lock. The lock fails with:
OSError: [Errno 35] Resource deadlock avoided
This is translated to WriteLockFailed exception; However, the caller seems to ignore the exception and calls put() instead. It thinks that transaction is already active (since _active_txn is true) and fails when trying to access segments().

@theamk I fixed that already, see the 2nd commit in https://github.com/jborg/attic/pull/228


Had to restart in order for SSH agent to register with PowerShell

Installing SSH via the Settings -> Apps menu worked, but I also had to restart my machine in order for Powershell to detect it.

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

ID: 18a5a4e5-2604-23c2-1374-68a6974170bc
Version Independent ID: f93809cc-eb5e-6b9b-ed58-c8811a8ee21f
Content: Installation of OpenSSH For Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
GitHub Login: @maertendMSFT

my windows not show the ssh client/serer packages....help me please

Installing SSH via the Settings -> Apps menu worked, but I also had to restart my machine in order for Powershell to detect it.

Can happen sometimes when enabling and disabling OpenSSH via the PowerShell commands too, you intermittently need to restart Windows for the feature to be enabled or disabled.

my windows not show the ssh client/serer packages....help me please

Try the steps from my comments in https://github.com/MicrosoftDocs/windowsserverdocs/issues/3970 and use PowerShell to install it.

I believe this is a support question, and a stale issue, and can be closed.

#please-close

#please-close


[BUG]: StripeElement--webkit-autofill class is not added to parent container after autofill

What happened?
According to the  docs class "StripeElement--webkit-autofill" should be added (or an override if specified) to the parent container when the input is autofilled. This is useful to adjust the background color to the container since padding is not allowed to be customised.
In either case the docs specify that it should work for Chrome and Safari and it isn't. The "StripeElement--complete" is there though as shown below

Environment
Chrome 104.0.5112.79 (Official Build) (arm64) on MacOS
Reproduction
No response

Hi @drevisios! Are you able to provide a link to a reproduction so I can take a look? Also - what are you using to autofill? I just tested with the Card Element on Chrome using regular Chrome autofill, and was able to see the StripeElement--webkit-autofill show up.

Hello @jackieosborn-stripe. I don't have a public url available at the moment. Once I have one I will provide it. Let me add that I don't use the CardElement but the separate CardNumberElement, if this makes a difference.

Just tested - I'm able to see it on the CardNumberElement as well!

Ok, it has to do with 1password. When it autofills using the browser credit cards it works properly. Thanks for the immediate response! As a note may I add that padding property in inputs would solve so many visual issues but it's covered in another feature request so thank you for the support!


Tetsuya Yamamoto: Watch (?:1\W*)?705\W806\W2021

Tetsuya Yamamoto requests the watch of the watch_keyword (?:1\W*)?705\W806\W2021. See the Metasmoke search here and the Stack Exchange search here.

!!/approve


Fix preflight response status in access logs

What does this PR do?
This PR adds an explicit WriteHeader call in case of a response to a preflight request for the access status code probe to be able to catch it.

Motivation
fixes #10064

More

[x] Added/updated tests
[ ] Added/updated documentation

Additional Notes
Co-authored-by: lbenguigui<EMAIL_ADDRESS>

@ldez Indeed, this was due to a typo in the test fixture.


Support Editing Commit Messages Generated by DevChat

I wanna edit this message before commit it.

The feature is no longer in use; the /commit workflow has been supported with editing capabilities by Charmark.


Accent insensitive search on search and smart-select

This is a (multiple allowed):

[x] bug

[x] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: "framework7": "^1.5.4",

Platform and Target: cordova browser

Live Link or JSFiddle/Codepen:

What you did
Created a Vue listitem with accented strings (ex.: Coração):
<f7-list-item smart-select smart-select-searchbar smart-select-open-in="popup" title="Operações" smart-select-searchbar-placeholder="Pesquisar">
				<select name="car" multiple="multiple" >
					<optgroup v-for="operation in operations" v-bind:label="operation.text">
						<option v-for="item in operation.children" v-bind:value="item.id">{{item.text}}</option>
					</optgroup>
				</select>
			</f7-list-item>

Expected Behavior
While searching the list, when typing coracao, it should find the option Coração (case and accent insensitive).
Or at least give me an event to program it.
Actual Behavior
coracao filters Coração

@JCKodel
I used this to this
`
Dom7.removeDiacritics
$$.removeDiacritics(text) - Replace diacritics in specified text string with standard latin characters
text - string - Text string
var text = $$.removeDiacritics('ÁÓL');
console.log(text); //-> 'AOL'
`

chit
เมื่อ 24 ส.ค. 2017 14:52 "Luke Wheatle"<EMAIL_ADDRESS>เขียนว่า

@JCKodel https://github.com/jckodel
I used this to this
`
Dom7.removeDiacritics
$$.removeDiacritics(text) - Replace diacritics in specified text string
with standard latin characters
text - string - Text string
var text = $$.removeDiacritics('ÁÓL');
console.log(text); //-> 'AOL'
`
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nolimits4web/Framework7/issues/1558#issuecomment-324561192,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT6XhZkTJ8lyBovsF7xHeV2eCbz9bDTiks5sbSu_gaJpZM4NcyoQ
.

I know how to remove accents from a string, but there is no event (AFAIK) to do so on F7 autocomplete.


restrict users to logging in from certain IP address (possibly a range of IP addresses)

I may have a requirement that certain users log in from a certain range of IP addresses in addition to the password requirement that we already have.
I have not been able to find a reasonable way to do this from looking at the Wiki (I'm a non-Ruby dev who inherited a existing Ruby project to maintain and develop - while learning Ruby :) ).  I am still using Rails 3/Devise 2.2.8 (and cancan) if that matters.  I'm also in the process of preparing to update from Rails 3 to Rails 4, so I am starting the process of updating all of the misc. gems as well if that makes a difference.
Any help/pointers would be appreciated.
Thanks,
Sharon

You can override the create action of sessions controller (this would be documented in the wiki or readme), call super and perform the checking of your IP white list there.

Thanks - I'll give that a shot!


amber and ayaka radius update

amber e explosion now has radius of 3m and centers on primary target
ayaka q (bloom) now centers on primary target
ayaka q (cutting) now has radius of 3m and centers on primary target
ayaka c2 (bloom) now has radius of 3m and centers on primary target
ayaka c2 (cutting) now has radius of 1.5m and centers on primary target
ayaka e now has radius of 4.5m
ayaka charged attack now has radius of 1m and centers on primary target

ayaka c2 positioning will probably have to get more fancy later on


Error at start up of omnisharp: mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded

System: ubuntu 20.04

Open Visual Studio Code v1.52.1
Open my (very big) C# project

Omnisharp mentions me the following errors
Actually, one project of my solution uses these assemblies (microsoft.codeanalysis.*****). But, something interesting: if I put appart the project that references assemblies that seem to be the source of the problem and open it alone in VSCode, I don't get the problem.
Information: my solution is VERY big.
Starting OmniSharp server at 1/5/2021, 11:31:21 AM
    Target: /home/stephane/Documents/Sources/PMS/src/FundProcess.Pms.sln

OmniSharp server started.
    Path: /home/stephane/.vscode/extensions/ms-dotnettools.csharp-1.23.8/.omnisharp/1.37.5/run
    PID: 9520

Can't find custom attr constructor image: /home/stephane/.nuget/packages/microsoft.codeanalysis.csharp.features/3.8.0/lib/netcoreapp3.1/Microsoft.CodeAnalysis.CSharp.Features.dll mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.
Can't find custom attr constructor image: /home/stephane/.nuget/packages/microsoft.codeanalysis.csharp.workspaces/3.8.0/lib/netcoreapp3.1/Microsoft.CodeAnalysis.CSharp.Workspaces.dll mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.
Can't find custom attr constructor image: /home/stephane/.nuget/packages/microsoft.codeanalysis.features/3.8.0/lib/netcoreapp3.1/Microsoft.CodeAnalysis.Features.dll mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.
Can't find custom attr constructor image: /home/stephane/.nuget/packages/microsoft.codeanalysis.csharp.features/3.8.0/lib/netcoreapp3.1/Microsoft.CodeAnalysis.CSharp.Features.dll mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.
Can't find custom attr constructor image: /home/stephane/.nuget/packages/microsoft.codeanalysis.csharp.workspaces/3.8.0/lib/netcoreapp3.1/Microsoft.CodeAnalysis.CSharp.Workspaces.dll mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.
Can't find custom attr constructor image: /home/stephane/.nuget/packages/microsoft.codeanalysis.features/3.8.0/lib/netcoreapp3.1/Microsoft.CodeAnalysis.Features.dll mtoken: 0x0a000006 due to: Cannot resolve dependency to assembly because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.

I'm having the same issue. Omnisharp stops functioning more often. I noticed I am spending lots of time trying to get intellisense up and running again. Restarting omnisharp sometimes helps but not always.

I have exactly the same issue.. have you fond a solution for this ?

I have exactly the same issue.. have you fond a solution for this ?

I stopped using VS Code and started using Rider. Of course that doesn't solve this specific problem but I simply can't wait for the developers of OmniSharp to come up with a fix. I have deadlines too to catch..

Same problem here... Ubuntu 20.04 and Arch Linux (Yes, I know it's not supported ..)

Running into this problem myself. This basically renders VS Code useless for C# projects.

I've found a solution for this problem:
the problem is due to a missing dlls from current .net version.. but same dlls are present in the previous version.
The problem is only on linux distro.. on dotnet 5.
My solution is:

install previous version of SDK for example 3.1.113
find installation folder /usr/lib64/dotnet/sdk/3.1.113 and go to subdir /usr/lib64/dotnet/sdk/3.1.113/Sdks/Microsoft.NET.Sdk.Razor/tasks/net472
copy alll the dlls in this folder to the .net missing folder /usr/lib64/dotnet/sdk/5.0.104/Sdks/Microsoft.NET.Sdk.Razor/tasks/net46

this fixed the problem for me. If you need i can provide missing dlls from my pc

Having same issue, but assemblies causing it are my own, from referenced F# projects. So I can't just drop those away to fix the problem.  Rider behaves correctly with the same solution.
Bottom line is: it's 2021 and dotnet 6 upcoming, yet I can't smoothly develop on Mac (no idea if Windows users are luckier) 🤦♂.  I just don't have any simple and efficient tool to work all day long without stress (and time losses). VS Code is not working. Rider is overkill for me (I just don't need all those fancy features! and BTW it eats all the RAM. I have it and use occasionally when get tired of VS Code crashing, but that can't be a solution long-term). Visual Studio Mac is the last resort, yet it seems to be a poor trade-off so far.
I feel really desperate seeing this issue with no real reaction/solution for months. As of today VS Code (at least on Mac) is NOT for dotnet development (whatever ridiculous it may sound). It's a very cool IDE, but for something else other than dotnet. So either switch to other OS/IDE, or just develop in typescript/haskell etc LOL.

There have been lots of improvements since this issue was opened. Please open a new issue with logs if you are still running into this.


Passing multiple inputs (scalar and columns) to the parametrized_input decorator

Is your feature request related to a problem? Please describe.
I would like to pass multiple inputs (scalar and columns) to the parametrized_input decorator.
Describe the solution you'd like
I would like to create a series of rules for different columns with all parameters in one place for easier management and readability.
Example of a rule is whether any values in the column has exceeded a threshold value, however this could be extended to include any numeric comparisons such as <,>, = or combinations thereof e.g. between value_1 & value_2 (inclusive).
import pandas as pd
from hamilton.function_modifiers import parametrized_input

INV_PARAMS = {
     #input var        (# output var,   # threshold_value          ,  # description of new outputs)
     'inventory', ('inventory_geq_1000', 1000, 'inventory greater than or equal to 1000'),
     'inventory', ('inventory_leq_215', 215, 'inventory less than or equal to 215'),
}

@parametrized_input(parameter='inventory', assigned_inputs=INV_PARAMS )
def inventory_geq(inventory: pd.Series) -> pd.Series:
    pass

Thanks so much!

So, I think this makes sense. The question is whether you want to do this just on the inventory column, or on a bunch of different columns (E.G. different types of inventory). If you do this on just the inventory column, you could do something like this:
INV_PARAMS = {
    ('inventory_geq_1000', 'inventory greater than or equal to 1000') : 1000,
    ('inventory_leq_215', 'inventory less than or equal to 215'): 215,
}

@parametrized(parameter=threshold, assigned_output=INV_PARAMS)
def inventory_geq(inventory: pd.Series, threshold: int):
    return inventory[inventory > threshold)

This works if you have one column inventory that you want to filter. If you have multiple columns, then you could create the above for each one (a little verbose, but there's something nice about being specific/making things readable), or we could look at combining @parametrized with @parametrized_input as you suggested. Does this make sense?

Thanks for clarifying my example. I agree that it is a bit verbose but
definitely gets the job done.
Can I naturally extend this example to use 2 threshold values by passing an
array of threshold values instead of just one int? This would be good for
identifying column values between the two threshold values.
On Wed, Nov 17, 2021 at 12:12 AM Elijah ben Izzy @.***>
wrote:

So, I think this makes sense. The question is whether you want to do this
just on the inventory column, or on a bunch of different columns (E.G.
different types of inventory). If you do this on just the inventory column,
you could do something like this:
INV_PARAMS = {
('inventory_geq_1000', 'inventory greater than or equal to 1000') : 1000,
('inventory_leq_215', 'inventory less than or equal to 215'): 215,
}
@parametrized(parameter=threshold, assigned_output=INV_PARAMS)def inventory_geq(inventory: pd.Series, threshold: int):
return inventory[inventory > threshold)
This works if you have one column inventory that you want to filter. If
you have multiple columns, then you could create the above for each one (a
little verbose, but there's something nice about being specific/making
things readable), or we could look at combining @parametrized with
@parametrized_input as you suggested. Does this make sense?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stitchfix/hamilton/issues/25#issuecomment-971197943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB46N5MJDZJVFDFBR4IMS53UMM2UDANCNFSM5IEUDUOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Regards,
Lorre Atlan, PhD

Thanks for clarifying my example. I agree that it is a bit verbose but definitely gets the job done. Can I naturally extend this example to use 2 threshold values by passing an array of threshold values instead of just one int? This would be good for identifying column values between the two threshold values.
On Wed, Nov 17, 2021 at 12:12 AM Elijah ben Izzy @.***> wrote: So, I think this makes sense. The question is whether you want to do this just on the inventory column, or on a bunch of different columns (E.G. different types of inventory). If you do this on just the inventory column, you could do something like this: INV_PARAMS = { ('inventory_geq_1000', 'inventory greater than or equal to 1000') : 1000, ('inventory_leq_215', 'inventory less than or equal to 215'): 215, } @parametrized(parameter=threshold, assigned_output=INV_PARAMS)def inventory_geq(inventory: pd.Series, threshold: int): return inventory[inventory > threshold) This works if you have one column inventory that you want to filter. If you have multiple columns, then you could create the above for each one (a little verbose, but there's something nice about being specific/making things readable), or we could look at combining @parametrized with @parametrized_input as you suggested. Does this make sense? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#25 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB46N5MJDZJVFDFBR4IMS53UMM2UDANCNFSM5IEUDUOA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
-- Regards, Lorre Atlan, PhD

Yep you can! We're not opinionated on the datatypes at all actually, except in the cases in which we've made it easier to use pandas dataframes. Config items can be anything.

@elijahbenizzy should this issue be closed? Or an updated added?

@skrawcz yeah, I think so. @latlan1, would love to get a sense of how the new changes to @parameterize woudl help you: https://github.com/stitchfix/hamilton/blob/main/decorators.md#parameterize.

OK, so I"m pretty sure this is now solved! @parameterize is far more sophisticated. @latlan1 let me know if there's something that hasn't been covered and we can reopen, otherwise I'm closing this!

Looks good-thank you!
On Sat, Oct 29, 2022 at 12:38 PM Elijah ben Izzy @.***>
wrote:

Closed #25 https://github.com/stitchfix/hamilton/issues/25 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/stitchfix/hamilton/issues/25#event-7697445432, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AB46N5KORTND63UWGPKAIVTWFVOKRANCNFSM5IEUDUOA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Regards,
Lorre Atlan, PhD


Incompatible with Dart >=2.0.0

Hey there!
I've been trying to use this lib for a simple test project but had the following issue trying to install it:
[discord_dart] pub get
Resolving dependencies...
The current Dart SDK version is 2.0.0.

Because nyxx >=0.24.0 depends on w_transport >=1.0.1 which requires SDK version <2.0.0, nyxx >=0.24.0 is forbidden.
So, because discord_dart depends on nyxx 0.24.0, version solving failed.
exit code 1

nyxx is the sole dependency I've got in my pubspec.yml.
I've just checked and the w_transport package (even in its 3.2.6 version) requires
environment:
    sdk: ">=1.14.0 <2.0.0"

so this issue is here. Do you have any plan to use something else?

Damn I'm sorry I've just found the dart2.0 branch :D
Do you have any idea of when you'll publish this new branch in place of the "older" one?

Alright! Take your time I'm not in a hurry.
I'll leave it up to you if you want to close the issue now or if you want to keep it as a reference for the updates.

Latest commit from this branch should all compatibility issues. Nyx is built with latest stable dart sdk. Make sure you are pulling nyxx from git:
dependencies:
  nyxx:
    git:
      url: git://github.com/l7ssha/nyxx.git
      ref: backend_rework

I'll leave this issue as refernce.

Fixed. https://github.com/l7ssha/nyxx/releases/tag/0.3.0


Table field type is missing

Hi!
I noticed, that the table field type is missing in generated types, would be great to have StoryblokTable type or smth with proper structure, but at least table: any would be nice to have
Thank you.

@wolkyura I would accept a PR for integration of tables. Currently I am backed up with work but interested to look into this when time allows.
You can currently set up a custom field for tables as shown in the docs: https://github.com/dohomi/storyblok-generate-ts#example-custom-parser


WORKPLACE TRANSFORMATION SITE provisioning failed

Workplace Transformation site
https://lookbook.microsoft.com/details/b0633491-127d-48bf-ba06-a0cdb6ec749f
correlation ID: 535669f4-a7cf-4232-b488-dbca890d4ba3

Hi,
Thanks for using the Look Book site. This seems to be a random hiccup on the SharePoint side which would imply that the template should work properly if you try it one more time. Seems that the provisioning did succeed partially. Can you test this one more time and, in case of any failure, share the updated Correlation ID (by creating a new issue) for us to double-check the flow. Thanks.


Documentation for encoding of control flow in VC generation

@bkragl :  Do you understand why this build is failing?

@keyboardDrummer :  The problem appears to be the one related to libraries on Ubuntu. Any suggestions?

@bkragl : Do you understand why this build is failing?

Nope, looks like an issue with GitVersion.
@keyboardDrummer also had to re-trigger a build in PR #319. Was that the same issue?

@bkragl : Do you understand why this build is failing?

Nope, looks like an issue with GitVersion.
@keyboardDrummer also had to re-trigger a build in PR #319. Was that the same issue?

Yes that was the same issue. I guess we should revert all or a part of PR #319

I am triggering a rebuild just to see if problem gets fixed deterministically with a rebuild.  I suppose we should try to get to the bottom of the problem.

I am triggering a rebuild just to see if problem gets fixed deterministically with a rebuild. I suppose we should try to get to the bottom of the problem.

Looks like it worked.
This was the error message in the failed build:
/home/travis/.nuget/packages/gitversiontask/5.5.1/build/GitVersionTask.targets(10,9): error : NameConflictException: failed to write reference 'refs/heads/pull/320/merge': a reference with that name already exists. [/home/travis/build/boogie-org/boogie/Source/UnitTests/TestUtil/TestUtil.csproj]

Note that for each PR there are actually 4 builds: {Branch, Pull Request} x {Debug, Release}
Only one of them failed.
I searched for the error and found https://github.com/GitTools/GitVersion/issues/864, where people also reported this problem happening sporadically on pull requests. Supposedly it was fixed in GitVersion 5.3.3.
It's a long shot, but we could try to call dotnet build explicitly before dotnet pack. If I remember correctly, there was a glitch in #319 that dotnet pack does not compile the unit tests, which is now done by dotnet test. Maybe compiling everything together works (because the error shows up for a test project).


Not getting IPv6 address on WAN interface at boot

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
My WAN interface is configured as PPPoE for IPv4 and DHCPv6 for IPv6. For the DHCPv6 client configuration, the prefix is set to 56 and Use IPv4 connectivity is checked, the others are unchecked. When I reboot, I get a IPv4 address, but not an IPv6 one unless I reload/reconfigure the WAN interface. After that everything works.
To Reproduce
Expected behavior
An IPv6 address should be assigned on boot.
Describe alternatives you considered
I have created a cron script that runs frequently and check if I have an IPv6 address, if I don’t it runs rc.configure_interface wan.
Relevant log files
2024-04-13T02:53:27	Notice	opnsense	/usr/local/etc/rc.newwanipv6: ROUTING: keeping inet6 default route to fe80::1%pppoe0
2024-04-13T02:53:26	Notice	opnsense	/usr/local/etc/rc.newwanipv6: IP renewal starting (address: misconfigured, interface: wan, device: pppoe0)
2024-04-13T02:53:05	Notice	opnsense	/usr/local/etc/rc.newwanipv6: ROUTING: keeping inet6 default route to fe80::1%pppoe0
2024-04-13T02:53:05	Notice	opnsense	/usr/local/etc/rc.newwanipv6: IP renewal starting (address: misconfigured, interface: wan, device: pppoe0)
2024-04-13T02:53:02	Notice	kernel	<118> WAN (pppoe0) -> v4/PPPoE: x.x.x.x/32
2024-04-13T02:52:55	Notice	dhcp6c	dhcp6c_script: REQUEST on pppoe0 renewal
2024-04-13T02:52:55	Notice	dhcp6c	dhcp6c_script: REQUEST on pppoe0 executing
2024-04-13T02:52:54	Notice	kernel	<118>setup pppoe0 [egress only]
2024-04-13T02:52:54	Notice	opnsense	/usr/local/etc/rc.newwanip: plugins_configure monitor (execute task : dpinger_configure_do(,WAN_PPPOE))
2024-04-13T02:52:54	Notice	opnsense	/usr/local/etc/rc.newwanip: plugins_configure monitor (,WAN_PPPOE)
2024-04-13T02:52:54	Notice	kernel	<118>Reconfiguring IPv6 on pppoe0
2024-04-13T02:52:53	Notice	opnsense	/usr/local/etc/rc.newwanip: IP renewal starting (new: x.x.x.x, old: , interface: wan, device: pppoe0, force: yes)
2024-04-13T02:52:53	Notice	kernel	<118>Reconfiguring IPv4 on pppoe0
2024-04-13T02:52:47	Notice	opnsense	/usr/local/etc/rc.newwanipv6: IP renewal deferred during boot on 'pppoe0'
2024-04-13T02:52:44	Notice	opnsense	/usr/local/etc/rc.newwanip: IP renewal deferred during boot on 'pppoe0'
2024-04-13T02:52:43	Notice	kernel	<5>ng_pppoe[15]: no matching session
2024-04-13T02:52:42	Notice	kernel	<5>ng_pppoe[15]: no matching session
2024-04-13T02:52:42	Notice	kernel	<5>ng_pppoe[15]: no matching session
2024-04-13T02:52:35	Notice	kernel	<6>ng0: changing name to 'pppoe0'
Environment
OPNsense 24.1.5_3-amd64
Intel(R) Xeon(R) E-2278G CPU @ 3.40GHz (2 cores, 2 threads)
Intel® PRO/1000 PT Quad Port Server Adapter

I am also having IPV6 issues.
My WAN is set to DHCPv6 like it was in PFsense and I set my LAN to track.
My LAN now has a valid IP and if I SSH into the router I can ping6 google and get around 4ms latency.
But devices on my LAN are not being delegated an IP. This is with the 'Allow manual adjustment of DHCPv6 and Router Advertisements' disabled.
Upon enabling it, and trying various settings, I cannot get devices on my LAN to get an IPV6 address. I've tried DHCPv6 with all the different RA settings and rebooting after changes, but nothing.
Yet, in PFSense I was able to get IPV6 working with no problems. I must be going crazy or its broken in OPNSense

I must be going crazy or its broken in OPNSense

you are not crazy. its been broken like this for at least a year. every time i update or restart the router for whatever reason it breaks again and its a fight between restarts, wan interface disable/enable, settings changes to get it back. my isp has perfect ipv6 support and provides a /56 prefix. opnsense falls flat on its face and i wonder if i should just find a replacement.

I must be going crazy or its broken in OPNSense

you are not crazy. its been broken like this for at least a year. every time i update or restart the router for whatever reason it breaks again and its a fight between restarts, wan interface disable/enable, settings changes to get it back. my isp has perfect ipv6 support and provides a /56 prefix. opnsense falls flat on its face and i wonder if i should just find a replacement.

My issue ended up being the bridge I was running as the LAN. I've switched back to PFsense, removed the bridge and put up with having each of my switches and sub devices be in a different subnet.
Now IPv6 is working perfectly. I get 19/20 points on https://ipv6-test.com. I am using SLAAC only too, no DHCP6 nonsense.

All I can see is that it fails to find a primary IPv6:
https://github.com/opnsense/core/blob/81676259d357fc444a78d511f6f1b11f98d94af7/src/etc/rc.newwanipv6#L92
It would help to see what the correct address from the ISP is, but it sort of suggests you want an address but you don’t get one (ia-na) for an unknown reason.
you know, guys, there is a debug setting for dhcpv6 client to enable since these issues are always entangled with the setup and ISP behaviour.

you know, guys, there is a debug setting for dhcpv6 client to enable since these issues are always entangled with the setup and ISP behaviour.

I didn't;t have issues getting an IPv6 from my ISP via WAN, my issue was that even thought he LAN interface had a real global /64 prefix from the /56 prefix, RA AND DHCP6 were both broken.
Turns out my issue was due to using a LAN bridge. Been an issue before https://redmine.pfsense.org/issues/8429

Sounds like a FreeBSD kernel issue with multicast forwards then if not done on purpose by the bridge implementation, perhaps a scope mismatch of link local advertisements.

Hello, I've got some more logs with dhcpv6 logging set to debug when the router boots and doesn't get a wan ipv6https://pastebin.com/a2SsvYuK. I can also post some logs after i reload services and it gets the ip if that's helpful.

All I can see is that it fails to find a primary IPv6:
https://github.com/opnsense/core/blob/81676259d357fc444a78d511f6f1b11f98d94af7/src/etc/rc.newwanipv6#L92
It would help to see what the correct address from the ISP is, but it sort of suggests you want an address but you don’t get one (ia-na) for an unknown reason.
you know, guys, there is a debug setting for dhcpv6 client to enable since these issues are always entangled with the setup and ISP behaviour.

i suspect the problem is not on the ISP side, but i will upload logs when i have this problem again. as i said before, usually some combination of disabling ipv6 and re-enabling on the interfaces, and restarting opnsense, resolves the issue.

some combination of disabling ipv6 and re-enabling on the interfaces, and restarting opnsense, resolves the issue

Which indicates timing inconsistencies with the ISP sometimes giving out the proper prefix and address and sometimes not depending on the previous lease state.
The fun part is the IP detection is just diagnostics since 24.1.5 which would further indicate the ISP is giving out the address/prefix sometimes, but not always.
The fact that it works for other distributions might be simply that the request rate is lower causing this to not fail as much.
Cheers,
Franco

Hi, I'm having similar issue. Also PPPoE for ipv4 and DHCP6 for ipv6 and /56 from ISP to delegated to LAN (track interface), without specific WAN ipv6.
I noticed that switching from "track interface" to "SLAAC" and back to "track interface" fixes it. I narrowed it down to dhcp6c not starting, (maybe it's a matter of services start order against PPPoE? Not sure...). Starting dhcp6c manually solves the problem in my case and ipv6 is assigned at once. @AtomicDude can you try to start dhcp6c after fresh boot and see if it solves it for you?


Improve documentation

We should make it as easy as possible to get started with Apicula. Part of that is writing documentation, part of that is working with toolchain distributions to package Apicula
User:

Review and update oss-cad-suite distribution
Write documentation about all command line options
Write documentation about all supported primitives
Streamline the readme for getting started

Developer:

https://github.com/YosysHQ/apicula/issues/282#issuecomment-2424656059
someone asked me about explaining the architecture of how the chipdb is built and flows between apicula and nextpnr but I forgot where

it seems like oss-cad-suite ships with nextpnr-himbaechel and apicula so it looks like there isn't anything to be done there
https://github.com/YosysHQ/oss-cad-suite-build/blob/main/default/scripts/apicula.sh
https://github.com/YosysHQ/oss-cad-suite-build/blob/main/default/scripts/nextpnr-himbaechel-gowin.sh

The readme seems fine honestly, I've removed some outdated parts in adab34b53bf357afc1532d1561348c2c6ccf9f54 and I guess I'll add a link later to the wiki where I plan to host most of this.
Or idk, I'm open to opinions. We have a docs folder, which is all developer documentation. So maybe I'll keep adding developer stuff there but put user documentation in the wiki so it's easy to browse?

Alright I've added documentation for all the command line options https://github.com/YosysHQ/apicula/wiki

and all the supported primitives https://github.com/YosysHQ/apicula/wiki/Supported-primitives

Updated readme


External dependencies upgrade - Spring Boot 2.7.16 and Spring Cloud 2021.0.8

Updates external dependencies to align with Spring Boot version 2.7.16 from 2.7.14 and Spring Cloud version 2021.0.8 from 2021.0.8.
Release notesSourced from spring-boot releases.⭐ New Features Add TWENTY_ONE to JavaVersion enum #37362🐞 Bug Fixes Invalid Accept header produces HTTP 500 in WelcomePageHandlerMapping #37455 PrivateKeyParser doesn't support ed448, XDH and RSA-PSS keys #37237 Parsing OCI image names that are invalid due to the use of upper case letters is very slow #35657 Using https with elliptic curves other than secp384r1 fails #34232 Saml2RelyingPartyAutoConfiguration ignores sign-request when metadata-url is used #33747 Leaking file descriptor / socket within DomainSocket tooling #32423📔 Documentation Correct the description of spring.artemis.broker-url #37260 Add default value metadata for management.metrics.export.signalfx.published-histogram-type #37210 Document that PKCS8 PEM files should be used whenever possible #37170 Polish javadoc #37112🔨 Dependency Upgrades Upgrade to Elasticsearch 7.17.13 #37286 Upgrade to Jetty 9.4.52.v20230823 #37287 Upgrade to Lombok 1.18.30 #37486 Upgrade to Micrometer 1.9.15 #37245 Upgrade to Reactor Bom 2020.0.36 #37246 Upgrade to Spring AMQP 2.4.16 #37247 Upgrade to Spring Data Bom 2021.2.16 #37248 Upgrade to Spring Framework 5.3.30 #37249 Upgrade to Spring GraphQL 1.0.6 #37250 Upgrade to Spring Integration 5.5.19 #37251 Upgrade to Spring Security 5.7.11 #37414 Upgrade to Spring WS 3.1.7 #37415 Upgrade to Tomcat 9.0.80 #37288❤ ContributorsThank you to all the contributors who worked on this release:@dependabot[bot], @hakan-krgn, @izeye, @mdeinum, and @quaff
This PR is created by GitHub Actions: https://github.com/Azure/spring-cloud-azure-tools/actions/runs/6306342519

API change check
API changes are not detected in this pull request.

/azp run java - spring - tests

/azp run java - cosmos - tests

/azp run java - keyvault - tests

/check-enforcer override


Temporarily switch from apparition to cuprite

This is a workaround for https://github.com/twalpole/apparition/issues/64.

@seand7565 can it be that you're running a rubocop from outside the bundle?

I don't think so, I'm running bundle exec rubocop. What's weird is it's also not recognizing some of the new cops. I made sure my rubocop version was updated. 🤔
seandenny@MacBook-Pro solidus_paypal_commerce_platform % bundle exec rubocop
Warning: unrecognized cop Layout/FirstArgumentIndentation found in .rubocop.yml
Warning: unrecognized cop Layout/FirstArrayElementIndentation found in .rubocop.yml
Warning: unrecognized cop RSpec/MultipleMemoizedHelpers found in .rubocop.yml
Warning: unrecognized cop Rails/Inquiry found in .rubocop.yml
Warning: unrecognized cop Rails/ActiveRecordCallbacksOrder found in .rubocop.yml
Warning: unrecognized cop Rails/FindById found in .rubocop.yml
Warning: unrecognized cop Rails/MailerName found in .rubocop.yml
Warning: unrecognized cop Rails/MatchRoute found in .rubocop.yml
Warning: unrecognized cop Rails/NegateInclude found in .rubocop.yml
Warning: unrecognized cop Rails/Pluck found in .rubocop.yml
Warning: unrecognized cop Rails/PluckInWhere found in .rubocop.yml
Warning: unrecognized cop Rails/RenderInline found in .rubocop.yml
Warning: unrecognized cop Rails/RenderPlainText found in .rubocop.yml
Warning: unrecognized cop Rails/ShortI18n found in .rubocop.yml
Warning: unrecognized cop Rails/WhereExists found in .rubocop.yml
Inspecting 60 files
..CC....C..C..C.C.C.........C...C..CCC.....CCCC...CC.....C..

🤔 this straight after a bundle update or bin/setup?
I'm getting this:
solidus_paypal_commerce_platform:master ⤑ be rubocop -ESP
The following cops were added to RuboCop, but are not configured. Please set Enabled to either `true` or `false` in your `.rubocop.yml` file:
 - Rails/AfterCommitOverride (2.8)
 - Rails/SquishedSQLHeredocs (2.8)
 - Rails/WhereNot (2.8)
 - Performance/Sum (1.8)
For more information: https://docs.rubocop.org/rubocop/versioning.html
Inspecting 62 files
..............................................................

62 files inspected, no offenses detected
solidus_paypal_commerce_platform:master ⤑                                                                  ~/C/N/solidus_paypal_commerce_platform


Update dbms-guide-ibm.md

Is it possible to add the name WSCS as I think it is more widely known?

@msjuergent
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


chore(docs): Fix grammar and spelling in the "Self Hosting" section

Update grammar and spelling
Service Adapters
Service Adapters are responsible for executing the request with the LLM and standardizing the request/response format in a way that the Copilot Runtime can understand.

Nice @NathanTarbert!


fix superbuild cmake < 3.17

issue: #192

CPM_INDENT (first commit) :
I agree with you. set(CPM_INDENT "CPM:" CACHE INTERNAL "") seems to be appropriate.

include(FetchContent):
Since the problem only occurs when using cmake < 3.17, I suggest to check the current cmake version and call include(FetchContent) for each include(CPM.cmake) when necessary.

During a future upgrade of the minimal cmake version, this can be noticed and removed easily.

Hey thanks for the PR! Yeah, the version fallback seems like a good approach. I wonder if we should also include(CMakeParseArguments), as it's used by CPM.cmake functions as well.

For include(CMakeParseArguments) I didn't notice any problem. Since CMakeParseArguments only serves to manipulate variables I don't see how there would be an internal bug in cmake with this use case.
If I notice anything strange while continuing my project I will bring up the problem anyway.

Fix is live in v0.28.3! 🎉


fix(deis-logger/manifests/deis-logger-svc.yaml): make the logger service run on port 80

Fixes #136
Ref deis/deis#4853
cc/ @jchauncey

@krancour

@jchauncey or @helgi were either of you able to test this by chance?

no but i will really quick

so i thought I had the proper env var for the service port in the controller but i was wrong. I submitted a PR to fix that before this one can be merged.

@jchauncey the PR is https://github.com/deis/workflow/pull/476, right?

yes

@arschles ok the other PR has been merged

@jchauncey thanks. I'm gonna wait til the image is built to merge this


Failed to load resource when JS is minified.

Hi,
I'm busy working on integrating Cesium into the Openlayers module for Drupal.
Everything works fine until the JS is no more aggregated, see the screenshot.

When I enable the JS aggregation, I add the variable CESIUM_BASE_URL to the page, with the path to the Cesium library and I'm getting these two errors and the map doesn't shows up.

You can see the problem, live at this address: http://dev.through-my-eyes.be/openlayers_examples/openlayers_examples_map_cesium
The Cesium JS library is installed at: http://dev.through-my-eyes.be/sites/all/libraries/cesium/
The CESIUM_BASE_URL is properly filled in if you look in the page HTML source:

The Cesium JS library has been installed following these steps:
user@server:~/domains/dev.through-my-eyes.be/public_html/sites/all/libraries$ wget http://cesiumjs.org/releases/Cesium-1.17.zip
user@server:~/domains/dev.through-my-eyes.be/public_html/sites/all/libraries$ mkdir cesium
user@server:~/domains/dev.through-my-eyes.be/public_html/sites/all/libraries$ mv Cesium-1.17.zip cesium
user@server:~/domains/dev.through-my-eyes.be/public_html/sites/all/libraries$ cd cesium
user@server:~/domains/dev.through-my-eyes.be/public_html/sites/all/libraries/cesium$ unzip Cesium-1.17.zip
Can you please let me know what is going on and how I could fix this ?
Thanks!

To get this working you need to generate a JS file on the fly containing:
var CESIUM_BASE_URL = '/sites/all/libraries/cesium/Build/Cesium/';
So you have to pay attention to two things:

It's not enough to provide the base library path (/sites/all/libraries/cesium/), you must provide the path + '/Build/Cesium/'

The string must end with a slash or else it won't work.


feat: update main.js to be equal to transifex utils

Description

Update main.js to be equal to frontend-platform's transifex-utils.js file. This is needed to fix studio-frontend push translation job that uses reactifex's main.js to compile comments instead of frontend-platform(not installed in it)
Enable quality test on CI
Update package.json bin to rename main.js usage in edx/reactifex via $(npm bin)/edx_reactifex

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Default metadata field names in PagePdfDocumentReader are can't be parsed in a filter expression

The field name file_name is not compatible with the filter expression parsing.
		SearchRequest searchRequest = SearchRequest.defaults()
				.withTopK(4)
				.withFilterExpression(PagePdfDocumentReader.METADATA_FILE_NAME + " == 'medicaid-wa-faqs.pdf'");

where `public static final String METADATA_FILE_NAME = "file_name"
throws the exception
Caused by: org.antlr.v4.runtime.NoViableAltException: null
	at org.antlr.v4.runtime.atn.ParserATNSimulator.noViableAlt(ParserATNSimulator.java:2014) ~[antlr4-runtime-4.13.1.jar:4.13.1]
	at org.antlr.v4.runtime.atn.ParserATNSimulator.execATN(ParserATNSimulator.java:445) ~[antlr4-runtime-4.13.1.jar:4.13.1]
	at org.antlr.v4.runtime.atn.ParserATNSimulator.adaptivePredict(ParserATNSimulator.java:371) ~[antlr4-runtime-4.13.1.jar:4.13.1]
	at org.springframework.ai.vectorstore.filter.antlr4.FiltersParser.booleanExpression(FiltersParser.java:556) ~[spring-ai-core-1.0.0-SNAPSHOT.jar:1.0.0-SNAPSHOT]
	at org.springframework.ai.vectorstore.filter.antlr4.FiltersParser.where(FiltersParser.java:199) ~[spring-ai-core-1.0.0-SNAPSHOT.jar:1.0.0-SNAPSHOT]
	at org.springframework.ai.vectorstore.filter.FilterExpressionTextParser.parse(FilterExpressionTextParser.java:147) ~[spring-ai-core-1.0.0-SNAPSHOT.jar:1.0.0-SNAPSHOT]
	... 46 common frames omitted

Underscore seems to be the issue.  Suggest we change to use camel case for document readers that add metadata fields.

Is this specific to PGVectorStore only or others too? It works fine for me when I use the Filter.Expression record. For e.g., the below code works (I am using Spring AI version 1.0.0-M1) -
final var expression = new Filter.Expression(Filter.ExpressionType.EQ, new Filter.Key("file_name"),
                new Filter.Value("sample_file.pdf"));

        List<Document> similarDocuments = vectorStore
                .similaritySearch(SearchRequest.query(message)
                        .withFilterExpression(expression));

Here are some more observations on Metadata filtering -

When I use the PgVectorFilterExpressionConverter, the above code does not work. For e.g., the below code does not work -

FilterExpressionConverter converter = new PgVectorFilterExpressionConverter();
final var expression = new Filter.Expression(Filter.ExpressionType.EQ, new Filter.Key("file_name"),
                new Filter.Value("sample_file.pdf"));

        final var convertExpression = converter.convertExpression(expression);

        List<Document> similarDocuments = vectorStore
                .similaritySearch(SearchRequest.query(message)
                        .withFilterExpression(convertExpression));

I see the following error in the logs -
Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed: org.springframework.ai.vectorstore.filter.FilterExpressionTextParser$FilterExpressionParseException: Source: <unknown>, Line: 1:7, Error: no viable alternative at input '.'] with root cause

org.antlr.v4.runtime.NoViableAltException: null
	at org.antlr.v4.runtime.atn.ParserATNSimulator.noViableAlt(ParserATNSimulator.java:2014) ~[antlr4-runtime-4.13.1.jar:4.13.1]

The 'IN' filter type does not seem to work at all. For e.g. the following code does not work -

final var expression = new Filter.Expression(Filter.ExpressionType.IN, new Filter.Key("file_name"),
                new Filter.Value(List.of("file1.pdf", "file2.pdf", "file3.pdf")));

        List<Document> similarDocuments = vectorStore
                .similaritySearch(SearchRequest.query(message)
                        .withFilterExpression(expression));

I get the following error -
 Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed: org.springframework.jdbc.BadSqlGrammarException: PreparedStatementCallback; bad SQL grammar [SELECT *, embedding <=> ? AS distance FROM vector_store WHERE embedding <=> ? < ?  AND metadata::jsonb @@ '$.file_name in ["file1.pdf","file2.pdf","file3.pdf"]'::jsonpath  ORDER BY distance LIMIT ? ]] with root cause

org.postgresql.util.PSQLException: ERROR: syntax error at or near " " of jsonpath input
  Position: 110
	at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2725) ~[postgresql-42.7.3.jar:42.7.3]

I tried with a metadata without an underscore and still face the same error.

Thanks for the investigation, there is a deeper bug lurking here.

I have raised https://github.com/spring-projects/spring-ai/pull/1483 with a fix to the original issue - that unescaped underscores were not accepted in filter expression keys.
For the 2nd issue mentioned above - "When I use the PgVectorFilterExpressionConverter, the above code does not work" I would say that it is valid to throw an error in that case - where the results of PgVectorFilterExpressionConverter. convertExpression are passed to  SearchRequest. withFilterExpression as the portable SearchRequest is not designed to be aware of PgVector style expressions.


Create Data Transfter Objects for Weapons Detection

Commit 2f1a1d3517e1b04321d41ac41a7cd1733fc0e14b is part of this task

Commit f048a25f9cdbb4e86feefb9013e05bac8347a4d7 is part of this task.

Intial DTO's and required entity changes made for supporting weapons detection as a machine learning service


More closely match Axum + Actix Leptos server implementation

Why is this needed?

More closely match how optional params are handled in Axum and Actix  server implementations
Match route context implementation in Axum server

Testing

Tested with a Spin app using local path dependency on leptos_wasi and it still works as expected + deploys to Spin cloud

@raskyld @benwis I appreciate reviews when ya'll have time!

LGTM!

LGTM!

Thank you both for the quick reviews! I don't think I have the ability to merge sadly


[CT-3518] [Bug] Retry should not print out warning when state path is not specified

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
We decided to add a warning when --state path == --target-path https://github.com/dbt-labs/dbt-core/issues/8160
This would cause dbt core raise a warning in retry when dbt tries to load the previous state. But actually dbt do not need to load all previous state in this case and should not raise such warning
Expected Behavior
We should only load the previous run results, and not print out a warning.
Steps To Reproduce
...
Relevant log output
No response
Environment
- OS:
- Python:
- dbt:

Which database adapter are you using with dbt?
No response
Additional Context
No response

@graciegoheen after looking at the code closely, I realized my previous description was wrong. Retry would still function in that case but would just always print out an irrelevant warning and load stuff that it do not need.
I updated the issue to make sure it is correct. Hope you don't mind.

Taking off sev 1 as it is only false alarm warning.


[update-checkout] Add support for swift-4.0-branch

Add support for swift 4.0 branch.

@swift-ci Please smoke test and merge


Fetching from Private NPM Repo Doesn't Work/Misleading User Feedback

When I try to pull down a repo from my org's private NPM repository with the following command:
NPM_CONFIG_REGISTRY="https://ghe.myorg.com/_registry/npm/@myorg/package" DENO_AUTH_TOKENS="ghp_tokentokentoken@ghe.myorg.com/_registry/npm/@myorg/package"  deno cache --node-modules-dir cache_test

I get this output
Download https://ghe.myorg.com/_registry/npm/@myorg/package/vary (59/116)

I have tried a few different permutations of the domain for DENO_AUTH_TOKENS. I can verify that using my token in a header with cURL does in fact return npm package info.
At worst private package imports are broken. At best this user feedback via the CLI isn't helpful - Download is green, there's no troubleshooting information, and it looks as if the process is just hung.
Repro with this command:
NPM_CONFIG_REGISTRY="https://google.com" deno cache --node-modules-dir googs

Moved https://github.com/denoland/deno/issues/20195


Want to add full steps to clone and Pull request

Want to add full steps to clone and Pull request
these steps are missing
I am contributing from Gssoc24

Hi @Harleen-786,
thanks for your interest in contributing to our project College.ai. I am assigning this issue to you.


[DO NOT MERGE] ManagedInstance api for restoredDroppableDatabases (restoredDroppableManagedDatabases)

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

Automation for azure-sdk-for-node

@anuchandy , I've fixed the error and validation is now passing.

@dejandundjerski thanks for fixing the semantic error. I see "don't merge" in title, is it because you have more changes coming or service is not yet in production?

Service is not yet in production. ETA for production is 3 weeks.

Tested the change on stage. Returned results as expected:
PS C:\Users\dejandu> Get-AzureRmResource -ResourceId /subscriptions/8cfb8b62-bcd6-4713-89ad-18097f75cc5b/resourceGroups/clperftesting_sneu_rg/providers/Microsoft.Sql/managedInstances/stlazitest/restorableDroppedDatabases -ApiVersion 2017-03-01-preview
Name              : cdb15000,131793381718870000
ResourceId        : /subscriptions/8cfb8b62-bcd6-4713-89ad-18097f75cc5b/resourceGroups/clperftesting_sneu_rg/providers/Microsoft.Sql/managedInstances/stlazitest/restorableDroppedDatabases/cdb15000,131793381718870000
ResourceName      : stlazitest/cdb15000,131793381718870000
ResourceType      : Microsoft.Sql/managedInstances/restorableDroppedDatabases
ResourceGroupName : clperftesting_sneu_rg
Location          : northeurope
SubscriptionId    : 8cfb8b62-bcd6-4713-89ad-18097f75cc5b
Properties        : @{databaseName=cdb15000; creationDate=2018-08-03T08:51:39.463Z; deletionDate=2018-08-21T15:16:11.887Z}
...
PS C:\Users\dejandu> Get-AzureRmResource -ResourceId "/subscriptions/8cfb8b62-bcd6-4713-89ad-18097f75cc5b/resourceGroups/clperftesting_sneu_rg/providers/Microsoft.Sql/managedInstances/stlazitest/restorableDroppedDatabases/cdb15000,131793381718870000" -ApiVersion 2017-03-
01-preview
Name              : cdb15000,131793381718870000
ResourceId        : /subscriptions/8cfb8b62-bcd6-4713-89ad-18097f75cc5b/resourceGroups/clperftesting_sneu_rg/providers/Microsoft.Sql/managedInstances/stlazitest/restorableDroppedDatabases/cdb15000,131793381718870000
ResourceName      : stlazitest/cdb15000,131793381718870000
ResourceType      : Microsoft.Sql/managedInstances/restorableDroppedDatabases
ResourceGroupName : clperftesting_sneu_rg
Location          : northeurope
SubscriptionId    : 8cfb8b62-bcd6-4713-89ad-18097f75cc5b
Properties        : @{databaseName=cdb15000; creationDate=2018-08-03T08:51:39.463Z; deletionDate=2018-08-21T15:16:11.887Z}

This api is expected to land on Canary on 8/23 + two weeks for whole production

@anuchandy , @ravbhatnagar - what should be my next step with this request?

API has landed on canary, pilot and initial. Remaining on heavy and geo1 and geo2 regions.

@anuchandy , @ravbhatnagar - are we fine now to do the merge? thanks in advance, Dejan

@anuchandy , @ravbhatnagar , folks are we good to proceed with merge now? Thanks in advance, Dejan

@dejandundjerski merging this. you can update the readme when you are ready to release the SDK
cc. @jaredmoo

Automation for azure-sdk-for-js


version bump for v0.5.1 release

Codecov Report

Merging #161 (ce24f03) into master (db59a74) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #161   +/-   ##
=======================================
  Coverage   56.50%   56.50%
=======================================
  Files          20       20
  Lines        3235     3235
  Branches      479      479
=======================================
  Hits         1828     1828
  Misses       1288     1288
  Partials      119      119

Impacted Files
Coverage Δ

csep/_version.py
100.00% <100.00%> (ø)

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 db59a74...ce24f03. Read the comment docs.


Sign up page

Sign in and sign up pages are added and connected with main page

Hey! @Pushti-R I'm not able to find where you have linked Sign up with the website.

@ishitakapoor26 sign up is different when you click on sign in it will direct you to your main page
You have to open signup page differently
Should I link it somewhere?


Refactor notification for indexer

extract from #4001

Codecov Report

Merging #5111 into master will increase coverage by <.01%.
The diff coverage is 55.1%.

@@            Coverage Diff            @@
##           master   #5111      +/-   ##
=========================================
+ Coverage    37.4%   37.4%   +<.01%
=========================================
  Files         307     308       +1
  Lines       45552   45586      +34
=========================================
+ Hits        17037   17050      +13
- Misses      26060   26080      +20
- Partials     2455    2456       +1

Impacted Files
Coverage Δ

models/issue_comment.go
47.87% <ø> (-0.5%)
:arrow_down:

models/issue.go
48.1% <ø> (-0.05%)
:arrow_down:

models/pull.go
51.16% <ø> (-0.11%)
:arrow_down:

routers/repo/issue.go
32.04% <0%> (-0.11%)
:arrow_down:

models/issue_indexer.go
62.35% <100%> (-5.47%)
:arrow_down:

routers/api/v1/repo/issue_comment.go
58.06% <100%> (+0.54%)
:arrow_up:

modules/notification/notification.go
37.77% <100%> (+9.68%)
:arrow_up:

modules/notification/indexer/indexer.go
53.84% <53.84%> (ø)

modules/process/manager.go
76.81% <0%> (-4.35%)
:arrow_down:

... and 4 more

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 8db3bdc...a2545f5. Read the comment docs.