Bing Map issue

Hi
the Below Bing map is not clear, unable to see the Road.
https://www.openstreetmap.org/edit#map=16/11.4852/79.7535

We can't do anything about the Bing imagery - they source their images from various places and this spot happens to be on a boundary between higher and lower quality.
But you can switch which imagery provider iD uses. In this area, Mapbox Satellite looks better. The available list is in the "Background" pane and can be switched like this:


habitat resource improvements

provider improvements
This commit introduces several improvements to the hab_package
provider, and one fix for the hab_install provider.
First, set the default hab_path to "/bin/hab" in the hab_install
provider. When the Habitat "install.sh" script runs, it uses "hab pkg
binlink" to put the hab binary in the $PATH, which is by default
"/bin/hab", not "/usr/local/bin/hab". This is idempotent but not
convergent, we don't need to rerun the install every time.
Next, improve the hab_package provider. This has several components to
it.

Ensure we use_inline_resources
Validate the package name doesn't contain the version/release
Clarify the message about the remove action
Fix package installation convergence
Change the test cookbook to use different package names

Ensure we use_inline_resources
We're using the updated custom resource DSL in this, so we need to
use_inline_resources so that we get the correct behavior.
Validate the package name doesn't contain the version/release
It is not idiomatic chef recipe code to specify package resources
with the version as part of the name, despite that being passed as a
string to the underlying command-line tool. While it is possible for
Habitat users to do this:
hab pkg install core/redis/3.2.3

In Chef this conflates to overloading the package name and it makes
detecting the idempotence of the resource more difficult, if not
impossible. Thus, the ability to pass the version/release to the
package name is not supported, and we validate this by raising an
exception if it's not the case.
Use the following forms for specifying Habitat package resources:
hab_package "core/redis"

hab_package "core/redis" do
  version "3.2.3"
end

hab_package "core/redis" do
  version "3.2.3/20160920131015"
end

Note that when specifying only the version, and not the release, the
provider looks at the remote depot for a candidate version, and if the
installed release does not match, the package will be installed from
the remote depot. To ensure that a specific release is used, specify
it as part of the version per the last example above.
Clarify the message about the remove action
The best way to handle removing a package is to add that functionality
in the hab CLI, rather than do something potentially dangerous in
this provider.
Fix package installation convergence
Previously, the install action would reinstall packages every Chef run
because we weren't checking for the existence of the current resource.
In this commit, we have implemented version_requirement_satisfied?
to determine whether there is a current version installed and if it
matches the version specified in the new resource. Because the Habitat
packages have a "version" and a "release" that both combine to be a
valid version identifier, it can be tricky to determine reliably which
version is installed, so we do our best here by splitting the
identifier on /.
Change the test cookbook to use different package names
Because we no longer accept versions or version/releases as part of
the package name, we need to use unique package names in order to
avoid resource cloning issues.

@lamont-granquist this should be updated based on our conversation. Thanks!


docs: add Buffers entry to Nerd Fonts v3  source selector example

I recently reconfigured my terminal and editor, so I'm using v3 nerd fonts. Using the winbar source selector, I noticed icons were broken, so I applied the config suggested in the readme. I realized that the buffers entry was missing, so I determined that the existing code point was 0xf719, nf-mdi-file_document_box:
https://github.com/nvim-neo-tree/neo-tree.nvim/blob/7f6fa04dbd8e8c79d1af33bc90e856b65d8641da/lua/neo-tree/sources/buffers/init.lua#L14
The equivalent glyph in v3 has a different name, nf-md-text_box, and is at 0xf021a.
This PR adds a line to the readme config example to fix the buffer icon for V3 nerd fonts.

@cseickel The reason I filed this against branch v2.x is because that's what is currently displayed on the main project page (https://github.com/nvim-neo-tree/neo-tree.nvim), whereas it's kind of useless on main since this already has support for V3 nerd fonts, so the workaround in the README can be removed.


Add reload test and extend update test

This PR adds test for crypto_policies_reload variable, we check that sshd service is restarted when this variable is set to 'true'. Moreover, test for setting policy module is added to "tests_update".

The CI failure is not your fault. I am not sure why the yamllint fails on a symlinks like this.

[citest commit:2c6f09180fdc712dfdf067b168a781d59df059d7]

The CI failure is not your fault. I am not sure why the yamllint fails on a symlinks like this.

Looks like a bug in crypto_policies: vars/CentOS_8.yml -> RedHat_8.yml but RedHat_8.yml does not exist.  Did you forget to add this file?

The CI failure is not your fault. I am not sure why the yamllint fails on a symlinks like this.

Looks like a bug in crypto_policies: vars/CentOS_8.yml -> RedHat_8.yml but RedHat_8.yml does not exist. Did you forget to add this file?

Thanks for second look. I completely missed. that. Now, it is removed in #4.

[citest commit:7f4e1d3447830ead03d2fd41f9eec378e00983ac]

[citest commit:a336f59576a1f2e6d115d77bb111b690e3b57bf8]

[citest commit:a336f59576a1f2e6d115d77bb111b690e3b57bf8]

[citest commit:cc7d9e76e09382a56199313d173ba54e062de1d9]

No idea why the centos-8 images fail while rhle-8 work as expected.

No idea why the centos-8 images fail while rhle-8 work as expected.

Are you talking about the centos-8/ansible-2.8 tests?  Yeah, not sure - https://fedorapeople.org/groups/linuxsystemroles/logs/linux-system-roles-crypto_policies-pull-linux-system-roles_crypto_policies-3-cc7d9e7-centos-8-20201215-101810/artifacts/ansible.log
TASK [Check the sshd was not reloaded] *****************************************
task path: /tmp/tmpv2jyf7v7/tests/tests_reload.yml:25
fatal: [/cache/centos-8.qcow2]: FAILED! => {
    "assertion": "sshd_pid_before.content == sshd_pid_after_first.content",
    "changed": false,
    "evaluated_to": false,
    "msg": "Assertion failed"
}

would be useful to add a task to dump sshd_pid_before.content and sshd_pid_after_first.content to see how they are different.

would be useful to add a task to dump sshd_pid_before.content and sshd_pid_after_first.content to see how they are different.

This is visible in the output

ok: [/cache/centos-8.qcow2] => {"changed": false, "content": "MTQ0OAo=", "encoding": "base64", "source": "/var/run/sshd.pid"}
...
ok: [/cache/centos-8.qcow2] => {"changed": false, "content": "NTQ2MAo=", "encoding": "base64", "source": "/var/run/sshd.pid"}

But it does not help much with debugging:
$ echo "NTQ2MAo=" | base64 -d
5460
$ echo "MTQ0OAo=" | base64 -d
1448

But it does not help much with debugging:

Are you able to reproduce the failure locally, using the qemu based tests with local qcow2 images?

But it does not help much with debugging:

Are you able to reproduce the failure locally, using the qemu based tests with local qcow2 images?

I think Ondrej tried to get the information what images are used in the CI but without success.
Using official centos images CentOS-8-GenericCloud-8.3.2011-20201204.2.x86_64.qcow2 and CentOS-8-GenericCloud-8.2.2004-20200611.2.x86_64.qcow2 work as expected.

Correct. I will try to workaround it by using bind or libreswan instead of sshd that is clearly customized in the CI image which caused this issue (I think, I have no access to CI harness, it is a private stuff, apparently). Anyway, I suspect bind and libreswan won't be installed in the image. Is there a way to define packages required by a playbook?

Correct. I will try to workaround it by using bind or libreswan instead of sshd that is clearly customized in the CI image which caused this issue (I think, I have no access to CI harness, it is a private stuff, apparently). Anyway, I suspect bind and libreswan won't be installed in the image. Is there a way to define packages required by a playbook?

- name: Install packages
  package:
    name: bind librewan
    state: present

https://github.com/linux-system-roles/crypto_policies/blob/master/tasks/main.yml#L17

Using official centos images CentOS-8-GenericCloud-8.3.2011-20201204.2.x86_64.qcow2 and CentOS-8-GenericCloud-8.2.2004-20200611.2.x86_64.qcow2 work as expected.

CI is using the 8.2 image - we need to update to 8.3 - @jharuda - but that isn't the problem
AFAICT, the problem is only with ansible 2.8, not ansible 2.9?  Have you tried to reproduce the problem with ansible 2.8 locally?

Correct. I will try to workaround it by using bind or libreswan instead of sshd that is clearly customized in the CI image which caused this issue (I think, I have no access to CI harness, it is a private stuff, apparently). Anyway, I suspect bind and libreswan won't be installed in the image. Is there a way to define packages required by a playbook?

- name: Install packages
  package:
    name: bind librewan
    state: present

Almost - use package: [bind, libreswan] - the argument to package must be a list if installing more than one package

AFAICT, the problem is only with ansible 2.8, not ansible 2.9? Have you tried to reproduce the problem with ansible 2.8 locally?

I can reproduce with ansible 2.8:
$ cd crypto_policies/tests
$ python -mvenv .venv
$ . .venv/bin/activate
$ pip install 'ansible==2.8.*' 'jinja2==2.7.*'
$ TEST_SUBJECTS=/path/to/centos8.qcow2 ansible-playbook ....

here's the output of the "Ensure required packages are installed" task from ansible 2.8:
TASK [linux-system-roles.crypto_policies : Ensure required packages are installed] ***
task path: /home/rmeggins/linux-system-roles/crypto_policies/tasks/main.yml:17
changed: [/var/lib/libvirt/images/CentOS-8-GenericCloud-8.2.2004-20200611.2.x86_64.qcow2] => {
    "changed": true,
    "rc": 0,
    "results": [
        "Installed: crypto-policies",
        "Installed: gnutls-3.6.14-6.el8.x86_64",
        "Installed: crypto-policies-20200713-1.git51d1222.el8.noarch",
        "Installed: crypto-policies-scripts-20200713-1.git51d1222.el8.noarch",
        "Installed: openssh-8.0p1-5.el8.x86_64",
        "Installed: openssh-clients-8.0p1-5.el8.x86_64",
        "Installed: openssh-server-8.0p1-5.el8.x86_64",
        "Removed: gnutls-3.6.8-10.el8_2.x86_64",
        "Removed: crypto-policies-20191128-2.git23e1bf1.el8.noarch",
        "Removed: openssh-8.0p1-4.el8_1.x86_64",
        "Removed: openssh-clients-8.0p1-4.el8_1.x86_64",
        "Removed: openssh-server-8.0p1-4.el8_1.x86_64"
    ]
}

here's the task output from ansible 2.9
TASK [linux-system-roles.crypto_policies : Ensure required packages are installed] ***
task path: /home/rmeggins/linux-system-roles/crypto_policies/tasks/main.yml:17
ok: [/var/lib/libvirt/images/CentOS-8-GenericCloud-8.2.2004-20200611.2.x86_64.qcow2] => {
    "changed": false,
    "rc": 0,
    "results": []
}

