add event number 9

This is the draft blurb for event number 9. Sorry for the tardiness - it's been all-hands on deck for re:Invent since last week (I'm just managing to sneak this PR in at 06:00 local time before heading to the Game Day). 🌄
I don't have a building environment on my local machine so I've no idea if this renders…

Pull requests that don’t have one approved review, with no changes requested can’t be merged on a phone.

😐


Mostly fix score display on Py3

ok good progress, turning off the grid numbers along the top and side enables the scores to show in the cells. Can't fathom exactly why but may be something with layers covering other layers, as the arrows still look different between python versions, and sometimes hide a score here or there on py3. But overall it's usable on py2 and py3 now:
py2 & py3

Not sure if it makes any difference, but I've fixed division to be the same as it was before - int output where there's no from __future__ import division.

Changes Unknown when pulling fac356384213431a44330dd0df8a702e24bd671d on tomviner:fix-more-things into ** on ldnpydojo:master**.

That's really weird. :-)


Asset import needs additional configuration inputs

Currently the asset import utility assumes that the path should include an SFTP URL which includes username and password, but using that will also disclose the password everywhere.  Also, in the event of using SFTP, it would be helpful to add controls over connection timeout.

He everyone.
We need this feature in our project.
I have forked acs-aem-commons, and I'm going to fix it in the following branch: https://github.com/cylinder-y/acs-aem-commons/tree/feature/Fix-1578.
When feature will be fixed I will create pull request that described in CONTRIBUTING.md
--Yaraslau

Hello @badvision,
Could you please review pull request?
--Yaraslau

Having a look now, thanks!  I just got back from holiday.

Looks great!  Thanks for helping improve these tools @cylinder-y !


Improve export by adding NuttX current target (arch, chip, board)

Summary
Add variables that describe NuttX current build (board, arch, chip)
Fix export: startups files are not added at link step
Impact
Testing
picopi:usbnsh with an external project
visual check on export archive files

@acassis the CI is broken but I cannot see how it is related to my changes... Any idea?

@acassis the CI is broken but I cannot see how it is related to my changes... Any idea?

It is going fine now


Add Dora read fallback to UFS logics

Support dora read to fallback to UFS if any error occur during read.

Can you add some tests for the new fallback behavior?

alluxio-bot, merge this please


[feaLib] Class-to-class kerning zeroed out

I've noticed that for class-to-class kerning rules, if one rule has a value of zero some other ones get cleared out. For example this feature source:
@public.kern1.K = [K X];
@public.kern2.O = [C O];
@public.kern2.V = [V W];

feature kern {
    pos @public.kern1.K @public.kern2.O -20;
    pos @public.kern1.K @public.kern2.V 0;
} kern;

produces this output:
$ hb-shape out.otf K
[K=0+619]
$ hb-shape out.otf KO
[K=0+619|O=1+781]

with this GPOS:
  <GPOS>
    <Version value="1.0"/>
    <ScriptList>
      <!-- ScriptCount=1 -->
      <ScriptRecord index="0">
        <ScriptTag value="DFLT"/>
        <Script>
          <DefaultLangSys>
            <ReqFeatureIndex value="65535"/>
            <!-- FeatureCount=1 -->
            <FeatureIndex index="0" value="0"/>
          </DefaultLangSys>
          <!-- LangSysCount=0 -->
        </Script>
      </ScriptRecord>
    </ScriptList>
    <FeatureList>
      <!-- FeatureCount=1 -->
      <FeatureRecord index="0">
        <FeatureTag value="kern"/>
        <Feature>
          <!-- LookupCount=1 -->
          <LookupListIndex index="0" value="0"/>
        </Feature>
      </FeatureRecord>
    </FeatureList>
    <LookupList>
      <!-- LookupCount=1 -->
      <Lookup index="0">
        <!-- LookupType=2 -->
        <LookupFlag value="0"/>
        <!-- SubTableCount=2 -->
        <PairPos index="0" Format="2">
          <Coverage Format="1">
            <Glyph value="K"/>
            <Glyph value="X"/>
          </Coverage>
          <ValueFormat1 value="0"/>
          <ValueFormat2 value="0"/>
          <ClassDef1 Format="2">
          </ClassDef1>
          <ClassDef2 Format="1">
            <ClassDef glyph="V" class="1"/>
            <ClassDef glyph="W" class="1"/>
          </ClassDef2>
          <!-- Class1Count=1 -->
          <!-- Class2Count=2 -->
          <Class1Record index="0">
            <Class2Record index="0">
            </Class2Record>
            <Class2Record index="1">
            </Class2Record>
          </Class1Record>
        </PairPos>
        <PairPos index="1" Format="2">
          <Coverage Format="1">
            <Glyph value="K"/>
            <Glyph value="X"/>
          </Coverage>
          <ValueFormat1 value="4"/>
          <ValueFormat2 value="0"/>
          <ClassDef1 Format="2">
          </ClassDef1>
          <ClassDef2 Format="2">
            <ClassDef glyph="C" class="1"/>
            <ClassDef glyph="O" class="1"/>
          </ClassDef2>
          <!-- Class1Count=1 -->
          <!-- Class2Count=2 -->
          <Class1Record index="0">
            <Class2Record index="0">
              <Value1 XAdvance="0"/>
            </Class2Record>
            <Class2Record index="1">
              <Value1 XAdvance="-20"/>
            </Class2Record>
          </Class1Record>
        </PairPos>
      </Lookup>
    </LookupList>
  </GPOS>

One can work around this by simply not including class-to-class rules with value of 0 in the feature source. Still, it seems like a bug.

Just discovered that makeotf does the same thing. @adrientetar suggested that maybe this should be filed towards the AFDKO.

cc @readroberts @miguelsousa does this look right?

Humm.  What's the bug again?  The generated XML looks correct to me, even if not optimal.

I didn't see anything wrong with the XML myself. But the output font is definitely not what I expect. If I have a kerning rule defined in feature source:
pos @public.kern1.ba.ethiopic @public.kern2.be.ethiopic -10;
I expect for kerning to be applied for sequence [ba.ethiopic|be.ethiopic]. But that's not what's happening.

Sorry, I just realized that this actually does work with makeotf. Not sure how I messed that up before. So it's only a feaLib problem. Here is the diff from makeotf output to feaLib output:
--- out_fdk.ttx	2016-07-12 11:28:53.806119200 -0700
+++ out_fealib.ttx	2016-07-12 11:24:41.663352389 -0700
@@ -32,12 +32,34 @@
       <Lookup index="0">
         <!-- LookupType=2 -->
         <LookupFlag value="0"/>
-        <!-- SubTableCount=1 -->
+        <!-- SubTableCount=2 -->
         <PairPos index="0" Format="2">
           <Coverage Format="1">
             <Glyph value="K"/>
             <Glyph value="X"/>
           </Coverage>
+          <ValueFormat1 value="0"/>
+          <ValueFormat2 value="0"/>
+          <ClassDef1 Format="2">
+          </ClassDef1>
+          <ClassDef2 Format="1">
+            <ClassDef glyph="V" class="1"/>
+            <ClassDef glyph="W" class="1"/>
+          </ClassDef2>
+          <!-- Class1Count=1 -->
+          <!-- Class2Count=2 -->
+          <Class1Record index="0">
+            <Class2Record index="0">
+            </Class2Record>
+            <Class2Record index="1">
+            </Class2Record>
+          </Class1Record>
+        </PairPos>
+        <PairPos index="1" Format="2">
+          <Coverage Format="1">
+            <Glyph value="K"/>
+            <Glyph value="X"/>
+          </Coverage>
           <ValueFormat1 value="4"/>
           <ValueFormat2 value="0"/>
           <ClassDef1 Format="2">
@@ -45,11 +67,9 @@
           <ClassDef2 Format="2">
             <ClassDef glyph="C" class="1"/>
             <ClassDef glyph="O" class="1"/>
-            <ClassDef glyph="V" class="2"/>
-            <ClassDef glyph="W" class="2"/>
           </ClassDef2>
           <!-- Class1Count=1 -->
-          <!-- Class2Count=3 -->
+          <!-- Class2Count=2 -->
           <Class1Record index="0">
             <Class2Record index="0">
               <Value1 XAdvance="0"/>
@@ -57,9 +77,6 @@
             <Class2Record index="1">
               <Value1 XAdvance="-20"/>
             </Class2Record>
-            <Class2Record index="2">
-              <Value1 XAdvance="0"/>
-            </Class2Record>
           </Class1Record>
         </PairPos>
       </Lookup>

Humm.  Ok.  When I read this it was using ethiopic!  Let me reread.

Ok, yes, this is a bug in feaLib.  In short, when a subtable matches the first glyph, all kern pairs for that glyph must be specified in that subtable.
This actually gives me a hint as to how to generate optimal subtables.  It's a much easier problem than it is without this constraint.

Humm.  Actually scratch that.  I'm not sure of what I just wrote :).  Let me dig.

Ok, I just confirmed that with HarfBuzz, a class-based kern subtable always "succeeds" if the first glyph matches even if the second glyph doesn't.  That's what class0 is for...
James, can you email me the sample font?  I want to test with Uniscribe.

Ok, I verified that Uniscribe works similarly.  So this is a feaLib bug.  Assigning to Sascha.  So, a PairPos lookup must not have two class-based subtables (format=2) that haver overlapping class1 members.

Tricky. Here’s the root cause: Currently, feaLib generates ValueRecords of different formats for the two pos statements below. The -20 gets compiled to a ValueRecord format 4, the 0 currently to a ValueRecord format 0. Because these ValueRecords have different formats, feaLib can not put them into the same subtable.
feature kern {
    pos @kern1.K @kern2.O -20;
    pos @kern1.K @kern2.V 0;
} kern;

