proposal: Go 2: introduce a 'check' built-in function to reduce error handling boilerplate

Now that the try proposal (#32437) has been withdrawn, I have been thinking about whether there might be a middle-ground proposal which would still save significant error handling boilerplate but be more acceptable to those who disliked try.
After studying various alternative proposals and the factors which led to try's demise, I have come up with the following which is even simpler and might fit the bill. There have been many similar ideas (notably #32811 which was a major influence) though probably not in the exact form used here.
The proposal.
I propose the introduction of a new built-in function called check. This takes a single parameter of a particular interface type (usually error) and does not return anything.
check can only be used inside a function with at least one result parameter where the last result is of the same interface type.
If the argument to check is nil, then it does nothing and control passes to the next statement.
However, if the argument is not nil, then it triggers an immediate return from the enclosing function. The final return parameter of the function is set to the argument to check and any other return parameters are given their zero values (or current values if named).
So, this:
check(err)

turns into the following inlined code:
if err != nil {
    return x1, x2, ...xn, err
}

where the x's are the zero values (or current values) of any other return parameters apart from the final interface parameter.
Aim of the proposal
The aim of this proposal is exactly the same as try, namely to cut down on boilerplate in a significant number of cases.
As in the case of try, all errors can be decorated in the same way using defer or the current if err != nil approach can and should be used in all other cases.
As a new built-in function is being used, it would still be backwards-compatible and would still be open to possible future improvements by simply adding a further parameter or parameters to check.
Example
The following example taken from the try proposal may help to make all this clear:
func CopyFile(src, dst string) (err error) {
    defer func() {
        if err != nil {
             err = fmt.Errorf("copy %s %s: %v", src, dst, err)
        }
    }()

    r, err := os.Open(src)
    check(err)
    defer r.Close()

    w, err := os.Create(dst)
    check(err)
    defer func() {
        w.Close()
        if err != nil {
            os.Remove(dst) // only if "check" fails
        }
    }()

    _, err = io.Copy(w, r)
    check(err)

    check(w.Close())
    return nil
}

Note that a function which only returns an interface value of the relevant type (such as w.Close()) can be passed directly to check().
Advantages
Compared to the try proposal, the present one has the following advantages:

It's simpler and should therefore be easier to implement, understand and document.

Custom error interfaces can now be coped with - though I believe try could have been altered to do the same.

It will be easier in most cases to refactor between code using check and the present approach, which should make debugging easier. Also the former doesn't look out of place where both approaches are used in the same function.

The word check is perhaps more suggestive that a change in flow control may be triggered than try was.

As check doesn't return anything, this effectively rules out the possibility of there being several independent errors on the same line (i.e. try expressions, nested or otherwise) which a lot of people disliked (albeit others saw as an advantage).

Disadvantages
Compared to the try proposal, the present one has the following disadvantages:

It doesn't save as much boilerplate.

check is probably used more than try as an identifier in existing code and is longer.

Finally
I should perhaps add that I'm not totally sold on this proposal (or for that matter any other alternative proposal) myself as it's questionable whether it saves enough boilerplate to be worth doing but it certainly looks more compact.
Incidentally, I've given up hope that go fmt will be changed to permit single statement if's to be written on one line which would have made even the present approach look more compact.
Also this proposal doesn't address (or even attempt to address) the problem of error decoration any better than try did because, quite frankly, I think this is best done with the existing if err != nil  approach unless all errors can be decorated in the same way when defer can be used.
However, I think it's arguably clearer what's happening with this proposal and so I thought that I should at least put it forward to see whether there's any support for it. If not, then it's something else we can rule out once and for all.

Not sure if I like this more than the try proposal, but it's probably worth mentioning that a big advantage to this is being able to annotate errors at each call site:
func CopyFile(src, dst string) (err error) {
    r, err := os.Open(src)
    check(errors.Wrap(err, "error opening copy source"))
    defer r.Close()

    w, err := os.Create(dst)
    check(errors.Wrap(err, "error creating copy destination"))
    defer func() {
        w.Close()
        if err != nil {
            os.Remove(dst) // only if "check" fails
        }
    }()

    _, err = io.Copy(w, r)
    check(errors.Wrap(err, "error copying data"))

    check(errors.Wrap(w.Close(), "error closing destination file"))
    return nil
}

Without this call-site annotation, it could potentially be very difficult to figure out if an error came from the os.Open or os.Create since under the hood, they both just call os.OpenFile.

Haben Sie Einweisung bitte für pkg Lösung die simple ist 🤯

Ich benutze pkg terminal

@wowa1234567 Please stop posting off-topic comments.  If you continue I will be forced to block you.

@ccbrown

Without this call-site annotation, it could potentially be very difficult to figure out if an error came from the os.Open or os.Create since under the hood they both just call os.OpenFile.

Thanks for making the above point which is a good one.
Although it had occurred to me that err could be decorated before being passed to check, the trouble is that it would come with a cost, namely that the decorator (errors.Wrap in your example) would still need to be called even if  err were nil.
You could, of course, avoid this cost with the present way of doing things:
    _, err = io.Copy(w, r)
    if err != nil {
        return errors.Wrap(err, "error copying data")
    }
    err = w.Close()
    if err != nil {
        return errors.Wrap(err, "error closing destination file")
    }

On the other hand the cost might not be so great as probably the first thing a decorating function will do is to return nil if its err argument is nil.
Also this is something that try couldn't do at all unless it was being applied to a function which only returned an error. So, yes, it is another relative advantage of this proposal.

I'm going to close this now as it's already clear there's little support for it and I don't want to clutter up the Go 2 proposals list with any more 'no hope' error handling proposals than there already are.
Thanks to those who did support it or made a constructive comment.
I was a supporter of the try proposal, not because I thought it was perfect - it wasn't, but because I thought it was the only realistic chance of reducing error handling boilerplate in a simple way. The earlier check/handle draft design was far too complicated in my opinion.
This proposal was even simpler than try but didn't really save enough boilerplate to be worth doing. I have also come to the conclusion that any proposal which doesn't also offer a way to decorate errors where they arise is simply not going to fly.
It would, of course, be easy enough to add a second optional argument (a string) to check which, if the first argument were nil, would be used to decorate the first argument in some predetermined fashion. So instead of:
 check(errors.Wrap(err, "error opening copy source"))

you'd have:
check(err, "error opening copy source")

However, unless this were confined to just prepending  the second argument  to the string value of the first, it would inevitably involve calling some wrapping function under the hood (hardly ideal for a language feature) and you'd end up with the same problems which beset #32811.
Frankly, I find it difficult to see how any alternative proposal could ever deal with in place error decoration in a better and more flexible way than if err != nil does and I shall therefore be supporting the status quo in future.


Get-DbaLastGoodCheckDb example included

ExcludeDatabase parameter didn't have an example.

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[X] Documentation
[ ] Build system

Purpose
Show how to use the "-ExcludedDatabse" parameter to limit the output.

tearing it up!!!


Made remote client send and receive requests.

Responses are not yet available in the REPL for some reason.

Thanks!


Error with nuget while adding package with #r command.

I have error with nuget, when I try to add package with #r command. And this happens with adding of any packages.
dotnet: 5.0.100
dotnet-interactive: 1.0.157409+69b86441f75e5663aa2c1568ae971127de9cd02e
python (venv): 3.6.8

But at this time in RoslynPad everything works.

Can you try with a different syntax for specifying the package version?  The slash / between the package name and version number isn't supported by .NET Interactive, but a comma , is.
E.g.,
#r "nuget:PuppeteerSharp,2.0.4"

Thanks a lot, yes, the slash / was the problem. Everything works well now. Even without specifying a version.


Properly initialize canned APs

@lzhao-skyhook - please review.

Note that I arbitrarily increased the number of canned APs from 4 to 10.


Desktop font sizes

Gave up on MUI responsiveFontSizes, it only works >1rem

it only works >1rem

Is that because they multiply the value, so anything below 1 when you multiply gets smaller? 😬

Fonts of less than 1rem are just skipped...


search with max mix duration only return a very few traces

jeager 1.14 all-in-one. when search traces without max min duration, a lot of traces found, but with min max duration(eg 1m-1000ms), only find very few（about 6） traces, although a lot of traces  duration in this duration but still be  filtered. do someone konwn why? thanks！

@zziawan the max/min duration filter is applied on a span level, not a trace level. This issue came up in the past as well with us, and I think we could improve the usability, perhaps improving the label text.

@albertteoh  thanks for you reply！ but i found many  filtered traces only has one span  and the duration also in the search min max duration.  and the search result is the same every time! the search limit also no effiect no matter how bigger it is,  fixed num traces returned everytime.  thanks！

hmmm... did you try increasing the lookback time period? Perhaps those traces are quite old and I think the default lookback is an hour.

@albertteoh  mybe it's a bug of v1.14. the same db and query, v1.17 no such problem. but i can not run v1.14 on my machine, so i can not found why。

@zziawan what storage backed are you using? ES/Cassandra or badger?
We are not planning to backport fixes to 1.14.0. Please close this issue if you cannot reproduce with newer version.

@pavolloffay  we use badger。ok we try other verion.


cargo crev review <crate> <version> should work even outside of any project

It should be possible to call it from anywhere.

Actually ... I'm not so sure about it. cargo is used only in context of a Rust project ...


Develop a Test Platform

We need some sort of tests to ensure correctness of the code.
Unfortunately, I'm no good at designing tests, so if anyone has some good ideas, put them here!
Thigs that should be tested:

Reading and writing classes
Readingand writing jars
Using chasm without transformers shouldn't chage any classes
Valid transformers that can be applied without conflicts must be able to be applied
Valid transformers that conflict must be detected
Probably a lot more later

There exists now a test that parses and writes single classes.
This test verifies that we don't lose any information by applying chasm.
I would appreciate adding some more complex test classes though, to find bugs in edge cases.

I'm not entirely sure how to do the transformation tests, but simple regression tests are probably good enough.
If anyone would be interested in implementing that, that would be highly appreciated!
Quiltflower supposedly has a good system for that as reference.

I could try to implement something similar to what Quiltflower has

Please do :)


What if I get the slot elements? For example, implement something like the space component, with a div over each element.

I am interested in helping provide a feature!
Yes
Which generators are impacted?

[x] All
[ ] Angular
[ ] HTML
[ ] Qwik
[ ] React
[ ] React-Native
[ ] Solid
[ ] Stencil
[ ] Svelte
[ ] Vue
[ ] Web components

What problem does this feature solve?
import { Slot } from<EMAIL_ADDRESS>
export default function MySpace(props) {
  return (

      <Slot />

  )
}

