Exception has been thrown by the target of an invocation.

This issue was submitted by the add-in's exception handler.

Exception:
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.Runtime.InteropServices.COMException: Error HRESULT E_FAIL has been returned from a call to a COM component.
   at Microsoft.Office.Core.IRibbonUI.Invalidate()
   at XLToolboxForExcel.Ribbon._excelApp_SheetSelectionChange(Object Sh, Range Target)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Delegate.DynamicInvokeImpl(Object[] args)
   at System.Runtime.InteropServices.ComEventsMethod.DelegateWrapper.Invoke(Object[] args)
   at System.Runtime.InteropServices.ComEventsMethod.Invoke(Object[] args)
   at System.Runtime.InteropServices.ComEventsSink.System.Runtime.InteropServices.NativeMethods.IDispatch.Invoke(Int32 dispid, Guid& riid, Int32 lcid, INVOKEKIND wFlags, DISPPARAMS& pDispParams, IntPtr pvarResult, IntPtr pExcepInfo, IntPtr puArgErr)
   at Microsoft.Office.Interop.Excel.Sheets.get__Default(Object Index)
   at XLToolbox.WorkbookStorage.Store.get_StoreSheet() in XLToolbox\WorkbookStorage\Store.cs:line 130
   at XLToolbox.WorkbookStorage.Store.ReadFromWorksheet() in XLToolbox\WorkbookStorage\Store.cs:line 468
   at XLToolbox.WorkbookStorage.Store.set_Workbook(Workbook value) in XLToolbox\WorkbookStorage\Store.cs:line 63
   at XLToolbox.WorkbookStorage.Store..ctor(Workbook workbook) in XLToolbox\WorkbookStorage\Store.cs:line 204
   at XLToolbox.Export.Models.Preset.FromLastUsed(Workbook workbookContext) in XLToolbox\Export\Models\Preset.cs:line 54
   at XLToolbox.Export.QuickExporter.ExportSelection() in XLToolbox\Export\QuickExporter.cs:line 42
   at XLToolbox.Dispatcher.Execute(Command cmd) in XLToolbox\Dispatcher.cs:line 63

Message: Exception has been thrown by the target of an invocation.
Inner exception:
System.Runtime.InteropServices.COMException (0x80004005): Error HRESULT E_FAIL has been returned from a call to a COM component.
   at Microsoft.Office.Core.IRibbonUI.Invalidate()
   at XLToolboxForExcel.Ribbon._excelApp_SheetSelectionChange(Object Sh, Range Target)

Inner exception message: Error HRESULT E_FAIL has been returned from a call to a COM component.
Environment

Parameter
Value

Version
7.2.13

OS
Microsoft Windows NT 10.0.15063.0, 64-bit

Excel
2013 SP1 (15.0.4989), 32-bit

CLR
4.0.30319.42000, 32-bit

VSTO runtime
10.0.40820

ClickOnce
False

Bovender
<IP_ADDRESS>

FreeImage
3.16.0

Identifier
c2b4c91a25bcce4a8079b12f136e623ab76698514c733fcd7a0671e0065ab73d

Another report

This issue was submitted by the add-in's exception handler.

Exception:
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.Runtime.InteropServices.COMException: Error HRESULT E_FAIL has been returned from a call to a COM component.
   at Microsoft.Office.Core.IRibbonUI.Invalidate()
   at XLToolboxForExcel.Ribbon._excelApp_SheetSelectionChange(Object Sh, Range Target)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Delegate.DynamicInvokeImpl(Object[] args)
   at System.Runtime.InteropServices.ComEventsMethod.DelegateWrapper.Invoke(Object[] args)
   at System.Runtime.InteropServices.ComEventsMethod.Invoke(Object[] args)
   at System.Runtime.InteropServices.ComEventsSink.System.Runtime.InteropServices.NativeMethods.IDispatch.Invoke(Int32 dispid, Guid& riid, Int32 lcid, INVOKEKIND wFlags, DISPPARAMS& pDispParams, IntPtr pvarResult, IntPtr pExcepInfo, IntPtr puArgErr)
   at Microsoft.Office.Interop.Excel.Sheets.get__Default(Object Index)
   at XLToolbox.WorkbookStorage.Store.get_StoreSheet() in XLToolbox\WorkbookStorage\Store.cs:line 130
   at XLToolbox.WorkbookStorage.Store.ReadFromWorksheet() in XLToolbox\WorkbookStorage\Store.cs:line 468
   at XLToolbox.WorkbookStorage.Store.set_Workbook(Workbook value) in XLToolbox\WorkbookStorage\Store.cs:line 63
   at XLToolbox.WorkbookStorage.Store..ctor(Workbook workbook) in XLToolbox\WorkbookStorage\Store.cs:line 204
   at XLToolbox.Export.Models.Preset.FromLastUsed(Workbook workbookContext) in XLToolbox\Export\Models\Preset.cs:line 54
   at XLToolbox.Export.QuickExporter.ExportSelection() in XLToolbox\Export\QuickExporter.cs:line 42
   at XLToolbox.Dispatcher.Execute(Command cmd) in XLToolbox\Dispatcher.cs:line 63

Message: Exception has been thrown by the target of an invocation.
Inner exception:
System.Runtime.InteropServices.COMException (0x80004005): Error HRESULT E_FAIL has been returned from a call to a COM component.
   at Microsoft.Office.Core.IRibbonUI.Invalidate()
   at XLToolboxForExcel.Ribbon._excelApp_SheetSelectionChange(Object Sh, Range Target)

Inner exception message: Error HRESULT E_FAIL has been returned from a call to a COM component.
Environment

Parameter
Value

Version
7.2.13

OS
Microsoft Windows NT 10.0.16299.0, 64-bit

Excel
365 (16.0.9029), 32-bit

CLR
4.0.30319.42000, 32-bit

VSTO runtime
10.0.60301

ClickOnce
False

Bovender
<IP_ADDRESS>

FreeImage
3.16.0

Identifier
c2b4c91a25bcce4a8079b12f136e623ab76698514c733fcd7a0671e0065ab73d


Two new inscription editions and an attempt at translating them

Might have messed up the pull request again - another abortive test?

Dear Jonathan,
Sorry for those silly mistakes, and thank you for pointing out where they
were. I've just made a new commit for both files and have pushed it to the
origin from my desktop GitHub app, although I'm not sure if this has been
automatically merged with the original pull request. Just in case, I
thought I'd email you to let you know.
All best,
Noah
On Mon, 8 Mar 2021 at 21:19, Jonathan Prag @.***> wrote:

@JonPrag requested changes on this pull request.
Dear Noah
Absolutely the right idea and broadly correct - well done. The issues here
are almost all tiny peccadilloes (and all are noted in comments
individually). In writing code the precision is however important: from a
purely syntactical perspective in XML, the spaces must only be in the right
places inside the tags; values on attributes must be in "" not in ' '; and
a value on a ref needs the # on it. In the text, could you use u and v,
rathe rthan just v; and could you aim to close up all space, except between
individual words?
As I say, all very small things. Once the corrections are made commit them
and repeat the pull request process, which will simply add them tot he
existing pull request, and then I will approve and incorporate.
many thanks!
Jonathan
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589652119:

@@ -37,6 +37,10 @@

  <respStmt>

                 <name xml:id="system">system</name>

                 <resp>automated or batch processes</resp>

           </respStmt>

           <respStmt>

               <name xml:id='NCash' ref='https://orcid.org/0000-0002-8767-3368'>Noah Cashian</name>

"NCash" not 'NCash', and " " for the ref also
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589652524:

@@ -172,7 +176,8 @@

  <change when="2020-11-26" who="#SS">Simona Stoyanova restructured bibliography</change>

  <change when="2020-12-22" who="#system">Updated Zenodo DOI</change>

         	<change when="2021-01-19" who="#SS">renumbered files, uris and references</change>

          <change when='2021-03-04' who='#NCash'>Noah Cashian refined inscription edition and provided English translation</change>

again, must use " " not ' '
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589652685:

@@ -193,46 +198,38 @@

 </facsimile>

 <text>

" "
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589653412:

                   <ex>arcus</ex>

               </expan>

               Antonius<lb n="3"/>Syriacus vivus<lb n="4"/>hoc monumentum

               <lb n="5"/>

               <expan>

                   sib

                   <ex>i</ex>

               </expan>

               et suis

               <expan>

                   fec

                   <ex>it</ex>

               </expan>

               .

       <lb n="1"/>

       <expan>d<ex>iis</ex></expan>

normally dis not diis
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589657374:

                   <ex>i</ex>

               </expan>

               et suis

               <expan>

                   fec

                   <ex>it</ex>

               </expan>

               .

       <lb n="1"/>

       <expan>d<ex>iis</ex></expan>

           <expan>m<ex>anibvs</ex></expan>

               <expan>s<ex>acrum</ex></expan>

       <lb n="2"/>

       <expan>M<ex>arcvs</ex></expan>

           <g ref="#interpunct"/>

               Antonivs

I understand the thinking (there is only the letter 'v') in Latin, but it
is normal lexical practice to distinguish consonantal 'v' from vowel 'v',
and so to use 'v' for the consonantal letter and 'u' for the vowel (but
always to use V for both in majuscule, i.e. diplomatic editions).
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589658585:

               <expan>

                   fec

                   <ex>it</ex>

               </expan>

               .

       <lb n="1"/>

       <expan>d<ex>iis</ex></expan>

           <expan>m<ex>anibvs</ex></expan>

               <expan>s<ex>acrum</ex></expan>

       <lb n="2"/>

       <expan>M<ex>arcvs</ex></expan>

           <g ref="#interpunct"/>

               Antonivs

       <lb n="3"/>

       Syriacvs

           <g ref="#interpunct"/>

