file upload request entity error

I want to upload a file of 100 MB on my redmine server.
But it is giving "Request Entity error".
I have set max file size to 500 MB in Redmine Setting.

Any help?

In assets/runtime/config/nginx/redmine
## Increase this if you want to upload large attachments
client_max_body_size {{NGINX_MAX_UPLOAD_SIZE}};

Searching source for NGINX_MAX_UPLOAD_SIZE
README.md
593:- **NGINX_MAX_UPLOAD_SIZE**: Maximum acceptable upload size. Defaults to `20m`.

assets/runtime/env-defaults
123:NGINX_MAX_UPLOAD_SIZE=${NGINX_MAX_UPLOAD_SIZE:-20m}

Thanks alot.

I modify the size in assets/runtime/env-defaults.
but dose not work.

I solved this issue by docker-compose.yml modification


[rmlui] Update to version 4.3

What does your PR fix?
Updates the RmlUi library to version 4.3.

Which triplets are supported/not supported? Have you updated the CI baseline?
Triplet support should be unchanged, tested x64-windows

Does your PR follow the maintainer guide?

Yes

All features are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

Thanks for the update!


Wiki: update entry for Core Features

Most of the features have changed since the first version of FXGL. The page needs to be updated to use the correct terminology, remove features that are not supported any more, add new features that have been introduced since last time.

Page updated to include some new features: https://github.com/AlmasB/FXGL/wiki/Core-Features-(FXGL-11)
Now need to group by module name, e.g. fxgl-entity

Completed.


Update Auth0 role IDs for staging when reseeding DB

 BUG: new user invitations do not work on staging


Use different response types

So, looks like openapi supports using a schema for error types, including different schemas for each error code.
See: https://swagger.io/docs/specification/2-0/describing-responses/
I can see we can add error descriptions as per: https://paperclip.waffles.space/actix-status-codes.html
Looking at the code doesn't look like we can do it but I'll ask anyway in case I missed it:
Is it possible to add a type to the error?
Can we specify a type for each status code?

@dunnock would you happen to know?

Ok so after having a better look, looks like it really doesn't exist.
Missing things:

ability to add error codes not just to an error type but to a handler
Why? If we have a generic error type, will all the possible error codes, then all handlers will report an error response which may not make sense for a given handler.
ability to associate an error type with the status code/description pair which gets added to the response, allowing us to send back errors with additional information in the body

Any thoughts?

I got a noddy example going with adding response codes to the handler. I wonder if that work in isolation, as in, not related to the api_v2_errors, or if that should work as a "filter".
Eg, if no responses are specified in the handler then you get all the error codes from your error type; if response codes are specified in the handler then those codes are the only ones from the error type that actually make it into the spec?

I've raised #307 to experiment. Here's how the error type works now:
#[api_v2_errors(
    code = 400,
    code = 401,
    description = "Unauthorized: Can't read session from header",
    code = 500,
    scheme = "RestError2",
    code = 404,
   // default_scheme = "ErrorFormat
)]
pub struct Error { ... }

And of course Error format must impl the schema:
#[derive(Apiv2Schema)]
pub struct ErrorFormat { ... }

The resulting response:
"responses": {"200":{"description":"OK","schema":{"type":"array","items":{"$ref":"#/definitions/Nexus"}}},"201":{},"400":{"description":"Bad Request"},"401":{"description":"Unauthorized: Can't read session from header"},"404":{"description":"Not Found"},"500":{"schema":{"$ref":"#/definitions/ErrorFormat"}}

👍

Would it be possible to do something like the overloads for Json like JsonCreated used in response part?

I haven't really used those Json types, but do you mean having an error wrapper type for the actual error, eg: Response<Json, Error> ?

Yes, something like that. If I've understood properly the Json and CreatedJson etc classes are basically type wrappers which provide more type information hints to the introspection engine.

In the error case it may need to be a union object of some type because of the possibility of returning one of several different codes

It should be possible to provide a macro that receives the codes/descriptions/schemes and creates a type similar to those you mentioned, eg:
    #[api_v2_error_overlay(
        400,
        description = "Sorry, bad request",
        code = 401,
        code = 403,
        schema = "PetErrorScheme1",
        description = "Forbidden, go away",
        500,
        description = "Internal Server Error",
        default_schema = "PetErrorScheme2"
    )]
    #[derive(Debug)]
    struct MyErrorOverlay {} // gets translated to struct MyErrorOverlay<T: HttpResponse + Ser + ...>(pub T);

    #[api_v2_operation]
    async fn echo_pet_with_errors(body: web::Json<Pet>) -> Result<web::Json<Pet>, MyErrorOverlay<PetError>> { }

The inner type must implement Serialize, HttpResponse and perhaps even Apiv2Error.

Here's my wip: https://github.com/wafflespeanut/paperclip/pull/307/commits/097de616bf7aa379812a369e39f7da8d916d8c8d.
It works, now we I just need to filter out the error codes.

The alternative macros: One for a specific type and another for a generic
macro_rules! api_v2_error_overlay_filter_typed {
        ($error:ident - $($status:literal)+ => $name:ident) => {
            #[derive(Debug)]
            struct $name(pub $error);

            impl std::fmt::Display for $name {
                fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
                    std::fmt::Display::fmt(&self.0, f)
                }
            }

            impl actix_web::error::ResponseError for $name {
                fn status_code(&self) -> actix_web::http::StatusCode {
                    self.0.status_code()
                }
                fn error_response(&self) -> actix_web::HttpResponse {
                    self.0.error_response()
                }
            }

            impl paperclip::v2::schema::Apiv2Errors for $name {
                const ERROR_MAP: &'static [(u16, &'static str)] = &[];
                fn update_error_definitions(op: &mut paperclip::v2::models::DefaultOperationRaw) {
                    $error::update_error_definitions(op);
                    for code in &[ $($status,)+ ] {
                        op.responses.remove(&code.to_string());
                    }
                }
                fn update_definitions(
                    map: &mut std::collections::BTreeMap<
                        String,
                        paperclip::v2::models::DefaultSchemaRaw,
                    >,
                ) {
                    $error::update_definitions(map);
                }
            }
        };
    }
    macro_rules! api_v2_error_overlay_filter {
        (- $($status:literal)+ => $name:ident) => {
            #[derive(Debug)]
            struct $name<T: actix_web::error::ResponseError + paperclip::v2::schema::Apiv2Errors>(pub T);

            impl<T: actix_web::error::ResponseError + paperclip::v2::schema::Apiv2Errors> std::fmt::Display for $name<T> {
                fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
                    std::fmt::Display::fmt(&self.0, f)
                }
            }

            impl<T: actix_web::error::ResponseError + paperclip::v2::schema::Apiv2Errors> actix_web::error::ResponseError for $name<T> {
                fn status_code(&self) -> actix_web::http::StatusCode {
                    self.0.status_code()
                }
                fn error_response(&self) -> actix_web::HttpResponse {
                    self.0.error_response()
                }
            }

            impl<T: actix_web::error::ResponseError + paperclip::v2::schema::Apiv2Errors> paperclip::v2::schema::Apiv2Errors for $name<T> {
                const ERROR_MAP: &'static [(u16, &'static str)] = &[];
                fn update_error_definitions(op: &mut paperclip::v2::models::DefaultOperationRaw) {
                    T::update_error_definitions(op);
                    for code in &[ $($status,)+ ] {
                        op.responses.remove(&code.to_string());
                    }
                }
                fn update_definitions(
                    map: &mut std::collections::BTreeMap<
                        String,
                        paperclip::v2::models::DefaultSchemaRaw,
                    >,
                ) {
                    T::update_definitions(map);
                }
            }
        };
    }

    api_v2_error_overlay_filter_typed!(PetError2 - 600 700 => PetErrorOverlay2);
    api_v2_error_overlay_filter!(-600 700 => PetErrorOverlay3);

@dunnock any input on whether this should be a proc-macro(https://github.com/wafflespeanut/paperclip/commit/097de616bf7aa379812a369e39f7da8d916d8c8d) or macro_rules macro (last comment)?


Travis: VADER_OUTPUT_FILE=/dev/stderr for neovim

Without this there is no output from Neovim on Travis.

Didn't know that, thanks!


Show cache key when logging compilation cache hits/misses

Fixes #22548

Example log lines (not in chronological order):
2024-07-22 11:56:03,008 - DEBUG - Writing jit_matrix_multiply to persistent compilation cache with key 'jit_matrix_multiply-fa478e4ead6882a4c14cc1366d30f8efc783165c9cb16536e025357428373990'
2024-07-22 12:24:05,601 - DEBUG - PERSISTENT COMPILATION CACHE MISS for 'jit_convert_element_type' with key 'jit_convert_element_type-92121ef34fa61ce74b4dad69c9ff3d93d790dbdf2ac0c7664ab8933c8cb46c27'
2024-07-22 12:24:05,834 - DEBUG - PERSISTENT COMPILATION CACHE MISS for 'jit_matrix_multiply' with key 'jit_matrix_multiply-fa478e4ead6882a4c14cc1366d30f8efc783165c9cb16536e025357428373990'
2024-07-22 12:23:09,163 - DEBUG - Persistent compilation cache hit for 'jit_matrix_multiply' with key 'jit_matrix_multiply-fa478e4ead6882a4c14cc1366d30f8efc783165c9cb16536e025357428373990'

@ayaka14732 Could you enable the tests to run here please?

I believe the CI failure is not related to my changes, as I'm seeing similar failures in other PRs.


Replace using of appseting.json with env variables for BE campaigns

Replace using of appseting.json for getting db connection string with environment variables.
Currently module campaign in BE reading db connection string from configuration which read it from appsetings.json. The PR replace reading of this JSON with reading env variables and build the string.

@dguguchkov is this PR still active?

Closing this in favor of a new repo where it's fixed


codeblock indentation on uninstall instructions

What does this PR do?
Fixes some markdown rendering in uninstall instructions.  A missing indent (may have been tab/space issue) was causing problems with a code block rendered on the instruction page.  Anyone copying code directly from the code blocks would end up missing 2 terminal commands from step 4.
What issues does this PR fix or reference?
NA
Is it tested? How?

View rendering after fix here: https://github.com/lglussen/devworkspace-operator/blob/149a2bbca13049eccd51ea4e1758485b886e69a2/docs/uninstall.md
Look at the step 4 commands.
Compare against current rendering: https://github.com/devfile/devworkspace-operator/blob/main/docs/uninstall.md
PR Checklist
NA - just a markdown tweak

@lglussen Thank you for catching this and submitting a PR :)

