RFE: lift root requirements for some bootc commands

Feel free to push back on this, but there are a number of rpm-ostree commands that can be run w/o sudo access. rpm-ostree status is what comes to mind. I believe we have policykit  (or something like that!) to enable certain commands from non-privileged users. It would be nice if we could do the same for bootc.
I think we want any user w/ shell access to be able to querry:
$ bootc status
ERROR Status: Querying root privilege: This command requires root privileges

Thoughts?

Yeah, this one is a bit harder as it heavily intersects with https://github.com/containers/bootc/issues/2 which is a big change.
An issue with rpm-ostree in general is I think not all use cases will want to necessarily make the system state accessible to unprivileged users - especially as we're likely to expand what data is there.
My offhand instinct here is the easiest thing would be:

Add a bootc-readonly-unprivileged.{path,service} unit that, when enabled, monitors for changes in e.g. /ostree (for which we bump the mtime on changes) and basically just does bootc status --json > /run/bootc/status.json that is world-readable
Teach bootc status when run unprivileged to just parse that JSON

Obviously whether it's enabled by default would be an important property.  I could get behind it being on by default upstream.

I think we want any user w/ shell access to be able to querry:
$ bootc status
ERROR Status: Querying root privilege: This command requires root privileges

Yep. This one trips me up every time because it's what I'm used to with rpm-ostree.

An issue with rpm-ostree in general is I think not all use cases will want to necessarily make the system state accessible to unprivileged users - especially as we're likely to expand what data is there.

I kind of liken it to rpm -qa. You can run that unprivileged. If there is data that's in there we don't want shown then can we just leave it out in the unprivileged path? If admins don't want unprivileged access to this data then maybe we could make that part configurable?

I kind of liken it to rpm -qa.

I agree that that being accessible unprivileged by default sets a big precedent, as does rpm-ostree having its status be accessible by default.
Another highly relevant example here is systemd allowing anything to introspect all system units by default. It looks like tools such as systemd-sysext run unprivileged too for querying state. And I guess other basic things like the Linux kernel shows all block devices (including USB etc.) by default.
However...let's take a contrasting case here which is podman and container images; today /var/lib/containers is a mode 0700 directory and while there is a remote API for podman it doesn't distinguish from read-write operations. So AFAIK there's no "list all system images" for podman available today to unprivileged users by default. flatpak has a dbus service for its storage.
Tying these together slightly, bootc now also has its own c/storage instance for LBIs and if you want complete status in the general case we'd probably want a mechanism to get access to those too.
I guess all of this kind of argues that podman and bootc are the odd ones out here and indeed we should enable this by default via something like the service above. I guess it's notable here at the implementation level that most of the things above are just reading files for status (with the notable exception of systemd units which are dbus). On the podman side arguably there should be a podman-ro.socket or so...I think libvirt did something like that.

I guess all of this kind of argues that podman and bootc are the odd ones out here and indeed we should enable this by default via something like the service above.

💯
Another example here of an unprivileged user not being able to see version information is OpenShift and it is super frustrating when trying to report issues as the version is one of the most important pieces of information to report.

https://issues.redhat.com/browse/OCPBUGS-8783
https://issues.redhat.com/browse/WRKLDS-1042

Great points! On the rpm-ostree side I always felt like "status" was up there w/ /etc/os-release or /etc/redhat-release in terms of checking what version I'm on. Do we consider this information:
 booted:     image:       image:         image: [image name]         transport: registry       version: [version]       timestamp: null       imageDigest: sha256:[digest]
"at the same level" as os-release? I tend to put it in that bucket, but I might be the only one.
This isn't meant to counter any previous point, just add a lens that may or may not be useful.
Cheers,


Add a signup and login feature

Feature Request: Add Signup/Login Functionality in E-commerce Application
Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to manually enter my personal details and payment information every time I make a purchase on our e-commerce platform. This repetitive process is time-consuming and can lead to errors. Additionally, the lack of a user account system makes it difficult to track orders, manage returns, and access past purchases.
Describe the solution you'd like
I would like to implement a signup and login functionality in our e-commerce application. This would allow users to create an account with their personal information, securely store their payment details, and log in to access their profiles. With a user account, customers could:

Save their shipping and billing addresses.
Store and manage payment methods.
View order history and track shipments.
Manage returns and refunds more efficiently.
Receive personalized recommendations and offers.

Additional context
Adding a signup/login functionality can enhance the overall user experience by making the checkout process quicker and more convenient. It also opens up opportunities for personalized marketing and customer engagement. Below is a mockup of the proposed login/signup page:

I want to work on this project . Please assign me this issue @Niharika0104 under ssoc24


Fix bug in documentation

Add the missing useLogging parameter

Thanks for contributing! If you don't want to go through the paperwork, I can just make this change for you.

I tried to fill in the paperwork, but I just get:
A server error occurred, please try your request again.
So I don't know if I can report that bug somewhere.  You can go ahead and
make the change yourself since it just doesn't seem like Google wants to
cooperate with me.  :P
On Tue, Oct 25, 2016 at 1:33 PM, Jonathan Amsterdam <
<EMAIL_ADDRESS>wrote:

Thanks for contributing! If you don't want to go through the paperwork, I
can just make this change for you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-go/pull/407#issuecomment-256104548,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABkYxyMaQVfgRt_QaF-obYbRgiYUfqSAks5q3j1YgaJpZM4KgKVA
.

Sorry about that!

Fix is in. Thanks again, closing this now.


merge previous pull requests and more

Hello @cboddy it seems like you don't have much time to merge pull requests,
I liked this script, so I merged and reviewed the pull requests #22 #23 #25 #26 #27 (keeping the history to show the contributors)
then I simplified the window name detection (the priority is written in readme),
added verbose startup
allowed workspace 0 renaming (I use it, even if not standard)
reviewed readme to match the previous
I would suggest you to release a new version to unlock the situation
Thanks for your time

Hey, thanks for this. I'll take a look in the next few days and get back to you

Thanks a lot for this @CastixGitHub @bastidest @rand0x0m @justyn @sersorrel .
These changes are available on pypi in release 0.15.0. If anything looks odd do shout/raise a PR.


Where is the drawer navigation?

Maybe I'm missing something obvious but on all the renders for this boilerplate framework there is a burger menu visible. However, I can't seem to find it anywhere. Pretty much everything is in place for navigation, but there is literally nothing on the front-end for navigation. Am I missing something here?

The hamburger menu you see is a part of the dev screens, which is currently only an option on the Andross boilerplate.

Side note, if you're looking for how to do a drawer navigation in a Bowser project, check out the documentation here: https://reactnavigation.org/docs/en/drawer-based-navigation.html


Potential Incorrect Charging Cost Calculation

Describe the bug
Hi, please see the attached photo below.

I am wondering if the calculated "charger cost" of 2.7c is an over-optimistic calculation?
The way I see it:

the solar cost is 2.25c (my feed in tariff), and the current grid energy tariff is 8.2c.
my current solar production is 8.3kW, of which the preferential use are consumption (1.7kW) and home battery charging (3.0kW).
I am also importing 3.6kW from the grid currently.
therefore, the car charging is really using 3.6kW from the grid and 3.6kW from the solar.
If you weight the charging cost appropriately, my charger cost should have been (3.6 * 2.25+3.6 * 8.2)/(3.6+3.6) = 5.2c  rather than the over-optimistic 2.7c shown in this screen.

I am guessing that the reason EVCC has produced 2.7c estimate is because it puts the car charging as the preferential user of the solar energy instead of my home battery, therefore of the 8.3kW solar production, 1.7kW went to the house, and 6.7kW went to the car. Therefore the car charging cost was calculated as (6.7 * 2.25+0.5 * 8.2)/(6.7+0.5) = 2.7c.
I wonder if this is an incorrect way of calculating the charging cost. If I wasn't charging the car, these solar excess would have gone to the battery first, there's no way I would have imported grid energy to charge the battery as this calculation is proposing.
Steps to reproduce
See above.
Configuration details
not relevant

Log details
not relevant

What type of operating system are you running?
Docker container
Version
v0.122.1