for those symbols for which we have the unicode character, we add the
symbol and place it within the g tags, so:
·
You can find a list of all the ones we so encode (so far - if you find
something new, tell me!), at:
https://github.com/ISicily/ISicily/blob/master/alists/charDecl.xml
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589659279:

       <expan>M<ex>arcvs</ex></expan>

           <g ref="#interpunct"/>

               Antonivs

       <lb n="3"/>

       Syriacvs

           <g ref="#interpunct"/>

               vivvs

       <lb n="4"/>

       hoc

           monvmentvm

       <lb n="5"/>

       <expan>sib<ex>i</ex></expan>

           et

               svis

                   <g ref="#interpunct"/>

                       <expan>fe<hi rend = 'small'>c</hi><ex>it</ex></expan>

 (i.e. close up the spaces except between the element
name and attribute and use " not ')
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589662573:

             </ab>

Sacred to the Manes of Marcus Antonius Syriacus, who while alive made this monument for himself and his relatives

Strictly speaking, can't be 'of marcus...', since he is in the nominative.
So, 'Sacred to the spirits of the underworld. Marcus Antonius Syriacus made
this monument for himself and his relatives while living'. I suppose if I
were being really picky, I'm not sure if we should prejudge that 'suis'
means just relatives, but not something to worry about here!
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589662647:

             </ab>

" " again
In inscriptions/ISic000329.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589663415:

           <g ref="#interpunct"/>

               Antonivs

       <lb n="3"/>

       Syriacvs

           <g ref="#interpunct"/>

               vivvs

       <lb n="4"/>

       hoc

           monvmentvm

       <lb n="5"/>

       <expan>sib<ex>i</ex></expan>

           et

               svis

                   <g ref="#interpunct"/>

                       <expan>fe<hi rend = 'small'>c</hi><ex>it</ex></expan>

                           <g ref="#leaf"/>

I fear I'm being super fussy, but could you close up the spacing in the
lines, so that there are no carriage returns/linebreaks in the text of any
individual line, and only a space between elements where there is a space
between two words?
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589667153:

             <ab>

               <lb n="1"/> Aeliae Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>coniugi

               <lb n="3"/>sancitissimae

               <lb n="4"/>Publius Aelius Augusti libertus

               <lb n="5"/><supplied reason="lost">N</supplied>igr<supplied reason="lost">ia</supplied>nus

               <lb n="1"/>

                   Aeliae

                       <g ref='#interpunct'/>

               Aug<supplied reason="lost">e</supplied>

indeed it's unclear, but what is hard to read in this copy is that CIL
proposes Aug[e] (there's a small italic e after the AVG), so second name. I
agree with your suggestion, and in many ways find it possibly more
attractive (but Augusti or Augustae libertae)
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589667486:

             <ab>

               <lb n="1"/> Aeliae Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>coniugi

               <lb n="3"/>sancitissimae

               <lb n="4"/>Publius Aelius Augusti libertus

               <lb n="5"/><supplied reason="lost">N</supplied>igr<supplied reason="lost">ia</supplied>nus

               <lb n="1"/>

                   Aeliae

                       <g ref='#interpunct'/>

               Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>

                   conivgi

again, coniugi rather than conivgi, please, and below 'Publius'
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589667613:

             <ab>

               <lb n="1"/> Aeliae Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>coniugi

               <lb n="3"/>sancitissimae

               <lb n="4"/>Publius Aelius Augusti libertus

               <lb n="5"/><supplied reason="lost">N</supplied>igr<supplied reason="lost">ia</supplied>nus

               <lb n="1"/>

                   Aeliae

                       <g ref='#interpunct'/>

               Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>

                   conivgi

               <lb n="3"/>

                   sancitissimae

spelling
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589668020:

               <lb n="3"/>sancitissimae

               <lb n="4"/>Publius Aelius Augusti libertus

               <lb n="5"/><supplied reason="lost">N</supplied>igr<supplied reason="lost">ia</supplied>nus

               <lb n="1"/>

                   Aeliae

                       <g ref='#interpunct'/>

               Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>

                   conivgi

               <lb n="3"/>

                   sancitissimae

               <lb n="4"/>

                   <expan>P<ex>vblivs</ex></expan>

                       <g ref='#interpunt'/>

                           Aelivs

                               <g ref='#interpunt'/>

interpunct, not interpunt; and as with the previous, add the mid-dot; and
" not ' (and u not v...)
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589746682:

               <lb n="4"/>Publius Aelius Augusti libertus

               <lb n="5"/><supplied reason="lost">N</supplied>igr<supplied reason="lost">ia</supplied>nus

               <lb n="1"/>

                   Aeliae

                       <g ref='#interpunct'/>

               Aug<supplied reason="lost">e</supplied>

               <lb n="2"/>

                   conivgi

               <lb n="3"/>

                   sancitissimae

               <lb n="4"/>

                   <expan>P<ex>vblivs</ex></expan>

                       <g ref='#interpunt'/>

                           Aelivs

                               <g ref='#interpunt'/>

                               <expan>A<ex>vgvsti</ex></expan>

I think it's Aug(usti)?
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589747830:

@@ -202,8 +220,10 @@

             <app loc="line 5"><note>Auria 1697: [..]IGR[..]NVS[..]; Gualtherus 1624: ΛIGRANIVS</note></app>

             </listApp>

Publius Aelius Nigrianus, freedman of the emperor, you enlarge (this) for his most sacred wife Aelia

surely a second name 'Auge' (Greek name):
http://clas-lgpn2.classics.ox.ac.uk/cgi-bin/lgpn_search.cgi?namenoaccents=ΑΥΓΗ
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589749007:

@@ -202,8 +220,10 @@

             <app loc="line 5"><note>Auria 1697: [..]IGR[..]NVS[..]; Gualtherus 1624: ΛIGRANIVS</note></app>

             </listApp>

Publius Aelius Nigrianus, freedman of the emperor, you enlarge (this) for his most sacred wife Aelia

chaste or virtuous (or pure), rather than 'sacred'.
In inscriptions/ISic000543.xml
https://github.com/ISicily/ISicily/pull/123#discussion_r589750458:

@@ -202,8 +220,10 @@

             <app loc="line 5"><note>Auria 1697: [..]IGR[..]NVS[..]; Gualtherus 1624: ΛIGRANIVS</note></app>

which leaves us with an interesting question, as to whether we think it's
Nigranius or Nigrianus; the latter is however better attested from a quick
look in Clauss Slaby.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ISicily/ISicily/pull/123#pullrequestreview-606586104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS36IXLJK77TXE3PFZ5M7M3TCU5M5ANCNFSM4YVERGHQ
.


Frame rate and rendering are messed up when rotating an iOS device while in XR mode

I believe this is a recent regression, and I've reproed it in the BabylonReactNative Playground app, but haven't tried in the BabylonNative Playground app. On an iOS device (also I've only tested in Release builds), if you enter XR mode, and then rotate the device, a couple weird behaviors happen:

The reported frame rate increases (from 60 to 72ish) but fluctuates some.
There are some weird frame hitches while moving the device around.

Sometimes rotating the device back to portrait mode returns rendering to a normal state, but not always.

This seems to be fixed by https://github.com/BabylonJS/BabylonNative/pull/591


git: change extraConfig from str to attrs

This makes it possible to configure git like this:
programs.git {
  enable = true;
  extraConfig = {
    pull = {
      rebase = true;
    };
  };
};

Instead of specifying the string like this:
programs.git {
  enable = true;
  extraConfig = ''
    [pull]
    rebase=true
  '';
};

Unfortunately, it doesn't allow to define sections in multiple modules, e.g. if it will be another module which will have programs.git.extraConfig.pull = { ff = true; }, then the final configuration will only contain one of the options:
[pull]
ff=1

At the same time, string value doesn't allow to extend individual sections either.
For now couldn't find the reason to use string value instead of attributes, correct me if I'm wrong.

I don't know why I didn't use an attribute set here to start with. I imagine it having to do with the point you make with options being overwritten. In general it shouldn't be too much of a problem. But I still think it is quite severe, in particular it is impossible to, e.g., say
programs.git.signing.key = "blahblah";
programs.git.extraConfig.commit.verbose = true;`

because programs.git.signing.key will write to the commit section of the INI file.
I think it should be possible to have extraConfig as an attribute set and still have the above case work by changing your PR as follows:
diff --git a/modules/programs/git.nix b/modules/programs/git.nix
index 88e0dd4..f1f6bb5 100644
--- a/modules/programs/git.nix
+++ b/modules/programs/git.nix
@@ -69,30 +69,43 @@ in
         default = {};
         description = "Additional configuration to add.";
       };
+
+      iniContent = mkOption {
+        internal = true;
+        type = types.attrsOf types.attrs;
+      };
     };
   };

   config = mkIf cfg.enable (
-    let
-      ini = {
-        user = {
+    mkMerge [
+      {
+        home.packages = [ cfg.package ];
+
+        programs.git.iniContent.user = {
           name = cfg.userName;
           email = cfg.userEmail;
-        } // optionalAttrs (cfg.signing != null) {
-          signingKey = cfg.signing.key;
         };
-      } // optionalAttrs (cfg.signing != null) {
-        commit.gpgSign = cfg.signing.signByDefault;
-        gpg.program = cfg.signing.gpgPath;
-      } // optionalAttrs (cfg.aliases != {}) {
-        alias = cfg.aliases;
-      } // cfg.extraConfig;
-    in
-      {
-        home.packages = [ cfg.package ];

         home.file.".gitconfig".text =
-          generators.toINI {} ini;
+          generators.toINI {} cfg.iniContent;
+      }
+
+      (mkIf (cfg.signing != null) {
+        programs.git.iniContent = {
+          user.signingKey = cfg.signing.key;
+          commit.gpgSign = cfg.signing.signByDefault;
+          gpg.program = cfg.signing.gpgPath;
+        };
+      })
+
+      (mkIf (cfg.aliases != {}) {
+        programs.git.iniContent.alias = cfg.aliases;
+      })
+
+      {
+        programs.git.iniContent = cfg.extraConfig;
       }
+    ]
   );
 }

If this is OK then the problem of backwards compatibility still remains. Unfortunately it wouldn't work very well to have condition = isString config.programs.git.extraConfig or something because it will error out during evaluation. What I can imagine is to now make a news entry, conditioned on extraConfig being non-empty, that say that the option in the near future will change type…

I changed the code according to your diff, and now it works as it should, thank you for the suggestion! As for backward compatibility, I managed to support both types using types.either. So now both old and new configuration styles are supported until the sting type option will be removed. I also added news entry with the corresponding condition.

Very nice! This looks good and works well for me (both with string and attrs).
Maybe change the commit message to "git: change extraConfig from lines to attrs"? Since it actually is of type lines :-)

I agree with suggestions, changed news entry and commit message.

Good work! Rebased to master in 0f096f9ad40f2fb055f4b696f16450d7487cd895. Many thanks :-)


Change contact form checkbox to LFPH instead of LF info

Requested by Dan: change the checkbox at the bottom of https://dev-lfph.pantheonsite.io/about/contact/ to say that you want to receive material from LFPH, not from the LF.

Submitted issue to DMO

this is done


Inquiry About Maintenance and Potential Collaboration

@Roemer
Discussed in https://github.com/FlaUI/FlaUI/discussions/661

Originally posted by jose-artur November 28, 2024
Hello @Roemer ,
My team and I are exploring automation tools, and we are particularly interested in using FlaUI for our project.
We  wanted to inquire about its current maintenance status. Additionally, we also want to understand if there are opportunities for collaboration.
Would it be possible to arrange a conversation with you to discuss this further? Our team is also open to contributing to the project, potentially as maintainers.
Looking forward to your response!

I just answered in the discussion. Somehow I don't get mail notifications for them. I'll close the issue here so we can stick to the discussion in https://github.com/FlaUI/FlaUI/discussions/661


connection to the guest agent was closed unexpectedlyunexpected EOF

Description
colima version && limactl --version && sw_vers
colima version 0.6.6
git commit: 9ed7f4337861931b4d0192ca5409683a4b7d1cdc

runtime: docker
arch: aarch64

limactl version HEAD-cac97ed

ProductName:		macOS
ProductVersion:		14.1.1

I've recently been getting an issue where docker commands start failing because the connection through the forwarded socket is getting closed. It appears that the ssh process dies for some reason. While the socket is working lsof shows the ssh process has it open. But when the socket becomes unresponsive, lsof no longer shows this and the process id no longer exists.
from ~/.colima/_lima/colima/ha.stderr.log :
{"error":"unexpected EOF","level":"warning","msg":"connection to the guest agent was closed unexpectedlyunexpected EOF","time":"2023-11-30T23:21:26-05:00"}

Maybe in this scenario the port forwards could at least be re initialized. But it'd be great if we could avoid the ssh process dying altogether.

With qemu or vz ?

@terev Will you be able to attach the log for this ?
We recently updated guest agent communication. Technically the connection should be re-established and this should not have any issues with ssh / port forwarding

@balajiv113 sure. just the host agent log?

Just reproduced again. But this time the guest agent isn't responding at all:
{"error":"unable to connect to guest agent via vsock port 2222","level":"warning","msg":"connection to the guest agent was closed unexpectedlyunable to connect to guest agent via vsock port 2222","time":"2023-12-01T01:51:03-05:00"}

@terev Are you able to ssh into guest ?? If so, try to collect logs from
sudo journalctl -u lima-guestagent.service -f

@balajiv113  Oh thanks! Yeah I was just able to get a log by tailing:
sudo journalctl -u lima-guestagent.service -f -n 100
Dec 01 07:22:57 colima systemd[1]: Started lima-guestagent.service - lima-guestagent.
Dec 01 07:22:57 colima lima-guestagent[1126]: time="2023-12-01T07:22:57Z" level=info msg="event tick: 3s"
Dec 01 07:22:58 colima lima-guestagent[1126]: time="2023-12-01T07:22:58Z" level=info msg="serving the guest agent on vsock port: 2222"
Dec 01 07:23:01 colima systemd[1]: Stopping lima-guestagent.service - lima-guestagent...
Dec 01 07:23:01 colima systemd[1]: lima-guestagent.service: Deactivated successfully.
Dec 01 07:23:01 colima systemd[1]: Stopped lima-guestagent.service - lima-guestagent.
Dec 01 07:23:01 colima systemd[1]: Started lima-guestagent.service - lima-guestagent.
Dec 01 07:23:01 colima lima-guestagent[1820]: time="2023-12-01T07:23:01Z" level=info msg="event tick: 3s"
Dec 01 07:23:01 colima lima-guestagent[1820]: time="2023-12-01T07:23:01Z" level=info msg="serving the guest agent on vsock port: 2222"
Dec 01 07:30:14 colima systemd[1]: lima-guestagent.service: A process of this unit has been killed by the OOM killer.
Dec 01 07:30:14 colima systemd[1]: lima-guestagent.service: Main process exited, code=killed, status=9/KILL
Dec 01 07:30:14 colima systemd[1]: lima-guestagent.service: Failed with result 'oom-kill'.
Dec 01 07:30:14 colima systemd[1]: lima-guestagent.service: Consumed 47.606s CPU time.
Dec 01 07:30:14 colima systemd[1]: lima-guestagent.service: Scheduled restart job, restart counter is at 1.
Dec 01 07:30:14 colima systemd[1]: Stopped lima-guestagent.service - lima-guestagent.
Dec 01 07:30:14 colima systemd[1]: lima-guestagent.service: Consumed 47.606s CPU time.
Dec 01 07:30:14 colima systemd[1]: Started lima-guestagent.service - lima-guestagent.
Dec 01 07:30:44 colima systemd[1]: lima-guestagent.service: Main process exited, code=killed, status=9/KILL
Dec 01 07:30:44 colima systemd[1]: lima-guestagent.service: Failed with result 'signal'.
Dec 01 07:30:44 colima systemd[1]: lima-guestagent.service: Consumed 16.029s CPU time.
Dec 01 07:30:44 colima systemd[1]: lima-guestagent.service: Scheduled restart job, restart counter is at 2.
Dec 01 07:30:45 colima systemd[1]: Stopped lima-guestagent.service - lima-guestagent.
Dec 01 07:30:45 colima systemd[1]: lima-guestagent.service: Consumed 16.029s CPU time.
Dec 01 07:30:49 colima systemd[1]: Started lima-guestagent.service - lima-guestagent.
FATA[0531] exit status 255

@balajiv113 Should we maybe adjust the OOM score for the guest agent?  It seems like a very important process and in this scenario it would have made more sense for the docker daemon to be killed. I'd be interested to see the memory consumption of this process though because its not expected to consume much memory right?

I'd be interested to see the memory consumption of this process

@terev I doubt if guest agent was consuming more memory i think its the other process due to which guest agent was closed

@balajiv113 yeah I watched top while running the build to reproduce, and the guest agent didn't take up much memory from what I saw.
maybe we should set the oom score adjustment to a negative number like docker does on the daemon https://docs.docker.com/config/containers/resource_constraints/#understand-the-risks-of-running-out-of-memory . this would it less likely it's the process killed. though I still didn't figure out why the container processes using the majority of memory weren't killed.

I am experiencing same / similar issue. It is pretty frequent. I am using lima 0.20.1 and qemu 8.2.1. My use case is multi-node Kubernetes cluster and did not have this issue previously with older lima + qemu combination.


fix defaults for wizard

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)
Closes #2474

/retest

/retest

/retest


zplug cannot load package containing pmodload from prezto

Describe the bug
In prezto, some modules contain dependencies when loading by using pmodload, such as module/git. For these modules, zlug only load the first one with pmodload and then stop load remaining code in init.zsh.
Of course, this problem can be fixed with use tag to load other files by myself, but I think this should be done by zplug automatically. I try to add use tag to load scripts by myself, but nothing  happended.
To Reproduce
source $ZPLUG_HOME/init.zsh
zplug "module/git",  from:prezto
zplug load --verbose

I get  Load "$ZPLUG_HOME/repos/sorin-ionescu/prezto/modules/helper/init.zsh" (module/git), but the alias.zsh in modules/git is not loaded which run in modules/git/init.zsh by source "${0:h}/alias.zsh".
And other modules like spectrum have the same problem.
Expected behavior
Load all dependencies and the module.
Env (please complete the following information):

zplug version: 2.4.2
zsh --version: zsh 5.7.1 (x86_64-pc-linux-gnu)
uname -a: Linux nixos 4.19.78 #1-NixOS SMP Mon Oct 7 16:57:29 UTC 2019 x86_64 GNU/Linux

Minimal zshrc (with less than 30 lines)
See To Reproduce

Now I deal with this problem by load whole prezto and use prezto to manage its modules.

If I'm not mistaken, same thing happens with plugins from OMZ - I had to remove them all after it started to take up to 30-40 seconds for zplug to load.


Pub 0.0.21 -> 0.0.22 failure

Running flutter update-packages (or anything else that invokes pub like flutter run) triggers:
Running 'flutter packages get' in automated_tests...
The pubspec for flutter 0.0.21 from sdk has version 0.0.22.
pub get failed (65)

That package's pubspec.yaml says
name: flutter_automated_tests
dependencies:
  flutter:
    sdk: flutter
  flutter_test:
    sdk: flutter
flutter:
  uses-material-design: true

The pubspec.lock says
...
  flutter:
    description: flutter
    source: sdk
    version: "0.0.21"
...

Removing the like from pubspec.lock fixes it.
[✓] Flutter (on Mac OS, channel master)
    • Flutter at /Users/xster/development/flutter
    • Framework revision 72fa281f12 (48 minutes ago), 2017-02-22 11:32:22
    • Engine revision ab09530927
    • Tools Dart version 1.23.0-dev.0.0

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.2)
    • Android SDK at /Users/xster/Library/Android/sdk
    • Platform android-25, build-tools 25.0.2

[✓] iOS toolchain - develop for iOS devices (Xcode 8.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.2.1, Build version 8C1002
    • ios-deploy 1.9.1

[✓] Android Studio (version 2.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Gradle version 2.14.1

[✓] IntelliJ IDEA Community Edition (version 2016.3.4)
    • Dart plugin version 163.13137
    • Flutter plugin version 0.1.10

[✓] Connected devices
    • Nexus 5 • 031462ee213e73b9 • android-arm • Android 6.0.1 (API 23)

You can work around this locally by running flutter packages update.  We should probably file a bug against pub to have looser locking behavior for sdk sources.

I just ran into the same issue, and flutter packages upgrade (not update, BTW) solved the problem.

Oops, you're right./

FYI @munificent

flutter packages upgrade solves it partially. There's a bunch of flutter projects in the git root directory. One has to go to each one and flutter packages upgrade. Probably less of an issue for external users that just flutter creates one project.

For flutter hackers there is a default-hidden command to upgrade everything flutter update-packages --upgrade

That worked. Thanks!
Perhaps the AI here is just to automate and/or document a way to update when the flutter major/minor version changes? Or would this be handled in pub?

We should make this sort of change smoother, either in pub or in the flutter tool.  Let's start by talking with @munificent about whether he thinks the change should be in pub.

@munificent The underlying issue is as follows:

My package has an SDK dependency whose version gets written to the pubspec.lock file.
I update my SDK to a version that has a newer version of that package.
pub get gives me an error because it can't give me the version of the package that my pubspec.lock files says I should get (because it only has one choice---the one version in the SDK that doesn't match the pubspec.lock file).

@nex3 knows more about this than I do.
I think it's probably reasonable for pub get to unlock the SDK dependency if it doesn't match in the same way that it unlocks regular dependencies if you change the pubspec and run pub get again.

Yeah, this looks like a bug to me. Can you file an issue against pub?

Sure thing.

Done: https://github.com/dart-lang/pub/issues/1519

I think we're going to need to either work around this somehow, or announce to users.  @Sfshaza also hit this this evening and was confused.

What workflow are people using when they run into this issue?  I understand how people on the Flutter team hit this issue because they manage their flutter.git directly, but if you follow our documented workflow and use flutter upgrade to update your Flutter SDK, you shouldn't hit this issue.

@abarth that seems correct.  I just tried:
cd /src/flutter
git checkout -b old_version f9c2d7d9aa7af787ab06e44b5c507ac5ddfbf654
git branch --set-upstream-to=upstream/master
cd /tmp
flutter create foo
cd /tmp/foo
flutter packages get
flutter upgrade

And it successfully completed. So yes, this may only be an issue for flutter team members.

People building flutter projects in fuchsia environment are all affected, since we don't use the flutter upgrade command. The flutter SDK is updated via jiri update command, which checks out the HEAD commit from GitHub.
Also, even when following the documented workflow using flutter upgrade, this issue still applies when multiple flutter projects are sharing the same flutter SDK installation.
(running flutter upgrade from one project, and then running flutter packages get from another project which already had pubspec.lock file would show the same error.)

Good point.

This is perhaps a non-issue for external users. I may have triggered it by git pulling upstream master directly.

This was fixed in pub.


[Bug]: ODC 4.3.0 would fail to start in client mode

ODC version
4.3.0
OB version

What happened?
ODC 4.3.0 would fail to start in client mode.

What did you expect to happen?
Start normally.
How can we reproduce it (as minimally and precisely as possible)?
Clear ODC completely, including h2 db files. Then install ODC 4.3.0.
Anything else we need to know?
No response

this issue is involved by this pr #2699


feature: allow extending auto triggers

Did you check the docs?

[X] I have read all the which-key.nvim docs

Is your feature request related to a problem? Please describe.
I have the following keymap
vim.keymap.set({'c', 'i'}, '<D-r>', '<C-r>')

I would like <D-r> to show the Registers plugin like it happens when using <C-r> in command/insert mode with the default configuration.
Describe the solution you'd like
A way to add <D-r> to the default "auto" triggers in command and insert modes.
Describe alternatives you've considered

Setting triggers={"<D-r>"} doesn't work. It doesn't show Registers in insert/command mode.
If I try to use wk.register(mappings, opts), I can't use the provided registers plugin. As far as I can tell, I would need to reimplement the plugin externally.

Additional context
No response

What you could do, is before running which-key setup, change the value of actions here:
https://github.com/folke/which-key.nvim/blob/main/lua/which-key/plugins/registers.lua
so require("which-key.plugins.registers").actions

Thank you very much!
I'm new to Lua and I didn't think require worked like this. The following when loading with Lazy is working for me:
  {
    'folke/which-key.nvim',
    opts = {
      triggers_nowait = { "<d-r>" }
    },
    config = function()
      local actions = require("which-key.plugins.registers").actions
      table.insert(actions, { trigger = "<d-r>", mode = "c" })
      table.insert(actions, { trigger = "<d-r>", mode = "i" })

      require("which-key").setup()
    end,
  },

Hey @santileortiz
I'm trying to do something similar where I remapped <c-r> to <C-G>u<c-r>
My config is the following
-- Lazy stuff
  {
    'folke/which-key.nvim',
    config = function()
      local register_actions = require("which-key.plugins.registers").actions
      table.insert(register_actions, { trigger = "<c-r>", mode = "i" })
      table.insert(register_actions, { trigger = "<C-G>u<c-r>", mode = "i" })

      require("which-key").setup()
    end
  }

--
-- ...
--

-- Does not work either
-- Remap Ctrl R to create undo point
-- vim.keymap.set('i', '<c-r>', '<C-G>u<c-r>', { noremap = true, silent = false })

-- Using which key
local wk = require("which-key")

wk.register({
	["<c-r>"] = { "<C-G>u<c-r>", "Paste", noremap = true }
}, { mode = "i" })

Do you notice something different, or point me to the right direction?


StdlibStubs: Favor swift::Lazy over static local initialization.

To minimize code size and VM live set, we try to funnel all one-time initialization through swift_once instead of mixing it with the C++ runtime's support for lazy static initialization.

@swift-ci Please test and merge

@shahmishal The OSX bot seems to be having some SDK-related problems; is this a known issue?
https://ci.swift.org/job/swift-PR-osx/2240/consoleFull#-1201360137c38edd5a-00f9-40a2-abac-1db59f7a90ca

@swift-ci Please test and merge

@jckarter Yes, this was failing on master branch after updating to new Xcode 8 Beta 2. Fix has been applied to master branch now.

@swift-ci Please test Linux

@swift-ci Please test Linux


Reward programs guides

Thanks @fboucquez for the review, but this is still a draft PR, bound to be heavily edited :)

Under section Super-node program.requirements Bonded Deposit defines almost imperceptibly the word tier.
The next section, Rewards, explains them breaking them down in 'tiers', assuming a knowledge of what tier means.
I suggest a hyperlink to its definition for those stepping directly into the rewards section.

I do not understand what is the part that is obscure. First it talks about "tiers" and says (see below), so you know that they will be defined later. Then tiers are defined as "Supernodes are divided in 3 tiers according to their staked symbol.xym: More than 1 million, more than 2 million and more than 3 million."
What exactly would you change?

Closing this PR. It has been merged with #581 into #586. Both PR were related and it was hard to review them separately.


Custom Cosmetic not accurate in-game to unity

as you can see in unity it looks how I want it.

But in-game it's way off and I have no idea how to fix it....

I have tried literally everything from blender to unity and it just refuses to rotate....

I have tried everything in blender and unity it just refuses to rotate

Are you moving, scaling, or rotating the gameobjects titled “DO NOT MOVE”?

Are you moving, scaling, or rotating the gameobjects titled “DO NOT MOVE”?

Yes, I did find a a fix by adding a rotational constraint to the model

Are you moving, scaling, or rotating the gameobjects titled “DO NOT MOVE”?

No I did the model that was placed there underneath that like the tutorial says, but I found a fix by adding a rotational constraint to the model

may i know which moment to put the rotational contraint? (blender or unity) and which model? (cosmetic or scavenger model)


Proxy: produce test requests based on the API description

User story
With one command, I’d like to validate status codes and response schemas (from an actual server implementation) against all the endpoints and operations in an OpenAPI description.
Problem
To test the consistency of a server implementation with its API description (using the validation proxy), users must decide which requests to make on a per-API basis and write individual scripts. It would be nice to provide a suite of requests to send through the validation proxy based on the API description.
Solution
To achieve this, we need to parse the OpenAPI description, gather the endpoints and operations, and make requests (both valid and invalid) to those endpoints with different parameters, request bodies, etc. through the validation proxy.
Through this we will, with one command, validate an API against its OpenAPI description and determine if the server-side implementation and/or the OpenAPI description need to be updated.
My team is currently researching tools that may be helpful in pursuit of this goal, and we may want to contribute this to Prism over the next few months. Is the Prism community interested in this feature, and will it accept our contribution?
Follow-up
This feature would be much improved (for my team’s purposes) if the validation proxy does not log errors for requests. We want the validation proxy to pass the request to the server as-is and validate the response. Would it be simple to disable request validation in the validation proxy?

Hey,
thanks for the message, super appreciated. This is a feature that has been popping up now and then by our users, which is a little bit what Dredd is doing (throwing request at your entire specification looking for inconsistencies).
I am not opposed to the feature (this is actually something that Prism v2 supports with the help of Stoplight's platform through Scenarios) but it must planned carefully (gathering  the requirements, understanding the use cases).
Just to give you an example, here are some of the points that we should think about:

How should Prism go through the operations?

By the way they appear in the document?
What if the order is not representative of the real operation order?
Shall we use a configuration file?

Should Prism store some intermediate state to use in the successive calls, or shall we assume that the server will figure out how to respond correctly to calls that might not make sense?

Maybe Prism should be programmable with an hook model like the same that Dredd is currently using?

These are just some of the things that need to be sorted out, but likely there are way more.
I think the first step would be for you to write your requirements, what are you trying to solve, and go from there.
Cheers!

Vincent,
I appreciate the quick and thorough response!
Dredd is something we are investigating as well. Dredd seems to align directly with our use case but may require a similar amount of effort to provide OpenAPI 3 support.
I am leaning toward Prism because of the excellent organization and documentation of the project, the interesting opportunity for live request validation, and communication (for example, your response on this issue).
We will take some time to address the points you raised and provide another response here.

but may require a similar amount of effort to provide OpenAPI 3 support.

Dredd supports OpenAPI 3.0! Although the original author has left Apiary/Oracle (I used to work there, by the way) I still think it has a decent level of support and at least bug fixes. I'd give it a try.

I am leaning toward Prism because of the excellent organization and documentation of the project, the interesting opportunity for live request validation, and communication (for example, your response on this issue).

Aha thanks, I appreciate. We try to do our best.

This is something we've discussed, as removing the need for people to need tocreate scenarios was part of the plan for v3.0, which we've started with contract testing and hope to continue with a few other options.
We'll need to plan this one out internally, so leave it with us. If anyone is looking at this thinking "I too would like to see this" I'd ask you to use the 👍 button up top to avoid bumping it, as we'll get to it when our roadmap cools off a little. We've got some huuuuuuge stuff on our plate for the next month or two. 🥴

I've spent a lot of time thinking about this one. I¡m glad we focused on making mocking and proxying the most effective we could, and it's pretty great that the community came to us with a suggestion of how to mix mocking and proxying in the easiest most standard way possible. That was one last hurdle for us to make Prism be a cohesive thing which does mocking and proxying with a bit of contract testing in there, and all works in a not "loads of things lumped in together" kinda way.
So now, let's focus back on the idea of having a test runner in Prism, which makes simple requests to an API instance similar to how the proxy works, then spits out errors when it finds them much like the SL-Violations currently do, but with an interface closer to Dredd.
I see it come up now and then like in this thread https://apisyouwonthate.slack.com/archives/C6EEULS87/p1599554457100700 where people are looking to have a brand new test suite that they dont have to spend much time making. I personally prefer to mix contract testing into the existing test suite when there is one, but I do understand the use case of improving a legacy API, having some sort of test runner to go through the docs, create example requests, and see what happens is at least some kind of testing, because more than just contract violations will appear, maybe some 500's escape, syntax errors, or even authentication errors!
Even if it's just a stepping stone towards eventually getting that codebase under control and adding a real test suite, having a test runner you don't have to create, or "keep in sync" is pretty handy in certain situations.
Is there anyone out there in open-source land who'd be interested in taking this on? We can work out a tree related bribe no doubt.


bug 1292937 - upgrade pip in docker

Coverage remained the same at 86.098% when pulling 343e02052d26c943dc69459c7cdb7764ae57305b on peterbe:bug-1292937-upgrade-pip-in-docker into 43d811cb7349027a3fce61796e1f642496f13127 on mozilla:master.


Fix wait for thread not started

At least the one I could find. Noise and AudioDriverDummy was happening for me on Win11, a couple where reported and iOS didn't match the android counterpart.
Fix https://github.com/godotengine/godot/issues/76456

The CI still has a few (not sure how many are actually duplicates):

https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334013?pr=76511#step:9:30
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334013?pr=76511#step:9:901
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334013?pr=76511#step:12:24
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334013?pr=76511#step:13:29
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334133?pr=76511#step:11:207
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334133?pr=76511#step:15:53
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334133?pr=76511#step:15:143
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334133?pr=76511#step:16:123
https://github.com/godotengine/godot/actions/runs/4822242570/jobs/8589334133?pr=76511#step:17:32957

Found a few more. Audio driver and a couple of manager that have a finish function that could potentially be called without a init (which happens in a few tests). I can't run all the same configs as the CI, so we'll see if I missed some more :)

Thanks!


task: improve error logs

Description
Send any beforeTransactionError error data to embed-ui, so it can be exposed in the onEvent - transactionFailed log info.
Ticket: https://gr4vy.atlassian.net/browse/TA-7168
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own changes
[x] I have run yarn lint to make sure my changes pass all tests
[x] I have run yarn test to make sure my changes pass all linters
[x] I have pulled the latest changes from the upstream main branch
[ ] I have tested both the react and the CDN versions on local and integration environments
[x] I have added the necessary labels to this PR in case a new release needs to be published after merging into main (e.g. release and patch)

Contribution guidelines
For contribution guidelines, styleguide, and other helpful information please
see the CONTRIBUTING.md file in the root of this project.

:rocket: PR was released in v2.27.1 :rocket:


feat: Adopts more Vue-specific features

Howdy.
Last weekend I had a closer look at this example and noticed that it does not use some of Vue's useful features. This PR changes the sample project so that it makes more use of Vue-specific features. For example:
TodoList is not rendering the TodoItems itself. Instead the consumer of TodoList has to render the item. TodoList merely provides the todo item which has to be rendered via scoped slots. This allows the consumer to have full control over the todo list item which comes in handy later when deletion and editing of todo items come into play:
<TodoList :todos="doneTodos" @selectionchange="$event.done = false">
  <template #todo="{todo}">
    <TodoItem
      :todo="todo"
      @delete="handleRemove(todo)"
      @edit="handleEdit(todo)"
    />
  </template>
</TodoList>

As you can see: Now the consumer has the actual todo within the same scope and thus can use this to register for delete- and edit-events. This means that the deleted/edited todo-item has no longer to be determined by looking it up in the local state. It is just there.
TodoList still emits a selectionchange-event but it includes the todo in question in the event's payload. This allows the consumer (App.vue) to react on selection-changes by simply accessing the payload via $event. In the snipped above you can see that $event.done is set to false when the selection changes.
Also, App no longer has two different todo-arrays. Instead there is one list called allTodos. Each todo has an additional property called done. Then there are two computed props:

todos: Contains all todos where done = false
doneTodos: Contains all todos where done = true

This simplifies the lookup of todos. Only one list has to be consulted in order to find a todo by a given id.
You can simply move a todo from one computed list to another list by changing it's done-prop.
I have also removed most refs, since this is usually only something that should be used as a last resort in Vue.
I have removed most refs by using v-bind and v-on.
A word of caution:
The sample no longer works properly. You will notice that you sometimes have to click twice in order to move todos.
Also you will notice errors beeing logged.
I have noticed that the ui5-list does not emit selectionChange when something is deselected. Which might explain the need for multiple clicks.
Also I have noticed that there was seemingly a bug in the ui5 text area which was fixed just recently.
It may also very well be that I simply introduced additional bugs. :D
If you think the merits of this PR are good I suggest we work together on the issues this PR has. I would love to know why the sample does not behave as nicely as before my changes. So do you have time to go through my changes with me together? I am sure all issues could be fixed or somehow explained by working together. :)
~ Chris

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

Hi Chris,
thanks for the PR, your experience is much appreciated. Let's work on getting the functionality stable, other than that your proposal looks much better than the initial sample.
Based on your description, we'll have a look to see what are the issues with the web components. Feel free to also open an issue there directly, if you find the APIs or events lacking for the improved Vue consumption code.
Thanks,
Peter


fix: screenshot - different Key for different screens

Impacted files:

app_test.dart: added the "reinvention" page; set different "next" keys; handled tooltip on KP pages
knowledge_panel_page_template.dart: specific "next" keys; added "tooltip" key
next_button.dart: new nextKey parameter
onboarding_bottom_bar.dart: added nextKey parameter used during screenshot generation
preferences_page.dart: specific "next" key
reinvention_page.dart: specific "next" key
sample_eco_card_page.dart: specific "next" key
sample_health_card_page.dart: specific "next" key
scan_example.dart: specific "next" key
welcome_page.dart: specific "next" key

What

Now there's a different Key for all screenshot screens: no more ambiguity.

Screenshots
This is the last screenshot:

Little question: should we show the tooltip for the screenshots?

Codecov Report

Merging #2798 (307af11) into develop (2ea0da3) will decrease coverage by 1.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2798      +/-   ##
==========================================
- Coverage     8.86%   6.99%   -1.87%
==========================================
  Files          161     218      +57
  Lines         6623   10723    +4100
==========================================
+ Hits           587     750     +163
- Misses        6036    9973    +3937

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
8.73% <0.00%> (-23.57%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
62.26% <0.00%> (-20.72%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
15.11% <0.00%> (-19.10%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
21.68% <0.00%> (-9.75%)
:arrow_down:

packages/smooth_app/lib/main.dart
14.65% <0.00%> (-3.24%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

...ackages/smooth_app/lib/pages/scan/scan_header.dart
2.50% <0.00%> (-2.27%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

... and 237 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Restore the SQL Server 2019 notebook download

This restores the SQL 19 notebook download related to https://github.com/microsoft/azuredatastudio/issues/20947.  The custom gulp task was failing with latest build lib updates from vscode merge.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.005%) to 41.959%

Files with Coverage Reduction
New Missed Lines
%

extensions/machine-learning/src/common/processService.ts
2
81.48%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
52.04%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
28860

Relevant Lines:
63888

💛  - Coveralls

@caohai will send a updated PR that resolved the CI breaks later today.


renderer: rewrite a bit the shader selection

use smaller stage struct
prevent the engine to use the PBR shader when it is disabled (do not crash)
improve the wording in various places

This is a commit that was sitting on my hard drive for a while, but it now also fixes a bug revealed when testing:

https://github.com/DaemonEngine/Daemon/pull/1073

The bug was that the engine was trying to use the PBR shader varient on PBR surfaces when r_physicalMap was disabled.


gnuradio3_8.pkgs.osmosdr: reduce closure size in total by ~300MB

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@SuperSandro2000 what builds did you test with this changes?

gnuradio3_8.pkgs.osmosdr

Error: detected mismatched Qt dependencies:

What package is the source of this error?


cmd/cgo: does not compile for iOS simulator arm64

What version of Go are you using (go version)?

$ go version
go version go1.19.3 darwin/amd64

Does this issue reproduce with the latest release?
I am using the latest release
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/darioklepoch/Library/Caches/go-build"
GOENV="/Users/darioklepoch/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/darioklepoch/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/darioklepoch/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.19.3"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/d8/p77zrhr12g1_xc5y4bl_0rmh0000gn/T/go-build10530953=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I am trying to compile a go project to iOS c libraries and then create a xcframework out of those libraries.
This works great for iOS arm64 and simulator x86. However, when I try to compile for simulator arm64 I encounter problems.
The library built for arm64 simulator seems to not target the simulator but a real iOS device. This is problematic for using the simulator on new apple silicone devices.

What did you expect to see?
I expected the library to be built for an arm64 simulator.
What did you see instead?
The library was built for an arm64 iOS device.

iOS simulator on arm64 is not a supported platform yet. You're welcome to file a feature request (or make this issue to), or even better, work on it to make it possible. Thanks.

Thank you for the quick response. As I do not have any experience implementing such a feature I cannot really help, I will wait for an implementation then.

What version of Go are you using (go version)?
$ go version
go version go1.19.3 darwin/amd64
Does this issue reproduce with the latest release?
I am using the latest release
What operating system and processor architecture are you using (go env)?
go env Output
What did you do?
I am trying to compile a go project to iOS c libraries and then create a xcframework out of those libraries. This works great for iOS arm64 and simulator x86. However, when I try to compile for simulator arm64 I encounter problems. The library built for arm64 simulator seems to not target the simulator but a real iOS device. This is problematic for using the simulator on new apple silicone devices.

What did you expect to see?
I expected the library to be built for an arm64 simulator.
What did you see instead?
The library was built for an arm64 iOS device.

I also encountered this problem, and now there is no iphone as a debugging device, the project cannot go on.

Is there any news about Silicon Simulator support? I was able to create a framework from Go code and I have a weird problem with the Simulator. Sometimes it just hangs on the start and the best that I achieved is that the App runs, but when I get to the point that the Go code is being called, it hangs again, but if I pause from Xcode the code and resume it, it snaps out and it continue.
All works ok if I edit the scheme and I disable "Debug executable", but as a developer I think you can imagine that I can't work without debugging.

Is there any news about Silicon Simulator support? I was able to create a framework from Go code and I have a weird problem with the Simulator. Sometimes it just hangs on the start and the best that I achieved is that the App runs, but when I get to the point that the Go code is being called, it hangs again, but if I pause from Xcode the code and resume it, it snaps out and it continue.
All works ok if I edit the scheme and I disable "Debug executable", but as a developer I think you can imagine that I can't work without debugging.

Do you have a solution to this problem?

I would like to add a vote to bump this feature to be added in the dev queue. Development on Mac computers becomes fast an Apple silicone exclusive.

I found a fix to build with correct (arm64) simulator runtime on Apple Silicon
in your clangwrap.sh add
if [ "$SDK" = "iphoneos" ]; then
  export TARGET="-target $CARCH-apple-ios$MIN_VERSION"
elif [ "$SDK" = "iphonesimulator" ]; then
  export TARGET="-target $CARCH-apple-ios$MIN_VERSION-simulator"
fi

and include the $TARGET part of your xcrun like below
exec $CLANG -arch $CARCH $TARGET -isysroot $SDK_PATH -mios-version-min=10.0 "$@"

you can arrive correct $MIN_VERSION from your simulator sdk by running xcrun --sdk iphonesimulator --show-sdk-path

I found a fix to build with correct (arm64) simulator runtime on Apple Silicon
in your clangwrap.sh add
if [ "$SDK" = "iphoneos" ]; then
  export TARGET="-target $CARCH-apple-ios$MIN_VERSION"
elif [ "$SDK" = "iphonesimulator" ]; then
  export TARGET="-target $CARCH-apple-ios$MIN_VERSION-simulator"
fi

and include the $TARGET part of your xcrun like below
exec $CLANG -arch $CARCH $TARGET -isysroot $SDK_PATH -mios-version-min=10.0 "$@"

you can arrive correct $MIN_VERSION from your simulator sdk by running xcrun --sdk iphonesimulator --show-sdk-path

Hi, this workaround worked for me to be able to create xcframework that does compiles on M1/M2 simulators. But I still have the issue with app handling on app splash screen (probably since my JSI install method happens very early in the app lifecycle and go being called).
Gomobile - this project seems to support m1/m2 simulators for a while now (according to their github discussions), but they bind the files for Java/Swift which I'm not interested in (need the native compilation).
Can somewhat do the workaround with killing xcode/etc, but would be nive if anyone has suggestion how to fix that. Thx!

Still hanging on the splashscreen too

@albertoAround try this. I was facing similar issue for my RN app and this has solved the problem.
 <EnvironmentVariables>
         <EnvironmentVariable
            key = "GODEBUG"
            value = "asyncpreemptoff=1"
            isEnabled = "YES">
         </EnvironmentVariable>
      </EnvironmentVariables>

Make sure you are setting this only for Debug scheme


Custom claims NoSuchMethodError

[REQUIRED] Step 2: Describe your environment

Library version: 5.6.0
Firebase Product: auth

[REQUIRED] Step 3: Describe the problem
I just upgraded to 5.6.0 version to try custom claims and now I can't get user because I get NoSuchMethodError
Exception in thread "firebase-default-0" java.lang.NoSuchMethodError: org.json.JSONObject.toMap()Ljava/util/Map;
	at com.google.firebase.auth.UserRecord.<init>(UserRecord.java:85)
	at com.google.firebase.auth.FirebaseUserManager.getUserById(FirebaseUserManager.java:112)
	at com.google.firebase.auth.FirebaseAuth$3.call(FirebaseAuth.java:266)
	at com.google.firebase.auth.FirebaseAuth$3.call(FirebaseAuth.java:263)
	at com.google.firebase.tasks.Tasks$1.run(Tasks.java:82)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)

Steps to reproduce:
So what I did.
I wanted to keep previous claims so I created copy of map and add new key-val.
    @Override
    public void addUserCustomClaims(final String userId, String key, String val) {
        final UserRecord user = getUser(userId);
        final Map<String, Object> customClaims = new HashMap<>(user.getCustomClaims());
        customClaims.put(key, val);
        final Void aVoid = Single.fromFuture(
            firebaseAuth.setCustomUserClaimsAsync(userId, customClaims))
            .blockingGet();
    }

It was inserted as I see but now I can't get user because it's throwing exception.
With breakpoint set to https://github.com/firebase/firebase-admin-java/blob/82510ac1fc518dd80061ccfbe9a5131ae938ac29/src/main/java/com/google/firebase/auth/UserRecord.java#L85
I see values
customClaims={"u_type":"USER"}
this.customClaims=null

Your app is using an incompatible version of the org.json:json dependency. You will have to inspect your dependency tree and ensure that there are no conflicting or incompatible artifacts. Here's what Firebase uses: https://github.com/firebase/firebase-admin-java/blob/master/pom.xml#L423

This problem is still happening. Problem with Spring Test boot which have the vaadin dependency causing the problem

@jfzr try to add this
configurations {
    compile.exclude module: "android-json"
    compile.exclude module: "com.vaadin.external.google"
}


Add a donation button

Hey @xunhuang, setting up the project here. Can you please add my account to firebase? It's<EMAIL_ADDRESS>Thank you.

added
On Wed, Mar 25, 2020 at 9:12 PM Shuo Shen<EMAIL_ADDRESS>wrote:

Hey @xunhuang https://github.com/xunhuang, setting up the project here.
Can you please add my account to firebase? It's<EMAIL_ADDRESS>Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/issues/7#issuecomment-604218180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF2VQSTNRA2FVAHCWBTRJLIZPANCNFSM4LTW4RFQ
.

Thank you!
Do we have % of users on desktop web vs msite? If most traffics are from msite, I can prioritize Apple Pay and Android Pay.

Maybe you can go to firebase to dig around. I remember seeing more details before but I can't find it

that said, i was so thinking about a Direct payment thing but more like a donation button that leads to Paypal or gofundme or a facebook fundraiser type places. That said I know very little this type of things.

You were not thinking about direct payment?

pros/cons?
On Thu, Mar 26, 2020 at 8:59 PM Shuo Shen<EMAIL_ADDRESS>wrote:

You were not thinking about direct payment?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/issues/7#issuecomment-604802569,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGFZA2DZVP5EDRWOOBEDRJQQBXANCNFSM4LTW4RFQ
.

3rd party fundraiser is easier to build, but properly lower conversion.

I'll say let's stay simple first, and measure the conversion. we can do
more work if it doesn't convert
On Thu, Mar 26, 2020 at 9:05 PM Shuo Shen<EMAIL_ADDRESS>wrote:

3rd party fundraiser is easier to build, but properly lower conversion.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/issues/7#issuecomment-604803860,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF7GGHCPHUIKXPMHGWDRJQQ2DANCNFSM4LTW4RFQ
.

hey shuo, finally got around to this. (motivated by an actual large bill from google)
ko-fi.com/covid19direct
Can you please update the code to put a small Red button on the top right corner of the site?
Thanks!

Hey, are you visioning a button on the right of the first row? Sounds like
more or less trivial. Feel free to go ahead and put it there.
I can help work on improving conversion if needed.
On Sat, Apr 4, 2020 at 12:07 PM xunhuang<EMAIL_ADDRESS>wrote:

hey shuo, finally got around to this. (motivated by an actual large bill
from google)
ko-fi.com/covid19direct
Can you please update the code to put a small Red button on the top right
corner of the site?
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/issues/7#issuecomment-609075042,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADKKUONUA2IGBAQ5X7LEK3RK6ANZANCNFSM4LTW4RFQ
.

added. done.


Certain imports not found in common module (PainterResource, AlertDialog..)

So, I've created a project, similar to the multiplatform example, but I'm having trouble using certain functionalities. I'm developing for desktop and android.
First of all, I'm aware that I can use these in the seperate android and desktop modules, but most of my implementations (Java and Compose specific) are in the common module.
And because the desktop and android module call code in the common module, I cannot call functions from them (it would create circular dependencies)
My question is - am I having the wrong approach to this? I'd like to load and display images located in the shared resources folder in common, but I can't achieve this without having access to PainterResource. Same goes with displaying dialogues (AlertDialog)

Common resource management still isn't supported by Compose Multiplatform. But you can implement this workaround. Also, please follow that issue for [painterResource]. and this issue for AlertDialog

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Ensure omnibar shows on reset - WIP

Task/Issue URL: https://app.asana.com/0/414730916066338/726263852905235/f
Description:
Fixes issues where omnibar could be hidden away and inaccessible
Steps to test this PR:

Go to a webpage
Scroll the page up to hide the bars
Press the back button to go home.
Observe that the bar is shown

Internal references:
Software Engineering Expectations
Technical Design Template

@brindy hold off on this one, I can still make the issue happen

False alarm @brindy, android studio just wasn't deploying the new code. This is good to go.


Add user callback on struct CanardRxSubscription

I think CanardRxSubscription need add a function call, named user_callback.
typedef void (*user_callback)(CanardInstance *ins, CanardTransferKind kind, CanardTransfer *transfer);
So that when after receive done, it will call user callback, so user can decode transfer and do something else.

This would go against some high-integrity coding guidelines that generally prohibit function pointers. We use function pointers for memory allocation/deallocation routines because there is no sensible alternative, whereas what you suggested is adequately covered already by canardRxAccept(). If you want a callback-based interface, write a simple wrapper on top of canardRxAccept().


[STYLE]: Added profile skeleton loader on Homepage

Description
On homepage where all the profiles are loaded, a profile skeleton loader has been added to show until the profile data loads.
Related Issues
Issue #498
Changes Proposed
Created a new skeleton component having similar structure as profile card. Rendering this until the profile data loads.
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

@ravi2048 Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


NBT described in the Pose tag of armor_stand is not a compound tag but a List tag.

Example:{Pose:{Head:[0f,0f,0f]}}

:tada: This issue has been resolved in version 1.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:

It hasn't been fixed :(
The error is the same.

This bug is not fixed, the error is the same

Looks like the dependency is not updated correctly.

:tada: This issue has been resolved in version 1.8.2 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


discovery: add per-host HTTP headers to discovery functions

The discovery library doesn't have support for passing auth credentials so it will fail in servers that have authentication enabled.
This PR modifies the discovery functions to accept a map with per-host headers. If the host in the discovery URL is specified in this parameter, the corresponding headers will be applied when doing the discovery.
As an alternative, we could change the parameter to be just a header and leave converting the app to a hostname and selecting the right header to the library caller.
This is not ready for review/merge, I just want to get some feedback on the general idea.
Related: https://github.com/coreos/rkt/issues/1516

Seems pretty reasonable. @sgotti ?

Added a test

LGTM after minor cleanup

Updated


Typo "pass@word1"→"pass@word1"

Bold with escape characters
https://docs.microsoft.com/en-us/windows/deployment/windows-10-poc-sc-config-mgr

(just to point more directly to the affected paragraph/section)

https://docs.microsoft.com/en-us/windows/deployment/windows-10-poc-sc-config-mgr#configure-the-network-access-account

BTW, this is less of a "typo" and more of a "code character missing the escape character" issue, but not a big deal. To a MarkDown aficionado, I guess this counts as a typo.

#sign-off


PI: default array editor

If, for whatever reason, a PI for an array doesn't load, com.livecode.pi.array should be the failsafe, not com.livecode.pi.string, or whatever it is, now.

probably should have a failsafe path for each type to progress, but in the end, should always be able to modify contents.


feat: HTTPConnectionManager settings for XFF headers on ClientTrafficPolicy

What type of PR is this?
feat: HTTPConnectionManager settings for XFF headers on ClientTrafficPolicy
What this PR does / why we need it:
Implements support for specifying XFF header handling configuration xff_num_trusted_hops and use_remote_address to ClientTrafficPolicy allowing proxies for listeners to be configured as required to trust XFF headers such as x-forwarded-for and x-forwarded-proto
Which issue(s) this PR fixes:
Related to #2252

thanks for building this out @davidalger . Here's how I'm hoping we can make progress in this PR

get some consensus that ClientTrafficPolicy is the right home for this field vs in EnvoyProxy
Youve already highlighted this - since EnvoyProxy attaches to a GatewayClass, it can affect multiple Gateways and corresponding Envoy Proxy fleets, so introducing the field in ClientTrafficPolicy does make sense, im a +1 for this
I'd prefer if we created a top level field called headers to hold all fields that affect proxy header transformations. Once we have consensus on this, we'd need to make that change first @liorokman
I'd prefer if we skipped introducing the use_remote_address field in this PR

I'd prefer if we created a top level field called headers to hold all fields that affect proxy header transformations. Once we have consensus on this, we'd need to make that change first @liorokman

I'm +1 for that. In fact, I've already proposed this in #2500 .

I'd prefer if we created a top level field called headers to hold all fields that affect proxy header transformations. Once we have consensus on this, we'd need to make that change first @liorokman

I like the idea of organizing things better, but not 100% sure headers is the right place to put it since XFF isn't just about header transformation and falls more into general IP detection territory.
@arkodg Since HCM supports either a combination of use_remote_address and xff_num_trusted_hops or using original_ip_detection_extensions for this, what are your thoughts on the following structure?
originalIpDetection:
  useRemoteAddress: true    # True by default, false if `extensions` is specified (not present in CRD, as suggested)
  xffNumTrustedHops: number # Mutually exclusive with `extensions` configures `xff_num_trusted_hops` built-in
  extensions:
    # Configures type.googleapis.com/envoy.extensions.http.original_ip_detection.custom_header.v3.CustomHeaderConfig
    customHeaders:
      headerName: string
      rejectWithStatus: enum
    # Configures type.googleapis.com/envoy.extensions.http.original_ip_detection.xff.v3.XffConfig
    xff:
      numTrustedHops: number

Supporting config for the custom headers IP detection extension will be of use to me in deployments of EG, so even if precluded from this MR for sake of keeping it single-purpose I would like to consider planning for it and can introduce it in a future change set.

so we have two options here, (please ignore the exact names of the fields)

add this config under a newly created headers field

headers:
  xForwardedFor:
    numTrustedHops:

add this config under a dedicated field around IP detection / network topology

ipDetection:
  numTrustedHops

cc @envoyproxy/gateway-maintainers
will bring this up in the community meeting this week

discussed this in the community meeting today and folks prefer 2 over 1 (generic headers section)

brainstorming some names - what should the top level field be - remoteIP or originalIP or remoteIPDetection or originalIPDetection or originatingIP....

sidenote (not related to this PR) - should proxy protocol config be moved under this field ? cc @envoyproxy/gateway-maintainers

brainstorming some names - what should the top level field be - remoteIP or originalIP or remoteIPDetection or originalIPDetection or originatingIP.... or clientIP..... Whichever term we decide will need to be used in other APIs (https://github.com/envoyproxy/gateway/issues/2250 and RateLimit)

clientIPDetection makes sense to me given the term 'client IP' lines up with what's used used to document sourceCIDR on RateLimit already and also has widespread use across the broader industry (Google LBs, CloudFlare, Nginx, etc) and it's meaning should be widely understood

brainstorming some names - what should the top level field be - remoteIP or originalIP or remoteIPDetection or originalIPDetection or originatingIP.... or clientIP..... Whichever term we decide will need to be used in other APIs (#2250 and RateLimit)

clientIPDetection makes sense to me given the term 'client IP' lines up with what's used used to document sourceCIDR on RateLimit already and also has widespread use across the broader industry (Google LBs, CloudFlare, Nginx, etc) and it's meaning should be widely understood

sg @davidalger, lets go with clientIpDetection !

@arkodg Implemented first pass here with the following structure:
clientIPDetection:
  xffNumTrustedHops: number # Mutually exclusive with `extensions` configures `xff_num_trusted_hops` HCM built-in
  extensions:
    # Configures type.googleapis.com/envoy.extensions.http.original_ip_detection.custom_header.v3.CustomHeaderConfig
    customHeader:
      headerName: string
      rejectWithStatus: number
    # Configures type.googleapis.com/envoy.extensions.http.original_ip_detection.xff.v3.XffConfig
    xff:
      numTrustedHops: number

use_remote_address maintains it's default of true while being set to false if original IP detection extensions are used since the two are mutually exclusive
Once https://github.com/envoyproxy/envoy/pull/31831 is merged, I would see there being an additional field to control the trusted CIDRs:
clientIPDetection:
  extensions:
    xff:
      trustedCIDRs:
      - "<IP_ADDRESS>/22"
      - "<IP_ADDRESS>/16"

Would it be better to keep things a little more flat given this won't support arbitrary extensions?
clientIPDetection:
    xffNumTrustedHops: 2
    customHeaderExtension:
      // ...
    xffExtension:
      // ...

@davidalger curious if you know the difference between the numTrustedHops field in the top level vs extension setting in envoy ?

curious if you know the difference between the numTrustedHops field in the top level vs extension setting in envoy?

At the moment the only difference I believe is the extensions were marked incompatible with use_remote_address, but otherwise the XFF extension is identical in function currently.
It's been asked by maintainers to implement additions to XFF handling in original_ip_detection_extensions and in the coming days it's looking like sourceCIDRs support will be added to the XFF extension (https://github.com/envoyproxy/envoy/pull/31831) and notably it will evaluate the x-forwarded-for header either by a fixed number of trusted hops, or by evaluating the client IP address against a list of trusted addresses.
So something that could be considered is supporting numTrustedHops at the top-level and sourceCIDRs (potential future feature) would use the extension but without any plans to support the seemingly duplicate numTrustedHops field.
Something like this I imagine:
clientIPDetection:
    xForwardedFor:
      # Top-level xff_num_trusted_hops field with no support for same field in XFF extension
      numTrustedHops: 2

      # Future ability to skip appending to xff header
      skipAppend: true

      # Future ability to evaluate xff header based on trusted CIDRs rather than number of hops (uses extension)
      sourceCIDRs:
      - "<IP_ADDRESS>/22"
      - "<IP_ADDRESS>/16"

    # Configures type.googleapis.com/envoy.extensions.http.original_ip_detection.custom_header.v3.CustomHeaderConfig
    customHeader:
      headerName: x-client-ip-address
      rejectWithStatus: 403

Anything other than numTrustedHops would result in use_remote_address being set to false due to incompatibility with extensions, otherwise the current default of true would apply.
I'll update this MR to only add the ability to set numTrustedHops and tackle customHeader separately after this is complete.

curious if you know the difference between the numTrustedHops field in the top level vs extension setting in envoy?

At the moment the only difference I believe is the extensions were marked incompatible with use_remote_address, but otherwise the XFF extension is identical in function currently.
It's been asked by maintainers to implement additions to XFF handling in original_ip_detection_extensions and in the coming days it's looking like sourceCIDRs support will be added to the XFF extension (envoyproxy/envoy#31831) and notably it will evaluate the x-forwarded-for header either by a fixed number of trusted hops, or by evaluating the client IP address against a list of trusted addresses.
So something that could be considered is supporting numTrustedHops at the top-level and sourceCIDRs (potential future feature) would use the extension but without any plans to support the seemingly duplicate numTrustedHops field.
Something like this I imagine:
clientIPDetection:
    xForwardedFor:
      # Top-level xff_num_trusted_hops field with no support for same field in XFF extension
      numTrustedHops: 2

      # Future ability to skip appending to xff header
      skipAppend: true

      # Future ability to evaluate xff header based on trusted CIDRs rather than number of hops (uses extension)
      sourceCIDRs:
      - "<IP_ADDRESS>/22"
      - "<IP_ADDRESS>/16"

    # Configures type.googleapis.com/envoy.extensions.http.original_ip_detection.custom_header.v3.CustomHeaderConfig
    customHeader:
      headerName: x-client-ip-address
      rejectWithStatus: 403

Anything other than numTrustedHops would result in use_remote_address being set to false due to incompatibility with extensions, otherwise the current default of true would apply.
I'll update this MR to only add the ability to set numTrustedHops and tackle customHeader separately after this is complete.

sounds like a good plan ! this keeps any envoyisms out of the API increasing its ability to be moved into the upstream gateway api in the future

@arkodg updated and should be ready for review

PR looks good @davidalger, added a minor comment around optional fields

need a user doc for this?

need a user doc for this?

was part of this PR https://gateway.envoyproxy.io/latest/user/client-traffic-policy/#configure-client-ip-detection


Issue with table user not found

I have setup this sample. not sure if need to create db and table in mysql separately ?  I am getting below error:
{
"code": "ER_NO_SUCH_TABLE",
"errno": 1146,
"sqlMessage": "Table 'sampledb.user' doesn't exist",
"sqlState": "42S02",
"index": 0,
"sql": "SELECT * FROM user WHERE username ='test'"
}
is there any db script that needs to be executed? DB i create manually, but not sure about table details.

/documents/sql-script/script_v1.sql has CREATE TABLE statement. The syntax was off for my version of MySQL, but that if you run that script in any query tool, you should be good to go.

@monikagupta78  you can check all models which ever table used can create or else you can comment code for that particular route ex : user  and can create your own table.
Any way thanks for updating i will change in readme.

@curtisolson  thanks for mentioning those point.


Add support for special characters in username/password for pip

Pull Request Check List
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!

[x] Added tests for changed code.
[ ] Updated documentation for changed code.

Note: If your Pull Request introduces a new feature or changes the current behavior, it should be based
on the develop branch. If it's a bug fix or only a documentation update, it should be based on the master branch.
If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!

Per #1249 and pip #6776, newer versions of pip require require the username and password to be properly encoded/quoted.  This PR simply quotes the URL when calling pip.
As referenced in #1249, we do not want to pre-quote the strings as that fouls up the metadata fetch which does not use pip.

Does this bug also exist on master? If so, then this PR should be merged into master and not into develop.

See my comment on #1402 , the fix should be applied to master branch as well, but the code changed between the two versions and this patch can not be applied as is to master

As @oleastre mentioned, there is a significant code divergence between master and develop so the patch cannot be applied to master. My recommendation is to not let perfect be the enemy of good and just merge both #1402 and this PR.
Honestly, this master vs develop situation is a giant pain and, as highlighted by this situation, is impeding work. It's only getting worse since there was a major refactor in develop that changes many core functions.

Thanks!


You?

Your an ad. Please check yourself $msft. You’re helping no one, how’s the pay check you brainwashed goat. 👎 :feelsgood: 🚼

Also change your profile name, no one can copyright the skyline, or I’ll send dogs in for your soul. glhf twirp, #Legion find out him and his family of followers and fleas.


[BUG] Templater is not applied

Describe the bug
When applying aliases to a note that doesn't exist, and then creating the note. The Templater metadata that was supposed to be applied, is not, and all the metadata there is are the aliases
To Reproduce
Steps to reproduce the behavior:

Create a link with alias to a note that doesn't exist yet
Open that link (which will create the note)

Expected behavior
The note should be created with the appropriate template, and after that the aliases should be applied. But only the aliases are applied and no template
Screenshots

The expected behavior:

What happens:

I am not using integration with templater, so it is not in my interest to learn how templater works in order to be able to make my plugin compatible with templater plugin.
However I agree that it would be nice to have it compatible. So a pull request with the fix is welcome. I will gladly check it and merge if possible.


Created Custom Resolution Functionality

Major Changes
Added 2 Functions to manage the custom resolutions
Modified the FPS functions to allow custom Menu FPS
Added and Modified support in the Lang and Window for these features.
Minor and Accidental
Forgot to revert the change to the Transparency to a Magenta background, I removed it for my personal build.
removed FPS from the start of 2 functions, since I used them for my dialog boxes. Both call in the XAML.
Uncompleted
I should have added references to the language template for the newly added lines.
But there was only RU and an unapproved ESP lang set so I don't feel so horrible about it.

Merged, but please disable insertion of spaces next time before committing, it led to more than 150 merge conflicts.


Installation instructions contain incorrect Github URL

The Module Installer directions in the Readme contain the following URL for the SDK:
github.com/checkout-sdk-go@{version}
This is incorrect as it does not contain the organization name.  It should be:
github.com/checkout/checkout-sdk-go@{version}
The Import code snippet should similarly be updated.

Hi @michael-taylor-cko, this was solved on #53  let us know if you face any other issue.


Add Folder Permissions For Github Version [Feature Request]

For those of us downloading or cloning the Github version why not add proper folder permissions to mirror bludit.com's official download version. In my personal experience i have to manually change folder and subfolder permissions to access bludit-master on my localhost.
find path/to/bludit-master -type d -exec chmod 755 {} \;

For content/ folder
chmod 777 content

Hi, I documented this issues on this page. http://docs.bludit.com/en/getting-started/installation-on-gnu-linux
Thanks!


implement popscope to ignore backbutton android

just implement popscope to ignore backbutton,
this is weird when cool alert show loading type is called, but user can pop the loading dialog

+1


Add additional labels based on checkbox state

Allows for more fine-grained filtering.

Tested manually with various applications ✅


Replace amp module with lodash

+1

+1, tests pass locally. this repo doesnt have a travis.yml file so thats why travis isnt passing.

Merged and v0.2.3 tag pushed. Need publishing to npm from @latentflip or @HenrikJoreteg :smile:

done @lukekarrys

Published as v0.2.3


Make sure the line begins with a (forward) slash ('/')

Make sure the line begins with a (forward) slash ('/') - otherwise we risk a situation where a full path to zsh is commented in, i.e.:
#/usr/local/bin/zsh

Ping.

This is a trivial change and an improvement over what is there now. Any chance to get this one in?

This commit has been picked up in #5169 and it will make it into master once it's merged. Hopefully by tomorrow it'll be done.

Hey @rjc, as I said yesterday the installer at #5169 is finished and ready to test. If you want to help speed up the testing and merge (which should happen early next week), you can wget or curl it from the URL https://raw.githubusercontent.com/mcornella/oh-my-zsh/refactor-installer/tools/install.sh. Read the comments at the top of the script to know how to call it.
Thanks!

Closed by #5169.


mobile: support notifications and events

What I'm trying to do
Ensure that mobile apps using Aries Agent Mobile can receive notifications for topics they are subscribed to or events they are part of.
Expected result
A communication "channel" that mobile apps can subscribe to in order to get notifications about the topics they care about.
Sample code / test case
TBD
Checklist

[ ] Interface for notifier/observer/event/websocket/webhooks... #2170

@soluchok


Feature/expose getting single page methods

Description of Change

Methods for getting individual response pages:

sdk.detectionlists.departing_employee.get_page()
sdk.detectionlists.high_risk.get_page()
sdk.users.get_page()
sdk.devices.get_page()
sdk.legalhold.get_matters_page()
sdk.legalhold.get_custodians_page()
sdk.alerts.get_page()

Issues Resolved
INTEG-1117
Testing Procedure
Call the methods above
PR Checklist
Did you remember to do the below?

[x] Add unit tests to verify this change
[x] Add an entry to CHANGELOG.md describing this change
[x] Add docstrings for any new public parameters / methods / classes

@unparalleled-js  @timabrmsn one thought I had: rather than adding separate methods to get single pages, would it be better to have the existing generators have optional start_page and end_page args (which you could set to be the same if you only wanted a single page)?

@unparalleled-js @timabrmsn one thought I had: rather than adding separate methods to get single pages, would it be better to have the existing generators have optional start_page and end_page args (which you could set to be the same if you only wanted a single page)?
start_page=2, end_page=5 - generator for pages 2-5, inclusive
start_page=None, end_page=5 - generator for first page through page 5, inclusive
start_page=2, end_page=None - generator for page 2 to last page, inclusive

So, to do paging, I would have to iterate using two iterators, like start_page=1, end_page=1, then to get the next page, I'd have to do start_page=2, end_page=2. Do-able I guess, but I just want to be able to request an individual page at a time, preferably not as a generator

@unparalleled-js is there a reason orgs isn't getting this functionality as well?

@unparalleled-js is there a reason orgs isn't getting this functionality as well?

No, I would like orgs for sure, so I will put that in, thanks
Sorry! I had accidentally edited your comment when I meant to reply before

I think I'd prefer keeping two methods get_page and get_all to keep the api simple (calling get_all with a start and end of the same page, to get a generator that you then have to call next() on is not how I'd want to get one page's results).
I do like the idea of adding a start_page param to the get_all methods, that allows us to move the logic for whether to start at 0 or 1 to each caller.
I don't think end_page would be necessary though, since the result is a generator, if you want a subset of results you can just stop iterating after your desired number.

I think I'd prefer keeping two methods get_page and get_all to keep the api simple (calling get_all with a start and end of the same page, to get a generator that you then have to call next() on is not how I'd want to get one page's results).
I do like the idea of adding a start_page param to the get_all methods, that allows us to move the logic for whether to start at 0 or 1 to each caller.
I don't think end_page would be necessary though, since the result is a generator, if you want a subset of results you can just stop iterating after your desired number.

we discussed offline and came to similar conclusions, though we haven't added a start_page param at this time.


upgrade @openzeppelin/contracts

fix #24

Just an upgrade
Tests have passed

There may be breaking changes, but so far it looks ok. I will investigate further.

oh is it working without any problem to mint ?

yeah, i think so