Is it possible to skip fetching git submodules for some repositories?

When we have a dependency from a github repository, we have two options:
git+https://github.com/owner/repo@commit
Or
https://github.com/owner/repo/archive/commit.zip
Using git is probably preferred, but it does submodule update at the beginning, and quite often this isn't needed for package building, and sometimes packages even depend on enterprise closed-source packages in their .gitmodules, making git option impossible
Would it be possible to add it to [tool.uv.sources], something like
[tool.uv.sources]
httpx = { git = "https://github.com/owner/repo", gitmodules=false }

(Another thing to add could be a flag to enable shallow git clone (--depth=1) for example?)
The second approach is to use URL, which doesn't fetch full git repo history and can navigate to subdirectories too
It works great and we fixed some cases in it recently, but here github gives us the bummer: it sets max-age=0 on any kind of archive download I could find (api, regular urls, anywhere). So it means that uv spends some time (600ms for me with 1-2 git repos) on each uv sync/uv run invocation. I know I can probably disable sync, but it would be ideal to instead somehow mark those URL dependencies as excluded from sync or something?
In any case, any of those 2 improvements would greatly speed up our workflow as it relies on many pinned commits in different source repos (the owners of which don't want to publish to PyPI unfortunately)
Thanks in advance!
P.S. Just for context of some gitmodules log (I don't need any of them to build the package, they are for people who want to build GUI app for example, and even then people can install libraries by their own, but it takes forever to download and even fails)
--- stderr
      Submodule 'contrib/electrum-locale' (https://github.com/Electron-Cash/electrum-locale) registered for path 'contrib/electrum-locale'
      Submodule 'contrib/libevent' (https://github.com/libevent/libevent.git) registered for path 'contrib/libevent'
      Submodule 'contrib/openssl' (https://github.com/openssl/openssl.git) registered for path 'contrib/openssl'
      Submodule 'contrib/secp256k1' (https://github.com/Electron-Cash/secp256k1.git) registered for path 'contrib/secp256k1'
      Submodule 'contrib/tor' (https://github.com/EchterAgo/tor.git) registered for path 'contrib/tor'
      Submodule 'contrib/zbar' (https://github.com/mchehab/zbar) registered for path 'contrib/zbar'
      Submodule 'contrib/zlib' (https://github.com/madler/zlib.git) registered for path 'contrib/zlib'
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/electrum-locale'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/libevent'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/openssl'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/secp256k1'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/tor'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/zbar'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/zlib'...
      Submodule 'boringssl' (https://boringssl.googlesource.com/boringssl) registered for path 'contrib/openssl/boringssl'
      Submodule 'krb5' (https://github.com/krb5/krb5) registered for path 'contrib/openssl/krb5'
      Submodule 'pyca.cryptography' (https://github.com/pyca/cryptography.git) registered for path 'contrib/openssl/pyca-cryptography'
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/openssl/boringssl'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/openssl/krb5'...
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/openssl/pyca-cryptography'...
      Submodule 'src/ext/rust' (https://git.torproject.org/tor-rust-dependencies) registered for path 'contrib/tor/src/ext/rust'
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/tor/src/ext/rust'...
      fatal: unable to update url base from redirection:
        asked for: https://git.torproject.org/tor-rust-dependencies/info/refs?service=git-upload-pack
         redirect: https://gitlab.torproject.org/tpo/team
      fatal: clone of 'https://git.torproject.org/tor-rust-dependencies' into submodule path '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/tor/src/ext/rust' failed
      Failed to clone 'src/ext/rust'. Retry scheduled
      Cloning into '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/tor/src/ext/rust'...
      fatal: unable to update url base from redirection:
        asked for: https://git.torproject.org/tor-rust-dependencies/info/refs?service=git-upload-pack
         redirect: https://gitlab.torproject.org/tpo/team
      fatal: clone of 'https://git.torproject.org/tor-rust-dependencies' into submodule path '/Users/alex/.cache/uv/git-v0/checkouts/93d858e1820fa8e2/6fc11cd9/contrib/tor/src/ext/rust' failed
      Failed to clone 'src/ext/rust' a second time, aborting
      fatal: Failed to recurse into submodule path 'contrib/tor'

@charliermarsh Sorry for ping! What is your stance on this feature? Technically I can try to open a PR to add gitmodules = false support as I know some Rust and I think it shouldn't be very complex. About second approach might be more complex though

I think something like this is reasonable (we've also tried to support shallow clones in the past), but it might be somewhat difficult to implement right now since we share a Git clone across packages. So we'd need some way to detect whether the repository was cloned with gitmodules=false or similar.


Cleanup ModelPage

The ModelPage was the first page thrown together and could use some love, pulling out components adding cleaning up the code

I can take a look at this in conjunction with this issue: https://github.com/covid-projections/covid-projections/issues/118

Duplicate of #118


Add SND_PCM_FORMAT_{DSD_U32_LE, DSD_U16_BE, DSD_U32_BE} formats

I would like to add some DSD formats. I use one of them (DSD_U32_BE) in a very basic audio player written in Rust. It plays DSF files (which contain DSD data) by outputting the DSD data directly to a DAC without having to convert to PCM.
I have a corresponding proposed update to diwic/alsa-rs which I will submit if you accept this one.
Thank you for all your work on this library — I’ve been having fun doing some audio programming in Rust and your efforts have made that all possible! :smiley:

Merged, thanks!


Links (a) display in one row (wrapping not working)

Will be fixed in the newest NuGet package. Please download the current sources and try the current implementation in one of the sample applications (SampleUwpApp or SampleWindowsStoreApp).

PS: HtmlTextBlock/FixedHtmlTextBlock have been renamed to ScrollableHtmlView/HtmlView.

Tried it and it works. Thanks a lot! :)

NuGet package will be released as soon as the final Windows 10 SDK is available...


Fails to open after Building and Deploying

Environment
Windows build number: 18362.327
Windows Terminal version (if applicable):Build from VS 2019, 01:12 2019/08/31

Steps to reproduce

I have selected CascadiaPackage as the Startup Project
I have set x64 and done Build and Deploy
When I try to run Windows Terminal, I get a window outline and shadow, and a crash dialog

Expected behavior
I expect the app to build, deploy, and be able to run
Actual behavior

Psychic debugging: VS 2019 Update 3 Preview?

Psychic debugging: VS 2019 Update 3 Preview?

Preview 2

I'm just going to repurpose this one for its real cause. Thanks.

Just cross posting from #2587 - This was working on Preview 2, then broke after some updates to 2 I guess. It is also broken in 3. Everything I've tried is not working.

Exception thrown at 0x00007FFDF6C8A839 in WindowsTerminal.exe: Microsoft C++ exception: winrt::hresult_error at memory location 0x000000DB2290DB38

#2587 is not fixed with this set of changes. The terminal still crashes on startup.

I have just successfully Cleaned, Built and Deployed the project, and I am able to run the terminal now!

#2587 is fixed now. Thanks!


openconnect: Start doesn't work after timeout

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
My openconnect server will automatically disconnect after a period of time without opnsense disconnecting. When this happens, starting the plugin again will not cause the interface to come back.
To Reproduce

Start openconnect
From the server side, disconnect the client.
Start openconnect client again and observe an ip address does not appear in interfaces.
Stop and start openconnect again and observe an ip address appears.

Diagnosis
I've determined that since the server side disconnects the client, the call to openconnect_stop is not made. Critically, the step to rename the ocvpn0 interface to tun3000 is not done. When starting the client again, it is unable to find the tun30000 interface because the stop procedure was not run.
https://github.com/opnsense/plugins/blob/58d06c7cc42dbb701a799507e8b6b875c70c42de/security/openconnect/src/etc/rc.d/opnsense-openconnect#L38
Potential solutions:

In the startup procedure, attempt to detect if the stop routine was not called and rename an existing ocvpn0 interface back to tun30000 before proceeding.
Remove the renaming of the ocvpn0 interface to tun30000. I am unaware of the implications of doing so and what the purpose of renaming the interface is in the first place. However, if this could be done, this would be the simplest solution.

I have the same issue

solved in this way:
I moved the IF configuration scripts from the launch file to the vpnc script hooks.
in the specific way:
from /usr/local/etc/rc.d/opnsense-openconnect  I removed

ifconfig ocvpn0 name tun30000

from the stop function and

sleep 5
ifconfig tun30000 name ocvpn0
ifconfig ocvpn0 group ocvpn

from the start function
Then, I created the hooks files under /usr/local/etc/vpnc  (create vpnc folder in case) with this tree
-- vpnc
---- attempt-reconnect.d
------ restore-ifname
---- disconnect.d
----- restore-ifname
--- post-attemp-reconnect.d
----- set-ifname
--- post-connect.d
----- set-ifname
with the following content for the files:
set-ifname

#!/bin/sh
ifconfig tun30000 name ocvpn0
ifconfig ocvpn0 group ocvpn

restore-ifname

#!/bin/sh
ifconfig ocvpn0 name tun30000

This change allows to reset the interface at original state if the connection drops on both sides (client or server) instead of only on client side.
To automatically re-establish a stable connection after a disconnection, I'm using Monit.
To better understand tha changes done, I created a PR ( #4126 ) with my changes, hope it helps !


Breakpoints don't work in C#

Godot version:
v.3.2.1.stable.mono.official
OS/device including version:
Windows 10
Issue description:
Set a breakpoint in C#, it gets ignored.
Steps to reproduce:
Create a minimal project, add a C# script, put break in _Ready function then run it. Breakpoints are ignored.
Minimal reproduction project:
bug-report-080420.zip

The built-in script editor only features basic C# support. Debugging isn't supported there, and there are no plans to support it.

I switched godot editor to Visual Studio Code but all I got was a blank editor in Godot. Wasn't clear what to do next.

Also assign the editor like this:
Editor (from top bar menu) > Editor Settings > Mono > choose external editor

One of the things I prefer with Godot over Unity3D is that I don't have to use an external editor for my C# code...
I would certainly welcome better C# editor support.

I think the only thing we can reasonably do for now is removing the ability to set breakpoints when editing a C# script. Implementing C# debugging support in the Godot editor would be a very large task for something that external editors can already do well.

@h5n1xp @imekon @Calinou
How is Godot so portable?

Godot comes with programming language, IDE and tools build in. These are bundled separate and are rather large in other engines (Unity and Unreal).
Godot design in general is a lot less bloated than Unity or Unreal, so features are implemented in simpler ways (See how scene system, resource system, scripting, rendering, shader language, 2D engine, etc. is implemented. They are all very simple and straight to the point in Godot, while complex and full of hacks/workarounds both in Unity and Unreal). This does not mean Godot is less powerful, just that it does the same in a less convoluted way. Likely reason is that we just had more time and freedom to think about it, experiment and rewrite systems as many times as necesary until we got them right. Proprietary engines are generally enslaved by their own design decisions, as hurting backwards compatibility can hurt their profits. In our case, both developers and community are more interested in the best possible engine over having a completely stable API.


Automated CI

Ideally need automated CI. To start with that, need multiple test accounts.
Types of repos:

empty repo
repo with:
** a filename that matches a report
** a file whose contents match a report
** team member who has done work in repo
** non-team member who has done work in repo

#142


how to check if marker is clustered?

when user select a marker I save a reference to it. but when user zoom out and marker goes into cluster and I want to do anything to marker get NullPointerException. How can I find that the marker is in cluster or is null? or have any listener to it when it cluster?

You might get a quicker response on StackOverflow with the tag google-maps. If you post there please link and close this issue. Thanks!

Easiest way to avoid NPE was to check if the marker with the saved reference is not null when getting it from the CustomRenderer:
if(customRenderer.getMarker(lastClickedMarker) != null) doSomething


When use a ScrollerView inside

I used a ScrollerView inside MotionLayout, and motionLayout won't respond to my actions when I try to swipe right. Please help me, Thinks.

I've solved this problem.


Re-implement HexWriter

Given a T: fmt::Write, HexWriter implements std::io::Write and writes the source bytes to its inner T as hex characters.
Fix: #84

Sorry for the disruption.
Last time, we discussed possibly storing additional state in HexWriter so that in the case where:

we successfully write some bytes,
but write_str eventually returns an error on some nth byte

we put HexWriter in an error state so that the next call to std::io::Write::write immediately returns an error instead of trying a write_str again.
@Kixunil raised performance concerns, I hope this is what he refers to.

The previous PR for this was #109 which mysteriously has been erased from history. (But it is in my browser history so I know it was there.)
I guess I need to start locally archiving github threads..
anyway c1f14567e47b450516f4b830ed1bb4ed8d29f93a looks good to me. As discussed in the previous thread, if the formatter somehow fails halfway through writing a byte, we don't detect this ... but there is also no way in the io::Write API for us to signal this, so I think we'll have to live with it.
...Actually, on further thought, we could signal it by setting a flag in the HexWriter that the user could check. But I think this is not worth the API complexity. In practice if users worry about half-written bytes and want to respond to it other than by giving up, they should probably write their own custom converter. It's not too hard.

As discussed in the previous thread, if the formatter somehow fails halfway through writing a byte, we don't detect this ... but there is also no way in the io::Write API for us to signal this, so I think we'll have to live with it.

My hope is that by using std::fmt::Write::write_str, it tells the formatter, either write the byte, or fail, don't write half a byte.
It works in the case where arrayvec::ArrayString is the formatter, but I am not sure about other formatters.

Let's give @Kixunil two weeks on this (from when the PR was opened) because he had some strong opinions on the last thread.

@tankyleo do you want this one or #117 to go in first?

Rebase:

Encoding performance PR broke this PR, fix it.
Convert HexWriter to a struct, to make the style more consistent with BufEncoder and BytesToHexIter.


Enable Travis CI

.travis.yml added.  @nadako could you turn this on at travis-ci.org when you get a moment?

Eh, i totally missed this issue somehow, sorry!

No worries.  Wasn't urgent.


Add version to databases path

close #882

Download last version of mesg-core (v0.8.1)
Deploy a new service with mesg-core
List services with ./dev-cli service list

Expected:
empty list of services
Actual:
➜  core git:(feature/db-versioning) ✗ ./dev-cli service list
rpc error: code = Internal desc = grpc: failed to unmarshal the received message unexpected EOF

mine works

@antho1404
This PR is only about adding path to core and mine is work as well


Kudu fails to select npm version on Azure using zipdeploy

I'm opening this issue because Kudu fails to select the NPM version when using zipdeploy.
This was working fine but today it stopped working with no changes in the code or any configuration.
I've tried with a new (clean) Web App and simple project (1 file, 0 dependencies) and the problem persists.
I've also tried with other Node versions (set in WebApp Application Settings), results are the same.
Repro steps.

create a new Web App (select NodeJS runtime in 10.1 version)
zip source files
post to web app using zipdeploy (...azurewebsites.net/api/zipdeploy)
zipdeploy script fails

Project structures.
files
index.js (simple http server from https://blog.risingstack.com/your-first-node-js-http-server/)
package.json (no dependencies)
package-lock.json

The log/error given by the failure.
2019-12-18T15:42:04  Fetching changes.
2019-12-18T15:42:04    Cleaning up temp folders from previous zip deployments and extracting pushed zip file /tmp/zipdeploy/d027c7b6-fc27-4af6-b5b3-879a26f49852.zip (45.90 MB) to /tmp/zipdeploy/extracted
2019-12-18T15:42:08  Updating submodules.
2019-12-18T15:42:08  Preparing deployment for commit id '0663cdb236'.
2019-12-18T15:42:08  Generating deployment script.
2019-12-18T15:42:08    Using the following command to generate deployment script: 'azure site deploymentscript -y --no-dot-deployment -r "/tmp/zipdeploy/extracted" -o "/home/site/deployments/tools" --node'.
2019-12-18T15:42:11    Generating deployment script for node.js Web Site
2019-12-18T15:42:11    Generated deployment script files
2019-12-18T15:42:12  Running deployment command...
2019-12-18T15:42:12    Command: "/home/site/deployments/tools/deploy.sh"
2019-12-18T15:42:12    Handling node.js deployment.
2019-12-18T15:42:12    Kudu sync from: '/tmp/zipdeploy/extracted' to: '/home/site/wwwroot'
2019-12-18T15:42:12    Copying file: 'npm-shrinkwrap.json'
2019-12-18T15:42:13    Detecting node version spec...
2019-12-18T15:42:14    Using appsetting WEBSITE_NODE_DEFAULT_VERSION value: 10.1
2019-12-18T15:42:14    ENOENT: no such file or directory, open '/opt/nodejs/10.1.0/npm.txt'
2019-12-18T15:42:14    Node.js versions available on the platform are: 4.4.7, 4.5.0, 4.8.0, 6.2.2, 6.6.0, 6.9.3, 6.10.3, 6.11.0, 6.17.1, 8.0.0, 8.1.4, 8.2.1, 8.8.1, 8.9.4, 8.11.2, 8.12.0, 8.15.1, 8.16.2, 9.4.0, 10.1.0, 10.10.0, 10.14.2, 10.17.0, 12.13.0.
2019-12-18T15:42:14    Resolved to version 10.1.0
2019-12-18T15:42:14    Detecting npm version spec...
2019-12-18T15:42:14    An error has occurred during web site deployment.
2019-12-18T15:42:14    select node version failed
2019-12-18T15:42:14    ENOENT: no such file or directory, open '/opt/nodejs/10.1.0/npm.txt'
/opt/Kudu/Scripts/starter.sh "/home/site/deployments/tools/deploy.sh"
2019-12-18T15:42:14

Mention any other details that might be useful.
I think this is somehow related with #3057 .
Thank you

The current workaround for this can be found here.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Formatting JSX fragments results in @JSX internals

Link: https://rescript-lang.org/try?code=LYewJgrgNgpgBAIQgF2SAdnAvHA3gKDjgAEAnGAQwGNkA6KkYABwxnWULlmTmAoGt4OABQBKbAD48nIgB4ARijToJBIuriyJ+AJA7cAJUo1aAZ2SkAlugDmwgETIY5+6IC+uuQHptGt7K9FVAxfOA8PIA
module Button = {
  @react.component
  let make = () => {
    <button>{
      <>
		{React.string("test")}
	  </>
    }</button>
  }
}

reformats to:
module Button = {
  @react.component
  let make = () => {
    <button> {@JSX list{{React.string("test")}}} </button>
  }
}

Thanks for the report! We already fixed this in master: https://github.com/rescript-lang/syntax/commit/db0c6b903eea21a4b18aab06f2469b0349bd5daa Will go out soon in the next compiler release.

Thanks. It would be awesome if we can get a playground that tracks the master branch, so we can easily double check these things against master :)


chore(deps): update terraform terraform-ibm-modules/observability-instances/ibm to v2.17.0

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/observability-instances/ibm (source)
module
minor
2.15.0 -> 2.17.0

Release Notes

terraform-ibm-modules/terraform-ibm-observability-instances (terraform-ibm-modules/observability-instances/ibm)
v2.17.0
Compare Source
Features

added support to create an IBM Cloud Logs atracker target.

Root level module has new input at_cloud_logs_targets. The new input in the activity_tracker submodule is called cloud_logs_targets

The following root level module variables have been renamed:- cos_targets -> at_cos_targets- eventstreams_targets -> at_eventstreams_targets- log_analysis_targets -> at_log_analysis_targets
source_id and source_name have been removed from the cloud_logs_existing_en_instances input variable (#558) (0f9d68c)

v2.16.0
Compare Source
Features

Add new outputs cloud_logs_ingress_endpoint and cloud_logs_ingress_private_endpoint for Cloud logs (#566) (e687aa3)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

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


Better API error messages for malformed tag names

x-ref https://github.com/aai-institute/lakefs-spec/issues/282.
tl,dr: We have an internal user that got stuck with API errors creating a tag with spaces in the name, which is apparently not allowed. The error message, however,
lakefs.exceptions.BadRequestException: code: 400, reason: Bad Request, body: {'message': 'argument tagID: tag id: invalid value: validation error'}

is not helpful. I'm guessing it comes from the server, since otherwise we would see pydantic client-side errors. This could use a better hint at what went actually wrong.
What would be even cooler is if the OpenAPI-generated pydantic type got a custom validator that catches exactly this, since then we wouldn't even make it to the API request. Another (less nice) way to achieve this would be to place a quick validation handler into the tag creation method in the Python wrapper.

The actual validation logic is defined here, for anyone who's willing to work on this: https://github.com/treeverse/lakeFS/blob/857e22bc0b6c0aa3e54729a70037371dc1c04327/pkg/graveler/validate.go#L49-L80


Fix cypher for SSL error in ESP32-lwmqtt example

Hi.
As shown in #221, the current ESP32-lwmqtt example gives a cypher for SSL error.
In this PR, I will fix this problem by modifying root_cert and setting root_cert to WiFiClientSecure instance.
This solution is mentioned in https://github.com/GoogleCloudPlatform/google-cloud-iot-arduino/issues/221#issuecomment-817110811 .
Thank you.
[E][WiFiClientSecure.cpp:133] connect(): start_ssl_client: -1
not connected
Settings incorrect or missing a cyper for SSL
Connect with mqtt.2030.ltsapis.goog:8883
ClientId: projects/***
Waiting 60 seconds, retry will likely fail

Tested on ESP32, confirmed to work, merging.


Implementation (optional parts)

[ ] Linkifies JIRA identifiers [SENG-1234] in GitHub UI - Commit Messages, Reviews, Issues and Pull Requests
[ ] Linkifies JIRA identifiers [SENG-1234] in GitHub notifications - email digest
[x] Associates JIRA project to 1 or more GitHub repositories
[x] Associates one GitHub repository with 1 or more JIRA projects

Hi @andymckay (again). Added the to triage label to this one as well for my team to look into 😄

Hi @andymckay. I've created a ticket in our backlog for someone on my team to investigate implementing your suggestions. We'll keep you posted on the progress. Please stay tuned.


In CellMark

The Taxon should be on the proteins not the GO terms.


The "PerResolve" lifetime is not properly respected when the resolving interception behaviors for an object

I've noticed some recent activity on these repos so I'm going to shoot my shot.
I am upgrading some code (that's an understatement) from Unity 3 to Unity 5 and have encountered what is either a bug or a significant behavioral difference regarding interception between the versions.
In Unity 5.11 (and likely earlier versions, I haven't explored deep enough to find where the issue starts), Unity will re-resolve objects registered with the "PerResolveLifetimeManager" when resolving objects needed for any defined interception behavior. I have verified that the same object will be used across the object graph for any reference that is not part of interception behavior.
Here is a test case that should demonstrate the issue. I tried another lifetime ("PerThread") to demonstrate that it does seem to specifically be an issue with "PerResolve".
UnityPerResolveInterceptionBrokenPoc.zip
I've pulled the interception repo and have been trying to uncover the issue, but I'm quite a bit out of my depth. Depending on the scope of the fix (I would have to evaluate fixing this vs other options for my team to consider), I'd be willing to submit a PR if I can get pointed in the right direction.

If you could provide a PR, I would be happy to release a new version with the fix. Interception is not very high on my priority list at the moment, I could be useful only in an advisory capacity.

If your need is great enough to hire me as a consultant you will have my complete and undivided attention until the problem is fixed.


Add special handling of nonzero constraints

During the preprocessing phase the plonkish backend is constructing the full SNARK expression that will later be "sumchecked" (actually, zero-checked using the sumcheck protocol). Normally, the expression represents the circuit wiring, lookups, etc., and also custom gates. In Halo2 frontend custom gates are normally represented as expressions that are zero on the evaluation domain. E.g., if the goal is to constrain a to be equal to b, the gate looks like a - b = 0. In this hyperplonk implementation such Halo2 gates are included into the final SNARK expression as a random linear combination of "zero checks on every row". For example, if the gate is a - b = 0, the circuit may look something like this:

Col
a
b
Constraint eq.

Row0
3
3
3-3=0

Row1
5
5
5-5=0

The important point here is that each row is checked independently due to the random linear combination of constraints (see the link above). This is essential for regular use cases where each row represents some mathematical identity of interest. After the "distribute powers" operation and multiplication by eq the linear combination of constraints may look like this: (3-3) + rnd_num * (5-5) = 0. Meanwhile, in our case each row is simply a user balance value (let's not take the range check into account for a moment). We would benefit from a "vertical" constraint "along the column", but applying it directly using large rotations is not feasible. Instead, we can modify the plonkish backend logic to allow for special expressions that are excluded from random linear combination with eq and, therefore, summed directly. For example, if we want to prove the sum of two balances 3 and 5 being equal to 8, our desired circuit may look like the following:

Col
Balance

Row0
3

Row1
5

Row2
-8

As you can see, if we had a way to transform such circuit into an expression by directly querying and summing the cells, we would satisfy a sumcheck of 5 + 3 = -8. The little trick here is to put the negative grand total at the last row.
This pull request enables the plonkish backend to construct such nonzero expression by handling nonzero gates in a special way. A nonzero gate is a Halo2 gate that would be considered invalid in a typical Halo2 circuit because its expression does not evaluate to zero. For example,
meta.create_gate("Nonzero gate", |meta| {
                let mut nonzero_constraint: Vec<halo2_proofs::plonk::Expression<Fp>> = vec![];
                nonzero_constraint.push(meta.query_advice(balance, Rotation::cur()));
                nonzero_constraint
            });

As you can see, the nonzero_constraint here is just the query to a single cell. In the plonkish backend we can detect such a constraint by parsing the expression and observing that it does not have any arithmetic operations (this is a simple approach I came up with, although it's definitely not exhaustive). Then, we can exclude this nonzero constraint from random linear combination.
Note that the gate defines a single expression that holds along all the rows of involved columns. If we have multiple currencies with respective negative totals in the last row, we may combine their nonzero expressions into one sum across the columns. Consider the following circuit:

Col
Balance0
Balance1

Row0
3
2

Row1
5
7

Row2
-8
-9

If we combine the column nonzero expressions when including them into sumcheck, it will look like the following: (3+2)+(5+7)+(-8-9)=0, and the identity still holds.

Looks good! Thanks for the detailed explanation about the attack vector in the inline comment.


Fix PHP 8.1 return type deprecation notices.

Thanks @alexei-bykovski

2.2.2 released :+1:

@zbateson, thanks a lot 🤗


Shade protobuf-java in runners/google-cloud-dataflow-java

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Coverage decreased (-0.003%) to 70.477% when pulling c668642ab5b4e5df0b0db93d6325191cdfd11b88 on vikkyrk:df_shade into 78a4b8e4fb014d40f7f6ea3117931f185b9e742b on apache:master.

Retest this please

Coverage decreased (-0.1%) to 70.368% when pulling c668642ab5b4e5df0b0db93d6325191cdfd11b88 on vikkyrk:df_shade into 78a4b8e4fb014d40f7f6ea3117931f185b9e742b on apache:master.

Better to remove the single dead code usage which has protobuf in CustomSources


[Bug]: API Key created in https://console.cloud.google.com/apis/credentials cannot be used

File Name
https://cloud.google.com/vertex-ai/generative-ai/docs/start/quickstarts/quickstart-multimodal#gemini-setup-environment-drest
What happened?
Do we must use gcloud auth print-access-token?
Relevant log output
It return `status: 401`, invalid_token\

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You can't use API keys for Vertex AI, you must use IAM. gcloud auth print-access-token is one way of accessing.

Note: If you are familiar with Gemini API in Google AI Studio, note that Gemini API for Vertex AI uses Identity and Access Management instead of API keys to manage access.

But the IAM will be expired which cannot build a robust and stable AI service from DEV side


Update Education icon to match rating in section

https://github.com/Gabriel439/post-rfc/blob/master/sotu.md#education has the rating as Mature and I updated the icon accordingly.  I'm not able to know which is correct.

Yeah, I think it's fine to call it mature at this point


rying to add a disposable to a DisposableStore that has already been disposed of. The added object will be leaked!

todays insiders
Seeing the error on the console. The trace points to this line: https://github.com/microsoft/vscode/blob/b2f203113c155f81a92fa5448b69b6170d16b2f7/src/vs/workbench/contrib/searchEditor/browser/searchEditorInput.ts#L151

workbench.desktop.main.js:17 Error: Trying to add a disposable to a DisposableStore that has already been disposed of. The added object will be leaked!
    at h.add (workbench.desktop.main.js:17)
    at Ji._register (workbench.desktop.main.js:17)
    at workbench.desktop.main.js:2482
    at async Ji.save (workbench.desktop.main.js:2481)
    at async Promise.all (index 0)
    at async Object.G [as settled] (workbench.desktop.main.js:68)
    at async x.save (workbench.desktop.main.js:2507)

Wasn't able to repro but pushed something that should fix.

verified in source


Glitchy mess when using viewport scaling and gl_compatibility renderer

Godot version
4.0.rc1
System information
Linux, gl_compatibility
Issue description

The outside of the viewport is not cleared or black like with other renders
Steps to reproduce
Make a new project with the gl_compatiblity renderer and turn display/window/stretch/mode to viewport or canvas_items
Minimal reproduction project
Minimum Repro.godot.zip

Duplicate of https://github.com/godotengine/godot/issues/71799


Filter through Europeana object to show only completeness [10]

User Story
As a user I need a result that has complete data in order for me to know the location and details about a piece of work.
Acceptance criteria
When my search results only logs objects with full completeness of data


GitHub Raffle at SOOC24 - Leave a comment to enter

Random Name Picker for GitHub prize draw at State of Open 2024!
:ticket: To enter the draw, add a comment at the bottom of this issue👇
🏆  The prize will be drawn on Wednesday Feb 7th at 13:30
If you would like a free GitHub certification voucher we also have 100 to give away (first come first served).   The Actions workflow pulls entrants from commenters of a GitHub Issue
:pray: Finally, huge shout-out to @icelam for a great and simple name picker we could fork and use for the GitHub booth!

A couple of T&C's:

Entry into this raffle will NOT constitute your consent to be contacted for anything else
We will not be adding you to our marketing database (but if you want to subscribe to our newsletter, you can do so here)
The raffle prize cannot be swapped or changed,
The winner of the prize at State of Open 2024 MUST be present when we draw a winner, otherwise we will redraw!

Good luck!

Santhosh was here

Hi

Hello :)

Hello

Pick me 🙏🏻

Hello!

Hi I am at GitHub table

Hi 👋🏻

Great event

Awesome!

Hello! 🚀

Hello!

🙏🙏

🌱

git push

I’m in!

Thanks, nice stickers!

Great stickers

+1

Hi

:rocket: wohoooo

:octocat:

+1

Hey :)

Cool Swags!

I literally created a GitHub account so I could put my name in the draw! 🤣 My tech peeps will be thrilled.

Thanks for your support of Open Source!

[x] ✍ leave a message
[x] 🗳 this gets converted as an entry
[ ]  🤞 wait till my name comes up because everyone else is eating lunch and forgot to come to the stand 🤭 (wait, have I forgotten it too??)

Hii!! GitHub is cool!!

Ping!

Heya 👋🏽

🎉 Reminder we’ll be doing the draw at 13:30 today!


Changed to directory-based scan

Changed script to have cast.exe scan directory paths instead of individual files to improve performance

My testing shows this is significantly faster - letting cast.exe walk the directory rather than trying to search it via PS1


Bladebit 3.1.0-beta sometimes double the time for one plot or more

Hi have to report some issues with my system
ryzen7 7800X3D
128GB RAM
1x1TB tmp nvme
gtx 1080
my command to plot with is
   .\bladebit_cuda.exe  -z 4 -f farmerkey -c contract  cudaplot --disk-128 -t1 l:\temp\  l:\temp\1\
in a powershell loop to dont have problems with 128+ plots error
my result normally is:
` Bladebit Chia Plotter
Version      : 3.1.0-beta1
Git Commit   : 076eba490f1c08b3a7bf10ea0a08f80be758c7b9
Compiled With: msvc 19.29.30151
[Global Plotting Config]
Will create 1 plots.
Thread count          : 16
Warm start enabled    : false
NUMA disabled         : false
CPU affinity disabled : false
Farmer public key     :
Pool contract address :
Compression Level     : 4
Benchmark mode        : disabled
[Bladebit CUDA Plotter]
Host RAM        : 127 GiB
Direct transfers: true
Selected cuda device 0 : NVIDIA GeForce GTX 1080
CUDA Compute Capability   : 6.1
SM count                  : 20
Max blocks per SM         : 32
Max threads per SM        : 2048
Async Engine Count        : 5
L2 cache size             : 2.00 MB
L2 persist cache max size : 0.00 MB
Stack Size                : 1.00 KB
Memory:
Total                    : 8.00 GB
Free                     : 7.02 GB
Allocating buffers (this may take a few seconds)...
Kernel RAM required       :<PHONE_NUMBER>0  bytes ( 87787.46  MiB or 85.73  GiB )
Intermediate RAM required :<PHONE_NUMBER>   bytes ( 4160.07   MiB or 4.06   GiB )
Host RAM required         :<PHONE_NUMBER>8  bytes ( 27008.00  MiB or 26.38  GiB )
Total Host RAM required   :<PHONE_NUMBER>28 bytes ( 114795.46 MiB or 112.10 GiB )
GPU RAM required          :<PHONE_NUMBER>   bytes ( 5855.03   MiB or 5.72   GiB )
Allocating buffers...
Table pairs allocated as pinned: false
Generating plot 1 / 1: 5b31db7c2d978d53aa30c60d7dfdc1fc2f0a81ccefd9e473ce26ffe6ffe194aa
Plot temporary file: l:\temp\1\plot-k32-c04-2023-09-12-18-54-5b31db7c2d978d53aa30c60d7dfdc1fc2f0a81ccefd9e473ce26ffe6ffe194aa.plot.tmp
Generating F1
Finished F1 in 6.65 seconds.
Table 2 completed in 24.46 seconds with<PHONE_NUMBER> entries.
Table 3 completed in 38.76 seconds with<PHONE_NUMBER> entries.
Table 4 completed in 62.86 seconds with<PHONE_NUMBER> entries.
Table 5 completed in 49.92 seconds with<PHONE_NUMBER> entries.
Table 6 completed in 44.92 seconds with<PHONE_NUMBER> entries.
Table 7 completed in 34.99 seconds with<PHONE_NUMBER> entries.
Finalizing Table 7
Finalized Table 7 in 14.45 seconds.
Completed Phase 1 in 277.48 seconds
Marked Table 6 in 11.08 seconds.
Marked Table 5 in 11.43 seconds.
Marked Table 4 in 11.08 seconds.
Marked Table 3 in 10.89 seconds.
Completed Phase 2 in 44.48 seconds
Compressing Table 2 and 3...
Step 1 completed step in 21.95 seconds.
Step 2 completed step in 21.18 seconds.
Completed table 2 in 43.13 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 80.08% ).
Compressing tables 3 and 4...
Step 1 completed step in 18.71 seconds.
Step 2 completed step in 22.66 seconds.
Step 3 completed step in 22.59 seconds.
Completed table 3 in 63.97 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 80.69% ).
Compressing tables 4 and 5...
Step 1 completed step in 19.55 seconds.
Step 2 completed step in 22.62 seconds.
Step 3 completed step in 22.67 seconds.
Completed table 4 in 64.84 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 82.24% ).
Compressing tables 5 and 6...
Step 1 completed step in 18.11 seconds.
Step 2 completed step in 22.58 seconds.
Step 3 completed step in 23.61 seconds.
Completed table 5 in 64.30 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 86.45% ).
Compressing tables 6 and 7...
Step 1 completed step in 18.24 seconds.
Step 2 completed step in 24.36 seconds.
Step 3 completed step in 27.49 seconds.
Completed table 6 in 70.11 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 100.00% ).
Serializing P7 entries
Completed serializing P7 entries in 12.30 seconds.
Completed Phase 3 in 318.66 seconds
Completed Plot 1 in 640.62 seconds ( 10.68 minutes )
l:\temp\4\plot-k32-c04-2023-09-12-18-54-5b31db7c2d978d53aa30c60d7dfdc1fc2f0a81ccefd9e473ce26ffe6ffe194aa.plot.tmp -> l:\temp\4\plot-k32-c04-2023-09-12-18-54-5b31db7c2d978d53aa30c60d7dfdc1fc2f0a81ccefd9e473ce26ffe6ffe194aa.plot
Completed writing plot in 5.23 seconds
Final plot table pointers:
Table 1:                0 ( 0x0000000000000000 )
Table 2:<PHONE_NUMBER> ( 0x000000004cc986e4 )
Table 3:<PHONE_NUMBER>9 ( 0x0000000261cf6219 )
Table 4:<PHONE_NUMBER>4 ( 0x00000005a958d694 )
Table 5:<PHONE_NUMBER>9 ( 0x0000000900dc21e3 )
Table 6:<PHONE_NUMBER>9 ( 0x0000000c83c405af )
Table 7:<PHONE_NUMBER>4 ( 0x0000001092d401ae )
C 1    :             4096 ( 0x0000000000001000 )
C 2    :          1719508 ( 0x00000000001a3cd4 )
C 3    :          1719684 ( 0x00000000001a3d84 )
Final plot table sizes:
Table 1: 0.00 MiB
Table 2: 8528.37 MiB
Table 3: 13432.59 MiB
Table 4: 13688.21 MiB
Table 5: 14382.49 MiB
Table 6: 16625.00 MiB
Table 7: 16870.63 MiB
C 1    : 1.64 MiB
C 2    : 0.00 MiB
C 3    : 1226.96 MiB
`
and after some uptime/ploting hous i get double the time for one plot
`
Bladebit Chia Plotter
Version      : 3.1.0-beta1
Git Commit   : 076eba490f1c08b3a7bf10ea0a08f80be758c7b9
Compiled With: msvc 19.29.30151
[Global Plotting Config]
Will create 1 plots.
Thread count          : 16
Warm start enabled    : false
NUMA disabled         : false
CPU affinity disabled : false
Farmer public key     :
Pool contract address :
Compression Level     : 3
Benchmark mode        : disabled
[Bladebit CUDA Plotter]
Host RAM        : 127 GiB
Direct transfers: true
Selected cuda device 0 : NVIDIA GeForce GTX 1080
CUDA Compute Capability   : 6.1
SM count                  : 20
Max blocks per SM         : 32
Max threads per SM        : 2048
Async Engine Count        : 5
L2 cache size             : 2.00 MB
L2 persist cache max size : 0.00 MB
Stack Size                : 1.00 KB
Memory:
Total                    : 8.00 GB
Free                     : 7.02 GB
Allocating buffers (this may take a few seconds)...
Kernel RAM required       :<PHONE_NUMBER>0  bytes ( 87787.46  MiB or 85.73  GiB )
Intermediate RAM required :<PHONE_NUMBER>   bytes ( 4160.07   MiB or 4.06   GiB )
Host RAM required         :<PHONE_NUMBER>8  bytes ( 27008.00  MiB or 26.38  GiB )
Total Host RAM required   :<PHONE_NUMBER>28 bytes ( 114795.46 MiB or 112.10 GiB )
GPU RAM required          :<PHONE_NUMBER>   bytes ( 5855.03   MiB or 5.72   GiB )
Allocating buffers...
Table pairs allocated as pinned: false
Generating plot 1 / 1: dd6ca6a96a81ec7f175ac2a9b094a654296c941a1f84321f714d126fdc33725f
Plot temporary file: l:\temp\2\plot-k32-c03-2023-09-12-19-26-dd6ca6a96a81ec7f175ac2a9b094a654296c941a1f84321f714d126fdc33725f.plot.tmp
Generating F1
Finished F1 in 14.68 seconds.
Table 2 completed in 47.41 seconds with<PHONE_NUMBER> entries.
Table 3 completed in 74.53 seconds with<PHONE_NUMBER> entries.
Table 4 completed in 136.37 seconds with<PHONE_NUMBER> entries.
Table 5 completed in 99.61 seconds with<PHONE_NUMBER> entries.
Table 6 completed in 87.10 seconds with<PHONE_NUMBER> entries.
Table 7 completed in 61.83 seconds with<PHONE_NUMBER> entries.
Finalizing Table 7
Finalized Table 7 in 52.03 seconds.
Completed Phase 1 in 574.51 seconds
Marked Table 6 in 18.27 seconds.
Marked Table 5 in 18.29 seconds.
Marked Table 4 in 18.21 seconds.
Marked Table 3 in 18.35 seconds.
Completed Phase 2 in 73.13 seconds
Compressing Table 2 and 3...
Step 1 completed step in 42.87 seconds.
Step 2 completed step in 40.29 seconds.
Completed table 2 in 83.16 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 80.09% ).
Compressing tables 3 and 4...
Step 1 completed step in 43.68 seconds.
Step 2 completed step in 49.10 seconds.
Step 3 completed step in 42.94 seconds.
Completed table 3 in 135.73 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 80.70% ).
Compressing tables 4 and 5...
Step 1 completed step in 42.63 seconds.
Step 2 completed step in 50.41 seconds.
Step 3 completed step in 47.84 seconds.
Completed table 4 in 140.88 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 82.25% ).
Compressing tables 5 and 6...
Step 1 completed step in 45.47 seconds.
Step 2 completed step in 61.61 seconds.
Step 3 completed step in 56.34 seconds.
Completed table 5 in 193.20 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 86.46% ).
Compressing tables 6 and 7...
Step 1 completed step in 45.06 seconds.
Step 2 completed step in 56.76 seconds.
Step 3 completed step in 51.47 seconds.
Completed table 6 in 153.29 seconds with<PHONE_NUMBER> /<PHONE_NUMBER> entries ( 100.00% ).
Serializing P7 entries
Completed serializing P7 entries in 24.33 seconds.
Completed Phase 3 in 730.60 seconds
Completed Plot 1 in 1378.24 seconds ( 22.97 minutes )
l:\temp\1\plot-k32-c03-2023-09-12-19-26-dd6ca6a96a81ec7f175ac2a9b094a654296c941a1f84321f714d126fdc33725f.plot.tmp -> l:\temp\1\plot-k32-c03-2023-09-12-19-26-dd6ca6a96a81ec7f175ac2a9b094a654296c941a1f84321f714d126fdc33725f.plot
Completed writing plot in 0.51 seconds
Final plot table pointers:
Table 1:                0 ( 0x0000000000000000 )
Table 2:<PHONE_NUMBER> ( 0x000000004cda6524 )
Table 3:<PHONE_NUMBER>0 ( 0x00000002c8bc94d4 )
Table 4:<PHONE_NUMBER>0 ( 0x00000006106c04a6 )
Table 5:<PHONE_NUMBER>5 ( 0x0000000968373603 )
Table 6:<PHONE_NUMBER>5 ( 0x0000000ceba06ab1 )
Table 7:<PHONE_NUMBER>5 ( 0x00000010fb946b3f )
C 1    :             4096 ( 0x0000000000001000 )
C 2    :          1720980 ( 0x00000000001a4294 )
C 3    :          1721156 ( 0x00000000001a4344 )
Final plot table sizes:
Table 1: 0.00 MiB
Table 2: 10174.14 MiB
Table 3: 13434.96 MiB
Table 4: 13692.70 MiB
Table 5: 14390.58 MiB
Table 6: 16639.25 MiB
Table 7: 16885.09 MiB
C 1    : 1.64 MiB
C 2    : 0.00 MiB
C 3    : 1228.01 MiB
`

Most likely your SSD, what SSD are you using?

I solved it by splitting the temp to 2 nvme.... Maybe theyre dieing


add flag INVERT_COLOR_LOGIN_SPLASH_VIEW_SIGN_IN_BTN

Description
OSPR-2025
The text color of the button Already have an account? Sign in in OEXLoginSplashViewController.xib is by default white.
Sometimes the background image used in that view doesn´t work with the white color and manually the text color needs to be updated to use another color, generally black.
With the flag INVERT_COLOR_LOGIN_SPLASH_VIEW_SIGN_IN_BTNwe can invert the color so instead of everytime changing the color manually it can all be done by configuration.
How to test this PR
Add the flag INVERT_COLOR_LOGIN_SPLASH_VIEW_SIGN_IN_BTN and set it to True. The text color of the button should be black.
Reviewers

[ ] Code review: @saeedbashir
[ ] Code review: @salman2013

Thanks for the pull request, @jagonzalr! I've created OSPR-2025 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
If you like, you can add yourself to the AUTHORS      file for this repo, though that isn't required.     Please see the CONTRIBUTING     file for more information.

@edx/ios-devs Take a look.

@jagonzalr We use a configuration file to configure and control the flow of the application. It's not a good idea to add customization(color, font etc) of controls via the config file. We might need to come up with some other exciting idea to achieve it.

@saeedbashir Are you rejecting this pull request?

@nedbat Yes, I'm. I wasn't close this PR because on one of his old PR's, he was concerned about I never gave him a chance to clarify.

Ok, maybe not on the config.yml l should come this but on the colors.json
I remember a while back that this PR was rejected even though it doesn't use the config file but the json file.
Maybe it is a good opportunity to revisit this idea @saeedbashir @nedbat

@jagonzalr It's not a good idea to configure every control via a configuration file because it will result in a messy code base. We may come up with an idea of theming instead of configuring a single control.

@jagonzalr Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.


MSID: 94142 Version: 1 DOI: 10.1101/2<IP_ADDRESS>7239

MSID: 94142
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>7239
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=94142
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>7239

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/94142v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>7239
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/94142/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Thomas Hildebrandt incorrectly giving Michael Brecht's corresponding author email address

Hmm the authors have this in the PDF:

So I'm not certain if there should be one or two corresponding authors. I'm going to change it to one (assume the asterisk is a mistake), and we can update if the authors flag it.
Package uploaded to s3://prod-elife-epp-meca/94142-v1-meca.zip.


add an unit test for sequential yaml validation

What's changed?

add unit test for issue#3270 to further debug
What's your motivation?

Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?

@timtebeek
Have you considered any alternatives or workarounds?

Any additional context

Checklist

[x] I've added unit tests to cover both positive and negative cases
[x] I've read and applied the recipe conventions and best practices
[x] I've used the IntelliJ IDEA auto-formatter on affected files

Thanks for getting this started @RossZhang-cpu ! I've updated your PR with the latest changes to the main branch, as your fork fell behind. Let me know if you find anything whilst debugging, or need help seeing this through!

The underlying issue has since been closed; closing this PR as well then. Thanks again!

https://github.com/openrewrite/rewrite/issues/3270


Servers should check log consistency before granting vote

Votes should only be granted when the candidate's log is at least as up-to-date as the voting server's log. See section 5.4.1 of the tech report.

Fix for this, (to be pull-requested soon):
https://github.com/NetSys/sts2-applications/commit/d908f4a9ae61a14e0bfbc583cdebb016f63e1c76


test: do not run tests when inside containers/CI as it won't work

Tests won't work as libusb won't be able to initialize when run in containers. Check for the presence of the CI environment variable which is the default to signal that it's running in a CI environment, and thus likely in container.

Appveyor: https://www.appveyor.com/docs/environment-variables/
CircleCI: https://circleci.com/docs/2.0/env-vars/#built-in-environment-variables
GitLab CI: https://docs.gitlab.com/ee/ci/variables/predefined_variables.html#variables-reference
Travis CI: https://docs.travis-ci.com/user/environment-variables/#default-environment-variables

Change-type: patch
Signed-off-by: Gergely Imreh<EMAIL_ADDRESS>

@balena-ci retest please

@balena-ci retest

@balena-ci retest please


Error when installing via Homebrew on Yosemite

I get an error when trying to install via Homebrew on Yosemite:
$ brew cask install aerial
Error: wrong number of arguments (1 for 0) while loading '/usr/local/Library/Taps/caskroom/homebrew-cask/Casks/aerial.rb'
Please report this bug:
    https://github.com/caskroom/homebrew-cask/issues
<main>:6:in `name'
<main>:6:in `<class:Aerial>'
<main>:in `<main>'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/source/path_base.rb:65:in `eval'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/source/path_base.rb:65:in `rescue in load'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/source/path_base.rb:62:in `load'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/source/tapped.rb:28:in `load'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask.rb:124:in `load'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/cli/install.rb:19:in `block in install_casks'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/cli/install.rb:17:in `each'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/cli/install.rb:17:in `install_casks'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/cli/install.rb:6:in `run'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/cli.rb:81:in `run_command'
/usr/local/Cellar/brew-cask/0.48.0/rubylib/cask/cli.rb:121:in `process'
/usr/local/bin/brew-cask.rb:42:in `<top (required)>'
/Library/Ruby/Site/2.0.0/rubygems/core_ext/kernel_require.rb:54:in `require'
/Library/Ruby/Site/2.0.0/rubygems/core_ext/kernel_require.rb:54:in `require'
/usr/local/Library/brew.rb:72:in `require?'
/usr/local/Library/brew.rb:152:in `<main>'

Just tried here and it installed fine through cask.
You could try running the command again with brew cask install --verbose aerial and it might give you more information about where it's failing and maybe try running brew doctor and fixing any issues that brings up as well.
But I'd recommend installing the latest beta. John has fixed a number of issues and added a caching feature since the 1.1 release.

No reply after eight months, closing.


feat: implement 0x13 - SGT Opcode

Pull Request type

Please check the type of change your PR introduces:

Feature

What is the current behavior?

Resolves: #8
What is the new behavior?

Implementation of SGT EVM opcode
Does this introduce a breaking change?

No

@enitrat  everything should be ok


It is about heading towards the Waypoints but realising that the game does not want you to continue and will do everything it can to stop you.

How to play your game?

Example  : Press WASD to move, get to the waypoints which are the circles with a blue outer ring and a red core.

Code
Check off the items that are true.

[ *] The game was made using the Sprig editor.
[ *] The game is placed in the in the /games directory.
[ *] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[* ] The game runs without errors.
[ *] The name of the file/game contains only alphanumeric characters, -s, or _s.
[* ] The game name is not the same as the others from gallery

Image (If an image is used)

[* ] The image is in the /games/img directory.
[ *] The name of the image matches the name of your file. Example: sokoban_plus.js -> sokoban_plus.png.

Thanks for PR!

I liked the idea! What's your name?

would you rather I put wyn-cmd or Wynston in the author field?

I would prefer wyn-cmd.

I added another game called The Pet, I wasn't sure how exactly to add a different game via the pull request.


DEPLOY-401

Integrates the ACS image containing identity service support into the DBP and disables the RM and sync service AMPs as they are not compatible with 6.0 yet.

Good catch @svidrascu, README updated.


Analytics on what pages are clicked

we want to know which pages are most popular/visited, to know what to put most effort into
also if we add search, analytics on what search results are incorrect would b nice

lol let's get some google analytics set up on the pitt csc email

finished in in commit 26e291d


initialize options before configuring sources

fixed in feature/type-erasure cherry pick to master or close when merging that branch to master


Add organisations fields

https://trello.com/c/Hf90xZZq/219-drop-down-for-organisations-in-eu-exit-statutory-instrument-specialist-publisher-document-type
Adds the abilities to select Primary publishing organisation and organisations for Statutory Instruments.

@elliotcm I've added a commit to include the primary publishing org with the organisations in the links hash.

@elliotcm good spot - updated.


Switch to new crossplane build module, consume KCL

Description of your changes

Switch to crossplane/build module
Consume KCL cli
Adjust Makefile
Prettify with local help

I have:

[x] Read and followed Upbound's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested
make e2e locally plus uptest below

/test-examples


Fix hashicorp-vault extension

I'll set this to draft until I figure out the missing shaded classes. Strange that I don't see the failures happen locally...

What's the status on this?

What's the status on this?

I can try to revisit it for 3.15.0.
I was hoping to not have to include most of spring-web because it's a magnet for CVEs. But, I'll probably just include it as-is to get things working again.

Reworked things to use the unaltered spring-web.


Evaluation and introduction of additional tools

Brief feature description
For making iceoryx more stable, we need to look for better tooling to do code analysis and support the developer.
We should also take a deeper look into the ROS side what there is used, e.g. ament_lint.
Proposals are welcome!
List of common tools from ROS2:

ament_cmake_copyright <https://github.com/ament/ament_lint/blob/master/ament_cmake_copyright/doc/index.rst>_ : a copyright linter which checks that copyright statements and license headers are present and correct

ament_cmake_cppcheck <https://github.com/ament/ament_lint/blob/master/ament_cmake_cppcheck/doc/index.rst>_ : a C++ checker which can also find some logic tests

ament_cmake_cpplint <https://github.com/ament/ament_lint/blob/master/ament_cmake_cpplint/doc/index.rst>_ : a C++ style checker (e.g. comment style)

ament_cmake_lint_cmake <https://github.com/ament/ament_lint/blob/master/ament_cmake_lint_cmake/doc/index.rst>_ : a cmake linter

ament_cmake_uncrustify <https://github.com/ament/ament_lint/blob/master/ament_cmake_uncrustify/doc/index.rst>_ : a C++ style checker

I think we already have ament_cmake_copyright since #346.
Since we already use clang-format I'm not sure if we need uncrustify. A pre-commit hook for clang-format would be nice, though

I think we already have ament_cmake_copyright since #346.
Since we already use clang-format I'm not sure if we need uncrustify. A pre-commit hook for clang-format would be nice, though

We currently have Address, Leak & Undefined Behaviour Sanitizers.
Clang has some more tools in their basket.

https://clang.llvm.org/docs/ThreadSanitizer.html
https://clang.llvm.org/docs/MemorySanitizer.html (all dependent libs including libc++ should be recompiled with special flags)

We currently have Address, Leak & Undefined Behaviour Sanitizers.
Clang has some more tools in their basket.

https://clang.llvm.org/docs/ThreadSanitizer.html
https://clang.llvm.org/docs/MemorySanitizer.html (all dependent libs including libc++ should be recompiled with special flags)


allow user to define temperature

[x] Modify the formula to accomodate a custom temperature
[ ] Add a temperature box to the GUI

I exposed Ion.temperatureC so we just need to add a GUI box to let the user set it.
https://github.com/swharden/JLJP/blob/906bfe5bf5e0f7249f828fec9b4565e143bf5124/src/Ion.java#L8-L13

Handling the temperature is not staightforward. The main effect of temperature changes on the liquid junction potential is to alter the diffusion coefficients (or equivalently the mobilities). In turn, the variation of diffusion coefficient is mostly determined by i) the variation of viscosity of the solvent and ii) the variation of the interaction between the ion and its solvation shell. The term kT in the formulas represent only a minor effect of temperature. Thus it is necessary to warn the user that the mobilities automatically filled by the program refer to room temperature, although different values of kT can be ideally input.

mobilities automatically filled by the program refer to room temperature

I did not consider this! It seems that setting temperature is a lot more complicated than I initially thought, so I'm happy leaving things as they are.
Thanks for looking into it!

Being able to calculate at various temperatures would be very useful. If the user knows the mobilities at a given temperature, he/she can input the values, but then he/she must also change the temperature T used for calculating k_B T. I suggest the following. First add a field where the user can add the temperature. Then, when the calculation is performed, the program checks if the temperature is set accordingly to the temperature at which the ion mobilities are given and, possibly, gives a warning. Currently, the mobilities are at 25°C, so, if the user sets - say - 50°C, the program warns: "the ion mobilities are defined at a temperature different from 50°C". If we implement the ion mobility table as a database, including the temperature at which the mobility is taken, the program could check the coherence between the set T and the temperature at which the ion mobility refers to.

I experimented with the ion set in the JLJP screenshot and ran LJP calculations at temperatures 0C through 50C using mobilities for 3 different temperatures (from Langes Handbook of Chemistry).

Interestingly, the KT effect from a swing of 25C is about 4 times larger than the effect of using mobilities reported 25C apart. I take this to mean that both are important, but adding user input for KT temperature has a bigger effect on final LJP than having them modify the mobility table.
Do you think discussion on this topic would be useful for the manuscript?
code used to produce this graph


GOAL: iot.atsign.dev

This document has been used to encompass most of this work.

[ ] Homepage

[x] Define content
[x] Refine content @thenobleone14 @XavierChanth
[ ] Designs @rtilney
[ ] Implement designs @XavierChanth

[ ] Industries section - @anthonyvprakash @Jesseadriano10

[x] Layout for content @Jesseadriano10
[ ] Define content

[ ] Tools section - @L1AMcB

[ ] Layout for content - in progress by @L1AMcB

[ ] Docs section - TBD if any work is needed
[ ] Showcase page - @TylerTrott

[ ] Define demos to be included
[ ] Add content for each

[ ] Partners section - @jenmonroe to define the content on these pages

[ ] Design area to display existing partner "badges"
[ ] Partner contact form or guide to connect with us

@XavierChanth -- @tinashe404 and I discussed this today. I have an updated contract ready, and @rtilney is working on the atVerified badge/bug. I recommend pulling the Commission program out of the larger Dev program (which originally included programming, content, and events) and just launching the atVerified and Commission portions. Should we look for some time to meet, or want to work on this asynch? Thanks!

legacy due to #94


I hope you can add support for empty frames.

Windows 10 x64
tensorflow 1.31
python 3.71
Traceback (most recent call last):
File "src/openpose_3dpose_sandbox_vmd.py", line 484, in
tf.app.run()
File "F:\Program Files (x86)\Miniconda3\lib\site-packages\tensorflow\python\platform\app.py", line 125, in run
_sys.exit(main(argv))
File "src/openpose_3dpose_sandbox_vmd.py", line 211, in main
center_2d_x, center_2d_y  = camera_center(openpose_output_dir) #動画の中心座標（動画の解像度の半分）
File "src/openpose_3dpose_sandbox_vmd.py", line 403, in camera_center
x = float(sf.readline().replace('\n', '')) / 2
ValueError: could not convert string to float:

I am sorry to have added trouble to you, but I think this is reasonable.
The person in the video I used is incomplete, so it will have empty frames, I hope you can add support for empty frames.

Hi i would suggest to run it in ubuntu enviroment It works fine in Ubuntu

嗨我建议在ubuntu环境中运行它在Ubuntu中工作正常

Hi，I know the problem is there. The above is that I didn't seriously read the self-reported results. I have solved the problem. But I still want to be able to skip this error.

最初のフレームに人数分のデータがありません。
There is no data for the number of people in the first frame.

ok  i hope it solves your issue . There are some modifications to be done , if you want to run the original imnplementation of the 3d-pose-baseline. I havent managed to run this version the code or to generate an animation. I am not a game developer , so i dont know .  Did you manage to generate an animation in unity based on the original 3d -pose estimation

Oh no, I don't have any knowledge.


MM: Revert to human on void out to prevent softlocks

Happens by default, except when not having Deku

Fixed


[BXMSPROD-1881] dependabot for release and blue branches

Please include 8.30.x also.
And the next productized branch should be included in there. Gonna think about something :)

we talked about it on our mtg that the new dependabot scanning should be done only on long lived branches, 8.30.x will not be used anymore and we move to new one for future releases so everything what is scanned on main is the vehicle there


Links to paper

Once the paper is out at IOP, we should add links to it throughout the web app. I am thinking about a question mark, maybe on top of the cards next to the Chevron menu, that will take you to the section of the paper describing that particular method.
Any thoughts on this @tdbjacobs @mcrot @jotelha @lthimons ?

I love your idea of linking WITHIN the paper (not just having each section
link to the top-level document)
On Fri, Jul 29, 2022 at 7:34 AM Lars Pastewka @.***>
wrote:

Once the paper is out at IOP, we should add links to it throughout the web
app. I am thinking about a question mark, maybe on top of the cards next to
the Chevron menu, that will take you to the section of the paper describing
that particular method.
Any thoughts on this @tdbjacobs https://github.com/tdbjacobs @mcrot
https://github.com/mcrot @jotelha https://github.com/jotelha @lthimons
https://github.com/lthimons ?
—
Reply to this email directly, view it on GitHub
https://github.com/ContactEngineering/TopoBank/issues/871, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFOUYWUUZYFEWBUINQHBVTLVWO6SRANCNFSM55AOTYDQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Tevis Jacobs
Mechanical Engineering and Materials Science
University of Pittsburgh
[Sent from my phone. Please forgive typos.]

We could additionally link to original publications, see issue #171


Update index.md

Updating the website with MMM-GPT information and link

:books: Documentation preview :books:: https://pymc-marketing--433.org.readthedocs.build/en/433/

@twiecki

Let's reuse the text from https://github.com/pymc-labs/pymc-marketing/pull/434.

@nialloulton @twiecki is this one still relevant?

@nialloulton I think we can close this one as it seems a duplicate by https://github.com/pymc-labs/pymc-marketing/pull/434. we can always re-open it :) Is just we are doing some backlog cleaning 🙏


[system] Fallback to value if theme's value is an array and index missing

Fixes #16260

@stasiukanya Thank you, do you think that you could look at adding a test?

@oliviertassinari Yes, I've added test for shadows and moved it to separate file

Thanks


Vite doesn't seem to honor .gitignore and tries to parse unrelated html files

Is your feature request related to a problem? Please describe.
I have some .html files lying around in my project, which should be ignored by Vite. But when I try to run vite, I get the following error:
$ vite
error when starting dev server:
Error: The following dependencies are imported but could not be resolved:

  testlib123.es.js (imported by /home/fabd/Code/github/some-project/src/assets/__fab/__test.html)

Typically I like to exclude files with double underscore in my gitignore, but in this case I also have a .gitignore IN this folder where __test.html is :
# private files
*
.gitignore

So for all intents and purposes it seems vite does not ignore unrelated html files, even when they are not called index.html.
I also tried to place my test file outside of the src/ folder. For example from the project's root folder ./__foo/__test.html, same error.
Describe the solution you'd like
Vite should probably honor the .gitignore, so that I can keep unrelated html files in my project's folder.
Describe alternatives you've considered
...
Additional context
...
Thanks!

.gitignore has nothing to do with Vite so this behavior doesn't look surprising to me.
Even if you put something in .gitignore, files are still here but are not taken into consideration by Git.
There probably is a Vite option that does what you want tho (I mostly use Vite with the default configuration so I can't tell you what it is).

As @Deckluhm explained, .gitignore only affects git. Maybe you are loading that page in your browser, if not, the Vite dev server shouldn't access it. Closing this issue, but we can discuss further in Vite Land.

Same issue here, please reopen this issue.
I have some html in a directory that prefixed with # for example: project/#foo/bar.html, for some reason, vite tried to load this file, but it will throw an error for some reason..

I think it tries to parse any html that references a javascript file?
In my case, the html file pointed to a js that no longer exists, but either way I don’t know why Vite reads it.
Assuming the multiple entries scenario is something you have to opt in, then Vite should only parse a index.html close to the root of the project right? Then it can ignore anything else.
I have ‘t had time to test yet (on my ipad now) I want to disable plugins just in case they cause this? Plugins are : eslint, tailwind jit, rollup strip, vite plugin md.
The config docs says

By default, Vite will crawl your index.html to detect dependencies that need to be pre-bundled. If build.rollupOptions.input is specified, Vite will crawl those entry points instead.

So it seems other entry points is opt in.

It seems the code does look for any html files, not just index.html
https://github.com/vitejs/vite/blob/bc35fe994d48b2bd7076474f4a1a7b8ae5e8f401/packages/vite/src/node/optimizer/scan.ts#L60
I could use optimizeDeps.entries I suppose.
In order to make it easy for beginners, would it make sense to ignore any html files besides the first one found, in the default scenario when user does not explicitly set any entries options?

@fabd @yaquawa this is already being tracked here https://github.com/vitejs/vite/issues/2599


PublishToRedis should have string configurationName

For now configurationName is NameString which validates values as valid graphql type
I want to use MyStitchingRedisPrefix.MyStitchingService with . separator for redis key and it is impossible for now

For now, you can use an underscore instead of a .. I will evaluate a change when we work on 11.1.

I am using . for all my services for better redis gui experience
For now i am using stitching configurationName
I will wait for 11.1, thanks)

Is this a duplicate of something? Should i close issue?

No, leave it open.

I thought you did report this already, but it might be you told me this on slack.


Update Tachiyomi-Lavender.json

Fixed these two unthemed things

Thanks! I apologize for the delay merging this, appreciate your help!

My pleasure 😁


Add ADR035: User info fetcher CRD changes

Closes https://github.com/stackabletech/opa-operator/issues/478

Thanks a lot @NickLarsenNZ!


demo: clean up when finished

Hello @jazzdan,
Please review the following commits I made in branch nicks/ch585/clean:
6928b2ca93e05f4d1ffd6126f9a1b499a7cc744e (2018-10-19 16:51:34 -0400)
demo: clean up when finished

nah, those get cleaned up automatically when the process closes


Publishing v0.12.0-pre.7

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.01%. Comparing base (3ce5335) to head (f417e99).
Report is 12 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #447      +/-   ##
==========================================
- Coverage   88.61%   88.01%   -0.61%
==========================================
  Files          39       38       -1
  Lines        9109     8282     -827
==========================================
- Hits         8072     7289     -783
+ Misses       1037      993      -44

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


refactor: certificates generation package and secrets

Description
This PR refactors the generation utilities in the admissionregistration package,
removing duplication and simplifying the current implementation.
It renames admissionregistration in certs and reactors the existing reconciler unit test in integration tests clauses.
Additional info

This PR also changes the golangci configuration to add an exception for the ok map index in the varnamelen linter. However we should uniform and clean up the golangci configurations across all the KW repos, this is tracked by: #778

There are a few incongruences with the naming, for instance, this status  https://github.com/kubewarden/kubewarden-controller/blob/030edce6f7c57d54b88453054e39f9baa20377cc/pkg/apis/policies/v1/policyserver_types.go#L132 should be probably named CertSecretReconciled instead. Fixing this is out of scope for this PR and I will create an issue about it.

Fixes: #775, #664

@flavio I've addressed the comments in https://github.com/kubewarden/kubewarden-controller/pull/777/commits/c3cbdf4e9759e6e0fa2900d2343bcde0865cfe43.
I tracked all the naming-related comments in: https://github.com/kubewarden/kubewarden-controller/issues/782

Blocked: waiting for 1.14 release


Ddoc: add class "simple-cheatsheet"

So that DDOX can choose not to display them.
.simple-cheatsheet tables play the same role as the listings that DDOX
generates. Having them both, and right after another, is confusing for
the reader, and doesn't add any value.
There are similar tables in

std.experimental.ndslice.{iteration,selection},
std.format,
std.net.curl, and
std.range.primitives,
but they contain additional information or are embedded in text. So their
presence should be less confusing.

Companion pull request for dlang.org: https://github.com/dlang/dlang.org/pull/1323

How about a CHEATSHEET macro?

How about a CHEATSHEET macro?

Sure. Added it in the dlang.org PR (https://github.com/dlang/dlang.org/pull/1323), and here. I've called it SIMPLE_CHEATSHEET, though, as more complex ones may add value over the DDOX listings.

Hmm, on second thought I'm not sure about this.
I only realized this when looking at the final diff: the cheatsheets include minimal examples that provide a much better overview of the functions than the first paragraph of their DDoc. Keeping in mind that some day we might throw away the Phobos DDoc documentation and use DDox exclusively, it would mean throwing away the cheat sheets completely.
Perhaps we should instead go in the other direction (hide the automatically-generated DDoc overview in the modules with cheat sheets)? That has its disadvantages too, e.g. forgetting to update the cheat sheet would mean that a symbol is not reachable via HTML links.

Here's an experience report from a new user who didn't like the cheat cheet for std.algorithm.comparison:
<EMAIL_ADDRESS>

FWIW I disagree very strongly with that assessment. Perhaps longer English descriptions are more useful for people new to programming, but as you acquire experience across multiple programming languages, reading the documentation becomes more of a task of "finding the name of the function which I already know what it does". This is also the use case for people who already are familiar with Phobos' repertoire, but just don't remember the exact names for all functions. Having small examples is a much simpler way to confirm that the function does what you think it does, and removes the ambiguities of English phrasing or the need to parse English at all.

Having small examples is a much simpler way to confirm that the function does what you think it does, and removes the ambiguities of English phrasing or the need to parse English at all.

Makes sense, but the DDOX listing may need more meat to be useful.
I've been toying with the idea of changing the actual summaries of the functions to something more similar to the cheat sheet snippet. I.e., make the summary one sentence + one small example. The more complete description would follow, and then the rest as usual.
That would eliminate the schism between cheat sheet and generated listing, but is it
a) acceptable in the actual function documentation (not just the cheat sheet), and
b) enough information for the generated listing?

It's one solution. IIRC Sönke suggested more options in another thread.

IIRC Sönke suggested more options in another thread.

Yeah, over here<EMAIL_ADDRESS>He suggests a special section or macro that DDOX would recognize and extract for the listing. He also mentions the problem: Another instance of duplication, because Ddoc wouldn't be able to build the cheat sheet that way.

@aG0aep6G are you still interested in pushing SIMPLE_CHEATSHEET?

Uhh, no. I had pretty much forgotten about this. If anyone decides that this PR here is the way to go, I'm up for fixing any issues it might have. But I don't care either way. If you want to go the other direction, by all means, go ahead.

Uhh, no. I had pretty much forgotten about this. If anyone decides that this PR here is the way to go, I'm up for fixing any issues it might have.

Hmm when I look at http://dlang.org/library/std/range.html, it still looks ugly due to the duplicated information. If there's an easy way to disable the listing with Ddox when a cheatsheet macro is present, I would say we go for it (we have to maintain the cheatsheets anyways).
If it turns out to be difficult to teach Ddox about the existing cheatsheets, I am also ok with the approach to disable the macros instead which is taken here.

But I don't care either way. If you want to go the other direction, by all means, go ahead.

Same here. I am going to make the decision easier for you as I am closing this PR - it's full of merge conflicts and submitting a new PR wouldn't be an overhead and would highly increase the chances of it being merged.
With the hopefully upcoming DDox transition this topic needs to be revisited again anyways, so maybe the future is more gentle to us.

I am going to make the decision easier for you as I am closing this PR

You should probably revert https://github.com/dlang/dlang.org/pull/1323 as it doesn't make sense without the PR here.

You might actually just want to change the cheatsheet to embrace the idea of duplicated information, like caption it "Most common functions organized by category" (well, maybe not literally say that but word it somehow) then at the end be like "Complete listing follows".
So yes, it duplicates in some sense, but it also reorganizes and prioritizes the information in the table. In ddoc, the complete listing is the all on one page stuff, and in ddox it is a more detailed navigation, but still shows all functions alphabetized rather than categorized/prioritized by hand.
I mean, as it is, the cheatsheets don't show everything.... and indeed, I don't think they should, it is a "cheatsheet", not a "textbook"!


Docker container version

Containerizes spliit2 using docker. You can now create an image and run the app in a cluster (db + app). Instructions on how to run it in a container were added to the README.

Brilliant!

Could this also be added to docker hub? This will aid local (and potentially homeassistant) deployments

@ChristopherJohnston totally, I created an issue https://github.com/spliit-app/spliit/issues/60


Smooth Scrolling to next position

Is it possible to fix velocity so on scrolling, only items changes and position remain constant. Like moveToNextPos() & moveToPreviousPos();

I would appreciate a method for one by one smooth scrolling as well, any chance to see this in the future?

Hi,
Use the below code for smooth scrolling on item select :
ValueAnimator valueAnimator = ValueAnimator.ofFloat(-1f * getAngleForPosition(mRawSelectedPosition), (-1f * getAngleForPosition(rawPosition)));
valueAnimator.addUpdateListener(new ValueAnimator.AnimatorUpdateListener() {
@Override
public void onAnimationUpdate(ValueAnimator valueAnimator) {
setAngle((Float) valueAnimator.getAnimatedValue());
}
});
valueAnimator.setDuration(2000);//animation duration
valueAnimator.start();
It work for me. :)


server.database.path not being used

I have seen where this issue is supposed to be resolved, yet is it still occurring for me.
I'm running an embedded server in Windows 7 using Eclipse Luna 1a(4.4.1) / Java 1.7
Please correct me if there is something I'm missing.
The code is fairly simple:
OServer server = OServerMain.create();
server.startup(configFileStream);
server.activate();
ODatabaseDocumentTx db = new ODatabaseDocumentTx("plocal" + dbName);
db.create();
dbName is a string - the configFileStream is an InputStream.  The database is created in the directory where the process has been started.
My log shows:
Mar 03, 2015 11:00:25 AM com.orientechnologies.common.log.OLogManager log
INFO: OrientDB auto-config DISKCACHE=1,079MB (heap=894MB os=4,021MB disk=37,062MB)
Mar 03, 2015 11:00:25 AM com.orientechnologies.common.log.OLogManager log
INFO: Loading configuration from input stream
server.database.path : C:/workspace/SMS_SERVER/com.tdkc.statemodel.OrientDB/databases/
2015-03-03 11:00:26:140 INFO  OrientDB Server v2.1-SNAPSHOT is starting up... [OServer]
2015-03-03 11:00:26:162 INFO  Databases directory: C:\workspace\SMS_SERVER\com.tdkc.statemodel.OrientDB\databases [OServer]
so the log shows the correct location based on the server.database.path set in the orientdb-server-config.xml file that is being passed in to the server.startup() method.
ORIENTDB_HOME is not set, but even when I set it as a system property it does not change anything.

hi @puggimer
this:
ODatabaseDocumentTx db = new ODatabaseDocumentTx("plocal" + dbName);
db.create();
create the database in local not on the running server.
you should use OServerAdmin for create the database to the server have a look to :
http://www.orientechnologies.com/docs/last/orientdb.wiki/Server-Management-Java.html
closing this issue is invalid

Thanks.  Very counter-intuitive to have to use a remote connection to an embedded server.
When would you use the ODatabaseDocumentTx.create() method - if you need to create it via the OServerAdmin then why does this method exist?
I am finding the documentation very lacking, even after reading Tesoriero's book ( https://www.packtpub.com/big-data-and-business-intelligence/getting-started-orientdb ) (which I thought was going to help).

After the db has been created, you can access to it using "plocal:" as well. It's just the first creation of DB the "problematic" part.

The only way I have been able to access it via "plocal" is to give it the entire path to the database - which kind of defeats the purpose of setting server.database.path in the first place.

You can use env variables in path. Example:
"plocal:${ORIENTDB_HOME}/databases/test"


Improve README, Update features to v1.0.0

Next time these updates won't be smashed together ;)
v1.0.0 includes...
UPDATES:
Updated README for v1.0.0 features and better document each feature in
general.
Updates to app, bean, controller, service, layout, subsystem and view
that include fixes to typos, more informative metadata hints which are
used in the help feature with CommandBox, restructuring of code and
also going through and "modernizing" some of the code base. Since
CommandBox runs on Lucee, we wanna code like it's 2015 right? :)
fw1 create subsystem now assumes you are using FW/1 3.5 or higher so
it will look for /subsystems directory to follow Subsystems 2.0
conventions. Specify the directory in the appropriate parameter to
override this.
ADDITIONS:
In light of FW/1 3.5 being released and finalizing the release of the
Subsystems 2.0 convention, fw1 create app now has 2 separate subsystem
skeletons: "Subsystem" and "Subsystem-Legacy".
Added the ability to generate Clojure controllers and services to be
used with FW/1 3.5+.
Added MIT License.

:+1:


LSP messages

Good to see some activity here, recently migrated to lualine but would be very happy to come back given el is maintained again.
Do you think your could add some native support for displaying messages from LSP? Something like http://github.com/arkav/lualine-lsp-progress. I think el previously required lsp-status.nvim installed?

I've pared this down to a more minimal (not configurable) component that may be more appropriate for express line: https://gist.github.com/clason/1fe0f77ed3cc810ff75429edf1c56409


SemanticChecks: Avoid repeated type resolution of [ordered]

PR Summary
Avoid unnecessary attempts to resolve [ordered]'s underlying type (which isn't there) in post-parse checks

PR Context
As highlighted in https://github.com/PowerShell/PowerShell/issues/17308, parsing the cast expression [ordered]@{} is exactly as slow as parsing [NonExistingType]@{}!
Most of this overhead is incurred during post-parse analysis conducted by the SemanticChecks visitor, when repeated attempts are made to resolve the underlying runtime type represented by [ordered]. Since [ordered] is a special attribute and not really a type literal, the type resolution fails, which in itself is a relatively costly (and non-cacheable) operation.
This PR makes no functional changes to the flow of analysis, but reduces calls to TypeName.GetReflectionType() in situations where we've already assess the target type name is [ordered].
This alone cuts the wallclock time for parsing [ordered]@{} compared to [nonexisting]@{} by 45-50%:

PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[X] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

If I understand correctly the intention was to do optimization for all non-resolvable type names:

To the 2nd question, the same performance degradation would happen to another non-resolvable type name in the script, so it would be nice to resolve this perf issue in the root -- not looking up a non-resolvable type name multiple times during parse would provide a significant boost to performance.

I must have misunderstood @daxian-dbw

@IISResetMe No, you are not 😄 Resolving the root issue will of course be the ultimate goal, but that will obviously require more time and yet still turn out to result in behavior changes. Also, it's possible that no one will pick up that work (or the work to make ordered a real type accelerator because that involves changing parser and compiler) for a long time.
The short term fix you discovered will solve the immediate problem right away with very minimal changes. I think it's reasonable to take it now.

There's a couple other mitigation ideas that came up while discussing the issue earlier today:

Please use a separate PR for further optimization, especially if the target is to solve the perf issue regarding non-resolvable type in general.


consider using factory functions in unit test mock data

I see we have some single reference exports in test data shared across tests. This could be more flexible by making these into factories, e.g.:
function generateObj(options) {
  return {
  };
}

so that they can be clean per-test via explicit call or beforeEach:
it('something', function() {
  const obj = generateObj();
  // ...
}

beforeEach(function() {
  // not sure if mocha shares an object like jasmine, but on `this` or in closure
  this.obj = generateObj();
});

I started doing this in the test data. See the set of createTestFOO functions in test/data/linodes.js such as createTestLinode.


CommandBus setup

Hi,
I'm trying to use tactician-bundle in a Symfony 3 project.
I've installed through composer.
Then I've added bundle in the kernel class.
Maybe I left some config step?
When I try to get command bus in the controller with $this->get('tactician.commandbus') I'm getting an error, It doesn't exist.
I've tried to add manually in my services.yml, but I need to setup each middleware.
Are this the correct path?
Thanks a lot,
Albert

Hi Albert,
Sorry to hear you're having problems! Can you let me know which version of
Tactician you're using and the exact error message you're seeing? That
would help a lot.
There is usually some config to place inside config.yml, you can see some
examples here:
https://github.com/thephpleague/tactician-bundle/blob/master/README.md#configuring-middleware
Thanks in advance and sorry for the issues :)
On Feb 8, 2017 10:50 PM, "Albert Moreno"<EMAIL_ADDRESS>wrote:
Hi,
I'm trying to use tactician-bundle in a Symfony 3 project.
I've installed through composer.
Then I've added bundle in the kernel class.
Maybe I left some config step?
When I try to get command bus in the controller with
$this->get('tactician.commandbus') I'm getting an error, It doesn't exist.
I've tried to add manually in my services.yml, but I need to setup each
middleware.
Are this the correct path?
Thanks a lot,
Albert
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/thephpleague/tactician-bundle/issues/37, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAI9TgYkCAWURB5KyUEghmIfhhpSFSV2ks5rajiPgaJpZM4L7aLx
.

Hi,
I'm in v1.0.2.
Now, trying to get last error to post it here, I got an other error, non related with tactician.
Now tactician.commandbus it seems to exist.
Sorry for disturb and thanks


Treat the last statement in a block the same as if it were an expression

This PR is an attempt to minimize the difference between a trailing expression vs trailing statement in a block.  When the last line in a block is followed by a ; it's a statement and when it isn't then it's an expression.  In both cases, the inferred type of the block is the type of the last statement/expression.  The reason for not requiring undefined; when there's nothing to "return" is that this would require adding this to all blocks where that's the case including functions, try-catch, etc.
Since an if (without an else) could appear at the end of a block, we need to allow that an infer a sensible value for this it.  This PR does this, inferring its type as T | undefined if the consequent's value is T.
Variable declarations are now allowed at the end of a block.  While introducing a variable there might not make much sense since it will go unused, the initializer could be doing useful work.  I'd rather this be a warning instead of an error.
NOTE: This behaviour is quite different from TC39's do-expression proposal which prevents the last statement in a do-expression block from being an if or a let.

Codecov Report

Merging #356 (72192f4) into main (f6f4b1f) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #356      +/-   ##
==========================================
- Coverage   88.87%   88.82%   -0.05%
==========================================
  Files          40       40
  Lines        5770     5764       -6
==========================================
- Hits         5128     5120       -8
- Misses        642      644       +2

Impacted Files
Coverage Δ

crates/crochet_infer/src/infer_expr.rs
95.09% <100.00%> (-0.08%)
:arrow_down:

crates/crochet_infer/src/lib.rs
97.47% <100.00%> (+<0.01%)
:arrow_up:

crates/crochet_parser/src/lib.rs
94.52% <100.00%> (+0.08%)
:arrow_up:

crates/crochet_infer/src/type_error.rs
63.63% <0.00%> (-2.28%)
:arrow_down:

crates/crochet_codegen/src/js.rs
93.95% <0.00%> (-0.42%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Introduction of a union type

The stdlib is currently missing one of the essential types i.e. Either<a, b>.

@KristianBalaj we've actually had a long convo about this before and we probably won't have an Either type in stdlib. There's been conversations about just supporting Result<a, b> but then that's another conversation about how to treat errors as values which just isn't the case at the moment. In any case, Either is more a candidate the prelude instead of stdib. One thing to keep in mind is that just cause something is commonly considered "essential" in some functional programming languages that may not be the case for aiken. I don't actually have a haskell background for example so I never even considered having Either in the prelude.


What is a "pipeline"?

It would help people new to cloud computing and, in particular, ISB-CGC, to get a high-level description of what a pipeline is.  Some background and a high-level overview would greatly enhance the onboarding of users.

Hi there, just wanted to let you know that I recently added a "Terminology Clarification" section to help with understanding what is meant by "pipeline" in this context.  I'm still in the process of working on more thorough documentation, so I will leave this open until I have a chance to provide a more comprehensive update.  Thanks for the feedback!