Compile

Thanks for the share, the only problem i found is while compiling the apk using APK Easy Tool.

the problems source come from the style.xml at Rafel-Rat-main\story\res\values-v28
lines:
styles.xml:18
styles.xml:20
styles.xml:8
styles.xml:10
Edit:
I just found out the problem, it is from the whole APK easy tool and not the decompiled apk, since i decompiled a fresh new IsntaStorySaver.apk v1.0.1 and recompiled it instantly but it failed ...
I guess the best solution to go with the first method

ok.. First Uninstall your apktool and download this Apktool v1.57 Here
Now Turn off your windows defender real-time protection and download again the zip file from my github repo and compile again


Aurora crashes on startup with any GHUB

Expected behavior:

Aurora should start
Actual behavior:

Aurora crashes on startup
How often does this problem occur?

This problem occurs every time
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Start Aurora
Wait
Aurora crashes after 1 second or doesnt start

What is the version of Aurora you are experiencing this issue with?

My version is 0.6.3-488
Does the previous version of Aurora have this issue?
The previous version, (0.6.3), have this issue.
Please include any logs, screenshots, or files that are related to this issue:

Log: https://pastebin.com/8bWuHd83

this version fixes this issue

Thank you very much it worked! But i'm curious why in download page corrupted version is available. This makes a lot of trouble for new users.


Sentinel attacks players even when in spectator

Title says it. On my server I have this NPC that gets triggered when you hit him. When you then go into spectator mode, the NPC doesn't go away or anything, it just keeps smashing the air where you're flying. I believe this is a bug and it should simply act like the player is not there.
https://gyazo.com/2d3d59bceae66e0e40f68a194256236a

Patched for build DEV-161+ on http://ci.citizensnpcs.co/job/Sentinel/
Thanks for the report!

Amazing! I will test it out right now.


termite: Add support for macOS

Replace fetchgit with fetchFromGitHub now that it supports fetching
submodules. Remove unnecessary postPatch to add <math.h> as termite
already includes . Add a patch to include <errno.h> and remove
the --as-needed flag from ld on macOS.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build termite

Success on x86_64-linux (full log)
Attempted: termite
Partial log (click to expand)
gzipping man pages under /nix/store/iq7a2jcsjr6li2h8jqiy3nnl4r9n687v-termite-13/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/iq7a2jcsjr6li2h8jqiy3nnl4r9n687v-termite-13/bin
patching script interpreter paths in /nix/store/iq7a2jcsjr6li2h8jqiy3nnl4r9n687v-termite-13
checking for references to /build in /nix/store/iq7a2jcsjr6li2h8jqiy3nnl4r9n687v-termite-13...
shrinking RPATHs of ELF executables and libraries in /nix/store/lyb2fjin82an69z890a8c0fb7w2xj6q2-termite-13-terminfo
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/lyb2fjin82an69z890a8c0fb7w2xj6q2-termite-13-terminfo
checking for references to /build in /nix/store/lyb2fjin82an69z890a8c0fb7w2xj6q2-termite-13-terminfo...
/nix/store/iq7a2jcsjr6li2h8jqiy3nnl4r9n687v-termite-13

Success on aarch64-linux (full log)
Attempted: termite
Partial log (click to expand)
gzipping man pages under /nix/store/17l6q4ck09ilr2gkdjs8m3sl8nx76ldv-termite-13/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/17l6q4ck09ilr2gkdjs8m3sl8nx76ldv-termite-13/bin
patching script interpreter paths in /nix/store/17l6q4ck09ilr2gkdjs8m3sl8nx76ldv-termite-13
checking for references to /build in /nix/store/17l6q4ck09ilr2gkdjs8m3sl8nx76ldv-termite-13...
shrinking RPATHs of ELF executables and libraries in /nix/store/mmzqcwqdsw2vhw2y3w4qwaananvqvqwi-termite-13-terminfo
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/mmzqcwqdsw2vhw2y3w4qwaananvqvqwi-termite-13-terminfo
checking for references to /build in /nix/store/mmzqcwqdsw2vhw2y3w4qwaananvqvqwi-termite-13-terminfo...
/nix/store/17l6q4ck09ilr2gkdjs8m3sl8nx76ldv-termite-13

Success on x86_64-darwin (full log)
Attempted: termite
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/i62ahhmb8a7j9pymak90k7qicw1fgnzs-termite-13/bin/termite
gzipping man pages under /nix/store/i62ahhmb8a7j9pymak90k7qicw1fgnzs-termite-13/share/man/
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/i62ahhmb8a7j9pymak90k7qicw1fgnzs-termite-13/bin
patching script interpreter paths in /nix/store/i62ahhmb8a7j9pymak90k7qicw1fgnzs-termite-13
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/2n5k1lqw99k41lbr5zri871m3cgjl7gh-termite-13-terminfo
/nix/store/i62ahhmb8a7j9pymak90k7qicw1fgnzs-termite-13

@GrahamcOfBorg build termite

Success on x86_64-darwin (full log)
Attempted: termite
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
Wrapping program /nix/store/29k45b79lh00bzdaw7zzhbi6mljpn2gv-termite-13/bin/termite
gzipping man pages under /nix/store/29k45b79lh00bzdaw7zzhbi6mljpn2gv-termite-13/share/man/
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/29k45b79lh00bzdaw7zzhbi6mljpn2gv-termite-13/bin
patching script interpreter paths in /nix/store/29k45b79lh00bzdaw7zzhbi6mljpn2gv-termite-13
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/8r8gxin2smmcg581vki5wir81xdxcvks-termite-13-terminfo
/nix/store/29k45b79lh00bzdaw7zzhbi6mljpn2gv-termite-13

Success on aarch64-linux (full log)
Attempted: termite
Partial log (click to expand)
gzipping man pages under /nix/store/m4ny8qdwpg4a760fdyp73drgvf8gs94n-termite-13/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/m4ny8qdwpg4a760fdyp73drgvf8gs94n-termite-13/bin
patching script interpreter paths in /nix/store/m4ny8qdwpg4a760fdyp73drgvf8gs94n-termite-13
checking for references to /build in /nix/store/m4ny8qdwpg4a760fdyp73drgvf8gs94n-termite-13...
shrinking RPATHs of ELF executables and libraries in /nix/store/azmdj2zgy5k15yvly8rm6dns1brafrvv-termite-13-terminfo
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/azmdj2zgy5k15yvly8rm6dns1brafrvv-termite-13-terminfo
checking for references to /build in /nix/store/azmdj2zgy5k15yvly8rm6dns1brafrvv-termite-13-terminfo...
/nix/store/m4ny8qdwpg4a760fdyp73drgvf8gs94n-termite-13

Success on x86_64-linux (full log)
Attempted: termite
Partial log (click to expand)
gzipping man pages under /nix/store/lsqi3p5qp000xka0ppw3hiziqgsskc05-termite-13/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/lsqi3p5qp000xka0ppw3hiziqgsskc05-termite-13/bin
patching script interpreter paths in /nix/store/lsqi3p5qp000xka0ppw3hiziqgsskc05-termite-13
checking for references to /build in /nix/store/lsqi3p5qp000xka0ppw3hiziqgsskc05-termite-13...
shrinking RPATHs of ELF executables and libraries in /nix/store/zy25fihv5jzfdmwy3p1ap0gfdvjkbvqd-termite-13-terminfo
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/zy25fihv5jzfdmwy3p1ap0gfdvjkbvqd-termite-13-terminfo
checking for references to /build in /nix/store/zy25fihv5jzfdmwy3p1ap0gfdvjkbvqd-termite-13-terminfo...
/nix/store/lsqi3p5qp000xka0ppw3hiziqgsskc05-termite-13

regarding the errno header patch: is this actually Nix related? Otherwise we should file a patch upstream right?

@Ma27 I believe it's macOS related, I will file an upstream patch later and if it gets merged I'll remove the patch from Nixpkgs.


[wip] - add support for displaying ob kiosk items in explorer

Description
Needs some more discussion
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Current dependencies on/for this PR:

main

PR #11904

PR #11943   👈

This comment was auto-generated by Graphite.


WIP: Security vulnerabilities fixed updating dependencies versions

Some dependencies where updated to fix vunerabilities. However, the update have
some icons missing in the UI showed on the attached picture. The status of this
PR is:
- Fix 13 vulnerabilities (7 moderate, 6 high) from dependencies since
commit 7647d8b1c660263efc4c5000d79d1a962090e39d.
- Have icons missing because some dependencies were updated.

The UI should be include the icons as the snapshot screen below from the commit
7647d8b1c660263efc4c5000d79d1a962090e39d showed it:


getCount for new StumbleUpon