MSG:

Nothing to do

I'm assuming sshd restarted when the openssh packages were installed, which explains why sshd pid is different.
So I'm investigating what is the difference between the package module in ansible 2.8 and 2.9

so the crypto-policies package is really a "meta" package:
$ rpm -q --provides crypto-policies
config(crypto-policies) = 20200619-1.git781bbd4.fc32
crypto-policies = 20200619-1.git781bbd4.fc32

That is, the rpm crypto-policies is really just a provide of the "real" packages crypto-policies-20191128-2.git23e1bf1.el8.noarch and crypto-policies-20200713-1.git51d1222.el8.noarch.
I suspect ansible 2.8 has some sort of bug installing "meta" packages that causes it to think it needs to do an upgrade.
@Jakuje when the role runs, do you want it to check if crypto-policies exists, install it if not, and upgrade it if it already exists?

@The-Mule depending on what @Jakuje wants to do, you might not be able to use the test "was sshd restarted?" - is there some other test you could use to verify the result is as expected?

so the crypto-policies package is really a "meta" package:
$ rpm -q --provides crypto-policies
config(crypto-policies) = 20200619-1.git781bbd4.fc32
crypto-policies = 20200619-1.git781bbd4.fc32

That is, the rpm crypto-policies is really just a provide of the "real" packages crypto-policies-20191128-2.git23e1bf1.el8.noarch and crypto-policies-20200713-1.git51d1222.el8.noarch.
I suspect ansible 2.8 has some sort of bug installing "meta" packages that causes it to think it needs to do an upgrade.
@Jakuje when the role runs, do you want it to check if crypto-policies exists, install it if not, and upgrade it if it already exists?

Thank you for looking into that! The crypto policies package should really be part of minimal operating system (it is dependency of all crypto libraries) so I considered that as a sanity check, making sure it is installed (and making sure the tools in RHEL 8.1+ are available from crypto-policies-scripts package). I am not sure how good idea is to update packages (probably not) from various roles.
But it is great that it uncovered an issue of old images here.
Given the above findings, I see that a simple workaround in the test would be first running the role without arguments and recording the sshd pid only after that.
But given the sheer difference in behavior between ansible versions, I think it might be something worth investigating.

I think https://github.com/linux-system-roles/crypto_policies/pull/5 will fix the problem.

As I said in #5, I do not think that updating a package is that big issue for crypto policies role. But it indeed might be an issue for customers. This test can workaround the issue by swapping the phases (first check for reload=true and then for reload=false). Additionally, we could try write one more test for checking the package was not updated while running the role (probably as part of #5).

@The-Mule can you fix the last comment from Rich and rebase on top of current master so we can check it works?

Absolutely, I'll do that tmrw.
On Sat, Dec 19, 2020, 23:20 Jakub Jelen<EMAIL_ADDRESS>wrote:

@The-Mule https://github.com/The-Mule can you fix the last comment from
Rich and rebase on top of current master so we can check it works?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/linux-system-roles/crypto_policies/pull/3#issuecomment-748532908,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACT6OBD52IS4MAXPW6A7QH3SVURLXANCNFSM4UXEFNTQ
.

This test can workaround the issue by swapping the phases (first check for reload=true and then for reload=false).

I would prefer not to do this. It would affect the test for reload - reload in CP or associated role can be broken and the bug will be hidden by reload caused by CP update.
I did what you suggested originally - to add one more role invocation in the beginning.

[citest commit:c0108cf8bbe1b5b92954a3e2edc6dfd5afff3f51]

[citest commit:b98fe45cfbee64a46f0f5acdf3b01e253c15267e]


Deploying a model with kfserving continuously deploys 100's of models

When I deploy a custom container mode on kfserving, it continuously deploys 100's of models until the model is deleted.
namespace = 'modelserve'
service_name = "kfserving-custom-model"

predictor = V1beta1PredictorSpec(
    min_replicas=1,
    containers=[{
        "image":"docker.io/XXX/XXX:latest",
         "name":service_name,
        "resources": {
            "limits": {"cpu": "500m", "memory": "4Gi"},
            "requests": {"cpu": "250m", "memory": "2Gi"},
        }
    }],
)

isvc = V1beta1InferenceService(
    api_version=constants.KFSERVING_V1BETA1,
    kind=constants.KFSERVING_KIND,
    metadata=client.V1ObjectMeta(
        name=service_name, namespace=namespace
    ),
    spec=V1beta1InferenceServiceSpec(predictor=predictor),
)

Envrionment: Azure AKS
Kubeflow Config: https://raw.githubusercontent.com/kubeflow/manifests/v1.2-branch/kfdef/kfctl_istio_dex.v1.2.0.yaml
KFserving Config: https://github.com/kubeflow/kfserving/releases/download/v0.6.0/kfserving.yaml
Notes:

The test docker container is public, so access should not be an issue.
I tested this exact set-up a few months ago and it all worked perfectly.
I have reinstalled the environment 5+ times and have not been able to get it to work.
I have tried setting the max number of replicas to 2, this does not stop it.
I run the above code from a Jupiter notebook on kubeflow.

Let me know if you require any particular logs and the command to get the logs please :)

@rileygreen022 Did you install KServe with kubeflow or standalone, seems like you installed two version of KServe?

Ah I see, when I tried to deploy a model with the kfctl_istio_dex.v1.2.0.yaml standalone, I don't think it had the V1beta1 API.
Should I remove the following lines below when installing and then complete the same steps installing 0.0.6?
  - kustomizeConfig:
      repoRef:
        name: manifests
        path: kfserving/installs/generic
    name: kfserving

Anything else I need to do?

@rileygreen022 I'd highly recommend you us kubeflow 1.3 or recently released 1.4 which gets you to v1beta1 API


feat(ajax): adds ajax methods from rx-dom.

@trxcllnt why is this necessary to be in core versus just as a separate module?

@mattpodwysocki .... I assume by "separate module" you mean "different npm package"? or "different repo"?
Now that everything is modular, there's really no reason to have them all separated. There are some consistency benefits to having them all in one repository.

@blesh @trxcllnt I'm less concerned about the bringing in what you need, I'm more interested in the versioning and the separation of concerns. By default, RxJS is a platform neutral library with no real hooks into the DOM (outside of schedulers).
Instead, I'd like to see this as a separate module that can age/version on its own and be combined with other DOM features.  That and bringing Rx via NPM or other means won't download things you won't ever use, such as Ajax, especially if you're using it on the server-side.  Just adds extra bloat to the node_modules folder without any real gain.

I'd like to keep everything in this repository. Whether or not they should be in separate npm packages is subject for debate. Here's a decent article on the advantages of the monolithic repository. While, I don't want to go Full-Facebook on it. I think it has merit on a smaller scale.
The HARD part of this is figuring out the best way to manage supporting 3 different delivery styles: CJS, ES6 and globals if we're making these separate packages.

@trxcllnt for now I've merged this into a new branch rxjs-dom. I'm going to continue working against that to fix a few minor issues and refactor a bit to match the rest of the library. We'll merge it over when we decide how we're going to organize it.


Copy code-icon.svg to near where it's used

https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/walkThrough/electron-browser/walkThroughPart.css#L144
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/welcomePage/electron-browser/welcomePage.css#L263

The icon should be copied close to the CSS where it is being used, just like every icon and image we have.
The only reason this is working now is that @alexandrudima enabled SVG inlining when the SVG is small enough and the loader actually is able to escape the vs namespace during bundling.

Removed candidate label since not user visible, up to Christof to decide wheather to fix today.

@alexandrudima @joaomoreno What is the size limit an SVG will be inlined? If this doesn't directly affect 1.9, I'll fix it for February.

Looks like the limit is 5k (3k for the editor), the files involved are below 1k (before minification). Moving to February.

Verified via code review


Ubuntu support: Enable creation of gppkg on Ubuntu

This commit makes necessary changes to create a gppkg on Ubuntu. The
default behavior when MADlib is built on Ubuntu is to create a .deb
installer. If we want to create a gppkg, then we need an RPM due to
limitations in gppkg. We now create an RPM on Ubuntu (assuming package
alien is installed on Ubuntu) if the right cmake flag is specified. Once
an RPM is created on make package, we can now go ahead and create the
gppkg using make gppkg.
The cmake flag to use if we want to create an .rpm instead of .deb on
Ubuntu when we run make package is:
-DCREATE_RPM_FOR_UBUNTU=True
Co-authored-by: Orhan Kislal<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/657/

Thank you for the comments @reductionista , I have updated the comment. Please do have a look at it.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/658/

So this requires Alien, but we do not automatically download or bundle Alien, correct?

Thanks @njayaram2 for the clarification.

Also:  a user who downloads the madlib gppkg for Ubuntu must install Alien first, before running gppkg -i on it.


