net/http: ServeMux uses URL.Path instead of URL.EscapedPath(), leading it to treat %2F as a path separator

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
go version go1.6 darwin/amd64

What did you do?
http://play.golang.org/p/60StDlOnw8
It doesn't work on play.golang.org, but when you run it on local machine it results in:

2016/03/14 14:40:10 http: panic serving <IP_ADDRESS>:58546: /a/b

What did you expect to see?
I expected HTTP handler to get a request with URL equivalent to URL in original request, i.e. /a%2F%2Fb. Per https://tools.ietf.org/html/rfc3986#section-2.2 /a/b and even /a//b are NOT EQUIVALENT to /a%2F%2Fb

This happens because http.ServeMux uses URL.Path, which already contains wrong data /a//b, does some normalization on that and issues a redirect to /a/b, so user handler is never invoked during the initial request, only after this redirect.
I suggest one of these ways to fix this:

Make net/url percent-decode only unreserved characters, making it compliant with RFC3986
Use EscapedPath() in http.ServeMux, so it will see the data before net/url spoiled it

Both options (and probably any other) break API in one way or another.
Yes, I saw the response on https://github.com/golang/go/issues/8248 that this is documented behavior, but it directly contradicts relevant Internet standard, so it's worth to be changed.

This comes up repeatedly in various forms, but compatibility and not breaking old programs likely prevents us from doing anything here.  Any such proposal needs to be thoroughly investigated to consider its impact on existing programs.
/cc @rsc for any opinions.

I can hardly imagine an application that would rely on %2F to be decoded and used as path separator. In most cases you either don't care about those weirdos sending you requests with %2F or want it to be preserved as is and not used as path separator. But that's just my opinion based on a very limited sample.

You cannot just leave the %2F's in place, because that will encourage people to unescape the URL.Path again, which will lead to double-unescaping problems. And you do need to unescape the other things. You do want /%78yz to turn into /xyz. Also if you decide not to unescape %2F then you also can't unescape %252F, and because you're not unescaping that, you can't unescape %25252F, or %2525252F, and so on. It gets into a rathole pretty quickly. And if we started doing this all of a sudden then you'd have to worry about people seeing %2F and not knowing if it was a preserved %2F or a decoded %252F being process by an earlier version of Go. There are no good answers here.
Fundamentally, the idea of 'Path string' is at odds with the "internet standard". It should have been 'Path []string' and people who wanted the full path would have to strings.Join(u.Path, "") or something like that. But obviously we can't do that now.
For now I think we should just leave things alone. In the long term I think very limited wildcard support in ServeMux might help provide an API with the convenience of the single string form and still have the ability to pull out these kinds of mangled paths from individual elements. I wrote more about this at https://www.reddit.com/r/golang/comments/46bd5h/ama_we_are_the_go_contributors_ask_us_anything/d05lewi.
The workaround is to pass your own handler to Serve or ListenAndServe and have that handler pick off the %2F paths you care about before handing the rest to ServeMux. One reason the root handler is just an http.Handler instead of a ServeMux is so that you can do this when it is necessary. (Sadly, this unexpected usage of REST has made it more necessary than we anticipated.)

Also if you decide not to unescape %2F then you also can't unescape %252F, and because you're not unescaping that, you can't unescape %25252F, or %2525252F, and so on. It gets into a rathole pretty quickly. And if we started doing this all of a sudden then you'd have to worry about people seeing %2F and not knowing if it was a preserved %2F or a decoded %252F being process by an earlier version of Go.

True. However, RFC3986 defines a concept of equivalence, which goes as far as unescaping unreserved characters. Problem with URL.Path is that it maps URLs from distinct equivalence classes to the same string value, and any change in how things are unescaped for URL.Path is API-breaking change. So it's either URL.Path is left broken or someone's code exhibits behavior change; you can't avoid both of these because there may be existing code that relies on this particular brokenness.
It is possible to preserve equivalence while mapping URL paths to string values by unescaping only unreserved characters (and "%" is not unreserved, it's funny^Wannoying that "reserved" + "unreserved" sets don't cover full ASCII), and then having UnescapeNonUnreserved(string) string function to be used on separate path components. But it would be less fragile to expose parsed and unescaped path as []string.

https://www.reddit.com/r/golang/comments/46bd5h/ama_we_are_the_go_contributors_ask_us_anything/d05lewi.
Similarly, the decision to use unescaped URLs in the API has been overall very helpful: no handler has to worry about turning index%2Ehtml into index.html, for example.

