[PowerToys Run] Highlight color is too close to background color in dark mode

ℹ Computer information

PowerToys version: 0.25
PowerToy Utility:
Running PowerToys as Admin: Yes
Windows build number: Win10 1909 (18363.1198)

📝 Provide detailed reproduction steps (if any)

Set PowerToys to dark mode
Launch PT Run

✔ Expected result
The current selection highlight color should be lighter.
❌ Actual result
The color for the current selection is too close to the background color, noticing the difference at a glance is difficult.
📷 Screenshots

The highlight color is your accent color

Clearly it's not:

It's a darker shade of the accent color maybe, but it still too dark.

It is darker because you are using the dark theme

Maybe it should be the opposite

Is this not using the Accent colour with an opacity?  I know its WPF and not UWP.  The opacity should probably be something closer to 40-50% so it stands out better in both Light and Dark themes.

Is this not using the Accent colour with an opacity?  I know its WPF and not UWP.  The opacity should probably be something closer to 40-50% so it stands out better in both Light and Dark themes.

Could be, yeah.
This issue has been reported before. We should maybe consider using the Accent brushes in ModernWPF. We can then also replace our own implementations of the UWP styling.

Related to #4208, if I am not mistaken

@Sov3rain
please add your comments to #4208, so we have only one issue tracking this problem.
Thanks.


Add support for API keys

#3 exists, but seems to assume that these API keys will deprecate existing auth methods, which is not the case in the initial roll out of API keys
see cmu-delphi/epidatr#79 for the R implementation; the Python implementation might be slightly different (e.g. use .env and the dotenv package)

Closed in f7930a9

Closed in f7930a9


Fixed #1273 : Add a battery low mode in settings

Description
Image will not load if battery is low
Related issues and discussion
#1273
Screen-shots, if any

Travis-CI test has been failed just because "The job exceeded the maximum time limit for jobs, and has been terminated.".

Hey @PrajwalM2212,
Your changes looks good to me!

Hey @PrajwalM2212,
Your changes looks good to me!

Hey @PrajwalM2212,
Your changes looks good to me!

Hey @PrajwalM2212,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because something unrelated to the tests, like a problem a dependency or the build process itself.


Blur image

Hi,
Blur image > https://docs.microsoft.com/en-us/azure/azure-sql/database/media/authentication-aad-configure/aad.png

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

ID: 2d4a33b0-87d0-f60c-1b52-aeafa889e657
Version Independent ID: c281bdb7-c98c-6853-5077-41b3dd44d3a2
Content: Configure Azure Active Directory authentication - Azure SQL Database & SQL Managed Instance & Azure Synapse Analytics
Content Source: articles/azure-sql/database/authentication-aad-configure.md
Service: sql-db-mi
Sub-service: security
GitHub Login: @GithubMirek
Microsoft Alias: mireks

@divyabhushan Thank you for the feedback. This is being assigned to the content author to evaluate and update as appropriate.

Hi @divyabhushan , thanks for your feedback! My apologies, I'm not sure I understand your request. Are you asking us to blur this image? Or are you saying the image is blurry?
Thanks!
Masha from the SQL Docs team

Hi @divyabhushan , thanks for your feedback! My apologies, I'm not sure I understand your request. Are you asking us to blur this image? Or are you saying the image is blurry?
Thanks!
Masha from the SQL Docs team

Hi @MashaMSFT,
Sorry, maybe my message wasn't clear. I mean the image is blurry and is not very readable.
Thanks

Hi @divyabhushan , thanks again for contacting Microsoft and helping improve our docs! My apologies for the delay but we've processed a change to update the image, it should be live later this week. I'll go ahead and close out this Git Issue but please feel free to comment should you need further.
Thanks once more, and I hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#reassign:MashaMSFT
#please-close


Set Modal window height based upon browser window size using Bootstrap

how to set the max-height for the modal window to be 80% of the current browser window size using bootstrap.

Per CONTRIBUTING.md please do not use the issue tracker for personal support requests. Stack Overflow (twitter-bootstrap-3 tag) or Slack, or IRC are better places to get help with these subjects.


Can't build with typescript if moment is not installed

Environment

Tech
Version

material-ui-pickers
rc2

material-ui
beta.35

React
16.3

Browser
chrome

Platform
win10

Typescript
2.7.1

Steps to reproduce

implement date picker like described in the docs
Build the project

Expected behavior
Compilation should finish without warnings or errors.
Actual behavior
[at-loader] ./node_modules/material-ui-pickers/_shared/DateTextField.d.ts:5:24
    TS2307: Cannot find module 'moment'.

[at-loader] ./node_modules/material-ui-pickers/_shared/PickerBase.d.ts:1:22
    TS2307: Cannot find module 'moment'.

[at-loader] ./node_modules/material-ui-pickers/typings/date.d.ts:1:24
    TS2307: Cannot find module 'moment'.

Live example
The picker works, but there are typescript compile errors.
Update:
The typings have multiple import { Moment } from 'moment'; I can see the reason for this. But if moment is imported, it MUST be installed at build time. Even if its not used. I would suggest a generic solution or even an any type (if there is REALLY no other solution).
The date implementation is now generic and optional. What is really a cool thing. But so should be the typings.

Let me also add that the labelFunc property of <DateTimePicker> is locked to a Moment object, so that's another factor that's blocking TypeScript users. :)

Fixed in rc-3


Crashed on Android with RN 0.34.1

When running ImagePicker.showImagePicker the picker appears and allows me to make a selection, but the app crashes after I choose Library/Camera. It crashes before returning / logging anything.

Downgrade to RN 0.32.1. I just tried most of the later versions, including 0.35.0-rc.0, and launchCamera() seems to crash hard on these later versions. 0.32.1 seems fine for me though.

I can confirm @hasseio 's finding.
It works on RN 0.32.1 but not on anything higher than that.

Thanks for the help. With RN moving / changing so fast maybe listing which version a module was last confirmed to support should become a standard in developer documentation? Managing modules seems to be the biggest pain point right now. Great community might make up for it though.

I just dug in on this personally for my own project, and found the format for customButtons must have changed. Previously I had specified options.customButtons as just a simple hash:
options.customButtons = {
  'Upload Video': 'upload_video',
}

After changing it to the below it now works:
options.customButtons = [
  { title: 'Upload Video', name: 'upload_video' },
];

/cc @hasseio @FearMediocrity @dangnelson

Currently this package won't even compile, digging into it and providing a solution.
<...>/node_modules/react-native-image-picker/android/src/main/java/com/imagepicker/ImagePickerActivityEventListener.java:6: error: cannot find symbol
import com.facebook.react.bridge.BaseActivityEventListener;

@robmoorman It compiles fine for me -- I'm using RN 0.34.1

@robmoorman Try this. Make sure to run a gradle clean after the change.

I'm working on developing a standard for displaying version support for React Native modules:
https://github.com/dangnelson/react-native-version-support-table
Would love some feedback.

@pthrasher I'm not using custom buttons at all.

I'm using version 0.21.5 and after upgrading to react native 0.34 the app is crashing every time that I select the "Choose from Library..." option.
This is happening on Android and iOS.

I'm having the same issue (crashes when picking Choose From Library option) but I've only tried it on iOS. I'm running RN v0.32.0.

If this file has this line of code, you can try to comment.
android/app/src/main/java/[your package]/MainActivity.java
`
public void onActivityResult(int requestCode, int resultCode, Intent data){

    super.onActivityResult(requestCode, resultCode, data);

    //mReactInstanceManager.onActivityResult(this, requestCode, resultCode, data);

}

`
I just do it, it’s working

If you're having iOS issues it's likely related to the new iOS 10 permissions requirements.
http://stackoverflow.com/questions/38498275/ios-10-changes-in-asking-permissions-of-camera-microphone-and-photo-library-c
I successfully used this to solve it on iOS:
https://github.com/yonahforst/react-native-permissions

In my experience you have to both add the permissions to the info.plist and explicitly request them using a method like the component uses above or the app will crash in iOS.

Need an available version ASAP, pls.

Thanks @dangnelson got it working with your suggestions!

@lucassaldanha why are you using verson 0.21.5? The latest is 0.22.9, try upgrading.

@dangnelson are you still having issues on Android?

I'm about to try again, will let you know.

Just got it working on a fresh project with RN 0.35.0 & image-picker 0.22.9
Only issue I ran into was permissions weren't added to the AndroidManifest.xml file automatically by react-native link

Glad to hear it. Since you were the original poster and it's now working, I'm going to close this issue since it's gone kind of all over the place. If anyone else is still having an issue, please open a new one with your specific problem.

Just leaving this here in case somebody has the same problem, I got an error:
java.lang.AbstractMethodError: abstract method "void com.facebook.react.bridge.ActivityEventListener.onActivityResult(android.app.Activity, int, int, android.content.Intent)"
if you get this problem, this is what worked for me:
Upgrade to rn 0.35
Upgrade image-picker to 0.22
cd android && ./gradlew clean

to add to ospfranco's comment:
I had the same error, but his steps did not solve my problem.
I figured out that another RN package was doing reactContext.addActivityEventListener(this);, but the package was using an (old) ActivityEventListener, rather than a RN 0.35+ BaseActivityEventListener.  The ImagePicker event listener was firing correctly, but RN would loop through all of the listeners, encounter the old listener and crash.  The stack trace would just show the error that ospfranco posted, so it wasn't clear which package was causing the problem


Dynamic options refetch

Docs for https://github.com/Altinn/altinn-studio/issues/7888
Also added latest entry for app-frontend

Should probably mention https://github.com/Altinn/altinn-studio/issues/7903?

Contributor

Yeah, good suggestion. I'll add some documentation for this.

Should probably mention Altinn/altinn-studio#7903?

Yeah, good suggestion. I'll add some documentation for this.


scrape: Allow to disable end-of-run staleness marker injection for targets

Use case
Suppose a project uses scrape package as a library in a cluster where targets are sharded across a number of instances. When a new instance joins the cluster, some portion of targets will typically be handed over to that new instance. The previous owning instance of a target that moved will attempt to inject staleness markers at the end of the scrape loop run (after delay of ~2 intervals). If out-of-order ingestion is enabled, writing staleness markers will succeed and results in occasional gaps in metrics.
Grafana Alloy faces the above issue in practice.
Proposal
Expose a method on scrape manager that allows users to disable end-of-run staleness for user-specified targets.
In above scenario, users of the scrape package will be able to detect that given target has moved to another instance and ensure that there are no unnecessary staleness markers written by the previous owning instance.
More generally, giving users control over when the staleness markers injection should or should not happen, can allow to implement more use cases correctly and with fewer data issues.

