Issue when trying to clone the repository

Cloning into 'C:\Users\zolta\OneDrive\Documents\GitHub\cardano-client-lib'...
remote: Enumerating objects: 1478, done.
remote: Counting objects: 100% (1444/1444), done.
remote: Compressing objects: 100% (652/652), done.
remote: Total 1478 (delta 567), reused 1206 (delta 357), pack-reused 34
Receiving objects: 100% (1478/1478), 298.95 KiB | 1023.00 KiB/s, done.
Resolving deltas: 100% (567/567), done.
Submodule 'cardano-serialization-lib' (https://github.com/bloxbean/cardano-serialization-lib.git) registered for path 'cardano-serialization-lib'
Cloning into 'C:/Users/zolta/OneDrive/Documents/GitHub/cardano-client-lib/cardano-serialization-lib'...
remote: Enumerating objects: 1416, done.
remote: Counting objects: 100% (271/271), done.
remote: Compressing objects: 100% (214/214), done.
remote: Total 1416 (delta 169), reused 97 (delta 57), pack-reused 1145
Receiving objects: 100% (1416/1416), 685.37 KiB | 1.98 MiB/s, done.
Resolving deltas: 100% (891/891), done.
fatal: reference is not a tree: 8c0f517ec39c333369462659b6c350223619973b
Unable to checkout '8c0f517ec39c333369462659b6c350223619973b' in submodule path 'cardano-serialization-lib'
Would you like to retry cloning ?

It's working for me. Can you please try again.
bash-3.2$
bash-3.2$ git clone https://github.com/bloxbean/cardano-client-lib.git
Cloning into 'cardano-client-lib'...
remote: Enumerating objects: 1556, done.
remote: Counting objects: 100% (1522/1522), done.
remote: Compressing objects: 100% (684/684), done.
remote: Total 1556 (delta 599), reused 1271 (delta 376), pack-reused 34
Receiving objects: 100% (1556/1556), 316.60 KiB | 17.59 MiB/s, done.
Resolving deltas: 100% (599/599), done.
bash-3.2$ cd cardano-client-lib/
bash-3.2$ git submodule update --init --recursive
Submodule 'cardano-serialization-lib' (https://github.com/bloxbean/cardano-serialization-lib.git) registered for path 'cardano-serialization-lib'
Cloning into '/Users/satya/work/release/cardano-client-lib/cardano-serialization-lib'...
Submodule path 'cardano-serialization-lib': checked out '8c0f517ec39c333369462659b6c350223619973b'
Submodule 'binaryen' (git@github.com:WebAssembly/binaryen.git) registered for path 'cardano-serialization-lib/binaryen'
Cloning into '/Users/satya/work/release/cardano-client-lib/cardano-serialization-lib/binaryen'...
Submodule path 'cardano-serialization-lib/binaryen': checked out '547e2be14d1f8f9b21905baee87b1946045b0fe7'
bash-3.2$
bash-3.2$

Well, what I did was: I downloaded the full repository from GitHub as ZIP and placed it into the very same directory to where I tried to clone and then asked GitHub Desktop just to sync and since then it seems to be fine.
I have successfully built it for Windows,I have a small correction for you guys: the "build-windows-x86_64.bat" file inside the "scripts" folder needs a "cd .." to the very first line.

Good to know that it's working now.
The script needs to be executed from the project's root folder.  That's how the current CI scripts are configured. So we may not need cd ..


api: v2alpha1: Add labels_per_unit to NetworkService.Info endpoint

https://github.com/spacemeshos/api/pull/365

bors merge

bors merge

bors merge

bors merge


need for adjoint fixes maintenance?

Every now and then I receive email having to do with updated versions of the MITgcm / ECCO adjoint, recompilations, and so-on. Creating an issue to share them here seems like the most useful thing I am able to do at this point. Not sure who is the owners is in charge of this part so I am just pinging @MITgcm/mitgcm-pr-mergers here.
In relation to this, I suggested a few times in the past to the MITgcm owners that maybe adding a maintainer for the adjoint to @MITgcm/mitgcm-pr-mergers would be useful. I don't think we really reached any conclusion on that but I am including this idea here with the secret hope that someone might volunteer to take on this role.

One adjoint related issue reported by @emmomp is https://github.com/ECCO-GROUP/ECCO-v4-Configurations/issues/39

Hi Gael -- I was going to email mitgcm-devel but since you opened this up it might be a good way to ask regarding gentim2d. I don't think this is an outright bug, but i think a change in expected usage. I recently used gentim2d and adjoint sensitivities are not being written to the file specified in data.ctrl (adxx_gentim2d1) but instead to adxx_gentim2d1.effective, while adxx_gentim2d is all zeros. Do you know what could potentially cause this behavior? thanks,dan

@dngoldberg I added this as an issue - and pinged @timothyas!

@gaelforget etc... definitely feel free to post the emails as issues, if sender is OK. I think some people prefer one on one emails. Thats OK too, but then hard for anyone else to know about. mitgcm-support is a good middle ground for folks who aren't comfortable posting issues.

I think this issue can be closed now. Thanks to @gaelforget and @christophernhill


Convert to default installation of cargo and replace --with-cargo option with --without-cargo option

Closes #81
Also see the discussion in https://rust-lang.zulipchat.com/#narrow/stream/217417-t-compiler.2Fcargo-bisect-rustc/topic/unrecognized.20option.20json
This PR converts the --with-cargo optional behavior to default behavior to address  Unrecognized option: 'json' errors in tests as bisection proceeds back in time past a defined date (I believe that this is ~ July 2019'ish era).  The current default leads to incorrect automated detection of the regression commit due to this error.  These changes also remove the --with-cargo option.

Looks good but I wonder if we do want a --without-cargo kind of flag to turn this off.

Looks good but I wonder if we do want a --without-cargo kind of flag to turn this off.

Anyone opposed to this?  I'll add it this weekend if this is the only thing blocking the merge here.

Rebased on master @ https://github.com/rust-lang/cargo-bisect-rustc/pull/98/commits/c5e86ee1c72f061bfbaaf80978737baa4bc1e19f

--without-cargo flag added in 2b7088c7f28fbf2dda7023a8fc30c092ab7764a1

@spastorino Please let me know if there is anything else that you need for this PR.  Thanks Santiago!


[Azure.AI.TextAnalytics.Authoring] Standalone package for the authoring client

As per our agreement, we decided to publish the TA authoring client as a standalone beta package until we have the chance to consolidate both the authoring and runtime packages into one package.
Remaining work:
Recording the tests.

One main question: all the work you did here could easily have been done in the existing package. We don't have separate authoring packages for the other Language clients. Why release a separate package when even less work is required than this to add it to the existing package?
Who was this discussion about releasing a separate package with?

This discussion was done a while ago (almost a month ago) in a long email thread titled "Authoring Client Status". I'm not sure whether you were included but Kareem, Krista, and Johan were there, and they came to the conclusion that currently it is better to have a separate package for the authoring clients for both Python and .NET as they had concerns about the maintainability of the authoring client in the same package as the runtime one.
Unfortunately, my internship has ended and I no longer have access to my work email to forward the thread to you. The SDK is handed over to @AhmedLeithy to continue working on and you can talk to Kareem or Krista to have a better understanding of the situation.

Thanks. I'll take this offline with them.


chore: migrate instructions to learn

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the main branch of freeCodeCamp.
[X] I have tested these changes either locally on my machine, or GitPod.

Closes #43836

This PR migrates the last of the python project instructions to /learn.

I noticed https://github.com/freeCodeCamp/boilerplate-cat-and-dog-image-classifier/pull/4 Do we want to remove the instructions from all the boilerplates?

Well, they mostly all have the instructions you moved here. The book recommendation project for example, has all the same instructions that will be added here. You're saying we don't want to remove those?

It's fine to leave them for now, just wondering if we want to or not.

I could remove the instructions, but we should probably leave the "watch this video about collab" link because it's a helpful resource and would make less sense on /learn IMHO.


Creating Worlds

Hi, I got a new Problem,
I wanted to create a world with this command:
/world create SkyClaims -d minecraft:overworld -g overworld -m sponge:skylands
And all I get is an Error where the Plugin seems to be asking for a "GAMEMODE", I wonder why a world would want a gamemode for creation, well yeah I dont seem to get it working.
Please Help.

Sounds like Nucleus, not ProjectWorlds. Most likely a command conflict


Add adopter logos

@estesp This isn't all of them, as some of the projects don't appear to have official logos just yet, but I think that this is a good start. You can see the new section here:
https://deploy-preview-27--containerd-io.netlify.com/#adopters

Looking good; based on comments on the containerd graduation proposal in the CNCF, we should add Kata containers logo, and now AWS announced "Firecracker" early this week, which uses containerd as well--so we might as well add their logo too.

@estesp Done. There were a few projects that didn't have logos AFAICT. I'll be on the lookout for those, but this should be good to go for now.

You'll need to squash your commits

@crosbymichael Done

LGTM


Time Input

Time and Date-Time Patterns
Problem Statement
We have developed a compositional date picker component. However, there are new requirements to support time input and date-time input. To address these requirements effectively, we need to identify and implement appropriate design patterns.
Requirements

Time Input:

Users should be able to input and select time values.
The component should support various time formats (e.g., 12-hour and 24-hour formats).
The time input should be intuitive and user-friendly.

Date-Time Input:

Users should be able to input and select both date and time values in a single component.
The component should seamlessly integrate date and time selection.
The date-time input should be flexible and accommodate different use cases.

Does anyone have an existing use-case ? If so, please comment on this issue

The native time input and also StepperInput may offer an interim solution
Pushing to Q1 until business priorities align and this is required.


A framework friendly satisfies-like (ability to disregard type information for specific inference).

Suggestion
🔍 Search Terms

Specific
Satisfies
Extract type information
Ignore type information
Endless other searches on SO and this repo, even tried ChatGPT!

✅ Viability Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
Thanks to the great work of @a-tarasyuk, we now have satisfies in typescript, for which the explanation goes:

TypeScript developers are often faced with a dilemma: we want to ensure that some expression matches some type, but also want to keep the most specific type of that expression for inference purposes.

satisfies works great for apps, where the whole codebase is in your control. But a framework wishing to extract specific type information from an argument is at the mercy of consumers using satisfies (and often as const in the case of arrays).
In other words, consumers are forced to use satisfies and const in order for the framework to work correctly.
I'll leave it for others to come up with the best solution, but the first thing that came to my mind is a utility such as Untype or Literal that takes a type and strip any type information - returning what would have been a const literal (for the inference of specifics).
📃 Motivating Example
A a minimal example can be seen in this SO thread.
⚠ I think it would be for the benefit of everyone if those who think this is doable without satisfies or as const post their solutions on the SO thread rather than in this issue.
Here, I'll share a real use-case.
Consider an editor that can be extended using an arbitrary number of extensions.
interface Extension {
  actions: Record<string, Action>,
  // ...
}

interface Editor {
  // a merge between all the `actions` in `Extensions`.
  actions: Record<string, Action>,
}

Usage:
const editor = createEditor([{
  actions: {
    a: () => {},
    b: () => {}
  }
},{
  actions: {
    c: () => {},
  }
}]);

const { actions } = editor;
// actions = {
//    a: () => {}
//    b: () => {}
//    c: () => {}
// }

The last example would work if we add as const satisfies Extension[].
But more often than not extensions will be imported from elsewhere, where authors will have to write:
export myExtensions = {
  //..
} satisfies Extension;

instead of
export myExtensions: Extension = {
  //..
};

Further, extensions may be broken down further (say actions in a separate script) so each type related to extensions will have to use satisfies.
The crude idea is to be able to do something like this:
// TBuildReturnType gets a type as if `extensions` were provided as a const literal (no type information).
function createEditor<T extends Extension[]>(extensions: T): TBuildReturnType<Untype<T>> { ... }

💻 Use Cases
To overcome this issue, TipTap uses global module declarations, which works if you only have one editor (or identical editors) in your app.
It should also be noted that such feature should help reducing multiple function declarations, such as those (from redux), since you could extract specific type information from an arbitrary list of items:
export default function applyMiddleware(): StoreEnhancer
export default function applyMiddleware<Ext1, S>(
  middleware1: Middleware<Ext1, S, any>
): StoreEnhancer<{ dispatch: Ext1 }>
export default function applyMiddleware<Ext1, Ext2, S>(
  middleware1: Middleware<Ext1, S, any>,
  middleware2: Middleware<Ext2, S, any>
): StoreEnhancer<{ dispatch: Ext1 & Ext2 }>
export default function applyMiddleware<Ext1, Ext2, Ext3, S>(
  middleware1: Middleware<Ext1, S, any>,
  middleware2: Middleware<Ext2, S, any>,
  middleware3: Middleware<Ext3, S, any>
): StoreEnhancer<{ dispatch: Ext1 & Ext2 & Ext3 }>
export default function applyMiddleware<Ext1, Ext2, Ext3, Ext4, S>(
  middleware1: Middleware<Ext1, S, any>,
  middleware2: Middleware<Ext2, S, any>,
  middleware3: Middleware<Ext3, S, any>,
  middleware4: Middleware<Ext4, S, any>
): StoreEnhancer<{ dispatch: Ext1 & Ext2 & Ext3 & Ext4 }>
export default function applyMiddleware<Ext1, Ext2, Ext3, Ext4, Ext5, S>(
  middleware1: Middleware<Ext1, S, any>,
  middleware2: Middleware<Ext2, S, any>,
  middleware3: Middleware<Ext3, S, any>,
  middleware4: Middleware<Ext4, S, any>,
  middleware5: Middleware<Ext5, S, any>
): StoreEnhancer<{ dispatch: Ext1 & Ext2 & Ext3 & Ext4 & Ext5 }>
export default function applyMiddleware<Ext, S = any>(
  ...middlewares: Middleware<any, S, any>[]
): StoreEnhancer<{ dispatch: Ext }>

// TBuildReturnType gets a type as if `extensions` were provided as a const literal (no type information).
function createEditor<T extends Extension[]>(extensions: T): TBuildReturnType<Untype<T>> { ... }

This is already possible since Typescript 5.0, see const type parameters. The only thing you need to change is to add const before the declaration of the generic type T, and it will act as if the argument to the function had an as const.
function createEditor<const T extends Extension[]>(extensions: T): TBuildReturnType<Untype<T>> { ... }
//                    ^^^^^

Also, I don’t think you ever need to use satisfies. You can actually remove all uses of satisfies in your code, and it will still typecheck properly. It is just helpful sometimes for catching potential issues, but it only affects the place where it is used, not the rest of the program.

Thanks @guillaumebrunerie. As for const - you are right; I have removed it from the question.
As for satisfies, please feel free to jump in on the SO thread, ideally with a working example.
Note that the issue here is not about type checking, it is about inference of specifics. Soon as you provide type the argument (say actions: Record<string, Action>), whatever you do, typescript will return Record<string, Action> for actions - never the actual list (a, b, c) - so you could call actions.invalid even though it isn't an action in any of the extensions.

@Izhaki Does this work?
Depending on what you want, it may make sense to throw an error for duplicate nicknames instead of overriding them, but that's what I chose to do.
interface Language {
   nicknames: Record<string, string>;
}

type _GetNicknames<L extends readonly Language[], Acc extends Record<string, string> = {}> =
   L extends readonly [infer First extends Language, ...infer Rest extends readonly Language[]]
   ? _GetNicknames<Rest, {
    [K in (keyof Acc | keyof First['nicknames'])]: K extends keyof First['nicknames'] ? First['nicknames'][K] : Acc[K];
   }>
   : Acc;
type GetNicknames<L extends readonly Language[]> = _GetNicknames<L>;

function getNicknames<const L extends readonly Language[]>(languages: L) {
    const nicknames = languages.reduce<Record<string, string>>((acc, curr) => {
        for (const nickname in curr.nicknames) {
            acc[nickname] = curr.nicknames[nickname];
        }
        return acc;
    }, {});
    return nicknames as GetNicknames<L>;
}

const r = getNicknames([
  {
    nicknames: {
      william: 'bill',
    },
  },
  {
    nicknames: {
      alexander: 'sasha',
      william: 'override'
    },
  },
]);
console.log(r);

I don't understand from the examples here how something like satisfies is supposed to fill the gap you're describing.
The example as posted can be typed as:
const editor = createEditor([{
  actions: {
    a: () => {},
    b: () => {}
  }
},{
  actions: {
    c: () => {},
  }
}]);

declare function createEditor<const K extends string>(obj: ReadonlyArray<{ actions: Partial<Record<K, () => void>> }>): { actions: Record<K, () => void> };

const { actions } = editor;
actions.a();
actions.b();
actions.c();

Sorry for being unclear.
@conorbrandon @RyanCavanaugh both your examples pass untyped objects - absolutely no problem extracting specifics from them.
The problem is with typed arguments, as it seems that typescript has no way introspecting the specifics - it just picks up the provided types.
I've amended the original post usage example to clarify this.
Here is Ryan's approach with typed arguments and it return Record<string, Action>.
Here is the change (and the challenge):
const extension1: Extension = {
  actions: {
    a: () => {},
    b: () => {}
  }
}

const extension2: Extension = {
  actions: {
    c: () => {},
  }
}

const editor = createEditor([extension1, extension2]);

const { actions } = editor;
// actions = {
//    a: () => {}
//    b: () => {}
//    c: () => {}
// }

Since extension1 and extension2 are typed, actions ends up as Record<string, Action>. You cannot extract a, b or c.
Myself (18 hours) and others cannot find a way around it, other than typing each extension with satisfies Extension rather than : Extension.
I hope this is clearer now.

And just spare others the time, I'm aware that a wrapper function would solve this (as you can pass object literals and have const and satisfies within.
Still don't see it as an elegant (or safe) solution. And while I can elaborate on the whys, it is little relevant to the feature request.

Yes, once a variable has been assigned a certain type, then there is indeed no way to extract a more specific type out of that variable (you can’t reexaminate how the variable was created). This is a fundamental property of how any type system works, for many different reasons, for instance performance. You don’t want any use of any variable to need to inspect exactly how that variable (and all other variables it is using, and the variables they are using, etc.) was created, that would make type-checking prohibitively expensive. Also, with such a feature, it would be very easy to start leaking out/depend on implementation details. Plus it would probably not be easy to implement, and it would also be harder to work with (now you need to remember that every variable has two types, the declared type and the "introspected" type). Not to mention that it would need to interact with every other Typescript feature, for instance narrowing.
If you need a more specific type, you need to define your variable with a more specific type to begin with. Either as const (no type safety), or as const satisfies …, or a wrapper function. I personally think that a wrapper function is rather elegant (and safe), especially if you need specific types only in some places and not in other places, then the wrapper function can give you a way to express exactly that.

Thanks @guillaumebrunerie for the detailed explanation.
One final comment:

I personally think that a wrapper function is rather elegant (and safe).

No doubt it will be type-safe, but I meant that it does not prevent consumers from typing the object passed, and I have just checked by if you type a variable and tuck on satisfies on it - it is still seen as typed.

@guillaumebrunerie said what I was going to say, but I'll restate it again for coverage.
Let's make some terminology
const x: T = expr;

What you have here is T, the annotated type, expr, the initializing expression, and (implicitly) typeof expr, the type of the initialization expression.
There's no operation that can take x and give you back the type of the initialization expression. Once you assign it to x: T, it's gone in the wind. I don't think we'd ever add something that says "Give me the type of the expression that initialized this binding". It just doesn't make sense as an operator. It seems like you're stuck on writing const x: T = when the : T part is making your type inference worse -- maybe that's worth XY-probleming on


Inform others modules that MMM-Selfieshoot is in use

Hi, I'm trying to add some rules with other modules at start using this module but it's not available.
I just add this part of code

Thanks! I'll check this out in the next day or two.

Is it just that one line of code? That seems easy enough. Question: is there a reason why you didn't send a notification when the selfie is complete?

Doh! Good point.
Code looks good. I'll merge now.


Current Jquery version

What is the current version for jquery? I am updating an older site which is using the jquery dependent version. I've included the latest version into the site without jquery dependency, but we are still using jquery and it isnt working correctly.
My specific issue is with the change callback. This was previously called through the initialization, but this method doesnt work anymore:
 $('#pickup_date').pickmeup({
    format : 'm/d/Y',
    hide_on_select: true,
    calendars : 2,
    change : function (formatted_date) {
       $('#return_date').val(addDays(formatted_date,2));
    }
  });

The docs recommend this method, but it doesnt work with jquery:
pickmeup(element);
element.addEventListener('pickmeup-change', function (e) {
    console.log(e.detail.formatted_date); // New date according to current format
    console.log(e.detail.date);           // New date as Date object
})

What is the correct jquery method for the change callback?

Why can't you just call addEventListener and need jQuery here?

If you desperately want jQuery the same can be done with jQuery, but what is the point of doing so?:
$(element).on('pickmeup-change', function (e) {
    console.log(e.detail.formatted_date); // New date according to current format
    console.log(e.detail.date);           // New date as Date object
})

Why can't you just call addEventListener and need jQuery here?

I am not sure how to do that.

I am not sure how to do that.

You've quoted the exact piece that does it last time:) Anyway, you can use jQuery version I've posted earlier.

I understand what you are saying. What I originally quoted wasn't working.   Which spurred me to ask for help.
This gets a response, but returns "TypeError: e.detail is undefined"
$("#pickup_date").on('pickmeup-change', function (e) {
console.log(e.detail.formatted_date); // New date according to current format
console.log(e.detail.date);           // New date as Date object
});

This is better, so event is fired and captured by jQuery, just e.detail property isn't there while it should.
Add debugger statement at the beginning of the callback and open Dev Tools in your browser. Than you can inspect e variable and see what it inside.

Just so you can get a full view of what I am trying to do:
This previously worked:
    var d = new Date();
    pickmeup('#pickup_date',{
        format : 'm/d/Y',
        hide_on_select: true,
        calendars : 2,
        min : d,
        change : function (formatted_date) {
            $('#return_date').val(addDays(formatted_date,2));
        }
    });

Now that the above no longer works, I am exploring new options. This doesnt work yet:
    $("#pickup_date").on('pickmeup-change', function (e) {
        $('#return_date').val(addDays(e.formatted_date,2));
    });

Here is another demo that works: https://jsfiddle.net/qx152r6r/176/
Maybe you have very old jQuery or something else.

Add debugger statement at the beginning of the callback and open Dev Tools in your browser. Than you can inspect e variable and see what it inside.

Thank you. You got me on the right track so I can sort this out.


Picture Element Lazy Loading

Firstly, thanks fo creating this amazing tool. I use it on almost every website I build. I would, however, like to open a discussion about the way that picture elements are lazy-loaded.
Why not just put the class lozad on the picture element and add then, when it enters the viewport, change all data-srcset attributes to srcset attributes on the source elements and on the img element, do the same but also change data-src elements to src elements.
The markup would then just become
<picture class="lozad">
    <source data-srcset="image.png">

</picture>

Adding the data-iesrc attributes to the picture elements etc just seems confusing to me and I'm not sure what benefit it brings.
I would be very happy to submit a PR if this is something you are interested in?

I agree with you that this implementation is very confused.
Other lazyloading plugins solved this in such better ways. Would be very satisfying to solve this.

Agreed. Especially the fact that you can't add , making it unable to have css classes. And also it loses backward compatibility with older browsers because the  added by the library doesnt have src attribute.
@ApoorvSaxena

I also agree with @Jab2870's suggested solution. I ran into the same problems when I wanted to assign a class to the img tag which currently isn't possible

As someone who is working with picture tags all the time, this implementation feels much better than the current one. Already using the patch on my end. Thumbs up @Jab2870.

I'm having an issue where it's saying it's loading but is in fact only entering an empty  tag. But that fact that you can't have CSS on the image tag is enough of a reason for me to switch.

I like the implementation proposed by @Jab2870 as well, having the ability to attach to the img element for styling is very important.

Correct me if I'm wrong, but here's an idea:
<picture>
  <source srcset="/image-1280.jpg" media="(min-width: 1280px)" />

</picture>

Output:
<picture>
  <source srcset="/image-1280.jpg" media="(min-width: 1280px)" />

</picture>

Don't add a lozad class directly to the <picture> tag, but to the  tag. This way you can add custom styles (such as aspect-ratio or additional classes) to the <picture> and  tags. You also avoid the confusing data-iesrc attribute.

As someone who is working with picture tags all the time, this implementation feels much better than the current one. Already using the patch on my end. Thumbs up @Jab2870.

Hello mate, you wouldn't happen to have that modified file would you? I don't know how to patch lozad.js with this diff file.

Hey @teebling, just use patch-package - its super easy and well documented. By the way: you can get patch files from any pull request by just adding ".patch" to the end of any pull request URL.
See this PR im using? https://github.com/ApoorvSaxena/lozad.js/pull/145
Just add ".patch" to the end of the URL and see what happens. :-)

I'd rather avoid maintaining my own patched version of any 3rd party software. As a workaround, I've used this:
<picture class="picture-lozad">
  <source data-srcset="image.png">

</picture>

This has the benefit that you get to use any classes you might need for layout at the  and also to place the 'alt' attribute right where it belongs. I use 'data-srcset' insted of just 'srcset' in the <source> elements because modern browsers will fetch their preferred image from the 'srcset' even if the  itself has no 'src', thus bypassing the whole lazy image loading behavior that was intended in the first place.

I'd rather avoid maintaining my own patched version of any 3rd party software. As a workaround, I've used this:
<picture class="picture-lozad">
  <source data-srcset="image.png">

</picture>

This has the benefit that you get to use any classes you might need for layout at the  and also to place the 'alt' attribute right where it belongs. I use 'data-srcset' insted of just 'srcset' in the <source> elements because modern browsers will fetch their preferred image from the 'srcset' even if the  itself has no 'src', thus bypassing the whole lazy image loading behavior that was intended in the first place.

Nice solution!


Evening out branches

Docs Build status updates of commit 10fa38b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

.openpublishing.redirection.json
:white_check_mark:Succeeded

DocsLegal/docfx.json
:white_check_mark:Succeeded

[Warning] Use provisioned locale en-ZA for docset intune instead of locale en-us in publish configuration
[Warning] No file found, nothing will be generated. Please make sure docfx.json is correctly configured.
[Warning] No content file found. Please make sure the content section of docfx.json is correctly configured.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Issue with GOG games

SuccessStory_2022-06-19.zip
Success story doesn't seem to work with certain GOG games (in my case it's BioShock Remastered).

Does the game actually have GOG achievements?
I've noticed that if there isn't, SuccessStory attempts to pull the achievements anyways and fails with an error. 'Ignore game' fixes it.


remove start() and close() from Timeout class

I was trying to avoid rewriting the functions to replace this. Please take another look.


Optimize images with extension in ignore case maner

#9

@all-contributors please add @yarikwest for bug and code .


1.4.2 breaks Azure Gov environments

Storage Explorer Version: 1.4.2
Platform/OS Version: Windows
Architecture: ia32 -->
Regression From: Yes, 1.4.1
Steps to Reproduce:

Connect to a Storage Account in the Azure Gov environment
Click to expand a Storage Account.

Expected Experience:
You should be able to browse/expand to access storage resources within a Storage Account.
Actual Experience:
Storage Accounts are visible, but when you attempt to expand them they are empty.  I removed my Gov account and re-authenticated--same behavior.

Sorry about that! We've recently changed the api-version to support a new blob type but Azure Government doesn't support that yet.  I'll add the workaround to my next comment on this thread.

To workaround the issue

Locate AzureStorageAccountAttributeLoader.js
Locate the variable assignment v2AttributesApiVersion = "2018-07-01"
Change to v2AttributesApiVersion = "2017-10-01";
Restart all Storage Explorer instances.

We double checked and the version is supposed in Azure Government ... except US Gov Iowa. Are you targeting US Gov Iowa?

No, the target is Virginia.  I will do some more digging in and follow back up.

Changing the v2AttributesApiVersion as suggested corrected the issue.  Thanks much for the quick response.  This is the second time I've seen a new version break Gov.

Sorry about that, we'll come up with a way to make sure we don't break it in the future. Btw it will self-correct it self soon. It turns our Azure Iowa will still impact some calls for Azure Virginia if the calls are across regions.
I'm going to leave this issue open until the new Api-Version works for Iowa.

Awesome. I know how much of a pain Gov is. Thanks again for the prompt workaround!

@JakeRadMSFT Hi Jake. Azure China (Mooncake) also have this issue, would the issue be fix automatically in the future?

@vivianxing, yes

This is still an issue with US Gov Texas.  The fix referenced above did remedy the situation, however.

@rider158, Thanks for reporting! I'll see if I can get an ETA.

We'll come up with a better workaround for our next release.

Same with Azure Germany.

Azure Government (specifically Texas) is still broken in 1.4.3 with the same issue (i.e., unable to browse storage accounts within an Azure Government subscription). The fix as provided by JakeRadMSFT still works.

Resolved (Reverted) and it will be rolled out when available everywhere.


Better work distribution

Currently all queries are executed on all processes (except for combine queries with no grouping). This might not be ideal when the number of continuous queries is high because there's a lot of constant overhead per query execution. Furthermore, since data is distributed to all procs (not true for workers since we always insert a full batch for each worker), aggregation per node might not be very high.
Investigate other models of distributing work between processes and have the scheduler dynamically change policies based on system load.

Measure the per-query execution overhead.


Doc comments (and other attributes) within intrusive_adapter! macro

It would be nice to be able to have attributes within intrusive_adapter! macro invocations.
I have #[deny(missing_docs)] on the module in which I use the macro, and I'd like to #[allow(missing_docs)] the stuff the macro produces, or even to have doc comments on the produced code. When I try the first, I run into a recursion limit reached while expanding the macro error.
Minimal code to reproduce:
struct Toy {
    link: RBTreeLink
}

intrusive_adapter!(
    #[allow(missing_docs)]
    pub ToyAdapter = Box<Toy> : Toy { link: RBTreeLink }
);

How easy would it be to modify the macro to preserve attributes?

I think this should be fairly simple to support since e.g. lazy_static supports this.


Timezone definition incorrect

Hi,
Awesome library, however, this week, a bug was revealed in timezone config.
In the README.md, and also in your lookup server, it is stated that timezone for Berlin/western eu is:
CET-1CEST,M3.4.0/2,M10.4.0/3

However, the actual definition is 'last sunday of the month'. In march 2019, the month had 5 sundays, so the clock on my night table is off for an hour this week.
According to the spec, '5' means 'last of the month', so I've also took the liberty to update the definition in my code to
CET-1CEST,M3.5.0/2,M10.5.0/3

Can you update README.md and probably your server accordingly?

If you upgrade to 0.7.10, you'll likely notice the problem is fixed...

Fixed the documentation now also...

drat, i did check the commits, but missed the "last week patch" comment, sorry.
Good to know it's fixed, thanks!


Allow the Wizard to run also as part of the page, not only inside a modal.

[ ] bug
[ ] feature request
[x] enhancement

Expected behavior
Sometimes a need is rising for the Wizard component to be part of the page, while opening the wizard in a modal doesn't fit, since it breaks the user's navigation flow.
Actual behavior
Wizard open in modal only. No way (AFAAK) to configure it to run as an inline, page-integrated component.
Reproduction of behavior
Wizard's documentation can be found here: https://vmware.github.io/clarity/documentation/wizards
Environment details

Angular version: 2

Clarity version: Latest

OS and version: Mac Sierra

Browser: all

thank you @jaffoneh for taking a look at this 👍

Hi @neoswf , I'm having the same requirements and I did some CSS overriding to achieve this. Please refer to the plnkr below:
https://plnkr.co/edit/HvFMLyeXw0dsL023xg0Z?p=preview

@yaolu and @neoswf : This is a version of what @yaooluu did with a lighter approach to the CSS selectors. Just make sure to add the .clr-wizard--inline classname to your wizard HTML.
https://plnkr.co/edit/sv4Shf7lAzQJoFX4gCyv?p=preview

This was addressed in #156
It is accomplished with CSS classnames applied to clr-wizard. An example can be found here in the demo app: https://github.com/vmware/clarity/blob/master/src/app/wizard/wizard-inline.demo.html
This will remain undocumented because it is not a recommended use case for the wizard. But the code is there to accommodate users.


Allow assistive technologies to properly skip the mirror element

About aria-hidden:
http://www.w3.org/TR/wai-aria/states_and_properties#aria-hidden
Compare how assistive technologies will handle the mirror-element in the different versions with the Accessibility Developer Tools

Ace! Thank you very much for the contribution :+1:


TypeError: Roles.setUserRoles is not a function

setFinancialRole(user,role){
var loggedInUser = Meteor.user();
if (!loggedInUser && !Roles.userIsInRole(loggedInUser, ['Automation', 'Financial'])) {
      throw new Meteor.Error(403, "Access denied")
    }
  Roles.setUserRoles(user, role)
},
removeFromRole:function(user,role){
  Roles.removeUsersFromRoles(user,role);
}

I'm using this method in a react JSX server/method.jsx file.
Can anybody explain why I'm getting this error?

Not sure why you're seeing that but it looks like the Roles package export is not accessible in your JSX file.
I'd start debugging from there.  See if there are other package exports that you can access in the JSX and see if there's a difference with how the roles package does things.
(Also I think you should check the logic in your IF statement.)

No feedback on this one. Closing.

Yes, I was calling the function and I needed to pass it down to the child prop before I could pass it to the server method. Thank you for pointing me in the right direction.


Does keda-operator ClusterRole need access to serviceaccounts?

Discussed in https://github.com/kedacore/keda/discussions/2348

Originally posted by gdamjan-loka November 25, 2021
Does the the keda-operator ClusterRole need access to serviceaccounts resources too?
I was getting these errors:
E1125 13:48:04.476470       1 reflector.go:138] pkg/mod/k8s.io/client-go@v0.20.8/tools/cache/reflector.go:167: Failed to watch *v1.ServiceAccount: failed to list *v1.ServiceAccount: serviceaccounts is forbidden: User "system:serviceaccount:keda:keda-operator" cannot list resource "serviceaccounts" in API group "" at the cluster scope

