Updated for compliance with Laravel 5.2 core objects.

http://laravel.com/docs/5.2/upgrade: "Some of the core events fired by Laravel now use event objects instead of string event names and dynamic parameters."
Jarno

Thank you very much! I didn't have the time to dig in Laravel 5.2 yet.


Remove reference to old 'Backstory' product name

The 'Backstory' name has been deprecated and should no longer be used.

LGTM. Thanks!


add .github/dependabot.yml for chore management

If I did, I do not remember... :sweat_smile:
Maybe someone else?

If I did, I do not remember... sweat_smile
Maybe someone else?

Ah Sorry, that was for Renovate: https://github.com/SovereignCloudStack/generics
But maybe we can revive the generics repository?


IOBroker stoppt den Emporia Adapter regelmäßig, teilweise mehrmals täglich

Hallo und guten Abend, zunächst einmal möchte ich mich recht herzlich bei dir für den Adapter bedanken. Ich bin erst seit 3 Wochen dabei und habe mir einen Raspi gekauft, den IOBroker installiert und auch den Emporia.  Mein Emporia ist auf dem aktuellen Stand, der Helpdesk hat das gestern bestätigt. Mein Adapter ist auch auf der neusten Version. Dennoch bekomme ich immer wieder Fehlermeldungen, der Adapter stürzt ab. Warum er das tut kann ich nicht nachvollziehen, bei meinem Freund, der sich den Emporia auf meine Empfehlung hin ebenfalls gekauft hat, läuft dieser seit 1 Woche ohne eine Störung durch. Kannst du mal in das Logfile reinsehen? Ggf, wirst du ja daraus schlau und könntest mir einen Tipp geben, was ich versuchen kann. Ich habe keine Firewall oder ähnliches installiert. Der Raspi hängt über LAN direkt am Netz und hat volle Rechte. Danke und Gruß :-)

is it possible to see data from the webportal of emporia. Are the user credentials correct ?
Seems you have a DNS problem. Try to check if the route to api.emporia.org works.
regards Chris

Dear Chris, thanks for your answer. The Emporia app works fine. Also Empora webaccess via browser. No problems, all data there. Also the IOBroker data exchange works perfect. In a nutshell no issue. But from time to time the adapter in IObroker crashes and is stopped. This happens 3-4 x a day. I have a script which automatically restarts the emporia adapter after this crash. Therefore, I cannot understand the problem, everything works fine 99,5 % of the day. Concerning this background, can you explain what you mean by check the route to emporia.org? Thanks and kind regards!

Thanks for your answer. I checked the traceroute in Windows and in the Raspi Pi4. It seems to be a problem... but on the other hand the data transfer Emporia -> IObroker is working nearly the whole day. I processed your workaround without understanding what this is doing. Maye it helps :-) Thanks a lot!

Ok
That would Pass the DNS Server using the ip directly. The error you get seems to have something to Do with DNS and AWS Service.
Monitor it and let me know
Regards Chris

Dear Chris, thanks for your reply. I will monitor if the workaround is OK. In the meantime, there is another issue after proceeding with the above mentioned change. Kind regards and have a nice weekend!

It seems that the ip you used is not reachable from your system ?
That could be the Problem.
Try all the ips you get with ping whether they are reachable.
Regards Chris

That is maybe a problem but on the other hand it is not.
Your Emporia adapter is working 99,5 % of the time. No issues.
If I do CMD and typ "ping api.emporiaenergy.com" the IP is always changing. Now it is <IP_ADDRESS>.
Since the failure from 12:36 h - no problems. The adapter running, data is comming.
I will monitor this topic... Thanks!

That is maybe a problem but on the other hand it is not. Your Emporia adapter is working 99,5 % of the time. No issues. If I do CMD and typ "ping api.emporiaenergy.com" the IP is always changing. Now it is <IP_ADDRESS>. Since the failure from 12:36 h - no problems. The adapter running, data is comming. I will monitor this topic... Thanks!

Try the new version if this more stable
regards Chris

close it now seems there is no issue anymore?


Add project to OpenUPM or make cleaned up version of just the plugin

I see that all of the scripts are already in the plugins folder. I'd reccommend to restructure the project so that it can be added with unity package manager or through OpenUPM or as a github submodule. All the Unity project bloat can probably be removed or put into a separate examples/demo repo.
Really good work though!!!

+1 on this, would be really nice to see this awesome package as, well, a unity embedded package for Package Manager. Makes the projects much more neat

Thanks to @frutbn now we have UPM support. Cheers 🎉


A710839966323293 additional contact points

Depends on:

https://github.com/openprocurement/openprocurement.auctions.core/pull/132

Pull Request Test Coverage Report for Build 67

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 78.917%

Totals

Change from base Build 66:
0.09%

Covered Lines:
539

Relevant Lines:
683

💛  - Coveralls


Introduce a Reset button

Summary
The reset button allows users to reset all settings to their default values. When the button is clicked, all settings are reset to their default values.
Related to the: Introduce a Reset button
Bugs
When a user clicks the Reset button values of the settings get reset but the radio buttons are not updated. I tried some things but couldn't figure out the unspagetti way to do it.

RE: Icons
We are using Lucide for this project. Here's the Reset Icon reference in the Tailwind Play -> https://lucide.dev/icon/rotate-ccw

Hey! Thank you for checking out PR! Here are my ideas on the things you mentioned:

If you do not feel comfortable using this framework and variant props, I can help you out.

I am using shacdn/ui in my projects too but I would gladly accept and help with tailwind and UI. I have added the default button to start on this issue and get feedback overtime. For button choice the first option a small button looks better with next to Settings text.

Reset Settings
You are passing arbitrary values in this function, in config/defaults you can find the default values that are used by the state reducer. I would add these ones instead. It will be easier to maintain and understand what happens when clicking this button.

I will make the necessary changes and update the code. I agree with you on this one.

The huge huge thing to figure out in this PR is the onload value for: insetColor and isDark value because these are dynamically generated depending on the image.
This is important because when we click on the button, I would expect to get what I started with and these values are calculated in utils/color and applied in the callback in ClipboardImage component.
Maybe we can store them in a ref and use them when the reset button is applied? Wdyt? I'm open for ideas here. If this results challenging we can take it in a separate issue and address here only the UI and using the default default values as declared in config/defaults.

Storing in ref and using them as a default value for reset button instead getting from config/defaults is a nice idea. But also we could only reset the values except insetDolor and isDark. This would be the painless way to implement but since there is an issue for implementing last 10 images from clipboard https://github.com/ekqt/screenshot/issues/13 we gonna end up storing those values somewhere.
I think creating another issue for storing those values will be much cleaner process. Let me change the default values and adding reset button with Settings text. Let's check deployed product and if we satisfy with result merge it to main close this issue and start implementing the storing values.
Let me know your thoughts. @ekqt

RE: Resetting Radio Button does not make in unchecked.
If user clicks Reset button if the Radio button is checked on Auto . It resets the setAspectRatio value back to 16:9 but the Auto radio button is stays checked. Couldn't find a way to implement this one properly. What are your thoughts on this one?

@kodbilenadam RE: Resetting Radio Button does not make in unchecked.

Not sure about that one from the top of my head but sounds like I must've left this as a uncontrolled input. Maybe ditching defaultValue for value instead and grabbing this from the settings.aspectRatio?

I think with this commit we are ready to go live. What do you think? @ekqt

Looks great!

This was fun! Thank you for the constructive feedbacks. 😄

@kodbilenadam great work! I'm planning on getting us a domain so we can make this project look a bit more official. Loving the ideas and the collaboration! Huge thanks! Make sure to recommend folks this web app whenever you see unframed screenshots haha Getting stars on GH will help us get traction. 🔥


Update homepage sections and spacing for new design

What
Create new view templates and stylesheets that are used when new_design is true, when requested the layout is updated so that:

The "More on GOV.UK" is moved next to "Government Activity"

Update the styling of the "More on GOV.UK" section so the links appear in a single column

Section heading font-sizes are increased from m to l
Section spacing is updated so that spacing for the top and bottom of each section adds up to 50px on mobile and 60px on desktop
Removed "Departments and organisations" section
Updated index_section_count and indexes_section values are also updated to reflect the new position on the page, this will ensure correct reporting in Google Analytics
Updated the Chevron card styling so that all content is vertically centred and the top border is removed from the first chevron-card element

Why
This is part of changes that will be gradually rolled out to the homepage. As these changes are not going to be made live immediately, I have attempted to split the code in a way that will allow them to be easily toggled by the new_design parameter.
Trello card
Visual Changes
Sections
Before

After

Chevron cards
Before

After

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

@andysellick would you also be able to help review this PR? thinking specifically about the updates to the sections on the homepage and the related index_section and index_section_count for Google Analytics when using the new design - https://github.com/alphagov/frontend/pull/3796/files#diff-b0533c1f365ffe916d27cb02d89deee3e7e1d586f289692af1c69b6036d88449

This is a minor accessibility thing, I'm wondering if clickable area/touch target of the chevron cards should be adjusted so there's some space between them. They have a pseudo element applied to them which makes the whole element clickable but if you are using zoom it might make distinguishing between the cards difficult.

This is a minor accessibility thing, I'm wondering if clickable area/touch target of the chevron cards should be adjusted so there's some space between them. They have a pseudo element applied to them which makes the whole element clickable but if you are using zoom it might make distinguishing between the cards difficult.

Thanks @patrickpatrickpatrick good spot!, I've updated the padding in a recent commit to ensure that the target size for the chevron-card is the same as before, and updated the padding on chevron-card__wrapper to ensure the total padding size is 24px at the top and bottom as needed in the design.

I think the spacing here

isn't the same as in the design https://www.figma.com/file/InXsSt89C5y5QRT1DZ3dPM/Homepage-redesign---2023?type=design&node-id=2522-77874&mode=design&t=7uUIoRtetLK4e6ac-0

Thanks @patrickpatrickpatrick I've updated this section so it now closely matches the Figma design

Looks good to me now, thanks Martin!


Enable milvus version of ChatQnA

Description
Enable milvus version of ChatQnA
Issues
https://github.com/opea-project/GenAIExamples/issues/886
Type of change

[ ] New feature (non-breaking change which adds new functionality)

Dependencies
No newly introduced 3rd party dependency.
Milvus is in GenAIComps already, but not enabled.
Tests
Verify ChatQnA milvus with UI and RAG functionality.

The PR dependes on
https://github.com/opea-project/GenAIComps/pull/858
858 Fixed milvus in retriever and data-prep and adapt milvus on TEI embedding.


[RAINCATCH-1263] Move keycloak realm json file to core templates

Motivation
We should move the raincatcher keycloak realm file within the core templates to make it available to use with a keycloak image.
Description
Move raincatcher-realm.json file from the raincatcher-keycloak repo to the core templates.
Progress

[x] Move raincatcher-realm.json file to the core templates

Additional Notes
Related JIRA - https://issues.jboss.org/browse/RAINCATCH-1263