To fix this, feaLib should obviously compile the above 0 into a ValueRecord format 4 with an xAdvance of 0. However, it’s not that easy. In other contexts, makeotf is compiling the exact same 0 to a ValueRecord format 0. An example is the SinglePos feature kern {pos A 0;} kern; which makeotf (and also feaLib) currently compile into the following:
    <LookupList>
      <!-- LookupCount=1 -->
      <Lookup index="0">
        <!-- LookupType=1 -->
        <LookupFlag value="0"/>
        <!-- SubTableCount=1 -->
        <SinglePos index="0" Format="1">
          <Coverage>
            <Glyph value="A"/>
          </Coverage>
          <ValueFormat value="0"/>
        </SinglePos>
      </Lookup>
    </LookupList>

So we can’t just change the compilation result for 0. If we did, we’d simply have the same bug with other statements — at least SinglePos would be broken (different from makeotf) for zero values, but other constructs might be affected too, such as contextual kerning rules which often implicitly generate SinglePos lookups. So I guess my next step will be to systematically check all places where feature files may contain value expressions, and find out when exactly makeotf is compiling 0 into a ValueRecord of format 0, and when it compiles that exact same 0 expression into a ValueRecord format 4.

Done.


unrecognized bind variable :p3 passed to the bind with Oracle

