Error related to Sourceror when starting up

Hi,
This tool looks so awesome! I'm really excited to try it out, but I'm seeing this error in the VSCode extension output logs:
[Error - 2:28:45 PM] Server not started: Error: Command failed: export MIX_ENV=prod && cd /Users/oliverswitzer/.cursor/extensions/gp-pereira.refatorex-0.1.24/refactorex && elixir --sname undefined -S mix run --no-halt -e "Application.ensure_all_started(:refactorex)" -- --port 51809
     error: Sourceror.TraversalState.__struct__/1 is undefined, cannot expand struct Sourceror.TraversalState. Make sure the struct name is correct. If the struct name exists and is correct but it still cannot be found, you likely have cyclic module usage in your code
     \x{2502}
 261 \x{2502}     {quoted, %{acc: acc}} = Macro.traverse(quoted, %TraversalState{acc: acc}, &{&1, &2}, fun)
     \x{2502}                                                    ^
     \x{2502}
     \x{2514}\x{2500} lib/sourceror.ex:261:52: Sourceror.postwalk/3

== Compilation error in file lib/sourceror.ex ==
** (CompileError) lib/sourceror.ex: cannot compile module Sourceror (errors have been logged)

So far I have tried running mix deps.get in the ~/.cursor/extensions/gp-pereira.refatorex-0.1.24/refactorex path
I have also tried running mix deps.compile sourceror --force there, and got the same error output as the one I get when I re-open Cursor and RefactorEx starts up:
❯ mix deps.compile sourceror --force
==> sourceror
Compiling 6 files (.ex)
     error: Sourceror.TraversalState.__struct__/1 is undefined, cannot expand struct Sourceror.TraversalState. Make sure the struct name is correct. If the struct name exists and is correct but it still cannot be found, you likely have cyclic module usage in your code
     │
 261 │     {quoted, %{acc: acc}} = Macro.traverse(quoted, %TraversalState{acc: acc}, &{&1, &2}, fun)
     │                                                    ^
     │
     └─ lib/sourceror.ex:261:52: Sourceror.postwalk/3

== Compilation error in file lib/sourceror.ex ==
** (CompileError) lib/sourceror.ex: cannot compile module Sourceror (errors have been logged)

Any help debugging this would be greatly appreciated! Thanks

Hi there. Can you send me your Elixir/OTP version pls?


How to run "composer install"?

Hello,
I have these rules:
- name: Run composer
  uses: nick-zh/composer-php@master
  with:
    action: 'install'

But it fails with this error:
Run nick-zh/composer-php@master
/usr/bin/docker run --name c27d3162fb786884fd45869746552576b0b052_334f10 --label c27d31 --workdir /github/workspace --rm -e INPUT_ACTION -e INPUT_OPTIONS -e HOME -e GITHUB_JOB -e GITHUB_REF -e GITHUB_SHA -e GITHUB_REPOSITORY -e GITHUB_REPOSITORY_OWNER -e GITHUB_RUN_ID -e GITHUB_RUN_NUMBER -e GITHUB_ACTOR -e GITHUB_WORKFLOW -e GITHUB_HEAD_REF -e GITHUB_BASE_REF -e GITHUB_EVENT_NAME -e GITHUB_WORKSPACE -e GITHUB_ACTION -e GITHUB_EVENT_PATH -e RUNNER_OS -e RUNNER_TOOL_CACHE -e RUNNER_TEMP -e RUNNER_WORKSPACE -e ACTIONS_RUNTIME_URL -e ACTIONS_RUNTIME_TOKEN -e ACTIONS_CACHE_URL -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/WooCommerce-Correios-Calculo-de-Frete-na-Pagina-do-Produto/WooCommerce-Correios-Calculo-de-Frete-na-Pagina-do-Produto":"/github/workspace" c27d31:62fb786884fd45869746552576b0b052  "install" ""
Invalid argument . Use "composer require " instead to add packages to your composer.json.

How can I run composer install with this action?
Thank you!

@Luc45 i assume it has a problem that option is missing, i'll check if this can be improved. I know there is also a problem if you add to options to the options parameter.

@Luc45 should be ok now, let me know if you need more helpt :v:


improvements for ssh, allow hostnames other than "crowbar" for admin

Î have a few more, am just grouping them a bit "by subject" to not drop in big batches at a time :-)


Investigate if Crossplane RBAC-Manager lets us use less boilerplate in compositions

What: It was suggested Crossplane RBAC-Manager may let us use less boiler-plate in our compositions. However, it must still let us meet our multi-tenancy requirements so it needs investigating
Why: This may help us reduce the 70/80 lines of RBAC boilerplate code in each of our compositions

[ ] Understand RBAC-Manager @ https://github.com/crossplane/crossplane/blob/master/design/design-doc-rbac-manager.md
[ ] Attempt to use it with a composition
[ ] Confirm/Deny that it still allows teams to only see their own resources at namespace level and cluster level like our current implementation
[ ] Create new issues to modify existing compositions if suitable or close this issue and take no action

Quote from Goals section:
"Platform consumers access to interact with the set of composite resource claims their platform operator(s) deem appropriate."
This doesn't state anything regarding platform consumer access seperation, i.e. this solution is created with multi-tenancy in mind at all.

In agreement with @rifisdfds the RBAC-manager doesn't meet our requirement.


Sentry: add kubernetes strong identities

This PR adds strong identity validation between the sidecar and Sentry using Kubernetes service account tokens.
Closes #1004

Looking at the code, it is hard to validate the scenario unless we hack the sidecar to change the environment variable unless my understanding is correct

Looking at the code, it is hard to validate the scenario unless we hack the sidecar to change the environment variable unless my understanding is correct

I would want to have a test that fakes the sidecar anyway and find a way for us to to manual injection testing, but for this PR I just pushed tests with fake k8s clients.

/ok-to-test

@youngbupark seems like the cluster was out of resources for this test run

/ok-to-test

/ok-to-test

@yaron2 this will run e2e test in the other cluster (maybe 2 or 4)

/ok-to-test

/ok-to-test

If this test run doesn't pass I'll buy @Haishi2016 pizza.


Take Notes Feature

ISSUE #152

Describe the changes you've made
Added a floating action button to take notes while attending lectures.

Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)
NA

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

Provide a Deployed link of route/page that needs to review
Preview: https://deploy-preview-267--festive-allen-a7f545.netlify.app/mycourses/:coursename

@vasudev19126 @Abhishek-kumar09  Please review

When I click on note add button the backtoTop button also get pops out.

When I click on note add button the backtoTop button also get pops out.
image

I'm not sure why is this issue happening, for me the backtotop button only shows up when scroll down. I'm trying to resolve this issue. If you could give a hint on how to resolve this?

Remove the back to Top button completely, it's not needed there I guess

The User experience will not be good here since it is a very small text area, where people could barely write visible notes of 3 4 sentences.
How is the idea of using a Modal sheet: https://www.npmjs.com/package/react-modal-sheet

The User experience will not be good here since it is a very small text area, where people could barely write visible notes of 3 4 sentences.
How is the idea of using a Modal sheet: https://www.npmjs.com/package/react-modal-sheet

yes, right. Sure I'll use this package and implement this.

Please review now @Abhishek-kumar09 . I have updated with react bottom modal

Looks cool.
That green success snackbar was looking very nice in the previous UI. Can we use it here?

yes, sure. I'll add that again.

Done, @Abhishek-kumar09

Thanks @sakshi-choudhary


Custom fonts for pdf and projector

@normanjaeckel Please review again.

@normanjaeckel Hi, Im stuck at assigning the new permission to all necessary groups. See this part of the traceback:
File "/home/finn/OpenSlides/OpenSlides/openslides/core/migrations/0007_auto_20180130_1400.py", line 29, in delete_old_logo_permission

  File "/home/finn/OpenSlides/.venv/lib/python3.5/site-packages/django/db/models/fields/related_descriptors.py", line 881, in add
    self._add_items(self.source_field_name, self.target_field_name, *objs)
  File "/home/finn/OpenSlides/.venv/lib/python3.5/site-packages/django/db/models/fields/related_descriptors.py", line 1018, in _add_items
    (self.model._meta.object_name, obj)
TypeError: 'Permission' instance expected, got <Permission: core | config store | Can manage logos and fonts>

I don't know, why I cannot add this permission. The debugger says that type(perm) -> <class 'django.contrib.auth.models.Permission'>, which is correct. I've tried two ways: First (as propsed) let Django create all permissions and get the new one to assign. The second way is just create the neede permission.
Both ways lead to this error above.
To test this: Create a db in the current master (and maybe save it that you can just do the migration several times). Switch to this branch and migrate.

Sorry I do not have any time now to test this manually.

Successfully retested - with db migration.


Releasing the mouse back to the UE

Thanks for all the work put into this project. I couldn't find an example of releasing the mouse back to UE. I noticed that once BLUI has it, it keeps it. Before I dig around further I thought I would ask if I am missing something simple. For example after pressing shift-F1 and interacting with BLUI display.
Thanks