Changes Unknown when pulling<PHONE_NUMBER>cbbbd15127ae67554a15d37938b996 on JameelB:RAINCATCH-1263 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling<PHONE_NUMBER>cbbbd15127ae67554a15d37938b996 on JameelB:RAINCATCH-1263 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling c727e993c17b95b71b00ef3ca9e1419025cbba1b on JameelB:RAINCATCH-1263 into ** on feedhenry-raincatcher:master**.

Changes Unknown when pulling c727e993c17b95b71b00ef3ca9e1419025cbba1b on JameelB:RAINCATCH-1263 into ** on feedhenry-raincatcher:master**.


AVT-937,AVT-938,AVT-939 : [Spectron][Windows] Close window when 'Minimize on Close' is ON

Description
Add test Close window when 'Minimize on Close' is ON  AVT-937
Add test Verify by deselecting Minimize on Close option once the application is launched AVT-938
Add test Verify Minimize on Close option once the application is installed AVT-939
Run
Please refer attachment for more details
Jest Test Report.zip
QA Checklist

[] Unit-Tests
[x] Automation-Tests

Jest Test Report.zip
My result


Setting has an invalid type, expected "string"

VSCode Version: 1.45.0
OS Version: Windows_NT x64 10.0.19041

Steps to Reproduce:

Change Font Weight to any value instead of "normal"
Then you will get this error
Setting has an invalid type, expected "string". Fix it in JSON.

Does this issue occur when all extensions are disabled?: Yes

/duplicate of https://github.com/microsoft/vscode/issues/97192


[Feature request] Add more social links to profiles

Currently only twitter is supported, but I think it'd be cool to add other options, notably mastodon or bluesky.

I'd be interested in adding up to e.g. 10 links to my profile, regardless of what they belong to. Icons can automatically be assigned if the URLs match some regex, much like GitHub:

This might be useful for us too, to future-proof us with new or niche social media.

Tom Aarsen


Reland "Remove obsolete drawShadow bounds workaround (#127052)"

This workaround was created 6 years ago with no links to bug databases to track. As best we can determine, the issue is no longer present in SkPicture or DisplayList and is most likely obsolete. More importantly, though, non-rendering primitives are ignored by the DisplayList construction and so this workaround will just be ignored anyway. If a problem surfaces about this issue that we haven't discovered by a thorough code search of the current code base, then workarounds should be installed within the relevant implementation modules rather than in the framework (and documented with bugs filed in either or both of Flutter's github repos, and/or the Skia issue database).
Workaround originally created in this PR: https://github.com/flutter/flutter/pull/9654

The original caused a golden image change that was not applied during the submit. The control script has been updated to fix potential problems in that workflow so this change should generate a golden failure to triage and hopefully merge successfully when that golden change is accepted.

Pushed a simple rebase to ToT to see if the testing will get past the Google Tests step.

It looks like the 2 Google testing failures were on web, but those should be now covered by https://github.com/flutter/engine/pull/42304 so another rebase should pass cleanly as soon as that fix rolls forward.

Hopefully final rebase pushed...


Project wide validators

What are you adding/fixing?

This pull request adds a .forRoot(options) method to the NgxErrors modules so that you can pass global application wide settings.  Currently the only option added was to have a default when condition that would get applied automatically without having to specify the when parameter or hasErrors() condition parameter every time.
I have found that I almost always use [when]=['touched'], this new code allows me to set this initially for the entire application and never have to set it again unless I want to override it for a control.  Cleaner HTML and easier to make global changes
Have you added tests for your changes?

I have updated the tests.
Will this need documentation changes?

I have update the documentation but please look it over and add or change if needed.
Does this introduce a breaking change?

Yes, this is a breaking change because you now have to add a NgxErrorsModule.forRoot() to you app module.
Other information
Please give it a test and let me know if you and other users find this useful.

Awesome, this looks great! I'll review when I can in the next few days! :)

@toddmotto Curious if and when you will approve this PR.  I need to know if I can use this feature in our app or if I need to revert back to the old methodology.

Hoping this gets pulled in soon!  Great module


Weird time values for bry forcing

ROMS actually likes the bgc bry file! 🎉
start_time=dt.datetime(2012,1,1,0,0,0)
end_time=dt.datetime(2012,3,1,0,0,0)
roms_bry = rt.BoundaryForcing(
    grid=roms_grd,
    start_time=start_time,
    end_time=end_time,
    physics_source={"name": "GLORYS", "path": datadir+'GLORYS/mercatorglorys*.nc'},
    bgc_source={"name": "CESM_REGRIDDED",
                "path": datadir+"CESM-climatology_lowres_regridded.nc",
                "climatology": True}
)

... however the values for bry_time look off to me, I assume related to climatology=True? My run is supposed to be in january 2012, but
bds=roms_bry.ds["bgc"].to_dataset()
[dt.datetime(2000,1,1,0,0,0)+dt.timedelta(seconds=float(bds['bry_time'].values[i]*1e-9)) for i in range(len(bds.bry_time))]

returns
[datetime.datetime(2000, 1, 16, 0, 0),
 datetime.datetime(2000, 2, 15, 0, 0),
 datetime.datetime(2000, 3, 15, 0, 0),
 datetime.datetime(2000, 4, 15, 0, 0),
 datetime.datetime(2000, 5, 15, 0, 0),
 datetime.datetime(2000, 6, 15, 0, 0),
 datetime.datetime(2000, 7, 15, 0, 0),
 datetime.datetime(2000, 8, 15, 0, 0),
 datetime.datetime(2000, 9, 15, 0, 0),
 datetime.datetime(2000, 10, 15, 0, 0),
 datetime.datetime(2000, 11, 15, 0, 0),
 datetime.datetime(2000, 12, 15, 0, 0)]

meanwhile for physics:
pds=roms_bry.ds["physics"].to_dataset()
[dt.datetime(2000,1,1,0,0,0)+dt.timedelta(seconds=float(pds['bry_time'].values[i]*1e-9)) for i in range(len(bds.bry_time))]

returns:
[datetime.datetime(2012, 1, 1, 12, 0),
 datetime.datetime(2012, 1, 2, 12, 0),
 datetime.datetime(2012, 1, 3, 12, 0),
 datetime.datetime(2012, 1, 4, 12, 0),
 datetime.datetime(2012, 1, 5, 12, 0),
 datetime.datetime(2012, 1, 6, 12, 0),
 datetime.datetime(2012, 1, 7, 12, 0),
 datetime.datetime(2012, 1, 8, 12, 0),
 datetime.datetime(2012, 1, 9, 12, 0),
 datetime.datetime(2012, 1, 10, 12, 0),
 datetime.datetime(2012, 1, 11, 12, 0),
 datetime.datetime(2012, 1, 12, 12, 0)]

as expected.
So all of the bgc bry values are in 2000, when my run is in 2012, and there doesn't appear to be anything in the metadata telling ROMS that this is a special case. I would expect ROMS to error out on initialisation in this case, but it doesn't. I'm not sure the behaviour that follows within ROMS is expected, just that there isn't an error. Could you explain? Thanks!

The bry_time variable has an attribute cycle_length that is filled with the value 365.25. If cycle_length is present, ROMS will just cycle through these dates (iteratively adding cycle_length) until the dates hit the actual range of the run.
In case of a climatology (as here), bry_time will just hold the year 2000 values, or whatever is closest to the reference time. If this is confusing, I can change it to year 2012 values or whatever is the first covered year of the simulation (based on start_time).
By the way, instead of doing
bds=roms_bry.ds["bgc"].to_dataset()
[dt.datetime(2000,1,1,0,0,0)+dt.timedelta(seconds=float(bds['bry_time'].values[i]*1e-9)) for i in range(len(bds.bry_time))]

you could have just looked at roms_bry.ds["bgc"]["abs_time"], which should hold the same values. ;)

abs_time and bry_time output the same thing for me!
xarray.DataArray
'abs_time'

    bry_time: 12

    array([<PHONE_NUMBER>000000,<PHONE_NUMBER>000000,<PHONE_NUMBER>000000,
<PHONE_NUMBER>000000,<PHONE_NUMBER>0000000,<PHONE_NUMBER>0000000,
<PHONE_NUMBER>0000000,<PHONE_NUMBER>0000000,<PHONE_NUMBER>0000000,
<PHONE_NUMBER>0000000,<PHONE_NUMBER>0000000,<PHONE_NUMBER>0000000],
          dtype='timedelta64[ns]')

    Coordinates:
        abs_time
        (bry_time)
        timedelta64[ns]
        15 days 45 days ... 349 days
        bry_time
        (bry_time)
        timedelta64[ns]
        15 days 45 days ... 349 days

Oh true, in the case of a climatology... Sorry!

Can this issue be closed @dafyddstephenson? Or what do you think about this:

In case of a climatology (as here), bry_time will just hold the year 2000 values, or whatever is closest to the reference time. If this is confusing, I can change it to year 2012 values or whatever is the first covered year of the simulation (based on start_time).

It can be closed, though I'd maybe suggest an attribute that indicates it's a climatology in a more human-readable format, and for abs_time to be formatted differently in this case - What do you think?

Both good ideas! I will work on those.

The attribute should maybe be global, rather than one of bry_time?


[MRG] avoid using deprecated functions in public API

Fixes #5578

This should work. But I get this trace, that is confusing me a bit.
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> PDB set_trace (IO-capturing turned off) >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
> /home/sik/code/mne-python/mne/io/tests/test_raw.py(58)_test_raw_reader()
-> assert_allclose(data1, data2)
(Pdb) w
  /home/sik/miniconda3/envs/mne/bin/pytest(11)<module>()

  ...

  /home/sik/code/mne-python/mne/io/eeglab/tests/test_eeglab.py(66)test_io_set_raw()
-> montage=montage)
> /home/sik/code/mne-python/mne/io/tests/test_raw.py(58)_test_raw_reader()
-> assert_allclose(data1, data2)

(Pdb) np.where((data2-data1)>0)
(array([10, 10, 10]), array([125, 214, 264]))
(Pdb) xx, yy = np.where((data2-data1)>0)
(Pdb) (data2-data1)[xx,yy]
array([1., 1., 2.])
(Pdb)

it breaks here, and I don't really understand what is data1 and data2.
https://github.com/mne-tools/mne-python/blob/fa119971035fb8cf0f92acc98c846645d5f410d2/mne/io/tests/test_raw.py#L56
check out the 3 different values of the trace.
any hints?
ping: @mmagnuski @larsoner @agramfort

@jona-sassenhagen @mmagnuski can you give a try to this branch and see if it fixes your problems with eeglab files? thanks heaps

Sure, I can take a look today. I didn't have much problems before, only the deprecation warning (and problems would arise if I followed what the deprecation message was suggesting).

Thx, all. (specially @agramfort) it was harder than I anticipated.

So there was a test we removed ???