<MySpace>

1

2

</MySpace>

You want to achieve the following effect

1

2

What does the proposed API look like?
You can get the slot elements and then iterate over each element to process it individually
Additional Information
No response

Duplicated https://github.com/BuilderIO/mitosis/issues/945

Another issue is that I may need to get the slots to make a determination
Examples:
vue

xxx

react
{props.xxx &&
xxx
}

like @kingzez mentioned, we do not plan on implementing a Children API like React's.
As for the 2nd question, you can currently do that like this: https://mitosis.builder.io/?outputTab=G4VwpkA%3D&code=JYWwDg9gTgLgBAbzgZQBYQO5wL5wGZQQhwBEAAgEYCuwANgCYCmUAdMBAPQjAwQDOwPiQDcAKFGMAHpFhwmeAIZVa8PFQB2AYxjt1cALIBPAMJFI6xupgAKMITB8AlIlFw4URjCpQ91127gAHjRMOAxUSwBeBDsIBxZNVDp6D3VsAD5%2FAKD6YAA3dIAJRlpaCECOXIKsoI4QjEy3RzFsIA%3D%3D
It unfortunately doesn't consistently work across all frameworks, because they do not all have a way to access the value of their slots (Qwik doesn't for example). But it does work for Vue, React, Svelte, and many others.


add adler32 functionality and tests

exposes the adler32 function of libdeflate.  Implementation is almost a copy of the crc32 implementation, only the initial value is 1 in adler32 (where crc32 has 0).

While you're at it, you could also correct the typo "alder32" in the commit comment.

Everything else looks pretty good and as expected, thanks for the PR!

Thanks for the review and thanks for spotting the mistake with the assert.


IE9 - tab characters are not supported

Because they're rendered with an unpredictable length in the mirror div.

IE9 will hopefully soon become a completely forgotten memory even among those hanging on to Windows XP.
Patches welcome, but the core maintainers won't be working on this.


Floating point math (Pow)

I'm trying to use pow and running into link errors (undefined reference to 'pow'). What flags do I need to change to get floating point math working?

You could try adding -lm to the linker parameters (LDFLAGS) here:
https://github.com/dvhdr/launchpad-pro/blob/master/Makefile#L35
Funnily enough, I don't get the undefined reference to pow. I get an undefined reference to __errno. To hack it, you can declare "int __errno;" somewhere in your code, but, um, ymmv!


Fix incorrect boolean type cast

Boolean properties were cast to boolean here:
https://github.com/nunomaduro/larastan/blob/master/src/Properties/ModelPropertyExtension.php#L233
So we need to compare $column->readableType with boolean instead of bool

Hi, thanks for the PR!
Can you explain why this is necessary? Cause here the type resolver would convert both bool and boolean to the correct type. So I don't think there will be an issue.

@canvural In old logic, if $column->readableType is boolean, the getReadableAndWritableTypes method will return [mixed, mixed] instead of [bool, bool].
The core of this PR is this line: https://github.com/nunomaduro/larastan/pull/513/files#diff-8c4331b9e7c6c4544f1895befdc48f1dR198

In that case, you are right :+1: Thank you!


Create git tags for releases

Create a tag in the format vx.y.z for each new release.

I should backfill the changelog, too, and make a habit of keeping that.

I've gone through and added annotated tags. I'll raise a separate issue to create a tag during the gem push action.


Make reminder prototype

Implemented notifier functionality. https://github.com/maddevsio/comedian/tree/master/notifier


ShowDefinitionForSelection shows up at wrong coordinates

In reference to https://github.com/brave/muon/commit/62dec3b9561bc5d35976549317c40b1b6fc46b4e
content/browser/renderer_host/render_widget_host_view_mac.mm
In showLookUpDictionaryOverlayFromRange(from ShowDefinitionForSelection)
gfx::Rect root_box = renderWidgetHostView_->GetViewBounds();
will get 0,0 0x0
comparing to showLookUpDictionaryOverlayAtPoint (from three finger tapping)
will get 0,22 1440x838
from the same selected word.

I suppose this could potentially be a webview bug. When we have some time we should try to repro in a Chrome webview

It starts from
atom/browser/api/atom_api_web_contents.cc
void WebContents::ShowDefinitionForSelection() {
#if defined(OS_MACOSX)
  const auto view = web_contents()->GetRenderWidgetHostView();
  if (view)
    view->ShowDefinitionForSelection();
#endif
}

and then
guest_bounds and embedder_boundslook fine in
RenderWidgetHostViewGuest::ShowDefinitionForSelection()  of content/browser/frame_host/render_widget_host_view_guest.cc

was there a ticket for the original issue? If not you can add it here


Make crate path configurable in contractimpl

What
Make crate path configurable in contractimpl
Why
So it is possible to use the contractimpl macro in places where the crate path needs to be modified.
Other macros already support configuring crate path, so this brings the contractimpl macro up to date.

@sisuresh I could have been more specific and detailed on the use cases, there are several.

As your suggested, it allows the macro to be used inside the SDK, such as in the testicles to provide stock contacted for use in tests.

Other SDKs that would like to embed the soroban SDK such as the one @willemneal is working on would be able wrap the macro in another macro and use a modified crate path to reference the reexported soroban SDK, without users of that wrapping SDK needing to import the soroban SDK themselves.

My immediate use cases is the first.


Custom setup

Eh wat?

Failed experiment in any case 😆


Test it out on phones and tablets

Need to host it somewhere so we can do some spot testing.

This is easy enough with the chrome developer tools.


Android license status unknown should warn when the JDK version is incompatible

On a fairly frequent basis, people come into the community flutter discord, and complain about this specific issue:
Exception in thread "main" java.lang.NoClassDefFoundError: javax/xml/bind/annotation/XmlSchema
        at com.android.repository.api.SchemaModule$SchemaModuleVersion.<init>(SchemaModule.java:156)
        at com.android.repository.api.SchemaModule.<init>(SchemaModule.java:75)
        at com.android.sdklib.repository.AndroidSdkHandler.<clinit>(AndroidSdkHandler.java:81)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:73)
        at com.android.sdklib.tool.sdkmanager.SdkManagerCli.main(SdkManagerCli.java:48)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.annotation.XmlSchema
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:636)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:182)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:519)
        ... 5 more

This problem happens because the user's default Java install, or the Java install pointed to by JAVA_HOME, is not compatible with Android tooling.
Flutter doctor should detect the JDK that will be used, and, if incompatible, automatically inform users of this fact, and eventually recommend them troubleshooting steps, such as:

Point JAVA_HOME to the bundled JDK inside of Android Studio
Install JDK8, and update JAVA_HOME to point to it
Maybe a link to some JDK download, depending on the OS?

Updates since 2021, flutter doctor -v prints the version of java. We default to the version of java installed in android studio, there is a config you can set to modify the version of java flutter uses for your project, flutter analyse --suggestions can let you know about agp, java, and gradle compatibility, a pr is up for warning in flutter create when the installed java version does not match the templates https://github.com/flutter/flutter/issues/130515.


Fix flatten

@ojii added values checks too

@yakky, thanks for the updated PR. Did you check this against django CMS to see if it actually fixes the issue you had there?

@ojii yes. Well, CMS has its own share of issues in this area, but I'm nailing them down in divio/django-cms#4998

Tentatively scheduling a 0.7.2 to include this by EOW.

Merged and released as 0.7.2, thanks @cyberstar and @yakky


feat: gitee issues, issue, pullRequests screens

Added Issues, PullRequests, Issue screens. PullRequest screen not done.
In Issues - creating an issue, commenting on an issue done
changed number prop to subtitle in issue_item.dart
Added a new prop prefix to comment_item.dart. CommentItem uses UserName which has the github prefix. Added a prefix prop so it works with sites other than github
In ge_pulls.dart, commentCount is set to 0 because the response from gitee doesn't have a comment count property. See this example response.
Screenshots:

Is the commenting on issue implementation fine? I just used the plus action button to open a new screen to type in the comment.

Is the commenting on issue implementation fine? I just used the plus action button to open a new screen to type in the comment.

I'm also thinking of adding the edit comment and delete comment actions. Which one do you think is better?
1 - adding two different buttons side by side below the comment's text (or I could use text labels instead of icons)
2  - adding the three dot icon with menu options that have add/delete
Should I open a separate issue to discuss about this?