Just had a look with google translate (don't speak german) - it's a different calculation issue.
I thought it's the natural order that solar should be considered to be used household consumption > home battery > car battery; as the first two are what get the solar when the car is not plugged in.
As it stands, the car charger expense is presumed to have used up the solar excess first (hence very cheap) before the home battery which is not really logical.

I think in that issue there is a link to the docs. Here the englisch version:
https://docs.evcc.io/en/docs/guides/setup/#savings-calculation

evcc assumes that the house battery is primarily used to store excess, self-produced solar power. If the house battery also discharges to satisfy other loads, or charges from grid supply, this assumption isn't always correct.

Looking in your screenshot I see that the battery is loading from the grid.

By the way an additional comment here.
The battery is a hybrid battery where it has a maximum inverter of 5kW. Therefore, even though the solar production of 8.2kW, it can only convert 5 of it to AC and the rest is supplied to charge the battery.
Therefore there is this illusion of "i am charging the battery with grid energy" which is not really the case.

The battery is a hybrid battery where it has a maximum inverter of 5kW. Therefore, even though the solar production of 8.2kW, it can only convert 5 of it to AC and the rest is supplied to charge the battery.
Therefore there is this illusion of "i am charging the battery with grid energy" which is not really the case.

But this seems to be the issue here. I would have assumed that the charger would reduce power to not introduce grid import. Or did you set a min of 7.2kW?
\cc @premultiply

That bit is intentional, my solar excess is not sufficient so I chose to fast charge with the cheap day time grid energy.

Any more thought thought from @StefanSchoof or @andig  now that I have explained my rationale in more detail?

I do agree with @changyang1230  - for savings calculation car charging energy should be considered "last in line" to get solar energy, no matter what the reasons or settings are.

Depending on your configured log level you can see the individual values in stdout. Do build aggregates per charging session to calculate the price per charging session but these all all in memory. We only persist the result to db (sessions table in evcc.db) when charging is finished.


traefik white list did not work as expect

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
I create d whitle list:
spec:
  ipWhiteList:
    ipStrategy: {}
    sourceRange:
      - <IP_ADDRESS>/24

What did you see instead?
my local ip is: <IP_ADDRESS>   when I access the web site, it still blocked.
What version of Traefik are you using?
2.2.8
What is your environment & configuration?
apiVersion: apps/v1
kind: Deployment
metadata:
  name: traefik-2
  namespace: kube-system
  selfLink: /apis/apps/v1/namespaces/kube-system/deployments/traefik-2
  uid: 775978bb-df43-4463-9ec8-8cb3ad57fe34
  resourceVersion: '47727123'
  generation: 7
  creationTimestamp: '2020-08-26T14:16:55Z'
  labels:
    app.kubernetes.io/instance: traefik-2
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: traefik
    helm.sh/chart: traefik-9.1.0
  annotations:
    deployment.kubernetes.io/revision: '5'
    kubectl.kubernetes.io/last-applied-configuration: >
      {"apiVersion":"apps/v1","kind":"Deployment","metadata":{"annotations":{"deployment.kubernetes.io/revision":"1","meta.helm.sh/release-name":"traefik-2","meta.helm.sh/release-namespace":"kube-system"},"creationTimestamp":"2020-08-26T14:16:55Z","generation":4,"labels":{"app.kubernetes.io/instance":"traefik-2","app.kubernetes.io/managed-by":"Helm","app.kubernetes.io/name":"traefik","helm.sh/chart":"traefik-9.1.0"},"name":"traefik-2","namespace":"kube-system","selfLink":"/apis/apps/v1/namespaces/kube-system/deployments/traefik-2","uid":"775978bb-df43-4463-9ec8-8cb3ad57fe34"},"spec":{"progressDeadlineSeconds":600,"replicas":1,"revisionHistoryLimit":10,"selector":{"matchLabels":{"app.kubernetes.io/instance":"traefik-2","app.kubernetes.io/name":"traefik"}},"strategy":{"rollingUpdate":{"maxSurge":1,"maxUnavailable":1},"type":"RollingUpdate"},"template":{"metadata":{"creationTimestamp":null,"labels":{"app.kubernetes.io/instance":"traefik-2","app.kubernetes.io/managed-by":"Helm","app.kubernetes.io/name":"traefik","helm.sh/chart":"traefik-9.1.0"}},"spec":{"containers":[{"args":["--global.checknewversion","--global.sendanonymoususage","--entryPoints.traefik.address=:9000/tcp","--entryPoints.web.address=:8000/tcp","--entryPoints.websecure.address=:9443/tcp","--api.dashboard=true","--ping=true","--providers.kubernetescrd","--providers.kubernetesingress"],"image":"traefik:2.2.8","imagePullPolicy":"IfNotPresent","livenessProbe":{"failureThreshold":3,"httpGet":{"path":"/ping","port":9000,"scheme":"HTTP"},"initialDelaySeconds":10,"periodSeconds":10,"successThreshold":1,"timeoutSeconds":2},"name":"traefik-2","ports":[{"containerPort":9000,"hostPort":9000,"name":"traefik","protocol":"TCP"},{"containerPort":8000,"hostPort":8000,"name":"web","protocol":"TCP"},{"containerPort":9443,"hostPort":9443,"name":"websecure","protocol":"TCP"}],"readinessProbe":{"failureThreshold":1,"httpGet":{"path":"/ping","port":9000,"scheme":"HTTP"},"initialDelaySeconds":10,"periodSeconds":10,"successThreshold":1,"timeoutSeconds":2},"resources":{},"securityContext":{"capabilities":{"drop":["ALL"]},"readOnlyRootFilesystem":true,"runAsGroup":65532,"runAsNonRoot":true,"runAsUser":65532},"terminationMessagePath":"/dev/termination-log","terminationMessagePolicy":"File","volumeMounts":[{"mountPath":"/data","name":"data"},{"mountPath":"/tmp","name":"tmp"}]}],"dnsPolicy":"ClusterFirst","hostNetwork":true,"nodeSelector":{"app-type":"online-app"},"restartPolicy":"Always","schedulerName":"default-scheduler","securityContext":{"fsGroup":65532},"serviceAccount":"traefik-2","serviceAccountName":"traefik-2","terminationGracePeriodSeconds":60,"volumes":[{"emptyDir":{},"name":"data"},{"emptyDir":{},"name":"tmp"}]}}}}
    meta.helm.sh/release-name: traefik-2
    meta.helm.sh/release-namespace: kube-system
status:
  observedGeneration: 7
  conditions:
    - type: Available
      status: 'True'
      lastUpdateTime: '2020-08-26T14:16:55Z'
      lastTransitionTime: '2020-08-26T14:16:55Z'
      reason: MinimumReplicasAvailable
      message: Deployment has minimum availability.
    - type: Progressing
      status: 'True'
      lastUpdateTime: '2020-09-11T03:00:36Z'
      lastTransitionTime: '2020-08-26T14:16:55Z'
      reason: NewReplicaSetAvailable
      message: ReplicaSet "traefik-2-59db755cd8" has successfully progressed.
spec:
  replicas: 0
  selector:
    matchLabels:
      app.kubernetes.io/instance: traefik-2
      app.kubernetes.io/name: traefik
  template:
    metadata:
      creationTimestamp: null
      labels:
        app.kubernetes.io/instance: traefik-2
        app.kubernetes.io/managed-by: Helm
        app.kubernetes.io/name: traefik
        helm.sh/chart: traefik-9.1.0
    spec:
      volumes:
        - name: data
          emptyDir: {}
        - name: tmp
          emptyDir: {}
      containers:
        - name: traefik-2
          image: 'traefik:2.2.8'
          args:
            - '--global.checknewversion'
            - '--global.sendanonymoususage'
            - '--entryPoints.traefik.address=:9000/tcp'
            - '--entryPoints.web.address=:8000/tcp'
            - '--entryPoints.websecure.address=:9443/tcp'
            - '--api.dashboard=true'
            - '--ping=true'
            - '--providers.kubernetescrd'
            - '--providers.kubernetesingress'
            - '--accesslog=true'
            - '--accesslog.filepath=/data/logs/access.log'
            - '--accesslog.bufferingsize=100'
          ports:
            - name: traefik
              hostPort: 9000
              containerPort: 9000
              protocol: TCP
            - name: web
              hostPort: 8000
              containerPort: 8000
              protocol: TCP
            - name: websecure
              hostPort: 9443
              containerPort: 9443
              protocol: TCP
          resources: {}
          volumeMounts:
            - name: data
              mountPath: /data
            - name: tmp
              mountPath: /tmp
          livenessProbe:
            httpGet:
              path: /ping
              port: 9000
              scheme: HTTP
            initialDelaySeconds: 10
            timeoutSeconds: 2
            periodSeconds: 10
            successThreshold: 1
            failureThreshold: 3
          readinessProbe:
            httpGet:
              path: /ping
              port: 9000
              scheme: HTTP
            initialDelaySeconds: 10
            timeoutSeconds: 2
            periodSeconds: 10
            successThreshold: 1
            failureThreshold: 1
          terminationMessagePath: /dev/termination-log
          terminationMessagePolicy: File
          imagePullPolicy: IfNotPresent
          securityContext:
            capabilities:
              drop:
                - ALL
            runAsUser: 65532
            runAsGroup: 65532
            runAsNonRoot: true
            readOnlyRootFilesystem: true
      restartPolicy: Always
      terminationGracePeriodSeconds: 60
      dnsPolicy: ClusterFirst
      nodeSelector:
        app-type: online-app
      serviceAccountName: traefik-2
      serviceAccount: traefik-2
      hostNetwork: true
      securityContext:
        fsGroup: 65532
      schedulerName: default-scheduler
  strategy:
    type: RollingUpdate
    rollingUpdate:
      maxUnavailable: 1
      maxSurge: 1
  revisionHistoryLimit: 10
  progressDeadlineSeconds: 600

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.


refactor analyzer and add unit test

What
Simplifies the signature of the ResourcesAnalyzer#analyze and add appropriate tests.

LGTM!


Does revery support Chinese char?

When I input Chinese, it doesn't display.

Oh, I have found the reason why Chinese can't display is the font. After using Microsoft Yahei font family, the Chinese can display.


Implementation of Fibonacci Series in Python

🛠 Issue (#641 )

Issue no #641
👨💻 Changes proposed
Implemented the python file of fibonacci series in both recursive and dynamic approach.

✔ Check List (Check all the applicable boxes)

[✔ ] My code follows the code style of this project.
[ ✔] This PR does not contain plagiarized content.
[ ✔] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
This time the code will not give the entire sequence rather the nth term that user want to find.

📷 Screenshots

@sitamgithub-MSIT Kindly implement all languages of same algorithm in the same PR immediately

@sitamgithub-MSIT I think I am clear to you that all program files should be in the same PR, or else none of them would be merged


Poor performance with optional columns

Sadly, this is more of a design problem, and only a breaking change could fix it: when we make a column optional using OptionalCellValueReader, and the column is actually missing, importing N lines with K missing columns will throw and catch N*K exceptions, dramatically degrading performance.
Proposed solution
Instead of throwing an exception, ReadCellValueResult could have a Success property that tells if the cell was read. The exception should be thrown only at the end of the chain, before invoking the cell value mappers, if none of the readers have reported success.

Happy to take the breaking change if you can show perf numbers and submit a PR :)

THB I decided to fork this project many times, but never had the time :) As soon as I have some capacity, I'll do that PR

