NEED HELP - cosign v1.13.1

Signed-off-by: Mathieu Benoit<EMAIL_ADDRESS>Bump cosign from v1.3.1 to v1.13.1.
Fixing the errors:
verify signature for: devopps/alpine:signed
2023/02/07 00:34:19 http: panic serving <IP_ADDRESS>:34976: creating root cert pool: retrieving trusted root; local cache may be corrupt: initializing root client: tuf: failed to decode root.json: encoding/hex: invalid byte: U+002D '-'
goroutine 9 [running]:
net/http.(*conn).serve.func1()
	/usr/local/go/src/net/http/server.go:1801 +0xb9
panic({0x1b9fa40, 0xc000257c20})
	/usr/local/go/src/runtime/panic.go:1047 +0x266
github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioroots.initRoots()
	/go/pkg/mod/github.com/sigstore/cosign@v1.3.1/cmd/cosign/cli/fulcio/fulcioroots/fulcioroots.go:67 +0x235
github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioroots.Get.func1()
	/go/pkg/mod/github.com/sigstore/cosign@v1.3.1/cmd/cosign/cli/fulcio/fulcioroots/fulcioroots.go:45 +0x17
sync.(*Once).doSlow(0xc0003ac138, 0x18)
	/usr/local/go/src/sync/once.go:68 +0xd2
sync.(*Once).Do(...)
	/usr/local/go/src/sync/once.go:59
github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioroots.Get()
	/go/pkg/mod/github.com/sigstore/cosign@v1.3.1/cmd/cosign/cli/fulcio/fulcioroots/fulcioroots.go:44 +0x31
github.com/sigstore/cosign/cmd/cosign/cli/fulcio.GetRoots(...)
	/go/pkg/mod/github.com/sigstore/cosign@v1.3.1/cmd/cosign/cli/fulcio/fulcio.go:197
main.validate({0x218ee50, 0xc000562000}, 0xc000560300)
	/go/src/github.com/developer-guy/cosign-gatekeeper-provider/provider.go:72 +0x408
net/http.HandlerFunc.ServeHTTP(0x7fabe0ff4aa0, {0x218ee50, 0xc000562000}, 0xc000562000)
	/usr/local/go/src/net/http/server.go:2046 +0x2f
net/http.(*ServeMux).ServeHTTP(0x0, {0x218ee50, 0xc000562000}, 0xc000560300)
	/usr/local/go/src/net/http/server.go:2424 +0x149
net/http.serverHandler.ServeHTTP({0x2182d08}, {0x218ee50, 0xc000562000}, 0xc000560300)
	/usr/local/go/src/net/http/server.go:2878 +0x43b
net/http.(*conn).serve(0xc00041c3c0, {0x2199b40, 0xc000651980})
	/usr/local/go/src/net/http/server.go:1929 +0xb08
created by net/http.(*Server).Serve
	/usr/local/go/src/net/http/server.go:3033 +0x4e8
verify signature for: devopps/alpine:signed
one of verifier or root certs is required

I'm still getting this message with my own public signed image:
verify signature for: devopps/alpine:signed
no matching signatures:
no certificate found on signature

But that's the same message when I run this command COSIGN_EXPERIMENTAL=1 cosign verify devopps/alpine:signed:
Error: no matching signatures:
no certificate found on signature
main.go:62: error during command execution: no matching signatures:
no certificate found on signature

But at least we are making progress with this PR.

I'm able to build locally this container image and test it in my Kubernetes cluster, no problem.
One of the Build / test check in the CI is green/successful.
Where I NEED HELP is to help me understand and fix the second Build / test check and the Test / test check in CI... not sure what I should do. Anyone here to help? Thanks!

Thanks @Dentrax.

I think we should prioritize merging https://github.com/sigstore/cosign-gatekeeper-provider/pull/24 first to make it pass.

I agree with you.

Woot! Woot! All checks have passed with #24.
I will soon work on the changes requested.

Good news! Looking for your changes. I will be sneaking around the merge button.

We should be could to go, thanks for your review @Dentrax!


Manual Test Approval

This is a manual test approval.
Please approve or deny this workflow.
Minimum Approvals: 1
Approval Words: approve, approved
Rejection Words: deny, denied, reject, rejected

Deny

Rejected by ekeel.


Attention-viz with new models and datasets?

Very nice tool!
I would like to use AttentionViz on other models and datasets.
I am working with Karpaty's Llama2.c project and a dataset of Covid-19 research papers.
Do you have a tool which generates the:
tokens.json
agg_attension.json
layer{n}_attention{m}.json
from a trained model along with the input sentences?
https://github.com/karpathy/llama2.c
https://allenai.org/data/cord-19

Thank you so much for checking out AttentionViz and apologies for missing this issue until now!
Feel free to check out these notebooks: generate_data.ipynb and generate_vit_data.ipynb. We are not actively maintaining these, however, and you may need to tweak them slightly to fit your needs. But I hope they will be helpful in showing how to generate similar data files!


Black screen instead of screensaver on Arch

When running via ./flying-toasters -windowed, everything works as expected, but when added to .xscreensaver like so:
  GL:                           hextrail --root                             \n\
                                /usr/local/bin/flying-toasters     \n\

...only a black screen appears when clicking the preview button.
I am running EndeavourOS / Arch.

Hi. Which version of toasters are you running. Is it v1.1.1?

Yes, it's the latest. I also tried building from source.

Cool, thanks for confirming. I’m not planning to close this issue, but since I don’t have Arch installed, it will take some to debug. I’ll take a look on other xscreenaver modules to find out what can be a fix here.

Please check if https://github.com/torunar/flying-toasters-xscreensaver/releases/tag/v1.2.0 works for you. I changed the way the screensaver window is initialized using the implementation from xscreensaver itself, hope it fixes the issue.

This works! Thank you.


community: added Browserbase loader

Thank you for contributing to LangChain!

[x] PR title: "community: added Browserbase loader"

Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..."  for template changes, "infra: ..." for CI changes.
Example: "community: add foobar LLM"

[x] PR message: Delete this entire checklist and replace with

Description: Adds remote browser web-page loader powered by browserbase.com
Issue: -
Dependencies: (optional) browserbase
Twitter handle: @browserbasehq

[x] Add tests and docs: If you're adding a new integration, please include

a test for the integration, preferably unit tests that do not rely on network access,
an example notebook showing its use. It lives in docs/docs/integrations directory.

[x] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

Make sure optional dependencies are imported within a function.
Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
Most PRs should not touch more than one package.
Changes should be backwards compatible.
If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, hwchase17.

Thank you so much! Ready for another round 😄


README.md: Update for 0.7

README.md: Update for 0.7

@baleboy ptal

do we still need the alpha disclaimer?

no, i don't think so

ok, remove that and then LGTM

squash merged


Can't fetch stderr of a job with container exit code 1

ETA: 2022Q4
Running an invalid command (resulting in container exit code 1) should return stderr/stdout. Instead, there's no results?
Example:
❯ go run . --api-port=$BACALHAU_API_PORT_0 --api-host=localhost docker run ubuntu -- bash -c 'rm /'
cb635bf6-215a-4ca8-b6f2-85229a583095

❯ go run . --api-port=$BACALHAU_API_PORT_0 --api-host=localhost list --wide
 CREATION_TIME      ID                                    JOB                         STATE  RESULT
 22-08-17-22:57:03  cb635bf6-215a-4ca8-b6f2-85229a583095  Docker ubuntu bash -c rm /  Error

❯ go run . --api-port=$BACALHAU_API_PORT_0 --api-host=localhost get cb635bf6-215a-4ca8-b6f2-85229a583095 --output-dir ./enrico
[...]
[2207] INF bacalhau/get.go:35 > Fetching results of job 'cb635bf6-215a-4ca8-b6f2-85229a583095'...
[2207] DBG ipfs/downloader.go:42 > No results to download

❯ ls -l enrico
ls: enrico: No such file or directory

Error: job has error state Error on node QmdZQ7ZbhnvWY1J12XYKGHApJ6aufKyLNSvf8jZBrBaAVL
I think that instead of returnining that a certain node, has error it should instead give the actual error

Thread: https://filecoinproject.slack.com/archives/C02RLM3JHUY/p1660770923525699

As a work around I've found that doing something like -- bash -c "erroring-thing | true" forces the execution to succeed and the stderr is returned.
But the fix is to return the stdout/err even upon failure.

Fixed by #1545.


Fixes

What is the purpose of this pull request? (put an "X" next to item)
[ ] Documentation update
[X] Bug fix
[ ] New feature
[ ] Other, please explain:
What changes did you make? (Give an overview)
Added a missing i18n string
Fixed the incorrect path for the Wi-Fi Forget function that disconnects the device from wifi networks.
We have an interceptor on the Axios requests that logs to console whenever there is an error or non-200 return code. So I removed the console.log commands from the .catch blocks because they are duplicates.
Which issue (if any) does this pull request address?
Is there anything you'd like reviewers to focus on?

great fixes here! anything you want me to test for specifically?

great fixes here! anything you want me to test for specifically?

Not particularly. It's sometimes helpful to have a chance to see what's getting merged in to be able to keep a big picture.


[Request] TrueNAS ready docker-compose

It would be nice to have a docker-compose file ready to use for TrueNAS for a easier setup. Otherwise I'm testing a way to make this work on TrueNAS. If I find a way I will share it.

Truenas scale uses helm for it application deployment. There's a project called truecharts that has unofficial charts, that's the project that should be looked at. The only problem I can see is it uses a dependency system so  it would cover nginx/caddy and postgres but I don't know what it look like for the 3 microservices pipedfrontend, ytproxy and piped.

https://github.com/TeamPiped/Piped-Kubernetes :O


2.0.0.rc2 does not redirect automatically

I just observe that I'm getting 302 redirect with
 :headers=>
  {"location"=>
    ["https://xyz:8443/console/"],
   "date"=>["Mon, 01 Feb 2016 17:21:21 GMT"],
   "content-length"=>["86"],
   "content-type"=>["text/html; charset=utf-8"]}

max_redirects are at 10. Any idea why is this? Would 2.0.0 GA be released anytime soon?

I found the issue. In AbstractResponse#return! method, there is a check for request method. Issue is that it only checks for [:get, :head] while if one specifies GET or HEAD that doesn't catch it.

Thanks for raising this! I ended up fixing this in a way that addresses more of the underlying issues as well. The old code had no good reason to be passing request args separately from the request when the request makes its args hash accessible.


Unexpected NaN in TestBSpline::test_bspline_derivative_jumps