I totally agree that unescaping (more precisely – normalization https://tools.ietf.org/html/rfc3986#section-6 ) is very useful and simplifies user code, but it was taken a bit too far and now it's too late to fix it in-place without breaking someone's code.

In the long term I think very limited wildcard support in ServeMux might help provide an API with the convenience of the single string form and still have the ability to pull out these kinds of mangled paths from individual elements.

In my case it doesn't mess the routing (i.e. it invokes the exact handler I want it to), but once it does, there's already no way to distinguish between / (path separator) and %2F (escaped "/" character). So unless you plan to bring preserving of escaped reserved characters (i.e. using URL.RawPath instead of URL.Path) along with wildcards, it won't help.

It should have been 'Path []string' and people who wanted the full path would have to strings.Join(u.Path, "") or something like that. But obviously we can't do that now.

How about adding URL.ParsedPath []string? Just like URL.RawPath was added in 1.5. It may require some considerations about what to do when it gets inconsistent with URL.Path and URL.RawPath, but I don't see any fundamental reason why it's not possible. Oh, and before joining path components you'd need to escape non-unreserved characters in each component, so it's a bit more complicated than simple strings.Join.
It's also possible to add it as URL.ParsedPath() []string (that will work off URL.EscapedPath()) and transition to use that instead of URL.Path in http.ServeMux and other standard packages.

The workaround is to pass your own handler to Serve or ListenAndServe and have that handler pick off the %2F paths you care about before handing the rest to ServeMux.

Yeah, for now I'm stuck with my own clone of http.ServeMux that preserves only %2F in the path.

I don't think we're going to add more complexity to url.URL now. RawPath suffices to preserve all the information one might need to derive or need control over. As of Go 1.5, clients in this (not everyday) situation can build what they need from the pieces provided. When it's time for a rethink of the overall big picture, we'll be sure to consider this. But I'd rather not add more piecemeal.

Okay, then what do you think about making http.ServeMux use EscapedPath() ? Cause my issue right now is that after passing through http.ServeMux information I need is not preserved even in RawPath.

I understand the issue you are having, but I don't think we can change
anything today. http.ServeMux is defined to issue certain redirects back to
the client and it is doing that (the request for /a/b%2f%2fc redirects the
client to /a/b/c; that second request is what your handler sees). If you
need to get in ahead of http.ServeMux and prevent the defined redirects,
the way to do that today is to define your own mux in front of it and pass
that to http.Serve or http.ListenAndServe.
On Wed, Mar 30, 2016 at 4:53 AM Maxim Ignatenko<EMAIL_ADDRESS>wrote:

Okay, then what do you think about making http.ServeMux use EscapedPath()
? Cause my issue right now is that after passing through http.ServeMux
information I need is not preserved even in RawPath.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/14815#issuecomment-203328530


Field published_by should be mandatory, to avoid sources not appearing in the SOURCES tab

It seems that the sources' name does not appear in the grapher chart's SOURCES tab. The field that does appear is published_by. But we have datasets that only have source name in the metadata, and not published_by, which leads to charts where the SOURCES tab actually does not mention the source.
So we could:

Make published_by a mandatory field in the sources metadata.
Let published_by copy the content of source_name if the former is empty.

I suppose option 1 implies changing several datasets, and it also implies more work when creating the metadata. So maybe 2 is a better option.
I think a better solution (for the future) could be to rename the source's name field short_reference and the published_by field full_reference. The mandatory field should actually be full_reference, and short_reference, if not given, would copy full_reference.

Discussed this in tech tea, @Marigold already submitted a PR. Seems like a pragmatic workaround for now, the data catalog project will probably bring more structure and clarity into this area of our data model in the medium term.


Documentation

[x] configure the gh workflow correctly so that it can find the tf files that it needs and populate the README
[x] README: write an example
[x] README: write a description
[x] add "description" fields to all the variables

[x] ec2-vslurm variables
[x] ec2-vslurm outputs
[X] ec2-vslurm/node variables
[x] ec2-vslurm/node outputs

[x] standardize the variable descriptions

documentation is up to date with commit 263606c9d35c4d0958dbd2ff7b05d9d92514427f
from now on i need to maintain the variable descriptions and the diagram
i'll need to write some more examples too, but that's not a task i can do now


some RESTt helpers

Description

Links to Tickets or other pull requests

Changes

Deployment

New Repos, NPM pakages or vendor scripts

Approval for review

[ ] All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

Notice: Please remove the WIP label if the PR is ready to review, otherwise nobody will review it.

Link to Definiton of Done
More and detailed information on the definition of done can be found on Confluence

just some helper functions to get user info per rest requests. Can be integrated in many re-usable steps.


tests: net: all: Fix COAP enablement in prj.conf

We had a left over from the old ZOAP lib in pjr.conf.  So enable the
COAP lib instead of ZOAP.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

Merging #7809 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7809   +/-   ##
=======================================
  Coverage   55.01%   55.01%
=======================================
  Files         483      483
  Lines       53946    53946
  Branches    10493    10493
=======================================
  Hits        29678    29678
  Misses      19982    19982
  Partials     4286     4286

Continue to review full report at Codecov.

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


Incorrect root? value after save

It seems the root? method incorrectly returns true after a model was updated/saved.
I will try to provide more details when possible.
I am working on running the specs, but it's not very straightforward as I am not using rbenv and it seems there is some issue with rails edge gemspec.

Update: doing model.reload after a save or update fix the issue (more of a workaround).

There are tests for root? before and after save, so without a worked example, I'm closing this.


plainToConstructor convert Date to ISO string

If the datatype of a plain json was already a date object, plainToConstructor will convert it to ISO String.
export class A {
    @Type(()=>Date)
    somedate:Date = new Date();
}

let json = {
    somedate: new Date()
};

let a = plainToConstructor(A, json);
console.log(a.somedate.constructor.name); // print String

fixed in 0.0.17


Refresh individual repos instead of org

Still need some more tests; waiting to assign reviewers until then.
dockstore/dockstore#3485
When user clicks Refresh Organization button, refresh each repo
individually instead of invoking the soon to be disappearing endpoint
that refreshes the whole org.
Need access to orgEntryObject, so passing that to component instead
of org/namespace.
Did not do tools, although did some of the prep work for it.
Added new method in WorkflowService to update a workflow, sort of
confusing (to me) in that there are two Akita stores that need to get
updated.
Also changed RefreshOrganizationComponent to extend Base and eliminate
some code duplication.
Test to ensures that clicking refresh org refreshes the individual
workflow.
Noticed that only one test was running in myworkflows.ts due
to an it.only; removed that and then added skip to all tests
that were failing. Created a followup ticket, dockstore/dockstore#3500
for followup on addressing skip tests.

Please create a ticket to follow up and remove the underlying endpoint afterwards

Please create a ticket to follow up and remove the underlying endpoint afterwards

dockstore/dockstore#3523


0.9.0: passing options to toc (for example opts.filter), and expose TOC helper to template

Because currently it is not exposed for use with {%= toc() %}, you can do only using html comment.
Also passing custom options to toc isn't possible, overwriting also is not possible, even through verb config - for example verb.toc.footer does not overwrite the default footer of toc.
<!-- toc('*.md', {filter: fn}) -->

closing since there is no toc logic verb. If necessary, create this issue on verb-readme-generator, wich uses verb-toc middleware. Options are already passed to the middleware, but it doesn't do the things you probably want. You might better off creating a custom toc middleware or helper that does what you want.

Options are already passed to

And how to pass filter option? Anad what options "works".

And how to pass filter option? And what options "works"?

I can make sure options are passed to markdown-toc if that's what you mean. but that would be an issue for verb-toc


Form fields for union types

Assume one needs form fields for the following type Account:
type alias CompanyName = String

type Account
    = Private
    | Business CompanyName

I assume the correct form for the UI would be a radio selection between Private and Business, with a text input for the company name appearing if Business is selected.
What would the correct approach be with elm-simple-form?
The part which is unclear to me is how to set up the validation for have one field depend on another.

Gosh, I'm afraid I don't have any satisfying answer for this, sorry... Not possible to do dependant validations at the moment.
We could look at how Json.Decode does that, implementation should be similar.

I did something like this:
type alias CompanyName =
    String

type AccountType
    = Private
    | Business CompanyName

type alias Registration =
    { email : String
    , accountType : AccountType
    }

validateRegistration : Validation () Registration
validateRegistration =
    let
        company =
            "company" := oneOf [ string, emptyString ]
    in
        Validate.succeed Registration
            |: ("email" := email)
            |: (("accountType" := validateAccountType) |: company)

validateAccountType : Validation e (String -> AccountType)
validateAccountType =
    string
        |> map
            (\s ->
                case s of
                    "business" ->
                        \x -> Business x

                    _ ->
                        \_ -> Private
            )

This works but is not ideal, as it allows Business "" as a valid value.

Okey, you could try using andThen like in Json.Decode, because Validation's andThen follows the same logic:
http://package.elm-lang.org/packages/elm-lang/core/4.0.5/Json-Decode#andThen

This is the first thing I tried. However if the validation is like
validateRegistration : Validation () Registration
validateRegistration =
      Validate.succeed Registration
          |: ("email" := email)
          |: ("accountType" := string `andThen` accountType)

accountType : String -> Validation e AccountType
accountType s =
    case s of
        "business" ->
            succeed Business |: ("companyName" := string)

        _ ->
            succeed Private

then andThen shifts the scope of companyName under accountType (the errors include accountType.companyName: InvalidString). Is this intended? In the case of Json.Decode the "scope" is not shifted.

Okey, that's not intended, as fair as I can remember.

I made an example: https://gist.github.com/nmk/e9fd97dd1a62c04fc52545fec5a9b4ef

Thanks for the example. I'm writing tests first as this change will break things.

Oh... could you try adding parenthesis like this:
(("accountType" := string) `andThen` accountType)

otherwise andThen is applying on string, not on the whole field getter!

 That works and solves the problem nicely. It would be nice if we could get this in the documentation. I will try to get a PR in next week.

Just for completeness, here is a link to a working gist: https://gist.github.com/nmk/e9fd97dd1a62c04fc52545fec5a9b4ef


Bump serde_yaml to 0.9

Closes #980

hm, this fails on the stupid kubectl compatibility test that the empty string is a valid kubeconfig (because everything is optional).
not sure if that's actually true in yaml land or not, but ultimately we get a doc in the loop which fails when trying to deserialize it:
for doc in serde_yaml::Deserializer::from_str(text) {
        let value = serde_yaml::Value::deserialize(doc).map_err(KubeconfigError::Parse)?; //<- fail here
        let kubeconfig = serde_yaml::from_value(value).map_err(KubeconfigError::InvalidStructure)?;
        documents.push(kubeconfig);
}

and the Deserializer is some weird internal progress struct that implements Iterator.
Not entirely sure what's wrong here, but we can probably work around this somehow. Not today though :zzz:


Update spec to reflect lfs.url and lfsurl

Corrects some configuration documentation, reported in #239

Oh it looks like this was also reported in https://github.com/github/git-lfs/pull/201 but CLA issues prevent it from being merged. I'll merge this one and close out the other #201.


401 Unauthorized

Hey guys,
I'm using this plugin for http://syntaxcenter.com, and for one instance the plugin works great. Unfortunately if I try it on another computer or browser (instance), the server returns 401 on the GET request. Any ideas as to why this is happening?

Fixed the problem, forgot to exclude the autocomplete task from before_action :authenticate_admin! within my controller.


Beacon [Universal banner]: add usage rules

The following usage rules should pass:

Universal banner must always be above the masthead
Test for any content overflow. If there are content overflow, because the universal banner has a max height, the overflown content will not be accessible.

Hey team! Please add your planning poker estimate with ZenHub @annawen1 @ariellalgilmore @emyarod @IgnacioBecerra @kennylam


Input: Don't add margin-top when input has no Label

Description

When a Label is not set, a margin-top is not set. This fixes that inputs with no label won't align with other items.
This removes the margin-top from input text variants, but instead incorporates a new class (mud-input-{variant}-with-label), that adds margin top only if the input has a Label set.
fixes https://github.com/MudBlazor/MudBlazor/issues/8262

Before this change:

How Has This Been Tested?

Added a Viewer test to have all inputs affected by this fix in one place
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Nice one! Looks great from the light testing in my app.
cc @henon

For completeness, can you post a before picture that shows the same MudStack with misaligned text fields?

I have another question. When you have textfields with labels, textfields without labels and buttons they will still all align well?

This is a great question, and you're right,
because they will have different heights, they won't align right away (if you take the bottom line as base for alignment).

You could accept it as expected behavior because "they will have different heights" and the dev has two main options to fix it:
Fix this aligning using baseline and not center (But if you look close, the radio, checkbox and button are not aligned now).

Or Fix it with a mt-4 in each input that has no label. Which could be a good option because intentionally you want to add an input with label and an input with no label aligned (different "styles" in the same row).

Alternativity, we could change the approach and add a negative margin to the label itself (never add the margin to the input), and there will always align.
![image](https://github.com/MudBlazor/MudBlazor/assets/40799354/1d1eb0f7-8fa4-4fbd-b9e4-69ca95cafce3)
But you get this kind of problems

Which could be solved by the dev with a margin top to the input or to the Stack (depending on how is structure the layout)
But still, I don't think it should be how any component should work. Is like an intentional bug that the dev should "fix" when using it.

Ok, I think the solution we have now is the most reasonable of all the options.

Added to v7.0.0 Migration Guide #8447

@ralvarezing I think this broke MudAutocomplete visually. Could you check on your end?
v7:

v6:

Oooh yeah I think I know what happened... i'll breake it down
Almost all the inputs use the MudInput to implement the "field" itself, this MudInput is the one that implements the margins.
But it does not have the access to the Label that has been set from the parent (ex. MudTextField). So, when trying to build the classes, this evaluation !string.IsNullOrEmpty(baseInput.Label) is always false.
This is were this comes in:

And has to be set in all the inputs that uses this MudInput... So preatty sure I missed some (like the AutoComplete).
I'll see to get that fixed and before opening a new PR, double check that I don't miss any other

That should do it,
Is a really small PR but, for what I've seen, this should complete all inputs

@ralvarezing I think this broke MudAutocomplete visually. Could you check on your end?
v7:

v6:

This issue still exists in 7.0.0-preview3 with

@ralvarezing I think this broke MudAutocomplete visually. Could you check on your end?
v7:

v6:

This issue still exists in 7.0.0-preview3 with MudTextField

Sorry, can you provide some example in a playground or where do you have this issue.
I am not being able to reproduce this in the tag for the release 7.0.0-preview3 or later commits

@ralvarezing Noticed this in the switches in this DataGrid examples:
https://dev.mudblazor.com/components/datagrid#visual-styling

Before:
https://mudblazor.com/components/datagrid#editing

Do you think it's related to this PR?

And the sorting example

Thanks @ralvarezing, glad it's an easy fix.
I would love if you could look at #9009! Got semi approval here https://github.com/MudBlazor/MudBlazor/issues/8988#issuecomment-2118709268


Unable to have the readthedocs theme active

Hello,
Details

Project URL: https://gitlab.com/mindflow/current/
Build URL (if applicable): http://mindflow.readthedocs.io/fr/latest/
Read the Docs username (if applicable): albanc

Expected Result
I just want to have the documentation display the awesome official readthedocs theme
Actual Result
I have been trying various configurations during a whole day before opening this issue, without any success.
I have created a sphinx based documentation which displays nicely localy using the official RTD theme. But when I upload it to readthedocs.org (thru gitlab pulling), the documentation builds properly, but the RTD theme will never show up. It seems like there is no theme applied on the documentation.
I have tried switching the html_theme to 'default' as suggested in your doc, without success. I have also tried html_theme = 'sphinx_rtd_theme' and I have tried regenerating a fresh conf.py using sphinx-quickstart. Same result always applies.
Here is the conf.py : https://gitlab.com/mindflow/current/blob/master/docs/source/conf.py
The only specific stuff I can see in my conf.py is the following directive :

source_encoding = 'iso-8859-15'

But I assume the docs wouldn't build without it, as my accented caracters would raise many errors. Anyway it does build, but the default RTD theme won't show up. Either I am missing something trivial, or there is some bug on the server side.
Could you give a hint ? Thanks

I tested it locally by downloading the repo, installing all the common RTD packages and running make html and I can see the RTD theme properly. Marking as a bug.

Hello,
It looks like the issue has been solved. Thanks a lot for your investigations humitos.


Add BitKeep Wallet

Can you point me to documentation for Bitkeep's support for Solana?

Can you point me to documentation for BitKeep's support for Solana?

Document pace

Can you point me to documentation for BitKeep's support for Solana?

https://docs.bitkeep.com/dapp-development/connect/solana-connect
In order to be compatible with the existing Sol DApps, we directly adopted a scheme similar to MathWallet, which allows Dapps to connect directly in BitKeep and initiate transactions.


Proposal: add type PickPrototype

TypeScript Version:  2.1.5
Pseudocode (for now it's not works)
type PickPrototype<T, K extends keyof T['prototype']> = {
    [P in K]: T['prototype'][P];
}

Expected behavior:
This feature will allow "picks" class prototype, and this will be resolve problem like this.

The problem in your linked SO question is that MyClass is both a type and a value. What you want is to treat MyClass as a value and then take the type of that value. To do that use typeof as follows:
class MyClass
{
  one(c: boolean){};
  two(){};
}

declare const test: PickPrototype<typeof MyClass, 'one'>;

type PickPrototype<T extends { prototype: any }, K extends keyof T['prototype']> = {
    [P in K]: T['prototype'][P];
}

Note that you have to constrain T to something that has a prototype property.

Wow! It works! Thank you very much. I could mark your answer on SO as resolved question (if you want).
But perhaps proposal of PickPrototype does not close, as well?

Thank you, but I don't have an active SO account. You can post it yourself if you wish.
I don't think the TS team can do much more about it. The typechecking is working as it is, maybe they could include it in the stdlib, but I don't see that happening as the usecases are rather niche.
PS: Nitzan Tomer's pieces of advice are really good.


Predicted Vs Actual is not setup to work ?

Hi there,
I am trying to set up a regression performance dashboard that includes the predicted vs actual metric, just like what is shown in the docs here. However, looking through the codebase it seems the predicted vs actual metric is not fully implemented? There is only a metric file for it and not a calculation, monitor or analyzer?
For context, I have integrated evidentlyai with Prometheus and Grafana, so have this dashboard json file pointing at the regression performance analyzer.

Hi @Joshwlks, you caught us in the middle of refactoring as we move to the new API (where you can get a Report comprised of individual Metrics). All the metrics will be implemented there very soon.
You can still get the regression performance dashboard, including this plot, as shown in the docs, using the old Dashboards API and calling the RegressionPerformanceTab https://docs.evidentlyai.com/features/dashboards/generate_dashboards

Thanks for the reply! Which version of the repo do you recommend using to achieve what I am looking for?

Hi @Joshwlks!
Apologies, as I missed you specifically referring to the Grafana example.
I am afraid there is no easy way to pass this plot to the Grafana in the existing example. The thing is that Predicted VS Actual is not technically a "metric" (there is nothing additional that Evidently computes). Both "target" and "prediction" are passed to Evidently as inputs in the first place.
Evidently generates the Predicted VS Actual visualization in the Evidently HTML reports output, but it does not produce any specific JSON output that can be useful in real time.
If you use Grafana for visualization, you'd need to write some custom code to implement and pass this data (predictions and target) directly to Prometheus from your service, bypassing the Evidently service.


Add FN_ prefix to config options (ie API_URL --> FN_API_URL)

Doesn't everything else prefix with FN? This ones hidden and already causing confusion.
Other ones that'll need FN_ :
| DB_URL | The database URL to use in URL format. See [Databases](databases/README.md) for more information. | sqlite3:///app/data/fn.db |
| MQ_URL | The message queue to use in URL format. See [Message Queues](mqs/README.md) for more information. | bolt:///app/data/worker_mq.db |
| API_URL | The primary Fn API URL to that this instance will talk to. In a production environment, this would be your load balancer URL. | N/A |
| API_CORS | A comma seperated list of domains to enable [CORS](https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS) for | None |
| PORT | Sets the port to run on | 8080 |
| LOG_LEVEL | Set to DEBUG to enable debugging | INFO |
| DOCKER_HOST | Docker remote API URL | /var/run/docker.sock:/var/run/docker.sock |

Can we vote and if passed change these before people get used to the way they are now?
Related tickets that also discussed/implemented FN_ prefix:
https://github.com/fnproject/fn/issues/329
https://github.com/fnproject/fn/pull/299
https://github.com/fnproject/fn/pull/5

Extensions may need additional environment variables (in the same namespace). For example the metrics API extension requires something like PROM_URL. Should this be FN_PROM_URL or  FN_EXT_METRICS_PROM_URL?

DOCKER_HOST will need care as this is used by docker in env. But we can take this as FN_DOCKER_HOST and set DOCKER_HOST env ourselves in that fn server to isolate it.

DOCKER_HOST makes sense to leave raw, and in any case it's easy enough for a user that wants to complicate things and run 2 docker daemons on the same host to set this variable at runtime or in a config instead of exporting env vars. the rest seem like what every other config with env vars project seems to do.


Add PyPI to developer

PyPI is the main Python package repository. Access to it allows people to upload/edit packages used by many other sites and tools, which has serious security implications, yet they do not offer 2FA.

Thanks for the pull request @edmorley! :+1:


[Feature]  in the "header.html" file tag is convering whole header.

Problem
The  tag in the "header.html" file is covering the whole width.
Description
The  tag in the "header.html" file is covering the whole width. It navigates to http://creativecommons.org/ on clicking anywhere on the header which isn't correct. We should navigate to http://creativecommons.org/ only by clicking the logo.
Alternatives
To implement this all we need to change is:
In header.html

To

Implementation

[x] I would be interested in implementing this feature.

Kindly Assign This Issue to Me i will be happy to help.

Thanks for letting me know. I will close this issue.


🐛 [firebase analytics] Missing homepage in doc - gives -10 on pub.dev score

pub.dev shows 0/10 points: Provide a valid pubspec.yaml
Homepage URL doesn't exist.
under "follow dart file conventions"

It's likely that pub tried to reach the URL when the site was down for some reason.
The URL works now.

@neiljaywarner
The score has been updated and homepage url is working now, as also confirmed by Remi above.
https://firebase.flutter.dev/docs/analytics/overview/ is accessible.
Closing this as resolved.

Thanks!
....sent from my phone
On Fri, May 21, 2021, 2:50 AM darshankawar @.***> wrote:

@neiljaywarner https://github.com/neiljaywarner
The score has been updated and homepage url is working now, as also
confirmed by Remi above.
https://firebase.flutter.dev/docs/analytics/overview/ is accessible.
Closing this as resolved.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FirebaseExtended/flutterfire/issues/6200#issuecomment-845738895,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXBGYNEAV2QGWNPPSL2XI3TOYGFFANCNFSM45HQDTKQ
.


emit metrics for each completed batch

Proposed Changes

emit metrics when export batches are completed. These are tagged in such a way that we will be able to write alerts for individual export chains not making progress.

Release Note
Additional metrics for export batch creation.

/lgtm


How to disable Perfwatson2.exe from Visual Studio 2022

I've selected VS22 in the version selector but there isn't a VS22 version of this doc. The instructions don't work for VS22.

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

ID: d08893df-9565-a578-3dda-9fb5c7d3528c
Version Independent ID: 23d30ca6-71e7-aece-d032-9b0ea5c81f58
Content: Customer Experience Improvement Program - Visual Studio (Windows)
Content Source: docs/ide/visual-studio-experience-improvement-program.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @anandmeg
Microsoft Alias: meghaanand

@WilliamAntonRohm  thanks for the response but none of those resources show how to disable Perfwatson2.exe in VS2022. I'd like to update my SO post on how to disable  Perfwatson2.exe.. As you can see, that's a very popular SO thread.


Add test in compliments module for remotFile option

nothing fancy here, just a simple test after @khassel's changes to the test setup :-)

Codecov Report

Merging #2932 (7d1e823) into develop (8212d30) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2932   +/-   ##
========================================
  Coverage    63.82%   63.82%
========================================
  Files            9        9
  Lines          293      293
========================================
  Hits           187      187
  Misses         106      106

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Output scale not being used with te_gemm in FP8

Hey, I'm using the te_gemm function defined in the PyTorch extensions here, and I'm trying to apply a scaling factor to the output. My gemm inputs are in fp8e4m3 and the output is in bf16.
For the D_scale argument, I am passing in tensors like torch.tensor([4.0], device='cuda') but changing the value of the scaling factor has no impact on the output. Am I doing something wrong here? Or is the scaling factor only applied when the output is of a certain dtype?

Hello @snarayan21. Yes, the purpose of this argument is to be the scaling factor for the output when the operator is producing FP8. The cuBLAS API does not otherwise have a parameter for scaling of the output. There is a potentially close thing in alpha: cublas performs D = alpha * A * B + beta * C. Currently in te_gemm we do not give a way to specify alpha (and beta is set to 1 by the accumulate option which performs D = A * B + D).
What is the usecase you are interested in that would benefit from that scaling?

Oh I see. I think that since the model I have doesn't use bias terms, it would be nice to just specify alpha...but would I get the same result by just modifying the A_scale_inverse in the te_gemm function? As in, I could multiply the existing A_scale_inverse by alpha to get what I want? The model has a few scaling factors that I would like to fuse into the FP8 gemms, if possible.

Yes, if you don't have bias that would be possible - you just need to be careful to not overwrite the scale inverse given as an input and instead create a new tensor there to pass to the gemm.


Adding support for full netmasks instead of subnet numbers

Hoi tehmaze,
I've added support to ipcalc for calculating the subnet number /xx using the actual netmask, because this was something I needed :)
If you want you can pull this from my fork, I think it's a useful feature for others people too. Please let me know if I should make any changes before it is ready for pulling.
Cheers,
Edwin

Thanks, this has been fixed in commit 7d89574442fc9ab62a37ce81183e72235e7901ba also dropping the need to include the types module. If you want to check if the input is a string, use:
    if isinstance(text, basestring):
        print "yay string"


Consolidate Swift 2.3 & Swift 3.0 support

I noticed there are two separate 2.3 & 3.0 branches. Is it too cumbersome to use the Swift version preprocessor facilities to keep 2.3 & 3.0 support in the same branch?  For example:
#if swift(>=2.3)
    return .none
#else
    return .None
#endif

Seems untenable to maintain multiple branches for each new Swift version.  Also, keeping it in the same repo allows people who are gradually adopting 3.0 to also use SwiftCheck's 3.0 API when possible.  Thoughts?

I was/am going to make an issue for making an strategy for support going forward. The plist involvement complicates the preprocessor strategy a little bit.

Thinking about this more, I don't think that a preprocessor approach would be desirable. The changes to the standard library, for one, mean that we would need to have significant amounts of code in these blocks.

You're probably right, then is the code effectively frozen for Swift 2.2 &
2.3—barring any major hotfixes for 2.3? Doesn't make sense to try to
maintain both in parallel.
On Mon, Jun 20, 2016 at 3:54 PM TJ Usiyan<EMAIL_ADDRESS>wrote:

Thinking about this more, I don't think that a preprocessor approach would
be desirable. The changes to the standard library, for one, mean that we
would need to have significant amounts of code in these blocks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/typelift/SwiftCheck/issues/167#issuecomment-227250614,
or mute the thread
https://github.com/notifications/unsubscribe/AAbHOX0StwWo_9ZFc1XOTqapHxSObPhYks5qNu_egaJpZM4I5AK4
.

I think that the plan should be two releases once Xcode 8.0 and Swift 3.0/2.3 are official.
One with 2.3 support

I think that the correct path forward is to support 2.3 until the official release of 3.0.
The codebase works with 2.2 without issue when updated to 2.3. This means that we can officially support 2.3 right now without causing any undue stress. Once 3.0 is official, we 'move on' from 2.3 barring hot fixes.

The strategy is the same one Typelift has always had.  We are going to support Swift 3, not Swift 2.3. This framework will have one more compatibility release for 2.3 if it hasn't already then our support for legacy Swift will end entirely and we will continue work on the 3.0 branch until it gets merged for master.  Because we haven't found any showstopping bugs during the beta period, I don't see a reason to maintain an entire 2.3 branch or introduce a bunch of ifdefs to try and scavenge legacy support back.


Celery Queues in stock CKAN ?

By default CKAN creates single celery queue, ckanext-qa and ckanext-archiver use queue named bulk if no queues are given in paster command. Does these extensions actually work with stock CKAN ?
In opendata.fi we cherry-picked ckan commit from data.gov.uk which enabled defining multiple queus https://github.com/yhteentoimivuuspalvelut/ckan/commit/85ef19c73faa7e841e9e0b94ec1044dbb6fee389 but we do not actually want to support this method in our new instances.
Celery is deprecated in future CKAN but we need to get these extension work for now, so is this just a configuration error in our end or should these work with CKAN 2.5.2 ?

Does this alternative way of starting celery solve it? The command is in ckanext-archiver, which is required for ckanext-qa.
paster --plugin=ckanext-archiver celeryd2 run all -c development.ini

Hi,
that actually did solve, might be good idea to to document it to ckanext-qa too :)

Great to hear it. I've added a note in the readme now. Thanks :)


Daily algorithm receiving extra bar at the end of a backtest

In this simple algorithm, the number of bars emitted are 4 instead of 3:
    /// <summary>
    /// This algorithm is a test case to count expected data points with forex symbols at daily resolution.
    /// </summary>
    public class ForexDailyDataPointsRegressionAlgorithm : QCAlgorithm
    {
        private int _count;

        public override void Initialize()
        {
            SetStartDate(2013, 12, 10);
            SetEndDate(2013, 12, 12);
            SetCash(100000);

            AddForex("EURUSD", Resolution.Daily);
        }

        public override void OnData(Slice data)
        {
            foreach (var kvp in data)
            {
                _count++;

                var symbol = kvp.Key;
                var barTime = kvp.Value.Time;
                var barEndTime = kvp.Value.EndTime;
                var timeZone = Securities[symbol].Exchange.TimeZone;

                Log($"AlgoTime: {Time}, Symbol: {symbol.Value}, BarTime: {barTime}, BarTime(UTC): {barTime.ConvertTo(timeZone, TimeZones.Utc)}, BarEndTime: {barEndTime}, BarEndTime(UTC): {barEndTime.ConvertTo(timeZone, TimeZones.Utc)}, {kvp.Value.Price}");
            }
        }

        public override void OnEndOfAlgorithm()
        {
            // one point per day
            const int expectedDataPoints = 3;
            Log($"Data points: {_count}");

            if (_count != expectedDataPoints)
            {
                throw new Exception($"Data point count mismatch: expected: {expectedDataPoints}, actual: {_count}");
            }
        }
    }

Relevant portion of log file:
20180222 20:31:53.417 Trace:: AlgorithmManager.Run(): Begin DataStream - Start: 12/10/2013 12:00:00 AM Stop: 12/12/2013 11:59:59 PM
20180222 20:31:53.447 Trace:: FileSystemDataFeed.GetEnumerator(): Begin: 12/10/2013 5:00:00 AM UTC
20180222 20:31:53.842 Trace:: FileSystemDataFeed.Run(): Data Feed Completed at 12/13/2013 5:00:00 AM UTC
20180222 20:31:53.870 Trace:: Log: AlgoTime: 12/10/2013 12:00:00 AM, Symbol: EURUSD, BarTime: 12/9/2013 12:00:00 AM, BarTime(UTC): 12/9/2013 5:00:00 AM, BarEndTime: 12/10/2013 12:00:00 AM, BarEndTime(UTC): 12/10/2013 5:00:00 AM, 1.37569
20180222 20:31:53.873 Trace:: Log: AlgoTime: 12/11/2013 12:00:00 AM, Symbol: EURUSD, BarTime: 12/10/2013 12:00:00 AM, BarTime(UTC): 12/10/2013 5:00:00 AM, BarEndTime: 12/11/2013 12:00:00 AM, BarEndTime(UTC): 12/11/2013 5:00:00 AM, 1.375935
20180222 20:31:53.874 Trace:: Log: AlgoTime: 12/12/2013 12:00:00 AM, Symbol: EURUSD, BarTime: 12/11/2013 12:00:00 AM, BarTime(UTC): 12/11/2013 5:00:00 AM, BarEndTime: 12/12/2013 12:00:00 AM, BarEndTime(UTC): 12/12/2013 5:00:00 AM, 1.37933
20180222 20:31:53.875 Trace:: Log: AlgoTime: 12/13/2013 12:00:00 AM, Symbol: EURUSD, BarTime: 12/12/2013 12:00:00 AM, BarTime(UTC): 12/12/2013 5:00:00 AM, BarEndTime: 12/13/2013 12:00:00 AM, BarEndTime(UTC): 12/13/2013 5:00:00 AM, 1.374775

From a purist's perspective, I completely agree with this, but I'm wondering if it would just be confusing to users using daily data. It's a reasonable expectation that they would receive the daily bar generated on the final day of the backtest. From the purist's perspective, that final bar is technically emitted after the backtesting range since it's available at 00:00 on the 13th, but it contains data from the 12th.
Also, the first bar we receive contains data from the 9th, but is included because it's emitted at 00:00 on the 10th. Again, from the purist's perspective, this makes sense and is consistent, but might be unexpected to a user.
@jaredbroad -- thoughts?

Comment here; https://github.com/QuantConnect/Lean/pull/1606
Closing for now.


Redesigning the members page

I was thinking of redesigning the members page using card UI layout. How does this look?

If it's OK, I'll proceed.

Use 2px border radius

@studenton It's looks ok for me. :ship:  :+1:
@AbsoluteZero273  @anarchyrucks  What do you say ?

Looks great. Can you think of a good colour scheme too while you are at it? 😃

@anarchyrucks I was playing around with material design colors. I'll see what I can come up with.
Referring: http://flatuicolors.com/ and https://material.io/guidelines/style/color.html. See if you like something there.
Current site color blue and grey looks a bit dull.

Yeah, everything looks nice except that color seems a bit off

@AbsoluteZero273 The background grey?

yeah, probably that. I am not a color expert. So I really don't know what color looks good without hit and trial. :sweat_smile:

Same here. I play around with the chrome developer tools and check what looks good.

Same here :joy:


Add --skip-git to allow skipping git initialization within the newly created folder

Feature Request (minor)
Is your feature request related to a problem? Please describe.
I am trying to add an operator into an already existing project - thus already having the git repo initialized in the root folder.
Describe the solution you'd like
We could simply add a --skip-git flag into operator-sdk new command, so the git repo won't be initialized.

@petomalina The operator-sdk new command has a --skip-git-init flag. Let me know if you run into any problems with the flag, thanks.

Ahh, thank you @LiliC I updated the CLI from master and you are right. Closing this. Thank you :slightly_smiling_face:


dont autospawn appliance_console

This goes along with #1.
Don't automatically run the appliance_console.
Instead just give them a bash prompt and let them invoke the appliance_console.
We want to merge this at the same time / before we merge the changes to appliance_console.
This was pulled out of ManageIQ/manageiq#792

/cc @dmetzger57 @carbonin want to keep you all in the loop

feedback from @abelotti - we agreed that a more subdue login would be more professional
fun:
Welcome to the
                   _ _
  __ _ _ __  _ __ | (_) __ _ _ __   ___ ___
 / _` | '_ \| '_ \| | |/ _` | '_ \ / __/ _ \
| (_| | |_) | |_) | | | (_| | | | | (_|  __/
 \__,_| .__/| .__/|_|_|\__,_|_| |_|\___\___|
      |_|   |_|            _
  ___ ___  _ __  ___  ___ | | ___
 / __/ _ \| '_ \/ __|/ _ \| |/ _ \
| (_| (_) | | | \__ \ (_) | |  __/
 \___\___/|_| |_|___/\___/|_|\___|

For a menu, please type: appliance_console

vs
Welcome to the Appliance Console

For a menu, please type: appliance_console


Dedicated RPC method for liveness and readiness probe

Is your feature request related to a problem? Please describe.
Right now db initialization can cause a very long delay before standard RPC calls are responsive.

based on this article https://developers.redhat.com/blog/2020/11/10/you-probably-need-liveness-and-readiness-probes#example_4__putting_it_all_together
I think we could do something like they have in example
livenessProbe:
  exec:
    command:
    - /bin/sh
    - -c
    - "[ -f /run/nginx.pid ] && ps -A | grep nginx"
  initialDelaySeconds: 10
  periodSeconds: 5
readinessProbe:
  httpGet:
    scheme: HTTPS
    path: /index.html
    port: 8443
  initialDelaySeconds: 10
  periodSeconds: 5

but for readiness I would go with our RPC call taraxa_protocolVersion, so something like
curl -X POST \
-H 'Content-Type: application/json' \
-d '{"jsonrpc":"2.0","method":"taraxa_protocolVersion","params": [],"id":1}' https://rpc.testnet.taraxa.io

fixed by https://github.com/Taraxa-project/taraxa-node/pull/1027


packing error with pyinstaller

Hi rdbende
i tried creating a exe for my python code with sun valley theme . I am using auto-py-to-exe to create the exe file.  i have added the theme using collect data but on opening the exe file i get the following error.
Traceback (most recent call last):
File "Interlock.py", line 107, in
File "sv_ttk_init_.py", line 19, in wrapper
_tkinter.TclError: couldn't read file "C:\Users\X0132832\AppData\Local\Temp_MEI197282\sv_ttk\sv.tcl": no such file or directory

So the solution in #38 doesn't work for you.
Some Windows and Pyinstaller experts needed. I have no idea about this 🤷

@shandy7 I was able to make it work using this exact PyInstaller command on Windows:
py -m PyInstaller --onefile --collect-data sv_ttk CalculatorApp.pyw
This is the same solution as shown in #38 so maybe it won't help but I was having the exact same error as you before I tried this.

Same here #38 doesnt works for me

Make sure the sv_ttk not only installed in the specific venv, but also your global python environment

Try adding the package to your app.spec file e.g. in the datas=[] list

Oh，You need to add this parameter after pyinstaller:--collect-data sv_ttk

If you're installing sv_ttk inside a .venv, then you also need to make sure that pyinstaller is installed and launched from within the same .venv, along with all other packages that you need for your application.
Never launch pyinstaller as global, that will take ALL packages that you've installed to your system, and package it with your app (depending on how good the auto trimming is, might take a good chunk of it away though, but still ill advised to run as global).


KAFKA-10033: Throw UnknownTopicOrPartitionException when modifying a non-existent topic's config

Committer Checklist (excluded from commit message)

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

@cmccabe this is ready for review.
The test location doesn't quite fit as it's in the module test dynamic config changes on the broker, but it's "near" the old error. Any ideas on where it might be better situated?

Thanks for the PR @bdbyrne, I'm curious why we don't consider fixing the AdminZkClient#validateTopicConfig thrown exception type, but fix its caller instead. And in fact we have two callers for that path as well, so maybe a more comprehensive fix is to fix AdminZkClient IIUC.

Hi Boyang -- there is a fix for the AdminZkClient at https://github.com/apache/kafka/pull/8715. This PR can be seen as an alternative/supplement in the case that changing the exception has other ramifications. I'm fine with dropping this if you feel the other PR is better.

@bdbyrne Can you rebase the PR?

ok to test

retest this please

KAFKA-9838

retest this please

retest this please

retest this please

ok to test


Support maxCol / maxRow simulatenously

Hi!
Is there a reason why max_cols and max_rows are not supported simultaneously?
I was hoping to achieve a similar result to:

Best,

Yes. By applying both max cols and max rows, it would then require lots of extra logic to figure out whether an item can still fit in the grid when it is added. The same result can be achieved by using max_cols and visible_rows which applies the same resizing logic, without adding any limitations to the number of items in the grid.

Hi @BTMorton,
Thanks for the reply!
I see you point.
We have tried to use visible_rows but it seems that the calculation is being done take the window as basis and in our case the grid doesn't occupy the entire screen (as it happens on your own example). Would you accept a PR where we would use elementRef.nativeElement instead of window?
Best,
José

Yes, I can't see there being too much issue with that

OK, I have added a new config property element_based_row_height which allows you to enable using the native element to calculate the visible row. It also no longer overrides the height with the max row as it assumes you are setting that yourself. Apologies this took so long to get to.


Zoom in/ out implementation

@andfanilo Thanks for this great library. I was trying to implement zoom in/out of the canvas and I got it to work great on canvas objects (see file attached). For some reason, however, the background image does not respond to mouse events.  Any chance you might know what is causing it?
DrawableCanvas_mod.txt

Hello @agporto ,
Are you able to make a PR, just to make it easier for me to view the code changes? We can close it afterwards if you then want to do a proper one.
Judging from https://stackoverflow.com/questions/16980273/html-5-how-to-zoom-canvas-background-image, It might be you need to redraw a scaled version of the image on the background canvas, though I didn't dig further
Anyway that looks awesome, hope it works out for you!
Have a nice day,
Fanilo

Closing for now, since I submitted the PR. #112


How to add a confirm prompt before leave?

We're having to add confirmation before leaving, I see that Navigo has the before, but it's not receiving event.
Is there an alternative solution for this?

I'll suggest to use the newest 8.0.0 beta version. There is a leave hook that accepts a callback.
router
  .on(
    "/about",
    (match) => {
      render("About");
    },
    {
      leave: (done) => {
        if (confirm("You are about to leave the About page. Are you sure?")) {
          done();
        } else {
          done(false);
        }
      },
    }
  )

When the user lands on /about and tries to go out will see that confirmation window. The router does nothing until you call the done function. (the full example here)
P.S.
Version 8.0.0 is not officially released but can be installed via npm install navigo@beta or yarn add navigo@beta. Migration guide is available here https://github.com/krasimir/navigo/blob/big-rewrite/CHANGELOG.md#migration-guide and the new documentation here https://github.com/krasimir/navigo/blob/big-rewrite/DOCUMENTATION.md.


Problem using Invidious as youtube URL

I am trying to play youtube videos using Invidious with the local=true parameter, which allows me to play them directly from invidio.us without having to touch googlevideo.com.  In order to do this, I set youtube_video_url in the config file to "https://invidio.us/watch?v=%s&local=true&autoplay=1&nojs=1". I also disabled DASH playback since this is not supported when playing directly from invidio.us. However, I noticed when selecting a video from youtube-viewer, I get output like this:
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------
* URL: https://invidio.us/watch?v=dQw4w9WgXcQ&local=true&autoplay=1&nojs=1
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------
                                                          =>> Rick Astley - Never Gonna Give You Up (Video) <<=

-> Channel   : RickAstleyVEVO
-> ChannelID : UC38IQsAvIsxxjztdMZQtwHA
-> Category  : Music
-> Definition: HD
-> Duration  : 03:33
-> Likes     : 3,810,399
-> Dislikes  : 169,044
-> Comments  : 591,721
-> Views     : 560,364,787
-> Published : 25 Oct 2009
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Playing: https://r4---sn-ab5l6n67.googlevideo.com/videoplayback?id=o-AEVETMi1wEVcUx7QQ0-ECbi1y83yEr4BZzeRTpauC89-&itag=18&source=youtube&requiressl=yes&mm=31%2C26&mn=sn-ab5l6n67%2Csn-p5qs7ned&ms=au%2Conr&mv=m&pl=24&ei=XKzdXK6MMJGkhgbwuKKwDA&gcr=us&initcwndbps=1782500&mime=video%2Fmp4&gir=yes&clen=15915462&ratebypass=yes&dur=212.091&lmt=1544406936248308&mt=1558031383&fvip=4&beids=9466588&c=WEB&txp=5531432&ip=<IP_ADDRESS>&ipbits=0&expire=1558053052&sparams=ip%2Cipbits%2Cexpire%2Cid%2Citag%2Csource%2Crequiressl%2Cmm%2Cmn%2Cms%2Cmv%2Cpl%2Cei%2Cgcr%2Cinitcwndbps%2Cmime%2Cgir%2Cclen%2Cratebypass%2Cdur%2Clmt&key=yt8&signature=928B21303FF6C3651ED81C47E57633BDE2B36887.15B887DD90DDF670063C8BDA0C736412C82598AD
 (+) Video --vid=1 (*) (h264 640x360 25.000fps)
 (+) Audio --aid=1 (*) (aac 2ch 44100Hz)

Somehow, between youtube-viewer printing that URL line and playing the video with mpv, the google video URL ends up getting passed into mpv. This doesn't seem to be the fault of mpv, since I can just dompv https://invidio.us/watch?v=dQw4w9WgXcQ&local=true&autoplay=1&nojs=1 from the terminal and it works as expected. Any idea why youtube-viewer is doing this? I can understand that youtube-viewer probably wasn't designed for this, but it doesn't seem to just be passing the URL directly to mpv like I would expect.

Hi. Please see: https://github.com/trizen/youtube-viewer/issues/227, https://github.com/trizen/youtube-viewer/issues/145, https://github.com/trizen/youtube-viewer/issues/109.

That worked. Thanks!


Updated packaging format

Closes #605.

@qingyun-wu thank you for catching this - installing in editable mode without setup.py requires a relatively recent version of pip. I added an update command to pip prior to the editable installation in the relevant section of the contribution guidelines.

Would it be better rather use pyproject.toml as setup.cfg is quite outdated... see: https://www.reddit.com/r/learnpython/comments/yqq551/pyprojecttoml_setupcfg_setuppy_whats_the/

It's a good time to revisit this issue since py3.6 support has been dropped. If pyproject.toml can be used without causing compatibility issues, it's OK to change.


Use Net::HTTPClientException instead of Net::HTTPServerException

As we have used Net::HTTPServerException and it is deprecated as per the link https://bugs.ruby-lang.org/issues/14688.
https://github.com/chef-cookbooks/audit/blob/584ed7af59d552c5bda1ee77f7ad9f6e06a7f54f/libraries/helper.rb#L47
So need to replace Net::HTTPServerException with Net::HTTPClientException

@vsingh-msys This looks valid. Would you mind submitting a PR for us to get this added? Thanks!

Okay sure @kekaichinose will create a PR.


Bump err-derive

It adds some features we can use, but no changes but the bump right now.

Coverage increased (+1.7%) to 71.043% when pulling 6e8bce92ac20e4e2e72f5d93dee8df9916a54d37 on rust-av:err-derive-bump into 37e3b59c0185a4e13bd67f010cf61569603128fb on xiph:master.


Add section for enabling/disabling components by groups

This PR resolves https://github.com/zowe/docs-site/issues/3932 by taking the table within that PR and inserting it in the page about how to manage components and extensions.
That page already had a topic on enabling and disabling components, so it was a small addition to just add a table suggesting which components are related and what they do.
With this, I was able to remove the older APIML-specific page, simplifying the doc slightly.

Do NOT merge this yet - I am discussing how best to clean references to the page that this PR replaces.


:bookmark: release 1.10.0

Description
Prepare release 1.10.0
Checklist

[x] My changes follow the contributing guidelines of this project.
[x] I have performed a self-review of my code.
[x] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my changes work.
[x] New and existing unit tests pass locally with my changes.

Coverage remained the same at 94.517% when pulling 821a9f2cdf12f55d4c6d224a307a592b18c4c1a8 on release/1.10.0 into 0083e8dd53576d0d81e89c1e2a0dbe382616b6ab on main.


How to get the connected iOS device model?

Hey @artehe ,
I have checked the library and we are able to get the OS version as 15.8 as like that. Do we have the option to get the device model such as IPhone 7 or IPhone 15 ?

Yes you can.
I made a change which adds getting the internal device model which comes across as a string like this "iPhone16,2" you should be able to use this like:
using (LockdownClient lockdown = LockdownClient.CreateLockdownClient(udid)) {
            string internalModel= lockdown.Product;
}

If you wanted to get the more friendly name you'd have to add a mapping of some kind to translate this. So from the example above the value "iPhone16,2" would translate to "iPhone 15 Pro Max"

@artehe ,
Thanks for the suggestion. We have identified with the same code and we got the mobile model identifiers. We have also got the list from this site. We are checking for the public API to get the device model based on the model identifier. Let me know if there is any available.

One thing you could do is just add another property to LockdownClient which is something like ProductFriendlyName. Then internally there's just a list of all the models somewhere and their nice names which are returned.
This sound reasonable to you?

@artehe ,
Yeah, I can do that. I have the list and we can print the ProductFriendlyName.

@artehe,
I have raised the PR for iPhone Models. Please take a look and let me know if there is anything to be updated.


Getting started trouble

I ran these commands to get started, but when I try anything mugen wants a missing 'bin' directory. Any ideas?
mkdir 3_try
cd 3_try
git clone https://github.com/scherroman/mugen.git
cd mugen
conda env create -f environment.yml
everything downloads...
source activate mugen
cd src/bin/
python cli.py --help
Recieve this error:
Traceback (most recent call last):
File "cli.py", line 9, in
import bin.constants as cli_c
ModuleNotFoundError: No module named 'bin'

Hey intoro,
This is an oversight on my part caused by not having mugen install as a proper package.
To fix this you should add the mugen package directory to your PYTHONPATH environment variable, so that python can find it.
export PYTHONPATH=$PYTHONPATH:PATH_TO_MUGEN_REPO/src

Depending on where you've installed the mugen repo on your computer, PATH_TO_MUGEN_REPO will look something like /Users/jeremy/Documents/github_repos/mugen


[RFC] template load performance, Blueprint.render_template

The loading of template tried too much times(count of blueprints), after setting with EXPLAIN_TEMPLATE_LOADING config.
Maybe Blueprint.render_template  is more effective.

In huge application, there are too much Blueprints, the performance is important.
An independent  module should always use independent template and Blueprint. Just like below in a view.py

mod = Blueprint('standalone', __name__, url_prefix='/evil', \
  template_folder='standalone/templates', \
  static_folder='standalone/static')

@mod.route('/')
def index():
    # add a function into Blueprint
    return mod.render_template('index.html')

@mitsuhiko Thoughts?
I personally would like to see a PoC and benchmarks that show any significant change in performance. I suspect that template loading is not an actual bottleneck.

I'd like to see a benchmark too.
But blueprint-local templates would be useful in any case. I'm currently using a slightly ugly hack (*) to have virtual folders for blueprints inside the merged template folders which means IDEs like PyCharm won't be able to link templates nicely since they don't know about it - with something builtin into Flask that'd probably change.
But even with blueprint-local templates it'd be important to keep the ability to access other templates (at least from the main app) from a blueprint template, e.g. to inherit from a base template or to include common macros - so using completely separate jinja environments most likely won't be an option.

(*) See the virtual_template_folder code in https://github.com/indico/indico/blob/master/indico/web/flask/wrappers.py#L173 for the hack I'm using.

For reference, other issues discussing blueprint / template handling:
https://github.com/pallets/flask/issues/1361
https://github.com/pallets/flask/issues/266

Is the issue that template loading in general is slow or just when the debug feature is enabled? I'm not entirely sure I understand the bug report.


gn: Use existing targets to manipulate icudtl.dat and V8 snapshots

Instead of repeating the same checks for icu_use_data_file and
v8_use_external_startup_data along with changes to sources and
deps in many different files, we can leverage the existing upstream
targets whose purpose is to do the exact same thing.

@heke123

LGTM


Review Data folder in Airlock Import Review VM Empty

When provisioning a VM to review import data the data is not in the review folder.

Resolved by #3215.
The storage account of Airlock's "import in progress" (stalimip) has 2 private endpoints, one to enable airlocks flow, and the other for the review VM. These 2 endpoints used the same private DNS zone, which caused the DNS entry to be overwritten when one of them was deployed/removed.
In this particular case, the DNS entry had the value of the first endpoint, causing the review VM to resolve to the wrong private endpoint IP.


Adds teosd.service to run teos as daemon

This builds on top of the work from the work by @decentralizedb and @jochemin in #138. It is mainly just re-arranging the code and readmes to fit https://github.com/talaia-labs/rust-teos/pull/138#pullrequestreview-1195570409, but they did all the heavy lifting.
Close #138

@jochemin @decentralizedb you may want to give this a look

Hey @jochemin @decentralizedb @lunaticoin, this is the only thing blocking v0.2.0, can you give it a try?

I tried this configuration, and it worked.
Just one comment, the lines:
After=bitcoind.service
After=network.target
It's equivalent to having:
After=network.target bitcoind.service
I suggest changing it because it's clearer to read putting both options on the same line.
For the rest, everything worked without any problem.

Addressed comments and rebased master


Added auto log rotation.

To address issue #66 I have added two features: auto log rotation and function mostRecentLogFiles().  The following outlines their usage.
The following properties are used to configure auto log rotation:
open var rotation = Rotation.none
open var rotationFileSizeBytes = 1024 * 1024  // 1M
open var rotationFilesMax = 1
open var rotationFileDateFormat = "-yyyy-MM-dd'T'HH:mm"
open var rotationFileHasSuffix = true

Auto log rotation occurs when the log file size exceeds rotationFileSizeBytes.  Log rotation can happen at the start time of the app (Rotation.onlyAtAppStart) or at every write (Rotation.alsoWhileWriting).  It is strongly advised that asynchronous logging (log queue) is used with .alsoWhileWriting.
The rotation files are placed in the same directory of the log file.  When the number of rotation files exceeds rotationFileMax, the older files are deleted.   It is strongly advised that a designated directory is created for the log file and its rotations.
The rotation files are named as -yyyy-MM-ddTHH:mm., if a suffix, such as .txt is found in the log file name.  If the log file name includes "."(dot) but no suffix, rotationFileHasSuffix should be set to false.
A new function mostRecentLogFiles are added to return specific number of the newest log files.  As an example, this function can be used to send the most recent log files to the developer.
Please feel free to make changes, especially styling changes to fit in. For the testing case added, I can't figure out how to make the 5-digit test number. So I just used xxxxx.

Forgot to say, please feel free to make changes, especially styling changes to fit in.

Is this likely to merge in anytime soon?

I'll be happy to do it if I'm given write permissions.

Any info about release version  with that cool feature?

By the way, my fork is currently the same with Dave's repo, plus the feature.  You are welcome to try the fork.

mostRecentLogFiles() has a internal access so it's not available when you import.

Hi maciejpigulski, Please send your podfile and include the exact error message.  I built and tested with xcode.  As an amateur app developer, I haven't used cocoapod yet.

When you install something using Cocoapods even from Github as a source then it's added to the project as a Framework. If you want to access methods of the Framework from a project you are developing you need to have those methods with open or public access modifier otherwise they won't be visible. You use open if you want to allow overriding of for your method and public otherwise.

Hi maciejpigulski, you know a lot of cocoapods and I know none.  How about you fork my repository and make the necessary fix?  Then you can send a pull request to me.  That's learning opportunity for me, too.  Thanks!

Automatic rotation would be great.
Is changing public to open the only thing that prevents this from being merged?
I'd be happy to help.

Hi Zyphrax,  Thanks for offering help!  As far as I know, that's the only issue.  I'll be able to learn from your fix.  However, I see that the code base hasn't changed since 9/16.  That means the owner Dave might be too busy to integrate.  But at lease people can use the feature in their own forks.  Thanks again!

Hi Zyphrax, Have you got a chance to looked at the build problem with cocoapods?  Happy new year (belatedly)!

Hi @xyzisinus, a happy new year to you too (I think I'm allowed to say that even in late January :))
When it comes to CocoaPods: XCGLogger isn't really part of the project but it is compiled into a Swift module. Like @maciejpigulski said, if you define your methods as something else than open or public other people using the library won't be able to call your methods.
It is fine for some methods to be private, or internal. But if you want to allow other people to call mostRecentLogFiles for instance, then you'll have to make it open or public. Another example is logFileDirectory, I see that it is internal, so only your code and XCGLogger's other classes will be able to use it (if that is not what you intended, than you should make it open or public).
The difference between open and public is that open is overridable and subclass-able and public isn't.
Would it be possible for our codebases to merge into one nice solution?
Is there a specific feature that you think my code lacks?
I don't think that we should put this into the file logger. Separating the two has a few benefits:

this keeps the FileLogger as fast as possible, if you don't need rotation the code won't be slowed down by it
you can set specific properties on the LogDirectoryLogger that don't make sense for a FileLogger
you keep the file name generated by the LogDirectoryLogger consistent, the pattern determines what the file name should be
the LogDirectoryLogger is more complex than the FileLogger, mixing the two could introduce bugs in the FileLogger causing frustration for people who aren't interested in rotating the log file

Hi @Zyphrax, thanks for the instructions.  I, too, build the Logger into frameworks using xcode and then include them in my project.  But I wonder why I don't see the problem.  Anyway, I'm very happy to make the changes.  Don't you mind inspecting/testing the changes for me because I can't see the bug?  To avoid consuming others' time, I'd like to move the discuss off github.  I'll send you a separate email.  See you there.

BillZong, This repo hasn't seen activities since 9/2016.  If you need the log rotation feature, you are welcome to use my fork which is this repo plus log rotation.  Please be prepared to see some build errors, as others have pointed out that certain functions should be marked as open or public.  Because I don't use cocoapods, however, I don't see those issues.  If you see build problems, please let me know the details and I'll address them quickly.  I'm a new ios hobbits and appreciate all the help :)

OK, I guess the better way is to fork this repo and add your own features. Adding cocoapods/cathage support is more convenient for people who need this framework.

The problem with mostRecentFiles has been fixed.  I couldn't see the problem because I was using the same named function on the top level object XCGLogger which offers this convenience function.
To use my fork with Carthage, use the following line in Cartfile:
github "xyzisinus/XCGLogger" "c78fd34acb9aaeb0e3b47acdbb721ec4355e7bbd"

@xyzisinus @Zyphrax I'm going through these two PR's (#163, #181) and will have a combination of them in shortly.

@DaveWoodCom Hi Dave, great to see you're back! If we can do anything to help, please let us know!

Hi @xyzisinus and @Zyphrax,
I've gone through the two branches. I had some concerns with both, especially with regard to performance. I've pushed (to develop) a new destination AutoRotatingFileDestination that should accomplish what you were looking for.
I still need to write up docs, and add some sample code before I can push it to master. But take a look and let me know if you have any concerns.

Hi @DaveWoodCom, Thanks for your hard work.  I did a quick look at the AutoRotatingFileDestination.  It's very nice code!  I have two questions about archivedFileURLS().  Does the function assume any order of the file URLs returned?  Is the current log file included in the returned files?  I hope that the answers are yes and yes :)  Thank you again.

Hi @xyzisinus,
the archivedFileURLs() method does not return them in any order, nor does it include the current log file (since it's not yet archived). You can get the current URL using the writeToFileURL property. The sort is done as late as possible to avoid sorting the array and then discarding it because there aren't enough to worry about cleaning.
You can see the sort done in the cleanUpLogFiles() method.
var archivedDetails: [(url: URL, timestamp: String)] = []
for archivedFileURL in archivedFileURLs {
    guard let timestampOptionalData = try? archivedFileURL.extendedAttribute(forName: XCGLogger.Constants.extendedAttributeArchivedLogTimestampKey) else { continue }
    guard let timestampData = timestampOptionalData else { continue }
    guard let timestamp = String(data: timestampData, encoding: .utf8) else { continue }
    archivedDetails.append((archivedFileURL, timestamp))
}

archivedDetails.sort(by: { (lhs, rhs) -> Bool in lhs.timestamp > rhs.timestamp })

What's happening here is that I'm using extended file attributes to store the identifier of the logger that archived the file, as well as the timestamp that the file was archived. So the code above is retrieving the timestamps, and caching them with the URL's, then using them to sort.
By using the extended file attributes, it lets us get a correct sort order, regardless of the date format or filenames used for the archived log files.
I guess it would be handy to be able to request a sorted list of archivedURL's so I can refactor a little and add that in.

Thank you, Dave, for explaining.  Yes, a chronologically sorted list will be very helpful.  In my app, the user can send the most recent log files to me when reporting trouble.  It will be great if the code  can get the files directly instead of having to sort them.  Thanks!

@xyzisinus I've made it so the URL list is always sorted. Figure you're right in that it will almost always need to be sorted. In the cases where we'll sort the list and then discard it, there will be very few items anyway so the sort shouldn't be a big deal.
Thanks for your work on this.
It's now on master (5.0.1).

@DaveWoodCom Great work! Few questions / points of feedback:

Good idea to use the file attributes!

String manipulation in Swift is horrible, removing the file extension must not have been fun to code. The URL class has some nice helpers though, that could reduce the code a bit, like deletingPathExtension().

The writeToFile initializer argument seems a bit strange. I guess its only purpose is to satisfy the initializer of the FileDestination class (I had some issues with that too). A writeToFolder argument would be more logical in my opinion.

The comments of targetMaxTimeInterval say that if 0, no maximum, but the interval is set to 600? Maybe that if-statement should be removed from the didSet and a guard-not-zero-statement should be added to the shouldRotate method?

For the archive date formatter it might be a good idea to use a neutral locale:  NSLocale(localeIdentifier: "en_US_POSIX"). Log files shouldn't have a different name based on the locale of the user's device. And is a custom date formatter really necessary? Perhaps a custom format string is enough (the developer can always inherit from the class to override it)?

It might not be necessary to use the currentLogFileSize variable to track the file length. The offsetInFile variable of the FileHandle created in the FileDestination class should tell you the size of the file? Might increase performance a bit too, not having to count the message size.

Why are you writing do { ... } catch let error as NSError { ... } instead of
just do { ... } catch { ... }?

Thanks!

@Zyphrax Wow, thanks for taking such a good look at the code.

Yeah, not fun, will revisit and update with deletingPathExtension().

There were a couple of reasons I went with the file instead of the folder as you had. a) Wanted it to be as compatible with the FileDestination as possible, was tempted to just add it to that single class instead of creating a sub class. b) There needs to a consistent current log file name anyway, so once we have that, we can default to archiving the logs in the same place. A big difference to your implementation was that your current log file would have the date in it, so each log file would be stamped with the start date vs the archived date.

Oops. :) Good catch, will be fixed in the next point release.

A date formatter is consistent with other parts of the framework, and it provides much more flexibility. See #198.  I don't want to presume what anyone is using the logger for, most use it for debugging their apps locally, so their own Locale/TimeZone makes sense, others are using it in server code (or comparing to server logs) and will need to use their server's time zone (or GMT) etc. A dev may want all logs from users to be in the dev's locale for example.

Great idea. I was so focused on avoiding checking the file attributes for the size that I didn't think of using the handle's own info. I'll switch this over in the next release as well.

Mostly just style, so I can be consistent and make use of error codes etc if needed.

I should double check that again. My thinking was that the write happens on the logQueue, so the cleanUp would automatically be on the same one. But I should make sure, especially for the case where there is no logQueue.

Thanks again for taking such a good look at it!


Use of VQ-VAE in test.py

I am wondering if there is a discrepancy between the how inference is explained in the paper and how it is implemented in test.py. To be specific, the paper says "During inference, only Gs and Gt are used." but the VQ-VAE model is loaded in test.py and used to do inference (see here and here for example).
I am new to tensorflow and not super familiar with VQ-VAE, so I might be missing something, but I think since the VQ-VAE encoder is getting the full image as input (see here) there might information leak from the full image to the inpainting module at inference time. Please correct me if I'm wrong.
Thank you.

Thanks for your good questions. Here are our responses.

Even though we load the hierarchical encoder and the decoder of VQ-VAE in test.py (see here and here), we don't use them in the inpainting procedure. We only use the fixed structural codebook (see enc_gt['embed_t'] at here) to map the predicted index onto the predicted feature vector at the current position.

Similar to the first response, we don't use the encoder or decoder of VQ-VAE, except for the fixed structural codebook (a 512*64 matrix). If you are concerned about the information leak from the full image, you can replace the batch_pos with masked (see here) or even an all-zero tensor. The performance will be the same because we only use the fixed structural codebook.

Best wishes,
Jialun

Thank you Jialun. It clarified my confusion.
Best,
Saeid.


 LETS GOOOO

code written: 1:54 pm. pr made: 3:50 pm. great work


What does a class (or a type) need in order to be designable by out-of-process winforms designer ?

What exactly makes a type designable by the (out-of-process) winforms designer ?
To further explain, if I have a custom type that does not inherit from any of the system.windows.forms's designable types (Component, Control, etc), what would it take to make that type designable in the winforms designer ?
Is implementing a proper designer and placing the Designer attribute on the type enough to make it designable, or is there something more required to accomplish this ?
Palli

Never mind. This question does not capture the actual context/subject, as it is basically related to what a type needs in order to be dropped on a form. I guess it's related to implementing the IComponent interface or something similar, so I'll repost when I can be more specific.


failed to deploy react project to netlify

Describe the bug
failed to deploy to netlify
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-4e5giq
Steps to reproduce
project runs successfully but cannot deploy to netlify
Expected behavior
Project should deploy successfully
Screen Recording / Screenshot
Failed to deploy my react code to netlify after several tries
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 56081899,
  "usedJSHeapSize": 52769967,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = jasonpython50
Chat ID = 50a35f5041a8

Additional context
Error:
Something went wrong whilst creating your site on Netlify.

we're aware of this issue it's due to netlify rate limiting us we're trying to resolve the issue. Closing this issue in favour of tracking at https://github.com/stackblitz/bolt.new/issues/365 instead.


[FLINK-24586][table] SQL functions should return STRING instead of VARCHAR(2000)

What is the purpose of the change
There are some SQL functions which currently return VARCHAR(2000). With more strict CAST behavior from FLINK-24413, this could become an issue.
The following functions return VARCHAR(2000) and should be changed to return STRING instead:

JSON_QUERY
JSON_OBJECT
JSON_ARRAY

There are also some more functions which should be evaluated:

CHR
REVERSE
SPLIT_INDEX
PARSE_URL
FROM_UNIXTIME
DECODE

Brief change log

Updates the return type of the SQL Function SqlJsonArrayFunction, SqlJsonObjectFunction and SqlJsonQueryFunction.
Updates the return type of the SQL Function CHR, REVERSE, SPLIT_INDEX, PARSE_URL, FROM_UNIXTIME and DECODE.

Verifying this change

Updates the JsonFunctionsITCase to verfiy the return type of the SQL function JSON_QUERY, JSON_OBJECT and JSON_ARRAY is STRING.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@Airblader , could you please take a look at this pull request? Thanks.

@slinkydeveloper , IMO, the previous tests include the return types of REVERSE, CHR, SPLIT_INDEX, PARSE_URL, FROM_UNIXTIME and DECODE because in this pull request I have updated the testProcessingTimeInnerJoinWithOnClause.out file. Does it need to add new tests for these functions?

@slinkydeveloper @Airblader , thanks for the detailed review. I have addressed the above comments. But the tests for CHR, REVERSE, SPLIT_INDEX, PARSE_URL, FROM_UNIXTIME and DECODE couldn't add to StringFunctionsITCase which is tested for the BuiltInFunctionDefinition not SqlFunction.

@flinkbot run azure

@slinkydeveloper , the failed test has nothing to do with this change. Please check again.

@twalthr , thanks for the detailed review. I have addressed the last comment above mentioned. Please help to review again.


[ownership] Fix a corner case in the linear lifetime checker.

Specifically, if we had a value that was consumed in the same block that it was
produced in, we ignored use-after-frees in subsequent blocks. We did check for
use-after-frees in the same block though. The general case was handled
correctly, this was just an incorrect early exit.
On master, this only found one problem (namely the one fixed in:
14d39c0cf951aaca3b8d782b0738bf55bf6672ab). I do not expect this corner case to
have more impact after cherry-picking to 5.1.
rdar://49794321
(cherry picked from commit 6552a0221ef2514b0f9fe3d79a314ffc02870335)

@swift-ci test


jalali dates ??

how can we do localization with this ??
i want datepicker to show jalali dates ??
can it be done ?

I don't know how that can be implemented currently but this being an open source project features are only limited to contribution.
The original projects documentation might be helpful http://www.daterangepicker.com/


Kerman, CA, US

# us-ca-city_of_kerman-1552501361813.geojson.gz
ok - 4006/4526 (88.5%) valid addresses
ok - 0 outside of boundary (dropped)
ok - 0 have an unsupported number format
ok - 520 are missing street names
ok - 0 could not be parsed
ok - 0 suffered misc failures


Adds dashboard for ocs client operator

@bipuladh I hope client operator is adding label to the openshift-storage NS as well: https://github.com/red-hat-storage/odf-console/blob/master/packages/odf/features.ts#L237 then only UI will poll from odf-managed prom, else it will poll from OCP prom.

@bipuladh I hope client operator is adding label to the openshift-storage NS as well: https://github.com/red-hat-storage/odf-console/blob/master/packages/odf/features.ts#L237 then only UI will poll from odf-managed prom, else it will poll from OCP prom.

We need to poll from OCP prom

/lgtm


Cannot import package

Love this package :smile: . However, I'm getting the following error using Node 18.12.1. I'm assuming this is because the file that's being distributed as esm/ directory is compiling as a CommonJS module.
I've tried importing by doing
// causes error
import { RequestInterceptionManager } from 'puppeteer-intercept-and-modify-requests';

// causes error
import pkg from 'puppeteer-intercept-and-modify-requests';
const { RequestInterceptionManager } = pkg;

// causes error
const { RequestInterceptionManager } = await import('puppeteer-intercept-and-modify-requests')

Importing
const { RequestInterceptionManager } = await import('puppeteer-intercept-and-modify-requests')

I get the following error.  I'm assuming
/code/node_modules/puppeteer-intercept-and-modify-requests/esm/main.js:14
import { promisify } from 'util';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:360:18)
    at wrapSafe (node:internal/modules/cjs/loader:1088:15)
    at Module._compile (node:internal/modules/cjs/loader:1123:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1213:10)
    at Module.load (node:internal/modules/cjs/loader:1037:32)
    at Module._load (node:internal/modules/cjs/loader:878:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:169:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)
    at async file:///code/openrss/api/browser/manager.js:5:40

Node.js v18.12.1

Hi @markcellus! thanks for flagging this. Next version should have this fixed.


Best practice for a blog with images.

Hi,
I'd like to manage a blog with a main image for every post.
To avoid polluting the 'static' directory with all the images for the posts, I understand that I could have a subfolder for each post and inside a file index.md and the corresponding main image:

Inside my index.md:
---
title: 'This is a sad post.'
date: '2020-06-22'
draft: false
tags: ["sad"]
categories: ["Social"]
excerpt: Introduction to a sad post.
image: 'main_image.jpg'
---
This is the markdown content of the sad post.

Then, inside the template for the page, I should have something like:
{{ asset(page.image) }}

But unfortunately this is not working, the file is not found.
Is there a better way to do that ?
Thanks a lot, Cecil rocks!

Hello @thomascenni, sorry for the delay!
Unfortunately it's not possible to store media files in a folder other than static (see https://cecil.app/documentation/content/).

Hello @thomascenni, sorry for the delay!
Unfortunately it's not possible to store media files in a folder other than static (see https://cecil.app/documentation/content/).

So the only way I understand is to create subfolders for each posts in the static directory.

So the only way I understand is to create subfolders for each posts in the static directory.

Yes, you can create as many subdirectories as you want and then refer to the image path in the front-matter of your Markdown file, or directly in the body of the text.
The advantage of this approach is that Cecil is compatible with the Media Manager of Forestry and Netlify CMS.

And more, if you use Typora as Markdown editor, you can configure the relative path to the media path in the front-matter:
typora-root-url: ../../static

Documentation: https://support.typora.io/Images/#relative-path-to-certain-folder.


example broken build

This PR is not labeled with ok-to-test and author of PR is not collaborator or owner of repository. You need to wait untill collaborator or owner of repository will label this PR with ok-to-test.


enable multi-value in spec intepreter fuzzing

Apologies for submitting another PR so soon after my previous one, but this turned out to work pretty quickly.

No worries, thanks for this!


Fix: Destination IP should be different for SNAT port reuse

Example given in the table to illustrate SNAT port re-use is wrong.
SNAT port reuse is possible only if destination IP and/or destination Port is different.
It seems to be a regression introduced in this commit: https://github.com/MicrosoftDocs/azure-docs/commit/499bd37958216f7786b7eb29c393cdc115cc6075

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

@asudbring: Hello Allen, could you please help to check this PR?

@scoulomb , thanks for bringing this to light. Looks like a typo when I made updates to this doc in my last PR. I'm in agreement that the destination IP must be different. I am in agreement with your change.

#sign-off

#sign-off

@scoulomb , thanks for bringing this to light. Looks like a typo when I made updates to this doc in my last PR. I'm in agreement that the destination IP must be different. I am in agreement with your change.

Thanks a lot for the confirmation @aimee-littleton 😊


Pkg.add("Combinatorics") not working

the following error is raised:
julia> Pkg.add("Combinatorics")
WARNING: fetch: GitError(Code:ERROR, Class:Net, SSL error: error:140E0114:SSL routines:SSL_shutdown:uninitialized)
ERROR: Base.Pkg.PkgError("Combinatorics can't be installed because it has no versions that support 0.5.0-dev+1190 of julia. You may need to update METADATA by running Pkg.update()")
in resolve at ./pkg/entry.jl:420
in edit at pkg/entry.jl:414
in anonymous at task.jl:447
in sync_end at ./task.jl:413
[inlined code] from task.jl:422
in add at pkg/entry.jl:57
in add at pkg/entry.jl:78
in anonymous at pkg/dir.jl:31
in cd at file.jl:22
in cd at pkg/dir.jl:31
[inlined code] from pkg/dir.jl:25
in add at pkg.jl:26
in eval at ./boot.jl:264
My julia version:
_
_       _ ()_     |  A fresh approach to technical computing
()     | () ()    |  Documentation: http://docs.julialang.org
_ _   | |  __ _   |  Type "?help" for help.
| | | | | | |/ ` |  |
| | || | | | (| |  |  Version 0.5.0-dev+1188 (2015-11-08 18:30 UTC)
/ |_'|||_'_|  |  Commit e9af3f6 (0 days old master)
|__/                   |  x86_64-linux-gnu

Yes, there is no currently tagged version for julia 0.5. You will need to do Pkg.clone("Combinatorics") until JuliaLang/METADATA.jl#3981 is merged.

... doesnt work either. see here:
julia> Pkg.clone("Combinatorics")
INFO: Cloning Combinatorics from git://github.com/jiahao/Combinatorics.jl.git
ERROR: GitError(Code:ERROR, Class:Net, SSL error: error:140E0114:SSL routines:SSL_shutdown:uninitialized)
[inlined code] from libgit2/error.jl:98
in clone at libgit2/repository.jl:189
in clone#363 at libgit2.jl:310
in clone at pkg/entry.jl:195
in clone at pkg/entry.jl:221
in anonymous at pkg/dir.jl:31
in cd at file.jl:22
in cd at pkg/dir.jl:31
[inlined code] from pkg/dir.jl:25
in clone at pkg.jl:37
in eval at ./boot.jl:264

JuliaLang/julia#13472

@jiahao   thanks a lot. works for me know . thumbs up :-)


Things to think about for "pass 2"

"Differential reddening" of fainter lensed image by lensing galaxy, where the source image is reddened by the lens itself!
Extrapolate SDSS colors to higher redshift using an underlying SED model (since QSO SEDs are all ~similar)
...

Edited this issue to emphasize the thought about differential reddening. Colors need work first, though!


chore: handle events before lnclient is initialized

Title

can this happen?

can this happen?
It could happen if your NWC is restarted and in a locked state, and a connected app tries to make payments. I think it might be good to return a specific error here so it's easier to debug when something goes wrong.

it could happen if your NWC is restarted and in a locked state, and a connected app tries to make payments. I think it might be good to return a specific error here so it's easier to debug when something goes wrong.

then the NWC app is also not listening for that messages and will not be able to reply.

@bumi yeah, you're right. It's impossible for this to work with the new setup.

@im-adithya I will close this for now - see above


[Feature Request]: Integrate BGE-M3 Model

Feature Description
https://huggingface.co/BAAI/bge-m3
A powerful embedding model which supports these features:

dense retrieval, multi-vector retrieval, and sparse retrieval
multi-linguality, and up to 8192 seq length

Reason
No response
Value of Feature
No response

Am I wrong, or can this already be used with our existing HuggingFaceEmbedding class?

Like integrated BGR-Reranker node post-processor, to use this model, you
need to install FlagEmbedding, and inference by this package provided
function.
*Best Regards, *
Zhichao Lin
Logan @.***>于2024年1月31日 周三11:34写道：

Am I wrong, or can this already be used with our existing
HuggingFaceEmbedding class?
—
Reply to this email directly, view it on GitHub
https://github.com/run-llama/llama_index/issues/10366#issuecomment-1918320302,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBO37H4W6KMJYDDD6CTCV3YRG3TDAVCNFSM6AAAAABCSJR4CWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJYGMZDAMZQGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

here is the FlagEmbedding implementation https://github.com/FlagOpen/FlagEmbedding/blob/master/FlagEmbedding/bge_m3.py

We have support for flag embedding reranker
https://github.com/run-llama/llama_index/blob/main/docs/examples/node_postprocessor/FlagEmbeddingReranker.ipynb

But I suppose no normal embedding class for flag embedding

Is anyone working on this? I am interested in this too.

Nobody currently working on it. Would welcome a PR

That's working for me to use

HuggingFaceEmbedding(model_name="BAAI/bge-m3")

Yea tbh I expect that to work fine. Going to close this

That's working for me to use

HuggingFaceEmbedding(model_name="BAAI/bge-m3")

This solution won't use the full functionalities of BGE-M3. the sparse and multi-vector retrieval scores of BGE-M3 will be ignored.


Issuing credits to Ryan

Issue 120,192 credits to @rmcgranaghan.

 Follow up
On Wednesday, 28-Feb-24 17:40:42 UTC by Gov4Git dev
Your directive cannot be parsed.


Delete wcf host and test projects.

We're not supporting this in v2 :)

YES!.
Next do the SelfHost one too!

This is the kind of diffs I like the most :heart:


Add event handling feature

Cc: @DevilsAutumn
Expected functionalities:

[x] Admins should be able to create events from admin panel.
[ ] Users should be able to register in the events by providing their emails.
[ ] Admins should be able to send email to all registered users of one event from admin panel.
[x] For events fields will be something like: Name, Cover picture, description, venue etc.
[ ] For registering for event form should contain: First name, last name, occupation (working professional/student/freelancer), gender, email etc.

Hey @DevilsAutumn , @salty-ivy
Where should we store the cover images? Store on server or add in some 3rd party storage (like GCP, or something). Storing on backend server isn't a good idea imo.

@HarshRaj2717 We'll be storing them on AWS.

Storing on backend server isn't a good idea imo.
Yes definitely, Are we planning to keep the AWS storage code on front-end directly keeping the architecture partitional?
thought?

My original thought was to keep the direct upload and storing mechanism in react itself and whatever ID has been formed after storing, we can store that in table.
Since whole thing is based on API that's directly upload to backend would break the standard of it.

Admins should be able to create events from admin panel.

But we aren't making any frontend for creating events right ? I can add the code to upload to AWS on the backend itself.

Admins should be able to create events from admin panel.

But we aren't making any frontend for creating events right ? I can add the code to upload to AWS on the backend itself.

I see so client side would only be fetching the details? then it seems great!

My original thought was to keep the direct upload and storing mechanism in react itself and whatever ID has been formed after storing, we can store that in table.

Yes but that might add little complications on frontend as one api call be there to upload to aws bucket and then another to backend for updating the ID. Lets upload files to aws from backend itself.

But we aren't making any frontend for creating events right ?

No, we'll be using django's inbuilt admin panel.

Yeps, client will focus on interacting with the end user only.