feat: add support for file delete, file create and diff file created by GitHub pr diff

see title.
and change the prefix of meta in test data, from space to tab (according to https://www.gnu.org/software/diffutils/manual/html_node/Detailed-Unified.html)
close #30

Thanks, this looks very good! But I would like to ask you to add at least one test which demonstrates that its now possible to create a file which did not exist before and one, which uses a github diff. There are in the tests/ folder.

Very nice! Thanks. This will be in a new release soon


fix(comp:button): buttonGroup disabled not working

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines
[x] Tests for the changes have been added/updated or not needed
[x] Docs and demo have been added/updated or not needed

What is the current behavior?
buttonGroup设置 disabled，button实际未禁用
What is the new behavior?
修复以上问题
Other information

dup #1508


feat: update smartevents to v2

Updating smartevents mgmr to use v2 of their openapi spec file
the v2 is already present in the repo

so we are upgrading the api in the SDK, are we planing this for the other SDKs?

@dimakis
So we are upgrading the api in the SDK, are we planing this for the other SDKs?

in what regard is that? if the other APIs are bumping their version?
This is coming from a request I got to bump the SDK version for smartevents. They want the same package, just different version so this seems the most seamless way to do so

in what regard is that? if the other APIs are bumping their version?

Well if they want to. bumb the version of this dont we want our. sdks in sync shouldnt this also be applied to other SDKs. But maybe i am misunderstanding.

in what regard is that? if the other APIs are bumping their version?

Well if they want to. bumb the version of this dont we want our. sdks in sync shouldnt this also be applied to other SDKs. But maybe i am misunderstanding.

sorry, you mean other languages? yes indeed we do that is correct.
Thats what I was hoping to achieve by "assigning" this to you last week. I've never used the assign thing before though so not sure if you got it or not 😅

Oh I see that you okay no problem will merge this, should we not re-generate sdk in pr aswell?

Oh I see that you okay no problem will merge this, should we not re-generate sdk in pr aswell?

I was going to do that manually. Usually this is automated as per https://github.com/redhat-developer/app-services-sdk-js/actions/runs/3490454820
I've not changed the Open API spec file like this before though so I was just going to do a manual regen of the SDKs before releasing just to make sure.


Create DE.xaml

Thank you for your work but you forgot a little file: https://github.com/Sophia-Community/SophiApp/blob/master/SophiApp/SophiApp/Resources/UIData.json 😹
Actually, you're rushing things: the description isn't finished yet and wil be improved. But I won't close the PR until all localization will have the complete form.

Thank you for your work but you forgot a little file: https://github.com/Sophia-Community/SophiApp/blob/master/SophiApp/SophiApp/Resources/UIData.json 😹 Actually, you're rushing things: the description isn't finished yet and wil be improved. But I won't close the PR until all localization will have the complete form.

I have already seen the file and have not forgotten it, but unfortunately it is not so small 😮

Welcome to our world))

When you will be able to translate the whole UI? We're very near the stable release, aiming on March.

@Henry2o1o do you need help with this? Are you still working on the translation?
I could offer some help.

@csidirop, https://github.com/Sophia-Community/SophiApp/discussions/32#discussioncomment-2128926
You may join him. but only after we will be confident that this is the final translation version. We're testing the first build supported Windows 11. We will inform you all when you may start translating.

@csidirop, https://github.com/Sophia-Community/SophiApp/discussions/32#discussioncomment-2128926
You may join him. but only after we will be confident that this is the final translation version. We're testing the first build supported Windows 11. We will inform you all when you may start translating.

Okay, just mention me when you've figured it out.

Hi, guys. We're pretty close to the first full stable release. As soon as I will be confident that that build will be RTM 🤣 , I will let you know, providing you the configured JSON to add the DE laocalization.

@csidirop, @Henry2o1o, are you ready? 😸
https://github.com/Sophia-Community/SophiApp/tree/master/SophiApp/SophiApp/Localizations
https://github.com/Sophia-Community/SophiApp/blob/master/SophiApp/SophiApp/Resources/UIData.json
You may get in touch me me if you have any questions via Telegram: https://t.me/sanctuary_d

@csidirop, hi. We just wonder how much time will take you to finish it?

@csidirop, hi. We just wonder how much time will take you to finish it?

Hi, I wasn't at home this weekend but i can start this week. Because i have a fulltime job I can only work afterworks.
But its not that much to translate, so maybe a few days. Is that okay?

@csidirop, if you think, this is not much, OK )). As for me, I do not think so. As you may see, we already prepared the DE.xml file for you. So it's OK for us of course to wait until you finish it up. There are still a few bugs left in <IP_ADDRESS> build. :)

@csidirop, hi! How is your progress? :) Or you haven't got to translating?

Hi @farag2, I already finished DE.xaml but only around one third through UIData.json.

@csidirop, great! thanks. As I might see, Ive changed a few strings. FYI.
https://github.com/Sophia-Community/SophiApp/commits/master/SophiApp/SophiApp/Resources/UIData.json

@csidirop, is there an end in sight? :)

Actually not. I'm still at about 30% of UIdata. And I wont have more time till mid next week.
I am really sorry!

@csidirop, no problem. Do it at you free time, of course. I'm just controling thr process. :) We still fixing the last 2 bugs.

@DAmNRelentless, you may team up with @csidirop. Two heads are better then one. 👍I think, we did our best and finished polishing the release version. Just waiting for the Ukranian translation and German one.

@csidirop, hi, any chance to see the result this week? 🌵

@csidirop Hi, again. As you may see, we've totally updated the JSON file by adding Ukranian translation and many more fixes. Do not forget to update it before editing!

The German translation was ordered and implemented. Closing...


Release 0.6.0

/test-all

Don't pay attention to the extra commit. It is already part of the target branch.

/test-all

/test-all

/test-all

/test-all

/test-conformance


LED identification not working

The board provides a user LED we should make use of it.

I tried to use the user LED via /sys/class/gpio7 according to Galileo Gen2 Board, which is very similar to IOT2000. This Link  from Siemens Support points to gpio7 too. I also tried to use other GPIO Pins but failed.
At the moment I don't know where to look for the problem. If you help me where to start I can try to fix this.

Today I tried again and could manage to blink the LED on GPIO46.

ping @spanceac @telphan about this

This is not forgotten, just low on the priority list.

This happens due to a lack of dependency in our kernel. It is fixed by:  https://github.com/resin-os/meta-resin/commit/1e06fe5a53872c6a46b8de933a1675147da71540.
Once meta-resin is released we can  go ahead and fix this issue.


OSX: Bundle without access to RenderTarget still draws black onto screen

I have a project doing shadow-volume-like stuff, which works correctly on Ubuntu:
https://github.com/ucarion/gfx-draping
(The screenshots in the README reflect what it looks like on my Ubuntu machine.)
However, on my Mac, there seems to be this bug where the bundle that's supposed to set up the depth stencil is also drawing black to the color target wherever the depth test passes. Here's a screenshot:

It seems like the stencil buffer is being set up as expected -- the polygon (the square with two holes in it) is getting draped onto the terrain correctly. But the surrounding black rectangular prism is the bug I'm talking about. In src/lib.rs can comment out the line that goes:
        bounding_box_bundle.encode(encoder);

To confirm that polyhedron_bundle, which doesn't even have "access" to any RenderTargetView (only a DepthStencilView), still manages to draw to the screen.

Not sure if this is still an issue? Please re-open otherwise.


Event "object-selected" should return the object

https://github.com/tvjsx/trading-vue-js/blob/e881bdb5c3ec3b890d21e3059cb6b3ef85a47432/src/components/primitives/pin.js#L157
Event "object-selected" should return the object so listener can identify which object emitted it.
this.comp.$emit('object-selected', this)

tool-selectedevent returns the originating tool which emitted the event.
object-selected does not.
On the object-selected event, the chart.data.selectedis not populated, so the following is not possible:  this.chart.get(this.chart.data.selected)
So far the method that does work is to use a watcher.
    'chart.data.selected': function(newValue, oldValue) {
      if (newValue) {
        console.log("TVJS: selected:",newValue)
        const selected = this.chart.get(newValue)
        console.log("TVJS: Object Selected:",selected)
      }
    }


Update months and weeks to singular. fixes #33

Minor fix for deprecation warning

Will merge as soon as you fix the .gitignore.  Thanks!

@bramski sure, good idea.

Merged.


Factor out the pre-registered MemoryBuffer functionality

form SerializedModuleLoader into its own ModuleLoader class. (NFC-ish)
This gives better control over the order in which the various module
load mechanisms are applied.
(cherry picked from commit bc1e9003f6d6cb436bd93463da3bf591ec0eb159)

test with https://github.com/apple/swift-lldb/pull/1492
@swift-ci smoke test

test with https://github.com/apple/swift-lldb/pull/1492
@swift-ci smoke test

test with https://github.com/apple/swift-lldb/pull/1492
@swift-ci smoke test

test with https://github.com/apple/swift-lldb/pull/1492
@swift-ci smoke test

test with https://github.com/apple/swift-lldb/pull/1492
@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - 2d15e2dd390f940707db973b37d4268f602b5d96

Build failed
Swift Test OS X Platform
Git Sha - 2d15e2dd390f940707db973b37d4268f602b5d96


[BUG] Cannot use a 'where' statement on a projection used in SELECT.from(...).columns()

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I use the projection syntax in a SELECT to expand associations. Unfortunately, the proposed solution at the CAP documentation (combination of projection and tagged template strings) is not valid TypeScript syntax. Here the example from the documentation:
SELECT.from ('Authors', a => {
   a.ID, a.name, a.books (b => {
     b`.*`,
     b.suppliers`[city='Paris']`('*')
   })
})

I tried a workaround which is working:
SELECT.from ('Authors', (a) => [
   a.ID, a.name, a.books ((b) => [
     b('.*'),
     (b.suppliers((s) => [s('*')]) as SELECT<unknown>).where(`city = 'Paris'`)
  ])
])

but unfortunately a cast to SELECT<unknown> is necessary.
Expected Behavior
The expected behaviour is allowing the "where" w/o casting:
SELECT.from ('Authors', (a) => [
   a.ID, a.name, a.books ((b) => [
     b('.*'),
     b.suppliers((s) => [s('*')]).where(`city = 'Paris'`)
  ])
])