To get rid of the error I had to add serviceaccount here somewhere (I edited the release yaml file):
https://github.com/kedacore/keda/blob/main/config/rbac/role.yaml#L19-L23

may be related to https://github.com/kedacore/keda/issues/837#issuecomment-789037326

Hi @dominhhai
I have several clusters with KEDA running there so I believe that it's not necessary.
Could you run kubectl get pods -n {KEDA_NAMESPACE} -o jsonpath="{..imageID}" and paste the output please?

What type of triggers are you using?

And how did you install KEDA? via HELM?

FWIW, we also ran into this issue on our test cluster with a fresh Keda installation. We install Keda via Helm (chart version 2.5.0) and tried to use the TriggerAuthentication pointing to a service account with IAM permissions (via IRSA). The ScaledObject had a trigger of type aws-sqs-queue. This failed in the error described above. Once we gave the operator access to read our queue, removed the TriggerAuthentication and instead used identityOwner: operator the scaling worked.
If possible, we'd like to use the aws-eks pod identity in Keda in the future, so that permissions can be given granularly by the teams operating their services.

If @zroubalik agrees, are you open to send a PR for this @mKeRix ?

I comment on this because I had the same issue and solved it by giving permissions to list and watch service accounts to the keda-operator cluster role.
I deployed KEDA via Helm (v2.5.0) and ran into this when when using ClusterTriggerAuthentication with identityOwner: oeprator.

