fix(cilium): add taint and install cilium first - WIP

This is a WIP PR to fix #270 .
I've just had a few minutes to look at this so far so please consider this as me sharing my idea.
When I've had more time to test and finalize, I'll mark this ready for review.
However, please do not hesitate to edit if someone else finds time earlier than me 👍 .
I've added ToDo comments where I left off.
Edit: Done force pushing, sorry for that.

Very good start @t1murl, thank you :) Just would love you to keep the HelmChart and values logic, as it's ideal IMHO. If not, why?

@mysticaltech , the k3s HelmChart controller spawns a pod to perform the helm chart installation. If we taint the node with the cilium NoExecute the aforementioned pod does not tolerate the taint and will be stuck pending, leading to the chart never being installed. Please note that with my current draft you do not loose the customization through values option.

Also just noticed the k3s docs stating:

K3s agents can be configured with the options --node-label and --node-taint which adds a label and taint to the kubelet. The two options only add labels and/or taints at registration time, so they can only be added once and not changed after that again by running K3s commands.

It seems like the taint is not re-applied after a node reboot, potentially leaving the initial problem description unresolved by what I've done so far.
I'll look up what we may be able to do with kured, i.e. before it issues the reboot it should taint the node to extend the current draft.

Got it! We can execute arbitrary commands with Kured. So maybe a custom bash would help resolve the situation?!

Or a startup script for Cilium nodes, MicroOS offers that kind of functionality easily.

@mysticaltech , that also sounds promising! I am not really familiar with MicroOS.. I reckon you mean this? Any help on this is more than welcome :)

@t1murl Combustion or Ignition are for first boot. Am thinking more in terms of https://www.suse.com/c/easy-running-scripts-boot-and-shutdown/

FYI: Tainting the node while cilium is still running will result in cilium instantly deleting the taint again. I feel like we might want to taint the node when it is in a state where cilium is not running (anymore) and before pods start coming up.

@t1murl Combustion or Ignition are for first boot. Am thinking more in terms of https://www.suse.com/c/easy-running-scripts-boot-and-shutdown/

@mysticaltech From quickly reading into this, this indeed sounds great to me for correctly timing the tainting! Very good idea in my opinion 🙂👍
I feel like we should try this. Only need to find a neat way to run e.g. kubectl taint node $(hostname) "node.cilium.io/agent-not-ready:NoExecute" or the respective API call. Currently there does not seem to be a kubeconfig ready to use on worker nodes.

Perfect!! Yep, sounds good 😊
About kubeconfig, it is already being downloaded from the first control plane node, if I remember correctly in init.tf, so we could reupload that to worker nodes for instance!

@t1murl I just came back from a break; I'll try to help with this ASAP.

Let me reflect back my understanding of the problem here.
What we're trying to avoid containers that are not managed by the Cillium i.e. they were launched before cillium was fully initialised? From what I understand Cillium uses two techniques for this, firstly you can use this taint, or secondly you can restart the pods. We probably don't want to be randomly restarting pods, so the taint is a nicer option.
However adding this taint means that the HelmController won't schedule the pod because the tolerations for bootstrap: true do not include the special taint. So we need a way to get the helm chart to run.
The approach here is to use the helm CLI rather than the controller. Other approaches might include, adding a pod to add the toleration to the controller, to use the cilium cli (which is basically the same at the helm chart), or we could render using helm template somewhere in this module

@PurpleBooth I did not realize that the Helm controller was in play even after Cilium is installed. Good catch!
@t1murl Does that make sense to you?

@PurpleBooth I did not realize that the Helm controller was in play even after Cilium is installed. Good catch!
@t1murl Does that make sense to you?

@PurpleBooth , thanks for sharing your reflection on this problem and thanks for opening https://github.com/k3s-io/helm-controller/issues/151! 👍
I feel this will help to understand if / how we can use the HelmController to install the Cilium chart instead of my proposal using plain helm + kubectl apply (or one of the other approaches you mentioned that avoids the HelmController) when tainting the nodes.
Please note that with the proposal in this MR, the initial deployment is working as expected (simply by installing Cilium before anything else).
We still have to implement something that accounts for node reboots after the initial deployment, i.e. the problem description I have provided in https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/issues/270 where I've described how pods become unmanaged by Cilium after a node reboot by e.g. kured because they have been scheduled already and start faster than the Cilium pods when the nodes comes back up.
To account for this, my understanding of a potentially feasible solution would be to use @mysticaltech proposal mentioned in https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/pull/275#issuecomment-1214331320 to realize tainting the node again when a reboot occurs (also see https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/pull/275#issuecomment-1214331757 regarding the timing & https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/pull/275#issuecomment-1214352160 ). Of course this only accounts for a 'safe/regular' node reboot and not power loss or unexpected shutdowns that bypass @mysticaltech proposed solution of a shutdown script - which I would find acceptable.
cc: @mysticaltech
PS: Unfortunately even my weekends are busy currently (birthdays & weddings to attend) so more days will pass until I can personally try out @mysticaltech proposal. Happy to hear you are back @mysticaltech to assist with this! 👍

@t1murl Thanks for clarifying! Helps a lots. As soon as any of us starts working on this, let's signal it here, so that we do not overlap on the same things. I'll do my best ASAP, if @PurpleBooth doesn't get this sorted before (she tends to be fast when she starts!) And looking forward for a reply to the issue she opened.

For restarts we could do use Kured https://github.com/weaveworks/kured#adding-node-labels-before-and-after-reboots

Excellent find, @PurpleBooth; That's another obstacle out of the way!!

I believe it's now worth tweaking this PR and giving it a try as is. Maybe the Cilium containers will start after all. And if they successfully remove the label, that's it unless I do not see things as clearly as you. It's also important to remember that not all nodes restart simultaneously, and in an HA setting, at least two CP nodes would already have been bootstrapped and Cilium running on them. @PurpleBooth What do you think?

@PurpleBooth , @mysticaltech to avoid confusion: node labels != node taints ? :O

@t1murl My bad, I was confusing the two!

@t1murl @PurpleBooth If the Cilium cli can solve our issues, then let's move away from the helm controller if needed. We can install it on the CP nodes at the moment when the k3s SELinux RPM is installed so that no additional reboots are needed (because of the transactional system).

And I have verified how to run startup scripts, it's basically via systemd, as explained here https://www.golinuxcloud.com/run-script-at-startup-boot-without-cron-linux/.

Good news folks, there is a good chances that custom tolerations can be specified via the helm chart, in our case via the HelmChart definition. See this example for Longhorn, and other Rancher project do the same. https://github.com/longhorn/longhorn/issues/2664

Last but not least, various tolerions that Cilium support, as also supporting by the helm controller jobs. https://github.com/k3s-io/helm-controller/search?q=toleration&type=commits

@t1murl I will close this as I will redo most, to keep the current HelmChart method. But it's good inspiration, thank you!

Changed my mind, merging instead so I have you in the git histor @t1murl, it's very important to me as you've helped so much 🙏. I will compare from master and do the required changes.


Update TypeScript-Design-Goals.md

Thanks for the PR @mattratliff-xc  - I think we might keep the language as-is for now, but I appreciate you taking the time to help improve the docs!


Safer page length