Hi,
I am working on new share: StumbleUpon, I cannot seem to get the share counter right :(
         stumbleupon: {
            label: "StumbleUpon",
            logo: "fa fa-stumbleupon",
            shareUrl: "http://www.stumbleupon.com/submit?url={url}&title={title}",
            countUrl:  "https://www.stumbleupon.com/services/1.01/badge.getinfo?url={url}",
            getCount: function(data) {
                return parseFloat((data.match(/\"views\":([.0-9E]+)/) || [])[1]);
            }

when I do https://www.stumbleupon.com/services/1.01/badge.getinfo?url=https://www.google.com
this is what is returned:
{"result":{"url":"https:\/\/www.google.com\/","in_index":true,"publicid":"1krtS0","views":14834,"title":"Google","thumbnail":"http:\/\/cdn.stumble-upon.com\/mthumb\/647\/5593647.jpg","thumbnail_b":"http:\/\/cdn.stumble-upon.com\/bthumb\/647\/5593647.jpg","submit_link":"http:\/\/www.stumbleupon.com\/badge\/?url=https:\/\/www.google.com\/","badge_link":"http:\/\/www.stumbleupon.com\/badge\/?url=https:\/\/www.google.com\/","info_link":"http:\/\/www.stumbleupon.com\/url\/https%253A\/\/www.google.com\/"},"timestamp":1455201235,"success":true}

What is de function / regex to get the views out of this response?
I'm really lacking Javascript knowledge :(
If I get this working and confirmed by a user I will do a PR for the changes :)

I believe that this is just JSON, so we can get it like return data.result.views.

The PR adding StumbleUpon would be great! Thanks for your efforts!

Hi, already tried the return data.result.views but that didn't work.
It turned out (learn something new every day :)) that the returned format was not jsonp.
Needed to call the return url with the prefix url: https://cors-anywhere.herokuapp.com/ (same as google plus).
now it works :)
One 'StumbleUpon' PR coming up :)


Fix build error of ‘exchange’ is not a member of ‘std’

Fixes #1010

@SaschaWillems Hi Sascha, do you have some time to review this tiny fix? thx in advance!

I'm not the sole reviewer, but I'll try to a take a look this weekend.

Looks to be already fixed by https://github.com/KhronosGroup/Vulkan-Samples/pull/984 merged a few hours ago.


cgroupv1: do not run tests for version >= 4152

Flatcar with systemd >= 256 does not support cgroupv1 anymore.
See: https://github.com/flatcar/scripts/pull/2145

Thanks, this can be done on the kubeadm test too:
https://github.com/flatcar/mantle/blob/28a2bfa833e25322d5b4ebee0a01f9009e330d6e/kola/tests/kubeadm/kubeadm.go#L148-L166

Yes, that needs to be addressed, but do not know where is the best place to compare with a max version aka EndVersion. I think it should be done in a unified way, only defined once and used accordingly.

Hello @flatcar/flatcar-maintainers, I would like to have this PR reviewed, to unblock https://github.com/flatcar/scripts/pull/2145


Improve CI times

Description of changes:

Do not needlessly re-trigger the standard regression suite on "tag". Only on push of commits to main. This eliminates some wait time in the release process from being throttled on too many CI tasks running at once.
(Minor change to setting RUST_BACKTRACE)
Disable the stdlib test on macos for the moment. When someone has time to speed this up, we should re-enable it. Tracked in #1578

Resolved issues:
Call-outs:
Testing:

How is this change tested? ci

Is this a refactor change? no

Checklist

[ ] Each commit message has a non-empty body, explaining why the change was made
[ ] Methods or procedures are documented
[ ] Regression or unit tests are included, or existing tests cover the modified code
[ ] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Well, it cut us from 38 minutes to 31 minutes. I'm baffled by why macos is so slow in CI.

This run is from a while back, but notice how every single step seems to be slower on macos-11.


EMR EKS DAG stabilisation for astronomer-providers

Take care of cleaning up following resources:

[ ] VPC created by the DAG
[ ] OIDC created by the DAG

just see whether it is worth the time because we are planning to deprecate stuff soon.

Even if we deprecate, wouldn't we keep running the integration tests? That would help us test the RCs right?

IMO , since we already have integration tests dashboard from Amazon on OSS, once we point our provider to use OSS one, we should remove integration tests as well - as we no longer maintain our version of providers.

yes, that would be nice, we would save cost on spinning up AWS resources in that case

OIDC deletion logic fixed in PR

as per discussion with @phanikumv I have raised request to increase VPC limit to 15

@vatsrahul1001 do you think we can close this issue?

@pankajkoti no I still see some failures due to this

Closing as the provider is being deprecated


how to manually clear cached page's input data?

hi,
I have a page component with a CreateReminder form in it. Currently, after creating one Reminder, when i navigate to the CreateReminder page again (fresh nav using link: i.e. not using back btn, nor due to redirect), it shows the previous Reminder's input in the form.
I figured the simplest solution would be to just clear the cached input data for that component when the form is submitted, but i'm not seeing any documentation on how to either disable caching for a route completely (not preferred in this case), or how to clear a specific cached route from the cache, preferably with a way to specify only clearing the associated input data, and not the whole component.
Or, how are we intended to handle forms with this router? I'm new to SPAs, so i could easily be missing something basic.
Thanks

Hi,
there are a couple of solutions.
The easiest option is to disable caching on that specific page by calling setResumable(false) as documented here.
This would however dispose of the form content immediately, therefore going back wouldn’t recover the form state.
Another option is to manually reset the form fields once the submission succeeds, this way the user will always be able to recover the previous state.
In general form handling is a matter that goes beyond the scope of this (any?) router

ok, thanks


level 2 healthcheck checks redshift cluster and ecs query runner

level 2 healthcheck checks redshift cluster and ecs query runner

Looks like the existing unit tests will send network requests. Ideally we'd mock these out to minimize unit test run times.


FIX: ensure property keys are always serialized in the correct order

Fixes #90

Let's close in favor of either #92 or #93


Ensure image tags

Summary
This small enhancement ensures that at least one tag/version exists for a given image is available and returns an error if that is not the case.
Use Cases
Safety net for non existing tags/versions.
Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

This prevents a panic, that would occur a few lines later, right?
https://github.com/paketo-buildpacks/jam/blob/93afef1f1115241e40b046b400d758f62d159e59/internal/image.go#L95

@modulo11 if you rebase against the main branch the linting failure should be resolved here and we can get this merged in!


Expand & Integrate Cardinality checking

[ ] Kevin.  how do we leverage [ASSIST] (https://github.com/ge-high-assurance/RACK/wiki/RACK-Assist-Bin-Tools)
[ ] Abha
[ ] sadl rules examples seem too difficult to maintain

Goal is to expand restriction-checking:

to must be objectOf at least 1
unique?
re-check qualified cardinality?
abstract class like REQUIREMENT ?

Targets:

the simplicity of SADL cardinality : domain experts not semantic experts.
simple to run : plug into SPARQLgraph restriction-checker
stretch: the power of SHACL
clear documentation using OWL terminology of what is checked, including retro documenting ingestion in these terms (e.g. qualified cardinality):  build on [data-verification wiki] (https://github.com/ge-high-assurance/RACK/wiki/Data-Verification)

For the model, does SADL/OWL have these (I see inverseOf).   Ideally we want a clean way to model the constraints and a clean way to run them (currently we run standard cardinality in SemTK with Java building SPARQL).
Don't want to use inverseOf and add it to the model in a way that it might be instantiated and now all the queries need to be complex or need an inference engine.

Hope to have a POC of SHACL for the release.
If it is successful we can use the Sep-Mar time to improve the tooling


text is not updated

Hello,
I have a problem on an application, where I change the value of a text but the text is not updated (visually).
I can not reproduce the problem yet by using a simple test case, but here are the results of my experimentations.
The following is the trace that happen on the texture that is not updated:
TextTexture[13329] - set text Plus belle la vie
TextTexture[13329] - _updateSource (isUsed:true)
...
TextTexture[13329] - becomesUnused
...
TextTexture[13329] - _performUpdateSource (isUsed: false)
...
TextTexture[13329] - becomesUsed

The problem is that, when the texture becomes used again, the _performUpdateSource has already been called, but the texture has not been updated because, between the _updateSource and the _performUpdateSource, it has been unused.
By looking the method _performUpdateSource, there is the comment that said

If, in the meantime, the texture was no longer used, just remember that it must update until it becomes used

But I don't understand how this special case is handled.
_performUpdateSource(force = false) {
        // If, in the meantime, the texture was no longer used, just remember that it must update until it becomes used
        // again.
        if (force || this.isUsed()) {
            this._mustUpdate = false;
            let source = this._getTextureSource();
            this._replaceTextureSource(source);
        }
    }

According to me, if the texture is not used, we should make sure the _mustUpdate flag is set to true, so when the texture will become used again, the load() method will call the _performUpdateSource (by using the this.source) ?
On my side, the following works:
_performUpdateSource(force = false) {
        if (force || this.isUsed()) {
            this._mustUpdate = false;
            let source = this._getTextureSource();
            this._replaceTextureSource(source);
        } else {
            // If, in the meantime, the texture was no longer used, just remember that it must update until it becomes used again.
            this._mustUpdate = true;
        }
    }

There are other places where we could fix the problem, and I can't really tell you how to reproduce the problem.
I just want to know if this fix looks acceptable for you.

Hi @salmon-charles,
Thanks for reporting.
Is _changed() method called between becomesUnused() and performUpdateSource()? Because it should've been called if texture source is currently unused and set _mustUpdate flag to true. How exactly are you changing value of text?
Best regards

@salmon-charles any updates? Or can we close this ticket?

Closing for now, please reopen if still relevant.


Create a community metrics dashboard

Description
Beam has a rich community metrics dashboard (discussion). Would it help to have something similar?

OpenSearch Project has one https://github.com/opensearch-project/opensearch-metrics and I'm one of the maintainer for this metrics project. Happy to drive this here and be part of this discussion :).

Project Insights Dashboard

Project Ops Metrics Dashboard

Thank you.

@prudhvigodithi - that's great! Your help is very welcome, I wouldn't really know where to start if I was doing this myself.

To begin with we need:

A server that can run and host the metrics portal (the frontend).
The application that will daily use the Github API to scrape the issues, pulls, contributions etc and save it to a database or any distributed service.
- Since Github API's are limited, the best possible way is to collect daily incremental data (to begin fetching the data from last successful collection).
- If required to use any external datasource, should be leveraging its own API's to fetch the data.
- An existing Github bot can be used as well, either to use the bot token or push on an Github event.
A side car (or a process in the same server) that will use the raw saved data from database or any distributed service and create the desired metrics that will be displayed by the frontend.

Thank you

That sounds like a plan! I'd be thrilled if you were driving this @prudhvigodithi - thank you for offering your help.
It would also be great to get feedback from others on which metrics they'd find most useful.

Thanks @stefanvodita, I would like to have the maintainers of the repo to take the initial stab at this metrics dashboard (come up the server, app, choosing the database and deploying the resources), and later as mentioned I will be happy to contribute to the dashboard design and features.


Allow Pathname arguments to be escaped

An error is raised by using a Rails project that is configured with:
config.autoload_paths << config.root.join('app/decorators')

And then running rspec bisect:
Running suite to find failures...
/usr/local/bundle/gems/rspec-core-3.7.0/lib/rspec/core/shell_escape.rb:20:in `escape':
  undefined method `shellescape' for #<Pathname:/app/app/decorators> (NoMethodError)

Can you add a spec for this?

@myronmarston it looks like everything is green but JRuby is having trouble running. Do you want to see if it passes by restarting the build?

We can ignore the JRuby failures, they're caused by Travis build limits and broken rvm

Thanks, @avit!


Fix: change tracking delete test case with v1.0.7

Older implementation resulted in 0 entries in changeView
https://github.com/cap-js/incidents-app/blob/259abc1e41142f2d2f27d3c8cb09af0ee1e9aed1/xmpls/change-tracking.test.js#L125-L130
With upcoming v1.0.7, there seems to be change and this fix works.
(To be merged after change-tracking v1.0.7 release, as this fails in older version).

Created issue in plugin to confirm the behavior
https://github.com/cap-js/change-tracking/issues/101


fix: auth route conflict

Description
This PR fixes #982
Summary
This PR does fix authenticated routes with different method but same path causing conflict
PR Checklist
Please ensure that:

[x] The PR contains a descriptive title
[x] The PR contains a descriptive summary of the changes
[x] You build and test your changes before submitting a PR.
[x] You have added relevant documentation
[x] You have added relevant tests. We prefer integration tests wherever possible

Pre-Commit Instructions:

[x] Ensure that you have run the pre-commit hooks on your PR.

todo:

fix tests
please comment on the approach @sansyrox

todo:

[x] fix tests
[x] please comment on the approach @sansyrox

Hey @VishnuSanal , can you fetch the latest changes here??

Hi all, same issue here.
Any blockers for merging this? Let me know if I can help somehow!

Hey @alissonpelizaro 👋
Apologies for the delay in review. I planned to review this over the weekend. If possible, could you do an initial review please?


What about REST server?

I scoured the documentation and the source code to find information about the REST Api and apparently it only includes a client.  You can find a .NET REST Server here.

Colore has full support for using the Chroma REST API in version 6.0 (it's one of the major new features). You can create an IChroma instance that uses the REST API as backend with ColoreProvider.CreateRestAsync.


Custom getter fix

[x] Closes #8902
[x] Tests added / passed
[x] Passes pre-commit run --all-files

Great, Ian! I especially like the test, which is the main reason why I did not just submit a PR. Figuring out where the test goes would have taken me ten times longer 😁

I'll probably continue to use inline_array=True in my custom getter use case, in order to support 2022.4.0...


build(package-lock.json): fix for npm pkg ip vulnerable to SSRF

:tada: This PR is included in version 5.0.17 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


make self contained packages

To not depend on system version of ruby, package everything as standalone installation.

http://phusion.github.io/traveling-ruby/
https://github.com/chef/omnibus
http://enclose.io
http://nts.strzibny.name/making-a-ruby-executable-with-ruby-packer/

would prevent #24, https://github.com/3scale/3scale_toolbox/issues/31

How about delivering docker image?
What's the use case? Linux users and OSX users?
I am being influenced by openshift client oc CLI. Delivery based on tarballs with ruby shipped (as proposed by Travelling-ruby solution) might be a good candidate. Although I do not like shipping third party binaries.

There are some CLI that use docker (circleci, codeclimate) and work quite well. It could definitely be an option.

Omnibus

active project
few limitations (if none)
OS-specific packaging. Each OS and each flavor of the same OS requires one package release. Each OS build process requires platform specific tools to be installed and maintained.
Heavyweight (several VM's must be launched for building), slow (one packaging process for supported OS version)
Several configuration DSL files required.

Ruby-packer

Packing application into a single executable
Based on read-only file system using sqashfs where all required dependencies are stored.
OS-specific packaging (linux, osx, windows)
Native extensions supported
last release Aug 2017 (0.4.0)
self contained Ruby 2.4. Already requested 2.5.1
Evaluation PoC

packed tool


server: add structural diffing to comparator

I removed the diff library from this PR due performance concerns. The string comparisons scores are now based on an edit distance with a timeout of one second per comparison, in which case a correct but less accurate score is reported. One thing to note that is this change can dramatically change the score for strings and object and arrays containing strings.

Since we didn't end up changing the payload (adding the diff or diffString api), is the added value here reduced to providing more accurate string comparison match rates?

I think in general it might more sense to perform diffing client-side, due to the smaller payload sent to the client and the smaller load on the server.
In that case, how would the server provide high level insights and organize, filter, sort the submitted test results?


Add CUMO_NVCC_GENERATE_CODE environment variable to configure nvcc --generate-code option

I've found that it sometimes takes time on booting CUDA program.
It was caused by first-time JIT compilation of PTX into cubin.
By specifying --generate-code with real architecture, we can create cubin beforehand and avoid JIT compilation.
Specifying multiple --generate-code options takes much of time on compilation, but one --generate-code does not increase build time so much (and we originally specified -arch=sm_35, so the compilation time does not change).

ref. https://qiita.com/sonots/items/1c60ccb6ffec6d04b223


Stop dask state change on Iris import

We have chosen to set default Iris behaviour to use only one thread when processing dask graphs initially. This was done by setting the global dask state using dask.set_options(). This is far from ideal as it means Iris is changing dask state; a situation that should not be. This PR updates the system for setting dask processing options so that behaviour is maintained without changing dask state.

Closes #2739.

@pelson I had a sneaky extra 10min, so I implemented the changes you suggested 😎

Deferring for now. #2879 removes the default change, and we can re-introduce a different default when we have a "smoking gun"/evidence of need to divert from dask's defaults.


block init: empty metadata in RPC response (maybe you are not using an archive node)

With the last version in master generates the following warning and doesn't seem to index anything:
$ ./tzindex run --rpcurl https://hangzhounet.smartpy.io
2022/04/07 00:10:49.500952 WARN MAIN Missing config file, using default values.
2022/04/07 00:10:49.501023 INFO MAIN Blockwatch tzindex v12.0 dev
2022/04/07 00:10:49.501031 INFO MAIN (c) Copyright 2018-2022 Blockwatch Data Inc.
2022/04/07 00:10:49.501042 INFO MAIN Starting tzindex.blockwatch.cc/v12.0.dev on 8 cores
2022/04/07 00:10:49.501049 INFO MAIN Go version go1.16.2
2022/04/07 00:10:49.506028 INFO MAIN Using bolt database ./db
2022/04/07 00:10:49.526745 INFO BLOC Initializing blockchain crawler in sync mode.
2022/04/07 00:10:49.526816 INFO BLOC Creating blockchain storage.
2022/04/07 00:10:49.532403 INFO BLOC Creating account index.
2022/04/07 00:10:49.670791 INFO BLOC Creating balance index.
2022/04/07 00:10:49.711302 INFO BLOC Creating contract index.
2022/04/07 00:10:49.790161 INFO BLOC Creating constant index.
2022/04/07 00:10:49.858943 INFO BLOC Creating block index.
2022/04/07 00:10:49.912165 INFO BLOC Creating op index.
2022/04/07 00:10:49.969932 INFO BLOC Creating chain index.
2022/04/07 00:10:50.032731 INFO BLOC Creating supply index.
2022/04/07 00:10:50.105097 INFO BLOC Creating bigmap index.
2022/04/07 00:10:50.215616 INFO BLOC Creating metadata index.
2022/04/07 00:10:50.284062 INFO BLOC Initializing account index.
2022/04/07 00:10:50.311522 INFO BLOC Initializing balance index.
2022/04/07 00:10:50.318882 INFO BLOC Initializing contract index.
2022/04/07 00:10:50.330374 INFO BLOC Initializing constant index.
2022/04/07 00:10:50.338913 INFO BLOC Initializing block index.
2022/04/07 00:10:50.354079 INFO BLOC Initializing op index.
2022/04/07 00:10:50.376303 INFO BLOC Initializing chain index.
2022/04/07 00:10:50.392760 INFO BLOC Initializing supply index.
2022/04/07 00:10:50.402942 INFO BLOC Initializing bigmap index.
2022/04/07 00:10:50.414582 INFO BLOC Initializing metadata index.
2022/04/07 00:10:50.421569 INFO BLOC Connecting to RPC server.
2022/04/07 00:10:50.737055 INFO BLOC Fetching genesis block.
2022/04/07 00:10:50.825675 INFO BLOC Crawling Tezos Hangzhounet2.
2022/04/07 00:10:50.893755 INFO BLOC Block cache with 1 entries built in 56.043664ms
2022/04/07 00:10:50.895104 INFO BLOC Starting blockchain crawler.
2022/04/07 00:10:50.895245 WARN SRVR HTTP Server reachable on localhost only
2022/04/07 00:10:50.895426 INFO BLOC Starting blockchain sync from height 1.
2022/04/07 00:10:50.895461 INFO BLOC Starting blockchain ingest.
2022/04/07 00:10:50.895587 INFO BLOC Starting blockchain monitor.
2022/04/07 00:10:50.899000 INFO SRVR Starting HTTP server at <IP_ADDRESS>:8000
2022/04/07 00:10:52.652406 WARN BLOC crawler: RPC connection error: block init: empty metadata in RPC response (maybe you are not using an archive node)
2022/04/07 00:11:01.768824 INFO BLOC Processed 1 block in 12.24s (0 transactions, height 0, 2021-11-04 15:00:00 +0000 UTC, q=0, t=1.753921ms, s=syncing)
2022/04/07 00:11:02.080203 WARN BLOC crawler: RPC connection error: block init: empty metadata in RPC response (maybe you are not using an archive node)
2022/04/07 00:11:10.532895 WARN BLOC crawler: RPC connection error: block init: empty metadata in RPC response (maybe you are not using an archive node)
2022/04/07 00:11:20.364669 INFO BLOC Processed 2 blocks in 18.59s (0 transactions, height 0, 2021-11-04 15:00:00 +0000 UTC, q=0, t=14.830497ms, s=syncing)
2022/04/07 00:11:20.540038 WARN BLOC crawler: RPC connection error: block init: empty metadata in RPC response (maybe you are not using an archive node)

I've tried with several nodes with the same result.
Any idea what may be wrong?

It seems the node(s) you are running the indexer against have pruned the metadata of early blocks. This means they do not run with --history-mode archive enabled.
If you look at https://hangzhounet.smartpy.io/chains/main/blocks/2 you can see that the metadata field in the block header and metadata fields in all operations are missing. The indexer needs this data to know the effects of each operation such as balance updates, contract call results, etc.
The following works as expected:
./tzindex run --rpcurl https://rpc.hangzhou.tzstats.com

I'm not sure who else other than us runs testnet archive nodes.

This is weird — I'm quite sure it was working properly with these nodes a few days back, and we are indeed running a hanghzounet node in archive mode and have the same behavior on that node.
Anyway, thanks for looking into this!

It seems Smartpy have upgraded to Octez v12.2 (https://hangzhounet.smartpy.io/version) which is known to strip metadata when size is above a limit. They call it a feature, but it creates more harm than good IMHO.

ok, thanks a lot for the info!


QRL Users

This is the main PR document for the QRL Mining Pool

First run through completed and uploaded to the next cloud.  Will place into review for a few days.  For these first documents with a fragmented team, let's have two or three development periods for documentation until we build trust and understand mission statements.

Use PR Documentation.docx to review.  Add comments here for review.  Using agile practice, all comments here will be addressed and the issue moved back or forwards based on the input.

Added Pre-Testnet Reddit Message.  Would like to send out soon to get more more people involved.

Throw a link to join the discord chat in general signature?

"Our mining pool will directly move everyone over from the testnet."

I don't know that this is going to work yet

Highlight the potential to continue earning from mining rewards after the fork to PoS

I'm going to make some additions to this Doc for Email campaigns and future mailing list content.
I think we should notify our customers via the mailing list when we are:

launching
Monthly updates if things have changed
Announcement for PoS


fix the raspios link

Changelog: None
Signed-off-by: Ole Petter<EMAIL_ADDRESS>

Missing to update the links from Downloads section

No, ur looking at staging.
For 3.2. they're there: https://staging.docs.mender.io/3.2/downloads


Add "edit" button to every page section

More details forthcoming (just didn't want to forget).

Won't repo!

Hello Monday! 😆


Update NuGet from command line

Hi,
I have installed NuGet to a particular folder from command line by using following command.
install "Packagename"  -Version "version"  -Source "source"  -o "output directory".The package has been installed successfully . Now i need to update the packages in this folder for this i have used the following command
Update "Pacakge name"  -Source "source"  -RepositoryPath "directory in which previous packages are installed". But am unable to update since iam not having any packages.config or solution file. it shows the following message.
No packages.config, project or solution file specified. Use the -self switch to update NuGet.exe.
SO how can i update packages without packages.config or solution folder

This issue was moved to NuGet/Home#2781


Connection going via FDP don't appear in conntrack

Which kinda breaks scope.

This may be relevant https://lwn.net/Articles/633401/

This may be relevant https://lwn.net/Articles/633401/

Yes, seems so.  Though there is the usual issue that it takes a while for kernel changes to propagate widely enough that we can rely on them.

Is there anything we can do here, besides waiting for the kernel changes to propagate, that doesn't require Herculian effort?

@dpw says we may be able to work around this by introducing an intermediary bridge.

Does it break the same way when using Docker's new network, which is also using VXLAN?

Does it break the same way when using Docker's new network, which is also using VXLAN?

I wouldn't expect so.  It's not due to vxlan.

I have completed a simple performance test comparing existing fast datapath with fastdapath + intermediary bridge e.g.:

veth --> weave <-- network --> weave <-- veth
veth --> docker0 <-- veth --> weave <-- network --> weave <-- veth --> docker0 <-- veth

Tested on an Intel i7 5820K @ 3.3 GHz using two Ubuntu 15.04 Virtualbox VMs as Docker hosts. docker0 was connected to weave by a veth on each host, and then a container was started on each, taking care to ensure that the eth0 docker interfaces were allocated different addresses in the same subnet. That way, it was possible to run iperf -s in a container on one host (binding to both the weave and Docker interfaces) and then run iperf -c in a container on the other, targetting either the weave or Docker IP of the remote container.
The performance results on the bare hosts:
$ iperf -s
------------------------------------------------------------
Server listening on TCP port 5001
TCP window size: 85.3 KByte (default)
------------------------------------------------------------
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 42088
[  4]  0.0-10.0 sec  1.45 GBytes  1.24 Gbits/sec
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 42095
[  5]  0.0-10.0 sec  1.57 GBytes  1.35 Gbits/sec
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 42096
[  4]  0.0-10.0 sec  1.42 GBytes  1.21 Gbits/sec

Performance results via weave:
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 33922
[  4]  0.0-10.0 sec  1.05 GBytes   902 Mbits/sec
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 33923
[  5]  0.0-10.0 sec  1.10 GBytes   946 Mbits/sec
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 33924
[  4]  0.0-10.0 sec  1.20 GBytes  1.02 Gbits/sec

Performance with an intermediary bridge:
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 33960
[  5]  0.0-10.0 sec   958 MBytes   803 Mbits/sec
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 33961
[  4]  0.0-10.0 sec  1022 MBytes   855 Mbits/sec
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 33962
[  5]  0.0-10.0 sec   972 MBytes   813 Mbits/sec

Based on this measurements the intermediary bridge restores conntrack functionality at the cost of a ~15% performance reduction.

The use of the docker0 bridge may well skew the results here.

Good point. I will re-run the tests once we've implemented with our own intermediary bridge.

I will re-run the tests once we've implemented with our own intermediary bridge.

Same test fixture as before. Bare host performance:
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 32786
[  4]  0.0-10.0 sec  1.47 GBytes  1.27 Gbits/sec
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 32787
[  5]  0.0-10.0 sec  1.41 GBytes  1.21 Gbits/sec
[  4] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 32788
[  4]  0.0-10.0 sec  1.37 GBytes  1.18 Gbits/sec

Performance with WEAVE_NO_BRIDGED_FASTDP=1 weave launch:
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 59091
[  5]  0.0-10.0 sec  1.25 GBytes  1.07 Gbits/sec
[  6] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 59092
[  6]  0.0-10.0 sec  1.15 GBytes   982 Mbits/sec
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 59093
[  5]  0.0-10.0 sec  1.15 GBytes   987 Mbits/sec

Performance with dedicated intermediary bridge + fastdp:
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 59075
[  5]  0.0-10.0 sec  1.05 GBytes   904 Mbits/sec
[  6] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 59076
[  6]  0.0-10.1 sec  1.21 GBytes  1.03 Gbits/sec
[  5] local <IP_ADDRESS> port 5001 connected with <IP_ADDRESS> port 59077
[  5]  0.0-10.0 sec   958 MBytes   802 Mbits/sec


How to automatically switch service to access to shared table between HANA and SQLite?

Dear team,
Thanks for your great contribution to share multitenancy application and instruction.
I am implementing multitenancy application based on your repository and I have question for shared database container.
In your explanation, we need two CDS for both HANA for production and SQLite for local testing.
Currently I extended your service to access to shared table and manually change CDS file /code/db/sqlite/data-model to /code/db/hana/data-model before deployment to Cloud Foundry.
Is there smarter way to switch between two CDS files?
I could not find good example of this use case. So I am helpful if you advise me.

Please check this example: fiori/package.json#L42

Hi @gregorwolf,
Thanks for your answer. That is helpful and I already checked this code sample.
But my question is how to refer to db CDS from service CDS.
For example, when I want to add Shared entity to public-service.cds, now I am writing like below.
using {susaas.common as common} from '../../db/hana/data-model';
...
service PublicService @(path : '/catalog/PublicService', impl: 'srv/public-service') {

  entity Shared as projection on common.Shared
  ...

Hence I am specifying db/hana/ directory, I cannot use it in for local SQLite.
How to automatically switch path in service CDS, or something alternatives?

Hi Dear @kaztakata san,
Thank you very much for your contribution!
To be honest, when we designed this "common container" approach, I personally only considered the database layer usage.
But your requirement is valid and makes sense. I think it might be useful for the rest of our valued susaas community members too.
I have a nice solution proposal for the problem you have.
This solution proposal should not change the behavior of the repository and should also fix the problem you have.
The main reason we have two files is that Shared  entitys DB artifacts are already created on deployment; therefore, we would like to mention it to HANA to avoid errors, but for sqlite we ne need to create the tables.
Can you please confirm if the changes on the fix/cc-localdev branch fix the problem you have?
I also linked the relevant PR to this issue.

Hi @alperdedeoglu san,
Thanks for your solution. I think extend CDS is effective for this situation.
I used your code and changed package.json to switch between production and development profiles like previous answer by Gregorwolf then working.


PAM Authentication not configurable

Gitea version (or commit ref): 1.14.6 (Noticed it on earlier versions as well!)
Git version: 2.20.1
Operating system: Debian 10

Database (use [x]): (irrelevant, saw it on different setups)

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes
[x] No - Can't login to try.gitea.io via GitHub OAuth, will attach screenshot.

Log gist:

Description

The documentation (link) mentions, that is is possible to use PAM as an authentication provider. It is not possible to select PAM as a Provider in the Web GUI.
Furthermore there is no sign in the app.ini that I can configure it there.
If the feature was planned and scrapped halfway through, this part should be removed from the docs. I'd love to use this feature, so if it would be implemented at a later point in time I'd appreciate it.
If this feature is available, the documentation should be updated as well to further describe, how to correctly configure it.
In the mean time I'll update my instance to the latest release and update the issue accordingly.
Screenshots

Updated my instance.
Problem still persists.

AFIAU PAM cannot be built statically therefore we do not provide it within the binaries you can download.
You need to build gitea yourself with the pam tag in addition to your normal build tags.
See https://docs.gitea.io/en-us/install-from-source/#build

Hello @zeripath,
thanks for your response.
This should definitely be made more clear in the documentation article (link). If someone is not installing it from source like me, he won't come across this step and wonders, why it is not working.
Is it possible to put it on the agenda?
Kind Regards
truzzon

Please feel free to send a PR improve the documentation on this.


gmtime_r and mbedtls_time on Windows

This PR fixes a couple of issues:

mbedTLS should not be calling gmtime_r. Even with a mutex, the call is simply unsafe (other threads in the application could be calling it, obviously without holding mbedTLS's mutex). It's simply not appropriate to be using non-thread-safe APIs in a security library.
On Windows, the mbedtls_time platform abstraction does not work, because for some reason the time() API is not used on Windows? I can't think of any reason not to use it, the Windows time implementation ought to go through the same platform abstraction layer as any other feature.
The second commit fixes the build with -std=c99. There are a couple of existing places in the code where this is broken, plus gmtime_r adds another one.

Issue: https://github.com/ARMmbed/mbedtls/issues/704

@NWilson Thank you for your continuous contributions!
We will have a look at it.

I've updated the PR by rebasing it, removing the mutex definition, and re-running the tests. Thanks for the review.
Sorry for lumping two different issues together here (gmtime_r and -std=c99 build), I only did them together because gmtime_r has various macro visibility requirements, and I wanted to test with those, and found that the rest of mbedTLS doesn't respect the POSIX visibility requirements!

Thank you for a quick response (a lot quicker than ours, sorry for the delay on our side)! The PR looks good now; I will arrange for the ultimate reviews once we've got a result from the CI.

Bump - there are no existing tests for compilation using -std=c99, is it necessary to add such tests for the PR to be accepted?

I'm a bit concerned how this PR will work with Mbed OS, and what support there is in Mbed OS for gmtime_r(). There is a related PR over in Mbed OS to enable RTC support, and I'd like this not to break that.
cc: @k-stachowiak
Also in response to @NWilson's question concerning the --std=c99 test, if it passes the CI and all.sh it's fine.

I can't find the relevant mbedOS documentation. What libc does it use? It looks like it might use newlib in future, but doesn't currently, and I can't find any reference to the standard C functions provided by mbedOS.
Unless mbedOS is living under a rock, gmtime_r should be available if gmtime is.
This shouldn't have any relation to Real-Time Clock support; gmtime is a hardware-independent algorithm that doesn't need to know about the current time at all.
I agree we don't want to break mbedOS, but I wouldn't expect it to (I can't test given I don't have any ARM boards or simulators set up).

I have verified in practise that the gmtime_r function produces results consistent with gmtime and therefore I sustain my approval for the PR. @sbutcher do you think the PR can be approved?

Following @NWilson additional commits, can you please re-review?

@NWilson - As ever, your contributions are very welcome. If you don't have time for responding to the review comments or doing any rework, please let us know and we'll pick this up and finish it off for you.

Sorry, it just took me a while to get round to it! The changes were small, I've done them now and re-run the build. Also rebased to fix a conflict in the ChangeLog file.
I didn't edit the commits, the changes are contained in the final commit for ease of reviewing. Thanks!

Thanks @NWilson! Hopefully we can get it in this time.

retest

I have investigated the FreeBSD build issue on a VM. The problem is, that, contrary to what has been documented in some places, _XOPEN_SOURCE >= 500 is not always enough to enable snprintf( ).
Looking here reveals, that in order to unlock this part of the API, we need to get the __ISO_C_VISIBLE symbol to resolve to at least 1999, which based on this leads to a conclusion that setting _POSIX_C_SOURCE to at least 200112 should do.
@NWilson For your convenience, I have submitted a PR with a relevant change to your clone. If you don't have any objections, you may merge it there, which in turn should lead to the commit being submitted here.

retest


Change of testing framework

It looks like Demitasse has been deprecated: https://github.com/pipobscure/demitasse
I would be happy to help with migration to something else. Lately I have been using Jest but don't really have much preference.

That would be great! I don't know if any of the maintainers have a preference either, but any reasonably popular test harness that works with TypeScript sources ought to be fine.

Great I should have a PR up this week

@corbinu - Thanks! Do you use VS Code?  If so, then please update .vscode/launch.json too for IDE debugging of tests.  If you don't use VS Code, then I or another maintainer can help you figure out what launch.json changes are needed. At a minimum you should change the name of the config below.
Here's the current debug config used for Demitasse tests:
https://github.com/js-temporal/temporal-polyfill/blob/f885253fdc0d16115b0a9d986a47e3fa35e50878/.vscode/launch.json#L39-L48

I do use VS Code and will make sure to update. Thanks so much for the suggestion!

For context: what do the Demitasse tests do that the test262 test suite doesn't? Could we instead focus this effort on having a single set of 262 tests?

Over at the original proposal-temporal repo we're in the process of converting the Demitasse tests into test262 tests, which is going to be a requirement for the proposal to move to Stage 4. We should definitely import those tests and run the polyfill against them, because that's what ensures the polyfill is compliant with the specification.
But test262 tests are cumbersome and time-consuming to write and edit due to the format they have to be in. So, if there are any sort of tests that are specific to the polyfill, such as non-ISO calendars, non-UTC time zones, or some mechanism for loading alternate sources of time zone data, then I think we'd be better off maintaining tests for those in a different format than test262.

That makes sense. Sorry this is my first time working on a pollyfill. Perhaps I should wait until the 262 tests are set and then I can convert any left over Demistasse tests.

While I'm thinking of it, another thing that we could implement in non-test262 tests is tests for certain out-of-memory conditions — for example, I'm learning from WebKit reviewers while trying to implement Temporal.Instant in JavaScriptCore that when you have error messages like ${string} is not a valid ISO 8601 string for Temporal.Instant then you have to be robust against things like Temporal.Instant.from('BAD'.repeat(2 ** 32)).


Async/await ES5 throws Cannot read property '0' of undefined

TypeScript Version:  Version 2.1.0-dev.20160911
Code
// A *self-contained* demonstration of the problem follows...
async function test(skip: boolean) {
    if (!skip) {
        await 1
    }
    else {
        throw Error('test')
    }
}

test(false)

Works fine with target=es6
Error is thrown in this part of generated code (op is undefined)
                switch (f = 1, op[0]) {
                    case 0: case 1: sent = op; break;
                    case 4: return _.label++, { value: op[1], done: false };
                    case 7: op = _.stack.pop(), _.trys.pop(); continue;

//cc: @rbuckton

related #10751

do not see how this is related to  #10751

no its not, but i got at least somebody look at it


git submodule integration

Excellent job, it works on my machine too !!!
I had a couple of elf patching error but at the end I can compile....
I wonder how could we use your git repo as a esp-hal cargo project using git submodule maybe ?
If you flatten all files in the root of your repo, it could be used as a submodule just by creating a symlink to shell.nix from the root of the host repo (I guess).

That is great to hear! :) I also get ELF-patching warnings, it seems OK.
I haven't tried to package it as a submodule before, but if you are just looking for a 'portable' shell.nix you can use this one :) (this is the one I normally use).
{ pkgs ? import <nixpkgs> {}}:
let
    esp-rs-src = builtins.fetchTarball "https://github.com/leighleighleigh/esp-rs-nix/archive/master.tar.gz";
    esp-rs = pkgs.callPackage "${esp-rs-src}/esp-rs/default.nix" {};
in
pkgs.mkShell rec {
    name = "esp-rs-nix";

    buildInputs = [
        esp-rs
        pkgs.rustup # I can't remember why I needed this
        #pkgs.espflash # broken, install with cargo instead!
        pkgs.rust-analyzer
        pkgs.pkg-config
        pkgs.stdenv.cc
        pkgs.systemdMinimal
    ];

    shellHook = ''
    # optionally reload your bashrc to make the ugly nix-shell prompt go away
    #. ~/.bashrc

    # this is important - it tells rustup where to find the esp toolchain,
    # without needing to copy it into your local ~/.rustup/ folder.
    export RUSTUP_TOOLCHAIN=${esp-rs}
    '';
}

That is great to hear! :) I also get ELF-patching warnings, it seems OK.
I haven't tried to package it as a submodule before, but if you are just looking for a 'portable' shell.nix you can use this one :) (this is the one I normally use).
{ pkgs ? import <nixpkgs> {}}:
let
    esp-rs-src = builtins.fetchTarball "https://github.com/leighleighleigh/esp-rs-nix/archive/master.tar.gz";
    esp-rs = pkgs.callPackage "${esp-rs-src}/esp-rs/default.nix" {};
in
pkgs.mkShell rec {
    name = "esp-rs-nix";

    buildInputs = [
        esp-rs
        pkgs.rustup # I can't remember why I needed this
        #pkgs.espflash # broken, install with cargo instead!
        pkgs.rust-analyzer
        pkgs.pkg-config
        pkgs.stdenv.cc
        pkgs.systemdMinimal
    ];

    shellHook = ''
    # optionally reload your bashrc to make the ugly nix-shell prompt go away
    #. ~/.bashrc

    # this is important - it tells rustup where to find the esp toolchain,
    # without needing to copy it into your local ~/.rustup/ folder.
    export RUSTUP_TOOLCHAIN=${esp-rs}
    '';
}

thanks I like your shell.nix example ! may i ask where are you using it ?

I haven't got a public repo that uses it at the moment - but I've got several commercial projects using embassy-rs, which use a shell.nix file like that :)


Question: Can't download model checkpoints from Azure blob model registry

I am attempting to download model checkpoints from the Azure Blob Model Registry associated with this repository. However, access denial errors appear to suggest that there's no public access to that storage, but only access via accounts that are added as external to openAI's tenant.
filename: model_registry.py
_MODEL_SPECS: dict[str, StandardModelSpec] = {
    # GPT-2 series
    "gpt2-small": StandardModelSpec(
        model_path="az://openaipublic/neuron-explainer/subject-models/gpt2/small"
    ),
    "gpt2-medium": StandardModelSpec(
        model_path="az://openaipublic/neuron-explainer/subject-models/gpt2/medium"
    ),
    "gpt2-large": StandardModelSpec(
        model_path="az://openaipublic/neuron-explainer/subject-models/gpt2/large"
    ),
    "gpt2-xl": StandardModelSpec(
        model_path="az://openaipublic/neuron-explainer/subject-models/gpt2/xl"
    ),
}

Could you please verify whether the aforementioned Azure Blob storage is intended to be publicly accessible for model checkpoint retrieval?

#10


Feature: Update all the package versions (Besides CJS to ESM)

Goal of PR
Updating all the packages that are used in the hub, reader and admin.
Implementation
Updating packages and making the appropriate changes in the code.
Manual Testing
Set up a gaia hub on:

GCP
Azure
AWS
and see that it works the same as before

Automated Testing
Unit tests
Submission Checklist

[x ] Based on correct branch: feature submissions should be on develop, hotfixes should be on master

[x ] The code passes our eslint definitions, unit tests, and
contains correct TypeFlow annotations.

[x ] Submission contains tests that cover any and all new functionality or code changes.

[ x] Submission documents any new features or endpoints, and describes how developers
would be expected to interact with them.

[x ] Author has agreed to our contributor's agreement.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: bjorger:x: fritzsimaYou have signed the CLA already but the status is still pending? Let us recheck it.

@bjorger , thanks for this pr. Please update the branch so conflicts are resolved.


How to remain auto play top video till its visiblity is more than 50% during scrolling.

I connected to toro library to auto play top video of Recyclerview using "compile 'com.github.eneim:Toro:1.2.0" gradle. It is working fine. When we start scroll from down to top then second video is playing automatically. Now I want that When the visibility of top video is going to less that 50% then second video auto play start and during scrolling top to bottom when top video get visibility more than 50% then second video auto play should be off and top video auto play should be start.
For achieving this functionality I am using this method
@Override
    public boolean wantsToPlay() {
        return visibleAreaOffset() >= 0.5f;
    }

    @Override
    public float visibleAreaOffset() {
        Rect videoRect = getVideoRect();
        Rect parentRect = getRecyclerViewRect();
        if (parentRect != null && !parentRect.contains(videoRect) && !parentRect.intersect(videoRect)) {
            return 0f;
        }
        float area = videoRect.height() * videoRect.width();
        float viewArea = getVideoView().getWidth() * getVideoView().getHeight();
        return viewArea <= 0f ? 1.f : area / viewArea;
    }

but during scrolling when the top video visibility is less than 100% then second video auto play start.
How can we achieve functionality to auto play top video till its visibility is more than 50%?

@shailesh2208 Take a look at your ToroStrategy too. It may be the one that will prior the View with higher visibility. Use the first one for your expected behaviour. I also suggest you to move on Toro 2.1.0 for latest support. Thanks for using Toro.


xds: Bump version to latest

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message: xds: Bump version to latest
Additional Description:
this includes latest xds version which will reduce the warning noise when building protos
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

/lgtm deps

/retest


OCM-8072 | test: automate id:60082,42188 ci-fix

`$ ginkgo --focus 60082 tests/e2e/
Ginkgo detected a version mismatch between the Ginkgo CLI and the version of Ginkgo imported by your packages:
Ginkgo CLI Version:
2.14.0
Mismatched package versions found:
2.17.1 used by e2e
Ginkgo will continue to attempt to run but you may see errors (including flag
parsing errors) and should either update your go.mod or your version of the
Ginkgo CLI to match.
To install the matching version of the CLI run
go install github.com/onsi/ginkgo/v2/ginkgo
from a path that contains a go.mod file.  Alternatively you can use
go run github.com/onsi/ginkgo/v2/ginkgo
from a path that contains a go.mod file to invoke the matching version of the
Ginkgo CLI.
If you are attempting to test multiple packages that each have a different
version of the Ginkgo library with a single Ginkgo CLI that is currently
unsupported.
Running Suite: e2e tests suite - /home/aaraj/ocm-forked-code/rosa/tests/e2e
Random Seed:<PHONE_NUMBER>
Will run 1 of 92 specs
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS2024/05/30 19:55:34 &{arn:aws:kms:us-east-1:301721915996:key/d57a63f2-1ab9-42c9-8422-fe1a478c6dc5 }
•SSSSSSSSSSSSSSSSSSSSSSSSSSS
Ran 1 of 92 Specs in 6.638 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 91 Skipped
PASS
Ginkgo ran 1 suite in 19.325901517s
Test Suite Passed
`
`$ ginkgo --focus 42188 tests/e2e/
Ginkgo detected a version mismatch between the Ginkgo CLI and the version of Ginkgo imported by your packages:
Ginkgo CLI Version:
2.14.0
Mismatched package versions found:
2.17.1 used by e2e
Ginkgo will continue to attempt to run but you may see errors (including flag
parsing errors) and should either update your go.mod or your version of the
Ginkgo CLI to match.
To install the matching version of the CLI run
go install github.com/onsi/ginkgo/v2/ginkgo
from a path that contains a go.mod file.  Alternatively you can use
go run github.com/onsi/ginkgo/v2/ginkgo
from a path that contains a go.mod file to invoke the matching version of the
Ginkgo CLI.
If you are attempting to test multiple packages that each have a different
version of the Ginkgo library with a single Ginkgo CLI that is currently
unsupported.
Running Suite: e2e tests suite - /home/aaraj/ocm-forked-code/rosa/tests/e2e
Random Seed:<PHONE_NUMBER>
Will run 1 of 87 specs
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS•SSSSSSSSSSSSSSSS
Ran 1 of 87 Specs in 4.536 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 86 Skipped
PASS
Ginkgo ran 1 suite in 9.968965786s
Test Suite Passed
`

/test e2e-presubmits-pr-rosa-hcp

/lgtm


Search Bar Not Showing

I install Ghost on my computer. I installed Dawn Theme But the search bar Not showing.  I added gh_search_key in code injection but the search bar is not showing!!

Hey @PasinduPiumal1,
The API key for search function now uses the custom theme setting in Settings > Design > Site-wide instead of code injection method which was used previously. Sorry about the lack of docs, it'll soon be updated!


fix: Change INSTALL_PREFIX placeholder notation to templating notation

The former shell notation $INSTALL_PREFIX could result confusing. As
this is a placeholder that's replaced in the make, we switch it to a
template notation {{ INSTALL_PREFIX }}.
Also, fixes one occurrence of the placeholder in restic_backup.sh not replaced (was using ${} notation and not replaced by sed).

Most of the changes are a massive replace of $INSTALL_PREFIX to {{ INSTALL_PREFIX }} as commented here https://github.com/erikw/restic-automatic-backup-scheduler/pull/127#issuecomment-1851010961, but I split the PR in atomic commits to ease the review.


Blank pages thaumnomicom

Hi Aza, sorry to bother you again... even though I've used many times Thaumcraft i cant remember by hard recipes, especially alchemy ones, and some of my Thaumnomicom some pages are black, for example, if i want to see the alchemical brass recipe, the page opens but it's blank. Some others, when they are many tabs, shows one blank, and in the page of the following item, the recipe that should have been in the blank one....

if there is no fix for this, is there anywhere where i could see the recipes? kinda need them...

i think i found the issue and its related to thaumic pheripery, hold on i'll try changing the config to see if i can fix it...


Node version is displayed as 1.33.0 for the 'vasil-testnet-v1' tag

Internal/External
Internal   if an IOHK staff member.
Area
Other   Any other topic (Delegation, Ranking, ...).
Summary
A clear and specific description of what the bug is.
Steps to reproduce
Steps to reproduce the behavior:

build the node, using Cabal, from the tag 'vasil-testnet-v1'
run cardano-node --version cli command

Expected behavior
the node version should be accurate (in this case it should be at least 1.34.1 - the last released version)
System info (please complete the following information):

OS Name: [e.g. Ubuntu]
OS Version [e.g. 20.04]
Node version - 'vasil-testnet-v1' tag
CLI version - 'vasil-testnet-v1' tag

PR failed-->  https://github.com/input-output-hk/cardano-node/pull/4035

this is fixed in 1.35.0-rc3


How much energy is feed to grid?

Hola,
is it possible to get the value for how much energy i feed into the grid?

Sorry, for the question, i'm dumb. Found it. I've should read the docs.

Sorry, for the question, i'm dumb. Found it. I've should read the docs.


Big code cleanup

spa day for the angery boi
There was a lot to change, quite a bit of duplicated code, or functions stubbed by config checks.
I've not looked into the Overlay properly yet, and I feel it could be simplified quite a bit, but I'll be checking that later; done quite a lot today

Thanks a lot for the work so far. I'm going to merge this so that I can do a bugfix and work on a different feature, without merge conflicts in the future


[Bug]: Unable to build the VectorStoreIndex from simple PDF documents

Bug Description
I was following the documentation and trying to load the simple PDF document using SimpleDirectoryReader.
documents = SimpleDirectoryReader(r"MyLocalPath\InputFiles").load_data()
I am using a valid Azure OpenAI keys with turbo 3.5 model and text-embedding-ada-002 model for embeedings.
The data loading is done successfully and post that the vector indexing is not happening and returning the RetryError[]. Tried the documents directly from wikipedia using download_loader, but still the same error as below.
vector_index = VectorStoreIndex.from_documents(documents)
Tried the other way as below:
From the documents I have created nodes and tried to index those nodes too. But still the same error.
Error:
InvalidRequestError                       Traceback (most recent call last)
File [c:\\AppData\Local\Programs\Python\Python310\lib\site-packages\tenacity\__init__.py:382](file:///C://AppData/Local/Programs/Python/Python310/lib/site-packages/tenacity/__init__.py:382), in Retrying.__call__(self, fn, *args, **kwargs)
    381 try:
--> 382     result = fn(*args, **kwargs)
    383 except BaseException:  # noqa: B902

File [c:\AppData\Local\Programs\Python\Python310\lib\site-packages\llama_index\embeddings\openai.py:167](file:///C://AppData/Local/Programs/Python/Python310/lib/site-packages/llama_index/embeddings/openai.py:167), in get_embeddings(list_of_text, engine, **kwargs)
    165 list_of_text = [text.replace("\n", " ") for text in list_of_text]
--> 167 data = openai.Embedding.create(input=list_of_text, model=engine, **kwargs).data
    168 return [d["embedding"] for d in data]

File [c:\\AppData\Local\Programs\Python\Python310\lib\site-packages\openai\api_resources\embedding.py:33](file:///C://AppData/Local/Programs/Python/Python310/lib/site-packages/openai/api_resources/embedding.py:33), in Embedding.create(cls, *args, **kwargs)
     32 try:
---> 33     response = super().create(*args, **kwargs)
     35     # If a user specifies base64, we'll just return the encoded string.
     36     # This is only for the default case.

File [c:\\AppData\Local\Programs\Python\Python310\lib\site-packages\openai\api_resources\abstract\engine_api_resource.py:149](file:///C://AppData/Local/Programs/Python/Python310/lib/site-packages/openai/api_resources/abstract/engine_api_resource.py:149), in EngineAPIResource.create(cls, api_key, api_base, api_type, request_id, api_version, organization, **params)
    127 @classmethod
    128 def create(
    129     cls,
   (...)
    136     **params,
    137 ):
...
--> 326     raise retry_exc from fut.exception()
    328 if self.wait:
    329     sleep = self.wait(retry_state)

RetryError: RetryError[]

Version
llama-index 0.8.8
Python 3.10.0
Steps to Reproduce
Try to import the document form wikipedia and load it using download_loader
from llama_index import download_loader  WikipediaReader = download_loader("WikipediaReader")  loader = WikipediaReader()  documents = loader.load_data(pages=["2023 in science"], auto_suggest=False)
Post getting the documents loaded. Now try to create a vector indexer using the below code.
vector_index = VectorStoreIndex.from_documents(documents)

I am using Azure openAI and the keys and endpoints I have provided are correct. The reason I also tried to create the knowledge graph indexer via Neo4J and it is working and I am able to get the result for the same document. But the error exists only with the VectorStoreIndex.

@mukundhareddy1996 the kg index doesn't use embeddings.
If you are using azure, can you double check your setup follows this guide? Setting the global service context is pretty important for azure
https://gpt-index.readthedocs.io/en/stable/examples/customization/llms/AzureOpenAI.html

Thanks @logan-markewich
I followed that document and changed the code a little bit and it worked.
import os
import json
import openai
from langchain.embeddings import OpenAIEmbeddings
from llama_index.llms import AzureOpenAI
from llama_index import LangchainEmbedding
from llama_index import VectorStoreIndex, SimpleDirectoryReader, ServiceContext
import logging
import sys
from IPython.display import Markdown, display
logging.basicConfig(
    stream=sys.stdout, level=logging.INFO
)  # logging.DEBUG for more verbose output

logging.getLogger().addHandler(logging.StreamHandler(stream=sys.stdout))
openai.api_type = "azure"
openai.api_base = "Your Azure OpenAI deployment URL"
openai.api_version = "2023-03-15-preview"
os.environ["OPENAI_API_KEY"] = "Your Azure OpenAI key"
openai.api_key = os.getenv("OPENAI_API_KEY")

//llm = AzureOpenAI(engine="turbo35", mode="gpt-35-turbo")
llm = AzureOpenAI(
    deployment_name="Your turbo 3.5 Deployment Name",
    model_name ="gpt-35-turbo",
    temperature=0,
    openai_api_version=openai.api_version,
    model_kwargs={
        "api_key": openai.api_key,
        "api_base": openai.api_base,
        "api_type": openai.api_type,
        "api_version": openai.api_version,
    },
)
llm_predictor = LLMPredictor(llm=llm)

//You need to deploy your own embedding model as well as your own chat completion model
embedding_llm = LangchainEmbedding(
    OpenAIEmbeddings(
        model="text-embedding-ada-002",
        deployment="Your Embeddings deployment name",
        openai_api_key=openai.api_key,
        openai_api_base=openai.api_base,
        openai_api_type=openai.api_type,
        openai_api_version="2023-03-15-preview",
    ),
    embed_batch_size=1,
)

service_context = ServiceContext.from_defaults(
    llm_predictor=llm_predictor,
    embed_model=embedding_llm,
)

set_global_service_context(service_context)

When trying to define llm = AzureOpenAI(deployment_name = ....), I am getting the error "you must define an engine parameter". Any ideas how to fix that? I'm using llama-index 0.8.10 and langchain 0.0.274.

@yuvansharma try to follow the guide I sent above closely. Usually setting a global service context solves most issues

@logan-markewich Yup, I followed the guide, and I'm getting the same issue that the author was receiving, of not being able to make a VectorStoreIndex. I have set all environment variables correctly. This is my code, any insight would be greatly appreciated:
`llm = AzureOpenAI(engine="gpt-4-default-version", model="gpt-4", temperature=0)
llm_predictor = LLMPredictor(llm=llm)
embedding_llm = LangchainEmbedding(
OpenAIEmbeddings(
model="text-embedding-ada-002",
deployment="gpt-4-default-version",
openai_api_key=openai.api_key,
openai_api_base=openai.api_base,
openai_api_type=openai.api_type,
openai_api_version=openai.api_version,
),
embed_batch_size=1,
)
documents = SimpleDirectoryReader(input_dir="UploadedFiles").load_data()
service_context = ServiceContext.from_defaults(
llm_predictor=llm_predictor,
embed_model=embedding_llm,
)
set_global_service_context(service_context)
index = VectorStoreIndex.from_documents(documents)
query_engine = index.as_query_engine()`

The embeedings model name which you provided is gpt-4. Please deploy the model text-ada-embeedings 002 and use that. Hope this resolves your issue.

oh I see. Thank you so much! How can I deploy the text-ada-embeddings model in Azure? I only see options for gpt-3,gpt-4, gpt-3.5.

@yuvansharma

Also you can refer this - https://gpt-index.readthedocs.io/en/stable/examples/customization/llms/AzureOpenAI.html

I was running into a weird error saying resource not found:
This is what fixed the issue.
openai.api_version = "2023-05-15"


AsyncStorage cannot work on synchronous way.

async function _localStorage() {
	await AsyncStorage.getItem("g-value").then(v => {
		if (v) {
			global.gValue = v;
		} else {
			global.gValue = 'hello world!';
			AsyncStorage.setItem("g-value", global.gValue);
		}
	});
}
_localStorage().done();

alert(global.gValue); //get a undefined value.

It's called AsyncStorage because it's asynchronous. You cannot use it synchronously.
@facebook-github-bot answered

You can only get the value from inside the async function, make sure all the code that depend on that global is executed inside the async function.
async function _localStorage() {
    const v = await AsyncStorage.getItem("g-value");
    if (v) {
        global.gValue = v;
    } else {
        global.gValue = 'hello world!';
        AsyncStorage.setItem("g-value", global.gValue);
    }

    // Continue here...
    alert(global.gValue);
}
_localStorage();

How I can use AsyncStorage as redux InitialState of my application?

You can dispatch an action that restores the application state after you get it from AsyncStorage and have your app render nothing until it is loaded. You can also use something like https://github.com/rt2zz/redux-persist that will take care of saving / restoring state for you.

@max-mykhailenko
I do it this way,

dispatch an action LOAD_SAVED_DATA_REQUEST.
load the data from AsyncStorage when I receive this action
dispatch an action LOAD_SAVED_DATA with the payload as the new state when loading succeeds, or LOAD_SAVED_DATA_FAILED when loading fails
when I get LOAD_SAVED_DATA action with new state, I replace the current state with the new state in my reducer

You can also have a state isRestoringData or something to indicate if you're loading the data, so that you can show a splash screen or spinner, and then set it to false when you get LOAD_SAVED_DATA. Usually it is fast enough that I just return null in my component instead of showing a spinner.


Build tests on various distros only happens for Node client, not other clients

The Node client has some scripts to test that it is installable on 6 major distros. The other clients don't yet have that.
We should have the same sort of scripts that test installability for every client.

@sentientwaffle I don't think we run cross-distro tests for the Go or C# client (the latter maybe not as important, but the Go one is)

Oops, this got auto-closed when https://github.com/tigerbeetledb/tigerbeetle/pull/284 merged.

decided to not do this: this is a test for how to install node on distro x, not for how tigerbeetle works on distro x. We test on alpine, ubuntu, windows and mac, which gives us enough confidence that things generally work


metrics.sklearns don't work with DDP: Tensors must be CUDA and dense

🐛 Bug
Using sklearn-like metrics (from metrics.sklearns) works fine in DP mode, but results in the following error in DDP mode: Tensors must be CUDA and dense
To Reproduce
This colab notebook has a full example: https://colab.research.google.com/drive/1uxihs4pztF_qTN-gkO1QOaX0npzE_uSZ?usp=sharing
Mentioning @justusschock

@SkafteNicki @justusschock mind have look? 🐰

@borda PR https://github.com/PyTorchLightning/pytorch-lightning/pull/2528 seems to solve it (tried out the colab notebook with that branch and error was gone). Could you please review so we can get it merged?


sender.history() doesn't work

the sender.history function throws an exception on chats that are too long

Yes, that is a known limitation for big groups.
For future bug reports, "an exception" is not specific enough.


We should provide a comprehensive list of CFML resources.

I've started a resources section on the site. Let's build this up so it's useful for newbies and pros alike.

Did this ever get actioned? Does it still need to be open?


add errors doc

@critesjosh I wonder if you have some idea on

if we should add each error heading into the sidebar?
Or how to best create hyperlinks for the errors.
PS - lmk if you think the errors doc doesn't belong where I added it

I don't think we should add each heading to the sidebar, there are too many.
I think that's a fine place to add it.
As far as creating hyperlinks for the errors, where do we define them and how are they turned into markdown? Is it just building a string from our file definition? For reference, here is how hardhat does it.

Closing this since docs is moved to the monorepo


Can not disable Global Media Control any longer

System Details

OS: Windows 10
Thorium Version 123.0.6312.133 64-bit

Problem
I used to use the --disable-features=GlobalMediaControls flag on the startup shortcut to disable the Global Media Controls panel. Unfortunately, this no longer seems to work with the recent update. The #global-media-controls-updated-ui flag in the settings (and any other) also does not remove the button.
Screenshots

Additional Notes
I've tried to browse through the source for more insight on how the --disable-features= flag is parsed, and understand why it no longer works, however I can't seem to find anything regarding this in the source. I see some sporadic references to kEnableFeatures but that also doesn't seem actually used anywhere. The only file where that is used is a browser test .cc that makes references to various missing files such as content/shell/common/shell_switches.h. What is going on here?

Does the 123 version of Chrome or Chromium have the same problem?

@Banderi Try also disabling GlobalMediaControlsUpdatedUI

@Banderi Try also disabling GlobalMediaControlsUpdatedUI

I tried disabling that flag as well as hardware-media-key-handling and enable-web-app-system-media-controls-win and other combinations from comments I found on the net but I couldn't get it to work sadly.
Do you know perhaps where in the code the --disable-features= flags are parsed? I can't find it anywhere and the Chromium src search on github is bugged (stuck on indexing)

@Banderi Use the source code search for Chromium here > https://source.chromium.org/chromium
Most of these features are controlled by this file > https://source.chromium.org/chromium/chromium/src/+/main:media/base/media_switches.cc

@Banderi Use the source code search for Chromium here > https://source.chromium.org/chromium
Most of these features are controlled by this file > https://source.chromium.org/chromium/chromium/src/+/main:media/base/media_switches.cc

Ok, I think I understand my confusion now. I couldn't figure out why the searches weren't returning anything, and after looking through the source I couldn't figure out why there was whole sections missing from the folders. Now I realize that this fork doesn't include the Chromium source in its entirety, and requires third party tools to fetch the rest (depot_tools).
Also, there isn't a single file that tailors and lists all the feature flags recognized by Chromium as I initially assumed, instead the BASE_FEATURE(...) command is invoked in the files that pertain to those features (e.g. kGlobalMediaControls inside media_switches.cc) and there it is initialized and recorded to the engine. (This command and its internals are defined in the Chromium source but not in the fork since they haven't been touched.)
I'm gonna try to look now at the source directly now and see if anything changed with the usage of the kGlobalMediaControls flag and why it's not working on my end...

I'm using latest build from today and this is still broken on windows release. Disabling the flag #global-media-controls-updated-ui does not remove the icon when playing a video in YouTube for example

@Nicols0Mart @Banderi Yeah, it still doesn't seem to be able to be disabled, even with --disable-features=GlobalMediaControls


In TFnighty TFLite  Interpreter missing setUseNNAPI "wrapper" method  to call Interpreter.Options.setUseNNAPI

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): WIndows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: Android emulator
TensorFlow installed from (source or binary):  org.tensorflow:tensorflow-lite-gpu:0.0.0-nightly'
TensorFlow version (use command below): nightly
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
Making call to Interpreter's setUseNNAPI method which no longer exists in the tfnightly version as of Oct. 12.   The method it should call Interpreter.Options.setUseNNAPI still exists but, not in the Interperter itself.
Describe the expected behavior
Either change the documentation to reflect this method does not exist or fix this.
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
tfLite = new Interpreter(loadModelFile(assetManager, modelFilename));

if (tfLite != null) tfLite.setUseNNAPI(isChecked);

Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

Interpreter.Options is a class inside Interpreter. It was implemented back in 2018. To use it, you have to create an instance of Interpreter.Options. That is, for your code, it should be something like:
Interpreter.Options tfliteOptions = new Interpreter.Options();
tfliteOptions.setUseNNAPI(isChecked);
tfLite = new Interpreter(loadModelFile(assetManager, modelFilename), tfliteOptions);

I already know that Intrepreter.Options has this method and that is a work around to set the options in the constructer.  My issue report was that prior Oct. 12 (or there abouts) like the setNumThreads it was implemented in BOTH the Interpreter class and the Interpreter.Options.  This allowed the modification of the Interpreter object after construction.  Also, note: this is performed by the sample in the new TF2 Object Detection API mobile app as they have a GUI element that can toggle ON/OFF the use of the NNAPI for acceleration and I believe others will complain.

I understand now from the documentation you linked to it is removing of a deprecated method however, given this I suggest that the TF2 Object Detection sample provided be reworked and the option removed or altered to recreate with GUI event each time the Interpreter.   Otherwise, that team will get reports on broken sample code.

@grewe Got your point. I don't work for Google, just happened to use Interpreter.Options before. Tag Jared @jdduke who removed  Interpreter.setUseNNAPI method in https://github.com/tensorflow/tensorflow/commit/89c2f7f062ee125bd3783035b1be07b5c0f9ea73 according to git log.

The object detection sample has already been updated to use the Interpreter.Options variant. It may not be as convenient, but since 2019, using Interpreter.setUseNNAPI had a few nasty corner cases (e.g., you couldn't disable it once it has been enabled and used, and the first inference result is almost always incorrect).
We'll also likely be removing the Interpreter.setNumThreads() setter as well. It's note quite as flawed, but it does have some unfortunate corner cases as well.

If we don't use Interpreter.setNumThreads() as deprecated already then Interpreter.Options automatically collects data ? or new alternative method arrived.


operator kubernetes-imagepuller-operator (1.0.1)

Signed-off-by: David Kwon<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

/merge possible

/test all

/merge possible

/merge possible

@dkwon17 please be aware that your operator is unable to start on OCP 4.11. Please test it and prepare some future version to handle it.

/merge possible

Hello @J0zi, I had some trouble reproducing the issue on a 4.11 cluster. Could the failure be caused by the 60 Wait for the catalogsource pod to start up attempts in build-log.txt? I'm new to this so I'm not sure what could be causing that. Are there some other logs I should be looking at?


Update package.json

Allow newer versions of mongodb dependency required for node 6.x

@johnwalicki we need to be very careful making a change like that. It would allow the mongo module to make a major version number change, which implies breaking changes, and we would pick it up regardless. This is why we are generally overly prescriptive on the version numbers - the mqlight version needs tighten up.
Do you know what specific version of the mongo module is needed to resolve this?
Whilst we should tidy it up, worth pointing out the 'error' isn't a fatal one - just means the bson module will fall back to its pure js version, rather than the native code.


fix issue 19475 - add a staticIndexOf overload taking a pred

This overload allows to find the first index or the last index of an element of a alias sequence that verifies a predicate. Current staticIndexOf are principally specialized for types and are not adapted, let's say, to work on structures used as UDA.

Approval from @andralex is needed as this is a new public symbol.
Also as this is a new public symbol, a changelog entry is required.

I don't think that the remaining requested change is good. Maybe it would be possible to use the same signature (and then abandon firstStaticIndexOf and lastStaticINdexOf  and make the difference using constraints but i cant manage to right now.

This is counterproductive. In the usual work environment, how odd would this be?

Engineer comes with a good initiative
Project manager says, "cool but let's make these API changes before merging"
Engineer disagrees and throws the work away

This is a collective waste of everybody's time. Work has been put into implementing this. Work has been put in reviewing this. And now it's all gone. We really need a collective adjustment of attitude with regard to contributing to dlang. cc @thewilsonator

In the usual work environment, how odd would this be?

This is not a work environment.

This is a collective waste of everybody's time. Work has been put into implementing this. Work has been put in reviewing this. And now it's all gone.

The work is not gone. The diff is still there. BBasile hasn't even deleted his branch yet. You can still use the patch and improve on it in any way you like. The only thing that is gone is BBasile's willingness to work on this.

We really need a collective adjustment of attitude with regard to contributing to dlang. cc @thewilsonator

I suggest you check your own attitude. You don't get to boss volunteers around like you apparently want to. Deal with it. By calling for more servitude you will only push contributors away.

What you describe here is not totally what happened, as I said I may reopen this PR later.

Envoyé: vendredi 21 décembre 2018 à 15:15
De: "Andrei Alexandrescu"<EMAIL_ADDRESS>À: dlang/phobos<EMAIL_ADDRESS>Cc: BBasile<EMAIL_ADDRESS>"State change"<EMAIL_ADDRESS>Objet: Re: [dlang/phobos] fix issue 19475 - add a staticIndexOf overload taking a pred (#6801)
This is counterproductive. In the usual work environment, how odd would this be?
Engineer comes with a good initiative
Project manager says, "cool but let's make these API changes before merging"
Engineer disagrees and throws the work away

This is a collective waste of everybody's time. Work has been put into implementing this. Work has been put in reviewing this. And now it's all gone. We really need a collective adjustment of attitude with regard to contributing to dlang. cc @thewilsonator
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


Add support for build version qualifiers.

Coming from https://github.com/opensearch-project/opensearch-build/issues/1632, add support for -Dbuild.version_qualifier=.
The distribution build process now supports passing additional optional qualifiers other than the 3 digit versions, e.g. 2.0.0-alpha1. Add the ability to pass additional qualifier to build scripts, and generate artifacts with a version that includes the qualifier.

Hi @joshuali925, I see you handled the OpenSearch plugin, are you working on the OpenSearch Dashboards side as well.
Unfortunately for OpenSearch Dashboards we don't have a nice build param as OpenSearch side has yet. You will be able to build and connect successfully to OpenSearch Dashboards yarn plugin_helpers build --opensearch-dashboards-version=2.0.0-alpha1) with an OSD plugin <IP_ADDRESS> but this will fail within the distribution build.
As for 2.0.0-alpha1 we ask plugins to bump their version [here] to be <IP_ADDRESS>-alpha1 to be supported within the build distribution.
Let me know if you have any follow up questions.
Thanks!

@kavilla should opensearchDashboardsVersion be 2.0.0-alpha1? i checked core repo didn't find alpha1 qualifier in its version
{
  "id": "reportsDashboards",
  "version": "<IP_ADDRESS>-alpha1",
  "opensearchDashboardsVersion": "2.0.0-alpha1",

Up to you but I might share different opinions than others.
My opinion, is that leave opensearchDashboardsVersion untouched. You can then utilize OpenSearch Dashboards main. Within the build repo we build OpenSearch Dashboards and plugins with a qualifier. This will set OpenSearch Dashboards to be 2.0.0-alpha and within plugins `"opensearchDashboardsVersion": "2.0.0-alpha1"". So it will be compatible and succeed in the building of distributions.

I see, so in plugin builds opensearchDashboardsVersion can be overridden by build args but version cannot. PRs updated.

Correct! When I have allocation I will try to get that functionality in here: https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1398

we will convert from alpha1 to Rc after core is done with the change.


Concurrency issue in route.go

Hi,
There is a concurrency issue in route.go:104
at:
for k, v := range r.Pattern {
vars.v[req][v] = ss[k]
}
vars should have a mutex lock enabled before getting ss[k].
I'll see if I have time to create a pull request.

Eelco

If you just look a few lines above, it actually holds the lock the entire time it is accessing those variables already.  I believe the code you have here will deadlock.
At any rate, I have proposed some changes here which holds the lock for even less time on <go1.7 and doesn't need to use locks at all on go1.7.


Hovering over maps is jerky and flickers instead of smooth

Description

When mousing over features on a geoJSON layer the hovering on my local machine, in ALL browsers flickers instead of being smooth. However when other machines this problem does not occur
https://gph.is/g/E00Kb6m
In the gif above I move the cursor smoothly over the section and it flickers instead of staying highlighted.
Any idea how to fix this. It makes developing very annoying.
Repro Steps

Environment (please complete the following information):

Framework Version: [e.g. deck.gl 6.3.0]
Browser Version: [e.g. Chrome 71.0]
OS: [e.g. Mac OS X 10.4]

Logs

I haven't notice this when I tested locally. When you say it only happens on one particular machine, can you give more details of your machine. What version of deck.gl you are using.
Also, are you just rendering one Layer or more, is it possible there is other geometry at the same location with picking or autohighlight off? Any console warnings/errors?

@1chandu Since this is on Mac, possible Intel GPU issue? Perhaps the other machines have external GPUs?

Thank you for the replies. I am wondering if ibgreen is right about it being a hardware issue. As even the deckGL example for geoJSON polygons give me this issue.
I am using DeckGL 7.0.10
I am on a Mac Book Air. Mojave 10.14.5
Intel HD Graphics 6000 1536mb
There are no console warnings and I got rid of all layers except for the standard map and my GeoJSON. It still happened.

Ok, yeah possibly Intel GPU issue either a precession/shader instruction issue. I will try to reproduce and report back here.

I was able to get hold a system with Intel GPU HD Graphics 630 1536MB, tried with geoJson example on deck.gl website (documentation page) , but still not able to reproduce. I am trying to find similar machine as yours.

@Ventronik , I tested on same exact GPU as yours, but I am not able to reproduce, I am using demo on GeoJSON layer doc page : https://deck.gl/#/documentation/deckgl-api-reference/layers/geojson-layer

Any thing different in out setups?, If you haven't can you try above demo.

http://www.giphy.com/gifs/PlyVhrEwEqitYy9ohi

The only example I can get similar flashing to happen is in the Example for GeoJSON's and Their hovering tooltips. I also included a screen shot of my machine specs. Hopefully that helps.
When dong the demo it seems to be working fine, as evidenced in the GIF. Is the code for the demo on Github?

Will re-test after 7.3 is released, need to get hold of a similar machine.

@vectorijk , can you retest with latest release 8.0, please assign in back if you still see this, will try to get hold of a machine with similar GPU.


Include Axon Server

Axon Server is a zero-configuration message router and event store for modern app development.
This PR aims at introducing it as a test container since it is being used more and more.

Default docker image - Checked
Module dependencies - Checked
API - Checked

Very good initiative.
At the moment we have a home-grown version of this, we can test a pre-release of this module as well.
/ping @maartenjanvangool

Hi! I'm Ivan from AxonIQ (the company behind Axon Server and Axon Framework).
I would love to continue working on this PR so that we can offer an official test containers module for Axon Server (event store/database).
Is this still a valid way? Please let me know.

Hi @idugalic, the module can be hosted by Axon org itself and be listed in the Official Module Catalog by adding it in this repo. I recommend this as the way to go because you are the experts on it and would be the place to answer questions related to it. Feel free to reach out to us for feedback on the new module.