1 - looks something like this (I'll have to add the emoji also):

I'm also thinking of adding the edit comment and delete comment actions. Which one do you think is better?
1 - adding two different buttons side by side below the comment's text (or I could use text labels instead of icons)
2  - adding the three dot icon with menu options that have add/delete
Should I open a separate issue to discuss about this?

1 - looks something like this (I'll have to add the emoji also):

adding the edit comment and delete comment actions

Great! Since there are enough changes in this PR, I think it's better to submit a new one to implement this.

1 - adding two different buttons side by side below the comment's text (or I could use text labels instead of icons)
2 - adding the three dot icon with menu options that have add/delete

Both would be fine. Considering the edit/delete is not very commonly used, maybe option 2 is better, in my opinion.

adding the edit comment and delete comment actions

Great! Since there are enough changes in this PR, I think it's better to submit a new one to implement this.

1 - adding two different buttons side by side below the comment's text (or I could use text labels instead of icons)
2 - adding the three dot icon with menu options that have add/delete

Both would be fine. Considering the edit/delete is not very commonly used, maybe option 2 is better, in my opinion.

maybe option 2 is better

Alright. I think this is better too cause someone might click on the buttons accidentally. Also, we could add 'Quote reply` in the future and it would look very cluttered if there are more than 3 buttons(including the emoji) at the bottom.

maybe option 2 is better

Alright. I think this is better too cause someone might click on the buttons accidentally. Also, we could add 'Quote reply` in the future and it would look very cluttered if there are more than 3 buttons(including the emoji) at the bottom.

maybe option 2 is better

Alright. I think this is better too cause someone might click on the buttons accidentally. Also, we could add 'Quote reply` in the future and it would look very cluttered if there are more than 3 buttons(including the emoji) at the bottom.

maybe option 2 is better

Alright. I think this is better too cause someone might click on the buttons accidentally. Also, we could add 'Quote reply` in the future and it would look very cluttered if there are more than 3 buttons(including the emoji) at the bottom.


Chair Layering Broken Again

its back
#8412
sit on a chair, move around, broken

Didn't you already make an issue https://github.com/space-wizards/space-station-14/issues/10326

i thought it was fixe then bork again


How to provide windows architecture parameter in puppet code ?

I would like to condition to install 32bit or 64bit package..in my code.Is it possible?
If so..could you pls provide an example?

Perhaps more context - this is completely possible and documented in Chocolatey now, and is supported using install_options with the provider, so more context will help draw out what your needs are.

install_options => ['--forcex86'], will get you the 32 bit architecture no matter what. For 64bit architecture, if the package supports it, it is automatic.

Closing as this has gone stale with no response @rammaram06 if you have further questions or wish to clarify what you were/are looking for, please comment and we can get this reopened.


Do you plan on supporting scala-cli style using directive?

So far we have using lib and using file, and we can add more as we need them. It also depends on external contributions, as I'm restricted wrt time.

for reference: https://scala-cli.virtuslab.org/docs/guides/using-directives/


Issuer option is not passed to otpauthURL in generateSecret

In function generateSecret, 'issuer' option is not passed to the otpauthURL function as an option.

The git repo includes issuer option in generateSecret function.
Can you specify "speakeasy": "git://github.com/speakeasyjs/speakeasy.git#e63b936d562fa16c5970d0821a221cf1a59f559b" in package.json to pull the latest source code and pass issuer in generateSecret function?
The wiki https://github.com/speakeasyjs/speakeasy/wiki/General-Usage-for-Time-Based-Token
includes an example to pass issuer value to generateSecret.

hey @railsstudent thanks for the answer I've tried the code version you've sent and it works. When we can expect to be available through npm?

What do you mean by wrong?
The verify function does not return correct value when code is passed in?

Verify function returns correct value but scanned QR code with Google Authenticator generates the wrong value.

The secret values are different. Regression?

Here's the problem: when you run the code "generateSecret()" and pass in an "issuer" as part of the options, the issuer gets dropped due to the following lines:
Line 474 in index.js
// add in the Google Authenticator-compatible otpauth URL
  if (otpauth_url) {
    SecretKey.otpauth_url = exports.otpauthURL({
      secret: SecretKey.ascii,
      label: name
    });
  }

As you can see, even if you pass in "issuer" the issuer won't be passed to the otpauthurl api that generates the google authenticator compatible otpauth URL.

Ahh, this fix appears to already exist in the current branch - what can we do to get this pushed up to NPM?

@Cheprer @BrandonCopley Have time to try https://github.com/speakeasyjs/libotp ?

@jakelee8 as @BrandonCopley mentioned the problem is that the issuer is not passed to otpauthURL constructor so when I've modified code to something like this then it works (of course it needs some check and defaults on the beginning):
if (otpauth_url) {
    SecretKey.otpauth_url = exports.otpauthURL({
      secret: SecretKey.ascii,
      label: name,
      issuer: options.issuer
    });
  }

works for both https://github.com/speakeasyjs/libotp and https://github.com/speakeasyjs/speakeasy

@railsstudent @jakelee8 hey guys I can see this code correctly fixed on the repository on master branch but the NPM package doesn't have it. Can you please update NPM package please and close this issue? Much appreciated, Cheprer

@markbao

+1

+1

+1

+1 it's not on NPM yet

Also:
if you take the response to generateSecret and then make your own URL you have to pre-encode the label your self as the library isn't. To answer @Cheprer
Which as he reports results in:

The correct function call is:
var secret = speakeasy.generateSecret({
    length: 20,
    name: 'Barry Carlyon',
    issuer: 'Some Issuer'
});

secret.otpauth_url = speakeasy.otpauthURL({
    secret: secret.ascii,
    label: encodeURIComponent('Barry Carlyon'),
    issuer: 'Some Issuer'
});

The fault when scanning the QR code only failing as the URL contains unencoded spaces
For example:

node create_user.js
otpauth://totp/Barry%20Carlyon?secret=SOMESCRET
otpauth://totp/Barry Carlyon?secret=SOMESCRET&issuer=Some%20Issuer

@markbao controls the NPM package

@BarryCarlyon thanks for wrapping it up. I hope guys will make it stable and put on NPM soon.

The fault when scanning the QR code only failing as the URL contains unencoded spaces

I'm having the same issue. A string with a space in breaks the url from otpauthURL.

@LukeXF I answered the problem in my solution code:
secret.otpauth_url = speakeasy.otpauthURL({
    secret: secret.ascii,
    label: encodeURIComponent('Barry Carlyon'),
    issuer: 'Some Issuer'
});

It still adds in %20 with encodeURIComponent()

@LukeXF  That what encodeURIComponent() does, it  replaces space with %20

@railsstudent Yes, but the %20 is showing up on Google Authenticator. Project%20Name for example instead of Project Name

Sounds like double escaping

On Oct 6, 2017, at 10:11 AM, Luke Brown<EMAIL_ADDRESS>wrote:
@railsstudent Yes, but the %20 is showing up on Google Authenticator. Project%20Name for example instead of Project Name
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@LukeXF Are you using the npm 2.0.0 version of the package, or the latest master from git?
In the npm version, label is not URL-encoded: https://github.com/speakeasyjs/speakeasy/blob/4e0aa91dee166306e07ea11e371d5a5df95f4391/index.js#L644
In the latest master from git, it is: https://github.com/speakeasyjs/speakeasy/blob/master/index.js#L704
In the latest master from git, issuer should be URL-encoded, since we pass in an object to url.format and that runs querystring's stringify.

@markbao I'm using NPM 2.0.0. But even with using encodeURIComponent() myself, the %20 still outputs as %20 and not a space. Any suggestions?

Problem was with base32 encoding. Because I'm using a base32 secret, I needed to specify an encoding so that it didn't load ASCII be default.
See https://github.com/speakeasyjs/speakeasy/issues/95#issuecomment-334955321 for reference.
var url = speakeasy.otpauthURL({
	secret: secret.base32,
	label: req.user.local.email,
	issuer: 'The Spaces Work',
	encoding: "base32"
});

I've used @LukeXF 's way and it works. Big thanks but even though the function generateSecret doesn't work properly still because it generates ASCII secret and then from it it generates the base32. Maybe there is a glitch. If I'll have time. I'll try to make a pull request with fix but not sure when. So I would keep this issue still open.

@markbao any plans on releasing a new version where the issuer is passed to the otpauthURL function on generateSecret?

Hi all, old issue but the issuer still misses in NPM on the generateSecret() method. Any plans to implement it?

This is still in the NPM package. Please update!

No releases i 4 years, no progress on v.3 i 3 years.  Is project dead?

I think yes, @jlous. @mlogan did a fork, but I think that project is dead too. I tried to contact @markbao on Twitter, but he didn't answer me, so I guess that could be great if someone fork again and implement these features/bug fixes.

Alternatively, https://www.npmjs.com/package/otplib seems to be alive and well.
Will consider migrating my code.


bugfix: check if table meta cache should be refreshed in AT mode

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
同步 https://github.com/seata/seata/pull/4734
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #5313 (426d306) into 2.x (207dda4) will decrease coverage by 0.01%.
The diff coverage is 45.23%.

:exclamation: Current head 426d306 differs from pull request most recent head 70bb469. Consider uploading reports for the commit 70bb469 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #5313      +/-   ##
============================================
- Coverage     48.59%   48.59%   -0.01%
- Complexity     4195     4198       +3
============================================
  Files           764      765       +1
  Lines         26870    26899      +29
  Branches       3352     3357       +5
============================================
+ Hits          13058    13071      +13
- Misses        12430    12440      +10
- Partials       1382     1388       +6

Impacted Files
Coverage Δ

...ec/mysql/MySQLInsertOnDuplicateUpdateExecutor.java
32.32% <0.00%> (ø)

...o/seata/rm/datasource/sql/struct/TableRecords.java
60.00% <32.00%> (-6.99%)
:arrow_down:

.../rm/datasource/exec/BaseTransactionalExecutor.java
55.68% <50.00%> (-0.79%)
:arrow_down:

...ta/rm/datasource/exception/TableMetaException.java
62.50% <62.50%> (ø)

.../seata/rm/datasource/exec/MultiUpdateExecutor.java
77.21% <100.00%> (ø)

...va/io/seata/rm/datasource/exec/UpdateExecutor.java
85.48% <100.00%> (ø)

...datasource/exec/mysql/MySQLUpdateJoinExecutor.java
77.07% <100.00%> (ø)

...seata/rm/datasource/undo/AbstractUndoExecutor.java
69.01% <100.00%> (ø)

...very/registry/zk/ZookeeperRegisterServiceImpl.java
61.02% <0.00%> (-0.74%)
:arrow_down:

...erver/storage/file/session/FileSessionManager.java
49.04% <0.00%> (-0.64%)
:arrow_down:

... and 2 more


Synchronize all reloaders for the first time

Otherwise the web handler won't have the proper config to serve until a reload is triggered once.
Currently the config handler actually panics. I didn't realize during dev earlier, as I constantly reloaded :grin:

Thank youuuuuuu


Added message when no containers match the filter query

Hey @ehazlett, let me know what you think of this... I can do the other pages if it's all good!

hmm i thought i already fixed that.  LGTM


Localization FirstRowType Value

Now we can't customize the value of FirstRowType.None &  FirstRowType.All.

Hello, @hstdt upgrade your pod to the latest version 2.0.9 and you'll be able to use custom strings as FirstRowType.

@rushisangani  Thanks, it works.


Move symfony/lock to require

symfony/lock is required by audit:schema:update

@gbere Thanks for the feedback but it is a duplicate of #104


re-add scalameta to 2.13 community build

@xeno-by you can see the failure details at https://scala-ci.typesafe.com/job/scala-2.13.x-integrate-community-build/318/consoleFull — it's because scala-parallel-collections is a module now. see https://github.com/scala/scala-parallel-collections#usage (which I just wrote today) for instructions on how to deal with it in your build and code
this isn't necessarily urgent, since (even in the 2.12 build) scalameta's only dependent project is scalafix. kinda up to you whether you want early notice of when Scala 2.13 changes break scalameta, or if you'd rather deal with it later all at once

closing because tons of projects have been removed from the 2.13 build, we don't need individual tickets


Expose types implementing serde::Serializer and Deserializer

This is sort of a proof-of-concept PR, I would like your input on it even though it's still in draft.
The goal here is to expose types implementing serde::Serializer and Deserializer so that bincode could be used with https://docs.rs/erased-serde. Serializer is straightforward, but Deserializer can be exposed in two ways:

Expose some type D, where for<'a> &'a mut D: Deserializer<'de>. This is done e.g. in postcard. The problem here is that using such types generically involves passing around an ugly bound on lifetimes, which cannot be encapsulated in a trait. Also, dealing with such types in erased_serde is somewhat complicated (albeit possible).
A much more convenient API is exposing a type D: Deserializer<'de>. The downside of this is that in the format implementations (like bincode) serde::Deserializer is generally implemented for &mut Something, or a wrapper of it (SerdeDecoder in bincode's case), because it has to be passed to nested list/struct/etc deserializers. So creating another implementation for D involves writing down the massive trait impl one more time.

So, I would like your input on the way to proceed with this. I see the following options:

This is not something you want to see in bincode at all. Feel free to close the PR then.
Expose a type D, where for<'a> &'a mut D: Deserializer<'de>. This requires minimum changes, but creates problems for the users, as described above.
Expose a type D: Deserializer<'de>, writing down another Deserializer impl (of which there are already two in the codebase).
Expose a type D: Deserializer<'de>, relying on a helper crate that I made (https://docs.rs/serde-persistent-deserializer), which is implemented in this PR. The reason I put it there is that I want to create a similar PR for other formats, and I would like to avoid writing Deserializer impls in each of them. I understand if you want to minimize the number of dependencies, then the contents of that crate can be brought in verbatim.

(This PR currently does not expose Serializer, but that's easy, and I'll add it later if you decide not to go with Option 1).
A few things about the PR itself.

I would like to remove the Deserializer impl in de_borrowed, and it's almost possible by just replacing de_borrowed::SerdeDecoder with de_owned::SerdeDecoder , but a single test fails because these impls are slightly different. I am not sure if it's possible to merge them.
I am not sure I understand the comment in de_owned::decode_from_slice() ("Note that this does not work with borrowed types like &str or &[u8]. For that use [borrow_decode_from_slice].") Seems to be working fine, as I can just call de_borrowed::borrow_decode_from_slice() from it.

After some experimentation, I decided to not use serde-persistent-bytes at this level and make it the user's responsibility (although its use can be added later, making API a little more convenient).
The PR exposes two new entities: BorrowedSerdeDecoder and OwnedSerdeDecoder, with as_deserializer() methods allowing one to get access to an object implementing serde::Deserializer.

Sorry for the late response, the last month has been very busy. We're still thinking on if we want this change in bincode or not. Thank you for your patience 🙏


workflows/eval: Request reviews from changed package maintainers

Request reviews from maintainers of changed packages, mirroring what ofborg does, but without having to wait for it anymore. Part of https://github.com/NixOS/nixpkgs/issues/355847
Things done

[x] Tested: https://github.com/Infinisil-s-Test-Organization/nixpkgs/pull/37#event-15696464782

This work is funded by Tweag and Antithesis :sparkles:
Add a :+1: reaction to pull requests you find important.

Still works: https://github.com/Infinisil-s-Test-Organization/nixpkgs/pull/37#event-15709044331

@JohnRTitor A bit unfortunate but unfixable right now, because the workflow for this PR itself uses the version from the base branch, but fetches the code from the PR, which now takes an extra argument not passed in the workflow

@JohnRTitor A bit unfortunate but unfixable here, because the workflow for this PR itself uses the version from the base branch, but fetches the code from the PR, which now takes an extra argument not passed in the workflow

This is the reference to look at: https://github.com/Infinisil-s-Test-Organization/nixpkgs/pull/37#event-15696464782


client_max_body_size

I use the reverse proxy gen with webdav (nextcloud) but it will generate an error with large files. That seems because the parameter client_max_body_size 0;  is missing in the nginx proxy template and thus nog generated. How/where to add this?

See https://github.com/jwilder/nginx-proxy/issues/29 for some discussion around this

See this comment in #981.

Hi @pdwonline.
Maybe makes sense helping this issue being closed as soon as possible, since there are currently almost 400 open issues.
Not sure what's the suggested way in github, maybe editing the title and adding "[RESOLVED]" or "[DUPLICATED]" at the beginning?
Many thanks.


⚠ Slack has degraded performance

In 45b25a2, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 110 ms

Resolved: Slack performance has improved in 1025a2d.


Otherworld wood not dropping

In Minecraft 1.19.2 after planting otherworld dappling and getting third eye effect, I'm getting Minecraft oak wood instead of occultism otherworld wood.

I'm using forge 43.3.0
Mod version: occultism-1.19.2-1.90.0

What are you breaking the trees with?
I recall that in the older versions, if you do not break it with an axe or by hand, it just drops normal oak

I'm breaking the tree with hand or axe, neither works

So it just started working out of blue. I started testing some things on creative, and now everything works as it should on survival.


CI: Update libdav1d to 0.7.0-dmo1

Now that dav1d 0.7.0 is out, bump our CI dependency before the previous version is removed from the pool.

Coverage increased (+0.2%) to 82.382% when pulling 80e80954206a719f5977c3230610a48fa0df2921 on barrbrain:fix-ci into f0e09c84f165e21f2d38c9ec44a6f5ba2ca3894c on xiph:master.


Dynamic sound not working properly on native targets.

To demonstrate the problem, I have created this example project https://github.com/PaulGene/PitchShiftingSound
Flash: everything works as expected.
Windows: Works most of the time but sometimes SampleDataEvent.SAMPLE_DATA stops being fired when lime_sound_channel_needs_data(__handle); returns false at https://github.com/openfl/openfl-native/blob/master/flash/media/SoundChannel.hx#L114
Antroid: SampleDataEvent.SAMPLE_DATA gets fired about 3 times then we have the same problem as Windows, lime_sound_channel_needs_data(__handle); returns false at https://github.com/openfl/openfl-native/blob/master/flash/media/SoundChannel.hx#L114
iOS: lime_sound_channel_create_dynamic(request.data, soundTransform); always returns null at https://github.com/openfl/openfl-native/blob/master/flash/media/Sound.hx#L186

I too had problem with dynamic audio, I was able to fix them all using this fix: https://github.com/openfl/openfl/pull/488
I used http://www.tanjent.se/labs/tanjentxm.html to test the functionnality, got it working on Android, iOS, Windows, Mac, Flash


shallow does not reach created hook

Is this an expected behaviour? In that case? How can I stub child component without loosing the execution of created hook?
Thank you

I added this yesterday. I think it should be expected behavior. There currently isn't a way, but I can add an option to pass to shallow if you have a good use case.
 You can have a separate test for the component with the created event to check it is behaving as expected.
But like I said, I'm happy to add an option in to bypass stubbing child lifecycle events if you find it valuable. What is your use case?

Of course, I allways test component in isolation. In my case I want to test only one component but stubbing the child components inside it.
Is in this main component when created is never reached. I need to call some api (mocking it in my case) when created. But it is never mocked.
Why created hook does not exists in this case?

Oh sorry, I misunderstood. So created on the component you're testing is being stubbed? This isn't expected behavior, it's a bug. I'll look into it

Bug should be fixed in 2.1.2 👍

Thank you. Awesome work!


fix: add change method param on callback

Closes INS-1665.
Description
Currently, present in the latest develop. The method dropdown selection change does not change the actual dropdown selection.
Steps to test

create a graqhql request or http request
change the method dropdown to a different method

After this fix
The UI needs to change the method to the selected method for the change.
Before this fix
Nothing happens.

I tried adding unit tests. Every time I attempt, I get blocked or spend so much time on adding some mock stores for Redux Provider. I will skip this one for now.


Complete CI actions and update README

Hello! I am working on integrating this with our Javascript SDK and wanted to include install instructions in my setup documentation. I saw that the CI was a WIP from @karthik2804's PR #27 and wanted to finish up that work. I took those changes, updated the actions to remove warnings from the action runs and updated the README to reflect those changes. You can see a sample run on my fork here. Once this PR is merged it should create a canary release and we can also tag a v0.2.0 release to match the version from pluginify.toml.

@jpflueger could you please squash and sign your commits, please?
Thanks.

@radu-matei commits should be squashed and signed, ready to be merged but I don't have write permissions.


item-sliding fireSwipeEvent method

From @BugsBunnyBR on July 22, 2016 2:7
Short description of the problem:
The method fireSwipeEventis listed in the docs here but is marked as private here.
It would be nice to sync the docs and source.
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)
Plus, I would like to have a way to slide an item to open programatically. It is not always intuitive to users the slide movement, but a button with an action that animates the slide could help to show how. To enable that, a public version of something like fireSwipeEventwould help.
Which Ionic Version? 2.x
Thanks
Copied from original issue: driftyco/ionic#7398