No - In my PR, @mmagnuski suggested I add a test, and you and I decided we don't need it ...

arfff silly me... maybe I thought you needed a new file in testing but actually we don't.
@massich can you resurrect this test and add this to the PR? thx

The test was never removed. There was an error raised whenever latencies < 0. But in #5407 we add the particular case when latencies == -1 and decided that we should warn instead but let people the process go.
And yes, @mmagnuski detected that the code was not covered and we decided that was not problematic. Maybe it is not still. Because it might be something completely impossible.
My question remains, does anyone have an example where latencies are even -1? I don't understand how a latency could even be -1.

let's move on then.
Good to go then?

@massich regarding negative latencies, I can't found the original issue, but you can see some explanation here:
https://github.com/mne-tools/mne-python/pull/5377#issuecomment-408932492

green !
thanks @massich


Forward ref of component rendered by Async

Description
This PR adds ref forwarding for the component rendered by Async that is returned by the createAsyncComponent function.
Type of change
It's somewhat debatable whether this is a new feature or a bug fix. I'd tend to categorize the fact that refs are not being properly forwarded currently as a bug (unexpected behaviour for most consumers).

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

Checklist

[x] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

I am biased against refs, since I have never used them for anything outside of UI component code (where you are generally not in a place to use the async library), so take this feedback with a grain of salt.
That's definitely fair, I share the sentiment, though I don't think that debate necessarily needs to be re-hashed here. There are legitimate use-cases and I don't think react-async should take an opinionated stance here.

I find the use of ref here to be particularly confusing in this context, because a) there is no obvious signal we can send to the parent component for when this ref is going to be set, and b) the ref does not apply to loading or error state, which is maybe a sensible default, but seems like it leaves a pretty obvious edge case in solving this (IMO) existing edge case.

Agreed, I don't think there's an elegant solution around this possible given how the loading and error markup is rendered.
I also tend to think it would be acceptable to optimize around the success-scenario here for refs. If the component is loading or there was an error, the ref will be undefined. If the component has mounted successfully, refs should behave as they normally would.

In online-store-web, we recently went through a re-write of our RichTextEditor component from scratch. We're planning to run a beta to conditionally render the legacy / next versions.
We had normalized some public API methods for this component such as blur and focus behind an imperative handle ref. This was done to minimize duplication in how we render rich text settings.
Something like:
function RichTextSetting() {
  const editorRef = useRef();
  const Component = betaEnabled ? NextRichTextEditor : LegacyRichTextEditor;

  return (
     <Label onClick={() => editorRef.current.focus()}>
         <Component ref={editorRef} />
     </Label>
   )
}

Both of these components weigh upwards of 30kb so they are async imported. Our code broke when we changed to async imports for these components.
This limitation forced us to change our approach and have lots of duplicated logic in both LegacyRichTextEditor and RichTextEditorNext instead.

Its gone stale. lets close it unless @clauderic wants to try to bring this back.


[TypeScript] Implement interface should be (partially) customizable

When creating a class which implements an interface, an error is shown if there are missing properties. Accessing the quick fixes prompts you to implement the interface, which fills in properties and methods from the interface.

// INTERFACE
interface IClassName {
  someprop: propType;
  someMethod(args: argType): returnType;
  /* ... */
}

// BEFORE QUICK FIX
class ClassName implements IClassName {
// Class 'ClassName' incorrectly implements interface 'IClassName'.
//   Property 'someprop' is missing in type 'ClassName' but required in type 'IClassName'. ts(2420)
}

// AFTER QUICK FIX
class ClassName implements IClassName {
  someprop: propType;
  someMethod(args: argType) {
    throw new Error("Method not implemented.");
  }
}

However I would prefer in many cases (such as throwing together temporary classes to ensure consuming code is type safe) to have the methods return null as unknown as returnType, basically "shut up it's a string, I'll implement it later" or something.
As the quick fix is already aware of the interface and the types within, it would be possible for the output to look like this;
// AFTER PROPOSED ALTERNATIVE QUICK FIX
class ClassName implements IClassName {
  someprop: propType;
  someMethod(args: argType) {
    /* line could act as a soft error in case this falls into the hands of peers, or if the author has forgotten. */
    console.error("Method not implemented.");
    return null as unknown as returnType;
  }
}

Important Note: This proposed alternative should not replace the existing "implement interface" quick fix. Instead, it could show up as another option, perhaps labelled "implement interface (unsafe)". The work "unsafe" would do enough to ward people away from using this as a common choice, and those who have the use case (such as myself) will be able to use it.
How urgent is this? Nice to have, idea formed from evening laziness.

I think having the ability to modify many quick fixes would be very useful to a lot of developers. I find myself often making the same small tweaks over and over to the quick fixes provided, simply because of my preferred style of programming (I stay within reason of course, I'm not advocating for spaghetti code).
As an alternative to this, maybe some inspiration can be taken from the Snippets system. A way to modify quick fixes, a default file and a user file for each language with the normal priority rules for such pairs.
If you find yourself making the same tweaks every time you use a quick fix, just write that tweak into [quick fixes dir]/[language].json.
I understand that this would take a huge amount of effort though, and wouldn't be surprised if it's not deemed worth it. The initial suggestion here though, I believe would be useful for many people.

Since you proper adding a new quick fix, this sounds like a good candidate for a typescript service plugin
( big 👎  for auto inserting code that casts away type errors from me though)

This QF has been around a very long time and this is the first we've seen of a request for different behavior; I don't think this meets the bar in terms of demand.


Receiving errors continuosly

Hi everyone,
From the last downtime that happened some days ago, I am experiencing the following two errors in some APIs from time to time:
504 Gateway Time-out
502 Bad Gateway
Localtunnel has looked very unstable these last few days. Is anyone else experiencing those problems?
Many thanks!

It's started working again.


files.upload issue

I wanted to get a command to upload files to slack and found this : https://api.slack.com/methods/files.upload
When I am trying to upload a file by following instruction, link https://slack.com/api/files.upload is reading nothing but error:
{"ok":false,"error":"not_authed"}
Please be notified that I am passing token in my command.
command I am running:
curl -F file='/home/testsend.txt' -F channels=#channel_name -F token='tokenNumber' https://slack.com/api/files.upload

Command is perfect, I was just using the wrong token number. Closing the issue as this is no more exist.


Add Supergroup Support

Since Supergroup updated the id to 1e13, default i32 will suffer a panic because it is too big for i32.

Also new optional fields added to the Message object: supergroup_chat_created, migrate_to_chat_id, migrate_from_chat_id and channel_chat_created, and #27

Fixed in #29


AWSSDK.CloudFront is slower than  AWSSDK.S3 - Signed URL Generation

Describe the bug
Previous issue
Because of the above issue, we are just wondering if there is any issue on the CloudFront as well. Because compared to S3 the CloudFront Response time is x6 or x5 times slower than S3.
These two methods (PlayerSummaryFull and UpdatePlayerProfilePicture) use S3 or CloudFront to Generate Signed URLs.
The Following two benchmark snapshots are by using S3 and CloudFront respectively as media content providers.

AWSSDK.S3 - 3.7.304.7

Benchmark log: Benchmark-using-S3.log

AWSSDK.CloudFront - 3.7.301.11

Benchmark log: Benchmark-using-CloudFront.log

Sorry for not having Multiple version comparisons or more details. The reason for providing these details is FYI to identify if there is any performance degradation on the CloudFront as well.
Expected Behavior
No response
Current Behavior
No response
Reproduction Steps
No response
Possible Solution
No response
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
AWSSDK.CORE - 3.7.300.18
AWSSDK.S3 - 3.7.304.7
AWSSDK.CloudFront - 3.7.301.11
Targeted .NET Platform
.NET Framework 7
Operating System and version
Windows 10 Enterprise

@SonalCalcey Good morning. The fix merged in other issue https://github.com/aws/aws-sdk-net/issues/3122 was made in AWSSDK.Core and hence should have propagated to CloudFront package as well. Do you have performance benchmarks between different versions of AWSSDK.CloudFront rather than comparing it with AWSSDK.S3?
Thanks,
Ashish

Hi @ashishdhingra Good morning. yes, I have a couple of benchmarks using AWSSDK.CloudFront.
AWSSDK.CloudFront - 3.7.301.12

AWSSDK.CloudFront - 3.7.300.5

AWSSDK.CloudFront - 3.7.0

All are using AWSSDK.CORE - 3.7.300.19
Thanks,
Sonal

@SonalCalcey Thanks for the information. Could you please share sample code to demonstrate what API you are calling to generated signed URL using S3 and CloudFront?
Thanks,
Ashish

@ashishdhingra  These are the sample codes that we used to generate signed URL.
S3

CloudFront

Thanks,
Sonal

Hi @ashishdhingra,
Just to Make sure that you understand the issue we have we have done a direct benchmark between CloudFront and S3. With the results of the below benchmarks, our main concern is that compared to the S3 signed URL generation time,  the CloudFront signed URL generation time is higher.
Just to let you know, is this behavior normal or are there any other methods or ways to get the CloudFront Signed URL without performance degradation that we are unaware of?
Benchmark result:

Code used for the benchmark:
[MemoryDiagnoser]
public class CloudFrontRunner
{
	// CloudFront
	private static string _sourceKey;
	private static string _cloudFrontKeyId;
	private static string _cloudFrontPrivateKey;
	private static string _cloudFrontDistributionDomain;

	// S3
	private static string _accessKeyId;
	private static string _secretAccessKey;
	private static string _bucketName;
	private static AmazonS3Client _s3Client;

	[GlobalSetup]
	public void GlobalSetup()
	{
		var configuration = new ConfigurationBuilder()
			.AddJsonFile("appsettings.json", optional: false)
			.Build();

		_sourceKey = configuration["AWS:SourceKey"]!;

		// CloudFront
		_cloudFrontKeyId = configuration["CloudFront:KeyPairId"]!;
		_cloudFrontPrivateKey = configuration["CloudFront:PrivateKey"]!;
		_cloudFrontDistributionDomain = configuration["CloudFront:DistributionName"]!;

		// S3
		_accessKeyId = configuration["AWS:AccessKeyId"]!;
		_secretAccessKey = configuration["AWS:SecretAccessKey"]!;
		_bucketName = configuration["AWS:BucketName"]!;
		_sourceKey = configuration["AWS:SourceKey"]!;
		_s3Client = new AmazonS3Client(_accessKeyId, _secretAccessKey, RegionEndpoint.APSoutheast1);
	}

	[GlobalCleanup]
	public void GlobalCleanup()
	{
		_s3Client.Dispose();
	}

	[Benchmark]
	public string CloudFrontGenerateSignedUrl()
	{
		return AmazonCloudFrontUrlSigner.GetCannedSignedURL(
			AmazonCloudFrontUrlSigner.Protocol.https,
			_cloudFrontDistributionDomain,
			new StringReader(_cloudFrontPrivateKey),
			_sourceKey,
			_cloudFrontKeyId,
			DateTime.Now.AddMinutes(5)
		);
	}

	[Benchmark]
	public string S3GenerateSignedUrl()
	{
		GetPreSignedUrlRequest request = new()
		{
			BucketName = _bucketName,
			Key = _sourceKey,
			Expires = DateTime.Now.AddMinutes(5)
		};

		return _s3Client.GetPreSignedURL(request);
	}
}

Versions:
AWSSDK.Core - 3.7.300.28
AWSSDK.CloudFront - 3.7.301.21
AWSSDK.S3 - 3.7.305.3
.Net - 7
Thanks,
Sonal


Support for bulk extension

https://springbot.github.io/json-api/extensions/bulk/
There is an official bulk extension for PATCH/POST requests. I was wondering if there was a way to format the bulk payload in and generate Rails-like params with the current version of the gem if not, is there a possibility to add this support?

Ok here is how I did it.
You can create a new set of ActionController::Parameters from each data object in the bulk payload.
payload_object = ActionController::Parameters.new(data: params['data'].first)
payload_object.from_jsonapi
=> Works

Then you can validate as you wish.


Styling issues on Chromium 87

Current Behavior:
Player doesn't draw correctly, but buttons work. (no JavaScript issues, just CSS)
Expected Behavior:
Gets drawn correctly, since browser support is there: https://caniuse.com/custom-elementsv1
Steps To Reproduce:

Open in Chromium 87
See that the player doesn't get drawn correctly

Environment:

Framework: Any
Device: Any
OS: Any
Browser: Chrome@87

Anything Else?
How the player looks on Chromium 87:

Hey AgentSmith! Chromium 87 is outdated and no longer supported. However, we might still remove the :where selectors as it's causing issues with CSS resets. Follow that issue for any updates.

I'll come back to considering a CSS file without :where, but can't guarantee because Chromium 87 is so old. You can show native controls by adding the controls to the player:
<MediaPlayer controls>


After update to Solo 4, controller do not find solo

I have followed the instruction to update to Solo 4, but after update the drone and controller were restarted and controller is only in state "waiting for solo". Please, I need a help if someone have already faced this kind of issue. Thank you.

Did you repair it? (Hold solo pair pin for 3 seconds then follow controller instructions)

I am having this issue as well. Holding fly and power does not put it into update mode either. leds are green red and two yellow which after a time turn blue. Any ideas?

there are plenty of instructions on the internet about what to do if the process doesn;t work... the short answer is repeat the process from the beginning,


refactor: optimize Fuse.js option

I tried optimizing Fuse.js options to get the results want more.
Try it if you like.
refer: https://fusejs.io/api/options.html
minMatchCharLength
The default of 1 may include unintended results.

shouldSort
To sort the result list, by score.
ignoreLocation / distance
Too long strings often contain extraneous characters that can become noise.
⚠ However, this setting may prevent from getting what used to be able to get.
keys
Add key for weighted search.
searchTerm is for AND search across keys.

cypress/videos/e2e.ts.mp4

Oh… I committed it by mistake.
I'll delete it later.

This is a great pull request👍
Thank you!
I'll try out the config of this branch for a week or so, and if all is well, I'll merge it in!

I just tried it for a bit and it's so good that I decided to merge it without the need to see how it goes XD.
Thanks so much for the awesome pull request！！！

cypress/videos/e2e.ts.mp4

I'll add this one to gitignore 🙏


Update Twitter url to X url to be consistent with X logo

hey thanks for this, however it looks like the x.com url just 302s to twitter.com anyway so probably best if we link to the final destination


Need Update over 2.0.0 release timeline.

Hi Team,
We have a use case where we need the pyspark xgboost integration feature #8020. Kindly need an update over xgboost 2.0.0 release timeline.

Please see https://github.com/dmlc/xgboost/issues/8282 .


Support for promises in mocha-like tests

The Mocha framework detects when a test returns a promise, which allows developers to write asynchronous tests without repeating if (err) return done(err) all over the place (see https://github.com/mochajs/mocha/pull/329):
it('reads file via promise', function() {
  return fs.readFileAsync('foo.txt')
    .then(function(text) {
      expect(text).to.contain('foo');
    });
  });
});
// the test runner waits until the returned promise is resolved