First off, thanks for the great tooling, thanks so much! Just dropping in my own note to re-attest that the performance hit is large with optional columns. For my Excel file which had around 44,000 rows, by removing the "MakeOptional()" and assigning a default value after the process, the time went from 5-10 minutes down to under 10 seconds. I could get away with it because I was just using that mapping to assign a default value. If you actually need the functionality I am not yet sure of the work around.
I did look at the code where the try /catch block is used and I am not yet sure how it could be restructured. If I can figure something out, I will indeed put in a pull request.

What do you think about making the result nullable. i.e.
ReadCellValueResult? GetValue(ExcelSheet sheet, int rowIndex, IExcelDataReader reader)

Or we could have
bool TryGetValue(ExcelSheet sheet, int rowIndex, IExcelDataReader read)

See #44


Interruptible monadic regions

The main feature of this pull-request is interruptible monadic regions:

A new interruptible combinator, which has no effect when used alone, but which can open up a "hole" of interruptibility inside of an uninterruptible region, which can be fully recovered from inside the uninterruptible region
New region-based, vertically composable semantics for interruptible status

As a result of these changes, bracket and bracketExit have simple implementations that do not involve forking. I suspect there are other operations that can benefit.
To track regions, a high-performance StackBool implementation is used. Precision of region tracking has been improved—it is now resistant to finalizer failure (it was not before, although it did not really matter; now it does matter).
This pull request is now ready for more tests and detailed feedback! /cc @mvv @regiskuckaertz
Fixes #706

@mvv Please take a look when you get a chance. Need more tests & such!

@iravid @regiskuckaertz
Looks like ensuring is no longer primitive:
trait ZIO[R, E, A] { self =>
  def ensuring(f: UIO[_]): ZIO[R, E, A] =
    self.interruptible.run.flatMap(e => f *> ZIO.done(e)).uninterruptible
}

It always bothered me that the implementation of ensuring depended on non-interruptibility, but was not previously expressible in terms of that.
Looks to me like ensuring = foldCause + (un)interruptible.

All right, this is interesting. The above definition of ensuring is not quite the same. Mainly because it is possible that self is not interruptible, due to an enclosing effect.

Will take a closer look at stack unwinding tomorrow.

I think the final thing to deal with (aside from making core operations interruptible & above name discussion) is exception handlers.
In Haskell, the exception handler is not interruptible. Probably this makes sense as a good default. I need a failing test to identify this issue and then fix it here.

@iravid 🤦♂

Will do annihilation laws in a separate PR.


Table with horizontal scroll

Markwon version: {4.6.1}
I am using TablePlugin for showing content in the table format. Small content is showing properly. But when big content is showing improper like below image.

I want to show content in the table with a horizontal scroll. I see your example with the recycler view that is working. But I do not want to use the recycler view. so can anyone help me by showing content in the table with a horizontal scroll.
SC-1\n\n| POS     | URL     | Name     |\n| ---------- | ---------- | ---------- |\n| 1
| https://www.google.com/search?ei=ChD4X_GmB86P9QO7k554&q=markdown+table+render+in+markwon&oq=markdown+table+render+in+markwon&gs_lcp=CgZwc3ktYWIQAzIHCCEQChCgAToECAAQRzoECAAQDToGCAAQDRAKOggIABANEAUQHjoKCAAQCBANEAoQHjoICAAQCBANEB46BggAEA0QHjoFCAAQzQJQ05XqAliv6OoCYOWK6wJoAHACeAGAAYcDiAGWG5IBCDEuMjMuMS4xmAEAoAEBqgEHZ3dzLXdpesgBCMABAQ&sclient=psy-ab&ved=0ahUKEwjx0_mV7ovuAhXOR30KHbuJBw8Q4dUDCA0&uact=5
| Release Notes -  Widget Deployment       |\n| 2       | https://www.google.com/search?ei=ChD4X_GmB86P9QO7k554&q=markdown+table+render+in+markwon&oq=markdown+table+render+in+markwon&gs_lcp=CgZwc3ktYWIQAzIHCCEQChCgAToECAAQRzoECAAQDToGCAAQDRAKOggIABANEAUQHjoKCAAQCBANEAoQHjoICAAQCBANEB46BggAEA0QHjoFCAAQzQJQ05XqAliv6OoCYOWK6wJoAHACeAGAAYcDiAGWG5IBCDEuMjMuMS4xmAEAoAEBqgEHZ3dzLXdpesgBCMABAQ&sclient=psy-ab&ved=0ahUKEwjx0_mV7ovuAhXOR30KHbuJBw8Q4dUDCA0&uact=5
| WEB-Support Release     |\n| 3
| https://www.google.com/search?ei=ChD4X_GmB86P9QO7k554&q=markdown+table+render+in+markwon&oq=markdown+table+render+in+markwon&gs_lcp=CgZwc3ktYWIQAzIHCCEQChCgAToECAAQRzoECAAQDToGCAAQDRAKOggIABANEAUQHjoKCAAQCBANEAoQHjoICAAQCBANEB46BggAEA0QHjoFCAAQzQJQ05XqAliv6OoCYOWK6wJoAHACeAGAAYcDiAGWG5IBCDEuMjMuMS4xmAEAoAEBqgEHZ3dzLXdpesgBCMABAQ&sclient=psy-ab&ved=0ahUKEwjx0_mV7ovuAhXOR30KHbuJBw8Q4dUDCA0&uact=5    | Article       |\n| 4       | https://www.google.com/search?ei=ChD4X_GmB86P9QO7k554&q=markdown+table+render+in+markwon&oq=markdown+table+render+in+markwon&gs_lcp=CgZwc3ktYWIQAzIHCCEQChCgAToECAAQRzoECAAQDToGCAAQDRAKOggIABANEAUQHjoKCAAQCBANEAoQHjoICAAQCBANEB46BggAEA0QHjoFCAAQzQJQ05XqAliv6OoCYOWK6wJoAHACeAGAAYcDiAGWG5IBCDEuMjMuMS4xmAEAoAEBqgEHZ3dzLXdpesgBCMABAQ&sclient=psy-ab&ved=0ahUKEwjx0_mV7ovuAhXOR30KHbuJBw8Q4dUDCA0&uact=5     | Article       |\n| 5       | google.com       | Article       |\n| 6       | www.facebook.com      | \n "QA test title")\n\nSC-2\n\n| Header1     | Header2     | Header3     |\n| ---------- | ---------- | ---------- |\n| Cell1       | Cell1       | Cell1       |\n| Cell2       | Cell2       | Cell2       |\n| Cell3       | Cell3       | Cell3       |\n\n\n\nTest number list\n1. Number1\n2. Number2\n3. Number3\n4. Number4\n\nTest bullet list\n- Bullet1\n- Bullet2\n- Bullet3\n- Bullet4\n"
Thanks
Dileep

@noties  can you help with this?

@noties  can you help with this?

Hello @chaurasiadilip ,
it is possible to do, but it would require an effort. At this point it is much easier just to use the RecyclerView version. I have a feature planned that could've solved your issue, but I have no estimate for it, so I would not put your hopes high. If you are inclined to use a LinearLayout (or similar ViewGroup), then you could:

parse markdown and obtain Node
then use MarkwonReducer to create a list of nodes
then use a special widget for each node in that list (TableBlock would appear here) and create a view for it, which you append to your ViewGroup contaner

so far I'm describing the logic that markwon-adapter is using.

take TableEntry and see what it does, there is no magic — it is using all public constructs, methods and properties, and adjust it to fit your needs

For the future, please format your code snippets accordingly, it is super easy with markdown, surround your code with three backticks (```):
Hello **there**, _hey_!

Hello @chaurasiadilip ,
it is possible to do, but it would require an effort. At this point it is much easier just to use the RecyclerView version. I have a feature planned that could've solved your issue, but I have no estimate for it, so I would not put your hopes high. If you are inclined to use a LinearLayout (or similar ViewGroup), then you could:

parse markdown and obtain Node
then use MarkwonReducer to create a list of nodes
then use a special widget for each node in that list (TableBlock would appear here) and create a view for it, which you append to your ViewGroup contaner

so far I'm describing the logic that markwon-adapter is using.

take TableEntry and see what it does, there is no magic — it is using all public constructs, methods and properties, and adjust it to fit your needs

For the future, please format your code snippets accordingly, it is super easy with markdown, surround your code with three backticks (```):
Hello **there**, _hey_!