I guess https://github.com/prometheus/prometheus/issues/13302 is partially related?

grafana/prometheus has an implementation of this which could be upstreamed, unless there some proposals how to do it better

Related: https://github.com/prometheus/prometheus/issues/12941


Some labels are marked as N/A why?

As an initial step for cost-based optimizer, I want to add SSD model along with FasterRCNN.
I wonder why some labels are marked as N/A for FasterRCNN like here?

@jiashenC https://pytorch.org/docs/stable/torchvision/models.html#object-detection-instance-segmentation-and-person-keypoint-detection

Interesting. So COCO has 81 classes including background. What is 91 classes? And the model is defined to be 91 classes, 10 of those are actually not used (return N/A).


Helm Release Detail

Describe the bug
Hi,
When I want to view the details of any of the Helm releases, the details of the first opened release after the application starts are always displayed.
To Reproduce
Steps to reproduce the behavior:

Go to 'Helm > Releases'
Click on 'Any Release on the List'
Click on 'Any Other Release on the List'
You will see the release detail, which you click on the first detail.

Expected behavior
I'm waiting to view the release detail I clicked on, but I suspect there is a cache here.
Screenshots
Not available.
Environment (please complete the following information):

Lens Version: 2023.1.110749-latest
OS: OSX
Installation method (e.g. snap or AppImage in Linux): Downloading Lens Desktop File

Duplicate of https://github.com/lensapp/lens/issues/6721


Are functions that take object parameters represented/callable?

No idea if this is a "typescript thing" or a "eventhorizon thing" or a "me thing" but I'm using the pirate fort sample (basically because most of what I want to use seems built already) and looking at eg BabylonJS's Mesh.CreateBox:
const size = 1;
BABYLON.Mesh.CreateBox("box", size, scene);

This call is represented in the C# no problems, and I can use it to create cubes. I'm fiddling with how to create cubiods using this "overload" cribbed from https://playground.babylonjs.com/#6XIT28#5:
const options = {
    width: 1,
    height: 2,
    depth: 3
};
const box = BABYLON.MeshBuilder.CreateBox("box", options);

I achieved what I want in TypeScript at https://playground.babylonjs.com/#6XIT28#781
Wondered if I could get it working in C# just by duplicating and altering the existing CreateBox:
        public static Mesh CreateBox(string name, decimal size, Scene scene = null, System.Nullable<bool> updatable = null, System.Nullable<decimal> sideOrientation = null)
        {
            return EventHorizonBlazorInterop.FuncClass<Mesh>(
                entity => new Mesh() { ___guid = entity.___guid },
                new object[]
                {
                    new string[] { "BABYLON", "Mesh", "CreateBox" }, name, size, scene, updatable, sideOrientation
                }
            );
        }

Additional overload:
        public static Mesh CreateBox(string name, object options, Scene scene = null)
        {
            return EventHorizonBlazorInterop.FuncClass<Mesh>(
                entity => new Mesh() { ___guid = entity.___guid },
                new object[]
                {
                    new string[] { "BABYLON", "Mesh", "CreateBox" }, name, options, scene
                }
            );
        }

No dice (hah) though; no box adds to the scene when I use this C#:
var box = Mesh.CreateBox("box", new { width = 1, height = 2, depth = 3, faceColors = new[] { green, green, green, green, blue, red } }, scene);

.. which I believe is the equivalent to the working TS:
        let options = {
            width: 1,
            height: 2,
            depth: 3,
            faceColors: [ green, green, green, green, blue, red ]
        };

        var box = BABYLON.MeshBuilder.CreateBox("box", options, scene);

Where did the chain break down? Bug in my c#? Misunderstanding of what EventHorizon can call/the process needed to get an alternative overload working?

Thanks for Taking an interest in this project! And finding a deficiency in the sister project this library depends on!
The sister project does not have logic to handle the arrays in the options, it has to loop the arguments and replace them with the cached version. For performance the whole object is not passed between the layers, just a unique id.
I have a fix for this, but will require me to pass it down the layers. As for your code, you can use the MeshBuilder.
The below code should work after I get the Update through. I will update this ticket when I get the update through to here.
var columns = 6m;
var rows = 1m;
var faceUV = new Vector4[6];

for (var i = 0; i < 6; i++)
{
    faceUV[i] = new BABYLON.Vector4(i / columns, 0, (i + 1) / columns, 1 / rows);
}

var mat = new BABYLON.StandardMaterial("mat", scene);
var texture = new BABYLON.Texture(scene, "https://assets.babylonjs.com/environments/numbers.jpg");
mat.diffuseTexture = texture;

var alpha = 1;
var red = new BABYLON.Color4(1, 0, 0, alpha);
var blue = new BABYLON.Color4(0, 0, 1, alpha);
var green = new BABYLON.Color4(0, 1, 0, alpha);

var options = new
{
     faceUV = faceUV,
     wrap = true,
     width = 1,
     height = 2,
     depth = 3,
     faceColors = new List<Color4> { green, green, green, green, blue, red }
};

var box = BABYLON.MeshBuilder.CreateBox("box", options, scene);
box.material = mat;

@CaiusJard The new version  is published, I also created a sample page with the gist of your playground.
https://wonderful-pond-05f7b3b10.azurestaticapps.net/mesh/builder
You can see the source here: https://github.com/canhorn/EventHorizon.Blazor.TypeScript.Interop.Generator/blob/05dedfdd7469b2ada3fc210df7d8496f53fe82ef/Sample/EventHorizon.Blazor.BabylonJS/Pages/MeshBuilderExample.razor.cs

@canhorn amazing; thanks for this


No notification triggered on FB Messenger mobile device c#

Hello to all,
When I use Conversations.SendToConversationAsync, the message arrives well on FB messenger but it not there no notification.
How I can make to triggers notification on mobile device for FB messenger?
Thank you

That is entirely up to the messenger client.  All we do is call the apis to deliver.
Note: Please only submit Bot Builder SDK bugs here. If you have general questions or implementation issues, please submit a question on Stack Overflow at http://stackoverflow.com/questions/tagged/botframework


[DatePicker] multi language support

Hi!
It will be extremely useful to set Days of the Week and Months in different languages (spanish in my case).
I dont think that it should be that hard, the interface should be something like this:
<DatePicker
  hintText="Portrait Dialog"
  daysName={['Sunday', 'Monday', ...]}
  monthsName={['January', 'February', ...]}
  />

I believe that the value outstands the complexity of implementing this :smile:
Thanks!

+1
I think it would be more enjoyable an object of culture, like
<DatePicker
  hintText="Portrait Dialog"
  daysName={['Sunday', 'Monday', ...]}
  monthsName={['January', 'February', ...]}
  />

+1
I think it would be more extensible an object of culture, like
var cultureInfo = {
    day: {
        name: ['Sunday', 'Monday', ...],
        abbr: ['Sun', 'Mon', ...]
    },
    month: {
        name: ['January', 'February', ...],
        abbr: ['Jan', 'Feb', ...]
    }
};

<DatePicker
  hintText="Portrait Dialog"
  cultureInfo={cultureInfo}
  />

+1

@mmrtnz @hai-cea How do you guys feel about utilizing http://momentjs.com/ for material-ui's date/time needs? We would be able to extract almost all of the datetime utils and have il8n for 'free'.

@jkruder +1

+1

+1

@rkmax It's still an open issue - Please see #537

+1
I can help fixing if needed
Thanks

Thanks @Griffosx. I need get solve some problems with my local dev folder to starting to contribute to mterial-ui

@hai-cea In April I proposed using momentjs to enhance the date picker. It has i8l support so we could lean on their implementation and just expose some props. What do you think?

I did something very quickly

let moment = require('moment');
moment.locale('es');

module.exports = {

  addDays(d, days) {
    let newDate = this.clone(d);
    newDate.add(days, 'days');
    return newDate.toDate();
  },

  addMonths(d, months) {
    let newDate = this.clone(d);
    newDate.add(months, 'months');
    return newDate.toDate();
  },

  addYears(d, years) {
    let newDate = this.clone(d);
    newDate.add(years, 'years');
    return newDate.toDate();
  },

  clone(d) {
    return moment(d);
  },

  cloneAsDate(d) {
    let clonedDate = this.clone(d);
    return clonedDate.toDate();
  },

  getDaysInMonth(d) {
    let resultDate = this.clone(d);

    return resultDate.daysInMonth();
  },

  getFirstDayOfMonth(d) {
    let clonedDate = this.clone(d);
    clonedDate.startOf('month');
    return clonedDate.toDate();
  },

  getFullMonth(d) {
    let clonedDate = this.clone(d);
    return clonedDate.format('MMMM');
  },

  getShortMonth(d) {
    let clonedDate = this.clone(d);
    return clonedDate.format('MMM');
  },

  getDayOfWeek(d) {
    let clonedDate = this.clone(d);
    return clonedDate.format('dddd');
  },

  getWeekArray(d) {
    let dayArray = [];
    let daysInMonth = this.getDaysInMonth(d);
    let daysInWeek;
    let emptyDays;
    let firstDayOfWeek;
    let week;
    let weekArray = [];

    for (let i = 1; i <= daysInMonth; i++) {
      dayArray.push(new Date(d.getFullYear(), d.getMonth(), i));
    }

    while (dayArray.length) {
      firstDayOfWeek = dayArray[0].getDay();
      daysInWeek = 7 - firstDayOfWeek;
      emptyDays = 7 - daysInWeek;
      week = dayArray.splice(0, daysInWeek);

      for (let i = 0; i < emptyDays; i++) {
        week.unshift(null);
      }

      weekArray.push(week);
    }

    return weekArray;
  },

  format(date) {
    let m = date.getMonth() + 1;
    let d = date.getDate();
    let y = date.getFullYear();
    return m + '/' + d + '/' + y;
  },

  isEqualDate(d1, d2) {
    return d1 && d2 &&
      (d1.getFullYear() === d2.getFullYear()) &&
      (d1.getMonth() === d2.getMonth()) &&
      (d1.getDate() === d2.getDate());
  },

  isBeforeDate(d1, d2) {
    let date1 = this.cloneAsDate(d1);
    let date2 = this.cloneAsDate(d2);

    return (date1.getTime() < date2.getTime());
  },

  isAfterDate(d1, d2) {
    let date1 = this.cloneAsDate(d1);
    let date2 = this.cloneAsDate(d2);

    return (date1.getTime() > date2.getTime());
  },

  isBetweenDates(dateToCheck, startDate, endDate) {
    return (!(this.isBeforeDate(dateToCheck, startDate)) &&
            !(this.isAfterDate(dateToCheck, endDate)));
  },

  monthDiff(d1, d2) {
    let m;
    m = (d1.getFullYear() - d2.getFullYear()) * 12;
    m += d1.getMonth();
    m -= d2.getMonth();
    return m;
  },

  yearDiff(d1, d2) {
    return ~~(this.monthDiff(d1, d2) / 12);
  }

};