I am proposing to implement this support in node-tap too, I am happy to contribute the implementation myself.
Thoughts?

Seems fine to me, as long as tap itself doesn't have to start using promises all over the place.  How does mocha detect?  Just check for a .then method on the returned object?

As long as tap itself doesn't have to start using promises all over the place.

Yes, my intention is to make a small isolated change just to support promise-returning test functions, see below.

How does mocha detect? Just check for a .then method on the returned object?

Yes, exactly like that. See lib/runnable.js#L265-L279
var result = fn.call(ctx);
if (result && typeof result.then === 'function') {
  result
    .then(function() {
      done()
    },
    function(reason) {
      done(reason || new Error('Promise rejected with no or falsy reason'))
    });
}

Frankly, I won't have time to contribute this feature as soon as I originally thought. Feel free to close this issue so that it doesn't clutter project's issue list.

I'm happy to leave it open in case someone comes along and wants to do it.  That someone may even be me one day :)

+1
I use is-promise to detect promises.
One HUGE benefit of adding support for returned promises is that it lets node-tap work out of the box with async functions. An async function always returns a Promise. Async arrow functions are also a thing.
Here's an example of a random mocha test that uses babel:
// ... imports

describe('Request:Users', () => {
  describe('GET /api/users/current', () => {
    it('returns current user', async () => {
      const user = await factories.createUser()
      const headers = buildUserHeaders(user)

      const res = await request
        .get('/api/users/current')
        .set(headers)
        .expect(200)

      expect(res.body).to.eql(JSON.parse(JSON.stringify(user)))
    })

    it('creates a user for you', async () => {
      await request
        .get('/api/users/current')
        .expect(200)
    })
  })
})

Oh, and it'd also play nicely with co, if someone doesn't want to use babel to transpile their code:
const co = require('co')
const assert = require('assert')
const asyncThing = require('./asyncThing')

describe('co test', function () {
  it('does async thing', co.wrap(function * () {
    const result = yield asyncThing()
    assert.equal(result, 'success')
  }))
})

Is this still free for grabs in terms of implementing?

Is this still free for grabs in terms of implementing?

I think it should be still free. The author of the issue (means me) is certainly not working on the implementation.