Pre-train model release

Hi,
Thanks for all the great work. You said you will put pre-train model in core_script folder in reply of another  issue. But I can't find it

Hi Steven，
Do you mean our pre-train model?

The model you used for the result

Yes. There are two ways you can do it.
1.python eval.py --input_path /your/path/to/sample.wav --model_path /your/path/to/your_model.pth the eval.py file is provided.
2. https://zinc.cse.buffalo.edu/ubmdfl/deep-o-meter/landing_page
using this platform to test out.

can u upload weights here instead of on website as open source

Hey
Would be great indeed to have the weight of the trained model in the repo :)
Thanks

Thank you all. The weight of the trained model has been uploaded to the readme with a Google Drive link.
https://drive.google.com/file/d/1TWdsCFKP2luAfhpB91N9X4z1gsJMvvhI/view?usp=drive_link
Please let me know if you have any questions.

Thank you all. The weight of the trained model has been uploaded to the readme with a Google Drive link. https://drive.google.com/file/d/1TWdsCFKP2luAfhpB91N9X4z1gsJMvvhI/view?usp=drive_link Please let me know if you have any questions.

any instruction for to use it for custom as I'm working on project with this dataset

Also, can you share other evaluation matrices?
we are trying to deploy it on real time so we need to know inference speed and percision

Thank you all. The weight of the trained model has been uploaded to the readme with a Google Drive link. https://drive.google.com/file/d/1TWdsCFKP2luAfhpB91N9X4z1gsJMvvhI/view?usp=drive_link Please let me know if you have any questions.

any instruction for to use it for custom as I'm working on project with this dataset

To use the pretrained weight, you can run 'python eval.py --input_path /your/path/to/sample.wav --model_path /your/path/to/your_model.pth'
you can also check the code in eval.py to make any modifications according to your custom requirements

Also, can you share other evaluation matrices? we are trying to deploy it on real time so we need to know inference speed and percision

You can check our paper for any evaluation metrices, the inference speed is fast enough for real time detection, inference time should be less than 1s for about 10s of speech, after loading the weight.

Also, can you share other evaluation matrices? we are trying to deploy it on real time so we need to know inference speed and percision

You can check our paper for any evaluation metrices, the inference speed is fast enough for real time detection, inference time should be less than 1s for about 10s of speech, after loading the weight.

there's  is no such matrix just reduce error rate


SECURITY - restrict server folder in deployment documentation

Describe the problem
I tried to deploy sveltekit with Plesk+nodejs adapter and it works great !
But I discovered that event if nginx redirect all request to sveltekit, if you try https://mywebsite.com/server/chunks/private-ade19f8d.js, it gives you the content of the file ! This mean that all your secrets are accessible from the web !!!
Describe the proposed solution
Since this is a big security flaw (not directly related to SvelteKit but rather to its use), I think we should add a page in the documentation that indicates that you really need to block access to /server.
For example, this is the nginx config I added for Plesk :
location ~ /(server|client|tmp|node_modules|env\.js|handle\.js|shims.js|index\.js) {
    deny all;
    return 404;
}

Alternatives considered
No response
Importance
i cannot use SvelteKit without it
Additional Information
No response

After careful consideration we're closing this issue as non-actionable. Documenting this won't help, rather confuse most people - or rather those people who actually read this close enough. This feels like something that should be documented on the hosting side (like; here's how you secure your server files in Plesk). The fact that other don't observe this behavior hint to me this is also not a common problem.


Added alternative term to areal energy density

Added alternative term specific power to areal energy density and added my user ID 0027. Issue #893

@HeddaElisabethGardian if this PR is ready for review, please request reviewers to check everything. If it isn't ready yet, you can convert the PR to draft to indicate that the PR is still in progress. Thanks ;)

The class areal power density is defined int the module (file) oeo-physical.omn, so your change should appear in that file. However, you committed and pushed only the file oeo.omn.
Additionally, please also update the file CHANGELOG.md.


cooldown listener issues?

I can't seem to trigger a warning with a command with a cooldown so i set up a listener like this
const { Listener } = require('discord-akairo');

class CommandCooldownListener extends Listener {
    constructor() {
        super('commandCooldown', {
            emitter: 'commandHandler',
            event: 'commandCooldown'
        });
    }

    async exec(message, command, reason) {
        console.log(`${message.author.user} triggered a cooldown from using ${command.id} because of ${reason}`);
    }
}
module.exports = CommandCooldownListener;

and this triggers a AkairoError [INVALID_TYPE]: Value of 'emitter' was not an EventEmitter error
if i use client as the emitter that does nothing.
I read somewhere that you need to add extra classes as classToHandle under the ListenerHandle but I couldn't seem to get that to work either. am I making some rookie mistake or is something changed about the cooldown system? I can't find any other examples online

What version are you on? If on master/beta, you have to specify the emitters yourself (except client) with ListenerHandler#setEmitters

Adding on the previous comment, there's also the listener guide that will help you set it up.
Also CommandHandler#commandCooldown (on stable) has been renamed to CommandHandler#cooldown (on master).

okay cool, finally got it, that was confusing. I ended up running into another problem with zucc that silently stopped my bot from connecting which was a pain but for future people who come across this issue this is what I changed (Master version) In the listener:
class cooldownListener extends Listener {
    constructor() {
        super('cooldown', {
            emitter: 'commandHandler',
            event: 'cooldown'
        });
    }

and in my client constructor I have this now:
this.commandHandler.useInhibitorHandler(this.inhibitorHandler);
this.commandHandler.useListenerHandler(this.listenerHandler);
this.listenerHandler.setEmitters({
    client: this, //don't know if this is required but doesn't error when included
    commandHandler: this.commandHandler,
    inhibitorHandler: this.inhibitorHandler,
    listenerHandler: this.listenerHandler,
});
this.commandHandler.loadAll();
this.listenerHandler.loadAll();
this.inhibitorHandler.loadAll();

Thanks for you help guys :) so many very helpful people in this community <3

Happy to help!
Also, the client: this is useless since it's automatically added here, in the ListenerHandler constructor.


[#2173] feat(remote merge): support netty for remote merge.

What changes were proposed in this pull request?
Support netty for remote merge. Use direct ByteBuf to replace with byte[] when netty is enable. And optimized code structure to avoid memory leaks
Why are the changes needed?
Fix: #2173
Does this PR introduce any user-facing change?
No.
How was this patch tested?
unit test, integration test, real job in cluster.

@jerqi Can you please review this PR?


Vue.js files don't show Vue icon

Files with the .vue extension don't show their correct icon:

Do you have an extension that provides support for Vue files ?

just installed and that fixed it, thanks!


Test failure when rspec run from inside .gem sub directory

My $GEM_HOME is ~/.gem (I believe this is the default when using chruby). One of the rspec tests fail when run beneath a hidden directory.
Expected behavior
No rspec failures when the code is stored beneath a hidden directory.
It seems this line is the cause of the match returning false: https://github.com/bbatsov/rubocop/blob/master/lib/rubocop/path_util.rb#L35
Can the deprecated behaviour be removed yet?
Actual behavior
Failures:

  1) RuboCop::Config#file_to_exclude? when the passed path matches any of patterns to exclude returns true
     Failure/Error: expect(configuration.file_to_exclude?('bar.rb')).to be_truthy

       expected: truthy value
            got: false
     # ./spec/rubocop/config_spec.rb:234:in `block (4 levels) in <top (required)>'

Finished in 27.62 seconds (files took 5.31 seconds to load)
9425 examples, 1 failure, 3 pending

Failed examples:

rspec ./spec/rubocop/config_spec.rb:228 # RuboCop::Config#file_to_exclude? when the passed path matches any of patterns to exclude returns true

Steps to reproduce the problem

Move the RuboCop source directory beneath a hidden directory e.g. ~/.gem/rubocop
run rspec in the ~/.gem/rubocop directory

RuboCop version
$ rubocop -V
0.39.0 (using Parser <IP_ADDRESS>, running on ruby 2.3.0 x86_64-darwin15)

Can the deprecated behaviour be removed yet?

For sure. Feel free to update this.

For sure. Feel free to update this.

Scary deprecating code in a project that a lot of people rely on, but there you go.


KeyValuePairConverter produces result incompatible with DataContractJsonSerializer

JsonNet uses KeyValuePairConverter (https://github.com/JamesNK/Newtonsoft.Json/blob/master/Src/Newtonsoft.Json/Converters/KeyValuePairConverter.cs) which explicitly sets Key and Value field names as, respectively, "Key" and "Value". However, the json, that is produced, cannot be deserialized back to KeyValuePair<TKey,TValue> with DataContractJsonSerializer, because in the actual KeyValuePair .NET type there are private fields key and value, and Key and Value are public getters. So DataContractJsonSerializer throws an exception that it cannot find "key" and "value" in the json.
KeyValuePairConverter should be updated to produce json with "key" and "value" fields instead of "Key" and "Value"

That would be a huge breaking change for anyone relying on the output of KeyValuePairConverter (which would probably be anyone actually using it at the moment).  What could make sense would be to either
a) Allow a flag, which defaults to false, to use lowercase field names when serializing; or
b) Provide your own DataContractCompatibleKeyValuePairConverter' (basically a clone of the KeyValuePairConverter.csfile) and modify the two private const string fields; or c) A mix of (a) and (b) in spirit...  ModifyKeyValuePairConverterso that it getsKeyNameandValueNameby calling a virtual protected function and then you can just overrideKeyValuePairConverter` and implement just that function