@jkruder Instead of using momentjs, what do you think about adding a DateTimeFormat and a locale as optionals prop to the DatePicker?
The DateTimeFormat would follow this spec.
We can update https://github.com/callemall/material-ui/blob/master/src/utils/date-time.js to provide the default interface (locale=en-US obviously).
Then if someone wants to change the locale, he can provide Intl.DateTimeFormat if it's available in the browser or this polyfill.

@oliviertassinari I like the idea of using Intl.DateTimeFormat, lighter than depending on momentjs. However, I do not think we should be recreating the wheel. If Intl.DateTimeFormat can provide all of the formatting that we need then we may not need a library like momentjs. I took a look at the polyfill but did not see anything that indicated Safari webkit support. Maybe I missed it? If we can confirm the polyfill works for Safari I see no reason why we could not move forward with Intl.DateTimeFormat over momentjs.

I think that the intl polyfill support every browser since they use the data of http://cldr.unicode.org/ then do some massage to produce the output.

However, I do not think we should be recreating the wheel

I was proposing to update date-time.js to not introduce breaking changes (adding a polyfill of Intl for browsers that don't support it).

I read through the readme of the polyfill more closely and I believe that it does support Safari (although I did not see anything that explicitly confirms that).
In regards to recreating the wheel, there may be some functionality in date-time.js that already exists in momentjs. For locale support if we only need to consume the Intl.DateTimeFormat API then I have no issue with it. Folks should not be consuming date-time.js directly so any changes we make (or replace it with something else) should be hidden from the consumer of material-ui.

Example usage:
<DatePicker
    hintText="Fecha final"
    container="inline"
    DateTimeFormat={ Intl.DateTimeFormat }
    locale='es-ES'
    mode="landscape" />

Hey, I'm using Material UI date picker for reactjs 4 years after this thread, is there multi language support now?

@raboija https://github.com/mui-org/material-ui-pickers.

What is the simplest way to change language of the calendar to french?
                                        <MuiPickersUtilsProvider utils={DateFnsUtils}>                                             <Grid container justify="space-around">                                                 <KeyboardDatePicker                                                     margin="normal"                                                     id="date-picker-dialog"                                                     //label="Date picker dialog"                                                     format="dd/MM/yyyy"                                                     placeholder="Jours/Mois/Année"                                                     locale='fr-FR'                                                     value={this.state.startDate}                                                     onChange={this.handleChange}                                                     KeyboardButtonProps={{                                                         'aria-label': 'change date',                                                     }}                                                 />                                             </Grid>                                         </MuiPickersUtilsProvider>

I tried locale, didn't work

this seems a bit over complicated for what I wanna do, isn't there a simpler way? https://material-ui-pickers.dev/localization/moment


New-CsBatchPolicyPackageAssignmentOperation not Working anymore

Hi,
I have this problem for two weeks now, I can't assign any package through the Teams Module to any users of my tenant, I tried to reinstall the module on an other computer, I tried with the full mail @, I tried with the user ID but none of it is working.
The other commands works fine and I'm using the latest version (1.1.6)
I wonder if I'm the only one having this issue...
PS C:\WINDOWS\system32> New-CsBatchPolicyPackageAssignmentOperation -Identity 1900a2e2-3441-4857-b8fe-95df44b18808 -PackageName Education_Teacher
New-CsBatchPolicyPackageAssignmentOperation : Une ou plusieurs erreurs se sont produites.
Au caractère Ligne:1 : 1

New-CsBatchPolicyPackageAssignmentOperation -Identity 1900a2e2-3441-4 ...

  + CategoryInfo          : NotSpecified: (:) [New-CsBatchPoli...gnmentOperation], AggregateException
  + FullyQualifiedErrorId : System.AggregateException,Microsoft.TeamsCmdlets.PowerShell.Custom.NewCsBatchPolicyPacka
 geAssignmentOperation

Any help would be gladly accepted :)
Cordially,
Gabe

Hello, is there any news concerning this issue ? Is it only me or is this a general problem ?

Hello @gfournier04
I tested the cmdlet to see if we need to impact any documentation based on your feedback, I am using module version 1.1.6, and had no issues at all.
I recommend opening a support ticket as here we are limited to documentation issues.
Hope this information helps you.
Thanks

Thanks a lot :) now I know the problem comes from me or my tenant.
I will try to fix my issue then
Best regards.

Thanks a lot :) now I know the problem comes from me or my tenant.
I will try to fix my issue then
Best regards.

Thanks for returning and sharing your feedback.


[occm] possible to disable shared-lb feature

What this PR does / why we need it: Currently there is no way to disable "shared-lb" feature that was added in https://github.com/kubernetes/cloud-provider-openstack/pull/1648
However, the current solution updates annotations in services. This makes gitops tools to behave incorrectly and we are already seeing problems with this. We have basically endless loop going on.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

[openstack-cloud-controller-manager] shared-lb feature can be disabled by setting `max-shared-lb` to 0

/cc @ramineni

However, the current solution updates annotations in services. This makes gitops tools

@zetaab I assume because we keep updating annotation of service, the tool like ArgoCD keep seeing the service got update then keep syncing, correct?
I think it's reasonable to set to 0 in order to disable the feature ..
/lgtm

@jichenjc exactly tools like argocd does not like when we modify annotations like this.

/cc @lingxiankong

@jichenjc exactly tools like argocd does not like when we modify annotations like this.

@zetaab This doesn't make sense to me, could you please describe your issue in detail? The resource annotation is meant to be used like this, the other cloud providers are also updating Service annotations for their own management purpose.
If using ArgoCD, it's common to use ignoreDifferences to solve issue like this.
/hold

@lingxiankong as I see it, OpenStack cloud provider should NOT change the behaviour that people are already using with argocd or another tools. If people really want this feature, they could enable it. I do not see the reason why this is enabled by default, it will break peoples CI pipelines. No - we are not using argocd. We are using own tool which support immutable way of deployment manifests. This feature is breaking that idealogy.

@lingxiankong as I see it, OpenStack cloud provider should NOT change the behaviour that people are already using with argocd or another tools.

The development of upstream project shouldn't be restricted or limited by the external systems using it, especially the new features are implemented using the recommended way in the upstream Kubernetes.

If people really want this feature, they could enable it. I do not see the reason why this is enabled by default, it will break peoples CI pipelines.

This feature is not designed as opt-in, even without load balancer sharing use case, we still need to use the annotation to efficiently manage the Service, using the load balancer name induced by Service namespace and name will bring trouble in some use cases. As far as I know, other cloud providers like DigitalOcean cloud provider is also dong the same thing, see https://github.com/digitalocean/digitalocean-cloud-controller-manager/blob/master/docs/getting-started.md#load-balancer-id-annotations.

No - we are not using argocd. We are using own tool which support immutable way of deployment manifests. This feature is breaking that idealogy.

I assume you are using GitOps to manage the cluster resources, from my point of view, it should be a common feature that any GitOps tool supports to ignore some particular resources or resource fields, e.g. either ArgoCD or FluxCD supports such configurations.
I can give you another use case showing that updating resource by some controllers inside the cluster is common. In Istio, Virtual Services require a weight when two destinations are listed in a route. If a user creates the Virtual Service using GitOps and another controller modifies those weights to shape traffic, the Virtual Service will be out of sync with what is stored in Git. If there is a Sync, the Virtual Service will be revert to the Git state and then the other controller should revert it back.
I believe you are suffering the same issue with openstack-cloud-controller-manager. which I think your GitOps tool should be fixed.

In my opinion that is wrong if any cloud-provider will touch the service spec itself. There is Status field for things like this. https://github.com/kubernetes/api/blob/master/core/v1/types.go#L4194 if it does not support custom things, it can be added to upstream by making PR. It is not great solution to modify spec annotations.

I'm afraid what you said is wrong, the above examples could explain well. Additionally, Status is not supposed to be used in this case. The status describes the current state of the object which is mutable and transitional, apparently the load balancer ID of a Service is not going to be changed through its lifecycle.

Everyone has own opinions, this is my opinion. I made PR that makes it work like I want, still the other way (solution made by @lingxiankong ) works in similar way after this. I do not see the huge issue here?

I totally agree that Everyone has own opinions. However, this PR removed the lb ID annotation which should always be there moving forward, that also makes the feature implementation in the future more complicated.


"New Tab" in New Tab

When I click on it, it closes the new tab. It shouldn't be visible by itself

It's useful to show the current "New Tab" page, as you can see where it is in relation to other tabs. Plus it's meant to be a list of all open tabs.
I can see how it could be confusing to close the page when clicking on it though. Perhaps it should not be a clickable link, or at least not close the page.

Fixed in https://github.com/maxmilton/new-tab/pull/2082 and released in https://github.com/maxmilton/new-tab/releases/tag/v0.22.0.
The updated version has been submitted to the Chrome Web Store, so it will start rolling out to user automatically soon.


Platforms table loads empty

http://www.chibi.ubc.ca/Gemma/arrays/showAllArrayDesigns.html
Platforms are only loaded when the "hide orphans" checkbox in top right corner is unchecked.

Faulty release has been rolled-back, new version does not contain the problem anymore.


Edited.

The following does work:
# focus window
hyper - h : yabai -m window --focus west
hyper - j : yabai -m window --focus south
hyper - k : yabai -m window --focus north
hyper - l : yabai -m window --focus east

# move window
cmd + shift - h : yabai -m window --warp west
cmd + shift - j : yabai -m window --warp south
cmd + shift - k : yabai -m window --warp north
cmd + shift - l : yabai -m window --warp east