@jkrems I think @aredridel was interested in taking a look at how to make tap promisey, not necessarily specifically around the mochalike tests (which, tbh, should probably be moved to a separate module or something, because it turns out that Mocha's BDD interface is actually quite huge).
I'm not saying this to discourage you, but to say that there may be some overlap in what you two are doing, so chat about it maybe?

+1

Thanks! I just realized that I actually misread the issue and didn't realize this was about the mocha-like interface in particular. Oops. What I had in mind was just a small addition to the basic interface to handle t.end and failing tests. Example: https://github.com/isaacs/node-tap/compare/master...jkrems:jk-promise-return?expand=1#diff-dd2e881af3311f834199c7b515cff0c7R398. Sorry for storming ahead with a first pass at implementing it, definitely happy to discuss better or more widely useful approaches.
I modeled it after blue-tape. Should I create a separate issue and/or PR to discuss this since it's technically not matching this issue's description?

@jkrems Neat.  It seems like that wouldn't support stuff like t.test(...).then(...).then(...), would it?  (And, is that a thing to actually want to do?  Should those promise-returning blocks be additional tests, or what?)

That's correct. Right now this could not be used to control the order of tests but from my current understanding that's handled by tap out-of-the-box (ensuring tests run in sequence).
For me the main use case of this feature would be async functions (example using blue-tape and babel). But maybe I'm missing a use case where t.test(...).then would be important.

How about if t.test returned this, so you could do this?
t.test('first', function (t) {
  return somePromise().then(function () {
    return someOtherPromise()
  }).then(function () {
    return yetAnotherPromise()
  })
}).test('second', function (t) {
  ...
}).end()

Ugh, actually having t.test return t is weird, and I wish I hadn't suggested it.  Nevermind.


Concretised flow title condition

Codecov Report

Merging #25 into 0.1 will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##                0.1      #25   +/-   ##
=========================================
  Coverage     36.56%   36.56%
  Complexity      255      255
=========================================
  Files            17       17
  Lines           856      856
=========================================
  Hits            313      313
  Misses          543      543

Flag
Coverage Δ
Complexity Δ

#php
36.56% <0.00%> (ø)
255.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

src/Model/Question.php
66.26% <0.00%> (ø)
22.00 <0.00> (ø)

Continue to review full report at Codecov.

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


App Gateway Redirection more info needed

Can you include some information on >> 'Include query string' and 'Include path' Options

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

ID: c327ae28-e9cf-38cf-f4b5-db72d4a870c3
Version Independent ID: 7cbbf426-829a-849f-b7fb-192073cd4ea9
Content: Redirect overview for Azure Application Gateway
Content Source: articles/application-gateway/redirect-overview.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@nitbhatt  Hi - can you clarify the request?  I don't see any references to 'Include query string' and 'Include path' Options in the article.

Please respond if there is still an issue. Thanks!
#please-close


[Pricing] add CustomMultiscaleQuantizeTransform

Add a transform that allows to select or exclude specific features for quantization
@hectorgon

:+1:


Second pass of name resolution

Hello
I found a bit of more time for the name resolution (#16) and put some more stuff there.
However, I'm afraid the whole chapter is above my abilities (or time, at least) to document reasonably ‒ I'm reading through the code and still don't understand most of it and have no idea about which part of code relates to the other bullet points in the issue description. Therefore, if there's someone else who might want to have a look at it, I'd like to release the lock on the task and I'll find some other way to contribute to rust (pull requests to random crates I use seem to have better cost-effect ratio for me).

If it helps:

There's also usually a "Help wanted" section in https://this-week-in-rust.org

And there are also sometimes mentored issues in the rust repo itself.


refactor: reduce coupling to test framework

Explanation
Reduce coupling to test framework.

/cherry-pick release-0.2


Webhook: Validate unique port, protocol per hostname for gateway listeners.

What type of PR is this?

/kind feature
What this PR does / why we need it:

Add validation to validate protocol is unique per hostname for gateway listeners
Add validation to validate port is unique per hostname for gateway listeners

Which issue(s) this PR fixes:

Fixes #847
Does this PR introduce a user-facing change?:

NONE

@robscott I make it a draft PR because I have some question:

As you mentioned in #847:

The validating webhook should ensure that port, protocol, and hostname are unique among Gateway listeners

but then you give an example:

"2 listeners for foo.com would be valid if they were on different ports or protocols, but not if port and protocol were also identical."

They are a bit inconsistent, but my understanding is the webhook should separately validate unique port, protocol per hostname for gateway listeners. Is it correct? Is it needed to verify unique port, protocol across hostnames? Is it needed to verify unique hostname per listener?

I add validations in v1beta1 apis, should I add them in v1alpha1 apis too?

@gyohuangxin where are we at with this one? Is there anything we can do to help the progression here? Looks like maybe we're still waiting on some feedback requested from @robscott?

Yes, it's on hold because I'm still confused with the first question,  I'm not sure my thought is close to the reality.

@robscott I make it a draft PR because I have some question:

As you mentioned in Webhook: Validate unique port, protocol, and hostname per listener #847:

The validating webhook should ensure that port, protocol, and hostname are unique among Gateway listeners

but then you give an example:

"2 listeners for foo.com would be valid if they were on different ports or protocols, but not if port and protocol were also identical."

They are a bit inconsistent, but my understanding is the webhook should separately validate unique port, protocol per hostname for gateway listeners. Is it correct? Is it needed to verify unique port, protocol across hostnames? Is it needed to verify unique hostname per listener?

Yes, it's on hold because I'm still confused with the first question, I'm not sure my thought is close to the reality.

Ok, I put this on next weeks community sync to discuss as part of the "old PR roundup" so we can talk about it there.

Thanks for the work on this @gyohuangxin! Sorry I lost track of this, don't hesitate to ping me if I can help explain anything better.

Thanks @gyohuangxin ! This Looks good to me.
We probably want to change the PR description to match the new logic - ensuring Listener's port-protocol-hostname combinations are unique within an individual Gateway.
over to @robscott for a final review and approval.

Thanks @gyohuangxin ! This Looks good to me. We probably want to change the PR description to match the new logic - ensuring Listener's port-protocol-hostname combinations are unique within an individual Gateway.
over to @robscott for a final review and approval.

Sorry for missing the PR description, updated. Thanks for reminding 💯

Based on this comment it seems like we should take into consideration the tls properties attached to a listener
https://github.com/kubernetes-sigs/gateway-api/issues/1842#issuecomment-1484615419
'

Thanks for your comments @dprotaso . As said in @robscott 's https://github.com/kubernetes-sigs/gateway-api/issues/1842#issuecomment-1496560365, the "Hostname" can be different for shared Port and Protocol, like the TLS example on the website: https://gateway-api.sigs.k8s.io/guides/tls/#wildcard-tls-listeners
So, I think it can cover the use cases. Please let me know if there are some use cases that use the same combination of port, protocol and hostname but use distinct tls fields.

Thanks @gyohuangxin!
/approve
/ok-to-test

/lgtm
/approve


storage/driver/azure: Update vendored Azure SDK

This change refreshes the updated version of Azure SDK for Go
that has the latest changes. Also addresses the memory leak
problem.
I manually vendored the new SDK (github.com/Azure/azure-sdk-for-go)
and I removed management/ core/ packages manually simply because
they're not used here and they have a fork of net/http and crypto/tls
for a particular reason. It was introducing a 44k SLOC change otherwise...
This also undoes the include_azure flag (actually Stephen removed the
driver from imports but forgot to add the build flag apparently, so the
flag wasn't really including azure. :smile: ). This also must be obsolete
now.
Fixes #620, #175.
Signed-off-by: Ahmet Alp Balkan
cc: @dmp42 @stevvooe

LGTM
@ahmetalpbalkan Thanks! Sorry about disabling the driver temporarily.

LGTM.
Thanks @ahmetalpbalkan !

Awesome, thanks for reviewing.

LGTM

Could you also update docs/building.md to remove the include_azure build tag info?

I already did. :-)

@BrianBland Could we get an LGTM here?

I see the change to docs/storage-drivers/azure.md but not docs/building.md. Everything else is good

@BrianBland oh missed that one, sorry. I fixed that now.

Awesome, thanks for the quick update

@ahmetalpbalkan Thank you for the good PR!

You're welcome @stevvooe, always a pleasure! :sunglasses:


Find-EntraPermission needs to have 2 parameter sets as Find-MgGraphPermission

Find-MgGraphPermission has two parameter sets. -All and -SearchString are in separated sets.
Find-EntraPermission has only one parameter set. A user can tab-complete -SearchString and -All, try to use them both, and command will fail at the runtime.
PS> Find-EntraPermission -SearchString "user.read" -All
Find-MgGraphPermission: Parameter set cannot be resolved using the specified named parameters. One or more parameters issued cannot be used together or an insufficient number of parameters were provided.

@alexandair please confirm this is resolved via this PR https://github.com/microsoftgraph/entra-powershell/pull/1145


ModelID is missing from Bridge.Config.Info

modelid	string 6..32	1.8	This parameter uniquely identifies the hardware model of the bridge (BSB001, BSB002).
http://www.developers.meethue.com/documentation/configuration-api

Added ModelId and BridgeId, will be released with next version.


Merge JSONFileCache change from v2 to v1

In v2, a constructor argument dumps_func was added to allow control over serialization (in particular, to allow the CLI v2 to cache AWS SSO tokens with actually ISO8601-compliant timestamps). However, this code has not been ported over to botocore v1, and because of this, aws-sso-util is still writing timestamps that are not ISO8601-compliant. The reason ISO8601-compliant timestamps are important here is because of this issue in the Go SDK: https://github.com/aws/aws-sdk-go/issues/3324

Apologies @benkehoe, this appears to have been lost in the fray. The fix was merged back in February with #2302, I'll plan to close this tomorrow if you don't have other concerns. It looks like you managed to get this added to aws-sso-util.

Yep, I should have closed this when I saw that merge and started using it


AttributeError: 'function' object has no attribute 'urlparse'

Traceback (most recent call last):
File "C:\Test\weibo-chaohua-daily-master\main.py", line 300, in
main()
File "C:\Test\weibo-chaohua-daily-master\main.py", line 225, in main
chaohua_list = chaohua_list + resolve(card_group)
File "C:\Test\weibo-chaohua-daily-master\main.py", line 87, in resolve
query = urlparse.urlparse(scheme).query
AttributeError: 'function' object has no attribute 'urlparse'
C:\Test\weibo-chaohua-daily-master>pause
Press any key to continue . . .

I have fixed the bug under python3 in this commit


Implement unwind information reset

This implmementation has several shortcomings, and doesn't have any rate-limiting in place. We will continue to improve this and have plans to revisit how we store chunks (as this can be optimized) as well as how the unwind information is laid out to try to prevent expensive state resets.
Test Plan
Ran on busy hosts that used all shards. The state was resest and profiling continued as usual.

cc @gmarler as this should fix the bug you reported. Let me know if that's not the case!


Make bin/server awares non-GAE dev environment

fix #1513

I don't believe this is the right change to do. isDev() is independent from isDevServer(). I'd like to keep it that way. Check the func's doc.

Then the optional left is to let the gulpscript define environment variable to control the isDevServer? That seems a solution?

Yes. In fact it was there if you look at gulpfile before refactoring into gulp_scripts/backend.js.
I forgot to put it back.

Yes. In fact it was there if you look at gulpfile before refactoring into gulp_scripts/backend.js.
I forgot to put it back.

and sorry about slow responses. as you can see i have a "great" connection over here, which doubles my comments :)


Added navBar with MLH logo and animation on scrolling, as well as links to existings sections

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Screenshot of the Navbar added -

@PAOK-2001 and @Mayank17M  please review the PR
Closes #8

The navbar looks much better and clean.
A few things can be changed to make it look even better.

Change the MLH logo to the MLH prep logo: The prep logo gives the website a better representation.
The navbar color can be changed from white to any other color that goes well with the header background. Check out #15 for reference.
The text size on the navbar("Meet the pod"...) can be increased as it looks smaller wrt the navbar size.

Other than this, I think this is good and the code looks clean.
Let's wait for some more people to review this!

Changes -

[x] Logo changed to MLH Prep
[x] Increased the font size of the navbar links
[x] Changed the color of navbar

Here's a screenshot of how it looks now -

@Mayank17M @PAOK-2001 please review the changes as requested by @Mayank17M

LGTM! 🎉

The Nav Bar looks great! Totally agree with the change of color! Excelent job. Looks ready

@suraj-singh127 a few conflicts, maybe merge main and look at it.

@suraj-singh127 , you can clear the build cache maybe and retry the deploy again?

Is this ready to get merged @suraj-singh127?

I am also trying to solve #20 so that it solves the profile images overflowing.

I am also trying to solve #20 so that it solves the profile images overflowing.

I think you mean #31 *.

@yashika51 This PR is good to go!


Fix an installation issue with Composer

Fix #84.

Coverage increased (+2.3%) to 80.323% when pulling ab23d91320c961ba5a43cb414ac05dc724407f36 on hotfix/fix_composer_install into 9172c66fb21987b628cec7465cb99bc8d2aad67a on master.


@mikehardy/lint battles

A bunch of little battles I just won with eslint + prettier ;-)
Each comment has what I learned and the reference
what a pain in the ... haha
yarn lint:all is now completely clean with these, including in a real project where there are switch statements and export const enums and such

Hmmm :thinking: - obviously the goal is clean lint runs and I have that locally.
I'll re-examine the lint failure a bit later to make sure this goes green in CI

Ah, it's on a component that doesn't exist anymore after #32 - or shouldn't - it does seem like the develop branch may be in a strange state ?
/home/runner/work/luna/luna/template/src/app/components/LearnMoreLinks/LearnMoreLinks.tsx

Error:    1:1   error    Delete `⏎`                                                                                prettier/prettier
Error:   17:31  error    Replace `'button'` with `"button"`                                                                                prettier/prettier
Warning:   17:31  warning  Unexpected usage of singlequote                                                                                jsx-quotes

Yep, it also seems to me that I messed something up with the develop branch, I'm trying to figure it out,

I'm doing the last changes for Luna and I'll try the firebase example ASAP.
Unfortunately, I'm still on a vacation and my family demands my attention 😂 Trying to hop on the laptop whenever I have some spare time.

Oh hang with the family every time. Laptop will always be there... Cheers man :-)


TC-SC-5.2 failing with group id 0x0101 (and only) (CON-974)

Describe the bug
When running TC-SC-5.2 on an official Matter TH, the test case fails at step TH sends ViewGroup command with status value mismatch: expected 0 but got 139. Using group id 0x0103 in the other steps works flawlessly.
DUT logs do not show any obvious errors. Adding and viewing groups with commands works, except for the tested id "0x0101".
Any hints?
Are there any definitions for the status codes, which the device responds with? Like 139 and 195? I think '0' is SUCCESS? I didn't find anything in the Matter specs...
Environment

ESP-Matter Commit Id: dd4f34ec98ee1d5467bcfa8f6c885d53de58e50e
ESP-IDF Commit Id: e088c3766ba440e72268b458a68f27b6e7d63986
SoC (eg: ESP32 or ESP32-C3): ESP32
Device Logs (Please attach the log file):
Host Machine OS: Linux version 6.1.29-1-MANJARO (builduser@fv-az292-908) (gcc (GCC) 12.2.1 20230201, GNU ld (GNU Binutils) 2.40) #1 SMP PREEMPT_DYNAMIC Wed May 17 14:00:55 UTC 2023
Commissioner app and versions if present: TH Version: v2.8.1-official | TH SHA: 1446f09
Commissioner's logs if present:
TC-SC-5_2_2024_01_22_14_29_36.zip