References
https://cap.cloud.sap/docs/node.js/cds-ql#columns
Versions

Package
Version

Node.js
v20.11.0

@sap/cds
7.9.2

@sap/cds-compiler
4.9.4

@sap/cds-dk
7.9.2

@sap/cds-dk (global)
7.6.0

@sap/eslint-plugin-cds
3.0.3

@sap/cds-mtxs
1.18.0

@cap-js/cds-types
0.2.0

Anything else? Logs?
I can also contribute if you need.

Hi @mutschosW ,
you seem to be using a .where fragment within a projection. I don't think I have ever seen this syntax and I couldn't find it in CAPire either. Could you please point me to where you found this syntax?
Best,
Daniel

Hi @daogrady ,
I also cannot find any code example, but it is the only working possibility I have found for realising the infix filter
b.suppliers`[city='Paris']`

in Typescript. Have I overseen another solution?
Best regards,
Marcel


[Node.js SDK, Code Examples] "Node.js quick start" example uses incorrect env variables for MICROSOFT_APP_ID and MICROSOFT_APP_PASSWORD

Bot Info

SDK Platform: Node.js
SDK Version: 3.13.1
Active Channels: MS Teams
Deployment Environment: Azure Bot Service

Issue Description
Using the example code at https://docs.microsoft.com/en-us/bot-framework/nodejs/bot-builder-nodejs-quickstart#edit-your-bot , the webchat/emulator fails when deployed to Azure Bot service.  The example lines:-
var connector = new builder.ChatConnector({

    appId: process.env.MICROSOFT_APP_ID,

    appPassword: process.env.MICROSOFT_APP_PASSWORD

});

use environment variable names that are different from an Azure Bot deployment.  The environment variables generate for an Azure deployment are MicrosoftAppId and MicrosoftAppPassword.
Actual Results
When deploying the example, it will not respond in WebChat or via ngrok (authentication issues possibly) until these lines are fixed.

Closing the issue assuming it as resolved. Please reopen a new issue if it still persists.


Added third callback parameter "rawData"

A possible solution for issue #19

For this issue, I think a good solution could be to have a ResultsCollection object, an array with a rawData property

Great idea, since in javascript, we can add properties to an array... I implemented this as PR #88, and like this even better, since it is promise compatible...


Use user token to login?

Is it possible to use a normal user account for this?
Normal discord search tries to be "smart" and does not let you do exact string searches.
For example, searching for "liquidate" pulls up "liquidity".
This causes me all sorts of problems.

Definitely, pushing this to a normal user account is the most ideal implementation of this project I think. I do mean to actually devote a good amount of time to building this, and I'll update the discord server when I do!


BLE Connect function does not clear connected device after failed connection.

Problem
In the connect function: https://github.com/crownstone/crownstone-lib-python-ble/blob/57e6b97dedf398ffdcc4d6516283abe271af4dc9/crownstone_ble/core/ble_modules/BleHandler.py#L63
If the connection with the Crownstone fails, and the BTLEDisconnectError is raised, the address is kept as connected.
When checking the connection with for example hcitool con , it is clear that there is no connection with the device then.
Solution
https://github.com/crownstone/crownstone-lib-python-ble/blob/57e6b97dedf398ffdcc4d6516283abe271af4dc9/crownstone_ble/core/ble_modules/BleHandler.py#L70-L72
Before the return False is issued, the address needs to be removed from self.connectedPeripheral and self.connectedPeripherals.
(Via a call to the disconnect function?)
Otherwise, a following attempt will result in "already connected", and this can mess up following code.

Since we've moved to a different bluetooth lib, I think this issue can be closed. If the issue remains in the new library, please open a new issue.


Fix GradScaler + Perceptual Loss

Reduced the init_scale of GradScaler to 2**10,  by lowering the scaling value, GradScaler no longer produces invalid values and ensures that optimizer.step() is executed correctly. This in turn resolves the warning for calling lr_scheduler.step() before optimizer.step().

Commited. Sorry for not merging your PR directly, I've mentioned you in the commit, as well as on the issue and on readme. Thanks.


[bug] displays lint errors in node_modules files that use xo

If a given package uses xo (listed in their (dev-)dependencies), the extension will lint them:

Large screenshot

This can make inspecting some packages unreadable. In addition, if xo is setup to format on save, any userland modifications to a package causes the extension to try formatting them.

v3.16.0 removes the ability for this extension to lint in the node_modules folder - if someone would like this ability, leave a comment and we can put it behind an option.


Bug: Editor is not able to import superscript and subscript styling from html string

Lexical version: 0.6.3
issue with importing superscript styling

$generateHtmlFromNodes() yields the following html string

1012

But When we try to populate the same html string to the editor using $generateNodesFromDOM() method, this results the following

issue with importing subscript styling

$generateHtmlFromNodes() yields the following html string

log210

But When we try to populate the same html string to the editor using $generateNodesFromDOM() method, this results the following

So, Editor is not able to regenerate subscript and superscript styling from the html string.

Duplicate of #2202.


Rancher create then delete service automatically during create Ingress

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Global-->Settings-->ingress-ip-domain
Custom ingress-ip-domain instead of xip.io , like test-2.demo.com

Workload
Create a simple workload nginx , with image nginx

Load Balancing
Add an ingress  with Automatically generate a .test-2.demo.com hostname

Watch in "Service Discovery"
You can see a service "ingress-xxxxxx" created but then disappeared soon

Found kube-proxy logs as below
{"log":"I1107 08:59:04.518888   13413 service.go:332] Adding new service port "default/ingress-ff53204a63e729b2b8734de61d74c257:" at <IP_ADDRESS>:80/TCP\n","stream":"stderr","time":"2019-11-07T08:59:04.519102432Z"}
{"log":"I1107 09:00:02.354143   13413 service.go:357] Removing service port "default/ingress-ff53204a63e729b2b8734de61d74c257:"\n","stream":"stderr","time":"2019-11-07T09:00:02.354269378Z"}

Result:
As service disappeared, the ingress will never work
Other details that may be helpful:
If I create the same ingress with Specify a hostname to use, this issue won't happen.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):

Rancher | v2.3.2
User Interface | v2.3.22
Helm | v2.14.3-rancher1
Machine | v0.15.0-rancher12-1

Installation option (single install/HA): HA

Cluster information

Cluster type : Deployed by RKE , HA installation , 3 Controller nodes, 3 Worker nodes.
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Machine type : Metal
CPU : 2P12C
Memory :

Master Node : 32 GB
Worker Node : 192GB

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.1", GitCommit:"4485c6f18cee9a5d3c3b4e523bd27972b1b53892", GitTreeState:"clean", BuildDate:"2019-07-18T09:18:22Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.5", GitCommit:"20c265fef0741dd71a66480e35bd69f18351daea", GitTreeState:"clean", BuildDate:"2019-10-15T19:07:57Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

Client:
 Version:           18.09.9
 API version:       1.39
 Go version:        go1.11.13
 Git commit:        039a7df9ba
 Built:             Wed Sep  4 16:51:21 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.9
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.11.13
  Git commit:       039a7df
  Built:            Wed Sep  4 16:22:32 2019
  OS/Arch:          linux/amd64
  Experimental:     false

We found this issue was caused by missing WorkloadID.
Just the same with the issue #16052

We found this issue is the same with
#16052

Close in favor of https://github.com/rancher/rancher/issues/24018


New version: Gyan.FFmpeg.Essentials version 7.0.1

Checklist for Pull Requests

[X] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/azp run


Fix make cookbook when certain cookbooks are guarded

This should fix issue #3520. These changes fix also  make cookbook_sphinx_linkcheck.
The command make cookbook still gives me warnings, though, but I don't understand if they are caused by my changes or if they were already there.
See warnings here: http://pastebin.com/chxfGvaq

Mmmh, I think you are pretty right. It was more like a "quick fix" than a proper solution. I refactored the code a bit to make it more readable and modular. This new approach should be better.
<off_topic>
Can I ask why we have to deal with missing dependencies? We could simply add these libraries as required to compile shogun, instead of setting up these "workarounds". Obviously, I'm sure that you have a bunch of good reasons for doing so. ;)
</off_topic>

Yes, an overwhealming number of good reasons for that :)
Come to IRC and we can discuss

I made the suggested changes @karlnapf.
By the way, I would definitely like to know more about these management choices about shogun dependencies. Are you shogun developers "always online" on IRC or there is a preferred time frame?

Fine to merge from my side. @vigsterkr ?

@geektoni IRC hours depend. I am UTC 11am-7pm usually. Others might differ.

go

Weird failures in travis, I just restarted to be safe.


Add doc for d2l-button-subtle.

The doc is modified from the original in the button repo.
I am conflicted about the screenshots to include.  In this case, it contains the normal, hover, and focus state screenshots, but I don't really want to do that for normal buttons and primary buttons, disabled states, etc.  It gets kinda crazy, especially if we do that for all components.  Animated gifs work okish, but the quality isn't amazing, and they are also much bigger.  I'm more inclined to include a simple screenshot, and if someone wants to see it in all the states with all the options and animations, they can run the demo.  So, I'm almost inclined to only provide one screenshot for subtle buttons - in the normal, non-hover, non-focus state.  Thoughts?
I will eventually add doc for the other buttons based on what we decide for screenshots.

Yeah I'm totally good with even just the normal state. Really folks just need to get a quick idea of what the component looks like so they know which one to explore further.


Add: Geckodriver support.

Refactor: WebdriverUrlInfo::version_urls now sorts drivers based on (major, minor) instead of (major).
Add: tarball extraction support based on url's suffix
Add: Geckodriver support

This breaks CI(no firefox installed), but #12 is still open so it will be fixed later when I handle #12.


docs: use open sans font and update slogan

Summary

Use open sans font in the website.
Update slogan, emphasize "Lightning Fast" instead of "Rspack-based".

before:

after:

Checklist

[ ] Tests updated (or not required).
[x] Documentation updated (or not required).

The Node.js new website) uses Open Sans and we like it 😄


Added Standard Page Objects

Added Standard Page Objects

I'm not a reviewer here, but I would like to point out a few things for your consideration.
Assertions
I don't think moving assertions into the page objects is going to make your life easier. You'll sooner or later be faced with assertions that cannot be easily built with the ones you've added. For example, how are users going to assert that:

A button becomes enabled when hovering it?
A links does not have a href attribute?
An element is visible for three seconds, disappears for five and then becomes green?

Probably through first retrieving the raw SelenideElement using element() and by then writing the assertions outside of the element's class themselves.
Rigid methods
All QTAF elements can be clicked, which is good. But:

Only buttons have a method isEnabled. Why can't users check whether an input is enabled?
Users can send keys to a FormElement. Why can't they send a key to a button, such as Enter?
A DateTextElement's content can immediately be retrieved as some Java date object. Why can't users retrieve a FormSelectElement's options as a date objects, e.g. for logging applications where a date can be selected from a dropdown?

Well, they can. By first retrieving the raw SelenideElement using element() and by then writing the interactions themselves using the Selenide element.
These page objects are way too rigid in my opinion. And some feel very arbitrary, like the NumberElement for Double only. Where's the Integer parsing? And with text and numbers covered, do we also need an ImageElement? There will be a lot of confusion for QTAF users when raw Selenide elements are scattered across the application.
All in all I think what bothers me is the following: If:

unwrapping the element offers more flexibility/makes the users' life easier
users won't be able to cover their entire application using QTAF elements

Why should they bother sticking to QTAF elements?


[Edit docs] un-escape quotes in examples

URL of the page to change: https://docs.stacks.co/docs/write-smart-contracts/clarity-language/language-functions

As seen in the screenshot above, the examples are showing up with the quotes escaped (\" instead of just ").

Thanks @obycode for pointing this out.

This issue is back.

Confirming this is currently fixed.


qx compile: add --qxpath option

In some cases, you'll need to use a path to the qooxdoo framework that is different from the one given in compile.json. For example, if you have the qooxdoo framework in ../qooxdoo/framework in your (committed) compile.json, but in your Travis CI travis.yml script, you need ./qooxdoo/framework. A --qxpath/-q option would allow to set this at runtime.

as Qooxdoo is a special case as libraries go, it would be appropriate to not list it in the libraries key at all; this would make it simple to swap qooxdoo repos, but is dependent on being able to reliably locate qooxdoo when not explicitly set (ie with --qxpath).
What would be really useful would be to persist data in a central location (eg ~/.qooxdoo/); configuration data could be in a json file in there, and could include things like the default or last used qooxdoo location.
This would be especially useful for sharing projects because the qooxdoo path is the one thing that always needs to be edited to suit local installations.  Because qx supports automatic location and download of contribs, the qooxdoo path is the one remaining thing which is not cross platform.
BTW, does qx support downloading missing contribs, eg like npm install downloads modules listed in package.json?

@johnspackman Re: missing contribs, yes, you just do qx contrib install and it will download the contribs listed in contrib.json.

We have a global qooxdoo path in the npm_modules path. The problem is that this path differs from installation to installation. What's about adjusting this path during compilation?

not longer needed in new compiler.
use qx config qx.library instead


chrome comand line args

Is there a way to pass chrome comand line args like  --kiosk-printing to the app
Any help is highly appreciated :)

@diegonakamashi not for now, no. Help welcome 🙂. Closing as duplicate of https://github.com/jiahaog/nativefier/issues/131


Using rez versioning regex in renovate

Hi, I'm a maintainer of the renovate bot and we have a community contribution at renovatebot/renovate#10930 to integrate your rez versioning scheme.
For this we need some parts of your versioning regexes.  Are we allowed to do so?

Hey,
I certainly don't mind personally, but I suppose it's a matter of determining what you're able to do given both our licenses (GNU GPL3 for me, GNU AGPL3 for you).
It looks like you're ok though: https://en.wikipedia.org/wiki/Affero_General_Public_License#Compatibility_with_the_GPL
If all you need to do is grab some regexes, I would just do that, and include a comment linking to the rez project.
Cheers
Allan

Thanks for info

Hi @nerdvegas,
Long time no see. Hope you're well.
Opening this issue again. After some discussion on the PR using the regular expressions, the consensus is that copying the code with a simple comment won't be good enough to be license compatible, unfortunately. Ideally the original author of the regular expressions, you, can provide a license to use them. Creating a simple PR with the regular expressions and signing the CLA will do exactly that.
Any possibility you are willing to help us out here some more and commit that PR (non-functional) with just the regular expressions and sign the CLA for the renovate bot project? That way we can get renovate to understand rez licensing right away.
I am including steps to help make this as easy as possible:

Fork renovate bot
Create lib/versioning/rez/index.ts
Include the following text:

    version_range_regex = (
        # Match a version number (e.g. 1.0.0)
        r"   ^(?P<version>{version_group})$"
        "|"
        # Or match an exact version number (e.g. ==1.0.0)
        "    ^(?P<exact_version>"
        "        =="  # Required == operator
        "        (?P<exact_version_group>{version_group})?"
        "    )$"
        "|"
        # Or match an inclusive bound (e.g. 1.0.0..2.0.0)
        "    ^(?P<inclusive_bound>"
        "        (?P<inclusive_lower_version>{version_group})?"
        "        \.\."  # Required .. operator
        "        (?P<inclusive_upper_version>{version_group})?"
        "    )$"
        "|"
        # Or match a lower bound (e.g. 1.0.0+)
        "    ^(?P<lower_bound>"
        "        (?P<lower_bound_prefix>>|>=)?"  # Bound is exclusive?
        "        (?P<lower_version>{version_group})?"
        "        (?(lower_bound_prefix)|\+)"  # + only if bound is not exclusive
        "    )$"
        "|"
        # Or match an upper bound (e.g. <=1.0.0)
        "    ^(?P<upper_bound>"
        "        (?P<upper_bound_prefix><(?={version_group})|<=)?"  # Bound is exclusive?
        "        (?P<upper_version>{version_group})?"
        "    )$"
        "|"
        # Or match a range in ascending order (e.g. 1.0.0+<2.0.0)
        "    ^(?P<range_asc>"
        "        (?P<range_lower_asc>"
        "           (?P<range_lower_asc_prefix>>|>=)?"  # Lower bound is exclusive?
        "           (?P<range_lower_asc_version>{version_group})?"
        "           (?(range_lower_asc_prefix)|\+)?"  # + only if lower bound is not exclusive
        "       )(?P<range_upper_asc>"
        "           (?(range_lower_asc_version),?|)"  # , only if lower bound is found
        "           (?P<range_upper_asc_prefix><(?={version_group})|<=)"  # <= only if followed by a version group
        "           (?P<range_upper_asc_version>{version_group})?"
        "       )"
        "    )$"
        "|"
        # Or match a range in descending order (e.g. <=2.0.0,1.0.0+)
        "    ^(?P<range_desc>"
        "        (?P<range_upper_desc>"
        "           (?P<range_upper_desc_prefix><|<=)?"  # Upper bound is exclusive?
        "           (?P<range_upper_desc_version>{version_group})?"
        "           (?(range_upper_desc_prefix)|\+)?"  # + only if upper bound is not exclusive
        "       )(?P<range_lower_desc>"
        "           (?(range_upper_desc_version),|)"  # Comma is not optional because we don't want to recognize something like "<4>3"
        "           (?P<range_lower_desc_prefix><(?={version_group})|>=?)"  # >= or > only if followed by a version group
        "           (?P<range_lower_desc_version>{version_group})?"
        "       )"
        "    )$"
    ).format(version_group=version_group)

Create a PR
Sign CLA

This is purely your choice, of course. Let us know if you have any questions or concerns we can help with.
Cheers,
Sebastian
P.S.: Here is the original thread about the license related concerns.

Hey Sebastian!
No worries, I'll get this done for you in the next few days.
Cheers
A
On Tue, Aug 3, 2021 at 5:20 AM Sebastian Kral @.***>
wrote:

Hi @nerdvegas https://github.com/nerdvegas,
Long time no see. Hope you're well.
Opening this issue again. After some discussion on the PR using the
regular expressions, the consensus is that copying the code with a simple
comment won't be good enough to be license compatible, unfortunately.
Ideally the original author of the regular expressions, you, can provide a
license to use them. Creating a simple PR with the regular expressions and
signing the CLA will do exactly that.
Any possibility you are willing to help us out here some more and commit
that PR (non-functional) with just the regular expressions and sign the CLA
for the renovate bot https://github.com/renovatebot/renovate project?
That way we can get renovate to understand rez licensing right away.
I am including steps to help make this as easy as possible:

Fork renovate bot https://github.com/renovatebot/renovate
Create lib/versioning/rez/index.ts
Include the following text:

version_range_regex = (
    # Match a version number (e.g. 1.0.0)
    r"   ^(?P<version>{version_group})$"
    "|"
    # Or match an exact version number (e.g. ==1.0.0)
    "    ^(?P<exact_version>"
    "        =="  # Required == operator
    "        (?P<exact_version_group>{version_group})?"
    "    )$"
    "|"
    # Or match an inclusive bound (e.g. 1.0.0..2.0.0)
    "    ^(?P<inclusive_bound>"
    "        (?P<inclusive_lower_version>{version_group})?"
    "        \.\."  # Required .. operator
    "        (?P<inclusive_upper_version>{version_group})?"
    "    )$"
    "|"
    # Or match a lower bound (e.g. 1.0.0+)
    "    ^(?P<lower_bound>"
    "        (?P<lower_bound_prefix>>|>=)?"  # Bound is exclusive?
    "        (?P<lower_version>{version_group})?"
    "        (?(lower_bound_prefix)|\+)"  # + only if bound is not exclusive
    "    )$"
    "|"
    # Or match an upper bound (e.g. <=1.0.0)
    "    ^(?P<upper_bound>"
    "        (?P<upper_bound_prefix><(?={version_group})|<=)?"  # Bound is exclusive?
    "        (?P<upper_version>{version_group})?"
    "    )$"
    "|"
    # Or match a range in ascending order (e.g. 1.0.0+<2.0.0)
    "    ^(?P<range_asc>"
    "        (?P<range_lower_asc>"
    "           (?P<range_lower_asc_prefix>>|>=)?"  # Lower bound is exclusive?
    "           (?P<range_lower_asc_version>{version_group})?"
    "           (?(range_lower_asc_prefix)|\+)?"  # + only if lower bound is not exclusive
    "       )(?P<range_upper_asc>"
    "           (?(range_lower_asc_version),?|)"  # , only if lower bound is found
    "           (?P<range_upper_asc_prefix><(?={version_group})|<=)"  # <= only if followed by a version group
    "           (?P<range_upper_asc_version>{version_group})?"
    "       )"
    "    )$"
    "|"
    # Or match a range in descending order (e.g. <=2.0.0,1.0.0+)
    "    ^(?P<range_desc>"
    "        (?P<range_upper_desc>"
    "           (?P<range_upper_desc_prefix><|<=)?"  # Upper bound is exclusive?
    "           (?P<range_upper_desc_version>{version_group})?"
    "           (?(range_upper_desc_prefix)|\+)?"  # + only if upper bound is not exclusive
    "       )(?P<range_lower_desc>"
    "           (?(range_upper_desc_version),|)"  # Comma is not optional because we don't want to recognize something like "<4>3"
    "           (?P<range_lower_desc_prefix><(?={version_group})|>=?)"  # >= or > only if followed by a version group
    "           (?P<range_lower_desc_version>{version_group})?"
    "       )"
    "    )$"
).format(version_group=version_group)

