BREAKING CHANGE: xHyper-V renamed to HyperVDsc: Renamed all DSCResources, Examples, Modules and Tests where applicable

Pull Request (PR) description
BREAKING CHANGE: xHyper-V renamed to HyperVDsc: Renamed all DSCResources, Examples, Modules and Tests where applicable
This Pull Request (PR) fixes the following issues

Fixes #62

Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[x] Resource documentation added/updated in README.md.
[x] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[x] Localization strings added/updated in all localization files as appropriate.
[x] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Community Testing Guidelines.
[x] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ ] New/changed code adheres to DSC Community Style Guidelines.

This change is

@johlju while I've got your attention, idk if you saw this!

Will look at it when I have some time. 😊

@nickgw just a few tiny comments. 🙂

@nickgw just a few tiny comments. 🙂
I'm not 100% on your first comment. I pulled azure-pipelines.yml from another repo that was updated more recently. I don't know if something changed concerning wuauserv in W2022 that changes that behavior.

I've updated the other two comments though.

Okey then I think you can safely remove the task in azure-pipelines.yml in this repo as it was not used prior. Guessing that was needed in the other repo to run a certain test.

Okey then I think you can safely remove the task in azure-pipelines.yml in this repo as it was not used prior. Guessing that was needed in the other repo to run a certain test.

Removed!

I updated pipeline files and added resource README files to enable Wiki documentation.
@nickgw can you rebase this when you have a chance?

@nickgw suggest you take azure-pipelines.yml from branch master and paste in into this PR. It will clear everything up I think.

You currently have the code coverage steps twice in the one in this PR. 🤔  So above suggestion might just be the easiest to fix it.. 🙂

@nickgw awesome work on this, thank you!  I waiting for the API key then I merge this.


python312Packages.google-cloud-trace: 1.13.4 -> 1.13.5

Changelog: https://github.com/googleapis/google-cloud-python/blob/google-cloud-trace-v1.13.5/packages/google-cloud-trace/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

4 packages built:

python311Packages.google-cloud-trace
python311Packages.google-cloud-trace.dist
python312Packages.google-cloud-trace
python312Packages.google-cloud-trace.dist


Allow defining store mixins via a spec syntax similar to Component mixins.

Fixes #32.
I found that I was defining similar functionality on a number of stores, especially if they listened to the same data.
Similar to React components, this will merge actions, chain functions, and attempt to merge objects/arrays. It will not, however, merge the result of functions (like createMergedResultFunction) as we have no such use case.
All the mixin functions are assigned directly to createStore so they can be overridden by users if necessary; for instance, I could see users wanting to define a different merge policy, such as deep merge.

This is awesome, mixins are a must for stores! +1!

@STRML I just merged this against current master in my fork and I had to use component.bindActions instead of merge here: https://github.com/BinaryMuse/fluxxor/pull/37/files#diff-32584de7a758f3cebf1bd66000a8b7caR47
I think .bindActions should do the job of merging or am I missing something? @STRML you've put .merge here for a reason, right?

@evindor .bindActions may indeed work, in this case I thought it made more sense to simply merge the object and let Fluxxor call bindActions when initializing the store. I am not sure why it is not working for you; which version of Fluxxor did you merge this to?

@STRML merged to current master.

@STRML Seems like Fluxxor just never calls .bindActions anywhere else. Anyway it works for me now. Hope we'll get it merged soon!

@evindor I've rebased to latest master and it passes tests again.
When I wrote this PR, around version 1.4, createStore didn't call .bindActions but instead just merged into the __actions__ object. Around 1.5 it changed the behavior. This PR now uses the new behavior so it should work just fine.

Just wanted to pop in for a quick update here.
There's no doubt that this functionality is quite useful from a reusable abstractions standpoint, but I've always been hesitant to merge it in due to the complexity that mixins bring (similar to the complexity they brought to React). As you probably know, there's been a big push toward native ES6 classes from the React team, and using native classes in React 0.13 precludes the use of mixins (as provided by React.createClass), with the goal of utilizing more idiomatic JS patterns to implement that functionality in the future (or using composition instead).
Furthermore, a future version of Fluxxor is likely to provide a more flexible method of store creation, which will include less "magic," and I'm not sure how this particular feature will fit in to that effort. So, for now, I'm going to leave this unmerged, but rest assured that making reusable store patterns easier to implement is definitely on my mind going forward.


task has failed: more than one PersistentVolumeClaim is bound

Expected Behavior
the task to execute from pipelineRun as it executes from taskRun
Actual Behavior
task backup-prometheus-snapshot has failed: more than one PersistentVolumeClaim is bound
pod for taskrun prometheus-snapshot-run-8cdnx-backup-prometheus-snapshot-fpcgx not available yet
Tasks Completed: 2 (Failed: 1, Cancelled 0), Skipped: 0
Steps to Reproduce the Problem
apiVersion: tekton.dev/v1beta1
kind: PipelineRun
metadata:
  creationTimestamp: null
  generateName: prometheus-snapshot-run-
spec:
  serviceAccountNames:
    - taskName: new-prometheus-snapshot
      serviceAccountName: prometheus-backup
  podTemplate:
    nodeSelector:
      kubernetes.io/hostname:
  params:
  - name: oc-image
    value: quay.io/openshift/origin-cli:4.5.0
  - name: oc-prometheus
    value: prometheus-k8s-1
  pipelineRef:
    name: prometheus-snapshot
  workspaces:
  - name: prometheus-pvc
    persistentVolumeClaim:
      claimName: "prometheus-k8s-db-$(params.oc-prometheus)"
  - name: backup-pvc
    persistentVolumeClaim:
      claimName: bkp
status: {}
---
apiVersion: tekton.dev/v1beta1
kind: TaskRun
metadata:
  creationTimestamp: null
  generateName: oc-snapshot-bkp-run-
spec:
  podTemplate:
    nodeSelector:
      kubernetes.io/hostname:
  params:
  - name: oc-image
    value: quay.io/openshift/origin-cli:4.5.0
  resources: {}
  serviceAccountName: ""
  taskRef:
    name: oc-snapshot-bkp
  workspaces:
  - name: prometheus
    persistentVolumeClaim:
      claimName: prometheus-k8s-db-prometheus-k8s-1
    subPath: prometheus-db/snapshots
  - name: backup
    persistentVolumeClaim:
      claimName: bkp
status:
  podName: ""

Additional Info

Kubernetes version: v1.18.3

Tekton Pipeline version:
Client version: 0.12.1
Pipeline version: v0.17.2
Triggers version: unknown

  workspaces:
  - name: prometheus
    persistentVolumeClaim:
      claimName: prometheus-k8s-db-prometheus-k8s-1
    subPath: prometheus-db/snapshots
  - name: backup
    persistentVolumeClaim:
      claimName: bkp

Could these two workspaces use the same PersistentVolumeClaim? I see one of them is using a subPath, could the other use another subPath of the same PVC? It does not work well to use two PVCs in the same Task, if you only use a single datacenter, it should work if you disable the affinit assistant but if you use a cloud provider this most likely will not help since the two PVCs may live within different Availability Zones in the cluster - this may also be the case in spanned clusters on-prem. In general, try to only use a single PVC for each task.

The requirement for the Task itself is to save the snapshot to some other remote location, and the subPath is used to simplify my logic in the Task that is execute. And yes currently the setup is with different Availability Zones.
I've tested the same logic on
Kubernetes Version: v1.18.3+2fbd7c7
Tekton Pipeline version: v0.11.3
and initially it was failing almost the same.
With exception of log translating TaskSpec to Pod: serviceaccounts "pipeline" not found.
To try and counteract that I've added spec.serviceAccountName: default inside PipelineRun yaml file.
After I did that the failing task run successfully and the pipeline executed as well.
don't know if this was the original error but different log output.
As explained this might go against best practices but I don't see other option currently regarding PVC usage.

And yes currently the setup is with different Availability Zones.

I don't see other option currently regarding PVC usage.

I think your use case is interesting, but also challenging. In this case, you need to disable the affinity assistant, and you also need to make sure that your PVCs are in the same Availability Zone, how this can be done depends on your storage system. Is there a way to set Zones for your PVs ? Or at least, enforce so that all Pods that use those PVCs are always scheduled to the same zone?
The VolumeBindingMode  for your StorageClass may also affect if this is possible or how to do this. Especially the field waitForFirstConsumer.
For some storage systems and volume binding mode, the volume is first scheduled to a Zone, then the Pod follow to that zone - as how I have understood it - at least with WaitForFirstConsumer: Immediate. This is a difficult field that I had to learn the hard way :) In your case, you need to find a configuration so that the two volumes are in the same AZ - otherwise it can not be mounted by a Pod at the same time. Or if you have any storageClass that is available in all your AZs?

@jlpettersson I want to thank you for pointing out affinity assistant after disabling it on the initial cluster where I started testing everything worked. The backup storage that I use is zone independent and does not pose any problem as it allows RWX access mode. I'm closing this as it's not really a bug but lack of understanding on my side.


Fix Ansible Galaxy versions

Need to switch to semver versions in requirements.yml due to the Ansible Galaxy upgrade.

Fixed by #47


background execution on iOS

Hi @dudipsh, i don't know if this is a bug but i don't get the plugin work in background for iOS. It works fine on Android (even in a background service), but on iOS keep stucked after the  "start" method is called without any kind of response in the console, do you know if something like this is posible on iOS?

@MCorzo
Hi
I don't remember if i test it on ios in the background
If you want you can create a demo in https://play.nativescript.org
and I will check it out.
In the meantime
If you have not deleted the ios platform try to remove/prepare/run

ths remove ios
tns prepare ios
tns run ios

hello @dudipsh, I did not get my example to work in ns playground so create a private repository with the example and send you an invitation I hope it can help


WIP - Adding support for chubby75 hardware.

This is a WIP pull request to add support to the HDMI2USB-litex-firmware for the RV901T LED "Receiver Card". The board is a very cheap FPGA board which has;

Spartan 6 FPGA (LX16, 14k 'logic cells', 9112 LUTs)
SDRAM
SPI Flash
2 x Gigabit Ethernet
Lots of IO drivers

See more info below and at q3k's chubby75 repository.

RV901T LED "Receiver Card"

This repository contains bits and pieces about the Linsn RV901T HUB75 LED driver card, also known as a "Receiver Card". Its stock function is to receive and forward framebuffer data using a proprietary protocol (from a "Sender Card") and blit out control signals to LED panels (via shields, like a HUB75 shield).
As it contains a user-reprogrammable Spartan 6 FPGA (LX16, 14k 'logic cells', 9112 LUTs) and 2x GbE, it has potential to be usable as a general purpose FPGA development board, an interface card for various purposes, or a logic analyzer.
Chubby75 is a project to reverse engineer, document and provide tools based on this card.