Laravel Version
11
PHP Version
8.2
Database Driver & Version
Oracle
Description
Hi.
i'm using the laravel 11
everything works well at MySQL
but as soon as i connect the framework to the Oracle, my query for searching on some data, has problem.
there is an error while i want to get the data base on 'time_log' :
oci_bind_by_name(): ORA-01036: unrecognized bind variable :p3 passed to the bind callHelp: https://docs.oracle.com/error-help/db/ora-01036/ (Connection: oracle_development, SQL: select * from (select * from "CAS_DATA" where "TIME_LOG" < 2024-10-11 06:09:15 and "TIME_LOG" >= 2024-10-10 00:00:00 and "CRAWLER_ID" = 1 order by "TIME_LOG" desc) where rownum = 1)
Steps To Reproduce
here's my code :
        $fields = $validator->validated();
        $targetDateTime = $fields['time_log'];
        $startTime = Carbon::parse($targetDateTime)->subDay()->startOfDay();
        $endTime = Carbon::parse($targetDateTime)->addDay()->endOfDay();
        $crawler_id = $fields['crawler_id'];

        // get data before and after time_log
        $queryBefore = CasData::query()
            ->where('time_log', '<', $targetDateTime)
            ->where('time_log', '>=', $startTime)
            ->where('crawler_id', $crawler_id)
            ->orderBy('time_log', 'desc')
            ->limit(1);

        $queryAfter = CasData::query()
            ->where('time_log', '>', $targetDateTime)
            ->where('time_log', '<=', $endTime)
            ->where('crawler_id', $crawler_id)
            ->orderBy('time_log', 'asc')
            ->limit(1);

        // merge two queries
        $results = $queryBefore->union($queryAfter)->get();```

here's my final query :
select * from (select * from "CAS_DATA" where "TIME_LOG" < '2024-10-11 06:09:15' and "TIME_LOG" >= '2024-10-10 00:00:01' and "CRAWLER_ID" = 1 order by "TIME_LOG" desc) where rownum = 1;
and even when i want to execute this on the Oracle (forgeDB) i get this error :
ELECTRONICJ> select * from (select * from "CAS_DATA" where "TIME_LOG" < '2024-10-11 06:09:15' and "TIME_LOG" >= '2024-10-10 00:00:01' and "CRAWLER_ID" = 1 order by "TIME_LOG" desc) where rownum = 1 [2024-10-18 19:15:28] [22008][1861] ORA-01861: literal does not match format string [2024-10-18 19:15:28] Position: 99

Hi there,
Thanks for reporting the problem you are encountering, but it looks like this is a question which may be better suited for a support channel. We only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repository you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Sounds GREAT but it won't open at all..

So.. i basically download the last version and run but nothing happen.. what i do wrong?? Something else to be well explained on how this work??
Ty :D

Hi, thank you for taking the time to report this.
There are a couple of things that may cause this off the top of my head. Mainly that GProfiles cannot locate your Logitech settings. Please keep in mind that for this application to work, you should also have Logitech G HUB installed in your system.
If you do have it installed, check that %LOCALAPPDATA%\LGHUB is an existing directory, and that there is a settings.db file in that folder.
I'm currently working on improving error handling, which should help further debug the issue in case the above did not solve it.
Cheers!

Hello and ty for the answer
Of course i have G HUB installed and the settings.db file in the appdata/LGHUB path.
My doubt was if it won't open because there's anything else to do, like if it's important to execute while LGHub is running or not, or it just work on edit properly the settings.db file and nothing else needed.
Btw no error or something like that.. just nothing open in the process manager when executing.
Cheers

Hello,
GProfiles doesn't need LGHUB to be running in the background for it to work properly. It only interacts with the settings.db file.
Furthermore, no other prerequisites should be required for the application to run correctly. It should be able to start as long as you have a valid settings.db in the default directory.
I just pushed out a new micro release which adds proper error handling for some cases, so let me know if anything shows up so I can further take a look into it.
Thanks!

Hello again
First of all thanks for support!!
So, i've just tried the new release and now i get the error %LOCALAPPDATA%\LGHUB is not a directory, but, ofc the directory is there and working like always.. what it could be at this point?

Hey,
Thanks for checking it out in such short notice. It looks like your LOCALAPPDATA environment variable might not be set correctly. Please verify that it is pointing to the correct location. You can check your environment variables by running the following command in a Windows Powershell instance:
Get-ChildItem -Path Env:

This will display all of your env vars formatted into a "Name" and "Value" columns. Simply find the row that says "LOCALAPPDATA" and ensure that it points to the same directory where your LGHUB folder is located.

Well, just checked and it's all right to where we need :/

Hey,
After some more debugging, I think I figured out what wasn't working correctly. I made a hotfix to release it. Please let me know if the issue was resolved.
Thank you very much in advance for your patience and cooperation.

Just tried and seems that APPDATA works now but i'm getting this double error insteat right after opening

Thanks. I updated the release with yet another hotfix. Sorry for the inconvenience.

Let me know if you come across any other problems, or if it's fine to close this issue.

So.. i'm trying it right now.. and IT WORK!! Good job mate!
I like a lot the interface but unfortunatelly there's still an issue..
There's no way to swipe between game profiles..
I'm trying with mouse scroll, by dragging, with kb arrows.. nothing seems to work
Any clue or just something is missing?

Thank you! I appreciate it.
You should be able to scroll through the carousel by either holding down shift and scrolling, or by dragging the little scrollbar which should be above the gray area, where you can edit the game profile properties.
Unfortunately, as far as I'm aware, horizontally scrolling using just the scroll wheel isn't natively available through the UI framework I'm using. Maybe I can come up with some hacky fix for better user experience in the future.
Cheers!

OOOOO niceeee.. the little gray scrollbar is barely visible.. i suggest to add somewhere a tip that tell you about the shift+scroll things..
So far, the only other issue i've found is regarding the windows resize that don't work when you expand.. the app area will remain the same size as it open..
I'll give you some more feedback in the next days after using it and doing some stuffs :D
Anyway really appreciate your work.. i (and a lot more people) really hate how Logi have ruined  and replaced their good old Logitech Gaming Software with this somehow trash GHUB..
Keep it up the good work!!
Cheers!

The window size being static is intentional. It mostly helps with preventing everything from breaking when resizing, and it also allows me to create a design which I know will look the same regardless of where the application is being used.
I would very much appreciate the feedback, thank you!
I'll close this issue for now then. Feel free to reopen it or create a new one.
Have a lovely rest of the day!


css-focus-within: Add Chrome status ID

https://www.chromestatus.com/feature/5363834508279808

Thanks for the humongous status ID update, @cvrebert! In the future though feel free to combine such non-contentious changes into a single PR. :)

You're welcome as always!


Improved: List and Grid (OFBIZ-11345)

According to the definition in widget-form.xsd the use of a combination of a form with type="list" is deprecated in favour of a grid.
Refactor various list forms into grids.
Refactor various list form references in screens.
Improved:
BlogScreens.xml: from form ref to grid ref
BlogForms.xml: from form definition with list ref to grid definition with list ref
additional cleanup

The changes committed in this pull request are in line with guidelines and best practices established over the course of the project under the wings of the ASF, which were applied on merged pull request as recent as 7 days ago.
If you want the guidelines changed, you are welcome to start a discussion in<EMAIL_ADDRESS>You can NOT unilaterally decide what will and will not be merged by other volunteers. That is dictatorship!

I did several of them and often crossed issues afterward, botched work relying on committers to assume, closing


Python 3?

Hi.
What's the story with flatten_json on Python 3.x?  You probably already know that Python 2.x will not be supported much longer.  I'm asking mostly because caniusepython3 identified flatten_json as one of the projects that need to be looked at.
If the compatibility is currently unknown, is there a test suite I can run with a Python 3.x interpreter to see how things look?
Thanks!

I'm using this in python3 without any issues.


Admission webhook can not map to correct AFU ID

Hi guys, I am trying Intel FPGA Device PlugIn with A10 FPGA.
The FPGA resource has report correctly, I checked it by kubectl describe nodes <IP_ADDRESS> | grep -A4 Alloc, the output is :
Allocatable:
  cpu:                                                 32
  ephemeral-storage:<PHONE_NUMBER>180
  fpga.intel.com/af-f7df405cbd7acf7222f144b0b93acd18:  1
  hugepages-1Gi:                                       4Gi

I have succeed run a pod with this yaml file:
apiVersion: v1
kind: Pod
metadata:
  name: test-nlb3
spec:
  containers:
  - name: test-nlb3
    image: intel/opae-nlb-demo:devel
    imagePullPolicy: IfNotPresent
    command: ['nlb3']
    securityContext:
      capabilities:
        add:
          [IPC_LOCK]
    resources:
      limits:
        fpga.intel.com/af-f7df405cbd7acf7222f144b0b93acd18: 1
        cpu: 1
        hugepages-2Mi: 20Mi
  restartPolicy: Never

But if I changed the resources section to:
    resources:
      limits:
        fpga.intel.com/arria10.dcp1.0-nlb3: 1
        cpu: 1
        hugepages-2Mi: 20Mi

The pod will be pending with an schedule error.
The related logs are the following:
Run kubectl logs intel-fpga-webhook-deployment-7f6b8474c-nd64g
2020/04/29 12:15:55 http: TLS handshake error from <IP_ADDRESS>:23786: remote error: tls: bad certificate

Run  tail -f kube-apiserver.log:
failed calling webhook "fpga.mutator.webhooks.intel.com": Post "https://intel-fpga-webhook-svc.default.svc:443/pods?timeout=30s": x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "ca")

Run kubectl describe pod test-nlb3:

Events:
  Type     Reason            Age   From               Message
  ----     ------            ----  ----               -------
  Warning  FailedScheduling  6s    default-scheduler  0/1 nodes are available: 1 Insufficient fpga.intel.com/arria10.dcp1.0-nlb3.
  Warning  FailedScheduling  6s    default-scheduler  0/1 nodes are available: 1 Insufficient fpga.intel.com/arria10.dcp1.0-nlb3.

Also, I wonder the notion of DCP, I noticed there are 3 version of DCPs, I have tried each of them, and had always the same error.  What does DCP exactly mean?

I see you are testing with a local setup. In this case the CA certificate is wrong usually as mentioned in fpga_admissionwebhook's README. You can try deploy the webhook with
$ ./scripts/webhook-deploy.sh --ca-bundle-path /var/run/kubernetes/server-ca.crt

The actual path to the correct CA bundle may vary in your setup.
Honestly I don't remember what DCP stands for, but it is a release of the Intel Acceleration Stack for an Arria10 device which includes firmware (or so called "blue bitstream"). Interface IDs differ for different firmware releases (AFU IDs don't).

Also you might want to follow #358 as it changes the format of the name mappings and the way how the webhook is configured.

@xxinran can you try with the latest master (and follow the new/improved instructions)?

Closing as name mappings format was changed in the latest release.


"Property 'default' does not exist on type '{}'." On Router-view and Router-link v-slot

Repository to reproduce the error
https://github.com/sibyl666/issue
When v-slot is used with router-view it gives the error "Property 'default' does not exist on type '{}'."
this wasn't an issue a while ago with the same vue-router version on another repo, When I check the other repo it shows the same error so that's why I think the issue is related to the extension. Repository created with
yarn create vite issue --template vue-ts

Vue Language Features (Volar) version v0.27.27
TypeScript Vue Plugin (Volar) version v0.27.27

Creating components.d.ts fixed for me. (https://github.com/johnsoncodehk/volar#using)
But I have a different error when noPropertyAccessFromIndexSignature is true.
Property 'default' comes from an index signature, so it must be accessed with ['default'].

Thanks, @sapphi-red! The issue is gone. But I won't close this issue because you have a another issue, or should I close it?

I have the same issue, and adding components.d.ts seems not working for me :(

@ndh103 I added components.d.ts in src folder and pasted
declare module 'vue' {
  export interface GlobalComponents {
    RouterLink: typeof import('vue-router')['RouterLink']
    RouterView: typeof import('vue-router')['RouterView']
  }
}

In it. Maybe you can share repo to reproduce the issue

I have red squiggles in all my v-slots/ #slots in this version. Nuxt + Composition API. Moving to 0.27.26 fixed it. Will comment later if it gets fixed with a future version.

@ndh103 I added components.d.ts in src folder and pasted
declare module 'vue' {
  export interface GlobalComponents {
    RouterLink: typeof import('vue-router')['RouterLink']
    RouterView: typeof import('vue-router')['RouterView']
  }
}

In it. Maybe you can share repo to reproduce the issue

Thank you, I was created the components.d.ts in another folder, putting it in the src folder help solve the problem

@sibyl666 I tried with components.d.ts and it still not work
Here are my repo branch that you can have a look
https://github.com/ndh103/person-hub/blob/features/error-with-volar/person-hub-web/src/views/home/events/EventDetails.vue

Having a similar, possibly related issue with locally-imported components:
<script setup lang="ts">
import { Menu } from '@headlessui/vue'
//...
</script>

<template>
  <!-- -->
  <Menu v-slot="{ open }">
    <!-- -->
  <Menu>
  <!-- -->
</template>

This is using v0.27.27, but I'm unsure if this is normal because v0.26.26 shows the type any instead.
Here is what the component in its d.ts looks like:

Details
export declare let Menu: import("vue").DefineComponent<{
    as: {
        type: (ObjectConstructor | StringConstructor)[];
        default: string;
    };
}, () => import("vue").VNode<import("vue").RendererNode, import("vue").RendererElement, {
    [key: string]: any;
}> | import("vue").VNode<import("vue").RendererNode, import("vue").RendererElement, {
    [key: string]: any;
}>[] | null | undefined, unknown, {}, {}, import("vue").ComponentOptionsMixin, import("vue").ComponentOptionsMixin, Record<string, any>, string, import("vue").VNodeProps & import("vue").AllowedComponentProps & import("vue").ComponentCustomProps, Readonly<{
    as: string;
} & {}>, {
    as: string;
}>;

Its source code is here if it's of any help.
If the "error" is intended on Volar's side, what's the workaround for me to hide the error?

Fixed in https://github.com/johnsoncodehk/volar/commit/d587ab162cf09fe79cec5a0f22624870b7ca08c0, please wait for release or use 0.27.26 for now, thanks!
@innocenzi it's also fixed in this commit.
Please keep this issue open, I will close when I release new version.


Subscribe socket from server

Hello,
I can't find a way to subscribe a socket to a channel from the server. SCSocket.subscribe(channel) is only available on the client. Is there a reason for that?
My use case is:

each channel represent a team ;
when a user do something, all teams he belongs to must be notified ;
a user can not choose which team he's able to subscribe to (this is managed by an admin).

Here is the code I wrote server side which does not work:
worker.scServer.on('connection', socket => {
    const teams = [ 1, 2 ];
    _.each(teams, team => socket.subscribe(`team:${team.id}`)));
});

How do you think I could implement it?

@jfsimon  Hey man use the exchange from the worker. For example:
var broadcastChannel = worker.exchange.subscribe('broadcast');
broadcastChannel.watch(function(data) { // Do something with data })

@happilymarrieddad should I understand there is no way to manage sockets/channels subscriptions from the server? Given your example, how can I send data to the right clients?

@happilymarrieddad I see a solution:

let client subscribe to channels
listen to subscription from server
us SCSocket.kickOut() to cancel unauthorized subscriptions

Let me know if you think there is a better way.
Thanks you for your help!

@jfsimon You can't subscribe on behalf of client sockets from the server directly - It encourages bad habbits.
You can do the approach as you described but the best way is to use the MIDDLEWARE_SUBSCRIBE middleware to block the subscribe request before it is even processed by the SC server. See http://socketcluster.io/#!/docs/api-scserver

@jondubois oh, it sounds great! If I understand, inside the middleware function, I call next() only if the client is allowed to subscribe to the channel, am I right?

@jfsimon next() to pass next(errorObject) to block. You SHOULD call next in any case (eventually) otherwise SC will just hang on to the connection until it times out (which works but isn't ideal).

@jondubois ah, okay. Thank you for the tips, very helpful! I close this issue.


Update name data to support new phase3 fonts.

Also adds support for parsing font file names with 'SemiCondensed',
'ExtraCondensed' variants.  Name expectations are not updated yet.

@marekjez86 this should take care of parsing issues.  The expectations for how the name tables should read are not updated yet (they expect 'SemiCondensed' and 'ExtraCondensed' where 'Condensed' appears, but the fonts have a space between 'Extra' and 'Condensed', and of course the fonts still generate the parts of the name in different locations from where lint expects them.

LGTM


Errors while Loading  Model

Summary
Running on Python 3.6 on MacOS
If a model file doesn't exist, I was having this error thrown:
Unable to create group (no write intent on file)
I then had an error with:
'Group' object has no attribute 'decode'
According to https://stackoverflow.com/questions/28583565/str-object-has-no-attribute-decode-python-3-error
This error was generated due to decoding a string that is already in UTF-8 and is safe to ignore.
This patch fixes both of these things.
Related Issues
https://github.com/keras-team/keras/issues/6005
PR Overview

[ Dunno] This PR requires new unit tests [y/n] (make sure tests are included)
[No ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[ Dunno] This PR is backwards compatible [y/n]
[No ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Could also be related to https://github.com/keras-team/keras/issues/12195

Upon further reflection, this issue is related elsewhere.  It maybe still necessary to open with 'a'


Adjust routing to get to the picture page when clicking on the own trophy score

Not necessary anymore.


Issue with SoundFingerprinting audio fingerprint builder

Hi i tried to use the library, i'm using the latest version on nuget and i'm having an issue with this code:
var hashedFingerprints = fingerprintCommandBuilder.BuildFingerprintCommand().From(pathToFile).UsingServices(service).Hash().Result;
Where it gives me this error:
Reference to type 'AudioSamples' claims it is defined in 'SoundFingerprinting.Audio', but it could not be found
Is it an error in the library or is it my code?
(If it helps it doesn't give the error until i add the pathToFile variable which is a string variable to the .From() method )
Thanks

Its been an issue with the versioning. Lately an effort was put to solve these. All libraries and dependencies has been aligned to v2.3.1 (available on NuGet).


ImportError: cannot import name 'repeat_kv' from 'transformers.models.llama.modeling_llama' when importing BetterTransformer

System Info
Optimum Version: 1.11.2
Transformers Version: 4.31.0

Python: 3.9.10

Who can help?
No response
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction (minimal, reproducible, runnable)
from optimum.bettertransformer import BetterTransformer
Expected behavior
it should work

It's fixed. Upgrade transformers to latest version.


Make spacer growable in order to align control elements

Test case

Open the test page
Load the Angel asset
Enter full screen mode

Expected
The current time element is left-aligned and other controls are right-aligned.
Actual
The current time element isn't left-aligned but has an indentation. For example:

I signed it!

Looks good! Let me run the tests.

Merged! Thanks for the contribution!


settings: Improve permissions notice banners.

Fixes: #24157

Screenshots and screen captures:
Banners for Custom emoji permissions.

Banners for User groups permission

Banners for adding bot permission.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

@alya I have reworded the banners for the following settings:

Invite users settings
user groups settings
custom emoji settings

I did not find any other setting for which banner should be reworded.
I was working on second point of issue too but I am not sure what do you mean by notice colour. Can you tell me any banner which is in notice colour ?

I was working on second point of issue too but I am not sure what do you mean by notice colour. Can you tell me any banner which is in notice colour ?

Ah, sorry that part of the issue wasn't very clear. I was thinking it might make sense to use a color from compose box banners, but let's hold off on that pending a broader settings redesign. So if you can post screenshots, I'll review the changes you've made. Thanks!

@alya I have added screenshots for each possible settings configuration. Please have a look. Thanks !

Thanks! I think the banners should always go in either increasing or decreasing permissions order (e.g., "members, moderators, admins" or "admins, moderators, members", but never "members, admins, moderators").
I'm not sure what's best between the two plausible options. Do we have a consistent pattern if you check other permissions banners?

Also, let's just use "administrators"; I don't think there's a need to say "organization administrators".

Thanks! I think the banners should always go in either increasing or decreasing permissions order (e.g., "members, moderators, admins" or "admins, moderators, members", but never "members, admins, moderators").
I'm not sure what's best between the two plausible options. Do we have a consistent pattern if you check other permissions banners?

I think we should follow because in  #24170  which introduces permissions banners for invitation setting also follows decreasing order.

I think we should follow because in https://github.com/zulip/zulip/pull/24170 which introduces permissions banners for invitation setting also follows decreasing order.

Sure, let's go with that.

@alya I have updated the PR and screenshots. Please have a look.

Thanks! The banners look fine, but they don't live-update if I change the permission settings.

Thanks! The banners look fine, but they don't live-update if I change the permission settings.

I think you must have tried changing permissions of user-groups because on changing permissions of bots and emojis banner gets live update. Regarding user-groups banner I have done discussion for it here.

Ah, I see, that's probably right.
Ok, in that case, @sahil839 could you please review this one? I did light-weight manual testing, but didn't try to validate all the cases or anything.

I tested and it works fine. Live-update too works fine for emoji and bot settings panel.
Regarding the bots panel, there is a PR #24198, which changes the behavior of banners in bots panel and both the PRs might have conflicts.

@sahil839 Does matter in what order we merge the two PRs?

I'd like not to show the banner to admins in personal settings.
@Ujjawal3 perhaps you can reach out to see if any help is needed with https://github.com/zulip/zulip/pull/24198?

I have opened #24562 addressing the changes to be done in #24198. I have to open a new PR since I cannot do changes in existing PR.

As #24562 is merged now I think we should mark it for integration review.

I think would be better to have the changes for showing the banner to admins always in a separate commit from commit doing the rewording and modify the commit messages to explain clearly what changes are being done in what panels.

I think would be better to have the changes for showing the banner to admins always in a separate commit from commit doing the rewording and modify the commit messages to explain clearly what changes are being done in what panels.

@sahil839 I have done the changes in a separate commit.

@Ujjawal3 Looks good to me. The commit message summary can be more specific, especially for the first commit, as we have separated the changes into two commits now.
@alya do you want to have a look at this again?

@alya do we want these to say "so that only administrators can do X"? Those particularly lines feel weird saying that administrators can do something you expect them to be able to do, and I'm not sure all users will understand that the point is that other users can't.

@alya do we want these to say "so that only administrators can do X"? Those particularly lines feel weird saying that administrators can do something you expect them to be able to do, and I'm not sure all users will understand that the point is that other users can't.

Yeah, I think you're right. Let's change that one to "so that only administrators can..."; no changes for the other permissions configurations.

I have done the changes and updated the screenshots.

The screenshots look good to me!
@timabbott Marking for integration review, but let me know if you think I should retest.

Merged, thanks @Ujjawal3!


chore: removed tmate session

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.4%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5087

Relevant Lines:
5733

💛  - Coveralls


androidenv: update packages

Motivation for this change
Update generate.sh to run using nix-shell. Also make it fail with meaningful output instead of writing empty output files.
Generate the latest Android packages.
Update composeAndroidPackages with versions that exist in the generated package set.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Actually this doesn't build when including system images; system-images-google_apis_playstore.nix is somehow using "28" as the key for "Q" images. This is probably in the XSLT, so I'll take a look in there.

Updated with fixes for convertsystemimages.xsl:

Use type-details/codename if it exists, falling back to type-details/api-level: this results in "Q" rather than "28" for preview images
Use <xsl:text> elements to control whitespace in the output.

Hi looks good so far, but I need a bit of time to review this. I need to check whether the testcases still pass and whether the frameworks built on top of it don't break.
Moreover, I may still have to set the default tools version to 25.x. 26.x is recommended nowadays, but it has the disadvantage that it does not include any ant-functionality. As a result, it will break the buildApp {}  function.

Ah, understood. I've been trying to work out how to make Gradle builds possible in sandboxed mode, and it's not an easy problem to solve.

How's this coming? (I re-ran update.sh on my local copy, and got a few duplicates, BTW).
For things like the NDK, I think we should include a couple older versions here (maybe back to r17?)

(BTW: if you want me to take over this PR, I'd be more than happy.)

@numinit I'm happy to integrate this, as long as the default tools package version remains a 25.x. Can you change the pull request so that this version becomes the default again?
Once we have decent gradle build support then the 26.1.x version can be made the default.
Making 26.1.x the default now will break the androidenv.buildApp {} function in the default installation, which IMO isn't very nice.

Any update here?

Most changes in this PR are now in PR #82067 (scripts generate.sh and related) and PR #82118 (actual package updates).
@tadfisher Could you please consider reviewing #82067 , and preparing a branch with any enhancements you envisage (e.g. I noticed you had amended the default versions - probably fruit of some testing) so we get most androidenv-related enhancements merged.

Most changes in this PR are now in PR #82067 (scripts generate.sh and related) and PR #82118 (actual package updates).

@tadfisher Could you please consider reviewing #82067 , and preparing a branch with any enhancements you envisage (e.g. I noticed you had amended the default versions - probably fruit of some testing) so we get most androidenv-related enhancements merged.

I'm closing as all mentioned PRs above were merged.


Add toggler / collapse components

Summary
Added controlled toggler component and uncontrolled collapse component.
The toggler can be used to hook up with a state object to hide and show content.
The collapse component controls its own state, but the initial state can be set by the consumer.
Additional Details
The collapse component serves as a simple button toggle that can also be used as an example for people want to create more advanced togglers (tree views, checkbox toggles, etc)
Deployment Link
https://terra-core-deployed-pr-475.herokuapp.com/
@cerner/terra

@mmalaker Renamed toggler to toggle and collapse to toggle-button

Functional verification completed. All toggle and toggle button examples display and function correctly.

@neilpfeiffer This is ready for UX review
https://terra-core-deployed-pr-475.herokuapp.com/static/#/site/toggle
https://terra-core-deployed-pr-475.herokuapp.com/static/#/site/toggle-button

(Looked at this last night but didn't get my comment added)
Tested: PR #475

 - Follows basic approved patterns for initial implementation of Toggle and ToggleButton

Pass

UX Review


Code : 1005,"Security Data : UsernameToken authentication failed"

I am facing this issue while encrypting card detail while running this sample.
Any help with this will be appreciated. Thank you.

You need to add the SOAP key in the encryption

Facing same issue, can you please check? or guide where I need to update what

You need to add the SOAP key in the encryption

I am using the SOAP transaction key and getting the same error. Any idea what I should use for api_login_id field?

You need to add the SOAP key in the encryption

I am using the SOAP transaction key and getting the same error. Any idea what I should use for api_login_id field?
as found here: https://github.com/CyberSource/cybersource-android-sdk

Managed to successfully encrypt card details.
The api_login_id is your Merchant ID which you can get from the business center

You need to add the SOAP key in the encryption

I am using the SOAP transaction key and getting the same error. Any idea what I should use for api_login_id field?
as found here: https://github.com/CyberSource/cybersource-android-sdk

Managed to successfully encrypt card details.
The api_login_id is your Merchant ID which you can get from the business center

I am getting same error when I am adding amount variable in 'MessageSignature'.
I checked out this repo and just performed basic transaction using this code. In the transaction dashboard, I can see the transaction details but not able to see amount and currency. So in the 'generateSignature' method, while creating string 'signatureComponents' I added amount. On clicking 'Encrypt' its giving this Error Code 1005
 ```
    String signatureComponents = stringHmacSha1
            + loginId + transactionObject.getMerchantReferenceCode()
            + amount    // ----------------- Adding this is giving error 1005 --------------------------------------
            + timeStamp;