Is there any way to programmatically call the fireSwipeEvent?
item-sliding is a really cool feature, but I'm not sure the slide action is intuitive to my users
A way to call the fireSwipeEvent on (click) would be brilliant
@BugsBunnyBR did you figure this out?

Fixed in master!

Hi @jjmax75 , I didn't find a proper way to do it. As this issue was moved to ionic-site I created another one just for the feature request (https://github.com/driftyco/ionic/issues/7440).
Thanks for the attention Mike! :-D

@jgw96 I have the same issue. I would like to programmatically fire swipe event on button click (button being against each list item). Could you pls share the code to do the same, and call fireSwipeEvent. Thanks!


Add possible limitations to guides

Serial requests
Throttling/debouncing

These are unsolved, important patterns!

Closing in favor of #165


maint: Update docker-compose.java.yml

Which problem is this PR solving?
When running the java front end service the following error is thrown because it cannot communicate with the collector.
frontend-service  | [otel.javaagent 2024-01-30 21:30:55:358 +0000] [OkHttp https://api.honeycomb.io/...] WARN io.opentelemetry.exporter.internal.grpc.OkHttpGrpcExporter - Failed to export metrics. Server responded with gRPC status code 16. Error message: unknown API key - check your credentials, region, and API URL

Short description of the changes

Added missing env variable for collector host

@nrichu-honeycomb is this something that needs to be done for all languages?

I had trouble replicating this. Can we confirm if this is still needed?

@nrichu-honeycomb I am going to close this for now as unable to replicate but if you are still interested in it please comment here or open an issue for further review. Thank you!


enforceFocus event.target - event is undefined

event.target should be e.target within enforceFocus method.

Tests passed. Automated cross-browser testing via Sauce Labs and Travis CI shows that the JavaScript changes in this pull request are: CONFIRMED
Commit: 9f2f51073cfbf3d2a4a16a901a60d184cbcfd4c4
Build details: https://travis-ci.org/twbs-savage/bootstrap/builds/102580546
(Please note that this is a fully automated comment.)

Argh. Good catch! Must've missed the difference in variable names when porting #18638 to v3.


INSTALL_PATH was dropped from Makefile in 075a35a6d390167b77b687e067dd0ba593e7f624

What steps will reproduce the problem?
1. Make leveldb on OS X
2. Note lack of absolute install path in `-install_name`
3. Attempt to build against leveldb dylib on OS X and fail

What is the expected output? What do you see instead?

g++  -dynamiclib -install_name libleveldb.dylib.1

instead of

g++  -dynamiclib -install_name /path/to/current/directory/libleveldb.dylib.1

What version of the product are you using? On what operating system?

Anything after SHA 075a35a6d390167b77b687e067dd0ba593e7f624

Please provide any additional information below.

It looks like when

$(SHARED3):
	$(CXX) $(LDFLAGS) $(PLATFORM_SHARED_LDFLAGS)$(SHARED2) $(CXXFLAGS) $(PLATFORM_SHARED_CFLAGS) $(SOURCES) -o $(SHARED3)

was made, the $(INSTALL_PATH) param was dropped completely.

The attached patch corrects this.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 21 Aug 2012 at 5:44

Merged into: #111

Attachments:

install_path.patch

Unclear how this go duplicated. Feel free to close this one in favor of 111.

Original comment by<EMAIL_ADDRESS>on 21 Aug 2012 at 5:53

Original comment by<EMAIL_ADDRESS>on 22 Aug 2012 at 6:08

Changed state: Duplicate


Add table of Medicare provider features

The Medicare Physician & Other Practitioners by Provider dataset provides summary stats on the patients the provider receives such as race composition, the services provided to those patients such as medical and prescription drug, and the dollar amounts reimbursed by Medicare (standardized and total) and quantity of services provided.

See #21 this will just be its own set of parquet files.


Bug: Pausing causes mute symbol to appear

Currently, pausing the game causes the mute symbol to appear over the sound, even though pausing does not affect whether the game is muted or not.
In theory, the fix should be that the mute button's appearance doesn't change when pausing. It also might be nice to have the pause button itself disappear when paused, since it doesn't have any functionality on the pause screen.

Update: I believe I've solved the problem with the mute button displaying oddly. Will submit a pull request accordingly.

Completed by @Germlord


[BUG] New test modal renders incorrectly when "no symptoms" box is checked

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
New test modal renders incorrectly when 'No Symptoms' checkbox is selected
Steps To Reproduce
From the conduct tests screen, search for a person and start a new test, notice that the modal splits and the 'continue' button occupies a separate box
Expected Behavior
Continue button does not get split into a second modal box when 'no symptoms' is selected
Additional context
https://drive.google.com/drive/u/0/folders/12KFSM-Hs9KdkyMuJ5rTYOYgqjYoFJURH
Does this issue have a priority label?

[X] I have added a priority label

Has issue been assigned to a team?

[X] I have added a team label

This was fixed!


Force-kill crawls when crawl is deleted.

Crawls can continue to run even after they have been deleted. This is a big problem and it could end up causing serious server problems. The solution is to force stop the crawl once it has been deleted.

This is really an issue for ache crawls, but it is still useful and important.


CMake build request.

I'm working for enable our IPEX CPU on Windows. Our project integrated libxsmm now, and libxsmm was integrated via ExternalProject: https://github.com/intel/intel-extension-for-pytorch/blob/main/csrc/cpu/CMakeLists.txt#L131-L150 . ExternalProject is not cross OS method.
At the same time, I found libxsmm only provide a MSVC solution file, https://github.com/libxsmm/libxsmm/tree/main/ide . It is not a good practice for CMake project.
Could you please enable origional CMake support for cross OS projects.

@jgong5

@alheinecke This is about the request for Windows build support for IPEX.

I'm not sure what you need, libxsmm has no cmake build, neither under windows, nor under Linux.
However, our MLIR compiler uses cmake and you guys might want to reuse the cmake scripts from there:
https://github.com/plaidml/tpp-mlir/blob/main/cmake/modules/xsmm.cmake

I'm not sure what you need, libxsmm has no cmake build, neither under windows, nor under Linux.
However, our MLIR compiler uses cmake and you guys might want to reuse the cmake scripts from there: https://github.com/plaidml/tpp-mlir/blob/main/cmake/modules/xsmm.cmake

@alheinecke Thanks for the pointer. @xuhancn Are we able to use the cmake to build libxsmm in IPEX?

I'm not sure what you need, libxsmm has no cmake build, neither under windows, nor under Linux.
However, our MLIR compiler uses cmake and you guys might want to reuse the cmake scripts from there: https://github.com/plaidml/tpp-mlir/blob/main/cmake/modules/xsmm.cmake

I still suggest you enable CMake in your libxsmm project.

On Windows, the MSVC solution file is limited MSVC version and runtime library version. CMake can solve this issue and genarate solution file on demand.
It is not bad MLIR has a example cmake file. But the parent project contains submodule's cmake, it is strange. I suggest move the example cmake file to libxsmm, it is the make sense solution.

We have no plan for supporting cmake, but you are more than welcome to submit a PR with cmake support.

We have no plan for supporting cmake, but you are more than welcome to submit a PR with cmake support.

https://github.com/libxsmm/libxsmm/pull/827 PR is here, Pls review it. @alheinecke , CC @jgong5


I want to see a list of all transactions

Unsure about how to create a method that will fire when the button is clicked

I don't understand what the Label is binding to/ what that binding is used for

How do I make another one of these with resharper? this is a property right?

@dtotino - adding a click event handler for a button is one of those very basic Visual Studio/WPF things you should be able to figure out with a google search or Pluralsight course.
For the label, the Text = ....syntax is doing the same thing as value.bind in Aurelia.  Whether you reference UserName in the code, it will update the label value.  When you update the UI, it will update UserName.
For the property, I normally type in prop (then Tab) to create an auto property.  An auto property just has getter and setter.  This makes it usable by others (for example the databinding in the UI) and allows you to intercept if needed.  In the example above, the get isn't doing anything special and there is no need to.  For the set, we make sure that the new value isn't the same as the old one before moving forward.  Then, we set the new value to the backing field (_userName).  Then, we call OnPropertyChanged.  Anyone who has subscribed to our PropertyChangedEvent (the UI has...this is the way WPF works), will be notified that we made a change.  At this point, the UI will get UserName (which just returns the value of _userName) and update the UI.
  With WPF, what you see is a very common pattern and becomes monotonous to enter it in all the time.  So, Resharper came up with a quick fix: you can read about it here:

@dukesta3 -- I am trying to figure out how you did the binding for the How Cool Am I part, but I don't remember how or where you started and the steps you took to make the binding happen. After understanding that, I want to try 2-way binding


Virtuemart

Is there a way to make it work in virtuemart?
I have  4 products per row . When i choose for itemselector the 'product class' or the 'row' class i get the message 'Sorry, we couldn't '
Any ideas?

Sorry to hear you're having trouble with Infinite Scroll.
Infinite Scroll v3 is out!. It's an all new re-write, so take a look at how to upgrade from v2.
I recommend upgrading to v3. Please open a new issue if you run into trouble with Infinite Scroll v3. 🌈🐻


Not Working on Windows 10 Build 1709

Hi Fyyre,
I am using your tool to disable patch guard on windows 10 build version 1709, after running the tool, I checked bcdedit and it is updated with patch guard disable and booted my PC using disable patchguard boot option, it seems that all is good, so to confirm it further I used https://gist.github.com/andronoob/d5882c5e6d21f3dea534a68434abe040 andronoob NTFS FSD hook filter and starting this service using following tool:
sc create FsFilter binPath= C:\temp\FsFilter1.sys type= kernel
sc start FsFilter
But I am getting blue screen and windows crashing, means patch guard is not disabled.
Can you please help me, what I am doing wrong? your quick response will be highly appreciated.
Thanks
BR,
Adeel

I tested with TitanHide and everything is working fine for me.

Hi Mrexodia,
Did you test it on Win10? if yes then which buid version, also could you please share .sys and inf files of TitanHide for windows 10?
How much time you ran the TintanHide to confirm patchguard ?
Thanks

Why would I post on an issue about win 10 1709 if I didn’t test there?
You can compile TitanHide yourself. As far as I know the vm is still running, but I’ll check in a moment.

Ah, the vm just bsod so it's not working...

So you are sure that with patchguard diable boot option on windows 10, you got BSOD, that means patchguard disable tool didnot work? right

Yes that is correct.
On Wed, 11 Jul 2018 at 12:06, adeelmunir<EMAIL_ADDRESS>wrote:

So you are sure that with patchguard diable boot option on windows 10, you
got BSOD, that means patchguard disable tool didnot work? right
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hfiref0x/UPGDSED/issues/30#issuecomment-404116384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWCmTSCj5IqFPR0NvEVcM9NcCyNTuxWks5uFc44gaJpZM4VI5br
.

Hi Fyyre,
could you please help us?
Thanks

Just to get the obvious out of the way, ..
.. FsFilter.sys is "FileSystem"-related, right ?
Maybe it's not meant to run in a VM and therefore not caused
by UPGDSED ?!

You should read the comments once more to not state the obvious:)
On Thu, 19 Jul 2018 at 08:35, nonspin<EMAIL_ADDRESS>wrote:

Just to get the obvious out of the way, ..
.. FsFilter.sys is "FileSystem"-related, right ?
Maybe it's not meant to run in a VM and therefore not caused
by UPGDSED ?!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/hfiref0x/UPGDSED/issues/30#issuecomment-406169953,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWCmZiZpXZqkx3B3tmBEXVIURqypG-Wks5uICiigaJpZM4VI5br
.

Thing is, i didn't state anything .. i did two of them 'question-things' .. obviously ;D

It is not unexpected. There was no guarantee that fast changing Windows 10 won't give any surprises. You should wait for @Fyyre reply, (as this project is entirely based on Fyyre researches). Or you can figure out patchguard changes on your own. Keep it mind that it possible to implement patchguard in a way which will render file patching useless, as system simple won't be able to boot without this piece of properly initialized DRM junk.

I admit to having lost much interesting in this project.  I am seeing it used by people mainly to cheat in online games, and credit is never correctly given to myself or hFiref0x for the work.

I admit to having lost much interesting in this project. I am seeing it used by people mainly to cheat in online games, and credit is never correctly given to myself or hFiref0x for the work.

Hi Fyyre please not stop update this project, is very useful for me, for malware analysis
I'm grateful, please not stop, this project is incredible !

Additionally to Fyyre words, you are free to fork, reverse fresh windows and pull request (optionally) with fixes/support whatever. We are not going to support this "forever". Closed.


Patch 1 Updated url in open_autojoomer function

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Any PRs should be done to the update branch only
Type of change
Please delete options that are not relevant.

[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)
[ ] This change requires a documentation update

Browsers Tested on?

[ ] Chrome
[ ] Firefox
[ ] Edge

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have checked the responsiveness of the component I built
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings

Hey @Nandana4, you missed one more place to update the url (in src/js/popup.js)
check this for the exact line:
https://github.com/Nandana4/AutoJoomer/blob/504a4f7db638d3e46f8c56c075203ff3c995d2f8/src/js/popup.js#L114
thank you

@Nandana4 reply on the main issue first to get assigned


Update to the latest version

Pull the latest version of u2f-api from https://github.com/google/u2f-ref-code/blob/master/u2f-gae-demo/war/js/u2f-api.js

This change is

Should we also update version in package.json?

I'll do the version bump in a separate commit.

I published this as v0.4.4.


[SPARK-26048][build] Enable flume profile when creating 2.x releases.

@dbtsai this doesn't need to be in the 2.4 branch but it would be nice to build the next RC with this change.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8395/
Test PASSed.

Test build #102922 has started for PR 23931 at commit e2d2e6c.

@vanzin Okay. I enabled -Pflume in my build for rc6. Thanks!

Test build #102922 has finished for PR 23931 at commit e2d2e6c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102922/
Test PASSed.

Hey, isn't flume support deprecated since 2.3? https://spark.apache.org/docs/2.3.0/streaming-flume-integration.html

ah got it. It's a followup of https://github.com/apache/spark/commit/a00181418911307725524641254439712e95445b#diff-01ca42240614718522afde4d4885b40d
Although we removed flume in 3.0, the release script should still include flume when releasing 2.x
LGTM

It's weird that with flume enabled, I got an error on signature validation on flume test jar when I published the jars to staging. Any idea?

Locally, or in the bundle that was pushed to Sonatype, do you see the hash / signature files for this module, or others? I'm not sure how it would be different, but I'd start there.

Interestingly, I download the files and run
gpg --verify spark-streaming-flume-assembly_2.11-2.4.1-tests.jar.asc

, and it works without any problem.
I guess it's a transient issue in sonatype. Do you know if I can ask sonatype to recheck the signature without building it again?

Hm, I suppose verification of the keys depends on it having your key. I wonder how it knows to download the KEYS file from Apache to verify? See https://central.sonatype.org/pages/working-with-pgp-signatures.html ; I wonder if you need to publish your signing key to one of these 'central' servers? What other log detail can you get about the error?


Fix-Spelling

Fixing the spelling of "Authentication" that was previously "Authetication".

Will submit a new one once master is updated.


CMake Error for zlib:x86-windows

.\vcpkg install curl[tool]
Computing installation plan...
The following packages will be built and installed:
curl[core,non-http,schannel,ssl,sspi,tool,winssl]:x86-windows -> 7.74.0#3

zlib[core]:x86-windows -> 1.2.11#9
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x86-windows...
Could not locate cached archive: C:\Users\Tanvi Palekar\AppData\Local\vcpkg\archives\f9\f9055bd2e31511a3e7cf0d2cda1af5ed91b357fd.zip
Could not locate cached archive: C:\Users\Tanvi Palekar\AppData\Local\vcpkg\archives\a6\a611b4a775a847047b5c58220a7f24f63edd9825.zip
Starting package 1/2: zlib:x86-windows
Building package zlib[core]:x86-windows...
-- Using cached C:/Users/Tanvi Palekar/vcpkg/downloads/zlib1211.tar.gz
-- Cleaning sources at C:/Users/Tanvi Palekar/vcpkg/buildtrees/zlib/src/1.2.11-63309e48e5.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source C:/Users/Tanvi Palekar/vcpkg/downloads/zlib1211.tar.gz
-- Applying patch cmake_dont_build_more_than_needed.patch
-- Applying patch 0001-Prevent-invalid-inclusions-when-HAVE_-is-set-to-0.patch
-- Applying patch add_debug_postfix_on_mingw.patch
-- Using source at C:/Users/Tanvi Palekar/vcpkg/buildtrees/zlib/src/1.2.11-63309e48e5.clean
-- Configuring x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:105 (message):
Command failed: ninja -v
Working Directory: C:/Users/Tanvi Palekar/vcpkg/buildtrees/zlib/x86-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
C:\Users\Tanvi Palekar\vcpkg\buildtrees\zlib\config-x86-windows-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:324 (vcpkg_execute_required_process)
ports/zlib/portfile.cmake:22 (vcpkg_configure_cmake)
scripts/ports.cmake:142 (include)
Error: Building package zlib:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: zlib:x86-windows
Vcpkg version: 2021-01-13-d67989bce1043b98092ac45996a8230a059a2d7e
Additionally, attach any relevant sections from the log files above.

@tp2043 ,Thanks for posting this issue,please send the error log C:\Users\Tanvi Palekar\vcpkg\buildtrees\zlib\config-x86-windows-out.log

config-x86-windows-out.log

@tp2043 ,CMake is not able to generate the project properly -
Check to make sure Visual Studio 2019 has the following installed:
Desktop Development with C++
One of the Windows SDKs, preferably the latest 19041
English Language Pack

Done and resolved. thank you


Hiro Explorer is missing additional inscriptions on satoshi ordinal page

Describe the bug
On https://ordinals.hiro.so/sat/1446791433368094 it only displays one inscription for this sat. However, there should be two since this API call returns two https://api.hiro.so/ordinals/v1/sats/1446791433368094/inscriptions
Transaction ID
a1c934dd35d5a3abeeba21ac026997e6eb9a0205a173ae32a999fc5b0818d66a
Address
bc1pgu7lvmsnw037v6g4693quckk8asz7mdh3rac4laftxzpv5e039uqc4h8qj
Block#
777530
Time stamp
FEBRUARY 20, 2023 AT 2:15:27 PM
To Reproduce
Steps to reproduce the behavior:

Go to https://ordinals.hiro.so/sat/1446791433368094

Expected behavior
It should display two inscriptions but it's only showing one.

FYI, @janniks, you may have some insights to share.


Domain name change isn't applied

Hello,
New domain name isn't applied after changing it in docker-compose.yml

Change

 # Management
  management:
    image: netbirdio/management:latest
    restart: unless-stopped
    networks: [netbird]
    volumes:
      - netbird_management:/var/lib/netbird
      - ./management.json:/etc/netbird/management.json
    command: [
      "--port", "80",
      "--log-file", "console",
      "--log-level", "info",
      "--disable-anonymous-metrics=false",
      "--single-account-mode-domain=nb.domain.tld",
      "--dns-domain=nb.domain.tld",
      "--idp-sign-key-refresh-enabled",
    ]
    logging:
      driver: "json-file"
      options:
        max-size: "500m"
        max-file: "2"

compose restart
container (netbirdio/management:latest) restart

My fault: docker compose up -d worked


Few questions

Hi, got a few questions:

Is it possible to reload the config without quitting and restarting the emulator? Would be good for trying sensitivity settings on the fly.
The Centering option doesn't seem to work (C hold). Also, can Alt+C be re-binded?
Is it possible to un-hide the cursor without quitting the emulator (when using KBM)?
How to force the emulator to use KBM if a gamepad is connected to the PC, it seems it always defaults to the gamepad.
Have you maybe tried your emu with RPCS3, it seems that something happens with sensitivity when you go fullscreen in that emulator, I apologize for asking for cross program compatibility, but if you have a minute or two just to check it out.

Hi,

Currently not possible, only keyboard & mouse profiles.
The latest version turns centering on and off on ALT + C (if it doesn’t work, make sure you have an English layout).
No, there is a problem with this, so it was done this way.
Currently there is no such function, but I will add it in the next version. You can restrict access to the program through HidHide as a temporary solution.
Perhaps RPCS3 has its own implementation of the keyboard and mouse, so if the sensitivity changes, the mouse is probably captured. I'll check.

Thanks for replying.
RPCS3 sees it as DS4, not as KBM (as is intended), just fullscreening it makes a change to sensitivity for some reason, which doesn't happen with either real KBM or gamepads, just a fyi.


Heightmap with modal gcodes

I don't know if I'm writing this correctly:
I have a post processor that creates a circle with modal values X Y I J such that it logically omits some, eg G3X0I10J0. If I apply Height map to such code, error 33 occurs on the first line with G3.
It seems to replace or add the missing value X to Z. So the line looks something like this:
G3X17Z0.196I-10J0
I tried different gcodes. I have no problem with this code before applying. Only after setting Y as nonmodal is everything fine. The problem is with all versions of the program.
For circle G3, must all values be non modal except Z?

Do you have an example (including a height map file)?

I will add the heigtmap and the modified file later:
Modal codes:
%
G21
G90
M6
S10000M3
G0Z6.000
G0X10.000Y5.000
G1Z-0.100F300.0
G17
G3X0.000I-5.000J0.000F100.0
G3X10.000I5.000J0.000
G0Z6.000
G0X10.000Y5.000
G0 X0.0000Y0.0000
M30
Nonmodal codes:
%
G21
G90
M6
S10000M3
G0Z6.000
G0X10.000Y5.000
G1Z-0.100F300.0
G17
G3X0.000Y5.000I-5.000J0.000F100.0
G3X10.000Y5.000I5.000J0.000
G0Z6.000
G0X10.000Y5.000
G0 X0.0000Y0.0000
M30

Original gcode:
%
G21
G90
M3 S10000
G0 Z6.000
G0 X20.000 Y10.000
G1 Z-0.100 F200.0
G17
G3X0.000I-10.000J0.000
G3X20.000I10.000J0.000
G0 Z6.000
G0 X20.000 Y10.000
G0 X0.0000 Y0.0000
M30
Heightmap:

0
0.029
0.032
0.06
0.064
0.049
0.072
0.073
0.092
0.105
0.081
0.109
0.133
0.132
0.143
0.147
0.162
0.169
0.174
0.179
0.181
0.193
0.2
0.198
0.196

Gcode after Apply:
G21
G90
S10000M3
G0Z6
G0X20Y10Z6.2
F200
G1Z3.15
G1Z0.1
G17
G3X19.01Z0.093I-10J0
G3X16.235Z0.082I-9.01J-4.339
G3X12.225Z0.059I-6.235J-7.818
G3X7.775Z0.025I-2.225J-9.749
G3X3.765Z-0.01I2.225J-9.749
G3X0.99Z-0.037I6.235J-7.818
G3X0Z-0.068I9.01J-4.339
G3X0.99Z-0.062I10J0
G3X3.765Z-0.052I9.01J4.339
G3X7.775Z-0.032I6.235J7.818
G3X12.225Z0.011I2.225J9.749
G3X16.235Z0.062I-2.225J9.749
G3X19.01Z0.088I-6.235J7.818
G3X20Z0.1I-9.01J4.339
G0Z6.2
G0X20Y10Z6.2
G0X0Y0Z6
M30