It also works on the real device:  ```        __   _ __      _  __
/ /  () /___ | |//
/ /__/ / __/ -)>  <
///_/_//|_|
SoC BIOS / CPU: LM32 /  75MHz
(c) Copyright 2012-2018 Enjoy-Digital
(c) Copyright 2007-2018 M-Labs Limited
Built Jun  5 2019 09:52:57
BIOS CRC passed (8e3a570a)
Initializing SDRAM...
SDRAM now under hardware control
Memtest OK
Booting from serial...
Press Q or ESC to abort boot completely.
sL5DdSMmkekro
Timeout
No boot medium found
BIOS>
Next I'd like to make the ethernet work. The problem is that I don't know how...

@danielkucera Let's start by merging what you have working so far. I'll review it shortly.

@mithro can you help a bit please? I cannot make the board load bios from SPI flash. I put it to 0x80000. You can check the last commit to see what I tried. On misoc it was working for me like this: https://github.com/danielkucera/flipsyfat/blob/sdstream/flipsyfat/targets/qm_xc6slx16_sdram.py#L162

I've enable the internal rom back to check if it can read from SPI correctly and it seems ok:
BIOS> mr 0x0 100
Memory dump:
0x00000000  98 00 00 00 d0 00 00 00 78 01 00 00 38 21 00 00  ........x...8!..
0x00000010  d0 e1 00 00 e0 00 00 3b 34 00 00 00 34 00 00 00  .......;4...4...
0x00000020  e0 00 00 00 34 00 00 00 34 00 00 00 34 00 00 00  ....4...4...4...
0x00000030  34 00 00 00 34 00 00 00 34 00 00 00 34 00 00 00  4...4...4...4...
0x00000040  e0 00 00 00 34 00 00 00 34 00 00 00 34 00 00 00  ....4...4...4...
0x00000050  34 00 00 00 34 00 00 00 34 00 00 00 34 00 00 00  4...4...4...4...
0x00000060  e0 00 00 00                                      ....

How do I say it to boot from there?


add buffer updating

currently there's no way to update buffers other than making new ones entirely. need dynamic/transient buffers for convenience and any kind of throwaway object (gui stuff, simple geometry, debug, sw skinning...)

transient buffers are almost complete as of 980bfb0a

update: transient buffers actually already work, my test was broken.
so now this issue is just about dynamic buffers

lowered priority to normal as the most common cases for this are handled by transient buffers


ValueError: GpuDnnConv images and kernel must have the same stack size

Hello,
I'm facing this error, I also followed your steps, any ideas ?
ValueError: GpuDnnConv images and kernel must have the same stack size

Apply node that caused the error: GpuDnnConv{algo='small', inplace=True}(GpuContiguous.0, GpuContiguous.0, GpuAllocEmpty.0, GpuDnnConvDesc{border_mode='valid', subsample=(1, 1), conv_mode='conv', precision='float32'}.0, Constant{1.0}, Constant{0.0})
Toposort index: 688
Inputs types: [CudaNdarrayType(float32, 4D), CudaNdarrayType(float32, 4D), CudaNdarrayType(float32, 4D), <theano.gof.type.CDataType object at 0x7f0094d97c50>, Scalar(float32), Scalar(float32)]
Inputs shapes: [(6, 480, 182, 34), (128, 64, 3, 3), (6, 128, 180, 32), 'No shapes', (), ()]
Inputs strides: [(2970240, 6188, 34, 1), (576, 9, 3, 1), (737280, 5760, 32, 1), 'No strides', (), ()]
Inputs values: ['not shown', 'not shown', 'not shown', <PyCObject object at 0x7f0081578c88>, 1.0, 0.0]
Inputs name: ('image', 'kernel', 'output', 'descriptor', 'alpha', 'beta')

Outputs clients: [[GpuElemwise{add,no_inplace}(GpuDnnConv{algo='small', inplace=True}.0, GpuReshape{4}.0)]]

HINT: Re-running with most Theano optimization disabled could give you a back-trace of when this node was created. This can be done with by setting the Theano flag 'optimizer=fast_compile'. If that does not work, Theano optimizations can be disabled with 'optimizer=None'.
HINT: Use the Theano flag 'exception_verbosity=high' for a debugprint and storage map footprint of this apply node.

Hey @abduallahadel  after googling this error, these seems answers seem to come :
https://github.com/fchollet/keras/issues/2678
and
https://github.com/fchollet/keras/issues/4697
Perhaps it has something to do with the drivers? or the Keras / theano dependencies.

Ok, I think it's CUDA problems with Theano, so I converted the code to work with Tensorflow as backend, it was an easy change.
The input image in Tensorflow should be ( w,h,depth)
In Theano : (depth,w,h)
So, I changed the input shape in the architecture and in the data generator


Do you have instructions for using this class?

Every time you want to add new functionality, the development process becomes difficult when you encounter interaction with this class in the diagram.It annoyed me.Can you give me a rough description of the process?

public class Feed : LifetimeTrackable {
public let lifetimeToken = NSObject()
fileprivate let backing = BackedStream<A>()
private let refreshTrigger : (BackedStream<A>) -> Void

public var output : OEXStream<A> {
    return backing
}

public init(refreshTrigger : @escaping (BackedStream<A>) -> Void) {
    self.refreshTrigger = refreshTrigger
}

public func refresh() {
    self.refreshTrigger(backing)
}

public func map<B>(f : @escaping (A) -> B) -> Feed<B> {
    let backing = BackedStream<A>()
    let result = Feed<B> { stream in
        self.refreshTrigger(backing)
        stream.backWithStream(backing.map(f))
    }
    return result
}

}

@Ben21hao It's an eyesore, aye! My understanding of your ask here is that you're asking for a working understanding of this code. Have I got that right?
The core concept here is Streams (OEXStream is read-only, Sync is writeable, Feed is a backed stream). All data in the app (okay, most) uses Streams. If you want to understand the core concept of Streams better, I recommend wrapping your head around these two stellar examples of streams in popular iOS libraries:

Streams in ReactiveCocoa: https://github.com/ReactiveCocoa/ReactiveSwift/blob/master/Documentation/Example.OnlineSearch.m
Streams in AFNetworking: https://github.com/deanWombourne/AFNetworking-streaming

Feed in this Open edX iOS codebase seems to mean: give me a stream that lets me pull-to-refresh or otherwise trigger a refresh. This is handy for user interfaces where you want to pull to refresh.
This specific implementation of Streams seems to have been rolled out fully-custom, relying on core concepts around streams and networking, but only adopted in this manner in this codebase, so documentation is scant.
Have you joined the mobile-apps slack channel? We'd love to see you there. You can get an invite to that chat room here: http://openedx-slack-invite.herokuapp.com/ and you'll find us in #mobile-apps.
I hope you contribute your feature upstream somewhere and comment here so we can check it out. Best of luck, Ben.


Reduce array allocations by usage MemoryStream.GetBuffer() instead of ToArray()

Check this document

Note that the buffer contains allocated bytes which might be unused. For example, if the string "test" is written into the MemoryStream object, the length of the buffer returned from GetBuffer is 256, not 4, with 252 bytes unused. To obtain only the data in the buffer, use the ToArray method; however, ToArray creates a copy of the data in
memory.

I think in this PR the author is reading just the amount of Length so having "test" in the buffer and backing array being of size 256 is OK as only the elements 0-buffer.Length are being written?

@lechu445 Please check if you can fix the following unit test

_/testcases/main/POIFS/FileSystem/TestNPOIFSFileSystem.cs#L724
System.UnauthorizedAccessException : MemoryStream's internal buffer cannot be accessed.

LGTM


Issues with multi-processing in Python

Now get this error when trying to download files:
AttributeError: module '__main__' has no attribute '__spec__'

A fix is mentioned here https://stackoverflow.com/questions/45720153/python-multiprocessing-error-attributeerror-module-main-has-no-attribute, i.e. doing something like this:
if __name__ == '__main__':
    __spec__ = None

but this doesn't work for me:
ImportError: No module named __main__

Resolved by installing using:
python -m pip install .

as python3 setup.py install is deprecated.


animationend and transitionend events do not fire in IE11

Description:
animationend and transitionend events do not fire in IE11
Versions affected:
0.9.11
Platforms affected:
IE11 / Windows
Reproduction:
You can view the issue on this playground link
Attaching the events manually works ok (playground link)

Apparently IE doesn't have HTMLElement.ontransitionend and friends, so ractive was deciding that those events aren't available. I've added a whitelist that seems to have resolved the issue, but since it's hard to test, I'm going to wait for verification before calling this resolved. If the fix is good, I'll pull it in to the changeset for 0.9.12.

If this becomes one of those inconsistently supported events, this might be a good candidate for a custom event. For instance, the custom tap event plugin is actually made out of focus and one of 3 other events (pointer, touch, or mouse).

Weirdly, it's fully supported by IE11 (and even 9 an 10 for the transitions, I think). It's just another IE-ism that they don't have an ontransitionend property on the elements, which is how Ractive determines whether or not the event is a DOM event.

This worked, thanks @evs-chris once again!


recover & print log

hi @fr05t1k
I use your package in our project but I have two question
first: When use  Sentry instead of Gokit log nothing to print. We need prints in log in development
second: What is the best practice for recover, can you send a sample code for recover

Hi @masihtehrani

We use multi logger in our projects and put Sentry Logger as one of the others. In this case you can tune your sentry logger as you want and combine it with an additional logger for stdout.

Here an example:

type MultiLogger struct {
	loggers []Logger
}

func (m *MultiLogger) Log(keyvals ...interface{}) error {
	for _, logger := range m.loggers {
		logger.Log(keyvals...)
	}

	return nil
}

func NewMultiLoggers(loggers []Logger) *MultiLogger {
	return &MultiLogger{loggers: loggers}
}

You can filter log messages and process only Error level for sentry logger.
var loggers []logutil.Logger
client, err := sentry.NewClient(
	sentry.ClientOptions{
		Dsn:         config.Sentry.Dsn,
	},
)
if err != nil {
	return nil, err
}

logger := sentrykit.NewSentryLogger(client)
loggers = append(loggers, level.NewFilter(logger, level.AllowError(), level.SquelchNoLevel(true)))

return log.NewSyncLogger(logutil.NewMultiLoggers(loggers)), nil

If you are talking about recovering from http server we use built-in middleware for net/http package

Example:
import (
	sentryhttp "github.com/getsentry/sentry-go/http"
	"github.com/gorilla/mux"
	"net/http"
)
r := mux.NewRouter()
http.Handle("/", sentryhttp.New(sentryhttp.Options{}).Handle(r))

Thank you

Hi @masihtehrani

We use multi logger in our projects and put Sentry Logger as one of the others. In this case you can tune your sentry logger as you want and combine it with an additional logger for stdout.

Here an example:
type MultiLogger struct {
	loggers []Logger
}

func (m *MultiLogger) Log(keyvals ...interface{}) error {
	for _, logger := range m.loggers {
		logger.Log(keyvals...)
	}

	return nil
}

func NewMultiLoggers(loggers []Logger) *MultiLogger {
	return &MultiLogger{loggers: loggers}
}

You can filter log messages and process only Error level for sentry logger.
var loggers []logutil.Logger
client, err := sentry.NewClient(
	sentry.ClientOptions{
		Dsn:         config.Sentry.Dsn,
	},
)
if err != nil {
	return nil, err
}

logger := sentrykit.NewSentryLogger(client)
loggers = append(loggers, level.NewFilter(logger, level.AllowError(), level.SquelchNoLevel(true)))

return log.NewSyncLogger(logutil.NewMultiLoggers(loggers)), nil

If you are talking about recovering from http server we use built-in middleware for net/http package

Example:
import (
	sentryhttp "github.com/getsentry/sentry-go/http"
	"github.com/gorilla/mux"
	"net/http"
)
r := mux.NewRouter()
http.Handle("/", sentryhttp.New(sentryhttp.Options{}).Handle(r))

thank you
I added Http recovery in our project but I don't have an idea about GRPC do you have experience with GRPC recovery?

@masihtehrani I don't have such experience. Anyway, you can recover manualy with built-it sentry recovery function. https://docs.sentry.io/platforms/go/panics/


GERSA II from BISTRITA NASAUD - typo

ENVIROMENT: ui.orase
URL: https://peviitor-ro.github.io/ui.orase/
Browser: Chrome
Device: pc
OS: Windows 10
STEPS TO REPRODUCE:
Open URL in browser
Type "Gersa II" in the search bar
EXPECTED RESULTS:
Gersa II, Bistrita Nasaud is spelled correctly.
ACTUAL RESULTS:
Gersa II has a typo - i instead of I

Fixed. Poti retesta

Retested. Pass


“your first time using LXD” is confusing

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

jenkins: test this please

This needs a "make i18n" and subsequent commit to update the translation templates. I can do it and then merge this.


Use @mock.patch decorator instead of calling it directly

Closes #17

Coverage remained the same at 83.2% when pulling 47f8ea87e83d6c28e4f6b1d1e584e601cb38b9c8 on tests into 50d2e9c69c005e405d7aa1e0e22e03739af7ae13 on master.


fix: handle uppercase property for normalize repeat

Coverage increased (+0.0002%) to 99.485% when pulling 046085981dfc3eab85eb9051ceb412d6c39a7a30 on evilebottnawi:ffix-handle-uppercase-repeat-style into fbb1385fa8eb88ff8865b4be106769596c1afb65 on ben-eb:master.

Close in favor https://github.com/ben-eb/cssnano/pull/500


Party Cookie - The party cookie should expire the day after the party date

The party cookie should expire the day after the party date. Ignore the party time.
Currently the party id expires after 30 days,

https://razoyo.atlassian.net/browse/TST-306


Exported time/date incorrect in CSV

Describe the bug
The dates in the exported CSV are incorrect. The dates are correct on the dashboard. But in the exported CSV, the date is always 07.06.1975.
To Reproduce
Steps to reproduce the behavior:

Export data
Open CSV and check "createdAt" column

Expected behavior
Exported CSV to display correct timestamp with date and time.
Environment (please complete the following information):

OS: Ubuntu 22.04
Architecture: amd64
Browser Firefox
Version v124.0.2

I couldn't reproduce this one, while it doesn't take into account the timezone maybe it has to do with your formatting if you're opening it in Excel?

Hi,
Actually you’re correct. My apologies.
If I open the file in Excel it has the correct date/time. And also if I open the file in a text editor the date/time is correct. I should have checked that before posting.
I generally use OnlyOffice and opening the CSV in the spreadsheets in OnlyOffice is where the date/time is wrong. Must be an issue on their side.
Thanks,
D

Also just checked with Google Sheets and it's formatted correctly there too.
Strange as never had problems with OnlyOffice before.

Thanks for confirming man. I'll close this ticket.


Sign In/Up UI flow

https://www.figma.com/file/MxA95ZHK5NqGX4ptFQljRJ/⭐-Design-System-3.0?type=design&node-id=5168-40915&mode=design&t=Vv6scKjpOL5E3wTh-11

duplicate
https://github.com/digma-ai/digma-ui/issues/748


Can't run Tensorflow-Lite in NDK

System information

OS Platform and Distribution: Linux Ubuntu 16.04
Mobile device: Samsung Galaxy S10
TensorFlow installed from (source or binary): binary
TensorFlow version: 2
Python version: 3.5.2
Bazel version (if compiling from source): 3.0.0
GCC/Compiler version (if compiling from source): 5.4.0

Describe the problem
I can build TFLite from binaries but I think it's not building correctly, or maybe only partially.
I can include the AAR / SO files into Android Studio, but I can't seem to use TFLite successfully in the NDK.
This should be a very basic task - I just want to run a simple inference in Android's NDK.
What am I missing here?
I tried to follow the basic examples from https://www.tensorflow.org/ or this repository's c++ examples, but the TFLite library does not seems to respond.
More information available at the StackOverflow post:
https://stackoverflow.com/questions/61523713/how-to-run-tensorflow-lite-in-android-studio-ndk

After failing to build the library as a shared / static one, I also tried to build the files straight from the IDE - see more info in my post.
Building fails with memory issues from toolchain / compiler.
It's possible that build system is missing some configurations - but the closest to successful build for arm64-v8a is done in Tensorflow's Docker container (from here) but the container uses a script, and not a configurable build system.
How can the TFLite library used in Android Studio's NDK environment - if not via static library?

Hi<EMAIL_ADDRESS>If I understand this correctly, you want some ndk code that links to the aar file?
From what I understand this is a bit tricky:  https://stackoverflow.com/questions/51930836/android-ndk-build-cmake-include-so-library-included-in-aar-library.
This may be a solution for you. If not, I suggest exploding the aar manually and adding a dependency on the shared object in your CMakeLists.txt (shared_library) or Android.mk (using PREBUILT_SHARED_LIBRARY).
I would, however, suggest that you will probably not get much performance benefit from this: the overhead of jni is probably not your bottleneck. I would take a look at possible optimizations such as quantization or pruning to reduce the model parameters.

Hi<EMAIL_ADDRESS>If I understand this correctly, you want some ndk code that links to the aar file?
From what I understand this is a bit tricky: https://stackoverflow.com/questions/51930836/android-ndk-build-cmake-include-so-library-included-in-aar-library.
This may be a solution for you. If not, I suggest exploding the aar manually and adding a dependency on the shared object in your CMakeLists.txt (shared_library) or Android.mk (using PREBUILT_SHARED_LIBRARY).
I would, however, suggest that you will probably not get much performance benefit from this: the overhead of jni is probably not your bottleneck. I would take a look at possible optimizations such as quantization or pruning to reduce the model parameters.

Hey @daverim,
Could you please elaborate on optimizations?
Do you think that NDK code won't reduce running times substantially?
The link you sent is all about extracting an .so from an .aar. That's not the issue for me :)
I already have both a shared library (.so) file and a static library (.a) file for the library I wish to include, which I'm adding in my Android Studio project's CMakeLists.txt file.
The problem is - when I include the library file, I get errors like so:
ninja: error: '/full/path/to/the/library/i/want/to/include/libtoinclude.so', needed by '.../app/build/intermediates/cmake/debug/obj/arm64-v8a/libnative-lib.so', missing and no known rule to make it
(when libnative-lib.so being the c++ library compiled from the NDK code).
Any ideas / suggestions?

Do you think that NDK code won't reduce running times substantially?

That's right, inference times between Java and C++ should generally be comparable, as long as you avoid using multi-dimensional arrays on the Java side for things like image inputs/outputs. The Java API is just a lightweight wrapper around the C++ API.

Do you think that NDK code won't reduce running times substantially?

That's right, inference times between Java and C++ should generally be comparable, as long as you avoid using multi-dimensional arrays on the Java side for things like image inputs/outputs. The Java API is just a lightweight wrapper around the C++ API.

I actually do have an image I'm trying to give as an input to the inference.
The output is also an image.
Currently the input image is loaded as a bitmap, and converted to an image-buffer before the inference.
How can I avoid using multi-dimensional arrays on Android's Java side?
What other options are there for image processing?

You can use a FloatBuffer or ByteBuffer of the appropriate size to provide your input data and extract output data. Most (maybe all?) of our examples use this approach. You can also use some of the helper utilities from the support library to do this, e.g., with our image classification example.

You can use a FloatBuffer or ByteBuffer of the appropriate size to provide your input data and extract output data. Most (maybe all?) of our examples use this approach. You can also use some of the helper utilities from the support library to do this, e.g., with our image classification example.

I didn't mention it's a high definition image, my bad.
It results in memory issues, and it's not related to this issue, but yes - I actually am using ByteBuffer, thanks.

@jdduke maybe this would better explain the issue:
I created an Android project from scratch and tried to build the C / C++ source files.
steps to recreate:

created an Android Studio Native C++ project
created basic C/C++ source files and headers directory structure in app/src/main/cpp and put the (A) tensorflow, (B) absl and (C) flatbuffers files there, so now it looks like this:

|-- CMakeLists.txt
|-- absl
|   |-- base
|   |   |-- attributes.h
|   |   |-- config.h
|   |   |-- internal
|   |   |   |-- identity.h
|   |   |   |-- inline_variable.h
|   |   |   `-- invoke.h
|   |   |-- macros.h
|   |   |-- optimization.h
|   |   |-- options.h
|   |   |-- policy_checks.h
|   |   `-- port.h
|   |-- memory
|   |   `-- memory.h
|   |-- meta
|   |   `-- type_traits.h
|   |-- types
|   |   |-- bad_optional_access.h
|   |   |-- internal
|   |   |   `-- optional.h
|   |   `-- optional.h
|   `-- utility
|       `-- utility.h
|-- flatbuffers
|   |-- base.h
|   |-- flatbuffers.h
|   `-- stl_emulation.h
|-- native-lib.cpp
|-- native-lib.h
`-- tensorflow
    |-- core
    |   `-- public
    |       `-- version.h
    `-- lite
        |-- allocation.h
        |-- c
        |   |-- c_api.cc
        |   |-- c_api.h
        |   |-- c_api_internal.h
        |   `-- common.h
        |-- core
        |   |-- api
        |   |   |-- error_reporter.h
        |   |   |-- op_resolver.h
        |   |   `-- profiler.h
        |   |-- macros.h
        |   `-- subgraph.h
        |-- delegates
        |   `-- nnapi
        |       `-- nnapi_delegate.h
        |-- error_reporter.h
        |-- experimental
        |   `-- resource
        |       `-- resource_base.h
        |-- external_cpu_backend_context.h
        |-- interpreter.h
        |-- interpreter_builder.h
        |-- kernels
        |   `-- register.h
        |-- memory_planner.h
        |-- model.h
        |-- model_builder.h
        |-- mutable_op_resolver.h
        |-- nnapi
        |   |-- NeuralNetworksTypes.h
        |   `-- nnapi_implementation.h
        |-- schema
        |   `-- schema_generated.h
        |-- stderr_reporter.h
        |-- string_type.h
        |-- type_to_tflitetype.h
        |-- util.h
        `-- version.h

with native-lib (native-lib.cpp / native-lib.h) being the compiled library and CMakeLists.txt being the cmake configuration file - which is left unchanged since it's "the only library".
3. All / most of tensorflow's header files include some #include "tensorflow/... lines, which couldn't be found by the compiler. So I changed all #include "tensorflow/... lines to either (A) relative paths or to (B) #include "/tensorflow/... paths (note the addition of a slash character, before "tensorflow/...").
4. In the app's build.gradle I added a no-compression line for the .tflite file:
aaptOptions {
        noCompress "tflite"
    }

and I control the build target via the ndk abiFilters option.
5. I added an assets directory to the app
6. In native-lib.cpp I add some example code from the TFLite website to use:
    char filename [] = "../assets/my_model.tflite";

    // Load the model
    std::unique_ptr<tflite::FlatBufferModel> model =
            tflite::FlatBufferModel::BuildFromFile(filename);

    // Build the interpreter
    tflite::ops::builtin::BuiltinOpResolver resolver;
    std::unique_ptr<tflite::Interpreter> interpreter;
    tflite::InterpreterBuilder(*model, resolver)(&interpreter);

    // Resize input tensors, if desired.
    interpreter->AllocateTensors();

    float* input = interpreter->typed_input_tensor<float>(0);
    // Fill `input`.

    interpreter->Invoke();

    float* output = interpreter->typed_output_tensor<float>(0);

Tried to build the library, source files included

errors

Build target is x86_64:
/path/to/Android/Sdk/ndk/20.0.5594570/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/include/c++/v1/memory:2339: **error**: undefined reference to 'tflite::impl::Interpreter::~Interpreter()'
memory, line 2339 is the "delete __ptr;" line:

_LIBCPP_INLINE_VISIBILITY void operator()(_Tp* __ptr) const _NOEXCEPT {
    static_assert(sizeof(_Tp) > 0,
                  "default_delete can not delete incomplete type");
    static_assert(!is_void<_Tp>::value,
                  "default_delete can not delete incomplete type");
    delete __ptr;
  }

Build target is arm64-v8a:
many errors of the following structure:

...
native-lib.cpp:32: undefined reference to `tflite::DefaultErrorReporter()'
native-lib.cpp:32: undefined reference to `tflite::FlatBufferModel::BuildFromFile(char const*, tflite::ErrorReporter*)'
...
native-lib.cpp:40: undefined reference to `tflite::impl::Interpreter::AllocateTensors()'
native-lib.cpp:45: undefined reference to `tflite::impl::Interpreter::Invoke()'
...

and the last one, again, being:
memory:2339: undefined reference to tflite::impl::Interpreter::~Interpreter()'`
additional info
The build line:
/path/to/app/build/intermediates/cmake/debug/obj/arm64-v8a/libnative-lib.so
: && /path/to/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android29 --gcc-toolchain=/path/to/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/path/to/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot -fPIC -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security   -O0 -fno-limit-debug-info  -Wl,--exclude-libs,libgcc.a -Wl,--exclude-libs,libgcc_real.a -Wl,--exclude-libs,libatomic.a -static-libstdc++ -Wl,--build-id -Wl,--fatal-warnings -Wl,--no-undefined -Qunused-arguments -shared -Wl,-soname,libnative-lib.so -o /path/to/app/build/intermediates/cmake/debug/obj/arm64-v8a/libnative-lib.so CMakeFiles/native-lib.dir/native-lib.cpp.o  /path/to/Android/Sdk/ndk/21.1.6352462/toolchains/llvm/prebuilt/linux-x86_64/sysroot/usr/lib/aarch64-linux-android/29/liblog.so -latomic -lm