Kindly let me know, if this is correct way to see amount in transactions.

I am getting same error when I am adding amount variable in 'MessageSignature'.
I checked out this repo and just performed basic transaction using this code. In the transaction dashboard, I can see the transaction details but not able to see amount and currency. So in the 'generateSignature' method, while creating string 'signatureComponents' I added amount. On clicking 'Encrypt' its giving this Error Code 1005

String signatureComponents = stringHmacSha1
        + loginId + transactionObject.getMerchantReferenceCode()
        + amount    // ----------------- Adding this is giving error 1005 --------------------------------------
        + timeStamp;

Kindly let me know, if this is correct way to see amount in transactions.

After getting user card and billing data, use the inapp sdk to encrypt the user data before passing it to your server for transaction processing, this will help reduce your PCI burden.
To encrypt the data you have to create a message signature (server side) using:

Your soap transaction key, get this from your sandbox/live environment
Merchant ID/API Login Id,
Merchant reference code (this can be any significant string like project name)
UTC timestamp of the transaction.

First create a HMAC-SHA1 of the soap transaction key. Then create a string concatenation of the HMAC-SHA1, your Merchant ID, a Merchant reference code and UTC timestamp:
String signatureComponents = transactionKeySha1 + Merchant Id + Merchant reference code + timestamp;
The MessageSignature class in the inapp sdk samples shows you how to generate the signature in java so you can transform the code to whichever language you need for your server. In my case I did the it in PHP.
After generating a signature you can make a call to the api to encrypt card data passing in your signature:
apiClient.performApi(InAppSDKApiClient.Api.API_ENCRYPTION,
transactionObject, signature);
If your signature is valid your call will be authenticated and processed.
I hope this helps.