but if I replace "cmd" with "hyper" in the second set of commands, it does not work. It seems that hyper +  doesn't work. Here, I've set up the hyper key via Karabiner.

Please delete.


Enable Tracking Total Number Of Blocks

At the moment we use a dedicated goroutine to track number of blocks in TemporalX. This will allow us to passively capture this information, and not need a goroutine to parse AllKeysChan. While this will add a miniscule overhead to general blockstore operations, it will be vastly more efficient

Whenever AllKeysChan is called, and it doesn't result in an error, with a count greater than 0 we override the metric with the result from that.
Any time Put or PutMany is called we increment the count
Anytime DeleteBlock is called we decrease teh count

That's actually a really good idea. I was doing some profiling and the system we have in TemporalX for tracking block size is actually one of the parts that allocates the most memory over long periods of time, it doesn't cause any serious memory usage but overtime it's a big driver of our allocations.
I'll do that in a followup PR after I merge this in


⚠ File access (beta) has degraded performance

In 7c8086b, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 12357 ms

Resolved: File access (beta) performance has improved in 3e85766 after 7 minutes.


Plausible Deniability: Add "Padding" property to submission payload

Currently, mobile apps are collecting keys based on their install date. When a user uploads his keys (after being tested positive), the payload will contain 1-13 keys. Although the traffic between mobile & server is secured, an attacker may still sniff the packages in the network and predict, based on the request size, how many keys are probably part of the submission request. This may then lead to additional information for the attacker in an attempt to deanonymize a user.
In order to mitigate this attack, add a new field to the submission payload, called padding of type string. The field shall be optional and must not break compatibility to the current/older mobile implementations (which should be given due to the use of protobuf). The server shall not process this field.

is amaizing one
On Thu, Jul 9, 2020 at 12:22 PM Christian Kirschnick <
<EMAIL_ADDRESS>wrote:

Currently, mobile apps are collecting keys based on their install date.
When a user uploads his keys (after being tested positive), the payload
will contain 1-13 keys. Although the traffic between mobile & server is
secured, an attacker may still sniff the packages in the network and
predict, based on the request size, how many keys are probably part of the
submission request. This may then lead to additional information for the
attacker in an attempt to deanonymize a user.
In order to mitigate this attack, add a new field to the submission
payload, called padding of type string. The field shall be optional and
must not break compatibility to the current/older mobile implementations
(which should be given due to the use of protobuf). The server shall not
process this field.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/corona-warn-app/cwa-server/issues/658, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYK7OIXN24HQQDHDUW6FH3R2WD6TANCNFSM4OVM5KTQ
.

@KevponSAP do we have any update on https://github.com/corona-warn-app/cwa-backlog/issues/22 this has been referred by @tkowark

@GPclips the padding feature was merged on July 20


clean_names breaks for MultiIndex Columns

MultiIndex columns returns expected str not tuple type error.
If you instead used,
df.rename(columns=lambda x: x.lower().replace('_', ''))
this would work for standard and MultiIndex DataFrames.
Can do PR if required.

@JoshuaC3 definitely would be a welcome addition! Would you be kind enough to put in a PR for this, including an example and test as well?

@ericmjl All done. Added MultiIndex tests and everything passes. Will do PR now.
FYI, I don't have the most experience with writing tests and using GitHub for PRs so please feel free to comment on anything you think I could improve on or should change.
Also, I didn't add any examples as anything using clean_names that works for the single Index DataFrame should also work for the MultiIndex.

Thanks @JoshuaC3! Looking forward to the PR :smile:


bug: whisper processing is not local?

Bug Report
Description
Bug Summary:
If you user whisper and turn of the internet connection it wont work.
Steps to Reproduce:
Turn off internet connection and try voice input with whisper.
Expected Behavior:
I don't know if this is as expected but in the settings its named "Whisper (Local)", so it should be working without internet since you can implement that with whisper.
Actual Behavior:
Does nothing, if you turn internet on and wait a few seconds the words you saind during offline time appear.
Environment

Operating System: [Windows 11]
Browser (if applicable): [Chrome]

Confirmation:

[x ] I have read and followed all the instructions provided in the README.md.
[x ] I have reviewed the troubleshooting.md document.

The local Whisper mode does require internet at least for the first attempt to use it because it does not download the model until then. Try again now that you have and report back please.

As @justinh-rahb mentioned, the bug you describe is the expected behaviour. Feel free to make a PR if you manage to find a way to embed the whisper model to the container image, Thanks!


App crashes after trying to access wallet or farm

See video https://www.youtube.com/watch?v=9z4-wsZUpYE

Android 8.1.0 – ABS 154
& it’s a BBB100-2, a Blackberry Keyone


AFE 1.1 Feedback menu

I guess text is very minimalistic. Instead of the link on MTC Excel discussion I'd give the link on related to AFE blog on MTC. Posts in common feed disapper in few hours, tag doesn't help a much. Plus add proper links on github and LinkedIn. Ideally it is better to submit feedback directly from AFE.

Good suggestions. The first ones are small fixes. We're looking into direct feedback but nothing certain yet.


Resource_name argument (confluent_kafka_acl resource) - have a list of strings instead of required string

Hi @linouk23!
It would be very helpful if instead of having resource_name as required string, to have a list of strings.
I am trying to pass a list of topics inside of it.
Could you please let me know if there is any plan to add this feature?
Many thanks in advance!

Thanks for creating an issue @alexandra-buica!
This question looks similar to https://github.com/confluentinc/terraform-provider-confluent/issues/9 and I'm wondering if you find this comment from @nick-osborn helpful.

Hello @linouk23,
Thank for the information that you have provided, that comment is indeed helpful but I think it will be much easier to have that list(string) for that field.

@mihneastaub thanks for the feedback! Could you also react to the first message so we could track the number of users interested in this feature?
I think once we confirm there's an interest, we might consider implementing it.


feat: add Buy Me Coffee

We should Add Buy me Coffee Button so that project can get funds

My self Harsh Upadhye , I have participated in GSOC 23 last year, I have good experience of working on open source projects. Please Assign this to me

@Harshu467 where are you planning to add this button? Frontpage is already cluttered

@ashutosh-rath02 Well Buy Me Coffee Button will be auto button generated and it will be fix at bottom right of screen , I can share sample button how it can look like

These are samples how button can look we can select color any whichever we want

Cool go ahead

@Harshu467 .vasudev You can reach out to me on discord for any creds you need.

@ashutosh-rath02 Can you share your discord link ?

@ashutosh-rath02 I have dm u via twitter share me script that I will need

@ashutosh-rath02 Can you share your discord link ?

Discord username .vasudev

@ashutosh-rath02 Can you share your discord link ?

Discord username .vasudev

I have added request check out, if could share profile link it will be better

@ashutosh-rath02 Can you share your discord link ?

Discord username .vasudev

I have added request check out, if could share profile link it will be better

Didnt receive anything neither on discord nor on twitter.
https://discordapp.com/users/519818032973873152

Any updates @Harshu467 ?

@Harshu467 updates?

Removing @Harshu467 due to inactivity


Use ESLint instead of TSLint

TSLint is officially deprecated and the way to go is ESLint.
https://github.com/palantir/tslint/issues/4534

@sgiradoa-psl v3 branch is using eslint. Still WIP so don't use


[UI] K3s clusters show etcd specifics labels when not applicable

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

import k3s cluster
go to cluster view page

Result:
shows "Etcd" in green box to indicate it is healthy
Desired:
don't show it for k3s clusters

It looks like if no component matching the label is found (in this case "etcd") we display it as healthy.
Support suggested we just make the label more general.

@westlywright Not sure how easy this is to update this. Let me know if there is a way to make this grey or disabled.

@deniseschannon I am just going to hide the status box and extend the other boxes to fill the space if we don't have an etcd status and its a k3s imported cluster. Are you okay with that?

@westlywright that's fine, but it's weird why we don't do something like that for AKS, which has 2 of the 4 boxes that aren't valid.

@deniseschannon Do we want to add it while I am in here? Is there good reason to show them as greyed out rather then hide them all together?

I have no good reason to grey them out (aka disable them) if it's not valid. I'd only worry about when you switch between clusters and how that looks. Someone might ask, why those don't show up, but maybe not.

This is reproduced with Rancher server rancher\rancher:v2.4.2
Steps:
Import a k3s cluster
View cluster
Result:

The feature is verified with Rancher server rancher\rancher:master-head version 03947c351 and rancher\rancher:v2.4-head version 03947c351
Etcd is greyed out:


add podSecurityPolicy to the post-delete-hook

issue: https://github.com/rancher/rancher/issues/32878
Problem:
The post-delete-hook pod fails to be scheduled because it does not have and can not use any psp.
Solution:
Add the missing permission and psp resource.
Note:
Rancher itself can be installed and uninstalled successfully because it uses the cluster-admin clusterrole.
Next Step:

[ ] Backport to the v2.5 branch.

The PR is already approved by 3 reviewers.
I just rebase the branch, hopefully, the build will pass this time.


API update script should be using github instead of gitlab

Feature or problem description
Challenge is that repo is private so standard way to get raw content will not work.
Instead of that we can use git export feature.
wget --no-check-certificate https://raw.githubusercontent.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/main/openapi/kas-fleet-manager.yaml

Soon to be obsolete by the SDK.


Patch release 5.0.4

Prerequisites

[x] DEV/QA: Define client versions and provide list of breaking changes for desktop/mobile team
[x] DEV/QA: Check new strings and align with clients
[x] DEV/DOCS: Create list of pending docs tasks
[x] DEV: Create branch release-x.x.x-rc.x

[x] DEV: bump ocis version in necessary files
[ ] DEV: changelog/CHANGELOG.tmpl
[x] DEV: ocis-pkg/version/version.go
[x] DEV: sonar-project.properties
[x] DEV: prepare changelog folder in changelog/x.x.x_????_??_??