Create a PR
Sign CLA

This is purely your choice, of course. Let us know if you have any
questions or concerns we can help with.
Cheers,
Sebastian
P.S.: Here
https://github.com/renovatebot/renovate/pull/10930/files#r678945686 is
the original thread about the license related concerns.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/1110#issuecomment-891271552, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSR2NYQDIXR5LDSFXSTT23VXTANCNFSM5BGAHNJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Hi Allan (@nerdvegas),
That's great. Thank you.
Cheers,
Sebastian

Done, apologies for the delay.
Note that CLAassistant is complaining but I did the signing. I notice it's
saying "ajohns seems not to be a GitHub user", even though the PR is
clearly done by nerdvegas, not sure what's going on here.
Let me know if there's anything more you need.
A
On Wed, Aug 11, 2021 at 7:36 PM Sebastian Kral @.***>
wrote:

Hi Allan @.*** https://github.com/nerdvegas),
That's great. Thank you.
Cheers,
Sebastian
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/1110#issuecomment-896672006, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSVKOXDGJ5APUSWJ223T4JADTANCNFSM5BGAHNJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

My username/email git conf was old, I didn't notice. This is fixed and I've
declined the previous PR and put in a new one:
https://github.com/renovatebot/renovate/pull/11634
On Wed, Sep 8, 2021 at 9:14 AM Allan Johns @.***> wrote:

Done, apologies for the delay.
Note that CLAassistant is complaining but I did the signing. I notice it's
saying "ajohns seems not to be a GitHub user", even though the PR is
clearly done by nerdvegas, not sure what's going on here.
Let me know if there's anything more you need.
A
On Wed, Aug 11, 2021 at 7:36 PM Sebastian Kral @.***>
wrote:

Hi Allan @.*** https://github.com/nerdvegas),
That's great. Thank you.
Cheers,
Sebastian
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/1110#issuecomment-896672006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOUSVKOXDGJ5APUSWJ223T4JADTANCNFSM5BGAHNJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Avoid single-line function definitions.

I'd like to keep okjson.rb as close to their version as possible. We should exclude okjson.rb from all Rubocop checks.


Update try_binary and checked_ops, and remove math_checked_op

Signed-off-by: remzi<EMAIL_ADDRESS>Which issue does this PR close?

Closes #2715.
Rationale for this change

try_binary should not panic on unequaled array length
The checked ops (checked_add, checked_mul ...) is better to return a Result. Because

we can return the error directly in try_binary and other try_... kernels
We dont need to wrap the ops in the compute kernel, which can allow us to do some simplification on those private functions

Remove math_checked_op because it is the same as try_binary
Add tests for float division

What changes are included in this PR?

Are there any user-facing changes?

Mark this as a draft now because I also need to update the math_checked_divide_op

Hmm, tests failed on SIMD. Let me find a good solution.

@viirya  @liukun4515 @tustvold  PTAL, thank you.

Currently running the benchmarks for this, just in case, LLVM is a fickle beast

The benchmarks regularly fluctuate by +- 25%, this is not all that surprising given we're talking microseconds, but importantly we are not seeing anything greater which would imply a failure to vectorise correctly or similar :+1:


🐛  CMake Error: REACT_NATIVE_UTILS_LIB set to NOTFOUND

What were you trying to do?
Build my RN app on the Android emulator to prototype with frame processors.
Note: I checked the Expo box below. I'm running Expo bare, having ejected from Expo managed awhile back.
What happened instead?
Gradle build failed with a CMake error when trying to finish tasks for VisionCamera.
Relevant log output
/Users/dchiu/Developer/midana/client/node_modules/react-native-vision-camera/android/CMakeLists.txt : C/C++ debug|x86 : CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
REACT_NATIVE_UTILS_LIB
    linked by target "VisionCamera" in directory /Users/dchiu/Developer/midana/client/node_modules/react-native-vision-camera/android

Device
Emulated Pixel 4 running API 30
VisionCamera Version
2.4.2-beta.9
Additional information

[X] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Hey! Thanks for the bug report.
Could you try opening the CMakeLists.txt file in node_modules/react-native-vision-camera and remove all references to REACT_NATIVE_UTILS_LIB? That only exists for RN 0.64+, and I am not sure if I even need that lib.
https://github.com/mrousavy/react-native-vision-camera/blob/01104e093290acf88854d32e419b239b7ab01d2f/android/CMakeLists.txt#L129

Cool! I think that solved the initial build issue.
Seems like something in the build configuration (probably Hermes?) is messing with a very delicate part of my app, though, so that's preventing me from easily confirming that the camera component is working properly.
I'll let you know if this worked for me once I get past that.
For reference, I'm on react-native ~0.63.4.

Not totally relevant, but this is the exact issue currently preventing me from making this work: https://github.com/facebook/hermes/issues/114
My app injects JS into WebViews and Hermes appears to have changed the way that calling toString() on a function behaves. My design is inherently brittle, unfortunately, but the WebView is powering registration / login flows so is critical and writing long scripts directly as strings is hard to maintain so I've been trying to avoid that route.

I don't think I fully understand, are you having a problem with Hermes? If so, why are you not just using JSC then? Also there have been some changes with the Function.toString() function lately, you could try building Hermes from source from their current master.

Oh, sorry! Should have clarified. The only reason my project has Hermes turned on at all is because that was a step described in React Native Reanimated v2's installation guide. And my project only uses React Native Reanimated because that's the backing for Frame Processors.
Hermes is specifically showing [bytecode] for the body of strings when calling toString() on them, which raises errors when injected into a WebView haha.

REA also works with JSC! Try disabling hermes if that's a problem for you

@chiubaka

Hermes is specifically showing [bytecode] for the body of strings when calling toString() on them, which raises errors when injected into a WebView haha.

Hermes, started from 0.8.1, adds a special directive "show source" to make toString returning original source code for this use case. Would you mind try it with React Native 0.65-rc.3? You can find more details at https://github.com/facebook/hermes/issues/114#issuecomment-887106990

Hey @Huxpro, nice to see you here!
Does JS support using multiple directives at once? So I just write them like
"use strict"
"worklet"
"show source"
...

?

@mrousavy yup, directive prologue is a sequence of directives (i.e. kleene star) ;)

Oh that's awesome! Appreciate the support on all of this. Will give these things a try :)


Add readme fix build

fixes build from using the webbrowser crate, because this is only used in macos according to the cargo.toml
adds readme with some very basic instructions

Also, I didnt include it in this PR, because i dont think Its up to me to decide, but could you add a license to this project please?

Hi, and thanks! https://github.com/not-fl3/egui-miniquad/pull/8 already added a README.md

I'll add a license and fix the non-mac builds shortly! Thanks for bringing it to my attention


overlap between datasets

Some datasets are built on similar resources.
There is a concern that some task instances might overlap with each other.
It might make sense to add tests that check whether any task pairs have overlap in terms of their instances (input/output)

This is a good idea; I just don't know how we would go about implementing it. What if the formatting between instances of different tasks (but from overlapping data) is different?
For example, let's say an instance from task 1 is "I love NLP because it is \nCompletion A: fun to study. \nCompletion B: something I don't love." to which the desired output is "Completion A" whereas an instance from task 2 has "I love NLP because it is fun to study. I love NLP because it is something I don't love." to which the desired output is "Sentence 1".
How would we go about finding overlap? N-gram similarity? What are some other ideas we can think of, in case the data is more paraphrased/summarized (yet still similar enough) than these examples?

This is a good idea; I just don't know how we would go about implementing it. What if the formatting between instances of different tasks (but from overlapping data) is different?
For example, let's say an instance from task 1 is "I love NLP because it is \nCompletion A: fun to study. \nCompletion B: something I don't love." to which the desired output is "Completion A" whereas an instance from task 2 has "I love NLP because it is fun to study. I love NLP because it is something I don't love." to which the desired output is "Sentence 1".
How would we go about finding overlap? N-gram similarity? What are some other ideas we can think of, in case the data is more paraphrased/summarized (yet still similar enough) than these examples?

Right, the's a legitimate concern.
My intuition is that we can automatically detect task pairs that:

(A) the definition/instructions of the two tasks are the same (say, their token overlap is > 70%). This initial pruning should eliminate many candidate task pairs.
(B) if (A) is true, decide if there are instances that are equivalent(say, if their token overlap of "input" + "output" is > 80%)

These kinds of rules can go a long way in detecting cases like the one here: https://github.com/allenai/natural-instructions-expansion/pull/367#issuecomment-933939890 (repeated tasks for the translation problem).
Obviously, this will have false positives and false negatives. It shouldn't be difficult to add exceptions for the false positives. For the false negatives, we will need human intervention, unfortunately.
Thoughts, @swarooprm?


⚠ Bofrost.at has degraded performance

In 57d1f7b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2096 ms

Resolved: Bofrost.at performance has improved in 6b2baae after 57 minutes.


Release taskcat [0.9.1]

