Crash when using WebListener and Negotiate auth scheme

Scenario :
Server is running with options.ListenerSettings.Authentication.Schemes = AuthenticationSchemes.Negotiate;
Client IE settings to automatically logon sites in the intranet zone. URL of site is not in the the sites list.
Client makes request to server and it crashes the server. Put the URL in the sites and auth takes place and all is well.
We ran DebugDiag to try and get a crash dump and below is the stack trace inside it (note this is reproducible on a customer installation currently 😢 ) :
An unhandled exception of type 'System.EntryPointNotFoundException' occurred in Microsoft.Net.Http.Server.dll

Additional information: Unable to find an entry point named 'CancelIoEx' in DLL 'api-ms-win-core-io-l1-1-1.dll'.

    Microsoft.Net.Http.Server.dll!Microsoft.Net.Http.Server.RequestContext.ForceCancelRequest()    Unknown
    Microsoft.Net.Http.Server.dll!Microsoft.Net.Http.Server.RequestContext.Abort()    Unknown
    Microsoft.Net.Http.Server.dll!Microsoft.Net.Http.Server.ResponseStreamAsyncResult.Fail(System.Exception ex)    Unknown
    Microsoft.Net.Http.Server.dll!Microsoft.Net.Http.Server.ResponseStreamAsyncResult.IOCompleted(Microsoft.Net.Http.Server.ResponseStreamAsyncResult asyncResult, uint errorCode, uint numBytes)    Unknown
    System.Private.CoreLib.ni.dll!System.Threading._IOCompletionCallback.PerformIOCompletionCallback(uint errorCode, uint numBytes, System.Threading.NativeOverlapped* pOVERLAP)    Unknown

OS? Windows 7 / 2008 R2?
Framework? .NET or .NET Core?
WebListener version?
api-ms-win-core-io-l1-1-1.dll is a missing API set not present on older Windows versions but can be bundled or installed. I'll work on a repro and a workaround.

(or in case of standalone applications the application was published for a newer version of Windows that it is running on e.g. an published for win8-x64 being run on win7-x64)

It is a self contained app published with win7-x64
It is .net core "Microsoft.AspNetCore.Server.WebListener": "1.1.0",
Running on Windows 2008 R2 SP1

Just a bit more info from the customer.
Their Local Intranet settings does in fact have a *.enterprise.com wildcard in Site List but when they visit https://server.enterprise.com we see the above stack trace. On a machine where they can edit these settings if they directly put https://server.enterprise.com in the Site List all is well.

That call stack looks like a secondary exception (which we should fix), but it doesn't tell us what the primary exception was. Can they reproduce it reliably enough to capture the first chance exception with the debugger?

Since it's a self-contained app, do you have 'api-ms-win-core-io-l1-1-1.dll' in your app directory?

The dll is there.
I can send the dump file if you like?
On Fri, 17 Mar 2017 at 19:35, Chris R<EMAIL_ADDRESS>wrote:

Since it's a self-contained app, do you have
'api-ms-win-core-io-l1-1-1.dll' in your app directory?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/HttpSysServer/issues/326#issuecomment-287451249,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGaps-SGTZoOt6qjXOUXzLMaguodJIfks5rmuCZgaJpZM4Md5OB
.

The dump file is here - https://drive.google.com/file/d/0B4zNlOLPdHfeYVdMT2lrT1FVOE0/view?usp=sharing

Any update on this, customer = large international bank

@jchannon urgent customer issues should be routed to Microsoft Support (https://support.microsoft.com/). The dev team does not have the bandwidth for all investigations.

I found the primary exception in the dump:
System.EntryPointNotFoundException: Unable to find an entry point named 'CancelIoEx' in DLL 'api-ms-win-core-io-l1-1-1.dll'.
   at Microsoft.Net.Http.Server.UnsafeNclNativeMethods.CancelIoEx(SafeHandle handle, SafeNativeOverlapped overlapped)
   at Microsoft.Net.Http.Server.RequestContext.ForceCancelRequest()
   at Microsoft.Net.Http.Server.RequestContext.Abort()
   at Microsoft.Net.Http.Server.ResponseStreamAsyncResult.FailSilently()
   at Microsoft.Net.Http.Server.ResponseStreamAsyncResult.IOCompleted(ResponseStreamAsyncResult asyncResult, UInt32 errorCode, UInt32 numBytes)}	System.Exception {System.EntryPointNotFoundException}

It's actually the same error triggered slightly earlier:
https://github.com/aspnet/HttpSysServer/blob/8d610264386397c8493efa05cf10586f1c0f0198/src/Microsoft.AspNetCore.Server.HttpSys/RequestProcessing/ResponseStreamAsyncResult.cs#L247
vs
https://github.com/aspnet/HttpSysServer/blob/8d610264386397c8493efa05cf10586f1c0f0198/src/Microsoft.AspNetCore.Server.HttpSys/RequestProcessing/ResponseStreamAsyncResult.cs#L270
FailSilently would be called when the client disconnected before the response was completed. It's not clear why this would be related to Negotiate, unless they were writing their own 401 error page?
However, it still comes down to not being able to load that dll. I have not been able to reproduce this issue with the 1.1.1 runtime, dev bits, and Win7x64. If you enable COREHOST_TRACE=1 is the specified dll in the trace? You should see something like Probed deps json and matched [C:\Program Files\dotnet\shared\Microsoft.NETCore.App\1.1.1\api-ms-win-core-io-l1-1-1.dll]

Thanks will get this setup on the customer's install.
Our application does have middleware that does return 401 in a couple of locations eg/XSRF checks, user validation

Attached is a new dump file - https://drive.google.com/file/d/0B3uxSfah57yBWUNjLXp1Z1BGUEE/view?usp=sharing

Here's a log with the env var set.  Thanks
https://drive.google.com/file/d/0B3uxSfah57yBamxoTWhmMzR6cDA/view?usp=sharing

Any update? Thanks 🙏

The log says the file was loaded but that it couldn't find the method within the file. I was not able to reproduce the issue. I don't think we'll be able to make any more progress here without directly debugging the customers app (or related repro app). Did you contact support? They're better set up for direct customer contact.

Thanks.
Just FYI, this is what I've got back from them

I didnt yet contact MS support because in the past I have found it impenetrable. Sorry for the painfully basic question but what category will .Net Core fall into? Our customer is a very large Microsoft customer and ideally we'd like to be able to ask them to have one of their MS contacts step in and help us negotiate our way thru what needs doing.

I'm not sure how support is structured, but organizationally .NET Core lives under Visual Studio, which is under Cloud and Enterprise.

@livarcocc Should we move this bug to SDK or CLI under dotnet?

@muratg At first sight, this seems more like a corefx issue.
@Petermarcu can you help take a look at this?

@gkhanna79 to help out.

I agree though, this may be better served going through support.

@Tratcher is correct that this binary was loaded - this is what was in the log:
[24/03/2017 18:52:41]
  Thread created. New thread - System ID: 4744
  C:\Program Files\Acano Manager\Portal\api-ms-win-core-io-l1-1-1.dll loaded at 0xf0d50000

The exception above is self-explanatory - the binary in question did not export the API.  Can you open a VS Developer Command prompt and do the following against the above binary:
dumpbin /exports C:\Program Files\Acano Manager\Portal\api-ms-win-core-io-l1-1-1.dll
This will list the APIs exported by the above DLL and should confirm that CancelIOEx is not present in it. If confirmed, we will need to understand why Microsoft.Net.Http.Server.UnsafeNclNativeMethods.CancelIoEx expects that API to be present in the above binary.
@jchannon Can you help with the above data collection? Also, what version of .NET Core is this on?

API Set docs: https://msdn.microsoft.com/en-us/library/windows/desktop/mt186421(v=vs.85).aspx
We have not been able to reproduce this issue on other machines.

@gkhanna79 Below is the output you asked for.
c:\AM 2.3 (58fad7)-windows\portal>dumpbin /exports api-ms-win-core-io-l1-1-1.dll
Microsoft (R) COFF/PE Dumper Version 14.00.24210.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file api-ms-win-core-io-l1-1-1.dll

File Type: DLL

  Section contains the following exports for api-ms-win-core-io-l1-1-1.dll

    00000000 characteristics
    5485138C time date stamp Mon Dec  8 02:57:16 2014
        0.00 version
           1 ordinal base
           3 number of functions
           3 number of names

    ordinal hint RVA      name

          1    0          CancelIo (forwarded to kernel32.CancelIo)
          2    1          CancelSynchronousIo (forwarded to kernel32.CancelSynchronousIo)
          3    2          GetOverlappedResultEx (forwarded to kernel32.GetOverlappedResultEx)

  Summary

        1000 .rdata
        1000 .rsrc

Note: We also appear to have api-ms-win-core-io-l1-1-0.dll in our folder and I have no idea how that relates to api-ms-win-core-io-l1-1-1.dll but below is the output from api-ms-win-core-io-l1-1-0.dll in case that is useful
c:\AM 2.3 (58fad7)-windows\portal>dumpbin /exports api-ms-win-core-io-l1-1-0.dll
Microsoft (R) COFF/PE Dumper Version 14.00.24210.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file api-ms-win-core-io-l1-1-0.dll

File Type: DLL

  Section contains the following exports for api-ms-win-core-io-l1-1-0.dll

    00000000 characteristics
    548513F4 time date stamp Mon Dec  8 02:59:00 2014
        0.00 version
           1 ordinal base
           7 number of functions
           7 number of names

    ordinal hint RVA      name

          1    0          CancelIoEx (forwarded to kernel32.CancelIoEx)
          2    1          CreateIoCompletionPort (forwarded to kernel32.CreateIoCompletionPort)
          3    2          DeviceIoControl (forwarded to kernel32.DeviceIoControl)
          4    3          GetOverlappedResult (forwarded to kernel32.GetOverlappedResult)
          5    4          GetQueuedCompletionStatus (forwarded to kernel32.GetQueuedCompletionStatus)
          6    5          GetQueuedCompletionStatusEx (forwarded to kernel32.GetQueuedCompletionStatusEx)
          7    6          PostQueuedCompletionStatus (forwarded to kernel32.PostQueuedCompletionStatus)

  Summary

        1000 .rdata
        1000 .rsrc

I see the same. Are API sets supposed to chain? Every doc I can find shows CancelIOEx in 1.1.1, and I haven't been able to repro this on my VMs.

I've finally been able to repro this error message by directly invoking CancelIoEx on app launch. It's still not clear why this hasn't reproduced any other way.  At least the fix is easy...


Clipped Text in TextBlock (Nightly Build)

Describe the bug
TextBlocks are being clipped very slightly on the left bounds.
To Reproduce
Steps to reproduce the behavior:

Add a TextBlock with text.

or

Open DevTools in Debug

Screenshots
The following shows this issue within the DevTools window in a new project with Inter font (no custom styling overrides).
Nightly Build (Issue)

11.0.5 (No Issue)

The issue is more prominent in our app

Environment

OS: macOS - Arm64
Avalonia-Version: 11.1.999-cibuild0042031-beta