LGTM

@lglussen Could you please sign off your commit so that the DCO github action passes? i.e git commit --amend -s Then you can do a force push git push -f

/test v8-devworkspace-operator-e2e, v8-che-happy-path
(tests are required on all PRs :upside_down_face:)

The "happy path" test is unhealthy, but DCO is still required (signed-off-by footer on commit message). To apply this:
git commit --amend -s
<save+exit editor>
git push origin patch-1 -f

Sorry for walking away from this - I thought this went through.

/test v8-devworkspace-operator-e2e, v8-che-happy-path

Thanks @lglussen!


v3.2.2 on node: throws an error, "MathJax retry", when I try to convert math with a "prime" symbol

Issue Summary
I'm using mathjax 3 for node for the first time. I used to use mathjax-node (which wrapped mathjax v2 so it could be used on node) and now I'm upgrading my node application. I'm not entirely sure I upgraded correctly, so it's possible I'm doing something wrong.
The issue is that when I send a string of mathml that contains an ' symbol (intended to be a "prime" like what you use in geometry), mathml throws an error.
I looked in the code, and it appears that the code's intention is to load an external entity file if mathjax doesn't recognize a character, and waif for it to load. But instead, it throws an error. This seems like a bug to me. I don't see how I can do anything with the error object (which contains a promise) that's thrown.
Steps to Reproduce:

Use this mathML string:

<math xmlns="http://www.w3.org/2010/Math/MathML"><semantics><mstyle scriptsizemultiplier="0.87"><msup><mi>A</mi><mo>'</mo></msup><msup><mi>B</mi><mo>'</mo></msup></mstyle><annotation encoding="text/plain"/></semantics></math>

Pass it into await htmlDocument.convert(mathML, {})

Any other information you want to share that is relevant to the issue
being reported. Especially, why do you consider this to be a bug? What
do you expect to happen instead?
Technical details:

MathJax Version: 3.2.2
Node 16.13.1

I did the following after looking at the examples linked from the documentation. Let me know if I'm doing something wrong:
const {mathjax} = require('mathjax-full/js/mathjax.js')
const {MathML} = require('mathjax-full/js/input/mathml.js')
const {SVG} = require('mathjax-full/js/output/svg.js')
const {liteAdaptor} = require('mathjax-full/js/adaptors/liteAdaptor.js')
const {RegisterHTMLHandler} = require('mathjax-full/js/handlers/html.js')

async function convert (mathML) {
  try {
    const adaptor = liteAdaptor()
    RegisterHTMLHandler(adaptor)

    const htmlDocument = mathjax.document('', {
      InputJax: new MathML(),
      OutputJax: new SVG({
        fontCache: 'none',
      }),
    })

    const node = await htmlDocument.convert(mathML, {})

    return adaptor.innerHTML(node)
  } catch (error) {
    console.log('=====> I threw an error: ', error)
    throw error
  }
}

Supporting information:
This exact same mathML string works fine with mathjax 2.6.1 (wrapped by mathjax-node)
This is the error:
Error: MathJax retry
    at retryAfter (/Users/xxxxxxx/node_modules/mathjax-full/ts/util/Retries.ts:99:13)
    at replace (/Users/xxxxxxx/node_modules/mathjax-full/ts/util/Entities.ts:510:17)

MathJax sometimes needs to load additional javascript file (for example, when a less-used TeX command is first encountered, it may require loading the code to process that command, or when certain entities are used in MathML input).  Since loading additional code requires the browser to wait for that code to arrive, MathJax has to stop processing the expression until the new code is available.  MathJax uses the "retry" error as a means of stopping the current processing so that it can be restarted when the asynchronous loading of code has completed.  That restarting needs to be handled by code higher-up in the processing chain.
The htmlDocument.convert() command you are using is a synchronous function, not an asynchronous one, so it does not trap the retry error itself.  MathJax provides a wrapper function that you can use to wrap the synchronous functions so that they will be retried if there is a need to load other files asynchronously.  So you could use
const node = await mathjax.handleRetriesFor(() => htmlDocument.convert(mathML, {}));

if you want your code to be able to handle the asynchronous actions of MathJax.  If a retry error occurs within the function passed to handleRetriesFor(), it will wait for the code that cause the retry to load and then will run the function again, repeating that until no retry occurs, and then return the result.

ok, thank you. I'll try that.  Is this documented somewhere?

Is this documented somewhere?

Well, the documentation for the component-based use of MathJax is in pretty good shape, but there is very little for the direct use of the MathJax module files themselves.  The handleRetriesFor() function is used in a number of examples (see here, here, here, and here).
There is a lot that needs to be done for that documentation.


Added Newton's law of gravitation program

It can be used to find unknown values from  Newton's law of gravitation.

should've implemented divide by zero error (when denominator is zero) and should report zero when one of the arguments is 0. using  keyword 0 as a checker is bad code practice @Ujjawal179 .

Read the Readme.md file of Python .
Guideline how to add py file read it again
PR accepted @Ujjawal179
Need to Star this repo incase not done.
Just Update the required will merge to main
🚀🚀👍


Cannot detect change of array from checkbox v-model?

Version
2.2.6
Reproduction link
https://fiddle.jshell.net/whoisbigZ/ec11h6gj/2/
Steps to reproduce
I use deep watch to trace an object. See source code, it's very simple.
What is expected?
I hope the setter of v-model should notice deep watch. Why not?
What is actually happening?
But I found the vue cannot detect the array change from checkbox.

I know some caveats about detect the changes of array, but I just use v-model in checkbox.

You're falling under this caveat: https://vuejs.org/v2/guide/list.html#Caveats
You can use an object to overcome it (in many ways). Ex: instead of an array for d use an object: {0: true, 1: false} You can also have an array of objects: d: [{model: true}, {model:false}]

I know everything that @posva said.
But I use v-model to bind a element of array to a input tag with type is checkbox, I don't directly set any element of array,  It been done by vue's v-model.
In my demo, I use v-model to bind a element of array to other input tag which type isn't checkbox, the change can be detected.
Why checkbox is NOK! In this case, why v-model don't use Vue.set to set the element of array.
Maybe it isn't a issue, or feature request, PLEASE use vue.set when use v-model to bind a element of array to a checkbox.

I'm giving it a deeper look. I'm not sure it's actually intended

I think I have the same problem:
https://jsfiddle.net/dq90hkb8/
Vue 2.2.1

@maximelebreton Because storedValues.three isn't exist.
You can ref: https://jsfiddle.net/y43nse2o/1/


Added Link to discord

Description
Added link to navigation

This PR fixes #70
Notes for Reviewers:

Signed commits

[✔] Yes, I am a Member of DEVSTRONS'

Discord ID (e.g.: DEVSTRONS'#0238)

! Raj#1813

@Rajkumar-stackcoder: Thanks for your contribution! LGTM :partying_face:

Hi @amino19 ,
Sry to tell but i dont know the npm and after seeing site today
then i found that without npm it will not make change so pls do that.

You've added an extra en at the beginning of the index.html which broke the website 😒
I'm currently working on fixing it. Next time, kindly check before submitting a PR.


Adding new kv type in which encryption/decryption will be handled using rest api call to third party encryption/decryption service

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?
This change is to add capability to have ecnryption/decryption using rest api call to third party HSM device like IBM Data Power.
Additional context
Files were updated and tested locally in my environment by using below variables in helm values.yaml for unsealer:args
args: [
"--mode",
"k8s-with-restapi",
"--k8s-with-restapi-secret-namespace",
"vault",
"--k8s-with-restapi-secret-name",
"bank-vaults-with-restapi",
"--k8s-with-restapi-keyidname",
"testapp",
"--k8s-with-restapi-encryption-url",
"https://test-vault-seal-key:8447/test/api/vault/v1/crypto/encrypt",
"--k8s-with-restapi-decryption-url",
"https://test-vault-seal-key:8447/test/api/vault/v1/crypto/decrypt"
]
Checklist

[x] Error handling code meets the guideline
[ ] User guide and development docs updated (if needed)

To Do

Error handling were handled to some extent. Please review and suggest if it is missed at some places.
Please suggest if any user guide that need to be updated for this small change.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.aditya gupta seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@adityagu0910 Please sign the CLA.

@pbalogh-sa  I did sign CLA

i am closing this pull request as i made some commits with unknown user email address so not able to sign CLA for that


Changing mesos_user does not apply

I have running Mesos/Marathon cluster. I added following file to  /etc/mesos-master/acls
{  "run_tasks": [{
      "principals":{ "type":"ANY" },
      "users":{ "values":[ "marathon" ] }
    }, {
      "principals":{ "type":"NONE" },
      "users":{ "values":[ "root" ] }
    }]
}

and corresponding flag to Marathon --mesos_user marathon
Then I restarted both Mesos and Marathon, but Marathon tries to launch tasks as previous user.
It worked when I changed zk prefix.
Do I need to set up new Marathon instance to change user?

Not sure, only a wild guess, did you tried to restart ZooKeeper, too?

I've just tried that, but nothing has changed (I've got running cluster changed mesos_user and restart marathon, then restart zk). Is there any order in which I should do it?

So, you restarted all Mesos-master/slaves, Marathon and ZK, right?
@gkleiman Any ideas?

Yep, one by one.

The order I would do it is:
Stop: Marathon, mesos-slaves, mesos-master, ZK
Start: ZK, mesos-master, mesos-slaves, Marathon

@aldipower Thanks, It's working. Do you think it should appear somewhere in doc, list of options that require mesos/zk restart.

@aldipower @gkleiman I tested this solution I can't figure out why it is required. I mean every component needs to be stopped. I thought maybe slaves do not require restarts but then it was not working.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-2860. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Can you show a working demo with a third person controller?

Post a video or playable demo

MotionMatching.webm


New update broke compatibility with Powercord

Describe the bug
The most recent update for slate that was supposed to fix a varity of issues https://github.com/DiscordStyles/Slate/commit/7e5e649bb96e19ce338a8a97a606aea9ab269512
Has broken compatibility with powercord
To Reproduce

Install Powercord
Install Slate
Observe the lack of change

Screenshots
https://www.youtube.com/watch?v=ljscyzbFFeQ
Infomation (please complete the following information)
Discord channel: Canary
OS: Windows
Mod: Powercord
Discord language: English

The bug got fixed with update https://github.com/DiscordStyles/Slate/commit/393e5c4ca990fa90ae86bf61b0b989925ce0f98b


Add zsh completion for multi-nodes 'node ps'