I am getting same error when I am adding amount variable in 'MessageSignature'.
I checked out this repo and just performed basic transaction using this code. In the transaction dashboard, I can see the transaction details but not able to see amount and currency. So in the 'generateSignature' method, while creating string 'signatureComponents' I added amount. On clicking 'Encrypt' its giving this Error Code 1005

String signatureComponents = stringHmacSha1
        + loginId + transactionObject.getMerchantReferenceCode()
        + amount    // ----------------- Adding this is giving error 1005 --------------------------------------
        + timeStamp;

Kindly let me know, if this is correct way to see amount in transactions.

You don't need to add amount. If you loo carefully in the MessageSignature.java, you will see that they did not use amount.


ipython example breaks at Tensorflow 1.0.1

I'm running tensorflow 1.0.1 and tried the ipython notebook in the example folder. But it break at several points. It seems code is broken because of incompatibility of tensorflow versions. Which version of tensorflow have you used to developed your code? And is it possible to migrate this to Tensorflow 1.0?
Thanks for the great work by the way. It's useful.

Hi there and thanks for pointing this out. I think the whole RNN-api has been restructured for the better and the TF-community has developed more good habits w.r.t how tf-code is written, something the ipynb at current doesn't reflect so the best is to just delete it.
Check out the branch wtte-rnn-0.0.2
and in particular the new examples.
The data-pipeline is something of a WIP but I think the discretely padded pipe is working quite good, I just haven't gotten varying-length sequences down yet without using batch_size=1.
I'm going to be mainly focusing on getting a pipeline with continuous data similar to the Phased LSTM going but there's some math around it. Blog post coming up.
Would dearly welcome PRs and comments and don't be shy to (especially) comment on style and structure of the repo.

Passes with tensorflow 1.0.1
 wtte-rnn/python/wtte/objectives/tensorflow.py
 wtte-rnn/python/tests/test_tensorflow.py


Required fields not honored when ordering a service

Go to the service catalog, choose a service to order that has a dialog attached to it with required fields. Ensure the required field(s) are not filled out, and observe that the service is still submitted "successfully".

@chalettu - can you take a look at this one?

@AllenBW here you go!

Tested again for @AllenBW since this is pretty old. Works now, so closing.


Clarify specs for Checkbox, Radio and Switch components

Following the message of @michael-zimmermann here we realised

a difference between specs in figma and zeroheight of the checkbox
a difference between what has been specified for checkboxes and radio buttons and switches

this task's goal is to understand why and to unify the specs

list of differences here : https://www.figma.com/file/0QchRdipAVuvVoDfTjLrgQ/Spark-Specs-and-Handoff?type=design&node-id=10014%3A24385&mode=design&t=dXwli0bltLrZhwtn-1

Specs have been updated


Study Week 2

Points
Extra Credit for Acting/Skit/Improv ASI:  0 to +2 in Seed, -1 for not participating.
GH Indi Page, Review Ticket, Crossover graded

[ ] MCQ results > 50%, (1 or 0)
[ ] FRQs results > 50% (1 or 0), evaluation considers Paper code pictures vs REPL running code.
[ ] PBL (0, .5, .75, 1).

Scoring guidelines for crossover graders 4.5/5.0 maximum

[ ] 3 points, use bullets as indicated above, 1 point per sub-bullet
[ ] 1 point, Participation: .20 for classroom presence and participation each day
[ ] .5 point, Quality of work:  grader should be looking at effort toward learning and understanding vs perfunctory bullet chasing (0, .25, .5).

Link to all

4.5/5

MCQ is completed, and corrections are also completed
FRQ isn't needed, not a test taker
All the links are centralized to a single location
Review ticket is well organized
Commits are named well, described,  and work towards the central goal of the PBL


Returning a question redirects to different intent

Since the Dialog interface isn't yet available, I'm home rolling my own with a whole bunch of ifs. The intent in question requires 4 inputs. I have cases in there for getting each one and saving the rest off to the session while gathering. You can see this intent here. There is similar intent with 5 inputs here.
I'm testing by trying to go sequentially: Alexa, ask infant tracker to add a diaper -> What is the child's name? as expected. But then when I give the name, I get a request to the AddChildIntent intent here instead of being sent back to the AddDiaperIntent. Why would this happen?

From my experience, having very short utterances where it's mostly a slot value can easily lead to this situation. If you're going to do short variable (slot-based) utterances as responses to questions, you'll need an intent designed specifically for receiving the value and then using a session attribute to decide what logic to trigger.
In your case you might need to make AddChildIntent  add {Name} a separate intent related to a prompt for a name that might be used across other states of the app. (The other AddChildIntent utterances are probably ok since they include more words than just a slot.)
I ran into this design issue previously when prompting a user for selecting a particular podcast program to play. My case was simpler, so the "select" intent just called the function dealing with handling the "play" intent. The "select" intent was an utterance defined to be match when a user spoke just the program name. (Technically, at any time for any reason, they could just say the name of a correct program and it would start playing it as a result, but this would allow us the spot to put the check for session attributes to see the state of the dialog if needed.)
Just an aside: Flask-Ask has really nothing to do with the intent chosen by Amazon, if people are wondering.

Yeah I figured this was on Amazon's end. I'm guessing the new Dialog interface would solve this problem.
So basically I need to split all of the dialogs out into separate intents? Then based on a session variable I route it to the proper intent?


chore: lets disable checks by default and enable desired ones using h…

…elm values

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rawlingsj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-bot-check
9e1785425982b584bbacd9eb36c5078421f78fd2
link
/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@rawlingsj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-bot-check
f9af497cf124dd463000611ed5ff7c9513325900
link
/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@rawlingsj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-bot-check
560aabfe5605e089c89acfa29f729a3e8427c651
link
/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest bot-check

/retest

@rawlingsj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-bot-check
560aabfe5605e089c89acfa29f729a3e8427c651
link
/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/test bot-check

@rawlingsj: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr-bot-check
560aabfe5605e089c89acfa29f729a3e8427c651
link
/retest

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


ARTEMIS-292 - fix isSameHost on TransportConfiguration

Ive renamed the current isSameHost method to isSameparams as thats what it checked and added a new method for isSameHost that cjecks the appropriate params for the connector. Ive changed ClientSessionFactoryImp to use this to correct the behaviour.
https://issues.apache.org/jira/browse/ARTEMIS-292

there's a typo on your commit, you want to fix it?


[WIP][Clojure] Add methods based on NDArrayAPI/SymbolAPI

Description

This PR generates new methods based on the newer NDArrayAPI and SymbolAPI. It's still a work in progress.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] NDArrayAPI methods, tests, (and when applicable, API doc)
[ ] SymbolAPI methods, tests, (and when applicable, API doc)

Reviewers
@gigasquid

@mxnet-label-bot [Clojure]

Thanks for looking into this. It will be interesting to see how it comes out. The new API might be more difficult to interop with than the regular data structures of the current one.

@gigasquid it does seem like the new API is somewhat more cumbersome in terms of interop. the main benefit i see is the type checking but while in scala you can call the functions with named optional arguments (e.g. Dropout(ndarray, p = Some(0.25), out = Some(outputArray))) it's not so straightforward to have this in clojure.

@kedarbellare Thanks so much for looking into this. I thought it might be the case that it might not be as nice to use. That's totally fine. We can stick to the other API for now and if for some reason we need to move off of it (like the Scala package deprecates it) , we can think of a different strategy, like moving a level down for NDArrary and Symbol and using the Scala Generator functions to generate our own functions directly using the JNI calls instead.
For example - getting all the backend functions by calling the JNI
https://github.com/apache/incubator-mxnet/blob/master/scala-package/macros/src/main/scala/org/apache/mxnet/GeneratorBase.scala#L63
and these to create the functions
https://github.com/apache/incubator-mxnet/blob/master/scala-package/macros/src/main/scala/org/apache/mxnet/GeneratorBase.scala#L73
Let me know what you think.

Makes sense. I'm working on unit tests for NDArray API right now which should give an idea of how we could start using it. I won't put too much effort into doing the same for Symbol API. It'll be great if we can start using JNI directly but I don't have much experience on that front.

@kedarbellare Thanks for tackling the toughest part of the whole Clojure package 💯  I think you have made some really nice progress and we can definitely leverage some of this work.
Since we are starting a new ndarray api, we have a nice opportunity to improve upon the current api and make it more useful for Clojure users. I agree with your assessment that the typing makes things better, but all the positional arguments and options make it more cumbersome. This is due to the philosophical differences between Clojure and Scala, where Clojure loves data structures and Scala loves types :) I'm thinking we should take a step back and think about what we (the clojure package) would want and then try to shape what you have into it. After looking at what you have, I think there is a path forward with maybe generating only the arity with all the parameters and then taking a data structure of vectors or maps to get there.
I think it would be nice to outline what that API should look like with a couple examples from the NDArray and Symbol API before we code it up too much. Since you have experience and good feeling for what is possible now with your work, are you up for helping to draft up the future look of the API on the wiki?