Else / el((s(e ?)?)?if / multi-word keywords

I don't think there are else and else if keywords/blocks. This is indeed minimalistic, but it might be more practical to have them. Otherwise could be an alias for else (without if). I don't have a clue for a single keyword for else if at the moment. In general, one word keywords are better maintainable. What if the parser expects a long keyword like take it to the top and the code accidentally misses a letter? An error should be thrown instead of unexpected behaviour. This is clear, but during implementation, one might forget this at some point and this is not too easy to maintain, if you always have to check that again...

However could be an alias for else if:
If My Life is nothing
  Whisper "it is nothing"
However My Life is true
  Say "it is true"
Otherwise
  Shout My Life


Create virtual environment failed

When the project packaged with the -v parameter is installed on the server, an error is reported: sh: virutalenv: command not found.

Fixed


when you switch tutorials (or any games) you are looking at the last game you ran

For eg, when I switch from walk to bean, walk is still in the player window

Good observation! It clears the game screen now. Could also fix by running games when loaded but don't want to run unchecked JS. Maybe if we decide on a preferable sandboxing method it would be okay.


No ViewManager defined for class LottieAnimationView

No ViewManager defined for class LottieAnimationView

I had the same issue. I resolved it like:

Re run : react-native run-android
This might create a build failure.
cd android && ./gradlew clean

Now run again.. it should wait.

I believe this should now be fixed in<EMAIL_ADDRESS>Please try it out and let me know if you still have trouble. Closing this issue for now. Thanks!

I did have the same problem using<EMAIL_ADDRESS>today

I ran into the same issue here and i found a work around below. Just for context i'm using a hybrid app (react + android) with other RN modules and react native v0.42. A doc on this would really help devs like me.

include modules under the settings.gradle file and resync.

include ':lottie-react-native'
project(':lottie-react-native').projectDir = new File(rootProject.projectDir, '../../node_modules/lottie-react-native/lib/android')

Override the react native host with LottiePackage in the activity where RN is initialized.

           @Override
            public ReactNativeHost getReactNativeHost() {
                    reactNativeHost = new ReactNativeHost(getApplication()) {
                      ...
                        @Override
                        protected List<ReactPackage> getPackages() {
                            return Arrays.asList(
                                  ...
                                    new LottiePackage()
                            );
                        }
                    };
                return reactNativeHost;
            }

Don't forget write the import line or will not compile.
import com.airbnb.android.react.lottie.LottiePackage;

please reopen, having this issue on android not on ios

i did have the same problem and 'ottie-react-native' of version is 1.1.1 。what can i do?

Same Issue

Same problem Here
RN : 0.48.4
Lottie : 2.2.7

True. Same issue, even after react-native link.


[meta] Handle large assets/scenes

This broadly needs more investigation.

An object with many children can jank the tree view
Refreshing some objects may cause a new toJSON() cycle in content, potentially causing large amounts of jank from serializing and sending giant strings of textures or buffers
Right now an entire scene is serialized on first request; should there be a difference between requesting the "overview" of an entity and the "data" (e.g. potentially very large base64 strings or buffers), and access the larger data lazily when needed?
If we wanted to modify the built in toJSON method for some type, like to only serialize the first n children, is there a way to do that without having to duplicate/rewrite the serialization logic for objects, materials, textures, etc...
Are we conservative in sending data over the wire? I think there's a duplicate request somewhere at least.
In the devtools, a comparison occurs to determine whether or not to emit an 'update' event -- can this be moved to the content and get rid of unnecessary wire transfers?

Now handled, explained https://github.com/threejs/three-devtools/issues/43#issuecomment-569558471


Replace assert_vector by eq_vector

Suggested by @aszepieniec.
By returning the result of the quintuple equality test instead of crashing the VM on inequality, a programmer can test whether a hash digest equals the Merkle root of any Merkle tree, for example the ones making up a Merkle Mountain Range. Other use cases might exist.
If desired, a crash can still be induced by calling already existing instruction assert on the result of eq_vector.

One of the most straightforward ways of arithmetizing the instruction-to-be eq_vec is
$$\texttt{st0'} - (\texttt{st0} - \texttt{st5}) \texttt{hv0} \cdot \dots \cdot (\texttt{st4} - \texttt{st9}) \texttt{hv4}$$
and an additional 10 polynomials ensuring that the helper variables hvi are indeed the inverse of $\texttt{sti} - \texttt{st(i+5)}$ (or zero if no inverse exists).
This gives rise to a total of 11 polynomials, 10 of which are of degree 3, one of which is of degree 10.

One way of arithmetizing instruction-to-be eq_vector is
$$ \begin{align}
(st0' - 1) &· (1 - (st0 - st5)·hv0)\\
&· (1 - (st1 - st6)·hv1)\\
&· (1 - (st2 - st7)·hv2)\\
&· (1 - (st3 - st8)·hv3)\\
&· (1 - (st4 - st9)·hv4)
\end{align} $$
$$ \begin{align}
&(st0 - st5)·hv0·st0' \\
&(st1 - st6)·hv1·st0' \\
&(st2 - st7)·hv2·st0' \\
&(st3 - st8)·hv3·st0' \\
&(st4 - st9)·hv4·st0' \\
&\ \
&(1 - (st0 - st5)·hv0)·hv0         \\
&(1 - (st0 - st5)·hv0)·(st0 - st5) \\
&(1 - (st1 - st6)·hv1)·hv1         \\
&(1 - (st1 - st6)·hv1)·(st1 - st6) \\
&(1 - (st2 - st7)·hv2)·hv2         \\
&(1 - (st2 - st7)·hv2)·(st2 - st7) \\
&(1 - (st3 - st8)·hv3)·hv3         \\
&(1 - (st3 - st8)·hv3)·(st3 - st8) \\
&(1 - (st4 - st9)·hv4)·hv4         \\
&(1 - (st4 - st9)·hv4)·(st4 - st9) \\
\end{align} $$
This gives rise to a total of 16 polynomials – 15 of degree 3 and one of degree 11.
In contrast, current instruction assert_vector requires 6 polynomials, all of which have degree 1.

eq_vector as a pseudo instruction:
dup 0
dup 5
eq
dup 2
dup 7
eq
mul
dup 3
dup 8
eq
mul
dup 4
dup 9
eq
mul
dup 5
dup 10
eq
mul

Using 19 instructions as opposed to making the AIR more complex seems like the better option.


fix: github release install silently fails if no urls found

I use this script for my own personal build and found that if the script does not find any valid rpms it will silently continue. This change creates an array and simply selects the first url (as the current script does). If there are no urls then rpm-ostree install should fail.

I use this script for my own personal build and found that if the script does not find any valid rpms it will silently continue. This change creates an array and simply selects the first url (as the current script does). If there are no urls then rpm-ostree install should fail.

@bcook254 I just stumbled onto this PR as it never seemed to cross my radar before.
Appreciate the simple solution, as the author of this script, I've run into the problem myself and hadn't fixed it yet.


Staking functions for the SDK

Working on creating staking functions for the SDK. First up, TS SDK, goal is to create something like
await contract.staking20.stake(20);

Same can be made for ERC-721 and ERC-1155 staking.

ERC-20 staking API complete, let me know if you have any questions.

/release-pr

@atharvadeosthale any updates on this? would love to see some unit tests

hey, was caught up with some devrel tasks, I can now focus on completing this in the coming couple of days.

@atharvadeosthale any updates here? do you need someone to take this over?


GITEA_RUNNER_GID and GITEA_RUNNER_UID show no effect

Possible follow up on #9.
I did set the user and group id environment variables in the docker-compose.yml but the container seems not to use them.
My docker-compose:
version: '3.8' # https://docs.docker.com/compose/compose-file/compose-versioning/

services:

  gitea_act_runner:
    image: vegardit/gitea-act-runner:latest
    container_name: act-runner
    environment:
      - GITEA_RUNNER_LOG_LEVEL=DEBUG
      - GITEA_RUNNER_UID=115
      - GITEA_RUNNER_GID=120
    restart: always
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock:rw
      - ./act_runner/data:/data:rw # the config file is located at /data/.runner and needs to survive container restarts
    environment:
      TZ: "Europe/Berlin"
      # config parameters for initial runner registration:
      GITEA_INSTANCE_URL: SECRET_URL
      GITEA_RUNNER_REGISTRATION_TOKEN: SECRET_TOKEN

And the log of the container start:
   _____ _ _                            _     _____
  / ____(_) |                 /\       | |   |  __ \
 | |  __ _| |_ ___  __ _     /  \   ___| |_  | |__) |   _ _ __  _ __   ___ _ __
 | | |_ | | __/ _ \/ _` |   / /\ \ / __| __| |  _  / | | | '_ \| '_ \ / _ \ '__|
 | |__| | | ||  __/ (_| |  / ____ \ (__| |_  | | \ \ |_| | | | | | | |  __/ |
  \_____|_|\__\___|\__,_| /_/    \_\___|\__| |_|  \_\__,_|_| |_|_| |_|\___|_|

GIT_REPO:    https://github.com/vegardit/docker-gitea-act-runner
GIT_BRANCH:  main
GIT_COMMIT:  dfc5a30 @ 2023-06-04 12:24:15 UTC
IMAGE_BUILD: 2023-06-09T09:24:23Z

2023-06-09 11:52:30 INFO [/opt/run.sh:26] Timezone is CEST +0200
2023-06-09 11:52:30 INFO [/opt/run.sh:27] Hostname: 33d70588a291
2023-06-09 11:52:30 INFO [/opt/run.sh:28] IP Addresses:
 - <IP_ADDRESS>
2023-06-09 11:52:30 INFO [/opt/run_runner.sh:10] Effective user: uid=1000(act) gid=1000(act) groups=1000(act),27(sudo),100(users),119(docker119)
time="2023-06-09T11:52:30+02:00" level=info msg="Starting runner daemon"

Any help greatly appreciated.

Have you tried it with only one environment block that contains all six variables? I ask because when I tested your posted compose file it reported an error with this command:
docker compose -f docker-test.yml config
parsing /home/wblew/docker/kulian-pihole/docker-test.yml: yaml: unmarshal errors:
  line 16: mapping key "environment" already defined at line 8

Maybe with your docker version it's just ignoring the first environment block?

No I haven't and this was obviously the cause of my error.
Placing everything inside the same environment as follows did the trick.
version: '3.8' # https://docs.docker.com/compose/compose-file/compose-versioning/

services:

  gitea_act_runner:
    image: vegardit/gitea-act-runner:latest
    container_name: act-runner
    environment:
      - GITEA_RUNNER_LOG_LEVEL=DEBUG
      - GITEA_RUNNER_UID=115
      - GITEA_RUNNER_GID=120
      - TZ:'Europe/Berlin'
      - GITEA_INSTANCE_URL:URL
      - GITEA_RUNNER_REGISTRATION_TOKEN:TOKEN
    restart: always
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock:rw
      - ./act_runner/data:/data:rw # the config file is located at /data/.runner and needs to survive container restarts

Thanks for you swift reply!


Right-button menu for Scene-file nodes has a bigger offset than normal

Godot version: 3.2.1 (stable)
OS/device including version: ArchLinux
Issue description: When clicking with the right-button on a node added to the scene via a Scene file, the inspector menu that appears will have a weird offset (as shown in the images). This is probably caused by the toggle options "Editable Children" and "Load as Placeholder", as they show more distant to the option label than the rest of the icons.
This is a (very) minor UI bug and I couldn't found something similar happening somewhere else in Godot.
Normal Menu:

Weird Offset Menu:

I don't think this is actually bug, just a limitation of the PopupMenu system which doesn't allow an icon to have both an icon and checkbox at the same place.

Damn. Isn't it better to change the issue tags from 'bug' to 'enchancement', then? Just to leave the issue so someone can tackle it in the future, if the PopupMenu is ever refactored.

Just decide which behavior to use:

checkbox and icon in the same place (with the limitation that the item cannot have both):

always leave space for the checkbox:

As they don't currently have icons I think the "use checkbox as icon" is probably the more elegant UI solution but likely the more complicated code solution.

@Calinou @akien-mga @groud Any executive decision on this on which way to go for this issue? I am fine with current implementation tbh.

Well, if we could make it so that we switch between the 2 behaviors (as proposed by @dankan1890) automatically, it would be perfect. The idea would be to check beforehand if there is at least one menu entry with both a checkbox and an icon.

I am not convinced that merging them into one column is a good idea. I think it would lose some visual clarity. Is it an icon or a checkbox? Observe: where are the checkboxes below? It takes time to find them.

I am not convinced that merging them into one column is a good idea. I think it would lose some visual clarity. Is it an icon or a checkbox? Observe: where are the checkboxes below? It takes time to find them.

Not that much if we make the checkbox have an "highlight" using a contour or something like when when hovering the option.
This could likely be done easily but requires a dedicated PR.


This macro is only reporting warning and doesn't work when called fro…

…m inside a osxcross generated environment.

Please don't tag me as a reviewer for systray. I don't have anything set up to test that with.

This is fixed by also upgrading to llvm v14 which require no code maintenance for us. Seems better.


ToSlice alternative: AddRange

@ahmetalpbalkan The idea here is to offer an alternative to people that are currently using ToSlice method to append the result slice. The .Net Framework offers the AddRange method (I personally don’t like this name, we need to find a better one) that implements this idea.

Hmm I'm not sure if we need an alternative to ToSlice(). Was the prior behavior really desirable?
Also is this really useful without other accompanying methods like GetRange, RemoveRange and InsertRange?

It looks like all the code is the same as ToSlice, except 1 line (index :=). If that's the case perhaps we can make ToSlice call AppendRange(v[:0]) or something like that. In any case, I doubt if there's a need for this method right now. We can take a look at this post-v3.

I was trying to write some use cases to AddRange and I realized they can be solved using the Concat method. 😄


ci: use codecov token

Is this still important? Looks like codecov has been working alright in other PRs?

Yes, it is still relevant. See, for example, https://github.com/uutils/uutils-term-grid/actions/runs/7956349330/job/21716900205, "Upload coverage results".


Remove wnp_count boolean

Please remove the wnp_count (Boolean?) variable, they are here to deploy with a windows nodepool, let’s not make them work harder to have that happen.

yes, agree I will remove it

Removed Boolean variable.


SemanticPass: Kill unames (almost)

The semantic pass added two properties to Variable and MemberExpression nodes:

uname — unique name of the variable or module export, to be used in compiled code
symbol — symbol table entry for the variable or module export

Since the symbol table entry already contained the unique name, setting the uname property on the nodes was redundant, and I decided to remove the uname assignment. Note this makes a lot of sense conceptually — the unique name is a property of the referenced object (variable, function, reducer,…), not of the reference to it, so it belongs to its symbol table entry, not to the node referencing it.
During the work it turned out that uname is also set for nodes which define symbols, such as FunctionDef or ReducerDef. I decided to make these consistent with Variable and MemberExpression and use the symbol property there too. I also did few other small cleanups.
I left the uname property assignment for nodes corresponding to procs as there are no symbols defined for them.
Prep work for #419.

@bkutil

+1


recoveryservicesbackup - Get unfound item returns 400 instead of 404 in 2023-02-01

Ref: https://github.com/Azure/azure-rest-api-specs/blob/3c639105c011765893db51cbb0dda056e34dc994/specification/recoveryservicesbackup/resource-manager/Microsoft.RecoveryServices/stable/2023-02-01/bms.json#L1297
Reported in https://github.com/hashicorp/terraform-provider-azurerm/issues/21663

This can also be reproduced with a simple rest method GET request using api version 2023-02-01. It will give the following output for both existing and non-existing resources:
Bad Request({"error":{"code":"BMSUserErrorCrossRegionRestoreNotEnabledInThisVault","message":"Cross Region Restore is not enabled in this vault. Please enable Cross Region Restore in this vault."}})

Meanwhile api version 2021-02-10 returns as expected:
Not Found({"error":{"code":"BMSUserErrorDataSourceObjectNotFound","message":"Item not found. Selected item might have been deleted. Please check if item is registered to the vault."}})

The issue seems to be resolved according to the latest comments from https://github.com/hashicorp/terraform-provider-azurerm/issues/21663, please feel free to close the issue once this is confirmed.

Adding Service team to confirm


Configure Travis CI to only build PRs

Right now we've got Travis CI configured to build on both PR and Pushses, which leads to what's effectively two independent builds whenever a PR is opened and whenever we push new updates to that PR.
There's discussion about this at https://github.com/travis-ci/travis-ci/issues/3241
I think we can get away with just switching the configuration to build PRs and get the behavior that we want, cutting down on the number of builds that need to be run without sacrificing testing.
I'm going to make this change and will open a dummy PR to confirm that we're still seeing at least one build done, and then close this issue if everything looks good.
CC: @addyosmani @gauntface

It looks like it's working:

There were builds triggered due to the initial https://github.com/GoogleChrome/sw-helpers/pull/192 and then a follow-on commit that was added to the branch while the PR was open, which is the behavior I expected.
I'm going to close this, but if there's some reason that anyone would prefer builds restored for each Push, it's easy enough to flip the switch back at https://travis-ci.org/GoogleChrome/sw-helpers/settings


[Typescript] Fix return type for itemToString under Actions interface

What:
[Typescript] Fixed the return type for itemToString under Actions interface

Why:
Types for itemToString() between DownshiftProps and Actions are not consistent.

How:
Changed the return type from void to string

Checklist:

[ ] Documentation
[ ] Tests
[ ] Ready to be merged

[ ] Added myself to contributors table

Thanks a bunch for the help!

Codecov Report

Merging #359 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #359   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines         330    330
  Branches       85     85
=====================================
  Hits          330    330

Continue to review full report at Codecov.

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

An interesting thing here: we have itemToString under Actions, although it’s not an action. It just happened to be a function.
I believe, next time we do something related to typings, we can refactor it. It’s not something causing a bug, but it’d be better to fix it in the future.


Invalid access of discarded (invalid, deleted) uWS.HttpResponse/SSLHttpResponse

Static files are served succesfully from webpack dist and public folders via
app.folder('/', "public")
app.file('/', "dist/index.html").folder('/', "dist")

However, when refreshing the page and cache (via CTRL+SHIFT+R), the server throws an error.
/Users/mdealiaga/code/io-2/node_modules/@sifrr/server/src/server/sendfile.js:165
      res.end();
          ^
Invalid access of discarded (invalid, deleted) uWS.HttpResponse/SSLHttpResponse.
Thrown at:
    at /Users/mdealiaga/code/io-2/node_modules/@sifrr/server/src/server/sendfile.js:165:11
    at emit (events.js:323:22)
    at endReadableNT (_stream_readable.js:1204:12)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)

There is a similar uWebSocket issue report https://github.com/uNetworking/uWebSockets.js/issues/123 where the fix was to "Look over the docs regarding onAborted - you have to track whether the response was aborted or not."
This link to onAborted is https://unetworking.github.io/uWebSockets.js/generated/interfaces/httpresponse.html#onaborted
If I serve only my dist folder, this is not an issue. It only happens when additionally also serving the public folder.

I think I was using onAborted, but will check again.

Is there any solution to this?
Unfortunately I have the same issue.


Short solution needed: "ffmpeg keyframe interval" (ffmpeg)

Please help us write most modern and shortest code solution for this issue:
ffmpeg keyframe interval (technology: ffmpeg)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to change keyframe interval using keyInt


Solarfactors notebook

See #8

Let me know what you think @kandersolar, and if I've missed doing something.

Thanks @echedey-ls!  The figure below doesn't look great by default, but I think it's okay since the figure is interactive (zoom, hover, etc).

Is there anything else you want to do before merging?  It is fine with me as-is.

Honestly, I wasn't able to make sphinx build the webpage, only the notebook. But if you can make it work, that's fine then.

Is there anything else you want to do before merging? It is fine with me as-is.

Nothing else from my side, no.

Thanks @echedey-ls!


python3Packages.torch-bin, python3Packages.torchvision-bin, python3Packages.torchaudio-bin: update cuda 11.6 -> 11.7

Description of changes
Updated the cuda version that torch-bin and torchvision-bin are built on from 11.6 to 11.7.
This is the cuda version currrently shipped in NixOS and it is the one that this version of pytorch is officially supporting.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Testing
Result of nixpkgs-review run on x86_64-linux 1

5 packages built:

python310Packages.torch-bin
python310Packages.torchaudio-bin
python310Packages.torchvision-bin
python310Packages.trainer
tts

@junjihashimoto

I want to add that this PR unbreaks the build process of torch-bin.
(currently broken on master)

One commit per attribute please.

Done :)

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

4 packages built:

python310Packages.torchaudio-bin
python310Packages.torchvision-bin
python310Packages.trainer
tts


Add native logging output

llama.cpp commit added output of the log to a callback
ea04a4c
Added native methods and a way to register though the existing Logger, although probably a temp solution
Order the current LogEnum to match to save making a duplicate.
Edit:
Ugggggh.. I think we have different tab sizes, do we have a preference?

Oh that's great, I was actually planning to investigate what it would take to replace the llama.cpp logging with a callback exactly like this!

Ugggggh.. I think we have different tab sizes, do we have a preference?

Yeah indentation seems to be very messed up in this PR.
Visual studio uses 4 spaces, so let's stick to that.

Thanks!


Fix search queries not opening in new tab

Fixes https://github.com/alyssaxuu/omni/issues/79

Can we also get an extension option to have this be the default behavior?
I'm trying to replicate Arc's (my main browser) experience when I'm working with Chrome (dev browser) and this would be really helpful 🤩


Admin sets and collections inaccessible

Double check to make sure the move to knapsack will not resolve this
Update: CONFIRMED - this is not be resolved with knapsack
Summary
Newly created user collections and admin sets on both staging and production are inaccessible after creation and save. From the Collection list on the dashboard, when we click on the new admin set or collection to edit the back end, we get this error:

The admin sets I created to test this error are here:
Staging - https://adl.s2.adventistdigitallibrary.org/admin/admin_sets/f9c791d8-687d-4e4f-86c7-d8e3599cc0f6?locale=en
Production - https://adl.b2.adventistdigitallibrary.org/admin/admin_sets/0e48eca3-e5cd-43a5-8a70-0e3f388591be?locale=en
Here's a staging user collection that I can't edit--I only get the error message: https://testing.s2.adventistdigitallibrary.org/dashboard/collections/dde6eae7-6778-4574-9473-fa3f72942d99?locale=en
I can't edit existing collections on prod. Here's a screenshot of my latest attempt (2023-11-20):

Acceptance Criteria

[x] Creating a collection does not throw an error on staging
[x] Creating an admin set does not throw an error on staging
[x] Editing a collection does not throw an error on staging
[x] Editing an admin set does not throw an error on staging
[x] Creating a collection does not throw an error on production
[x] Creating an admin set does not throw an error on production
[x] Editing a collection does not throw an error on production
[x] Editing an admin set does not throw an error on production

Investigation results
Ref:

scientist-softserv/adventist_knapsack#211

The error is being thrown because the forms are trying to render a partial that's missing. Specifically, app/views/hyrax/dashboard/collections/_current_thumbnail.html.erb.
This file was deleted on Oct 16 in commit:

https://github.com/scientist-softserv/adventist-dl/commit/032f58450daf90a6db8e88a1279979fe524a3e0d#diff-80ba3830edda78d3b05df34e72259c25036be18c64e093f834d07c83b40525b6

This file still exists in upstream Hyku.
The deletion seems unrelated to the other changes in the commit, so I'm guessing it was accidental? Restoring the file should prevent the error from happening.
Error trace
I, [2023-12-04T19:23:44.918218 scientist-softserv/adventist-dl#1]  INFO -- : [b7ce4b41a4af43bbf5f3f2c68680df59] Started GET "/dashboard/collections/4fc890b3-c81d-49c9-8c38-12ad2d30ac16/edit?locale=en" for <IP_ADDRESS> at 2023-12-04 19:23:44 +0000
I, [2023-12-04T19:23:45.106201 scientist-softserv/adventist-dl#1]  INFO -- : [b7ce4b41a4af43bbf5f3f2c68680df59] Completed 500 Internal Server Error in 173ms (ActiveRecord: 33.9ms)
D, [2023-12-04T19:23:45.110124 scientist-softserv/adventist-dl#1] DEBUG -- : [b7ce4b41a4af43bbf5f3f2c68680df59]   Account Load (1.1ms)  SELECT  "public"."accounts".* FROM "public"."accounts" WHERE "public"."accounts"."tenant" = $1 LIMIT $2  [["tenant", "public"], ["LIMIT", 1]]
I, [2023-12-04T19:23:45.221119 scientist-softserv/adventist-dl#1]  INFO -- : [b7ce4b41a4af43bbf5f3f2c68680df59] Sending event 36f429bcd5c84be289a05649a6cba7f8 to Sentry
D, [2023-12-04T19:23:45.239583 scientist-softserv/adventist-dl#1] DEBUG -- : [b7ce4b41a4af43bbf5f3f2c68680df59] Raven HTTP Transport connecting to https://o1008683.ingest.sentry.io
F, [2023-12-04T19:23:45.316581 scientist-softserv/adventist-dl#1] FATAL -- : [b7ce4b41a4af43bbf5f3f2c68680df59]
F, [2023-12-04T19:23:45.316647 scientist-softserv/adventist-dl#1] FATAL -- : [b7ce4b41a4af43bbf5f3f2c68680df59] ActionView::Template::Error (Missing partial hyrax/dashboard/collections/_current_thumbnail, hyrax/my/collections/_current_thumbnail, hyrax/my/_current_thumbnail, application/_current_thumbnail, catalog/_current_thumbnail, hyrax/base/_current_thumbnail with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:raw, :erb, :html, :builder, :ruby, :coffee, :jbuilder]}. Searched in:
  * "/app/samvera/hyrax-webapp/app/views"
  * "/usr/local/bundle/bundler/gems/iiif_print-5c5fa6880b3d/app/views"
  * "/usr/local/bundle/gems/blacklight_range_limit-6.5.0/app/views"
  * "/usr/local/bundle/gems/blacklight_advanced_search-6.4.1/app/views"
  * "/usr/local/bundle/gems/good_job-2.99.0/app/views"
  * "/usr/local/bundle/gems/devise_invitable-1.7.5/app/views"
  * "/usr/local/bundle/gems/devise-i18n-1.9.1/app/views"
  * "/usr/local/bundle/gems/devise-4.7.2/app/views"
  * "/usr/local/bundle/gems/qa-5.11.0/app/views"
  * "/usr/local/bundle/gems/flipflop-2.7.1/app/views"
  * "/usr/local/bundle/gems/bulkrax-5.3.0/app/views"
  * "/usr/local/bundle/gems/hyrax-2.9.6/app/views"
  * "/usr/local/bundle/gems/browse-everything-1.3.0/app/views"
  * "/usr/local/bundle/gems/hydra-editor-5.0.5/app/views"
  * "/usr/local/bundle/gems/blacklight-gallery-0.12.0/app/views"
  * "/usr/local/bundle/gems/mailboxer-0.15.1/app/views"
  * "/usr/local/bundle/gems/blacklight-6.25.0/app/views"
  * "/usr/local/bundle/gems/kaminari-core-1.2.2/app/views"
  * "/usr/local/bundle/gems/bulkrax-5.3.0/app/views"
):
F, [2023-12-04T19:23:45.316821 scientist-softserv/adventist-dl#1] FATAL -- : [b7ce4b41a4af43bbf5f3f2c68680df59]     43:                 Uploading an image here will replace a thumbnail selected from the above dropdown. For best results, use an image at least 500(width) x 900(height) px. If you would like to use an image from the dropdown above, click the "Clear upload" button below.
[b7ce4b41a4af43bbf5f3f2c68680df59]     44:               </p>
[b7ce4b41a4af43bbf5f3f2c68680df59]     45:               <div id="uploaded-thumbnail mb-2">
[b7ce4b41a4af43bbf5f3f2c68680df59]     46:                 <%= render "current_thumbnail" %>
[b7ce4b41a4af43bbf5f3f2c68680df59]     47:               </div>
[b7ce4b41a4af43bbf5f3f2c68680df59]     48:               <%= f.file_field :thumbnail_upload, class: "thumbnail-upload" %>
[b7ce4b41a4af43bbf5f3f2c68680df59]     49:               <%= link_to("Clear upload", main_app.delete_uploaded_thumbnail_path, method: :post, remote: :true, class: "btn btn-danger mb-4 mt-2", id: "clear-upload") %>
F, [2023-12-04T19:23:45.316856 scientist-softserv/adventist-dl#1] FATAL -- : [b7ce4b41a4af43bbf5f3f2c68680df59]
F, [2023-12-04T19:23:45.316879 scientist-softserv/adventist-dl#1] FATAL -- : [b7ce4b41a4af43bbf5f3f2c68680df59] app/views/hyrax/dashboard/collections/_form.html.erb:46:in `block in _app_views_hyrax_dashboard_collections__form_html_erb___1756626244783578818_559480'
[b7ce4b41a4af43bbf5f3f2c68680df59] app/views/hyrax/dashboard/collections/_form.html.erb:26:in `_app_views_hyrax_dashboard_collections__form_html_erb___1756626244783578818_559480'
[b7ce4b41a4af43bbf5f3f2c68680df59] app/middleware/no_cache_middleware.rb:13:in `call'
<IP_ADDRESS>, <IP_ADDRESS> - - [04/Dec/2023:19:23:45 +0000] "GET /500?locale=en HTTP/1.0" 500 1477 0.4008

Testing this on staging I was able to create a collection, create a admin set and edit and save both without error.
Testing this on production I was able to create a collection, create a admin set and edit and save both without error.
Video demo: https://share.zight.com/KoukdE4l
@KatharineV are you still seeing this issue on your end?

@aprilrieger sorry for the late reply on this. I was under a tight deadline that finally passed! I checked the collection functionality today and I am not seeing the issues we had before. It seems like restoring the missing file, referenced above, has restored functionality!


TypeScript Typing Issue with response.results[0].hits in v5

Summary:
The provided reproduction code successfully queries Algolia and returns search results, but TypeScript raises an error during compilation: "Property 'hits' does not exist on type 'SearchResult<ResultDTO>'. Property 'hits' does not exist on type 'SearchForFacetValuesResponse'.ts(2339)". Despite this, the code works as expected at runtime, indicating that this is likely a typing issue within the Algolia client.
Steps to Reproduce:

Clone the Reproduction Repository:
The reproduction code is available in the following GitHub repository: https://github.com/timotheeduran/algolia-repro.
git clone https://github.com/timotheeduran/algolia-repro.git
cd algolia-repro

Install Dependencies:
Ensure all required dependencies are installed by running:
npm install

Replace Algolia Credentials:
Update ALGOLIA_APPLICATION_ID and ALGOLIA_API_KEY in the src/index.ts file with your actual Algolia credentials.

Run TypeScript Compilation:
Execute the TypeScript compiler to check for any typing issues:
npx tsc

Observe the Error:
You should see the following TypeScript error in the terminal:
src/index.ts:24:32 - error TS2339: Property 'hits' does not exist on type 'SearchResult<ResultDTO>'.
  Property 'hits' does not exist on type 'SearchForFacetValuesResponse'.

Run the Code to Verify Runtime Behavior:
To demonstrate that the code works correctly at runtime, run the code using Node.js:
node dist/index.js

Ensure that you are using a real Algolia index and a valid query that is supposed to return hits. When you run the code, you will see that the hits property is correctly returned and accessible, despite the TypeScript error during compilation.

Expected Behavior:
The code compiles without any TypeScript errors.
Actual Behavior:
TypeScript raises an error indicating that the hits property does not exist on the type SearchResult<ResultDTO>. However, at runtime, the code works correctly and returns the expected search results with the hits property accessible and populated.
Conclusion:
The issue appears to be related to the TypeScript types defined in the Algolia client. The hits property is accessible and correctly returned at runtime, but the TypeScript typings do not reflect this. This discrepancy suggests that the typings in the Algolia client need to be reviewed and potentially updated to align with the actual runtime behavior.
Please let me know if you need any further information or assistance!

Hey @timotheeduran thanks a lot for testing the v5 version already and for the thorough issue description
Thanks to your reproduction steps, I managed to find the issue and also tried on v4, which also has this issue. I guess TypeScript is not able to assert if the Nth element of the array is actually of type Search or Facets (see union https://github.com/algolia/algoliasearch-client-javascript/blob/main/packages/client-search/model/searchResult.ts#L6), due to the lack of type guards.
We have two helpers that could help have a proper typing, if you know you won't mix hits and facets result, you can use the searchForHits and searchForFacets methods respectively.

We have two helpers that could help have a proper typing, if you know you won't mix hits and facets result, you can use the searchForHits and searchForFacets methods respectively.

Not in the liteClient from algoliasearch/lite, though, right? 😞

We have two helpers that could help have a proper typing, if you know you won't mix hits and facets result, you can use the searchForHits and searchForFacets methods respectively.

Not in the liteClient from algoliasearch/lite, though, right? 😞

Oh great point, I think it wouldn't increase the bundle size by a lot, I can give it a shot on Monday!

Hey @wKovacs64, this will be available in 5.1.0 in a few minutes :)

Fantastic, thanks @shortcuts !


Enhancement: Make swiping of plugins/sources less sensitive

Make swiping source/plugins less sensitive, related issue #1199
https://github.com/user-attachments/assets/de4136f1-1b25-4f5f-b869-43b1befae593

https://github.com/LNReader/lnreader/issues/1199
this would help too

Btw 62 is too much. It's cranky now. Can u try 40?

I have tried 50 and it was still triggering the swipe, i think this would be quite hard to balance

In ny opinion it's enough to lower the sensivity to 30-40 reducing the issue until a 30° angle thresold is implemented

It is currently hard coded but the best way is probably to turn it into a function of your phone's width

If you dont mind can i have your phone model

In my opinion, we shouldn't adjust its value based on lower-end or older devices. If it's working well for most devices, that's sufficient. He uses a Huawei P8 Lite 2017.

My device is very smooth when using plugin cards, it's not due to that.

My device is very smooth when using plugin cards, it's not due to that because there is no lag, surprisingly.

Ah its a misunderstanding i wanted to know your device so i can get the width and find a correlation

1080x1920. Maybe a % value would be better?

Tested on a Moto One 5G Ace, was able to get minor hitching when installing plugins, but nothing else.

I have a painfully slow tablet I can test on as well if it would be helpful.  However it barely runs a handful of Firefox tabs so I wouldn't say its indicative of the average low end device.

Here is my data, I used an autoclicked with precise coordinates, Macrorify.
-720x1280 (I changed it through settings)
-speed: 20
Value 35 (minimum non-responsive width: 65px)
https://github.com/user-attachments/assets/847b1b55-92ea-469c-8926-c7bbef435916
Value 62 (minimum non-responsive width: 115px)
https://github.com/user-attachments/assets/224b558c-4a34-437b-b735-2eba36a15efe

Practical test videos:
Value: 30
Comfortable, no unwanted trigger
https://github.com/user-attachments/assets/3adea854-c765-4ee5-8f6b-604e68a244e0
Value 25
Some unwanted trigger start to appear. 30 os better.
https://github.com/user-attachments/assets/424b6a04-596e-4183-b7a5-4fd295bc6fb1

The default value was 10(before this commit). Three times it reduces the problems by 90%.
Stopping completely unwanted triggers comes with the price of cranky plugin cards.
It's better to just reduce them.

Ok that sounds like a good compromise, pretty unrealistic to expect no triggers


Missing variable Names

Hi,
on booting, it seems some Language-Variables are missing:

Duplicate of #46


Streamlined blog process

Created a JSON file and associated JSON schema to contain a list of all blog posts
Made blog index dynamically generate the page from said JSON file
Created a script to automatically create the necessary folder structure for a new post and to add the new post to the JSON file

A quick writeup in the README about using the script would be worthwhile

Change to README with simple explanation of script usage, along with changing.
Want to add your own blog post? Send us a pull request. For a simple example of how to write a post, refer to [this post's code](https://github.com/MUNComputerScienceSociety/Website/tree/master/content/blog/Welcome everyone!)
to also note the new script.


Updated composer.json

Updated markguinn/silverstripe-email-helpers to v1.3 (latest version).  Thank you.

Just FYI: This wasn't really necessary, since the version constraint ~1.2 will match all 1.x versions (but not 2.x). Installing with ~1.2 will also install 1.3, 1.4 etc.


GH-40393: [Java] add precondition for NullVector constructor to check field type

Rationale for this change

See https://github.com/apache/arrow/issues/40393
What changes are included in this PR?

This changes constructors of NullVector and ZeroVector to always set the field type to the Null type, and marks constructors that accept a field or a field type as deprecated.
Are these changes tested?
This change should be covered by existing tests.

Are there any user-facing changes?

Yes

GitHub Issue: #40393

Looks like the formatter is unhappy:
Warning:  src/main/java/org/apache/arrow/vector/NullVector.java:[96] (sizes) LineLength: Line is longer than 120 characters (found 133).

Hmm.
https://github.com/apache/arrow/blob/b448b33808f2dd42866195fa4bb44198e2fc26b9/java/vector/src/main/java/org/apache/arrow/vector/ZeroVector.java#L28-L31
@barronw I guess the optimization you wanted originally does exist, it's just a subclass of NulLVector...? That also means the check here isn't possible to do. (This is making every test fail. I didn't realize we had this.)

I don't think they should have been subclasses, since Null IS a distinct type, and this was just an implementation convenience. Maybe other Java maintainers have context, @jduo or @emkornfield?

I don't recall either, I think this might have been just for convenience?


update 3d canvas to latest vedo

This PR updates the existing class to support vedo 3d renderings, also adds an example of usage.

Summary by CodeRabbit

New Features

Introduced a VedoViewerUI class for 3D visualization with points, axes, and a histogram.

Refactor

Updated code to use numpy and medfilt for data filtering in use_click.py.
Renamed self._filtrated to self._filtered across relevant files.
Transitioned from VTK to Vedo for 3D components, affecting class names, type annotations, and method implementations in multiple files.

Documentation

Updated documentation to reflect changes from VTK to Vedo in volume rendering modes.

Hi @marcomusy ,
That looks very nice! Thank you for your contribution.
I left two comments on the changes. Could you take a look at them? I think it's almost ready to merge.

It's very reasonable not to define another API on magicclass side, as vedo already has clean API 👍
I'll merge this PR, and I'll see if there's any better way of the deprecation warning stuff. Thanks!


Per-session kore-rpc server state

In order to use the kore-rpc server concurrently in a safe way, we modify the server to create a new server state per each session. This way, each client will get the same initial server state and calls to add-module will always be safe, because the added modules are per session and not shared between different clients. This of course means that each client has to send the required modules to the server separately.
Part of https://github.com/runtimeverification/hs-backend-booster/issues/383

The ability to share modules between clients in a controlled way can be very useful, I'd prefer a solution that enables that. Did you consider the proposal under https://github.com/runtimeverification/hs-backend-booster/issues/383#issuecomment-1831803781?

The ability to share modules between clients in a controlled way can be very useful, I'd prefer a solution that enables that.

Is this due to performance? Do you expect these modules to be very big?

Did you consider the proposal under runtimeverification/hs-backend-booster#383 (comment)?

I was working under the assumption that we simply want to avoid module clashes and tweaking the global state to only be a session state seemed like a much simpler solution. Using IDs and especially importing modules via IDs whilst maintaining module names completely disjoint from said IDs seemed a much more substantial change to both back-ends.


Theme for Semantic-UI

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
Only supports bootstrap and material css framework.
Expected behavior
Support for Semantic-UI framework.
Reproduction of the problem
No theme.css exists for semantic-ui framework, can't apply css easily, need to style by yourself currently. Also the icons used for sorting are in a <span> element instead of an <i> element (in pager they are in a i element).
What is the motivation / use case for changing the behavior?
Semantic-UI is used by a lot of people. I'm sure some would love to have this great component with semantic-ui styling (including pagination, etc.).
Please tell us about your environment:

Windows 7, Visual Studio Code, npm, angular-cli

Table version: 11.1.7

Angular version: 5.2

Browser: Chrome, IE11

Language: all

Maybe someone has already created a theme file to support semantic-ui?
If yes, please share it here so others can use it too.


Establish connection does not close operation on the specified timeout

Summary
When I add a timeout to establishConnection  the operation is not timed out within the specified time but will throw a BleDisconnectedException with error code 133 instead.
Library version
1.9.0
Preconditions
Have an existing bonding with a device and make sure you turn the bledevice off so it will not respond to the call.
Reproduced the issue on a Google Pixel (Android P) .
Steps to reproduce actual result
1. Make sure the device is switched off
2. Start the app and wait around  6 seconds to establish connection
Minimum code snippet reproducing the issue
    fun connectToDevice(deviceId: String): Observable<RxBleConnection> {
        Timber.d("RXBLE connecting with device $deviceId")
        return rxBleClient.getBleDevice(deviceId)
                .establishConnection(false, Timeout(6000, TimeUnit.MILLISECONDS))
                .doOnError {
                    Timber.d("RXBLE WHOOPS error connecting device: $deviceId error:( $it) ")
                }
                .doOnTerminate{
                    Timber.d("RXBLE terminator establish conn to device: $deviceId terminated ")
                }
    }

Logs from the application running with setting
2019-03-25 18:53:15.830 24619-24619/com.remonh87 D/MyBleClient: RXBLE connecting with device 12:34:55:66:77:88
2019-03-25 18:53:15.899 24619-24680/com.remonh87 D/RxBle#ClientOperationQueue: QUEUED   ConnectOperation(137016935)
2019-03-25 18:53:15.901 24619-24667/com.remonh87 D/RxBle#ClientOperationQueue: STARTED  ConnectOperation(137016935)
2019-03-25 18:53:15.902 24619-24667/com.remonh87 I/RxBle#ClientOperationQueue: RUNNING  ConnectOperation{MAC='12:34:55:66:77:88', autoConnect=false}
2019-03-25 18:53:15.918 24619-24682/com.remonh87 V/RxBle#BleConnectionCompat: Connecting without reflection
2019-03-25 18:53:37.154 24619-24637/com.remonh87 I/RxBle#GattCallback: MAC='12:34:55:66:77:88'  onConnectionStateChange(), status=133, value=0
2019-03-25 18:53:37.174 24619-24637/com.remonh87 D/RxBle#ConnectionOperationQueue: Connection operations queue to be terminated (MAC='12:34:55:66:77:88')
    com.polidea.rxandroidble2.exceptions.BleDisconnectedException: Disconnected from MAC='12:34:55:66:77:88' with status 133 (GATT_ERROR)
        at com.polidea.rxandroidble2.internal.connection.RxBleGattCallback$2.onConnectionStateChange(RxBleGattCallback.java:77)
        at android.bluetooth.BluetoothGatt$1$4.run(BluetoothGatt.java:262)
        at android.bluetooth.BluetoothGatt.runOrQueueCallback(BluetoothGatt.java:770)
        at android.bluetooth.BluetoothGatt.access$200(BluetoothGatt.java:39)
        at android.bluetooth.BluetoothGatt$1.onClientConnectionState(BluetoothGatt.java:257)
        at android.bluetooth.IBluetoothGattCallback$Stub.onTransact(IBluetoothGattCallback.java:71)
        at android.os.Binder.execTransact(Binder.java:731)
2019-03-25 18:53:37.180 24619-24679/com.remonh87 V/RxBle#Executors: Terminated (MAC='12:34:55:66:77:88')
2019-03-25 18:53:37.181 24619-24637/com.remonh87 D/MyBleClient$connectToDevice: RXBLE WHOOPS error connecting device: 12:34:55:66:77:88 error:( com.polidea.rxandroidble2.exceptions.BleDisconnectedException: Disconnected from MAC='12:34:55:66:77:88' with status 133 (GATT_ERROR))
2019-03-25 18:53:37.184 24619-24637/com.remonh87 D/MyBleClient$connectToDevice: RXBLE terminator establish conn to device: 12:34:55:66:77:88 terminated

Actual result
Operation will be terminated after approx 22 seconds
Expected result
Establishconnection should fail on the desired timeout of 6 seconds

Thank you for pointing this out.
Yup. It seems that the documentation is indeed misleading:

 * @param operationTimeout Timeout configuration after which the operation will be considered as broken. Eventually the operation
*                         will be canceled and removed from queue. Keep in mind that it will cancel the library's operation
*                         only and may leave Android's BLE stack in an inconsistent state.

It should be something like the below to match the current behaviour:

* @param operationTimeout Timeout configuration for operations scheduled using {@link RxBleConnection} emitted from this observable.
*                         Those operations will be considered broken after the specified timeout. Keep in mind that after the timeout
*                         those operations will canceled and may leave Android's BLE stack in an inconsistent state.

To have Establishconnection should fail on the desired timeout of 6 seconds one may simply use device.establishConnection(autoConnect).timeout(6, TimeUnit.SECONDS) — there is no need to bake it into the library API
What do you think?

Thanks for the quick reply:  yes II think that would match the current behavior better.
I also implemented the Rxjava timeout operator but I was hoping the timeout in RxBLE  would somehow  trigger resetting the android ble stack or the internal queue. At the moment I suffer from the fact that connecting to  multiple devices hang when one device is switched off).  I will dig further into this problem and create another ticket in case I see an issue in the library.

I will keep this opened until the Javadoc fix will be in the master branch.
As for your issue — connecting may be cancelled by the user externally (e.g. by using .timeout() operator) and the next connect may then proceed. Just keep in mind that there is an Android OS bug that may show itself then

I believe the proper link to the "Android OS bug" is https://github.com/Polidea/RxAndroidBle/issues/483
Thank you for mentioning it, by the way.

Fixed the link


Offer as webjar for simple Spring-HATEOAS (and Play & etc.) integration

Spring-HATEOAS is supporting HAL OOTB and it also has support for webjars as it's built on Spring MVC.  Wrapping up the hal-browser into a webjar would be a great way to deploy it side by side with any Spring-HATEOAS implementation.
All that would need to happen is for someone to make releases on github for each significant milestone and leave it for all eternity (a small commitment).  They don't even have to be major version numbers. 0.1.0 is fine...but they should follow semantic versioning.
I'm happy to put together the webjar package and submit it if a release is published on GH.

I second that request. A Webjar would be perfect for consumption within Java projects. However, at least creating some releases (tags?) would help downstream consumption of the browser. Currently you have to copy the files from an arbitrary commit and bundle it with your app somehow.

webjar would be good - it was my first thought after 'this all looks really great'. Cheers.

@andrekampert did the job https://github.com/Product-Foundry/hal-browser-webjar, but it seems a fork request to webjars.org is missing (https://github.com/Product-Foundry/hal-browser-webjar/issues/1)

Release request created at https://github.com/webjars/webjars/issues/946

Great, thanks guys

It's now listed in webjars at https://github.com/webjars/hal-browser so I suppose you could close this ticket.

ok :+1:

fyi, there's still some friction for webjars since this repo doesn't use revision / releases but yay!

I know. That's why I worked with @jamesward to push out a release to webjars. If this project would start doing releases, then its not hard to update the process.

You need me to start using github releases? That's no problem, really


Action Type or ActionClass not considered

The Listener is missing a check if the Action or ActionClass match the ones given in the executor.

Fixed by Release v.0.1.2.


PMINet Reload Issue

Dear authors, thanks for your amazing work!
When I was trying to reproduce the results based on this GitHub repo, I encountered an issue when reloading the pretrained PMINet weights, the warning indicated some mismatch between the pretrianed weights and the model architecture；

I'm wondering if I need to rename the keys like "ComplexesGraph.0.lin_src.weight" to "ComplexesGraph.0.lin.weight" to utilize the model weights fully.
Thanks for your attention, and looking forward to your reply.

Please refer to this issue: https://github.com/YangLing0818/IPDiff/issues/2
The authors used torch-geometric version which is 2.0.4, you can check your version and see if things go well after this operation.

Wow, solved it, awesome! Thank you very much.


jQuery.isArray is deprecated; use Array.isArray 🐛

Bug Description
While testing django-select2 with the latest version of jQuery there is a deprecation notice:
This is fixed in the latest version os select2 (4.1.0) but django-select2 is shipping with (4.0.13).
jQuery.isArray is being removed in jQuery 4.0.
Im Using: django-select2: 8.1.2
Also see: https://github.com/select2/select2/issues/6298
jquery.js:1 JQMIGRATE: Migrate is installed with logging active, version 3.4.1
jquery.js:1 jQuery Version 3.7.1 Loaded
jquery.js:1 JQMIGRATE: jQuery.isArray is deprecated; use Array.isArray <-- HERE.
a @ jquery.js:1
t.<computed> @ jquery.js:1
n._resolveLanguage @ select2.full.min.js:2
n.applyFromElement @ select2.full.min.js:2
e @ select2.full.min.js:2
d @ select2.full.min.js:2
(anonymous) @ select2.full.min.js:2
each @ jquery.js:1
each @ jquery.js:1
r.fn.select2 @ select2.full.min.js:2
initHeavy @ django_select2.js:48
(anonymous) @ django_select2.js:56
each @ jquery.js:1

        if (self.options.get('debug') && window.console && console.error) {
          // Check to make sure that the response included a `results` key.
          if (!results || !results.results || !$.isArray(results.results)) {             <--- HERE
            console.error(
              'Select2: The AJAX results did not return an array in the ' +
              '`results` key of the response.'
            );
          }
        }

Steps to Reproduce
Install jquery-migrate plugin.
Open browser console.
View warning:jQuery.isArray is being removed in jQuery 4.0.
Expected Behavior
I expect django-select2 to be prepared for the use of jQuery 4.0.0

Hi @jamesleesaunders,
Thank you for reaching out. However, I am afraid, the issue you are describing is in Select2. There is already an open issue regarding your question as well, right here: https://github.com/select2/select2/issues/6298
I am taking the liberty to close this issue. Since we don't bundle Select2 but point to the latest release, the issue will resolve itself.
Cheers!
Joe

Thanks @codingjoe understood.
The current version of django-select2 v8.1.2 is shipping with select2.js v4.0.13.
(where the latest is 4.1.0).
Understood you bundle the latest at point of release.
select2.js has already made a number of changes in preparation for jQuery4 removing deprecated jQuery functions, once https://github.com/select2/select2/issues/6298 is concluded and fully tested will you (django-select2) also be releasing a new version so that it also supports jQuery4?

I appreciate jQuery4 only came out a few days ago and is still in beta, this is just in preparation.

Thanks for providing django-select2 !!

We use Django's vendored version by default, but you can provide your own version, via the SELECT2_JS setting.
However, if you open a ticket on the Django project itself. I am happy to help you get it pushed into the next bugfix release of Django. Just post the link to the issue on code.djangoproject.com here, an I am happy to help.


Latest @hashgraph/proto causes UnhandledPromiseRejection

The latest update to @hashgraph/proto from 1.0.25 to 1.1.0 causes the SDK to throw an error when trying to execute a query/transaction.
Deleting node_modules and updating package.lock to use version 1.0.25, then running "npm ci" fixes the issue.
I'm not sure what the issue is, but on fresh installs the update to 1.1.0 currently makes the SDK unusable.
Error:
(node:36840) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'encode' of undefined
at _makePaymentTransaction (E:\PROJECTS\GITHUB\CRYPTO\hashgraph-bootstrap\node_modules@hashgraph\sdk\lib\query\Query.cjs:420:54)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async AccountBalanceQuery._beforeExecute (E:\PROJECTS\GITHUB\CRYPTO\hashgraph-bootstrap\node_modules@hashgraph\s    at async AccountBalanceQuery.execute (E:\PROJECTS\GITHUB\CRYPTO\hashgraph-bootstrap\node_modules@hashgraph\sdk\lib\Executable.cjs:217:5)
at async main (E:\PROJECTS\GITHUB\CRYPTO\hashgraph-bootstrap\main.js:42:23)
(Use node --trace-warnings ... to show where the warning was created)
(node:36840) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:36840) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I'm speechless, how was this not caught earlier? I've forgotten that we released 1.1.0 6 months ago, so all new releases were under the 1.0.X release. Either way, we'll release 1.1.1 to fix this issue.

I've released @hashgraph/proto v1.1.2 and tried using the latest SDK version within examples and it seems to work now. Can you confirm this resolves your issue @rocketmay

Beautiful. The bug appears to be fixed!
Thank you.


Cannot publish Bot to Azure

Describe the bug
We're trying to deploy out bot made with the Composer App with the publish workflow provided in the application. Our bot is also using LUIS services.
We're experiencing issues when launching the publish script and looks like it has to do with the proper LUIS resource configuration. We already did publish this specific bot in the past using the same process.
The problem began when updating to the last stable of the composer.
Version
4.7.0-preview-191208-1
Browser

[X] Chrome
[ ] Safari
[ ] Firefox
[X] Edge

OS

[ ] macOS
[X] Windows
[ ] Ubuntu

To Reproduce
Follow the publish procedure provided in the composer app.
This is the error i get.
Screenshots

@fathorMB Hi, did you meet the same error like this:

@hibrenda  This is caused by empty model error in luis build. Will be fixed in #2402

@zidaneymar Yes, that's an error we get before the lubuild succeeded message. All this happens before the 401 error.

addressed via https://github.com/microsoft/BotFramework-Composer/pull/2402


The order of keyalias commands in bbl file changes nondeterministically

This file reproduces the bug
The problem with this is, that latexmk (and possibly other tools) keeps running forever, because the bbl and bcf files keep changing.
The data of the entries shouldn't matter, though I tried first with minimal @misc entries and it didn't produce the bug. Maybe I made a mistake.
\documentclass{article}

\usepackage[backend=biber]{biblatex}
\begin{filecontents*}{main_biber.bib}
@InProceedings{Bacchus2002,
  author       = {Bacchus, Fahiem and Dalmao, Shannon and Pitassi, Toniann},
  title        = {Value elimination: Bayesian inference via backtracking search},
  booktitle    = {Proceedings of the Nineteenth conference on Uncertainty in Artificial Intelligence},
  year         = {2002},
  ids          = {bacchus2002value},
}
@InCollection{Bidyuk2003,
  author         = {Bidyuk, Bozhena and Dechter, Rina},
  title          = {Cycle-cutset sampling for Bayesian networks},
  booktitle      = {Advances in Artificial Intelligence},
  year           = {2003},
  ids            = {bidyuk2003cycle},
}
@InBook{Biere2009,
  author    = {Biere, Armin},
  title     = {Handbook of satisfiability},
  year      = {2009},
  publisher = {Ios PressInc},
  chapter   = {Model Counting},
  ids       = {Biere2009b},
}
@InProceedings{Bouchard2009,
  author       = {Bouchard, Guillaume and Zoeter, Onno},
  title        = {Split variational inference},
  booktitle    = {Proceedings of the 26\textsuperscript{th} Annual International Conference on Machine Learning},
  year         = {2009},
  ids          = {bouchard2009split},
}

@InProceedings{Boutilier1996,
  author       = {Boutilier, Craig and Friedman, Nir and Goldszmidt, Moises and Koller, Daphne},
  title        = {Context-specific independence in Bayesian networks},
  booktitle    = {Proceedings of the Twelfth international conference on Uncertainty in artificial intelligence},
  year         = {1996},
  ids          = {boutilier1996context},
}

\end{filecontents*}

\addbibresource{main_biber.bib}

\begin{document}
    \parencite{bacchus2002value}
    \parencite{bidyuk2003cycle}
    \parencite{Biere2009b}
    \parencite{bouchard2009split}
    \parencite{boutilier1996context}
    \printbibliography
\end{document}

See also  https://github.com/plk/biblatex/issues/618. The MWE does not work with BIbTeX, the ids are not resolved.  Not sure if that can be done with BibTeX.

I tried adding a sort before the hash that gets traversed at https://github.com/plk/biber/blob/a29463dbe2f565cf1986520107d885ce6cae5a4a/lib/Biber/Output/base.pm#L339.
But this didn't fix it. I don't understand what happens here:
https://github.com/plk/biber/blob/a29463dbe2f565cf1986520107d885ce6cae5a4a/lib/Biber/Output/bbl.pm#L167
I assume that the relevant code for this issue is in bbl.pl and base.pl.

I have corrected the non-determinism in the .bbl output - please try biber 2.8 DEV version (on Sourceforge). You will also need biblatex 3.8 DEV version to use this.

I haven't yet upgraded biblatex, but I'm getting this, and I think this is a Perl issue:

INFO - This is Biber 2.8 (beta)
INFO - Logfile is 'main.blg'
Use of bare << to mean <<"" is deprecated at /home/thomas/workspace/biber/lib/Biber/Output/bbl.pm line 42.
ERROR - Error loading data source package 'Biber::Output::bbl': syntax error at /home/thomas/workspace/biber/lib/Biber/Output/bbl.pm line 42, near "<<~"
Compilation failed in require at (eval 339) line 2.
INFO - ERRORS: 1

That means you are not using the pre-built biber - presumably you are using your own perl installation but biber 2.8+ requires perl 26.0+ and you seemingly don't have that (<<~ constructions are perl 26.0+).

Yes, I'm running the source from Git. Can I get a binary of the dev branch somehow?

Yes, in the development folder on Sourceforge: https://sourceforge.net/projects/biblatex-biber/files/biblatex-biber/development/

I can confirm that the order of the entries within the bbl file now stays the same with the current dev version.

I checked with my minimal example and with my project. latexmk fails to complete with the old code, and completes after three runs with the new code.

Ok, then I'll close this.

Ok, then I'll close this.


Number of extensions wrongly reported as zero on ios GLES3

2020 Ipad pro, ios 14.something
 glGetIntegerv(GL_NUM_EXTENSIONS, &num_ext);
gives no glGetError(), but num_ext is zero, (so extensions aren't initialiesd)
if I do print (char*)glGetString(7939) (GL_EXTENSIONS = 7939) in lldb, then it prints out
(char *) $2 = 0x000000010d067496 "GL_OES_standard_derivatives GL_KHR_texture_compression_astc_ldr GL_EXT_color_buffer_half_float GL_EXT_debug_label GL_EXT_debug_marker GL_EXT_pvrtc_sRGB GL_EXT_read_format_bgra GL_EXT_separate_shader_objects GL_EXT_shader_framebuffer_fetch GL_EXT_shader_texture_lod GL_EXT_shadow_samplers GL_EXT_texture_filter_anisotropic GL_APPLE_clip_distance GL_APPLE_color_buffer_packed_float GL_APPLE_copy_texture_levels GL_APPLE_rgb_422 GL_APPLE_texture_format_BGRA8888 GL_IMG_read_format GL_IMG_texture_compression_pvrtc "

Now, I can't remember if this is supposed to return null on ES 3.1...(as it's deprecated) but maybe it also suggests I haven't setup my GLKView properly (maybe I need to be explicit with level, as I do on osx's NSOpenglView)
Now, this might be a user(me) error... but maybe sokol should be falling back? (or detect this as the wrong ES version?)
I'm also trying to remember if I actually have rendered to a float before on native ios (I know it's missing from safari), but I thought I had... (maybe my old code fellback to half_float automatically)

Thanks for the bug report! I'll try to investigate tomorrow with sokol_app.h (on my iPad Mini4, I hope that's good enough). It will almost definitely work with the Metal backend, but the GLES3 on iOS might indeed be a bit "undertested".
Here is how sokol_app.h initializes GLES on iOS btw (_sapp_ios_view is a subclass of GLKView):
https://github.com/floooh/sokol/blob/1d747aaf56d73f6b27411bcdc8c270f9ba5ed83e/sokol_app.h#L3298-L3327
Unless GLES2 is forced, it first tries to create a GLES3 context, and if this fails falls back to GLES2.