ref. #26299
@thaJeztah Thx for the ping.
Signed-off-by: Steve Durrheimer<EMAIL_ADDRESS>

LGTM


Permalink Icon right on page

Describe the feature you'd like
I found this post on the permalink.  in itself a very good thing.  However, for the end user, not really intuitive.
https://www.bookstackapp.com/docs/user/content-permalinks/#:~:text=Simply%20select%20any%20block%20of,link%20with%20a%20single%20click.
Improvement idea 1: In the right area a link icon where you can create an external link.
Improvement idea 2: Switch the address in the browser URL to permalink via the configuration file.
Describe the benefits this would bring to existing BookStack users
Improvement idea 1: The operator will quickly find that a normal link often leads to problems.  he will find the link icon and see the tooltip text for the benefits of that link.  He will also notice that the link looks different and gives him advantages for external access.
Improvement idea 2: In a company, you can immediately avoid the operator copying a link from the browser that quickly no longer works after restructuring.
Can the goal of this request already be achieved via other means?
Via detours with the contribution to permalink described above.
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
0 to 6 months
Additional context
No response

Thanks for the request but this has already been discussed a few times, including within #2917 where you can find my thoughts on this.


Parameter Preview on Multiple Parametersets over same parameters fails partially

Hi,
Let's assume I have a function Get-Test with a dozen parameters, including the switch parameters $a, $b and $c.

Parameters $a and $b are both the sole members of ParameterSet "Test1".
Parameters $a, $b and $c are the sole members of ParameterSet "Test2".

Then in the console, I'll type this:
Get-Test -a -b -
and try using "CTRL" + "Space" to get your awesome preview of the options I have. Parameter "c" will not be in the list of options shown.
I'll admit that scenario is not all too common, but occasionally one of my functions ends up with this situation (I can define individual Output types per ParameterSet. PowerShell Studio can detect this correctly and provide the correct Primal Sense).
Not the most pressing of issues, but I thought I'd place it in the pipeline.
Cheers and thanks for all the awesome work,
Bosparan

This most likely has nothing to do with PSReadline, but I can't repro anyway, using:
function Get-Test
{
    param(
        [Parameter(ParameterSetName = 'Test1')]$a,
        [Parameter(ParameterSetName = 'Test1')]$b,
        [Parameter(ParameterSetName = 'Test1')]
        [Parameter(ParameterSetName = 'Test2')]$c
    )
}

Get-Test -a -b -

Closing as no repro.


RBAC error preview 3

Environment
Platform: AWS EC2
Kubernetes/Platform Version(s): k3s 1.23.4
Describe the bug
Kubernetes clusterrole "neuvector-binding-nvdlpsecurityrules" is required to grant delete,list permission(s) on nvdlpsecurityrules resource(s).
Cannot find Kubernetes clusterrolebinding "neuvector-binding-nvdlpsecurityrules"(kubernetes api: Failure 404 clusterrolebindings.rbac.authorization.k8s.io "neuvector-binding-nvdlpsecurityrules" not found).
To Reproduce
Steps to reproduce the behavior:

Install via helm
login

Expected behavior
No errors appear
Workaround (provided by FM)
Resolved by applying the dlp CRD YAML
kubectl apply -f https://raw.githubusercontent.com/neuvector/manifests/main/kubernetes/5.0.0/dlp-crd-k8s-1.19.yaml
Create the ClusterRole:
kubectl create clusterrole neuvector-binding-nvdlpsecurityrules --verb=list,delete --resource=nvdlpsecurityrulesand
Creating the ClusterRoleBinding with:
kubectl create clusterrolebinding neuvector-binding-nvdlpsecurityrules --clusterrole=neuvector-binding-nvdlpsecurityrules --serviceaccount=neuvector-system:default

I think this is resolved in b1.
If installing crd and core charts separate, make sure serviceaccount is the same in both.

I confirm this is resolved in b1.

confirmed using helm chart version v2.2.0-b1!! Thank you.


running tests in all browsers

to-do

[x] fix test errors in firefox with api customElements
[ ] enable CI to run tests in all browsers (chrome, firefox, and safari), using browserstack

tests broken on firefox because firefox dont support the api customElements v1
http://caniuse.com/#search=custom elements
So, in this PR, I will try offer a fallback
the library document-register-element probably works, but, their dont call methods from customElements v1, like

connectedCallback()
attributeChangedCallback()

the connectedCallback can be easiliy overwited, (see diffs in mn-input.spec.js:21).
now I need found a way to implement attributeChangedCallback

Coverage decreased (-12.9%) to 87.097% when pulling b2174729ace2e71a53015cfdbbb54d378dffacca on firefox-custom-elements into f51fbf2cb4a89f3feb3639b10e8a28b4c3f1428a on master.

@alencar13 approved?

I will remove the coverage, karma dont export coverage "yet"

Approved the first implement fix test errors in firefox with api customElements.

anyone else? if all approved, I will merge, after the current CI done

It's ok for me. You can merge without problems.


Why not using async fun ?

I was wondering if there was a particular reason for not using async fn in the LanguageServer trait since async function in trait are a thing now.

I guess this section aswer this sorry
async-lsp accepts &mut self for requests and notifications, and the former returns a Future without borrowing self. Requests borrows only immutable states and can be run concurrently, while still being able to mutate state (like snapshotting) during preparation.


I have an issue where I comme back to my server and process has failed

You have triggered an unhandledRejection, you may have forgotten to catch a Promise rejection:
TypeError: Cannot read property 'url' of undefined
at outstanding_bids.find.bid (/home/xx/xx/postpromoter/postpromoter.js:519:75)
at Array.find (native)
at /home/xx/xx/postpromoter/postpromoter.js:519:45
at /home/xx/xx/postpromoter/node_modules/steem/lib/api/transports/http.js:94:9
at
at process._tickCallback (internal/process/next_tick.js:160:7)

@JordanFAVRE I think this issue should be fixed in the latest version.


Ec2 custom policy

Coverage remained the same at 70.0% when pulling 09c402d420833a156a52c1ee7189b480a61f1eeb on swethanaretla:EC2_custom_policy into d111f3b8d80194443746fabc92167364239c4695 on MicroPyramid:master.

Coverage remained the same at 70.0% when pulling ce103e852520a5da7be9a8e0d1ed0929b6795e96 on swethanaretla:EC2_custom_policy into d111f3b8d80194443746fabc92167364239c4695 on MicroPyramid:master.

Coverage remained the same at 70.0% when pulling 097da4f4d2efaafac185085115820f0c46656621 on swethanaretla:EC2_custom_policy into d111f3b8d80194443746fabc92167364239c4695 on MicroPyramid:master.


Added a medium story link to FAQ

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #10229

Description

Updated answer to "How did Free Code Camp get started?" Link added to Medium story.

LGTM :+1:
Thanks for your first contribution to Free Code Camp :tada:


support for trivy v0.37.x - update scanner flag

it is require to bump trivy to latest v0.37.x and update command flags.
-replace -security-checks with --scanners

This will need release notes as a breaking change for people using a custom image unless they update their image to 0.37+

This will need release notes as a breaking change for people using a custom image unless they update their image to 0.37+

good point however  we can add support for old trivy versions as will , we did something  similar on different API change


⚠ Processors has degraded performance

In b9be5c3, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 43 ms

Resolved: Processors performance has improved in e957f4e.


mis-match in documentation and actual installation

It seems that the installation guide is no longer up-to-date with the current build:
https://egeria.odpi.org/open-metadata-resources/open-metadata-tutorials/building-egeria-tutorial/task-installing-egeria.html
Egeria is cloned from master and successfully build with maven. But when I was following the installation process, I noticed a few differences in the tutorial and the actual file structure:

there is no secondary omag-server* directory within the egeria-install/egeria-omag-2.4-SNAPSHOT/server folder. The jar file is located under server folder itself.
If one try to launch Egeria in egeria-install/egeria-omag-2.4-SNAPSHOT/server folder, a file not found exception would be triggered because java could not locate egeria-install/egeria-omag-2.4-SNAPSHOT/server/truststore.p12. The truststore.p12 file is located in the parent directory of server folder.
I have tried the following work around: a. copy truststore.p12 into server folder; b. launch the jar file from egeria-install/egeria-omag-2.4-SNAPSHOT folder as follow: java -Dloader.path=server/lib -jar server/server-chassis-spring-2.4-SNAPSHOT.jar.
Also, it appears that setting strict.ssl=false still tries to fetch truststore.p12.

Thanks.
Thanks

agreed - this should be corrected - feel free to create a PR, or if you prefer we can fix.
This is #3705 where there's a little more content. Unfortunately it's partly down to not completely finishing the work for #1403 - it should launch without error in my view, plus we need to ensure the documentation for configuring certs is adequate. The docker image we created will launch the jar without problem. i will try and look at this sooner.....

I've encountered this while automating the setup of an Egeria Integration Daemon configuration and it was costly time ways to resolve it. I realise the work is ongoing but perhaps changing the docs to move the following up to the parent (server) directory
from
java -Dserver.port=9443 -jar server-chassis-spring-2.6-SNAPSHOT.jar
to
java -Dserver.port=9443 -jar ./server/server-chassis-spring-2.6-SNAPSHOT.jar
could act as an interim fix.

Thanks for the comments.
Fixed the first two points with some minor clarifications in the docs (though I think we need to revisit them more broadly)
The strict.ssl makes a minor change to how certificates are validated in the spring based server.  (for self signed certs). However the code doesn't prevent the .p12 from being loaded - more significant changes would be needed for this.
For now I think it best to concentrate on ensuring it's clear how to start the server/document how to setup the certificates
We did consider some more custom code to setup the correct security context, but it added more complexity, and itself risk for error, vs sticking as close to standard java process behaviour as possible. If we really need to we can revisit.

Thanks for the comments.
Fixed the first two points with some minor clarifications in the docs (though I think we need to revisit them more broadly)
The strict.ssl makes a minor change to how certificates are validated in the spring based server.  (for self signed certs). However the code doesn't prevent the .p12 from being loaded - more significant changes would be needed for this.
For now I think it best to concentrate on ensuring it's clear how to start the server/document how to setup the certificates
We did consider some more custom code to setup the correct security context, but it added more complexity, and itself risk for error, vs sticking as close to standard java process behaviour as possible. If we really need to we can revisit.


Spelling Mistake???

Hi there, this looks like a very interesting container and I am going to be spinning it up and testing it.  One thing that first caught my eye is this
importatntservice:
is that spelled correctly or should it be
importantservice:
thanks

Thanks for reporting, fixed in b8a861f