TestBSpline::test_bspline_derivative_jumps has unexpected Nan for some input. Found in #7638.
https://ci.preferred.jp/cupy.win.cuda120/134985/
FAILED cupyx_tests\scipy_tests\interpolate_tests\test_bspline.py::TestBSpline::test_bspline_derivative_jumps[_param_0_{c=[-1, 2, 0, -1], extrapolate=True}] - AssertionError:
Not equal to tolerance rtol=1e-07, atol=0

x and y nan location mismatch:
 x: array([-1.206633,  0.363639, -0.413208,       nan])
 y: array([-1.206633,  0.363639, -0.413208,  0.      ])

This failure didn't appear in the full test after merge.
https://ci.preferred.jp/cupy.win.cuda120/135273/

Closing as duplicate of #7240


DSC Version unclear

As already stated, it is unclear which DSC Version should be used.

The Testbed/readme.md refers to the DSC documentation so that you check out the current main branch
The Testbed/DataspaceConnector/readme.md says version 5.1.2 (inside the zip file is the main branch some commits after version 5.1.2)
The linked Test Suite  in the readme.md says you have to use version 4.3.0

So it would be nice to use a defined version for the whole testbed. For example the latest version.

Thank you for your issue.

In the pull request, I have updated the Testbed/Readme.md to refer to the DSC v6.2.0 documentation.
The new directory architecture has replaced this and it is no longer an issue. The repository will use v6.2.0
I will bring this point internally.

Actually, we are not using the DSC in our test suite. The mentioned version 4.30 of the DSC in the README is optional and will be removed soon. Don't be confused with this point. We are using the DSC only to test our tests.

DSC in Test Suite Readme is optional. All other DSC version have been updated.


Two copies of the Eigen headers are installed