Cool, I'm using the default GLKView constructor (via swift! :), I'll do some specific initialisation like I do on mac;
https://github.com/NewChromantics/PopEngine/blob/master/src/SoyOpenglViewOsx.mm#L51
and see if my defaults aren't GLES3.
And then I may implement the glGetStrings() fallback, even if it's supposed to be deprecated, just to get stuff working :)

Oh scrub that, I'm explicitly making ES3 context :)
https://github.com/NewChromantics/PopEngine/blob/master/src/SoySokol_Ios_Gles.mm#L51
(lldb) print [mOpenglContext API]
(EAGLRenderingAPI) $0 = kEAGLRenderingAPIOpenGLES3

I guess this is 3.0 not 3.1, hence the GL_EXTENSIONS behaviour.
I'll implement something to read the GL_EXTENSIONS string as a fallback :)

Update #1:
I copy+pasta the strings handling from _sg_gl_init_caps_gles2(), which works
Update #2:
glGetIntegerv(GL_NUM_EXTENSIONS, &num_ext); now returns the number of extensions...
I can't explain why (I took out my other changes since having this problem)

Ok, I've tested on my iPad mini 4 with iOS 14.4, and the native version of this sample:
https://floooh.github.io/sokol-html5/pixelformats-sapp.html
This is initialization the GLES3 code path in sokol_app.h and sokol_gfx.h, the GL extension init code in sokol_gfx.h calling glGetIntegerv(GL_NUM_EXTENSIONS, &num_ext) works, I get 19 extensions back, and after the loop checking the extension, the flags related to float-rendering are as follows:

has_colorbuffer_float = false
has_colorbuffer_half_float = true
has_texture_float_linear = false
has_float_blend = false

This means that rendering is enabled for the following float pixel formats: R16F, RG16F, RGBA16F (screenshot just shows the RGBA16F format):

I'm not sure why the call to query the number of extensions would return 0, but I notices that sometimes the order of steps on macOS or iOS can make a difference, even if the API looks like the order of function calls doesn't matter.
I'll just close the issue for now. If you stumble over any other problems, just reopen or create a new ticket :)

PS: one unrelated thing I just noticed is that the iOS MTKView (so Metal, not GL) seem to ignore the view's contentScaleFactor property (the backing buffer is always full Retina resolution). I'll see if I can come up with a fix.

 (I do recall checking the error and it returning 0 though)

Hmm ok who knows :)
While I'm at it, I'm doing some more fixes for iOS and Metal, but only in sokol_app.h, I guess that's not relevant for you :)
(GLKView multisampling wasn't supported, I guess that was broken in an older iOS version and works now, and on Metal, the high_dpi selection didn't work anymore because MTKView seems to ignore the UIView's contentScaleFactor now).

Cool, yeah I'm all opengl at the moment.
We're into chat territory now, but this is my current bodge in my fork
SG_MAX_UB_MEMBERS = 1024,
I guess the aim is to use multiple uniform buffers,(I'm just writing to one atm) which is fine, but still not exposing the max uniforms... is this a bit of a todo, or a general cross platform limitation(16*4) I'm not aware of?

Ouch, hmm... SG_MAX_UB_MEMBERS is a "random" value for the capacity of varous nested arrays in C structs. Setting this to a big value will make your sg_shader_desc structs awfully big (see the sg_shader_uniform_block_desc struct).
Can you use arrays? Those count as one uniform block member. But note that there are various "unfixable" alignment issues, but float arrays and vec4 arrays should be save (while vec3 arrays are currently broken, and vec2 arrays might be broken).
For instance the sokol_shdc shader compiler flattens all GL uniforms in a shader stage into a single vec4 array, and that works just fine.

PS: you still might run into a "max number of vec4 uniforms" even with a single vec4 array, but I think at least on iOS with GLES3 those are now "big enough" (on old GLES2 iOS devices it was just a bit over 100 vec4's if I remember right).
If you need to put much more uniform data into vertex shaders, the currently best cross-platform option is to use an RGBA32F texture and sample that in the vertex shader (unfortunately that's the only cross-platform option if WebGL1 needs to be supported).

It might be big, but it works ;)
Yeah, I know about alignment of shaders etc, but I typically cram it all together when optimising, after I have things working :) (I only need about 30 atm)
Getting bool's & int's working is also on my todo list.
I'm loading shaders on the fly across all platforms, so maybe the shader compiler won't quite be appropriate, but that's a topic for later. (I already have some hacky "refactoring" in my shaders I need to get rid of)


Starscream.TCPTransportError.invalidRequest

I am using server-client based socket here. iPad will act as server and other payment device will be client in my app.
In that I am using below request.
var request = URLRequest(url: URL(string: "<IP_ADDRESS>:8030")!)//https://localhost:8080
request.timeoutInterval = 5
But I am getting Starscream.TCPTransportError.invalidRequest in didReceive method.
Can you please please help me

Sorry, I can help without more context. I'd recommend asking the question on StackOverflow. Also make sure you are using a web socket server on your iPad and not just a generic HTTP server. Best of luck!

Thanks daltoniam, How do we decided that we are using web socket server on iPad not just a generic HTTP server. Can you please help me to understand this.


Fix reStructuredText docstring warnings

This PR fixes most warnings and syntax mistakes I got when trying to generate the docs with Sphinx. There are still some things that don't generate correctly, like the math tex in some of the filters.
If you like this I can submit a new PR with a sphinx generator, which you can use to  host your docs on GitHub Pages or similar.
Anyway, here are the docs I've generated. They are a little rough around the edges (that's mostly due to the template I've used. This theme)

Download equideepdmri_docs.zip

Thanks a lot for your contribution. I'd appreciate a PR with a sphinx generator


Press Enter to Continue?? - When Installing OXAR on CentOS 7.5 on Digital Ocean

This time I had to press Enter 3 times:
This is just weird, cause it was not the case before... Possibly a quirk with the latest CentOS Bash?
Disconnected from Oracle Database 11g Express Edition Release <IP_ADDRESS>.0 - 64bit Production
Oracle installed. Press enter to continue...

* Installing APEX *

Disconnected from Oracle Database 11g Express Edition Release <IP_ADDRESS>.0 - 64bit Production
APEX installed. Press enter to continue...

* Installing node-oracledb *

INFO: Modified: /etc/ords/defaults.xml, setting: apex.excel2collection.useSheetName = true
ORDS installed. Press enter to continue...

This was added in some time ago by Martin.
e.g. https://github.com/OraOpenSource/OXAR/blob/master/build.sh#L116-L118
and added from this issue https://github.com/OraOpenSource/OXAR/issues/166
Doing a quick inspection, it looks like it toggles that on when you start the script with the debug flag (-d|--debug).

@tschf Oops I forgot I had edited and enabled the debug testing for item #254 and never put it back with the defaults of FALSE.
Sorry for the confusion, I think this item can be closed.