Build idevicesyslog, idevicescreenshot, and iproxy binaries and all dependent libraries with macOS  arm64 and  x86_64 architectures

Building these binaries and their dependencies with both architectures will allow the binaries to run natively on either x64 or arm64 developer Macs.
./autogen.sh in the ios_usb_dependencies recipe is called here:
https://flutter.googlesource.com/recipes/+/refs/heads/main/recipes/ios_usb_dependencies/ios-usb-dependencies.py#276
via the resources sh scripts, like:
https://flutter.googlesource.com/recipes/+/refs/heads/main/recipes/ios_usb_dependencies/ios-usb-dependencies.resources/libimobiledevice.sh#21
I believe exporting CFLAGS="-arch arm64 -arch x86_64" env variables will configure and later build/make with both binaries.
See how the libimobiledevice GitHub action does this:
https://github.com/libimobiledevice/libimobiledevice/blob/c797c2af6f4e2e8ae93b68f1c41abab1579563db/.github/workflows/build.yml#L129-L138
https://github.com/libimobiledevice/libimobiledevice/actions/runs/4059613727/jobs/6987855555#step:9:22
Part of https://github.com/flutter/flutter/issues/69157

Blocked on the failing ios-usb-dependencies builder https://github.com/flutter/flutter/issues/135757


update copy on pricing page

Description

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

now that there isn't too much text for contact us, @Yndira-E what do you think of making this a heading with the Form below it instead of on the side?

Hi @Hasmin-AC this can be part of the design review I'll be doing soon, pricing page will be the first one 😉. So feel free to merge as is and we'll handle that in a different PR.

Since I'm the last one approving, I'm merging it now.


How to access alerts from the past?

Hi thanks for this great repo! Star from me :)
I see the python-red-alert.src.main() will log current alerts happening, but I want to get all the alerts that have happened in the last 4 months (and save them to a file).
Is that possible to do?

need to integrate an api, if there is any. will look into it.

I would really like to have such a feature! Please let me know if you are able to find a way to integrate this


Setup KyberDao repository

Main contracts:

KyberDao
KyberStaking
KyberFeeHandler
EmergencyFeeHandler

missing files:
daoFuzzTests.js.
stakingFuzzTests.js
I See many code changes in js files.
did u run preteffier?

Yes, alr ran prettier on all js files
will add daoFuzzTests and stakingFuzzTests

We should move kybernetwork contracts file to contracts/mock/kyberNetwork


Fix Windows code to work with non-native code pages

Fixes the Windows OS issue with some code failing to list some files where name is given in non-native code page.

Filenames may have up to 32K chars (and eben more bytes) if they use UNC names like "\?\UNC-SHARE..."

If I follow correctly, this makes the dir() function always return UTF-8 strings. If that is the case, this is a breaking change — what happens when returning files with the native codepage and it happens to not be UTF-8? Shouldn't we be doing this converting based on the current locale?

Windows uses utf16 to unicode API. And if you use unicode then you can use path names longer than MAX_PATH

Cant we have a function argument on all lfs functions for using old behaviour (native codepage paths) or new one (utf-8 paths)?
Perhaps less verbose would be a library runtime flag for swtching behaviour.

@hishamhm "ñ" character will be returned in utf-8 as a two byte string and in current implementation as one byte only, but it is not a breaking change because it is "always" returning the same two bytes for "ñ".

@Adriatic1 wchar_t name[260]; is used for file name without prepending path but full path length is 32760
according to
https://msdn.microsoft.com/en-us/library/windows/desktop/ee681827(v=vs.85).aspx

The original code has MAX_PATH buffer limit, so I guess I adapted to it (see typedef struct dir_data definition). Feel free to build upon the patch and add modifications for longer paths, I am low on free times nowadays...

I would be pleased to apply unicode to utf8 changes but only if @hishamhm agrees with that.

https://github.com/sonoro1234/luafilesystem/tree/unicode
is the ffi lfs version I am using for this now

I would be pleased to apply unicode to utf8 changes but only if @hishamhm agrees with that.

That would be a great addition, but I think the right thing to do would be to apply these changes only if the active encoding (e.g. detected via setlocale?) is using UTF-8. This way we could support UTF-8 and keep compatibility for those using legacy codepages.

Using unicode you have the benefit of not depending on code page (Microsoft recomends using unicode for getting access to characters in whatever code page)
And then, having translation between unicode and utf8, we can use utf8 as a convenient unicode replacement in the Lua side.

I understand that UTF-8 in the Lua side + Unicode functions in the Windows API is better.
My concern is with backward compatibility. If I understand the Windows side of things in LuaFileSystem correctly, the current behavior is that, if a user is using Windows-1251 (aka Latin1, ISO8859-1) in the system locale, and they give to lfs a string encoded using it (e.g. "olé" as 6F 6C E9), they get the expected file.
Could we add a flag to switch lfs to UTF-8 mode in LFS 1.x, and then make it the default in LuaFileSystem 2.0?

this is a test on windows7 with "olé" and other file with caracter outside code page:
first cmd dir
c:\luaGL\frames_anima\luajit_test\dir>dir
 El volumen de la unidad C no tiene etiqueta.
 El número de serie del volumen es: 182F-9872

 Directorio de c:\luaGL\frames_anima\luajit_test\dir

23/02/2017  09:30    <DIR>          .
23/02/2017  09:30    <DIR>          ..
23/02/2017  09:06                 0 asc.txt
12/02/2017  17:15                 0 letra_?_letra.txt
23/02/2017  08:54                 0 olé.txt
               3 archivos              0 bytes
               2 dirs  69.161.799.680 bytes libres