N.B The last nightly build without this issue is 11.1.999-cibuild0041954-beta

I can't reproduce the issue here:

I'm on macOS rather than Windows. I did a quick test on Windows and the same project doesn't have the issue.
I'm using 11.1.999-cibuild0042033-beta which is the latest available on the nighly feed. I'll test ...39 when it's available.

The below screen grabs are from DevTools in the same project using 11.1.999-cibuild0042033-beta
Windows

maxOS


Updates test to be compatible with math#382

Submisison Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Update to test code that stopped compiling with fix to stan-dev/math#382.
Intended Effect
Fixes Stan test code.
How to Verify
Run tests.
Side Effects
None.
Documentation
None.
Reviewer Suggestions
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

If anyone wants to take a look, it's 4 changes to test code. It shouldn't take more than a couple minutes to ok it.

Jenkins, retest this please.


Started the ide setup doc with instructions for vscode

WHY:
Having an IDE helps in spending less effort in figuring our syntax and compilation issues.
WHAT:
Added steps for vscode. Folks could enhance the same and also add on steps for other IDEs.

This change is

@kwk that
True, that's what I've done- just do not have a nice way to put it. I agree,
/path/to/the/parent/directory/of/src
Would that makes sense?

See https://github.com/sbose78/almighty-core/pull/1/files

@kwk  Please review.

Merged changes from @pmuir  into my repo.
Let me know your thoughts on the changes, @kwk .


Segmentation fault in SplineConv

self.conv1 = SplineConv(n_in, n_hidden, dim=3, kernel_size=3, norm=False)
x = F.elu(self.norm(self.conv1(x, edge_index, pseudo), edge_index))
It runs perfectly well on my laptop (mac, CPU), but there is a segmentation fault on the line above when I switch to the server (linux-x86_64, gcc 4.9.0, python 2.7, cuda 8.0).
here is the test output:

================================================= test session starts =================================================
platform linux2 -- Python 2.7.5, pytest-3.8.2, py-1.6.0, pluggy-0.7.1
rootdir: /users/../libs/pytorch_spline_conv, inifile: setup.cfg
plugins: cov-2.6.0
collected 28 items
test/test_basis.py Segmentation fault (core dumped)

A segmentation fault may be a sign of an installation error  in pytorch_spline_conv. What PyTorch version are you using?
Please confirm that
python setup.py install
python setup.py test

in pytorch_spline_conv works and report again.

0.4.1
Install runs fine but it's the same problem in test.

Are you sure the kernel did rebuild? You can force this with
rm -rf build/ && python setup.py install

still, the test fails with this:
test/test_backward.py ....Segmentation fault (core dumped)

I had the same issue with all the mnist examples.

I have that all tests run successfully, but get problems when running my own model: It always runs for some epochs, but then i get problems:
Segmentation fault (core dumped)
or
free(): invalid size
Aborted (core dumped)
or
free(): invalid next size (normal)
Aborted (core dumped)
or
double free or corruption (out)
Aborted (core dumped)
or
Traceback (most recent call last):
File "first_test.py", line 101, in
loss = train(epoch)
File "first_test.py", line 83, in train
loss.backward()
File "/home//anaconda3/lib/python3.7/site-packages/torch/tensor.py", line 102, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph)
File "/home//anaconda3/lib/python3.7/site-packages/torch/autograd/init.py", line 90, in backward
allow_unreachable=True)  # allow_unreachable flag
RuntimeError: could not compute gradients for some functions
or
corrupted size vs. prev_size
Aborted (core dumped)
or
munmap_chunk(): invalid pointer
Aborted (core dumped)
I think those are all the different error messages I get without changing any of my code.
I was not able to track down the error further than knowing it is somewhere in the backward step if I use SplineConv. When doing with torch.no_grad for the SplineConv layer I don't have issues.

I think I know the issue. When feeding pseudo values which are not in [0,1], there is a disaster. Maybe there should be a test somewhere warning the user?

Yes, this is not allowed! I could add an assertion, but this would result in unnecessary computations. Any idea?

Mhh true. I think in the documentation there should be a warning.
And maybe when initializing the spline conv you could add a safe=True/False option. Depending on this it will do either
if safe:
def forward(...) safely
else:
def forward(...) as is...
this is ofc ugly and a dupication of code.
Then again i am bad at programming.

I will definitively add a note in the documentation. I think i will add an assertion in the forward pass which gets only called once.

Done!


kubescheduler Startup Issue [error: no kind "KubeSchedulerConfiguration" is registered for version "kubescheduler.config.k8s.io/v1beta1"]

This issue was closed in the thread Kube-scheduler so thought to create a new one. I am seeing the similar issue where my kube-scheduler is failing to start. Following is how my yaml looks like
apiVersion: kubescheduler.config.k8s.io/v1beta1 kind: KubeSchedulerConfiguration clientConnection: kubeconfig: "/var/lib/kubernetes/kube-scheduler.kubeconfig" leaderElection: leaderElect: true
After reloading and restarting I am still not able to restart the scheduler
● kube-scheduler.service - Kubernetes Scheduler
Loaded: loaded (/etc/systemd/system/kube-scheduler.service; enabled; vendor preset: enabled)
Active: activating (auto-restart) (Result: exit-code) since Thu 2022-01-20 11:42:38 UTC; 1s ago
Docs: https://github.com/kubernetes/kubernetes
Process: 9675 ExecStart=/usr/local/bin/kube-scheduler --config=/etc/kubernetes/config/kube-scheduler.yaml --v=2 (code=exited, status=255)
Main PID: 9675 (code=exited, status=255)
kube-scheduler[9675]: I0120 11:42:38.664840 9675 flags.go:27] FLAG: --scheduler-name="default-scheduler"
kube-scheduler[9675]: I0120 11:42:38.664847 9675 flags.go:27] FLAG: --stderrthreshold="2"
kube-scheduler[9675]: I0120 11:42:38.664853 9675 flags.go:27] FLAG: --use-legacy-policy-config="false"
kube-scheduler[9675]: I0120 11:42:38.664860 9675 flags.go:27] FLAG: --v="2"
kube-scheduler[9675]: I0120 11:42:38.664866 9675 flags.go:27] FLAG: --version="false"
kube-scheduler[9675]: I0120 11:42:38.664878 9675 flags.go:27] FLAG: --vmodule=""
kube-scheduler[9675]: F0120 11:42:38.665137 9675 helpers.go:119] error: no kind "KubeSchedulerConfiguration" is registered for version "kubescheduler.config.k8s.io/v1beta1"
systemd[1]: kube-scheduler.service: Main process exited, code=exited, status=255/n/a
systemd[1]: kube-scheduler.service: Unit entered failed state.
systemd[1]: kube-scheduler.service: Failed with result 'exit-code'.
My Kube version is
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.2", GitCommit:"81753b10df112992bf51bbc2c2f85208aad78335", GitTreeState:"clean", BuildDate:"2018-04-27T09:22:21Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
The wired thing that I notice is, even if i remove the kube-scheduler.yaml file from the /etc/kubernetes/config location I still see the same error.

I was able to figure out the Issue. It seems there was version compatibility issue with the api version. My scheduler was on v1.10.2 and my config was referring to kubescheduler.config.k8s.io/v1beta1. I then installed v1.21.0 and modified the kube-scheduler.yaml to refer to componentconfig/v1alpha1 and that made it work.

@amit4aws You dont have to run older version. if you want it to work in latest version(1.23.2) just change the line:
apiVersion: kubescheduler.config.k8s.io/v1beta1
with the line:
apiVersion: kubescheduler.config.k8s.io/v1beta2
and you done.


Front office returns 500 and 404s for authors attached to specific records

Bug
Front office returns 500 and 404s for authors attached to specific records
Steps to reproduce

Search for https://telefoonboek.prd.rad.ugent.be/nl/people/001995048308 in http://biblio.ugent.be
See all search results. There are two people found:
2.1. https://biblio.ugent.be/person/FFA3F874-F0ED-11E1-A9DE-61C894A0A6B4
2.2 https://biblio.ugent.be/person/26601476-38e0-11eb-9f20-922c3b627285
Click the authors, such as https://telefoonboek.prd.rad.ugent.be/nl/people/001995048308, or other authors.
3.1. Some return a status 500, some a 404.
3.2. None return a research page with ORCID

When I search for an other researcher this problem does not happen

Related

[x] #913

This case has been solved in the back office

