Isolated functions with .NET 8 - SignalR extension throws warning CS8785

After upgrading our application to .NET 8, I started receiving the following error:
CSC : warning CS8785: Generator 'FunctionMetadataProviderGenerator' failed to generate source. It will not contribute to the output and compilation errors may occur as a result. Exception was of type 'InvalidOperationException' with message 'TypedConstant is an array. Use Values property.'.

Steps to reproduce:

using Visual Studio 17.8.1
create a new Function app targeting .NET 8 Isolated
add the SignalR extension + a simple Negotiate method

[Function("negotiate")]
public string Negotiate([HttpTrigger(AuthorizationLevel.Function)] HttpRequestData req, [SignalRConnectionInfoInput(HubName = "TestFunctions", UserId = "{headers.user}",
            IdToken = "{headers.token}",
            ClaimTypeList = new []{"user_id"})] string connectionInfo
    )
    {
        return connectionInfo;
    }

Minimal repo:
https://github.com/Msirkovsky/signalR-net8-issue-CS8785

While we investigate, you can mitigate the issue by adding the property <FunctionsEnableWorkerIndexing>false</FunctionsEnableWorkerIndexing>.

While we investigate, you can mitigate the issue by adding the property <FunctionsEnableWorkerIndexing>false</FunctionsEnableWorkerIndexing>.

@satvu Are there any negative side effects of setting <FunctionsEnableWorkerIndexing>false</FunctionsEnableWorkerIndexing> ? I ran into the same problem with triggers for Timer, Storage Queues, and ServiceBus Topics

@christianhelle Setting it to false means that some cold start performance optimizations will not take place (see docs), but this should have no other impact (same behavior/performance as previous SDKs).

Thanks for clarifying @satvu

Can you share a minimal repro of the code causing errors for you and the error that you are getting? And could you also share the versions of the Timer, Storage, and ServiceBus extension packages you are using?

Sure, the Functions project file looks like this:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFramework>net8.0</TargetFramework>
    <AzureFunctionsVersion>v4</AzureFunctionsVersion>
    <OutputType>Exe</OutputType>
    <FunctionsEnableWorkerIndexing>false</FunctionsEnableWorkerIndexing>
  </PropertyGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.ApplicationInsights.WorkerService" Version="2.21.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.20.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.16.2" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Storage.Queues" Version="5.0.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Timer" Version="4.1.0" />
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.ServiceBus" Version="5.7.0" />
  </ItemGroup>
  <ItemGroup>
    <None Update="host.json">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
    </None>
    <None Update="local.settings.json">
      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
      <CopyToPublishDirectory>Never</CopyToPublishDirectory>
    </None>
  </ItemGroup>
</Project>

The build error was pretty much the same as the one in the description of this issue, except that instead of an InvalidOperationException I got a NullReferenceException

@christianhelle The NRE is likely a result of the timer extension package - upgrading to 4.3.0 should resolve it. If not, please open a new issue to track this separately. Thanks!

Original error is thrown from Roslyn when iterating over the named arguments of attributes here in our own stack. Retrieving named arguments using attributeData.NamedArguments will throw this error if an array is present.
Here is an example of a potential workaround implemented by the dotnet team.

SDK version 1.17.3-preview2 has been released with this fix.


feat: processing query params of url in put method

Signed-off-by: GaoJing<EMAIL_ADDRESS>PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.
What is the current behavior?

There are no query params in put method when calling device put commands.
Issue Number: #3033
What is the new behavior?
Processing query params of url in put method. In NewPutCommand function, when creating restful url, adding the query params in url.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

New Imports

[ ] Yes
[x] No

Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Codecov Report

Merging #3034 (4574322) into master (6045104) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3034   +/-   ##
=======================================
  Coverage   40.52%   40.52%
=======================================
  Files         158      158
  Lines       13775    13776    +1
=======================================
+ Hits         5582     5583    +1
  Misses       7872     7872
  Partials      321      321

Impacted Files
Coverage Δ

internal/core/command/put.go
71.42% <100.00%> (+2.19%)
:arrow_up:

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 6045104...4574322. Read the comment docs.

Codecov Report

Merging #3034 (4574322) into master (6045104) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3034   +/-   ##
=======================================
  Coverage   40.52%   40.52%
=======================================
  Files         158      158
  Lines       13775    13776    +1
=======================================
+ Hits         5582     5583    +1
  Misses       7872     7872
  Partials      321      321

Impacted Files
Coverage Δ

internal/core/command/put.go
71.42% <100.00%> (+2.19%)
:arrow_up:

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 6045104...4574322. Read the comment docs.


Element Selection Box In the workspace

Bug Description
If you place an element (shape, text, video, images) partially in the work space (dark gray area), the element should be clipped but the selection box should be visible so that the user is able to transform (resize, drag on left side etc) so that they are not forced to drag the element back into the page to transform.
Figma
Steps to Reproduce

Place an element (shape, text, video, images) partially in the work space

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Figma flow.
#2042 is a related issue

This got already fixed in #2285, closing.


Re-add Bold Underline and Italic options for 'LoadFont' function

The engine should not take responsibility for them. They should be provided as a separated font file by font providers.


Can jaeger set ssl_verify false when connect elastic search with ssl?

I set up elastic search with search guard plug in.
kibana can access successfully with below configure,
  # Set to false to have a complete disregard for the validity of the SSL certificate.
    elasticsearch.ssl.verificationMode: none

Fulentd can access successfully
host <IP_ADDRESS>
port 30920
scheme https
ssl_verify false
user admin
password mypassword
ssl_version TLSv1_2