Any additional details
Test also failed at official testing offing for licencing.

I came across https://github.com/project-chip/connectedhomeip/issues/29352 added the Groups.Endpoint = 0 thing and retestet. Still failing.
TC-SC-5_2_2024_01_22_14_44_28.zip

Of course, here is the TH report: TC-SC-5_2_2024_01_22_14_44_28-74.zip
DUT log is full of device-output (and even CAN-over-UART-bytes) and therefore hard to read. If you still want need it, I will retest and save the logs...

@PhLuReh I went through the logs shared by you  in that I can see step's are mismatching but still the step ./chip-tool groupkeymanagement key-set-remove "0x01a3" 1 0 should work.

I tried it at my end with same esp-matter commit id and it is working as expected!

Have you selected GroupKeyManagement PICS and Groups Cluster PICS while executinng this TC?

While generating PICS make sure that you have select mandatory Feature, Attribute and commands required for Groups Cluster otherwise step related feature or attribute or command skip while execution.

I try to answer your questions.
GRPKEY.S (Does the device implement the Group Key Management cluster as a server?) is marked as supported (true).
G.S (Does the device implement the Groups cluster as a server?) is marked as supported (true).
I rechecked the PICS, which I use in the Matter TH with the online PICS tool, and there's no obvious error related to this test case:
General Diagnostics Cluster Test Plan.xml: Validated 26 items with 0 warnings and 0 errors.
Group Communication.xml: Validated 6 items with 0 warnings and 0 errors.
Groups Cluster Test Plan.xml: Validated 22 items with 0 warnings and 0 errors.
Identify Cluster Test Plan.xml: Validated 17 items with 1 warning and 0 errors.
Multiple Fabrics Test Plan.xml: Validated 18 items with 0 warnings and 0 errors.
OTA Software Update Test Plan.xml: Validated 4 items with 0 warnings and 0 errors.
Scenes Cluster Test Plan.xml: Validated 35 items with 0 warnings and 0 errors.
Wi-Fi Network Diagnostics Cluster Test Plan.xml: Validated 36 items with 0 warnings and 0 errors.
Window Covering Cluster Test Plan.xml: Validated 82 items with 1 warning and 2 errors.
Access Control cluster.xml: Validated 11 items with 0 warnings and 0 errors.
Access Control Enforcement Test Plan.xml: Validated 1 item with 0 warnings and 0 errors.
Base.xml: Validated 179 items with 2 warnings and 0 errors.
Descriptor Cluster TestPlan.xml: Validated 10 items with 0 warnings and 0 errors.
General Commissioning Cluster Test Plan.xml: Validated 13 items with 0 warnings and 0 errors.
Device Management Test Plan.xml: Validated 126 items with 2 warnings and 0 errors.

the 2 relevant PICS files
I would like to try to run it manually. could you provide the chip-tool calls for this? I tried, but I failed already at the second and third step. How to write the ACLs?
./chip-tool accesscontrol write acl '[{"fabricIndex": 1,"privilege": 5,"authMode": 2,"subjects": 1,"targets": null },{"fabricIndex": 1,"privilege": 4,"authMode": 3,"subjects": [259],"targets": null}]'  1 0
Thanks in advance.

I have checked your PICS file and it is correct. Please find the attached TC-SC-5.2-steps file for manual test execution. I have also attached the device and chip-tool logs of TC-SC-5.2, which I have executed on my end with the same specifications used by you. I am getting the expected output for all the steps. Please refer to the following links for the logs and test case document:
TC-SC-5-2-chip-tool.log
TC-SC-5-2-DUT.log
TC-SC-5.2-Steps.ods

Don't get me wrong, I'm very happy about your support, but there's some quirks going on...
I manually run the steps from your TC-SC-5.2-Steps.ods:
./chip-tool accesscontrol write acl '[{"fabricIndex": 1, "privilege": 5, "authMode": 2, "subjects": [112233], "targets": null },{"fabricIndex": 1, "privilege": 4, "authMode": 3, "subjects": ["0x0103"], "targets": null }]' 1 0 > 2024-02-21-TC-SC-5_2.log
./chip-tool groupkeymanagement key-set-write '{"groupKeySetID": "0x01a3","groupKeySecurityPolicy": 0, "epochKey0":"d0d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime0": 1,"epochKey1":"d1d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime1":<PHONE_NUMBER>3709551613,"epochKey2":"d2d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime2":<PHONE_NUMBER>3709551614 }' 1 0 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groupkeymanagement write group-key-map '[{"groupId": "0x0103", "groupKeySetID": "0x01a3", "fabricIndex": 1},{"groupId": "0x0101", "groupKeySetID": "0x01a3", "fabricIndex": 1}]' 1 0 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groups remove-all-groups 1 0 >> 2024-02-21-TC-SC-5_2.log

The fourth call was unsuccessful. I then had a look at your TC-SC-5-2-chip-tool.log, where you ran:
./chip-tool accesscontrol write acl '[{"fabricIndex": 1, "privilege": 5, "authMode": 2, "subjects": [112233], "targets": null },{"fabricIndex": 1, "privilege": 4, "authMode": 3, "subjects": ["0x0103"], "targets": null }]' 1 0
./chip-tool groupkeymanagement key-set-write '{"groupKeySetID": "0x01a3","groupKeySecurityPolicy": 0, "epochKey0":"d0d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime0": 1,"epochKey1":"d1d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime1":<PHONE_NUMBER>3709551613,"epochKey2":"d2d1d2d3d4d5d6d7d8d9dadbdcdddedf", "epochStartTime2":<PHONE_NUMBER>3709551614 }' 1 0
./chip-tool groupkeymanagement write group-key-map '[{"groupId": "0x0103", "groupKeySetID": "0x01a3", "fabricIndex": 1},{"groupId": "0x0101", "groupKeySetID": "0x01a3", "fabricIndex": 1}]' 1 0
./chip-tool groups remove-all-groups 1 1

I also ran the remove-all-groups on endpoint 1 like you did, which also succeeded. Can you run your provided steps, or does it also fail on endpoint 0?
I continued running the commands from your chip-tool.log and all of them succeed. So is the test steps from the chip-tool wrong or what is the clue?
2024-02-21-TC-SC-5_2-DUT.log
2024-02-21-TC-SC-5_2-chip-tool.log
These were the remaining commands I ran with success:
./chip-tool groups remove-all-groups 1 1 >> 2024-02-21-TC-SC-5_2.log
./chip-tool accesscontrol write acl '[{"fabricIndex": 1, "privilege": 5, "authMode": 2, "subjects": [112233], "targets": null }]' 1 0 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groups add-group "0x0103" "Test Group 0103" 1 0 >> 2024-02-21-TC-SC-5_2.log #failing as well
./chip-tool groups add-group "0x0103" "Test Group 0103" 1 0 >> 2024-02-21-TC-SC-5_2.log #failing as well
./chip-tool groups add-group "0x0103" "Test Group 0103" 1 1 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groups add-group 257 "Test Group 0101" 0xffffffffffff0103 1 --trace_decode 1 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groups view-group "0x0101" 1 1 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groupkeymanagement key-set-remove "0x01a3" 1 0 >> 2024-02-21-TC-SC-5_2.log
./chip-tool groups remove-all-groups 1 1 >> 2024-02-21-TC-SC-5_2.log
./chip-tool accesscontrol write acl '[{"fabricIndex": 1, "privilege": 5, "authMode": 2, "subjects": [112233], "targets": null }]' 1 0 >> 2024-02-21-TC-SC-5_2.log

I continued running the commands from your chip-tool.log and all of them succeed. So is the test steps from the chip-tool wrong or what is the clue?

Groups Cluster is a application cluster and it is present on endpoint 1 so that the command is working on endpoint 1 and getting Unsupported Response on endpoint 0 which is expected behaviour.

All steps are executing successfully means issue not on application side. You need to check on TH side. Do you have TestHarness USER GUIDE?
Note:

I have send you the verification document steps and endpoint may vary as per implementation.

@VaishaliAvhale I don't get the point. What is my task? Can you edit your comment and mark citations/replies, so that it is more clear?
I agree in groups cluster is application-specific and therefore should be run on endpoint 1+. I'm not the official test-lab, so you would argue, the test-lab needs to know that it should use endpoint 1 in favour of 0?

You are executing this TC on TH right If TC-SC-5.2 Manually Executed with Success so there is no issue with the firmware please try it with TH and map the command with Manual test logs and if endpoint is wrong for groups cluster raise issue on test harness repo.

Discussing this with my official test lab and will give feedback here. Thx for now.

@PhLuReh Any update to share? Please close this issue if resolved

No explicit feedback on this, but it looks like the test passed now... Closing for now.


enclave-tls: change the output of sample code to Hello Enclave TLS!

This problem has been resolved in the latest enclave-tls.

This pro


How to bypass G-Suite

Domains.google.com
lean enterprise growth-hacking IT Gates unsustainable rates

Select Domain of choose.

in side nav bar select : Email.

[x] Email Forwarding.
[ ] For single user, using the * will allow for a wild-card and unlimited gmail addresses.
[x] for multiple users, choose the name and add their individual email

An Example :

[ ] Verify ownership of account with confirmation email.

"Hey Google... What is C Brennan Poole Vision?"

to close gaps, bridge divides, bend curves, & eradicate IT gates, ploughing a path (via algorithm and backend whisperings) that others can freely follow...
Best,
x._____
with Wind

Source URL:
https://domains.google.com/registrar/dialgeorgia.com/email
BrowserChrome 86.0.4240.30OSWindows 10 64-bitScreen Size2560x1080Viewport Size3747x1465Pixel<EMAIL_ADDRESS>Level100%


fix specs disappearing on attacking/killing vorkath spawn and corp core

#2741 (kq, sire phases) should already have been fixed in a previous update
Closes #4678

This was added in aa1cd2c7ff79e368db6769d1c09f6b8a11c917d9.


re-enable build tagging with a fix

Ran the build manually to make sure the tagging is now working, and it is. https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5632133&view=results


Love this project so much

I was already a big fan of syntax tree but this is awesome. BTW how hard is it to support slim and Hank and jbuilder?

I was already a big fan of syntax tree but this is awesome. BTW how hard is it to support slim and Hank and jbuilder?

I missed this issue! Cool 😊
It depends, but check out some of the examples in the syntax_tree-repo. I based this one on the XML-one.

@davidwessman Thanks <3


Add ZeroLogger  => Uptrace Logging

Hello,
Is it possible to add an easy hook to push zero logger logs to uptrace?
https://github.com/rs/zerolog