See anything that might cause this clang "memory" issue?

with native-lib (native-lib.cpp / native-lib.h) being the compiled library

Can you talk more about this? What exactly do you mean that this is the compiled library? Which library exactly? And how was it generated?
Note that we've had some internal discussions over the past week, and are planning to prioritize adding native CMake support in Q3 for TFLite. This should simplify native integration into non-bazel toolchains.

with native-lib (native-lib.cpp / native-lib.h) being the compiled library

Can you talk more about this? What exactly do you mean that this is the compiled library? Which library exactly? And how was it generated?
Note that we've had some internal discussions over the past week, and are planning to prioritize adding native CMake support in Q3 for TFLite. This should simplify native integration into non-bazel toolchains.

These are great news :)
Non-bazel toolchains integration would be excellent.
Also - I couldn't get Bazel to build TFL for arm64-v8a either, I was only able to build a static (.a) library for x86_64.
Regarding your question:
native-lib.cpp and native-lib.h are the C++ source file and header file, respectively, that I wrote to execute my program. They're compiled by Android Studio's build system (gradle) via the NDK extension (using cmake), and referred as native-lib. This is standard in Android Studio, and configured by default when creating a Native Project in Android Studio.
So what I was trying to do is to get gradle (Android Studio's build system) to build two libraries: (1) native-lib (my code) and (2) tensorflow-lite - and have my code in native-lib call the tensorflow-lite functions.
In the last trial depicted above, the native-lib.cpp did not have any special functionality - I was only trying to call the Tensorflow-Lite functions, to see that I can use TFL in native code.

Just for reference:

What I'm trying to do is to run a Tensorflow-Lite inference, from Android Studio's NDK.
The reason I was trying to build the tensorflow-lite library myself, was because it failed to be compiled from a static / shared library (for arm64-v8a)
Before trying to build and compile the code with gradle, I also tried using Android Studio's Bazel plugin to build the TFL library, with no apparent success.
Using a shared library is preferred, but I couldn't build an .so for arm64-v8a, or failed to have it successfully compiled / linked into the project.

@jdduke I tried running TFL in NDK using the C-API via these instructions and version 2.2 seems to work ok :)
I assume I have you to thank for - so thanks!
I still can't see any result from the inference but I just assume it's my bad somehow... regarding this issue specifically, I think that TFL is currently running in NDK, so I'll close it.
I still want to run the C++ API (this issue) and also check out GPU configuration capabilities of TFL, but I'll just open another issue for that.