I recommend you write your own KeyValue JsonConverter.

I think this should actually be fixed. The thing is - Json.Net works perfectly fine when deserializing KeyValuePair with either case, because the uppercase fields match the hard-coded field names, while the lowercase fields are properly mapped to corresponding private fields via reflection.
The below two examples both produce the same valid result:
KeyValuePair<string, string> lowercase = JsonConvert.DeserializeObject<KeyValuePair<string, string>>(@"{ key: ""key"", value: ""value"" }");
KeyValuePair<string, string> uppercase = JsonConvert.DeserializeObject<KeyValuePair<string, string>>(@"{ Key: ""key"", Value: ""value"" }");
So technically if Json.Net starts emitting lowercase key and value for KeyValuePair, it would not break anyone who is using Json.Net to deserialize that Json back. So the only scenarios this could break would be cases when people explicitly rely on the Key and Value being upper case. At the very least this could be made configurable via JsonSettings. I just feel that it's not correct to have such a fundamental type in .NET being serialized in a form not deserializeable with pure reflection (i.e. requiring a custom converter).

But it would brake anyone using Key/Value in JavaScript

Fair point. I wonder though how many people use KVPair output from non-.NET code.
Instead of just having a custom KVPair converter, do you think the behavior can be made configurable through JsonSettings? With perhaps 3 options: default (current), DCJS-compatible (lowercase) and "uber" (send out both upper and lower case, twice the payload size, but guaranteed to be compatible)? I can work on a pull request for that if you would consider taking such a change.


feat: update input schema docs

Updates the docs to include the new prefill property on Boolean fields in the input schema. When the field has a default value set now, we display the default chip, to differentiate between them and regular/prefilled fields.
Related to https://github.com/apify/apify-core/pull/16390/
Also, polite request - please merge this when the above PR is out - I won't be able to since I won't be at Apify anymore :)

FYI: @gippy , this is out right?

Yes, merging


Add --exclude option to the CLI script

Is your feature request related to a problem? Please describe.
I run the SVGO recursively on a single folder (-r -f), but I have some auto generated SVG files that I for structural purposes have placed in a nested folder in the given folder. However since I know this file is already optimized there is no reason to use resources on parsing it (as well the risk of accidental clearing of the file as it is only symbols), so I would like to exclude them when executing.
I have look in the code (and documentation) and I have not been able to find any way to do this.
Describe the solution you'd like
An option to exclude certain files (--exclude or -e), possibly as regular or glob expression (eg --exclude "auto/.+\.svg")
Describe alternatives you've considered
I have considered building my own script that uses the SVGO module that parses the files I need. But that seems redundant, since all other options would be the same and thus a simple exclude in the CLI would be simpler.
Additional context
N/A

Fixed in https://github.com/svg/svgo/releases/tag/v2.3.0


Fixed robot file, ready to check

Good first iteration.  I don't think the registry works, though.  Take a look at the comments, and fix and refactor where necessary.  Resubmit when ready.


Weird dragging behaviour when programmatically adding elements to Grid

Hello! First of all, thanks for the great plugin!
I have setup a grid layout with the following settings:
:is-draggable="true"
:is-resizable="false"
:vertical-compact="true"
:margin="[0, 0]"
:use-css-transforms="true"
If I load a lot of items at start, everything works flawlessly. However, If I have an empty grid, and then add items (by click or something), the grid elements have weird dragging behaviour. They don't move when other items are being draged over them. Below is a gif that demonstrates this weird behaviour.
I checked, and the items have valid properties (x, y, w, h, i). What could be causing the problem?
Thanks in advance

Please use the issue template and provide a code sample as well.

I am closing this issue because I couldn't fix it, therefore used another plugin.


Onagre doesn't scroll view when navigating with arrow keys

When scrolling with the arrow keys, the view doesn't scroll along with the focused item. The view scrolls just fine when using the mouse wheel.
OS: Arch Linux
WM: Sway

Thank you for pointing that out.
I am well aware of the issue, I am waiting for this PR to be merged in iced.
Once its done, this would be trivial to implement.


minor typo in README.md

Minor typo with the README.md linking to FreeTDS.  Markdown was consuming your comment as a link attribute.

Thanks!


04/12/2018 20:50 : Create Pull Request for branch prTesting_2018041220 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Segmentation fault

The program seems to crash at some point (after writing about 8.3Gb of data to the disk) with Segmentation fault (core dumped). Since there is no helpful message, do you know what could be causing this error?
Thanks!

I am running it on a machine with 256GB RAM; doesn't look like it should be the problem. I also ran it on a different node. But the error seems to come up in the same instance during the computation -- such that the output file is truncated to 8.3GB.

@cx0 On a side note, how much memory does it really need?

Hi, I realize this is much later than your original message but I've eyeballed some of the RAM usage so I thought I'd post.
With ~7M SNPs and 1000 reference individuals the "ldpred" command used 240 GB RAM.
With ~7M SNPs and 3.7K reference individuals the "ldpred" command usage increased to 250 GB ram.
I am now running it per chromosome with ~7M SNPs and 1000 individuals, and in total it is using 270G RAM.

I apologize for the high memory use.  The limiting factor is likely the LD step, where the memory usage scales quadratically in the size of the LD window.  Hence, if the memory usage is very large, please consider a smaller LD window.

hey, just a side point, I have been struggling with same thing today, and I wonder what you mean by "LD_FILE_PREFIX"? The ld file ends with extension ".LD"?

I apologize for the slow reply on this, but I didn't notice your last comment as I closed the issue.
Anyways, LD_FILE_PREFIX refers to the filename where the LD will be stored.
Also, I have made lots of changes to the code now, including bug fixes, so perhaps you can try the latest version.


Idea: Only have root words in word file

Due to the way this extension completes works, it may be worth removing words which differ only be a short suffix (such as plurals, etc.). Explanation/example below

When chal- is typed,
challenge and challenger will both be suggested
Upon selecting challenge no space will be added
So if one wanted challenger, one could just select challenge and press the key r
This has the same effect as using arrow keys to select challenger but has two main advantages

(Minor) this method usually will be faster to do, since r sits closer to where the fingers naturally lie than the arrow keys, this holds true for other suffixes
(Major) by not suggesting challenge and challenger

Space is saved on the suggestions box
This opens up room for more (different) suggestions, increasing the chance that the desired word will be shown
Potentially makes the suggestion box look less cluttered / stuffed full of words

Bonus idea (significant undertaking): Making it so that if a word that has multiple possible suffixes is selected, new suggestions for the suffixes appear.

Do you know you can even type chalr (chal and then r) to get challenger 😏

And another possible reason is the irregular words, e.g. try and tries.

Aaah, that's quite nice :) Since now the work doesn't seem to have sufficient benefit to warrant it lets close this issue.


Starting Development Server issue

I have nodeJS installed locally & when i tried to run it using ./server.js it threw this error:
events.js:353
throw er; // Unhandled 'error' event
^
Error: connect ETIMEDOUT <IP_ADDRESS>:443
at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1146:16)
Emitted 'error' event on ClientRequest instance at:
at TLSSocket.socketErrorListener (_http_client.js:475:9)
at TLSSocket.emit (events.js:376:20)
at emitErrorNT (internal/streams/destroy.js:106:8)
at emitErrorCloseNT (internal/streams/destroy.js:74:3)
at processTicksAndRejections (internal/process/task_queues.js:82:21) {
errno: -4039,
code: 'ETIMEDOUT',
syscall: 'connect',
address: '<IP_ADDRESS>',
port: 443
}
Can someone explain me that why am i getting this and what can i do solve this?

I have the same problem. Have you solved it?

@adityassharma-ss I have the same problem as you. Have you solved it?

Hi!
We've just migrated the CNCF landscape to v2 and this issue probably no longer applies.
Please feel free to reopen if needed 🙂


spike: re-implement the class-based resource in terms of function-based resources

This should remove quite a bit of code from /core, if possible, but the gist would kinda be:
// "Resource" today, maintaining instanceof compatibility
class ResourceState {
  static from(args) {
    return Resource(this, args);
  }
}

const Resource = (Klass, args) => resource( ({ on }) => {
  let instance = new Klass();

  return () => {
    instance.modify?.(args.positional, args.named);

    return instance;
  };
});

just a rough sketch, but this would allow fer deleting hundreds of lines of code from core.

This is done in ember-modify-based-class-resource


Add GroovyDSLImprover

This PR makes Compass use (or at least in part) GDI for DSL objects.
Notes

As GDI is a Groovy transformer, the DSL classes must be written in Groovy; this lead to the addition of the Groovy plugin and move of all classes from the java folder to the groovy folder. This can be done as Groovy supports cross-compilation with Java.

Nice. I approve


Invalid integer buffer

I try to encode number 1000, it should return 8203e8 but return 8103e8.
After traced step by step, I found that the buffer length of 1000 was wrong, it always return 1:
object(Web3p\RLP\Buffer)#323 (2) {
  ["data":protected]=>
  array(1) {
    [0]=>
    int(1000)
  }
  ["encoding":protected]=>
  string(4) "utf8"
}

Buffer should be
Buffer($length) => create the length of data
or
Buffer($integer) => encode data as hex