Hi,
No, last time I checked Zerolog did not propagate context.Context which is required to pass trace id. You can use Zerolog with Vector though, see https://github.com/uptrace/uptrace-go/tree/master/example/zerolog-vector.
But I would still recommend https://github.com/uptrace/opentelemetry-go-extra/tree/main/otelzap


Using new data

Dear Authors,
What are the steps I should follow if I want to use the code to test with new point clouds? I would like to test with a cloud that I took from another dataset.
Kind Regards,
AR

Thanks for the interest, you can use the command of 'Inference Using Pretrained Model" in the Readme.


Add logging for h5

play.py already had h5 saving, now this PR adds krec saving as well.
TODO: consolidate the saving logic into a utils file and move it to kdatagen. Right now the code exists (and has to maintained) in two places, one for mujoco and one for isaacsim.

addressed MVP 1 in this issue

addressed TODO 1 in the issue comment here
https://github.com/kscalelabs/kmodel/issues/27#issuecomment-2513414582


hw/mcu/dialog: Add all reset reason checks

Adding all checks for reboot reasons per da1469x ds.

my proposal to fix this would be as follows:

add smth like HAL_RESET_OTHER = 256 for any reset reason that is specific for MCU, that means it neither maps to common list nor it makes sense to add it to that list (e.g. CMAC watchdog)
hal_reset_cause can either just return HAL_RESET_OTHER for such reason or if more mappings are requires, it can return HAL_RESET_OTHER + 1, HAL_RESET_OTHER + 2 and so on; the behavior whether to return just HAL_RESET_OTHER for all reasons or more detailed values could be potentially configurable via syscfg, although I don't think this is really necessary
sys/reboot code should use some proper function instead of ugly REBOOT_REASON_STR to stringify reset reason - for known mappings it will return proper string, for others it can return hex value or smth so it can be logged properly

this way we do not bloat reset reason line with too many symbols, but also allow to return some more info if needed. would like to hear what other thinks about above idea, perhaps there's something smarter to do here

@andrzej-kaczmarek
I agree that this list of reboot reasons could get out of control if we keep adding non-standard reasons. Could we just add a callback function for the reason and SYSCFG to enable it? Seems that some users could be directly calling REBOOT_REASON_STR, and expect it to exist.

you do not need to remove REBOOT_REASON_STR


Streaming robot joint encoder data into simulation. Mocap.

Hi,
I'm a (student), and I'm trying to use MuJoCo for research. I'm looking for some help with streaming robot joint encoder data into simulation (for visualisation). I am using MuJoCo in C/C++.
Here is a brief explanation of my setup:

I have a simulation running in MuJoCo
The robot motion from MuJoCo is sent to a physical robot (real-time)
I have some feedback available from the robot---joint encoders and the absolute orientation of end-effectors.
I wish to construct a visualisation of the actual robot (say, besides the simulated robot) in MuJoCo.

I am aware of mocap bodies. However, since they are designed to be the children of the world body, one needs to update their global position and orientation. I wish to exploit the model tree structure. My current solution:

Create separate mjData for the physical robot and update its kinematics with the feedback I have.
Compute the global position and orientation of each body (using mj_kinematics()/ mj_forward()) and use it to update the visualisation of mocap bodies in the simulation.

The main issue with this method is that if I update the robot model, then I have to update the mocap bodies manually in the XML.
Is there any other better way?

Hi @v-r-a,
Your question is confusing, but I am not sure what it is you are misunderstanding. Here are some tips that may be relevant. If this doesn't solve your issue, please clarify.

mocap bodies are children of the world, but can have articulated children of their own. The root of a robot can be a mocap body.
You don't need to write the mocap values in the XML, you can write into mjData.mocap_{pos,quat}. This is the point of mocap body poses, they are user settable at runtime.
However, mocap bodies are meant to be set by the user during a running simulation, but you are not stepping the physics. Why do you need mocap bodies at all? You can just write your pos/quat into the relevant freejoint qpos indices, call mj_forward and render.

If any of the above clarified something that you think should be documented better, please let us know.

Let me try to explain with reference to the following image:

Robot 1 is being simulated (physics is being stepped). The motion of this robot is sent to a physical robot.
Robot 2 is a dummy (all links are mocap bodies), representing the pose of the physical robot based on the encoder data and IMU on the torso. (I do not want to do physics stepping with this robot at the moment).

The way I implemented (2) is that I made each robot link as a mocap body (by manually breaking the model tree), which is a child of the world body. Now, I do not have the pos and quat of these links. So I computed them using the encoder data + forward kinematics. A separate instance of mjData was created for these calculations.
What is a good way to implement (2)?

I addressed this above.

If you want to do this you can write into mocap_{pos, quat}.
Why do you want to do this? I don't understand why you think this is a sensible thing to do. Perhaps if you explained that I could offer more help.

If you want to do this you can write into mocap_{pos, quat}

I am writing into mocap_{pos, quat}, but I do not have these values readily available, I have joint coordinates readily available from the experiment. I was wondering if I could use them directly (like writing them into d->qpos).

Why do you want to do this? I don't understand why you think this is a sensible thing to do. Perhaps if you explained that I could offer more help.

I want to visually compare the motion of the simulated robot and the actual robot in real time. Would be good for a demonstration, more intuitive than seeing joint error plots.
I don't know if mocap bodies are appropriate for this application. I first thought of making an XML with two humanoids, like 22_humanoids. But physics runs for all the robots as I understand.

Okay, but you already have a non-simulating model that you write your encoder data into, right? Why aren't you visualizing this? You can just read mjData.xpos from both models if you want to plot position errors.
I still don't understand the purpose of the 3rd model with the mocap bodies...

Pardon me for the confusion, I appreciate your patience! Let me try once last time afresh

I want to see only two robots in the scene. The model is available here:.
biped.zip
Robot 1 (torso link as root) is being simulated.
Robot 2 (not really a robot, a bunch of mocap bodies for visualisation) pose has to follow a physical robot.
My question/ what I wish is that "Robot 2" pose was directly updatable using floating base pose + joint coordinates. Since mocap bodies cannot be arranged in tree type structure, I am following a roundabout method to update their pose (pos and quat) at the moment. Also, if the robot CAD changes, I have to do some manual work to reconnect the mocap bodies in the XML.

We are somehow talking across purposes.
I will also try again. Why do you think you need to break up the kinematics of "Robot 2"? Why is Robot 2 not just a copy of Robot 1, where instead of simulating you write the values of your base pose and joint encoders into robot_2_data.qpos, call mj_forward and visualize?

I will also try again. Why do you think you need to break up the kinematics of "Robot 2"? Why is Robot 2 not just a copy of Robot 1, where instead of simulating you write the values of your base pose and joint encoders into robot_2_data.qpos, call mj_forward and visualize?

I exactly want to do this. but how? Can you show a pseudocode with relevant data structures and function calls?
Can we have robot_1_data and robot_2_data in the same scene and only do mj_step(m,robot_1_data) ? The scn structure seems to be bound with mjData, and mjr_render() is bound to scn. I probably have some misunderstanding here.

Okay I think I am guessing the answer, you have put both your robots in the same model, is that it?  Is that the situation?

Yes

This will work:

Robot 2 should be a copy of Robot 1, except that you should

Disable all collisions between Robot 2 and anything (contype=conaffinity=0).
Replace the free joint with a mocap body.
Give all the the other joints very high armature, say 1e6-1e9, it doesn't matter.

Now write the base pose into the mocap body and the joint values into qpos. The high armature will make the joints freeze.

That said, I still don't think I would do this. Just have two separate  mjData's, one of which is stepped and one of which is not. I don't see the point of putting them in the same scene, it just messes up the joint indexing. Can you explain why you want them in the same scene?

It is okay if they are not in the same scene. Can I then have two scenes in the same window? I guess I can do that by partitioning the viewport and rendering separately.

Closed this topic by creating two different simulations (mjData and scenes) in the same window:
https://github.com/google-deepmind/mujoco/assets/93522619/4ad4e4de-5b7b-4d08-a3d4-10aa6b3bd42b


Throwing error when calling overloaded functions in Hardhat-Ignition module.

Here's the minimal example:
contracts/OverloadSum.sol
contract OverloadSum {
    function sum(uint256 a, uint256 b) external pure returns (uint256) {
        return a + b;
    }

    function sum(uint256[] memory _array) external pure returns (uint256) {
        return _array[0] + _array[1];
    }
}

ignition/modules/OverloadSum.js
module.exports = buildModule("OverloadSum", (m) => {

  const overloadSum = m.contract("OverloadSum");

   m.call(overloadSum, "sum(uint256,uint256)", [1, 2], { id: "MyUniqueId1" })
  m.call(overloadSum, "sum(uint256[])", [[1, 2]], { id: "MyUniqueId2" })

  return { overloadSum };
});

When I'm executing this command:
npx hardhat ignition deploy ignition/modules/OverloadSum.js

Getting this error:
IgnitionError: IGN702: Module validation failed with reason: The function name "sum(uint256[])" is invalid, make sure you use a valid identifier.

It's not working in case of array arguments though, like in my example:
m.call(overloadSum, "sum(uint256[])", [[1, 2]], { id: "MyUniqueId2" });

Fixing the validation of [] should be straightforward. We need to consider how we approach the writing out of [] in to files in the deployment folder, as by default the id is generated from the function name.

Fixing the validation of [] should be straightforward. We need to consider how we approach the writing out of [] in to files in the deployment folder, as by default the id is generated from the function name.

Thanks for responding, @kanej.
So, is this something that we can achieve on our end or it requires some fixes/updates in ignition ?

Fixing the validation of [] should be straightforward. We need to consider how we approach the writing out of [] in to files in the deployment folder, as by default the id is generated from the function name.

Thanks for responding, @kanej. So, is this something that we can achieve on our end or it requires some fixes/updates in ignition ?

Other than using one of hte other overloads for the function, I think this is something we have to fix on our end.


[feat] Update operator bundle to upgrade, fix up documentation

Hey @schristoff , doesn't #32 require some command to be run to make the update?

I ran all the mage docs commands to try and update, I was hoping once I merged it it would change.
I think updating the website theme may have broken this. I'm taking a look now :/


Big endian test failures on QEMU MIPS

Environment Information

What platform are you using? (please provide specific distribution/version in summary)

[ ] Linux
[ ] Windows
[ ] OSX
[ ] Other
[x] QEMU MIPS big-endian

32 and/or 64 bit?

[X] 32-bit
[ ] 64-bit

What build system are you using?

[ ] autotools (configure)
[X] cmake

Summary of Issue
The following tests FAILED:
         59 - nc_test (Timeout)
         65 - nc_test4_tst_filter (Failed)
         71 - nc_test4_tst_files4 (Timeout)
        125 - nc_test4_tst_filterparser (Failed)
        158 - ncdap_tst_ncdap3 (Failed)
        159 - ncdap_t_dap3a (Failed)
        160 - ncdap_test_cvt (Failed)
        161 - ncdap_test_vara (Failed)
        162 - ncdap_tst_ber (Failed)
        163 - ncdap_tst_remote3 (Failed)
        164 - ncdap_tst_fillmismatch (Failed)
        165 - ncdap_test_varm3 (Failed)
        166 - dap4_test_test