9356a56 Bump Version - Creating a new minor version
5b0c421 Removing tag requirement; resetting version to 0.9.0 in prep for 0.9.1
89193b2 Bump Version - Creating a new minor version
6ec0426 More revisions to travis logic
5e94239 Bump Version - Creating a new minor version
1729fe1 Simplify travis logic based on tags
1ffc257 Bump Version - Creating a new minor version
0114603 Updating Travis regex; closes #370
8e608e2 additional tests
b6be515 remove pyfiglet dependency
8e49abb Bump Version - Creating a new minor version
aca9c22 Adding skip_cleanup
f2cc0d4 Fixing if-statement in travis config
f840e02 add flag to keep only failed stacks
849a17f support legacy global overrides from ~/.aws/taskcat_global_override.json
5059c89 allow undefined parameters section in test. Fixes #355
46c73a1 Updating TravisCI Config to reflect build pipeline desired state
d6d3572 Add support for floating point parameters
473e5fa additional tests for Config
2df524d additional tests for Boto3Cache

Codecov Report

Merging #378 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #378      +/-   ##
==========================================
- Coverage   72.55%   72.51%   -0.04%
==========================================
  Files          30       30
  Lines        2700     2700
  Branches      460      460
==========================================
- Hits         1959     1958       -1
  Misses        646      646
- Partials       95       96       +1

Impacted Files
Coverage Δ

taskcat/_lambda_build.py
85.24% <0%> (-1.64%)
:arrow_down:

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 dea85a8...9356a56. Read the comment docs.


Expose cAdvisor WebUI through Caddy

Solves #272

Thank you, very nice. Do we a have to add this port also in docker-compose.yml for cadvisor in the expose list?

It looks like the docker-compose already exposes port 8080 for cadvisor

I mean for caddy -> ports

I mean for caddy -> ports

Actually - you're right.
Fixed it

I imagine we're going to see a lot of noise due to the 8080 port mapping because that's a very commonly used port but let's see how this goes.


Allow publications to be deposited into Scholarsphere from the last step/list in the new workflow

This implements the "Deposit" button described in https://github.com/psu-libraries/researcher-metadata/issues/715. Clicking this button will result in essentially the same behavior as submitting the Scholarsphere self-deposit form that already exists in RMD, so we should be able to reuse a lot of the same code/functionality.
If the publication has more than one associated file upload and only one is the version that can be deposited, then it should select that one file, or, if there is more than one file that is the version that can be deposited, then it should select one of those files arbitrarily. The selected file, along with the required metadata from the publication record should then be uploaded to Scholarsphere in the background. After the upload has succeeded, the publication's open access status and location information should be updated in RMD which will cause the publication to no longer appear in any of the lists and to no longer be picked up by any of the background jobs in this workflow. At least for now, the "Deposit" button should trigger a confirmation dialog.

The file sent won't necessarily be arbitrarily selected.  It will be the first file that is good to send to ScholarSphere.  Or in other words, the most recently uploaded file that is good to send to ScholarSphere.


Add "profile tag" to the profile, as currently you can't see it

As they are also a search filter, consider adding 'new tab click' to see everything with that tag?

Shows now


[test-framework] Various fixes for running with existing k8s environment

Also splitting out the kube settings/flags

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: nmittler, yutongz
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/OWNERS

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

/lgtm
/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ozevren
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers: nmittler, yutongz
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/OWNERS
tests/OWNERS

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


Substitute for 2024-12-10 Foundation board meeting