https://github.com/web3p/rlp/pull/10/commits/7d2b025f9c03ad23b379a3c10b95a9674ea3d792


docs: Added Tailscale access recipe to cookbook

This commit adds a cookbook recipe explaining how to use a container in a Dagger pipeline to access a Tailscale network.

@helderco  can you suggest the change needed to fix the lint error at https://github.com/dagger/dagger/actions/runs/5718235177/job/15493626865?pr=5532#step:4:514? Thanks.

❯ cd sdk/python
❯ hatch run fmt

Ping @kpenfound @jpadams @marcosnils for review

Do you want to rebase on top of main? The Rust SDK issue has been fixed via:

https://github.com/dagger/dagger/pull/5626

Do you want to rebase on top of main? The Rust SDK issue has been fixed via:

Fix sdk:rust:test running against nightly Rust #5626

Rebased. Ready for re-review @gerhard


Fix queries involving links to objects that are then deleted or moved

Update to accommodate Query::links_to taking a row accessor rather than a row index.
Fixes #1803.
This requires a new core release that includes realm/realm-core#1397.
/cc @jpsim @tgoyne

:+1:

:+1:


Split tests in different files

...and possibly we can also use pytest as the test runner (but we still want to support unittest).

https://github.com/giampaolo/pyftpdlib/pull/342


Update README.md

Modified the list of pages for email, as Installation should be first.
-> Next step = review by Attila (hence the assignation)

I could do the merge but I would prefer to have you reviewing it. I believe there is a setting that allows to have systematic reviewing. I'll look for it.

I am merging your changes now


[SPARK-36257][SQL] Updated the version of TimestampNTZ related changes as 3.3.0

What changes were proposed in this pull request?

As we decided to release TimestampNTZ type in Spark 3.3, we should update the versions of TimestampNTZ related changes as 3.3.0.
Why are the changes needed?

Correct the versions in documentation/code comment.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing UT

cc @beliefer as well

Test build #141476 has started for PR 33478 at commit 343383c.

Test build #141476 has finished for PR 33478 at commit 343383c.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141476/

jenkins, retest this please

Test build #141479 has started for PR 33478 at commit 343383c.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45994/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45994/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45996/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45996/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45994/

@MaxGekk Thanks, I have updated the code.

Test build #141486 has started for PR 33478 at commit 6ded562.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46004/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46004/

Test build #141479 has finished for PR 33478 at commit 343383c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merging to master

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141479/

Test build #141486 has finished for PR 33478 at commit 6ded562.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141486/


Missing Namespaces like Steamworks and LobbyCreated_t

hello,
I just pulled so I can look at your example, as I am having issues when following your tutorial:
https://www.youtube.com/watch?v=Fx8efi2MNz0
I have to say though, BIG BIG thank you for your tutorial, it helps a lot to understand more on the network manager and Mirror... but when i go to pull the: https://github.com/DapperDino/Mirror-Multiplayer-Tutorials
I get the following errors:
on script MyNetworkManager.cs: the type or namespace 'SteamWorks' cannot be found
this is also same message but different name space:
LobbyCreated_t
GameLobbyJoinRequested_t
AvatarImageLoaded_t
not sure if I am missing additional cs files or classes etc.?

instructions:
https://steamworks.github.io/installation/#unity-instructions
File:
https://github.com/rlabrecque/Steamworks.NET/releases/download/15.0.1/Steamworks.NET_15.0.1.unitypackage


fix: pass missing secret

Relates to: #47

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


GORA-649 Replace usage of deprecated API of MongoDB driver

Use new MongoDatabase and MongoCollection API as described on Databases and Collections
Use Codec and CodecRegistry interface instead of DefaultDBEncoder
MongoFilterUtil: Avoid changing query passed as reference so adapted to return an Optional with subfilter to apply
Use org.bson.Document as container everywhere (replace DBObject)
Use Filters and Projections helper methods to facilitate the creation of filter documents
MongoDBResult: pass cursor and size as constructor args

+1


Model: changing model js filename breaks preview

<App Model="PodcastApp.js">
</App>

Renaming the PodcastApp.js file and then changing the Model="..." results in crash which does not go away by "Try again"

Turns out you must omit .js


(suggestion) Being able to have a paper plugin for Legacy Crafting and Console Fly mechanics to work on the client.

I wonder if there would be a way to have a Paper plugin of this for the server so the server would know you are using the mod in either forge, fabric or quilt and then the client would allow the Legacy Console Edition crafting and fly mechanics. It would be cool so I wouldn't have to switch my server into a fabric server since I want to use all my plugins.

bump


Updated raffo.ttl with groupings

Added lv2 groups to make it more compatible with zynthian.

Thanks for the contribution @Pastitas!

 Thanks for the great synth!


 It appears
to introduce a double-initialization of metrics, causing relay
initialization failures.
Related: #22316

Hubble relay was successfully installed, so the issue is mitigated with this revert: https://github.com/cilium/cilium/actions/runs/3535227606/jobs/5933004240

Alright good enough for me, let's :ship: . (Incidentally, we still hit #22168 but that's unrelated to the failure this PR is addressing)


[Inky]  tags around .menu should add attributes

It would be useful to be able to wrap the menu component with <center> tags so that the align="center"  attribute is added. This would allow centering the menu easily given the width is reduced from 100%.

We'll close in favor of https://github.com/zurb/foundation-emails/issues/247


fix edge case where a file shadows a directory

Say you have a directory structure like this:
- a
  - b.js
- a.js

The path for "a/b" would be handled by a.js instead of a/b.js. To fix, I completely rewrote the code to load and find modules and put it in a new class called ModuleTree.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

231 of 245 (94.29%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.5%) to 83.086%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/server/module-tree.ts
220
234
94.02%

Files with Coverage Reduction
New Missed Lines
%

src/typescript-generator/repository.js
2
59.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
2473

Relevant Lines:
3042

💛  - Coveralls


How to do some action when tap on buttonbar or when swipe between screens.

How can we do some action or do some task like save the data when tapping on the buttonbar or when we swipe between screens?

You can do it inside of changeCurrentIndex or changeCurrentIndexProgressive


[linux] Add "Type": "wine"

On linux windows executable bots must be run thru wine.
I suggest to add a "Type": "wine" to the LadderBots.json handler that runs the bot file with "wine botbinary.exe"

https://github.com/Cryptyc/Sc2LadderServer/commit/c9f352ed9a30a82b3e4067f4e4fdc215afdee88d


Fix old anthology ingestion to handle multi line authors

S/N: 11
Title: Fix old anthology ingestion to handle multi line authors

Not relevant to this version of the ACL Anthology, moving to legacy ACL Anthology.


change route() to multiprocessing style

Added multiprocessing dependency for running distributeTask() in a subprocess.

Hi @michaeleekk, Thank you very much for setting the PR. I don't think this works for the following two reasons (please correct me if I am wrong):

This seems not the multiprocessing, each job quested from UI will send to server independently. Thus, at any time there is only one job in "Process". You can test this by send a UI job which take a bit longer, and while server is working on it, you send several more jobs from UI. You can then observe the server still working one job at a time.
Even this works for multiprocessing, I don't think this solves the "memory leak" problem, actually making it worse. Because now you have two or more parallel processes to do DEG on a large dataset (seems like diffxy making internal copy of data), then you will be out of memory. You can test this, by open two or more browser instances, and submit DEG on each one of them, (you can use this to observe the multi-process as well), if multi-process work, memory usage will be double (multiple) times.

let me know what's your thought.
Thanks again for the PR.
O'Young

@z5ouyang Thanks for the comments. Let me clear your doubts :)

The code is from multiprocessing Python module so I named the PR as is. The code could potentially work for processing multiple requests at the same time but I decided not to make a big changes on the repo. The main point that is prohibiting it is your original code of having the jobLock in route(). I believe you wrote that for a reason, for example other code would break if they are used at the same time. So I decided to keep the PR small by keeping the jobLock.
I guess we might have a misunderstanding here. Memory leak doesn't mean you can process a lot of processes at the same time with insufficient memory. Memory leak means that when one part of code is completed, the memory usage will go back to the point when it's entering that part of code, i.e. there are no remaining objects, etc. in the memory after that part of code is finished. So if you would like to process a lot of huge datasets at the same time, you still need to provide enough memory to do so, but the point of this PR is memory leak.

I hope I make myself clear on the code  :)

Hi @michaeleekk, thanks for explaining. I might misunderstand you at beginning. If I am understanding this correctly, the PR is not fix the memory leak issue, but let the "multiprocessing" handle/do memory cleaning (garbage collection)? If it is, I will try to find the real cause and attempt to fix the issue. Will keep you posted. If you have additional information (additional test), please let me know. Thanks again for this!

This is not a memory leak after some testing. It is related to how python's "garbage collection". The large data generated within a function is not freed at then end of function immediately. Just added "gc.collect()" in 32e72188dcaa4ac026e58b28fba2dd2ec313a36e  after task is done to force python to do "garbage collection" solved the memory increasing issue.
Thanks @michaeleekk for reporting this issue!


[Bug] Admin Sub Menu Items Not Workign

When using the menu builder on the admin menu to create a sub menu.
Then it can not open the sub menu on the admin panel.

PR #74 fixed this issue, but only for 2 level menu