When using the Generate Text option some lines (pages) cannot be pasted into the book because they are too long. Setting Number of lines per page under Advanced to 13 solves this issue but is not elegant and most pages will have an empty line at the bottom of the page.
I think an simpler solution might be to just subtract 1 word from each page and move it to the next page. This should make the page length safer to paste.
Or (please don't do this unless you hate yourself), measure the width of each character (and special characters the book supports) and calculate "exactly" how long each word/line will be.

All the available glyphs are already measured trough the MGD project , and this is how the program calculates how many words fit in a page. I've made many attempts now at making it fit perfectly, but there is some piece of the puzzle I'm missing. In any case I like your idea of removing a word, hadn't thought about that. However, seeing as we already have the widths available the goal would be to perfect the algorithm first, although I could also add this as a fail safe option.
For reference, see the code, #10 and #11

I found a pixel issue that is probably the result of the MGD project. The space is labeled as 4 pixels when it is actually 3 pixels.
The < char is also labeled as 4 pixels but you can see in the image below that that line is longer than the line with a space. The I char is labeled as 3 pixels and you can see that it is the same length as the line with the space.

I am not sure how to help with #10 at the moment. I'll continue looking at the code to see what can be done.

I think this is due to the spacing between characters. All characters except the space adds a single pixel spacing after it, so a characters with a width of 5 pixels is in practice 6 pixels. The space is 4 pixels wide but has no additional spacing. Thus, we can go trough each row like this (w = width, s  = spacing):
ROW 1
=====
i + space + i
(1w + 1s) + 4w + (1w + 1s) = 8px

ROW 2
=====
i + < + i
(1w + 1s) + (4w + 1s) + (1w + 1s) = 9px

ROW 3
=====
i + I + i
(1w + 1s) + (3w + 1s) + (1w + 1s) = 8px

This is were the additional pixel offset comes from in row 2. The code takes this into consideration:
https://github.com/TheWilley/Text2Book/blob/1bf9643b45e1b05738cae3e8ecb9db8bb9302668/src/utils/MinecraftBook.ts#L86-L90
In addition, the space width is defined within the spacing provider, which is set to 4. The space length is probably not an issue.


[SPARK-31882][WEBUI] DAG-viz is not rendered correctly with pagination

What changes were proposed in this pull request?

This PR fix an issue related to DAG-viz.
Because DAG-viz for a job fetches link urls for each stage from the stage table, rendering can fail with pagination.
You can reproduce this issue with the following operation.
sc.parallelize(1 to 10).map(value => (value ,value)).repartition(1).repartition(1).repartition(1).reduceByKey(_ + _).collect

And then, visit the corresponding job page.
There are 5 stages so show <5 stages in the paged table.

Why are the changes needed?

This is a bug.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Newly added test case with following command.
build/sbt -Dtest.default.exclude.tags= -Dspark.test.webdriver.chrome.driver=/path/to/chromedriver "testOnly org.apache.spark.ui.ChromeUISeleniumSuite -- -z SPARK-31882"

Test build #123360 has started for PR 28690 at commit 208cffc.

Merged build finished. Test PASSed.

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

Test build #123360 has finished for PR 28690 at commit 208cffc.

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

Merged build finished. Test PASSed.

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

Looks okay. cc @gengliangwang FYI

@sarutak Does the bug exist in branch 3.0?

@gengliangwang I've confirmed this bug exists in branch 3.0 too.

Then, let's merge to both master and 3.0 :)

Yeah, but #28627 isn't merged to branch-3.0 so we need to back port it first or open another PR for branch-3.0 to back port this change without the testcase.

Anyway, I'll open a PR for backporting #28627.

Awesome, let's merge this one after #28627 is backported 👍

Thanks, merging to master/3.0


GitHub Workflows security hardening

This PR adds explicit permissions section to workflows. This is a security best practice because by default workflows run with extended set of permissions (except from on: pull_request from external forks). By specifying any permission explicitly all others are set to none. By using the principle of least privilege the damage a compromised workflow can do (because of an injection or compromised third party tool or action) is restricted.
It is recommended to have most strict permissions on the top level and grant write permissions on job level case by case.

Thanks! The formatting and commit hygiene still needed some work, so I just went ahead and did that myself.


Problem with ubuntu xenial repository

E: The repository 'http://repo.radeon.com/rocm/apt/debian xenial Release' does not have a Release file.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Problem was:
http://repo.radeon.com/rocm/apt/debian should be replaced by
http://repo.radeon.com/rocm/apt/debian/debian


Discrepancy between Android and iOS getTag result

I'm getting different results when calling getTag on Android vs iOS. Is there any obvious reason behind this? Issue seems related/identical to #450.
Code:
await NfcManager.requestTechnology(NfcTech.Ndef)
const tag = await NfcManager.getTag()

Android:
{
  "canMakeReadOnly":true,
  "id":"046C70F2CB7281",
  "isWritable":true,
  "maxSize":137,
  "ndefMessage":[
    ...
  ],
  "techTypes":[
     "android.nfc.tech.NfcA",
     "android.nfc.tech.MifareUltralight",
     "android.nfc.tech.Ndef"
  ],
  "type":"NFC Forum Type 2"
}

iOS:
{
  "id":"046C70F2CB7281",
  "ndefMessage":[
    ...
  ],
  "tech":"mifare"
}

Expected result:
getTag: () => Promise<TagEvent | null>

...

export interface TagEvent {
  ndefMessage: NdefRecord[];
  maxSize?: number;
  type?: string;
  techTypes?: string[];
  id?: string;
}

Debug info:
react-native: 0.70.2
react-native-nfc-manager: 3.14.1

Hi @erikostling great question!
Regarding the ugly type TagEvent, it's mainly because:

iOS and Android handle things differently

they report different fields from native layer
some fields even requires extra API calls

different NFC technologies have different fields

So rather than putting efforts to make all fields identical in both OS, I only focused on following ones (if they present):

id: means the NFC tag's physical identifier
ndefMessage: the NDEF contained by scanned NFC tag

It's simply because In my perspective, id and ndefMessage are the most important fields for most use cases.

why can't I get the ID in the card when I read it from iOS?

how to get a tag id on ios. an empty array is now returned. although everything is fine on Android


Compact compiler buttons

Requesting a PR from Jared as this touches static and I don't want to cause any upset :)

looks ok to me (although js solutions to css problems make me twitch :D)
At this stage just anything modifying includes will hurt.

Right! If you know of a way to solve this in pure CSS I'm all ears!

for css you could find the point at which things break, record the screen width in devtools. Use a media query to change at that width .hideable { display: none }

@jaredwy that works for the screen size being that small. This monitors the size of a particular pane in the view, and if it's too small, hides the long elements. Unless it would spill onto two lines anyway, in which case there's no benefit to making it smaller... that's why the JS is there. TBH if I could just hide if an element was <= some size I'd do that, but as I had to crack out a JS solution anyway I thought I'd try and be "clever".

Ah, so there isn't a correlation between screen size and pane size?

Only that pane size <= screen size. The panes are the resizable panels (the editor, the compiler) so their size can be changed by resizing the whole browser window, or by dragging the edges of the panes to resize them.

👍 lgtm

Thanks :grinning: - always good to have this kind of feedback!


Revert Blazor circular references content

Remove the Avoid circular object references section when engineering has resolved the issue here and in the Call .NET from JS topic.
Tracking:

Circular references are not supported, take two (dotnet/aspnetcore #20525)
Proposal: Add mechanism to handle circular references when serializing (dotnet/runtime #30820)

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

ID: aada0049-99a8-2f91-5d60-6970474e2371
Version Independent ID: c29fb1a8-028f-6efd-f1bc-e889fb362311
Content: Call JavaScript functions from .NET methods in ASP.NET Core Blazor
Content Source: aspnetcore/blazor/call-javascript-from-dotnet.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

This looks like it's going to be on the backlog in engineering for quite some time. We'll open a new issue if/when they take it on. Closing for now.

This has gone forward for 5.0 release.

https://docs.microsoft.com/en-us/aspnet/core/blazor/call-javascript-from-dotnet?view=aspnetcore-3.1#avoid-circular-object-references
::: moniker range="< aspnetcore-5.0"

## Avoid circular object references

In versions of .NET Core prior to 5.0, objects that contain circular references can't be serialized on the client for either:

* .NET method calls.
* JavaScript method calls from C# when the return type has circular references.

For more information, see the following issues:

* [Circular references are not supported, take two (dotnet/aspnetcore #20525)](https://github.com/dotnet/aspnetcore/issues/20525)
* [Proposal: Add mechanism to handle circular references when serializing (dotnet/runtime #30820)](https://github.com/dotnet/runtime/issues/30820)

::: moniker-end


Use skypeforlinux

Self explanatory, there's a new version of skype for linux that's a wrapper around the web version with electron. Allows for more features. https://wiki.archlinux.org/index.php/skype#Skype_for_Linux_Alpha

Have created a separated project for Skype for Linux (alpha) https://github.com/sameersbn/docker-skype. Please help test it.


Is plan to asynchronously render features?

As you already know, if the number of features or the execution time of style function increases, there is a performance issue because they are interpreted synchronously.

https://openlayers.org/en/latest/examples/synthetic-points.html?q=points

So I wonder does ol plan to asynchronously render features.

In version 6, instruction groups for vector tile layers are created asynchronously already. That was a major refactoring. For vector layers, this would also be possible, but a huge effort.

@ahocevar That's good news for me. I'll find things to contribute


Not get status Pull Request

Please update get PR, currently I see your project only get status of branch.
/dashboard?id=repo&branch=develop
I am using PR so I can't get report, can you add get PR, the URL will be like this
dashboard?id=repo&pullRequest=69.
Thanks,

Hi @xuanan93! You can use branch input to specify the branch that this action will get the quality status from, you can configure the action similar to the below:
jobs:
  sonarqube:
    runs-on: ubuntu-latest
    steps:
      - uses: phwt/sonarqube-quality-gate-action@v1
        id: quality-gate-check
        with:
          sonar-project-key: ${{ secrets.SONAR_PROJECT_KEY }}
          sonar-host-url: ${{ secrets.SONAR_HOST_URL }}
          sonar-token: ${{ secrets.SONAR_TOKEN }}
          github-token: ${{ secrets.GITHUB_TOKEN }}
+         branch: main

In the background, this action will call the GET /api/qualitygates/project_status endpoint to retrieve the quality gate status, if the branch input is specified, the action will append the branch parameter to this endpoint, similar to the below:
GET /api/qualitygates/project_status?projectKey=your-project-key&branch=your-branch

For reference, the request to SonarQube API is handled in src/modules/sonarqube-api.ts file.

Hi again @xuanan93, I was sleepy and read your issue wrong, the support for pull request will be released with #30


Blank screen

First of, thank you so much for your images, I've been running them on our server!
Lately I've been having an issue with the g3 images. When I connect to it I get a blank screen and I cannot see the desktop environment.
sudo docker run --rm -p 6901:6901 accetto/ubuntu-vnc-xfce-g3
This does not happen when I use an old legacy image.
sudo docker run --rm -p 6901:6901 accetto/ubuntu-vnc-xfce
I would really appreciate it if you can help me with this matter.  Thanks in advance,
Ale

Hello @ale-sc,
thank you for your feedback. I didn't have such a behaviour but I'll try to give a few tips that could help you.
I see that you try to use the noVNC protocol, so I assume, that you access the container via a web browser. So this the first thing you can try to change. You can try a different web browser.
The next thing I would recommend, ist to use a different TCP port on the host instead of 6901. Try something like this:
docker run --rm -p 25000:6901 accetto/ubuntu-vnc-xfce-g3

Then point your web browser to http://localhost:25000 and click the Light or Full noVNC client hyperlink. Change the localhost to your server's name if you do not work on that server locally.
I've also noticed that you start your docker as sudo. You can try to configure Docker so that it doesn't require sudo, as it's described on this page - Linux post-installation steps for Docker Engine.
You can also try to acces the container with a VNC Viewer, e.g. TigerVNC Viewer. Creare the container as
docker run --rm -p 25001:5901 accetto/ubuntu-vnc-xfce-g3

and point the VNC Viewer to :25001.
You can also get more information for troubleshooting if you start the container with the --debug option like this:
docker run --rm -p 25000:6901 -p 25001:5901 accetto/ubuntu-vnc-xfce-g3 --debug

One of the reasons for black screens can be also that, the the display :1 is already in use. The image full readme file describes how you can override the VNC related parameters. However, you can override the display number also during container start, like this:
docker run --rm -p 25000:6901 -p 25001:5901 -e DISPLAY=:2 accetto/ubuntu-vnc-xfce-g3 --debug

Hopefully it'll help you to pinpoint the problem.
Regards,
accetto

Dear accetto,
thank you for your taking the time to answer my question!
I've tried your suggestions but it did not work. In all cases when running the image ubuntu-vnc-xfce-firefox-g3 after connecting with novnc or a vnc viewer I get a blank screen. I've changed the display port and the port number.
I want to point out that if I use the image debian-vnc-xfce-firefox-g3 everything works out perfectly. I just wanted to figure out why isn't it working to avoid that this happens in the future also for those images that now are working.
Additional info.
The output of the debug is the same for the two aforementioned machines and this is the output of the vnc.log file for the machine that is showing the blank screen
headless@6b3721b68df7:~$ cat /dockerstartup/vnc.log
Using desktop session xfce
xauth:  file /home/headless/.Xauthority does not exist

New '6b3721b68df7:2 ()' desktop is 6b3721b68df7:2

Starting desktop session xfce
xinit /etc/X11/Xsession startxfce4 -- /usr/bin/Xvnc :2 -depth 24 -geometry 1360x768 -rfbport 5901 -auth /home/headless/.Xauthority -desktop 6b3721b68df7:2 () -pn -rfbauth /home/headless/.vnc/passwd
_XSERVTransmkdir: ERROR: euid != 0,directory /tmp/.X11-unix will not be created.

Xvnc TigerVNC 1.13.0 - built Feb  3 2023 08:34:05
Copyright (C) 1999-2022 TigerVNC Team and many others (see README.rst)
See https://www.tigervnc.org for information on TigerVNC.
Underlying X server release 12013000

Fri Mar  3 09:02:30 2023
 vncext:      VNC extension running!
 vncext:      Listening for VNC connections on all interface(s), port 5901
 vncext:      created VNC server for screen 0
xinit: XFree86_VT property unexpectedly has 0 items instead of 1

Fri Mar  3 09:04:08 2023
 Connections: accepted: <IP_ADDRESS>::57116
 SConnection: Client needs protocol version 3.8
 SConnection: Client requests security type VeNCrypt(19)
 SVeNCrypt:   Client requests security type VncAuth (2)

Fri Mar  3 09:04:11 2023
 VNCSConnST:  Server default pixel format depth 24 (32bpp) little-endian rgb888
 VNCSConnST:  Client pixel format depth 24 (32bpp) little-endian bgr888
 ComparingUpdateTracker: 0 pixels in / 0 pixels out
 ComparingUpdateTracker: (1:-nan ratio)

while this is the output of the xsession-errors
headless@6b3721b68df7:~$ cat .xsession-errors
Xsession: X session started for  at Fri Mar  3 09:02:30 UTC 2023
localuser:headless being added to access control list
dbus-update-activation-environment: systemd --user not found, ignoring --systemd argument
dbus-update-activation-environment: setting NOVNC_HOME=/usr/libexec/noVNCdim
dbus-update-activation-environment: setting HEADLESS_USER_NAME=headless
dbus-update-activation-environment: setting FEATURES_FIREFOX_PLUS=1
dbus-update-activation-environment: setting FEATURES_SCREENSHOOTING=
dbus-update-activation-environment: setting HOSTNAME=6b3721b68df7
dbus-update-activation-environment: setting SHLVL=0
dbus-update-activation-environment: setting HOME=/home/headless
dbus-update-activation-environment: setting DESKTOP_SESSION=xfce
dbus-update-activation-environment: setting STARTUPDIR=/dockerstartup
dbus-update-activation-environment: setting HEADLESS_USER_ID=1000
dbus-update-activation-environment: setting FEATURES_BUILD_SLIM_TOOLS=1
dbus-update-activation-environment: setting FEATURES_BUILD_SLIM_XSERVER=1
dbus-update-activation-environment: setting DBUS_SESSION_BUS_ADDRESS=unix:abstract=/tmp/dbus-eOdHEHcJdk,guid=e6c0c69054e0ea7284fab1876401b7a6
dbus-update-activation-environment: setting FEATURES_FIREFOX=1
dbus-update-activation-environment: setting VNC_COL_DEPTH=24
dbus-update-activation-environment: setting VNC_VIEW_ONLY=false
dbus-update-activation-environment: setting _=/usr/bin/vncserver
dbus-update-activation-environment: setting FEATURES_BUILD_SLIM_XFCE=1
dbus-update-activation-environment: setting PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
dbus-update-activation-environment: setting FEATURES_BUILD_SLIM_FIREFOX=1
dbus-update-activation-environment: setting DISPLAY=:2
dbus-update-activation-environment: setting HEADLESS_USER_GROUP_NAME=headless
dbus-update-activation-environment: setting XDG_CURRENT_DESKTOP=XFCE
dbus-update-activation-environment: setting FEATURES_VNC=1
dbus-update-activation-environment: setting XDG_SESSION_DESKTOP=xfce
dbus-update-activation-environment: setting XAUTHORITY=/home/headless/.Xauthority
dbus-update-activation-environment: setting FEATURES_NOVNC=1
dbus-update-activation-environment: setting NO_AT_BRIDGE=1
dbus-update-activation-environment: setting NOVNC_PORT=6901
dbus-update-activation-environment: setting GDMSESSION=xfce
dbus-update-activation-environment: setting FEATURES_THUMBNAILING=
dbus-update-activation-environment: setting GPG_AGENT_INFO=/home/headless/.gnupg/S.gpg-agent:0:1
dbus-update-activation-environment: setting HEADLESS_USER_GROUP_ID=1000
dbus-update-activation-environment: setting PWD=/home/headless
dbus-update-activation-environment: setting VNC_RESOLUTION=1360x768
dbus-update-activation-environment: setting XDG_DATA_DIRS=/usr/share/xfce4:/usr/local/share/:/usr/share/
dbus-update-activation-environment: setting XDG_CONFIG_DIRS=/etc/xdg/xdg-xfce:/etc/xdg
dbus-update-activation-environment: setting FEATURES_BUILD_SLIM_NOVNC=1
dbus-update-activation-environment: setting FEATURES_VERSION_STICKER=1
dbus-update-activation-environment: setting VNC_PW=headless
dbus-update-activation-environment: setting VNC_PORT=5901
/usr/bin/startxfce4: X server already running on display :2
_IceTransmkdir: ERROR: euid != 0,directory /tmp/.ICE-unix will not be created.
xfce4-session: No SSH authentication agent found

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.280: Failed to spawn gpg-agent: Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.292: Unable to launch "xfwm4": Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.295: Unable to launch "xfsettingsd": Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.298: Unable to launch "xfce4-panel": Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.301: Unable to launch "Thunar": Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.304: Unable to launch "xfdesktop": Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.308: Unable to launch "xdg-user-dirs-update" (specified by autostart/xdg-user-dirs.desktop): Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:56): xfce4-session-WARNING **: 09:02:31.311: Unable to launch "xfsettingsd" (specified by autostart/xfsettingsd.desktop): Failed to close file descriptor for child process (Operation not permitted)

I've just published Release 23.0 with TigerVNC 1.13.1 bugfix release.
Please let me know if it has solved your issue.

I've tested it and got the same results,
the ubuntu image does not work, while the debian works fine!
Yes the xfce manager does not start on the ubuntu one, here is the output of pstree
headless@0b39ca24d68d:~$ pstree
tini-+-dbus-daemon
     |-dbus-launch
     |-startup.sh-+-bash---python3---python3
     |            |-bash---pstree
     |            `-xinit-+-Xvnc
     |                    `-xfce4-session---3*[{xfce4-session}]
     `-xfconfd---3*[{xfconfd}]

also I am not binging anything here is the exact command that I use to start both images
sudo docker run -it --rm -p 20000:6901 -e DISPLAY=:2 accetto/ubuntu-vnc-xfce-firefox-g3 --debug bash
I also tried the --user 0:0 option but no luck, the outcome is always the same, the debian image works while the ubuntu one does not.
thanks for your help!
Ale

Hello @ale-sc,
it's really very strange, but the Xfce4 stuff is not starting for some reason. Unfortunatelly I'm still not able to reproduce the issue.
I've just uploaded a special temporary image accetto/ubuntu-vnc-xfce-g3:debug-issue-36. I've excluded the pre-configuration files and set the user to root (0:0). Otherwise it's the latest image based on Ubuntu 22.04 LTS. Can you try it please?
Btw, have you the same issue with the image based on Ubuntu 20.04 LTS?
Can you also try to start the container without using sudo?
Regards,
accetto

Hello @ale-sc ,
those are the images with the 20.04 tags. For example, accetto/ubuntu-vnc-xfce-g3:20.04 is based on Ubuntu 20.04 LTS.
In between I was able to reproduce similar behaviour while working on other issues.
However, it happens only if I bind the complete directory /home/headless on Windows.
Are you also on WIndows and do you bind the complete /home/headless directory?
Nevertheless, I'm marking this issue as a bug and I'm already working on a fix.
Regards,
accetto

Hi, I was seeing the same behavior with a black/blank screen in noVNC running accetto/ubuntu-vnc-xfce-firefox-g3:latest  on a swarm node (Ubuntu Linux server), I tested it on a Windows WSL2 Docker client and it worked. I then tried  accetto/ubuntu-vnc-xfce-firefox-g3:20.04 on the server and it worked, so something weird going on with the 22.04 release it seems.

Hi, I've just updated the temporary image for debugging - accetto/ubuntu-vnc-xfce-g3:debug-issue-36-v2.
Can you test it please and let me know if it has helped?
Please let me also know, if you're on Windows and if you've bound the complete folder home/headless.

Hi,
ubuntu 20.04 works fine, unfortunately your last image does not. I run my images from an ubuntu server.
It seems that there are some issues with the x-system here is a portion of the .xsession-errors file
/usr/bin/startxfce4: X server already running on display :2
_IceTransmkdir: ERROR: euid != 0,directory /tmp/.ICE-unix will not be created.
xfce4-session: No SSH authentication agent found

(xfce4-session:61): xfce4-session-WARNING **: 19:49:47.591: Failed to spawn gpg-agent: Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:61): xfce4-session-WARNING **: 19:49:47.606: Unable to launch "xfwm4": Failed to close file descriptor for child process (Operation not permitted)

(xfce4-session:61): xfce4-session-WARNING **: 19:49:47.610: Unable to launch "xfsettingsd": Failed to close file descriptor for child process (Operation not permitted)

Interestingly, part of the x-system works, for example if I start thunar from the terminal I can see it through my browser but the window manager itself is not starting, here is a screenshot of the running thunar

Ale

FYI: I've moved the image accetto/ubuntu-vnc-xfce-g3:debug-issue-36-v2 to the development repository as accetto/headless-ubuntu-g3:debug-issue-36-v2.
I've also developed a similar image that fixes the behaviour on my environment.
It would be nice if you could test it, if it helps - accetto/headless-drawing-g3:debug-issue-37

Hi there,
this image has the same behavior as well.
About the permissions of the home folder.
I will try configure docker without sudo in the next days and see if that solves the issue
Ale

In the answer to the issue #37 I've described, that the reason for the black screen or a partially initialized desktop is the missing package libgl1 in the image accetto/ubuntu-vnc-xfce-g3. However, there was a binding of the whole $HOME folder involved. I was also able to reproduce the behaviour in such cases.
The image accetto/ubuntu-vnc-xfce-opengl-g3 already includes the package libgl1.
However, I'm not able to reproduce your issue.
If you'll closer describe your environment, I could try to install a similar machine. Maybe I'll be able to reproduce it then.

I'm closing this issue because I'm not able to reproduce the behaviour and the communication has staled.

I am running into the same issue.
I see mentioning a possible DISPLAY conflict. I do have a container running with host privileges, but it's on display 99. Given the above isn't passing in any additional permissions, I'd be really surprised if that was the conflict.


[WMI] Incorrect color of text literal in sql editor

Description
The text 'Manual' is highlighted with wrong color.

create connection to WMI
use code:

select *
from Win32_Service
where StartMode = 'Manual'

Expected: color is "SQL string color"
Actual: color is not "SQL string color"

DBeaver Version
Enterprise Edition 24.2.2-202409192322
Operating System
Windows 10
Database and driver
WMI
Steps to reproduce
No response
Additional context
No response

Thanks for the bug report


Reorder notes within board view

As a moderator I want to reorder notes within the board view by drag & drop.

[ ] On drag & drop, an indicator should appear at the current drop position (e.g. note-sized placeholder)

We should experiment with animations here and choose what's best.
JUST LIKE THIS EXAMPLE!

@Kraft16 is this design still up-to-date? :)

@CronJorian no, this design is not relevant.

I think this ticket is blocked until #1859 has been completed. We first have to research and decide which sorting method fits our needs best.


React sample and demo

Would be great have a react sample with this feature and a demo to test it.
Good job for the repo! Im going to try it

Hello, Thank you, I never used React so I don't plan to learn React for that. PR welcomed :)

@acemtp ok im going to try it, the sample link http://passwordless.meteor.com/ its down

would be easy send a link to log in the user? like slack


New Release into master

Fixes a few minor issues and support for other chains.

LGTM

@RoshanSyed : The changes are not merged into master


Embeds Weibenfalk video on Final Space API

Fixes #135
This PR adds the Weibenfalk video on Final Space API

Hey @adityatandon007
I have reviewed this PR and left comments
Please update it accordingly and let me know when you are done.
Thanks

@lelouchB couldn't find any comments on the commit. Can you mention me what needs to be updated

@lelouchB couldn't find any comments on the commit. Can you mention me what needs to be updated

Go under Files Changed section of the PR

https://github.com/lelouchB/final-space-api/pull/139/files

Have you found them?
Also, it would really help this project if you could star it.
Thanks

@lelouchB couldn't find your comments on the file

How about now?

@lelouchB updated the pr. Hope it is fine now

@adityatandon007
Awesome
I will review and merge it sometime tommorow.
In the meantime I will approve this PR
Great work 👏
Thanks

Hey @adityatandon007
You can embed videos using iframe, I have updated your file changed.
See it live here https://final-space-api.vercel.app/docs/slider
Thanks for your valuable contribution

@lelouchB yup I knew about that iframe implementation of embedding videos, but it is generally not the recommended way to embed videos in the .md files. I read that on stackoverflow because different markdown parser may render it in a different way, but thanks again for adding that iframe implementation. It still is better than the image over link thing 👍

Oh okay
Sorry I didn't knew that
I checked the PageSpeed and other stuff and it looked fine to me.
Making it responsive was a struggle.

@lelouchB can you add hacktoberfest accepted label on this PR. It says the invalid repository (repository not participating in hactobefest) but if you add hactoberfest accepted label on your PR then this might be considered eligible for their submission process.

Ofcourse
I removed the label from the repo because I thought the Hacktoberfest was over.
Sorry.


PHP Namespaces usage

Library class names should be refactored to use PHP namespaces.
Namespaces were introduced at PHP 5.3.
Swiftmailer 5.x requires PHP >= 5.3:
https://github.com/swiftmailer/swiftmailer/blob/v5.4.8/composer.json#L18
Swiftmailer 6.x requires PHP >= 7.0:
https://github.com/swiftmailer/swiftmailer/blob/v6.0.0/composer.json#L18
Pros:

a PHP library without namespaces looks outdated as namespace usage considered a practice, which good library should follow
using namespaces you can rely on Composer autoloader like ‘PSR-4’, making such files as swift_required.php to be redundant
using namespaced based class autoloading (e.g. ‘PSR-4’) will increase performance comparing with current custom autoload implementation.

For the namespace root swiftmailer or swift can be used. Thus definition of class Swift will look like following:
namespace swiftmailer;

abstract class Swift
{
    // …
}

Definition of Swift_Transport_SendmailTransport will look like:

namespace swiftmailer\transport;

class Sendmail extends AbstractSmtp
{
    // …
}

To keep BC and support migration to the namespace usage, you can create aliases for the old class names for the most-used classes. For example:
class Swift_Transport_SendmailTransport extends \swiftmailer\transport\Sendmail
{
}

Even one of the most conservative libraries – PHPUnit – has already migrated to namespaces usage.
It would be a shameful if swiftmailer 7.x will not.

This was mentioned in #566 and was rejected by owner see https://github.com/swiftmailer/swiftmailer/issues/566#issuecomment-302747726 .

Shameful...
:disappointed:

@fabpot as this was done for Twig can we reconsider this for SwiftMailer?

I think it worth it. Plus, it will fix bench of weird issue with custom auto-loading such as this one: https://github.com/maglnet/ComposerRequireChecker/issues/55#issuecomment-370135553

This is frustrating !
How can I use SwiftMailer from a custom class which is namespaced (like entire project) ?
I cannot make it recognize the class / methods from Swift

Closing as the namespaced version of Swiftmailer is now available... as Symfony Mailer.


WIP: Runtime annotations

This is a first cut at allowing the use of annotations to specify which tests to run at run-time as was considered in #17.  The code needs some clean-up, so consider this an initial proposal that is very open to everyone's input.
The gist is this:

Put an annotation (e.g. @smoke) on a feature or scenario in the .feature file.
Specify which annotations to run in config/environment.js in the appropriate environment section.  This could be made more flexible by passing in an arg on ember t or using environment variables to specify the annotations to run.
The acceptance test runner calls a user defined module (e.g. yadda-run-test-helper.js), passing it the annotations on the feature or scenario.  The user defined module has the logic to compare the annotations on the test to the requested annotations in the configuration and returns true/false.  If true, the acceptance test runner runs the test; if false it does not run the test.
The user defined module is configurable in ember-cli-build.js.  This configuration is used by the feature parser to import the module if it exists.

Using a user defined module keeps the test runner from enforcing rules on how annotations are used.

Please provide opinions and I will make updates to the code and this pull request as needed.

Very cool! Thanks so much for this. I'm not entirely clear on how this would work for the user, but that will get clearer with more study-ing. I have one question now:
Do I understand correctly that if people want to handle custom annotations they'll need to create a yadda-run-test-helper.js file? If so it might be nice to add this file to the default blueprint, so it's created with the skip annotation by default so people can see how it works.

Do I understand correctly that if people want to handle custom annotations they'll need to create a yadda-run-test-helper.js file?

Yes.  They can name it what they want and put what ever logic in it they want.  If we want to provide a default implementation we could provide that as a blueprint.

I think it would make sense to handle all annotations the same way.

I simplified some things:

Annotations are all (including @ignore) handled in tests/helpers/yadda-annotations.js.

The configuration in ember-cli-build.js has been removed.  The feature parser always imports tests/helpers/yadda-annotations.
See acceptance.js for how it is called.

A default implementation of yadda-annotations.js is in the blueprint and will be put in the consuming app's tests/helpers at install time.
The default implementation provides backwards compatibility.  See comments in yadda-annotations.js for more details.
If the consumer needs something other than than the default implementation, they can change it in tests/helpers/yadda-annotations.js.

I didn't know how to change the ember's ENV.annotations between running feature acceptance tests.
Currently config/environment.js needs to be changed manually and the tests rerun to test all annotation scenarios.  See comments in config/environment.js for more info.  Thoughts on how to automate this?
Does any of this break mocha consumers?
Let me know your thoughts on where this is going, and I will continue to adjust and clean it up.

Is this anywhere close to the concept you are thinking?

code here returns a function
code here executes it

With the way it is coded, there would need to be a decision on what params to pass to the functions.  Thoughts?

Thoughts on where this is going?  I'll clean it up and add some doc if so.

Yeah definitely, I'll try to have a look this evening. I think this is a very worth while extension. We'll have to look into mocha support and upgrade path, from the top of my head. I'll look into this asap. 😄

I looked at it yesterday and got kind of lost in getting ember try to install ember-cli-mocha to test the mocha stuff and then my npm / node install disintegrated. 👍
I really like what you've done here and I think it's the correct way to handle annotations. Giving sensible defaults and providing extensibility for future use.
@simonihmig and @adamjmcgrath and @mschinis If you guys have time and are still using ember-cli-yadda do you want to read through this and see what you think 😄

This branch is failing for mocha because of #53 if you rebase it against master I'll take a look

hey @albertjan, we're currently not using ember-cli-yadda 😊

I'm not sure I re-based this correctly :slightly_frowning_face:.
@adamjmcgrath can you try this out for mocha?

@albertjan @adamjmcgrath I think this is ready to merge.  Any objections?

Go for it.

From: sfbwalder<EMAIL_ADDRESS>Sent: Saturday, February 17, 2018 7:44:18 PM
To: albertjan/ember-cli-yadda
Cc: Albert-Jan Nijburg; Mention
Subject: Re: [albertjan/ember-cli-yadda] WIP: Runtime annotations (#52)
@albertjanhttps://github.com/albertjan @adamjmcgrathhttps://github.com/adamjmcgrath I think this is ready to merge. Any objections?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/albertjan/ember-cli-yadda/pull/52#issuecomment-366466386, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AASa38vtRtqpUwJppISB3odvrZoTCLMOks5tVywSgaJpZM4PbYGf.

@albertjan ready for new release, thx

@sfbwalder do you have a npm account?


Some SMARTS patterns from CDK are not working in RDKit

Describe the bug
I am trying to reimplement Chemistry Development Toolkit (CDK) SubstructureFingerprinter in RDKit. Out of 307 SMARTS patterns there, 304 are working well, but I get None for:
&#47[D2]=[D2]/
&#47[D2]=[D2]/
([-1,-2,-3,-4,-5,-6,-7]).([+1,+2,+3,+4,+5,+6,+7])

**To Reproduce**

patterns = [
"&#47[D2]=[D2]/",
"&#47[D2]=[D2]/",
"([-1,-2,-3,-4,-5,-6,-7]).([+1,+2,+3,+4,+5,+6,+7])"
]
for pattern in patterns:
print(MolFromSmarts(pattern))

**Expected behavior**
Regular molecules created from SMARTS patterns, for substructure matching.

**Configuration (please complete the following information):**
 - RDKit version: 2023.9.5
 - OS: Ubuntu 22.04
 - Python version (if relevant): 3.11
 - Are you using conda? no
 - If you are not using conda: how did you install the RDKit? pip

For the first two, this looks like an escaping issue in the CDK docs. Take a look at the CDK source directly at https://github.com/cdk/cdk/blob/main/descriptor/fingerprint/src/main/resources/org/openscience/cdk/fingerprint/data/SMARTS_InteLigand.txt
I think you want */[D2]=[D2]\* and */[D2]=[D2]/* which both work fine with MolFromSmarts.

For ([-1,-2,-3,-4,-5,-6,-7]).([+1,+2,+3,+4,+5,+6,+7]), the RDKit docs have a list of "SMARTS features that are not supported" which includes "component level grouping requiring matches in different components, i.e. (C).(C)".

@mcs07 I totally missed that, thanks!


feat: allow fields to be optional i.e. not required

follow-up of #92
fix #69

@samuelcolvin ready for review!

Feedback taken! Thanks for the review

I'll fix conflicts now, then review.

LGTM, thanks so much, this looks great.

Thanks so much.


fix: remove mseUpdating flag

fix: remove mseUpdating flag
Description
No need to use mseUpdating flag
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #63 into master will increase coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   52.02%   52.08%   +0.06%
==========================================
  Files         123      123
  Lines        1705     1703       -2
==========================================
  Hits          887      887
+ Misses        818      816       -2

Impacted Files
Coverage Δ

packages/griffith-mp4/src/mse/controller.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Cursor jumps around and scroll does not work

First of all, thank you for the software! It works great on my work machine, and does exactly what I want. Problem is with my home setup where I use a laptop with a hi-dpi monitor 3200x1800. I think something about the size scaling (font, apps, etc) feature of Windows 10 appears to throw off the mouse cursor location.

If it's set to 100%, it works fine, but when anything else, it throws the cursor off screen. Any help will be appreciated! If you can point towards where the problem might be, I could also take a crack at it...
Thank you!

I can confirm the same issue on Windows 8.1.

The issue has been fixed by @leohinojosa in #6, I'll create a new version asap.

The issue has been fixed by @leohinojosa in #6, I'll create a new version asap.

v2.1.0 is released. Testing with a single monitor went fine but I did encounter some strange behaviour with multiple monitors. I'll  create a new bug for these issues.

I can confirm it's fixed with 2.1.1. Thank you.


Firefox: Accessibility labels and event type labels on event preview cards not readble by a screen reader

What
On the events preview cards on the "whats on pages" the mac voice over, screen reader cant read the little labels that are indicating that it is "speech to text" etc
Where

https://wellcomecollection.org/#main

It seems that voiceover with firefox is ignoring any list items inside the anchor - not sure what to do about this as they are lists, so should be marked up as such

It seems that voiceover with firefox is ignoring any list items inside the anchor - not sure what to do about this as they are lists, so should be marked up as such


Heads up! Repo name changed to springtime

We decided to rename our project to "Springtime", for "Spatiotemporal Phenology Research with Interpretable Models". Since this repo more or less reflects our main workspace, I suggest renaming the repo to match the new project name.

Done! Heads up: to update your local clone of the repo, run:
git remote set-url origin git@github.com:phenology/springtime

For more info, see https://docs.github.com/en/repositories/creating-and-managing-repositories/renaming-a-repository
You may also want to change the folder name of your local clone:
mv phenologyX/ springtime

@mkhzadeh @SarahAlidoost please be aware of the changed repo name

Should be done by now


⚠ Inventory API Endpoint has degraded performance

In 3dd6bee, Inventory API Endpoint (https://inventory.roblox.com/v1/users/82738847/assets/collectibles?limit=10&sortOrder=Asc) experienced degraded performance:

HTTP code: 503
Response time: 2842 ms

Resolved: Inventory API Endpoint performance has improved in fb78b29.


Add test-unit gem dependency explicitly

Because Ruby 2.2 does not provide minitest with test-unit compatible layer.

Thanks!

:yum:


@retryable should specify how to retry

Excessive retries can lead to problems, especially when the operation is failing because it is already overloaded. I'm looking at the retryable example from the spec:
@error("server")
@retryable
@httpError(503)
structure ServiceUnavailableError {}

Without a specific strategy this may lead to retry storms. One way to get around this would be to specify strategies for retrying, for example:
@error("server")
@retryable(strategy: "TokenBucket") // Retries once for every 1000 successful requests
@httpError(503)
structure ServiceUnavailableError {}

@error("server")
@retryable(strategy: "ExponentialBackoff", times: 3)
structure FooError {}

@error("server")
@retryable(strategy: "OnceImmediately")
structure BarError {}

Of course best practices should be the default if nothing is specified, but in some cases a different design is warranted, and a one-size-fits-all approach doesn't make sense

Thanks for the suggestion! We really need to write up a doc on how clients should handle retries...
We have some design documents floating internally within AWS describing different approaches to retry strategies based on whether an error is a throttle vs if it's some other kind of retry. This is captured today in the @retryable trait through the throttling property. From my understanding, we were planning on only needing two strategies: one using a token bucket and AIMD, and another using exponential backoff. We'll followup here when there's more information.


Update --tree usage example to match option in the help info

For consistency. The top level example uses --showtree but the preferred option in the usage help below is --tree

Resolved by #3078


Projects task2 gayathrigundaveni

Install Task Capture from
https://vajrang.sharepoint.com/:u:/r/sites/UiPathProjects/Shared Documents/General/Softwares/task-capture-setup.exe?csf=1&web=1&e=cOXO3D

I will take this task

Completed the task and here is the PR #1835


[Suggestion] Improved submarine detection.. ballast tanks

Hi,
I was trying to make a sub, and after making my nice little sub I noticed it didn't work, so I went and looked at the video, and the way you are calculating it, well, it really doesn't allows you to free form your sub. Since this is minecraft and things really don't have to look real to work well, I was thinking why don't you add colored "ballast tanks" blocks in the same way balloon blocks work, just ask for a "ballast tank" ratio in your build and if it has it it will sink, and if it doesn't it will not, plain and simple.
thanks.
-Winsrp

Added to this, you could also add air blocks, so the player fill the walk-able space with air blocks so you know what parts to fill with air when the ship stops, and what to keep with water.

I was considering making it just a minimal not fillable block count like 24 or something...

Closed, the mod is essentially in a feature freeze for the time being. Focused on maintenance.


AKS: unclear handling of Premium SSD v2 config support

The section Set IOPS and throughput limits is unclear (especially the first sentence, "Input/Output Operations Per Second (IOPS) and throughput limits for Azure Premium v2 SSD disk is currently not supported through AKS"):

Is there a plan to support it in the future?
When I create a StorageClasswhere I set some values for IOPS and throughput limits, are these values initially applied?

If not, what default values are used?
If they are applied, does the first sentence mean that simply updating these values (in a PersistentVolume) is not supported?

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

ID: 1ae05620-ad2f-fa83-a3a4-05c8d099bc4c
Version Independent ID: f6758c5c-1072-f0ce-73f9-75f68011586c
Content: Enable Premium SSD v2 Disk support on Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/use-premium-v2-disks.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@MShekow
Thanks for your feedback! We will investigate and update as appropriate.

@MShekow
Thank you for your questions. I understand that the section "Set IOPS and throughput limits" in the document you referenced may be unclear.
Regarding your questions:
It is not clear from the document whether there is a plan to support IOPS and throughput limits for Azure Premium v2 SSD disks in AKS in the future. You may want to check the Azure AKS documentation or contact Microsoft Support for more information on this.
When you create a StorageClass and set values for IOPS and throughput limits, these values are initially applied to the PersistentVolumeClaim. However, the actual IOPS and throughput performance may vary depending on the size and type of the disk, as well as the VM size and region.
If the values for IOPS and throughput limits are not initially applied, the default values for the StorageClass will be used. The default values may vary depending on the StorageClass and the AKS version.
The first sentence in the section means that AKS does not currently support setting IOPS and throughput limits for Azure Premium v2 SSD disks. However, you can still set these limits in the PersistentVolumeClaim using the maxIOPS and maxThroughputMiBps parameters. Updating these values in a PersistentVolumeClaim should be supported, but you may want to test this in your environment to confirm.
I hope this helps clarify the section. Let me know if you have any further questions.

Hi, thank you for your response.

You may want to check the Azure AKS documentation or contact Microsoft Support for more information on this.

I thought that by creating a ticket here, you are clarifying it (presumably, you have "connections" to the AKS core team).

However, the actual IOPS and throughput performance may vary depending on the size and type of the disk, as well as the VM size and region.

I'm assuming that this is a general statement (not specific to Premium SSD V2). For the Premium SSD v2 disk type, I'd assume that the IOPS and throughput limits do not vary, but conform to what I'm paying for.

However, you can still set these limits in the PersistentVolumeClaim using the maxIOPS and maxThroughputMiBps parameters.

Are these parameters (maxIOPS, maxThroughputMiBps) documented anywhere? Google doesn't find these terms, at all. Could you please provide a complete YAML example of a PVC that uses these parameters? I wouldn't know where to exactly to place them.

@MShekow
Thank you for your feedback!
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)
Thank you for your time and patience throughout this issue.

Sad to see that you use AI to answer questions :( (quite obvious signals, including wrong links to MS Teams forums that have no relevance here)


Does GPTMD task handle overlap between Uniprot and other mod lists?

The Mods list has several things that are also represented in the Uniprot modifications, e.g. phophos. It doesn't look like these are hashed to give a unique set of modifications at GPTMDTask.cs @ line ~136. Leah and I are curious how these are being handled, i.e. what happens upon collisions.

Treated as unique. Need to do this for mainly because ptmlist does not include neutral loss info.

Which does it assign in the Gptmd database since overlaps would have the same mass? Do you recommend just using the mods one or birth?

*both. Sorry for accidentally closing/reopening. On my phone.

Recommend against using uniprot and unimod modifications for gptmd.

If multiple mods selected, all with corresponding masses would be added. It would be a mess

Got it. I would put an actual note of this at the top of where you check mods because it wasn't immediately obvious to me.

uniprot removed from search options in #1723 .


Upgrade to bazel 1.0.1

This pull request is based off of https://github.com/grpc/grpc/pull/20558 with the following additions:

Remove the toolchains //third_party/toolchains:local and //third_party/toolchains:local_large.
Remove the platforms :rbe_ubuntu1604, :rbe_ubuntu1604_large, :local and :local_large.
No longer inherit from @rbe_default//config:platform but instead use it directly. It is now the only non-windows platform.
When creating @rbe_default//config:platform directly with rbe_autoconfig, set dockerAddCapabilities and dockerPrivileged directly in the exec_properties field. No need to set dockerNetwork to "off" and dockerSiblingContainers to false since these are the defaults.
Also set gceMachineType = "n1-highmem-2" on the default platform. This value can be overridden by specific targets that want to use LARGE_MACHINE.
Use create_exec_properties_dict where appropriate.
Use custom_exec_properties to define LARGE_MACHINE.

Note that these changes do not address (at least not intentionally) the reasons because of which https://github.com/grpc/grpc/pull/20558 was rolled back in https://github.com/grpc/grpc/pull/20611.
Also, I wasn't able to test thoroughly that this PR does not break any existing targets. I was not able to run anything on windows/mac and I also don't have access to gRPC's RBE setup.

@eytankidron  I'm trying to get https://github.com/grpc/grpc/pull/20732 in first - it's mostly done but I'm facing a problem with std::regex bug in RBE docker image (I think).

@eytankidron I'm trying to get #20732 in first - it's mostly done but I'm facing a problem with std::regex bug in RBE docker image (I think).

Sounds good. When that PR is merged, the next step could be to implement these platform/toolchain simplifications.

@eytankidron  #20732 has been merged, can you please rebase?

@eytankidron #20732 has been merged, can you please rebase?

I am going to create a new pull request which only has the platform/constraints changes and does not attempt to migrate from Bazel 1.0.0 to Bazel 1.0.1 because that just adds noise.
Stay tuned.

Superseded by https://github.com/grpc/grpc/pull/20862


Implement relayRequestId security scheme

I wouldn't mind postponing this

Gets bigger with docs and example but yes

@kolenic-martin - to test this please run the relay-security-schemes airnode-examples integration. After making a request you should see in the console output that the requestId has been relayed.

Hi, @dcroote, I've been struggling here with some deployment-related issues and am investigating that at the moment, but I'll try to do that in the morning tmrw (UTC+2) and will get back to you as soon as I manage to get the deployments working.

@amarthadan - with the docs PR for this merged, should the "needs documentation" label be removed? Also should I add the tag myself in the future for issues that I know will need documentation?

@dcroote So, the flow for the tasks that need documentation is not finalized yet and I'm still trying different approaches. The main idea is that each issue in the Airnode repository that needs documentation (e.g. this one) has also a "sibling" issue in the docs repository (https://github.com/api3dao/api3-docs/issues/844). I initially thought that these sibling issues should be created by the developer once the task is done but now I'm thinking about creating them during the planning beforehand. The needs documentation label is currently just for me. I'm trying to figure out what would be a good way of marking such issues and will most likely later use them for some automation.

@amarthadan - sounds good, thanks for the clarification.

Hi @dcroote and @amarthadan, I've tested the issue by running the relay-security-schemes Airnode example, which successfully sent a request that was also fulfilled without any issues. Consider this tested.
PS: sorry about the delay with the message, I spent most of yesterday digging in AWS despite my hopes and expectations.


Change repo name chaikovsky => tchaikovsky

Tchaikovsky seems to be right way to spell the composer's name

Sure thing! Thanks for pointing out!


[tool_crash] ProcessException: No such file or directory Command: /home/kb/snap/flutter/common/flutter/bin/cache/dart-sdk/bin/pub, OS error code: 2

Command
flutter create FYP

Steps to Reproduce

...
...
...

Logs
ProcessException: No such file or directory Command: /home/kb/snap/flutter/common/flutter/bin/cache/dart-sdk/bin/pub, OS error code: 2
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:390:33)
#1      Process.start (dart:io-patch/process_patch.dart:36:20)
#2      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:44:22)
#3      ErrorHandlingProcessManager.start.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:664:24)
#4      _run (package:flutter_tools/src/base/error_handling_io.dart:544:20)
#5      ErrorHandlingProcessManager.start (package:flutter_tools/src/base/error_handling_io.dart:663:12)
#6      _DefaultProcessUtils.start (package:flutter_tools/src/base/process.dart:425:28)
#7      _DefaultProcessUtils.stream (package:flutter_tools/src/base/process.dart:444:35)
#8      _DefaultPub.batch (package:flutter_tools/src/dart/pub.dart:283:34)
<asynchronous suspension>
#9      _DefaultPub.get (package:flutter_tools/src/dart/pub.dart:223:7)
<asynchronous suspension>
#10     PubDependencies.update (package:flutter_tools/src/flutter_cache.dart:121:5)
<asynchronous suspension>
#11     Cache.updateAll (package:flutter_tools/src/cache.dart:546:9)
<asynchronous suspension>
#12     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1143:7)
<asynchronous suspension>
#13     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1043:27)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     CommandRunner.runCommand (package:args/command_runner.dart:196:13)
<asynchronous suspension>
#16     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:284:9)

[✓] Flutter (Channel stable, 2.2.1, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.1 at /home/kb/snap/flutter/common/flutter
    • Framework revision 02c026b03c (3 months ago), 2021-05-27 12:24:44 -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/kb/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /snap/android-studio/114/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for
      more details.

[✗] Chrome - develop for the web (Cannot find Chrome executable at
    google-chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2020.3)
    • Android Studio at /snap/android-studio/114/android-studio
    • Flutter plugin version 59.0.2
    • Dart plugin version 203.8292
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[!] Android Studio
    • Android Studio at /home/kb/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.59.0)
    • VS Code at /usr/share/code
    • Flutter extension version 3.25.0

[!] Connected device
    ! No devices available

! Doctor found issues in 4 categories.

Flutter Application Metadata
No pubspec in working directory.

hI @RitchieM0
duplicate of https://github.com/flutter/flutter/issues/77484. please follow https://github.com/flutter/flutter/issues/77484#issuecomment-896742636


Custom configuration is not translated to English

(sorry about the template again)

It's an issue about the theme config, so I guess you should submit it to the theme repo.
But as I can see this configuration doesn't support multi-language yet so I suggest you can translate and replace it at /Gridea/themes/<your_theme>/config.json.


404-Error-Fix

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


cusCastSetFormat symbol is missing in clarius-cast-v801-linux libraries

Hi Clarius,
While adding more options to caster example available on master branch, cusCastSetFormat appears to be missing from clarius-cast-v801-linux.zip libraries:
~/sources/github.com/clariusdev/cast/src/example/caster$ make
mkdir -p build/./
g++ -I./ -I./build -I../../include -std=gnu++14 -c main.cpp -o build/./main.cpp.o
g++ ./build/./main.cpp.o -o build/caster -L../../lib -lcast -lpthread
/usr/bin/ld: ./build/./main.cpp.o: in function `init(int&, char**)':
main.cpp:(.text+0x15d5): undefined reference to `cusCastSetFormat'
collect2: error: ld returned 1 exit status
make: *** [Makefile:24: build/caster] Error 1

This can be reproduced with this fork: https://github.com/daudrain/cast.

should be fixed now in 8.6.0 release.

cusCastSetFormat is still missing in clarius-cast-v860-linux
>nm -gD libcast.so | grep cusCast
0000000000416806 T cusCastConnect
00000000004162bd T cusCastDestroy
000000000041693d T cusCastDisconnect
0000000000416433 T cusCastInit
0000000000414c06 T cusCastIsConnected
0000000000416db0 T cusCastReadRawData
0000000000416cc0 T cusCastRequestRawData
0000000000416b01 T cusCastSeparateOverlays
0000000000414cfd T cusCastSetButtonFn
0000000000414d4f T cusCastSetErrorFn
0000000000414cd4 T cusCastSetFreezeFn
0000000000416a1b T cusCastSetOutputSize
0000000000414c82 T cusCastSetProcessedImageFn
0000000000414d26 T cusCastSetProgressFn
0000000000414c59 T cusCastSetRawImageFn
0000000000414cab T cusCastSetSpectralImageFn
0000000000416e96 T cusCastUserFunction

thanks for catching, this was caught recently, and our 8.7.0 release has fixed this, will update when any beta's are available.

should be fixed now with latest release, thanks for patience!


[HW] Hitesh Reddy Peram

Name: Hitesh Reddy Peram
Email<EMAIL_ADDRESS>

Attach the homework screenshots below for both step II and step III:

This is a duplicate of https://github.com/DataStax-Academy/Intro-to-Cassandra-for-Developers/issues/101 where we also have our conversation. I'll close this one out and use the other for your homework badge.


Fixed buggy AMD implementation

Implemented Universal Module Definition to support buggy existing AMD implementation.

@tHBp Thanks so much for your contribution. I would very much like to improve the library’s reusability by implementing UMD. However, this PR seems to have made some additional changes to the codebase, including changing the indentation from 2 spaces to 4, and adding ; at the end of some lines. If you are willing to re-implement the PR such that the only change modifies the module definition, I’d be happy to accept the changes. Otherwise, I’d be happy to implement this change at some point myself. Thanks

I have implemented the suggested changes and updated the PR. If it is reasonable to you, can you also publish the same to npm once changes are merged? That'd be great. Thanks.

Should be all set https://www.npmjs.com/package/chrome-tabs


Fix(config): do not allow that the same processor runs more than once

This PR fixes #110.
@henry-lp WDYT?

It's great :) . LGMT


Code error, please answer

Hello author, I downloaded your programme and the dataset today to use spyder for study and research, but your source code is running with an error, I'm using the pdrcnn dataset, and the error is reported as follows: (no change in code)
ValueError: Input 0 of layer conv1d_48 is incompatible with the layer: : expected min_ndim=3, found ndim=2. Full shape received: [None, 120]
I'm looking for an answer.

I encountered the same issue as you, The problem is that data is missing the feature dimension necessary for a Conv1D layer, which needs the input_shape=(timesteps, features). You can try adding an additional dimension with tf.expand_dims for x: x = tf.expand_dims(x, axis=-1), it works for me

I encountered the same issue as you, The problem is that data is missing the feature dimension necessary for a Conv1D layer, which needs the input_shape=(timesteps, features). You can try adding an additional dimension with tf.expand_dims for x: x = tf.expand_dims(x, axis=-1), it works for me

Hello, I used this method a few days ago, but the bug appeared again in the subsequent training, have you encountered the error since then

I didn't encounter such an issue again later， would you still report the same error?

I didn't encounter such an issue again later， would you still report the same error?

I have encountered other bugs during the reproduction process, and the program is not running at the moment, can you provide an email address for us to easily communicate?

I didn't encounter such an issue again later， would you still report the same error?

你后来遇到这个问题了吗：
nvalidArgumentError: 2 root error(s) found.
(0) Invalid argument:  indices[0,0] = 160001 is not in [0, 160001)
[[node functional_1/embed_ngram_1/embedding_lookup (defined at C:\Users\Zhao\Desktop\phishing detection\GramBeddings-main\train.py:315) ]]
(1) Invalid argument:  indices[0,0] = 160001 is not in [0, 160001)
[[node functional_1/embed_ngram_1/embedding_lookup (defined at C:\Users\Zhao\Desktop\phishing detection\GramBeddings-main\train.py:315) ]]

I also encountered this issue. It's caused by a mismatch in the input dimension of the embedding layer. You just need to change it to input_dim = self.vocab_size + 1, and it should run smoothly.

I also encountered this issue. It's caused by a mismatch in the input dimension of the embedding layer. You just need to change it to input_dim = self.vocab_size + 1, and it should run smoothly.

Big Brother thank you so much, now it is possible to train, I guess it is possible to run through it

I also encountered this issue. It's caused by a mismatch in the input dimension of the embedding layer. You just need to change it to input_dim = self.vocab_size + 1, and it should run smoothly.

Hi big guy, did you encounter any other problems while testing, I'm a rookie and have some confusion after training, can I ask you for some advice?

I also encountered this issue. It's caused by a mismatch in the input dimension of the embedding layer. You just need to change it to input_dim = self.vocab_size + 1, and it should run smoothly.

My model test result is very poor, please ask your model prediction how, can add you a qq or something to exchange

I also encountered this issue. It's caused by a mismatch in the input dimension of the embedding layer. You just need to change it to input_dim = self.vocab_size + 1, and it should run smoothly.

My model test result is very poor, please ask your model prediction how, can add you a qq or something to exchange

My program runs without any issue, and the results are quite promising. Certainly, you can add my QQ:<PHONE_NUMBER>, and we can discuss further.

我也遇到了这个问题。这是由嵌入层的输入维度不匹配引起的。您只需将其更改为input_dim = self.vocab_size + 1，它应该可以顺利运行。

我的模型测试结果很差，请问你的模型预测怎么样，可以加你一个qq什么的交流一下

你好，我可以加你联系方式吗，我也是个菜鸡，想交流一下


login at ru servers

python bot.py
config reloaded
11-07, 14:53:56 INFO Starting bot
11-07, 14:53:57 ERROR Login failed!
11-07, 14:53:57 INFO Sleeping for 698 secs
config
[credentials]
uni = 144
server = ru
username =<EMAIL_ADDRESS>password = ********

works after sudo pip install --ignore-installed six


Complete CSE2000 contributions

Resolves #12, #17, #18, #19, #20, #27
Note, for #12, only KFAC for multi-class is included. KFAC is not applicable to regression. KFLR would require access to pre-activation values, which Zygote does not provide.
This PR includes an extensive and systematic test-suite covering most combinations of backends (EmpiricalFisher or GGN), network subset types (all, subnetwork, lastlayer), batchsizes, and Hessian structures (full or kron), comparing Hessians computed to those of laplace-torch.
We have also included some additions to the pipeline, namely: a formatting check, and a benchmark pipeline.

Codecov Report

Merging #33 (a3b3a7f) into main (29982d4) will decrease coverage by 6.36%.
The diff coverage is 90.50%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #33      +/-   ##
==========================================
- Coverage   97.18%   90.83%   -6.36%
==========================================
  Files           9       10       +1
  Lines         249      611     +362
==========================================
+ Hits          242      555     +313
- Misses          7       56      +49

Impacted Files
Coverage Δ

src/LaplaceRedux.jl
100.00% <ø> (ø)

src/curvature/Curvature.jl
100.00% <ø> (ø)

src/data/Data.jl
100.00% <ø> (ø)

src/mlj_flux.jl
66.10% <66.10%> (ø)

src/baselaplace.jl
90.00% <90.90%> (-10.00%)
:arrow_down:

src/curvature/functions.jl
97.94% <97.83%> (-2.06%)
:arrow_down:

src/data/functions.jl
100.00% <100.00%> (ø)

src/laplace.jl
95.23% <100.00%> (-2.54%)
:arrow_down:

src/plotting.jl
97.77% <100.00%> (+6.66%)
:arrow_up:

src/utils.jl
100.00% <100.00%> (+10.00%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Some other things to address before we merge (TBC):

[ ] missing docstrings in a few places

Hi team,
As far as I can tell from this, you haven't added any updates to the actual /docs/, is that right? In that case, what constitutes your documentation? Just everything that's under dev?

Hi! This was addressed on the final presentation, but for the record: no, we did not update docs/, the supporting documentation we provide is in docstrings and notebooks. Sorry for the oversight with docstrings for KFAC, we'll fix that soon enough.


[Snyk] Upgrade io.grpc:grpc-api from 1.47.0 to 1.49.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade io.grpc:grpc-api from 1.47.0 to 1.49.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 7 versions ahead of your current version.
The recommended version was released a month ago, on 2022-10-01.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add "new_name" parameter to repo transfer request (#2787)

Resolves: https://github.com/google/go-github/issues/2787

@gmlewis thanks for the hint. I updated the code and run the generate command. Have to admit did not thought about the accessors. Very nice that you have generators in place for that.


Presenting at DC DevOps

A placeholder.

Link to the presentation
Talk about the experience

Welp, the time is a little past for this one.


P2Plus: SourceAcademy Mission Creation Interface

Description
Creates a way for GitHub-hosted missions to be created through the Source Academy website.
Subfeatures:

Ability to create and delete new tasks for a mission
Ability to create, edit and delete test cases for a mission
Ability to load test cases

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test

Checklist

[ ] I have tested this code
[ ] I have updated the documentation

Pull Request Test Coverage Report for Build 896148990

187 of 850 (22.0%)  changed or added relevant lines in 31 files are covered.
6 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.7%) to 28.71%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/application/Application.tsx
0
1
0.0%

src/commons/application/actions/SessionActions.ts
4
5
80.0%

src/commons/application/reducers/SessionsReducer.ts
0
1
0.0%

src/commons/controlBar/ControlBarGitHubButtons.tsx
0
1
0.0%

src/commons/githubAssessments/GitHubMissionCreateDialog.tsx
9
10
90.0%

src/commons/navigationBar/subcomponents/tests/snapshots/GitHubAssessmentsNavigationBar.tsx.snap
0
1
0.0%

src/commons/navigationBar/NavigationBar.tsx
0
2
0.0%

src/commons/sideContent/SideContentMissionEditor.tsx
12
14
85.71%

src/pages/playground/Playground.tsx
0
2
0.0%

src/commons/controlBar/ControlBarGitHubLoginButton.tsx
1
5
20.0%

Files with Coverage Reduction
New Missed Lines
%

src/commons/controlBar/ControlBarGitHubButtons.tsx
1
9.52%

src/commons/gitHubOverlay/FileExplorerDialog.tsx
1
71.43%

src/commons/sagas/GitHubPersistenceSaga.ts
1
14.86%

src/pages/playground/Playground.tsx
1
2.28%

src/features/github/GitHubUtils.tsx
2
2.7%

Totals

Change from base Build 892480455:
-0.7%

Covered Lines:
2940

Relevant Lines:
9459

💛  - Coveralls

I'm not sure why it's 541 commits. It should only be about 100. Most of those commits should already have been merged in with P2Minus.

To my knowledge, Octokit API for creating repositories under an organization does not exist for Javascript, only for Ruby.

I'm not sure why it's 541 commits. It should only be about 100. Most of those commits should already have been merged in with P2Minus.

I think github counts all commits in this branch from the time it was created. This includes all commits that have been merged from the main branch. That explains the number 541, I think. Don't worry about that.

To my knowledge, Octokit API for creating repositories under an organization does not exist for Javascript, only for Ruby.

What is the consequence of this for this project?

To my knowledge, Octokit API for creating repositories under an organization does not exist for Javascript, only for Ruby.

What is the consequence of this for this project?

I will make sure to write about this when updating the documentation.
I will also make an issue to track this issue after the merge. I think this is a problem that can be solved by sending our own GET requests to GitHub, perhaps through a similar microservice that is being used for logging-in right now, but making the change now might be cutting a bit too close to the end of our internship.


11529 - Bambara - Society lat-lon could be closer to songs and EA match

Current society lat-lon isn't incorrect per se, but since all of our songs assigned to 11529 are from Southern Mali, and that's where the EA and glottolog coordinates are too, it could be good to move the society lat-lon to the average song lat-lon: 12.60, -7.66.

Suggestion implemented


ERROR: Failed building wheel for pointnet2

After entering the directory .\utils\Pointnet2. PyTorch\pointnet2, I executed the build command pip install ., There were many errors in the middle, and I have been solving them until now. I'm not sure if it's because my Python version doesn't match or doesn't adapt to Windows 10. I have just come into contact with this issue, and if you are willing to point out my issue, I would greatly appreciate it.
The error is as follows:
Processing d:\project\vrcnet-main\utils\pointnet2.pytorch\pointnet2
  Preparing metadata (setup.py) ... done
Building wheels for collected packages: pointnet2
  Building wheel for pointnet2 (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py bdist_wheel did not run successfully.
  │ exit code: 1
  ╰─> [16 lines of output]
      running bdist_wheel
      running build
      running build_ext
      building 'pointnet2_cuda' extension
      creating D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build
      creating D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38
      creating D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release
      creating D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src
      Emitting ninja build file D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\build.ninja...
      Compiling objects...
      Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
      1.11.1.git.kitware.jobserver-1
      creating D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\lib.win-amd64-cpython-38
      "D:\software\Microsoft Visual Studio\VC\Tools\MSVC\14.36.32532\bin\HostX86\x64\link.exe" /nologo /INCREMENTAL:NO /LTCG /DLL /MANIFEST:EMBED,ID=2 /MANIFESTUAC:NO /LIBPATH:D:\Anaconda3\envs\GAN\lib\site-packages\torch\lib "/LIBPATH:D:\NVIDA GPU Computing Toolkit\lib\x64" /LIBPATH:D:\Anaconda3\envs\GAN\libs /LIBPATH:D:\Anaconda3\envs\GAN /LIBPATH:D:\Anaconda3\envs\GAN\PCbuild\amd64 "/LIBPATH:D:\software\Microsoft Visual Studio\VC\Tools\MSVC\14.36.32532\lib\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.18362.0\ucrt\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\\lib\10.0.18362.0\\um\x64" c10.lib torch.lib torch_cpu.lib torch_python.lib cudart.lib c10_cuda.lib torch_cuda_cu.lib torch_cuda_cpp.lib /EXPORT:PyInit_pointnet2_cuda D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/ball_query.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/ball_query_gpu.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/group_points.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/group_points_gpu.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/interpolate.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/interpolate_gpu.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/pointnet2_api.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/sampling.obj D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src/sampling_gpu.obj /OUT:build\lib.win-amd64-cpython-38\pointnet2_cuda.cp38-win_amd64.pyd /IMPLIB:D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src\pointnet2_cuda.cp38-win_amd64.lib
      LINK : fatal error LNK1181: 无法打开输入文件“D:\project\VRCNet-main\utils\Pointnet2.PyTorch\pointnet2\build\temp.win-amd64-cpython-38\Release\src\ball_query.obj”
      error: command 'D:\\software\\Microsoft Visual Studio\\VC\\Tools\\MSVC\\14.36.32532\\bin\\HostX86\\x64\\link.exe' failed with exit code 1181
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pointnet2
  Running setup.py clean for pointnet2
Failed to build pointnet2
ERROR: Could not build wheels for pointnet2, which is required to install pyproject.toml-based projects

thanks a lot😭

Now, I successfully ran it under Linux

Now, I successfully ran it under Linux

can you tell me how you sovled the problem,thanks

Now, I successfully ran it under Linux

can you tell me how you sovled the problem,thanks

Just change the operating system to Ubuntu 20.04 on liunx, and follow the author's requirements for other configurations


How to run sls wsgi serve with <IP_ADDRESS> binding

I am trying to bind to <IP_ADDRESS>, however it seems wsgi is ignoring the flask main app run command. This is required for me to locally access this service within my vm's guest from my host through port forwarding.
from flask import Flask

app = Flask(__name__)

@app.route("/")
def home():
    return "Hello, World!"

if __name__ == "__main__":
    app.run(host='<IP_ADDRESS>', debug=True, port=5001)

Serverless: Using Python specified in "pythonBin": python3
 * Running on http://localhost:5000/ (Press CTRL+C to quit)
 * Restarting with stat
 * Debugger is active!

Is there any way to bind the wsgi serve to specific IP ? through serverless.yml maybe?

Found it!
sls wsgi serve --host <IP_ADDRESS>


chore(renovate): add scheduled renovate to bump providers for examples

Feature or Problem

Unfortunately dependabot is not flexible enough other than the existing ecosystems. Although renovate-bot can do that with regular expressions.
Heads-up, you might need to onboard renovate app for the repository.
The PRs which will be open should look like this https://github.com/markkovari/wasmCloud/pull/36

update -> even should create only one PR from now on.

Feel free to edit the name/description/schedule, not sure what is the most convenient.
Related Issues

closes: #2339
Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

@markkovari this looks really cool! I'm not sure I quite understand how it works though, does it just go through and patch bump on a schedule that's separate from when we release providers?

@brooksmtownsend yes, that is kinda exactly how it works
this is the community cloud version of it, working after enabling the github app to read the repo and create PRs
Unfortunately it only has (AFAIK) a scheduled timer and a manual trigger on their UI, and cannot listen to other events - like releases and other events from GitHub. 😞 but it can keep these references up to date by asking ghcr if a newer version is available.

Would it be worth considering the Renovate GitHub Action instead of the App so that we can have more fine-grained control over how and when it's triggered?

Yeah, sure. Will add it soon. Not sure what on events should I add, but will add a few and will update based on feedback. 🙇

Well it is acting a bit silly on my fork for some reason. Let me debug/fix it there, before we have to do it here. 🤔

@joonas sorry for the delay, did some testing on my fork and unfortunately without the repositories it was not scanning the repository, and the config file was not fount - beside the input explicitly added for the action now it should be working.

@markkovari repositories seems totally reasonable and in fact something we would likely want, so that doesn't seem too bad of an addition.
That does bring up a question about whether we should limit running this to pjust wasmCloud/wasmCloud with an if statement](https://github.com/wasmCloud/wasmCloud/blob/main/.github/workflows/update.yml#L13), because I don't see how this would be useful to have run in forks.
Would you mind adding that to the workflow?


How to use my own video to extract feature?

Hi, I'm trying use my own video to extract feature to get the numpy array by using you extract_feature.py.
But I kept getting the following errors:
Traceback (most recent call last):
File "extract_features.py", line 94, in
run(mode=args.mode, root=args.root, load_model=args.load_model, save_dir=args.save_dir)
File "extract_features.py", line 40, in run
dataloader = torch.utils.data.DataLoader(dataset, batch_size=batch_size, shuffle=True, num_workers=8, pin_memory=True)
File "/home/nick/anaconda3/envs/i3d/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 262, in init
sampler = RandomSampler(dataset, generator=generator)  # type: ignore
File "/home/nick/anaconda3/envs/i3d/lib/python3.7/site-packages/torch/utils/data/sampler.py", line 104, in init
"value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integer value, but got num_samples=0
What's the potential reason to cause this error and how can I solve it?

How did you solve it?


Smpl joint order

Thanks a lot for the amazing code.
Im trying to make it to work in blender, to import the result that is stored in the pkl file.
I was following the infirmation from dmpl about the 23 bones on the character, but after applying it on the smpl fbx, the legs looked alright but the arms and the head were very odd.
Is there any difference from the default order of the SMPL data?
Tha ks a lot for the attention

Hi Carlos,
There shouldn't be a difference with respect to the default order of SMPL data. You can check the typical order of joints here, which has 23 joints + the root (pelvis). The 'global_orient' corresponds to pelvis, whereas the 'body_pose' field corresponds to the rotations of the other 23 joints, from left_hip to right_hand.
Please let us know if you still face any issues.

Oh great, thanks for that information.
I'm using this data

From this presentation
https://files.is.tue.mpg.de/black/talks/SMPL-made-simple-FAQs.pdf
And I think the problem might be because I should have used the global_orient as the pelvis data.
thanks a lot!!!

And that fixed the issue. thanks a lot!!!
showing what I've got

For other people that tries to import it in blender, you could try this script
import in blender.zip
with that script I was able to get this result
https://github.com/shubham-goel/4D-Humans/assets/4061130/3ad47927-d728-47c6-b066-a0ad6d44474f

This is very cool!! Thanks a lot for sharing the code and results.
Would you be interested in submitting a pull request so that we can integrate this 4D Humans -> Blender support? Or, we can also do some minor edits and add instructions (e.g., where to get he fbx files) for people to run this on their own.
Alternatively, if you want to host this in a repo of your own, we would be happy to point to this in the Readme.
I think more people would be interested in this functionality. :)

For now I could do a blend file and add some instructions, since the person can do almost it all using the script panel in blender. I can also make a quick video showing how to import.
Like this would be ok?

@geopavlakos
I made a tutorial at: https://youtu.be/WYdO0fOxEWU
and posted a sample code and blend file at: https://www.patreon.com/posts/86992009
I explain how to load multiple characters and how to fix odd motions
Hope it helps


FreeLibrary() does not work as expected: file still in use

The question is related to:

DllExport -version: 1.7.4

I am working on a plugin system where I dynamically load or unload DLLs at runtime via LoadLibrary()/FreeLibrary(). For DLLs developed with C++, either natively or with use of CLR, this works correctly.
When I use DllExport to create a plugin in C# (.NET 4.8, Visual Studio 2022), I find that FreeLibrary() does not work as expected:
After I call FreeLibrary(), the file is still in use (so it cannot be deleted or overwritten).
Since LoadLibrary()/FreeLibrary() is reference-counted, I tried calling it multiple times. I found that it can be called two times successfully, so apparently something inside the DLL had increased the reference-count as well.
However, even though FreeLibrary() fails after two calls, the file still remains in use, so it still cannot be deleted or overwritten.
I have tested with a minimal DLL which only exports some basic functions that return a string, and does not load any other assemblies, libraries, or creates any objects, threads or anything, to rule out any possible problems.
Then I just tested by doing LoadLibrary(), GetProcAddress(), calling the function to retrieve a string, and then FreeLibrary() again.
I am not sure if this is a bug or if I need to implement some special shutdown/cleanup code somewhere in my DLL for this to work correctly?

Hello,
It's a duplicate, #207, #157, #118
And since for current version we can't use something like modern loading via context (somehow),
I recommend the classic intermediate loading of the final PE module. That is, for example, direct memory if module is tiny or fs copy/rename same to vsix plugin system in Visual Studio IDE etc.


Dont’t wrap alt text in

On my test page, the following tag:
{% cloudinary /assets/test.png alt="beautiful!" %}

produced an img with this alt attribute:
alt="
beautiful!
 "

I can’t find it stated definitively anywhere, but markup doesn't belong in alt. e.g., all of the examples in the W3C’s official guidance are raw text.

You're right, it a bug, only the caption should run Markdownify


Network Interceptor retry on other url

Hi all, please can anyone explain me how can I retry a port or get request on different baseurl on connection timeout?
I try with NetworkInterceptor but it doesn't work

May this demo can help you. https://github.com/flutterchina/dio/blob/master/example/interceptor_lock.dart