Yep. I wholeheartedly agree about the philosophical differences between Scala and Clojure and I completely see the value in doing what's best for the Clojure package. I can try to take a stab at creating an outline so that we can add it to the wiki and ask for comments. Note, however, that I mostly work on this project during the weekends or when I have free time so I'll try my best to put something together soon (over the next week or so). Please let me know if you want to have something out there ASAP, in which case it may not make sense for me to work on it.

@kedarbellare Definitely no rush - whatever pace works well for you. I appreciate your efforts and your contributions to improve the Clojure project.
I don't have rights to create accounts on the wiki so I'll ask out on dev for access for you.
Thanks again!

@kedarbellare You should have access now - ping me if you have any trouble 😸

@mxnet-label-bot update [Clojure, pr-work-in-progress]

@gigasquid i've improved the api generation for both ndarrays and symbols. PTAL thanks!

Awesome work!!! I look forward to reviewing this 😸

@kedarbellare - I think the next step after this is to try to get https://github.com/apache/incubator-mxnet/blob/master/contrib/clojure-package/test/org/apache/clojure_mxnet/conv_test.clj to use the new symbol-api. It will probably shake out some further refinements.
It certainly is a much nicer api to use and would love to get it to the point were we could deprecate the old symbol and ndarray way of doing things 😸

I'm also not sure why the unix-gpu keeps failing from CI, might want to try to rebase on master and see if that helps

Oh @kedarbellare - one more thing... Would you mind marking the new api namespaces as Experimental ? That way users know that it is in active work and might change and not be stable yet.
Thanks!

@Chouffe @gigasquid : thanks for the detailed feedback! i'll take care of these issues soon (hopefully by end of the week).
@gigasquid : where should i mark the namespaces as Experimental? in the docs at the top or in the individual docs of the functions or somewhere else?

@kedarbellare marking it as Experimental in the namespace docstring of contrib/clojure-package/src/org/apache/clojure_mxnet/ndarray_api.clj - or a comment should be fine. And in the corresponding symbol-api namespace. Thanks!

We should probably start rewriting some of the examples with this once it gets merged.
I can help with this @kedarbellare @gigasquid :)

@gigasquid @Chouffe i've updated the PR with the following:

addressed cleanup comments
tagged the new apis as "Experimental"
modified the conv_test.clj to use the new symbol-api and fixed a small bug that i discovered during that
tweaked the coerce-param for symbol-api (especially for the attr param)
rebased to the latest upstream/master to see if that fixes the unix-gpu CI failure

PTAL thanks!


When will the code be released?

Hi Jiwoon Ahn, congratulations! I'm really interested in your code and can't wait to try it out. So when are you going to release the code? Thank you! The paper was great!

I deeply sorry about the delay... I can assure you the code will be released this week.

I deeply sorry about the delay... I can assure you the code will be released this week.

Great to hear! Thank you!


Update logging docs

We've gotten feedback these docs are confusing and we are trying to make improvements:

Tighten up the initial text to have fewer warnings, caveats and tips so we get to key concepts and examples faster
Lead off with a minimal example to establish basic concepts. I tried experimenting with including more concepts in the first example like DI or source generated logging, but it felt both easier to understand and more concise starting with a low concept count example and then building out additional concepts and variations from there.
Removed other extraneous C# features from the initial example as not all C# developers are familiar with them: BackgroundService, async/await, cancellation tokens.
Added an explicit recommendation to use source generated logging. We probably should use it more in other examples, but this is a start.
Tried to make the the article a little more balanced between hosted and un-hosted approaches. For the original ASP.NET docs it made sense to prioritize hosting and DI, but for the general purpose doc we should show this API can be equally applicable to all app models. I don't think it is even yet, but the balance is closer.

Internal previews

📄 File
🔗 Preview link

docs/core/extensions/logging.md
Logging in C# and .NET

docs/core/extensions/snippets/logging/aspnet-mvc/wwwroot/lib/jquery-validation/LICENSE.md
docs/core/extensions/snippets/logging/aspnet-mvc/wwwroot/lib/jquery-validation/LICENSE

@IEvangelist @reyang @tarekgh @samsp-msft @davidfowl - could you guys take a look at this and let me know what you think?

I just noticed too, now that we've updated the source a bit - we'll have to fix the range="" usage. With code examples that are not fully rendered, i.e.; if we're doing ranges only for a given file, we could instead use id="some-dentifier".
Here's an example:
// <main>
static void Main(string[] args)
{
    using ILoggerFactory factory = LoggerFactory.Create(builder => builder.AddConsole());
    ILogger logger = factory.CreateLogger("Program");
    LogStartupMessage(logger, "fun");
}
// </main>

// <logger-message>
[LoggerMessage(Level = LogLevel.Information, Message = "Hello World! Logging is {Description}.")]
static partial void LogStartupMessage(ILogger logger, string description);
// </logger-message>

We could render each section independently, by using an id.
:::code source="path/to/file.cs" id="main":::

This would render as:
static void Main(string[] args)
{
    using ILoggerFactory factory = LoggerFactory.Create(builder => builder.AddConsole());
    ILogger logger = factory.CreateLogger("Program");
    LogStartupMessage(logger, "fun");
}

I just noticed too, now that we've updated the source a bit - we'll have to fix the range="" usage. With code examples that are not fully rendered, i.e.; if we're doing ranges only for a given file, we could instead use id="some-dentifier".

I attempted the id technique in one spot but got this warning from CI: "Line 70: [Warning] It looks like your code snippet was not rendered. Try range instead." Its possible I did it wrong but I switched it back to range as suggested and I think the snippets are correctly displayed and highlighted now.

Fingers crossed all the feedback has been addressed. Let me know if there is anything else you'd like me to take care of. Thanks!


Bump the symbolic libraries and fix deprecations

This PR was not passing CI, so I reopened it.
@ChrisRackauckas I would greatly appreciate if PRs which change the code (not only compats) are merged after I look at them.

This PR passed all tests? And it was just a depwarn fix?

This PR passed all tests? And it was just a depwarn fix?

Yes, you can say so: it was failing "Downgrade" test because of the devs inconsistency

Downgrade CI currently has some false negatives because it needs a different resolver, which will hopefully be something added in the next few months.

Glad to hear about these developments in Downgrade. In this case, it actually spotted a mismatch, I have fixed it anyway.


Upgrade next from 12 to 13 #266

Used ,,npm i next@latest react@latest react-dom@latest eslint-config-next@latest" & ,,npx @next/codemod new-link" and works well.

could you check why build is failing?

could you check why build is failing?
I have a problem with Vercel account so Im not able to tell you now 🤷

I have a problem with Vercel account so Im not able to tell you now 🤷

you don't have access to vercel account so you can't see the logs, please try building the app locally. I strongly believe you can see the errors there. thanks

adriangasiorek@MacBook-Pro-Adrian projectmate % npm install
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/next
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer next@"*" from<EMAIL_ADDRESS>npm ERR!   node_modules/next-themes
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/next-auth
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer next-auth@"^4" from<EMAIL_ADDRESS>npm ERR!   node_modules/@next-auth/prisma-adapter
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/next
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/next-auth
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer next-auth@"^4" from<EMAIL_ADDRESS>npm ERR!     node_modules/@next-auth/prisma-adapter
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/adriangasiorek/.npm/eresolve-report.txt for a full report.
npm ERR!      /Users/adriangasiorek/.npm/_logs/2022-11-13T12_14_41_280Z-debug-0.log
adriangasiorek@MacBook-Pro-Adrian projectmate %
But it works after npm install --force

@Xoviec yeah we are aware of that issue. will fix it later. as of now --force is good to go.


Bucket Sort

Add comment to get assigned to your desired language.
Submission Title
Bucket Sort
Submission Type
Algorithm
Submission Description
Write the Bucket Sort Algorithm
Submission Directory
Algorithm => Sorting => Bucket _Sort => Bucket_Sort.extension
Language

[x]  C++
[x]  C
[x]  Python
[x] Java
[x] JavaScript
[ ] C#

Hello. I would like to be assigned to do this in C.

I would like to do this in java. Can you please assign it to me?

I would like to do this in java. Can you please assign it to me?

@Priyanshi-3
Assigned for Java

#103 BucketSort.java created. Please review.

#103 BucketSort.java created. Please review.

Done

please assign this to me for cpp

please assign this to me for cpp

@Abheetarora Assigned for C++

Can you assing this to me for python language

Can you assing this to me for python language

@leli-1 Assigned for Python

Add comment to get assigned to your desired language.
Submission Title
Bucket Sort
Submission Type
Algorithm
Submission Description
Write the Bucket Sort Algorithm
Submission Directory
Algorithm => Sorting => Bucket _Sort => Bucket_Sort.extension
Language

[x]   C++
[x]   C
[x]   Python
[x]  Java
[ ]  JavaScript
[ ]  C#

Can you assign this to me for JAVA

Can you assign this to me for JAVA

@Neha2827 Assigned for Java. Someone already submitted in Java, please name your file little different.

Can I take this for Javascript?

@redgardner Assigned for Javascript

Please check https://github.com/krishrahul98/DSA-Library/pull/279 @krishrahul98 @hemanth-kotagiri

can you assign it for c++?


support PeerTube

Fediverse YouTube!
Not sure how much we currently interop, but we did hit this error today when fetching this video post and converting it to AS1.

Example AS2, for https://spectra.video/videos/watch/1484abab-d649-45d0-8c50-d761382ac4e7 . A few thoughts on first glance:

No link to the video itself! The only way to actually watch this is to open the id URL in a browser. So much for other clients 😐
No author info! attributedTo is empty
No @context 😎
No url
type: Video but mediaType: text/markdown 🤪
The comments, likes, shares collection links are interesting!
duration is an ISO 8601 duration.  Full log:
Running send task<PHONE_NUMBER>099621783
Params: [('obj', 'ahBicmlkZ3ktZmVkZXJhdGVkcnMLEgZPYmplY3QiZ2h0dHBzOi8vc25hcmZlZC5vcmcvMjAyNC0wNC0zMF9idWlsZGluZy1icmlkZ2VzLXRvLXRoZS1mZWRpdmVyc2Utd2l0aC1yeWFuLWJhcnJldHQtZmxpcGJvYXJkLXBlZXJ0dWJlLTIM'), ('orig_obj', 'ahBicmlkZ3ktZmVkZXJhdGVkckoLEgZPYmplY3QiPmh0dHBzOi8vZmxpcGJvYXJkLnZpZGVvL3cvMGUwMTgwYjMtODFlNy00N2I5LTk4NTMtYTA2YWU0NDg0ZDI3DA'), ('protocol', 'activitypub'), ('url', 'https://flipboard.video/accounts/flipboard/inbox'), ('user', 'ahBicmlkZ3ktZmVkZXJhdGVkchkLEghNYWdpY0tleSILc25hcmZlZC5vcmcM')]
Sending https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2 AS1: {
  "objectType": "activity",
  "verb": "like",
  "published": "2024-04-30T21:08:49-07:00",
  "content": "Likes https://flipboard.video/w/0e0180b3-81e7-47b9-9853-a06ae4484d27.\n
\nBuilding Bridges to the Fediverse, with Ryan Barrett – Flipboard PeerTube\n
",
  "url": "https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2",
  "object": "https://flipboard.video/w/0e0180b3-81e7-47b9-9853-a06ae4484d27",
  "actor": {
    "objectType": "person",
    "displayName": "Ryan Barrett",
    "url": "https://snarfed.org/",
    "image": [
      {
        "url": "https://secure.gravatar.com/avatar/947b5f3f323da0ef785b6f02d9c265d6?s=96&d=blank&r=g"
      }
    ],
    "id": "snarfed.org"
  },
  "id": "https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2"
}
ActivityStreams 1 only supports single author; dropping extra attributedTo values: [{'type': 'Group', 'id': 'https://flipboard.video/video-channels/dot_social'}]
Determining protocol for id https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2
  Key('Object', 'https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2') owned by source_protocol web
Sending AS2 object: {
  "published": "2024-04-30T21:08:49-07:00",
  "content": "Likes https://flipboard.video/w/0e0180b3-81e7-47b9-9853-a06ae4484d27.\n
\nBuilding Bridges to the Fediverse, with Ryan Barrett – Flipboard PeerTube\n
",
  "url": "https://fed.brid.gy/r/https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2",
  "actor": "https://fed.brid.gy/snarfed.org",
  "id": "https://fed.brid.gy/r/https://snarfed.org/2024-04-30_building-bridges-to-the-fediverse-with-ryan-barrett-flipboard-peertube-2",
  "@context": "https://www.w3.org/ns/activitystreams",
  "type": "Like",
  "object": "https://flipboard.video/videos/watch/0e0180b3-81e7-47b9-9853-a06ae4484d27",
  "cc": [
    "https://flipboard.video/accounts/flipboard",
    "https://flipboard.video/video-channels/dot_social",
    "https://www.w3.org/ns/activitystreams#Public",
    "https://flipboard.video/accounts/flipboard/followers"
  ],
  "to": [
    "https://www.w3.org/ns/activitystreams#Public"
  ],
  "contentMap": {
    "en": "Likes https://flipboard.video/w/0e0180b3-81e7-47b9-9853-a06ae4484d27.\n
\nBuilding Bridges to the Fediverse, with Ryan Barrett – Flipboard PeerTube\n
"
  }
}
Signing with Key('MagicKey', 'snarfed.org')'s key
requests.post https://flipboard.video/accounts/flipboard/inbox {'data': b'...', 'auth': <httpsig.requests_auth.HTTPSignatureAuth object at 0x3e42a68b64d0>, 'headers': {'Date': '...', 'Host': '...', 'Content-Type': '...', 'Digest': '...'}, 'gateway': True}
Received 403: {
  "type": "about:blank",
  "title": "Forbidden",
  "detail": "ActivityPub signature could not be checked",
  "status": 403,
  "error": "ActivityPub signature could not be checked"
}


Amethyst Plus (Paid): What is it?

Origin
"I do not mind paying $5 (in Lightning) a month for Amethyst to get costly features. Maybe some form of paid Amethyst?"
<EMAIL_ADDRESS>note1u2hwm0petfn7m0snm70037ymfmevvcu8hqly0gmmqmqcp3kfa87q4gv9um
@vitorpamplona seems to not sleep, and codes 24/7 to give around an expected 1-2 releases per day. Amethyst users have expressed interest to support Vitor, and to potentially receive a few exclusive features in return for their patronage.
What could Amethyst Paid include?

Capability to upload media
Amethyst paid relay
Amethyst supporter badge

"Maybe a little nice icon or circle on profile pic and maybe access to a private relay for starters?"
note109emwktqsxldh0zr3lqnmwnxur82854tf6nkecellfpcnlqgj8tsdegz70

Make costs public [of Amethyst server costs], and add zap/pay capability
"Make the cost and a wallet that will pay for it viewable to the public and we will make sure the bills get paid."
note15fxv5nkymg0nejah7lzdx58kav34n63vqvyu2dgxrwxfzhhkvukqf5dnzj

What should Amethyst Paid not include
"I would want to pay for Vitor to work on Amethyst, not to distract him from it by requiring him to operate a relay or other services."
"The badge would suffice imho."
-@donnyxray
Please add suggestions to this ticket below

I would want to pay for Vitor to work on Amethyst, not to distract him from it by requiring him to operate a relay or other services.
The badge would suffice imho.

Thank you!
I really want to avoid creating classes of users.
What about this: Ship the exact same app for paid and unpaid users. Everything stays free and open source, fully open.
Those who can help, make monthly payments via Zaps to a new Amethyst npub and get an "Amethyst Supporter" (Sponsor?) badge on their profile pictures that shows up to everyone else using Amethyst. The list of sponsors is just the Zap list on the new account's profile page.
If this gets to a big amount (enough to spike the ire of the IRS), I will create some formal structure to receive the funds.
In that way, we make sure there is no preferred user base while assuring some resources to a never-ending app :)
Maybe this "display of goodwill" through badges can help other projects to finance themselves as well?

We can do streaming as well, but that requires an LN wallet in the app that needs to be recharged. :(

As you said, I really appreciate that everything is OSS and there is no "different user classes".
Just my 2 cents idea: what about displaying the "Amethyst supporter" badge, with whatever that means (something in the profile saying it, a different style for the avatar,... but mostly something symbolic), after doing a donation of XXk sats, and that will stay active for 30 days, and then it's gone. So, you will keep your supporter "benefits" as long as you manually send that amount.
This way you don't need to worry developing the recurring payment complexity, and just need to watch for a payment done by a certain user, and that user will receive the badge valid for 30 days starting by the payment day.
This would be an easy first step, and you could iterate over this idea after seen how the people like it and use it.

This would be an easy first step, and you could iterate over this idea after seen how the people like it and use it.

I can tell you that I would find this pretty annoying. The donation you make is essentially converted into code. That code does not expire after a set time, so I don't see why the badge should expire either. Which is not to say that I would not want to donate repeatedly, I know Vitor needs to keep eating, but I hate being a slave to my device and have it push me to do things.
My alternative proposal would be to create a new badge once a year or so. And perhaps additional ones at special occasions. That way people are encouraged to donate repeatedly without losing the appreciation that they donated before. In fact, it would reward those that started donating early by giving them a badge that later donations cannot buy.

I agree that a monthly manual payment is too annoying. Yearly might be too big? I am thinking 100K sats as the threshold for the yearly badge, which is around 25 dollars today: basically 2 dollars/mo.
What do you folks think? Too high? Too low?

What about introducing different level of yearly supporters? like 50k for a bronce, 100k silver, 200k gold and 500k platinum. And that will be reflected in the border of your avatar.
And, I like what @donnyxray said, if I do a donation, I like the idea of keeping it there forever, like a badge that won't get lost after time passes. So there are 2 things to consider: the current "supporter status" and the historical ones.

Closing this discussion since we went with the value4value "Zap the Devs" model


feat: add support for voice messages

Please describe the changes this PR makes and why it should be merged:
If applicable, please reference Discord API Docs PRs or commits that influenced this PR:

https://github.com/discord/discord-api-docs/pull/6082

This PR needs an update with recent additions from discord


[nrf5] support make -j

This commit adds build-chip to dependecies of application's link targets
so that it will not output error for missing those targets if -j is
given.
Problem
Run make -j under gets the following errors:
make: *** No rule to make target '/usr/local/google/home/xyk/Code/connectedhomeip/examples/lock-app/nrf5/build/chip/lib/libDeviceLayer.a', needed by '/usr/local/google/home/xyk/Code/connectedhomeip/examples/lock-app/nrf5/build/chip-nrf52840-lock-example.out'.  Stop.
make: *** Waiting for unfinished jobs....

Summary of Changes

add build-chip to dependencies of app's link targets
use $(MAKE) so that it can get make parameters.

 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.


An error with secret path

Hello,
I got a NPE right after launching the jar.
Based on this thread :
https://stackoverflow.com/questions/52100532/how-to-read-vault-kv-with-java-spring-boot/52145745#52145745
I used :
VaultResponse response = vaultTemplate.read("/secret/data/github");
To make it work.
Could you please update the spring guide and provide an explanation for that ?
Thank

If you would like us to spend some time helping you to diagnose the problem, please spend some time describing it. Providing Vault versions and some stack trace are a bare minimum.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Can not typing Chinese in Integrated terminal

VSCode Version: 1.2.0-insider
OS Version: OS X El Capitan 10.11.5

Steps to Reproduce:

by default settings

Cannot type cyrillic as well

Upstream issue https://github.com/sourcelair/xterm.js/issues/124

This is one of the things issues I want to focus on for v1.4


test: racheting coverage

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Ryan I know repokitteh auto-assigns but I'll leave this as a draft until it passes and ping you when it does.

OK, passing :-)


Sitemap can be read, but has errors