Are you open to send a PR for this?

sure thing!

A new Helm chart is out - https://github.com/kedacore/charts/releases/tag/v2.5.1
Thanks for reporting and fixing!


Add 1.17.1 and 1.16.8

Go 1.17.1 and 1.16.8 has been released. Would you please support them? (Ref. https://github.com/syndbg/goenv/issues/194)

Thanks!


Gamkers gpt

Oh no .
Running problems

We are pleased to inform you that the issue has been resolved. You can now continue to use our application as usual. Thank you for bringing this to our attention


Print total tree size, including inactive shards in rekor-cli loginfo

Signed-off-by: Priya Wadhwa<EMAIL_ADDRESS>
Summary

Ticket Link

Fixes
Release Note

Codecov Report

Merging #864 (4044029) into main (b2a5dca) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #864      +/-   ##
==========================================
- Coverage   46.46%   46.43%   -0.04%
==========================================
  Files          60       60
  Lines        5120     5128       +8
==========================================
+ Hits         2379     2381       +2
- Misses       2467     2473       +6
  Partials      274      274

Impacted Files
Coverage Δ

cmd/rekor-cli/app/log_info.go
1.98% <0.00%> (-0.12%)
:arrow_down:

pkg/types/rekord/v0.0.1/entry.go
48.84% <0.00%> (+0.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Gracefully handle login attempts by users with no EmailAddress

Currently, login attempts by users having no associated EmailAddress end up in a 500 error, due to the get query not finding any object. Why not handling it in a more graceful way? It could be easily achieved via filtering and an exists query.

Eagerly waiting for this to be officially released.


False positive: service.gmx.net

Fixed https://github.com/HorusTeknoloji/TR-PhishingList/commit/0f747621f38aa46eefa36c2bca5f1265636d0e22
Thanks for your help. 👍🏼

Fixed https://github.com/HorusTeknoloji/TR-PhishingList/commit/0f747621f38aa46eefa36c2bca5f1265636d0e22
Thanks for your help. 👍🏼


Create SECURITY.md

Add security policy so we that dependency security can be checked.

@Blank-Xu


Peity in IE7+ with explorercanvas (doesn't work)

Hi, thanks for great plugin!
It is pretty fine solution, but i couldn't optimize it for IE7 + IE8 - I am using this :
http://code.google.com/p/explorercanvas/wiki/Instructions
I follow the second reccomendation on their github wiki, bud doesnt work if i apply it.
I am giving the fix code into private function of plugin - createCanvas()
Rest of code and initialization is same as on your pages
I would be very grateful for any advice..
Thank you very much

I've attempted to get Peity working with IE via explorercanvas on a couple of separate occasions but with no luck. I'm going to give it another go, maybe together we can crack it.

Hi, thanks for quick reply!
Try this - hard recoded based on your plugin functions and it seems to work in IE8:
http://pastebin.com/a2Q2MHZT
May be the problem is on top start of your plugin code, that first IF condition should return TRUE in IE
because of excanvas.js automatic initialization (because of first IF there), but it returns FALSE
and code to create canvas element will not proceed..

// Peity jQuery plugin version 0.4.1
// (c) 2010 Ben Pickles
//
// http://benpickles.github.com/peity/
//
// Released under MIT license.
(function($, document) {
  var peity = $.fn.peity = function(type, options) {

    if (document.createElement("canvas").getContext) {

      this.each(function() {
        $(this).change(function() {
          var opts = $.extend({}, options)
          var self = this

          $.each(opts, function(name, value) {
            if ($.isFunction(value)) opts[name] = value.call(self)
          })

          var value = $(this).html();
          peity.graphers[type].call(this, $.extend({}, peity.defaults[type], opts));
          $(this).trigger("chart:changed", value);
        }).trigger("change");
      });

    }
    else{

      alert("Its IE!");

      // but it shloudnt pop up, because of excanvas which should create canvas ability in IE

      /* it is this starting part in excanvas.js > if (!document.createElement('canvas').getContext) {(*/

      // because is returned FALSE in IEs, the code above is not proceed and canvas element via createCanvas() is not created.. just testing alert pop-up

    }

    return this;
  };

I had a go with excanvas again and suddenly it works... sometimes. There seems to be an issue with excanvas, in my tests it works on the first page visit but not if the page is refreshed, if it's included after jQuery then it doesn't work, if it's a remote file then it doesn't work. Weird. A quick Google shows I'm not the only person experiencing this, I'll investigate to see if anything that can be done to make it work consistently.
Anyway I pushed to a separate branch. I tested in IE6.


[Request]: Dependency JS Update

Plugin
connectivity_plus
Use case
Many packages are using "js 0.7.0" but "connectivity_plus" is using older one. It's can update for the future?
Proposal
Because connectivity_plus 5.0.2 depends on js ^0.6.4 and flutter_native_splash >=2.3.11 depends on js ^0.7.0, connectivity_plus 5.0.2 is incompatible with flutter_native_splash >=2.3.11.
So, because flutter_app depends on both flutter_native_splash 2.3.11 and connectivity_plus 5.0.2, version solving failed.

Would probably be solved by dropping js dependency in favor of https://github.com/fluttercommunity/plus_plugins/issues/2600

@vbuberen shouldn't the js dependency removed entirely in favor of web.
To enable compiling to WASM?

It will be removed, but later. I have seen your issues about WASM, but we already keep track of it as I mentioned one of related issues.

Closed in #2621, the dependency to js has been removed


Step 10 redundunt links redirection

In the following paragraph :
Idling Resources in Practice
In our app we have wrapped the calls to Simple CountingIdlingResource...
I believe that we should have only one link redirection by concatenating the 2 words to be :
In our app we have wrapped the calls to SimpleCountingIdlingResource...

Closing as the associated paragraph was removed in the latest update.
There shouldn't be another instance of this kind of redundant link in the codelab now, but feel free to reopen and edit if we missed a spot.


Use type-only imports where possible

This is a small clean-up pass that utilizes TypeScript's "type-only imports" and exports. By enabling this flag in tsconfig.json, TypeScript will throw errors if you're only importing types from a specific module, and remind you to declare the imports as import type instead. By doing so, you statically declare that all of those imports will all be types. This is useful to tools like Babel as they can know for sure that those imports are intended to strictly be types. I also find it makes code more readable to me as a developer. Reading import { Json } from './json', you instantly know that it's just a type, and not a class, for example.
This can also help reduce chances of import cycles in some cases. If an import cycles is only needed at the type level, then using import type instead removes the import cycle at runtime, so tools like Node, Babel, or bundlers have an easier time.

Getting this merged in to rebase the deprecation work on top of that I'd love to push out today. As always, please do review and comment in retrospect! 🙏


⚠ Bofrost.at has degraded performance

In 2ca5e1f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1590 ms

Resolved: Bofrost.at performance has improved in 2fe2fbc after 19 minutes.


Add GDPR Supported Variable in C1X Adapter Doc

@bretg thanks for reviewing. I've add prebid_1_0_supported : true to the header

@bretg Just a quick update - C1X adapter PR was merged. Thank you for the review.

released with 1.17


[BUG] HTTP instrumentation traces for Node.js + Azure Monitor OTel not showing in AI resource

Library name and version
"@azure/monitor-opentelemetry": "^1.0.0",     "@azure/service-bus": "^7.5.1",     "@opentelemetry/api": "^1.4.1",     "@opentelemetry/exporter-trace-otlp-grpc": "^0.38.0",     "@opentelemetry/instrumentation-express": "^0.33.1",     "@opentelemetry/instrumentation-http": "^0.38.0",     "@opentelemetry/resources": "^1.12.0",     "@opentelemetry/sdk-metrics": "^1.12.0",     "@opentelemetry/sdk-node": "^0.43.0",     "@opentelemetry/semantic-conventions": "^1.12.0",

Describe the bug
Unable to get OTel HTTP instrumentation traces to show at the AI resource in Azure Portal. This is node.js app on AKS.
Verified that the connection string is correct. Verified using AI logging that HTTP instrumentation is active on pods.

.NET works fine and HTTP instrumentation shows correctly.
Here is the relevant code.
. . .
const client = new ResourceManagementClient(this._tokenCreds, subscriptionId); // Pick up subscription guid from /subscriptions/guid/...
const resource = await client.resources.getById(resourceId, "2020-02-02"); // await will resolve the Promise returned by getById().
this._connectionString = resource.properties.ConnectionString; // property is case-sensitive.
console.log(Resource ID ${resourceId}, subscription ${subscriptionId}, connection string ${this._connectionString});
    const customResource = new Resource({
        [SemanticResourceAttributes.SERVICE_NAME]: this._serviceName,
        [SemanticResourceAttributes.SERVICE_INSTANCE_ID]: this._serviceVersion,
    });

    const options = {
        azureMonitorExporterOptions: {
            connectionString: this._connectionString,
        },
        resource: customResource,
        credential: this._tokenCreds,
        instrumentationOptions: {
            http: {
                enabled: true
            }
        }
    };

    useAzureMonitor(options);
    console.log("Started Public Monitoring...");

Expected behavior
HTTP instrumentation traces show in AI resource "Transaction Search".
Actual behavior
Application map only shows .NET apps. Traces do not show anything from node.js app.
Reproduction Steps
AKS cluster with node.js. Using @azure/monitor-open-telemetry. Code added above.
Environment
AKS running Mariner 2.0. Client is .NET, Server is node.js app. Both apps work fine functionally.

Some more info after debugging.
Doc seems incorrect, see
Fixing the credentials field, I got further. One of the ingestion calls is working.
azure:identity:info ClientAssertionCredential => getToken() => SUCCESS. Scopes: https://monitor.azure.com//.default.
azure:identity:info ChainedTokenCredential => getToken() => Result for DefaultWorkloadIdentityCredential: SUCCESS. Scopes: https://monitor.azure.com//.default.
azure:core-rest-pipeline retryPolicy:info Retry 0: Attempting to send request 723ac412-78be-4a68-be2f-fdce1b2d46fd
azure:core-rest-pipeline:info Request: {
"url": "https://westus2-2.in.applicationinsights.azure.com/v2.1/track",
"headers": {
"content-type": "application/json",
"accept": "application/json",
"accept-encoding": "gzip,deflate",
"user-agent": "azsdk-js-monitor-opentelemetry-exporter/1.0.0-beta.16 core-rest-pipeline/1.12.1 Node/v18.17.1 OS/(x64-Linux-<IP_ADDRESS>-1.cm2)",
"x-ms-client-request-id": "723ac412-78be-4a68-be2f-fdce1b2d46fd",
"authorization": "REDACTED"
},
"method": "POST",
"timeout": 0,
"disableKeepAlive": false,
"streamResponseStatusCodes": {},
"withCredentials": false,
"requestId": "723ac412-78be-4a68-be2f-fdce1b2d46fd",
"allowInsecureConnection": false,
"enableBrowserStreams": false
}
azure:core-rest-pipeline:info Response status code: 200
azure:core-rest-pipeline:info Headers: {
"transfer-encoding": "chunked",
"content-type": "application/json; charset=utf-8",
"server": "Microsoft-HTTPAPI/2.0",
"x-content-type-options": "nosniff",
"strict-transport-security": "max-age=31536000",
"x-request-id": "0129ef82-492e-410a-a6cc-23edac0e6c1e",
"date": "Thu, 28 Sep 2023 22:00:53 GMT",
"connection": "close"
}
However another tracing ingestion one fails which might be the critical error. This is pretty bizarre since I am passing the credential during init and expect Azure Monitor exporter to use that throughout.
ApplicationInsights:Envelopes could not be exported and are not retriable. Error message: [
'ChainedTokenCredential authentication failed.\n' +
'CredentialUnavailableError: EnvironmentCredential is unavailable. No underlying credential could be used. To troubleshoot, visit https://aka.ms/azsdk/js/identity/environmentcredential/troubleshoot.\n' +
"AuthenticationRequiredError: invalid_scope: 70011 - [2023-09-28 22:01:38Z]: AADSTS70011: The provided request must include a 'scope' input parameter. The provided value for the input parameter 'scope' is not valid. The scope {Host}/v2.1/.default is not valid.\r\n" +
'Trace ID: 13a330f0-bbc9-48e0-bfd2-68e4a7966800\r\n' +
'Correlation ID: 99481608-e731-4961-af88-0032a271556d\r\n' +
'Timestamp: 2023-09-28 22:01:38Z - Correlation ID: 99481608-e731-4961-af88-0032a271556d - Trace ID: 13a330f0-bbc9-48e0-bfd2-68e4a7966800\n' +
"AuthenticationRequiredError: invalid_scope: 70011 - [2023-09-28 22:01:38Z]: AADSTS70011: The provided request must include a 'scope' input parameter. The provided value for the input parameter 'scope' is not valid. The scope {Host}/v2.1/.default is not valid.\r\n" +
'Trace ID: c331e26f-c3d9-4f77-ac5f-c13c5dcb8000\r\n' +
'Correlation ID: 79b8819a-69e9-4245-8908-8420ff43ab65\r\n' +
'Timestamp: 2023-09-28 22:01:38Z - Correlation ID: 79b8819a-69e9-4245-8908-8420ff43ab65 - Trace ID: c331e26f-c3d9-4f77-ac5f-c13c5dcb8000\n' +
'CredentialUnavailableError: Invalid scope was specified by the user or calling client\n' +
'CredentialUnavailableError: Error: Invalid scope was specified by the user or calling client. To troubleshoot, visit https://aka.ms/azsdk/js/identity/powershellcredential/troubleshoot.\n' +
"CredentialUnavailableError: Azure Developer CLI couldn't be found. To mitigate this issue, see the troubleshooting guidelines at https://aka.ms/azsdk/js/identity/azdevclicredential/troubleshoot."
]

@chitturs we updated official docs and fix of the different scope being used would be available in next release, let us know if you have any other issue with AAD integration with @azure/monitor-opentelemetry

@chitturs issue should be resolved now in latest version, we are also working on improving and simplifying official documentation.


docs(v2): Replace mbt-bundle by sicope-model

Motivation
I rewrote my application, rebranded, updated website, changed domain. I am still using Docusaurus 2. It's time to update the showcase.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Go to https://deploy-preview-xxxx--docusaurus-2.netlify.app/ and see if SICOPE Model appear on showcase page, and MBT Bundle disappear.
Related PRs
Not available

thanks!


Document SuppressWarnings support from scalac-scapegoat-plugin#8

I couldn't come up with a particularly nice example, so I just put in a simple one.
(I think it does need an example, to show the correct syntax.)

re https://github.com/sksamuel/scalac-scapegoat-plugin/issues/8

Thanks @RichardBradley


Fix blob puts for object-lock-enabled S3 stores

Ref #711

@redgoat650 PTAL. We'll submit upstream shortly.


Add initial support for Markdown

Dear Author,
It was a pleasure working with your code. I modified the code a bit, so that it will allow converting .md files into HTML.
Also, I added a  tag, so that all generated text will be bold.
Hope that sounds great !

I have modified Readme file.


Add a script demonstrating how to evaluate defenses against adversarial examples according to best practices

Is your feature request related to a problem? Please describe.
Hi all, I'm doing research on designing defensive methods. From the references I used and my own experience, a standard of the evaluation process could largely benefit the whole community. Since different evaluation process is conducted, it is hard for people to evaluate their proposed defense with the state-of-the-art. Moreover, some weakly designed evaluation may result in the obfuscated gradient effect which is already identified by @carlini.
Describe the solution you'd like
As a reference or a suggestion, CleverHans could offer an evaluation demo to the whole community. In this demo, a detailed evaluation plan could be offered which includes a selection of different adversarial examples, different evaluation flows for white-box/black-box defenses and even the computational cost of proposed defenses.

No progress has been made on this issue yet, but for now we recommend reading the following report: https://github.com/evaluating-adversarial-robustness/adv-eval-paper

Closing this issue for now since it's inactive for a while, and we currently don't have plans to design/implement a standard evaluation process. I'll reopen if more people are interested in discussing this.

Closing this issue for now since it's inactive for a while, and we currently don't have plans to design/implement a standard evaluation process. I'll reopen if more people are interested in discussing this.


Correctness fixes and (optional) better C stdlib integration

Add early NULL check to ta_free()
Add overflow checks in alloc_block() and ta_calloc()
Clear full block size to 0's in ta_calloc()
Optionally set errno to ENOMEM when out of memory
Optionally use memset() to implement memclear()
Add ta_getsize() to return allocated block size
Add ta_realloc() (optional, requires memcpy())

ta_getsize() and ta_realloc() are based on ideas from #11, but rewritten to fix various issues in that PR.

Updated with some more fixes:

Fix behavior of ta_alloc(0) (see #15)
Fix additional arithmetic overflows
Add fallback memcopy() when not using C stdlib
Add early return to ta_realloc() when size is similar

@jlindgren90 Thank you very much for this and the other PR - I will take a look asap, but likely will take a little while for me to review & test this...


Display does not persist across all spaces

As previously pointed out (in improperly named issue #62): keycastr only ever captures the "default screen" (not sure what it's called on Mac, but basically, it seems to be the screen with the app dock). As a result, you cannot cast from an application when in full-screen mode.
That in itself would not even be the worst, but sadly, the keys are always displayed in the bottom left corner, where they simply overlap with the system menu bar.
As a work-around (given, you don't need to capture keys on more than one screen at a time): Simply hide the app dock and hide the system bar, and you get the complete screen estate on the first screen, for your recordings.

You should be able to drag the display window around to place it wherever you'd like.  Can you confirm for me whether or not that feature is still working?

I understand what's happening with this issue now.  The app is "stuck" on whatever desktop / screen you launch it on (it follows the settings dialog).  For multi-monitor setups, this seems reasonable - just drag the display around wherever you want it - but when flipping through spaces, you'd expect the display to stick to wherever you place it on the screen.  Basically you'd expect the display to "float" across spaces.
I'm considering this a bug.  Will fix for 1.0.

I just checked the QuickTime recorder. It captures whatever you currently see, even when switching between desktops (including the transitions between desktops), but you only see the capture window and control buttons in the "main desktop".

Would leave to see this fixed. In the meantime, two clunky workarounds:

Set an Event Trigger in Quicksilver to relaunch KeyCastr whenever the active space is changed. Super annoying, but effective.
Assign apps for which you want KeyCastr active to desktop 1, and assign apps where it doesn't matter as much to desktop 2 or other desktops. This is what I've done, because I use multiple desktops with a disappearing dock and maximized windows, (Amethyst is great for this,) instead of using spaces. If you use spaces, then option 1, or look forward to this getting fixed.

Is there any update on this? I would really love to use Keycastr in full screen apps. 😞

Working across multiple spaces and overlaying a fullscreen app seem like two separate issues to me, although I see how they could be related.
Regarding full-screen apps, please have a look at this comment and the next one for additional workarounds.
Regarding displaying KeyCastr's visualizers across all spaces/desktops/fullscreen apps, I currently don't have a clear picture of how we would do this, nor do I have much time to investigate. Are there any apps known to the community (ideally open-source apps) which do something similar?

Regarding displaying KeyCastr's visualizers across all spaces/desktops/fullscreen apps

If restarting it works as suggested in https://github.com/keycastr/keycastr/issues/64#issuecomment-299666247, then maybe the visualizer could just be reinitialized some how for the new space. I switch spaces/desktops very frequently so this would be a massive help.
Great app though 🎉

Thank you for the pointer -- KeyCastr development has been on the back
burner for a while but this seems like it's definitely worth looking in to!


[BUG]  Grad_norm is nan and Loss is 0

Describe the bug
when train llama-vid (stage2, full-finetuning LLaMA) using deepspeed==0.14.0, and transformers trainer, grad_norm will be nan (or 1.414, with smaller lr, pink line) and loss will be 0 after few steps.
Same issue as described in #5242, but with AMD GPU.
I follow #5242, deepspeed==0.12.3 can work normally.
However, both versions of ds do not significantly speed up training when using multiple nodes.
grad_norm, circle means NAN:

Step
Stage2_1node_ds0.14.0

37
1.6166185140609741

38
1.5178347826004028

39
NaN

40
1.434411883354187

41
NaN

42
NaN

Step
Stage2_4node_ds0.14.0

0
23068.85156

1
24.89443588256836

2
22.727699279785156

3
23.45322036743164

4
NaN

5
NaN

Step
Stage2_1node_ds0.14.0_smaller_lr

113
1.500418203016511

114
1.0031307797956182

115
1.4142135623730951

116
1.4142135623730951

117
1.4142135623730951

118
1.4142135623730951

loss:

training speed:
# one node, ds 0.14.0
[default0]:  0%|          | 11/5964 [05:15<46:24:59, 28.07s/it]
[default0]:  0%|          | 12/5964 [05:40<45:14:48, 27.37s/it]
[default0]:  0%|          | 13/5964 [06:05<43:52:20, 26.54s/it]
[default0]:  0%|          | 14/5964 [06:50<53:08:24, 32.15s/it]
[default0]:  0%|          | 15/5964 [07:16<50:11:33, 30.37s/it]

# four nodes, ds 0.14.0
[default0]:  0%|          | 11/5964 [04:11<36:46:44, 22.24s/it]
[default0]:  0%|          | 12/5964 [04:32<36:07:52, 21.85s/it]
[default0]:  0%|          | 13/5964 [04:49<33:38:57, 20.36s/it]
[default0]:  0%|          | 14/5964 [05:20<38:58:16, 23.58s/it]
[default0]:  0%|          | 15/5964 [05:44<39:29:56, 23.90s/it]

(stage1, training Connector, all work normally, inculding  speed up training when using multiple nodes and ds version)
training speed:
# one node, ds 0.14.0
[default0]:  0%|          | 11/3086 [01:43<6:47:31,  7.95s/it]
[default0]:  0%|          | 12/3086 [01:51<6:58:28,  8.17s/it]
[default0]:  0%|          | 13/3086 [02:00<7:02:19,  8.25s/it]
[default0]:  0%|          | 14/3086 [02:06<6:33:13,  7.68s/it]
[default0]:  0%|          | 15/3086 [02:14<6:38:44,  7.79s/it]

# four nodes, ds 0.14.0
[default0]:  0%|          | 11/3086 [00:36<2:21:11,  2.76s/it]
[default0]:  0%|          | 12/3086 [00:39<2:27:00,  2.87s/it]
[default0]:  0%|          | 13/3086 [00:41<2:24:27,  2.82s/it]
[default0]:  0%|          | 14/3086 [00:43<2:12:09,  2.58s/it]
[default0]:  0%|          | 15/3086 [00:46<2:11:55,  2.58s/it]

I'm not sure if the training speed is related to issue #5242, but I think it's abnormal because with A100 and multiple nodes, I can achieve a significant speed improvement.
 Op compatibility means that your system
      meet the required dependencies to JIT install the op.
--------------------------------------------------
JIT compiled ops requires ninja
ninja .................. [OKAY]
--------------------------------------------------
op name ................ installed .. compatible
--------------------------------------------------
 [WARNING]  async_io requires the dev libaio .so object and headers but these were not found.
 [WARNING]  async_io: please install the libaio-devel package with yum
 [WARNING]  If libaio is already installed (perhaps from source), try setting the CFLAGS and LDFLAGS environment variables to where it can be found.
async_io ............... [NO] ....... [NO]
fused_adam ............. [NO] ....... [OKAY]
cpu_adam ............... [NO] ....... [OKAY]
cpu_adagrad ............ [NO] ....... [OKAY]
cpu_lion ............... [NO] ....... [OKAY]
 [WARNING]  Please specify the CUTLASS repo directory as environment variable $CUTLASS_PATH
evoformer_attn ......... [NO] ....... [NO]
fused_lamb ............. [NO] ....... [OKAY]
fused_lion ............. [NO] ....... [OKAY]
inference_core_ops ..... [NO] ....... [OKAY]
cutlass_ops ............ [NO] ....... [OKAY]
transformer_inference .. [NO] ....... [OKAY]
quantizer .............. [NO] ....... [OKAY]
ragged_device_ops ...... [NO] ....... [OKAY]
ragged_ops ............. [NO] ....... [OKAY]
random_ltd ............. [NO] ....... [OKAY]
 [WARNING]  sparse_attn is not compatible with ROCM
sparse_attn ............ [NO] ....... [NO]
spatial_inference ...... [NO] ....... [OKAY]
transformer ............ [NO] ....... [OKAY]
stochastic_transformer . [NO] ....... [OKAY]
--------------------------------------------------
DeepSpeed general environment info:
torch install path ............... ['/xxx/miniconda3/envs/llamavid_rocm5.6/lib/python3.10/site-packages/torch']
torch version .................... 2.2.2+rocm5.6
deepspeed install path ........... ['/xxx/miniconda3/envs/llamavid_rocm5.6/lib/python3.10/site-packages/deepspeed']
deepspeed info ................... 0.14.0, unknown, unknown
torch cuda version ............... None
torch hip version ................ 5.6.31061-8c743ae5d
nvcc version ..................... None
deepspeed wheel compiled w. ...... torch 2.2, hip 5.6
shared memory (/dev/shm) size .... 427.71 GB

System info (please complete the following information):

OS: SUSE Linux Enterprise Server 15 SP4
GPU count and types: one or four machines with x8 MI250X each
Interconnects (if applicable) [e.g., two machines connected with 100 Gbps IB]
Python version: 3.10.14
transformers==4.39.2

Launcher context
slurm -> torch.distributed.run
Additional context
following #5242, use ds 0.12.3, One Node:
grad_norm:

loss:

Setting overlap_comm to False can avoid this problem.

Hi @xxtars , we noticed this accuracy issue in 14.0 and did several fixes on accuracy later on. Could you try 0.14.2? thx

Setting overlap_comm to False can avoid this problem.

This works in my multi-node training scenario.

I have checked the problem in 0.15.0 and the problem still exists.
Another workaround is to increase the bucket size. For example, an increase from 5e7 to 2e8 can help tackle the problem.

I have checked the problem in 0.15.0 and the problem still exists. Another workaround is to increase the bucket size. For example, an increase from 5e7 to 2e8 can help tackle the problem.

Hi @jianshuod , is overlap_comm=False works on your side?

since it has been two weeks, will close it for now. Feel free to reopen if needed

overlap_comm=False

works for me.  deepspeed=0.15.1


Tool Openness not recognized

Thank you for all your work on this.  I've been trying to run the openness tool, but I keep running into the same error where it says whitebox_tools.exe doesn't contain a tool called "Openness" (exact message 'Unrecognized tool name Openness.').  Other tools that I've tried do seem to work.

The Openness tool is part of the WhiteboxTools General Toolset Extension. Running this tool requires a valid license. More info about licensing can be found at https://www.whiteboxgeo.com/services-whitebox-geospatial

I have just added a list of extension tools to the WBT open-core so that when someone without a license for one of the extension tools tries to run the tool, rather than providing the non-illuminating, 'Unrecognized tool name' error, it should now issue a more informative error: "This tool is part of a Whitebox extension product and there is a missing license. Please contact Whitebox Geospatial Inc<EMAIL_ADDRESS>to obtaina valid license key."

@jblindsay Sounds good. I will update the Windows binary for the ArcGIS frontend when it is updated. Other frontends should automatically pull the latest binaries from whitebogeo.com during the installation.

I'm afraid this feature won't be available until the next release of the WBT open-core though and so we will need to wait a bit of time. I don't yet know what the time-frame is for that release.

No worries. Take your time.


Please provide a JS client sample instead of .Net client Console app

[EDIT by guardrex to add the metadata]
Hello team;
Now that the gRPC is GA, could you update the docs to show how to generate ProtoBuf for JS client, i.e. Vue and also how to create Js client to comunicate a .Net Core gRPC server?
https://docs.microsoft.com/en-us/aspnet/core/tutorials/grpc/grpc-start?view=aspnetcore-3.1&tabs=visual-studio
Thanks!
..Ben

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

ID: 1d298a6c-8ddc-dbc8-d219-0fa7138442c2
Version Independent ID: b96be346-958a-cfcd-d80a-c352d64223e8
Content: Create a .NET Core gRPC client and server in ASP.NET Core
Content Source: aspnetcore/tutorials/grpc/grpc-start.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @JunTaoLuo
Microsoft Alias: johluo

Hello @BenHayat ... Please use the This page feedback button+form at the bottom of a topic to add metadata to the issue automatically. No worries on this one ... I'll add it manually.

Hello @BenHayat ... Please use the This page feedback button+form at the bottom of a topic to add metadata to the issue automatically. No worries on this one ... I'll add it manually.

I did. That how it brought me to page.

Not sure what happened ... you should see the metadata in the opening comment ...

Not sure what happened ... you should see the metadata in the opening comment ...

My fault. I accidentally deleted it. Sorry!

There are many platforms supported by gRPC (10+) and providing documentation with how to crease each isn't realistic.
If you want to see how to use gRPC from a Node app then you should view that gRPC platform's documentation.


VS code complains Project 'ABC' is missing required library: 'lib/titanium.jar'

updated to new VS code titanium extension, now I am getting this message:
Project 'ABC' is missing required library: 'lib/titanium.jar'.  There is nothing wrong my android sdk. since everything works fine with Atom (with Appcelerator extension).

@jason919 are you getting that error during a build of your application? I can't find any context for that error in our CLI build. Could you please provide a more complete log for us to help diagnose the issue?

it is not during the build, just when I open the project in VS code, the "problem" pane shows that error.

@jason919 Interesting, to my knowledge this extension will not be adding that to the Problem pane. I believe that the problem pane should state what extension is adding that warning, does it definitely point to this extension providing the warning?

@jason919, did you track down where that warning was coming from?

Somehow it magically disappeared. I can’t get it appear again.


Inventory Interactions

Characters should not be able to take items from another characters inventory unless they are 2m or less apart and it should cost a action to do so in combat.

I see no way to actually implement this, unfortunately. I'll need some scripting to do things like that, and even then, I might not be able to do it. This will have to go on hold until we get more tools to work with

This would be difficult. A script that activates in combat that sets inventory bound on all items, except for dropping or throwing an item. Or block right click transfer to character...but not sure if that is possible.


Multiple paths in config

My next step is to allow multiple paths in the config.   This will bring Vite-plugin-pages closer to config parity with Vite-plugin-components.    Is this a feature you'd be interested in?

Can you provide some use cases about that?

For me, I like to store code in feature areas.  Was trying to come up with a way to do it like this
features/account/pages
features/users/pages
features/admin/pages
But I just realized thats going to get messy fast.  Would have to have some way to change the source dir / route path.  Might be a bit more involved than I first thought.  I had a simple prototype working but it won't cover this use case.
Could do this but seems awkward
features/account/pages/account
features/users/pages/users
features/admin/pages/admin

I think we only need to change getPagesPath(options), use it to build some thing like is in the same folder, then pass to generateRoutes.

Sounds good, but that will be a little tricky as you are changing routes path, but still need to keep filePath to resolve components.


chore: pre-commit initial

I have also added myself as a CODEOWNER for .pre-commit-config.yaml.

Codecov Report

Merging #103 (7230dd3) into develop (75bdb13) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop     #103   +/-   ##
========================================
  Coverage    55.10%   55.10%
========================================
  Files           29       29
  Lines         1254     1254
========================================
  Hits           691      691
  Misses         563      563

Impacted Files
Coverage Δ

splunk_connect_for_snmp_poller/manager/poller.py
0.00% <ø> (ø)

...ect_for_snmp_poller/manager/static/mib_enricher.py
94.11% <ø> (ø)

..._connect_for_snmp_poller/manager/task_utilities.py
20.76% <0.00%> (ø)

splunk_connect_for_snmp_poller/manager/tasks.py
36.23% <0.00%> (ø)

tests/test_hec_sender.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

:tada: This PR is included in version 1.5.0-develop.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[wip] add bond for proposing link

This PR reserves a bond amount for every link proposal.

I think the return of the deposit should happen in link_anchors ext, but this requires storing the details of the proposer. Opening this draft to ensure I'm going the right way.


HTTP import test is brittle

http://jenkins.cgcloud.info/job/toil-branches/branch/releases%252F3.2.x/7/testReport/junit/src.toil.test.jobStores.jobStoreTest/FileJobStoreTest/testImportHttpFile/

This should be fixed on the 3.2.x branch and master.

I want it fixed on the 3.2.x branch to avoid this test from getting in the way of future point releases on that branch.


chore(main): release 0.9.3

:robot: I have created a release beep boop
0.9.3 (2024-12-19)
Bug Fixes

bump sanitize-html from 2.13.1 to 2.14.0 (#641) (c988923)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.9.3
:sunflower:


Inventory 0.17.2.final + Agent 0.20.0.final

Doing these two commits as a single PR so that we see the two coming together in a green build.

Thanks @metlos


Ayr 787/date filter validation styling || WIP

Changes in this PR

updated date_validator.py to retrieve specific date field errors,

updated date_filters_validator.py to return the specific date field errors,

updated routes.py to return date field errors to html template,

updated date-filters.html to show an error class if the date field returned error,

updated test cases in test_date_validator.py, test_date_filters_validator.py,

amended basic styling in _browse-consignment.scss to align date_from and date_to in filter tray

updated env.vars and readme.md to set correct variables

JIRA ticket
https://national-archives.atlassian.net/browse/AYR-787
Screenshots of UI changes
Before
After

[ ] Requires env variable(s) to be updated

updated .env.e2e_tests.template in this PR as it was not allowing e2e tests to work, please use the updated template to run e2e tests

updated date_validator (removed browse_consignment reference) , moved to date_filters_validator function


Print challenges in a level

Implementation of #10

Hey @Ishan1742 Thanks for the PR, I'll review this within the next day!

Hey @Ishan1742 , I ran
C:\Users\benth\Programming\test>git gud challenges
Printing challenges for level: "intro"

commits
branching

C:\Users\benth\Programming\test>git gud challenges intro
Printing challenges for level: "intro"

commits
branching

C:\Users\benth\Programming\test>git gud challenges merging
Printing challenges for level: "merging"

octopus

All that looks good.
I ran this:
C:\Users\benth\Programming\test>git gud load merging
Setting up challenge: "merging octopus"
Setup complete
Type "git gud instructions" to view instructions

C:\Users\benth\Programming\test>git gud challenges
Printing challenges for level: "intro"

commits
branching

C:\Users\benth\Programming\test>cat .git\gud\level
intro commits
C:\Users\benth\Programming\test>echo merging octopus > .git\gud\level

C:\Users\benth\Programming\test>git gud challenges
Printing challenges for level: "merging"

octopus

This failure is mostly my fault since switching levels doesn't work properly (as of the commit your code is based off of), so I manually set the level and the output was as expected. This part is good.
I ran this
C:\Users\benth\Programming\test>git gud challenges mergeing
Traceback (most recent call last):
  File "C:\Users\benth\Miniconda3\envs\git-gud\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Users\benth\Miniconda3\envs\git-gud\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "c:\users\benth\programming\git-gud\gitgud\__main__.py", line 244, in <module>
    GitGud().parse()
  File "c:\users\benth\programming\git-gud\gitgud\__main__.py", line 237, in parse
    self.command_dict[args.command](args)
  File "c:\users\benth\programming\git-gud\gitgud\__main__.py", line 169, in handle_challenges
    level = all_levels[args.level_name]
KeyError: 'mergeing'

This is unexpected input and the command should fail, but when git gud challenges mergeing fails, can you check for the KeyError manually and "fail gracefully" by stating "there is no level mergeing" and say that the levels can be viewed by typing git gud levels.
If you want, you can either exit right after that or print out the challenges for the current level. That last part is up to you.
Once all the commands work properly, I'll take a look at the code itself, and from there if all looks good, I can merge it! Thanks for the PR!

Ok sure I'll do that.

I did add a check for KeyError. But even without that it was working in my computer because I had put it in a try except block. I think it might be a windows issue?? Anyways check the current print challenges and do let me know if any more changes required. Thanks.

This was the output before the current commit.

Just ran it all. Output looks great. Code looks great.
Two things, then we're good to go:

If you wouldn't mind, can you change the flag from being 0/1 to False/True
When you get a KeyError, add a print statement that tells the user to run the  "git gud levels" command so if a user wants to figure out which level they're trying to talk about, they can.

I guess this should do it.

Yep. That does it! Thank you @Ishan1742 . Your changes are merged!

Thanks. Happy to contribute. I will continue with the next issue 😀.


TonelWriter must write methods "as-is" without using the source filter

TonelWriter relies on the #sourceString method that does some code formatting, which can change the originally compiled code.
We should use an equivalent version of #sourceString without passing the resulting source from the source filter.

Adriaan Van Os suggested using the following instead of #sourceCode.
CompiledMethod>>#sourceStringTonel
    "Return the source code for the receiver."

    "Bypassing CompiledMethod>>#sourceString that manipulates the source.
     https://vast-support.instantiations.com/support/tickets/80256
     https://github.com/instantiations/tonel-vast/issues/140"

    ^EmLibrary cache methodSourceAt: self filePointer decompress: true

I'd only change the selector to tonelSourceString.

Closed as per 9e84fbf commit.


After create AP. AP has internet. Machine does not

Running xbuntu 14.xx
The following commands are used to create my ap
sudo rfkill unblock wlan
sudo create_ap -m bridge wlan0 p1p1 nsa nosuchaccount

The ap is successful. Mobile devices connect and can access the internet.
The machine is no longer able to access the internet. I can ping internal addresses but not hosts names
A ping command to www.yahoo.com results in
ping www.yahoo.com
ping: unknown host www.yahoo.com

Is there a solution to this?

A feature from NetworkManager below 0.9.9 is missing and because of this some stuff are not working properly in Ubuntu 14. You need to update to Ubuntu 15 which has a newer NeworkManager or if you want to stay on 14, you can use Wicd insted of NetworkManager.
If you do one of the above and you still have problems, please reopen the issue.

Thank you.


[Bug]: Multi-Document Agents throws InvalidRequestError: Unrecognized request argument supplied: functions

Bug Description
Multi-Documents agent example in llamaindex documentation(https://docs.llamaindex.ai/en/latest/examples/agent/multi_document_agents-v1.html) gives "Unrecognized request argument supplied: functions" error.
Tried different versions of llamaindex from 0.8.48 to latest ones.
Initially it gave me error that engine or deployment_id is required. I resolved this error by making change in source code. That is by adding engine as argument in openai/api_resources/chat_completion.py. After this change i get above mentioned error when i execute "top_agent.query()"
Version
0.8.56
Steps to Reproduce
https://docs.llamaindex.ai/en/latest/examples/agent/multi_document_agents-v1.html
Getting error while executing top_agent.query()
Relevant Logs/Tracbacks
No response

Are you using Azure OpenAI? what API version are you using? It looks like you might be using an old version that doesn't have function calling support.

Are you using Azure OpenAI? what API version are you using? It looks like you might be using an old version that doesn't have function calling support.

I am using Azure OpenAI with model gpt-35-turbo 0301 and api version 2023-08-01-preview

Are you using Azure OpenAI? what API version are you using? It looks like you might be using an old version that doesn't have function calling support.

I am using Azure OpenAI with model gpt-35-turbo 0301 and api version 2023-08-01-preview

Changing model version from gpt-35-turbo 0301 to gpt-35-turbo 0613 helped to resolve Unrecognizeed request argument issue


Why we not implement db transaction to mongodb?

because mongodb does not support transactions.

@cebe transaction is possible on the client side,
when we started to write Yii2, it had the similar abilty

can you point to some docs about that? I could not find any...

here https://github.com/yiisoft/yii2/issues/11

I do not see anything about transactions in the linked docs...

hmm, maybe I misunderstood

The list is copy from SQL layer, the question mark indicates that there was no information on whether such feature is provided by mongo or not. And as far as I know there is no such feature in mongodb. If you find it, point us to the docs and we can try implement it.

Hm, maybe only Two Phase Commits
http://docs.mongodb.org/manual/tutorial/perform-two-phase-commits/
but it not easy, i think

https://techcrunch.com/2018/02/15/mongodb-gets-support-for-multi-document-acid-transactions/

It is totally suported now.
We should be able to use them ;)
https://www.mongodb.com/transactions

It's very important update.

Adding some examples in case anyone else runs into issues. I got hung while trying to add the session to the $options parameter in  the yii\mongodb\Collection methods.
With Yi2 Mongodb Extension
$client = Yii::$app->mongodb;
$session = $client->manager->startSession();
$session->startTransaction();

try {
    $client->createCommand()
           ->addInsert(['key' => 'value'])
           ->setWriteConcern(['session' => $session])
           ->executeBatch('COLLECTION');

    // Uncomment to prevent transaction
    // throw new \Exception('Force fail. Stop transaction.');

    $session->commitTransaction();

} catch (\Exception $error) {
    $session->abortTransaction();
    print_r($error->getMessage());
}

Without Yii2 Mongodb Exension
$client = Yii::$app->mongodb->manager;
$session = $client->startSession();
$session->startTransaction();

try {
    $bulk = new \MongoDB\Driver\BulkWrite;
    $bulk->insert(['key' => 'value']);
    $client->executeBulkWrite('DB.COLLECTION', $bulk, ['session' => $session]);

    // Uncomment to prevent transaction
    /// throw new \Exception('Force fail. Stop transaction.');

    $session->commitTransaction();

} catch (\Exception $error) {
    $session->abortTransaction();
     print_r($error->getMessage());
}

$client = Yii::$app->mongodb;
$session = $client->manager->startSession();

Manger is null here. Whats the issue?

If I remember correctly and depending on your environment, you may need to install ext-mongodb to add the core API functionality, along with the MongoDB Driver. Make sure the mongodb.so extension exists in your PHP.ini file.
Also, I believe transactions are only available with Replicas(at least with Mongo Atlas) and MongoDB 4.2.


Add missing linux installation pages

Description
Add installation process for CentOS
Add installation process for Fedora
Add installation process for Rocky
Add installation process for Redhat
Add installation process for Amazon Linux
Closes #667
Closes #668
Closes #669

[x] Check all content with Grammarly
[x] Perform a self-review of my code
[x] Make corresponding changes to the rest of the documentation (consult with the DX team)
[x] The build passes locally
[x] My changes generate no new warnings or errors
[x] Add a corresponding label
[ ] If release-related, add a product and version label
[ ] If release-related, add release note on product PR

I have updated the PR.


Android - Add Unity Build Scripts and Export

Following the README I could not finish the setup, and using the video and the example project I could complete.
To complete I did the following:
So I created the folder unity-classes and inside and put my unity-classes.jar file generated in the Unity Export step and this other file build.gradle

You will need to modify the first line of ./android/settings.gradle :
include ':app'

to :
include ':app', ':unity-classes'


Events can be read twice by the same system when a state change occurs

Bevy version
bevy = "0.7"
What you did
I've run into this bug a few times since Bevy 0.5. If an event-reading system is added to multiple SystemSets, and the event is written as a state change occurs, then the system can receive the event, once before the state change, and once after. This can occur when a system that handles an event that leads to a state change is reused between SystemSets or when a system that changes the state also sends other events. I imagine it can also lead to difficult to reproduce and diagnose bugs when an event is randomly written at the same time as a state change, but I haven't run into this as far as I know.
Here's a minimal reproduction of the bug:
use bevy::prelude::*;

#[derive(Clone, Debug, Eq, Hash, PartialEq)]
enum GameState {
    First,
    Second,
}

struct MyEvent;

fn send_event(mut my_events: EventWriter<MyEvent>, mut state: ResMut<State<GameState>>) {
    info!("Sending an event!");
    my_events.send(MyEvent);
    state.set(GameState::Second).unwrap();
}

fn event_listen(mut my_events: EventReader<MyEvent>) {
    for _ in my_events.iter() {
        info!("Now that's an event!");
    }
}

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .add_state(GameState::First)
        .add_event::<MyEvent>()
        .add_system_set(
            SystemSet::on_update(GameState::First)
                .with_system(send_event)
                .with_system(event_listen.after(send_event)),
        )
        .add_system_set(SystemSet::on_update(GameState::Second).with_system(event_listen))
        .run();
}

2022-06-29T18:18:34.414457Z  INFO bevy_test: Sending an event!
2022-06-29T18:18:34.415308Z  INFO bevy_test: Now that's an event!
2022-06-29T18:18:34.417425Z  INFO bevy_test: Now that's an event!

What went wrong
I would expect the event to only be read once, but it is actually read twice.
Additional information
I think the issue is because Bevy treats a system that is added to different SystemSets as multiple different systems when determining whether it has already received an event. If so, this could be resolved in a number of ways that I can think of:

Prevent a system from receiving an event if a copy of it has already received that event
Allow the user to manually specify that two systems are the same
Clear events when a state change occurs

Since I've run into this bug a few times, I've used many different workarounds:

Make your event generic and add it once for each SystemSet that it can be read in, and add a type parameter to your system to specify which copy of the event it should use
Manually clear events on state exit using ResMut<Events<T>>
Explicitly order your systems
Add a resource when the state changes that prevents your system from acting on the event, then remove it after the first frame
Remove your system from SystemSets entirely and add it using .add_system()

The first workaround adds a lot of redundant code, especially when your events are already generic. The third doesn't always work for some reason, but it does solve the above code (when the order of the systems is reversed). With the last one, you may need to change some of your Res<T>s to Option<Res<T>>s if that resource isn't always present.
Thank you!

No longer the case, as looping run conditions are gone.


Rcar mmc restore mmc after reset

This is very strange pull request. You are adding some functionality, but you are not describing why do you need this in the first place. Could you please write a proper commit descripion?
Also, why are you removing MMC_INIT_SD_CLK_DIV_SHIFT?

This is very strange pull request. You are adding some functionality, but you are not describing why do you need this in the first place. Could you please write a proper commit descripion?
Also, why are you removing MMC_INIT_SD_CLK_DIV_SHIFT?

done

"does not necessarily reset I/O settings to boot" - are you using "to boot" in sense "as well" there?'

"to boot" here is meant a state as after power cycle

"does not necessarily reset I/O settings to boot" - are you using "to boot" in sense "as well" there?'

"to boot" here is meant a state as after power cycle

In this case you need to write "to the boot state". Because "to boot" has not the meaning you expected :)

"does not necessarily reset I/O settings to boot" - are you using "to boot" in sense "as well" there?'

"to boot" here is meant a state as after power cycle

In this case you need to write "to the boot state". Because "to boot" has not the meaning you expected :)

done


fix: bounding box

Hey, are you sure the bounding box is implemented correctly?
I'd expect

but get

Hey, I accidentally typed Math.min when I want to use Math.max, can you try v0.1.4 to check whether it fixes this problem?

It does! Thx :heart:


add system:deployer ClusterRole to HCCO

Creation of this role was moved from the OAS to the OCM operator (which we don't run)
https://github.com/openshift/openshift-apiserver/pull/331
https://github.com/openshift/cluster-openshift-controller-manager-operator/pull/270
We need to create this ClusterRole in the guest cluster.  Lack of this role is causing all Server-Side apply conformance tests to fail.

/lgtm

/hold
Need to make the annotation match exactly otherwise backport will result in reconcile loop

/hold cancel
I just removed the openshift.io/description annotation.  It is a recipe for reconciliation loops and should be make backporting compatible since we shouldn't fight the OAS create in 4.12.

/lgtm


Formatting fixes

Update all code blocks to markdown code fences
Spell check and fix spelling
Fix broken links
Switch to prismjs dark theme

that looks great :) Is it on purpose that there is no syntax highlighting anymore :?

Thanks @stebo85. My mistake, I've added back the syntax highlighting now

oh, cool! Thank you - looks fantastic now :) Now we just need to fix a few littel code blocks that have the incorrect naming. I love the markdown style now. The old way didn't render nicely on github, but now it's nice on every platform.

Yeah, the old method got a bit messy. This should be a lot simpler going forward.
Just putting down a note here...
I think we should switch from prismjs to chroma (hugo's included syntax highlighter) once the docsy team adds the copy button to the chroma renderer. Tracking this issue here. https://github.com/google/docsy/issues/1267
Chroma has a github-dark theme, it looks really good and will keep the site looking very consistent.

ahh, great! Yes, the copy button is a crucial feature :D


Some multischemas initialize required contents as optional

Description
In 5.10.20, a fix for issue #2831 was published. However, it only seems to take effect if the property that uses a multischema pattern is named multischema. I read over the associated commit, but am honestly at a loss as to how this behavior came about.
Minimal Reproduction
https://stackblitz.com/edit/ngx-formly-multischema-required-as-optional-2
Your Environment

Angular version: 10.2.5
Formly version: 5.10.20

a_multischema_by_any_other_name is not required so its required properties will be mandatory only when   the value is an object (that's part of JSON-SCHEMA spec also you may check the original issue https://github.com/ngx-formly/ngx-formly/issues/2752#issue-834019827)
So the solution is to set a_multischema_by_any_other_name as required or set object as default value.


linking

(for README) Tiled Map Editor link is : http://www.mapeditor.org (not .com)

Thanks for alerting me to this. Fixed.


Cache configuration

Hi,
i started to use your library and I would like to manage caching configuration, but I didn't find anything. Is it possible to set for how much time to keep some data (both in memory and on disk)?

If you look into the Constructor of the L4J8 class you see, that there is a line which sets the CacheProvider of the DataCall.class.
I dont have much time right now, but maybe you can figure it out yourself if you inspect the DataCall code and thr CacheProvider etc.

I looked into my code and here is a short example how to use a FileSystemCacheProvider with custom TimeToLive times for specific endpoints.
L4J8 l4j8 = new L4J8(SecretFile.CREDS);
FileSystemCacheProvider cacheProvider = new FileSystemCacheProvider(getFilesDir().resolve(L4J8_CACHE_DIR));
CacheLifetimeHint hints = new CacheLifetimeHint();
hints.add(URLEndpoint.V3_CHAMPION_ROTATIONS, TimeUnit.DAYS.toMillis(7L)); // 1 week
cacheProvider.setTimeToLive(hints);
DataCall.setCacheProvider(cacheProvider);

The memory cache is kinda wonky at the moment, and does not support the cache hints properly.
As for how to set the TTL, see the comment above from manuelwaldner.
The default values are shown here https://github.com/stelar7/L4J8/blob/a639248e2509b91aee1cb64c493c4982881d24b0/src/main/java/no/stelar7/api/l4j8/basic/cache/CacheLifetimeHint.java#L38-L65

I have a fix for the memory cache in the works, but no ETA. Most likely late October...


Explore folders feature (backend side)

Why ?
We need to have this feature in order to be able to explore all folders and sub folders we need to scan to look for audio files on our seamlisten tool easily (like a JupyterNotebook)
How ?
I have build a method to gather the folder contents given the parent folder path using os.walk() method to do that effectively, and return all the folder contents in an array well constructed.
and this endpoint is called whenver the path the user enters is a path to a directory not a normal file.
Test plan
when running the frontend and the backend successfully, paste in the field to fetch a path to a folder and when pressing fetch, if you opened the console you should see an array of folder contents shown correctly.

@Mortimerp9 @Celebio

this is a screenshot showing the Tree View in action.
@Mortimerp9


Cross-link the perf review document

The developers document is a more prominent place
where lots of contributors hit.
We copy some of the material from the doc to improve
visibilty and point to the rest of the doc.

Summary

Ticket Link

Thank you @cwarnermm !


Build debug kernel

@shenki this change is to the kernel build scripts which implement a debug sparse build and structs check. I have updated the openpower.xyz job to already use m branch and it works correctly, generating the new debug files which we archive.
It also fixes a few small things like bug in generating the proxy config, only adding the user and group if they don't already exist in the cached layer, and not always running apt-get update.

Once merged we could set jenkins to build back from openbmc master.