I created PR #74 working with 2nd levels only because I needed that fix right away.
If we take a look at src/Models/Menu.php, we realize that views are hard coded on the model, so I guess most of the code will change. I'm sure @tnylea have plans for this?
I also noticed that some of the menu items (Tools and Settings) are coded at src/views/master.blade.php, if we moved this to the database, we would be able to manage this items using the Menu Builder.

@akaZorg: We could indeed move the menu items from the view to the database. It just requires that 2nd levels works first.
Also I think it would come in handy if the menu where build using views instead, that makes it way more customizable. I do not know if @tnylea have any plans so far.

Hello, (sry about delay).
2nd levels are working at the moment, just from the 3rd + is not. I would advance more work on that, but I need confirmation from project maintainer - hey @tnylea, are you alive? :dancing_men: any plans for the menu?
I noticed you post issue #108 regarding views, we should do it indeed!

This works now, but only for 2nd level menus.
Let's keep this open until more levels are supported.
I might try to solve this somehow in the near future.


[Pulsar Functions] Add support for multiple sinks/sources in a single NAR file.

Contribution Checklist
Address https://github.com/apache/pulsar/issues/3576
Motivation
Added logic to use the proper class loader when a Source or Sink is created from the command line and an "archive" and "className" are provided. Prior to this change, it was always assumed the the archive file specified was a standard "JAR" file.
Now, we check the file extension of the archive and choose the appropriate class loader based on that. This allows user to submit Pulsar connectors that are bundled inside a NAR file
Also added support for multiple sink/source definitions in NAR files, while also being 100% fully-backward complaint. Added test classes to validate that
Modifications
Modified both the
pulsar-functions/utils/src/main/java/org/apache/pulsar/functions/utils/SinkConfigUtils.java
pulsar-functions/utils/src/main/java/org/apache/pulsar/functions/utils/SourceConfigUtils.java
classes to check the file extension of the archive, and use the correct class loader
Modified the org.apache.pulsar.functions.utils.io.ConnectorUtils class to read both the old and new pulsar-io.yaml formats as shown below. Therefore this is NOT a breaking change
name: file
description: Reads data from local filesystem
sourceClass: org.apache.pulsar.io.file.FileSource
and
connectors:
                hdfs:
                description: The HDFS connector
                  sourceClass: org.apache.pulsar.io.hdfs.SoruceClassImpl
                 sinkClass: org.apache.pulsar.io.hdfs.SinkClassImpl
                redis:
                  description: The Redis connector
                 sourceClass: org.apache.pulsar.io.redis.SoruceClassImpl
                  sinkClass: org.apache.pulsar.io.redis.SinkClassImpl
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as

org.apache.pulsar.functions.worker.rest.api.v3.SinkApiV3ResourceTest
org.apache.pulsar.functions.worker.rest.api.v3.SourceApiV3ResourceTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): No
The public API: No
The schema: No
The default values of configurations: No
The wire protocol: No
The rest endpoints: No (Fixes existing broken usage of archive/className params)
The admin cli options: No (Fixes existing broken usage of archive/className params)
Anything that affects deployment: No

Documentation

Does this pull request introduce a new feature? No
If yes, how is the feature documented? not applicable
If a feature is not applicable for documentation, explain why? It's already documented

@david-streamlio so which multiple sources/sinks packaged in a single NAR I would use the specify the classname to run a particular one? e.g.
./bin/pulsar-admin source create --name test --destinationTopicName output --archive foo.nar --classname my.source

Most connectors require multiple supporting jars to be bundled with them. This allows you to submit either an Uber jar or a NAR file.
I used the pulsar-aws-nar-0.0.1.nar file for testing, which has 3 sink and 3 source connectors.

@david-streamlio this PR looks good to me, but I am unclear how the changes in the PR makes submitting a uber JAR with multiple sources/sinks different from submitting a NAR.  Do we need to support submitting with a NAR with multiple sources/sinks if submitting a uber JAR already covers the use cases of multiple sources/sinks in one package? I guess it doesn't hurt, however my concern is that we have always submitted NARs without specifying a classname since there is the a yaml file under META-INF that specifies the default source/sink classname as well as the name of the source/sink.  How does that work here?  with multiple sources and sinks.

There wouldn't be a difference between submitting an uber jar and a NAR. They would both be submitted in the exact same way. Each would require the user to specify an archive AND a classname.
bin/pulsar-admin source create --classname org.apache.pulsar.io.aws.sqs.SqsSource --archive /Users/david/clone-zone/streamlio/pulsar-connectors/pulsar-nar-bundles/pulsar-aws-connector-bundle/pulsar-aws-nar/target/pulsar-aws-nar-0.0.1.nar --className org.apache.pulsar.io.aws.s3.source.AwsS3Source --name s3-source --destination-topic-name s3_data --source-config-file /Users/david/Desktop/s3-config.yaml

@david-streamlio so for this NAR you built, pulsar-aws-nar-0.0.1.nar,  with multiple sources and sinks, what happens if I submit this NAR without a classname? what happens? I ask since currently submitting a NAR without specifying the classname is supported and the de factor way to submit a NAR.  Does it contain a yaml file under META-INF that specifies the default source/sink classname like the rest of the sources and sinks that we currently have?

@jerrypeng @david-streamlio - did you have any conclusions in #3576? It would be worth that you guys can conclude an approach in #3576 , so that everyone is aligned with the approach and other committers can also review it based on that approach.

@jerrypeng If you submit the pulsar-aws-nar-0.0.1.nar without specifying a classname, then the sink/source configured in the pulsar-io.yaml file will be used. You can think of that as the "default" connectors, that can be overridden by the className property.

@david-streamlio ok that sounds good to me. Just making sure all code paths are covered.

@david-streamlio please revert back the change in pulsar_tools_env.sh and I think we are good to go. Thanks!

rerun java8 tests

Closed as stale.


Builds crawl to a halt when upgrading from 0.15.23 to 0.15.24.

I've noticed my builds taking around 10x longer since  upgrading from 0.15.23 to 0.15.24. I've also noticed peak memory consumption is up from 360mb to 920mb. I've had a look at https://github.com/systemjs/builder/compare/0.15.23...0.15.24 but can't notice any obvious cause.
I'm using the jspm bundle command as follows: jspm bundle app/app.js. I've profiled this command with version 0.15.23 and 0.15.24 of builder. You can see there is a hotspot in 0.15.24 that uses 65.0% of ticks where it only uses 5.7% in 0.15.23. I've included the processed traces below.
0.15.23.txt
0.15.24.txt

could that be because global and register transforms were switched from traceur to babel ?
https://github.com/systemjs/builder/commit/b474ee71df9a7a4ddebfdb0d284c12c9794488d7
https://github.com/systemjs/builder/commit/051deb6aab1d0113e77b5a860e6dbbc00c2544ef

I'm having a really hard time pin pointing this issue. I can reliably reproduce the slow down when moving from JSPM 0.17.0-beta.24 to 0.17.0-beta.25 which actually corresponds to a move from version 0.15.24 to 0.15.25 of systemjs-builder. However, when I manually install builder 0.15.24 or 0.15.25 in JSPM's node_modules directory, both work fine :/

This sounds like it could have been a consequence of the upgrade to using
Babel over Traceur transforms internally.
On Sun, 14 Aug 2016 at 01:54, Rickert Mulder<EMAIL_ADDRESS>wrote:

Closed #663 https://github.com/systemjs/builder/issues/663.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/systemjs/builder/issues/663#event-754913861, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAkiygQCHYqVK7opJX_W3CXFeESVO88Wks5qflkvgaJpZM4JjjLj
.

I was able to reproduce this consistently when upgrading from JSPM 0.17.22 to 0.17.23. I the process if trying to find the exact commit of builder that caused this, NPM's deduping cause some confusion making me a bit making me unsure of what I was reporting. I'll try again and reopen this issue once I regain my confidence in the details.

It seems pretty clear this is a consequence of the move from Traceur to
Babel in the latest landed systemjs prs.
This will be tough to fix - either involving switching back to Tracuer
(which is not really an option as it's not well supported and doesn't
support code comments and has variable source maps support).
The only path forward then is probably to do some performance optimization
work on the Babel-plugin-transform-es2015-modules-systemjs package in Babel
core. That will not be an easy job though!
On Sun, 14 Aug 2016 at 21:24, Rickert Mulder<EMAIL_ADDRESS>wrote:

I was able to reproduce this consistently when upgrading from JSPM 0.17.22
to 0.17.23. I the process if trying to find the exact commit of builder
that caused this, NPM's deduping cause some confusion making me a bit
making me unsure of what I was reporting. I'll try again and reopen this
issue once I regain my confidence in the details.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/systemjs/builder/issues/663#issuecomment-239691575,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAkiyu7WlXDJ838c_cXzltO_F9Cy6l0Rks5qf2trgaJpZM4JjjLj
.

I was unable to reproduce this issue in isolation. It appears to have been a symptom of a specific version of some package I had installed. After I deleted the node_modules directory and reinstalled everything the problem disappeared for all versions of JSPM and SystemJS Builder.


[Feature Request] mod config to configure mods

Is your feature request related to a problem? Please describe.
I want to configure mods without going into mod menu
Describe the solution you'd like
mod config modname would open up the mod config screen like the modmenu cog button does.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Thank you!


Change color of pinned tags