@jdduke I tried running TFL in NDK using the C-API via these instructions and version 2.2 seems to work ok :)

Great news! Thanks for the feedback.

I still want to run the C++ API (this issue) and also check out GPU configuration capabilities of TFL, but I'll just open another issue for that.

Sounds good. Our CMake work this quarter should hopefully make it much easier to build from source directly from within Android Studio.


Revise Examples so that they follow the style guide.

Increasing Access
Consistency across examples and documentation increases cognitive accessibility.
Most appropriate sub-area of p5.js?
Examples
Feature request details
Examples should all follow the p5 style guide.

@nickmcintyre Will start working on this issue soon

Sounds good @zelf0! Feel free to suggest revisions to the style guide as you work.

@nickmcintyre Hey, I'm getting around to working on this, a lot of the examples need punctuation, grammar, and clarity edits to fit the style guide. But the other thing that's against the style guide is using // for multiline comments. I started changing it to /, but I don't see an explanation in the style guide for why / is better than //, so I'm wondering if I should just keep the //s and suggest a revision to the style guide because I personally feel that // is more common anyways and just as good. It's very possible there's a reason the style guide suggests /* over //, I just don't know what it is. (Also if that's the case, I would suggest for the reason to be indicated in the style guide like some of the other guidelines have)

@zelf0 nice! Thanks for bringing this up.
I believe most major programming languages have a dedicated syntax for multiline comments because it's tedious and error-prone to remember // for things like function descriptions. Modern IDEs make writing documentation with /* pretty nice.
All that said, let's discuss some specific examples. I think it'd be fine to use // for comments that span 2 or 3 lines. I'm curious how often comments that are more than ~3 lines appear. My sense is that the example code should be clear enough that big multiline comments (more than 3 or 4 lines) generally aren't needed.

Just to note there is a bit of nuance when it comes to multi line comments, mostly in terms of convention as JS don't really care that much about comments.
This style is usually quite common especially for shorter text, the reason being this is usually what text editor do by default when commenting lines with Cmd/Ctrl + /.
 Adding and removing them is usually not as easy as the above.
/*
Some comment that
go onto the next line
*/

This style is doing the same thing as above with JS only caring about /* */ pair of symbols, however the beginning /** is used by many documentation engine to differentiate between documentation (which starts with /**) and regular old multi line comments not meant to be documentation (starts with /* only). The additional decoration at the beginning of each line (*) are usually optional and just makes things neater.
/**
Some comment that
go onto the next line
*/

A small side effect with the third documentation comment style with decoration at the beginning of each line is that it is a bit more difficult to copy multiple lines of text without also copying the decorations (unless your text editor support a particular style of text highlighting):
/**
 * Some comment that
 * go onto the next line
 */

Right, so the style guide seems to be preferring that last way, with the * in the front of each line, which does seem a little tedious, but if that's the preferred way, there are a bunch of multiline comments, mostly using //, so I can change them all over to the /** if that's preferred.