[x] DEV: Check successful CI run on release branch
[x] DEV: Create signed tag [v5.0.4-rc.1](https://github.com/owncloud/ocis/releases/tag/v5.0.4-rc.1)
[x] DEV: Check successful CI run on v5.0.4-rc.1 tag / BLOCKING for all further activity
[ ] DEV: Merge back release branch
[ ] DEV: bump released deployments to vx.y.z-rc.x
[ ] DEV: update ocis.released.owncloud.works
[ ] DEV: https://ocis.team.owncloud.works/

[ ] DEV: needs snapshot and migration

QA Phase

[x] QA: Confirmatory testing (if needed)

[x] QA: Fix nats reconnects

[x] QA: Compatibility testing between 5.0.3 and 5.0.4

[x] QA: Performance test  see result here https://grafana.k6.infra.owncloud.works/?orgId=1&refresh=15m&var-test_id=69434ec2-b51f-42a7-91d6-ef9db204c00c

[x] QA: e2e with different deployment:

[x] QA: wopi against smoke test
failed test smoke/upload.feature:18 - due https://github.com/owncloud/web/issues/10810
failed test smoke/urlJourneys.feature:4

[ ] QA: Smoke test on Web Office (Collabora, Onlyoffice, Microsoft office)

[ ] QA: Collecting errors found

After QA Phase

[ ] Brief company-wide heads up via mail @tbsbdr
[ ] Create list of changed ENV vars and send to<EMAIL_ADDRESS>
[ ] Variable Name
[ ] Introduced in version
[ ] Default Value
[ ] Description
[ ] dependencies with user other components

[ ] DEV: Create branch release-x.x.x

[ ] DEV: bump ocis version in necessary files
[ ] DEV: ocis-pkg/version/version.go
[ ] DEV: sonar-project.properties
[ ] DEV: released deployment versions
[ ] DEV: prepare changelog folder in changelog/x.x.x_???

[ ] Release Notes + Breaking Changes @tbsbdr
[ ] Migration + Breaking Changes Admin Doc @mmattel
[ ] DEV: Create final signed tag
[ ] DEV: Check successful CI run on vx.y.z tag / BLOCKING for all further activity
[ ] Merge release notes

Post-release communication

[ ] DEV: Create a docs-stable-x.y branch based on the docs folder in the ocis repo @micbar
[ ] DEV/QA: Ping documentation in RC about the new release tag (for ocis/helm chart version bump in docs)
[ ] DEV/QA: Ping marketing to update all download links (download mirrors are updated at the full hour, wait with ping until download is actually available)
[ ] DEV/QA: Ping @hodyroff once the demo instances are running this release
[ ] DEV/QA: notify @michaelstingl @hosy @fmoc @jesmrec to publish client finals
[ ] DEV: Merge back release branch
[ ] DEV: Create stable-x.y branch in the ocis repo from final tag

Final 5.0.4  is published


WIP: feat(collection): updating find API

Find api is now updated and clarified. You can no longer pass in
fields, and you cannot pass in individual options as parameters.
BREAKING CHANGE:
find and findOne no longer support the fields parameter.
You can achieve the same results as the fields parameter by
using Cursor.prototype.project. Additionally, find does not
support individual options like skip and limit as positional
parameters. You must pass in these parameters in the options object
Work in Progress for the following reasons:

We may still need to support projection for findOne
We may want to document the options object for find more clearly

Note that I also added a commit that just fixes lint errors. Feel free to ignore that commit in the review

I stopped my review short because I want to focus on the idea of supporting project in the options for find.  Lets make that choice first, and then work from there.

I think supporting project in options could be good too. Is there anything comparable in another driver?

We may want to document the options object for find more clearly

I think we should also add code samples as well 👍


PatchCore  inference

when inference, patchcore model need "memory_bank" to compute distance ,which is used in models/patchcore/torch_model.py, line 158:
distances = torch.cdist(embedding, self.memory_bank, p=2.0)
I think I should save it when train,and use it when inference...but how i save it and how to use it?

Hi!,I just got the same issue,
could you please share how you solved this?

Hi, memory_bank is part of the TorchModel here.
When the model weights are loaded, the memory bank is loaded as well.


lib: prevent usage of unknown schema type

I don't think that actually using undefined types should behave like using undefined types :)

Thanks! A modified version landed in a4322c9c18b00d2ef5cf54d9557fcc6d3347a873.


tests: add case test_add_node_check_quorum

contribute to https://github.com/pingcap/raft-rs/issues/23
add test test_add_node_check_quorum
i use rustfmt 0.3.6(specified in the travis.yml) to format all the code, it seems the current code mostly was formatted by 0.2.* like the previous code in tikv,  and some code use other rustfmt version in the recent pr, maybe we must make sure which version to use and check it in CI.

Thank you! But please don't do the format yet, it's hard to review and distracting. We will format all the code later.

no format now

LGTM

@july2993 Thanks for your contribution! 🍺


Change ILoadBalancingPolicy to be async

What should we add or change to make your life better?
In LoadBalancingMiddleware there is a call to PickDestination method from ILoadBalancingPolicy.
https://github.com/microsoft/reverse-proxy/blob/52b31103f6a25f881282d7ceb5f285ca9f09d26e/src/ReverseProxy/LoadBalancing/LoadBalancingMiddleware.cs#L53
can you change this to be async method?
Why is this important to you?
we have some calls there to external API to decide the destination according to tenant geo region and we think using sync method might cause scaling\resources issues later

Making an external API call to load balance every request will also be a significant scaling limiter.
Possible alternatives:

Caching and syncing in the background
Custom middleware. The LoadBalancingMiddleware is there for convenience, it doesn't provide any unique functionality.
https://github.com/microsoft/reverse-proxy/blob/a6b19a913dd5f6c0d6b68e923d0f36baa6e2f037/samples/ReverseProxy.Code.Sample/Startup.cs#L100-L119

Triage: There are workarounds.
We do not believe it is a good idea to do it directly in YARP, because it will create too much pressure on pipeline and would create a bottleneck.
Given that it would be a breaking change, we are unlikely to take it after 1.0.

Thank you @Tratcher - your suggestion solved my issue


error catching for tf deletion, added missing metadata property

When we click "rerun" we're deleting the current taskflow, in once case we were not sending an empty metadata object which would clear the old taskflowId. One take away is that we should try to minimize the amount of redundant code between workflows we have.
Fixed how the simulation object updates. Added some get()'s to some toaster conditionals which could throw errors themselves.

@chetnieter @cjh1 PTAL

Current coverage is 61.18% (diff: 75.00%)

Merging #529 into release will decrease coverage by <.01%

@@            release       #529   diff @@
==========================================
  Files            59         59
  Lines          2940       2945     +5
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1799       1802     +3
- Misses         1141       1143     +2
  Partials          0          0

Powered by Codecov. Last update 7d2ce8d...1a7628c

:+1: For reducing duplicate code between workflows in the client, this problem is only going to get worse as more workflows are added. Is there some refactoring we can do to reduce the duplication?

@TristanWright I can rerun the viz, however, its not passing the data-dir or file-load option the second time around, did it for you?

👍 For reducing duplicate code between workflows in the client, this problem is only going to get worse as more workflows are added. Is there some refactoring we can do to reduce the duplication?

There used to be (still is?) an extends: [] property you could have in React components which you could use to inherit and extend from other components. I don't know it's status in React >15.0 though I plan to look into it.

@TristanWright I can rerun the viz, however, its not passing the data-dir or file-load option the second time around, did it for you?

I reran about a dozen times, thought I saw it a few times. I'll track it down now.

I was clearing the metadata which had the dataDir and file in it. I reproduced the issue you had and looks fixed now.

I just tested this as well and I now see the data loaded on rerun.


How to enable embedding policy?

Rasa Core version: 0.13.2
Issue:
I have explored embedding policy in Rasa Core library from the documentation. But I didn't find any details  for enabling the embedding policy in the docs.
Can any one help me how to enable the embedding policy?
Content of domain file (if used & relevant):

what do you mean by enable? you have to pass appropriate policies list, e.g. policies = [EmbeddingPolicy()] to your Agent

Yes so as @Ghostvv says you can't enable this via the train script, but you can load an agent like this here: https://rasa.com/docs/core/policies/#id1 and specify your embedding policy as stated later on that page

thanks @akelad
can you explain more for the embedding policy working?
https://rasa.com/docs/core/policies/#embedding-policy here its written that : it takes previous user inputs as a user memory and Embedding of previous system actions as a system memory.
1.Then from where did the it taking the previous user inputs?
2. How the Embedding policy works better then Keras policy?
3. How can I differentiate the Embedding and Keras policy output?

We'll be publishing a paper soon that explains all the details of this new policy. The main benefit of the embedding policy is that it learns to generalise on conversations better. I'll close this issue for now as this is more of a discussion topic. If you have more questions please post them on the forum


Bug: NullReferenceException when used from different thread

Bug Description
I have a singleton class that holds a reference to another class (a variable called _service).  In the singleton class, I have a System.Threading.Timer that runs and uses the reference to _service to call a method on _service, that looks like:
public async Task SomeMethod()
{
   using var connection = new Microsoft.Data.SqlClient.SqlConnection(dbConnString);
   await connection.OpenAsync();

   var requests = await connection.QueryAsync<DataRecord>(x => x.CompletedTime == null && x.ReceiveTime < DateTime.Now.AddHours(-1.5));
}

The timer callback method:
private void TimerTick(object state)
{
    _service.SomeMethod().Wait();
}

From somewhere else in the same singleton class, I also call:
await _service.SomeMethod()

which works fine.  But, when the timer callback calls _service.SomeMethod(), the line: connection.QueryAsync() throws a NullReferenceException.  When the exception happens, I verified that RepoDb.SqlServerBootstrap.IsInitialized is true.
Exception Message:
NullReferenceException
TargetSite: Void b__0(RepoDb.QueryGroup)}	System.Reflection.MethodBase {System.Reflection.RuntimeMethodInfo}
Stacktrace:
at RepoDb.QueryGroup.<>c__DisplayClass59_0.b__0(QueryGroup queryGroup)
at RepoDb.QueryGroup.<>c__DisplayClass59_0.b__0(QueryGroup queryGroup)
at RepoDb.QueryGroup.<>c__DisplayClass59_0.b__0(QueryGroup queryGroup)
at RepoDb.QueryGroup.GetFields(Boolean traverse)
at RepoDb.QueryGroup.Fix()
at RepoDb.QueryGroup.Parse[TEntity](Expression1 expression)    at RepoDb.DbConnectionExtension.ToQueryGroup[TEntity](Expression1 where)
at RepoDb.DbConnectionExtension.QueryAsync[TEntity](IDbConnection connection, Expression1 where, IEnumerable1 fields, IEnumerable1 orderBy, Nullable1 top, String hints, String cacheKey, Nullable1 cacheItemExpiration, Nullable1 commandTimeout, IDbTransaction transaction, ICache cache, ITrace trace, IStatementBuilder statementBuilder, CancellationToken cancellationToken)
Model:
[Map("MyTable")]
public class DataRecord
{
    public int Id { get; set; }
    public string Client { get; set; }
    public DateTime ReceiveTime { get; set; }
    public DateTime? CompletedTime { get; set; }
}