We need to select a substitute for the 2024-12-10 Foundation board meeting place of @rylev.
According to our policy, we need to make a public announcement (that's this issue) and agree by consensus to a substitute (also this issue).
cc @rust-lang/leadership-council

Closing as accepted. I remind @traviscross to coordinate ahead of time with the directors and the foundation to be ready for the meeting.


[BEAM-416] Use the beamhead label for containers

Use a special container label for Beam SDKs running E2E tests.

Note that we need to release the containers with this version to make sure the E2E tests will pass.
R: @robertwb, @aaltay please take a look

LGTM


Fix problem with duplicated VM update records

refer #1125

But still I think that if you prefer readability to compactness+efficiency (both are absolutely not critical in this case), then using Option is better than adding INVALID_BLOCK_ID and comparing it

Ok, I did the refactoring I had in mind but using Option, and rebased this. If the CI is happy and there no objections, I'll squash and push this later today.


Add support for multiple aliases and multiline comments

There are five individual commits in this pull request:

Converts the examples in the README into test cases. No functional change, just made it easier to verify my work in the later two commits.
Adds support for multiple aliases (to resolve https://github.com/daixiang0/gci/issues/1).
Adds support for multi-line comments (to resolve https://github.com/daixiang0/gci/issues/13).
Removes the completed TODOs from the README.md file.
Add support for imports appearing multiple times (at least once with and once without an alias).

Happy to split this changes into multiple pull requests if that would make reviewing easier. It was simpler while implementing them to build on top of the previous change with each new change.
My goal is that this is strictly backward compatible (so any file formatted with a current release of this formatter would not be changed with this version). It should resolve cases though where the previous current release of the formatter would have either removed comment information or duplicated an import statement.
Let me know if there is anything I can do to help clean up this change to make it easier to review.

I ran this against a large repository internally and discovered one more interesting side effect of this change.
If there is a comment associated with an import with multiple aliases, those import lines will be arranged next to each other in the final results and the comment will be above all of them. If there are multiple comments, they will be appended to each other.
That might not be the preferred behavior. I was going to look into updating the code tomorrow so comments are associated not just with an import package, but directly with the import package and alias pairing.


Implement generated client code for the single entity records

Description:
This task is to implement client code for the single entity record without any associations. This client code will get auto-generated by looking at the entities in the Ballerina project. each entity will have a Ballerina client.

Closing since completed in https://github.com/ballerina-platform/persist-tools/pull/15


WhatIf does not evaluate changes when using template specs in a Bicep template file

When using template specs for a module in my Bicep template, the what-if command seems to always return that there are no changes, even when there are obvious changes.
Doing a deployment works fine and the resources are updates as expected. However, there is now no way to verify changes before deployment.
I experienced this issue using both azcli and Powershell

Can you confirm that this only occurs with template specs? If you deployed the same template as a local module, does what-if work as expected?


fix: startup warning about deprecated TEMP_FAHRENHEIT and TEMP_CELSIUS in 2024.1.6

I am running HA Core 2024.1.6
Upon startup, I am seeing the following warnings.  Seemed simple enough to just fix instead of opening an issue.  This change (https://github.com/home-assistant/core/pull/81006) was made in HA core in 2022.11, so this shouldn't reasonably be introducing any backward compatibility issues.
2024-02-02 16:30:49.771 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from ha_carrier, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please create a bug report at https://github.com/dahlb/ha_carrier/issues
2024-02-02 16:30:49.782 WARNING (MainThread) [homeassistant.const] TEMP_FAHRENHEIT was used from ha_carrier, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.FAHRENHEIT instead, please create a bug report at https://github.com/dahlb/ha_carrier/issues
2024-02-02 16:30:49.805 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from ha_carrier, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please create a bug report at https://github.com/dahlb/ha_carrier/issues
2024-02-02 16:30:49.815 WARNING (MainThread) [homeassistant.const] TEMP_FAHRENHEIT was used from ha_carrier, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.FAHRENHEIT instead, please create a bug report at https://github.com/dahlb/ha_carrier/issues

Thanks for fixing this 😃


No data_transfrom function in the trainer of SGDetector

Hi, it seems that there is no data_transfrom function in the trainer of SGDetector.

It seems the data_transform() method was eventually found, haha. Cheers.


Meaning of LWW?

From the docs there's some quotes like:

data type UUID, e.g. lww,

I've not run across this term before, some cursory Googling Yandexing suggests it means "last write wins" which in the context of this log structured replicable data type would fit.
Could you clarify in the docs, or here, and I can send a Pr with the docs I'd have liked to have seen included?

https://github.com/gritzko/ron/blob/master/rdt/lww.go#L7
I added a comment like this.
Which docs do you like to be included?

Perfect, thanks - I missed it in the readme seeing lww five times :)


Added ability for MergeRequest Commits.

I added an ability to read the commits of merge requests.

@shivan Could you add some tests?

Has been merged with #46


doggo_setup.py doesn't work?

I use ODrvie V3.6-56V and burn the modified firmware. When I run the doggo_setup.py script, it just runs the motor calibration (beep~) and exit.  It seems haven't finished all the programs. Should it still have encoder calibration?. I check the errors in odrivetool, no error.

I found that it should use full_calibration(app_shutdown_token) rather than bare_bones_calibration(app_shutdown_token, reset=False).


Manpage erroneous "new executable project" example

In the examples section there's an example for "Create a new project that builds an executable", with the following code:
cargo new --init foobar
This is my first hour with Rust but I really think this should be:
cargo new --bin foobar
I'm on the latest stable, just installed via rustup.sh. I don't think this should be too hard to fix, but I don't want to jump to learning your contributing rules before learning anything beyond println!
(As an aside: wow, the learning curve to rust has surprisingly been very, very smooth, great job! The "Getting started" docs are a joy to work through. The manpages are still lacking, though, using --help for now.)

cc @carols10cents


Added sybase

I compiled it myself and it works, you can check my repo, it is part of php 8.3 too, it is compatible

I compiled it myself and it works, you can check my repo, it is part of php 8.3 too, it is compatible

All right. I'll test it beforehand and then implement it.


docker supplementary group not reloading causes VLAN configuration failure

When following the steps from https://github.com/nephio-project/docs/blob/main/content/en/docs/guides/user-guides/exercise-2-oai.md on a VM from start (including docker install) to finish, the below issue is encountered due to the docker supplementary group not getting reloaded.
[fedora@nephio-r2-e2e-vish-fedora-2 ~]$ ./test-infra/e2e/provision/hacks/vlan-interfaces.sh
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get "http://%2Fvar%2Frun%2Fdocker.sock/v1.24/containers/core-md-0-kw6vg-wqtfk-4chmk/json": dial unix /var/run/docker.sock: connect: permission denied
[fedora@nephio-r2-e2e-vish-fedora-2 ~]$ id
uid=1000(fedora) gid=1000(fedora) groups=1000(fedora),4(adm),10(wheel),190(systemd-journal) context=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023

The work around is to either logout and re-login into the VM or issue the newgrp docker command to reload the supplementary docker group
[fedora@nephio-r2-e2e-vish-fedora-2 ~]$ newgrp docker
[fedora@nephio-r2-e2e-vish-fedora-2 ~]$ id
uid=1000(fedora) gid=990(docker) groups=990(docker),4(adm),10(wheel),190(systemd-journal),1000(fedora) context=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023

The docs need an update to call this out so that users are aware.
cc: @electrocucaracha @rravindran123 @liamfallon

We also need to standardize the docker client calls to use or not sudo

https://github.com/nephio-project/test-infra/blob/main/e2e/tests/free5gc/007.sh#L55
https://github.com/nephio-project/test-infra/blob/main/e2e/provision/hacks/vlan-interfaces.sh#L25-L26

I think that this is not detected by the CI because the runuser instruction loads the latest information group (https://github.com/nephio-project/test-infra/blob/main/e2e/provision/init.sh#L155)


Add bash to PathLinker image

This updates the PathLinker Docker image to include bash as described in https://github.com/Reed-CompBio/spras/issues/130#issuecomment-2182842994
Before making the change, I versioned the prior image as v1 and pushed it to DockerHub. This update is now v2.

Expanding this pull request to also update AllPairs, which is Alpine based and has the same problem.


docs(*) Fix some typos

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/docs.konghq.com/blob/master/CONTRIBUTING.md
Summary
Fix spelling
Full changelog

Edited docs to fix spelling

Checklist:

[x] Commit message & atomicity checked
[N/A] Documentation
[x] Spellchecked my updates
[x] Ready to be merged

Good stuff, thanks very much @pushkyn !


Debug incompatible QoS

Sometimes, QoS policy incompatibilities arise when using foxglove bridge. When it happens, we cannot visualize the corresponding data on foxglove studio, whether it is coming from a ros bag/mcap or live publishers.
My bests guess for the moment: if foxglove bridge creates a subscriber on a certain topic, it will (supposedly) check the QoS policies of the other subscribers and will use the same one. If the foxglove bridge is already running when we launch our nodes, it could start before the others and use wrong QoS profiles.
Relevant parts of foxglove bridge source code:
choice of the QoS profile and PR where it was introduced

Thanks to this post I managed that in our case, the subscriber related to the line plots of Foxglove studio is the one selecting the incompatible policy (in this case durability that is set to TransientLocal instead of Volatile)


Support for MythicMobs

Hello
I downloaded the latest free build of MythicMobs but the console says your plugin doesn't support the current version.
ERROR:
[22:16:12] [Server thread/INFO]: Your MythicMobs version is not supported by Jobs! Supported versions: 4.9.1+
Jobs version:  <IP_ADDRESS>
Spigot version: (MC: 1.18.2) (Implementing API version 1.18.2-R0.1-SNAPSHOT) 3462-Spigot-ffceeae-2c47af0
Relevant plugins:
MythicMobs free version: 5.0.1-e876be12

Reopened this issue because your plugin is not working properly with MythicMobs.
Please update the dependencies.

Reopened this issue because your plugin is not working properly with MythicMobs.
Please update the dependencies.

Works perfectly fine with 5.0.4 free version and latest Jobs builds.


Refactor CAG refinement process

This refactor is purely for computational efficiency. Instead of iterating over the gene abundance zarr in every step, the CAG abundance is computed at each step and passed along to the next step in the process.

Passes local tests


Evaluation on Linemod_preprocessed

Hello,
I am able to train with your awesome work.
I trained on Linemod_preprocessed, now I want to evaluate on the same dataset.
Do I also have to run vanilla SegNet on Linemod? If I use python train.py --dataset_root=../datasets/linemod/Linemod_preprocessed I get an error message, saying:
FileNotFoundError: [Errno 2] No such file or directory: '../datasets/linemod/Linemod_preprocessed/data_syn/030571-label.png'

Do I need to change a path or is vanilla SegNet only for ycb?
Furthermore I also got some issues evaluating linemod with./experiments/scripts/eval_linemod.sh. After loading the object buffers I get this:
File "/home/katharina/Schreibtisch/DenseFusion-Pytorch-1.0/datasets/linemod/dataset.py", line 218, in mask_to_bbox
    _, contours, _ = cv2.findContours(mask, cv2.RETR_TREE, cv2.CHAIN_APPROX_SIMPLE)
ValueError: not enough values to unpack (expected 3, got 2)

solution in https://github.com/j96w/DenseFusion/issues/127#issuecomment-559854709

Evaluation works now, thank you.
But I still got the issue with vanilla SegNet as described above.

How did you solve the problem with the SegNet? -->
"Do I need to change a path or is vanilla SegNet only for ycb?"


Console command /locatebiome

Using that command the game breaks but using the /locate it seems to work fine. The world stops generating, you can't access chests and other inventories. When I quit the game it freezes in the "saving world" screen and I need to use task manager to close it. After that the launcher opens saying that the game crash with exit code 0 and when I look for a crash log report I don't find any.
Using fabric version - Terra-fabric-5.3.0-BETA+fd48f5f1-shaded-mapped.jar

Unable to replicate. You're also on an outdated version. Update to the latest version.

On the curse forge web site the 5.3.0 is the most recent release  but on the modrinth web site is a newer one.
I have download the newer one and the error continues here is a 15 minutes video of replicating the error a few times.
https://drive.google.com/file/d/13RnOEhgjrozo3esyVu_4gVzHJLfpkEEe/view?usp=sharing


Logo / Branding

Idea from Sheridan: maybe add a logo, signaling that the commons is like an App built on top of Ocean. I made some variations on a Commons logo that also refer to the Ocean logo as a source of inspiration.

Here is the .SVG file with all the layers selected (so all four variations together).
logo commons.zip


Separate GPU test jobs into general and Ampere-specific

This will be followed by switching the generic runners to be T4
machines. I think we need to roll things out this way with the extra
job also running on A100.
Part of https://github.com/openxla/iree/issues/14169

aaaand we're having capacity issues with a100s. Yet more reason to switch


Add a logo for choclo

The logo is based on the colorful corn varieties that exist in the Andes. Add it to the docs and the readme file.

Relevant issues/PRs:

Awesome logo (as always)! I'm merging this and we can start drafting the v0.1.0 release of Choclo. 🚀


Update issues

This PR updates the list of issues

marking the Weak Hash set crash as solved
adding the MingW Sys.rename issue

CI summary:

two threadomain crashes on Windows bytecode trunk and 5.0.0
a threadomain deadlock on Windows bytecode trunk
Linux 5.0.0 timing out since Lin DSL Weak HashSet test took a whopping 8505.5s


Client: get method with provider arguments

Summary
Follow up to https://github.com/Qiskit/qiskit-serverless/pull/1352
Adding provider argument to get method.
client = ...

function = client.get("my-func")
function = client.get("default/my-func")
function = client.get(title="my-func", provider="default")

But only title is never unique

so, if we only do api/programs/get?title="my-function" then we return Program named my-function where author is the user of request?
and if we api/programs/get?title="my-function"&provider="blah" then we return by function name and provider?

But only title is never unique. What do we do when user just requested my-function ?

If you request my-function from our current logic it will happen the next:

There is no provider specified so we are going to look for it by the author that is doing the request
author + title is unique, due to our logic in upload. If that title exists we update that function for that user, we don't create a new one

If you would like to get my-function from another user is something that we don't support by now and yes, if we would need to support that use-case definitely we would need to take a look to our entire logic.
Correct me if I'm wrong, Iskandar.

so, if we only do api/programs/get?title="my-function" then we return Program named my-function where author is the user of request?

and if we api/programs/get?title="my-function"&provider="blah" then we return by function name and provider?

Here I have more confidence about your Django knowledge than mine. I imagine that the end-point will need to look something like: api/programs/get/my-function?provider="blah" because Django is going to force us to do something like: get/{title}. I'm open to discuss if we should leave free that end-point for the id instead and we want to create another one like: api/programs/get_by_title/my-function?provider="blah"

CI is still failing due to https://github.com/Qiskit/qiskit-serverless/issues/1369


JS Basics Test

[x] Working HTML code cell in notebook based on wireframe
--> Math test code
[x] Working data types hacks in javascript code cell
--> Stores number answers in console
[x] Working javascript code cell in notebook that modifies elements in html code cell (DOM hack)
--> After clicking "submit," answers will show up in div
[x] Working javascript code cell in notebook based on javascript hack instructions
--> Creates submit button function so that the answers are both in console and printed
[x] Correcting errors hack that shows corrections made to the first three code cells in the 1.4 Correcting Errors page
--> Made changes to the given code located in the 1.4 Correcting Errors page

HTML Hack

The layout looks great, if you wanted to you could play around with the other tags. For example, adding links.
Data Types
Use of the console is great 👍  but there isn't much use of data structures other so maybe you could experiment with that later
DOM Hack
Awesome job referencing the tags in HTML and using js to modify the values.
It would be cool if you found a more efficient way add content to your answersDiv.innerHTML, especially if you wanted to expand on the code.
Javascript
Nice job with the submit button
Errors
Nice debugging skills 😎. I recommend adding more comments to your code because it can be easier to debug and understand what is happening in the code.


missing files for FANUC robots

i want to ask you if you you can add a folder with related files for following robots:

Fanuc R-2000iB 165F
Fanuc CR35iA

we want to use the examples from pinocchio.

I would be in favour of supporting a maximum number of robots.

@dadouz Could you add the robot inside this repo, it should be relatively easy.

I would also be in favored to add these two robots.
But remember that the mission of this package is to provide a set of robots for prototyping, unittesting and benchmarking.
Its intension is not to be used for real applications on robots.
@dadouz we are not planning to include ourself these robots. However, you're welcome to contribute on this topic.
Let us know if you are planning to integrate these robots, otherwise, I would kindly ask you to close this issue :)


Incorrect Path in .yoga/nexus.ts

Versions
OS: Windows 10
Yoga CLI: yoga/0.0.18
node : v10.11.0
Description
Inside the generated .yoga/nexus.ts  the context is imported using escape characters.
.yoga/nexus.ts
/**
 * This file was automatically generated by Nexus 0.11.3
 * Do not make changes to this file directly
 */
import * as ctx from "..\src\context"

Expected
import * as ctx from "../src/context"

Additional Info

This could be related to #128
Could possibly be an issue with nexus; Im not sure how this is generated

I wonder if this is connected to this feedback I see when I run node dist/src/index.js:
🚀  Server ready at http://localhost:4000/
GraphQL Nexus: Unable to find file or module /Users/devanbeitel/Projects/syllabus/api/dist/src/context.ts, skipping
GraphQL Nexus: Unable to find file or module /Users/devanbeitel/Projects/syllabus/api/dist/.yoga/prisma-client/index.ts, skipping
Missing required typegen import: ctx

@DevanB I get the same issue when running node dist/src/index.js but I am unsure whether they are related.
🚀  Server ready at http://localhost:4000/
GraphQL Nexus: Unable to find file or module C:\Users\willi\Documents\Atto-Byte\Development\api-server\dist\src\context.ts, skipping
GraphQL Nexus: Unable to find file or module C:\Users\willi\Documents\Atto-Byte\Development\api-server\dist\src\...yogaprisma-clientindex.ts, skipping
Missing required typegen import: ctx

This seems to be fixed but upgrading nexus to 0.11.6

This PR should fix this #105 @Weakky

PR merged, closing now 🙏


VCPKG installation ubuntu server 16.04 stuck

Hi all.
After installing the prerequisite programs as written in the readme. I ran
Linux:~/$ ./bootstrap-vcpkg.sh
Everything works as expected, but the programs gets stuck at building a CXX object:

-- The C compiler identification is GNU 7.4.0
-- The CXX compiler identification is GNU 7.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/g++-7
-- Check for working CXX compiler: /usr/bin/g++-7 -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- Configuring done
-- Generating done
-- Build files have been written to: /home/ubuntu/vcpkg/toolsrc/build.rel
[8/64] Building CXX object CMakeFiles/vcpkg.dir/src/vcpkg/base/strings.cpp.o

Any ideas how to fix this? Couldn't find anything on this problem online.
-EDIT: I thought the problem was fixed after restarting the os. But now it gets stuck at 17/64

Okay after retrying multiple times. I have also gotten stuck at 46,47. And in the end it did finish. However it is definitely not consistent.


Compilation failed with the newest LLVM build (PadTensorOP Removed)

Looks like they removed this Op in the newest LLVM build.
PadTensorOp was also removed from the doc
https://mlir.llvm.org/doxygen/structmlir_1_1linalg_1_1PadTensorOpTransformationPattern.html
below is the error message.
/home/spica/GitRepo/mlir-tv/src/analysis.cpp:295:30: error: ‘PadTensorOp’ is not a member of ‘mlir::linalg’; did you mean ‘InitTensorOp’?
295 | bool analyzeOp(mlir::linalg::PadTensorOp op, AnalysisResult &res) {
|                              ^~~~~~~~~~~
|                              InitTensorOp
/home/spica/GitRepo/mlir-tv/src/analysis.cpp:295:61: error: expected primary-expression before ‘&’ token
295 | bool analyzeOp(mlir::linalg::PadTensorOp op, AnalysisResult &res) {
|                                                             ^
/home/spica/GitRepo/mlir-tv/src/analysis.cpp:295:62: error: ‘res’ was not declared in this scope
295 | bool analyzeOp(mlir::linalg::PadTensorOp op, AnalysisResult &res) {
|                                                              ^~~
/home/spica/GitRepo/mlir-tv/src/analysis.cpp:295:65: error: expression list treated as compound expression in initializer [-fpermissive]
295 | bool analyzeOp(mlir::linalg::PadTensorOp op, AnalysisResult &res) {
|                                                                 ^
/home/spica/GitRepo/mlir-tv/src/analysis.cpp:295:6: warning: too many template headers for ‘{anonymous}::analyzeOp’ (should be 0)
295 | bool analyzeOp(mlir::linalg::PadTensorOp op, AnalysisResult &res) {
|      ^~~~~~~~~
/home/spica/GitRepo/mlir-tv/src/analysis.cpp:295:66: error: expected ‘;’ before ‘{’ token
295 | bool analyzeOp(mlir::linalg::PadTensorOp op, AnalysisResult &res) {
|                                                                  ^~
|

Thank you for reporting this issue!
We've been staying on previous LLVM build (b5a0f0f) for a while, so it won't compile with latest LLVM. There's a patch waiting to be merged, so it will hopefully be fixed soon :)


when training, the loss and depth loss are not convergent

Thanks for your brilliant work!
However, I print the loss and depth_loss when training and they have no tendency to converge. And I want to know what is wrong with my operation.
Thanks in advance for any help you can offer.
train:  55%|█████▌    | 1022/1856 [18:19<14:59,  1.08s/it, total_it=1021]
epochs:   0%|          | 0/80 [18:19<?, ?it/s, loss=10.9, lr=0.000101]
loss is tensor(19.3046, device='cuda:0')
loss_depth is 16.68878936767578
train:  55%|█████▌    | 1023/1856 [18:20<14:59,  1.08s/it, total_it=1022]
epochs:   0%|          | 0/80 [18:20<?, ?it/s, loss=19.3, lr=0.000101]
loss is tensor(8.4508, device='cuda:0')
loss_depth is 6.085319995880127
train:  55%|█████▌    | 1024/1856 [18:21<14:58,  1.08s/it, total_it=1023]
epochs:   0%|          | 0/80 [18:21<?, ?it/s, loss=8.45, lr=0.000101]
loss is tensor(7.5888, device='cuda:0')
loss_depth is 4.7100067138671875
train:  55%|█████▌    | 1025/1856 [18:22<14:56,  1.08s/it, total_it=1024]
epochs:   0%|          | 0/80 [18:22<?, ?it/s, loss=7.59, lr=0.000101]
loss is tensor(7.5831, device='cuda:0')
loss_depth is 5.188798904418945
train:  55%|█████▌    | 1026/1856 [18:23<14:56,  1.08s/it, total_it=1025]
epochs:   0%|          | 0/80 [18:23<?, ?it/s, loss=7.58, lr=0.000101]
loss is tensor(11.0090, device='cuda:0')
loss_depth is 8.783016204833984
train:  55%|█████▌    | 1027/1856 [18:24<14:55,  1.08s/it, total_it=1026]
epochs:   0%|          | 0/80 [18:24<?, ?it/s, loss=11, lr=0.000101]
loss is tensor(9.2116, device='cuda:0')
loss_depth is 6.269783020019531
train:  55%|█████▌    | 1028/1856 [18:25<14:55,  1.08s/it, total_it=1027]
epochs:   0%|          | 0/80 [18:26<?, ?it/s, loss=9.21, lr=0.000101]
loss is tensor(7.4908, device='cuda:0')
loss_depth is 4.79773473739624
train:  55%|█████▌    | 1029/1856 [18:26<14:52,  1.08s/it, total_it=1028]
epochs:   0%|          | 0/80 [18:27<?, ?it/s, loss=7.49, lr=0.000101]
loss is tensor(17.2144, device='cuda:0')
loss_depth is 14.055488586425781
train:  55%|█████▌    | 1030/1856 [18:27<14:52,  1.08s/it, total_it=1029]
epochs:   0%|          | 0/80 [18:28<?, ?it/s, loss=17.2, lr=0.000101]
loss is tensor(7.1547, device='cuda:0')
loss_depth is 4.631966590881348
train:  56%|█████▌    | 1031/1856 [18:28<14:50,  1.08s/it, total_it=1030]
epochs:   0%|          | 0/80 [18:29<?, ?it/s, loss=7.15, lr=0.000101]
loss is tensor(14.0801, device='cuda:0')
loss_depth is 10.272579193115234
train:  56%|█████▌    | 1032/1856 [18:29<14:50,  1.08s/it, total_it=1031]
epochs:   0%|          | 0/80 [18:30<?, ?it/s, loss=14.1, lr=0.000101]
loss is tensor(8.0110, device='cuda:0')
loss_depth is 4.475630283355713
train:  56%|█████▌    | 1033/1856 [18:30<14:48,  1.08s/it, total_it=1032]
epochs:   0%|          | 0/80 [18:31<?, ?it/s, loss=8.01, lr=0.000101]
loss is tensor(10.5358, device='cuda:0')
loss_depth is 7.642397880554199
train:  56%|█████▌    | 1034/1856 [18:32<14:49,  1.08s/it, total_it=1033]
epochs:   0%|          | 0/80 [18:32<?, ?it/s, loss=10.5, lr=0.000101]
loss is tensor(10.3984, device='cuda:0')
loss_depth is 7.659360408782959
train:  56%|█████▌    | 1035/1856 [18:33<14:48,  1.08s/it, total_it=1034]
epochs:   0%|          | 0/80 [18:33<?, ?it/s, loss=10.4, lr=0.000101]
loss is tensor(7.7434, device='cuda:0')
loss_depth is 4.804980278015137
train:  56%|█████▌    | 1036/1856 [18:34<14:46,  1.08s/it, total_it=1035]
epochs:   0%|          | 0/80 [18:34<?, ?it/s, loss=7.74, lr=0.000101]
loss is tensor(11.9586, device='cuda:0')
loss_depth is 9.924271583557129

Hi Gaozihui!
These are just loss values over 14 iterations. I recommend plotting the loss values over a longer duration using tensorboard to look at the trends. The training script should already output a .tfevents file you can use to plot.
Additionally, since we are using a small batch size (I assume batch size of 2 here), the loss values will vary quite a bit between iterations. However, the trend over a longer duration should be downwards.

Closing due to inactivity


Style: Update T+Cs

Updates T+Cs to 2022 version; retains previous version until transition - after which this can be removed.

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


Refactor for argocd

Touch up front end:

Use typer to handle env variables (Show in --help and do type casting/sanitising for things like export ENV_VARIABLE=false to read as boolean false)
Dynamic command hiding based on chosen backend and commands implemented
Consistant flow of user inputs for commands through backend rather than a mix of typer ctx, env reads, globals - to support validation, sanitising
Separate frontend (typer) from backend (particular: error handling)
Lazy import of monitor to reduce load time for everytime ec is called

Backend work:

Remove docker ("local" backend) and related
Implement an argocd backend
Remove handling of symlink charts for helm commands and replace with shared helm values
Allow changes to shared files to show as a new revision of a service
Only interact with "ec-services" - These are statefulsets which are tagged "ec_service=true"
Use custom errors over "typer error" so that targetted error handling can be done
Add schema validation and type hinting for polars schemas to interface more clear
Add tempfile context manager for file retension at debug and handle case of multiple temp files

After this is merged I proceed with the change the name of the project to ec-cli

Oh and I also tried it out on a k3s namespace.
I had issues that the PVCS were not getting created but that was nothing to do with you. So the helm template and ec work great for this use case too.

To clarify - Because we got rid of a lot of the complexity of the "local backend", the system tests no longer did very much & also started feeling over engineered with the minikube action


ImportError: cannot import name 'triu' from 'scipy.linalg'

After installing 'gliner' using the command "pip install gliner",  I received the following message: "Successfully installed flair-0.13.1 gliner-0.1.6" But I encountered the following error message when attempting to load the model with the following code:
from gliner import GLiNER
model = GLiNER.from_pretrained("urchade/gliner_base")
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.11/site-packages/gliner/init.py", line 3, in
from .model import GLiNER
File "/usr/local/lib/python3.11/site-packages/gliner/model.py", line 13, in
from gliner.modules.token_rep import TokenRepLayer
File "/usr/local/lib/python3.11/site-packages/gliner/modules/token_rep.py", line 4, in
from flair.data import Sentence
File "/usr/local/lib/python3.11/site-packages/flair/init.py", line 42, in
from . import (  # noqa: E402 import after setting device
File "/usr/local/lib/python3.11/site-packages/flair/models/init.py", line 1, in
from .clustering import ClusteringModel
File "/usr/local/lib/python3.11/site-packages/flair/models/clustering.py", line 14, in
from flair.embeddings import DocumentEmbeddings
File "/usr/local/lib/python3.11/site-packages/flair/embeddings/init.py", line 13, in
from .document import (
File "/usr/local/lib/python3.11/site-packages/flair/embeddings/document.py", line 16, in
from flair.embeddings.token import FlairEmbeddings, StackedEmbeddings, TokenEmbeddings
File "/usr/local/lib/python3.11/site-packages/flair/embeddings/token.py", line 10, in
import gensim
File "/usr/local/lib/python3.11/site-packages/gensim/init.py", line 11, in
from gensim import parsing, corpora, matutils, interfaces, models, similarities, utils  # noqa:F401
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/gensim/corpora/init.py", line 6, in
from .indexedcorpus import IndexedCorpus  # noqa:F401 must appear before the other classes
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/gensim/corpora/indexedcorpus.py", line 14, in
from gensim import interfaces, utils
File "/usr/local/lib/python3.11/site-packages/gensim/interfaces.py", line 19, in
from gensim import utils, matutils
File "/usr/local/lib/python3.11/site-packages/gensim/matutils.py", line 20, in
from scipy.linalg import get_blas_funcs, triu
ImportError: cannot import name 'triu' from 'scipy.linalg' (/usr/local/lib/python3.11/site-packages/scipy/linalg/init.py)
Additionally, when attempting to run "python -c 'import gliner; print(gliner.version)'", I encountered the same ImportError.

See https://github.com/urchade/GLiNER/issues/50

See #50
resolved, thanks


SGD names and descriptions updating?

Are we picking uop new SGD names and descriptions?
check out https://www.pombase.org/gene/SPBC8D2.16c
Saccharomyces cerevisiae | YGR283C | Putative methyltransferase more ... |
Saccharomyces cerevisiae | YMR310C | Putative methyltransferase more ...
buit at SGD
https://www.yeastgenome.org/locus/S000003515
UPA1 & 2

The SGD genes aren't updated automatically because I wasn't convinced that the script to query SGD InterMine would be reliable enough.  I've updated the data file now (for Monday morning) and added a note to the PomBase calendar to update it regularly in future.
https://github.com/pombase/pombase-chado/wiki/Updating-the-SGD-genes-file

Is  updated so closing


Data gets overriden on editor reload - closing and reopening

Godot version
4.0 beta rc5,rc4,rc3 ...
System information
Windows 10 Forward+
Issue description
When I close editor and open it again resources get overridden.
One material get changed to 1 other material that is constantly selected.
so it is like 1 material is constantly changed to 1 other material (I created 1 material and then another 3 and first got set into fourth created on meshes not affecting the second and third where it was setted)
(Using XR tools) when setting Viewport2Din3D scene it also gets overridden
example.
I set this

to scene called storage_ui_content.tscn
but it gets back to being settings scene after editor reopening
Also I searched is this bug described in other issue but I could not find it.
Steps to reproduce
Create multiple materials (I not sure how many 10, 15 mayby more)
Create double the amount of meshes - could be cubes
Set materials created before to cubes
Reopen editor
You should see some meshes having changed/not saved material you selected
Minimal reproduction project
N/A

Please provide an MRP. Asking contributors to create dozens of everything in hopes to reproduce the bug is not going to be productive.

ok gona try - it is preety much random, but I gona try

File: godot_bug_mrp.zip
Here is How scene should look

Here is how it looks on my side when I reload

Probably due to the same uid.

What method did you use to make both sets of materials? Did you use "Duplicate"?

I was duplicating this using rightclick, sometimes using the file system

by file system I mean cp paste to another folder

If you copied the files outside Godot then having duplicate UIDs is expected, because Godot can't detect and validate that.

still expected asset path was diffrent

The external dependencies are loaded by UID first and path is only used if the resource is not found under the cached path associated with the UID.
Finding duplicate UIDs is tricky. When you copy a file with UID, Godot will think that the file's path changed rather than it's a duplicate. The only way to detect duplicates is creating a map of all UIDs during a full scan, which is a very costly operation and there isn't really a good time to execute it (aside from manually).

mayby there should be system that says when loading project or importing new that found duplicate uids and what person does want do do with it - replace or make new

or button called regenerate uuid

What about counting all possible/importable assets and then checking number of existing uids
if number is higher then editor loaded/defined uids then it would look for duplicates mayby give some error and regenerate uids for duplicates.

I want to add to this that FileSystem dock does not support copy,cut and paste and that is why when somebody might need to copy multiple files they might use System File browser for that.

It does support all these operations via drag and drop or the "Move to..." menu option.
Protip: If you hold Ctrl while the file is being dropped, it will get copied instead of moving.

hm.. All I am gona say that that is not intuitive. When looking at "Move to..." menu option I see more like cut operation or drag and drop rather then copy, in rc5 it copied file that I moved keeping it references and creating unreferenced cp in folder from where I was copying.
Also I mistakenly typed there cut operation cause I just forgot it is just drag and drop operation.


Some unicode characters missing since 0.18.1-beta.42 (Minecraft versions >1.17.1-rc2, <22w11a)

This issue also happens without QSL or JourneyMap installed.
Expected behaviour:

Behaviour since beta.42:

Logs don't give seem to give any hint of what's happening
Logs across versions: https://gist.github.com/macbrayne/76b88173c36622dc5b9602a3184ec707
Reproduced on MultiMC and PrismLauncher, not sure if that matters
See: https://discord.com/channels/817576132726620200/1071067936304934952

This only happens with Force-Unicode enabled.
Further examples:

(renders fine without force-unicode)
Some languages on the other hand still render with the normal Minecraft font even with force-unicode enabled...

...but don't render with force-unicode disabled either (might be a separate issue).

Minecraft versions 22w11a (1.19) and up are not affected

Okay, this should be fixed in beta 68 - can you retry?

Fixed! ✔

Thanks for testing :)


#159065510 Build out REST Endpoints for Entries (CRUD)

What does this PR do?
Have CRUD on Entries Endpoints in MyDiary
Description of Task to be completed?
Have the following endpoints working

EndPoint
Functionality

GET  /api/v1/entries
Fetch all entries

GET  /api/v1/entries/<entryId>
Fetch a single entry

POST  /api/v1/entries
Create an entry

PUT  /api/v1/entries/<entryId>
Modify an entry

DELETE  /api/v1/entries/<entryId>
Delete an entry

How should this be manually tested?
To run your tests use
$ python manage.py test

To test endpoints manually fire up postman and run the following endpoints
What are the relevant pivotal tracker stories?
#159065510
#159065209

Nice work so far. I have clones it and tested the endpoints. All seems to be well. Just handle the quality review issues on codacy and you are good


Watch doesn't reload codegen.yml file on file changes

I'm trying to use watch mode on the codegen.yml file itself, because I'm adding a lot of new mappers.
But even though it outputs;
✔ Parse configuration
✔ Generate outputs

It doesn't take into account the newly added mappers.
When I quit watch mode and start it again, it correctly uses the new mappers.

Hi @koenpunt ! You are right, this is currently not supported.
But I think it might be simple to implement, we can add it in the future.

Fixed in 1.6.0.

Hi, I seem to be getting the same issue in 1.19.2

When making an update to codegen.yml the watcher outputs

but it doesn't actually generate any code.

@lioneltay this issue it outdated. Can you please report a new if this still relevant?
Thanks!


[460] migration remove unused feature flags

Context
This is the second PR, migration removing unused feature flags. The first PR is here.
Changes proposed in this pull request
Data migration to remove unused feature flags. It will be rebased and merged after the first one is merged
Guidance to review

Things to check

[x] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[x] Make sure all information from the Trello card is in here
[ ] Rebased main
[x] Cleaned commit history
[x] Tested by running locally
[x] Add PR link to Trello card

 Remove unused feature flags


css unit fix

Description
Fix typo use correct CSS unit for padding i.e. px instead of p
This PR fixes #282
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Thanks!


Upgrade to mbedtls 2.25.0

Upgrades the mbedtls-sys crate to version 2.25.0.
Warning The upstream mbedtls library deprecated the functions mbedtls_cipher_auth_encrypt and mbedtls_cipher_auth_decrypt to mbedtls_cipher_auth_encrypt_ext and and mbedtls_cipher_auth_decrypt_ext respectively. The new functions assume that the tag is appended to the ciphertext. The current PR exposes a similar requirement. This saves explicit memory allocation and copy operations, at the cost of a changed API.
See https://github.com/ARMmbed/mbedtls/blob/development/include/mbedtls/cipher.h#L1027
cc: @jethrogb

bors r+

bors r+


Update to new Mockito 2

Mockito released their second major version (v2.1.0). We could use some of the new features and should update ASAP.

Update is done in 2c2ecdba55befdbed89f97e33cc2b9adef9e3323