I noticed today that there are two copies of the Eigen headers in my vcpkg\installed\x64-windows\include directory. One directly in the "root" of that directory, the other one in the subdirectory eigen3/.
I've got the following packages installed (and boost, but I'm not pasting these here, as they should have nothing to do with it and make the list very long):
bzip2:x64-windows                                  1.0.6-2          High-quality data compressor.
clapack:x64-windows                                3.2.1-1          CLAPACK (f2c'ed version of LAPACK)
eigen3:x64-windows                                 3.3.4-2          C++ template library for linear algebra: matrice...
fftw3:x64-windows                                  3.3.7-2          FFTW is a C subroutine library for computing the...
liblzma:x64-windows                                5.2.3-2          Compression library with an API similar to that ...
metis:x64-windows                                  5.1.0-1          Serial Graph Partitioning and Fill-reducing Matr...
openblas:x64-windows                               0.2.20-2         OpenBLAS is an optimized BLAS library based on G...
opengl:x64-windows                                 0.0-5            Open Graphics Library (OpenGL)[3][4][5] is a cro...
pugixml:x64-windows                                1.8.1-3          C++ XML processing library
sqlite3:x64-windows                                3.21.0-1         SQLite is a software library that implements a s...
suitesparse:x64-windows                            4.5.5-4          algebra library
zlib:x64-windows                                   1.2.11-3         A compression library

What I'll be doing now is uninstalling everything, maybe I can find out more.

Ok: .\vcpkg.exe remove eigen3:x64-windows removed both copies. So it seems like the eigen3 package installs itself twice into two different directories...
In my opinion the eigen3 directory should be removed. Eigen should be used by including e.g. Eigen/Core, and not eigen3/Eigen/Core: http://eigen.tuxfamily.org/dox/GettingStarted.html.

I think this is still a hack to solve the problem that different dependent packages have different rules on the eigen3 base directory. https://github.com/Microsoft/vcpkg/issues/1284
Would be better to decide for one path and patch all includes in packages that do not respect this rule. However this is currently errorprone, as this problem does not show up during port build as the required include pathes are added by cmake, while once installed by vcpck it is assumed to find all includes from the base include directory and build will break.

Ok, I'm closing this in favour of #1284.


Nexus Schema Types don't recognize "Role" Prisma Type

        const seller = {
          name: `${first_name} ${last_name}`,
          role: "RESELLER",
          cpf: `${number}`,
          email: `${email}`,
          loginName: `${nickname}`,
        }

        await ctx.prisma.seller.create({
          data: {
            ...seller,
            role: "RESELLER",
          },
        })

This is a TS issue, to make it work as intended you need to say TS the seller is a read-only object. Other than that, you can inline the variable declaration to the data property, thus eliminating the read-only/mutable dilemma TS has.
const seller = {
  name: `${first_name} ${last_name}`,
  role: "RESELLER",
  cpf: `${number}`,
  email: `${email}`,
  loginName: `${nickname}`,
} as const;

Sorry, but I don't get your explanation. I changed a little the code, but I am still get error.
        const userPrisma = ctx.prisma.user.create({
          data: {
            ...user,
            accounts: {
              create: {
                nickname,
                platformUserId: mercadolibre_id.toString(),
                platform: "MERCADOLIBRE",
                details: userData,
              },
            },
            customer: {
              create: {},
            },
          },
        })

@felinto-dev To fix this you can use "MERCADOLIBRE" as const.

The problem was solved.
I just made a small mistake when writing my schema.prisma. You can see it in the print. It's working fine now.


Support Bit-manipulation extension

File this bug to track the progress of Bit-manipulation extension support.
Thanks for the contribution from Mellanox, we have got the basic Bit-manipulation instruction class ready (#518). Following work:

Create a target for B-extension
Add tests for B-extensions.
Get ovpsim setup and pass ISS sim.
Update document
Add functional coverage model for B-extension

Please leave your comment here if you have suggestion or feature request on B-extension testing.

A rv32imcb target is created, OVPsim setting is also updated.

I believe below steps are done already.

Create a target for B-extension
Add tests for B-extensions.
Get ovpsim setup and pass ISS sim.

I believe below steps are done already.

Create a target for B-extension
Add tests for B-extensions.
Get ovpsim setup and pass ISS sim.

Yes, these are done.

Add one item: Enable spike & ovpsim co-simulation flow for B-extension.
When you update the document, please provide the links for the required toolchain

Closing this bug as all required work is done. Thanks Weicai!


installation instructions fail

Getting the following message:
pear install pear.phpunit.de/phpdcd
Failed to download phpunit/phpdcd within preferred state "stable", latest release is version 0.9.3, stability "beta", use "channel://pear.phpunit.de/phpdcd-0.9.3" to install
Running with the other instructions yields:
pear install channel://pear.phpunit.de/phpdcd-0.9.3
phpunit/phpdcd requires package "phpunit/File_Iterator" (version >= 1.3.0RC1), installed version is 1.2.6
phpunit/phpdcd requires package "phpunit/PHP_TokenStream" (version >= 1.1.0RC1), installed version is 1.0.1
No valid packages found
install failed

There is no stable release of PHPDCD. If you want to use it anyway you need to have the latest versions of File_Iterator and PHP_TokenStream. A simple pear upgrade-all will fix this for you.


Language/markdown: Added LSP and fixed Glow not working

Description
I don't know if it's good practice to combine two commits into one PR like this, but they both touch on the same module, so I figured it would probably be okay. Correct me if I'm wrong.
The first commit adds marksman as a markdown LSP. It also provides the same options that the other language modules do.
The second commit fixes Glow, which I noticed wasn't working correctly (or at all). As with the first commit, new option(s) are provided. There is now an option for changing the preview keybind, which by default is the same bind that it was before (<Leader>p).
Type of change

Bug fix (I guess. Glow not working hasn't been reported as an issue, though)
New feature

Checklist

[x] My code follows the style and contributing guidelines of this project.
[x] I ran Alejandra to format my code (nix fmt).
[x] I have performed a self-review of my own code and tested it.
[ ] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] This change requires a documentation update.
[ ] I have updated the documentation accordingly.

Screenshots & Logs
The maximal config completely fails to build for me, so I replaced it with my own. When using my own, I get no new errors, nor any new warnings.

Thanks!


Missing relationship between 'dendrite arborization' and 'neuron projection arborization'

@paolaroncaglia pointed out that 'dendrite arborization' does not appear to be a child of 'neuron projection arborization' in GO.

To do: add the missing is_a relationship.
cc @RLovering @ukemi

Hi @paolaroncaglia @ukemi @pgaudet @vanaukenk
I made a simple edit yesterday (I manually added a relationship after @paolaroncaglia pointed out it was missing), and Travis checks passed (https://github.com/geneontology/go-ontology/pull/15413).
So I could just merge it, but now I wonder whether this relationship ('dendrite arborization' is_a 'neuron projection arborization') should actually have been inferred? Because a 'dendrite' is_a 'neuron projection'... So it seems to me that this should have been inferred? Unless the fact that 'dendrite' is also a part_of 'dendritic tree' which itself is_a 'neuron projection' too (screenshot below) causes some logical inconsistencies, hence the lack of inference?

And I also wonder whether 'dendrite arborization' should be renamed as 'dendritic tree arborization' (based on the CC ontology, a 'dendrite' is a branch of (part_of) the 'dendritic tree', so a 'dendrite' cannot grow branches/arborize, but a 'dendritic tree' can). 'Dendrite arborization' should probably still be kept as a broad synonym due to naming inconsistencies in literature.
Could you please advise?
(This is related to https://github.com/geneontology/go-ontology/issues/15417)
cc @RLovering

Hi @BarbaraCzub
Please see https://github.com/geneontology/go-ontology/issues/15417
If dendrite is_a neuron projection, then the arborization terms should automatically be stated by the reasoner, not sure why this isn't happening but I can't dig deeper at the moment, sorry.
I'm not sure about the renaming instead. I guess the tree originates from a dendrite anyway? But I understand your concern. @ukemi @dosumis could you advice please as I'll be away and won't have a chance to look at this in Protege for a few days. Barbara is trying to fix this for a poster that she's presenting on Tuesday. Many thanks!

Inference would require an EquivalentClass definition on 'neuron projection arborization'.  As there isn't one right now and adding one would not be trivial (you'd probably need to use a term from PATO as the differentium), adding by hand (as you've done) is the best way to do this.

Thanks @dosumis


Add Output panel to display ancillary information

Add an Output panel below the Globe panel to display ancillary information.

[x] Output panel should be resizeable
[x] Output panel should be collapsible

Use the Split.js library to create a splitter between the Globe and Output panels.

Added the Output panel below the Globe.  Following are screenshots showing the new adjustable panel:
Screenshot 1

Normal
Screenshot 2

Resized
Screenshot 3

Collapsed


Highlight WebGL shaders embedded in HTML file

In WebGL we can specify shaders in separate files or embedded in an HTML file like this:
<html>
  <script id="vertex-shader" type="x-shader/x-vertex">
    attribute vec4 vPosition;

    void main()
    {
      gl_Position = vPosition;
    }
  </script>
</html>

VSCode already has highlighting for embedded JS in an HTML file so I think this should be possible, although I'm not sure if it's within the realm of this extension or would require a change in the way VS Code parses HTML. The shader is identifiable as the text inside any

@stef-levesque How long will your sabbatical leave last?


how to reinit waterfall

i want to add a box at the top and first column of the waterfall ,and how can i do this? or how to reinit the waterfall.. thx

=.=


Add version 1.3.9

BWC tests are failing on 2.x due to the release of 1.3.8.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks @andrross , I think this step was automated, or not anymore?

Thanks @andrross , I think this step was automated, or not anymore?

@reta The version increment on the 1.3 branch is automated (#6163), but the change for BWC on the 2.x branch seems not to be automated (#4709 #5564).

@reta It is automated for the BWC version on the .x branch of the same major version (#6164) but seems to not look ahead to the next major version.

@reta It is automated for the BWC version on the .x branch of the same major version (#6164) but seems to not look ahead to the next major version.

Oh I see, thanks @andrross

Actually, it is automated for the main branch when a 2.x version is added. Seems there's just this one case of patch release for the previous major version that isn't automated.

@andrross do we need to bring this to 2.5 and main as well?

2.5 yes (#6176). main no because it doesn't have any support for the 1.x versions.


Upgrade Argo CD Operator to pick up TLS race cond fix

What type of PR is this?
/kind bug
What does this PR do / why we need it:

Update argocd-operator dependency to latest from master branch, to pick up https://github.com/argoproj-labs/argocd-operator/pull/1469

Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
N/A
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

/retest
Mainly failed on cluster setup, but also failed on:
        --- FAIL: kuttl/harness/1-063_validate_statefulset_restart (1264.11s)
        --- FAIL: kuttl/harness/1-083_validate_resource_customization_subkeys (1203.94s)
        --- FAIL: kuttl/harness/1-042_validate_status_host (1203.90s)
        --- FAIL: kuttl/harness/1-062_validate_extra_config (1203.96s)
        --- FAIL: kuttl/harness/1-058_validate_prometheus_rule (1206.35s)

/retest
 time="2024-08-14T19:48:19Z" level=fatal msg="Failed to run bundle: error waiting for CSV to install: timed out waiting for the condition\n"
{"component":"entrypoint","error":"wrapped process failed: exit status 1","file":"sigs.k8s.io/prow/pkg/entrypoint/run.go:84","func":"sigs.k8s.io/prow/pkg/entrypoint.Options.internalRun","level":"error","msg":"Error executing test process","severity":"error","time":"2024-08-14T19:48:19Z"}

/retest
        --- FAIL: kuttl/harness/1-080_validate_regex_support_argocd_rbac (1204.20s)
        --- FAIL: kuttl/harness/1-079_validate_vars_for_notificaitons (1203.15s)
        --- FAIL: kuttl/harness/1-063_validate_statefulset_restart (1248.45s)
        --- FAIL: kuttl/harness/1-039_validate_fix_argocd-tls-certs-cm (1203.32s)
        --- FAIL: kuttl/harness/1-052_validate_rolebinding_number (6.25s)
        --- FAIL: kuttl/harness/1-067_validate_redis_secure_comm_no_autotls_ha (329.75s)
        --- FAIL: kuttl/harness/1-104_validate_applicationset_tls_scm_volume_mount (126.80s)


Faker uses methods named the same as the class name

Hi
I'm currently working on using Faker for a project using PHP 5.3.2 and I'm running into an issue.
According to this documentation on PHP's constructor, PHP versions < 5.3.3 will use a method named the same as a class if it can not find a __construct method in that class.
This is an issue with Faker as some of the providers have a method in them with the same name as the class, which will cause these versions of PHP to call the method. e.g. the Faker\Provider\Address class has an address() method. It leads to errors such as:
PHP Fatal error:  Call to a member function parse() on a non-object in faker/src/Faker/Provider/Address.php on line 91
Defining a __construct method that just calls the parent constructor works, but throws a PHP strict standards notice:
PHP Strict Standards:  Redefining already defined constructor for class Faker\Provider\Address in faker/src/Faker/Provider/Address.php on line 93
Is there a solution that allows you to keep your architecture but ignores this php backwards compatibility shim?
Cheers,
Chris.

I thought the problem only existed with method names exatcly like the class name, which is not the case here, since the method names start with a lowercase character.
I'm surprised by the strict standars notice. I'll do further tests, and if there is no other simuton but to rename providers (probably from Address to AddressProvider), we'll have to do it.

I have the same issue, are there any plans to solve this?


Initial and maximum memory settings do not work

The settings "Initial memory" and "Maximum memory do not seem to have any effect on the memory usage of the game. I've tried it with a few different values but the memory usage pattern of Terasology on startup has remained around the same.
System information if it's useful:
OS Name:                   Microsoft Windows 10 Home
OS Version:                10.0.14393 N/A Build 14393
OS Manufacturer:           Microsoft Corporation
OS Build Type:             Multiprocessor Free
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
[01]: Intel64 Family 6 Model 78 Stepping 3 GenuineIntel ~2492 Mhz
BIOS Version:              American Megatrends Inc. F.42, 22/06/2016
Total Physical Memory:     8,103 MB
Virtual Memory: Max Size:  18,855 MB
Java 8 Update 111

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Tried to reproduce the and all seems good - as @Cervator already said =)
We'll update the labels to clarify that "Max Memory" is actually the maximum heap size for the JVM. We'll call this "Max Game Memory" in the future to make clear that this is not an upper bound to what the JVM will use (but only the game).


There is no parameter --num_scales

When I start to train the scn models, it has bug and show "trainer.py: error: the following arguments are required: --num_scales ", but I could't find this  parameter in the train.py. So how can I solve this problem?

Sorry I have found  it in scn.py  and  I'll set  it as in the paper.Thanks!


Creating a fixed width grid based on pixels instead of percentage.

I'm trying to create a fixed cell width and height independent of window size. I'd like to create a 20px by 20px grid where each widget can fluidly move up and down in 20px increments. I'm able to get the height fixed by passing cellHeight: 20, but width doesn't seem to work very well. This is what I'm passing into options:
var options = {
      float: true,
      width: 53,
      acceptWidgets: '.grid-stack-item',
      cellHeight: 20,
      verticalMargin: 0,
      disableResize: true,
      alwaysShowResizeHandle: false
  };

  $('#grid-1').gridstack(options);

I tried cellWidth: 20, but that didn't seem to work. I also modified the scss script to be calculated based on px instead of %.
.grid-stack > .grid-stack-item {
  $gridstack-columns: 60;

  @for $i from 1 through $gridstack-columns {
    &[data-gs-width='#{$i}'] { width: $i * 20px; }
    &[data-gs-x='#{$i}'] { left: $i * 20px; }
  }
}

This snaps grid items into the correct 20px grid locations, but horizontal dragging becomes less smooth. Is there something else I need to add to the .scss script so horizontal dragging maps to grid width? Or do I need to do something different for the width or cellWidthparam? I'd like to get away from any % calculations :)

@cblokker Did you ever figure out how to do this? I am also looking at how to implement widths based on pixels and not percentages. My won't even snap to the correct places. It seems to be off by a large margin.

This is the function that is causing me grief:
    GridStack.prototype.cellWidth = function() {
        return Math.round(this.container.outerWidth() / this.opts.width);
    };

In the case of a pixel based width, along with a css that defines the columns in pixels, I need this function to return only this.opts.width.
I think we would either need some smarts in gridstack to determine what model it was in (% or px) or have a configuration option to tell it what mode.

Would love this, as there is a problem with viewing this in 4k screens, sometimes you want the web page to be wide, in which case you have to set up a new grid where the sizes are pretty much halved, and then when you resize the page down to a resolution others will use the widths are way too narrow.

I too would very much like to see this feature being supported by gridstack

Same here, I would love to see that in grid stack.
In the meantime, I was able hack it for my simple use case (one row, disabledOneColumnMode, height=1)
To force all tiles 100px wide (all my tiles are width=1, it should not be hard to do for any width) :
In the CSS
    $tile-width: 100px; /*100px*/     .grid-stack-item[data-gs-width="1"]  { width: $tile-width; min-width: $tile-width; max-width: $tile-width; }     .grid-stack-item {       @for $i from 1 through 20 {         &[data-gs-x='#{$i}'] { left: $i * $tile-width; }       }     }
constructor:
            const width = $('.grid-stack').outerWidth();             this.grid = $('.grid-stack').gridstack({                 auto: false,                 cellHeight: 80,                 disableResize: true,                 verticalMargin: 0,                 width: Math.round(width/100),                 height: 1,                 disableOneColumnMode: true             }).data('gridstack');
Reset the width in the resize handler
    resizeHandler() {         const width = $('.grid-stack').outerWidth();         const grid = $('.grid-stack').data('gridstack');         grid.setGridWidth(Math.round(width/100), true);     }
the hack is to force the cellWidth to 100px (as set in the CSS) by adjusting the width  (column count) based on the current grid width (px).

You can set cellHeight to whatever you want, and you can use Javascript to determine dimensions before creating a container and passing options to gridstack. In this fashion, you can set whatever ratio you want and even set specific pixel-based widths and heights.


Typing for useRouteMatch is not nice to use in TS strict mode

currently typing for useRouteMatch always returns match<Params> | null which is not ergonomic when using TS strict mode.
To satisfy the compiler , we now have to write:
  const orgDetailPath = '/organisation/:organisationId'
  const { params } = useRouteMatch(orgDetailPath) as match<{
    organisationId: string
  }>

or
  const orgDetailPath = '/organisation/:organisationId'
  const match = useRouteMatch<{
    organisationId: string
  }>(orgDetailPath)
  if (!match) {
   return null
  // we could throw some kind of an error here that the component is used in wrong url path, but that is superflous, because if we omit this error handling the error on the destructuring is descriptive enough for any average programmer to understand
  }
  const { params } = match

It would be better, if the typing always returned the provided generic argument without nullability. Many other libraries do it like that when interfacing with type unsafe interfaces.
The routing with string paths cannot be made 100% typesafe as the TS compiler will never understand the route match strings. So in this case it is okay to rely on the user to trust that the generic argument is correct in the current context.

The types for 5.x aren't maintained here. Consult the support resources for DefinitelyTyped, which is where they are written and maintained.


Deploy to IIS

Hi,
I have created a small APP using the Angular2 Template. Works great and was very easy to use.
I published the App to Azure, just as an initial prototype test and it worked great using an Azure App Service deployment.  However, I now need to deploy the app to an IIS Server, but I can seem to work out how to do it.
When I xCopy the after creating a publish using dotnet publish
I place the files in the Application Virutal Directory, if I browse to the url i.e. http://localhost/someapp
I get an error page saying page can't be found
However, when I browse to an api call i.e. http://localhost/someapp/api/controller/action
I get a result.
What is the additional step I need to take to ensure the application is rendered?
when I compared the deployed files i.e. between App Service & Publish
it appears the same files are included  however, I get the feeling there is an additional configuration step for IIS that I am not aware of ?
I have created the APSNETCORE App pool and set it as UnManaged.
Do I need to set default document to something?
I have also gone through this guide
http://webmodelling.com/webbits/aspnet/aspnet-deploy-iis.aspx   and ensured all has been set up correctly in IIS

A couple of things you might want to try:

Temporarily disable server-side prerendering (remove the asp-prerender-module tag helper from your Index.cshtml) and check that things are fully set up to work as far as client-side rendering is concerned. If they aren't, you'll probably get more easily-debuggable errors, since it's running in your browser rather than remotely.
Try setting up the correct base URL - in your _Layout.cshtml, change <base href="/" /> to <base href="/someapp/" /> or similar
Similarly, try updating the baseUrl option in your boot-server.ts. Unfortunately there's no way for us to infer this automatically at runtime, so you will have to enter it yourself here.

I'll close this because AFAIK there's no bug here, but if you track this down and think there is a problem in the templates/libraries in this repo, please reopen with details! Thanks.


Parser: Move availability macro definition cache to a request

Update the availability macro definitions caching logic to use the request evaluator.
Follow up to https://github.com/swiftlang/swift/pull/76794.

@DougGregor If you have guidance to improve this request it would be appreciated. I'm not convinced by the use of the ASTContext as input and the related simple_display.

@swift-ci Please smoke test


[FEATURE] Set bot channel & Announcement channel through admin commands

Saw someone did this work, but we need to combine work. Need to combine work for the usability of those channels.
#22 pull request here relies on this feature.

This is implemented


Redesign

Hi guys!
I've sent you an email about this. I added a bit more styling/features compared to the screenshot, I think the only thing missing is the ability to change the fireteam members (not quite sure what the best place is for that) and an update for the Help overlay.

This is absolutely beautiful.

Cool, glad you like it :)
Small note:
In /app/views/shared/header.htmlat the bottom there is some ugly inline Javascript to initialize the Material Bootstrap theme, which is needed for the PSN/XBL toggle. My Angular knowledge is limited, so I couldn't find the correct place to put that initialisation. There is also quite some commented HTML in the views, which is all the old styling that I used to build the new screens.

Since this is the first time either of us have received a pull request here, we are currently trying to figure out the best way for us to work together on this redesign before merging.
Some notes:
No need for a "Skilled player" box under the name, I dont want to assume someone is good or bad and there have been times where ive gotten my ass handed to me by really low KD players. Also its a lot of space we can use for other content.
I noticed you reduced the amount of "Last Matches", Currently we are showing last 25 games which I would like to keep. Maybe we can display two rows of 12 to compensate if all 25 on one row doesnt look as nice.
LOVE the weapon columns, I noticed you removed the weapon perk names which is good for design but without the names its hard to tell what im looking at at first glance. I think we can display two nodes per row like
(icon) name    (icon) name
(icon) name    (icon) name
We would have to truncate the names, ive had issues with this in the current design
The last match and stats tabs have to be added in, I am heavily attached to those lol
The new search bar is awesome, I really like it. But like you mentioned in the email we need to find another solution for searching the other characters (and recent players)
Maybe add another row under the main search bar with the two additional members search?
Im so conflicted on the intellect/discipline/strength, it looks so good but me and Steff have been way too obsessive about being able to cram as much as we can to save space.
What if they were horizontal rows?
Intellect    Discipline  Strength

Overall this design is amazing, hope you dont mind the tweaks.
I want this on live ASAP :)

Personally I disagree with Vlad on the match history. We have the win percentage, so the match history honestly doesn't need to go back that far. The win streak indicator is just perfect for this also, as even if we only show 10 games, they could still be on a 25 game win streak (the max we would be able to show). And really, that's all we need to know.
As for the skill indicator, I think that's a fork in the direction we want to go with the website. I mean, it's just based on KD, so we are adding our interpretation of KD to the website. I like how our original design showed everything you could possibly want to know, and then let the user decide how to interpret it. It's also a lot of wasted space when KD is sufficient (in my opinion).
I also share Vlad's view on the Intellect Discipline Strength bars. They just waste too much space currently. I'd like to keep the website without scrollbars on a 'normal' resolution of 1080p.

I prefer the way you did the weapon rows vs. the way we do it. I think the name of the weapon is more relevant at first glance than the weapon perk names, and those are still available by mousing over the icons. Also, any avid user of the website or player of Destiny would likely recognize the perk icons, so it's nice to use just the icons, and it saves some space.

http://staging.destinytrialsreport.com

For reference, we are working on SteffanLong/Marruk-master.

Hi guys,
First of all, I'm from the Netherlands, so I have a bit of a differente time schedule :). Wall of text incoming!
Responses
My thought behind the "Skilled player" box is the idea that in one split second you can see whether you should look at the stats further. Three players "With other qualities"?: it doesn't really matter if they use shotgun or sniper. That's what I like about the "hazards" as well: DTR adds value to the data, making it more of a service.
On top of that I would love players on LFG to use terms like "You have to be an Above average player on Destiny Trials Report". That gives people a way to talk about DTR on other places, and puts DTR in an "authority"-role.
I agree that basing this only on KD is a bit weak, but I think we could improve on that a lot (with statusses like "Getting carried" (low KD, high win %).
In my ideal DTR world (an iteration on this concept) is that you can fill in your own fireteam, and based on the opponents DTR instantly gives you an indication whether it will be an "easy", "close" or "hard" match. Together with a summary of the opponents (two shotguns, one sniper, three sunsingers) DTR does all the work for you. But that might be something for later.

I wasn't sure how much value 13 more matches added in that streak, so I decided to keep them on one line. One Trials run a week is usually no more than 7-12 matches, so a run of 25 would mean matches from over 2 weeks ago. I believe that together with "Streak" and "Win rate" that gives you enough information about the player.

The idea behind the weapon icons is that the players that are interested in them are at a level of "hardcore-ism" that they recognise them.  Plus, design wise it took away a lot of clutter on the screen.

The reason I took out the "Last match" and "Stats" is because I don't think I have ever used them :). Do you maybe have Google Analytics stats on how much they are being used? I like the way the tabs look on staging now, so we can always add them.
As an alternative thought to tabs: we could embed "Last match" as a clickable toggle in the "Last matches" area, and the "Stats" as a clickable toggle in the Win rate / Streak / Rating area. It gives a nice layeredness to DTR: "Want to know more? Click on it and we service you!". It would also be an option to add the DestinyTracker link to there ("WANT TO KNOW EVEN MORE? Ok here you go").

I've tried it with two additional search bars underneath, you can see that commented out in header.html I believe. I removed it because in my opinion it took to much space for the amount of times it's going to be used (I've found that around 4 in 5 times the fireteam based on recent players is correct).
The alternative I had in mind is some kind of "Edit" or "Replace" button in the emblem area of the two fireteam members. By clicking that button the Gamertag (or whatever it's called on PS :)) could turn into an Input field, and "Level 34 bladedancer" into a smaller suggested players dropdown.
Other
A general thought on "cramming as much to save space": I think it's good practice to evaluate every pixel to see if it adds value to the user, but not at the cost of everything. As long as the most important information is available on first glance the user won't mind scrolling every one in a while, that type of data is only interesting in some cases.

What I would love as a responsive solution to the fireteam members is that instead of scrolling up and down, you are able to swipe left / right. On a tablet you can already see two members, swiping shifts the focus to the middle and right one. A jQuery plugin that has all the functionalities we need is http://kenwheeler.github.io/slick/, but I haven't been able to implement it yet (I can't find the right place for the javascript initialisations with the dynamically loaded data).

What's the reason that the player hazards are commented out, does it still need styling? I loved it as a functionality (again adding value to the data).

Cheers!

Merged with branch SteffanLong:Marruk-master


caused by: editions are unstable on Rust Edition 2018 RC1

I'm trying  Rust Edition 2018 RC1 on my project and part of it was removing cargo-features = ["edition"] line. Most tools works correctly, but both cargo-outdated and my own project that uses cargo as a library stopped working and print:
error: failed to parse manifest at `/home/dpc/lab/crev/crev-common/Cargo.toml`
caused by: editions are unstable
caused by: feature `edition` is required

or similar when opening workspace:
cargo::core::Workspace::new(&self.manifest_path, &self.config)?;

It looks like for whatever reason they didn't get the memo that cargo-features = ["edition"] is not required anymore.
It's a bit surprising since cargo on beta channel works just fine. Do I have to configure something in https://docs.rs/cargo/0.30.0/cargo/util/config/struct.Config.html ?

I guess this is related: https://github.com/kbknapp/cargo-outdated/issues/141
But I've tried the workaround from there (calling cargo::core::enable_nightly_features()) and it didn't help.

This isn't a bug in cargo the executable, nor is it really a bug per se in Cargo-the-library but moreso a feature request. In any case seems fine to discuss here!
The problem here is that the version of cargo-the-library that cargo outdated is using thinks that to use the edition key you need to have the edition feature listed. The enable_nightly_features function is not intended to solve this, it's a red herring.
The beta channel of Cargo is different source code than what's published on crates.io, only the stable channel's cargo-as-a-library is published. The beta channel Cargo doesn't require the edition feature to use the edition key. The stable channel, however, does.
This isn't a problem purely isolated to cargo oudated, though. Any subcommand that links to cargo-the-library will likely experience this. I think the best thing we can do is to add a specific function which is targeted at relaxing the restriction that nightly features need to be listed to use them. This function would never be called by cargo-the-binary itself. It would, however, relax the restriction that features need to be explicitly enabled.
This won't be a bulletproof solution but it would hopefully be a better solution than what we have now! For the time being there's unfortunately not a great solution otherwise that I know of :(

See https://github.com/kbknapp/cargo-outdated/issues/141 which is relevant, specifically cargo-outdated needs to carry the cargo-features into it's Manifest struct and deserialize with it, since cargo the library requires it.

@joelgallant I believe that's the same issue as this one, which should be solved by my above comment

Just for reference [Solved]:
I had a similar problem:
cargo build --release
error: failed to parse manifest at `/home/bitcoin/electrs/Cargo.toml`

Caused by:
  editions are unstable

Caused by:
  feature `edition` is required

this Cargo does not support nightly features, but if you
switch to nightly channel you can add
`cargo-features = ["edition"]` to enable this feature

I ran:
rustup update
The rustc version was upgraded to 1.32.0
  stable-x86_64-unknown-linux-gnu updated - rustc 1.32.0 (9fda7c223 2019-01-16)

This solved the issue.

@gabridome tried right now, it still doesn't work for me if edition is specified in Cargo.toml. Are you sure you haven't it commented out?

@vn971 update to cargo-outdated 0.8.0, it was released a couple of weeks ago: https://crates.io/crates/cargo-outdated/0.8.0

@gabridome tried right now, it still doesn't work for me if edition is specified in Cargo.toml...

Yes. In my case worked.
`4.15.0-45-generic #48-Ubuntu SMP Tue Jan 29 16:28:13 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux``

@dwijnand @gabridome it works for me as well now, thanks! The trick was indeed to update cargo-outdated itself, e.g.:
cargo install --force cargo-outdated


Retain initial request parameters until the logout completion in OIDC logout flow.

Description:
In the OIDC flow, once you are logged in to an application and when you click the logout button, it will pop up the consent and if you click on 'NO', then it will redirect to oauth2_error.do which mention the below message.

access_denied
End User denied the logout request

But it will be better if have the initial request parameters until the logout completion.
So, it would be better to fix the code and redirect to the post logout URI (if it was sent in the initial logout request) when the user denies the consent.
Suggested Labels:
Type/Improvement

@piraveena Whats should be the correct\recommended approach according to the spec?

@darshanasbg , According to the OIDC session management spec, it is mentioned that during RP initiated logout flow [1],

At the logout endpoint, the OP SHOULD ask the End-User whether he wants to log out of the OP as well. If the End-User says "yes", then the OP MUST log out the End-User.

But it does not say anything about how to handle if the end-user declines it. So IMO, that is OP specific
[1]https://openid.net/specs/openid-connect-session-1_0.html#RPLogout


Two of the colab notebooks are essentially empty

Hi @pengKiina,
The last two notebooks listed in the readme are essentially empty. Was this on purpose?

@pengKiina are you still working on this project? Your article on Medium and first notebook caught my interest. the last two notebooks are still empty. I'm interested in exploring the application of this method to topics modelling.
Hope your well!

Hi guys,
My apologies, my full time job is a tour guide in Lapland Finland. The winter season was very busy and tiring. Now I am back to work on this long-planed project.


handle sc profile changes

we had a case for handling twitter account changes, but not soundcloud.

@facing-n this this looks good, just one thing is what happens if line 144 for getSoundcloudProfile fails? Should we add a catch for that?

if you look at names.js at line 385 you can see that getSoundcloudProfile is implemented in a try catch and will return null on failure which we handle by saying that the profile on soundcloud has been deleted and therefor the verification should be set to active = false

Ok thanks, looks good to me.


Request for Issue Template

I would like to suggest creating a standardized issue template for this repository. Issue templates provide a structure for issues, making it easier for contributors to provide the necessary information.
I propose that we create a template that includes the following sections:

Bug report
Feature request
Other

@surajnarwade

@adityasimant  If you are creating an Issue template then do it for pull_Request too

ok

@adityasimant feel free to raise a PR

closing this as we have the issue/PR template now


⚠ PrivateHD has degraded performance

In 2d14c83, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2434 ms

Resolved: PrivateHD performance has improved in 2cfa7a9.


New-AzDataLakeGen2Item Does Not Appear to Be Working Correctly

The commandlet does not appear to be working.  Whenever I execute it to copy source files from a local source to an ADLS Gen 2 account, nothing happens.  No files are copied to the ADLS Gen 2 account and no errors are thrown.  When I use the Output property of the newitem object I see output but it doesn't appear that a file was copied.  At this point it appears that the cmdlet may be non-functional.

Hi @briancuster, Thanks for reaching out. Could you please share the document which you are currently following?

Hi @briancuster, If your feedback on a specific document, I would recommend you to please open up a new issue under that document using the below button.

In case if your query is not a feedback on document, I would recommend you to please open a new thread in Microsoft Q&A forum which is better suited forum for such queries. We have our engineers and other community members who will be happy to assist.
We will now proceed to close this thread. If there are further questions regarding this matter, please continue to comment. We will gladly continue the discussion.
Thank you.


ALLOWED_HOST_LIST not honored

Description
i have added the configuration below and rebooted the server.  however i still get webhook can only call allowed HTTP servers (check your webhook.ALLOWED_HOST_LIST setting), deny '<IP_ADDRESS>(<IP_ADDRESS>:8080)' when i hit the "test delivery" button in the webhook.  It's either a bug, or a result of very (very) poor documentation
root@gitea:~# cat /etc/gitea/app.ini
[webhook]
SKIP_TLS_VERIFY = true
DELIVER_TIMEOUT = 10
ALLOWED_HOST_LIST = *

Gitea Version
1.22.3
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
git version 2.39.2
Operating System
Debian 12
How are you running Gitea?
binary
Database
MySQL

Are you sure it is the content of your "app.ini"? It can't be that simple.

Are you sure it is the content of your "app.ini"? It can't be that simple.

It means maybe you edited the wrong config file which is not used.
For a possible way, you can get it from log.

@yp05327
what log?  there is in nothing for gitea in /var/log/ and there is nothing in /etc/gitea besides this app.ini i created myself.  strange that the certs do work when i put them in that file.  how can i get this log?
the service status :
jay@gitea:~$ systemctl status gitea
● gitea.service - Gitea (Git with a cup of tea)
Loaded: loaded (/etc/systemd/system/gitea.service; enabled; preset: enable>
Active: active (running) since Mon 2024-11-04 20:16:47 EST; 12h ago
Main PID: 1123 (gitea)
Tasks: 8 (limit: 2315)
Memory: 119.2M
CPU: 1min 8.209s
CGroup: /system.slice/gitea.service
└─1123 /usr/local/bin/gitea web

nevermind i found it!  maybe this step shoudl be included in your cheatsheet documentation?  would have saved me a couple days of pulling my hair out
root@gitea:/# find / -name app.ini
/var/lib/gitea/custom/conf/app.ini

thank you

nevermind i found it! maybe this step shoudl be included in your cheatsheet documentation? would have saved me a couple days of pulling my hair out

When Gitea starts, it always tells the users which config file it is using, and you could also check it from the admin panel.
And the document also mentions it https://docs.gitea.com/help/faq#where-does-gitea-store-what-file

what log? there is in nothing for gitea in /var/log/ and there is nothing in /etc/gitea besides this app.ini i created myself. strange that the certs do work when i put them in that file. how can i get this log?

The "log" is also covered by documents: https://docs.gitea.com/help/support

nevermind i found it! maybe this step shoudl be included in your cheatsheet documentation? would have saved me a couple days of pulling my hair out

When Gitea starts, it always tells the users which config file it is using, and you could also check it from the admin panel.
And the document also mentions it https://docs.gitea.com/help/faq#where-does-gitea-store-what-file

LOL

can't you see why that's misleading to someone just starting out.   it wouldn't be hard to include this in your cheatsheet page...
Log files are usually located here... but you can also find them by using "find / -name app.ini" or watching the logs at startup
Who knows how many people have been in my position and just given up because of the poor documentation.  If this was my product, I certainly wouldn't want folks to lose interest in my product because I can't property document the most common issues.
I'm a technical dev ops cloud native engineer supporting open shift kubernetes, google cloud, and cloud foundry for a company with about 50 thousand applications being hosted.  There's no way this should have been this difficult for me if the documentation was on par.

Sorry but I don't see why it misleads .... the screenshot you shown is in a section called "Can't create repositories/files", and it says it is for "Arch package systemd" and it is an issue (wrong config) which makes files non-writable, I do not think it is related to you question.

It seems that your question is about how to find the correct "app.ini" file. There is a section above called "Where does Gitea store what file", it says that INI (config file) is CustomPath/conf/app.ini, the CustomPath just matches your case /var/lib/gitea/custom. It should be able to answer all the questions. If you search "app.ini" in that page, it should be the first matched result.
The path rules are complex because in history many contributors have various requirements , so a lot of patches were added to it, then the problem becomes that the path system is too flexible. It really depends on how you installed the instance ......

About how to find the "app.ini" manually, I think find / -name app.ini is not a good approach in a production server, there might be a lot of files or event shared network volumes, etc. So the proper approaches could be:

Use systemd to see the logs (it could assume every systemd user should be able to know how to use systemd log system).
Go to admin panel's config page, some details are there (hmm .... maybe this is the easiest approach but never mentioned)
Take a look at Gitea's data directory first, there is something.

The documents are also maintained by community contributors, It is always welcome to propose PRs to improve them. https://gitea.com/gitea/docs/pulls
ps: I am just trying to explain some details, and I am not a native English speaker, so sometimes my expression might not be accurate or right, that's why I didn't spend time on the documents ......

Yes that looks perfect! Thank you for doing that!  I thought to submit a PR
myself but thought it might be rude since I only just now started using
Gitea.  I’m loving it though.  Using it to help my wife learn how to
automate processes using CI/CD and IaC.
Also your English seems great to me

Jac Flesher
(586) 838-6885
On Tue, Nov 5, 2024 at 10:18 AM wxiaoguang @.***> wrote:

@jacflesher https://github.com/jacflesher I managed to propose a PR to
update the FAQ
https://gitea.com/gitea/docs/pulls/92
Does it look good to you?
—
Reply to this email directly, view it on GitHub
https://github.com/go-gitea/gitea/issues/32420#issuecomment-2457449173,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOWMHTALSAWGV232MQUNYGTZ7DOUHAVCNFSM6AAAAABRFN57GWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJXGQ2DSMJXGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


ENH: Fix compilation of shared libraries with opencl on windows

Closes #765

When reviewing, it helps to ignore whitespace changes: https://github.com/SuperElastix/elastix/pull/769/files?w=1


Update dotnet new doc for 2.2

Fixes https://github.com/dotnet/docs/issues/12129
Fixes https://github.com/dotnet/docs/issues/12130
Add 2.2 SDK info to doc while I was adding the --dry-run option and substring matching details. Also fixed a few minor formatting issues.

Thanks @scottaddie. I’ll try to review this shortly. I’ve been super behind schedule for all the updates needed for the CLI.
One thing I’ve been doing though is to remove the tabs and call out when things were introduced among with other style changes (see an example here: https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-build)
But it’s probably faster if we add yours using the old format and then I update it later. Thoughts?

@mairaw I'm in favor of using the old format for now. That gets the content out to readers quicker.

@mairaw Changes made. Please take another look.


clean up prose, clarify some content

Issue #, if available: https://github.com/aws-samples/rancher-on-aws-workshop/issues/51
Description of changes:
Clean up prose & clarify content throughout.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Build ID: 1267e9ec-c9af-4e4c-8747-f480b85c7a3e
Successful build. 1 expected infrastructure error (single region workshop), can safely ignore.
Preview link (Amazon login required): https://studio.us-east-1.prod.workshops.aws/preview/be7b2281-57e8-44d0-a40e-1b7ea0d88f2a/builds/1267e9ec-c9af-4e4c-8747-f480b85c7a3e/en-US

Are we sure the "title: " fields should be single quoted? Not sure if VS Code auto-updated or if it was purposeful for the workshop environment? There's 15 or 16 occurrences.

Are we sure the "title: " fields should be single quoted? Not sure if VS Code auto-updated or if it was purposeful for the workshop environment? There's 15 or 16 occurrences.

VS Code auto-updated that. It looks like it rendered fine in the preview build, so I assume Workshop Studio can use either.


Scrolling Issues

If you are in the Keytag Array or Medallions tabs, there is a lot of space above and below the images.
It is easy to "lose" the image, by scrolling too far, above or below.

This is fixed in 5.0.1


Multiplayer server crash on startup

Hello, I just updated Underground Jungle (underground-jungle-forge-20.1.0.jar) for my multiplayer server, so I added the new dependencie, Phantom Loader (library-forge-20.1.4.jar), but now the server crashes on startup.
Here is the crash report :
crash-2024-01-12_22.58.42-fml.txt

I see the mod registry is trying to access the particles engine, which is only available in the client. Will try to fix that.

Fixed by 2584fbbba30ba0aaeb8818d477bed58b0ffbfc69
It was caused by Forge client events not being set to only be invoked on the client.


Add notes about serialization to models docs

📚 Documentation
Following https://github.com/pytorch/vision/issues/2923#issuecomment-719650811

it would be useful to add a note in the models documentation pointing to https://pytorch.org/docs/stable/notes/serialization.html and saying that we guarantee compatibility with serialized state-dict, but not if the model is entirely serialized.

Apologies for not discussing prior to sending a PR. I thought that the task was small enough. Okay so I wanted to know if this a fix is still necessary ? @vfdev-5


Adding an option to extend the life of the token

Is there an option to extend the token time from one to two hours?

Unfortunately not - the token is a standard GCP oauth token and is fixed at one hour.


error: identifier "gpuAtomicMax" is undefined

/root/miniconda3/lib/python3.8/site-packages/torch/utils/cpp_extension.py:788: UserWarning: The detected CUDA version (11.7) has a minor version mismatch with the version that was used to compile PyTorch (11.3). Most likely this shouldn't be a problem.
  warnings.warn(CUDA_MISMATCH_WARN.format(cuda_str_version, torch.version.cuda))
building 'transformers.models.bloom.custom_kernels.fused_bloom_attention_cuda' extension
creating build/temp.linux-x86_64-3.8
creating build/temp.linux-x86_64-3.8/src
creating build/temp.linux-x86_64-3.8/src/transformers
creating build/temp.linux-x86_64-3.8/src/transformers/models
creating build/temp.linux-x86_64-3.8/src/transformers/models/bloom
creating build/temp.linux-x86_64-3.8/src/transformers/models/bloom/custom_kernels
/usr/local/cuda-11.7/bin/nvcc -I/root/miniconda3/lib/python3.8/site-packages/torch/include -I/root/miniconda3/lib/python3.8/site-packages/torch/include/torch/csrc/api/include -I/root/miniconda3/lib/python3.8/site-packages/torch/include/TH -I/root/miniconda3/lib/python3.8/site-packages/torch/include/THC -I/usr/local/cuda-11.7/include -I/root/miniconda3/include/python3.8 -c src/transformers/models/bloom/custom_kernels/fused_bloom_attention_cuda.cu -o build/temp.linux-x86_64-3.8/src/transformers/models/bloom/custom_kernels/fused_bloom_attention_cuda.o -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_BFLOAT16_CONVERSIONS__ -D__CUDA_NO_HALF2_OPERATORS__ --expt-relaxed-constexpr --compiler-options '-fPIC' -arch=compute_80 -std=c++17 -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE="_gcc" -DPYBIND11_STDLIB="_libstdcpp" -DPYBIND11_BUILD_ABI="_cxxabi1011" -DTORCH_EXTENSION_NAME=fused_bloom_attention_cuda -D_GLIBCXX_USE_CXX11_ABI=0
src/transformers/models/bloom/custom_kernels/fused_bloom_attention_cuda.cu(68): error: identifier "gpuAtomicMax" is undefined

1 error detected in the compilation of "src/transformers/models/bloom/custom_kernels/fused_bloom_attention_cuda.cu".
error: command '/usr/local/cuda-11.7/bin/nvcc' failed with exit status 1
make[1]: *** [Makefile:18: install-transformers] Error 1
make[1]: Leaving directory '/root/autodl-tmp/text-generation-inference/server'
make: *** [Makefile:2: install-server] Error 2

GPU：A100-80GB

oh BUILD_EXTENSIONS=False make install  than Solved my problem,But I don't know why

The detected CUDA version (11.7) has a minor version mismatch with the version that was used to compile PyTorch (11.3). Most likely this shouldn't be a problem.
It seems that you have a old version of Torch installed.

Yes, it works @OlivierDehaene Thanks for your help


Reduce nom deps

This contains an API change to remove dbg_dmp_rest, a function for which I could find no public users.
This allows disabling nom default features, which reduces the dependency footprint of the various rusticata crates (from x509-parser down) a lot.

Hmm, turns out this isn't useful yet, der-parser and x509-parser do require nom::multi which allocates.
There's also nom::bitvec, with its heavy dependency tree, which is exposed by the as_bitvec methods in der-parser.
(Requires mostly trivial changes to test, having nom default features disabled:
Remove the default features on the nom dependency (on der-parser and x509-parser as well), and update various crates to refer to the unreleased der-parser v6 (oid-registry, x509-parser, the latter requires a small patch due to der-parser API changes).

https://github.com/rusticata/x509-parser/compare/master...g2p:reduce-nom-deps
https://github.com/rusticata/oid-registry/compare/master...g2p:reduce-nom-deps
https://github.com/rusticata/der-parser/compare/master...g2p:reduce-nom-deps)


Operator registration does not check for keyword sensitivity

When registering an operator there are a set of keywords that cannot be specified in various places throughout the registration process. One example is adding an argument named "name" builds fine but on importing in python results in this error:
Traceback (most recent call last):
File "", line 1, in
File "/home/ubuntu/engine_arch/python/mxnet/init.py", line 28, in
from . import contrib
File "/home/ubuntu/engine_arch/python/mxnet/contrib/init.py", line 27, in
from . import autograd
File "/home/ubuntu/engine_arch/python/mxnet/contrib/autograd.py", line 29, in
from ..ndarray import NDArray, zeros_like, _GRAD_REQ_MAP
File "/home/ubuntu/engine_arch/python/mxnet/ndarray/init.py", line 26, in
from . import register
File "/home/ubuntu/engine_arch/python/mxnet/ndarray/register.py", line 169, in
_init_op_module('mxnet', 'ndarray', _make_ndarray_function)
File "/home/ubuntu/engine_arch/python/mxnet/base.py", line 629, in _init_op_module
function = make_op_func(hdl, name, func_name)
File "/home/ubuntu/engine_arch/python/mxnet/ndarray/register.py", line 162, in _make_ndarray_function
exec(code, None, local)  # pylint: disable=exec-used
File "", line 2
SyntaxError: duplicate argument 'name' in function definition
Here is the example operator registration:
NNVM_REGISTER_OP(min_ex)
.describe("some description")
.set_num_inputs(1)
.add_argument("name", "int", "description")
.set_num_outputs(1)
.set_attr<nnvm::FInferShape>("FInferShape", MinExOpShape)
.set_attr<nnvm::FInferType>("FInferType", MinExOpType)
.set_attr<FCompute>("FCompute<cpu>", MinExForward);

We should probably add upfront checks, or at least clean up the runtime failure messages so they are more clear, in addition to better documenting keywords to avoid.

@mxnet-label-bot[bug, Exception Handling]

@samskalicky do we have a list of such keywords?

@anirudhacharya no we do not


Cert present in certifi but not in requests

https://github.com/certifi/python-certifi/blob/master/certifi/cacert.pem#L5026
Can the Entrust G2 certificate be added to the requests certificate store?

I think the core issue here is that requests' certificate store is somewhat out of date. I propose we ship a 2.8.1 containing a cert store update, specifically the weak.pem bundle that ships with certifi.
@hombremuchacho Someone I presume was your colleague or another team member (@aehlke) suggested that certifi was missing a certificate you needed, so you couldn't use it as a drop-in replacement. Did you try using certifi.old_where()?

(To be clear, the bundle pointed at by certifi.old_where() is exactly the same as the bundle I'm proposing in #2810, so if that works for you I'll know it's a good choice to move forward with.)

@Lukasa Thanks, I'll test this now.


Binaries should be created for each backing store to make users from a CLI

Each BackingStore should incorporate a program in bin that allows creation, deletion, and so on, of users from the command-line, using that BackingStore.

It's pointless to make a MemoryBackingStore CLI tool because it will be too short-lived to be any good.  Even if it dumps the output at the end, it's no better than a test or the FBS example in 0.5.2.

a4d5851 has the CLI tool for the FBS.


for new developers there should be an easy and well documented way to build the system

I have spent quite some time on windows trying to build the system until I found unix make is needed and node/make will not work. For node-sass you need python etc...
Installation should be much easier and only depend on npm and make.

Now, we are using rollup for build process.

@idc-developer I would close this issue, since rollup build works fine in windows. Let us know if you try the new release.


Make return type to dictionary for discoverability of vault types

Extension to #123

Dictionaries are more usable than arrays because you can index into them without having to iterate through the array to find an id, and you can still get an array from them anyway if you want by using dictionary.keys

But in this context it is only used for if item exists, which is Array.Contains in Cadence.
getSupportedVaultTypes().Contains(t)
vs
getSupportedVaultTypes()[t]!=nil
or even
if let exists = getSupportedVaultTypes(t){
   if t == true{
  }
}

somehow I feel it doesn't make sense here.

array.contains still has to iterate through the array under the hood to find the value, right? That is a O(n) operation, but with a dictionary, it is an O(1) operation I believe.

yeah but it matters only for big N; here we have something like N<10 maybe on worst case ( on most cases N=1 ) ( even iterating can be cheaper for some N < x )

but it would matter for something like the NFT standard, and I want to make the function signatures as similar as possible for these kinds of methods so using them can be similar. We're going to add something like this for NFTs also


TEP-0115: Update git resolver doc to use "revision" and "pathInRepo" fields

Part of TEP-0115
Update Git Resolver documentation to use "revision" and "pathInRepo" fields to reflect the changes in TEP-0115: Add git revision resolution support in Git Resolver and Add core Reconciler testing and fake resolver framework functions

/assign @jerop

/cc @abayer

/lgtm

@abayer I'm not an owner in this repo so can't approve, switch places? 😂
/lgtm

/approve
=)


solaar: 1.1.9 -> 1.1.10

Description of changes
Update solaar to the latest release.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] 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

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

Result of nixpkgs-review pr 266085 run on aarch64-linux 1

3 packages built:

solaar.udev (logitech-udev-rules)
solaar
solaar.dist


Appending to annotations without collisions

A point brought up in the previous discussion about race conditions was that multiple processes cannot modify/append annotations simultaneously. Consequences of this include (1) that only one set of modifications can be performed on a set of annotations at a time, and (2) a given task (e.g. calculating poses) cannot be parallelized. This latter point has come to attention with the fact that recent experiments with multipass acquisition data require a significant amount of time to calculate poses (>~16-20 hr):
In [1]: experiment_dir = '/mnt/9karray/Sinha_Drew/20190111_spe-9_B' # ... next command errored out
In [3]: %run segmentation_pipeline.py $experiment_dir # Segment and annotate all 'adult' stage poses
Image channels to be segmented: ['bf', 'bf_1', 'bf_2', 'bf_3', 'bf_4', 'bf_5', 'bf_6', 'bf_7']
scanning done after 69.39152073860168 s
segmenting done after<PHONE_NUMBER>7813416 s
annotation done after 54436.36029171944 s
(Note: (1) this was annotation of a portion of the whole dataset, and (2) back of the envelope calculations put pose calculation on the order of several seconds, suggesting that this isn't necessarily an optimization issue with pose calculation)
The proposed feature adds some infrastructure that would allow multiple tasks to be performed on a set of annotations at once. First, a function for "diff'ing" experiment annotations (diff_annotations) allows recognition of fields that change after a set of modifcations. Second, a set of functions mirroring the current read/write infrastructure uses diff_annotations and merge_annotations to safely append to a set of annotations. In particular, this appending functionality uses file locking to capture the latest set of annotations and rewrite the annotation file without fear of collision.

I think we'll stick with the pickle files for now, so we do need some synchronization. To discuss: instead of providing an append option, would it just be better to convert "write_annotations" to do a "read / merge / write" operation instead of a pure write?
In addition, for a "read / merge / write", I think the lock needs to be acquired for the whole operation -- not just the write part -- in order to ensure consistency. (Correct?) But fcntl has some weird weaknesses (like closing the lock if some code in a called function opens and closes the same file) that might crop up if we're not super careful. Also note that for this reason, fcntl locks are decidedly not threadsafe.
Last, what's the use case for the diff function?

Glorious. Let me talk about the diffing first, and we'll see if this makes for a good workflow (or it'll come out that I'm trying to make a hammer also a screwdriver).
To do the "merge" part of the read-merge-write, one needs to be able to only overwrite fields that make sense to overwrite. For example, for parallelizing pose calculations (e.g. by position), one would want each job to only update those pose fields it was working on. A naive dict.update/load_data.merge_annotations doesn't work here as each job would try to update based on its copy of annotations, but a given copy is out-of-date w.r.t. positions it wasn't working on. Having each job merge based on merge_annotations would yield a final set of annotations only containing a subset of pose annotations made by the last job that wrote out to file.
The diff functionaliity handles this situation by isolating those fields that were modified/appended (w.r.t. a reference set of initial annotations) allowing one to do the merge with a simple call to merge_annotations (i.e. with the latest set of annotations as one parameter and the diffs as the other).  In reality, the latter steps are what append_annotation_file does.
Regarding the read/merge/write vs. write, I'm on the side of read/merge/write  I do think that the read-merge-write would essentially become standard for what we do now (e.g. one could cast experiment_annotator related tasks in the read-merge-write framework and I think that would work swimmingly). In general, it's probably good to assume that read/merge/write always needs to happen. I do think there might be uses for bulk writing annotations. E.g. Will or I want to clear out poses from a set of annotations so that we can do reliability with the undergrads, and I just want to do that quick and in bulk. But, technically, that's just another read-merge-write problem anyways; it should be straightforward based on the other infrastructure.
As far as the thread-safety, good call. I have to think a little more on that.

Closing after discussion.


Write a Scala library for parsing dates

There are lots of different date formats in the Calm data – the Calm docs list the supported formats, and there are plenty more extra “features” that we’ve introduced.
We need to be able to parse these strings and turn them into a canonical representation.

Some possible thoughts on how we might do this:

Elasticsearch 5.2 supports date range fields, so we could use this to store ranges.
I’ve done a bit of work with pyparsing and have a bunch of existing code that we could adapt.
parboiled2 was mentioned in my interview (not by me) – I’d enjoy a chance to actually use it!


Region - region.id

Hello !
Thank you for this amazing app :-)
I encounter a small problem with "wavesurfer.on('region-in', function (region)" I use to know which region is playing and so be able to display a series of corresponding information (subtitle, notes...) that I extract from an xml. Everything is perfect (id of the current region is correct) with a classic url (like http://somesite.com/somepage.html).
However, when calling the application with the url followed by parameters, like http://somesite.com/somepage.html?ID=5 (I need this to kwow the xml where I can find the url of the mp3, the regions, and so on, ), the very first region.id is always the number of the parameters (here "5") and the following region.id are something  like wavesurfer_s53sq0fl06o, wavesurfer_oerbr9apuv8, ...
The other infos are ok, for example : Start = 18 End = 23 Loop = false Drag = false Allow resize = false Color = hsla(200, 100%, 30%, 0.1), everything is perfect for that
thank you very much for your help
Wavesurfer.js version(s):
https://cdnjs.cloudflare.com/ajax/libs/wavesurfer.js/1.3.0/wavesurfer.min.js
https://cdnjs.cloudflare.com/ajax/libs/wavesurfer.js/1.2.3/plugin/wavesurfer.regions.min.js
Browser version(s): Chrome (Version 56.0.2924.87)
Code needed to reproduce the issue: If needed, with pleasure :-)

@jj-quintin cant help you here but always try the latest wavesurfer (1.3.2) before reporting issues.

I won't know where the issue with the weird 5 region id is coming from but the wafesurfer_xxx type ids are intentionally created random ids by wavesurfer. They are created automatically if no custom ids are provided when adding a region.

Thank you for those informations that helped me find the problem, which was on my side. Indeed, the id variables sent when creating the regions were undefined :-/ What surprised me is to have first id.region exactly the value of the parameter transmitted with the url of the application (I have to look on this weird stuff). Sorry for the inconvenience and thank you again for this great application


Create CONTRIBUTING.md guide

Adds a contributing guide, inspired by the rails contributing guide, https://github.com/rails/rails/blob/master/CONTRIBUTING.md
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

will do!


rtabmap/rtabmap_ros: 0.19.3-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repositories rtabmap and rtabmap_ros to 0.19.3-1:

upstream repository: https://github.com/introlab/rtabmap.git, https://github.com/introlab/rtabmap_ros.git
release repository: https://github.com/introlab/rtabmap-release.git, https://github.com/introlab/rtabmap_ros-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.8.0
previous version for package: 0.17.6-0

Holding for kinetic sync


fixes tests, follow up commit of 195cf158a8eb83403b06273723228ebbf4c0a657

_raw column is now forbidden inside a where clause

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@db01a4f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #5850   +/-   ##
=========================================
  Coverage          ?   82.96%
  Complexity        ?    12083
=========================================
  Files             ?     1423
  Lines             ?    46381
  Branches          ?     5717
=========================================
  Hits              ?    38481
  Misses            ?     5226
  Partials          ?     2674

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 db01a4f...63939a5. Read the comment docs.


walltime app function parameter modifies task_record func_name

Describe the bug
When the walltime special parameter is passed to a Parsl app the task_record['func_name'] parameter is set to "wrapper" instead of to the function's name.
To Reproduce
Launch the code below using parsl version 1.2.0:
import parsl
print(parsl.__version__, flush = True)
from parsl.app.app import python_app, bash_app
from parsl.config import Config
from parsl.executors import HighThroughputExecutor

@python_app
def test(stdout='std.out', stderr = 'std.err', walltime = 5):
    from time import sleep
    sleep(1)
    Fail = 1/0
    return 'Hello'

def retry_handler(exception, task_record):
    print(task_record['func_name'], flush = True)
    return 1

if __name__ == '__main__':

    config = Config(
        executors = [HighThroughputExecutor()],
        retries = 2,
        retry_handler = retry_handler
    )
    print('Loading Parsl Config', flush = True)
    parsl.load(config)

    fut = test()

    print(fut.result())

It will print "wrapper" if the walltime parameter is present and test otherwise.
Expected behavior
The code should print the function's name (test).
Environment

OS: Linux
Python version: 3.10.4
Parsl version: 1.2.0

Distributed Environment

Where are you running the Parsl script from ? Local
Where do you need the workers to run ? Local

This bug is also present in more recent parsl master at 50044b7f67cbe34ec0c612a9f16507b1383e53a4
This behaviour also shows up in log messages:
parsl.dataflow.dflow:929 submit INFO: Task 0 submitted for App wrapper

should show the app name, but instead shows wrapper.

@avidalto this should be fixed by https://github.com/Parsl/parsl/pull/2450

Thanks! It is working after the fix


Questions about the files in ".txt" format used to train the "shift" model

I noticed videos from AudioSet were used to train the "shift" model. In "shift_params.py", there are params such as:
train_list = '/data/ssd1/owens/audioset-vid-v21/small_train.txt',
test_list = '/data/ssd1/owens/audioset-vid-v21/small_train.txt',
train_list = '/data/scratch/owens/audioset-vid-v21/train_tfs.txt',
test_list = '/data/scratch/owens/audioset-vid-v21/test_tfs.txt',
However, I didn't find any available files in ".txt" format at the website : https://research.google.com/audioset/
Could you provide those ".txt" files to train the "shift" model? Thank you for your help! Looking for your reply:)

Same question.

Same question.


Writing to amazon s3

Since soundfile works with file-like objects, can it write to s3? Currently I can get it to read from s3 using the following:
import tensorflow as tf

with tf.io.gfile.GFile('s3://mybucket/foo.wav', "rb") as fh:
    y, sr = sf.read(io.BytesIO(fh.read()), always_2d=True)

As long as the DFile object supports the usual write, seek, tell, etc., this should be no problem.

Great. So it sounds like I should be able to do something like:
import tensorflow as tf

with tf.io.gfile.GFile('s3://mybucket/foo.wav', "w") as fh:
    sf.write(fh, foo_array, 44100)

Looks like you can't write from s3 and then write using soundfile. If I do:
import tensorflow as tf

with tf.io.gfile.GFile('s3://mybucket/foo.wav', "rb") as fh:
    y, sr = sf.read(io.BytesIO(fh.read()), always_2d=True)

# do some manipulation of y

with tf.io.gfile.GFile('s3://mybucket/foo2.wav', "w") as fh:
    sf.write(fh, y, sr)

I get the following error:
From cffi callback <function SoundFile._init_virtual_io.<locals>.vio_get_filelen at 0x7f7d7711d710>:
Traceback (most recent call last):
  File "/home/ubuntu/anaconda3/envs/.../lib/python3.7/site-packages/soundfile.py", line 1196, in vio_get_filelen
    curr = file.tell()
  File "/home/ubuntu/anaconda3/envs/.../lib/python3.7/site-packages/tensorflow/python/lib/io/file_io.py", line 189, in tell
    self._prewrite_check()
  File "/home/ubuntu/anaconda3/envs/.../lib/python3.7/site-packages/tensorflow/python/lib/io/file_io.py", line 87, in _prewrite_check
    compat.as_bytes(self.__name), compat.as_bytes(self.__mode))
  File "/home/ubuntu/anaconda3/envs/.../lib/python3.7/site-packages/tensorflow/python/util/compat.py", line 87, in as_bytes
    (bytes_or_text,))
TypeError: Expected binary or unicode string, got PosixPath('s3:/mybucket/foo2.wav')

If I try to read from s3, perform some manipulations and then write to disk using soundfile, I get a seek error.

Turns out the problem was that I was using pathlib for the path instead of a string. Works now. Apologies for reopening.


--skip-bav doesn't skip open redirect scanning

Describe the bug
As I understand using the flag --skip-bav should disable the scanning of open redirect. While testing I discovered that this scanning is still being done (I think)
> dalfox url 'https://ditu.google.com' --skip-bav
[*] 🦊 Start scan [SID:Single] / URL: https://ditu.google.com
[G] Found Open Redirect. Payload: https://ditu.google.com?pleasedonthaveanamelikethis_plz_plz=pleasedonthaveanamelikethis_plz_plz
[G] Found Open Redirect. Payload: https://ditu.google.com
[G] Found Open Redirect. Payload: https://ditu.google.com?pleasedonthaveanamelikethis_plz_plz=pleasedonthaveanamelikethis_plz_plz
[G] Found Open Redirect. Payload: https://ditu.google.com
 ⠦  [SA: ✓ PA: ✓ ] Waiting for parameter and static analysis

Environment

Dalfox Version: v2.3.6
Installed from: go-get

Thank you for submit issue!
I tested it in the same version and --skip-bav didn't deliver the actual bav request. but Perhaps the problem is caused by not checking --skip-bav at the 943~945 line I'll fix it quickly!
https://github.com/hahwul/dalfox/blob/1f32f3494e1aa3312f84b3e2a836eb61a9ae9aac/pkg/scanning/scan.go#L944

[ ] check --skip-bav option Or modified to check only in bav request
[ ] Issue of detected unconditionally when scanning google domain

Hello again @pdelteil
I added code for #215 #217. I don't know if it was a good response, but I'd like to ask you to test it.
Test
$ git clone https://github.com/hahwul/dalfox
$ cd dalfox
$ go build

version check
$ ./dalfox verison
v2.3.7-dev

Test...
$ ./dalfox ~~~

Hello there @hahwul.
It works great!

Closing the issue because released version 2.3.7. If there is any other problem, please re-open it! Enjoy your day!


WebAssembly global not defined in extension on VSCode 1.86.2

I'm trying to run WASM in my extension, but the WebAssembly global is undefined, so I can't compile the module.
Commit: 903b1e9d8990623e3d7da1df3d33db3e42d80eda
Date: 2024-02-13T19:41:37.860Z
Electron: 27.2.3
ElectronBuildId: 26908389
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.1.0-18-amd64````

The WebAssembly global is defined, but it contains no functions. WebAssembly.compile is null.

Let keep the discussion in the VS Code issue.


Added Frontend Project Compare Page Template #4806

Title and Issue number

Frontend Project Compare Page Template #4806
Close #4806

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121
Kindly merge it.


Example for unioning source data

@fivetran-joemarkiewicz Yes, I need to update the readme. I wanted to make sure we were happy with the approach. I can do that and move the macros to fivetran-utils on Monday.

Hi,
any chance for this to get merged?
It is such a waited for feature 😄

@lizdeika I hav my calendar blocked this afternoon to hopefully finish this up! 😄 🎉

@lizdeika I will be merging this branch and cutting a new release v0.4.0 of the dbt_shopify package. This version can be expected to be live on the dbt hub by the end of today or early morning tomorrow. I hope this release helps you, let me know if you have any questions!

Hi all!
I have run into one issue when one of the shopify schemas does not have one table.
I have opened a PR which should fix the thing ( not tested, not idea how ): https://github.com/fivetran/dbt_fivetran_utils/pull/33


Compatibility is incorrectly reported on vcpkg.io

Describe the bug
Package compatibility is incorrectly reported on https://vcpkg.io/en/packages. For example, z3 shows supporting arm-uwp and x64-uwp despite the package not supporting UWP:

https://github.com/microsoft/vcpkg/blob/cf4ebef2294e164875ce17d7937f44d3e3ea156e/ports/z3/vcpkg.json#L2-L7
Environment

OS: All
Compiler: All

To Reproduce
Steps to reproduce the behavior:

Visit https://vcpkg.io/en/packages
Search for z3
Observe UWP compatibility listed

Expected behavior
Correct compatibility information should be shown.
Failure logs
N/A
Additional context
N/A

Now Compatibility field has been removed, see details from https://github.com/vcpkg/vcpkg.github.io/pull/154.


Provide Tool Tip with instructions

Not sure how to best achieve this because we can't link to github readme from within the app.
Is there some way to embed a markdown file with instructions in the app page with a local reference?

Maybe this isn't necessary since the sample transfer file has an example.
Could we include commentary in the sample file that is ignored by the CSV parser?

Could we include commentary in the sample file that is ignored by the CSV parser?

The fast csv parser support commenting:
https://c2fo.github.io/fast-csv/docs/parsing/options/#comment

Is there some way to embed a markdown file with instructions in the app page with a local reference?

There are react components to display markdown. But I dont know how nice it will look if we render them into a tooltip. We could try it though.
But then again I think the app is fairly straight forward to use. There are more complex gnosis safe apps without any instructions whatsoever.

Maybe we can just try and go with some comments in the sample file.

closing this as the new help modal solves this problem.


Refactor hostname logic

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ali Ahmed seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@aahmed-se - can you please fix the travis ci issue?


koord-scheduler: support kubelet reserved cpus

Signed-off-by: Joseph<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
The koord-scheduler plugin NodeNUMAResource supports kubelet reserved CPUs, and these kubelet reserved CPUs are no longer allocated when allocating to prevent CPU stacking.
the correlative PR #592

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

/lgtm

/approve


Manually set the artifactId for publishing

Transferred from FabricMC/fabric-example-mod#233.
By default, Gradle will use the project name configured in settings.gradle (rootProject.name), after which it will default to the project's directory name. The directory name is very often equal to the capitalised mod name. This can be problematic for GitHub Packages, since capitalised artifacts are not allowed. This PR prevents that from happening.
Edit: See this stackoverflow question for the error GitHub Packages returns.

It seems that this was implemented in the example mod in fabric-example-mod/commit/c8dba6d after my original PR was closed. That probably means this PR can be merged as well.

What am I saying, apparently this was also added to this repository outside of this PR. Can't complain 🤷♂.

Tfw #84 was merged instead.


some bugfixes and adjustments to new aws event datastructre

Hi Taylor,
thanks for publishing your lambda functions. I had to fix some parts, to make it work in our current setup. Hope they help you too.
Br
Christian

Sorry, for the late reply. I will take a look.

The variable akid is never used in the python file.
For the region, I just removed the hardcoded definition. I fetch the region from the event the function receives.


Update rmlocal.script

Added --delete-after to prevent accidental deletions if the transfer is interrupted. Did not add to config since rclone_options is used for copy also and --delete-after should only be used for move.

I may be wrong on this... researching.

According to Rclone docs it only removes the file if the copy has ended successfully

Yeah I closed because I realized.


Training is too GPU - power hungry

While using gtx 1070 8gb (over 6gb free) I can only train with batch_size=1 which seems ridiculous. Maybe code is somewhat not optimized or I should expect these kind of drawbacks because of my gpu? Any suggestions? Even setting MEMORY_FRACTION = 0.95 does not help...

You may need to check if there is anything wrong since even using one GPU with 12 GB graphic memory we can train the model with a batch size of 8. Besides, you may also try our ENet-Label-Torch model which is extremely lightweight and can achieve around 68.8 in CULane testing set.

what should I do when I use the CULane dataset in SCNN-Tensorflow?  I cpoy the list/train_gt.txt&val_gt,txt.  to the dataset path,but I always get the Error not found :xxx/xxx.MP4/xxx.jpg
For example,Not found:driver_23_30frame/05170932_0736.MP4/02780.jpg. but I am sure that dataset contains the folder


Updated import statements

Imports in index.js now refelect latest react-native appropiate syntax.
See https://github.com/facebook/react-native/releases/tag/v0.25.1

Requiring React API from react-native is now deprecated - 2eafcd4 0b534d1
Instead of:
import React, { Component, View } from 'react-native';

you should now:
import React, { Component } from 'react';
import { View } from 'react-native';

Thanks @buddingmonkey


Error's compile with Spring boot 2.0.3

Hello,
I was trying to migrate to the version of Springboot 2 but without success, errors come out in source code.
It would be great if I could migrate.
Thank you

Hi max,
I just had a look and it seems ResourceCondition has been changed. I'll create a branch and push a fix for Spring Boot 2.0.3

Hi max,
I have created branch using-spring-boot-2 of the project using Spring Boot 2.0.3. Please let me know if this works for you and I will release version 2.0.0 of the project.
Ed

Thanks So Much...
this Work is great... :)


Test and fix line buffer utils

current line detection was off on the first position of a following
line
find_current_line_end was not correctly aware of CRLF platforms
Increase of coverage for some old functions

nice