Library Version:
RepoDb v1.12.9 RepoDb.SqlServer v1.1.4
Microsoft.Data.SqlClient v3.0.1

We will look on this issue and if verified, will be a part of the next release. Do you have any timeline for this issue to get fixed? Thank you for reporting this issue.

Thanks for looking into it. I've committed to providing this functionality to my customer, but I can hold off for a brief/little while.

Hi @Dean-NC , we could not replicate this issue. Here we attached a small project for you to play and replicate this based on your scenario.
Project: Issue948.zip
Would you be able to update this and revert to us?
Note: We tested on the versions you mentioned and also to the latest version we had as of writing this (RepoDb v1.12.10-beta4, RepoDb.SqlServer v1.1.5-beta4, RepoDb.SqlServer.BulkOperations 1.1.6-beta2). We also tested on the latest main branch and everything works fine.

@mikependon Thanks for the update. I've tested my code with the latest RepoDB and also updated Microsoft.Data.SqlClient to 4.0, and the error is no longer happening. It might have been a fix in Microsoft SqlClient.

@mikependon Thanks for the update. I've tested my code with the latest RepoDB and also updated Microsoft.Data.SqlClient to 4.0, and the error is no longer happening. It might have been a fix in Microsoft SqlClient.


add templates for redis

This should probably be merged after #287 and bump the versions in the templates.

Are we okay with expecting a user to implement the class and export an object? If not, we could potentially add a spin.ts as we do for the http templates and wrap all that up where the user can just export one function.

Given they're technically part of the same SDK, I'd like the two to be consistent.

One part I would like to note though is that - the http handler uses the built in fetchEvent while redis and anything else would end up using generated bindings.


Chrome: Audio player fades out on tab blur

Using: OSX Chrome Version 46.0.2490.71 (64-bit)
When I use the keyboard to tab to the audio control buttons, when I tab out of the player, the player fades out. This has something to do with the "hideControls" function. This is reproducible on the mediaelementjs.com website homepage audio example.
This is not an issue in Firefox, maybe because is does not support focusout?
I have temporarily resolved this issue (i think) by modifying line 2842 adding a conditional for the focusout:
t.container.focusout(function (e) {
if( e.relatedTarget ) {
var $target = $(e.relatedTarget);
if (t.keyboardAction && $target.parents('.mejs-container').length === 0) {
t.keyboardAction = false;
// added this if statement
if (t.isVideo) {
t.hideControls(true);
}
// end added statement
}
}
});

Have you made a pull request for this? I've having the same issue.

@superuntitled Closing issue since it has been merged


Image preview pane zoom

📝 Provide a description of the new feature
mouse wheel scroll for zoom image

If you'd like to see this feature implemented, add a 👍 reaction to this post.

could be an extension to #5147