However,  jaeger always failed like:
{"level":"fatal","ts":1555073973.330164,"caller":"all-in-one/main.go:101","msg":"Failed to init storage factory","error":"failed to create primary Elasticsearch client: health check timeout: Head https://<IP_ADDRESS>:30920: x509: certificate is valid for <IP_ADDRESS>, not <IP_ADDRESS>: no Elasticsearch node available"

Here is jaeger deploy
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  annotations:
    deployment.kubernetes.io/revision: "3"
  creationTimestamp: 2019-03-05T06:50:38Z
  generation: 3
  labels:
    app: jaeger
    jaeger-infra: jaeger-deployment2
  name: jaeger-deployment2
  namespace: kube-system
spec:
  replicas: 1
  selector:
    matchLabels:
      app: jaeger
      jaeger-infra: jaeger-pod
  strategy:
    type: Recreate
  template:
    metadata:
      creationTimestamp: null
      labels:
        app: jaeger
        jaeger-infra: jaeger-pod
    spec:
      volumes:
      - name: volcert
        secret:
          secretName: certes
      containers:
      - env:
        - name: COLLECTOR_ZIPKIN_HTTP_PORT
          value: "9411"
        - name: SPAN_STORAGE_TYPE
          value: elasticsearch
        - name: ES_SERVER_URLS
          value: https://<IP_ADDRESS>:30920
        - name: ES_INDEX_PREFIX
          value: dev1ctmp
        - name: ES_USERNAME
          value: admin
        - name: ES_PASSWORD
          value: adminP
        - name: ES_TLS_CA
          value: /tmp/es-root-ca.pem
        image: jaegertracing/all-in-one
        volumeMounts:
        - name: volcert
          readOnly: true
          mountPath: "/tmp/"
        imagePullPolicy: Always
        name: jaeger
        ports:
        - containerPort: 5775
          protocol: UDP
        - containerPort: 6831
          protocol: UDP
        - containerPort: 6832
          protocol: UDP
        - containerPort: 16686
          protocol: TCP
        - containerPort: 9411
          protocol: TCP
        readinessProbe:
          failureThreshold: 3
          httpGet:
            path: /
            port: 16686
            scheme: HTTP
          initialDelaySeconds: 5
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 1
        resources:
          limits:
            memory: 2Gi
          requests:
            memory: 200Mi
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext: {}
      terminationGracePeriodSeconds: 30

Here is cm of key:
kubectl create secret generic certes --from-file=es-root-ca.pem -n kube-system

It looks like tls.Config type has a field InsecureSkipVerify. We can expose it as one of the --es.tls.* CLI flags.
Are you interested in submitting a PR?

Thanks you are trying to fix this issue.
Just a confirm: which parameter should I set in k8s config later, as below?
- name: ES_TLS_SKIP_HOST_VERIFY
value: TRUE

literally just made the exact commits into my fork as i didn't see this beforehand thanks~!

It looks like tls.Config type has a field InsecureSkipVerify. We can expose it as one of the --es.tls.* CLI flags.
Are you interested in submitting a PR?

was almost finished with the exact same changes in my fork and going to work on the tests when I saw this issue...Will be a good feature as often there may be reasons to have different host/cert on some es node you want to connect to in test
Thanks! @stefanvassilev

@jordo1138 what kind of tests do you have in mind? I have added a unit test case in plugin/storage/es/options_test.go

@stefanvassilev
your test looks good, was referring in my own test env I often have a es node that has no matching ssl cert bc it’s ip based and has changed etc. there’s other workarounds on both side but it’s a nice option to have when needed. Thanks


Support cash withdrawal transaction

A "Wire Received" transaction is currently supported. I don't have an actual wire withdrawal transaction.

@ipaulo Thank you for sharing the transaction data. I will update the code.


Add a Haskell package.

Build the package in CI. Cross-language builds are apparently not possible.

Ready for review.

Superseded by https://github.com/tree-sitter/haskell-tree-sitter/pull/23.


sign-up button functionality

WHEN I click on the sign-up button
THEN my user credentials are saved and I am logged into the site

solved


LSP client connection to Gauge LSP server

Expected behavior
I would like to try and integrate another LSP client with gauge. I start gauge at the  root of a gauge project like:
gauge daemon --lsp 9000

The process starts and listening for websocket connections on port 9000
Actual behavior
The process starts but there is no logging and the port 9000 is not listening
I expect to be able to open a WebSocket connection like:
ws://localhost:9000/

Gauge version
Gauge version: 1.0.6

Plugins
-------
html-report (4.0.8)
java (0.7.2)
js (2.3.5)
screenshot (0.0.1)

Am I missing something here, is this a supported?

Ok, so your LSP server is using gRCP over process inout/output and not TCP or WebSocket. I wasn't aware that LSP servers were using other transports.

thanx for the clarification


Provide a (better) changelog

First off, thanks a lot for this well maintained fork of javafaker. Very much appreciated!
IMO, this project is lacking a "proper" changelog. There is https://www.datafaker.net/releases/1.4.0/, but it's not linked from the Github releases page, so it's not that easily found when one has a very Github-centric workflow.
In fact, there are just tags, not "proper" releases here on Github.
I suggest create a release entry per actual release and place a link to the respective website page.
And to have a detailed list of changes I suggest to let GH auto-generate it (there is a button on the release creation page).
Also, being the maintainer of a GH project myself (https://github.com/gitflow-incremental-builder/gitflow-incremental-builder), I found Github milestones to be very handy.

Hi @famod , thanks for your suggestion, we'll take it into consideration. If you'd like to setup a workflow for this, we're happy to take PRs on board.


Error on flashing_script for esp32

Problem
bootloader_offset was changed to default None. It appears to be changed to an sdkconfig parameter, but CONFIG_BOOTLOADER_OFFSET_IN_FLASH does not appear in my sdkconfig. The result is a python exception.
Traceback (most recent call last):
File "examples/all-clusters-app/esp32/build/chip-all-clusters-app.flash.py", line 23, in
sys.exit(esp32_firmware_utils.Flasher(**DEFAULTS).flash_command(sys.argv))
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/firmware_utils.py", line 399, in flash_command
return self.parse_argv(argv).actions().status()
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/esp32_firmware_utils.py", line 442, in actions
if self.flash(flash).err:
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/esp32_firmware_utils.py", line 351, in flash
return self.run_tool(
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/firmware_utils.py", line 204, in run_tool
arguments = self.format_command(arguments, opt=option_map)
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/firmware_utils.py", line 329, in format_command
result += self.format_command(i, args, opt)
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/firmware_utils.py", line 329, in format_command
result += self.format_command(i, args, opt)
File "/workspaces/connectedhomeip/examples/all-clusters-app/esp32/build/firmware_utils.py", line 364, in format_command
raise ValueError('Unknown: {}'.format(template))
ValueError: Unknown: None
The terminal process "/bin/bash '-c', 'examples/all-clusters-app/esp32/build/chip-all-clusters-app.flash.py'" failed to launch (exit code: 1).
Proposed Solution
Add back the default, or specifically error when this value isn't in the sdkconfig with instructions on how to get it there (re-running menuconfig does not work for me)

We're in progress of reviewing older issues. Is this still an issue given how old the SDK this was tested against? Please re-open if so. Thanks!


Community

add function query_issues, update files

I accepted the changes, updated the functions, the code works


libsForQt5.kirigami-addons: 0.7.2 -> 0.8.0

Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 226156 run on x86_64-linux 1

17 packages built:

libsForQt5.alligator
libsForQt5.angelfish
libsForQt5.kalendar
libsForQt5.kasts
libsForQt5.kclock
libsForQt5.kirigami-addons
libsForQt5.krecorder
libsForQt5.ktrip
libsForQt5.kweather
libsForQt5.neochat
libsForQt5.plasma-dialer
libsForQt5.plasma-mobile
libsForQt5.plasma-phonebook
libsForQt5.plasma-settings
libsForQt5.qmlkonsole
libsForQt5.spacebar
libsForQt5.tokodon


Check error case for skip existing

Test that if you try to re-upload a different package with skip existing, you get an error before even trying to upload, as part of #7917 (first commit). Improve the test script configuration a bit (second commit).

The wording in the summary / title is a bit misleading, this just change test coverage (and some minor formatting of the error) right?


TLS 1.2 support

I'm passing the option: "--ssl=tls1.2" but this is the error I'm getting back "http: error: argument --ssl: invalid choice: u'tls1.2' (choose from 'ssl2.3', 'ssl3', 'tls1')"
Looks as though it's not supported. The version I'm using is 0.9.6. Can anyone give me the heads up here?
Regards,
Rashid

The actually available set of protocols may vary depending on your OpenSSL installation.
— https://httpie.org/docs#ssl-version

A quick way to find out what the available protocols are, is to run http --help. The --ssl option will list all the options actually supported by your system.  For example, on my macOS/Homebrew installation it looks like this:
$ http --help | grep ' --ssl'
  --ssl {ssl2.3,ssl3,tls1,tls1.1,tls1.2}

@rashthedude is this on Ubuntu 10.04? If so, the problem is your version of Python is so incredibly old that it's ssl module does not support tls1.2. You should really not be using lucid if you care about security.
You might be able to install PyOpenSSL, pyasn1, and python-ndghttpsclient from pip to get better support but if those are not installable, you should really abandon the notion of using 10.04.

@jkbrzt and @sigmavirus24 thanks a lot for the swift response. I think the only thing that makes sense at this point is upgrading and abandoning Lucid.


Making the python3 requirement more obvious

Its already in the description and wiki, the former being higher up in the readme than the dependencies are.
If anything I need to get around to simply rejecting installs on Python 2, still haven't figured out how other packages due that though I've not really given too much effort towards it.


where should the training files be placed?

I'm really appreciate your sharing of this project . But I just started using Tensorflow, so May i ask where should the training files be placed？ and I already download 91-Image/SET 5/SET 14 etc. but I can't run it success!

InvalidArgumentError (see above for traceback): You must feed a value for placeholder tensor 'label/hr' with dtype float and shape [?,?,?,1]
[[node label/hr (defined at /mnt/md0/xianan/VideoSuperResolution-master/Train/VSR/Framework/SuperResolution.py:124)  = Placeholderdtype=DT_FLOAT, shape=[?,?,?,1], _device="/job:localhost/replica:0/task:0/device:GPU:0"]]
[[{{node loss/Const_2/_39}} = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_279_loss/Const_2", tensor_type=DT_INT32, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]

@xia00100 Hi, you can check datasets.yaml.
let's say you download 91-image.tar.gz and decompress 91 images into /mnt/data/datasets/91-image, you just modify root and Path: 91-image sections in datasets.yaml. If you are familiar with Path.glob() function, it should be easy to understand.
After 'Root' and 'Path' are correctly set, combined them in Dataset section as I did.
At last, VSR can automatically parse datasets' files and start training.

Currently, try training ESPCN for a good to start.
If you want to train SRCNN and VDSR, currently add --add_custom_callbacks=upsample as an argument for run.py.

@Iatboo Did you use your VESPCN or my version? In paper, the author claimed optical flow ranges (-1, 1), representing movement across the entire image, which however leads to quite poor results.
So instead I inteprete optical flow ranges (-1, 1) as the absolute pixel, which means the movement is restricted up to 1 pixel.
By doing that way, my result is around 25.5dB (VID4), better than bicubic, EDSR, worse than Bayesian.
I trained with MCL-V and GOPRO dataset, you can find the download link in README file.
The pre-trained weights is on-the-way, please wait for some days (or some months...)

@LoSealL Thank you very much for your detailed answer.

closed since no more questions here


Can't listen for touchstart events on a draggable element

In my app, I use goog.fx.Dragger to scroll an element in the page (the element is both the target and the handle). However, this element contains children that have click listeners, and on touch devices these listeners won't be fired because of the preventDefault() call in goog.fx.Draggeer.prototype.startDrag.
Despite having found a workaround, I think it would be nice to make it work by default. Having done some testing, I have found that listening for goog.events.EventType.TOUCHMOVE instead of goog.events.EventType.TOUCHSTART in the constructor seems to perform the same AND it enables the use of click listeners on the draggable element.
I do not know about all the use-cases that goog.fx.Dragger was intended for (I do know that mine is probably not one of them), so maybe I am missing something and such a change is undesirable, but from the testing I have done everything seems to work just fine.

startDrag fires a goog.fx.Dragger.EventType.START event on touchstart. This change will affect that behavior by firing START on touchmove events instead of start. This is semantically incorrect and could break code relying on that assumption.
Beyond that, it looks like the code wasn't intended to fit your use case. It specifically calls preventDefault() to avoid bad user experiences like following a clicked link instead of dragging an element.


[WIP] Remove core Cinder and Swift Managers

The Cinder and Swift manangers in core aren't used and can be deleted

@miq-bot cross-repo-tests manageiq-providers-openstack#677,manageiq-ui-classic,manageiq-api

@miq-bot cross-repo-tests manageiq-providers-openstack#677,manageiq-ui-classic,manageiq-api

Cross-repo tests are green

Cross-repo tests are green

@agrare Do we need a schema migration?  I don't think so (and I think I asked this elsewhere), but was it possible to create a base CinderManager?

@agrare Do we need a schema migration?  I don't think so (and I think I asked this elsewhere), but was it possible to create a base CinderManager?

We already did a schema migration for swift: https://github.com/ManageIQ/manageiq-schema/pull/543
I thought Cinder was take care of a long time ago but I don't see anyone doing an explicit migration after https://github.com/ManageIQ/manageiq-providers-openstack/pull/242 introduced it.  I'll throw one together just to be safe.

We already did a schema migration for swift: https://github.com/ManageIQ/manageiq-schema/pull/543
I thought Cinder was take care of a long time ago but I don't see anyone doing an explicit migration after https://github.com/ManageIQ/manageiq-providers-openstack/pull/242 introduced it.  I'll throw one together just to be safe.

https://github.com/ManageIQ/manageiq-schema/pull/553 (still queued :frowning:)

https://github.com/ManageIQ/manageiq-schema/pull/553 (still queued :frowning:)


Logical error -maybe

Hello,
Firstly, I struggled so much trying to run the script. The problem was installing pcap module which I was encountering to pip errors. Finally I run it by installing pypcap module. Anyway!
I captured traffic of two clients behind NAT (Windows and Lubuntu) who browsed google, facebook and two other websites and that is it.
Total number of packets captured: 100
I run script like so: python plotpcap.py packetcapture.cap number ipid ip src <IP_ADDRESS>
but plot I get is weird. There were only two clients. why there are so many dots?
and the x-axis goes to 60000 while at max it can be 100 (total number of packets) even if we consider each packet matched the filter(s).
Any clue?

Sorry, that is my bad, it was a long working day, and very very surprisingly I confused x and y axises.
but however, can you explain me this plot.

How can this show me the number of clients behind NAT? as I know there are two
and what is Y axis?
and will you state which algorithm is it? is there a paper regarding this algorithm.
and here is the pcap file attached:
packetcapture.cap.zip

This technique relies on operating specific behavior that may have changed in recent versions of operating systems (I think I last used this code in 2013).  Last time I was working in the field Windows systems has linear incrementing ipid fields so you would see a line on the plot and for Linux and Mac they would use a random value (though Linux machines would increment the number for subsequent packets in the same flow creating small lines). I just took a quick Windows 10 packet capture and it looks like the ipid strategy has changed, I see something more like Linux where the field increments at the for subsequent packets in a flow but in randomized for each new flow.
I'd recommend trying your experiment again with a larger capture (I would generally have thousands of packets), make sure you include a tcp dump style filter on the command line (e.g. ip src <IP_ADDRESS>). For Linux like operating systems you can also count them using the TCP timestamp optional field with the tcptsval option to the script but this won't work on Windows because it doesn't include that field in its TCP traffic.
The first blog post on this technique has more details (including a bibliography): http://memeover.arkem.org/2012/02/identifying-computers-behind-nat-with.html
The key papers are:
Source attribution for network address translated forensic captures - https://docs.google.com/viewer?a=v&pid=explorer&chrome=true&srcid=0B9hc84IflFGbODRjMzc4ZjgtNWJiNS00NWRlLWJhYjQtZTk3Mjg1ODc0ODA3&pli=1#
Network forensics - A Practical Introduction - https://docs.google.com/viewer?a=v&pid=explorer&chrome=true&srcid=0B9hc84IflFGbZTkyODRjYzUtNzNiYi00MzNhLWI3OTEtM2M4ZWRkMjQzZDk4&pli=1#


Duplicate logs getting generated

Duplicate logs getting created
Issue
When we added the log in the code, the logs are getting created twice. I have tried adding the log on Delegate its still printing the logs twice in the console.

Xcode: 14.2 (Rosetta)
XCGLogger version: 7.2.0
swift: 5

enabledDestinations.count is 3 for one log registered which is printing two message on the console.

You can see the console the message Theater loaded successfully 🎉 theater: is printed twice on the console.
The LOG is triggered only once in the code.
But in the XCLogger Class the function
open func logln(_ level: Level = .debug, functionName: String = #function, fileName: String = #file, lineNumber: Int = #line, userInfo: [String: Any] = [:], closure: () -> Any?)

is creating duplicate logs in the terminal, which is causing issue in debugging the source code.
It looks like there is bug in the login function which is causing the logs to get generated twice.
Required Result
Print the log only once in the terminal and do not create duplicate of it

Even I am also facing the same issue.


Splunk otel collector own metrics?

Hi,
does splunk otel collector components expose their own metrics? If so could you please point me to the relevant configuration in Helm chart how to get it enabled? I would be specifically interested in any metrics that could tell me if the log shipping to Splunk Cloud goes fine, so would appreciate any hints on that.

At this time, it is not possible to collect metrics from the OTEL components.
However, assuming there are no configuration or communication issues which can impact the agent functionality, the assumption is that all the logs should make it to the destination.
In the event that you do not see logs being forwarded by the agent, you can use the agent logs to investigate any issues.
If necessary, you can enable debug mode to gather more detailed logs that can assist in your investigation.
Lastly, if you run into any technical difficulties with the above, I recommend you file a Support Case with Splunk at http://login.splunk.com/ for assistance to investigate such issues.
Below are links to documentation you may find useful.
Troubleshoot the Splunk OpenTelemetry Collector
https://docs.splunk.com/Observability/gdi/opentelemetry/splunk-collector-troubleshooting.html
Troubleshooting
https://github.com/signalfx/splunk-otel-collector/blob/main/docs/troubleshooting.md

okay I understand, could you please make a feature like that? I think it would be quite useful. I know you could probably get something setup from Splunk Cloud side to get alerted, but from the agent side it would be nice as well I think i.e. if volume of data/logs suddenly drops, timeouts, bottlenecks or errors occur impacting data shipment, it would be easy to build monitoring & alerting using well known Prometheus stack based on such metrics.

The best thing to do is to submit the idea at https://ideas.splunk.com
Thank You

@hgus-gushernandez  I also found this: https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/monitoring.md
so it would look like there are some metrics built in but maybe just not exposed on any port/endpoint yet in the Helm chart?

The Metrics or this document is specific to Splunk Observability and not Splunk Cloud
https://github.com/open-telemetry/opentelemetry-collector/blob/main/docs/monitoring.md
You can read more about each product in the links I have added for each product

Unfortunately, no.
They are different backends with different capabilities and the Splunk OTEL can be used to send data to either back end.
I would recommend you sign up for the free Splunk Observability trial and experiment with the Splunk OTEL in Observability to see the functionality first hand.
During your trial, you will be engaged with a Splunk Observability expert and who can address all your concerns regarding  Opentelemetry.

@hgus-gushernandez so I actually logged into one of the DaemonSet splunk-otel-collector-agent pods and was able to find out the otelcol process actually has the port 8889 opened and appears to be talking http and guess what? it even responds with metrics..  btw the port 24321 is not there, even tho it is in container spec and named http-metrics...
But okay I think I can overlay on top of the helm chart manifests to add that 8889 port to the container and then get it scraped by Prometheus.. But you guys should be exposing this and I should not be needing to hack this to find out this simple info...
/ # telnet localhost 8889
Connected to localhost
GET /metrics HTTP/1.1
Host: localhost

HTTP/1.1 200 OK
Content-Type: text/plain; version=0.0.4; charset=utf-8
Date: Fri, 26 May 2023 16:47:00 GMT
Transfer-Encoding: chunked

800
# HELP otelcol_exporter_enqueue_failed_log_records Number of log records failed to be added to the sending queue.
# TYPE otelcol_exporter_enqueue_failed_log_records counter
(...)

We scrape this endpoint as part of our deployment, please see here:
https://github.com/signalfx/splunk-otel-collector-chart/blob/ea3c4f9b9b55f3cd8543cff3a8d600131fed643f/helm-charts/splunk-otel-collector/templates/config/_otel-agent.tpl#L40

@atoulme okay, but where's this port enabled on the agent DaemonSet? On my Helm inflated manifests I don't see 8889 opened on containers anywhere, and looking in the chart values file I don't seem to find 8889 here?
https://github.com/signalfx/splunk-otel-collector-chart/blob/ea3c4f9b9b55f3cd8543cff3a8d600131fed643f/helm-charts/splunk-otel-collector/values.yaml#L313

The port wasn't added to the daemonset spec. I believe this was intentionally done because we only wanted the agent collecting it's own metrics.

https://github.com/signalfx/splunk-otel-collector-chart/blob/7698ca2498dffe121ec7776227c4bf77733a1dba/examples/default/rendered_manifests/configmap-agent.yaml#L255
https://github.com/signalfx/splunk-otel-collector-chart/blob/7698ca2498dffe121ec7776227c4bf77733a1dba/examples/default/rendered_manifests/configmap-agent.yaml#L179
https://github.com/signalfx/splunk-otel-collector-chart/blob/7698ca2498dffe121ec7776227c4bf77733a1dba/examples/default/rendered_manifests/configmap-agent.yaml#L283

@jvoravong but how do I then get those metrics into my Promethus in the cluster to alert me on say:
otelcol_exporter_send_failed_log_records

?
btw. I patched the ds & deployments to expose port 8889 and getting it scraped now, so are you saying I will now get all the metrics that go into Splunk Cloud also in local cluster Prometheus instance?


Don't know what category should be my PR

Hi, I would like to send you guys a PR for adding
(TresJs)[https://tresjs.org/] and (Cientos)[https://cientos.tresjs.org/]
Is a declarative ThreeJS  way using Vue Components. And Cientos is a helper library for TresJs
As this library is for 3D web experience, where should I put them?
(Open source)[https://next.awesome-vue.js.org/projects-using-vue-js/open-source.html]?
(Canvas)[https://next.awesome-vue.js.org/components-and-libraries/ui-components.html#canvas]?
Or create a new category for 3D stuff?
Thanks

I think this counts as an integration


implement ssl-skip-verify to forward to self-signed-certificates

we are using internally also SSL between our servers (nginx and Service fabric / load balancer) but these are self-signed-certiticates. Today clammit does not allow it
2021/11/04 23:18:04 Interceptor passed this request
2021/11/04 23:18:04 Forwarding to https://<IP_ADDRESS>:8001
2021/11/04 23:18:04 Failed to forward request: 'post' "https://<IP_ADDRESS>:8001/api/Files?api-version=1.0": x509: certificate signed by unknown authority

the PR allows to configure "ssl-skip-verify" (true, false).
If true, will skip SSL validation forwarding connection to use self-signed certitifates, default = false
ssl-skip-verify = false
2021/11/04 23:22:26 Interceptor passed this request
2021/11/04 23:22:26 Forwarding to https://<IP_ADDRESS>:8001
2021/11/04 23:22:26 Request forwarded, response 200 OK

test cases ran
test cases implemented
documentation adapted
(some build documentation adapted)
GitHub build action run
build v0.7.3+ssl-skip-verify on our forked repository

Any change on getting this merged?

Also another question/observation: Why is it not possible that self signed certs that have been added to the os level are considered valid? I am not familiar with the go ecosystem but i would expect that a cert that is trusted on the system level would be working. We tried this together with https://github.com/maxsivkov/clammit-docker (and after adding the necessary cert files) and when we curl inside the container it works but when clammit forwards the request it keeps complaining about invalid cert.


Next 14 - ERR_UNSUPPORTED_DIR_IMPORT

First off, lib looks awesome so very much looking forward to taking it for a spin when I can get this issue resolved.
You can pull this MRE to see the issue if needed.
Packages:

Next 14.0.3
React 18
@kuma-ui/core 1.5.4
@kuma-ui/next-plugin 1.1.14

Error
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import 'D:\Code\Rysk\kuma-mre\node_modules\next\dist\build\webpack\config\blocks\css' is not supported resolving ES modules imported from D:\
<EMAIL_ADDRESS>Did you mean to import next/dist/build/webpack/config/blocks/css/index.js?
    at new NodeError (node:internal/errors:405:5)
    at finalizeResolution (node:internal/modules/esm/resolve:320:17)
    at moduleResolve (node:internal/modules/esm/resolve:946:10)
    at defaultResolve (node:internal/modules/esm/resolve:1132:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:77:40)
    at link (node:internal/modules/esm/module_job:76:36) {
  code: 'ERR_UNSUPPORTED_DIR_IMPORT',
  url: 'file:///D:/Code/Rysk/kuma-mre/node_modules/next/dist/build/webpack/config/blocks/css'
}

Things to note:

Using Node 18.
Using type: module in package.json.

Thanks!

Ok so having actually do a little more digging, this issue seems to be ESM related. If I remove the type: module from package.json and setup my Next config using require/module.exports then things boot up.

Ok so having actually done a little more digging, this issue seems to be ESM related. If I remove the type: module from package.json and setup my Next config using require/module.exports then things boot up.


added EditAction::SelectNone

this PR adds a new edit action SelectNone as the opposite of SelectAll.

@cmyr okay, I got it.

@sysint64 ping on this?

@cmyr I'm still here, just a lot of work, I'll work on my PR's on the following weekends.

@cmyr does Movement::LeftOfLine do the same thing as should BeginningOfDocument? at least the current implementation seems to me what I have tried to do.

It may be the same now, but it shouldn't be the same when we are handling multi-line text correctly; if it is that's likely a bug in the current implementation.

@cmyr Sorry for the delay, I have implemented two methods - line_beginning and line_ending for EditableText, and added Movement::BeginningOfDocument and Movement::EndingOfDocument.

@cmyr fixed.

@cmyr it looks like there is an error on the master branch located in druid/src/box_constraints.rs

this is what I get after rebase:
error[E0428]: the name `tests` is defined multiple times
   --> druid/src/box_constraints.rs:375:1
    |
276 | mod tests {
    | --------- previous definition of the module `tests` here
...
375 | mod tests {
    | ^^^^^^^^^ `tests` redefined here
    |
    = note: `tests` must be defined only once in the type namespace of this module

fixed here: #1187


Question: Receiving Multiple Unmarshal Error

Is there a way to extract multiple errors from an unmarshal operation?
type user struct {
	ID    uuid.UUID `json:"id"`
	Roles Role      `json:"role"`
}

func main() {
	j := `{"id": "asdadsasd", "role": "JACK"}`
	var u user

	if err := json.Unmarshal([]byte(j), &u); err != nil {
		se := err.(*json.SemanticError)
		log.Fatalf("%s\n", se)
	}

	fmt.Printf("%+v\n", u)
}

The value for both id and role are not compliant with the type. Is there a way to get back that information instead of the first field that fails?

When designing v2, we talked about having a mode where we continue parsing as far as possible and collect up the errors, but didn't have a good API to collect up the errors. At the time errors.Join and the implicit interface{ Unwrap() []error } interface did not exist, so the calculus is different today. We decided back then that failing early on the first error was still the right default with the intention of eventually providing this ability.

For past discussion: https://youtu.be/tfWn54N6uwc?feature=shared&t=922

Ok cool. Thanks for getting back to me so quickly.


Specification of decimal equivalent of polynomial (g_matrix entry) for trellis

Hey,
I want to know how to specify the g_matrix entries as decimals. The problem is, I have the generator polynomials, but don't know how to convert them to decimal numbers.
Say one of the polynomials is 1+D^2+D^3, for a memory=3 encoder. Does this correspond to a binary number of 1011 (LSB first format) or 1101 (MSB first format)? In both the examples given in the documentation, the binary number is symmetric.

Hey @aklsh,
Thanks for your interest on this project! The format is MSB first (for instance 1+D <-> 3 <-> 011). In your example, you should use 13 or 0o15.
Please, let me know if you spot an error with this format but I'm quite sure that this is the right way. I'll add you're example to the example file and make some change to the doc to explicit this feature. Could you keep this issue open until then.

Okay. I just figured out what to use and was about to post the same.
Just a suggestion: In the Matlab equivalent convenc, we specify the polynomial as 1011 i.e., decimal 11 or 0o13 for 1+D^2+D^3. So, it would be nice if format used here can match that, so that people can have one standard format across platforms. I think it will only take an additional line of code to reverse the numpy array, got from dec2binarray()

Yes. It would be interesting for sure. I'm just wondering how to detect if 1101 means the polymonial with a binary format or the integer 1101. Any suggestion?

Another way of preserving the retro-compatibility while allowing to use the Matlab LSB format is to add a new argument that specified the format. It would be a string 'MSB'  or 'LSB' . This string would have a default value to 'MSB' to keep the current format. The doc should then highlights that 'LSB'  is the Matlab format.
Is this a goor option?

What I had in mind was preserving the specification of polynomials in form of decimals, but adding another argument (say, matlab=true) so that the binary array returned from dec2binarray can be reversed using a simple if statement.
But what you said also sounds good.

Currently, it is possible to specify the polynomial in hexadecimal/octal format.
I'll add the new argument as it seems to fit what we are looking to do here.

I just had the argument polynomial_format that accept 'MSB', 'LSB' and 'Matlab' to Treillis.


HessianTracer type improvements

https://github.com/adrhill/SparseConnectivityTracer.jl/blob/83cb0b07ea4973e95a9031c9468af50e0927d679/src/tracers.jl#L119-L121
Would it be more coherent to have keys of type eltype(S), since they are also indices, rather than always UInt64?

Sounds good. :)

As seen in #45, we should also

make the AbstractDict subtype parametric, e.g. to use SortedDict in conjunction with SortedVector
rename the type parameter T to I for integer wherever it still appears
add an inner constructor that checks eltype(S) == I

See also #51.


update to 1.5.2 and kube-state-metrics to 0.4.1

WFM (also running already on l8)
RFR @oponder @webwurst

LGTM if it WFY! :D


docs(network): update readme to mermaid diagrams

Add additional use case content
Cleanup diagrams using drawio direct file inclusion
https://github.com/opiproject/opi-api/issues/86

Diagram error not found error on the readme file when you click view file see https://github.com/opiproject/opi-api/blob/e6de02f10ddb5e1479f53c4584c8c6f08556b055/network/README.md#openconfig-model

Thanks, I must have missed removing that as a residual.  Will get that fixed now.


Standalone Table with Row Actions not working

Package
filament/tables
Package Version
v3.0.0-beta1
Laravel Version
v10.10
Livewire Version
v3.0
PHP Version
PHP 8.2
Problem description
I have the following table actions for my standalone table.
public function getTableActions(): array
    {
        return [
            Action::make('Edit')->form($this->getFormSchema())->modalWidth('md')->action(function ($data) {
                $this->company->benefits()->updateExistingPivot($data['id'], ['benefit_description' => $data['description']]);
            }),
            Action::make('Delete')->requiresConfirmation()->action(function ($data) {
                $this->company->benefits()->detach($data['id']);
            }),
        ];
    }

Not sure why, but clicking on the Edit or Delete actions does nothing.
No console errors but there is a network call and the response is always about close-modal. Screenshot of the network call response. Here is the video of what I am facing on the actual project https://share.cleanshot.com/4xZDqtxK

Here is the payload

Expected behavior
The Edit Modal or the Delete Confirmation Dialog doesn't show
Steps to reproduce
Try to add the above code to a standalone table builder
Reproduction repository
Please create a user on the database and visit the root / url to load the table and click the edit button
NA
Relevant log output
No response

Make sure your page contains the <x-filament-actions::modals /> component: https://beta.filamentphp.com/docs/3.x/actions/adding-an-action-to-a-livewire-component/#adding-the-action.
@danharrin could you add a reference of this to the table docs if you think that's needed?

The tables package should inject it by default


Cookbook: improve legend font recipe

This is probably working okay, but it's a bad example because the output looks poor in small figures. Simplify this if possible. Related: #4117
https://scottplot.net/cookbook/5.0/Legend/LegendCustomFontManualItems/

done in #4234


Static constants feature

I have been working with your library for a couple of months, but after trying to use $statics to add immutable constants and object literals grouping immutable constants to a project, I realized you could accidentally modify them.
I have  made a simple implementation that requires ES5 to work. I hope you consider it interesting enough to be merged to the master branch of your library. It includes a simple sample and the corresponding unit tests together with the main implementation.
Don't hesitate to contact me if you think the code can be improved.
Regards,
Federico Budassi.

Hi Federico,
$const looks great. Can you refactor the code to make $const is a plugin instead (I made a comment on your change). I'll merge the code and update README.
Thanks,
Tan

Hi Tan,
Done, $const moved to the Class.plugins.
Regards,
Federico.

Merged. This is a nice feature :+1: Thanks Federico!
Tan

You're welcome.
Federico.


Option to choose deployment type for services

Many of the Watson services have a deployment type option where a 'production' type gets deployed by default. A production deployment has at least two replicas of each pod to support production-scale workloads. This is not needed for trial or demo purposes.
The request is to allow choosing a deployment type option, or choosing 'starter' deployment type by default.
Relevant KC link: https://www.ibm.com/docs/en/cloud-paks/cp-data/4.5.x?topic=discovery-installing

Fixed, all cartridges now support installation_options in which additional attributes (or even full JSON constructs) can be specified to be added to the spec of the CR.


tests to verify toleration feature

Signed-off-by: Jeeva Kandasamy<EMAIL_ADDRESS>Summary:
Additions:

added test suite for tolerations tests
tolerations test suite has threes tests

TestAllInOneTolerations - verifies tolerations on all-in-one -- tolerations on all-in-one deployment
TestProdTolerations - verifies tolerations on production environment with external es cluster -- tolerations on collector, and query deployments
TestElasticsearchProdTolerations -  verifies tolerations on production environment with es self provision -- tolerations on collector, query, and es cluster deployments

NOTE: TestElasticsearchProdTolerations will be skipped in the Kubernetes environment, however in the Makefile, I have mentioned deploy-es-operator (e2e-tests-tolerations: prepare-e2e-tests deploy-es-operator es), hence es-operator will be deployed on Kubernetes environment (which is not useful for now). To avoid this I have to split the tolerations suite. I need your input here.

Changes:

To reuse all-in-one cr I have moved the CRs func from all_in_one_test.go to utils_cr.go
Added AllInOneSmokeTest to all the tests in all_in_one_test.go, to ensure the functionality of the deployment
minor change on AllInOneSmokeTest to support custom query base path, as a result created another func AllInOneSmokeTestWithQueryBasePath

@jpkrohling @kevinearls can you please review the changes

@jpkrohling @kevinearls a test failed due to the es node not available on time. Can you please re-run the tests again?

Test restarted.

@jpkrohling @kevinearls can you please review this PR?

@kevinearls Thanks for the review. I have updated the PR based on your comments.

@jkandasa  Sorry I didn't notice this earlier, but after discussing this with @jpkrohling  yesterday I was wondering if this needs to be an e2e test at all.  I wrote a unit test for this after adding the original code: https://github.com/jaegertracing/jaeger-operator/pull/1299
That test could be enhanced if I missed any cases you are covering.   Doing this in a unit test would be much cheaper than doing it in an e2e test.

@kevinearls np. I believe in the unit tests we can cover this only up to CR(D) level.
I would like to verify these changes on the deployment level.
I believe the e2e test only can do that.

discussed with @jpkrohling and @kevinearls about this PR and come to know that these e2e tests covered on the unit test itself.


How to set authentication for rtmp

!!! Before submitting a new bug report, please ensure you have searched for any existing bugs and utilized
the Ask AI feature at https://ossrs.io or https://ossrs.net (for users in China). Duplicate issues or
questions that are overly simple or already addressed in the documentation will be removed without any
response.
Describe the bug
A clear and concise description of what the bug is.
Version
Desribe your SRS Server version here.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

How to set authentication for rtmp

这个问题在Wiki中有，请看文档。该Issue会被删除，请先阅读FAQ：#2716


Extension not loaded when importing hvplot.pandas after running panel.extension()

Recording a bug observed on the HoloViz tutorial after upgrading its pins to the latest hvplot/panel/holoviews/etc. Not sure it's a hvplot bug so this issue might have to be transferred to the right place after more investigation.

Bug probably caused by the change to extension loading but the fix is indeed in hvPlot.

Hmm, actually pretty confused by this. Don't see how this ever worked given the current logic.

@maximlt hvplot is still on 0.8.0 in pkgs/main/osx-arm64channel. Would it be possible to update it? I'd been having this issue and randomly found this bug and saw that osx-arm64 channel still had the buggy version.

It is not possible to update that channel by anyone on the holoviz team, but you can install it through the pyviz channel: conda install -c pyviz hvplot=0.8.1.


Enable template to include features which have been deactivated from …

…base template.

Hi @nickcox, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Thank you for your contribution! It's an interesting PR I think, and I will check it out the coming days.

PR will require manual porting to new PnP-Sites-Core repo + merging with current code

Hi Nick,
Thanks for contributing this PR!
This PR has been open for a long time due to the fact that we've been restructuring the PnP repositories. Today the provisioning engine updates happen in the PnP-Sites-Core library and a lot of changes have been implemented over the last months.
Therefore it's very hard for us to merge this PR as is...we would like to ask you to resubmit this PR (if still relevant) as a new PR in the PnP-Sites-Repo. We apologize for the extra efforts this is resulting into.
This also means that we're going to close this PR in the PnP repo.
Best regards,
The PnP Core team

Thanks, Bert.
How should I stop this sort of thing from happening again? Is there some sort of roadmap where contributors can find out about upcoming major restructures?

The repo split is the only major restructuring we did over the last 1,5 years...and no new major restructuring is planned. It's unfortunate that it collides with your PR submission, apologies for that.


Make sure purged packages are removed from removed cache

Just fixed with adding a purge command (not pushed yet).


[loc][Query][vscode-azure-sphere-tools] Localizability of string: 0; "./src/resources.errorConfigUnavailable" #91349

Hi,
please help localization team to answer question from translator.
In case translator is not sure about context/meaning/localizability of string we raise a question to Core team.
Could you help please? Thanks!**
Source:
Error: Required azure sphere settings not found. Please configure in settings.json or run command 'Azure Sphere: Configure settings' to apply default settings.
File:
Src\vscode-extensions\vscode-azure-sphere-tools.xlf
Query:
Please, determine whether this command should be translated

@PDostalek please provide the link to the document which you are following so that I can better assist.

@PDostalek Please help with the link of document on which above document feedback request is created.
Under this repository we want to make sure any issue is linked to a doc so we can help others that may be having the same issue.

Since we have not heard back from you we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.
In case your query is not related to document content, open a new forum thread in MSDN forum or Stack Overflow as it could be better place to get help on your scenario.
These forum community members could provide their expert guidance on your scenario based on their experience. Thanks.


Unsafe checks only run one time after the label is applied

Bug Description
If the unsafe label is applied to a PR, it will run the checks once on the latest commit, but not on future commits. The label must be removed and then reapplied
Steps to Reproduce

Open a PR that triggers a change
Apply the label
Confirm the tests run
Push another commit
Confirm the tests don't run

Expected behavior
The unsafe checks should run on every subsequent commit

fixed in unified-fides


Galaxy S3 stock browser scores 380 + 3 bonus

But Chrome is listed as the top mobile browser with 371 + 11.

Non Google versions of the stock Android browser are now supported.


Create a helper method to get DeviceSpecimenType UUID (#3765)

BACKEND PULL REQUEST
Related Issue

Fixes #3765

Changes Proposed

Create a helper method within QueueMutationResolver to keep things DRY

Additional Information
N/A
Testing
N/A
Checklist for Primary Reviewer

[ ] Any large-scale changes have been deployed to test, dev, or pentest and smoke tested
[ ] Any content updates (user-facing error messages, etc) have been approved by content team
[ ] Any changes that might generate questions in the support inbox have been flagged to the support team
[ ] GraphQL schema changes are backward compatible with older version of the front-end
[ ] Changes comply with the SimpleReport Style Guide
[ ] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[ ] Any dependencies introduced have been vetted and discussed

@emyl3 finally following up on our conversation from this morning about how to unit test this :)
Generally, you'll want to follow a model like OrganizationMutationResolverTest. You'll need to create mocks for each Service used in QueueMutationResolver, and then assert things are happening to the mocked services instead of asserting on the new method directly.
You have two options for asserting on the mocks:

Assert that the parameters are what you expect (see this example in OrganizationMutationResolverTest)
Assert that the method is being called the number of times you expect. This is probably more helpful when you're not expecting anything to be called, such that you can verify the method is never called - here's an example.

If you have questions or want to pair on this, just let me know - working with mocks/Mockito can be tricky!

@emmastephenson @johanna-skylight @fzhao99 Ready for re-review along with some questions 😅 Thank you!!

@emyl3, awesome questions!! Let me know if you want to discuss anything further offline.


Feature Request: Redstone NOT gate option

Describe the feature you'd like to have added
On the Redstone cards could you program a NOT gate option, to automatically invert when signal should run or should not run. For example your episode today you were attempting to get a spawner to stop running when your blood alter was full and you had to add a lot of extra steps, with a NOT gate option on the redstone card you could have clicked the one button to flip the redstone output to what you needed.
Additional information
No response

I consider this one scope creep - I want to let redstone mods handle this functionality for now.


OpenTelemetry Tracing

Feature proposal: add support for OpenTelemetry Tracing in API commands, is it a good idea to implemenet in this lib

Feature proposal: add support for OpenTelemetry Tracing in API commands, is it a good idea to implemenet in this lib

why?

I needed to implement tracing in system and minio is a part in the system, but since there is not support from the box, i did my own wrapper
i suppose i could help potential users dont think about that

You can provide custom HTTP transports in Options, which seems like a natural place to have tracing.

You can provide custom HTTP transports in Options, which seems like a natural place to have tracing.

This.


minor css selector fixes to support aem 6.2 platform of bobcumber

bobcumber integration aem 6.2

approved


[Bug] Consumyroll: video sub does not match selected sub

when i play a video in the original dub the selected sub is ignored so first sub in the list is played

subs provided for dub versions are signs and songs in official crunchyroll (in the small anime i've tested) so it is same for consumet too. they don't cache the subs from subbed version to server with dubs like kamyroll


feat: abs func implementation + Unary Expr implementation

This is the implementation of the abs scalar function on sqlite. This is associated to the issue https://github.com/penberg/limbo/issues/144.
This is my first time working with a bytecode but I followed what sqlite do. it looks similar but they are not exactly the same
Sqlite:

Limbo:

One doubt that I have is if it is necessary to increase the program counter after each instruction function is finished as I saw in the vdbe we have an instruction called "Transaction" that is executed after "Halt" and it increase the program counter.

I think Once here is actually just caching the result for future invocations of the program. Probably not very useful for this example, but I can see how that would come handy if the executed function was more complex. We do want to implement that later, but the PR is good as-is.


Add Refer support.

Fix some errors in the Refer process.

@rlsutton1 @ghenry
General calls and Refer calls use the same EventCallXXX event and may need to separate the EventReferCallXXXX event?
like:
EventReferAccepted, EventReferCallFailed, EventReferRequestFailed.
To tell the refer initiator that the peer has successfully transferred


allocInstanceDataBuffer buffer entry count does not match parameter

In an application of mine, the instance data buffer returned by allocInstanceDataBuffer(n, stride) had its size inflated thanks to the logic in allocTransientVertexBuffer. I was creating a 100 element IDB and got a 101 element one out of the call.
If calling the unary setInstanceDataBuffer(idb) as in the 05-instancing sample, an additional instance with bogus data will be drawn.
In my application I worked around this by using the binary setInstanceDataBuffer(idb, n) form. I have yet to investigate if the 05-instancing sample accidentally works or if the phantom instance just happens to not show up.

Still didn't have time to look into this. My guess is that buffer alignment causes it. If you can change 05-instancing to reproduce issue you're having that would be the quickest way to fix it.

I confirm 1 extra element and a drawn visual artefact.
https://github.com/bkaradzic/bgfx/blob/master/src/bgfx_p.h#L1540
On this line, I don't understand the +1, but removing it seems fixing the problem...

+1 is to allow extra data in buffer for alignment.

(and since index buffer 32 bits caps, does sizeof(uint16_t) is still correct (line 1522) ?

Transient buffer doesn't support 32-bit indices now.

bonus : how is computed the value 80 for instanceStride in example 5 ?

  64 bytes for matrix
+ 16 bytes for
---------------------
= 80 bytes stride

I though it will be obvious from data in instance buffer here:
https://github.com/bkaradzic/bgfx/blob/master/examples/05-instancing/instancing.cpp#L190

I found what's the issue, I'll fix it soon.

great ! thanks for infos.


Fixing pagination for non-geometrical features; removing dead code fr…

…om settings.
@3nids When testing pagination in QGIS it appears that QGIS is unable to add the signalo_vl.officialsigntype layer even though it opens (and paginates) fine as of this PR. Maybe the absence of a geometry field is something that causes trouble to QGIS WFS3 provider?

Waiting on https://github.com/qgis/QGIS/pull/53011.


Custom binaries host not used when the package's name contains a dash

I am building an electron app unsing electron-builder. If I want to use custom binaries for a dependency, e.g. node-hid I am setting node-hid_binary_host = http://localhost:1234/ in .npmrc, and npm confirms this is set.
It seems that at some point the dashes in the variable names are replaced with underscores, so prebuild-install can not find the variable and does not use the alternate server.
By patching the getHostMirrorUrl function to replace dashes with underscores in the pkg.name I was able to get the desired behaviour, but I'm not sure this will not cause collisions at some point.

By patching the getHostMirrorUrl function to replace dashes with underscores in the pkg.name I was able to get the desired behaviour, but I'm not sure this will not cause collisions at some point.

Can you send a PR so we can see what that looks like?
Perhaps we can read both the original env var and the underscored variant.

https://github.com/prebuild/prebuild-install/pull/97

Hm, npm docs state that you can use : to separate the package name and config key. If so, maybe the PR is not needed.
Can you try npm config set node-hid:binary_host http://localhost:1234/?

That also results an underscore in the package name.

OK. Was worth a shot :)

I did a test on a mock package and replacing dashes indeed seems to be builtin npm behavior. I'm OK with the proposed fix (though we have 67 dependents with a dash in their name). The only use case I can think of that would break, is manually setting a npm_config_package-name_.. environment variable but I prefer following npm's behavior.


Remove fix sticky for save widget

Pin the save widget to the top of the screen and add a jump to the top link in the page.

Back to Top Link without additional metadata

Bach to Top link with additional metadata


Removed redundant debug tasks

Removed redundant debug tasks (METROAE-519).

This PR is no longer valid. Will be replaced with a new one.


⚠ Bofrost.at has degraded performance

In 2e7f069, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 935 ms

Resolved: Bofrost.at performance has improved in b01ea6d.


Adapter startet nicht mehr

The problem
der Adapter startet nicht mehr. Hier das Log
starting. Version 0.4.0-beta.3 (non-npm: ice987987/ioBroker.husqvarna-automower#6339d7b584f59c1874ad3c647536996077de3a0e) in /opt/iobroker/node_modules/iobroker.husqvarna-automower, node: v18.16.0, js-controller: 4.0.24
Version of "husqvarna-automower"-adapter
v0.4.0-beta.3
Husqvarna Automower Type
405x
Version of Node.js
v18.16.0
Version of ioBroker js-controller
9.5.1
Version of "admin"-adapter
6.3.5
Operating system incl. version running ioBroker
Linux Debian
Checklist of files to include below

[X] iobroker.current.log (Please share the logs in debug mode and check that there are no personal information)
[X] Contents of the JavaScript browser console (always include in cases of issues with the user interface)

Additional information & file uploads
No response

dublicate https://github.com/ice987987/ioBroker.husqvarna-automower/issues/117


Add a sample to connect to Azure Cloud

This samples demonstrates how to connect to Azure Cloud IoT hub which is based on MQTT protocol. User has to create an account in Azure Cloud and provide those details
using Kconfig options. This sample first acquires DHCPv4 address and opens a secure connection with Azure cloud. Then opens a MQTT connection and publish messages randomly.

@rlubos Thanks for the review. On overall I started modifying mqtt_publisher sample to work with Azure cloud. There are left overs from that sample.
Idea is nice to have end-to-end sample/demo with cloud. e.g. temp sensor connected to cloud.
I will work on your inputs and update the PR.

Ok, I've managed to run the sample finally and connect to the cloud. Some feedback from the Azure newbie, for consideration:

I think we should support qemu/native_posix target in the sample, so that it works out-of-the box. I've tested with qemu_x86 and had to apply some extra configs, the same as I've described some time ago here. I guess we need some board specific config or an overlay file for that.
It would be nice to have DNS enabled (for instance in qemu configuration), fetching the IP address manually is kind of troublesome. LWM2M engine has a nice mechanism to resolve IP address, so that you can provide either IP address directly or the server name - the engine will figure out which one he deals with:
https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/net/lib/lwm2m/lwm2m_engine.c#L4252
This is a remark more to the Azure documentation, but I guess we could cover this in our docs as well. I found the SAS token generation quite problematic, I've ended up installing VS Code Azure extension to have this done, after other methods described here failed (Azure CLI method simply didn't work, the documented command does not seem to be supported; the Python script method IMO is missing some docs on what are the parameters expected). @rveerama1 How did you generate this token, perhaps you know an easier way to generate them?

Ok, I've managed to run the sample finally and connect to the cloud.
Thanks, but I am working on your comments, I will update PR by Monday.

1. I think we should support qemu/native_posix target in the sample, so that it works out-of-the box. I've tested with `qemu_x86` and had to apply some extra configs, the same as I've described some time ago [here](https://github.com/zephyrproject-rtos/zephyr/issues/14790). I guess we need some board specific config or an overlay file for that.

That would be nice, I will give a try.

2\. It would be nice to have DNS enabled (for instance in qemu configuration), fetching the IP address manually is kind of troublesome. LWM2M engine has a nice mechanism to resolve IP address, so that you can provide either IP address directly or the server name - the engine will figure out which one he deals with:

I am working on it. I am investigating DNS with SOCKS5. So that I can modify sample properly.

3\. This is a remark more to the Azure documentation, but I guess we could cover this in our docs as well. I found the SAS token generation quite problematic, I've ended up installing VS Code Azure extension to have this done, after other methods described [here](https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-devguide-security#example) failed (Azure CLI method simply didn't work, the documented command does not seem to be supported; the Python script method IMO is missing some docs on what are the parameters expected). @rveerama1 How did you generate this token, perhaps you know an easier way to generate them?

Yes it was bit cumbersome. I tried to install https://github.com/Azure/azure-iot-sdk-csharp/releases windows installer on my windows laptop and generated SAS token.

@rlubos I fixed some of your comments and modified a bit. I am working on DNS addition.

@dbkinder Can you please take a look?

@dbkinder Updated as per comments. Please re-review.

ping @dbkinder, can you recheck this one


ts.getResolvedModule cannot be used to get ambient modules

Bug Report
🔎 Search Terms
getResolvedModule ambient module
🕗 Version & Regression Information
This repros on the latest version of TypeScript, and I don't believe it's ever worked on previous versions.
⏯ Playground Link
See repro steps in linked https://github.com/microsoft/rushstack/issues/3335.
💻 Code
// index.d.ts
import './modules';
export { Foo } from 'foo';

// modules.d.ts
declare module 'foo' {
    class Foo {}
}

// some_other_file.ts
ts.getResolvedModule(indexSourceFile, 'foo');  // returns `undefined`

The helper ts.getResolvedModule is found at https://github.com/microsoft/TypeScript/blob/v3.2.2/src/compiler/utilities.ts#L218.
🙁 Actual behavior
ts.getResolvedModule does not seem to return the resolved module for ambient modules.
🙂 Expected behavior
I'd expect this helper to return the resolved module.

Ambient modules are not “resolved” like other imports/references during module resolution, as we haven’t processed all files yet, and the declaration could be anywhere. Module resolution is concerned only with what can be found on the filesystem, and pulling those resolutions is what this API is supposed to do. Ambient module declarations get symbols in the global scope (just like any other declaration in a non-module file) during binding, and when a type checker is initialized, those symbols get merged into one big global symbol table. Only then do we know what ambient modules exist, and so module name resolution in the checker begins by querying those global symbols before trying the filesystem resolutions that you’re looking at. TLDR, the API is doing exactly what it’s supposed to, and it would be impossible for it to do what you want because it doesn’t accept a TypeChecker parameter.

Thanks for the quick response @andrewbranch, that makes sense.
Maybe you can help me find an alternative TS compiler API to solve my problem. The @microsoft/api-extractor package uses the TS compiler to (1) determine if import/export statements are "external" to the current package (via ResolvedModuleFull.isExternalLibraryImport) and (2) get the corresponding exported declaration for a given import/export statement (e.g. get Foo for import { Foo } from 'foo').
For an import/export of an ambient module, what helpful TS compiler APIs are available to do these two things? If this isn't the right forum for these questions, no worries (maybe the issue linked above is better). Thanks again!

For just module specifiers, should probably be using checker.resolveExternalModuleName, and check if any/all of the declarations of the symbol returned are in node_modules, or not in the project’s root files, or whatever your specific use case requires. For imported symbols like Foo in your example, you should use checker.getAliasedSymbol and do the same.

Is checker. resolveExternalModuleName exposed to the public? I'm on TS 4.5.5 and don't see it on the TypeChecker interface. There's also a code comment on the api-extractor repo that says:

Map the filename back to the corresponding SourceFile. This circuitous approach is needed because we have no way to access the compiler's internal resolveExternalModuleName() function

Hm sorry, I didn’t notice that function was internal. checker.getAliasedSymbol should cover all the cases where the import declaration has at least one name binding. I think you can use checker.getSymbolAtLocation on the string literal of the module specifier, which dispatches to resolveExternalModuleName.


Mitm fails with SEC_ERROR_REUSED_ISSUER_AND_SERIAL

I had a working application performing mitm proxy.
I upgraded Go from 1.10 to 1.12 along with move to go modules.  As such, I am not 100% sure I have correctly identified whether the issue is with Goproxy, GO or another dependency.  I have tried latest and slightly older version of Goproxy and I tried Go 1.11 and so far have not found a version which works.
When I retrieve pages, they initially arrive without error.  If I then refresh the page, Firefox reports the error: SEC_ERROR_REUSED_ISSUER_AND_SERIAL - the debug message from Goproxy is WARN: Cannot handshake client www.gstatic.com:443 remote error: tls: bad certificate
If I wait a minute, I can sometimes refresh the page, though some objects get the same error.
I suspect this issue is related to this old issue: https://github.com/elazarl/goproxy/pull/314 and and in another project which uses Goproxy: https://github.com/bettercap/bettercap/issues/514
Or more current - is this maybe the cause? https://github.com/elazarl/goproxy/issues/297
If there is no obvious answer, I will try and build the absolute minimum project which replicates the issue reliably.

I think we understand the issue now. This is because Go now takes more data
than it needs from the random source.
We're thinking of a solution now.
On Wed, May 8, 2019 at 1:45 PM richp10<EMAIL_ADDRESS>wrote:

I had a working application performing mitm proxy.
I upgraded Go from 1.10 to 1.12 along with move to go modules. As such, I
am not 100% sure I have correctly identified whether the issue is with
Goproxy, GO or another dependency. I have tried latest and slightly older
version of Goproxy and I tried Go 1.11 and so far have not found a version
which works.
When I retrieve pages, they initially arrive without error. If I then
refresh the page, Firefox reports the error:
SEC_ERROR_REUSED_ISSUER_AND_SERIAL - the debug message from Goproxy is
WARN: Cannot handshake client www.gstatic.com:443 remote error: tls: bad
certificate
If I wait a minute, I can sometimes refresh the page, though some objects
get the same error.
I suspect this issue is related to this old issue: #314
https://github.com/elazarl/goproxy/pull/314 and and in another project
which uses Goproxy: bettercap/bettercap#514
https://github.com/bettercap/bettercap/issues/514
Or more current - is this maybe the cause? #297
https://github.com/elazarl/goproxy/issues/297
If there is no obvious answer, I will try and build the absolute minimum
project which replicates the issue reliably.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/elazarl/goproxy/issues/341, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB7RIUBOB3FC7S23VGA5F3PUKVOBANCNFSM4HLQRMXA
.

I have found a fix for this. Let me know if you want this as a PR though it is very simple so you could just take from here.  The solution is to replace use of crypto/rand
The edit is to signer.go
import gitlab.com/NebulousLabs/fastrand
Replace block of code from ~line 47 with
hash := make([]byte, 20)
fastrand.Read(hash)
serial := new(big.Int)
serial.SetBytes(hash)

I am unqualified to comment on the cryptographic implications but this is working perfectly well for me and I no longer get the SEC_ERROR_REUSED_ISSUER_AND_SERIAL error.

I want to understand why does this work before using it.
It should have the same problem of reading excessive bytes on purpose by
the keygen utils.
On Tue, May 21, 2019 at 5:49 PM richp10<EMAIL_ADDRESS>wrote:

I have found a fix for this. Let me know if you want this as a PR though
it is very simple so you could just take from here. The solution is to
replace use of crypto/rand
The edit is to signer.go
import gitlab.com/NebulousLabs/fastrand
Replace block of code from ~line 47 with
hash := make([]byte, 20)
fastrand.Read(hash)
serial := new(big.Int)
serial.SetBytes(hash)
I am unqualified to comment on the cryptographic implications but this is
working perfectly well for me and I no longer get the
SEC_ERROR_REUSED_ISSUER_AND_SERIAL error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/elazarl/goproxy/issues/341?email_source=notifications&email_token=AAB7RIVQ7J2Z2QPOYQOFYTLPWQDW3A5CNFSM4HLQRMXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV4FCFA#issuecomment-494424340,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB7RIT75DHZWDK572BLA2DPWQDW3ANCNFSM4HLQRMXA
.

Hi,
Any progress on this issue?
Thanks

Currently we use a random serial number for each generation, closing this issue


Preserve non-ascii-compatible range utf8 in document titles for citations

See https://notes.ietf.org/rpat-20230123#handling-of-non-ASCII-symbols-in-titles-of-documents.
The reference should use whatever the draft used in its title, and not downgrade it to an ascii-compatible-range equivalent

I tried to debug this, but when I resubmitted a faked -07 of this doc, the resulting bibxml has the correct title.
I think this was only submitted as plaintext, and the secretariat may have manually adjusted the title? The document title on https://datatracker.ietf.org/doc/draft-briscoe-docsis-q-protection/ also has the (r)...

That's the issue. Right now, the bibxml at
https://datatracker.ietf.org/doc/bibxml3/reference.I-D.briscoe-docsis-q-protection.xml
contains
      <title>The DOCSIS(r) Queue Protection Algorithm to Preserve Low Latency</title>
Given what was submitted in the current version of the doc (the policy around whether what that doc contains is right or not is not what this issue is about),
It should contain
      <title>The DOCSIS® Queue Protection Algorithm to Preserve Low Latency</title>


Bind must be called on a function

Describe the bug:
When trying to boot up an Ember app, I get this error: Bind must be called on a function
The stack in the log points at some minified file, so I'm not sure how to best see what is going on -- but on a real machine, the same code runs just fine.
 - stack: TypeError: Bind must be called on a function
    at bind (<anonymous>)
    at Object.eval [as bind] (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:3:33198)
    at eval (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:3:37215)
    at Object.eval (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:3:37490)
    at r (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:3:613)
    at eval (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:3:805)
    at eval (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:3:815)
    at D (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:1:821)
    at Object.eval (/home/projects/gqaimkxpv.github/node_modules/esm/esm.js:1:1479)
    at Object.function (https://gqaimkxpv--github.jw.staticblitz.com/blitz.cb1eb6b301af424e8dffb694972509f06ea6b5db.js:11:15318)

Link to the blitz that caused the error: https://stackblitz.com/github/ember-cli/ember-new-output/tree/stable?file=package.json
Steps to reproduce:

Go to the linked blitz
run npm start
See error

Expected behavior:
dev server boots (there may be other errors between now and then, as the previous error was https://github.com/stackblitz/webcontainer-core/issues/53, so we'll see what we run in to next!)
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 91.0.4472.77 (Official Build) (64-bit)

I'm not entirely sure, but it looks like something ESM related. We don't support ESM yet unfortunately. But the Node.je ecosystem is adapting so it's definitely on the radar as one of the big first things we have to tackle.
The node-esm package, which looks like it's the one that's failing, doesn't work either. I haven't looked into why that is though so can't give you feedback yet on that front.

any updates? I'm very excited to try this out, but node ESM is pretty much a requirement for all my work <3

Hey there! Thanks for reporting this issue. It's indeed related to some internals on our end. I am actually working on ESM built-in support and part of this effort, I have fixed some issues that break the esm package you're using. So I'd say stay tuned and hang in for a bit longer. ESM is cooking 🧑🍳

Here is a minimal reproduction https://stackblitz.com/edit/node-fofpws?file=index.js

Sorry to bother, but any updates? This is the current known blocker for emberjs folks to use stackblitz ❤

I believe you tweeted it yourself, but this works now 🎉. Let us know if you run into any issues 🙏 .

Thank youuuuuu!


InAppBrowser.openWithSystemBrowser crash on iOS

Environment
Flutter version:  1.17.1
Plugin version:  3.2.0
Android version:
iOS version: 13.4.1
Xcode version:  11.5
Device information:  iPhone 7 Plus
Description
Expected behavior:
Open the system browser
Current behavior:
Application crashed
Steps to reproduce
use:
InAppBrowser.openWithSystemBrowser(url: url);

Fatal error: Unexpectedly found nil while unwrapping an Optional value: file /Users/pmaes/.pub-cache/hosted/pub.dartlang.org/flutter_inappwebview-3.2.0/ios/Classes/InAppBrowserManager.swift, line 37

Fix available in the next release!

thank you ;)

New version 3.3.0 released!


Build failing: dragdrop package

<EMAIL_ADDRESS>build /home/max/Projects/Continuum/DOCs/OSS-Docs/phosphor/packages/dragdrop
tsc

src/index.ts(226,7): error TS2322: Type 'Promise' is not assignable to type 'Promise'.
Type 'string' is not assignable to type 'DropAction'.

Hi @maxgamurar, running the tsc command directly uses the global typescript compiler, while npm run build uses the version local to that package.

You can run tsc --version to check what your global version is, and run npm install -g<EMAIL_ADDRESS>to get it in sync with what phosphor is using.

Yeah. Typescript doesn't really follow semver, and this error is due to a backwards incompatible change made in Typescript 2.4. This is why we pin to a minor version (~2.3.0 currently) and use that over a globally installed tsc in our build scripts.
As for the direct issue, it will be fixed with the next release of Phosphor when we upgrade to Typescript 2.5


fix issue 'no user found using id [-1]' if try to change status

This is fix error "No User found using id [-1] to check read access" on attempt to change task status when review was requested but nobody has reviewed it yet.
In case when new record is created in task_review table then reviewed_by is null initially. After that if user tries to change task status, error No User found using id [-1] to check read access will occur. This is because we are trying to add a notification for an unknown user.

Note: Fixes #741


Record GLSurfaceView

First of all, thank you for such a great work, there is a lot of useful info and examples here as well as answers by you on stackoverflow.
I've checked RecordFBOActivity and I see that you using simple SufraceView with your own RenderThread instead of GLSurfaceView to have more control over EGLSurface.
I'm quite new to openGL and maybe don't fully understand all nuances but as far as I see (in blit mode) you're activating framebuffer, then making draw call to execute all gl draw commands and then drawing it again twice, once for display and once for videoencoder.
Can it somehow be accomplished with GLSurfaceView ? I mean recording, since I want to use render thread of GLSurfaceView instead of adding my own RenderThread. I'm currently drawing camera texture and adding some other textures using glDraw calls so I want to record all this stuff. I know that glSurfaceView is manage all EGL context and surface by itself, but we can provide factory for it, although we still won't be able to access underlying EGLSurface.
Also what is the most performant way to record, I see that you use 3 modes, mode with gl 3.0 doesn't fit since it covers too few devices so what is the best way: draw twice method or FBO ?
Thanks.

It can be done with GLSurfaceView, but because of the way the class manages the EGLSurface it's a lot harder.
At one point I made a game recording patch for Android Breakout, but that had a serious flaw in its implementation (see this link).
I expect that copying rendered output will be faster than re-rendering the entire scene for most applications.  One advantage of "draw twice" is that you don't have to wait for the first render to complete, but that's not important for most apps.

What flaws does GameRecorder has ? Using SurfaceView with RenderThread does not have these flaws ?

It has a race condition, same as described in https://github.com/google/grafika/issues/36.

Got you, thanks

Everything works great, thanks. Can also audio be recorded alongside with video ?

@fadden I've been profiling opengl calls for a while and mentioned that binding offscreenTexture while recording with frame buffer for gl 2.0 is quite expensive, it takes a really long comparing to other operations. I'm using 1 texture atlas for all my sprites to not bind textures when it is not needed since passing textures to GPU is an expensive operation that should be avoided when possible but it seems like I just can't add generated offscreenTexture to the atlas and binding it on each frame is expensive.
I've search info about opengl textures optimization for some time, but didn't find a solution. My guess was somehow push to GPU both sprites atlas and offscreen texture for record on setup phase and then somehow just switch between them when I need.
Can it be optimized somehow ? Thanks.

Hi, I to want to achieve the same thing you want(ed): record camera preview and some textures added on top of it. Did you manage to do this? If so, can you show me your example?

@zoopolitic hi ! can you tell me how to record glsurfaceview ? it seems CameraCaptureActivity record surfacetexture instead glsurfaceview ? is it right? sorry for my poor english.

@shiguiyou I ended using SurfaceView instead of GLSurfaceView

@zoopolitic ...ok,thanks for reply.


Update pagerduty Authorization in app-config.yaml

Hey, I just made a Pull Request!
Fixes #6142.

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Please link this PR with https://github.com/backstage/backstage/issues/6142

Thank you! 🙌


Is there any plan to release the code?

Thank you for your attention to our work :-) We have just released the code.

@Hsu1023 nice job!


Disable identation

Hey I love the plugin!
Is it possible to disable identation?

You can now set auto_indent = false inside options to disable indentation!


Adds San Diego to React.js community meetups

San Diego JS holds a monthly meetup dedicated to React.js

@anthonyvalera thanks!


Missing uws dependency in engine.io server.js feathers dependency

Background
Installed a feathers api inside a vue client.
Feathers boots
Vue will not compile. It insists that a dependency "uws" is missing inside the api (feathers) libraries.
On an older feathers cli, the dependency was there, but empty with a warning it was discontinued.
On the very latest feathers cli, it's just not there at all, even though the feathers dependency engine.io still needs it.
Steps to reproduce
Create a new feathers api in /api with realtime and rest
Then create a vue client in  and add an api.js which uses these imports
import feathers from '@feathersjs/feathers'
import socketio from '@feathersjs/socketio-client'
import auth from '@feathersjs/authentication-client'
import io from 'socket.io-client'
Then boot them.
(First please check that this issue is not already solved as described
here)
googlefoo does not lead me here It leads me to a variety of places where this issue is talked about, mostly in vue contexts, not feathers cli contexts. It's possible I missed something and it's already solved. I'd like that to be the case, if only this can be solved easily.

[ ] Tell us what broke. The more detailed the better.
Consider this console report when booting the vue client reporting a compiler failure

This dependency was not found:

uws in ./api/node_modules/engine.io/lib/server.js
To install it, you can run: npm install --save uws

googlefoo suggests adding uws to the vue client package.json
(to me, that seems nonsensical because it's a feathers dependency, not a vue dependency)
Observation, and this is why I am here: I built the feathers twice, second time after I updated to the latest cli hoping this would go away (it didn't); it is still calling for an engine.io which insists on loading a version of uws which, according to the engine.io package.json is "uws": "~9.14.0"
But, there is no uws package either in the engine.io node_modules or in the outer api/node_modules.
With the older cli version, there was, in fact, a uws in api/node_modules but it was just a package.json with a lot of messages, and a readme that says the product was discontinued.
Expected behavior
What should happen is that Vue console will tell me it's up and running on port 8080
Actual behavior
The console above said it all; compiler error. Backtracking from the vue api.js to the innards of the feathers platform, it could not make connections by way of engine.io to engine.io's call to import uws. It's just not there. Why the feathers compiler didn't find that is a mystery to me, but then, this is relatively new to me.
System configuration
Tell us about the applicable parts of your setup.
Module versions (especially the part that's not working):
Here's what's in my feathers app
"@feathersjs/authentication": "^2.1.16",
"@feathersjs/authentication-jwt": "^2.0.10",
"@feathersjs/authentication-local": "^1.2.9",
"@feathersjs/configuration": "^2.0.6",
"@feathersjs/errors": "^3.3.6",
"@feathersjs/express": "^1.3.1",
"@feathersjs/feathers": "^3.3.1",
"@feathersjs/socketio": "^3.2.9",
In theory, that's what I got when I did a fresh npm i @feathers/cli before this
NodeJS version:
12.4.0
Operating System:
MacOs mojave
Browser Version:
we're not even to the browser yet
Module Loader:
npm 6.9.0

Since the Feathers CLI does nothing frontend related I'd recommend separating the server and frontend code.
The most likely reason for this error is that your app is pulling in @feathersjs/socketio instead of @feathersjs/socketio-client somewhere.

Indeed! They are separated.
App.js, made by vue cli does bring in @feathersjs/socketio. But, the codebase that is running on, made with older feathers cli, is online and running fine. Very confusing.

It should not. Server side code should not be built for the client, only Feathers client side modules.


Avoid using async functions from synchronous code

This changes the retrytimeout to use Thread.Sleep as we cannot use Task.Delay in this context due to risk of deadlocks.

I don't think it is a good idea to use Thread.Sleep. That would suspend the thread which might run many tasks in the host application and could cause even more harm.
I think I'd either use Task.Delay(x).GetAwaiter().GetResult() or remove the delay between retries all together because of possibly too many side effects.

I think I'd either use Task.Delay(x).GetAwaiter().GetResult() or remove the delay between retries all together because of possibly too many side effects.

Wouldn't that also block the current thread in the same way as a Thread.Sleep would do, as the GetResult() call is blocking the current thread until the Task ends?

I think I'd either use Task.Delay(x).GetAwaiter().GetResult() or remove the delay between retries all together because of possibly too many side effects.

Wouldn't that also block the current thread in the same way as a Thread.Sleep would do, as the GetResult() call is blocking the current thread until the Task ends?

Looked a bit more into this for a reference for my reasoning here. David Fowler at Microsoft has a great collection on guidance on async programming. He has a section on "sync over async" about that invoking an async method (for example Task.Delay) and then blocking the thread while waiting (GetAwaiter().GetResult()) for it to complete could be worse than just calling the synchronous version (Thread.Sleep):
https://github.com/davidfowl/AspNetCoreDiagnosticScenarios/blob/master/AsyncGuidance.md#warning-sync-over-async


Sort JSON object-keys for consistency

In your last commit (39ddea188b150f10ea5c504cf42e47acc2bab8f5, which included the Changes message "Fixed map data (South Kensington is not linked to Knightsbridge)"), you made these changes to the map data:
            "id" : "K014"
         },
         {
-           "link" : "H036,S012",
+           "link" : "H036,G003",
            "name" : "Knightsbridge",
            "line" : "Piccadilly:33",
            "id" : "K015"
@@ -1703,7 +1703,7 @@
         },
         {
            "id" : "S012",
-           "link" : "S006,G003,K015",
+           "link" : "S006,G003",
            "name" : "South Kensington",
            "line" : "Circle:31,District:30,Piccadilly:32"
         },

When I run this module's tests as they currently are on master, they fail. With the change in this PR, they pass again.
Your change affected:

G003 (Gloucester Road, aka GR)
H036 (Hyde Park Corner, aka HPC)
K015 (Knightsbridge, aka K)
S006 (Sloane Square, aka SS)
S012 (South Ken, aka SK)

Before your change, the K connectivity was as follows, which was correct:
SK - K - HPC

Afterwards (one right, one wrong - you're now skipping SK which is between K and GR):
GR - K - HPC

Before your change the SK connectivity was as follows, which was correct:
        K
       /
GR - SK - SS

Afterwards it is:
GR - SK - SS

In other words, you removed the Piccadilly line connection between SK and K, despite it being shown in your screenshot.

I have therefore updated this PR to undo that map change, and update the test so it passes with the re-corrected data.

@mohawk2 you found a bug in Map::Tube added in v3.65. I have now reverted that changes.
Also while doing that I fixed the missing links in the London map data.
In the end I fixed the unit test too.
I have uploaded all the recent changes, so you can checkout yourself.
Apology for all the trouble.

Thanks for your contribution as always. @mohawk2

Good heavens, that was quick! I hope you're not staying up all night!

I do most Sunday night as I get the weekly challenge out on Monday morning.

The motivation for me to adjust the JSON is it will be handy for me to spot diffs with my nearly-ready updating to include the Elizabeth line. That's causing what looks like an infinite loop on loading the map, so I'm still getting something wrong.

Good luck, please do let me know if you find any issues.


File explorer: type to focus.  …

Some hackiness needs to be removed but otherwise works great.
Fixes: #43358

Imho this should not be implemented on the level of the explorer but rather be a feature of the tree/list widget so that it works everywhere.

You are totally correct. I wanted to do a proof of concept. Having this in tree widget would be very useful although vscode UI code can be very daunting and complex.
Some pointers on where to add the keydown listeners and access the tree model would be useful.

@bpasero Is totally right.
@nojvek Try the list widget first. This is tough because both the tree and list don't, and shouldn't, know anything about what is rendered inside each row. You're going to have to add API in which a list user can provide the actual filter string to look for, per element. Also, focusing is not really interesting. We can do much better: filtering would be awesome.
This would be a good place to start.

as @Tyriar pointed out #10026 is a very requested feature.
Just being able to select by typing is powerful. Mac, Windows, Ubuntu all do this in their explorers.  Pycharm / IntelliJ does this as well. You don't need to reach for the mouse to select files.
Please leave this PR open. I feel strongly about this feature and motivated enough to see it through in the next release.
I like @bpasero's suggestion. Have this be part of list widget. File explorer implements an API in its controller which gives the text to search and enables typeToSelect feature of list widget.
This means it wouldn't be available automatically in every list, but quite easy to add it.

Sure. I closed it because I understood this was just a proof of concept and you were convinced about doing it at the widget level.

Pretty old PR. I was super motivated at the time, but now after having a baby, priorities have changed. I don't think I'll have time to re-visit this.

@nojvek congrats! Good luck with it 😃

Thanks

@nojvek Congratulations 🎆!!! Sorry for not getting to this in time, we'll get there eventually.

I use the outline pane every single day. The filter on that is pretty amazing since it starts the search as soon as I type anything in outline. The search box doesn't need explicit focus. As long as outline pane has focus, it works. It's a great UX.  I have a keyboard shortcut to focus outline pane.
I really hope that filter box is done at a tree-view component level. That would make things really really easy for any other treeview to have filter functionality.

So .... As an radical VSCode fan, it pains me that we still don't have Explorer search. But if the outline filter is implemented at tree-view level, I feel confident I can get explorer search working in a couple of hours. Do let me know.
I  ❤ VSCode, mostly because its open source and the team's focus on UX and perf. It's probably the only Microsoft product I can think of that has this religious focus. Keep up the great work.


[SERVER] - adding server building files

Objective
Files needed to build the smartpeak-server image.

Hi Ahmed. Do you know what is failing on the CI?

Hi Douglas, the latest OpenMS/Qt5::Sql wasn't applied.


fix fullLike (again)

I tried to use an ee.Image created by ee.Image.geetools.fullLike (with the latest modifications) inside an ee.ImageCollection and got an error complaining about "not being an homogeneous" collection. This PR solves that issue.

merging as the test error is not related to this PR