When I try to add sitemap in Google search console, I got an error. The error shows as follow, I don't know how to solve this.
Actually, I can visit my website by replace the loc item http://example.com/~ into https://mysite.top/~.

<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
<url>
<loc>http://example.com/Others/Useful-Websites-for-Programmers/</loc>
<lastmod>2020-12-20</lastmod>
</url>
<url>
<loc>http://example.com/about/index.html</loc>
<lastmod>2020-12-20</lastmod>
</url>
<url>
<loc>http://example.com/Research/Recommend-some-softwares-for-scientific-research/</loc>
<lastmod>2020-12-18</lastmod>
</url>
</urlset>

Okay, I solved it through updating url in _config.yml.


How to implement source codes for one item

Hello,
How can i implement your source codes for just one subitem?
Thank you

Hi,
If you want to use just one subitem then you need to modify in the viewDidLoad the fill of the items like in the following way:
for (var i = 0; i < 10; i++) {
        topItems.append("Item \(i + 1)")
        actualPositions.append(-1)

        var items = [String]()

        // You can change the number of elements here to 1 for example.
        for (var i = 0; i < 1; i++) {
            items.append("Subitem \(i)")
        }
        self.subItems.append(items)
 }

Note the comment in the code.
I hope this help you.

Hi,
Thank you very much for your help.
Best regards
Haydar


WIP for microsoft teams module

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:


Square boxType is pixelated on iPhone 6 Plus

Only found in iPhone 6 Plus, both simulator and device.
Screenshot was taken from device, lineWidth = 2 (tried 1 as well and looked the same).
Great work btw! Thanks!

I've seen the same thing, but with a 5s running iOS 9.2

This seems to be a duplicate of #13, and should be fixed with the release of 1.2.0.
Please reopen this bug if not fixed.
Thank to you two for your contribution!


go/ast: trailing func comments mishandled in CommentMap

Working example: https://play.golang.org/p/TlsDxVbGow
Prints: g: "trailing comment in f\n"
Seems like the correct output should either be nothing (if it is associated with _ = x), or f: "trailing comment in f\n".
What's currently happening is that the "_ = x" in func f is the most recent "important" node when we encounter the comment. Since the comment isn't on the line immediately following "_ = x", it doesn't
get associated with it. We don't look further up the "important nodes" stack to notice that it's within the bounds of f. Instead, we just give up and associate it with g.
/cc @griesemer

Feel free to take a shot at it.

Is there a way to have it associated with _ = x or f while still following the three current rules? Or would a new rule need to be added?
- g starts on the same line as n ends

Not applicable in this circumstance. This is the foo := bar // baz case, as I understand it.
- g starts on the line immediately following n, and there is
  at least one empty line after g and before the next node

The line break between _ = x and the comment prevents this one from applying.
- g starts before n and is not associated to the node before n
  via the previous rules

At this point, g can only be attached to a node after its appearance.


Feature Proposal: Layout Anchors

iOS has a great concept of layout anchors. You can for instant link control A's top anchor to the bottom of control B. So if control A moves down, control B's top will move down with it.
This is a bit like the relative layout panel, but without the need for a relative layout, and provides a lot of additional flexibility.
And it becomes a lot more advanced once you start exposing additional anchors in your control besides the normal top/bottom/left/right/center/width/height anchors.
For example, you can start exposing anchors of internal elements:
public class MyControl : Control
{
    public Anchor TopOfTextAnchor => _internalTextElement.TopAnchor;
}

Now it becomes possible to align other controls with specific parts of a custom control's internal design elements. This is extremely powerful to make great connected layouts, especially during animation etc, or if the internal text element gets larger, or moves.
public class FrameworkElement
{
    public Anchor TopAnchor { get; }
    public Anchor BottomAnchor { get; }
    public Anchor LeftAnchor { get; }
    public Anchor RightAnchor { get; }
    public Anchor HorizontalCenterAnchor { get; }
    public Anchor VerticalCenterAnchor { get; }
    public Anchor WidthAnchor { get; }
    public Anchor HeightAnchor { get; }
}

The anchor class would also provide an offset value. So you could align the top of one control with the bottom of another, and add a bit of extra margin, or ensure they have the same size. Example:
<TextBox x:Name="tb1" />
<TextBox x:Name="tb2"
       TopAnchor.ConstrainTo="tb1.BottomAnchor, Offset=50"
       WidthAnchor.ConstrainTo="tb1.WidthAnchor"  />

@chrisglein @MikeHillberg as FYI.

@chrisglein @MikeHillberg as FYI.

How different is this to the RelativePanel, which also does positioning based on other elements in the panel?

How different is this to the RelativePanel, which also does positioning based on other elements in the panel?

@mdtauk It would negate the need for relative panel (and be able to be used anywhere), but the main difference is in the first example I provided, where a control could have multiple custom anchors in addition to just the edges.

@mdtauk It would negate the need for relative panel (and be able to be used anywhere), but the main difference is in the first example I provided, where a control could have multiple custom anchors in addition to just the edges.

I think this is a great idea - this would create a ton of flexibility in layouts and make it easier to create more complex layouts. I also like the idea of internal elements having anchors, which would definitely help with animation, but also would generally provide new functionality in terms of aligning things exactly the way you want them.
Anchors may also provide some cool stuff in the way of navigation - I'm thinking about how in html/js you can use an anchor to scroll or navigate directly to a part of a page. This may be useful in WinUI as well, especially when I think of apps that use infinite scroll and things of that nature.
Conceptually, I'm thinking of this sort of being like making the entire app one big RelativePanel, but with more specific and targeted "relations" for each element. @ranjeshj Any thoughts on the performance impact of something like this?

I think this is a great idea - this would create a ton of flexibility in layouts and make it easier to create more complex layouts. I also like the idea of internal elements having anchors, which would definitely help with animation, but also would generally provide new functionality in terms of aligning things exactly the way you want them.
Anchors may also provide some cool stuff in the way of navigation - I'm thinking about how in html/js you can use an anchor to scroll or navigate directly to a part of a page. This may be useful in WinUI as well, especially when I think of apps that use infinite scroll and things of that nature.
Conceptually, I'm thinking of this sort of being like making the entire app one big RelativePanel, but with more specific and targeted "relations" for each element. @ranjeshj Any thoughts on the performance impact of something like this?


begin on rules is not optional

The following rule should be able to be parsed, but it generates a syntax error:
rule
  x := !x;
end;

Unsurprisingly the cause of this is an ambiguity in the grammar. Bison recognises the token x as an expression and expects ==> to follow. What is surprising is that it turns out this bug also exists in CMurphi. One of my attempts at fixing this was to make begin non-optional on simple rules, however some of the CMurphi examples actually depend on this being optional. I don't know if users coincidentally never ran into this bug or simply realised that adding begin worked around it and didn't think anymore about it.
I don't see how to resolve this in the current implementation, because it requires more than one token of lookahead which Bison does not support in the lalr1 skeleton. Ultimately it may even be preferable to use a hand written parser (yuck). Resolving this one as WONTFIX as it seems this is simply intended behaviour.

Re-opening as this is still an issue. We may need to switch to a GLR Bison parser to resolve this.

Just an update that this is still an open problem. Cursory attempts to switch to Bison’s GLR skeleton indicate that its C++ variant is immature. It does not support several options that we are using.


Building a book fails without gatsby actively serving

Hi @SaraVieira,
First of all, thanks for this awesome project 👍
I'd tried to play with it and build a book. Node failed with errors mentioning that localhost:8000 is not found.
So I'd run the serve command for gatsby and then re-run the build command - and it succeeded.
What I suggest is to add the note about this (necessary, I believe, if it's intended behaviour) pre-build part of the flow. Or add the prebuild script as an alternative.
Please let me know what you think about it, I can submit a PR with the most suitable option.

Hey!
Thank you so much for making an issue, seems many people felt confused about it so I wrote some new docs that add that and many other small quirks at https://saravieira.github.io/starter-book/#/ and added that to the readme.
I'm gonna close this since it's now there but again thank you for raising this


Wrong place into the frame

The function setupAutocompleteTable is not calculating the right place.
I pass a subview that is into a subview, and the frame is calculated improperly, I had to overwrite the function in order to fix this issue.
Is there an update to that fix? If not when I finish I will submit my solution.

Could you fix it? I have some problem too.

@ignacio-chiazzo any updates?

@omerbas92 @b-e-r-t-o I am too busy now, nonetheless, I would do it ASAP.

Remove the code that is in awakefromnib, override the method layoutSubviews and put it there - it then works

Closing this since the repo is not maintained anymore.


can #present and #notpresent work outside of a JSON doc and even for XML

this fails currently:
* def foo = <bar>baz</bar>
* match foo/ban == '#notpresent'

with the error assertion failed: path: /ban, actual: null, expected: '#notpresent', reason: actual xpath returned null

released 0.8.0

What is the solution???

@Rupal-Saxena ask on stack overflow please: https://stackoverflow.com/a/54851582/143475


Improve layout for address screen

Is this the same page as this one here, with the click-button for add ETC? https://github.com/ethereumproject/emerald-wallet/issues/127

that's a current Address page, just little rearrangement

Updated, Merged #143


Add support for accessing x86_64 registers

This PR adds support for accessing x86_64 registers. It requires libunwind >= 1.6.0, which contains the fix for the issue described in #16. It also contains a small fix to the build system which allows building with -rc releases of libunwind.

Looks like the build script change may have broken some of the version detection logic?

If you are talking about the failed CI checks, they are expected to fail with version 1.4 on x86_64. The required version, the one with the fix for #16, is 1.6.

I certainly do not expect to see segfaults when using 1.4. That does not appear to be the case on master: https://github.com/sfackler/rstack/runs/3077148196?check_suite_focus=true

The segfault is a symptom of the bug described in #16. It happens with 1.4 in the test case I added in this PR. The cause is a self-reference in the unwind cursor in that version of the library. That reference leads to a use-after-free, which was fixed in 1.6.

It sounds like we would need to disable this test on those versions then.

Closing this in favor of #22.