with this script
local sep = package.config:sub(1,1)
local dir = [[c:\luaGL\frames_anima\luajit_test\dir]]
lfs = require"lfs"
for file in lfs.dir(dir) do
    if file~="." and file~=".." then
        print()
        print(file)
        print(file:byte(1,#file))
        print(lfs.attributes(dir..sep..file))
        local handle,err = io.open(dir..sep..file,"r")
        print(handle,err)
        if handle then handle:close() end
    end
end

script output
asc.txt
97      115     99      46      116     120     116
table: 0x00321200
file (0x76072960)       nil

letra_?_letra.txt
108     101     116     114     97      95      63      95      108     101     116     114     97      46
116     120     116
nil     cannot obtain information from file `c:\luaGL\frames_anima\luajit_test\dir\letra_?_letra.txt'
nil     c:\luaGL\frames_anima\luajit_test\dir\letra_?_letra.txt: Invalid argument

olÚ.txt
111     108     233     46      116     120     116
table: 0x00329428
file (0x76072960)       nil

changing lfs for lfs_ffi (unicode)
asc.txt
97      115     99      46      116     120     116
table: 0x002b7490
file (0x76072960)       nil

letra_Ãó_letra.txt
108     101     116     114     97      95      199     162     95      108     101     116     114     97
46      116     120     116
table: 0x002bcd18
nil     c:\luaGL\frames_anima\luajit_test\dir\letra_Ãó_letra.txt: No such file or directory

ol├®.txt
111     108     195     169     46      116     120     116
table: 0x002bcca0
nil     c:\luaGL\frames_anima\luajit_test\dir\ol├®.txt: No such file or directory

1-lfs
lfs fails with letra_Ãó_letra.txt, lfs_ffi does not fail in any file.
so unicode seems better.
2-io.open is trickier:
io.open expects ascii extended with code page so:
io.open with lfs letra_?_letra.txt fails
io.open with lfs_ffi fails on letra_Ãó_letra.txt and ol├®.txt
with a function for utf8 to ascii extended olé.txt would succed.
a replacement of io.open unicode would succed with lfs_ffi but  would fail with lfs on  letra_?_letra.txt and with olé.txt (but olé.txt could be made to work with conversion function)

@sonoro1234 Thank you for the tests.
This confirm that simply adopting the UTF-8 conversion approach of lfs_ffi is not possible for lfs 1.x, because it breaks compatibility with io.open(). And even in lfs 2.x, I think most users will complain if the results of lfs.dir() are not directly usable with io.open() without a conversion step.
I read some more about the ugly world of Windows codepages, and my original plan wouldn't work because Windows does not support UTF-8 codepages, so we can't detect "UTF-8 mode" from setlocale.
I also noted that in the first script output it displays "olÚ.txt" instead of "olé.txt". This means the Lua interpreter is running with the DOS codepage (cp-850) rather than the Windows-1252 codepage (that dir used above). (This table helped me confirm this.)
I think (based on this) that if you put os.setlocale(".1252") at the beginning of the script, it would display "olé.txt".
Could you run another test, please? What happens if you run that script with lfs adding os.setlocale(".65001") in the beginning? Thank you!

os.setlocale(".65001") -> nil
so it is not working
os.setlocale(".1252") lets me see olé.txt from lua
I was wondering: Is io.open on linux being able to open any utf8 path?

perhaps provide lfs.open unicode-utf8 aware?

For folks finding this discussion from search results, start with the initial problem report in #56.

What is the current state of affairs with LFS and Windows?
As I understand it LFS on the Windows platform tends to use Windows 1252 character encoding in a US-centric environment which is a super-set of the the ISO 8859-1/Latin1 in that they both use a single 8-bit number to encode a range of characters that extend upward from the standard ASCII 7-bit (0-127) numbers although other parts of the World may use a different selection of characters for the range of numbers from 128 to 255.  UTF-8 uses the same numbers for the Unicode code-points as Windows 1252 but the way that the non-ASCII numbers (above 127) are encoded means that for characters with number from 128 to 255 (in fact all the way to 2047) use two bytes not one per characters.
The normal Windows i/o functions use whatever 8-bit codepage (mapping of character numbers 128-255 to characters) is currently selected but that is very fragile because if any values (above 127) are created by a different system/user for a file or path name the mapping between 8-bit values and characters - what string.byte() and string.code() use - can break either because a character in the original file/path name code page is not included in the end user's system or it is in a different place (in the range 128-255).
The only reliable way to pass non-ASCII file and path names to Windows i/o functions is to use the so-called wide_character versions that require/return UTF-16 arguments and which always use the same "number" (or sequence of numbers) to represent the same "character".
It is not possible to use UTF-16 with the current string library because the latter is predicated around 8-bit values for characters; it is possible to use UTF-8 with the current string library (for post Lua 5.1 versions) and for 5.1 and some prior versions, additional libraries can fill in the gaps in the library at those versions.
Q.E.D. LFS is broken for Windows for non-ASCII characters and fixing it means that previous code that sort of worked when fed/returning file/path names that used non-ASCII characters that were present in the selected code-page on WIndows because the system created and using those file/path name had the same encoding, will fail to work when the same data is read on a new version that uses UTF-8 internally to match how other OSes do it.  OTOH That file/path name data is more likely to be readable by those other systems that are already using UTF-8 internally.

I think it's rather simple and such solution is already used ib a lot of software including mine: library works with UTF-8. When it calls WinAPI, UTF-8 strings are recoded into UTF-16. When it calls Unix API, strings are used as is.

@SlySven as far as I can tell this is also how all file I/O in the Lua standard library works.
Should we go the direction suggested by @sonoro1234 and provide UTF-8-enabled replacements like lfs.open and lfs.remove?
I wouldn't be opposed to that in a LFS 2.0, but we'd need someone to contribute that code as I don't do Windows development myself.
Would we also need a lfs.toutf8 function for converting local-codepage strings into UTF-8 strings so that code that currently works using the local code page could have an easy path fpr converting? (Otherwise they would need some sort of WinAPI bindings defeating the purpose of LFS).
LFS is designed to be a portable interface. Should we ensure UTF-8 interfaces on non-Windows platforms as well? Most Linux systems currently run on UTF-8 but not all. Should a toutf8 function be available there as well, binding libiconv? That wouldn't be hard to do.

I'm particularly tied to Lua 5.1 because the project I code for cannot afford to move to a later minor version without breaking things for our current users who will have scripts and fragments of lua code that they may not personally be able to revise and we have given an undertaking not to break our own API in a non-backwards compatible way. That being the case and because we are aiming to achieve I18n for our next major version we have only recently started to include a recommendation of a UTF8 string module - although I understand that the equivalent is already in 5.3 if not 5.2 .
I am not sure what processes are needed to be available to transcode between UTF-8 and UTF-16 but it seems that the file i/o library will have to do this as close to the Windows calls as possible... it looks like luawinfile does this.

I recently discovered luawinfile. It's a fork of LFS that converts strings from UTF-8 to UTF-16 and uses the wide-string versions of the various file-related functions. It seems to work; I was able to open a file whose name contained non-ASCII characters. Perhaps it can be used as a guide for making luafilesystem Unicode-friendly on Windows.

I found that luawinfile uses Lua 5.3 functions and I gave not been able to backport it to 5.1 - I'm just hoping someone else has the skills (and inclination) to do it... :pray:

I created a fork that compiles under Lua 5.1, but doesn't quite work. 😬


language server diagnostic dialogs should hint at what the (number) is

Version: 1.55.1
Commit: 08a217c4d27a02a5bcde898fd7981bda5b49391b
Date: 2021-04-07T18:22:52.186Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0
Steps to Reproduce:

Install ms-azuretools.vscode-docker 1.11.0 (newer versions will fix this specific flavor which means you'd have to find your own language server alert)
Create a Dockerfile with the content from https://github.com/check-spelling/check-spelling/blob/ce3aff0bfdf1cbabeffecb5afc28cf6b1c719e25/Dockerfile#L17-L22
Hover over the squiggly line

Click View Problem

It isn't readily apparent what (34) is or why there's no space between it and dockerfile-utils.

I suppose I should define is. Both views already tell me what the warning is. They do not explain why they also show (34). Letting one hover over this or click to see a hint indicating that this is a language-server specific identifier for this warning category would be quite elucidating.
For instance, can I use the (34) to ignore the alert? I have no idea. Maybe I can, but only for certain language servers? These views are a good way to teach me what I can do w/ the content they're presenting.
Also, while you're in the area, if there was a way to cause dockerfile-utils to explain which extension(s) contributed it, that'd be really helpful. I eventually figured out that I wanted to file a bug against microsoft/vscode-docker, but it wasn't remotely obvious.

Does this issue occur when all extensions are disabled?: Yes/No
No (well, you need a language server, and I don't know if vscode has any built in)

This bug isn't about the language server (I already filed microsoft/vscode-docker#2865 about their contribution), it's about vscode's rendering of the report from the language server:
https://github.com/microsoft/vscode-docker/issues/2865#issuecomment-817340146
Fwiw, I currently use the reporter format for my GitHub Action, and at some point, I might look into making it available as a vscode extension, since I'm starting to live here, as do my colleagues.
GitHub Actions, afaict, do not render the identifier:
https://github.com/jsoref/examples-testing/commit/e4210b7459c477a3cbb04884d417823b7b34a8da

https://github.com/jsoref/examples-testing/runs/2316332417#step:3:21

Does this issue occur when all extensions are disabled?: Yes/No
No (well, you need a language server, and I don't know if vscode has any built in)

Visual Studio Code comes with built-in support for JavaScript and TypeScript so you can reproduce these "mystery numbers" without too much effort just by writing an incorrect JavaScript file.

This is information showing the source and error as provided by extensions. The pattern is <source>(<code>). I suggest to reach out to extensions individually, e.g VS Code has the ability to make the code portion a link for further explanation


Fix broken Validator:isValidMatch

Hello @vstelmakh, thanks for the response.
The current validation method along with its unit tests is quite weird. I'm using your library in my projects and I encountered some real-life examples where it gives incorrect results when tweaking unrelated options:
Example: matchEmails = true
<EMAIL_ADDRESS>-> true
https://example.com/ -> true
         example.com -> true

All correct. But set matchEmails = false and suddenly you have
<EMAIL_ADDRESS>-> false (as expected)
https://example.com/ -> true
         example.com -> false (wrong!)

Regarding the unit tests and matching "edge cases" with $this->getMatch(null, null, null, null) etc., the existing behavior is already incorrect. Example:
$this->getMatch('http', null, null, null) returns true. but it's obviously not a valid URL since it has no host. Note that Validator already receives strings that are formed like an URL and merely does some additional validation on them, so that's not the issue, same when we give it $this->getMatch(null, null, null, null) (because that would equate to an empty string).
If you want to be strict about it, you can add additional checking to isValidMatch method to verify it has all required fields present, but in my opinion it's not necessary.
Regarding matchByTLD, yes, you're correct. I inadvertently changed its behavior a bit and will fix that.

@vstelmakh ok, if this is the desired behavior, I've reduced my PR to just fixing broken handling of e-mails. Note that the original unit tests all pass despite the fix, which is kind of a bad thing.

@qqux I'm not able to reproduce this case:

All correct. But set matchEmails = false and suddenly you have
<EMAIL_ADDRESS>-> false (as expected)
https://example.com/ -> true
         example.com -> false (wrong!)

Here is a dataset I used for ValidatorTest::testIsValidMatch:
[
    [true, [], [], false, $this->getMatch(null, 'email', 'example.com', 'com'), false],
    [true, [], [], false, $this->getMatch('http', null, 'example.com', 'com'), true],
    [true, [], [], false, $this->getMatch(null, null, 'example.com', 'com'), true],
]

For me it works properly. How did you achieved your results?

Regarding the unit tests and matching "edge cases" with $this->getMatch(null, null, null, null) etc., the existing behavior is already incorrect. Example:
$this->getMatch('http', null, null, null) returns true. but it's obviously not a valid URL since it has no host. Note that Validator already receives strings that are formed like an URL and merely does some additional validation on them, so that's not an issue, same when we give it $this->getMatch(null, null, null, null) (because that would equate to an empty string).

Definitely cases mentioned above are not proper matches. As ValidatorTest is the unit test for Validator, there is no benefit to set complete UrlMatch objects. For each test case, only the required data is set. For example Validator expect match to be valid if there is scheme defined. Setting up full UrlMatch object will just blur the focus from the test case itself. That's why only scheme is set.
Take a look to ValidatorTest::getMatch also fullMatch, byteOffset and url properties are always omitted.
There is MatcherTest which takes care of testing how UrlMatch objects are created form text input.

@qqux I'm not able to reproduce this case:

Yes, I was writing from memory and I looked at the wrong parameter.
Set matchTLD = false, matchEmails = true and then you get this:
         example.com -> false (as expected)
<EMAIL_ADDRESS>-> false (wrong!)
https://example.com/ -> true

For me it looks like it behave the same as previous condition

@vstelmakh it's not the same. It would've been the same if the method returned true at the end.

Yes, that's expected behavior, as we can't be sure if this is an email or, for example, a filename<EMAIL_ADDRESS>And if the TLD check is disabled than it considered as false.
In this case only<EMAIL_ADDRESS>will match.
I know it's probably not very obvious. Maybe it make sense to mention this in the documentation :thinking:

In this case only<EMAIL_ADDRESS>will match.

It still won't because of https://github.com/vstelmakh/url-highlight/blob/7dffee7b2679d5d1ea98d0c3beaf316de3559e30/src/Validator/Validator.php#L58
😉

@qqux anyway, thanks for bringing this this topic out. I will think how to make it better.


Score matching

Hi devs!
Is there a way to control the threshold of score? For example, I would like to only proceed to specific QID when the score is above 20

Please refer to the implementation guide at https://docs.aws.amazon.com/solutions/latest/qnabot-on-aws/
More specifically check the section https://docs.aws.amazon.com/solutions/latest/qnabot-on-aws/modifying-configuration-settings.html with the ES_MINIMUM_SHOULD_MATCH parameter.

@dangollayan I believe score as determined by opensearch is only relative to the other qids identified in the current search. Its not a percentage or anything that can be compared between different searches. Hence there is no mechanism to configure a minimum score result. If keyword matching is enabled, the setting referenced by @fhoueto-amz could be used to control the minimum number of keyword matches required.

@dangollayan You can control the score thresholds when you using embeddings through settings EMBEDDINGS_SCORE_THRESHOLD, EMBEDDINGS_SCORE_ANSWER_THRESHOLD, and EMBEDDINGS_TEXT_PASSAGE_SCORE_THRESHOLD settings.
For more information, please Semantic question matching using text embeddings


Unable to Enable NATS as Messaging Service in LuckPerms REST API

I'm encountering an issue with enabling NATS as the messaging service in the LuckPerms REST API. Despite correctly configuring the LUCKPERMS_MESSAGING_SERVICE environment variable to nats and providing the appropriate address, the service fails to recognize NATS.
Environment Details:

LuckPerms REST API Version: [Specify version, e.g., latest]
Docker Compose Configuration:services:
  nats:
    container_name: nats
    image: nats:latest
    restart: unless-stopped
    ports:
      - "4222:4222"
    command: --T -VV
    networks:
      - shared-network

  luckperms-rest-api:
    image: ghcr.io/luckperms/rest-api
    environment:
      LUCKPERMS_STORAGE_METHOD: postgresql
      LUCKPERMS_DATA_ADDRESS: luckperms_db:5432
      LUCKPERMS_DATA_DATABASE: luckperms
      LUCKPERMS_DATA_USERNAME: user
      LUCKPERMS_DATA_PASSWORD: password
      LUCKPERMS_REST_HTTP_PORT: 4000
      LUCKPERMS_REST_AUTH: true
      LUCKPERMS_REST_AUTH_KEYS: ${SECRET_TOKEN}

      LUCKPERMS_MESSAGING_SERVICE: nats
      LUCKPERMS_MESSAGING_SERVICE_ADDRESS: nats:4222
    ports:
      - "4000:4000"
    networks:
      - shared-network
networks:
  shared-network:
    driver: bridge

Actual Behavior: The service fails to recognize NATS, and the following warning appears in the logs:
Messaging Service was set to nats, but nats is not enabled!
Messaging service 'nats' not recognised.

Please let me know if any additional details are needed. Thank you for your support!

Admittedly this is not in the documentation (oops - NATS was a pull requested addon, I guess the wiki updates were missed)
However, the NATS configuration is largely the same format as for redis and rabbitmq. here's an example from the tests:
https://github.com/LuckPerms/LuckPerms/blob/0fef481d480737528491fc0b3b8487eb1612f955/standalone/src/test/java/me/lucko/luckperms/standalone/MessagingIntegrationTest.java#L287-L289
I think if you set as follows, it should work
services:
  luckperms-rest-api:
    image: ghcr.io/luckperms/rest-api
    environment:
      LUCKPERMS_MESSAGING_SERVICE: nats
      LUCKPERMS_NATS_ENABLED: "true"
      LUCKPERMS_NATS_ADDRESS: "nats:4222"

yep, now everything worked, thank you


the influence of batch_size

  Hi, I run the code as: python train.py --config_file configs/VeRi/vit_transreid_stride.yml MODEL.DEVICE_ID "('0')" and don't change the arguments except  batch_size, because of limited GPU. I set batch_size as 12 and mAP for VeRi-776 dataset is 70.3%, 10% lower than the original.
  Is this normal or something wrong happened?

I think it is normal. Triplet loss needs proper batch size to learn discriminative feature. You can set batch to 32. It will not be so bad.

OK. Thanks a lot.


Setting support link does not change title of link

Internal Reference: SURE-4261
When a user sets a custom support link, we only change the link destination for the link shown on the home page - this is within a box called 'Community Support' under the File an Issue.

When a custom support link is set, we should change the title of the box to say 'Support' instead of 'Community Support'

on 2.6.5-rc2

Navigate to Global Settings --> Branding --> Support Links
Add in a URL. Save changes
Navigate to Home

- Navigate to Global Settings --> Branding --> Support Links
- Remove the URL. Save changes
- Navigate to Home
- Section is titles - Community Support


Update get-status-badge.md

Clarify that the anonymous badge access setting is only available for private repositories.
This confused me as I created a public repository and do not see the setting. Another person also mentioned this on #7074 asking why they can't see it. The only reason I was looking for it is because I wanted to be sure the badge would be visible to the public.

@RobertBColton : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @RobertBColton! I've sent a small change request to you.

@RobertBColton : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Got Next.JS and Prism to work together via Express

Addresses #3

Details for deploy on README
Removed ESLint and Prettier to make things simpler
renamed folders to respective functions

Since the folder changes made the commit a mess, here are the main files I changed:
https://github.com/MLH-Fellowship/0.3.2-team-3/pull/4/commits/2507eec78c838fbddf4e4b9be7f9004b819f1e41#diff-168726dbe96b3ce427e7fedce31bb0bc
https://github.com/MLH-Fellowship/0.3.2-team-3/pull/4/commits/2507eec78c838fbddf4e4b9be7f9004b819f1e41#diff-04c6e90faac2675aa89e2176d2eec7d8
https://github.com/MLH-Fellowship/0.3.2-team-3/pull/4/commits/2507eec78c838fbddf4e4b9be7f9004b819f1e41#diff-39c99e14ae3583582e17d154808e18cc
https://github.com/MLH-Fellowship/0.3.2-team-3/pull/4/commits/2507eec78c838fbddf4e4b9be7f9004b819f1e41#diff-341da547369e9651eee3b5767fcbc945


Update twitter.svg

new x logo

Hey @Sunnyy001, the icon needs to turn white in the dark mode and on hover it should turn blue like others. You may need to check out how this is working. You can get help from @akshaykaluse35


Updated to the latest QDK.

Thanks for making this update. It seems we didn't do this in April because we're going from the March release version (0.24.201332) directly to May.  LGTM.


HDDS-9008. Fix unchecked warnings in TableCache

What changes were proposed in this pull request?
Fix warnings in TableCache and related code caused by raw usage of CacheKey and CacheValue.
https://issues.apache.org/jira/browse/HDDS-9008
How was this patch tested?
CI:
https://github.com/adoroszlai/hadoop-ozone/actions/runs/5540872425

@ashishkumar50 @sumitagrawl please take a look

@adoroszlai, Thanks for fixing this, LGTM+1.

Thanks @ashishkumar50, @szetszwo for the review.


Fix linter errors

@herdigiorgi can you merge this one, so the PR on the httpx gets these linter fixes as well?

Yes, thanks! (:


Append slash to robots.txt disallowed urls

Hopefully will keep robots away from some more disallowed pages.
See discussion starting at https://groups.google.com/g/mo-developers/c/FwF4m2EarQw/m/oQGbv-GiAAAJ
Delivers https://www.pivotaltracker.com/story/show/177802249

Coverage remained the same at 93.014% when pulling 878c105d545db8082be4dd02a70a6a8811fc1957 on 177802249-robots_txt-slash into 1dbb61b3f0c1a6dc5479dddbcb783173b9425e4f on master.


how to use this

export channels that restrict saving content is my need,thanks ,but how to run?

thks but how to  export channels that restrict saving content

Send channel post to bot. If channel is private, send join link.

the bot is dead ? coz i've start it but no response from the bot

the bot is dead ? coz i've start it but no response from the bot

Not actually. Now days I'm busy. Can you explain problem exactly?


Bug: World renders blue

While in a space dim everything turns blue in a set area, however some areas are not blue.
https://youtu.be/pdNjynmgEpE
watch the first few mins, rest is junk

does this occur without the rest of the stuff in that pack?  Last I checked there were several mods in there that messes with world rendering in some way.

should have been fixed by lighting changes a few versions ago


ScannerRun: Store scan results by provenance

See individual commits.
This is still work in progress.

Replaced by the non-recursive / simpler re-implementation: https://github.com/oss-review-toolkit/ort/pull/7099


[BEAM-8165]Release-2.16.0 pass root as a param and keep default root

R: @markflyhigh

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Java_Examples_Dataflow PreCommit

Run Java_Examples_Dataflow PreCommit

Run Python PreCommit


Data to pass to ServiceWorker notifications ?

Hi, could you please explain more about how and what to pass to ServiceWorker notification ?
what I want to do:
I want to create a desktop notification when the tab or even the browser is closed.
I could not find any documentation or code example of that yet. So far I dig here, I realize that we must put something to ServiceWorker to make the magic available, so please advice.
thank you..

This is not possible atleast in a straightforward manner with the current implementation of Push.js.
You might want to look into service workers and server-side push notifications.


[BUG]: Bug in Stable-Diffusion Example, "Killed" when saving checkpoint

🐛 Describe the bug
I was trying to run the stable-diffusuin example, but found the program would be Killed at the saving process, Without any Error Message. Also found the config file "train_ddp.yaml" is OK. So maybe some bug in save_checkpoint, would you please fix this bug?
Environment
I'm using the Docker Image you provide "hpcaitech/colossalai:0.1.10-torch1.11-cu11.3"

Thanks for your issue, can you offer more detail such as the image?

I also get this bug when trying to save this model. This was in a conda environment using the provided environment.yaml and on Ubuntu 20.04. I ran multiple tests to try and fix it and noted that, even with batch size of one the training went from virtual memory usage 22 to 46 to 68 and then the process was killed. I noticed that using save_weights_only: True enabled saving to work but with the weights only I am unable to do sampling, nor could I continue training.

Hello, I also met the same problem. How did you solve it

we have update the stable diffusion to v2 (https://github.com/hpcaitech/ColossalAI/pull/2120), Now there is no problem of kill

@Fazziekey  But stable diffusion v2 has no pre-trained model, so is not useful. How to train with stable diffusion v1? I also met with this problem, the first epoch is normal , the second time to save checkpoints it is killed.

We have updated a lot. This issue was closed due to inactivity. Thanks.
https://github.com/hpcaitech/ColossalAI/tree/main/examples/images/diffusion


Get distribution from select

Is your feature request related to a problem? Please describe.
When using select it could be good to get the distributions of the different output.
Describe the solution you'd like
One could imagine something like select(options=..., name="john", return_distribution=True)
Then we could get it through lm["john.distribution"] or anything else.
Describe alternatives you've considered
Right now I don't know how to do that. I can try. I see how to get the logits for the generated text thanks to _cache_state but I don't know how to get the distribution for the not choosen one (except by forcing the LLM to output each options and compute it. I think this is the only way but this is heavy without a function to wrap all this code :D)

_cache_state["logits"] would be a vector of size 32000 lets say if you are considering llama2.
If I convert the options to tokens,
lets say John=32, Amy=27941, etc.
and then use these token IDs to index into logits, I would get a probability distribution over the options.
Question: The default temperature of the model is 0, so select should be returning the most probable option from the above distribution by default?
But I ran some experiments where the answer from select was not necessarily the same as the most likely one even with temperature = 0. Why would this happen?

Can you share an example @iamshnoo?
I think there is either a bug or (more likely) you're not accounting for token healing issues.
For example, if you run lm + This is a prompt ' + select(['dog', 'cat']), you need to look at the logprobs of ' dog' given 'This is a prompt', not only 'dog' given 'This is a prompt ' (notice the whitespaces)

Oh good point @marcotcr ! I was not accounting for token healing.
If I look at the probabilities of  ' dog' and ' cat' for your example, then I am getting 0.5 for each, which is expected at temperature=0 (uniform distribution). [As opposed to previously when for 'dog' and 'cat', I get different probabilities].
For reference, the code I am using to look at probabilities is the following:
# out_select is the output of select
lm = llama2 + prompt + f" {select(split_options, name='answer')}"
out_select = lm["answer"]

# out_sentence is the output of taking the max of the logits
# but note that probabilities are the same for each option,
# so out_sentence is just the first item of option_probs dict
all_logits = lm._cache_state["logits"]
split_options = [" " + o for o in split_options] # add a leading space to each option (undo token healing)
option_tokens = [tokenizer.encode(o) for o in split_options]
option_tokens = [o[1] for o in option_tokens] # tokenizer adds <s> to each option in the above line
option_logits = [all_logits[o] for o in option_tokens]
option_probs = torch.softmax(torch.tensor(option_logits), dim=0)
option_probs = [float(o) for o in option_probs]
option_probs = dict(zip(split_options, option_probs))
out_sentence = max(option_probs, key=option_probs.get)
out_sentence = out_sentence.strip() # strip the added space

Questions:

But if dog and cat are equally likely, then why does select choose one over the other? (i.e if guidance returns dog for the answer, then how can I justify that answer unless I have some difference in probability values compared to cat)
Also as noted in #449 , the select call is deterministic and does not change outputs even if I pass the temperature parameter to the model as a value other than 0?

@iamshnoo  I don't understand why you get the full distribution. It works only if the different options have different first tokens leading to the fact that choosing the first token is equivalent to choosing one of the option. Does this happen often?
I am not aware of exact tokens but you can imagine something like

option 1 = token_1 ...
option 2 = token_1 ...
option 3 = token_2 ...

In this case you need to generate every options and compute the likelihood for every option (not truly but you get my point I hope). Maybe I miss something but I really don't see how you can do it in one generation in every case.

@iamshnoo In option_tokens = [o[1] for o in option_tokens], the token at position [1] is a space. tokenizer.encode(' dogs')=[1, 29871, 26361]. The space is common to all options, which is why you are getting the same probability.


Not able to connect from 2 node to 1 node

I tried to build a graph for family tree, requirement is from father and mother i need to conenct to child(2 node to 1) using BuchheimWalkerConfiguration  algorith, but I'm not able to connect like that.

Hi @naaf-44 did you find the answer,?

Not yet sir I'm stuck in that, do you know any way for that
On Mon, Aug 30, 2021, 11:00 AM VengaiMovan @.***> wrote:

Hi @naaf-44 https://github.com/naaf-44 did you find the answer,?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nabil6391/graphview/issues/49#issuecomment-908041691,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASCQFYWMPDN6TX5ZZJ46VW3T7MJQ7ANCNFSM5CJDFLXA
.
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.

No, I'm also stuck on that. If you find any solution please inform.

ok

Keeping issue #51 open , thus closing this


Feature request: Ability for effect on the mount when mounted by a rider

Would it be possible to include a way for an effect to be applied to a mount while it’s mounted by a rider? This would allow a mount to benefit from effects in situations such as when using the Mounted Combatant feat in dnd5e.

So something like a new text field where riders can enter the effects they give their mount? Sure no problem.

Not sure if is something to put in this module there are to many differences between  Pf2e effect management and the other systems. A better solution maybe is to put some hook on the module to let the single system integrate some macro with it.

I will for now add something similar to the already exisitng riding effects that are applied to riders of certain mount. These  currently only work for Pf2e or with DFreds CE. There are allready hooks for "Mount" and "UnMount"

I have just released v1.12.0 which adds the token setting Mount effects


sync: from upstream master to master

Created by sync-fork workflow job to sync changes from upstream to downstream.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

17 of 54 (31.48%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 40.938%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/host/internal/sriov/sriov.go
6
10
60.0%

pkg/host/store/store.go
0
9
0.0%

pkg/helper/mock/mock_helper.go
0
10
0.0%

pkg/plugins/mellanox/mellanox_plugin.go
0
14
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
5604

Relevant Lines:
13689

💛  - Coveralls


npm install throws error

when doing npm install i keep getting errors about not being able to find some packages, current error points:
npm ERR! code E404
npm ERR! 404 Not Found<EMAIL_ADDRESS>
Tried installing through yarn, but got the same error.
edit: this only happens in mac, in windows 10 i was able to run npm install with no issues

this is old but if it is not solved can you share the logs.
But you can try this first:
npm config set registry http://registry.npmjs.org
and make sure that you are in the root directory

Updated the course, please let me know if you continue to have any issues!!


Dedicated API for uploading place files

Currently, I have to overwrite the default place file of an account using the uploadModel function. While it is sufficient, editing it's description or anything else will break it. uploadItem, as of currently, seems to be broken so I haven't used that either.
As of recently, i've switched to uploading places with rbxmlx, an XML format of rblx files, but it seems like uploadModel doesn't seem to accept rbxmlx. So, now I'm stumped. I'd really not like to make my own logic for this so i'd be very nice if noblox had some support for place file uploading.

Not sure what the API is for uploading places, but I suspect it is different. Could you find out what the API is?

Not planned


Open Accordion Elements Styling

Description
Hi, so i was wondering is there a way to style the opened elements of the accordion differently?
Thanks in advance

I can't understand what this question asks exactly.

the question is that if i open an element in the accordion, can i provide a custom styling to the open elements of the accordion which would not apply to the closed elements.

You can get the state of accordion using template slots https://ui.nuxt.com/elements/accordion#item. Or when the accordion opens there is a data-headlessui-state="open" html attribute added to it, you can use css selectors

Hi @shahsank,
@metkm linked correctly to documentation which you can accomplish what you want using the open slot (if I understood your question correctly). Since there has been no reply here, assume you got sorted with this 🎉  and therefore will close issue. If you have any remarks, please tag me and will re-open issue if necessary.


reverse proxy

i use nginx reverse proxy with ssl for the ipfs webui, and bind a domain for it,
the ipfs conf:
"API": "/ip4/<IP_ADDRESS>/tcp/6501"
"Gateway": "/ip4/<IP_ADDRESS>/tcp/6502"
on the Files tab, i can not download any file, the download url like:
http://<IP_ADDRESS>:6502/ipfs/QmRtUpjjJh3WavoYTBPXS2r7hUZuwvYYc5VvFc96k79Q6x
how can i change the url to my domain form

Did you ever find out how to apply this @wolfired

Did you ever find out how to apply this @wolfired

no

Hi - suggest moving this discussion to the official forums at discuss.ipfs.io to improve community visibility? Thanks!


Cleanup unused package scripts

Fixes #880. Removes several unused package scripts now that we've moved to Vite.
This is a nice step before lerna v6 which uses nx since nx config includes any script that can be cached. Just helps keep things clean.

That error seems to be from me adding roots to the unit test jest config. I added that to try to quiet jest's initial spam about duplicate mocks, but I removed it since it's not a big deal


minor doc add in odo.odo

It looks like the description of valid options for source had lopped off a line. Just added a reasonable conclusion, and a close parenthesis in the target description.

thx merging now ... those failures are not you .... probably a version change ... investigating now.


Add form fields dynamically

Hey,
it would be great to have possibility adding fields dynamically from code. Currently I don't see such functionality in source. It would be even better to have dynamic adding of another nested forms through template modifying.
I could take this task on myself if you'd like.

Any news on this topic?

Nothing to report beyond the "I don't have the time right now" in my previous comment.  You may have better luck with one of the projects listed in the README as alternatives.  Apologies again for not working on this, but unless someone wants to pay for the time I spend on it (the only way I was able to get it this far), I simply won't be able to do more for the foreseeable future.


Settings "search.include"

There is a setting "search.exclude", but not "search.include".
My problem is that search.exclude inherits from files.exclude.  I would like to be able hide linter configuration files like .prettierrc, while still being able to quick open or search in them.
{
  "files.exclude": {
    ".node-version": true,
    ".prettierrc": true,
  },
  "search.exclude": {
    "**/node_modules": true
  },
  "search.include": {
    ".prettierrc": true,
  }
}

This is very similar #55271 but it was closed in favor of #869.  #869 does not address this use case because the files.exclude inheritance can not be pattern matched against.
I would like to see either

the ability to disable search.excludes inheritance of files.exclude
a stand alone search.include setting

Please contribute to this discussion in https://github.com/microsoft/vscode/issues/869. Although your use case is different, the implications and implementation of a search.include should also consider the use cases that others have for creating more advanced file/search globbing. Thanks!
/duplicate https://github.com/microsoft/vscode/issues/869


Merge this repo with rules_nodejs

Because monorepo.
We do too much work to shuffle things around, set up duplicate dev tooling, etc.

/cc @buchgr
We might want to go even further someday and merge everything into the angular/angular monorepo.

I would not really like to have to fetch the angular/angular monorepo just to use rules_typescript.... Do you see this set of rules as mostly meant for angular?

If we merge them, we'd publish an artifact for you to fetch (at
@bazel/typescript), not depend on the source repository.
On Mon, Feb 26, 2018 at 11:33 AM Yohann Coppel<EMAIL_ADDRESS>wrote:

I would not really like to have to fetch the angular/angular monorepo just
to use rules_typescript.... Do you see this set of rules as mostly meant
for angular?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_typescript/issues/133#issuecomment-368620640,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC5I900wQusIpOjDuUk1CpX9oncATP4ks5tYwY-gaJpZM4RqDzZ
.

If we merge them, we'd publish an artifact for you to fetch (at @bazel/typescript), not depend on the source repository.
…
On Mon, Feb 26, 2018 at 11:33 AM Yohann Coppel @.***> wrote: I would not really like to have to fetch the angular/angular monorepo just to use rules_typescript.... Do you see this set of rules as mostly meant for angular? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#133 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AAC5I900wQusIpOjDuUk1CpX9oncATP4ks5tYwY-gaJpZM4RqDzZ .

Appreciate the work!
But I (and my company) want to use everything in source, not from published artifacts (whether or not this is justified is out of scope, we have decided to use things this way).
Currently rules_typescript can only be used when put under @bazel/typescript. If I put it anywhere else and modify local_repository in WORKSPACE to point to the new location, it would break.
I'd like to see this repository become usable both in source and as a published artifact.

To be more clear, this proposal was about governance of the projects (having integration testing between them, fewer CI/github repos to maintain, etc) and users would always fetch them as separate artifacts.
I agree with @pauldraper that it would be a smell if these were actually coupled though. We already have a layering violation where rules_nodejs knows to look inside the "typescript" provider for files. Having them in one repo might encourage more layering violations.
Ultimately the blocker for this proposal is that the typescript rules are maintained primarily in google, and the nodejs rules are primarily external, and our sync mechanism (copybara) has to have a single source of truth, so we cannot merge the repos.


Some README cleanup and extra hinting

There were a couple of phrases that needed rewording and some words that were misspelled or could use some highlighting.

Thank you! there's always need for someone with eye on details :+1:

Hehe, glad I can be of some help!


heatmap loss convergence

@xggIoU  Why the heatmap loss does not converge and what did you change? I am using DLA version and I had the same problem of convergence. I tried with mean_square_loss and focal_loss and still having convergence problem ? Some hints ?

Whether the loss converges mainly depends on the heat map produced. The main change is to ensure that the weight value outside the bbox is zero.

Thanks @xggIoU, I think if you use the adaptive gaussian raduis from cornernet which mainly calculate the radius based on the size of the bbox, it will never had this problem.


phf_map! on &'static [T; N] broken

I guess this is a result of issue #76 but it's no longer possible to use phf_map! with b"foo" syntax to get a phf::Map<&'static [u8; 3], T>. Using &'static [1, 2, 3] gives an ICE. [1, 2, 3] works but loses readability for ASCII byte blobs.
#![feature(plugin)]
#![plugin(phf_macros)]

extern crate phf;
extern crate phf_macros;

/* used to work, now type error */
static MAP1: phf::Map<&'static [u8; 3], usize> = phf_map! {
    b"foo" => 2,
};

/* it would be nice if this would work */
static MAP2: phf::Map<[u8; 3], usize> = phf_map! {
    *b"foo" => 2,
};

/* error: internal compiler error: Error constructed but not emitted */
static MAP3: phf::Map<&'static [u8; 3], usize> = phf_map! {
    &'static [4, 3, 2] => 2,
};

/* works */
static MAP4: phf::Map<[u8; 3], usize> = phf_map! {
    [4, 3, 2] => 2,
};

@sp3d it's been quite a while since this issue was opened. Try the latest release version? I can't reproduce the error.

Closing this as not reproducible (phf_macros isn't even a compiler plugin anymore)


feat: add math/base/special/ahavercosf

Resolves #649.
Description

What is the purpose of this pull request?

This pull request:

resolves part of #649 by adding C implementation for math/base/special/ahavercosf

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #649

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

Open to reviews!!
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@Planeshifter ready for review!

made the required changes, do i also have to add @stdlib/number/float64/base/to-float32 in manifest.json? if so, how? ill make the same changes in #3076

@aayush0325 Thanks for the changes! Not needed as a dependency in manifest.json since it is not used in the C source files.

i see, @Planeshifter. will keep that in mind, thanks.


Peers Indicator Design Suggestion: Signal Bars instead of Row of Dots

@hebasto originally proposed a change to the Peers Indicator component here: https://github.com/bitcoin-core/gui-qml/pull/128
This is being opened to move the conversation on this possible design change into designer native channels.
The proposed change is to have the peers indicator be represented by signal bars instead of a row of dots.
hebasto implemented the design in this branch: https://github.com/hebasto/gui-qml/tree/220603-peers.ALT
Which upon running, presents the following:

@GBKS also rendered this within the full block clock, which is not yet finished:

I like the blockclock design idea as a peers indicator from @GBKS as it fits in well with all the other already designed elements. With regards to the wording used within the peers indicator I am just wondering if there would be an alternative word to use for "blocktime".
Could there be perhaps another word we could use such as:

Peers connection
Connection

Isn't blocktime reffering to something else, namely the time block is mined (avg. 10 minutes)? Example here or here .
If I understand correctly and blocktime here reffers to block height, and there's another meaning for blocktime than used here, then it may be confusing for people already acquainted with those names

Blocktime and block height are the same thing. But yes the bars are conveying peer connections and the number is the blocktime.
I have a bit of different suggestion for this part of the app. I think the blocktime / clock should stay focused on that single concept of blocktime and we should move this indicator to a 'network' icon in the top left that users can tap on and see various network based info. One detail being peers connected. The bars / dots don't convey the whole picture as the amount of peers you can connect to can vary and there is also inbound / outbound peer types. Other things could be if the user is using Tor / I2P, data your using, network type etc. Users could click on each variable to view more detailed info too (such as latency of peers).
This icon could flash or have some kind of animation when it is still connecting / syncing etc. There is lots of info relevant to convey to users and a few dots or bars really doesn't offer that depth.
If you check out the wallet Phoenix they have a network icon in the top left of their app when you log in which does something similar to what I've mentioned (says if you lightning peer is connected, if you electrum is connected, Tor if enabled etc).

We have an onboarding screen that explains the block clock before users see it, to avoid confusion.

I think there's a good amount of depth that can be put into the dots. Seems to work just fine for wi-fi and cell connectivity on smart phones.

The peer connection dots should be tappable and show an overlay with details, like this (content is not right, but this type of modal). I am not a big fan of what Phoenix does, it draws too much attention to the connectivity.

Similarly, tapping the Blocktime could potentially toggle through different information (if we can identify things people would want to see there).
There's also the concept of widgets, which are just a rough idea. We could let users pick a few widgets that are shown below the block clock and visualize different types of data. If someone really loves to see connections details at all times, this would allow them to do so.

Some more explorations of widgets, heavily inspired by smart watch UIs (it is a block clock after all).

Here's a sample visual for that peer connection overlay. Tells you in plain text what is happening, and you can pull up numerical stats if you want.

I will close this issue as it has gone stale. A new one can be opened if there's more interest in exploring this.


update_features() fails when simple feature collection is passed

Error when attempting to update a feature attribute value in an existing feature layer.
Reprex:
library(FIESTA)
library(arcgis)

auth_binding()|>
  set_auth_token()
#> Token set to environment variable `ARCGIS_TOKEN`

furl <- "https://services1.arcgis.com/hLJbHVT9ZrDIzK0I/arcgis/rest/services/Madagascar_points/FeatureServer/0"

fl <- arc_open(furl)

features_sf <- arc_select(
  fl,
  where = "text_field_1 = 'feature 2'",
)

features_sf[1, "int_field_1"] <- 1658641

update_features(
  fl,
  features_sf
)
#> $error
#> $error$code
#> [1] 400
#>
#> $error$message
#> [1] "Cannot perform operation. Invalid operation parameters."
#>
#> $error$details
#> [1] "'features' parameter is invalid"
#> [2] "Object reference not set to an instance of an object."

Created on 2023-04-28 with reprex v2.0.2

resolved in main!
auth_code() |>
  set_auth_token()

furl <- "https://services1.arcgis.com/hLJbHVT9ZrDIzK0I/arcgis/rest/services/Madagascar_points/FeatureServer/0"

fl <- arc_open(furl)

features_sf <- arc_select(
  fl,
  where = "text_field_1 = 'feature 2'",
)

features_sf[1, "int_field_1"] <- -10

update_features(
  fl,
  features_sf,
  match_on = "name"
)


Deployment documentation as GitHub action is not up-to-date

Documentation https://probot.github.io/docs/deployment/#github-actions is mentioning main.workflow instead of main.yml with current Action syntax, and not working anyway for typescript bot.

I would suggest we remove the GitHub Actions section for now. Pull request welcome :)

Working bot:

Workflow config in README
Dockerfile


Use port of the first found tab if tabs aren't explicitly defined in gradle command

Right now if you call ./gradlew -Pcmd='-d' gradle task doesn't know which port to reveal in Docker container and so it binds none.

Not a problem once #569 is solved.


Evaluator error when works with Ciphertext in vectors

Hi all,
I encounter the following error, and may I ask for any help/suggestions about this.
First, I tried to generate a series of plaintext and ciphertext and store into a vector, under the BFV scheme.
...
vector<Ciphertext> matrixA(4*t);
for (int i = 0; i < 4*t; i++) {
    Plaintext temp_plain(uint64_to_hex_string(temp));
    Ciphertext temp_encrypted;
    encryptor.encrypt(temp_plain, temp_encrypted);
    matrixA.push_back(temp_encrypted);
}

vector<Ciphertext> matrixB(4*t);
for (int i = 0; i < 4*t; i++) {
    Plaintext temp_plain(uint64_to_hex_string(temp));
    Ciphertext temp_encrypted;
    encryptor.encrypt(temp_plain, temp_encrypted);
    matrixB.push_back(temp_encrypted);
}

Afterwards, I use the evaluator doing substraction:
vector<Ciphertext> matrixC(4*t);
    for (int i = 0; i < 4*t; i++) {
        Ciphertext temp_subtracted;
        evaluator.sub(matrixA[i], matrixB[i], temp_subtracted);
        matrixC.push_back(temp_subtracted);
    }

And then I received the following error:
libc++abi.dylib: terminating with uncaught exception of type std::invalid_argument: encrypted1 is not valid for encryption parameters

which should be come from the function evaluator.sub(..., ..., ...).
Then, I dive into the library and found out that is_metadata_valid_for() return false in the parameter check, and the 4 entries of the parms_id array are all 0 instead of some non-zero values.
I test with the following case (no vectors involved), which works well and no error:
... (initialize temp)
Plaintext temp_plain(uint64_to_hex_string(temp));
Ciphertext temp_encrypted;
encryptor.encrypt(temp_plain, temp_encrypted);

... (initialize tempB)
Plaintext tempB_plain(uint64_to_hex_string(tempB));
Ciphertext tempB_encrypted;
encryptor.encrypt(tempB_plain, tempB_encrypted);

Ciphertext temp_subtracted;
evaluator.sub(temp_encrypted, tempB_encrypted, temp_subtracted);

May I ask for any help/suggestions? Thank you very much!

You have a mistake in how you are using std::vector. Namely, the constructor you are using initializes your vector with 4*t empty ciphertexts, but this isn't what you are hoping to do, because now you're appending to that with std::vector<T>::push_back, so you end up calling Evaluator::sub on empty ciphertexts. Hence, it complains the parameters are not valid for this operation.
Instead, for each of the three matrices, use:
vector<Ciphertext> matrix;
matrix.reserve(4*t);


unnecessary reloading of the file history view?

GitLens Version: 9.9.3

VSCode Version: 1.37.1
OS Version: macOS 10.14.6

disable Git: Auto Fetch in vscode settings.
open the File History view for a file already tracked in a git repository.
navigate/click through the history commits
the file history view is reloaded many times

My guess is such kind of reloading can be avoided when there's no new commits?
Another minor problem is the currently selected entry is lost upon view reloading.

I can't seem to reproduce this. Please enable output channel logging by setting "gitlens.outputLevel": "debug" in your settings. This will enable logging to the GitLens channel in the Output pane. Once enabled, please reproduce the issue and attach the log from the GitLens channel. Thanks!

I am closing this issue because it needs more information and has not had recent activity.  Please re-open this issue when more details can be provided.
Thanks!