I belive I have fixed this, but I want to verify with a more comprehensive piece of gcode before release. This has to wait until I am back home later in January. Until then you may use File > Transform > Arcs to lines before applying the map as a workaround.

Can you try the latest edge version?, available from here.

Neither version 2.0.42p0 can be started, it reports an error.

I've tested the standard version after downloading the zip and it works for me.
Can you translate the error message?
Which controller are you using?

Info from 2.0.41
[VER:1.1f.20221101:]
[OPT:VNMZSL,35,1024,3,0]
[AXS:3:XYZ]
[NEWOPT:ENUMS,RT+,HOME,ES,TC,SED]
[FIRMWARE:grblHAL]
[NVS STORAGE:*FLASH]
[DRIVER:STM32F411]
[DRIVER VERSION:221014]
[BOARD:BlackPill]
The index is outside the field bounds. Probably the correct translation is : "Index Out of Range Exception".

Thanks, that helps. Fixed versions uploaded now.

Version 2.042p1 seems to work ! After loading the gcode, the circle appears as a hex, and after applying the heightmap, it appears as a "perfect" circle. Perfect, seems to be fixed, thanks...


fix(auth): use default scope account for Bitbucket auth

Fixes https://github.com/backstage/backstage/issues/22383
The scope team does not exist, a list of valid OAuth2 scopes can be found here:

https://developer.atlassian.com/cloud/bitbucket/rest/intro/#bitbucket-oauth-2-0-scopes

The passport-bitbucket-oauth2 package implementing OAuth2 for Bitbucket under the hood uses the /2.0/user API to retrieve the user information, which requires the account scope:

https://developer.atlassian.com/cloud/bitbucket/rest/api-group-users/#api-user-get
https://github.com/bithound/passport-bitbucket-oauth2/blob/master/lib/passport-bitbucket/strategy.js#L60

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@pjungermann

The test failure seems to be unrelated to the change as far as I can see.

The test failure seems to be unrelated to the change as far as I can see.

I've retriggered the failed job. All checks have passed now.

Thanks for fixing this & providing a good argumentation in the PR. I do trust @pjungermann with this being the right change of scope as I can't reproduce it for Bitbucket.
Open task: Please make sure to also adjust the respective test in the defaultApis.ts test-utils: https://github.com/backstage/backstage/blob/master/packages/test-utils/src/testUtils/defaultApis.ts#L186

Should have looked more closely for more occurences, thanks for the pointer! I additionally found https://github.com/backstage/backstage/blob/master/packages/core-app-api/src/apis/implementations/auth/bitbucket/BitbucketAuth.ts#L55 and pushed an update for this as well.

@pjungermann could you retrigger the failed checks please? There were some OOMs and segfaults that seem unrelated to the change.

@tudi2d Maybe you could have a second look if you got some time. From my side this should be ready to go now.


[Event Request] Codeunit 414 - "Release Sales Document" - OnBeforeCalcInvDiscount

Dear ALAppExtensions team,
through this issue we would like to request adding a parameter to event publisher OnBeforeCalcInvDiscount in Codeunit 414 "Release Sales Document" with the following signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeCalcInvDiscount(var SalesHeader: Record "Sales Header"; PreviewMode: Boolean; var LinesWereModified: Boolean; var SalesLine: Record "Sales Line"; var IsHandled: Boolean)
begin
end;

which would be used to control the execution of functionality related to SalesSetup."Calc. Inv. Discount" in procedure Code as follows:
local procedure "Code"() LinesWereModified: Boolean
   var
       SalesLine: Record "Sales Line";
       PrepaymentMgt: Codeunit "Prepayment Mgt.";
       NotOnlyDropShipment: Boolean;
       PostingDate: Date;
       PrintPostedDocuments: Boolean;
       ShouldSetStatusPrepayment: Boolean;
       IsHandled: Boolean;
   begin
       with SalesHeader do begin
           if Status = Status::Released then
               exit;

           IsHandled := false;
           OnBeforeReleaseSalesDoc(SalesHeader, PreviewMode, IsHandled, SkipCheckReleaseRestrictions);
           if IsHandled then
               exit;
           if not (PreviewMode or SkipCheckReleaseRestrictions) then
               CheckSalesReleaseRestrictions();

           IsHandled := false;
           OnBeforeCheckCustomerCreated(SalesHeader, IsHandled);
           if not IsHandled then
               if "Document Type" = "Document Type"::Quote then
                   if CheckCustomerCreated(true) then
                       Get("Document Type"::Quote, "No.")
                   else
                       exit;

           TestSellToCustomerNo(SalesHeader);

           IsHandled := false;
           OnCodeOnAfterCheckCustomerCreated(SalesHeader, PreviewMode, IsHandled, LinesWereModified);
           if IsHandled then
               exit;

           CheckSalesLines(SalesLine, LinesWereModified);

           OnCodeOnAfterCheck(SalesHeader, SalesLine, LinesWereModified);

           SalesLine.SetRange("Drop Shipment", false);
           NotOnlyDropShipment := SalesLine.FindFirst();

           OnCodeOnCheckTracking(SalesHeader, SalesLine);

           SalesLine.Reset();

           // >>>OLD>>>
           // OnBeforeCalcInvDiscount(PurchaseHeader, PreviewMode, LinesWereModified, SalesLine);
           // <<<OLD<<<
           // >>>NEW>>>
           IsHandled := false;
           OnBeforeCalcInvDiscount(SalesHeader, PreviewMode, LinesWereModified, SalesLine, IsHandled);
           // <<<NEW<<<
           SalesSetup.Get();
           // >>>NEW>>>
           if not IsHandled then
           // <<<NEW<<<
           if SalesSetup."Calc. Inv. Discount" then begin
               PostingDate := "Posting Date";
               PrintPostedDocuments := "Print Posted Documents";
               CODEUNIT.Run(CODEUNIT::"Sales-Calc. Discount", SalesLine);
               LinesWereModified := true;
               Get("Document Type", "No.");
               "Print Posted Documents" := PrintPostedDocuments;
               if PostingDate <> "Posting Date" then
                   Validate("Posting Date", PostingDate);
           end;

           IsHandled := false;
           OnBeforeModifySalesDoc(SalesHeader, PreviewMode, IsHandled);
           if IsHandled then
               exit;

           ShouldSetStatusPrepayment := PrepaymentMgt.TestSalesPrepayment(SalesHeader) and ("Document Type" = "Document Type"::Order);
           OnCodeOnAfterCalcShouldSetStatusPrepayment(SalesHeader, PreviewMode, ShouldSetStatusPrepayment);
           if ShouldSetStatusPrepayment then begin
               Status := Status::"Pending Prepayment";
               Modify(true);
               OnAfterReleaseSalesDoc(SalesHeader, PreviewMode, LinesWereModified);
               exit;
           end;

           OnCodeOnBeforeSetStatusReleased(SalesHeader);
           Status := Status::Released;

           LinesWereModified := LinesWereModified or CalcAndUpdateVATOnLines(SalesHeader, SalesLine);

           OnAfterUpdateSalesDocLines(SalesHeader, LinesWereModified, PreviewMode);

           ReleaseATOs(SalesHeader);
           OnAfterReleaseATOs(SalesHeader, SalesLine, PreviewMode);

           Modify(true);
           OnCodeOnAfterModifySalesDoc(SalesHeader, LinesWereModified);

           if NotOnlyDropShipment then
               if "Document Type" in ["Document Type"::Order, "Document Type"::"Return Order"] then
                   WhseSalesRelease.Release(SalesHeader);

           OnAfterReleaseSalesDoc(SalesHeader, PreviewMode, LinesWereModified);
       end;
   end;

Thanks in advance!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10879.


CompileTimeErrorCodes without tests

There appear to be no test for either CompileTimeErrorCode.MIXIN_INFERENCE_INCONSISTENT_MATCHING_CLASSES or CompileTimeErrorCode.MIXIN_INFERENCE_NO_MATCHING_CLASS.
These codes should either have tests associated with them or they should be removed.

Neither of these codes exist any longer.


Add about page

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
I want to add about page to the website as there is no about section, with better UI/UX  please assign me this issue under GSSOC'24.

Dupliacte issue


When will the MambaIRv2 code release?

Thx for your interest, the code and weights are updated now! enjoy yourself ;D


Structure changes

HIi Tobi, now for container, I've make seperate variable in taiwing config file for container. In future whenever we want to change container size, we can change at only one place and changes will reflect to evrywhere in our website:)

How's this now? @tobySolutions


Cannot multiple sort

How to use multiple order/ sort mongodb?
Thank's

Looks like this issue can be closed?


Can you specify the exact RBAC required

Can you specify the minimum RBAC required for accessing proxy/stats/summary k8s API?

Yup, @ketankr9 check out: https://github.com/yashbhutwala/kubectl-df-pv#requirements !!  If that's not clear enough, please free to send a PR.

Checked it already, it just says appropriate RBAC. I was hoping there would be definite permission narrower than cluster-admin.

"specifically, you need a service account with enough RBAC privileges to access api/v1/nodes/ from the kube-apiserver"... I didn't exactly explore what the yaml for that looks like, so you would have to play with the rbac yaml https://kubernetes.io/docs/reference/access-authn-authz/rbac/

You are right @fabiorauber permission over resource nodes/proxy worked for me 👍 . Forgot to update this thread.


Timespan is removed completely

I am very new to NVIVO and your helpful tool, so please forgive me if this is user error.
When I use your tool on my Team-generated docx file (which contains timespans, speakers and content) it creates a tab-delimited file of the format:
<speaker>\t\t<content>
I've tried poking around in the tool and file and can maybe bodge an importable file (although import doesn't like the fact i have no timespan). Is this due to a recent update in Teams transcription?
Attached is a mwe Teams transcription and the resulting NVivo txt
interviewmwe_2024-07-05.docx
interviewmwe_2024-07-05.txt

with experience, I've managed to overcome this - my problem was that I was editing (anonymising) the speaker label before conversion - I guess removing some vital spacing. If I do this after conversion it works really well.

Not an error.


Bugfix/1307 pythonic params

Fixes # .
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 14 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 97.037%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
2816

Relevant Lines:
2902

💛  - Coveralls


Allow code gen when only protos exist

@alexespencer can you document this behavior in the book eventually?
Thank you!


[BUG] Links in terminal maintain color, but not if we export to HTML

I'm working on a little cron-dashboard. This is what it looks like; from the terminal.