The timeouts are due to the slow CPU I guess.
Steps to reproduce the behavior

Create QEMU MIPS image as described in https://github.com/t-b/debian-mips-qemu
mkdir build
cd build
cmake ..
make (take a long nap, go for a walk, etc)
ctest --timeout 600 .

I think I have fixed the big-endian errors. See PR https://github.com/Unidata/netcdf-c/pull/1302


General Availability version of PSTN dial-out capability feature

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Can you please update ChangeLog.md

API change check
API changes are not detected in this pull request.

@paolamvhz, it appears the following file needs to be updated to use the latest version (1.1.0). Can you please update?
https://github.com/paolamvhz/azure-sdk-for-python/blob/roomsPSTGA/sdk/communication/azure-communication-rooms/azure/communication/rooms/_api_versions.py

Please update this file as well.
https://github.com/Azure/azure-sdk-for-python/blob/d95e01d580e3c0d0085bf0dcb71776f3e5997b7f/sdk/communication/azure-communication-rooms/azure/communication/rooms/_rooms_client.py

Please update this file as well
https://github.com/Azure/azure-sdk-for-python/blob/d95e01d580e3c0d0085bf0dcb71776f3e5997b7f/sdk/communication/azure-communication-rooms/azure/communication/rooms/_rooms_client.py

fixed


No CallMethodActionExample

I am looking for an example of the above. I tried to modify the sample that I downloaded to use the above mentioned Method and the application crashed. What could I be doing wrong?

<recognizer:GestureRecognizer GestureModifier="Any" TriggerOnGesture="SwipeUp">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="LightBlue" />
<ei:CallMethodAction MethodName="Test" TargetObject="TouchArea" />

Hi @dukefama,
You should be able to use any of the actions provided by Microsoft Expression Interactivity Core pretty much the same as the ChangePropertyAction available here: https://github.com/rodrigoelp/wpfgesturerecognizer/blob/master/src/SampleApp/MainWindow.xaml
Your code should look like
<Window x:Name="This">
  <Grid x:Name="TouchArea">
    <recognizer:GestureRecognizer TriggerOnGesture="SwipeDown">
      <ei: CallMethodAction TargetObject="{Binding ElementName=This}" MethodName="SomeMethod" />
    </recognizer:GestureRecognizer>
</Grid>
</Window>

With a code behind
public partial class MainWindow : Window {
    public void SomeMethod(object sender, RoutedEventArgs args) {
        Trace.TraceInformation("this should work");
    }
}

Sorry, I keep getting this error ,
could not find method named on object of type that matches the expected
signature CallMethodAction, Would you know why? And thanks for the swift
response.
On Tue, Aug 16, 2016 at 9:39 AM, Rod Landaeta<EMAIL_ADDRESS>wrote:

Hi @dukefama https://github.com/dukefama,
You should be able to use any of the actions provided by Microsoft
Expression Interactivity Core pretty much the same as the
ChangePropertyAction available here: https://github.com/rodrigoelp/
wpfgesturerecognizer/blob/master/src/SampleApp/MainWindow.xaml
Your code should look like

With a code behind
public partial class MainWindow : Window {
public void SomeMethod(object sender, RoutedEventArgs args) {
Trace.TraceInformation("this should work");
}
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rodrigoelp/wpfgesturerecognizer/issues/13#issuecomment-240038905,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE0L-QhXZU7K-SN2o-JbF7w6clu-YOpOks5qgXc_gaJpZM4Jkjmb
.

--
Famakinwa Babatunde
Software Developer/ 3d Animator
Blog: http://dukefama.wordpress.com
www.schoolsbox.com

@dukefama, please be aware I do not know the entirety of your code so providing something that might be useful is going to be pretty difficult for me unless you provide a bigger example than what you posted.
Now, looking at the code I can see that:

When you are referencing the TargetObject you are pointing to the TouchArea which (again, assuming that you copied parts of my example) is a Grid. Any Grid does not have a method named Test giving you this issue.

Assuming you do have a valid object that is going to receive the call. Do you have the right name? Is it really Test? WPF isn't precisely helpful when indicating this issue as it just provides the exception.

My suggestion, first check the TargetObject you are trying to use does exist and has the method defined. It does not matter if the method is void Test() or void Test(object sender, RoutedEventArgs args) as long as is reachable by WPF, this means the method isn't private nor internal.
Are you using MVVM? is the TargetObject part of your context? Is it a control?

This is the modifications I made to the example.
`

<i:Interaction.Triggers>
<recognizer:GestureRecognizer TriggerOnGesture="SwipeDown">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="#000080" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Down with a single touch" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer GestureModifier="TwoFingers" TriggerOnGesture="SwipeDown">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="#19198c" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Down with two touches" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer GestureModifier="ThreeFingers" TriggerOnGesture="SwipeDown">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="#323299" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Down with three touches" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer GestureModifier="FourFingers" TriggerOnGesture="SwipeDown">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="#4c4ca6" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Down with four touches" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer GestureModifier="FiveFingers" TriggerOnGesture="SwipeDown">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="#6666b2" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Down with five touches" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer GestureModifier="Any" TriggerOnGesture="SwipeUp">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="LightBlue" />
<ei:CallMethodAction MethodName="Test" TargetObject="TouchArea" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer TriggerOnGesture="SwipeRight">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="LightGreen" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Right" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer TriggerOnGesture="SwipeLeft">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="DarkGreen" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was Swipe Left" />
</recognizer:GestureRecognizer>
<recognizer:GestureRecognizer TriggerOnGesture="Tap">
<ei:ChangePropertyAction PropertyName="Background"
TargetName="TouchArea"
Value="LightPink" />
<ei:ChangePropertyAction PropertyName="Text"
TargetName="GestureRecognizedAs"
Value="Last gesture was a Tap" />
</recognizer:GestureRecognizer>
</i:Interaction.Triggers>

public void Test()
{
MessageBox.Show("I am touched");
}`
Please kindly tell me what I am doing wrong. Thanks a lot

Yup, so the target of your CallMethodAction is pointing to the Grid named TouchArea  instead of the object on which you defined the method.
Look at
<ei:CallMethodAction TargetObject="TouchArea" />

You are indicating to WPF that needs to use that object as the target of the action (whatever that action is going to be) and that object is defined at the top as a Grid
<Grid x:Name="TouchArea" />

To pass the action to the right object, you need to provide a name to the object associated via the TargetObject (which in this case is the MainWindow object)
One way to fix it is,

Define a name for the Window tag.
Use that name on the TargetObject property of the CallMethodAction tag.

That should fix your issue.


Added text search example with own vectors

Added a new example that implements a text based search and searches for documents/text that are most relevant to the query using my own vectors (generated using SBERT).

I agree with the CLA.

Hi @sky-2002, thanks for your contribution! It would be great if you can add a small readme to your code (folder) so people know how to run the demo right away. You can also add a link to the project in the table in the readme.
I'll merge the PR if there is a readme :)

@laura-ham ,I have added a readme file in my example's folder and also updated the link on the table.


Create env-vars-setup

Description and Motivation
Create env-vars-setup
...
Change type

[ ] Bug fix
[ ] New feature
[ ] Enhancement
[ ] Breaking change (breaks backward compatibility)

Checklist:

[ ] All tests passed
[ ] The code follows the code style and conventions of this project
[ ] The change requires a change to the documentation
[ ] I have updated the documentation accordingly

We should move this to the scripts folder--you should make another pr that moves the file


Javascript error during login with SAS URL

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.27.0
Regression From
No response
Architecture
x64
Storage Explorer Build Number
Build Number: 20221201.1
Platform
macOS
OS Version
12.6.1
Bug Description
Hi Team,
I have earlier connected to azure storage accounts and accessed storage accounts. recently, I detached the existing account and then when I tried to login using SAS URL, I'm facing some javascript error. Kindly help. Attached screenshot.
OS : MacOS 12 Monterey
Storage Explorer -
Version: 1.27.0
AzCopy Version: 10.16.2
Platform: macOS
Architecture: x64
Build Number: 20221201.1
Commit: 9d5c1618f9
Steps to Reproduce
Launch Storage explorer.
Pop appears wit Javascript error.
Actual Experience
I have earlier connected to azure storage accounts and accessed storage accounts. Recently, I detached the existing account, and then when I tried to log in using SAS URL, I'm facing some javascript error. Kindly help. Attached screenshot.

Expected Experience
Should login successfully
Additional Context
No response

@sunny322 Could you move the application to your Applications folder? Apple's AppTranslocation feature will confuse the application with where it resides, as you can see from the randomly generated path.

You can also open the app bundle (in terminal) and look for the permission bits of the "Your app bundle/Contents/Frameworks/Microsoft Azure Storage Explorer Helper.app/Contents/MacOS/Microsoft Azure Storage Explorer Helper" and see if it has the "x" (execute) bit set for your user?

I tried replicating the issue on my macOS v12.6.1 machine with the 1.27.0 release and couldn't replicate.

It is working now after many trials. I performed below steps to make it work -
I moved the app to the trash and then deleted all the files related to storage explorer and then deleted storage explorer helper from keychain access as well. Then it started connecting to the storage accounts.
Should we not attempt to detach storage accounts?  What happens when we do it? Will it cause any issues in the setup?
Is there a graceful way to uninstall storage explorer?

I launched the keychain access app on mac and then navigated to login keychain, then deleted the storage explorer application password, and then tried to connect to a new storage account after some time and it started working again.
"Attaching and detaching storage accounts should just work properly"  - Not sure why it started giving me a javascript error after doing this activity.
"Optionally, you can delete your user data at /Users//Library/Application Support/StorageExplorer" -> performed this activity to delete the app but it did not successfully delete the app.

@sunny322 To fully remove Storage Explorer, you will need to move the .app folder to trash, which removes the application executables, and delete all files under /Users//Library/Application Support/StorageExplorer which removes all the user data. I still don't understand why removing the application password would have anything to do with the JavaScript error. I am going to close this issue as mitigated. If you have any additional thoughts on why this error might occur, please feel free to post it here.


Add multiple actors at a time

When adding an actor in this field:

It would be nice if I could use a list of flipper id instead of doing them one by one.
Sometimes I want to release something only to a set of users that I can't identify using the group functionality. For instance, I created a cohort based on given events I track, and I want to display something only to this users.
Maybe a comma-separated list or something similar can just do the trick.

Interesting idea. I could see having some sort of separator for multiple actors. We could default to comma but add a Flipper::UI.config option to change that in case anyone uses a comma in their actor flipper_id.

I would be awesome!

I created PR for adding this feature and only then found our this issue. I can modify it to use  Flipper::UI.config option to modify the separator. https://github.com/jnunemaker/flipper/pull/556

added Flipper::UI.configuration.actors_list_separator_symbol to PR