@limzykenneth thanks for helping us to better understand the common patterns for comments. The various forms of /* have their place, but perhaps not in example code.
@zelf0 thanks again for bringing this up, and for helping us to keep it simple. I think typing * on each line isn't much of an improvement over //. It also introduces a possible syntax barrier for beginners. What do you think about opening an issue to change that style guideline with a rationale along the lines of "Using // for all comments increases consistency between examples."? Or something like that.
I found a few examples with comments that span several lines, such as edge detection. In this case, there are 42 lines of comments and 34 lines of code. What do y'all think about the length of the comments and the ratio of comments:code?

I think it's ok to have a high comments:code ratio if it's necessary. If I find examples where comments are too wordy, I'll simplify them. And it's possible some examples require more comments because they are overly complicated, in which case we can create a new issue to replace the example with something more straightforward.
And I'll open an issue to change the guideline.

Working in this branch: https://github.com/zelf0/p5.js-website/tree/edit-examples
In the Structure folder, there's an example called "Functions" But it seems to just be about the drawTarget() function. Should it be replaced with an example on functions in general, or should it be re-titled to "Draw Target"

Also, what do you think about using destructured assignment to condense consecutive variable declarations? For example, I came across this line in the examples:
let [total, child, young, adult, senior, elder] = [577, 103, 69,
122, 170, 113
];
And I'm wondering if it should be changed to:
let total = 577;
let child = 103;
let young = 69;
let adult = 122;
let senior = 170;
let elder = 113;

@zelf0 thanks for pointing this out. I think the current functions example could be improved in a couple of ways. It'd be helpful to demonstrate different patterns for parameters (none, a few, and defaults). It's also easier to focus on functions without loops or other other control flow structures present.
I also agree about avoiding destructuring–your change looks good!

@RohitPaul0007 here's some of the conversation regarding comments.
What do you think about editing a subset of the examples to bring them inline with the style guide? The examples in the Hello p5 section could be a good place to start.

Well if it is one line comment one should use "// ".  But if it is multi line comment, where some one is trying to explain the code one  should use "/* */ ". In this way it becomes more readable code base.  @nickmcintyre

@RohitPaul0007 you're welcome to suggest a change by opening an issue. We recently came to a different conclusion as you can see from the discussion above. The style we use in our documentation breaks with some common patterns when it's helpful to beginners.


Bottom bar overflow on mobile browsers

Works well on iPhone 11. But not on iPhone 8. Need to add something for device screen ratio.

Perhaps we should consider switching to a drawer for desktop as it would also match mobile

And we can focus on 4-6 icons instead of trying to fit 9

Easy fix will be to scale the icon sizes down.

Perhaps we should consider switching to a drawer for desktop as it would also match mobile

Probably can try that out

Scaling them down makes them harder to press

Remove up and down arrows for mobile

Related https://github.com/ShabadOS/desktop/issues/321

Do we want to show the Info button?


feat: upgrade esbuild to 0.17.6

closes https://github.com/esbuild-kit/core-utils/pull/37

@privatenumber

:tada: This PR is included in version 3.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Upgrade tenv to version 3.2.1

Hi!
The new tenv version 3.2.1 add improved error messages.
https://github.com/tofuutils/tenv/releases/tag/v3.2.11
I don't see any PR by renovate proposing an upgrade...
Is it possible to upgrade it manually?

Hi Seb,
The PR creation from renovate was rate limited, I juste manually triggered it: #444
I'll merge it once the CI passes
Merry Christmas :)


combine_estimations is taking too long to publish (3-12ms)

Probably due to waiting for new pose estimations published in /tf from aruco_board for each chaser.

Closed by 7ce74ef


make dice device enrollment tool compiling on linux

update both generator py and the makefile
fix is to ensure stdc++ is part of the link process. Other than that, use variable instead of repeating compile time args
silence make clean to ensure make clean && make is going to work.

Fixes https://github.com/DevKitExamples/DevKitDPS/issues/12


Add escrow error output to logging

When inspecting logs from Crypt, sometimes admins may see:
ERROR: Key escrow unsuccessful
This is a helpful message, but it only explains half of the story. It would be helpful to log the reason why the key escrow was unsuccessful. This PR adds the captured error value to the Crypt log(s). The goal of merging this PR would be to provide Crypt administrators broader knowledge about when and why key escrows could fail.

Yes indeed I do:
2023-04-14 12:11:38 AM - ERROR: Key escrow unsuccessful. Reason: curl: (28) Failed to connect to cryptserver port 443 after 151232 ms: Couldn't connect to server

hey @grahamgilbert! any thoughts on this one? thank you for your review!


Enhance sidecar-shim README

What this PR does
Over the time a few enhancements got implemented
for the sidecar-shim feature.
Amend its README.md for consistency.
Add a generated example of a VM
with hook sidecar ConfigMap.
Why we need it and why it was done in this way
Special notes for your reviewer
PR #10637 and #11272 introduced a few changes that are still not fully reflected
in cmd/sidecars/README.md , updating it for consistency.
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[X] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[X] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/lgtm
Thanks!

/test pull-kubevirt-e2e-k8s-1.30-sig-compute-migrations


Generic to/fromValue() method to move between zk and JS types

For O1 people: we discussed this internally @garwalsh
Draft PR with complete implementation: https://github.com/o1-labs/o1js/pull/1271

completed!


Indent a few docs/install.md examples.

To line up with the bullet points the examples live under.
/assign @suhanime

/lgtm


Improvement/9490 packages

Closes toggl/backend#9490
The main effort was made in this branch is to make pipes-api at least maintainable and readable to others.
I refactored all the packages structure, introduced layers separation, added testify and mocks, added a lot of unit tests and make it possible to understand what is this project all about. In other words, I got rid of Big Ball Of Mud in this project.
Integrations were tested on stage which successfully works:

[x] Asana
[x] GitHub
[x] Toggl Plan
[ ] Basecamp 2 (Didn't test because we have lost basecamp 2 account)
[ ] Freshbooks Classic  (Didn't test because we have lost classic account)

@nidarasheed Is your Basecamp 2 account still operational? Could Backend/Anton login to your account, change the email address and then use it to test out the integration?


Extend Provisionner to support Network Automation

The current version don't cover any use cases related to network devices.

For some particular reason .gitignore didn't ignore .vagrant.....

@tima can you assign to me

@IPvSean I'll assign to you though this ticket is tied to @victorock's PR in addition to yours and my related ones.


Revert "Do not use daemon for Jupyter servers (#9032)"

This reverts commit 867d2edd1e81e3d96fb15a307b8598e67ba800cf.
Fixes #9511
See original thread here https://github.com/microsoft/vscode-jupyter/pull/9032#issuecomment-1040668737
Basically @rchiodo  brought this up, and a client confirmed it worked without daemon, however we still have others that can't get it working , hence reverting this change.
I just wish we knew the root cause (as daemons were added later). Anyways, cannot break users.

Do you think we need to port this to release?

Do you think we need to port this to release?

this isn't in the release branch.
I deliberately merged this PR after release was cut to ensure we first test this in insiders. looks like that paid off


Add order preview model

Erst orderModel reviewen
https://github.com/mittwald/api-client-js/pull/192


Fix the brew build setup

hugo version outputs:
hugo v0.112.5+extended darwin/arm64 BuildDate=unknown

Downloaded from GitHub:
hugo v0.112.5-48e33e10af2feb2a0e1c32d04880ed016619f2d8+extended darwin/arm64 BuildDate=2023-05-29T07:27:36Z VendorInfo=gohugoio

Missing:

BuildDate
Commit hash
VendorInfo (brew would be a good value)

The issue was on the Homebrew side and does not need any codes change in Hugo.
I've merged a PR on Homebrew and it already looks better starting with v0.115.4:
$ hugo version
hugo v0.115.4-dc9524521270f81d1c038ebbb200f0cfa3427cc5+extended darwin/amd64 BuildDate=2023-07-20T06:49:57Z VendorInfo=brew

Thank you.


install dependencies on localhost

Currently the playbooks (e.g. remote_fp) assume that they can and should install certain dependencies on localhost. Since most people run ansible from their local machine (e.g. their desktop, laptop or macbook), this means that the current playbooks try to modify the machines that they run on and not only the remote machines they should deploy.
https://github.com/intel/container-experience-kits/blob/master/playbooks/infra/remote_fp.yml#L17-L23
https://github.com/intel/container-experience-kits/blob/master/roles/bootstrap/ansible_host/tasks/main.yml
In my opinion this leads to a couple of issues:

privilege escalation on localhost (so on my local machine) is something that i never want to grant to an ansible playbook downloaded from the internet (due to very basic security considerations) and imho it should never be needed
my local environment might look very different from what the playbook expects (e.g. using venv or having my python dependencies in a different path)
modifications on my local system should be left to me, or to be a bit more generic here: any modifications needed to run ansible playbooks should be mentioned in the docs, there might be a script or even an playbook to help, but this should not be forced by default

In conclusion i think the whole part of installing dependencies should be moved to documentation or at least made optional and be disabled by default.

agree...will review and update accordingly

Thanks again @jklare for your submission.  Our next release BMRA 22.01 will address this and contributor license will also be added to encourage further contributions.  However, we're not planning to patch the current 21.09 for this purpose.

see https://github.com/intel/container-experience-kits/releases/tag/v22.01


Improve error message from parse_dockerimage_string

Throw a ValueError rather than a bare Exception when a bad
character is found

Emit the problem-causing characters as part of the error message

thank you for the contribution @0xDEC0DE - nice improvement!


PERF: speed up from_ragged_array with single-pass cython implementation (Polygon and MultiPolygon)

This is a follow-up on the original PR adding from_ragged_array (https://github.com/shapely/shapely/pull/1559). As mentioned there, I started with a pure python implementation reusing existing building blocks, but for better performance we could later add a custom implementation in cython or C.
The main benefit is that it can then be done in a single pass, instead of (in case of MultiPolygons) first creating all LinearRings (as python objects), and from that all Polygons, and only then all MultiPolygons (with requiring to clone the geometries at each step).
For from_ragged_array, I opted to do this in cython, because we already have similar functionality in cython right now (when passing indices to the shapely.linestrings, shapely.polygons etc functions, then we don't use the C ufunc but the custom implementation in cython).
For now this PR only adds a custom implementation for (Multi)Polygon.
On the MultiPolygons of the nz-building-outlines datasets (3.3 million polygons, small polygons with on average only 8 coordinate pairs), this gives a speedup of around 4.5s to 1.5s (with constructing the same geometries from WKB taking around 2.5s).

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

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

Totals

Change from base Build<PHONE_NUMBER>0:
0.3%

Covered Lines:
2612

Relevant Lines:
2967

💛  - Coveralls

@sgillies @brendan-ward thanks for the reviews!


Error

Check the sql syntax

i can not fix it. i tried adding DataError event

u dont join relationships between tables in database

No

can  u help me fix my problem ?

I don't face this type of error, i have to go through the code, I'll help

are you done ?

Can u fix it ?
On Fri, 4 Oct 2024 at 09:06, Yogesh @.***> wrote:

I don't face this type of error, i have to go through the code, I'll help
—
Reply to this email directly, view it on GitHub
https://github.com/y0geshx/Collage-Management-System/issues/3#issuecomment-2392650868,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3S4UNPLXTS2QOK6YTXOE5LZZXZ2ZAVCNFSM6AAAAABPI6SVMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJSGY2TAOBWHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can u fix it ?


--self-upgrade is funky on Intel Macs

I've run into this before but it took me a while to realize that it only happens on my older Macs and not on the new one and I don't believe I've seen it on Linux.
~/local/bin/ubi --self-upgrade --debug
[ubi][DEBUG] munged args for self-upgrade = [["/Users/olaf/local/bin/ubi", "--project", "houseabsolute/ubi", "--in", "/Users/olaf/local/bin"]]
[ubi::ubi][DEBUG] Parsed --project houseabsolute/ubi = houseabsolute / ubi
[ubi::ubi][DEBUG] exe name = ubi
[ubi::ubi][DEBUG] matching =
[ubi::ubi][DEBUG] install path = /Users/olaf/local/bin/ubi
[reqwest::connect][DEBUG] starting new connection: https://api.github.com/
[ubi::ubi][DEBUG] current OS = macos
[ubi::ubi][DEBUG] matching assets against OS using (?i:(?:\b|_)(?:darwin|macos|osx)(?:\b|_))
[ubi::ubi][DEBUG] current CPU architecture = x86_64
[ubi::ubi][DEBUG] matching assets against CPU architecture using (?ix)
        (?:
            \b
            |
            _
        )
        (?:
            386 | i586 | i686
            |
            x86[_-]32
            |
            x86[_-]64
            |
            x64
            |
            amd64
            |
            linux64
            |
            # This is gross but the OS matcher will reject this on non-Windows
            # platforms.
            win64
        )
        (?:
            \b
            |
            _
        )

[ubi::ubi][DEBUG] matching against asset name = ubi-Darwin-aarch64.tar.gz
[ubi::ubi][DEBUG] matches our OS
[ubi::ubi][DEBUG] does not match our CPU architecture
[ubi::ubi][DEBUG] matching against asset name = ubi-Darwin-x86_64.tar.gz
[ubi::ubi][DEBUG] matches our OS
[ubi::ubi][DEBUG] matches our CPU architecture
[ubi::ubi][DEBUG] matching against asset name = ubi-FreeBSD-x86_64.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-aarch64-musl.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-arm-musl.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-i686-musl.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-mips.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-mips64.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-mips64el.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-mipsel.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-powerpc-gnu.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-powerpc64-gnu.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-powerpc64le.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-riscv64gc-gnu.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-s390x-gnu.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Linux-x86_64-musl.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-NetBSD-x86_64.tar.gz
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Windows-aarch64.zip
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Windows-i686.zip
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] matching against asset name = ubi-Windows-x86_64.zip
[ubi::ubi][DEBUG] does not match our OS
[ubi::ubi][DEBUG] only found one candidate asset
[ubi::ubi][DEBUG] picked asset from matches named ubi-Darwin-x86_64.tar.gz
[ubi::ubi][DEBUG] picked asset named ubi-Darwin-x86_64.tar.gz
[ubi::ubi][DEBUG] downloading asset from https://api.github.com/repos/houseabsolute/ubi/releases/assets/122258412
[reqwest::connect][DEBUG] starting new connection: https://api.github.com/
[reqwest::async_impl::client][DEBUG] redirecting 'https://api.github.com/repos/houseabsolute/ubi/releases/assets/122258412' to 'https://objects.githubusercontent.com/github-production-release-asset-2e65be/327105217/5cb33681-cc6d-4952-a317-12aa72b88bfd?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20230904%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230904T163304Z&X-Amz-Expires=300&X-Amz-Signature=755ee9a45830252bd20afbf5d49752fe1bc8996bfb2d3878f8a781dcdddcd6a1&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=327105217&response-content-disposition=attachment%3B%20filename%3Dubi-Darwin-x86_64.tar.gz&response-content-type=application%2Foctet-stream'
[reqwest::connect][DEBUG] starting new connection: https://objects.githubusercontent.com/
[ubi::ubi][DEBUG] archive path is /var/folders/fx/nhlkqy9s1gj0x98s1k3gxcgr0000gn/T/.tmpmEipfp/ubi-Darwin-x86_64.tar.gz
[ubi::ubi][DEBUG] extracting binary from tarball at /var/folders/fx/nhlkqy9s1gj0x98s1k3gxcgr0000gn/T/.tmpmEipfp/ubi-Darwin-x86_64.tar.gz
[ubi::ubi][DEBUG] found tarball entry with path GNUSparseFile.0/ubi
[ubi::ubi][DEBUG] extracting tarball entry to /Users/olaf/local/bin/ubi
[ubi::ubi][INFO] Installed binary into /Users/olaf/local/bin/ubi

ubi --version
/Users/olaf/local/bin/ubi: line 1: 2: command not found
/Users/olaf/local/bin/ubi: line 2: 0: command not found
/Users/olaf/local/bin/ubi: line 3: 5484544: command not found
/Users/olaf/local/bin/ubi: line 4: 5488640: command not found
/Users/olaf/local/bin/ubi: line 5: 2090208: command not found
/Users/olaf/local/bin/ubi: line 6: syntax error near unexpected token `('
/Users/olaf/local/bin/ubi: line 6: `��h
H__PAGEZERO�__TEXT@N@__text__TEXT�__⎽├┤␉⎽__TEXT__⎽├┤␉_␤␊┌⎻␊⎼__TEXT���__±␌␌_␊│␌␊⎻├_├▒␉__TEXT���__␌⎺┼⎽├__TEXT@�@^@�__␌⎽├⎼␋┼±__TEXT6M┘6M__┤┼┬␋┼␍_␋┼°⎺__TEXT�M,�M__␊␤_°⎼▒└␊__TEXT?N?N�_DATA_CONST@N@N__±⎺├__DATA_CONST@N@@N__␌⎺┼⎽├__DATA_CONST@@N(␤@@N│__DATASS@__┌▒_⎽≤└␉⎺┌_⎻├⎼__DATAS│S__␍▒├▒__DATA�84�__├␤⎼␊▒␍_┴▒⎼⎽__DATAS�S__├␤⎼␊▒␍_␍▒├▒__DATASH__LINKEDIT@TT�"0TP)P)T�TH89TX$Wk�]�
                                                                       P``�jjH�f
                                                                                /┤⎽⎼/┌␋␉/␍≤┌␍4▒␊Z3#┴┐Hѥ2
4*(�
    @'

Any idea what that file is? Can you run file on it to find out?

Also, can you manually download the release ubi downloaded and see if it works? Maybe the release itself is just broken for some reason.

file ~/local/bin/ubi
/Users/olaf/local/bin/ubi: data

If I just download the release, I get:
file ~/local/bin/ubi
/Users/olaf/local/bin/ubi: Mach-O 64-bit executable x86_64

in="$HOME/local/bin"
mkdir -p "$in"

if [ ! "$(command -v "$in/ubi")" ]; then
    curl --silent --location \
        https://raw.githubusercontent.com/houseabsolute/ubi/master/bootstrap/bootstrap-ubi.sh |
        TARGET=$in sh

fi

$ ls -al ~/local/bin/ubi
-rwxr-xr-x@ 1 olaf  staff  7578848 Aug 19 20:30 /Users/olaf/local/bin/ubi
$ ~/local/bin/ubi --self-upgrade
-rwxr-xr-x@ 1 olaf  staff  7575264 Aug 19 20:30 /Users/olaf/local/bin/ubi

No clue if this helps, but more ~/local/bin/ubi provides different results with working vs not.
Broken install:

Working install:

Ok, this looks like a known bug in the tar crate - https://github.com/alexcrichton/tar-rs/issues/295 and https://github.com/alexcrichton/tar-rs/issues/322
There's a fork of the tar crate I could use, but I'd really like to see this fixed in the original.

This should be fixed in v0.0.28.

Looks great. Thanks!


likecoin.bigdipper.live - unable to redelegate

Describe the bug
after clicking the redelegate button on the validator page, it shows "We're sorry - something's gone wrong. Please try to refresh the page and see if the problem is gone.". retried a few times over a few hours.
To Reproduce
Steps to reproduce the behavior:

login https://likecoin.bigdipper.live with ledger. check own wallet.

go:
https://likecoin.bigdipper.live/validator/2C021179E6F72C87943883CFB64F4E114A23E487
(or the validator which you delegated LikeCoin)

hit "redelegate"

We're sorry...

Expected behavior
choose validators to redelegate to
Screenshots

Desktop (please complete the following information):

Mac OS 10.15.6 on Mac mini
Chrome 85.0.4183.83
also Brave 1.13.82

Fixed in the latest #b715b48 commit in likechain branch.


Error thrown when using npx keycloakify initialize-email-theme and selecting KC-Version 24.0.8

Error: end of central directory record signature not found
    at /Users/user/kc-email-theme/node_modules/keycloakify/bin/33.index.js:1520:14
    at /Users/user/kc-email-theme/node_modules/keycloakify/bin/33.index.js:1964:5
    at /Users/user/kc-email-theme/node_modules/keycloakify/bin/33.index.js:1000:7
    at FSReqCallback.wrapper [as oncomplete] (node:fs:671:5)

After cloning the keycloakify starter and installing all dependencies.
I get hit with the error from above when selecting KC-Version 24.0.8.
Choosing KC-Version 25.x / 26.x / 23.x works fine, and the folder src/email is created.
Tested on keycloakify versions 11.3.6 & 10.0.0-rc.128

Hello @lucaa6i,
Thanks for reporting, it's fixed in 11.3.7.
I recomend however to initialize you email theme with the latest available version regardless of the Keycloak version you are curently targetting.


Leverage/configure LB with Proxy-Protocol Support

I'm looking for a way to use Kalm to deploy and manage pods but where the HTTP-based container applications will have access to X-Forwarded-For, X-Originating-IP, X-Remote-IP, and/or X-Remote-Addr.

Do you mean you want to access these header values(or equivalent) in your pod? Will add some docs about this. New docs will be released soon.

For clarity, I have an application running in a pod that needs to access the client's IP address. The pod is managed by Kalm and accessed through a route with HTTPS-only access. In order for the load balancer to add the client's IP address information to the HTTPS request, the load balancer must be configured to proxy TCP connections and to support proxy protocol. Additionally, Istio must be configured to handle TCP connections with proxy protocol support enabled.


chore: release 0.4.0

In GitLab by @jon-nfc on Jun 5, 2024, 21:06

In GitLab by @jon-nfc on Jun 5, 2024, 21:06
added 2m of time spent

In GitLab by @jon-nfc on Jun 5, 2024, 21:12
added 26 commits
23a06be3...cb09252b - 16 earlier commitsdd0c13a6 - fix(itam): ensure device type saves historyeb6ae13c - feat(core): support more history types735ac287 - test(settings): history view permission check for device typea4bc4b15 - test(settings): history view permission check for manufacturerff595b0c - test(settings): history view permission check for software category930e5aeb - test(settings): access permission check for app settingsd100c311 - test(api): placeholder test for inventoryc0173d6f - feat(api): move invneotry api endpoint to '/api/device/inventory'adeffff4 - feat(database): add mysql support4ee63473 - Merge branch '2024-06-05' into 'development'
Compare with previous version

In GitLab by @jon-nfc on Jun 5, 2024, 21:12
enabled an automatic merge when all merge checks for 4ee6347306dc5c67303e4e416c1be11ed4d9f519 pass

In GitLab by @jon-nfc on Jun 5, 2024, 21:13
added 2m of time spent

In GitLab by @jon-nfc on Jun 5, 2024, 21:24
mentioned in commit a79adb0bdd45a8b76e351cc57c11854afb7ec3d3

In GitLab by @jon-nfc on Jun 5, 2024, 21:58
mentioned in commit fa28fd436ef03b66c270ad460cadfd877434fd0d


Add sync.d.ts

Closes https://github.com/terkelg/tiny-glob/issues/53

Thank you

No problem! Do you think you might be able to cut a new release with the change?


CMake Update

Few patch to:

Ease the use of glog as subproject (i.e. adding alias glog::glog)
Better manage gflags dependencies
Rework VERSION management using project() provided variables
cf https://cmake.org/cmake/help/v3.0/command/project.html

I just remove the gflags::gflags / gflags management.
keep it on a dev branch on my fork when needed...

@googlebot rescan

Ok I moved the doc INSTALL.md and incorparate this patch with the glog::glog alias commit.
You can see the rendering there https://github.com/Mizux/glog/blob/master/cmake/INSTALL.md
Don't hesitate to comment / modify / whatever...

LGTM.


Clarification for Azure AD SSO URL Values For Cisco AnyConnect SAML

In the following link, you will find the "Configure Azure AD SSO" Which is used for configuring Cisco AnyConnect with Azure AD for MFA/SSO
https://docs.microsoft.com/en-us/azure/active-directory/saas-apps/cisco-anyconnect#configure-azure-ad-sso
In step 4 these could be more concise. When you are configuring this on the Azure UI they give you some example URL that you should be using.

The example should be given due to the examples on the Azure UI side, the examples are not clear and can lead to some errors as well.
Example 1
This one is good and straight to the point, a good example of how we can configure the Identifier (Entity ID)

Example 2
In this example we show the following URL which is not complete, the URL is missing ?tgname= followed by the TGTGroup name the was used in the Identifier URL shown above,  this URL is also case sensitive on the Cisco side, so using the following URL with https://YOUR_CISCO_ANYCONNECT_FQDN/+CSCOE+/SAML/SP/ACS?tgname=TGTGroup will result in a file not found error when trying to connect to over the VPN via AnyConnect.

All together this example should be https://YOUR_CISCO_ANYCONNECT_FQDN/+CSCOE+/saml/sp/acs?tgname=TGTGroup, we can leave the +CSCOE+ in because Cisco is expecting this on the AnyConnect side.

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

ID: d242e95c-09e5-91ae-6075-be736201df09
Version Independent ID: 95899322-7404-6e42-97f4-cf1726fade97
Content: Tutorial: Azure Active Directory single sign-on (SSO) integration with Cisco AnyConnect
Content Source: articles/active-directory/saas-apps/cisco-anyconnect.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@JustJordanT
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#please-close


Ensure that HTTP::WebSocket uses SNI, just like HTTP::Client.

Please see the manual spec for a use-case for this. Websockets over Cloudflare fail without SNI. I can modify the manual spec to whatever you guys would like.

@ysbaddaden A self-contained test would be great. Question is, how do we get a simple SNI server working to test this locally? I don't think HTTP::Server supports that.

Specs should pass when you're offline, for example on a long flight. If we are to spec this in this way, we should spawn the tls server locally.

I've open an issue for supporting SNI in the HTTP server. I'll get feedback on that (issue #5722), and redo this after the PR for that discussion has been submitted.
Thanks.

I think this is a quite valuable fix. So if we can't come to an agreement/plan how to spec this within two weeks I would vote to merge this with a stub/commented out/todo spec

Let's drop the spec and merge the fix. It was wonderful to have a test, but relying on external service availability is flaky.

there is already spec that uses external service: https://github.com/crystal-lang/crystal/blob/master/spec/manual/badssl_spec.cr

@kostya ugh, we should fix that too.

Thank you!


Azure AD profile cannot be imported

After importing exported Azure AD authentication profile, Azure VPN client states: Azure VPN client supports connecting to only Azure VPN Servers. Please validate your configuration.

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

ID: e9ef898a-0b72-e018-8a3d-6f14f62e1119
Version Independent ID: b62b5b8f-bc88-0868-a2f6-1020a5097460
Content: Configure a VPN client for P2S VPN connections: Azure AD authentication
Content Source: articles/vpn-gateway/openvpn-azure-ad-client.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@ondrejvaclavu , We are looking into this query and will update you as soon as possible.

@ondrejvaclavu , You need to upload the xml file which you have downloaded from the SAS URL .
If you look into the XML file, you can see the Server entry something like this: azuregateway-2ba9c18-36db0b4e3bfc.vpn.azure.com . When you import this XML, you should not get any error. Can you delete the config in your server and retry ?

Hi, the entry in exported XML file is like:
azuregateway-XXXX-someID-XXXXX.cloudapp.net
Not sure how to change to …vpn.azure.com

From: KARTHIK SRINIVAS<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 12:05:52 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ondřej Václavů<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure AD profile cannot be imported (#42410)
@ondrejvaclavuhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fondrejvaclavu&data=02|01||59c9726e58c048f85a5108d766971e53|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090671537358610&sdata=UMUNCwk8tRWewbyTUCShDy318tQs%2BZBoHw5uKbyj7Mo%3D&reserved=0 , You need to upload the xml file which you have downloaded from the SAS URL .
If you look into the XML file, you can see the Server entry something like this: azuregateway-2ba9c18-36db0b4e3bfc.vpn.azure.com . When you import this XML, you should not get any error. Can you delete the config in your server and retry ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F42410%3Femail_source%3Dnotifications%26email_token%3DAESI747AVRBLPIVE7UFF32TQTE4BBA5CNFSM4JKVZKR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDWO2VA%23issuecomment-552398164&data=02|01||59c9726e58c048f85a5108d766971e53|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090671537368615&sdata=iD%2BiIXTt2Z6%2FSvErSTM%2B5y3skBXjglcv8V%2FY88WAJII%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAESI74ZD6U5KQD5FD3IFTI3QTE4BBANCNFSM4JKVZKRQ&data=02|01||59c9726e58c048f85a5108d766971e53|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090671537388674&sdata=UCeHpUfMLYBHPqt5XlxdZd9sfiJrDpo0e5cA%2FU9m2VE%3D&reserved=0.

@ondrejvaclavu , Can you re-download the package and check the XML file?

Still …cloudapp.net

From: KARTHIK SRINIVAS<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 1:00:03 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ondřej Václavů<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure AD profile cannot be imported (#42410)
@ondrejvaclavuhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fondrejvaclavu&data=02|01||1f3dda9f6fde4c70efd208d7669eb00e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090704044063546&sdata=c64psJs%2FoR4PbwTGGMDhXSMg%2B96PKE1%2BMntqgUW%2Fd3k%3D&reserved=0 , Can you re-download the package and check the XML file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F42410%3Femail_source%3Dnotifications%26email_token%3DAESI745MR4ICD6MBFQNCO63QTFCMHA5CNFSM4JKVZKR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDWTFKY%23issuecomment-552415915&data=02|01||1f3dda9f6fde4c70efd208d7669eb00e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090704044063546&sdata=S4U2WXTTY8CYexYoD4ED4mD824rSLYXvyhA%2BwvB9BqQ%3D&reserved=0, or unsubscribehttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAESI745ZZSB7IMOT3OTOQSLQTFCMHANCNFSM4JKVZKRQ&data=02|01||1f3dda9f6fde4c70efd208d7669eb00e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090704044073554&sdata=pcYUVK%2BdpCjIdHPH2sJ6uYhm1p9mor1mbgUSlTW5KU0%3D&reserved=0.

Have you configured OpenVPn only in your VPN gateway?
What is your Gateway SKU?

Yes, tunnel type is OpenVPN (SSL) only
SKU is VpnGw1

From: KARTHIK SRINIVAS<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 1:10:08 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ondřej Václavů<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure AD profile cannot be imported (#42410)
Have you configured OpenVPn only in your VPN gateway?
What is your Gateway SKU?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F42410%3Femail_source%3Dnotifications%26email_token%3DAESI74Z4YDOSUEFQQ73S4SLQTFDSBA5CNFSM4JKVZKR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDWT72A%23issuecomment-552419304&data=02|01||1391af7e8e344440febd08d766a01878|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090710094580101&sdata=gMma0gWMhs7FtZEkOSC1LO%2BrIgZzTDpeZ4MrXR56WoI%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAESI747TNE4RSBBE4JMJNHTQTFDSBANCNFSM4JKVZKRQ&data=02|01||1391af7e8e344440febd08d766a01878|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090710094580101&sdata=Q4wkd8N78AohGy%2B7eTdRitZdiIhTWQSZLikIzWiBjY8%3D&reserved=0.

@ondrejvaclavu , Are you downloading the package via PowerShell command?
$profile = New-AzVpnClientConfiguration -ResourceGroupName AADAuth -Name AADauthGW -AuthenticationMethod "EapTls"
$PROFILE.VpnProfileSASUrl

Yes, exactly like this.

From: KARTHIK SRINIVAS<EMAIL_ADDRESS>Sent: Monday, November 11, 2019 1:15:46 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Ondřej Václavů<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Azure AD profile cannot be imported (#42410)
@ondrejvaclavuhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fondrejvaclavu&data=02|01||28e2e0f83df343e366cb08d766a0e229|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090713477869373&sdata=uleyYEjmqd3GWFVIqDpa3Vx7b6Kc%2FQjkGKkWdIHm854%3D&reserved=0 , Are you downloading the package via PowerShell command?
$profile = New-AzVpnClientConfiguration -ResourceGroupName AADAuth -Name AADauthGW -AuthenticationMethod "EapTls"
$PROFILE.VpnProfileSASUrl
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F42410%3Femail_source%3Dnotifications%26email_token%3DAESI744TJOEOS5PZWR5TQSDQTFEHFA5CNFSM4JKVZKR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDWUOLA%23issuecomment-552421164&data=02|01||28e2e0f83df343e366cb08d766a0e229|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090713477879382&sdata=KWUw%2B4TPYaMgA6VB7Njmombt%2FgnjbXJZpIC5Hk3yiQM%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAESI744JH6R6SBOHHIE64I3QTFEHFANCNFSM4JKVZKRQ&data=02|01||28e2e0f83df343e366cb08d766a0e229|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637090713477879382&sdata=36UN5869pFLUBTWIx527cwVawhJlLkJit0VFeIYEi8k%3D&reserved=0.

Can you drop an email to<EMAIL_ADDRESS>with your subscription ID? I will take a look and revert

Please upgrade to Client version 1.1042.24.0 from the Microsoft Store.

Please upgrade to Client version 1.1042.24.0 from the Microsoft Store.


Handle customizing

#3244

Customizing transport to create relevant files based on object name & object type

Change the customizing in one of the object and refresh the ABAGIT screen

Since AbapGit supports only the TADIR objects, it doesn't make sense integrate the support of customizing here.
As it requires several tweaks in the abapgit code, which may have some effects on the performance.
Also it make sense to create a new report to handle only the customizing content using the APIs, see https://docs.abapgit.org/development/api.html.


Sketch Dark Mode

Closes #546

Added Sketch Dark Mode Functionality

Light Mode

Dark Mode

AHH.. As far as the UI goes, it's dope
But logically speaking, if we introduce a dark mode one would have take extra efforts in order to get similar results to that of the light mode ( I mean the output of the sketches)
We can rather have an option to change the background to make this simple.
@anushbhatia @sanjay270899 what do you think?

But logically speaking, if we introduce a dark mode one would have take extra efforts in order to get similar results to that of the light mode ( I mean the output of the sketches)

Extra Efforts like?

change the background to make this simple.

Yeah, I will build this in future PR soon.
@smaranjitghose

I think you are confused with background. Currently the background is transparent. So, downloading image drawn in light and dark mode will be same. (And about the background, it is already in my todo.)

@smaranjitghose You got my point?

@smaranjitghose You got my point?

Yep! I do understand.
Try the following please:

Switch to dark mode for the sketch page
Set the pen color to Black
And draw atleast 7 sqaures one after the other (in progession)

I observe an expected change in background color to different shades of pink (magenta to be specific).
You can give it a try here for convinience

@smaranjitghose I didn't get you. Can you please provide me the screenshot?

Stage 1

Stage 2

Stage 3

Stage 4

Stage 5

Stage 6

@smaranjitghose How? 😅

@smaranjitghose Are you using any extension? How can the color change!
Background color is given with CSS. It has nothing to do with Canvas.

@smaranjitghose Maybe due to some extension in your browser it is coming like that. I have checked in Chrome as well as in Edge, also my friend has also tried this, But back ground color is not changing.
Background Color has been given by CSS. It has nothing to do with canvas.

@smaranjitghose Maybe due to some extension in your browser it is coming like that. I have checked in Chrome as well as in Edge, also my friend has also tried this, But back ground color is not changing.
Background Color has been given by CSS. It has nothing to do with canvas.

Ya, just checked my Firefox has an option to prevent canvas fingerprinting. It was weirdly interrupting with this.


LatestVersion.TSG-Buddy

IMPORTANT !!!

Do NOT delete this issue and keep its number as 8.
Keep this issue as Closed and Locked.

What is this for?
This is used to store & read the latest Test/Prod version of the TSG-Buddy.
Is the content confidential?
No. It will store just the latest version. Comments in this issue can be read anonymously.
How is this used?
A new comment is added by the TSG-Buddy's build pipelines whenever a new version of the app is released to Test/Prod.
The TSG-Buddy will check the latest comment (per Test/Prod) regularly of this issue.

Prod|1.0.9070.13841

Prod|1.0.9071.34588

Prod|1.0.9072.10482

Prod|1.0.9074.27401

Prod|1.0.9075.12059

Prod|1.0.9075.13524


Build: correct lookup for libIndexStore.dll on Windows

This is required for test discovery to work properly.  We would
previously incorrectly compute the location for IndexStore, and fail to
discover tests as a result.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test Linux platform

For platform bringup this seems fine.  What would be great in the future would be to parameterize all of these platform specifics.  I don't know if this is something that could come out of each platform's PlatformInfo.plist for example.  probably what we'll want to do after the bringup is a refactoring to do that kind of parameterization, but it might be too much overhead while things are changing quickly.

Agreed.  I know pretty confidently I don't know enough of the pieces that differ to be able to build up the platform settings.  But once we have nearly all the pieces in place, refactoring this becomes much more reasonable.


In Inerst mode use  mapping  not work

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Neovim version
NVIM v0.9.4
Branch info
main (Default/Latest)
Minimal (user) folder structure required to reproduce the issue
lua/user/keymap/core.lua

Minimal config with steps on how to reproduce the issue
This is my config('core.lua'):
return {
	["i|<C-BS>"] = map_cmd("<C-w>"):with_noremap():with_desc(""),
	["i|<C-H>"] = map_cmd("<C-w>"):with_noremap():with_desc(""),
}

Expected behavior
Equivalent to  in insert mode.
I can use nvim command mode and enter :imoremap <C-H> <C-w> and it will be successful.
Additional information
No response

Hi, it's because the imap of <C-h> is overrided by autoclose.nvim.

If you remap imap <C-H> <C-W> after this plugin loaded, it will works normally.

inoremap works for me too.

Suddenly I cam do it too, thank you


Clean up ArrayOfNumbersSerDe and test errors in the serde

Coverage increased (+0.04%) to 98.722% when pulling d975689ff40e3fb101ea121618fc1a98567a6741 on number_serde into ca23eb34df6ddf62b96193395ce4b4ccb8491421 on master.


docs: Improve documentation and comments

Comments
It would be nice to have a description explaining the responsibility of each declared class/module
https://github.com/mithi/bare-minimum-3d/tree/master/src

Documentation
Please check the wiki / documentation, to learn the arguments that you'll need to pass to BareMinimum3d's renderScene function.

SceneOptions
SceneSettings
ViewSettings

Hi . I would like to contribute for this.

Hi . I would like to contribute for this.

Sure! Thank you for your interest!
Don't forget to read the contributing guidelines and the recommended commit style guide! Thank you!
And let me know if you have any questions!

I couldnt push my changes to my branch. It says permission denied.
Can you help me with this? Sorry I am new to this

what did you try to do so far ?
On Fri, Oct 2, 2020 at 7:05 PM BertinoCanisius<EMAIL_ADDRESS>wrote:

I couldnt push my changes to my branch. It says permission denied.
Can you help me with this? Sorry I am new to this
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mithi/bare-minimum-3d/issues/19#issuecomment-702669974,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMX2FMVAPIOBOH63VAUKQTSIWXVZANCNFSM4QQYFT5Q
.

I couldnt push my changes to my branch. It says permission denied.
Can you help me with this? Sorry I am new to this

You can try following the steps here: https://github.com/mithi/hexapod/wiki/Simple-Git-Workflow-guide


wip: new(docs): add entry for 0.33.1 changelog

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind documentation

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

new(docs): add changelog for 0.33.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign andreagit97 for approval by writing /assign @andreagit97 in a comment. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

4c4e60d update(changelog): set release date for 0.33.1

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

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

LGTM label has been added.  Git tree hash: 9648bdce567ba8649507804ca7d83e4828ef2756

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

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


feat(coinbase): coinbase pay card shows only when adding funds

Description
A bug was found during 1.37 Bugapalooza where the Coinbase Pay card was visible in the withdraw flow.
This PR adds logic to limit Coinbase Pay card to the add funds flow.
https://user-images.githubusercontent.com/57046916/181386954-7757096c-d26a-4590-a42a-da830cd3f93d.mp4
Other changes
N/A
Tested
Screen recording above was taken on a mainnet build with Coinbase Pay feature flag turned on locally.
Added a unit test to check for coinbase pay card's visibility during both cash in and cash out flows.
How others should test
Perform the following steps with the coinbsePayEnabled feature flag turned on.
Steps:

Navigate to the Withdraw Funds flow.
Select CELO and an available amount.
Check if coinbase pay card is visible on Select Withdraw Method screen.

Expected behavior:
Coinbase pay card should not be visible on Select Withdraw Method screen
Related issues
N/A
Backwards compatibility
N/A

the "steps to test" don't align with the change that's made here-- there isn't anything in those steps about how to verify the fix to the select providers screen when withdrawing

Hey @MuckT we have verified the above task on latest iOS Test Flight build V: 1.38.0 and updated status in Test rails
Observation : Coinbase pay is not seen in withdraw CELO flow on select withdraw method page
Test Rail link:
iOS 14: https://valoraapp.testrail.io/index.php?/tests/view/57037
Device : iPhone 12(14.7.1)
CC: @awtsui
Thanks.!

Hey @MuckT, We have verified the above task on Android Internal Release Build V:1.38.0 and updated status in Test rails
Test Rail links:
Android 11: https://valoraapp.testrail.io/index.php?/tests/view/57051
Device : Google Pixel 2XL (11.0)
CC: @awtsui
Thanks.!!


Keyboard Input

I was having an issue with getting keyboard input on mobile using emscripten. I have a canvas that covers the entire viewport. While I'm rendering to the canvas using OpenGL ES the canvas no longer responds to JS event listeners, so I had no way of getting the user keyboard up using HTML5 APIs since it requires a user gesture for activation. Does this library fix this issue? When running my code using this library in the browser, can I activate the soft keyboard programmatically?

No, GLFM doesn't provide a way to show the soft keyboard programmatically on Emcripten (glfmSetKeyboardVisible works on native iOS and Android).

No, GLFM doesn't provide a way to show the soft keyboard programmatically on Emcripten (glfmSetKeyboardVisible works on native iOS and Android).

Ok thanks for clearing that up. I'll close this thread


Consider annotated getters in the generation of TS models

JPA and Jackson lets you set annotations on getters instead of fields.
JPA
For JPA that means if the getter has an annotation the field should not be used.
This can also be specified setting the AccessType with the Access annotation: https://javaee.github.io/javaee-spec/javadocs/javax/persistence/Access.html
In that case the defined AccessType should be used to access the state of the entity.
Jackson
Jackson has the JsonProperty, JsonGetter and JsonSetter annotations.
If these are used then also getter or setter should be used instead of the field.
Why do we want to use that?
There could be the case that we want to serialize a calculated field that is calculated in the getter of the DTO or Entity.
For example if we have a Order we want to calculate the total price of the order and serialize that to the client.

Added in PR #720


Add new improvements

Description
In this PR we updated the SDK version to 0.7.0 and modified functions and variables names in the different modules.
Fixes #59 and #60
Type of change

[X] Bug fix (non-breaking change which fixes an issue)

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my code
[X] I have commented on my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my change

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

42 of 42 (100.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
298

Relevant Lines:
298

💛  - Coveralls