You can zoom into an image (https://github.com/martinchrzan/FileExplorerGallery#zoom) if #5147 becomes a thing in PowerToys

Yeah, going to close this as #5147.  But this lacks a lot of the needed details as well as what the actual problem here is.  This is just a 'i want this'


Why you choose to use fbank instead of spectrogram?

Why you choose to use fbank instead of spectrogram?  Thank you!

Hi there,
I don't have a strong reason to choose one over another. But I felt that fbank is more standard for audio application as it provides some flexibility on the choice of filters (in this repo, mel filters). Having said that, I haven't formally studied if mel filters are better for general audio classification. I would expect the normalization stats for spectrogram would be different with fbanks, so if you want to choose to use spectrogram, please double check the normalization mean and std (especially for ImageNet pretrained models).
-Yuan


Small Variation issue

Hey again,
thx for ur great work !
I've noticed that if you variate a generation twice, seems like the code cannot make the difference between the first variation and the second one, so it takes the first one instead of waiting for the second one.

Maybe the code could check the first date/hour of the last message that look like the right anwser before choosing it as the right msg.
The rule could work cause it is not possible to start receiving an answer before the request has been sent ^^

Hey again,
thx for ur great work !
I've noticed that if you variate a generation twice, seems like the code cannot make the difference between the first variation and the second one, so it takes the first one instead of waiting for the second one.

please show the code.


Send message to user| Permanently identify user

UPDATE:
So i've been trying different things out and it appears, that i had to fill the From-Property in the message.
The solution is like this:
var conClient = new ConnectorClient();
Message message2 = new Message();
message2.From = new ChannelAccount { ChannelId = message.From.ChannelId, Address = botAddress};
message2.To = new ChannelAccount { ChannelId = message.From.ChannelId, Address = userAddress};
message2.Text = "Hello other user!";
message2.Language = "en";
await conClient.Messages.SendMessageAsync(message2);

Now i'd like to use this thread here to discuss another question, that i had for quite some time:
If i want to --permanently-- identify a user/bot, what should i use?
I am guessing that a combination of "ChannelID" and "Address" is an unique identifier of a user for a channel.
But what about the "id" ? I dont really understand the reason for that ID. What value does it have for me? Because it seems like the id is also a unique identifier for a user for just one channel.
What reasons are there to store the "id"?
Is it possible, to get all ChannelID-Address combinations of a user with the id? (if the id is globally unique)
Greetings!
--------------------------------------------Initial Question------------------------------------------------------------
Hello!
What i want to do is to message a user with my bot, only knowing the channelID/UserID/UserAddress.
Because i need to handle a message from my backend, which tells me something like "message that user", so i have no conversation info or anything.
So is it possible to just message a user?
If so, how does it work?
Im using the C# Builder.
Im currently testing it with skype.
What ive tried:
var conClient = new ConnectorClient();
var msg = new Message() { To = new ChannelAccount { Id = "hisID", ChannelId = "skype", Address = "his address", IsBot = false }, Text = $"Hello!" };
await conClient.Messages.SendMessageAsync(msg);

That doesnt work for some reason.
Am i missing a required message property? Thats what im thinking.
Greetings!

Hi there NanashiGit,
The ID currently represents the Bot Framework ID for this user. In upcoming releases we'll be simplifying the ChannelAccount data to just the channel service's user Id and Name; with ChannelId being at the message level.

Jim is correct.  The idea of id vs address was that id would be a global user id across addresses.  In our new version we have removed it, so channelId/Address is the unique id for a user on a channel

can we send messages to user just by knowing their email address now?


Hello!

Hi, I have some question that if i want to use DQN and DDQN to train the agent in the high_env, which json file should i choose ? I notice that there is a file named "no_dueling.json" in the /scripts/configs/HighwayEnv/agents/DQNAgent, but in the model.py there is not Unknown model type to match with the "no_dueling.json", what should i do ? thanks for you help!

The DuelingNetwork architecture is defined in https://github.com/eleurent/rl-agents/blob/master/rl_agents/agents/common/models.py
It is selected (or not) based on the model field of the configuration dictionary here (DQN agent):  https://github.com/eleurent/rl-agents/blob/master/rl_agents/agents/deep_q_network/pytorch.py#L17 and there (model factory): https://github.com/eleurent/rl-agents/blob/master/rl_agents/agents/common/models.py#L415
By default it is enabled in the baseline config https://github.com/eleurent/rl-agents/blob/master/scripts/configs/HighwayEnv/agents/DQNAgent/baseline.json#L4, and disabled in the do-dueling config https://github.com/eleurent/rl-agents/blob/master/scripts/configs/HighwayEnv/agents/DQNAgent/no_dueling.json#L4
Be careful: the other fields are also different (gamma, batch size, etc.), if you wish to only compare the architecture you should choose them equal.

Thanks, but i see in the no_dueling file, the type is the "FCNetwork", and in the model.py there is not "type" model match the "FCNetwork", how can i do ?

The DuelingNetwork architecture is defined in https://github.com/eleurent/rl-agents/blob/master/rl_agents/agents/common/models.py
It is selected (or not) based on the model field of the configuration dictionary here (DQN agent): https://github.com/eleurent/rl-agents/blob/master/rl_agents/agents/deep_q_network/pytorch.py#L17 and there (model factory): https://github.com/eleurent/rl-agents/blob/master/rl_agents/agents/common/models.py#L415
By default it is enabled in the baseline config https://github.com/eleurent/rl-agents/blob/master/scripts/configs/HighwayEnv/agents/DQNAgent/baseline.json#L4, and disabled in the do-dueling config https://github.com/eleurent/rl-agents/blob/master/scripts/configs/HighwayEnv/agents/DQNAgent/no_dueling.json#L4
Be careful: the other fields are also different (gamma, batch size, etc.), if you wish to only compare the architecture you should choose them equal.

Another question is that in the highway_env/envs/common/observation.py, the feature of the class KinematicObservation is ['presence', 'x', 'y', 'vx', 'vy'], i understand that the presence mean whether the vehicle exist, exist is "1", otherwise "0". the "x" and "y" means the relative distance between Longitudinal and Horizontal, the "vx" and "vy" means the relative velocity between Longitudinal and Horizontal, is this right ? thanks a lot !

That's right, don't forget that x,y,vx,vy are also normalized using the config feature_range. You can also set the configuration "absolute":true, in which case the fields x,y,vx,vy are absolute positions and velocity instead of relative.

That's right, don't forget that x,y,vx,vy are also normalized using the config feature_range. You can also set the configuration "absolute":true, in which case the fields x,y,vx,vy are absolute positions and velocity instead of relative.

That's right, don't forget that x,y,vx,vy are also normalized using the config feature_range. You can also set the configuration "absolute":true, in which case the fields x,y,vx,vy are absolute positions and velocity instead of relative.

Thanks, but i see in the no_dueling file, the type is the "FCNetwork", and in the model.py there is not "type" model match the "FCNetwork", how can i do ?

That's right, don't forget that x,y,vx,vy are also normalized using the config feature_range. You can also set the configuration "absolute":true, in which case the fields x,y,vx,vy are absolute positions and velocity instead of relative.

Thanks, but i see in the no_dueling file, the type is the "FCNetwork", and in the model.py there is not "type" model match the "FCNetwork", how can i do ?

Ah! I see, that's because I renamed the model to MultiLayerPerceptron without changing the config, sorry about that.

Ah! I see, that's because I renamed the model to MultiLayerPerceptron without changing the config, sorry about that.

thanks for your help !

My question is what the meaning of the file named no_dueling

In this file, a Multi-Layer Perceptron model is chosen instead of the Dueling architecture from DDQN (by default in baseline.json).

what the meaning of the "tau", "temperature" and "final_temperature"?

They are used to schedule the epsilon-greedy exploration: epsilon starts at "temperature" and converges to "final_temperature" with an exponential decay of time constant "tau" (unit: number of steps).

My question is what the meaning of the file named no_dueling

In this file, a Multi-Layer Perceptron model is chosen instead of the Dueling architecture from DDQN (by default in baseline.json).

what the meaning of the "tau", "temperature" and "final_temperature"?

They are used to schedule the epsilon-greedy exploration: epsilon starts at "temperature" and converges to "final_temperature" with an exponential decay of time constant "tau" (unit: number of steps).

If i want to use DDQN to train, i need to modify the model.py,is this right?

No, if you want to use DDQN you just have to run an experiment with the baseline.json config, which already selects the Dueling architecture in its "model" field.

Thanks a lot！So if I use the Dueling DQN to train，can i use the baseline.json? Can i both run the baseline.json to use the DDQN and Dueling DQN to train the agent？
Another question is that i caculate the success rate in 10000 episode，the vehicle count is 20，the duration is 20, the environment is env-medium,the episode which is success is about 4000，i put the gamma equal to 0.95，and other parameter keeps unchangeable in baseline.json，i define the success is the ego vehicle which is not crash in a episode，is that right？And the average reward during the train is rising consistently. I guess if i train the episode exceed the 10000 times，the success will better，is this right？Thanks for you help！！！

Yes, baseline.json has both Dueling and Double DQN (Double DQN is always activated and not configurable).
I'm not sure I understood your second question.

Yes, baseline.json has both Dueling and Double DQN (Double DQN is always activated and not configurable).
I'm not sure I understood your second question.

Thanks，so my question is if i want to use the DDQN to train, what command should i run? I notice that in the baseline.json, the "type" is "DuelingNetwork",and  what the different of the command if i want to use the DQN and Dueling DQN to train?
I'm very sorry, I want to explain the second question above: I use the Dueling DQN to train the agent in the enviromment which named "env_medium", I changed the  "vehicles_count" and  "duration" both equal to 20，and i want to calculate the ”success rate“ in the training，I trained a total of 10,000 episode，and i define the success episode is that ”the ego vehicle did not collide during a episode“ and in the 10000 episode the success episode is about 4000, is this result correct? Will this success rate increase if I train more episode? Thanks for you help！

Yes, baseline.json has both Dueling and Double DQN (Double DQN is always activated and not configurable).
I'm not sure I understood your second question.

Hi, there are some detail in the experiment which is described above: The success rate is keep about 40 percent，and the terminal tell me "[root:WARNING] NaN or Inf found in input tensor.",and the success rate and average reward begin decrease, How can i solve the problem? Are there any method to increase the success rate? Thanks ! !

i define the success episode is that ”the ego vehicle did not collide during a episode“ and in the 10000 episode the success episode is about 4000, is this result correct?

I don't know, you tell me ;) I haven't run this experiment specifically.

Will this success rate increase if I train more episode?

Maybe, maybe not. Make sure to check the schedule of exploration (random actions vs estimated optimal action): if you agent stops exploring too early (i think the default configuration is for about 1k episodes of exploration), it will be stuck in its current suboptimal behaviour.

terminal tell me "[root:WARNING] NaN or Inf found in input tensor."

That is very strange, I haven't run into this issue. It would be great if you could reproduce and investigate this bug. Apparently, the observation generated by the environment contains NaN or Inf. What type of observation are you using?

That is very strange, I haven't run into this issue. It would be great if you could reproduce and investigate this bug. Apparently, the observation generated by the environment contains NaN or Inf. What type of observation are you using?

   Hi，i use the “KinematicObservation” observation. The terminal tell me "[root:WARNING] NaN or Inf found in input tensor." i think maybe the reason is that I change the reward function, i have a question about the reward function, if the variance of each reward function is too large，it will be occur the "[root:WARNING] NaN or Inf found in input tensor." i put the reward function such as if the ego vehicle crash, the reward will given -10, and if ego vehicle lane change, i will given a reward equal to -0.1, and the velocity reward is 3, and i put distance reward is -(100 - distance)*0.03 ,which the distance represent the ego vehicle to the front vehicle  which in the same lane. And I change  the normalized form that the reward of every steps is
                                            "reward = (reward - average-reward)/std(reward)",

and i noticed that every episode of the reward have a big change , maybe -20 to 30 or 40 ,I think this is bad for model convergence, I think it the method that to normalize in to [0~1] is bettter.
I notice that if i change the duration time, and if i train the agent, the terminal also tell me the " [root:WARNING] NaN or Inf found in input tensor" sometimes. I will continue to study this issue in the future !
And another question is that if i want to compared with the different model which named "DDQN" and "Dueling DQN", what command should i run in turn ?
I'm very sorry to bother you again !

i define the success episode is that ”the ego vehicle did not collide during a episode“ and in the 10000 episode the success episode is about 4000, is this result correct?

I don't know, you tell me ;) I haven't run this experiment specifically.

Will this success rate increase if I train more episode?

Maybe, maybe not. Make sure to check the schedule of exploration (random actions vs estimated optimal action): if you agent stops exploring too early (i think the default configuration is for about 1k episodes of exploration), it will be stuck in its current suboptimal behaviour.

terminal tell me "[root:WARNING] NaN or Inf found in input tensor."

That is very strange, I haven't run into this issue. It would be great if you could reproduce and investigate this bug. Apparently, the observation generated by the environment contains NaN or Inf. What type of observation are you using?

Hi，i use the “KinematicObservation” observation. The terminal tell me "[root:WARNING] NaN or Inf found in input tensor." i think maybe the reason is that I change the reward function, i have a question about the reward function, if the variance of each reward function is too large，it will be occur the "[root:WARNING] NaN or Inf found in input tensor." i put the reward function such as if the ego vehicle crash, the reward will given -10, and if ego vehicle lane change, i will given a reward equal to -0.1, and the velocity reward is 3, and i put distance reward is -(100 - distance)*0.03 ,which the distance represent the ego vehicle to the front vehicle  which in the same lane. And I change  the normalized form that the reward of every steps is
reward = (reward - average-reward)/std(reward)
and i noticed that every episode of the reward have a big change , maybe -20 to 30 or 40 ,I think this is bad for model convergence, I think it the method that to normalize in to [0~1] is bettter.
I notice that if i change the duration time, and if i train the agent, the terminal also tell me the " [root:WARNING] NaN or Inf found in input tensor" sometimes. I will continue to study this issue in the future !
And another question is that if i want to compared with the different model which named "DDQN" and "Dueling DQN", what command should i run in turn ?
I'm very sorry to bother you again !

This NaN of Inf in the input tensor might come from your reward = ((reward - average(reward))/std(reward) formula when the std is either zero or Nan?
I cannot reproduce this issue myself, so you should use a debugger and add a breakpoint (or a print statement) whenever this exception is raised in order to check the content of the observation and reward tensors.

This NaN of Inf in the input tensor might come from your reward = ((reward - average(reward))/std(reward) formula when the std is either zero or Nan?
I cannot reproduce this issue myself, so you should use a debugger and add a breakpoint (or a print statement) whenever this exception is raised in order to check the content of the observation and reward tensors.

Hi, I'm very sorry to bother you again, the problem that i find maybe the distance reward function cause the total reward Unstable. Another question is that if i want to compared with the different model which named "DDQN" and "Dueling DQN", what command should i run if i want to train the agent which use the "DDQN"  and "Dueling DQN" ? Thanks !

For Dueling Double DQN:
python3 experiments.py evaluate configs/HighwayEnv/env.json configs/HighwayEnv/agents/DQNAgent/baseline.json --train --episodes=1000
For Double DQN:
python3 experiments.py evaluate configs/HighwayEnv/env.json configs/HighwayEnv/agents/DQNAgent/no_dueling.json --train --episodes=1000

But beware that this is not a fair comparison since other parameters are changed in the no_dueling.json config. In order to compare the two algorithms, you should copy every parameter from baseline.json to no_dueling.json except for the "model" parameter.

* For Dueling Double DQN:
  `python3 experiments.py evaluate configs/HighwayEnv/env.json configs/HighwayEnv/agents/DQNAgent/baseline.json --train --episodes=1000`

* For Double DQN:
  `python3 experiments.py evaluate configs/HighwayEnv/env.json configs/HighwayEnv/agents/DQNAgent/no_dueling.json --train --episodes=1000`

But beware that this is not a fair comparison since other parameters are changed in the no_dueling.json config. In order to compare the two algorithms, you should copy every parameter from baseline.json to no_dueling.json except for the "model" parameter.

Thanks ! If i only use DQN to train, can i use the no_dueling.json to train? thanks!!

* For Dueling Double DQN:
  `python3 experiments.py evaluate configs/HighwayEnv/env.json configs/HighwayEnv/agents/DQNAgent/baseline.json --train --episodes=1000`

* For Double DQN:
  `python3 experiments.py evaluate configs/HighwayEnv/env.json configs/HighwayEnv/agents/DQNAgent/no_dueling.json --train --episodes=1000`

But beware that this is not a fair comparison since other parameters are changed in the no_dueling.json config. In order to compare the two algorithms, you should copy every parameter from baseline.json to no_dueling.json except for the "model" parameter.

Hi,i see in the every metadataXXX.json, there is a "show_trajectories" config, how can i change the config ? Thanks ! !

@zhangxinchen123 sorry for the delay, I lost track of this issue.
the no_dueling config does not exist anymore, if you want only dqn you should now use the dqn.json config.
If you want to test the model you just trained, you should run the same command and replace the --train option by --test. You should see a message saying that the latest saved model has been loaded.

@zhangxinchen123 sorry for the delay, I lost track of this issue.
the no_dueling config does not exist anymore, if you want only dqn you should now use the dqn.json config.
If you want to test the model you just trained, you should run the same command and replace the --train option by --test. You should see a message saying that the latest saved model has been loaded.

Thanks for you reply! ! And I have a question is about that the lateral and longitudinal trajectory planning of the ego vehicle when it lane change, does this follow any rules? Thank you! ! !

It the ego vehicle's lateral position and longitudinal velocity are controlled by state feedback controllers implemented in steering_control() and velocity_control().
The resulting trajectory is obtained by integrating the kinematics model with these control laws. A lane change trajectory simply corresponds to a change of setpoint for the lateral position.

It the ego vehicle's lateral position and longitudinal velocity are controlled by state feedback controllers implemented in steering_control() and velocity_control().
The resulting trajectory is obtained by integrating the kinematics model with these control laws. A lane change trajectory simply corresponds to a change of setpoint for the lateral position.

Thanks for you reply ! !

I have a question to ask you, the observation value observed under this project of highway is a 5 * 5 matrix, I understand this. But in merge and roundabout are 3 * 10 * 3 matrices, etc., I have a little doubt, I do n’t know if you can answer it. Thank you in advance for your answers

@EnormousAdversity different types of observations are defined here.
The environments use different types as a default configuration, defined in their default_config(self) -> dict methods.
You can override them depending on your needs, by using:
env.configure({
"observation": {
"type": ,
:   # observation parameters
}
})


new azure retrievers

Related Issues

fixes #950

Proposed Changes:

This PR includes the following features:

Implement a BM25 retriever for Azure AI Search
Implement a hybrid search retriever for Azure AI Search

How did you test it?

Manual testing in Haystack pipelines
To do: add unit tests

Notes for the reviewer

Used the existing embedding retriever code as a template and primarily just changed the parameters in the calls to the Azure AI Search Python SDK.
Checklist

I have read the contributors guidelines and the code of conduct
I have updated the related issue with new insights and changes
I added unit tests and updated the docstrings
I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[BEAM-115] Unify Java and Python WindowingStrategy representations.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @tgroh

R: @kennknowles

(btw in commit message s/Stragegy/Strategy/)

We have serialization and round-tripping tests.

What I mean is my understanding is that we do not have any logic to serialize known types (so this new deserialization code is not exercised). I'm happy to be wrong though.

Ah, yes, the serialization logic is only available in Python.

PTAL

Jenkins: retest this please

retest this please

Jenkins: retest this please

Coverage increased (+0.1%) to 70.836% when pulling f5ff7275fd9882dc4d286694dc93df8b1f991a4d on robertwb:runner-windows into 36436768dba354ffbc68df0b3761a447c6814dfb on apache:master.

Connect to conjars.org:80 [conjars.org/<IP_ADDRESS>] failed: Connection timed out
retest this please

retest this please

Retest this please

Jenkins: retest this please (Apache Beam :: SDKs :: Java :: IO :: HBase failed downloading dependencies.)

Coverage increased (+0.01%) to 70.679% when pulling 97e72a07952ffc36e9ee239f63863aaecde0a346 on robertwb:runner-windows into 2df9dbd24f6fd3bcfb14316859f4b38c38d95eca on apache:master.

Jenkins is happy, PTAL.

Jenkins appears sad due to a typo in an attribute name.

Coverage decreased (-0.003%) to 70.678% when pulling 3b23198504ed6807da026d4bf19f23f6bc3574f4 on robertwb:runner-windows into 217f085f2769ac1956fd126c9eca0ef60a8d3d9b on apache:master.


fixes #852 should not be an array with null in it

I'm not sure if this is the intended behaviour, but my take on it as a convenience method it should be able to handle creating the initial commit on a empty repo.

:+1: Thanks @get-trucked


Refactor Event Logging to Provide Detailed Object Information

Current Behavior
Currently, Kubernetes events are logged using a generic format that includes the entire object.
er.Logger.Infof("event has been created %v", ev)
er.Logger.Debugf("event has been updated %v", ev)

Desired Behavior
Refactoring of the event logging mechanism to provide more detailed information about the involved object, including namespace, name, kind, event type, reason, and message.
Reason for Change
The current logging format making it difficult to quickly understand the context and severity of events. By adopting a more structured and informative logging format, we can improve readability, simplify troubleshooting, and enhance our overall monitoring capabilities.
Proposed Solution
Refactor the event logging code to log events in the following format:
er.Logger.Infof("event has been created object: %s/%s kind: %s type: %s reason %s message %s",
	ev.InvolvedObject.Namespace, ev.InvolvedObject.Name, ev.InvolvedObject.Kind, ev.Type, ev.Reason, ev.Message)

Sample Example:
{"severity":"info","time":"2024-04-15T01:30:49Z","message":"event has been created object: kubeslice-avesha/cluster1 kind: Cluster type: Normal reason ClusterCNISubnetUpdateSuccessful message cluster CNI Subnet updated successfully.","namespace":"kubeslice-system","sliceCluster":"cluster1","name":"worker-operator"}

Hello. can I work on this issue? kindly assign it to me.

sure @Geun-Oh. I have assigned this issue to you. Let us know @Geun-Oh if you need any help with this issue.


Background colors broken in CSS generated by Less v3

Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
main
Can you reproduce the issue with our latest release version?
no
Can you reproduce the issue with the latest code from main?
yes
Are you using the demo app or your own custom app?
demo
If custom app, can you reproduce the issue using our demo app?
N/A
What browser and OS are you using?
ChromeOS stable 98
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?

N/A; anything in the demo
What configuration are you using?  What is the output of player.getConfiguration()?

default
What did you do?

Open content in the latest nightly build
What did you expect to happen?
Close button should have a white background
What actually happened?

Close button has no background, CSS debugger shows that the background attribute is invalid: rgba(255 255 3%).

In #3991, I changed the syntax of our colors to a modern rgba syntax.  For example, rgba(255, 255, 255, 0.85) would become rgba(255 255 255 / 85%).  However, less v3 seems not to understand that properly, and performs division on the last two parts, resulting in output of rgba(255 255 3%), which is indeed invalid.
This doesn't affect any release branches, since #3991 hasn't been cherry-picked.


Possible bug in utility_meter

HASSIO on NUC version 0.87.0
By defaut
Yearly energy has unit_of_measurement: kWh
Monthly energy and weekly energy and daily energy have unit_of_measurement: W (not KWh and not even Wh)
Hourly energy has unit_of_measurement: kWh

utility_meter:
  hourly_energy:
    source: sensor.total_energy_spent
    cycle: hourly
  daily_energy:
    source: sensor.total_energy_spent
    cycle: daily
  weekly_energy:
    source: sensor.total_energy_spent
    cycle: weekly
  monthly_energy:
    source: sensor.total_energy_spent
    cycle: monthly
  yearly_energy:
    source: sensor.total_energy_spent
    cycle: yearly

sensor:
  - platform: integration
    source: sensor.efergy_8xxxx  # this sensor is in W and it seems working fine
    name: total_energy_spent
    unit_prefix: k
    round: 2
  - platform: efergy
    app_token: !secret efergy_token
    utc_offset: 60
    monitored_variables:
      - type: instant_readings
      - type: current_values
      - type: budget
      - type: cost
        period: day
        currency: E
      - type: amount
        period: day

In the screenshots (in the community thread) your utility_meters show a reset at different times.
Did you configure all the meters at once ? or dis you add some of them later ?

Yes I noticed that, but I configured all at the same time (I think)

Have a look
https://ibb.co/yVYPm8b
https://ibb.co/MsSjWm5
https://ibb.co/GnP2TD1
https://ibb.co/MDcHTjh
https://ibb.co/vqp3LJS

OK... I copy&paste your configuration into my system replacing sensor.efergy_*** by my own sensor which provides values every 2 seconds.
All is working correctly :(
Can you add debug information ?
logger:
  default: warning
  logs:
    homeassistant.components.utility_meter: debug

Are the values in W being updated, or have they freezed ?

Any updates on this ?


bugfix create new account navigation

Codecov Report
Patch and project coverage have no change.

Comparison is base (23cb9f1) 41.36% compared to head (5e075e2) 41.36%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #316   +/-   ##
=======================================
  Coverage   41.36%   41.36%
=======================================
  Files          14       14
  Lines         996      996
=======================================
  Hits          412      412
  Misses        545      545
  Partials       39       39

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Initial functionality implementation and tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b1edb04). Click here to learn what that means.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master     #2   +/-   ##
=======================================
  Coverage          ?   100%
=======================================
  Files             ?      2
  Lines             ?    790
  Branches          ?      0
=======================================
  Hits              ?    790
  Misses            ?      0
  Partials          ?      0

Impacted Files
Coverage Δ

src/EventCalendar.js
100% <100%> (ø)

test/unit/EventCalendarTest.js
100% <100%> (ø)

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 b1edb04...0c34830. Read the comment docs.


fix: Add containerd to the message type reference

Similar to the other types:

https://github.com/bryantbiggs/containerd/blob/c3694aaf879e7e1b6f17c6e723a1e46cc5e94f01/api/runtime/sandbox/v1/sandbox.proto#L67
https://github.com/bryantbiggs/containerd/blob/c3694aaf879e7e1b6f17c6e723a1e46cc5e94f01/api/runtime/sandbox/v1/sandbox.proto#L89

Without this, we are unable to derive the correct source for the Metric type in https://github.com/containerd/rust-extensions

/ok-to-test

/ok-to-test

merging now


fix(mysql): fix bug of internal key

Change-Id: Id1a0739739d282aa60e1588e696c8a87d22f98ae
What type of PR is this?
Bug Fix
What this PR does / why we need it:
Fix issues of gorm adaptor
Which issue(s) this PR fixes:
None
Special notes for your reviewer:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


add support for cookie-based sessions

Adds support for sessions.  This is based on the way that Rack's cookie-based session middleware is handled.
The secret is a required parameter that can be setup using:
Kemal::SessionCookieHandler::INSTANCE.config { |c| c.secret = "my secret" }
Currently the session only supports {} of String => String.

Maybe add a simple way to add it to the HANDLERS array, like enable_session, also maybe make it into it's own repo as a plugin so we can just add the shard if we want to ?

Have you seen this https://github.com/porras/session

@sdogruyol No, I did not.  Thanks for pointing it out.  I will close this PR.

I'm opening this up since this is a well tested Kemal middleware.

There's also this one which is more abstract to storage engines https://github.com/Thyra/kemal-session

@drujensen these 2 may give you a better idea about how to improve your middleware.

Thanks @sdogruyol.  Both examples are excellent work and well documented.

So are you going to improve this @drujensen ?

@sdogruyol for now, I will close this.  It looks like others are creating handlers separate from Kemal so if I continue to work on this, I will do that instead.