Here you can find a demo that uses the mouse (http://bit.ly/1ODRiTO). It requires UE4.7.


Editable redirects no longer work

@Siilwyn and I encountered editable redirects no longer work. The cause is unknown.
This could have been caused by:

the upgrade to the new Astro + Cloudflare adapter
the introduction of using Functions within Cloudflare Pages as the Cloudflare docs on Redirects states you need to put that in your own worker logic:

Redirects defined in the _redirects file are not applied to requests served by Functions, even if the Function route matches the URL pattern. If your Pages application uses Functions, you must migrate any behaviors from the _redirects file to the code in the appropriate /functions route, or exclude the route from Functions.

Cloudflare _redirects are based on Netlify _redirects. So we could stick with this format for rewrites and use the netlify-redirect-parser in a catch all route or middleware.

Maybe better to switch to our own (meaning not Cloudlare/Netlify), more explainable syntax and use a better documented package like https://www.npmjs.com/package/tiny-request-router


add instructions for okta authentication

Address issue: https://github.com/rancher/docs/issues/1027

LGTM

LGTM

@fyery-chen  Please don't forcepush over this branch. I have updated this PR.


Files with \r\n can be incorrectly tokenized

Hi,
I just noticed that this file is tokenized incorrectly, and I have a strong suspicion that it's because it has \r\n end lines. In particular, it has some inside comments.
9d694731f6291f4553fad6b8174e7c915e6219ec_TigerBNB.sol
If you compile it with solc the pragma solidity ^0.6.12; statement is correctly parsed. In solang it's part of a comment.

@seanyoung, I found more files that break like this. In case they are helpful, I'm posting them here:
54bef7b75e1bec4594ecbc8caad54328d653f5c1_Spark.sol.txt
71d0e03abd2b4b4fb9ef73d4c6f35a650124791e_BabyRock.sol.txt
85d1f31777c5c9ca544a1a73b83341872bdca45e_PUSHNOU.sol.txt
97f4706e90cbd5d882088a0a245c63cb6b5f27cf_Vitalic.sol.txt
719de440d870b18de38782ca191ed6a651a2e67a_TotoToken.sol.txt
909b4e9a086ba9feb85b3244fd83b84cf8b2ab83_Saddam.sol.txt
914a70c59340d18305a6b65f4049d18731e82fec_CodeWithJD.sol.txt
38606c5e2e7805dd763f1247bb71c08669a8b4f3_DOGEREADSFLOOR.sol.txt
69001d581C1BC73049e10a0574788902E10dED2C_SmallTiger.sol.txt
81396ce02b575382bc969ed970b8b658f0c7e3d0_Spark.sol.txt
086109f8ea01ac3D41a18405c212D555De0e5cC5_MetaverseGame.sol.txt
855322f5aeabd16ec1d5ef255504062661edc157_Spark.sol.txt
aa8a84097c721e1de0ac3004fd08d4d7b0c6f586_Spark.sol.txt
ac7f486b31098b87e67be5c998acec7e28a1b734_Address.sol.txt
b0DA2a6fCB1ff8850118948508E64De9EAd10b9d_RocketLink.sol.txt
b44f93a99b7615937e2515c4613ad26760617991_Spark.sol.txt
cd70fa5fba9f0996c83e0e95cabc1cad5edc205a_Spark.sol.txt
Db5Cede38Af7225562a4D2EE464c605a6601AC22_Web3T.sol.txt
e29d2560d861ec0e372c064f2aa00bf5ddd82ef3_HelloWorld123.sol.txt
e813091ac3cf86a2334d8f57d94c0598bf97a23a_TokenLauncher.sol.txt
eFEa63672F9942f9F793eb07BeB3b37520ef559e_SpongeBOB.sol.txt
f8097bdC3a4Bfd8a900F0e802D17dA95cD4B0e1A_Token.sol.txt
5fBd2bD682B1e490E79A75e5c2EdF0F5016b62d1_FirstETH.sol.txt
006f045e0f877a321474692839d897945fa62b45_BABYLINK.sol.txt
7Ea1A16c2B44BF2a7187F29D95f5ad3E03dF2A6A_MetaWorldV2.sol.txt
8a61E6D97cF885C94960298BBa374c6AE3D6555B_FirstMoon.sol.txt
8e28666cd07bcb39dfb939ff4e47014602c64cec_Spark.sol.txt
9c2e8bf7cf0dfe45e0ce2d4e53b2a398e5feac8c_BabyCHINA.sol.txt
9d694731f6291f4553fad6b8174e7c915e6219ec_TigerBNB.sol.txt
15bfe7fc335e9a5f5f2c519c242ffb6f4317f97f_SHIBTiger.sol.txt
21a58b28224c6191Ef1ed38A0CAb85D444E8E665_BNBGirl.sol.txt

@seanyoung, I found more files that break like this. In case they are helpful, I'm posting them here:

Great, thank you very much! I'll go through them and make sure they parse.

btw, I'm lexing the whole smart-contract-sanctuary repository and comparing the results with those of @solidity-parser/parser. It should be done by tomorrow. I'll let you know if I find more issues.

As promised, here are all the files from the sanctuary that caused errors:
failures.zip

As promised, here are all the files from the sanctuary that caused errors:
failures.zip

Thank you!

@seanyoung did #745 fix this?

This issue is not just what's in the description, it's all the files in the zip file. They should be re-tested

Ok, I'm not closing this then. Thanks!


[Bug] [k8s session] param set failed on flink configuraion

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
version: 1.2.0-rc5, when i set a param in cluster param management page,start a k8s session cluster,i can not find the apram in /opt/flink/conf/flink-conf.yaml file。
What you expected to happen
i can find the apram in /opt/flink/conf/flink-conf.yaml file。
How to reproduce
flink 1.18.1,dinky 1.2.0-rc5,k8s cluster,create a cluster conf in clusterParamManager page,

like this,then i can not find the param in /opt/flink/conf/flink-conf.yaml,

Anything else
No response
Version
1.2.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

启动是指 复用这个集群配置启动一个 session 集群, 不会变更你的本地的配置文件

When starting cluster instances through cluster configuration, parameters will not be written to the flink-conf.yaml file.


Show OneDrive sync status

It'd be helpful if Files UWP shows sync status for OneDrive. These are the different possible sync status:

Available when online
Available on this device
Syncing...
Can't sync
Not synced (excluded)

The technology behind OneDrive sync client is also used by iCloud for Windows.


Feature: ability to generate new Processing Libraries

Issue to discuss the development of the generating new Processing Libraries command (directly in Sublime Text w/o any Eclipse involvement). Related to https://github.com/b-g/processing-sublime/issues/61

I guess the --build command does that.


Build backend: Add fast path

Going through PEP 517 to build a package is slow, so when building a package with the uv build backend, we can call into the uv build backend directly. This is the basis for the uv build --list.
This does not enable the fast path for general source dependencies.
There is a possible difference in execution if the latest uv version is newer than the one currently running: The PEP 517 path would use the latest version, while the fast path uses the current version.
Please review commit-by-commit
Benchmark
built_with_uv, using the fast path:
$ hyperfine "~/projects/uv/target/profiling/uv build"
Time (mean ± σ):       9.2 ms ±   1.1 ms    [User: 4.6 ms, System: 4.6 ms]
Range (min … max):     6.4 ms …  12.7 ms    290 runs

hatcling_editable, with hatchling being optimized for fast startup times:
$ hyperfine "~/projects/uv/target/profiling/uv build"
Time (mean ± σ):     270.5 ms ±  18.4 ms    [User: 230.8 ms, System: 44.5 ms]
Range (min … max):   250.7 ms … 298.4 ms    10 runs

This is sick!

Sure:
For all the python build backend, we pay a cost for each import (see also https://peps.python.org/pep-0690/). A build backend is using a lot of features, so it gets slowed down by a lot of imports, even something manually optimized to have lazy imports such as hatchling.
For uv-against-uv, we paying the cost for setting up the build env, syncing it, spawning python, the spawning uv from python. We do this for each hook we call (should be build_sdist and build_wheel here).
Here's a uv-against-uv benchmark:
UV_PREVIEW=1 hyperfine "../../../target/profiling/uv build --find-links ../../../target/wheels/ --preview" "UV_PREVIEW=1 ../../../target/profiling/uv build --find-links ../../../target/wheels/ --preview --force-pep517"
Benchmark 1: ../../../target/profiling/uv build --find-links ../../../target/wheels/ --preview
  Time (mean ± σ):       9.7 ms ±   1.7 ms    [User: 4.7 ms, System: 4.9 ms]
  Range (min … max):     6.5 ms …  21.7 ms    265 runs

Benchmark 2: UV_PREVIEW=1 ../../../target/profiling/uv build --find-links ../../../target/wheels/ --preview --force-pep517
  Time (mean ± σ):      87.7 ms ±   4.2 ms    [User: 61.0 ms, System: 28.4 ms]
  Range (min … max):    79.6 ms …  98.5 ms    33 runs

Summary
  ../../../target/profiling/uv build --find-links ../../../target/wheels/ --preview ran
    9.08 ± 1.61 times faster than UV_PREVIEW=1 ../../../target/profiling/uv build --find-links ../../../target/wheels/ --preview --force-pep517

To illustrate the overhead from calling python, this is the cost for some std modules you'll want for building distributions:
$ hyperfine "python -c ''"
Benchmark 1: python -c ''
  Time (mean ± σ):       7.9 ms ±   1.2 ms    [User: 5.9 ms, System: 2.0 ms]
  Range (min … max):     6.2 ms …  10.4 ms    383 runs
$ hyperfine "python -c 'import sys'"
Benchmark 1: python -c 'import sys'
  Time (mean ± σ):       8.1 ms ±   1.2 ms    [User: 5.9 ms, System: 2.1 ms]
  Range (min … max):     6.3 ms …  10.7 ms    342 runs
$ hyperfine "python -c 'import sys, os, subprocess, pathlib, zipfile'"
Benchmark 1: python -c 'import sys, os, subprocess, pathlib, zipfile'
  Time (mean ± σ):      17.5 ms ±   2.1 ms    [User: 14.3 ms, System: 3.1 ms]
  Range (min … max):    15.1 ms …  24.6 ms    170 runs

The uv build backend imports sys and subprocess lazily, and has to spawn uv, so we can look at their isolated overhead:
$ UV_PREVIEW=1 hyperfine "python -c 'import sys, subprocess; from uv import build_wheel'"
Benchmark 1: python -c 'import sys, subprocess; from uv import build_wheel'
  Time (mean ± σ):      14.7 ms ±   1.8 ms    [User: 11.8 ms, System: 2.9 ms]
  Range (min … max):    12.4 ms …  20.2 ms    219 runs
$ hyperfine -N "uv build-backend --version"
Benchmark 1: uv build-backend --version
  Time (mean ± σ):       2.4 ms ±   0.3 ms    [User: 0.9 ms, System: 1.5 ms]
  Range (min … max):     1.6 ms …   3.1 ms    968 runs

Wow. So cool.


Highlight stub files with php syntax highlighter

refs. https://github.com/phpstan/phpstan-src/pull/181

@ondrejmirtes any chance to merge this ?


Gauge quality of search results in automated tests

We need a way to empirically evaluate the quality of our search results as we make changes to our Elasticsearch queries.
This entails compiling some sample search data and ensuring that the right documents float to the top.

Duplicate of #219.


Update ubuntu 20.04 kernel headers to 5.15{uname -r}

Tool name
/usr/include
Tool license
GPL
Add or update?

[ ] Add
[X] Update

Desired version
5.15
Approximate size
No response
Brief description of tool
No response
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[ ] Azure DevOps
[ ] GitHub Actions

Runner images where you need the tool

[x] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

It is ducumented Kernel Version: 5.15.0-1074-azure is used.
But run githubaction it shows, which is Less than 5.8
sudo cat /usr/include/linux/version.h
#define LINUX_VERSION_CODE 328959
#define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << [8](https://github.com/kmesh-net/kmesh/actions/runs/11926109999/job/33239358208?pr=1056#step:4:9)) + ((c) > 255 ? 255 : (c)))

Hi @hzxuzhonghu - Thank you for bringing this issue to our attention. We will look into this issue and will update you after investigating.


SerializationTraits: Remove unnecessary 2nd template parameter

Internal reference b/67717285

@vjpai I'll let you write it :P

I was just about to approve

I hope the meeting for that 5-year-old calendar invite went something like this:


rvintrin.h file name

This is a very general name.  Other extensions will also want intrinsics files.  Perhaps B extension intrinsics should be in a file with a name like rvb-intrin.h to make it clear that these are RISC-V B extension intrinsics.
If we are putting all intrinsics in one file, then we may need to conditionalize them based on whether that particular extension is enabled.  That requires a macro as per issue #28.

Yes, the idea is put in all intrinsics in one file, and conditionalize them based on enabled extensions, similar to x86intrin.h.

The x86 gcc port has about 80 intrinsics files, though it does look like x86intrin.h directly or indirectly includes most of them.  So the scheme there appears to be that each major extension has its own intrinsics file, and groups of related extensions have a combined header file, and then the top level x86intrin.h includes each major group.
We can always split apart the rvintrin.h file later if it gets too big to maintain.
ARM has only 5 intrinsics files, and doesn't have a master file that includes them all.

The x86 gcc port has about 80 intrinsics files [..]

Note that this is just an implementation detail. The user should always include <x86intin.h>, never any of the other intrinsics header. All the other intrinsics headers have an ifdef guard like the following:
#ifndef _X86INTRIN_H_INCLUDED
# error "Never use <ia32intrin.h> directly; include <x86intrin.h> instead."
#endif

I see this file in the elf path for toolchain but not in the linux toolchain path?
I need this for cross-compiling some bitcoin code.

I see it in my linux toolchain build.  In the install tree
rohan:2331$ find . -type f -name rvintrin.h
./lib/gcc/riscv64-unknown-linux-gnu/10.0.0/include/rvintrin.h
gcc finds it fine when I try a simple testcase.
rohan:2344$ cat tmp.c
#include <rvintrin.h>
rohan:2345$ build-install/bin/riscv64-unknown-linux-gnu-gcc -S tmp.c
rohan:2346$


Weird scrollbar behavior

I'm at the end of the table, and the scrollbar is not positionned correctly at the end (on Chome + IE11)

Code is pretty simple:
const StickerHeaderTable = ({ headers, rows }) => (

    <StickyTable stickyColumnCount={0} stickyHeaderCount={1}>
      <TableRow>
        {headers.map((header, i) => (
          <Cell key={i}>{header}</Cell>
        ))}
      </TableRow>
      {rows.map((row, i) => (
        <TableRow key={i}>
          {row.map((cell, j) => (
            <Cell key={j}>{cell}</Cell>
          ))}
        </TableRow>
      ))}
    </StickyTable>

);

 <StickerHeaderTable
        headers={[

column A
,

column B
,

column C
,
        ]}
        rows={[
          ['first', '1 b', '1 c'],
          ['2 a', '2 b', '2 c'],
          ['3 a', '3 b', '3 c'],
          ['4 a', '4 b', '4 c'],
          ['5 a', '5 b', '5 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['6 a', '6 b', '6 c'],
          ['last', '6 b', '6 c'],
        ]}
      />

No went with a custom solution instead which is not perfect but good enough for my usecase and not requiring any JS:
https://stackoverflow.com/questions/52000363/flexbox-row-with-inline-flex-fixed-item-width-fixing-flex-basis-bug-not-wo

#46 is same probably

@slorber @dominikdosoudil thanks for letting me know. They are the same issue. I think it's relatively straightforward to fix, I just haven't had the time. Happy to accept a correct PR. Hopefully I can get to it soon

@slorber I've made a pull request, would be great if you test if it works also in your project before the PR is checked by @maxhudson. Could you do it or you are already using something absolutely different?
To install sticky-table from fork:
npm install --save dominikdosoudil/react-sticky-table

sorry I've moved to a custom CSS only solution that works for me, I'm not using this lib anymore

What about the PR?

resolved in v3.0.1


[ci][fix] Fix cuda_exp ci

Unfortunately, in a very long time (since the merge of #4630), the ci tests for cuda_exp has not actually run.
Since the task name for both cuda and cuda_exp is cuda
https://github.com/microsoft/LightGBM/blob/702db13b01bfb090c3f8f3455790a498f105b28f/.github/workflows/cuda.yml#L14
in test.sh the cuda_exp option cannot be detected, and it will always run for device cuda!
https://github.com/microsoft/LightGBM/blob/702db13b01bfb090c3f8f3455790a498f105b28f/.ci/test.sh#L198-L199
That's a serious mistake. And I think we need to fix it right now for our on-going development of cuda_exp.

See https://github.com/microsoft/LightGBM/pull/5425#issuecomment-1225352237

I've added the label maintenance to this PR.
As I mentioned in https://github.com/microsoft/LightGBM/pull/5403#issuecomment-1205602981 and https://github.com/microsoft/LightGBM/pull/5413#issuecomment-1212065345, please add a labels when you create pull requests here, so your PR will be correctly categorized in the release notes.

I think doing the following would prevent adding new complexity to .ci/test.sh:

Thanks for the suggestion. Already applied via 0206da8 and eb34dd5.

We should never merge a PR that will result in CI being broken on master, so please do one of these:

push a fix for that issue to this PR
remove the cuda_exp CI jobs in this PR, document the work to add them back in a separate issue, merge this PR, and then push a follow-up PR fixing cuda_exp and adding back the CI jobs

I'd prefer the second choice, and already pushed the fixes via ca7df38.

As I mentioned in https://github.com/microsoft/LightGBM/pull/5403#issuecomment-1205602981 and https://github.com/microsoft/LightGBM/pull/5413#issuecomment-1212065345, please add a labels when you create pull requests here, so your PR will be correctly categorized in the release notes.

Thanks again for the reminder.

This is ready. @jameslamb @guolinke Could you please review this again. If it looks good to you, let's merge this so that we can work on related PRs. Thanks.

/gha run r-valgrind

/gha run r-solaris

@shiyu1994 Solaris support was removed in #5226. The r-solaris check no longer exists, and is no longer necessary.
But thank you for being so thorough!

@guolinke @jameslamb @StrikerRUS Thanks for reviewing this. I'll merge this since all tests have been passed and it is blocking many related PRs including #4827, #5425 and #4266.


Disable Kiosk mode

Hi, i followed the instructions in the release note and they worked perfectly. How do i disable the kiosk mode now though? Ideally at boot so that i can login to the user account if needed

You are asking for a feature that belongs to FydeOS Enterprise Upgrade. With this release of Chromium OS for Raspberry Pi, you can only undo what you did with the chrome_dev.conf and system-services.override to manually disable "forced" kiosk mode.


[flash_ctrl] Initial flash galois multiply

Initial draft of high storage overhead Galois multiplier

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

@sjgitty
Do you mind having a look? I was originally going to make this a prim, but the flash xex usage is a bit different from a generic use case.
Since the key is known ahead of time, I'm basically treating the matrix expansion as an initialization step and expand it all out.  So later during the multiply, only the select / xor-ing is really done.
The storage cost of this solution is really high...so I might only store every 4 or 8 column in the end, and expand between them on the fly.  This will of course create more logic, so I'll probably have to juggle a bit and see which one makes sense.

@sjgitty
Do you mind having a look? I was originally going to make this a prim, but the flash xex usage is a bit different from a generic use case.

I checked it out, but couldn't find the gfmul usage, that will help me get the bigger
picture. I might just be misreading it too.

I should probably also try the straightforward thing and just break this up into N stages. I'll give that a shot too.

alright, both draft implementations are in. You can see both in flash_xex_mult.
One is doing what i described earlier, the other is just breaking an operation into parameterizable stages.
I'll get some synthesis results the first chance I get this weekend.  Although i'm fairly sure the higher latency one will come in WAY smaller than the other... I'll try this at 32/64b to get a sense of how things scale.

hey all,
can i ask you got have a quick look at flash_gfmult.sv? Ignore everything else in this PR, most of it was done to try out an alternative implementation (which is way too big) and to do some quick and dirty dv.
Can you let me know how the module generally looks to you? It's mean to allow us to break down the operation into arbitrary stages (powers of 2 and divisible).  I've tested it with pyfinite at 32b, so I think it is at least somewhat functional.
If it seems reasonable, i can convert it to a prim, but wanted to make sure it made sense to you guys to do so as well (especially if @vogelpi can re-use it. If not, i would probably keep it in flash).

O wait i didn't see the 0 at the end of your $onehot0.  Let me try that.
On Wed, Mar 18, 2020 at 10:17 AM Eunchan Kim<EMAIL_ADDRESS>wrote:

@eunchan commented on this pull request.
In hw/ip/flash_ctrl/rtl/flash_gen_matrix.sv
https://github.com/lowRISC/opentitan/pull/1758#discussion_r394512242:

 end

 StDone: begin

   done_o = 1'b1;

 end

 default:;

endcase // unique case (st)
end

//////////////////////////////////////////////
// Assertions, Assumptions, and Coverpoints //
//////////////////////////////////////////////

`ASSERT(cntEnClrExclusive, (cnt_en | cnt_clr) |-> $onehot( {cnt_en, cnt_clr}))

Wait. do you mean ASSERT(CntEnClrExcllusive_A, $onehot0({cnt_en,
cnt_clr})) immediately failed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/1758#discussion_r394512242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXTNAP5CBBTAI3XDK3RID6ZJANCNFSM4LHX3LRQ
.

@vogelpi
btw, don't feel like you have to use my crappy module.  If you have something better planned in works, feel free to go a different route.

Thanks @tjaychen for doing this implementation, it's great that you are doing this in a parametrized way!
As you know, I also had this on my list for AES-GCM but did manage to start that yet. I think this iterative implementation would be a good match for my use case (I will probably use a Width of 128 and maybe do 8 cycles, i.e., 16 bits at a time). I think it would be a good idea to have this as a prim.
I think it would be useful to also cite a paper, have a reference where the architecture is from. I would like to have a second look at the actual gf_mult() and gen_matrix() with that reference at hand.
As a side note, the paper by Leilei et. al "Efficient Finite Field Serial/Parallel Multiplication" presents some more alternative architectures. I think it is worth to have a look.

yep, i'll add my sources in an update.

@vogelpi
i updated per your comments (especially around the naming), so hopefully it's less confusing now.
I decided not to split the sequential / combinational elements since it feels pretty simple to me at the moment.  My personal rule of thumb is that when i start having nested conditions, or more than 4 or 5 conditions, I start splitting up.  I checked our style guide just in case, and it seems to say if it's too complicated, it's more of a personal preference.  Hope this is okay with you.

yeah but i worry if we do that, it will multicycle from the input all the
way to the flop.
Whereas what we're really trying to say is that the matrix calculation is
multicycle, whereas the last flop calculation has to be 1.
Does that make sense?
On Fri, Mar 20, 2020 at 9:26 AM Eunchan Kim<EMAIL_ADDRESS>wrote:

@eunchan commented on this pull request.
In hw/ip/flash_ctrl/rtl/flash_gen_matrix.sv
https://github.com/lowRISC/opentitan/pull/1758#discussion_r395751247:

always_ff @(posedge clk_i or negedge rst_ni) begin

if (!rst_ni) begin

 cnt <= 'h0;

end else if (cnt_clr) begin

 cnt <= 'h0;

end else if (cnt_en) begin

 cnt <= cnt + 1'b1;

end
end

assign cnt_prev = cnt == 0 ? '0 : cnt - 1'b1;
assign prev_matrix_entry = matrix_o[cnt_prev];
assign next_matrix_entry = {prev_matrix_entry[0 +: Width-1], 1'b0} ^

                        (IPoly & {Width{prev_matrix_entry[Width-1]}});

always_ff @(posedge clk_i) begin

We could define it in SDC set_multi_path (forgot if it is
set_multiclock_path or set_multi_path or similar). As this output is just
comb output, SDC should find the terminal registers first then use the list
to define multipath.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/1758#discussion_r395751247,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSXMTC2HOKTGOTP6CJLRIOKNBANCNFSM4LHX3LRQ
.

yeah but i worry if we do that, it will multicycle from the input all the way to the flop. Whereas what we're really trying to say is that the matrix calculation is multicycle, whereas the last flop calculation has to be 1. Does that make sense?

I see. I thought that it can be defined as set_multi_path -from {signame} -to {FF.d}.

yeah that makes sense. So i guess we would maybe put a buffer on the output
of the matrix gen? and multipath from that point to the FF?
On Fri, Mar 20, 2020 at 9:41 AM Eunchan Kim<EMAIL_ADDRESS>wrote:

yeah but i worry if we do that, it will multicycle from the input all the
way to the flop. Whereas what we're really trying to say is that the matrix
calculation is multicycle, whereas the last flop calculation has to be 1.
Does that make sense?
I see. I thought that it can be defined as set_multi_path -from {signame}
-to {FF.d}.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/1758#issuecomment-601798379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSRIKZSMZZIFJNS2LHDRIOMEVANCNFSM4LHX3LRQ
.

actually sorry that's not what i meant to say.
What i'm thinking is...what if we set a max_delay through the matrix logic,
and then false path from the key input all the way to the flop?
That will tell the tool it should try to finish all the combo logic in a
certain amount of time (let's say...500ns), and also not attempt to time it
to the D input of the flop because it's technically going to be static by
the time we care.
The tool however will still time the other input to the flop, and with no
further information, it should treat that as 1 cycle.
On Fri, Mar 20, 2020 at 10:29 AM Timothy Chen<EMAIL_ADDRESS>wrote:

yeah that makes sense. So i guess we would maybe put a buffer on the
output of the matrix gen? and multipath from that point to the FF?
On Fri, Mar 20, 2020 at 9:41 AM Eunchan Kim<EMAIL_ADDRESS>wrote:

yeah but i worry if we do that, it will multicycle from the input all the
way to the flop. Whereas what we're really trying to say is that the matrix
calculation is multicycle, whereas the last flop calculation has to be 1.
Does that make sense?
I see. I thought that it can be defined as set_multi_path -from
{signame} -to {FF.d}.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/1758#issuecomment-601798379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSRIKZSMZZIFJNS2LHDRIOMEVANCNFSM4LHX3LRQ
.

@vogelpi
btw, don't feel like you have to use my crappy module. If you have something better planned in works, feel free to go a different route.

This is definitely not crappy @tjaychen ;-)
Thanks for adding the references. I have studied your implementation and the reference more in depth now. It is practically the same as the digit-serial implementation described in the paper I referenced before (with a digit size D equal to StagesPerCycle in your design). So I think I would have come up with a very similar implementation.
One more question regarding the other implementations discussed by your reference: the Fan-Hasan (FH) and Karatsuba (KA) implementation both seem to be around 30% smaller than the Mastrovito implemenation (corresponds to StagesPerCycle = Width). So we might also study these in more depth, but I am not sure if it makes sense to iteratively decompose those.  If you have something better planned in works, feel free to go a different route.

This is definitely not crappy @tjaychen ;-)
Thanks for adding the references. I have studied your implementation and the reference more in depth now. It is practically the same as the digit-serial implementation described in the paper I referenced before (with a digit size D equal to StagesPerCycle in your design). So I think I would have come up with a very similar implementation.
One more question regarding the other implementations discussed by your reference: the Fan-Hasan (FH) and Karatsuba (KA) implementation both seem to be around 30% smaller than the Mastrovito implemenation (corresponds to StagesPerCycle = Width). So we might also study these in more depth, but I am not sure if it makes sense to iteratively decompose those. Do you have any idea?

to be honest i haven't looked very closely at the other implementations.  When i started studying, it seemed to suggest that the Mastrovito had lowest latency, and i was immediately sold.
I do think it's worth investigating more though.
Btw, let me know if there are other things I should change in this specific PR (i think other implementation options we can slowly tack on as we study them).  Otherwise, I may start prepping a real PR where I try to convert this into prim_galois_mult.

On Wed, Mar 25, 2020 at 06:29:36AM -0700, Pirmin Vogel wrote:

References:

[1]: Mastrovito, "VLSI Designs for Multiplication over Finite Fields GF(2^m)",
https://link.springer.com/chapter/10.1007/3-540-51083-4_67
[2]: Song et al., "Efficient Finite Field Serial/Parallel Multiplication",
https://ieeexplore.ieee.org/document/542803

Further useful references:

[3] Patel, "Parallel Multiplier Designs for the Galois/Counter Mode of Operation",
https://pdfs.semanticscholar.org/1246/a9ad98dc0421ccfc945e6529c886f23e848d.pdf
[4] Wagner, "The Laws of Cryptography: The Finite Field GF(2^8)",
http://www.cs.utsa.edu/~wagner/laws/FFM.html

OMG, I should read all of them. Thanks for the materials!

Hey @tjaychen ,
I started to set up a scratch Verilator TB for the Galois multiplier primitive. It is still very primitive. But maybe it is already of use to you. You can find it on my branch here: https://github.com/vogelpi/opentitan-1/tree/gfmul-pre_dv
The testbench can be found under /hw/ip/prim/pre_dv/prim_galois_mult_tb . There is also a README showing how to build and use it.
The next steps on my list are:

Set up a DPI model using the Python package you referenced.
Randomize the inputs
Parametrize the TB to more easily test different configuration (Width + Polynomials).


Fixes 18300: Get last completed job run

Describe your changes:

Fixes 18300
Currently omd only ingest last successful dbt job, if there are failing dbt job due to dbt tests, the result will be never ingested by omd.
The change I am making to to let omd query last completed dbt job, either success (10) or error(20), so omd can ingest failed dbt tests and create incidents accordingly and correctly.
According to dbt official doc, we can set status__in with a set of 10, 20
Filters the results to a specific set of run statuses. 1: Queued, 2: Starting, 3: Running, 10: Success, 20: Error, 30: Cancelled
I have tested the API by using Postman
GET
https://cloud.getdbt.com/api/v2/accounts/<account_id>/runs/?project_id=<project_id>&job_definition_id=<job_id>&order_by=-finished_at&limit=1&status__in=[10,20]

Response:
{
    "status": {
        "code": 200,
        "is_success": true,
        "user_message": "Success!",
        "developer_message": ""
    },
    "data": [
        {
            "id":,
            "trigger_id": 349296538,
            "account_id":,
            "environment_id":,
            "project_id":,
            "job_definition_id":,
            "status": 20,
            "dbt_version": "versionless",
            "git_branch": "master",
            "git_sha": "33007b3127508ba1009884ca5fefe9bc3e6041eb",
            "status_message": "dbt command failed",
            "owner_thread_id": null,
            "executed_by_thread_id": "orc-dispatch-x3-1-57c87f96fb-tsnht",
            "deferring_run_id": null,
            "artifacts_saved": true,
            "artifact_s3_path": "prod/runs//artifacts/target",
            "has_docs_generated": false,
            "has_sources_generated": false,
            "notifications_sent": true,
            "blocked_by": [],
            "created_at": "2024-12-02 19:00:56.649426+00:00",
            "updated_at": "2024-12-02 19:08:39.157694+00:00",
            "dequeued_at": "2024-12-02 19:00:56.969450+00:00",
            "started_at": "2024-12-02 19:00:57.784528+00:00",
            "finished_at": "2024-12-02 19:08:39.050634+00:00",
            "last_checked_at": "2024-12-02 19:08:39.107842+00:00",
            "last_heartbeat_at": "2024-12-02 19:06:46.612584+00:00",
            "should_start_at": "2024-12-02 19:00:56.649426+00:00",
            "trigger": null,
            "job": null,
            "environment": null,
            "run_steps": [],
            "status_humanized": "Error",
            "in_progress": false,
            "is_complete": true,
            "is_success": false,
            "is_error": true,
            "is_cancelled": false,
            "duration": "00:07:42",
            "queued_duration": "00:00:01",
            "run_duration": "00:07:41",
            "duration_humanized": "7 minutes, 42 seconds",
            "queued_duration_humanized": "1 second",
            "run_duration_humanized": "7 minutes, 41 seconds",
            "created_at_humanized": "4 hours, 27 minutes ago",
            "finished_at_humanized": "4 hours, 20 minutes ago",
            "retrying_run_id": null,
            "can_retry": true,
            "retry_not_supported_reason": null,
            "job_id":,
            "is_running": null,
            "href": "https://cloud.getdbt.com/deploy//projects//runs//",
            "used_repo_cache": null
        }
    ],
    "extra": {
        "filters": {
            "limit": 1,
            "offset": 0,
            "state": "all",
            "account_id":,
            "project_id":,
            "job_definition_id":,
            "status__in": [
                10,
                20
            ]
        },
        "order_by": "-finished_at",
        "pagination": {
            "count": 1,
            "total_count": 1056
        }
    },
    "error_code": null
}

Type of change:

[x] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@SumanMaharana @ulixius9 PTAL

@TeddyCr I have applied formatting by using pre-commit, but some tests are failing. However, I am not sure whether it's due to my change since they are UI related tests. Can you take a look?

@TeddyCr @SumanMaharana thanks for merging the change into main branch. Can we know when and which release version this change will be in?


hyprland.conf syntax change

stylix is writing incorrect syntax to hyprland.conf for shadows
col.shadow = rgba(1a1a1aee)
is now:
shadow {
enabled = true
range = 4
render_power = 3
color = rgba(1a1a1aee)
}
the old syntax displays as an error in hyprland

Is this related to https://github.com/danth/stylix/pull/610?

it is

Closing as duplicate: https://github.com/danth/stylix/pull/610.


Link BCP-003-01 to https://github.com/AMWA-TV/nmos-secure-communication

Or should the link to BCP-003-01 be https://amwa-tv.github.io/nmos-secure-communication/ ?

@peterbrightwell The build failure is due to a version bump in a dependency of the CI workflow, hosted-git-info, 6 days ago, due to their commit as part of version 3.0.4: https://github.com/npm/hosted-git-info/commit/167cef296215ffffb7c2ba7b142be0ae35d44b50. Since 3.0.4 was working for us I think we could explicitly specify 3.0.4, or perhaps 2.8.8. Otherwise we need to force node v10.

Or should the link to BCP-003-01 be https://amwa-tv.github.io/nmos-secure-communication/ ?

To be determined... NMOS Steering are working on a strategy for memorable, AMWA-controlled, canonical URLs for all specs and versions thereof.

NMOS Steering are working on a strategy for memorable, AMWA-controlled, canonical URLs for all specs and versions thereof.

Has this been determined?

Thomas will talk with NMOS steering folks about this.

As per @peterbrightwell we should use https://amwa-tv.github.io/nmos-secure-communication/ for now, this might change later.

@NEOAdvancedTechnology, the merged commit makes the link to https://github.com/AMWA-TV/nmos-secure-communication not https://amwa-tv.github.io/nmos-secure-communication/

It will change later... as part of a larger plan for AMWA URL rationalisation

So not gtihub.io "for now" as suggested above? OK

@garethsb-sony will fix to https://amwa-tv.github.io/nmos-secure-communication/ I think Peter is talking about future change to something like specs.amwa.tv

yes exactly that


feat: Globalize Product ID

Describe the feature
Make all product ID unique for global.
Embed datasource_id into product_id field.

[x] #707
[ ] Remove datasource_id in Product
[ ] Remove datasource_id in Period
[ ] Remove datasource_id in PSR
[ ] Remove datasource_id in GSR
[ ] Remove datasource_id in AccountInfo's positions
[ ] Remove datasource_id in FundingRate
[ ] Remove datasource_id in Tick

Why do you need this feature?

We can view all products in a tree view
Easily compose accounts
Remove the 'datasource' concept (make it simpler)
Position can link to a certain product

Additional context
No response

#715

Some additional points. Using a tree-structured resource management model involves more than just concatenating datasource_id and product_id together. For instance:

Replace the previous calls to encodePath, decodePath with standard posix-style path libraries (in browser environments, these libraries can be used by configuring the packager, and the same consideration should be given when packaging plugins as IIFE).
Consider alternative methods for provideTicks.
Consider alternatives for decodePath within provideDataSeries.
After simplifying GSR into two fields, should it be directly merged into the IProduct interface? Or has it become meaningless altogether?

We will never again seek to impose any requirements on the path format of product_id.
We will never demand to extract any structured information from the hierarchical path of product_id.
Therefore, the specific structure within the product_id path is no longer important; what matters is:

Establishing jurisdiction. Each provider is responsible for offering a portion of the products.
Determining hierarchical and kinship relationships for certain classification statistics and data consumption.
To accommodate the limited capacity of the human brain, allowing for the management of the vast amount of diverse product information by breaking it down.


how to reference a div inside ReactDOMBuilder.render()

I've been converting this code:
https://github.com/Kotlin/kotlin-fullstack-sample/blob/master/frontend/src/org/jetbrains/demo/thinkter/HomeView.kt
to my own needs and got to a point where inside the render method I need to do:
val div = document.getElementById("map")
kotlinGoogleMap(div)

this comes from https://blog.frankel.ch/kotlin-front-end-developers/  so the actual line I'm trying to call is line 33 here https://github.com/nfrankel/kotlin-frontend/blob/master/src/script.kt#L33
Is there a good way to do this from my HomeView.kt render method?

I was able to solve this with componentDidMount:
   override fun componentDidMount() {
      super.componentDidMount()
      val div = document.getElementsByClassName("thegmap").item(0)
      kotlinGoogleMap(div)
    }

where render makes the thegmap div:
override fun ReactDOMBuilder.render() {
  div {
    div(classes = "thegmap") {
    }
  }
}


Append gmt to the install dir on Windows

Addresses the install path issues pointed out in #62
Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@leouieda I don't think this is the right solution.
Comparing the build.sh and bld.bat, I believe you should add
-D GMT_INSTALL_TRADITIONAL_FOLDERNAMES:BOOL=FALSE

to windows build script.

Good point @seisman, thanks!

We were missing the GMT_INSTALL_MODULE_LINKS:BOOL = FALSE as well. Don't need to create all of the shorthands for GMT4 compatibility.

@seisman what do you think of this?

It looks good to me.

There is always a warning when running gmt on Windows:
gmt: Warning: failed to release lock (Error <158>: The segment is already unlocked.)

I believe this warning can be fixed by setting FLOCK to on, but I know nothing about file locking. Maybe @joa-quim can help here.

I don't think there are file locking's on Windows. Again, I've seen that type of warning before. It happened when gmt tried to delete a temporary file but that file was not yet closed. However, I do not see that warning, so once again I don't know how to help.

@seisman is the warning happening to you locally?

@seisman is the warning happening to you locally?
Yes, it happens locally too.

@leouieda As the file lock warning doesn't affect GMT's functioning, we can ignore it at present. I think the PR is good to merge.

:+1: @seisman I just have to bump the build number. After that, we can merge then.


setCellProps changed following V3 update

I'm setting a cell's border by calling setCellProps in the column's options. This is the code:
setCellProps: (value) => { return { className: classnames ({ [classes.BorderCell]: value !== undefined, }) }}

BorderCell is simple CSS:
BorderCell: {
    borderRight: '1px solid gray',
},

In V2 it looked like this:

Now it V3 there's a border both for the cell and for the text in it:

What's the new way of doing it?

Looks like this is a bug, the props get spread on the inner cell as well as the table cell: https://github.com/gregnb/mui-datatables/blob/master/src/components/TableBodyCell.js#L136
I'll get a fix into the 3.2.0 release, which should either happen tonight or tomorrow (more info here: https://github.com/gregnb/mui-datatables/pull/1404)

Try with the new 3.2.0 version, this issue should be resolved there.


Add community page with avatars of all people

It is available under /community (not linked from anywhere), to help creating "Acknowledgement" slide

works on my machine and is gooooorgeous! I've added a tiny tweak to the language.


Singers won't load after updating to v<IP_ADDRESS>

Acknowledgement

[X] I have read Getting-Started and FAQ

🐛 Describe the bug
After updating OpenUtau to v<IP_ADDRESS>, hen I open a USTX or UST, the song loads, but when I try to select the singer, nothing happens. When I go to the 'Singers' window, none of the singers appear. In the files, the singer files are untouched. The preferences window also does not work as it just crashes the application once I click on it.
Explains how to reproduce the bug

Update OpenUtau to v<IP_ADDRESS> via the startup prompt
Open a USTX/UST (or just open the application)
Try to select a singer (or open the singer window)

OS & Version
Windows 11
Logs
log20230514.txt
^^^

Sorry, this likely has to do with the bug mentioned at the top of the Release Notes. The likely fix has been addressed, it just needs to be merged. Sorry for the inconvenience.


[PDAL] Plugins should be included

Is your feature request related to a problem? Please describe.
Using the vcpkg PDAL package to read pointcloud data has worked great for me, until a client supplied one in e57 format. After doing some cursory investigation, reading/writing this format is included as a plugin for PDAL, but none of the plugins are included in the vcpkg build.
Proposed solution
Please include plugins in order to support more pointcloud file formats.
Describe alternatives you've considered
The alternative is to return to building directly from source and setting the appropriate cmake options. Because of the myriad of dependencies this is probably not going to be a straightforward process imho.

Plugins should be added as features.


.xp doesn't show unicode characters on image

How it works:

This is because the drawing library I'm using doesn't support drawing unicode characters :c

Should we make an issue on the library your using instead? Because even with help this is out of your doing :P

the problem is within the font, the drawing library will draw nothing if the font doesnt support the characters

Tested this out on NadekoBot v2.10.1. Still present.

Update:
Rjt suggested to use Noto Sans. This font supports more characters what should solve this issue.

What about just using the Arial font? I tested it and it draws the unicode character perfectly

Nice try, but it doesn't work properly, and having the single font be huge % of the size of the whole bot is not viable. It works sometimes but most of the time not. It's better than what we currently have for sure, but definitely not worth it.

Tried it, it's not the font, it's the fact that the library has issues drawing them I believe.

What library is it drawing from? Because you can always just download Microsoft IMEs for the languages most commonly used in the usernames like JP and minimize storage or download all of them with more compatibility with usernames at the cost of more storage.
Though if you're already using it and it's still having problems with drawing from it, just ignore this message I guess.

How to change background like you?


conditional definition of _pointer methods (for Julia 1.6)

Codecov Report

Merging #17 into master will decrease coverage by 7.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
- Coverage   72.11%   65.04%   -7.08%
==========================================
  Files           9        9
  Lines         520      452      -68
==========================================
- Hits          375      294      -81
- Misses        145      158      +13

Impacted Files
Coverage Δ

src/ssubarray.jl
29.62% <0.00%> (-18.43%)
:arrow_down:

src/abstractarray.jl
75.00% <0.00%> (-11.37%)
:arrow_down:

src/broadcast.jl
70.00% <0.00%> (-11.36%)
:arrow_down:

src/linalg.jl
78.12% <0.00%> (-2.44%)
:arrow_down:

src/indexing.jl
83.96% <0.00%> (+0.30%)
:arrow_up:

src/convert.jl
58.06% <0.00%> (+0.92%)
:arrow_up:

src/HybridArrays.jl
97.91% <0.00%> (+1.55%)
:arrow_up:

src/arraymath.jl
75.00% <0.00%> (+15.00%)
:arrow_up:

Continue to review full report at Codecov.

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


OHIF-179: UI for successful posting of SR data to PACS.

PR Checklist

[ ] Brief description of changes
[ ] Links to any relevant issues
[ ] Required status checks are passing
[ ] User cases if changes impact the user's experience
[ ] @mention a maintainer to request a review

@dannyrb we have to create an issue to update the snackbar styles to tailwind since this will take some time. And possibly another one to reuse this logic of showing a blocking loading indicator using the dialog service.


[chore] Remove Extra File

Somehow an extra file was included in client about a month ago. This PR removes it.

Pull Request Test Coverage Report for Build 10793

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.604%

Totals

Change from base Build 10785:
0.0%

Covered Lines:
6793

Relevant Lines:
8642

💛  - Coveralls


JEI does not show cooking pot recipe

Hello.
Since is a very important part of the game, JEI doen't show the cooking pot recipes.

Why don't use REI？

Hello @RibsyCC,
Fabric version of Farmer's Delight has the REI compatibility, use it to have all recipe.


Issue with PyCurl

Receive the following error:
Traceback (most recent call last):
  File "/Users/####/Sites/####/####/dl.py", line 3, in <module>
    from defillama.tvl import get_protocol_tvl
  File "/Users/#####/Sites/####/#######/venv/lib/python3.11/site-packages/defillama/tvl.py", line 2, in <module>
    from ._utils import get
  File "/Users/####/Sites/#####/#######/venv/lib/python3.11/site-packages/defillama/_utils.py", line 1, in <module>
    import pycurl
ImportError: pycurl: libcurl link-time ssl backends (secure-transport, openssl) do not include compile-time ssl backend (none/other)

I checked my pycurl
Name: pycurl
Version: 7.45.2
Summary: PycURL -- A Python Interface To The cURL library
Home-page: http://pycurl.io/
Author: Kjetil Jacobsen, Markus F.X.J. Oberhumer, Oleg Pudeyev
Author-email<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>License: LGPL/MIT
Location: /Users/###/Sites/###/###-###/venv/lib/python3.11/site-packages
Requires:
Required-by: DeFiLlama-Curl

Fixed it with this
https://stackoverflow.com/a/74173308/1240798

Does this require any source code/deps modifications?

Closing this because its resolved and a PyCurl issue at the end.


Azure Policy installs Qualys extension to the AKS Nodes

Hello,
I created the following Azure Policy for the automatic installation of the Qualys extension to the VMs but it also installs to the AKS nodes. Will it be a problem for the AKS cluster? How can I exclude the AKS nodes in this Policy?
https://github.com/Azure/Azure-Security-Center/tree/master/Remediation scripts/Enable the built-in vulnerability assessment solution on virtual machines (powered by Qualys)/Azure Policy
As far as I know, AKS nodes don't support Qualys Vulnerability Assessment extension installation in Security Center.

Hi @serxansherif
When assigning a policy definition, you can define a scope to exclude from the assignment. Doing so, you can make sure that the DINE policy is not applied to a particular scope, such as the AKS nodes' resource group. You can find further information here and here.

hi @TomJanetscheck
are there any options to excluding in the Policy definition without assigning exclusion to the Resource Groups?
I want to build a more dynamic Policy instead of assigning RGs.
In the following screenshot, Policy includes VMs, does it possible to exclude only AKS Nodes here?

A policy definition will only define what the policy will do (and for what resource type), not where. The assignment is where you decide for which resources a policy will be effective. You can assign the policy on a level up to your Tenant Root Management Group, and exclude the resources, resource groups, and/or subscriptions you do not want to be touched by the policy.

Hi @TomJanetscheck
Thanks for the clarification. I have another issue related to this Azure Policy. Policy installed Qualys extension to the 4 VMs then it stopped to install the extension, other VMs (about 20) are shown non-compliant in the Policy.

The policy will run a remediation task for all resources which are non-compliant. However, it may take a while to pick up remediation for all resources. If the issue persists, I suggest to open a support ticket as this issue then is related to the Qualys installation itself, not the DINE policy.
Closing this GitHub issue since answers to the initial request have been provided.


QuickLook on iOS Chrome does not work with .reality files, even with correct Content-Type Headers

Description
A similar issue came up in #534, however, it seems that this time Chrome actually has an issue.
QuickLook supports both .usdz files (model/vnd.usdz+zip) and .reality files (model/vnd.reality). However, even with QuickLook chrome integration enabled, only .usdz files properly display in AR, .reality files always get downloaded (they work fine in Safari).
Expected: .reality opens in QuickLook in iOS Chrome when pressing AR button
Actual: .reality is downloaded in iOS Chrome when pressing AR button
Live Demo
https://prefrontalcortex.de/labs/model-viewer/penguin-audio/
(can't use glitch since it does not serve correct Content-Type anyways)
Browser Affected

[X] Chrome
[ ] Edge
[ ] Firefox
[ ] Helios
[ ] IE
[ ] Safari

OS

[ ] Android
[X] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

@soraryu thanks for catching this. This may, in fact, be considered a feature request by the Chrome team (just speculating). IIRC they had to manually implement support for .usdz, so it is likely that they would have to do the same for .reality.
My recommendation would be to file a bug at https://crbug.com. If you do, I will reach out to some folks I know on the iOS Chrome team and see what they have to say 👍

I reported a bug here: https://bugs.chromium.org/p/chromium/issues/detail?id=1051609
Thanks for forwarding this!

Hello. Any news on .reality support in model-viewer?

This bug is about .reality file issues in iOS Chrome. At this time, I do not know of a reason why .reality files would fail in iOS Safari. @DA-NDI Can you confirm that .reality files fail with ` in iOS Safari (not Chrome)?

Quick update: just got a notification that the bug is fixed in Chrome, so hopefully at some point in the future this will work properly.
https://bugs.chromium.org/p/chromium/issues/detail?id=1051609#c4
https://chromium.googlesource.com/chromium/src.git/+/e4e00438348f37dd35349aeb084564eee2e53502

Seems this will release in Chrome 85, already in Canary.


Resdesigned the contact page and made it responsive for all devices

Changed the contact page design and made it responsive for all the devices screens.
Changes :

@achintyasrivastav  Looks Good To Me

@achintyasrivastav Looks Good To Me

Thanks! I request you to please assign this under the label of Hacktoberfest 2024 & GirlScript Summer of Code 2024.

It's #423

@achintyasrivastav Good!


A new StatevectorSampler times slower that the V1 implementation on a small number of qubits

What should we add?
The implementation of the Sampler primitive is significantly slower than the V1 implementation on a small number of qubits.
Here is a script:
import time

import matplotlib.pyplot as plt
import numpy as np
from qiskit import QuantumCircuit
from qiskit.circuit.library import EfficientSU2
from qiskit.primitives import StatevectorSampler, Sampler as RefSampler

def time_ref_v1_sampler(qc: QuantumCircuit, data: np.ndarray, retries=100):
    ref_sampler = RefSampler()
    start = time.time()
    for _ in range(retries):
        ref_sampler.run([qc] * len(data), data).result()
    elapsed = time.time() - start
    print(f"Reference V1 Sampler: {elapsed:0.2f} sec")
    return elapsed

def time_ref_v2_sampler(qc: QuantumCircuit, data: np.ndarray, retries=100):
    sampler_v2 = StatevectorSampler()
    pubs = [(qc, data[i, :]) for i in range(len(data))]

    start = time.time()
    for _ in range(retries):
        sampler_v2.run(pubs).result()
    elapsed = time.time() - start
    print(f"Reference V2 Sampler: {elapsed:0.2f} sec")
    return elapsed

def run_comparison():
    v1_elapsed = []
    v2_elapsed = []
    qubits = [5, 6, 7, 8, 9, 10]
    for num_q in qubits:
        qc = EfficientSU2(num_q)
        qc.measure_all()
        data = np.random.random((10, qc.num_parameters))

        v1 = time_ref_v1_sampler(qc, data)
        v1_elapsed.append(v1)
        v2 = time_ref_v2_sampler(qc, data)
        v2_elapsed.append(v2)

    plt.plot(qubits, v1_elapsed, label="v1")
    plt.plot(qubits, v2_elapsed, label="v2")
    plt.xlabel("Num qubits")
    plt.ylabel("Time")
    plt.legend()
    plt.show()

if __name__ == '__main__':
    run_comparison()

Here is the result on Mac.

On the larger numbers, e.g. 15 qubits, the results are comparable.

After some benchmarking with @ElePT it seems that two possible culprits could be

that the old Sampler still has a caching of circuits
the StatevectorSampler seems to have some new overhead in type conversions
That being said, in your example a significant chunk of time is spent in the removal of final measurements which could maybe be sped up (e.g. by porting to Rust).

I checked the script and found that the removal of final measurements takes around a half of runtime. If it is urgent, I can write a patch to add a cache to StatevectorSampler (and StatevectorEstimator if possible).
We didn't include that because QuantumCircuit does not have a hash function (because it is mutable) and we made a private utility function as a workaround.
https://github.com/Qiskit/qiskit/blob/fa3d6df04f57d6a5bcc658a7f1a6e5a99984b949/qiskit/primitives/utils.py#L133-L134


SVG logo scaling

The swiML logo on the HTML page does not scale with the rest of the text. Its size should be determined by en or em. This will make it relative to the overall text size which is the basis for all other size units used.

Just to demonstrate the problem. When you use a web browser to increase/decrease the zoom the text and all other elements scale. The SVG logo at the bottom stays at the exact same size.


Fix PR builder for identity-apps PRs.

PR builder is failing for identity-apps PRs.

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3171809351

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3171809351
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3279844574

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3279844574
Status: success


Presenters and automatic reload raises an exception

I am opening an issue to know if I should do something else in my project or if it is possible to improve the shortcode gem.
I created my presenters in the app/presenters directory in my rails app to get automatic reload in development environment so I don't have to restart the server when I change something in a presenter. The presenters are registered in an initializer.
However, when I modify a presenter, on the next load of a page using shortcode, I get that exception:
A copy of ExamplePresenter has been removed from the module tree but is still active!

I was able to fix this temporarily by registering my presenters in my controller, but I don't like the idea.
The other solution I think would be good is to change the register implementation to store string of the class name instead of the class itself. So the class can be reloaded and called over and over dynamically.
def register(presenter)
  validate presenter
  [*presenter.for].each { |k| presenters[k.to_sym] = presenter.to_s }
end

Then each time the class is needed, we'll have to use  "Classname".constantize. Maybe we should make a getter method...
I'm not 100% sure this will fix the problem, so I will be working on it. If that's not the solution, I will tell you. If it is, I will create a PR. If you have a better idea, I'd like to try it.

I have the same issue, would it be possible to merge this? I understand @kernow had some objection for using this fix in production, but I'm not sure what the issues could be?
@sophiedeziel I suggest using safe_constantize instead of constantize

@mtarnovan I'll have a look at this later today, the main reason I haven't merged is because I've not had the time, there were a few tweaks I wanted to make before merging that I haven't got around to yet.

Ok, let me know if I can help.

Ok so the issue is that contstantize is a rails method so it can't be used. Another way will need to be found to solve this issue.

How about Module.const_getor cherry-picking ActiveSupport's constantize?

Oh, I did not know that about constantize. Module.const_get seems an excellent approach. It would make us use symbols instead of strings, which is also good.

I think this will cause more problems for other users as const_get does not work for all situations e.g. Foo::Bar. Could you try registering your presenters as ::ExamplePresenter rather than ExamplePresenter to see if that fixes the situation?

I will experiment with every scenarios I can think of and write the proper tests for it. Thanks for the feedback :smile:

@sophiedeziel I meant trying out Shortcode.register_presenter(::CustomPresenter) rather than changing anything within the gems code. See if this resolves your problem first.

That would be awesome if it's solving it. I can't remember, I think I tried a couple of months ago and it didn't. But honestly, it's hard to remember.

What about:
2.0.0-p353 :003 > module Foo; module Bar;class Baz; end;end;end
 => nil
2.0.0-p353 :004 > 'Foo::Bar::Baz'.split('::').reduce(Module, :const_get)
 => Foo::Bar::Baz

I think this is a problem with the rails class reloading in development that's not related to Shortcode. If your presenters are in the /app directory they they are set to reload on each request. If this issue is only happening occasionally then it's likely that it's something outside of Shortcode causing the problem.
If it's only happening occasionally (and not on every request) then restarting the rails app isn't much of an issue. I'm going to close this issue for now.

@mtarnovan to reproduce it, change something in your presenter file. Each time the class changes, it is reloaded but the reference we keep remains the same. I'll have a look on other gems to see what strategy they use when they have to register classes and keep a reference to them.
Another solution would be to have the presenters in the lib/ directory, but we'll have to restart our server each time we modify our presenters.

For some reason this issue happens for us from time to time even without changing a presenter file, and after it happens a server restart is required for it to go away. Obviously not a blocker, but rather annoying, so I'd like to find a fix other than moving the presenters out of autoload_path.


autoplay doesn't work with embed

User was trying to setup auto play here:
Iframe looks like, which has the right param in iframe (and extra on url, not sure if needed)
<iframe width="560" height="315" src="https://odysee.com/$/embed/let%27s-cook-wagyu-enhanced-brisket/498bec51b0ddc7f0411d7c1f923fba3650967f4c?r=5bSHWQjmQaAAz3TLKy6sTN1c46xUTmVT&autoplay=true" title="Odysee video player" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen=""></iframe>

Not familiar with the codebase, but here shouldAutoplay is set to false if embedded is true.

You're right, will adjust that today.


Report is only text, what am i missing to see the IU when I open the file

Hello, I've ran the command to get the audit report, but when I open the file (using IE, Edge, Chrome) from the base file explorer location, I only get the json text, no user interface.  Am I missing a plugin?
Thanks in Advance.

I found a work around.  I created a script to run in task runner, when executed the file is correctly created and I am able to view the file in Edge with the displayed interface.

Hello @NicoleBWan, thanks for the report!
Can you share with us details about your environment? Whats OS, Browser, and NodeJS version are you using?


[Question] Does this also supports HTML inside PHP?

When I installed this extension and I couldn't find the 'Visual Editor to the Side' button.
So I was wondering if this supports PHP file.

This extension is specifically designed for editing HTML files saved locally, rather than handling the process of generating or creating HTML. Its purpose is to allow users to work with existing static HTML files directly.
However, if you're working within PHP files and including local HTML files, you may be able to leverage this extension to edit those included HTML files. Just note that it won’t support dynamic HTML generation from PHP itself, as it is intended solely for pre-existing HTML files.


Logging is a bit invasive

I get a lot of Using cached... in the logs and given I am already using trace-forms, this causes a little bit of mess in the console.
I was wondering if there is a way to disable this for instance or if they will just go away with the new release 0.8.0.
The rest seems very smooth, we are using the reg-sub facility and I confirm it works and looks great!

I wonder if logging these should be a closure-defines option?

Probably it should, at the moment I have two:
:closure-defines {"goog.DEBUG" true
                            "clairvoyant.core.devmode" true}

so a third one for re-frame would be nice as well.

logs should now be clean.


Release 5.1.2

The Release 5.1.2 with an urgent security fix is missing.

You can upgrade to 5.2.2 or 5.3.1 instead.

As this is our production server I plan to upgrade I would like just a minor fix upgrade. And do further Tests on our staging server before moving our production server to another feature update.

Seems legit.
I am not able to push images to Docker Hub. Maybe @jasonblais or @cpanato are able to build and push an image for 5.1.2 (just need to change the app/Dockerfile)

@pichouk i can build and push

@pichouk @liepertt image is available https://hub.docker.com/r/mattermost/mattermost-prod-app/tags/

Do we need to update anything in our build process for future? Curious why 5.2.2 was pushed to Docker Hub but not 5.1.2.
@cpanato @pichouk

the tag 5.1.2 was not created, i guess

Right, but do we know why? May that happen in the future?

no sure. @pichouk any thoughts?

(Sorry for the delay). I think I know what happen.
5.1.2 was released the same day (2018, September 16th) than 5.2.0 so we upgraded the image without noticing that the 5.1.2 version was missing. In fact when a new minor (eg. 5.2.X) version is released, we do not maintain the previous one if there is a patch version (eg. 5.1.Y).
We should be better at maintaining some Git branches for previous versions in order to easily continue to upgrade patch versions. IMHO @liepertt use case is legit.


Remove multi sentence method comments that mess up with doc generation

Temp fix for #11

Current coverage is 100% (diff: 100%)

Merging #15 into master will not change coverage

@@           master   #15   diff @@
===================================
  Files           4     4
  Lines          29    29
  Methods        28    28
  Messages        0     0
  Branches        1     1
===================================
  Hits           29    29
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 102abd5...4a62158

LGTM, :shipit:


Add openSUSE Leap 15.2 and move Uyuni Master to it

What does this PR change?
Add openSUSE Leap 15.2 and move Uyuni Master to it
Libvirt image: https://build.opensuse.org/package/show/systemsmanagement:sumaform:images:libvirt/opensuse152
OpenStack iamge: https://build.opensuse.org/package/show/systemsmanagement:sumaform:images:openstack/opensuse152

I have tested on libvirt and LGTM

Ok, merging, merging susemanager-ci PR... and let's hope nothing will break :-D


Moved Porter to architecture section and updated description

I'd like to propose reorganization of Porter under the architecture section as I feel it is a much better fit there. Its main purpose is to provide structure through its abstraction for data imports. This isn't the same thing as data structure, it's an architectural structure because we're more concerned with structuring the program and less about the structure of data. This is especially true of the upcoming v4 which removes or migrates almost all implementations and replaces them with interfaces, but I have a hard time justifying its current classification even now.

Thanks.


feat: instrument using open-telemetry

Fixes #13

This is WIP, need to test properly and add errors and attributes to spans


 But the contract creation transactions are already shown in browser. Next time I run this, it gives nonce is not correct error :(

What is the L2 you are running against. Is this an open source repo that you can point us to to help diagnose the issue?

fuji throws same error when using other modules in single module and calls after deployment
`   const { contract } = m.useModule(TimelordsModule);
const { contract: Factions } = m.useModule(FactionsModule);
m.call(contract, 'setContracts', [Factions.address, Players.address])
m.call(Factions, 'setContracts', [contract.address, Guilds.address])`

fuji throws same error when using other modules in single module and calls after deployment
const { contract } = m.useModule(TimelordsModule);
    const { contract: Factions } = m.useModule(FactionsModule);

    m.call(contract, 'setContracts', [Factions.address, Players.address])
    m.call(Factions, 'setContracts', [contract.address, Guilds.address])

You shouldn't need to destructure the address with passing a contract future into a call, the following should work, assuming Factions and Players are both contract deployment futures (i.e. they were created with const Factions = m.contract("Factions")):
m.call(contract, 'setContracts', [Factions, Players])

Closing for now. If more info comes in we can reopen.


search-sample: texture downloaded but doesn't apply into mesh, shown as black.

In search-sample, when load some models like:

Burger: https://poly.google.com/view/eke7qcu_FR2
Fox: https://poly.google.com/view/10u8FYPC5Br

The result shown as black model without texture, but texture downloaded sucessfully, please see below screenshot:

Dear @mrdoob
Have you found any feasible solution for this problem yet?
There are quite a lot models with texture got into this problem ( even these models published by the original Poly by Google ).

I've not been able to investigate this yet. Hoping to have a look tomorrow.

newmtl Hamburguer_Mat
illum 4
Kd 0.00 0.00 0.00
Ka 0.00 0.00 0.00
Tf 1.00 1.00 1.00
map_Kd Hamburger_BaseColor.png
Ni 1.00

newmtl Fox_Mat
illum 4
Kd 0.00 0.00 0.00
Ka 0.00 0.00 0.00
Tf 1.00 1.00 1.00
map_Kd Fox_BaseColor.png
Ni 1.00

These are difficult mtl files. Kd 0.00 0.00 0.00 ends up in material.color.setRGB( 0, 0 0 ) in three.js. I don't know if MTLLoader can be improved.
I'm hoping Poly will add GLTF2 support and these kind of issues will go away.
Until then, you can do this:
scene.traverse( function ( child ) {

    if ( child.material === undefined ) return;
    if ( child.material.color.getHex() === 0 ) child.material.color.setRGB( 1, 1, 1 );

} );

Thanks! I tried ignoreZeroRGBs options in MTLLoader and it works too 😄

Thanks! I tried ignoreZeroRGBs options in MTLLoader and it works too 😄

Oh! Hah! I forgot MTLLoader had that option.

Fixed. Thanks!


May be worth considering use of different tensorflow library for the Pi Zero

Running into some issues getting model prediction running from a freshly built image, only error message is "Illegal instruction". Came across this thread that may provide Pi Zero-specific performance improvements. If this ends up causing less issues or results in performance improvements, it may be worth introducing separate build stages specific to Pi Zero and sense hat devices that only install libraries necessary to each, with other minor changes such as potentially removing some references to the sense hat from specific scripts that throw up errors when a sense hat is not detected. Currently bumping into that with focus.sh on the Pi Zero. There was a simple if statement I put together for the raspi-config access point menu that may come in handy to prevent some of these.

After fighting tooth and nail with the armv6l architecture, we've ended up going to the Raspberry Pi Zero 2 W which adopts armv7l. Unbelievably glad for that, closing the issue


fix(sdk): add domain validation

This PR will add following changes

remove preference link for sidebar as it is not working properly.
Add domain validation in add domain
Add confirmation for domain delete
fix delete team modal close button link
remove hardcoded names from delete team and delete project page.
Show Member instead of userIds in invite member error message

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.292%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5768

Relevant Lines:
11469

💛  - Coveralls


SCVMM - updated specs, generate_data.rb and generate_inventory* files.

The SCVMM rspec files read inventory data from the  get_inventory_output.yml file. This file was outdated and therefore so was the SCVMM refresher_spec.
This PR updates theget_inventory_output.yml file and the rspec tests.
The script that creates the inventory data file, generate_data.rb, also needed updated to work with the latest SCVMM infra_manager.rb file
https://github.com/ManageIQ/manageiq/issues/2820

@blomquisg Please review

I am aware of this test failure.

The issue might be that there are duplicate named vms and the tree comparator probably is not aware of ordering with them.
diff --git a/Users/jfrey/Desktop/expected.txt b/Users/jfrey/Desktop/actual.txt
index 73e1c85..19a79d6 100644
--- a/Users/jfrey/Desktop/expected.txt
+++ b/Users/jfrey/Desktop/actual.txt
@@ -1,9 +1,9 @@
-         - EmsFolder: Datacenters
-           - EmsFolder: SCVMM
-             - EmsFolder: host
+         - EmsFolder: Datacenters ({:is_datacenter=>false})
+           - EmsFolder: SCVMM ({:is_datacenter=>true})
+             - EmsFolder: host ({:is_datacenter=>false})
                - EmsCluster: hyperv_clus
                - ManageIQ::Providers::Microsoft::InfraManager::Host: dhcp129-212.brq.redhat.com
-             - EmsFolder: vm
+             - EmsFolder: vm ({:is_datacenter=>false})
                - ManageIQ::Providers::Microsoft::InfraManager::Template: WS2008R2Core_Tem
                - ManageIQ::Providers::Microsoft::InfraManager::Template: cfme-hyperv-5.4-2.x86_64
                - ManageIQ::Providers::Microsoft::InfraManager::Template: cfme-hyperv-<IP_ADDRESS>-1.x86_64
@@ -68,15 +68,15 @@
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: scvmmaz
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: smis_netapp_dnd
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: sprint_demo_vm
-               - ManageIQ::Providers::Microsoft::InfraManager::Vm: taigotest
-               - ManageIQ::Providers::Microsoft::InfraManager::Vm: taigotest
+               - ManageIQ::Providers::Microsoft::InfraManager::Vm: taigotest ({:ems_ref=>"24e42dc3-d121-42c1-88b6-cf3adcac28a7"})
+               - ManageIQ::Providers::Microsoft::InfraManager::Vm: taigotest ({:ems_ref=>"d0681c50-4e31-4204-a2ff-8bc4a32fce7c"})
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: test_retire_prov_0BjSoMhMl8
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: testvm
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm1
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_auto_provisioning
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_13
-               - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_4
-               - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_4
+               - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_4 ({:ems_ref=>"b11fdee7-1c82-4a1e-a504-c1b978cae690"})
+               - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_4 ({:ems_ref=>"18a0e51e-6abb-49e4-99e9-492298bd738d"})
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_7
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_8
                - ManageIQ::Providers::Microsoft::InfraManager::Vm: vm_extrap_9

However, regardless, the numbers are too wild 191,757 additions and 47,370 deletions., and it seems your test server has been used by others.  For example the VmOrTemplate count jump from 28 to 93.  I don't think we should merge this as is because of this.
Can you modify this PR to hardcode the cassette changes specific to this PR, and we will have to investigate rebuilding the SCVMM cassettes with a dedicated test environment that cannot be manipulated by others.
cc @blomquisg

Closing this as per summary above by @Fryguy


checkov: 3.2.136 -> 3.2.137

Diff: https://github.com/bridgecrewio/checkov/compare/refs/tags/3.2.136...3.2.137
Changelog: https://github.com/bridgecrewio/checkov/releases/tag/3.2.137
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages built:

checkov
checkov.dist


CV screening for interview

Description
As a PDX Finder developer, I want to have a competent Software Developer to join PDXFinder team.
Deadline: 11 September 2020 EOD
Acceptance criteria

[x] @CsabaHalmagyi selected top 20 candidates
[x] @zperova selected top 20 candidates

Hackerrank results finished September 8.
Carryover to the next sprint.

This is now done.


Databases not being updated #51, possibly #11

This implements logic to pull only the most up to date database for a type. Each database is an entity with possible different states (cvd, cld, cud). In my findings ".cud" is used purely for unsigned databases, and should not actually be used when referring to the main items used by freshclam (main, daily, bytecode). The main motivation for not treating each database as a different file in every combination (daily.cvd, daily.cld) is databases can get rather large and the run context of lambda is 512mb. During testing if the scanner had access to both a cvd and a cld it would just ignore the older file.

I have tested this, and it doesn't appear to fix #51, the cld file is uploaded, but not the cvd file.

@andybkay The CLD and CVD databases are exclusive. The CVD is pretty much only used on a fresh download. After the initial download, diffs are downloaded going forward and then applied to the CLD/CVD to create an updated CLD only. The process originally already only uploads changed files. During my testing in my production environment, before I had code in place to not download all of the files every time I would get a complaint from clamav that the CLD was newer than the CVD and that the CVD was going to be ignored.
Can you check to see if the CLD is being updated? If the CLD is updated then the CVD most likely will not receive updates. The reason I had to put this fix in, in the first place is because freshclam was downloading diffs creating a CLD, then discarding all of that work because it never gets merged back into the CVD (by their design). Should freshclam deem a new CVD necessary it will download it, and the process will upload it to the s3 bucket because the md5sum would have changed.
This process is working correctly in my production environment and clamav does not complain that my databases are out of date. If you verify that your CLD is being updated but clam is still complaining about database out of date, we may need more information from your setup. Please also make sure you have all of the latest changes from my branch as I had a number of commits.

I've implemented @adminrobert changes into my environment, the updates run well and the scan does not complain about the databases being out of date anymore.
Thank you!

@wilqq @adminrobert - I just joined the project so I'm catching up here. But let me take a look at this PR and bring it in line with master and we'll see what we can do.

@adminrobert - I've updated this PR to what's on master. I'm going to write a few tests to make sure this does what we expect. Then I'll give it a thumbs up for merging.

I'm pretty happy with the tests around update_defs_from_s3 now. I'm still working on tests for upload_defs_to_s3. I think once we have test coverage for both of them we can merge with confidence. I apologize for the big code push to this branch but hopefully its helpful.

@jaygorrell @denniswebb - I think this is actually good to go. The changes to upload_defs_to_s3 doesn't really modify anything but the filename. I've got to test this in AWS but then I think it ought to be good to go.
The only issue I see is that the license/cla needs to be signed by @adminrobert or we won't be able to merge.

@chrisgilmerproj thanks for getting this in line with the main line. I had since lost access to the entire setup with which I wrote this. I signed the license. Let me know if there is anything else you need from me.

This is great! Thanks guys.


created rishi-garg.md

please merge my Pull Request for hactober fest 2020.

hey @wajahatkarim3  pls merge my pull request for hacktober fest 2020.


Suggestion: allow instantiation of type aliases

Currently a type aliased class cannot be instantiated. Also there is no way to alias a type with type parameters. Please consider making type aliases as capable as the types that they represent, basically capturing the type expression under a different name.

We'd need some new type system machinery here. Consider something like this:
class Foo<T, U> {
	constructor(public x: T, public y: U) { }
}
var f = Foo;

The type of f, from the type system's perspective, is this:
{
  new<T, U>(x: T, y: U): { x: T; y: U; };
}

Note that this type itself is not generic, so the expression Foo<number, string> in a value position (basically referring to typeof Foo<number, string>) has no meaning. We'd need some rule that says when you can and can't "push down" type parameters into construct signatures.
This is to say nothing of class-like things that have been modeled in .d.ts files as interface/var pairs instead of classes. What should this code do?
interface Foo<T, U> {
  x: T;
  y: U;
}
var Foo: {
  new<S>(n: string): Foo<number, S>;
  new<S, T>(n: number, m: string, z: any): Foo<T, U>;
  new<S, T, U>(n: number): Foo<T, U>;
}
import q = Foo<number, number>; // Is this even legal? What does it mean?
var j = new q('hello'); // Valid?

Thanks, it's good to know. Clearly there is no straight way of doing it by the means TS currently has.

What should this code do?

import q = Foo<number, number>; // Legal. It means that q  is  { new (n: number, m: string, z: any): Foo<number,  number>; }
var j = new q('hello'); // Invalid since q is  { new (n: number, m: string, z: any): Foo<number,  number>; }

Does #1616 cover this?

@danquirk id does if, besides being capable of type parameters, you add to it being capable of aliasing a class so that instantiating of such alias is possible

Yeah that's what I thought. I'll modify the title here to specifically reflect the instantiation aspect and we can use that issue to track the type parameters part since it already has a bunch of discussion and upvotes.

Is there any chance the proposal for this can be completed? This feature is really important in my honest opinion.

there is a workaround for classes
class MyClass<A, B, C> {}
class MyCertainClass extends MyClass<string, number, boolean>

Needs Proposal

Here's a proposal that should address this quite nicely.

We'd need some new type system machinery here

@RyanCavanaugh I'm guessing it didn't exist back when this written, but the machinery and semantics around subclassing do exactly what we want here. In particular:

import q = Foo<number, number>; // Is this even legal? What does it mean?

It would mean exactly what class q extends Foo<number, number> means, except the value of q would be equal to Foo rather than a subclass of it. In particular:

If constructor overloads matching Foo<number, number> have different return types, it's an error (ts2510)
Otherwise, the instance type of q is the return type from the constructor, and the constructor type of q is the same as Foo's, but with new overloads filtered based on the number of type parameters.

I don't imagine that being too controversial, since it mirrors existing behavior. I think the only remaining question would be syntax.
Rather than extending import (which doesn't quite fit the semantics), I'd propose extending the class syntax, like this: class Foo = Bar<Baz>. This is backwards compatible, seems unlikely to conflict with future JS syntax, conveys the meaning well, and nicely mirrors the type syntax.

I ran into this and just realized I could just replace
type Foos = Collection<Foo>;

with
class Foos extends Collection<Foo> {}

adding a use case: type definition files.
When I want to declare a class that extends an imported class, I currently have to do this:
// Aclass is `class` but A is `type`
type A = import('./a-class').AClass

// @ts-ignore -- complains that A methods aren't implemented
declare class AC implements A {}

declare class B extends AC {
}

This seems to work but is ugly


Adding HaltMotion (Or new suggested name) to physics plugin

Signed-off-by: Tomas Lorente<EMAIL_ADDRESS>🎉 New feature
Summary
Adds the functionality to stop a model's joint's movement in other cases than the battery being 0. By adding a HaltMotion component to the model, its joints movement can be turned on/off. Also added some QoL to kineticEnergyMonitor plugin with profilers and some checking.
(Updates)
19acc8d adds the capability to turn movement halting off if you only need the advice (and making it backward compatible).
Test it
TO-DO tested it with SubT worlds for now
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

IMO, the kinetic energy monitor should not be concerned with halting a vehicle. If we adhere to the separation of concerns principle, I would think another System in SubT (eg. GameLogicPlugin) would monitor the output of the kinetic energy monitor and set the state of the HaltMotion component.

I do agree with moving the HaltMotion logic into the GameLogicPlugin, but that would require adding A PreUpdate there. @nkoenig Do you have a strong feeling for either of them?

IMO, the kinetic energy monitor should not be concerned with halting a vehicle. If we adhere to the separation of concerns principle, I would think another System in SubT (eg. GameLogicPlugin) would monitor the output of the kinetic energy monitor and set the state of the HaltMotion component.
I also left some comments in case we decide to proceed with keeping that logic in the kinetic energy monitor.

Agreed. @Lobotuerk , can you remove the logic for halting motion from the Kinetic Energy Monitor and place it in the SubT Game Logic Plugin?

I do agree with moving the HaltMotion logic into the GameLogicPlugin, but that would require adding A PreUpdate there. @nkoenig Do you have a strong feeling for either of them?

Adding a PreUpdate to the GameLogicPlugin sounds good.

@azeey When moving the halting logic to GameLogic, should I leave the HaltMotion component creation inside KineticEnergyMonitor?

@azeey When moving the halting logic to GameLogic, should I leave the HaltMotion component creation inside KineticEnergyMonitor?

IMO, We don't need to modify KineticEnergyMonitor at all. All the logic can be in GameLogic and Physics.

I'm wondering if this functionality couldn't be achieved using components::JointVelocityReset / components::JointPositionReset?

It's better for a long time halt to simply stop the physics layer before applying forces, than trying to reset the joint every single timestep