Expected Behaviour
A pinned tag should look different than a non-pinned one. Please change the color of a tagged pin. The color should change back, if the user unpins a tag.
Actual Behaviour
The color does not change.
Steps to Reproduce

Pin a tag
Color does not change

Pinned tags will now have a pin on their left side, so they can be distinguished from non-pinned tags.


tsx support

Currently leasot doesn't support tsx extension. Is the support for tsx planned for upcoming releases? #93

https://github.com/pgilad/leasot/commit/5a17146e4cd2755816355cf7e318757232a72ba9


Climate data retrieval go lang service creation

Current weather information retrieval using weather API go lang service

I have pushed the go lang service for data retrieval


Keep selected tab through screen rotation

Kind (Required)

Improvement

Overview (Required)

Since this is the state of FeedScreen, it may be possible to keep it in ViewModel

Links

I would like to challenge this issue.　@takahirom

@Index197511 please challenge it🚀  We assigned you 👍


2-Phase Commit

TODO: Implement 2-phase commit.

Completed. Can now pull 2-phase commit from latest version in the repo.


Wrong slash commands branch git URL in FAQ

The bot currently gives this command to install pycord with slash commands:
pip install git+https://github.com/Pycord-Development/pycord@slash

This points at the wrong branch, as the branch is actually called feature/slash

The correct url is pip install git+https://github.com/Pycord-Development/pycord@feature/slash

this issue is with a different bot, but when the code from that bot is merged into this one, it will be resolved.


Update jsoniter-scala-core, ... to 2.13.13

Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.13.12 to 2.13.13.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #275.


Add explicit instruction to clone Zulip Repository

Added instruction for cloning the proper repository in order to make the "Installing Zulip Development Environment" more explicit & clear.

Automated message from Dropbox CLA bot
@lunacodes, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here and update the thread so we can consider merging your code.

Hi @smarx I've now signed the CLA - Thanks!!

Awesome, thanks for the contribution!  lgtm and merged!


Failing to create a oneview logical interconnect group with real networkUris

Scenario/Intent
Creating a oneview_logical_interconnect_group resource with networkUris without their network names.
Environment Details

Puppet module for HPE OneView Version: v2.9.0
OneView SDK Version: 5.17.0
OneView Appliance Version:  5.50.00-0426657
OneView Client API Version: 2200
Puppet Version: 5.5.17
Ruby Version: 2.4.5
Platform: Oracle Linux Server 7.7

Steps to Reproduce
Create a oneview_logical_interconnect_group resource with the following parameters. Its important to have real uris in the networkUris parameter.
oneviewconf::oneview::ethernet_networks:
  VLAN0002:
    ensure: present
    data:
      name: VLAN0002
      vlanId: 2
      purpose: General
      smartLink: false
      privateNetwork: false
  VLAN0003:
    ensure: present
    data:
      name: VLAN0003
      vlanId: 2
      purpose: General
      smartLink: false
    privateNetwork: false
  VLAN0141:
    ensure: present
    data:
      name: VLAN0141
      vlanId: 2
      purpose: General
      smartLink: false
      privateNetwork: false
  VLAN0023:
    ensure: present
    data:
      name: VLAN0023
      vlanId: 2
      purpose: General
      smartLink: false
      privateNetwork: false
  VLAN0142:
    ensure: present
    data:
      name: VLAN0142
      vlanId: 2
      purpose: General
      smartLink: false
      privateNetwork: false
  VLAN0143:
    ensure: present
    data:
      name: VLAN0143
      vlanId: 2
      purpose: General
      smartLink: false
      privateNetwork: false

oneviewconf::oneview::logical_interconnect_groups:
  LIG_TEST_NET_2:
    require: Oneview_ethernet_network[VLAN0002, VLAN0003, VLAN0141, VLAN0023, VLAN0142, VLAN0143]
    ensure: present
    data:
      name: LIG_TEST_NET_2
      enclosureType: SY12000
      enclosureIndexes:
      - 1
      - 2
      - 3
      interconnectBaySet: 3
      redundancyType: HighlyAvailable
      name: LIG_TEST_NET_2
      interconnects:
      - bay: 6
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 1
      - bay: 3
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 2
      - bay: 6
        type: Virtual Connect SE 40Gb F8 Module for Synergy
        enclosure_index: 2
      - bay: 3
        type: Virtual Connect SE 40Gb F8 Module for Synergy
        enclosure_index: 1
      - bay: 3
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 3
      - bay: 6
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 3
      uplinkSets:
      - networkType: Ethernet
        networkUris:
        - VLAN0003
        - VLAN0002
        mode: Auto
        name: Uplink_Set_Sync
        ethernetNetworkType: Tagged
        lacpTimer: Short
        uplink_ports:
        - bay: 3
          port: Q1:4
          enclosure_index: 1
          type: Virtual Connect SE 40Gb F8 Module for Synergy
        - bay: 6
          port: Q1:4
          enclosure_index: 2
          type: Virtual Connect SE 40Gb F8 Module for Synergy
      - networkType: Ethernet
        networkUris:
        - VLAN0141
        - VLAN0023
        - VLAN0142
        - VLAN0143
        mode: Auto
        name: Uplink_Set_Mgmt
        ethernetNetworkType: Tagged
        lacpTimer: Short
        uplink_ports:
        - bay: 3
          port: Q1:1
          enclosure_index: 1
          type: Virtual Connect SE 40Gb F8 Module for Synergy
        - bay: 6
          port: Q1:1
          enclosure_index: 2
          type: Virtual Connect SE 40Gb F8 Module for Synergy
  LIG_TEST_NET:
    require: Oneview_ethernet_network[VLAN0002, VLAN0003, VLAN0141, VLAN0023, VLAN0142, VLAN0143]
    ensure: present
    data:
      name: LIG_TEST_NET_2
      enclosureType: SY12000
      enclosureIndexes:
      - 1
      - 2
      - 3
      interconnectBaySet: 3
      redundancyType: HighlyAvailable
      name: LIG_TEST_NET
      interconnects:
      - bay: 6
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 1
      - bay: 3
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 2
      - bay: 6
        type: Virtual Connect SE 40Gb F8 Module for Synergy
        enclosure_index: 2
      - bay: 3
        type: Virtual Connect SE 40Gb F8 Module for Synergy
        enclosure_index: 1
      - bay: 3
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 3
      - bay: 6
        type: Synergy 20Gb Interconnect Link Module
        enclosure_index: 3
      uplinkSets:
      - networkType: Ethernet
        networkUris:
        - "/rest/ethernet-networks/991083c7-be11-44bc-94bc-a4d285ff1b78"
        - "/rest/ethernet-networks/6207cf55-e15d-4cd3-a10d-b649ec2b4fac"
        mode: Auto
        name: Uplink_Set_Sync
        ethernetNetworkType: Tagged
        lacpTimer: Short
        uplink_ports:
        - bay: 3
          port: Q1:4
          enclosure_index: 1
          type: Virtual Connect SE 40Gb F8 Module for Synergy
        - bay: 6
          port: Q1:4
          enclosure_index: 2
          type: Virtual Connect SE 40Gb F8 Module for Synergy
      - networkType: Ethernet
        networkUris:
        - "/rest/ethernet-networks/20e9c7bb-7384-4556-8dcc-e3d4842ed211"
        - "/rest/ethernet-networks/f7b7ef24-0a7f-42e6-950a-a599e4ad3184"
        - "/rest/ethernet-networks/812ca27c-4394-4b38-b47f-4b1b79f6095c"
        - "/rest/ethernet-networks/3aacc7b5-e1eb-4b71-b1d9-06e6394a42a6"
        mode: Auto
        name: Uplink_Set_Mgmt
        ethernetNetworkType: Tagged
        lacpTimer: Short
        uplink_ports:
        - bay: 3
          port: Q1:1
          enclosure_index: 1
          type: Virtual Connect SE 40Gb F8 Module for Synergy
        - bay: 6
          port: Q1:1
          enclosure_index: 2
          type: Virtual Connect SE 40Gb F8 Module for Synergy

Expected Result
Creating  two oneview_logical_interconnect_group resource with networkUris.
Actual Result
The oneview_logical_interconnect_group named LIG_TEST_NET_2 with network names in the networkUris parameter will create normally. See the following picture.

But the oneview_logical_interconnect_group named LIG_TEST_NET with network uris in the networkUris parameter will create without any network. See the following picture.

Quickfix and Guess
When you look in the oneview_logical_interconnect_group provider you can see that everytime a real uri is used it will skip that element in the array of the networkUris parameter and all uris are deleted.
https://github.com/HewlettPackard/oneview-puppet/blob/6eedd764f39ca8fdc4fbf27606e36167f4289d82/lib/puppet/provider/oneview_logical_interconnect_group/c7000.rb#L112
To get it also working with uris  you can replace the above line with the following. Here it adds the uri element first and skips after that.
      if network_uri.to_s[0..6].include?('/rest/')
        net = net_class.new(@client, uri: network_uri)
        uplink_set.add_network(net)
        next
      end

Hi @riconem , Thanks for pointing us to this issue by adding your analysis and also providing the solution helps us a lot. We will add these changes and raise a PR soon.

Hi @riconem , Thanks for pointing us to this issue by adding your analysis and also providing the solution helps us a lot. We will add these changes and raise a PR soon.