The idea is that my app will keep track of where all the logs are and that you can easily find the logs of the jobs that failed. But I don't just want to have users find logs in the terminal. I also want to export these to HTML. So I was super stoked when I learned rich basically just supports this.
I've printed the table via a Console which I've set to record and I've taken the liberty to also export it to HTML. This is where there is a subtle bug. Notice the subtle difference in the link-line under the red x.

I might imagine that we'd like to keep the colors consistent here. The link in the browser should carry the color of the parent. I think this is a bug, but if this is by design I'll gladly hear it.
Code
This is the code that is used inside the table.
f"[red][link={make_link()}.html]x[/link][/red]"

Hi Vincent,
I think that's due to the order of the  and  tags. It's probably generating X So the link style is taking preference. It should probably be X .
I'll have a look a that shortly. Unless you want to tackle that yourself? It should be an easy fix in export_html.

I've had a quick look but the code-base looked a tad bit too much to take in all in one go. I might try again later, but don't let me hold you!

Got a fix. Will be in the next version.

Please try 9.11.1


Add cornerRadius(_:) modifier

only CGFloat, no Extended support
border is not adjusted (also clipped)

Does Layer need corner radius or shape properties in order to adjust border?

Hey, sorry about that! This was just an experiment. I intended to create the pull request on a fork but got a bit distracted. I'm closing it now.


Added applications info to stop distillery from failing

distillery fails with the following error when I try to create a release. Adding applications to the app.src file fixes it. I am not sure if distillery is at fault here. Please let me know if this is an incorrect fix
==> Failed to build release:

    detergent: Missing parameter in .app file: applications

Closing stale PR, @devinus please reopen and merge if this is still an issue :slightly_smiling_face:


Enable airport load without need to gulp build

In order to avoid turning off potential airport contributors from being scared away by npm, it would be good to have the ability to add airport json files that can be loaded on a page refresh without requiring rebuilding of the bundles.
This way, potential contributors who want to see their home airport added can do the work on it, and then the merging collaborator can simply take their work and put it where it belongs, rebuild the bundles, and off you go-- without them having to deal with the (pretty significant) hassle of installing npm and all the modules and knowing how to deal with all these command line things.

Resolved by #731. Closing.


Polish the CorsFilter based on resteasy + add it to cli

Tests are failing I need to investigate since it was working on my machine.

I have no idea because I didn't copy paste the entire code and rewrote some part. Maybe I should add a comment and a link to the resteasy file saying that I was inspired by their implemementation?

I faced a similar situation with MIT code elsewhere in the code base and retained the copyright even if the code evolved. https://github.com/baremaps/baremaps/blob/main/baremaps-postgres/src/main/java/com/baremaps/postgres/jdbc/CopyWriter.java
Notice that we shouldn't integrate portions of GPL code as it is more restrictive than ASL, MIT or BSD.

Regarding the tests, it is failing with the integration tests are you running mvn install -P integration?

I figured it out just after writing my comments. I'll work on it a bit to fix the test.

I added the reateasy licence header.
I removed the test for CORS because jdk11 restrict setting some http header (https://github.com/AdoptOpenJDK/openjdk-jdk8u/blob/master/jdk/src/share/classes/sun/net/www/protocol/http/HttpURLConnection.java#L276-286) It saddens me in every possible way...

Interesting, I havn't used the new HTTP client very often. Do you think this is because CORS headers are a browser thing?

I have no idea why these headers are restricted... Do you want me to use another HTTP client lib ? I'm pretty rusty when it comes to choosing Java libs.

The openapi module uses Jersey Test. Here is an example produced by @b4l:
https://github.com/baremaps/baremaps/blob/tilesets/baremaps-openapi/src/test/java/com/baremaps/openapi/services/StylesServiceTest.java
In general, when writing basic tests, I simply use HTTPUrlConnection which allows to set custom headers:
URL url = new URL("http://example.com/");
HttpURLConnection conn = (HttpURLConnection) url.openConnection();

In general, when writing basic tests, I simply use HTTPUrlConnection which allows to set custom headers:
URL url = new URL("http://example.com/");
HttpURLConnection conn = (HttpURLConnection) url.openConnection();

As you can I'm using the "new" Http client that appear in JDK9: https://github.com/baremaps/baremaps/pull/310/files#diff-79032f8befc30a3655833a26b20cb85c4851129484c47c0cf06651f329583316R125
it's quite elegant and works well


SILGen: Emit "trivial" property descriptors for properties that withhold no information about their implementation.

Client code can make a best effort at emitting a key path referencing a property with its publicly exposed API, which in the common case will match what the defining module would produce as the canonical key path component representation of the declaration. We can reduce the code size impact of these descriptors by not emitting them when there's no hidden or possibly-resiliently-changed-in-the-past information about a storage declaration, having the property descriptor symbol reference a sentinel value telling client key paths to use their definition of the key path component.

@swift-ci Please test


Firmware Updates

TomTom seem to bring out firmware updates very regularly, and as the current fix involves modifying ttwatch.c and recompiling, which may concern some users, any chance of enabling a default setting?
The most likely default would be the last entry in the MANIFEST_DEFINITIONS table, since that is what most people do.  If this is considered too dangerous to do automatically, then at least it could be an option to enable it.  In all cases it should still print out that the firmware version isn't found.
What are your thoughts?

I've thought about this before, but thought it would be too dangerous. I could possibly allow the program to read settings, but not write them if the firmware version was not found. Perhaps a "force" option could be added if people really wanted to write settings.

Yep, understandable, and yes, read-only would be a reasonable solution.

I have implemented the ability to read settings if the firmware version has not been found. It still prints out that the firmware version has not been found. A --force option has been added to allow writing them as well, if people are feeling brave...


#2764 Fix inferring urls

Closes #2764
This PR fixes the inferring of the URL as follows: In case a user uses deployment strategy direct or blue_green_service, the DeploymentURIsLocal and DeploymentURIsPublic are always inferred regardless they are already present in the incoming event.
Signed-off-by: agrimmer<EMAIL_ADDRESS>

@warber Thanks for the hint! I added respective tests.


Added JASMINE env when running unit tests

The JASMINE environment variable can be used to determine when
in unit test mode in case you wanna make sure something doesn't
happen when unit testing, like, perhaps, making an un-mocked API call.

Coverage remained the same at 87.48% when pulling 3000c508d0157f4dd168bd47ddcb84ba4281619c on job13er:add-jasmine-env into 40962b42e4cbb5969d62531d7da9209730b6097f on cyaninc:master.

Looks good :+1:


Add .readthedocs.yml

Our RTD builds have been failing for weeks; trying to fix.

FYI this wasn't our fault: https://github.com/readthedocs/readthedocs.org/issues/7605


Template help: wrap tags in brackets

Hi!
First of all, amazing plugin, exactly what I was looking for.
I'm trying to edit the template a bit. I'd like the tags to be links to notes, so every tag wrapped in [[{{tag}} ]]
The nunjuck docs aren't helping me... Maybe someone here can?

Found out how to do it!
Change the {{ }} after Topics:: to this:
{{ '[[' + tags | join("]], [[") + ']]'}}


Do we want to change to mergo to merge manifest json in v5?

Take a deeper look at: https://github.com/imdario/mergo/

I guess it's time to close this issue as there was no real interest in the migration.


"cvssv3.1_qr" Qualitative Severity Rating Scale should say "Medium" instead of "MODERATE"

From the linked docs, its seems to me "MODERATE" should actually say "Medium":
https://github.com/nexB/vulnerablecode/blob/d7d3206ba1af98410b91655bfc96a147175eca75/vulnerabilities/severity_systems.py#L81-L82

@sschuberth thank you for your eagle eyes!


maccy: 0.28.0 -> 0.31.0

Description of changes
Update maccy to 0.31.0. Changes: https://github.com/p0deje/Maccy/releases and https://github.com/p0deje/Maccy/compare/0.28.0...0.31.0
Things done

Built on platform(s)

[ ] ~x86_64-linux~ (n/a)
[ ] ~aarch64-linux~ (n/a)
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 323636 run on x86_64-darwin 1

1 package built:

maccy


Azure MFA integration with ADFS required?

PIN reset from Windows login screen works for me only after enabling Azure MFA on ADFS (2019) for a federated domain. Is this an official prerequisite?

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

ID: d9dae9da-eaf0-c209-7c65-be8d2c1ada12
Version Independent ID: 28b6788d-7982-23c2-5f19-f80690ebc0ce
Content: Pin Reset - Microsoft 365 Security
Content Source: windows/security/identity-protection/hello-for-business/hello-feature-pin-reset.md
Product: w10
Technology: windows
GitHub Login: @mapalko
Microsoft Alias: mapalko

Hello @pmeuser ,
Hybrid Windows Hello for Business deployment is official prerequisite for Pin reset.
Windows Hello for Business in turn requires MFA deployment (it could be Azure MFA or third-party MFA)
More detail are in Windows Hello for Business
Thank you

@pmeuser - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by MaratMussabekov  so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


How to Determine the Dataset Used for Pretrained Models from Checkpoints?

Describe the bug
Pretrained model checkpoints are often used in pyabsa, but it's not always clear which datasets were used for their training. How can one determine the datasets used to train these checkpoints?
请问怎么样可以知道代码中的checkpoints是在哪些数据集上训练所得的呢？

比如aspect_opinion_sentiment_category_extraction这一任务

Refer to: https://huggingface.co/spaces/yangheng/PyABSA/blob/main/checkpoints-v2.0.json

Thank you. I have found the corresponding training dataset in checkpoint file.
But i wonder are those datasets used for APC, ATEPC or text classification in https://github.com/yangheng95/ABSADatasets/tree/v1.2 suitable for ACOS task?
Or is there anyplace storing dataset for ACOS task?

https://github.com/yangheng95/ABSADatasets/tree/v2.0/datasets/acos_datasets

thank you soooo much :)


Potential BUG in pack.py?

I encountered an AssertionError while using the code in the algorithm branch to quantize the LLaMA3.1-8B model with the setting num_res_centroids > num_centroids. I believe that setting num_res_centroids > num_centroids is a valid configuration. It seems that this issue might be related to line 128 in pack.py. Specifically, I propose the following change:
#res_indices = ((unpack_indice >> index_bits) & ((1 << index_bits) - 1)).view(torch.uint64).to(torch.int64)
res_indices = ((unpack_indice >> index_bits) & ((1 << res_bits) - 1)).view(torch.uint64).to(torch.int64)

Could you please review this part of the code and confirm if this modification is appropriate?

Let me check, thanks for your feedback!

I think it is a bug, and can you help me to pull a request to fix it. You can be a contributor of the project. Thanks!

merged! Thanks for your contribution!