e.g. go to https://biblio.ugent.be/person/1A8A0462-F0EE-11E1-A9DE-61C894A0A6B4 and it crashes like this, with this error in /var/log/biblio/deployment.log:
[15681] error @0.206455> [hit #21]request to GET /person/1A8A0462-F0EE-11E1-A9DE-61C894A0A6B4 crashed: core - template - undef error - Can't use string ("") as a HASH ref while "strict refs" in use at /opt/biblio/lib/Biblio/AppHelper/Routes.pm line 39.
 at local/lib/perl5/Dancer/Template/Abstract.pm line 91. in local/lib/perl5/Dancer/Handler.pm l. 98

ok, I can reproduce this locally with the copied and pasted index from production.
Note to self:
has nothing to do with that call to person_url in the view, but all with that automatic
installed subroutine (so hard to find) person_url here

Found it: https://github.ugent.be/Universiteitsbibliotheek/biblio/blob/master/views/person.tt#L83. One of the persons in the list of co authors could not be found in the authority database, but the code in this view ignored that and called person_path anyway, leading to above error.
Some context: you can find the list of co-authors in the blue right bar under the title "Frequent UGent co-authors".

e.g. https://biblio.ugent.be/person/11466D46-F0EE-11E1-A9DE-61C894A0A6B4: co-author "Jurgen van Limmen" with identifier "26601476-38e0-11eb-9f20-922c3b627285" does not exist with that id in the authority table "person"

@nics I tried looping over the production publication records (using the /frontoffice/publication handler) to check if there are other contributor id's that cannot be found in the authority table "person", and there seem to be a lot. My script stalled at around 22000 contributors not found, as the handler returned a "bad gateway", so I am not sure how many more there are (see issue #958 ) for that. Or am I checking this in the wrong way?

@nics whoops, there was a bug in my check script that caused that high number (which was unprobable). Will give you an update.

32061 out of 33122 referenced contributors (or 96.80%) were found in the authority table.
See the 1061 referenced contributors that could not be found here:
biblio_referenced_contributors_not_found.csv

check_authors_exist_es.pl.txt
this was my test script that returned that csv result

@nicolasfranck seems like an indexing problem?
e.g. https://biblio.ugent.be/publication/8759239 references the nonexisting person id FFA3F874-F0ED-11E1-A9DE-61C894A0A6B4
while the same record in the backoffice metadata references the correct id 26601476-38e0-11eb-9f20-922c3b627285
can you investigate how to best fix the index (without losing the gismo timestamps)?

@nics unfortunately not. I redid the same check, but now using the frontoffice handler. I did not exhaust it (as it would return a "bad gateway").
See these examples:
person_id,example_publication_id
6D1420E2-B25E-11E6-B104-2D2FD0AF0289,257322
6D05710A-B25E-11E6-B104-2D2FD0AF0289,222949
6D0AD80C-B25E-11E6-B104-2D2FD0AF0289,223459
8F6A35F0-B25E-11E6-B104-2D2FD0AF0289,381985
6EFC83FE-B25E-11E6-B104-2D2FD0AF0289,243562
7098EB9E-B25E-11E6-B104-2D2FD0AF0289,376631
8A88E126-B25E-11E6-B104-2D2FD0AF0289,438658
8CF86A08-B25E-11E6-B104-2D2FD0AF0289,335906
6D08DEE4-B25E-11E6-B104-2D2FD0AF0289,229269
6DB8AED2-B25E-11E6-B104-2D2FD0AF0289,311300
7313A292-B25E-11E6-B104-2D2FD0AF0289,365688
7135C1D0-B25E-11E6-B104-2D2FD0AF0289,393091
711BB830-B25E-11E6-B104-2D2FD0AF0289,391317
79FCC1E2-B25E-11E6-B104-2D2FD0AF0289,369572
7E8CE07A-B25E-11E6-B104-2D2FD0AF0289,375318
74AE77DA-B25E-11E6-B104-2D2FD0AF0289,393563
870E6004-C053-11E6-8B10-4038B5D1D7B1,663059
6FC6CF56-B25E-11E6-B104-2D2FD0AF0289,427757
95C402DC-B25E-11E6-B104-2D2FD0AF0289,423553
6D144FEA-B25E-11E6-B104-2D2FD0AF0289,445749
6DC0419C-B25E-11E6-B104-2D2FD0AF0289,437337
6D2F9C14-B25E-11E6-B104-2D2FD0AF0289,445049
6DF5CF42-B25E-11E6-B104-2D2FD0AF0289,443322
6D146624-B25E-11E6-B104-2D2FD0AF0289,445789
6DF7C0C2-B25E-11E6-B104-2D2FD0AF0289,435230
6DF56804-B25E-11E6-B104-2D2FD0AF0289,439908
6D2F4DD6-B25E-11E6-B104-2D2FD0AF0289,444750
87E91DFC-C053-11E6-8B10-4038B5D1D7B1,848993
711A7D8A-B25E-11E6-B104-2D2FD0AF0289,251450
73C3CFC8-B25E-11E6-B104-2D2FD0AF0289,430690
8C0DBF76-B25E-11E6-B104-2D2FD0AF0289,369265
8C0DC020-B25E-11E6-B104-2D2FD0AF0289,369265
8C0DC084-B25E-11E6-B104-2D2FD0AF0289,369265
6CC31DF0-B25E-11E6-B104-2D2FD0AF0289,906568
834532DE-B25E-11E6-B104-2D2FD0AF0289,363401
94597A12-B25E-11E6-B104-2D2FD0AF0289,369269
6EAEA22E-B25E-11E6-B104-2D2FD0AF0289,419439
770C0B3C-B25E-11E6-B104-2D2FD0AF0289,426390
9459D868-B25E-11E6-B104-2D2FD0AF0289,369567
8AE3CEF6-B25E-11E6-B104-2D2FD0AF0289,370343
7518E7BE-B25E-11E6-B104-2D2FD0AF0289,345588
817340FE-B25E-11E6-B104-2D2FD0AF0289,351259
947E1BA6-B25E-11E6-B104-2D2FD0AF0289,358623

First publication here https://backoffice.biblio.ugent.be/frontoffice/publication/257322
reference author 6D1420E2-B25E-11E6-B104-2D2FD0AF0289 who is NOT in
the authority table person

check_authors.csv
this is a list of publications (based on previously imported data) compared with
with live authority table persons. The publications are not live, but as I understood,
these problems only exist in older records.
Interpretation:

publication_id: publication identifier (can be repeated)
person_id: person identifier as found in the publication record
role: person role as found in the publication record
fixed_person_id: new person identifier. This is done either by matching on ugent_id (rare, sometimes found in contributor, and matches rarely), or by matching on both first_name and last_name.
match_strategy: matching strategy used (ugent_id or name)
match_hint: match value used in the matching
match_error: when matching failed, why. Can be "zero_results" or "multiple_results" (where there are more than 1 results)

@mietcls I cannot "solve" this without a manual check. Most of the fixed authors (half of it) were fixed by matching on their first and last name, and making sure there were only one. Can a reviewer have a check on this?

@nics I can ask them for help and assist as well. These are 3000 records – what should they check on specifically so I can give them a good briefing to fix it and help them figure out whether/in which timeframe this is feasible (they have deadlines)?

@nicolasfranck I can ask them for help and assist as well. These are 3000 records – what should they check on specifically so I can give them a good briefing to fix it and help them figure out whether/in which timeframe this is feasible (they have deadlines)?

@mietcls mm, I will maybe add a link to the phonebook. There are a lot of them, but if you sort on person_id you will see a lot of repetition (one line per found contributor per publication).

This time with phonebook links:
check_authors.csv
A lot of them are not in the phonebook anymore (inactive), so I do not know if that helps.

@nicolasfranck @mietcls
The idea here is to have the review team loop over the CSV file - publication by publication - and replace the old, defunct  contributors with a new one for each publication.  The CSV file gives a hint / suggests which contributor might be a suitable replacement for an old, defunct contributor.
If you pick an publication from the CSV file, go the "contributors" tab and then try to edit the corresponding contributor, you'll notice that it will error with a request id popup.  (e.g. first item in the list 1029837 / Bouquerel => check it out in Biblio Backoffice).
The vast majority of suggestions in the CSV file are generated by going through the current list of persons in the person service, and looking for an alternative through matching the "first name / last name" of the old, defunct contributor. The problem to this approach is that this is prone to errors (e.g. potential for multiple "Jan Jannsens" who aren't the same person).
Fixing requires manually verifying if the proposed alternative in the CSV list is the exact, same person to the original contributor. So, this has the potential to gobble up a lot of time. I also expect that it won't be possible to solve all cases the CSV list either.
The popup error with the instruction to "send the request to the biblio team" + the biblio transfer commands help us mitigate these issues as they gradually appear; instead of pro-actively trying to fix them all in one go.
I think we need to discuss this in person / via slack before we proceed.

OK TBD on Wednesday

We will split this up into actionable tickets, this ticket has become too vague

Not able to reproduce, we will have to fix this with the person service if it happens again. @nicolasfranck do you still have any news about your CSV?


TEAMNADO-4579 - Update packages

This doesn't run or compile successfully right now - I think its because you are importing InventoryRoutes which doesn't exist. That should probably be ManifestRoutes

I did fix this issue, i got it to run. I believe this is resolved.

This doesn't run or compile successfully right now - I think its because you are importing InventoryRoutes which doesn't exist. That should probably be ManifestRoutes

I did fix this issue, i got it to run. I believe this is resolved.

That does look fixed - I am still running into some errors when trying to open the application though. I get this whenever I navigate to manifests.
Uncaught Error: Objects are not valid as a React child (found: object with keys {$$typeof, _payload, _init}). If you meant to render a collection of children, use an array instead.


Account for token burns caused by spending in non-SLP transaction

This is a lower priority item.
The individual token UTXO status stored within the token graph object property _graphTxns does not properly label the outputs if they were involved in a token burn incident caused by spending in a non-SLP transaction.  For complete token graph labeling accuracy, this should eventually be fixed.
However, the token graph object's tokenStats and tokenUtxos properties do account for these burned tokens.  Hence why this is a low priority item.

This shouldn't be considered a low priority item.  This issue also causes the burned token graph to not be updated and containing out of date statistics.

This issue is currently handled on SLPDB start-up in commit 4e04917.
A subsequent commit to update at each block will allow us to close this issue.

This is handled in 0.11.0.  Non-SLP burns are detected on block events and startup events.


Not able to fetch underlying link data

Hi Guy,
I'm writing to you as a last resort to help solve my current issue. I have used your code earlier to fetch the underlying links data and it worked fine. Currently when I'm trying to use it, it is not working.
Not sure why.
Below is my code: I have included all the required dependencies and compared the code with your example. Everything seems fine.
JSON:
	{ 		"amount": 0, 		"year": 2017, 		"month": 3, 		"assessmentDate": "2017-03-07T18:30:00.000+0000", 		"transactionBalance": null, 		"_links": { 			"self": { 				"href": "http://localhost:8080/totalcharges/1046" 			}, 			"currentCharges": { 				"href": "http://localhost:8080/totalcharges/1046" 			}, 			"homeId": { 				"href": "http://localhost:8080/totalcharges/1046/homeId" 			}, 			"communityInfo": { 				"href": "http://localhost:8080/totalcharges/1046/communityInfo" 			}, 			"hoaId": { 				"href": "http://localhost:8080/totalcharges/1046/hoaId" 			}, 			"assessmentRuleType": { 				"href": "http://localhost:8080/totalcharges/1046/assessmentRuleType" 			} 		} 	}
JS Code:
SpringDataRestAdapter.process($http.get('/totalcharges/search/findByHoaId_hoaId?hoaId=1289','homeId',true)).then(function (processedResponse) { 		  console.log("Inside adapter all"); 		  $scope.totalCharges = processedResponse._embeddedItems; 	      console.log('attendance users '+ JSON.stringify($scope.totalCharges)); 	  });
Can you help me with some troubleshooting steps?
Thanks,
Kiran

I think you placed your closing parenthesis to arguments to late.
On Apr 16, 2017 9:30 PM, "kkolli1127"<EMAIL_ADDRESS>wrote:

Hi Guy,
I'm writing to you as a last resort to help solve my current issue. I have
used your code earlier to fetch the underlying links data and it worked
fine. Currently when I'm trying to use it, it is not working.
Not sure why.
Below is my code: I have included all the required dependencies and
compared the code with your example. Everything seems fine.
JSON:
{ "amount": 0, "year": 2017, "month": 3, "assessmentDate":
"2017-03-07T18:30:00.000+0000", "transactionBalance": null, "_links": {
"self": { "href": "http://localhost:8080/totalcharges/1046" },
"currentCharges": { "href": "http://localhost:8080/totalcharges/1046" },
"homeId": { "href": "http://localhost:8080/totalcharges/1046/homeId" },
"communityInfo": { "href": "http://localhost:8080/totalcharges/1046/
communityInfo" }, "hoaId": { "href": "http://localhost:8080/
totalcharges/1046/hoaId" }, "assessmentRuleType": { "href": "
http://localhost:8080/totalcharges/1046/assessmentRuleType" } } }
JS Code:
SpringDataRestAdapter.process($http.get('/totalcharges/
search/findByHoaId_hoaId?hoaId=1289','homeId',true)).then(function
(processedResponse) { console.log("Inside adapter all");
$scope.totalCharges = processedResponse._embeddedItems;
console.log('attendance users '+ JSON.stringify($scope.totalCharges)); });
Can you help me with some troubleshooting steps?
Thanks,
Kiran
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/guylabs/angular-spring-data-rest/issues/35, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGiCi3HPPJgVeDN4P6J2EIlaGTRiGgx4ks5rwurUgaJpZM4M-3Ia
.

Sorry about the inappropriate word...I posted from my phone and auto
correct replaced it. 😐
On Apr 17, 2017 8:22 AM, "Matthew Valli"<EMAIL_ADDRESS>wrote:

I think you placed your closing parenthesis to arguments to late.
On Apr 16, 2017 9:30 PM, "kkolli1127"<EMAIL_ADDRESS>wrote:

Hi Guy,
I'm writing to you as a last resort to help solve my current issue. I
have used your code earlier to fetch the underlying links data and it
worked fine. Currently when I'm trying to use it, it is not working.
Not sure why.
Below is my code: I have included all the required dependencies and
compared the code with your example. Everything seems fine.
JSON:
{ "amount": 0, "year": 2017, "month": 3, "assessmentDate":
"2017-03-07T18:30:00.000+0000", "transactionBalance": null, "_links": {
"self": { "href": "http://localhost:8080/totalcharges/1046" },
"currentCharges": { "href": "http://localhost:8080/totalcharges/1046" },
"homeId": { "href": "http://localhost:8080/totalcharges/1046/homeId" },
"communityInfo": { "href": "http://localhost:8080/totalch
arges/1046/communityInfo" }, "hoaId": { "href": "
http://localhost:8080/totalcharges/1046/hoaId" }, "assessmentRuleType":
{ "href": "http://localhost:8080/totalcharges/1046/assessmentRuleType" }
} }
JS Code:
SpringDataRestAdapter.process($http.get('/totalcharges/searc
h/findByHoaId_hoaId?hoaId=1289','homeId',true)).then(function
(processedResponse) { console.log("Inside adapter all");
$scope.totalCharges = processedResponse._embeddedItems;
console.log('attendance users '+ JSON.stringify($scope.totalCharges));
});
Can you help me with some troubleshooting steps?
Thanks,
Kiran
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/guylabs/angular-spring-data-rest/issues/35, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGiCi3HPPJgVeDN4P6J2EIlaGTRiGgx4ks5rwurUgaJpZM4M-3Ia
.

Hey Matt,
Thanks for pointing it out. I corrected it now. I see that it is trying to fetch the data from the underlying link (in the Network tab), but it is failing with the below error.
Possibly unhandled rejection: {"data":null,"status":-1,"config":{"method":"GET","transformRequest":[null],"transformResponse":[null],"jsonpCallbackParam":"callback","url":"http://localhost:8080/totalcharges/749/communityInfo","headers":{"Accept":"application/json, text/plain, */*"}},"statusText":""}

You may have already solved the problem, but with my experience that error
occurs when the API server has either not configured a GET Request for the
provided route or the API had some form of security feature disabling the
request.
For example, the team I work with uses Spring Boot, where both scenarios
have appeared.
On Apr 17, 2017 9:25 PM, "kkolli1127"<EMAIL_ADDRESS>wrote:

Hey Matt,
Thanks for pointing it out. I corrected it now. I see that it is trying to
fetch the data from the underlying link (in the Network tab), but it is
failing with the below error.
Possibly unhandled rejection: {"data":null,"status":-1,"
config":{"method":"GET","transformRequest":[null],"
transformResponse":[null],"jsonpCallbackParam":"callback","url":"
http://localhost:8080/totalcharges/749/communityInfo
","headers":{"Accept":"application/json, text/plain,
/"}},"statusText":""}
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/guylabs/angular-spring-data-rest/issues/35#issuecomment-294672899,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGiCi6yPrlw-0oU56itMBhuvNARFEPYDks5rxDsugaJpZM4M-3Ia
.

I'm using the Spring Boot too. Thanks for all your help. Will investigate the problem.

Hi @kkolli1127
sorry for my late response. Looks like @mattvalli said: an issue with the back end server configuration of the /totalcharges/749/communityInfo endpoint. Did you have more insights or did you solve it?
@mattvalli Thanks a lot for the help! Really appreciated!
Thanks and regards
Guy


NixOS 22.11 - Zero Hydra Failures on i686

Part of #199919.
I've got a permission @mweinelt to open this discussion to try and improve the i686 support somewhat in the upcoming release.
Since i686 is a legacy platform that's getting rarer every year, there are plenty of calls for dropping support for it outright, but I feel like the way to go for the immediate future is just refocusing the support for it somewhat.
In 22.11 we would get the DEs working out of box on i686 again.
Among the remaining failures we should decide, which packages do we really care enough to try and support, since plenty of them(or their dependencies) are dropping or would be dropping x32 support in the foreseeable future or are just too costly to maintain in terms of manpower in proportion to the actual userbase.
The remaining 200+ failures should probably be first categorized by whether the failure is i686 exclusive. There are plenty of cases where it isn't.
Then we can roughly split the remaining failures by purpose:

General use software. One currently failing example is keepassxc, but it's in the broken everywhere category. Ideally we want to maintain at least the most popular tools.
Developer and server software. We maybe want to maintain some of those, but they can be somewhat costly to do so in terms of work. Currently nginx and at least some mariadb versions do not work.
Specialized and rare software. It ranges from MuseScore(fails everywhere) to things like rss2email(generic failure too) and nitter. This is the most obvious category for us to look into cutting.

Then there's another category, software that currently does not have a Hydra job, but someone cares about it enough and is willing to maintain. Personally I want to get ncdu and streamlink running again.
Finally, if you are a i686 user and (want to) use any of the software that's currently failing on Hydra, please comment here, so that we would know for sure that that there are still users who care about those particular packages.

Hope nobody minds me keeping this issue open for now, since I still have a few packages that I want to get working again with the end goal of reaching that coveted ZHF state.

22.11 is done. Is there still anything left here? Otherwise I would close this.

Sure, lets close it, I'm still planning to fix a few other things on i686, but this can be done outside of this.


Rectangles instead of stars

I work with Symfony 3.2.
My code is the following:

{{ product.rate|rating(5, "fa-3x") }}

as an output, I had the 5 rectangles instead of stars. How can I solve this issue?

@diriy how are you getting font awesome onto your page?

Same for me, I'm working with symfony 2.8, I have fonts loaded and working, forms also won't render properly


Merge Editor: setting to show only conflicts in overview rulers

This PR resolves #161980 by adding a new setting, mergeEditor.showOnlyConflictingChangesInOverviewRulers, that is false by default, so all changes, conflicting or not, will be highlighted in the Merge Editor's overview rulers by default. If the user chooses (e.g., the author of #156037 has many changes to merge and wants to focus on conflicting changes), this setting can be enabled to highlight only conflicting changes in the overview rulers.
Testing
The easiest way to test this PR is to start a merge from the command line, e.g., .\scripts\code.bat --merge --wait a.txt b.txt c.txt d.txt, where:
Example of non-conflicting change

a.txt: One
b.txt: Two
c.txt: One
d.txt: One

This change should be highlighted in the overview rulers only if the setting is disabled.
Example of conflicting change

a.txt: One
b.txt: Two
c.txt: Three
d.txt: Three

This change should always be highlighted in the overview rulers.
Notes

This PR also makes the description of mergeEditor.showDeletionMarkers localizable. I can remove this quick fix if the reviewer prefers it to not be in this PR.
There are three overview rulers displayed in the Merge Editor, so the setting name and description refer to "rulers", not "ruler", but I can change them to use the singular form if the reviewer prefers.

@microsoft-github-policy-service agree


⚠ Samsung has degraded performance

In 3623ced, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2383 ms

Resolved: Samsung performance has improved in b0c3ba8.


ImportError: libreadline.so.5: cannot open shared object file: No such file or directory

I am completely confused.
I'm using Kali, used the README command, and always get this when using the "ghost" command.
I always get this;
ImportError: libreadline.so.5: cannot open shared object file: No such file or directory

apt-get install libreadline-dev


fix longRunningRequestRE to something that doesn't match pretty much all requests

Fixes #9840
cc @liggitt

Since fixing this could drastically change how incoming requests are bucketed, this should get attention from whoever is doing performance tuning to decide if the MaxRequestsInFlight default of 400 still makes sense

@bprashanth are you a good person to comment on MaxRequestInFlight?

@liggitt addressed comments. Please take another look.

Thanks for fixing this. We pretty much never hit our max-inflight-requests limit, it's too high, probably why this hasn't been noticed. LGTM.

I take that back, this hasn't been noticed because we ignore everything. 400 shold be high enough for most of our scalability testing through.

should be good to merge

added comment and simplified.

LGTM

So if I use watch as a query parameter (which server supports and was supposed to replace /watch ) am I no longer allowed to be treated as a long running operation?

The regex is/was only checked against the URL path. Detecting long running requests via params is a separate change and worth another issue


Changing from int to str

Changing from int to str since the OozieFlowScheduleRecord class itself is already expecting a str.
  public OozieFlowScheduleRecord(Integer appId, String flowPath, String frequency, Integer interval,
    String cronExpression, Long effectiveStartTime, Long effectiveEndTime, String refId, Long whExecId) {
    super(appId, flowPath, frequency, interval, cronExpression, effectiveStartTime, effectiveEndTime, refId, whExecId);
  }
}

This is causing issue: https://github.com/linkedin/WhereHows/issues/903

Coverage remained the same at 26.96% when pulling 76e8b25953469f16bb5e096bb23b103093ce57b8 on avvo:oozie_type_error into fc6ff917a5bae1553083526c9ddf4e1787c07341 on linkedin:master.

Codecov Report

Merging #904 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #904   +/-   ##
=======================================
  Coverage   36.06%   36.06%
=======================================
  Files         309      309
  Lines        8295     8295
  Branches     1546     1546
=======================================
  Hits         2992     2992
  Misses       5303     5303

Continue to review full report at Codecov.

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

@mars-lan , thanks for merging!
Do you happen to know when a new docker image with this change will be pushed up? I still see that the last pushed image on dockerhub was a month ago...I'm trying to compile it myself, but am having issues even though I've downloaded the recommended files in /extralib. Any tips on that if you guys aren't going to be pushing the new build up soon?
Thanks again!

@visokoo Could you open an issue and attach the exact error message you got when building docker images? Thanks.

@mars-lan, I actually resolved it on my own already. Thanks!

I don't think the type needs to change. I ran into the following error because of the change.
TypeError: wherehows.common.schemas.OozieFlowScheduleRecord(): 4th arg can't be coerced to java.lang.Integer

The 162 line in OozieExtract.py calls the OozieFlowScheduleRecord. The 4th arg in OozieFlowScheduleRecord is int, so as the 4th arg in OozieFlowScheduleRecord.
If you go into the super class of OozieFlowScheduleRecord, you can find that the arg name in OozieFlowScheduleRecord is wrong, but the sequence is right. We should follow the arg sequence, not the arg name.
OozieFlowScheduleRecord(self.app_id, row['app_path'], row['time_unit'], str(row['frequency']),  None, row['start_time'], row['end_time'],  row['ref_id'], self.wh_exec_id)
 public OozieFlowScheduleRecord(Integer appId, String flowPath, String frequency, Integer interval,     String cronExpression, Long effectiveStartTime, Long effectiveEndTime, String refId, Long whExecId) {     super(appId, flowPath, frequency, interval, cronExpression, effectiveStartTime, effectiveEndTime, refId, whExecId);   } }


Discord API v7 cant send message

Please describe the problem you are having in as much detail as possible:
I am trying to code a cleverbot integration from Cleverbot; I keep getting this error and I don't know what to do; it keeps happening and I honestly don't know what to do.
Include a reproducible code sample here, if possible:
    if (args.length < 1) {
        return msg.reply("Add a argument!")
    } else {
      cleverbot.configure({botapi: cfg.api_keys.Cleverbot});
      cleverbot.write(args.join(" "), function(response) {
          msg.channel.startTyping();
          setTimeout(() => {
              msg.channel.send(response.output).catch(console.error);
              msg.channel.stopTyping();
          }, Math.floor(Math.random() * 6 + 3));
      });
    }

Further details:

discord.js version: v11.2.1
node.js version: v8.4.0
Operating system: Ubuntu Server
Priority this issue should have – please be realistic and elaborate if possible:
Getting this error:

0|Yumi     | { DiscordAPIError: Cannot send an empty message
0|Yumi     |     at item.request.gen.end (/root/YumiBoat/src/node_modules/discord.js/src/client/rest/RequestHandlers/Sequential.js:71:65)
0|Yumi     |     at then (/root/YumiBoat/src/node_modules/snekfetch/src/index.js:263:21)
0|Yumi     |     at <anonymous>
0|Yumi     |     at process._tickCallback (internal/process/next_tick.js:188:7)
0|Yumi     |   name: 'DiscordAPIError',
0|Yumi     |   message: 'Cannot send an empty message',
0|Yumi     |   path: '/api/v7/channels/265156286406983680/messages',
0|Yumi     |   code: 50006 }

[ ] I found this issue while running code on a user account
[ X ] I have also tested the issue on latest master, commit hash:

My spidersenses are telling me this is not a lib problem.
You tried to send an empty message. Check if response.output actually contains smth.


replace imports with new aliases

Replace deprecated module-alias with built-in imports map. It is present since node v12 and will be soon available in bun (it works in bun-canary). My goal is to make this app runnable with bun and that's the first step to make it happen.

I like this, but it will break some third-party modules because they use the module aliases.
Here is an example: https://github.com/KristjanESPERANTO/MMM-PublicTransportHafas/blob/fafe3a369ae1e356d36d82030d56ffba499024bf/node_helper.js#L1

I got to agree with @KristjanESPERANTO : As much as I like replacing 3rd-party-packages with build-in-fucntionality, breaking compatibility with modules is something that should be avoided at all costs.
I cant imagine that there is some backward compatible way to replace module-alias?

I guess one could create real npm modules named node_helper (fetch and logger are already taken) and proxy it to internal one. Using fetch and logger aliases while real modules already exist seems risky.

I guess one could create real npm modules named node_helper (fetch and logger are already taken) and proxy it to internal one.

That sounds very hacky and hard to maintain...
But whats the problem with bun and module-alias? Maybe there is something we can fix otherwise....

What if we switch MagicMirror and its built-in modules but still leave module-alias in there for the 3rd party modules? This will not get @thesebas to his original goal, but we would have the core a bit more modern and no longer directly dependent on module-alias.

But whats the problem with bun and module-alias?

module-alias does some hacky tricks to achieve its goal and that's not possible in bun,
bash-5.2$ bun serveronly/index.js
12 | var modulePaths = []
13 | var moduleAliases = {}
14 | var moduleAliasNames = []
15 |
16 | var oldNodeModulePaths = Module._nodeModulePaths
17 | Module._nodeModulePaths = function (from) {
   ^
TypeError: Attempted to assign to readonly property.
      at /Users/thesebas/projects/MagicMirror/node_modules/module-alias/index.js:17:0

and additionally this: https://github.com/ilearnio/module-alias/issues/113

What if we switch MagicMirror and its built-in modules but still leave module-alias in there for the 3rd party modules?

I'm really new to the project and wasn't aware that many 3rd party plugins are relying on this module, so, as you said, won't fix the problem, but ofc it would improve a bit the project.

After few hours on this my original goal (make it run on bun) is now even less achievable, seems that there are many hidden incompatibilities (like console-stamp trying to use console.Console constructor that is missing in bun's console implementation and few others), but trying to run it with a bit more restrictive bun exposed some ambiguities that could also be improved (like server = new (require("https").Server)(options, app); instead of server = require("https").Server(options, app); or missing const here or there).

@rejas, what do you think about pursuing this further if @thesebas undoes the removal of module-alias in the js/app.js and package.json?
Old 3rd party modules should still be able to use it that way, but the core wouldn't depend on it anymore. And new modules could use the new approach.

note that we added the package.json alias because npm implemented some security checking for modules found in node modules, but not in package-lock. it removes them
so it kept being deleted.

still appreciate the idea but the mentioned problems in the comments are not solved and no prgress since 11 months so closing this.


bind: 9.12.1-P2 -> 9.12.2

https://ftp.isc.org/isc/bind9/9.12.2/RELEASE-NOTES-bind-9.12.2.html
Bug and security fixes, features.  See Release notes linked above.

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] 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)
[ ] Fits CONTRIBUTING.md.

It appears all NixOS tests currently fail
(extra-utils refers to libidn2 and apparently that's
bad/new/unexpected, dunno haven't looked into it yet)
so haven't tried the "bind" NixOS test before/after.

@GrahamcOfBorg test bind
@GrahamcOfBorg build bind dnsutils

Success on x86_64-linux (full log)
Attempted: bind, dnsutils
Partial log (click to expand)
strip is /nix/store/fzsyclvxz1p2ia83caniy93v8cwarg0j-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/bi5x7ha5lrwa2affvb5z0m5610hdw0aw-bind-9.12.2-dnsutils/bin
patching script interpreter paths in /nix/store/bi5x7ha5lrwa2affvb5z0m5610hdw0aw-bind-9.12.2-dnsutils
checking for references to /tmp/nix-build-bind-9.12.2.drv-0 in /nix/store/bi5x7ha5lrwa2affvb5z0m5610hdw0aw-bind-9.12.2-dnsutils...
shrinking RPATHs of ELF executables and libraries in /nix/store/7plc93b8ghnx995mwyhqn87pg9f6hhgs-bind-9.12.2-host
shrinking /nix/store/7plc93b8ghnx995mwyhqn87pg9f6hhgs-bind-9.12.2-host/bin/host
strip is /nix/store/fzsyclvxz1p2ia83caniy93v8cwarg0j-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/7plc93b8ghnx995mwyhqn87pg9f6hhgs-bind-9.12.2-host/bin
patching script interpreter paths in /nix/store/7plc93b8ghnx995mwyhqn87pg9f6hhgs-bind-9.12.2-host
checking for references to /tmp/nix-build-bind-9.12.2.drv-0 in /nix/store/7plc93b8ghnx995mwyhqn87pg9f6hhgs-bind-9.12.2-host...

Success on aarch64-linux (full log)
Attempted: bind, dnsutils
Partial log (click to expand)
patching script interpreter paths in /nix/store/7v65by0n9607fch471wxn5kbwawkk7l4-bind-9.12.2-dnsutils
checking for references to /build in /nix/store/7v65by0n9607fch471wxn5kbwawkk7l4-bind-9.12.2-dnsutils...
shrinking RPATHs of ELF executables and libraries in /nix/store/ymhwmm396b7k7ah449qxl1kgrkxyh7if-bind-9.12.2-host
shrinking /nix/store/ymhwmm396b7k7ah449qxl1kgrkxyh7if-bind-9.12.2-host/bin/host
strip is /nix/store/h0nc84ybvb8g724fwysvcghh1675wjli-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/ymhwmm396b7k7ah449qxl1kgrkxyh7if-bind-9.12.2-host/bin
patching script interpreter paths in /nix/store/ymhwmm396b7k7ah449qxl1kgrkxyh7if-bind-9.12.2-host
checking for references to /build in /nix/store/ymhwmm396b7k7ah449qxl1kgrkxyh7if-bind-9.12.2-host...
/nix/store/6784vf44s3kc4mwfwz7nsgs1jr2si278-bind-9.12.2
/nix/store/7v65by0n9607fch471wxn5kbwawkk7l4-bind-9.12.2-dnsutils

(FWIW I confirmed the NixOS test passes before/after on master and cherry-picking this commit over)

Success on aarch64-linux (full log)
Attempted: bind
Partial log (click to expand)
these derivations will be built:
  /nix/store/asc8sw7abaa612nhxflr67bf2kz22030-bind-9.12.2.drv
waiting for locks or build slots...
/nix/store/6784vf44s3kc4mwfwz7nsgs1jr2si278-bind-9.12.2

Failure on aarch64-linux (full log)
Attempted: tests.bind
Partial log (click to expand)
cannot build derivation '/nix/store/h9px9b7i03mx2gmcs3m3a174l7q7la95-system-units.drv': 1 dependencies couldn't be built
building '/nix/store/gcnfvjlrj4sjmll9lrsah7cmllp6nc0c-user-units.drv'...
cannot build derivation '/nix/store/csy8mddmhx7zscrbxz5vazigs3nd65qi-etc.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/l4s5g8gqm3f7zk0x8impv72f6z7gqvyn-nixos-system-machine-18.09.git.d6a9a93.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/d1cfixigynw3wnyivh34v2fnwi2ar1yd-closure-info.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/yrhp4md66bvrnykhn0gad11ns9gspzb7-run-nixos-vm.drv': 3 dependencies couldn't be built
cannot build derivation '/nix/store/9ncsb7qhr0lcfg4gywdab7qjl8a294hq-nixos-vm.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/929mnb3598v3idsky93vhhjiiflz2vb8-nixos-test-driver-bind.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/pikyddh3rkaf9k0r8iihii3ym3grxzsj-vm-test-run-bind.drv': 1 dependencies couldn't be built
error: build of '/nix/store/pikyddh3rkaf9k0r8iihii3ym3grxzsj-vm-test-run-bind.drv' failed


Porting in React

I created a porting in React: https://codesandbox.io/s/usesyncscroller-3r6tu

use
useEffect(() => {
    if (names[id]) {
      names[id].push(ref);
    } else {
      names[id] = [ref];
    }
+    names[id] = names[id].filter((n) => n.current !== null);
  }, [id, ref]);

This will remove any references that have been created but not yet assigned. It would be better if the hook deletes its own link when it's done. During synchronization, you just need to skip the null references.

use
useEffect(() => {
    if (names[id]) {
      names[id].push(ref);
    } else {
      names[id] = [ref];
    }
+    names[id] = names[id].filter((n) => n.current !== null);
  }, [id, ref]);

This will remove any references that have been created but not yet assigned. It would be better if the hook deletes its own link when it's done. During synchronization, you just need to skip the null references.

use
useEffect(() => {
    if (names[id]) {
      names[id].push(ref);
    } else {
      names[id] = [ref];
    }
+    names[id] = names[id].filter((n) => n.current !== null);
  }, [id, ref]);

This will remove any references that have been created but not yet assigned. It would be better if the hook deletes its own reference when it's done. During synchronization, you just need to skip the null references.


feat: fix superchainweth docs

#231  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


Split OCI Image Index from Docker Manifest List

Move implementation of the index from the manifestlist package to the ocischema package so that other modules making empty imports support the manifest types their authors would expect. This is a breaking change to distribution as a library but not the registry.
As OCI 1.0 released the manifest and index together, that is a good package from which to initialise both manifests. The docker manifest and manifest list remain in separate packages because one was released later.
The image index and manifest list still share common code in many functions not intended for import by other modules.
This now supports (omits) empty platform objects in OCI Image Indices.
This adds support for annotations on OCI Image Indices.
fixes #3836
fixes #2652

Codecov Report
Patch coverage: 77.94% and project coverage change: +0.24 :tada:

Comparison is base (0c95801) 56.58% compared to head (4cee737) 56.82%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3869      +/-   ##
==========================================
+ Coverage   56.58%   56.82%   +0.24%
==========================================
  Files         105      107       +2
  Lines       10636    10716      +80
==========================================
+ Hits         6018     6089      +71
- Misses       3946     3953       +7
- Partials      672      674       +2

Impacted Files
Coverage Δ

registry/storage/manifestlisthandler.go
48.33% <61.53%> (+5.47%)
:arrow_up:

registry/storage/ociindexhandler.go
62.50% <62.50%> (ø)

manifest/ocischema/index.go
75.58% <75.58%> (ø)

manifest/manifestlist/manifestlist.go
78.31% <88.88%> (+6.88%)
:arrow_up:

manifest/ocischema/manifest.go
75.00% <100.00%> (+0.80%)
:arrow_up:

registry/storage/manifeststore.go
77.77% <100.00%> (+1.03%)
:arrow_up:

registry/storage/registry.go
88.57% <100.00%> (+0.27%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@thaJeztah do you want to give this a pass, too, please? In case I missed something, which I'm pretty sure I did 😄

@thaJeztah do you want to give this a pass, too, please? In case I missed something, which I'm pretty sure I did 😄

Yes, I can give this one a pass; currently hanging out at KubeCon and was looking at it  from my phone (bit hard to read the diff 😆).
I recall I looked at some of this, and seem to remember some "docker" types could potentially be just aliases for the OCI types if we get rid of this mostly useless "Describable" interface; https://github.com/distribution/distribution/blob/0c958010ace2e0c2a87f1bf9915b7c74157dfb62/manifests.go#L73
Which was implemented in various places "for convenience" https://github.com/distribution/distribution/blob/0c958010ace2e0c2a87f1bf9915b7c74157dfb62/blobs.go#L91
(I may have a branch that gets rid of that)

This approach is fine with me. I'm waiting for @thaJeztah and @squizzi to have a look.

Ping @thaJeztah @squizzi

ping @wy65701436

Ping @deleteriousEffect


lib: introduce foreach, combined and combined map idioms

This PR adds foreach and combined functions. They are repainted
versions of following standard lib functions:

foreach        == flip map
foreach        == flip mapAttrs when applied to attrset
combined       == concatStrings
combined "sep" == concatStringsSep "sep"
combined map   == flip concatMapStrings
combined "sep" map == flip (concatMapStringsSep "sep")

The idea behind this rename is to encourage flipped usage of functions
in templaters and other script generators.
Flipped version are often easier to read. They are also more consistent with
existing usage of optional, optionalAttrs, mkIf in sense that
usually first argument is shorter (syntatically) than second.
I rewrote two files in Nixos to show how these new functions should be used.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] 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
[ ] Fits CONTRIBUTING.md.

I'm okay with the foreach function, its two forms can be explained as "Takes an iteratable data structure and a function that gets executed for each element". But combined feels just like a mess, I have no idea what those 4 forms are supposed to be and I think it would be more confusing than anything.

@Infinisil I found that most unreadable code is in script generators. And those use concatMapStrings and concatMapStringsSep, and exactly here is where code is :eyes: bleeding. An option to solve that problem is to replace usages with flip concatMapStrings and flip (concatMapStringsSep "sep"). I found many places where this was already written.
So yeah, all 4 forms do have separate names, and we can live with that... except that it still looks ugly to me.
Maybe I should change description, to make it more user-friendly:
/* `combined` concatenates collection to a string. If supplied with a separator
   and collection, it uses that separator. `combined map` is an idiom to concatenate
   collection, but apply a function to each element of collection (function is supplied
   in last argument).

      combined map collection (element:
         ...here element is converted to a string...
      )

  `combined "sep" map` is a variant of `combined map`, which uses separator "sep"
  for concatenations.
*/

I can split it into two functions combine (concatenate) and combined map (concat with flipped map). I'm also interested what others think about variadic arguments functions and semantics based on argument type.

@danbst Care to push this ahead with just foreach for now and defer decisions on the other functions for later? Or come to a decision on all of the functions now?

@aanderse done. I've removed varargs behavior to keep it simple.
Overall, I still love my combined and combined map approach, even if it sounds complicated. So ruby. Maybe also embed https://github.com/NixOS/nixpkgs/pull/57584 into combined map
And while we are here, the support from Nix for python-styled generators would also be great. https://github.com/NixOS/nix/issues/2543

Closing this now, maybe another time.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/would-you-like-to-use-nix-instead-of-consul-template-templating-language/5145/1


Need a way to disable security question based password recovery in accountrecovryendpoint web app.

We can disable notification based password recovery from the web.xml file as follows, but there is no option to disable security question-based password recovery from a config file.
<context-param>
      <param-name>EnableEmailNotification</param-name>
      <param-value>false</param-value>
  </context-param>

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Publish book to GitHub Pages via Travis CI

pchote has already enabled this repo in the openra account on https://travis-ci.org, now we just need a .travis.yml file that will actually build and publish the book.

Everything except #1 has been done in 5b7786bac6f375c3135c316708cec42a69c65715.

http://www.openra.net/book/


Ockam Release 04-05-2023

Ockam release

Failing lint https://github.com/build-trust/ockam/actions/runs/4887326773/jobs/8723776635?pr=4872


Errors sudo crontab rpi-clone sda -u

Hi,
Thanks for this great project, that's exactly what I was looking for! It's working flawlessly when I run it manually.
When I tried to schedule the following command via sudo crontab:
/usr/local/sbin/rpi-clone sda -u
I got the following error output via email:

Command not found: parted      Package required: parted
Command not found: fdisk       Package required: util-linux
Command not found: fsck.vfat   Package required: dosfstools
Do you want to apt-get install the packages?  (yes/no): Aborting!

Am I missing something or there's an issue here?
Thanks!!

I was also having this same issue and discovered that the script has no PATH set and thus the cron user is unable to properly locate those commands. By editing the script with the following it fixed it:
vi /usr/local/sbin/rpi-clone
add the following under the first line of "#!/bin/bash":
#!/bin/bash
export PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/root/bin"

Thanks for the hint, I'll test it and let you know!

Many thanks @charlie-young! I was having this exact issue as well 👍


Find First and Last Position of element in sorted array in Python

🚀 Feature
I would like to add :

Find First and Last Position of element in sorted array in Python

Have you read the Contributing Guidelines on Pull Requests?
Yes
Pitch
To solve this problem we can use Binary Search Algorithm

/assign


Monitor on training_epoch_end with ModelCheckpoint

What does this PR do?

Fixes #4797
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified; Bugfixes should be including in bug-fix release milestones (m.f.X) and features should be included in (m.X.b) releases.

Did you have fun?
Make sure you had fun coding 🙃

Hello @tchaton! Thanks for opening this PR.

In the file tests/checkpointing/test_model_checkpoint.py:

Line 1069:14: E225 missing whitespace around operator

Do see the Hitchhiker's guide to code style

(misclicked approve instead of comment) :upside_down_face:


Add ignore header

Closing this, since https://github.com/signalfx/signalfx-ruby-tracing/pull/7 handles identifying collector requests without the header


istioctl: Replace package alias 'protio' with package name 'proto'

Please provide a description of this PR:
Replace package alias 'protio' with package name 'proto'.
AS-IS
import (
  protio "istio.io/istio/istioctl/pkg/util/proto"
)

filteredClusters := make(protio.MessageSlice, 0, len(clusters))

TO-BE
import (
  "istio.io/istio/istioctl/pkg/util/proto"
)

filteredClusters := make(proto.MessageSlice, 0, len(clusters))

@jongwooo: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

bookinfo-build_istio
fe99ed1358ac2b137e582cd0ba93685ce0a9f331
link
true
/test bookinfo-build

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 kubernetes/test-infra repository. I understand the commands that are listed here.

/test bookinfo-build


Importing Diffuse texture and applying to a Mesh

Hi,
I'm trying to apply a texture to a mat but I could not figure out how to do it:
here is a succession of what I've tried:
a direct translation of the c code taken from examples:
m_LP_SpookyTree = LP_SpookyTree.GetModel;
m_LP_SpookyTree.materials[0].maps[MaterialMapType.MAP_ALBEDO].texture = Raylib.LoadTexture("Ressource/Textures/PlaceHolders/mossy_rock_diff_1k.png");

the last test
m_LP_SpookyTree = LP_SpookyTree.GetModel;
var mat = m_LP_SpookyTree.materials;
mat[0].maps[MaterialMapType.MAP_ALBEDO].texture = Raylib.LoadTexture("Ressource/Textures/PlaceHolders/mossy_rock_diff_1k.png");

The problem comes from the Intptr, how could I get the material array from it?
thanks for your time.

just found yesterday night:
                Material* materials = (Material*)model.materials.ToPointer();
                MaterialMap* maps = (MaterialMap*)materials[0].maps.ToPointer();

                Texture2D d_LP_SpookyTree = Raylib.LoadTexture("Ressource/Models/LP_SpookyTree.Diffuse.png");
                maps[(int)MaterialMapType.MAP_ALBEDO].texture = d_LP_SpookyTree;

                Raylib.GenTextureMipmaps(ref maps[(int)MaterialMapType.MAP_ALBEDO].texture);


No just classification task

Hello, I want to generate some text adversarial examples that disturb the features in the Intermediate level of model. However, I don't know how to modify the code in Victim Module, the victim is asked to ouput the 1 or 0.

I want to generate some adversarial examples whose features are away from the benign ones just like the adversarial example for images with attacking the feature space.


Update Windows Admin Center to 1.2.24130.0 (Product version 2007)

Previous manifest fails with a hash mismatch due to a new GA release of WAC on 7/21: https://aka.ms/wac2007
Removed specific local from URLs

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

I seem to get a 1603 error and the app is not installed.


about Museum description

Is there any museum description about fishes, bugs, fossils, sea creatures, arts?

See https://github.com/alexislours/acnh-message/blob/master/TalkSNpc_USen/owl/SP_owl_Comment_Fish.csv . They won't be added here.


Add documentation to README.md

What the script does.
How to configure it via properties.
How to run it and override defaults via command line parameters.
Example use cases and how to use the script to accomplish them.
How to install it.

Revision 20 does all of this.


SSR server forwarding cookies to apollo server?

This is a following issue from https://github.com/Akryum/vue-apollo/issues/400.
I am using vue-cli-plugin-ssr together with vue-cli-plugin-apollo and try to achieve proper SSR which prefetch data from the apollo server according to the authentication jwt from cookie.
My ssr server runs on localhost:8000 and apollo server runs on localhost:4000.
After using cookie instead of http request headers to pass the jwt to the ssr server, I can see the cookie is successfully passed to the ssr server:

But when the ssr server prefetches from the apollo server via its own http request, I can see it passes no cookie to it, therefore the prefetch is still no properly authenticated.
So now the problem is how to forward this cookie from the ssr server to the apollo server. The source code creating the ssr server here (https://github.com/Akryum/vue-cli-plugin-ssr/blob/master/lib/server.js#L4-L21) provides no interface to inject express middleware like cookie-parser, and I also couldn't find a way to export from here the express req object, which I believe is necessary in my entry-server.js file to customize the server-side ApolloProvider.
Any suggestions about that?

I sent a PR having the same problem #23 , it allow to add middlewares like cookieparser
Do you think it can revolve yours ?

@zickat That's wonderful! And I just found a minute ago that the context parameter in entry-server.js just include req.headers.cookie which is the raw cookie string from the client. Before your PR I was afraid I had to parse the cookie string myself, and then here is your PR. :))

Thanks ! Hope it will be merged soon 👍

When I have some spare time :smile_cat:

@beeplin @zickat I'm having a similar issue trying to achieve proper ssr using vue-cli-plugin-apollo and vue-cli-plugin-ssr. Obviously it's using localstorage by default which I have overridden in vue-apollo.js getAuth option, so that is using cookies now... My confusion is how to then 'customize' apolloProvider inside entry-server.js to use the jwt from the cookie? I've found very little information outside of this ticket and the one other linked ticket about this although it seems it would be a common use case in ssr. Many thanks if someone can please elaborate a bit on how to achieve this.


[Question]:  Regarding the funsd data set preparation script for custom training

请提出你的问题
Hi @authors,
Could you add the FUNSD training set preparation script for ernie-layout custom training.
This data set - "https://bj.bcebos.com/paddlenlp/datasets/funsd.tar.gz"  is preformatted and could you add the conversion script/data prepartion script for the same.
Thanks,

@chinnaGuggilla Hi,  Thanks for your interest on ERNIE-Layout!
You can just follow our data format directly, the data format is universal for all the NER tasks. The conversion scripts is not universal and will not be open-sourced

I want to load a custom dataset to train earnie-layout on. Instead of using load_dataset from "datasets" library, how can I load files from my custom dataset folder and split it into train test val? My custom dataset would have same format as funsd dataset so that it will have image files and annotation files.


the production link in the README gives 404

Describe the bug
the production link in the README gives 404
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/Steinbeck-Lab/cheminformatics-python-microservice/blob/main/README.md
Click on Production: https://api.naturalproducts.net/latest/docs
See 404 error

Expected behavior
A HTTP 200 reply with OpenAPI conten

@egonw Thanks for the bug report it is now fixed.


device-to-device.md: create draft titles and subtitles.

I pushed a commit to create draft titles and subtitles. But I'm not sure these titles can be described by plgd-dev/sdk test code. Review overall content titles and comment me.

@ondrejtomcik
I pushed new commits added device-to-device guide based on OCF Client sample in plgd/sdk. And also git clone branch in the create-device guide was changed from master to plgd.dev of iotivity-lite to fix bugs communicating with plgd/sdk, plgd/go-coap. Review plz.

Trishia, when you merge the MR please remove also your branch.

Trishia, when you merge the MR please remove also your branch.

Can I merge this MR now? If so, I will merge this MR and then delete this branch.

@ondrejtomcik
I pushed new commit to add code snippet for how to make own OCF Client application as a separate section. How about this kind of titles and flow ?

@ondrejtomcik
I pushed new commit to fix minor changes as you commented. Can I merge this MR now ?


DO NOT MERGE

@wixysam this is another attempt to execute docworks on all modules.
can you quickly verify that the previous issues (with typescript etc...) are resolved ?
specifically for wix-ui-santa, i(which now uses a rege for the relevant files), should it indeed have so many changes ?

It's closer, but there are definitely still things that need to be looked at. No big issues pop out at me.


Update Webpack Config to Exclude i18n strings for translation

Feature Request
Right now when you run npm run build, the code is obfuscated and doesn't work with WP CLI's makepot feature because the variables are overwritten to be something else.
Adding this to webpack.config.prod.js allows for the i18n strings to not be obfuscated so that a user can run WP CLI's makepot command and have the JavaScript strings added to their POT file.
mangle: {
	safari10: true,
	except: ['__', '_n', '_x', '_nx' ],
},

When a user runs npm run build, the user can immediately run WP CLI's makepot command afterward and have the strings show up in their POT file.
An example of creating a POT file is here: https://developer.wordpress.org/cli/commands/i18n/make-pot/
Reference PR is: #182

@ahmadawais
Here's my original POT without the i18n exceptions and the one after it.
The JS without the exclusions was a reasonable 66KB, and with the exclusions, 67KB.
I used the following WP CLI command:
wp i18n make-pot . languages/metronet-profile-picture.pot --exclude="/js/,/src/block/"

You can find my updated POT file with the exclusions here: https://raw.githubusercontent.com/ronalfy/user-profile-picture/dev/languages/metronet-profile-picture.pot
I believe this will also help with .org translations as well since it can parse the JS.

+1000
I was always struggling while trying to i18n my GB blocks. Since yesterday i can do that at last but was afraid of using JSX, ES6, Webpack and any tools that would begin minifying the .js files just because i suspected those would break the i18n process...
This bugfix sounds really promising, have to try it now!

@ronalfy
Just tried the proposed fix but it does not seem to work on my end. Here is what i have done:
0. npx create-guten-block my-block

modfied webpack.config.prod.js just as you suggested
npm start
write the code ready /adding load_plugin_textdomain() and wp_set_script_translations() in init.php/
npm run build
wp i18n make-pot ./ languages/my-block.pot --exclude="/js/,/src/block/"
the .pot file will contain the extracted strings with proper references (dist/blocks.build.js)
translate with PoEdit to get the .po and .mo files
wp i18n make-json my-block-de_DE.po --no-purge
rename the json file to my-block-de_DE-my_block-cgb-block-js.json

I can confirm that after applying the suggested fix by running the command in 5. the .pot file contains the strings that reside in the .js file and need to be translated (which is not the case without the fix proposed here)
However at the end no strings that come from blocks.build.js will get translated in GB, despite the path i coded for the languages directory must be right as strings from the php files (ex plugin header) get translated under the plugins screen. Meaning load_plugin_textdomain() works alright, but wp_set_script_translations() seems to fail despite they search the translation files in the same directory.
Can you confirm if your method results in a working translation of strings coming from script files?
If so, do you have any idea what i might be doing wrong?
If i develop GB blocks with no dev tools, no JSX, no minifying, etc then i can successfully translate also strings coming from .js files.

I haven't tried to create a JSON file. The recommended way I believe is
here:
https://make.wordpress.org/core/2018/11/09/new-javascript-i18n-support-in-wordpress/
I just released a new version of a plugin and the translated strings are
now showing up:
https://translate.wordpress.org/projects/wp-plugins/metronet-profile-picture/dev/en-ca/default/
On Sun, Jun 2, 2019 at 7:50 PM Peter B<EMAIL_ADDRESS>wrote:

@ronalfy https://github.com/ronalfy
Just tried the proposed fix but it does not seem to work on my end. Here
is what i have done:
0. npx create-guten-block my-block

modfied webpack.config.prod.js just as you suggested
npm start
write the code ready /adding load_plugin_textdomain() and
wp_set_script_translations() in init.php/
npm run build
wp i18n make-pot ./ languages/my-block.pot
--exclude="/js/,/src/block/"
the .pot file will contain the extracted strings with proper
references (dist/blocks.build.js)
translate with PoEdit to get the .po and .mo files
wp i18n make-json my-block-de_DE.po --no-purge
rename the json file to my-block-de_DE-my_block-cgb-block-js.json

I can confirm that after applying the suggested fix by running the command
in 5. the .pot file contains the strings that reside in the .js file and
need to be translated (which is not the case without the fix proposed here)
However at the end no strings that come from blocks.build.js will get
translated in GB, despite the path i coded for the languages directory must
be right as strings from the php files (ex plugin header) get translated
under the plugins screen. Meaning load_plugin_textdomain() works alright,
but wp_set_script_translations() seems to fail despite they search the
translation files in the same directory.
Can you confirm if your method results in a working translation of strings
coming from script files?
If so, do you have any idea what i might be doing wrong?
If i develop GB blocks with no dev tools, no JSX, no minifying, etc then i
can successfully translate also strings coming from .js files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ahmadawais/create-guten-block/issues/183?email_source=notifications&email_token=AAE3M2PZYVDIPQ75SJMPQI3PYRTF7A5CNFSM4HSBG6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWYBW2A#issuecomment-498080616,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE3M2NBJGXDC6CYXCG4BSTPYRTF7ANCNFSM4HSBG6MA
.

--
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error, please notify Ronald Huereca at
<EMAIL_ADDRESS>You can only delay the inevitable.
www.ronalfy.com

Thanks for the quick reply!
Just checked your code, as you call wp_set_script_translations( 'mpp_gutenberg', 'metronet-profile-picture' ); you just do not specify any location for a json file as in your case the translation comes from the WP plugin repo.
As i do not commit my plugins there, it is not an option. That is why i pass the location of a json as 3rd argument. It seems that the issue i am facing is then related to the content of the .json file.
So took a closer look inside the json, and found a part "source":"dist\/blocks.build.js" inside it.
Made 2 assumptions:
a) There might be a bug that prevents using several dots in the file name. So i just renamed blocks.build.js to block.js and modified the content of the json accordingly. Did not make any difference.
b) Then i supposed the folder structure might be the issue then, so i just moved block.js to the root folder of the plugin and modified the content of the json file accordingly. That did not help either...
Find it very strange as the .po and .mo files located in the same directory work as expected but they just do not affect strings in js files. Even in the console does not show errors so i am clueless why it works using old-school js without any dev tools and why things break otherwise...

Care to share your code and I'll take a look?

@ronaldhuereca
Just uploaded the pure ES5 version and also a CGB implementation of the same plugin.
Added you as collaborator, so you should have insight to both versions.
Also listed up the workflows step-by-step.
I really wonder what i might have missed in the second version...
A working example of a i18n-ized GB block that stores the translation files in the languages folder (and not in WP plugin repo) would be highly appreciated!

@idea--list I created a branch called i18n with a working example.

Hey, @ronaldhuereca ! Where would that branch be located?

Here:
https://github.com/idea--list/Callout-block-with-CGB/tree/i18ntest?files=1
On Mon, Jun 10, 2019 at 01:22 Rodrigo D’Agostino<EMAIL_ADDRESS>wrote:

Hey, @ronaldhuereca https://github.com/ronaldhuereca ! Where would that
branch be located?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ahmadawais/create-guten-block/issues/183?email_source=notifications&email_token=AAE3M2NPEXC7SPYJYOO5UJTPZXXLNA5CNFSM4HSBG6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXJA5IY#issuecomment-500305571,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE3M2LHKGILHFAJY75WOEDPZXXLNANCNFSM4HSBG6MA
.

--
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error, please notify Ronald Huereca at
<EMAIL_ADDRESS>You can only delay the inevitable.
www.ronalfy.com

Thank you very much, my friend! I'll take a look to it :)
On Mon, Jun 10, 2019 at 3:33 AM Ronald Huereca<EMAIL_ADDRESS>wrote:

Here:
https://github.com/idea--list/Callout-block-with-CGB/tree/i18ntest?files=1
On Mon, Jun 10, 2019 at 01:22 Rodrigo D’Agostino<EMAIL_ADDRESS>
wrote:

Hey, @ronaldhuereca https://github.com/ronaldhuereca ! Where would
that
branch be located?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/ahmadawais/create-guten-block/issues/183?email_source=notifications&email_token=AAE3M2NPEXC7SPYJYOO5UJTPZXXLNA5CNFSM4HSBG6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXJA5IY#issuecomment-500305571
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAE3M2LHKGILHFAJY75WOEDPZXXLNANCNFSM4HSBG6MA
.

--
This email and any files transmitted with it are confidential and intended
solely for the use of the individual or entity to whom they are addressed.
If you have received this email in error, please notify Ronald Huereca at
<EMAIL_ADDRESS>You can only delay the inevitable.
www.ronalfy.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ahmadawais/create-guten-block/issues/183?email_source=notifications&email_token=ACCE4SPC6VSDSQPSCXFXC53PZXYURA5CNFSM4HSBG6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXJBOOQ#issuecomment-500307770,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACCE4SPWXXPLQ7WJGONHFNTPZXYURANCNFSM4HSBG6MA
.


[HOLD for payment 2024-08-01] [$250] Settings - Left hand menu and RHP do not open instantly when opening from link

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.3-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to https://staging.new.expensify.com/settings/preferences/priority-mode
Note that the left hand menu and the priority-mode pop-up does not open instantly
Click Home
Click Account settings
Navigate through the settings

Expected Result:
In Step 1, when opening the settings from link, the left-hand menu and RHP will open instantly
In Step 5, the selected tab will be highlighted
Actual Result:
In Step 1, when opening the settings from link, the left-hand menu and RHP do not open instantly
In Step 5, the selected tab is not highlighted
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/0a0a8a14-333d-4765-9481-24a112c14b39
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~011fca7dda4a362b49
Upwork Job ID:<PHONE_NUMBER>550694979
Last Price Increase: 2024-07-05
Automatic offers:

ishpaul777 | Contributor | 103070943

Issue OwnerCurrent Issue Owner: @RachCHopkins

@marcaaron  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Waiting on proposals, Melvin

Please re-state the problem that we are trying to solve in this issue.
Settings - Left hand menu and RHP do not open instantly when opening from link
What is the root cause of that problem?
The root cause of the issue is that the necessary data for rendering the left-hand menu and right-hand panel (RHP) is not immediately available when navigating directly to a specific URL, causing a delay in rendering and resulting in incomplete UI display. This occurs because the components try to render before the data is fully loaded, leading to a delay in the UI being displayed and the selected tab not being highlighted.
What changes do you think we should make in order to solve the problem?
To resolve this issue, ensure that the component explicitly handles the loading state and waits for the necessary data before rendering the main content. Here’s how you can do it:

Add a Loading State:

Use a state variable to track whether the component is still loading.
Use a useEffect hook to update this state once all required data has been fetched.

Conditional Rendering:

Render a loading indicator while waiting for the data.
Render the main content only after all necessary data is available.

const [isLoading, setIsLoading] = useState(true);
useEffect(() => {
if (priorityMode !== undefined && user !== undefined && preferredTheme !== undefined) {
setIsLoading(false);
}
}, [priorityMode, user, preferredTheme]);
if (isLoading) {
    return (
        <ScreenWrapper>
            <Text>Loading...</Text>
        </ScreenWrapper>
    );
}

video:- https://drive.google.com/file/d/1OP8jCA7-NAS8CtGB8jPgdhnhJKyKhS9q/view?usp=sharing

Please re-state the problem that we are trying to solve in this issue.
When opening the settings from link, the left-hand menu and RHP do not open instantly.
The selected tab is not highlighted
What is the root cause of that problem?
The LottieView component in the lottie-react-native package causes an infinite re-render loop when the autoPlay property is set to true. This prevents the Settings page from rendering properly. Due to the continuous re-rendering of the component, the highlighting is also not working. The problem occurs not only on the Preferences page but also on all settings pages that have Lottie animations, such as the Security page.
What changes do you think we should make in order to solve the problem?
I initially tried to control the Lottie animation manually as recommended in the documentation. However, this approach did not work as expected.
useEffect(() => {
  // Attempt to control the animation programmatically instead of using autoPlay
  // animation.current?.play();
}, []);

To avoid the infinite re-rendering, I used a state variable autoPlayDelay that temporarily holds the autoPlay value. This conditionally renders an empty view which prevents the infinite re-renders.
const [autoPlayDelay, setAutoPlayDelay] = useState(autoPlay);
useEffect(() => {
    const ac = new AbortController();
    if (autoPlayDelay) {
        setTimeout(() => setAutoPlayDelay(false), 0);
    }
    return function cleanup() {
        ac.abort();
    };
}, []);

// If the image fails to load or app is in background state, we'll just render an empty view
// using the fallback in case of a Lottie error or appState.isBackground to prevent
// memory leak, see issue: https://github.com/Expensify/App/issues/36645
//
// autoPlayDelay is using to prevent infinite re-renders when autoPlay is enabled
if (isError || appState.isBackground || autoPlayDelay) {
    return <View style={[aspectRatioStyle, props.style]} />;
}

https://github.com/Expensify/App/assets/10712881/5890a4f9-aecd-4890-b7bb-c97e96eaf9ff

@abdulrahuman5196 please review the proposals above as a next step, thanks!

Hi @MitchExpensify, I am having lesser availability now. Kindly re-assign the labels for another C+. Unassigning myself from the issue.

Looking for a volunteer here https://expensify.slack.com/archives/C02NK2DQWUX/p1720649838512589

@MitchExpensify what about our proposals?

hey @ishpaul777 I will take a deeper look tomorrow. But from my experience there is no single way of debugging these things.
Could be that some underlying library is not ready React concurrent (which we didn't know about) or could be our code.
Out of context Im afraid I can't help much with this, but I will spend a bit of time tomorrow to take a look and write some kind of response.
Meanwhile other people could still try to find the solution - in the end we want to stick to Strict Mode

@ishpaul777  Today, I pulled code, and while it works fine, there is a one-second delay. What do you want me to do next regarding this issue?
screen-capture.webm

Priority mode is used for day to New Expensify use so I think this fits into #wave-collect. Accessing it via the URL directly isn't mainline so labelling it polish

Reassigning as I'm headed on leave 🙇

Hey! I’m Wojtek from Software Mansion, an expert agency, and I’d like to work on this issue.

Sounds good!

I think we're just waiting on payment for @ishpaul777 at this point.

Payment approved in UpWork for @ishpaul777 - sorry for the delay Ishpaul!

Payment Summary:

Contributor: @ishpaul777 paid $250 via Upwork
Contributor+: @WojtekBoman does not require payment (Contractor)

Contributor has been paid, the contract has been completed, and the Upwork post has been closed.


[BUG] Visual tracking tool plots incorrectly with projection other than EPSG:4326

When you used the plot_batch_splitter function as described in the documentation with an other projection than EPSG:4326, the function doesn't plot correctly because get_bbox_list function will convert splitter's coordinates to EPSG:4326 as its return but crs.pyproj_crs will get the origin crs of splitter and set it as the crs of gdf. Consequently, the function is plotting EPSG:4326 coordinates value on another crs plane, in this case the EPSG:3035.
Input:
def plot_batch_splitter_bug(splitter):
    gdf = gpd.GeoDataFrame({
        'status': [info['status'] for info in splitter.get_info_list()],
        'geometry': splitter.get_bbox_list()
    }, crs=splitter.crs.pyproj_crs())
    print('gdf coords with splitter.get_bbox_list(): ', gdf['geometry'][0])
    print('gdf crs: ', gdf.crs)
    ax = gdf.plot(column='status', legend=True, figsize=(10, 10))

    area_series = gpd.GeoSeries(
        [splitter.get_area_shape()],
        crs=splitter.crs.pyproj_crs()
    )

    area_series.plot(ax=ax, facecolor='none', edgecolor='black')

splitter = BatchSplitter(batch_request=batch_request)

plot_batch_splitter_bug(splitter)

Output:

To make the tool works, the crs of gdf needs to be specified as "EPSG:4326" and then apply gdf.to_crs(splitter.crs.pyproj_crs()) to convert the gdf to the correct crs as shown below:
Input:
def plot_batch_splitter(splitter):
    gdf = gpd.GeoDataFrame({
        'status': [info['status'] for info in splitter.get_info_list()],
        'geometry': splitter.get_bbox_list()
    }, crs="EPSG:4326")

    gdf = gdf.to_crs(splitter.crs.pyproj_crs())

    ax = gdf.plot(column='status', legend=True, figsize=(10, 10))

    area_series = gpd.GeoSeries(
        [splitter.get_area_shape()],
        crs=splitter.crs.pyproj_crs()
    )

    area_series.plot(ax=ax, facecolor='none', edgecolor='black')

Environment:
Python 3
Desktop:

OS: macOS 11.1
Browser: chrome

In the new package version 3.4.0 we made changes of BatchSplitter class and fixed plot_batch_splitter function. The function will now always transform bounding boxes into a CRS of the input area of interest.


C#: Remove @precision for new date queries

Removing the @precision annotation until we know how precise these queries actually are for projects on LGTM.com.

We should create a Jira ticket to check these queries, reenable the precision and add release notes.


Is there a Todo list or roadmap for the software?

@MichalPrincNXP What is the future of the software?

Hello @Pillar1989 , the erpc project development continues, the goal is to provide at least two releases per year (aligned with NXP MCUXpressoSDK releases). The list of tasks, todos, bugs is maintained internally. We are trying to address this Github community issues and pull requests first. Except of bugfixing and accepting new community pull request, new transports are planned for the next release. Hope it helps.

Thank you for your answer. When is the next release? What feature is new transports? Can you divulge?  @MichalPrincNXP

The next release is planned in 6 months (beg. of January). New transports means we are going to add support for new transport layers: except of RPMsg_Lite, UART, SPI, serial, TCP, MU - based transports (currently available) there is a plan to support transports based on Linux SPIdev and I2C.

Thanks