Make the data parameter of LoadEvent be a promise instead of a direct value, preventing a waterfall and allowing optimistic UI

Describe the problem
If you have both a server load function and a shared load function, the latter can get the results of the former by reading the data part of the load event.
The problem is that the shared load function (as well as the rendered page) have to wait for the server result of the server load function. This results with two problems:

Forcing a waterfall by waiting to the server load function results before processing the shared load function, when it can be processed in parallel.
When not on SSR, this prevents optimistic UI, since the page/layout couldn't start be rendered before the full response of the server. Notice that if you put the entire load logic only inside the shared load function, this optimistic UI mechanism is possible.

Describe the proposed solution
Just change the value of the data parameter to be a promise resolving to the server results. (Yes! Such a simple solution!)
This will also allow optimistic UI since the shared load function (if not in SSR) might decide to pass the promise of data directly to the page/layout component (while avoiding promise unwrapping of SvelteKit of course).
Alternatives considered
Forcefully avoid using server load functions, and only use a shared load function for this relevant context.
Importance
would make my life easier
Additional Information
In some sense it's a "blocker" for the other optimistic UI issues requested in #7213 and #7175.

We can take it a step further and say that the data will be a function returning a promise, meaning that if the shared load function doesn't call it, the request to the server load function wouldn't be called at all.

+1. This is related to https://github.com/sveltejs/kit/discussions/7119#discussioncomment-3781744

I'm not sure this gives us any speed boost given the current semantics: We rerun the shared load function only when it's deemed out of date. Part of that decision depends on the server load response - if it's null or type: 'skip' and other things are also determined be to up to date, the shared load function isn't called again. We cannot make this decision upfront if we wouldn't wait for the server data. If we wouldn't wait, this would mean potentially running the shared load function only to discard its execution mid-way, resulting in needless work.

I don't follow how this is a blocker for #7213 or #7175 (or #7119)? If we do implement some kind of loading skeleton screen mechanism (whether that's in the +layout.svelte as proposed in #7213, or a separate +loading.svelte file) then it would be completely independent of load functions.

We rerun the shared load function only when it's deemed out of date. Part of that decision depends on the server load response - if it's null or type: 'skip' and other things are also determined be to up to date, the shared load function isn't called again. We cannot make this decision upfront if we wouldn't wait for the server data.

The client does know which server load functions are invalid, because the client is the one telling the server which load functions are invalid (via the x-sveltekit-invalidated header when it requests server data). The only wrinkle is that the server might unexpectedly decide to throw an error or redirect, in which case shared load functions would be wasting work. That's arguably not the thing to optimise for though.
So the real question for me is more about ergonomic trade-offs than performance ones. Does needing to do this sort of thing...
export async function load({ data }) {
+ const foo = await load_some_stuff();
  return {
-   ...data,
-   foo: await load_some_stuff()
+   ...(await data),
+   foo
  };
}

...pay for the ability to prevent waterfalls in cases like this?
// src/routes/photo/[id]/+page.js
async function load({ data, params }) {
  await preload_image(params.id);
  return data;
}

I don't follow how this is a blocker for #7213 or #7175 (or #7119)?
This is a "blocker" using a hack I was hinting in the description:
You can bypass the promise unwrapping of data loading, simply by encapsulation of the promise on an object (i.e. {result: {promise: externalFetch()}}) , and then the invocation of the load function in this hack is non-blocking. Yes, I know, this is ugly but I thought it might be a good starting point for improving the API.

If we do implement some kind of loading skeleton screen mechanism (whether that's in the +layout.svelte as proposed in #7213, or a separate +loading.svelte file) then it would be completely independent of load functions.

I'm trying to make a clear suggestion, so I won't go into details, but my further suggestion is similar to the +loading.svelte file. I would like if we can drop out entirely the server-only loading function, and replace them by "resource providers"(you can think on them conceptually as "memoization of fetch request" per session, meaning that if on the same session the same provider is called twice with the same argument, it will be fetched only once) that are not dependent on the pages routing structure.
This means that there will be left only shared loading functions, server-side "resource providers"(that are publicly available to be queried both in server and client), and the server handle method that return the initial session value (which is passed to the shared loading function). Notice that such design cancels completely the waterfall issues. (There is another idea how to further store the result in the shared client/server side, but need to process it tough)
Will be glad to see if there was such a discussion.

So the real question for me is more about ergonomic trade-offs than performance ones. Does needing to do this sort of thing...
export async function load({ data }) {
+ const foo = await load_some_stuff();
  return {
-   ...data,
-   foo: await load_some_stuff()
+   ...(await data),
+   foo
  };
}

...pay for the ability to prevent waterfalls in cases like this?
// src/routes/photo/[id]/+page.js
async function load({ data, params }) {
  await preload_image(params.id);
  return data;
}

There are cases that you want to query data both from your SvelteKit server and another server (your own other/main backend, or 3rd party service), so this induces waterfall.

I would like if we can drop out entirely the server-only loading function, and replace them by "resource providers"

That's... not going to happen 😀 .

There are cases that you want to query data both from your SvelteKit server and another server (your own other/main backend, or 3rd party service), so this induces waterfall.

If you have a server-only load function anyway, why not fetch the external data there? That way everything gets batched into a single request.

I would like if we can drop out entirely the server-only loading function, and replace them by "resource providers"

That's... not going to happen 😀 .

Why strictly not? In my suggestion, when the load function is being computed on the server side, calling this "resource provider" is just calling to a async function(in the SvelteKit echo system). The idea is that also the client can call this. You can view these providers as a decomposition of the currently server-only loading function into (atomic?) parts.

Because that approach involves making multiple independent requests instead of having everything neatly batched up by the framework. The atomicity you describe is a bug, not a feature. Since these resource providers aren't linked to a request, they don't get access to the RequestEvent that contains things like event.locals (unless you run handle for each resource provider, which would be very wasteful in the case where handle does things like retrieve a user object from a session cookie). Getting rid of server-only load functions would also be a very significant breaking change.
We discussed this among the maintainers just now and decided that the cases where the current behaviour introduces an unnecessary waterfall are rare enough in practice that the ergonomic tradeoff isn't worth it. In those rare cases it's possible to avoid the waterfall by not having a +page.server.js alongside a +page.js; the +page.js load function can (for example) request data from a +server.js file. As such we've decided to take this off the 1.0 milestone


♻ Update SwaggerUI

Update SwaggerUI to 5.17.12.

📝 Docs preview for commit 9d42097afb8a4b0732fc7abb7d4dba28cc2d7144 at: https://3af7c6f2.fastapitiangolo.pages.dev

📝 Docs preview for commit 0d756e7a125b3b5dab6808fc98c2e1e1ec74e3b6 at: https://cbdd39e4.fastapitiangolo.pages.dev

Thanks @arttiger but this one was addressed in #11459 so we'll close this one but thanks for your interest. 🤓


Refresh token not saved after closing the app

Describe the bug?
When logging in we receive a Token that has a refresh token. After refreshing, the refresh token will be changed but after closing the app, the refresh token will be the same as when we logged in. This means that we cannot refresh again if we close the app (the Credential.refresh() function will go into the .failure case of the switch).  This seems like a storing problem
What is expected to happen?
Refresh token should be changed after refreshing and closing the app
What is the actual behavior?
Refresh token is the same as when logging in
Reproduction Steps?

Log in
Refresh the token as many times as you want
Close the app and enter again
Try to refresh again

Additional Information?
P.S.: This happens for the whole Token
SDK Version(s)
1.1.0
Build Information
No response

I can't reproduce this with our sample application. Would you be able to share some code samples showing how you're using this feature?
If you wanted to try in our sample app, you can open the OktaMobileSDK.xcworkspace and try the WebSignIn (iOS) sample.
When you store a token using [Credential.store](https://okta.github.io/okta-mobile-swift/development/authfoundation/documentation/authfoundation/credential), it will store it within the keychain. Once a refresh succeeds, it should automatically update the value in the keychain with the new tokens.

The default property will automatically be assigned when the first token is stored.  For example:
try Credential.store(token)

print(Credential.default) // Prints out the credential created above

For more information about managing credentials, and in particular how to use the default property, check out the following documentation: https://okta.github.io/okta-mobile-swift/development/authfoundation/documentation/authfoundation/managingusercredentials/#Working-with-the-default-credential
That being said, when a refresh is triggered, it should automatically update the corresponding value in the credential storage. Let me try something on my end to see if I can reproduce your issue.

This issue turned out to be difficult to reproduce, but very easy to fix.  I have https://github.com/okta/okta-mobile-swift/pull/108 up to address this issue, and a release will be submitted shortly.

@mikenachbaur-okta -Thanks for pushing the fix. I quickly tested this on sample app here https://github.com/okta/okta-idx-swift.. but Credential.default is still presenting the old values of token


Openstack terraform changes: Add (optional) dns_nameservers, add volu…

…me size parameters, and allow ssh_user passthrough to modules.

Does this require users to manually set the ssh-user? I think we should have some kind of default.

This looks like a good cleanup! Thanks @paulfrybarger-tgt.

Thanks!


fix: endpoint body handles properly internal links

Changelog
Fixed

Internal links are properly replaced inside body template

@nubinub Je te fais confiance :+1:


Use snapshot's .d file as source inputs in Gradle build.

If we don't yet have a .d file (first build), fall back to using the
.dart files in the current directory. This enables us to detect changes
in dependent source files (Flutter framework, packages outside the
source directory, etc.), and re-generate the snapshots as needed.
Unfortunately, Gradle requires knowing the source files before executing
the task, and can't update them after building, so Gradle considers the
second build to be out-of-date (because it has more input files than the
first build). Sub-sequent builds have the correct dependency
information, and will be skipped if the source files haven't changed.
Also added a dependency on gen_snapshot. The snapshot ABI isn't stable,
so we need to re-generate the snapshots when we roll the Dart SDK
dependency.
Fixes #8315
Fixes #8687
Fixes #8607

FYI @jason-simmons @abarth


Hide warning messages for skipped repositories explicitly excluded from workspace

First of all thank you for sharing this awesome tool with the community.
Environment

Output of tsrc version: tsrc version 0.9.2
Output of python --version: Python 2.7.15
Operating system: MacOSX Catalina 10.15.1

Manifest.yml
repos:
        - src: projects/project-1
          url: git@myrepo-url:projects/project-1.git
        - src: projects/project-2
          url: git@myrepo-url:projects/project-4.git
        - src: projects/project-3
          url: git@myrepo-url:projects/project-3.git
        - src: projects/project-4
          url: git@myrepo-url:projects/project-4.git
        ...
        - src: projects/project-100
          url: git@myrepo-url:projects/project-100.git
groups:
        group1:
                repos: [projects/project-1, projects/project-2]
        group2:
                repos: [projects/project-3, projects/project-4, projects/project-5]
        ...

Init only for group1
tsrc init git@myrepo-url:projects/meta-repo-.git --group group1

Command ran
either with or without --quiet mode:
$ tsrc -q foreach -c 'git status'

Actual output
$ tsrc -q foreach -c 'git status'
:: Running `git status` on 2 repos
* (1/2) projects/project-1
$ git status
On branch feature/x-1
nothing to commit, working tree clean
* (2/2) projects/project-2
$ git status
On branch feature/x-1
nothing to commit, working tree clean
Warning: The following repos were skipped:
Warning: * projects/project-3
Warning: * projects/project-4
...
Warning: * projects/project-99
Warning: * projects/project-100

Expected result
I would expect to see only the warning messages regarding the repos of group1. Warning messages about skipped repositories are too much since I explicitly initialized my workspace only for group1 and not interested in other repos.
:: Running `git status` on 2 repos
* (1/2) projects/project-1
$ git status
On branch feature/x-1
nothing to commit, working tree clean
* (2/2) projects/project-2
$ git status
On branch feature/x-1

First of all thank you for sharing this awesome tool with the community.

You're welcome.
This issue will probably get fixed when we tackle the big "Revamp groups UX" milestone.
If you want you can give your opinion on the #180 issue


Update ControlsV2 scrollbar template

This commit updates our scrollbar template to microsoft-ui-xaml at ceeab5f.

This is great! Does it fix any issues? What are the upstream reasons for the changes?
Should we service it?

This is great! Does it fix any issues? What are the upstream reasons for the changes?

It contains just a single fix. This one: https://github.com/microsoft/microsoft-ui-xaml/pull/7081
The remaining changes are just me making it a bit easier for us to incorporate upstream changes in the future (in my personal opinion).


Inconsistent "DROP TABLE YES NO" warning

Description
When you attempt the DROP TABLE …; statement execution, dBeaver shows up a warning… usually.
But it seems its appearance depends on the table name.
DROP TABLE cache; - no warning.
DROP TABLE `cache`; - warning pops up.
DBeaver Version
<IP_ADDRESS>404211624
Operating System
Windows 10
Database and driver
MySQL 8.0, using default driver.
Steps to reproduce
No response
Additional context
No response

Thank you for bug report.
It's a bug in JSQLParser library, which can't parse drop table cache query.


Errors when publishing to Digital Ocean Container Registry

Testing this for a project where I need to publish images to Digital Ocean's container registry and running into an error.
I can build and image and manually push it, like this:
dotnet publish -p:PublishProfile=DefaultContainer -p:ContainerRepository=registry.digitalocean.com/practical-asp-net/blazordemo
docker push registry.digitalocean.com/practical-asp-net/blazordemo

With that working I figured I'd trying using dotnet to push the image as well.
My assumption was that I'd be able to do this:
dotnet publish -p:PublishProfile=DefaultContainer -p:ContainerRegistry=registry.digitalocean.com  -p:ContainerRepository=practical-asp-net/blazordemo

(hopefully I got that syntax right).
However, I'm getting a Bad Request response when I do.
See attached binlog for details.
msbuild.zip
Thanks,
PS: I'm using .NET 8 Preview 7

Hi @jonhilt - I've been able to reproduce this failure on my test harness with the latest version of the tooling - I think this may be a quirk/bug in how we do basic auth but I haven't been able to confirm that.
For now, I think the best thing to do would be to push to a local docker daemon (by removing the ContainerRegistry property) and then use docker tag and docker push to push the generated container to Digital Ocean.

Yeah - the problem is an interesting chain of behaviors:

we authenticate successfully to repository A
we start upload layers to A
we try to make the process faster by asking repository A to 'mount' the layer from repository B, if it exists
DigitalOcean sees that we haven't requested access to B, and tells us we need a key that needs both write scope for repository A and read scope for repository B
We attempt to get that key, and DO tells us we have an invalid request (because we passed a scope with a space in it: scope=repository%3achethusk-sdk-container-demo%2fsdk-container-demo%3apull%2cpushrepository%3adotnet%2faspnet%3apull)
We blow up

I checked what the Docker CLI does, and it doesn't even attempt the 'mount' operation - it always attempts to upload the layer from the local machine to the destination. I think we should make the 'mount' operation specifically resilient to authentication failures (i.e. catch the UnableToAccessRepositoryException) and continue on - if we do so then we flow right to the same upload-from-local behavior that the Docker CLI does (though it would also be great to understand why DO doesn't seem to handle scopes like this.

We are likely not going to be able to get this into .NET 8.0.100 release itself - we're about to create branches for RC2 and that's historically when we shift into a much higher bar.  We can look at this enhancement for 8.0.200 in the Jan/Feb time frame, however - though it's lower priority than some of the big rocks we have like tackling multi-RID publish, etc. This one would be a great candidate for an engaged user to contribute to the tooling - some potential fixes:
More registry-level quirk knobs
We already have 'quirk' modes for a few features, like parallel upload of Layers, and chunked upload of Layers. Various registries support different modes, and we have environment variables that let users control the quirks more directly. We could add another quirk for "should we attempt to 'mount' the Layer from the registry if it exists". If yes, we would do the current pattern of "try mount, else pull from the source and upload from the local machine". If no, we would always "pull from the source and upload from the local machine"
Try to handle this kind of error directly in the "try to mount Layer" call
Alternatively, we could put more robust error handling into the Mount Layer API call to try and detect this kind of failure and recover from it (effectively returning false from that API call for more scenarios than it does today). From there the rest of the system would continue uploading the full layer.
I lean towards the first approach, and I'm sure there are more.


Implement a dummy DeviceInfo component for react-native-web usage.

When using this package under react-native-web, the DeviceInfo component is not found, because it's not implemented by react-native-web. Instead, under web, supply our own dummy.

@davepack could you please look at this? Although I think https://github.com/react-community/react-native-safe-area-view/pull/18 might be a better solution, but at least merge one of them? Thanks.

landed #18 instead, thanks!


fix(versions): update Activiti/activiti-cloud-full-chart-security versions

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.390

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.390


0000-Hold_Rewards_for_Denylist_Hotspots

Perhaps a rethink from the ground up how HNT is distributed each Epoch?  New code at point of release to send a very small percentage of HNT each Epoch to a reserve pool (held in trust by the Foundation or Nova labs)  used for compensation purposes. any unused tokens in the pool after a year are then redistributed to all as rewards. when a hotspot is wrongly put on the Deny list an then removed it should be a simple calculation to work out how long in days they have been on it. Crowdspot could notify the a 'Compensations Panel' with the Trustees to issue compensation from the compensations pool to the wallet that miner is in. Either at a variable day earnings average i.e 0.1 HNT right now or by a flat rate per day or any other method voted as fair.

Needs some tweaking, but I like the core concept. 🙂👍🏻

Renaming to avoid confusion with HIP-76

Thanks for the suggesting @samgutentag I merged it in, please also feel free to edit the authors to add yourself to the list. I do not have all the answers to the other enquiries right now, I'll ask the fellow authors how they feel and update accordingly. However outside contribution is greatly encouraged.

hey @robputt there's still some things that need to be resolved on this. Can you see the changes requested?

Hi.  Let's collaborate.   I think we are working toward the same goal. Reduce the wait on crowdspot, streamline the appeal process, and find a way to compensate false positive gotspots.
I have a doc that I wrote here.  Feel free to pull any ideas from this.
https://docs.google.com/document/d/1YTHrD4CDyT4dT9lYXMZTwqlvM7Me3gyeY_usu49NQz4/edit?usp=drivesdk

@hiptron
Rob has confirmed this PR can be closed.

Closing this on behalf of the HIP Editors.


GPS hdop

@gervaisK @DonLakeFlyer we used the position accuracy in meters so far (which seems more intuitive since you can judge wether you think its accurate enough or not). However, the MAVLink spec (that I wrote myself) calls for hdop / vdop, and this PR changes to use them.
I'm not sure this is an "improvement", but I think consistency is more important in that case. PX4 will continue to use the position accuracy in meters internally to determine when its safe to set home and lock.

@LorenzMeier : I´m not sure, if its understandable, why I propose this change. That wouldn´t be required, if one tends to stick to former PX4 users. Won´t be a problem for brand new users as well, who think to get a along with a very limited resolution or don´t care at all. I always recommended to stick to U-Center standards.
While it might not be that important for planes, with copters reception above a HDOP of 1.8 is not usable for GNSS related modes, imo requires a red area representing huge position jumps. That is the reason, I´m in favour of (better) visualizing the usable HDOP range  from <1 to 1.8
Same problem with analysis graphics with linear splitted HDOP, e.g. from 1-16, while the interesting HDOP range from 0.5 to 2 always sticks at the bottom of the plot. (e.g. LogMuncher)
At APM, MP used to show PDOP values, calling them HDOP about 2 yrs and the confusion was big, because the U-Center always showed better values. Meanwhile this has been corrected and users got along well with the switch.

Users should never look at decimal values. But that's a different issue.
What exact label and numeric value do you want? The receiver delivers pdop by default.

Rebased and applied to master.


Bump versions

Describe your changes

Migrate nx to the latest version (17.2.7)
nx migrate latest
yarn
yarn nx migrate --run-migrations
delete migrations.json
remove gh-pages (now we have pipeline)
Update yarn version
Testing/validation

Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)

[ ] I have commented my code, in hard-to understand areas.
[ ] I have made corresponding changes to README or wiki.
[ ] For front-end changes, I have updated the corresponding English translations.
[ ] Ran yarn run mini-ci locally to validate format + lint.
[ ] If there were format issues, I ran nx format write to resolve them automatically.

Why did the zips in .yarn/cache/ change when the version didn't change in commit 0d7c51206a1d7fd5c412bed179cea5ddd3e6f193?

I honestly don't know. It is jumping from yarn v3 to v4 so I assumed there is something there.

fyi https://typescript-eslint.io/blog/announcing-typescript-eslint-v6/#rule-breaking-changes

Several rules were changed in significant enough ways to be considered breaking changes:
Previously deprecated rules are deleted (chore(eslint-plugin): remove deprecated rules for v6):
@typescript-eslint/no-duplicate-imports, replaced by imports/no-duplicates

Been working on the imports/no-duplicates conversion for quite some time, a lot of weird errors popping up, maybe this version bump is more work than anticipated...


Add strings.isDate(dateFormat)

We don't have a easy check in the ringo/utils/strings module if a string matches a specific date format string.

Work in progress: https://github.com/ringo/ringojs/tree/issues/341-342-stringdates

Fixed in #343


Fix model download permissions

This PR depends on https://github.com/SubstraFoundation/substra-chaincode/pull/79

Adds a new check on permission preventing model downloads from nodes who don't have permissions
Relies on a new chaincode smartContract to retrieve the permissions associated with a model.

To reproduce the original issue:

run test_permissions_denied_model_process from substra-tests
get the model url from the traintuple whose algo is xxxxxxx_test_permissions_denied_model_process - Algo 1 This model should only be processable by MyOrg2MSP.
curl the URL using the node1 to node2 credentials: MyOrg1MSP:nodeSecret2w1

You should get a 200 without the fix and a 403 with the fix.

@Kelvin-M Good catch, thanks a lot!

@Kelvin-M Tested and approved: I can only retrieve the model with the owner's node credentials, and never with user credentials.


Add script to safely merge daria databases

I rule and have completed some work on Case Manager that's ready for review!
We are ready to go here! The idea here is:

we make a new fund in the instance
we set that fund's id and that fund's current db creds as env vars
we run something like rails migrate_db_multitenant

This then copies db rows over and offsets everything to preserve relations/refs, including in papertrail. It's pretty fast (I've been running it against a prod db for some stress testing and it takes roughly a minute).
How to test drive:

run rails db:seed, then run this sql to clean out your db entirely except for funds: delete from call_list_entries; delete from events; delete from patients; delete from archived_patients; delete from configs; delete from clinics; delete from users; delete from lines; delete from calls; delete from versions; delete from notes;  delete from practical_supports; delete from external_pledges; delete from fulfillments;
get the first fund id with echo "Fund.first.id" | rails c
export MIGRATION_FUND_ID=(that id)
export MIGRATION_DB_URL=(sandbox db url, assuming sandbox is set up to have one tenant currently)
rails multitenant_db_merge
reset the password of<EMAIL_ADDRESS>log in to your local and ensure it has parity with the remote db you just loaded in. it should!

This pull request makes the following changes:

postgres migration script

no view changes
It relates to the following issue #s:

Bumps #1817

For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

one remaining problem: created_at / updated_at are -5h as they're getting converted to the same time in eastern. an easy fix: just add 5 hours to every timestamp?

per offline discussion, I feel good about where this is at, so merging!


we need unit tests and coverage reports

We have unit tests; but we still don’t have coverage reports.

@v0lkan with https://github.com/vmware-tanzu/secrets-manager/pull/377 MR merged, it will show coverage as below:
ok      github.com/vmware-tanzu/secrets-manager/core/log        0.300s  coverage: 13.8% of statements ok      github.com/vmware-tanzu/secrets-manager/core/probe      (cached)        coverage: 12.5% of statements ok      github.com/vmware-tanzu/secrets-manager/core/template   (cached)        coverage: 90.0% of statements ok      github.com/vmware-tanzu/secrets-manager/core/validation (cached)        coverage: 100.0% of statements
If this coverage is enough then we can close #78 now.

Yup. we can close this one.
We need a place to save those coverage reports, but that can be a separate feature.


Delete obsolete AzureBlobStorage instead of Microsoft.Bot.Builder.Azure.Blobs

Fixes #
Proposed Changes

Delete obsolete AzureBlobStorage instead of Microsoft.Bot.Builder.Azure.Blobs

Testing

Thanks @ThePiz

Thanks @ThePiz


Add OWNERS

/assign @radoslawc

/lgtm

asked @aravind254 because I don't have r/w here, OWNERS added, this can be closed


Got benchmarks?

Have you released any benchmarks for comparison to other ML frameworks?
Thanks!

that would be interesting to see :+1:

Thanks for the suggestion guys. Gluon by itself is just an API. We know that the implementation on top of MXNet is almost as fast and scalable as MXNet symbolic, thanks to Hybridization (and as long as your layers can be hybridized).
I'm closing this issue since this is an implementation question, and not an API spec question.
Feel free to reopen if you think otherwise.


Add leex/heex to projectionist

This adds the html.leex/html.heex files to projectionist.
Only odd part on this is that we can't have multiple alternates so :A will jump to the test which will jump back to the .ex file.
Alternatives/Questions

We could have the alternate file be the .ex file instead of the test.
Do we want a basic ["<div>", "", "</div>"] template or just leave it blank?

Usage
Due to the ambiguity these would mostly be used via :Eleex and :Eheex to switch to the file or create it if it does not exist.

@zolrath when https://github.com/elixir-tools/elixir-tools.nvim/pull/108 merges, you should have a foundation for adding projectionist tests.
you can run the tests with bin/test or bin/test path/to/file.
You should run this with >= 0.9 for it to work well.
I'm copy pasting this comment to your other PR

Only odd part on this is that we can't have multiple alternates

I think you can
						*projectionist-alternate*
"alternate"
        Determines the destination of the |projectionist-:A| command.  If this
        is a list, the first readable file will be used.  Will also be used as
        a default for |projectionist-related|.

@zolrath let me know if you can get to this in the next week or so. if you can, i'll wait to cut the next release, but if not, i'll cut it and this can go in the next one


fix(i18n): add mapping locale rules

ML-190

Deploy preview ready!
Built with commit dffdce702f56e5654928e0674238d72d926ec024
https://deploy-preview-364--mcslite.netlify.com

Deploy preview ready!
Built with commit dffdce702f56e5654928e0674238d72d926ec024
https://deploy-preview-364--mcs-lite-introduction.netlify.com

Deploy preview ready!
Built with commit dffdce702f56e5654928e0674238d72d926ec024
https://deploy-preview-364--mcs-lite-ui.netlify.com


Fix bug in triggers emitted on key value pair changes and sensor spawn/exit.

We were publishing the triggers with just name before. This failed lookup when creating trigger instances. So basically this feature never worked before.
I added tests for sensor spawn and exit but for KVP, the code has multiple levels of abstraction.

@manasdk found the root cause. So praise him. I am just a donkey writing code. No brain involved.

Overall LGTM.

Good catch.
This one was pretty nasty since triggers would still get dispatched on the queue, but the lookup / creation of trigger instance in the rules engine would fail, right?

LGTM

+1


NO MRG: Test CIs

Had some issues with openssl being installed in the environment due to linking issues. This attempts to add an empty commit to see if the same issues exist on the CIs.

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

Closing as no longer relevant.


New NEO gui.  Is there a Wallet .db3 file that I can download to catch up?

I have downloaded the new NEO gui on my PC Windows 10.  It has been over 20 hours now and I have only 1414935/1239872.   The download freezes...continually "waiting for new block", but does nothing.  I have found that I have to exit  and restart the wallet program.  After this is done, the download restarts.  However, the download shortly thereafter freezes again.  Is there a file I can download or can anybody help in another way?  Thanks.

Same here, I don't have a .db3 file like the antshares gui wallet.

If you previously had been using the AntSharesCore gui, you can speed up sync by copying the Chain folder from the root of the AntSharesCore directory to the root of your new Neo directory.  The client must be shutdown when you do this.  When you launch neo-gui, the chain should already be up to the height it was the last time you ran AntSharesUI.exe and only have to sync blocks since then.
As for the db3 file, if you already had one you should be able to simply open it up with neo-gui and your SystemShares (NEO) will be there.

What if I never had an AntShares gui before? I did not. Then what world I do to speed up the process?
Sent from ProtonMail mobile
-------- Original Message --------
On Aug 11, 2017, 21:23, LanceS wrote:

If you previously had been using the AntSharesCore gui, you can speed up sync by copying the Chain folder from the root of the AntSharesCore directory to the root of your new Neo directory. The client must be shutdown when you do this. When you launch neo-gui, the chain should already be up to the height it was the last time you ran AntSharesUI.exe and only have to sync blocks since then.
As for the db3 file, if you already had one you should be able to simply open it up with neo-gui and your SystemShares (NEO) will be there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Download https://www.antshares.org/client/chain.acc.zip
Unzip in the neo directory and then launch neo-gui

There we go...Thx!!
Sent from ProtonMail mobile
-------- Original Message --------
On Aug 11, 2017, 23:19, LanceS wrote:

Download https://www.antshares.org/client/chain.acc.zip
Unzip in the neo directory and then launch neo-gui
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

it seems the problem has been resolved. I will close this issue.

Thanks guys.  Yes, new .db3 installed.  I imported my antshares wallet into the new wallet, so now I have 2 wallet addresses.  Can I delete the one I just created on the new database and keep my trusty old one?  Or is that way too much of a pain and I just don't need to use the new wallet address now that the old one is linked. Thanks again!

Make sure that there is no asset in the address you want to delete. Otherwise, there is no chance to recover them. You must to be very careful.

I am not going to mess with it.   The Oracle told me I am not the One.  :(   ;D

New URL: https://neo.org/client/chain.acc.zip


ERROR  There are no commands defined in the "lrd" namespace.

Describe the bug

To Reproduce
Steps to reproduce the behavior:

composer create-project laravel/laravel:^9 laravel
composer require rakutentech/laravel-request-docs
php artisan serve
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows, Mac] win
Browser [e.g. chrome, safari] edge

Versions (please complete the following information):

Laravel Version: 9.19
PHP Version: 8.1
Laravel Request Docs Version:

Additional context
Add any other context about the problem here.

lrd:generate is not longer in v2.


fix: add lost state for pb actors

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1379
#1378  👈 (View in Graphite)
#1376
#1375
#1364
#1362
#1356
#1355
#1354
#1349
#1326
#1325
#1324
#1304
#1317
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


DO NOT MERGE! Fast Forward stable to c6eeaf8cec12d92dc66e7152b2d1484ebbf450a5.

This PR will be merged automatically once checks are successful.

This change is

Jenkins job mixer/postsubmit passed


Implement "Users become inactive after a year" rule using FakeGithub adapter

This is done! See f40feb01ba32e8736a83959516fa221305321146


Using a large BAI or CSI Index with Composite Function

Hello! I have a very large set of chromosomes (749Mb is the largest) I'm working on which BAI indexes are too small for, and this seems to be throwing and error with methylartist composite specifically:
2023-02-09 15:51:39,395 starting methylartist with command: /anaconda3/envs/methylartist/bin/methylartist composite -s HAL1-3_ME.BED31.bed -r assembly.fasta -t HAL1-3_ME_Consensus.fasta -b Methyl.bam --motif CG 2023-02-09 15:51:39,864 fetching mod types from Methyl.bam, if this takes awhile MM/ML tags may be missing. 2023-02-09 15:51:39,892 found mods: m 2023-02-09 15:51:39,899 found mod names: m 2023-02-09 15:51:39,899 using the one available mod: m 2023-02-09 15:51:39,899 composite output filename: Methyl.HAL1-3_ME.BED31.composite.png 2023-02-09 15:51:39,957 profiling Methyl.bam: chr1:10701-12136:+:m phase: None Traceback (most recent call last):   File "/anaconda3/envs/methylartist/bin/methylartist", line 5313, in <module>     main(args)   File "/anaconda3/envs/methylartist/bin/methylartist", line 5010, in main     args.func(args)   File "/anaconda3/envs/methylartist/bin/methylartist", line 4655, in composite     elif c[4] in ('-', '+'):          ~^^^ IndexError: list index out of range
I've managed to successfully run methyartist segplot with the same bamtools BAI indexes without any issues. Is there a way around this where I can use a CSI index or the bamtools forced BAI index? Thank you!

Hey, I bet this is a bug. Looking at the error message what I think is going on here is that the file HAL1-3_ME.BED31.bed doesn't have enough columns in it. the -s option to composite should be fine with a BED3+1 but it appears that I'm trying to check the 5th column (c[4]). Will fix but as a quick workaround, try adding something as an additional column (i.e. make it BED3+2) - doesn't really matter what. Let me know if that works.
If that doesn't solve it I'll test with a .csi index but they should be supported as I'm just using pysam to do the .bam parsing.

Thanks so much for the quick response! Unfortunately there still seems to be a hangup - I tried moving around the strand information to column 5, adding more information, and copying the strand to multiple columns. I get the same error each time, but not the index error:
strand (+/-) not found in cols 4 or 5 of HAL1-3_ME.BED32.bed
Up until this point, it gives the exact same output as above prior to the "Traceback (most recent call)....".

848c7be should fix the index oob error. Not sure about the further issue here but assume it was a record in the .bed file not complying with chrom, start, end, strand, Please re-open if still an issue you're contending with.


Fixed unused uri for bad gateway test.

My cell phone provider in Germany use the IP address <IP_ADDRESS> for the ByteMobile Inc. BMI service. It is a service not claimed by me as I know to save traffic by modify content and compress images. So this IP is used and breaks the test and the build.
I've simply tried <IP_ADDRESS> and it's working for me.

That's pretty funny. LGTM!


how to connect to Oracle as SYSDBA?

↑

you can specify the following in connection string DBA PRIVILEGE=value
the accepted value is one of the following
SYSDBA
SYSOPER

I add a new commit that will add SYSDBA automatically if you connect with sys user

Thanks


Store qualified counters in dedicated capacity limited cache

@didierofrivia here is what I am thinking...

These are all the changes... By default, this now tries to use 70% of the available memory at server startup to store "qualifies counters", it'll emit a WARNing level log if you don't explicitly set the cache's size using -c, like so:
[2023-08-04T14:51:22Z WARN  limitador_server] No cache size provided, aiming at 70% of 6019MB, i.e. 17259155 entries

Left some comments, nothing blocking TBH, feel free to address them if it makes sense or just comment :)

I don't think anything should be addressed, but feel free to disagree... One thing which might be useful is setting a hard limit on what we know would be limitador's memory requirement. But we don't as of now... I could open an issue for that so that we at some point measure and address that. But right now, 70% isn't any better or worse than the previously hardcoded value... it's as bad, just differently. But at least, now, someone can set it to something that makes sense to their use-case.


Fix robot page exception on undefined dataset

This PR is to fix the robot page exception that customer hit in buildModelMetadata function.
For better user experience, we should find a way to check all the props, and return meaningful error messages when certain property is undefined. Maybe we can achieve that directly inside AML studio UI code.
Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

This can be fixed in AzureML repo. Close this PR.


[Backend] API to Get the influencers data form their Social Handles added

Description
End point is added. Now you can get the data using that end point. eg: localhost:3000/social/:id
Issue
Issue Link resolve #2
Screenshot or video
Video Link
api response

@Kajol-Kumari  or @RohitGupta1237  can you please check and merge it.


Door Lock controls

Can you add the lock and unlock commands to the lock ?

Yes, i am on it. Problem with lock and unlock, kia doesn't generate a response immediately and response arrives with push notification over google cloud messaging.
The response can be negative or positive, so you car might receive and process your message or generate a failure. To receive this response, i am looking at options;

subscribing google cloud messaging notifications
polling command status to find out status

I am closer to the second one, still investigating

Ok, I see. I think the status update would work ? FYI, Bluelinky does get a confirmation message, not sure how they do it.

I will double check bluelinky implementation
On Sun, Apr 11, 2021, 18:11 holdestmade @.***> wrote:

Ok, I see. I think the status update would work ? FYI, Bluelinky does get
a confirmation message, not sure how they do it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/fuatakgun/kia_uvo/issues/10#issuecomment-817331774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUSN7VXF2UMH3G7OGMGNS3TIHC2FANCNFSM42XYBP2A
.

should be implemented here: https://github.com/fuatakgun/kia_uvo/commit/f8c1c6474ff8fdf61e372238ae5d450dfd1d36b9
it is still a bit buggy, needs improvement

Only the binary_sensor is still showing and not a lock entity ?

again my bad, commit has been merged, please refresh with latest code.
it is locking/unlocking with data will not be updated immediately.

The lock works fine but never updates the status
I tried unlocking and immediately calling the update service and it updated right away, not sure what delay you are using ?

So, i have implemented 10 seconds waiting time because when i send a lock/unlock command, i am receiving a response as 'command delivered' but not 'command executed'. If you get the latest code, you can see that the state will be updated in 10 seconds after sending the command

please get the latest version, it should be all fixed.

Thanks, nearly works. When unlocking, the state is updating but when locking, the state will not update, only with a force refresh
Also noted that if I do more than 2 locks/unlocks, it fails with a Duplicate request error from Kia

because lock/unlock/force_update is time consuming, they are not happening immediately.
I will extend force_update request to 3 seconds after sending lock request.
and if it fails, it will try again in 30 seconds for 5 times

get latest version, error will be gone.
I am not sure why lock/unlock is behaving weird. It is same for me when I am using UVO app.
Can you check your experience with UVO app and let me know if it is different?

With the Uvo app I need to specify my PIN in order to carry out any remote actions on the vehicle (e.g. lock/unlock, or turning on the AC), does that need to be implemented here?

interestingly no, I did not find any actual requirement to get;

location data
lock/unlock
get latest data

I do not know why we are forced to use PIN here. So far, it works fine for me without PIN

get latest version, error will be gone.
I am not sure why lock/unlock is behaving weird. It is same for me when I am using UVO app.
Can you check your experience with UVO app and let me know if it is different?

Yes UVO is similar, seems to be limited to how many times in succession you can perform lock/unlock

let me know if we can resolve this

interestingly no, I did not find any actual requirement to get;

location data
lock/unlock
get latest data

my car does not support remote start/stop, i will try and send this command over API and share my findings.
I do not know why we are forced to use PIN here. So far, it works fine for me without PIN

In bluelinky nodered, in the config you have to add Brand (Kia or Hyundai) and PIN

interestingly no, I did not find any actual requirement to get;

location data
lock/unlock
get latest data

my car does not support remote start/stop, i will try and send this command over API and share my findings.
I do not know why we are forced to use PIN here. So far, it works fine for me without PIN

In bluelinky nodered, in the config you have to add Brand (Kia or Hyundai) and PIN

I know, still, I am not able to find a need for asking for PIN.

All seems to work now, with approx 10 secs delay for update


EntityUtils assemblyOfEntities should be distinct

Console.WriteLine("Same assembly passed more than once, you may one to check arguments of EntityUtils!");


Object cache support

Blocked by https://github.com/WordPress/gutenberg/issues/55853

https://wordpress.org/plugins/sqlite-object-cache/ could be a workaround for this.

With the move to Playground this won't be possible in the short term, but perhaps later on. See https://github.com/WordPress/wordpress-playground/issues/207
Closing as maybelater


Options to overflow lists of items into a separate dialog/screen

Unsure whether I should have them only show the first few items + an "expand" button to make them bigger, first few items with a "view more" button leading to a dialog or activity, ONLY show a "view" button (without the first few items), or some combination of the above.

Think I have a good idea about how I'll go about this. There will be ONE attribute that controls this feature: overflow (an integer, >= -1). Basically, if overflow="-1", all items will be shown in the list. If overflow="5", and the amount of items in the list is greater than 5, an "expand" arrow will be displayed at the bottom that opens all of the items in a new dialog. If overflow="0", the only item that will be displayed is a "View %s" button ("%s" being the title attribute of that list).


not collecting jira_issue_commits but we should be?

The code is being called but no data is in the DB. I think this code is still being used but I need confirmation from @klesh who wrote this 22 days ago.
Steps to reproduce:

Run all services locally (make dev)
Collect from Jira board 8
Check the db for records in jira_issue_commits

Expected:
There should be data
Actual:
There is no data

@joncodo i think it might be caused by the diff sync  feature, try to truncate the jira_issues table and recollect.

Can we close this ?

I think so, feel free to reopen if problem still exists


Getting Started refers to config.js and admin.js

See:
https://github.com/marmelab/ng-admin/blob/master/doc/Getting-started.md
At times it mentions config.js, at others it mentions admin.js, which is confusing. Are they supposed to be the same thing (and if so, the inconsistency should be addressed) or are they actually two different files?

Thanks for the heads up. There is only one file, named admin.js. The tutorial is now fixed.


id.google.com

I have no idea what this is: id.google.com
I think we dont need this. But 1 problem, id.google.xx have many domain extensions. As you use google.nl, than you have id.google.nl as you use google.be, then you have id.google.be
Who can tell me more about this? Is it possible to block this?

Hi all, id.google.com

I have no idea what this is: id.google.com
I think we dont need this. But 1 problem, id.google.xx have many domain extensions. As you use google.nl, than you have id.google.nl as you use google.be, then you have id.google.be
Who can tell me more about this? Is it possible to block this?

It has something to do with ADS. You can block this. ;)

Alright, I know this is an old and closed ticket, but it caught my attention. Searching my Blocklist Tools website shows a number of lists blocking id.google.com. The oldest record I have is the Bjornstar: Hosts project which has had the domain blocked since at least 2013-08-28T15:57:44.000Z - which is the oldest version I'm aware of. ANYWAYS, what caught my eye is that id.google.com was blocked in the unified list for a single day Mar 23 2018 -> Mar 24 2018. What is even more interesting is that the dates didn't line up with any of the other included lists. After reviewing the commits, I discovered that the Bjornstar list was actually included in the unified project for a single day.
Added: https://github.com/StevenBlack/hosts/issues/543
Removed: https://github.com/StevenBlack/hosts/issues/548
Fascinating (at least to me 😆)
Back on topic. If you are determined to block id.google.com but do not want to make the entry yourself, there are a number of other lists that you can easily include in your clone of this repo:

Bjornstar: Hosts
Jerryn70: GoodbyeAds
Llacb47 mischosts: Blacklist

Thanks Dan @lightswitch05 I find this interesting too 😄


Simplify history.CreateAccounts() to avoid select statement

The CreateAccounts , CreateHistoryClaimableBalances ,  CreateAssets, and CreateHistoryLiquidityPools functions are used to insert records into the history_accounts, history_claimable_balances, history_assets, and history_liquidity_pools tables. The functions are implemented by first executing insert statements into the respective tables followed by executing select statements to lookup the rows which where just inserted.
However, the select statements can be avoided because we can add a RETURNING clause to the insert statements which will return the row which was just inserted.

The RETURNING clause in an INSERT statement only returns the newly inserted rows. To return all rows, including existing ones, we can force an update using the ON CONFLICT UPDATE SET...  clause. However this additional update will likely have a negative impact on performance.
To improve reap performance, we plan to add a last_Reference_ledger column to these tables. This column will be updated every time which will allow us to use the RETURNING clause. However, when reingesting older ledgers, we may not want to update the last_Reference_ledger if it is greater than the one we are reingesting. So at this point I'm not sure that we can combine the two statements.


MINOR ImageTest: Failing tests with fixture 'imageWithMetacharacters'

The test methods ImageTest->testGeneratedImageDeletion() and ImageTest->testMultipleGenerateManipulationCallsImageDeletion() fail to report that the resampled image could not be deleted.
Bug:

Run GDImageTest
Tests run without failures
assets/ImageTest_resampled/SetHeight100-SetWidth200-test_image).png still exists
assets/ImageTest_resampled/SetWidth200-test_image).png still exists

I'm uncertain how to fix this and why there should be a bracket in the filename.

Neither of those tests exists anymore in https://github.com/silverstripe/silverstripe-assets


Windows / Super key support

I can't find any sign of support for the windows/start/super key modifier. Is the library able to generate this?

Got similar issue for Mac and Windows.

@lyneca: this should be possible with autopy.key.tap(autopy.key.Modifier.META, []), I've updated the docs accordingly: https://github.com/autopilot-rs/autopy/commit/86c4488bb3d3543868352925d4ad36298ac82c97.

https://github.com/autopilot-rs/autopy/issues/27#issuecomment-468876832 is a duplicate of #29.


msystem: MING32 seems unusable

I have two jobs, one with msystem: MING32 and one with msystem: MING64. They are identical, except that they install i686 and x86_64 packages respectively:
jobs:
  build_msys2_64:
    runs-on: windows-latest

    steps:
    - uses: actions/checkout@v2
    - name: Init submodules
      run: git submodule update --init

    - name: Setup Msys2 environment
      uses: numworks/setup-msys2@v1
      with:
        msystem: MINGW64
        update: true

    # Update again. Needed to actually get all updates.
    - name: Update msys2
      run: msys2do pacman --noconfirm -Syuu

    - name: Install deps
      run: msys2do pacman --noconfirm -S --needed
        base-devel
        git
        mingw-w64-x86_64-SDL_net
        mingw-w64-x86_64-cmake
        mingw-w64-x86_64-glib2
        mingw-w64-x86_64-libsndfile
        mingw-w64-x86_64-ninja
        mingw-w64-x86_64-toolchain
        zip

    - name: Build
      run: |
        cd libretro
        msys2do ./github_msys2_build.sh
        msys2do zip -9 windows-x64.zip dosbox_core_libretro.dll

    - uses: "marvinpinto/action-automatic-releases@latest"
      with:
        repo_token: "${{ secrets.GITHUB_TOKEN }}"
        automatic_release_tag: "latest_build"
        prerelease: true
        title: "Latest Build"
        files: libretro/windows-x64.zip

  build_msys2_32:
    runs-on: windows-latest

    steps:
    - uses: actions/checkout@v2
    - name: Init submodules
      run: git submodule update --init

    - name: Setup Msys2 environment
      uses: numworks/setup-msys2@v1
      with:
        msystem: MING32
        update: true

    # Update again. Needed to actually get all updates.
    - name: Update msys2
      run: msys2do pacman --noconfirm -Syuu

    - name: Install deps
      run: msys2do pacman --noconfirm -S --needed
        base-devel
        git
        mingw-w64-i686-SDL_net
        mingw-w64-i686-cmake
        mingw-w64-i686-glib2
        mingw-w64-i686-libsndfile
        mingw-w64-i686-ninja
        mingw-w64-i686-toolchain
        zip

    - name: Build
      run: |
        cd libretro
        msys2do ./github_msys2_build.sh
        msys2do zip -9 windows-x86.zip dosbox_core_libretro.dll

    - uses: "marvinpinto/action-automatic-releases@latest"
      with:
        repo_token: "${{ secrets.GITHUB_TOKEN }}"
        automatic_release_tag: "latest_build"
        prerelease: true
        title: "Latest Build"
        files: libretro/windows-x86.zip

The build_msys2_64 works fine. build_msys2_32 on the other hand fails with errors like:
cmake: command not found
strip: command not found

The msys2 update and package installation steps work, but using any of the tools is not possible. They seem to not be in the path?

Ah, ffs. As soon as I posted this and went over the text, I noticed I used MING32 instead of MINGW32. Sorry for the noise.

I wonder if this issue can be reopened as an enhancement:

It might be possible to enumerate the allowed values in action.yml.
Alternatively, it should be possible to check it in msys2do.cmd.


[NotificationService] NS support consuming message since earliest or latest version

Describe the feature

NS should support consuming message since earliest or latest version like kafka, latest by default.
Describe the solution you'd like

Client could pass version as earliest or latest when start_listen_event, Notification Master handles the race condition and return messages since specific version.
Describe alternatives you've considered

Additional context

AIFlow has been moved to https://github.com/flink-extended/ai-flow.
Please reopen the issue there if necessary.


Multiple sensors

Is it possible to use multiple instances for multiple pairs of sensors?
(e. g. basement and attic)
thx.

Yes you can. Just click "Configuration" -> "Integrations" and add the integration a second or third time...
I will update the instructions next week.

Fixed with 99fd797751321efaf51147614774d858de1c3b27


Updated helm chart repository url for prometheus to reflect the recent changes

https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/3124/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/3127/

Can you check if we need to modify any sample documentation.

checked , there is  nothing to correct in the sample document

We have one failure on parallel run.

it is not related to my change and looks intermittent
did not find file /u01/shared/domains/domainonpv-lqaw/servers/admin-server/data/store/default/_WLS_ADMIN-SERVER000000.DAT in pod domainonpv-lqaw-admin-server in namespace ns-lqaw ==> expected:  but was:


Update helm and kube config maps to follow new format, update versions.

Changes of note

Helm 2.8.2 -> 2.9.1
kubectl 1.9.4 -> 1.13.4
Minikube 0.25.0 -> 0.35.0
Kube and Helm config maps updated to new format.
Helm config map is now templated to range over origins and caches in values.

A thought I had while doing this is that we could put the comments of what each thing is in values.yaml in stead of the configmap like prometheus does in their chart. Thoughts?

Comments in the values.yaml are preferred over the configmap, I think. You can always link to trickster docs for anything too complex.

Two more things:

You'll want to update the version/appVersion in the chart.yaml
Not sure if you want to update docker image version now


Fixed ClassCastException in AndroidScanner #1474

Local classLoader typed to ClassLoader, not PathClassLoader
Removed unnecessary cast causing error.

Thanks for investigating and fixing this! Truly appreciated!


[SPARK-35445][SQL] Reduce the execution time of DeduplicateRelations

What changes were proposed in this pull request?

This PR reduces the execution time of DeduplicateRelations by:

use Set instead Seq to check duplicate relations

avoid plan output traverse and attribute rewrites when there are no changes in the children plan

Why are the changes needed?

Rule DeduplicateRelations is slow.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Run TPCDSQuerySuite and checked the run time of DeduplicateRelations. The time has been reduced by 77.9% after this PR.

cc @cloud-fan @maropu Please take a look, thanks!

Test build #138712 has started for PR 32590 at commit f085918.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43233/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43233/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43233/

Test build #138712 has finished for PR 32590 at commit f085918.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class ReferenceEqualPlanWrapper(plan: LogicalPlan)

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138712/

Test build #138737 has started for PR 32590 at commit ef50b77.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43260/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43260/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43260/

Test build #138737 has finished for PR 32590 at commit ef50b77.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138737/

Test build #138782 has started for PR 32590 at commit 47c603d.

Thanks, merging to master

thanks all!

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43299/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138782/


How to acquire the KEGG KO information file?

Here is the enquiry email sending from a current user which might help those who have similar concerns.

Hi
I tested for one file, generated the ##.species.txt from HSFinder.py.
How do I generate the ##.species_ko.txt ?
Creating Heatmap ?

Hi Glad you have worked it out. The KEGG KO file is acquired from KEGG KO BLAST engine https://www.kegg.jp/ghostkoala/ , which is pretty straightforward to use. Simply submit the protein data and the email to receive file, you will receive the KO file. I also detailed and demonstrate the steps in HSDFinder tutorial, Please find the Step 6 from the link: https://github.com/zx0223winner/HSDFinder/blob/master/Tutorial/Tutorial for HSDFinder.pdf
g10.t1  K07566
g11.t1
g12.t1
g13.t1
g14.t1
g15.t1  K09481
g16.t1  K00472
Once you have the KO file, you can either compare different thresholds of HSDs in one species in a heatmap or HSDs from different species (if you have respective HSD result file and KO file) in a heatmap (examples attached).
~Xi

Q: Can't we do it by command line version in KEGG ?
A: Did you mean the heatmap or the KO file? The KO file seems can only be acquired from KEGG. If you worried about the speed of online heatmap option in HSFinder web server ( if you have tried, it is actually not that slow. e.g., 10 mins for human genome). I can send you the heatmap script but it might have to be comfortable with command lines environment. ~Xi

Did you mean the heatmap or the KO file? The KO file seems can only be acquired from KEGG. If you worried about the speed of online heatmap option in HSFinder web server ( if you have tried, it is actually not that slow. e.g., 10 mins for human genome). I can send you the heatmap script but you might have to be comfortable with command lines environment. ~Xi

Here is the enquiry email sending from a current user which might help those who have similar concerns.

I can only submit one job at a time to the KEGG KO BLAST
Its very slow, maybe it's because of the big data set perhaps I think.

Here is the enquiry email sending from a current user which might help those who have similar concerns.

Can't we do it by command line version in KEGG ? I had a min of 50,000 protein sequences in each genome. Don't we have access to the command line version of KEGG KO BLAST to speed up the process ?

Did you mean the heatmap or the KO file? The KO file seems can only be acquired from KEGG. If you worried about the speed of online heatmap option in HSFinder web server ( if you have tried, it is actually not that slow. e.g., 10 mins for human genome). I can send you the heatmap script but you might have to be comfortable with command lines environment. ~Xi

I can only submit one job at a time to the KEGG KO BLAST Its very slow, maybe it's because of the big data set perhaps I think.

ou can submit KEGG jobs with different emails. It is slow but definitely worth it. I could not find an easier way to do it so far.


PHASE 1: Update claim popup text

https://xd.adobe.com/view/3aa8b6ec-58e7-4364-810c-ebfcb78921ab-6ffb/
*** RELEASE ONLY WHEN PHASE 1 IS LAUNCHED ***

@LiavGut @tomerGD
what goods and services? maybe we should write something more realistic

@yaroslav-fedyshyn-nordwhale don't start this task right now, please wait until ready

@sirpy
The text should be  "Claim your G$'s today!"

@sirpy  the text displayed according to the requirement
Checked on DEV V0.25.3
LG V30+ // Android 8.0 // Google Chrome 83.0.4103.106
Desktop: Windows 10 // Google Chrome 83.0.4103.116


Usage of unencrypted HTTP protocol

Platform:

[x]  Dart
[x]  Flutter Android or iOS
[ ]  Flutter Web

IDE:

[ ]  VSCode
[x]  IntelliJ/AS
[ ]  XCode
[ ]  Other, which one?

split-debug-info and obfuscate (Flutter Android or iOS) or CanvasKit (Flutter Web):

[ ]  Enabled
[x]  Disabled

Platform installed with:

[x] pub.dev
[ ] GitHub

Output of the command flutter doctor -v below:
The output goes here...
The version of the SDK (See pubspec.lock):
4.0.1

I have the following issue:
Hello guys,
I'm working on one app related to investments which is having requirements of security testing. Well, tester has submitted the security report using two different tools

MOBSF: https://mobsf.github.io/docs/#/requirements
ImmuniWeb: https://www.immuniweb.com/

Both reports looks almost identical and having common errors and warnings related to sentry. I've search related to those security details and couldn't found any good source to deal with the report.
Can anyone help me to resolve to get a good score in security testing. Here are some snaps from the report.

Steps to reproduce:

Integrated latest library

Actual result:

It should bypass security test

Expected result:

Showing unecrypted error.

hey @babulpatel1309 thanks for reporting.
regarding https://cwe.mitre.org/data/definitions/319.html if you are using a https DSN to set up Sentry, it's not an issue at all, the connection class is actually HttpsURLConnection and not HttpURLConnection, as we support both cases, we have to use the parent class, see:
https://github.com/getsentry/sentry-java/blob/81594398ee623b40eb9df1a847dbe21d9e4c25ef/sentry/src/main/java/io/sentry/transport/HttpConnection.java
both are pretty much the same, see the Java impl. https://docs.oracle.com/javase/7/docs/api/javax/net/ssl/HttpsURLConnection.html
I don't think we can do anything about that, it's already using SSL features, just be sure to use a https DSN

Also worth noting the new DSN's from sentry o123.ingest.sentry.io don't support plain http.

@marandaneto thanks for your prompt response. Well, I'm using https DSN in the application. It just that the security testing tool was capturing the HTTP class and was marking it as vulnerability.

@marandaneto thanks for your prompt response. Well, I'm using https DSN in the application. It just that the security testing tool was capturing the HTTP class and was marking it as vulnerability.

@babulpatel1309 yep, I know, it's not the 1st issue like this, these tools are pretty good but it's known for false positives, happy to clear this up.


Quick and Dirty Multi Platform Testing | GUI

This can be a basis of quick multi platform testing if you support many platforms (https://github.com/Microsoft/BashOnWindows/issues/1) when developing on Windows. Since many of the current system are GUI based it would be great if you can support windowing also.

Running GUI based apps is something we get quite a bit.  We have a very similar user voice here.
Closing this out since there has not been any traffic for 22 days.


HTTP connection with Sunray App 1.0.153 is broken

The HTTP adapter fails with 400 Bad Requests on Sunray App 1.0.153 because the content-type of the requests is now correctly text/plain. This clashes with the work arounds for the previously sent application/x-www-form-urlencoded content-type.

Quick fix is a work around on top of the other work around in the forked ESPAsyncWebServer library: https://github.com/timotto/ESPAsyncWebServer/commit/473f108b733e238e651e148ce5cbd81decadd4f6


Many floating artifacts in the air.

Hi, Congrats for your great work!
But when I reproduced your code on the Scannet scan6, I found that there were many floating artifacts in the air. Why does this happen? And how to avoid this problem?

I would be grateful for any help you can provide!

And I think the background is also a little strange. This link contains the results I reproduced.

Looking forward to your reply!

Hi, I think the reason is:
In the preprocessing step, the world coordinates are transformed to a unit cube, i.e., all the camera poses are transformed, and the final marching cube is conducted in this cube. If I remembered correctly, the size of this unit cube is the maximum value of x, y, and z in all the original camera poses.
However, in this scene all the training views only cover a part of the cube, i.e. the floating areas are areas that haven't been optimized in the training, and never be observed in all training views.

Hello, thanks for your reply.
These floating areas are places that are not observed from the input perspective, but the reconstructed scene I got using MonoSDF under the same input data is closed and does not have so many floating objects. In addition to the training views, what is the other problem? Is it also related to bg regularization or SDF transform method (NeuS or VolSDF)? You have a discussion about SDF and rendering method in supp (E.2. Backbone Ablation Study). Is there any conclusion on this issue at that time?

I think you're right. That's another reason. Before the unified framework SDFStudio came out, I used to try replacing VolSDF in MonoSDF to NeuS, and for the indoor scenes, the structures outside the room (unobserved in training) produced by two methods behave in a different way. NeuS would tend to have much more floating artifacts in the unobservable regions.
The difference includes:

implementation of eikonal loss
ray sampling
volume rendering transformation

I didn't do thorough ablation experiments after that, but NeuS and VolSDF would perform very differently in the unobservable regions.

I see, thanks for your reply!


Forgot password - Red dot doesn't appeared after lot of request

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch Desktop App
On the login screen, enter a existing email and click/tap continue
On the password page, Click/tap on the "Forgot?" link
Tap on forgot link again 3-4 times

Expected Result:
Red dot and error so many attempts should be appeared.
Actual Result:
Red dot and error doesn't appear. Able sent link a lot of times
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/213815529-95b2c997-8e36-481a-bbc1-f32da30ac8e2.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

not overdue, assigned over weekend. I'll do this today

I think we should expect some throttling to prevent spamming the action

Ok, cool thanks!  Do you agree that this can be completed by 'External'?

No, I think this has to be throttled in the API.

An option is to also do nothing if spamming this endpoint doesn't matter much. I think we send an email each time the user clicks, and I'm not sure if that can be a problem.
I'll ask in engineering.

Just a note that I wanted to add is maybe it is due to the login issues that the site was experiencing and IPs were whitelisted manually as mentioned here: https://expensify.slack.com/archives/C01GTK53T8Q/p1670957921425999
And without this whitelisting, the site would give 429 - too many requests error which might be the rate limiting factor. Although I am not sure, but maybe the reason that you are not getting the 429 error while requesting password after so many attempts is because the IP maybe whitelisted and the backend may not rate-limit it

@esh-g I also had that thought. I only tested in dev without the staging API. It may be true that at some point I would get 429 - too many requests if I do this with production/staging.
Thanks for the input!
It was decided in the slack conversation to do nothing for now because:

Once #passwordless is implemented, this won't exist.
This is not being abused at the moment.

UP job posting closed!

@abekkala do we have to do anything in regards of the UpWork job https://www.upwork.com/jobs/~01924de2228436f496? close it manually?


add ArrayHelper::unique()

This implemented often faster than native array_unique()
benchmark array_unique() vs \yii\helpers\ArrayHelper::unique()

My benchmark results are:
array_unique($arr)              to 666667 in 5.1696400642395
foreach + array_keys            to 666667 in 0.42175698280334
ArrayHelper::unique($arr)       to 666667 in 0.25078105926514
ArrayHelper::unique($arr, true) to 666667 in 0.29126000404358

I think this is a good idea, but I think case with non-scalar data should be handled with exception

Probably, you should add a note to changelog

Verified that indeed this method is faster. Any idea why?

Verified that indeed this method is faster. Any idea why?

because array_unique works not this way. it does sorting on the array and dependend on the sorting different strings are considered equal or not. http://php.net/manual/en/function.array-unique.php
We should carefully check if this is the expected behaviors in all places where the new option is used.

another issue that comes to mind is that ArrayHelper::unique([0, false, "", 1, "1", true]) will return [2 => 0, 5 => 1]

ArrayHelper::map() this isn't the same as array_map()
I use both depending on the situation.
ArrayHelper::unique() is not intended to completely replace the native array_unique(). ArrayHelper::unique() - accelerates the most frequent use case.

the documentation should be more clear about this and explain the difference to php array_map in detail.

So apart from those missing annotations, this pull request would be good to go? Or can I help in any way by providing some additional tests/functionality?

It's good apart from what I've commented on.


Get rid of extra indirections (WIP)

Setting out to implement the ideas in #12.

Store a Vector_ in the root node instead of an
Array of Vector_s. This simplifies almost everything
and should make the code smaller. This seems especially
important for folds, which we generally want to inline. But
we get one extra indirection per vector until we actually
unlift everything.

Make Vector_ a GADT. This lets us use the type system to
enforce the height balance invariant. Ultimately, this is going
to have to look different, but I think it's not a bad first step.

Add a validity test to check the rest of the data structure
invariants.

A couple thoughts:

I think I actually want to work mostly with UnliftedArray# rather than UnliftedArray. So I probably want a module providing relatively user-friendly operations on UnliftedArray# (using Int rather than Int#, using ST with Box rather than explicit state-token passing, etc).

To make things as backwards compatible as possible, use a data family rather than an injective type family. But we want to discard those newtype constructors with as little fuss as possible. I'm developing a pretty decent sense of how I think I want that to work.

There is another alternative to working mostly with UnliftedArray#, I think. The idea is to generalize Array and UnliftedArray to a unified type. This will be annoying, but it might shift the burden of ugliness into a simpler space.
-- Keep the kind open to support unboxed arrays?
data L
data U
type family Blob (l :: Type) (a :: Type) :: TYPE 'UnliftedRep
type instance Blob L a = Array# a
type instance Blob U a = UnliftedArray# (Unlifted a)

data Array (l :: Type) (a :: Type) = Array (Blob l a)

What's this do? It lets us yank an Array out of an Array 'U without having to determine whether it's actually an Array# or an UnliftedArray#.

No, that second thing isn't quite right. We need to add in an assumption that an Array U holds an Array x... hmm... Might not work out.

Yeah, I think I'm going with the first idea. It sucks, but at least I know how to make it work.


i18n(zh-cn): Update 3-components/4.mdx

Description (required)
Update 3-components/4.mdx
Related issues & labels (optional)

#9240
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

zh-cn/tutorial/3-components/4.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


check 0-rtt packet length when queueing

When we queue 0-rtt packets in undecryptable_pkts, check
the packet length (payload_len) and its actual buffer size
and return CryptoFail error if the packet size is smaller than its
length field in the packet payload.
This is to prevent the case that actual packet length is smaller
than the size in the payload (packet length field) which may
cause a runtime panic. Such packet is undecryptable in any way,
so drop it when received.

bussasassera

เมื่อ พ. 5 พ.ค. 2564 เวลา 16:07 Alessandro Ghedini @.***>
เขียนว่า:

@.**** approved this pull request.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cloudflare/quiche/pull/918#pullrequestreview-652082246,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGOMMRJU7ADGKBCW3FSYSDTMEDFRANCNFSM44BK4EAA
.

--
null


Add sanity check to ensure that all data decoded by parity_scale_codec are consumed

I don't think this is an issue at all

Marin, [20.04.22 12:54]
the byte representations match for VersionedQueryResult and paginated version

Marin, [20.04.22 12:54]
to a point

Marin, [20.04.22 12:54]
if it was the other way around, it would fail. If you were encoding QueryResult and decoding PaginatedQueryResult

Marin, [20.04.22 12:55]
if there was something after if in the encoded form, it would fail as well

Aleksandr Petrosyan, [20.04.22 12:55]
[Forwarded from Aleksandr Petrosyan]
Thankfully it doesn't decode QueryResult, but the fact that it happily works with types containing VersionedQueryResult is a little worrying

Marin, [20.04.22 12:55]
but seems that scale happily disregards trailing bytes if it finished decoding an object

Aleksandr Petrosyan, [20.04.22 12:56]
[In reply to Marin]
It works more like a parser than a binary serialisation format.

Marin, [20.04.22 12:56]
we can inject a check I guess

Marin, [20.04.22 12:56]
to see if all bytes were consumed

Marin, [20.04.22 12:57]
check that parity_scale_codec::Input is depleted

Aleksandr Petrosyan, [20.04.22 12:58]
Letting unrecognised data into a blockchain is a ticking time-bomb.

I'll give an example of what's happening here. Let's say we have two structures:
struct Struct1 {
    a: u32,
    b: u16,
}
struct Struct2 {
    a: u32,
}

where Struct2 is a subtype of the Struct1 in the encoded format meaning that all valid encodings of Struct1 will successfully decode into Struct2 and there will always be 2 trailing bytes left at the end of the encoded array. I'm not sure this is a security issue or how could this be exploited, or if it's a feature or a bug.
If this is not a feature that we need/want then we have to check that the parity_scale_codec::Input is depleted after decoding. This can be done by wrapping the Decode::decode<I: Input>(input: &mut I) method with a wrapper and make sure that all the decode calls go through the wrapper

I think we can do it by introducing our own macro of (de-)coding and use it in derives


Options repeat themselves for every instance of the date range picker

I have three date range pickers in my application and the ranges seem to be copied everytime there in a new instance of the date picker. For example my first picker has one copy of the ranges, the second has two copies, and the third has three copies. Do you know why it would be doing this? I have tried giving all the date pickers separate names/IDs and separate option lists but that hasn't worked.

I have the same issue


500 server error on malformed UTF-8 characters

Hey,
Nice little rss reader 🤟
It seems one of my feed have encoding problems, so the app shows error 500.
Can you fixed it?
Thanks in advance.

[2022-07-04 11:21:51] production.ERROR: Malformed UTF-8 characters, possibly incorrectly encoded {"exception":"[object] (InvalidArgumentException(code: 0): Malformed UTF-8 characters, possibly incorrectly encoded at /app/vendor/laravel/framework/src/Illuminate/Http/JsonResponse.php:91)

Thanks @droM4X,
Do you perhaphs have a feed URL to share that causes such a scenario? Just need to be able to reproduce the issue.

@ssddanbrown in my case<EMAIL_ADDRESS>is problematic. this url is came from here<EMAIL_ADDRESS>

@ssddanbrown in my case<EMAIL_ADDRESS>is problematic. this url is came from here<EMAIL_ADDRESS>
URL encoded address is just fine to me.

At the moment, I got 500 for multiple feeds what is worked in the last 24 hours.
For example:
https://24.hu/feed/
https://telex.hu/rss
24.hu is the biggest hungarian news site
Error is same in laravel.log:
production.ERROR: Malformed UTF-8 characters, possibly incorrectly encoded {"exception":"[object] (InvalidArgumentException(code: 0): Malformed UTF-8 characters, possibly incorrectly encoded at /app/vendor/laravel/framework/src/Illuminate/Http/JsonResponse.php:91)

Also sometimes I got js error and blank screen and cache clear isn't help at all.
Got this yesterday evening, but it worked again in the morning. At the moment I get this error again. Maybe its cause this problem?

VM113:1 Uncaught (in promise) SyntaxError: Unexpected end of JSON input
at JSON.parse ()
at Nf (app.ab3aa705.js:12:898)
at app.ab3aa705.js:12:1766

@droM4X Thanks for the further info.
Using the provided feeds, I identified a case where this could occur, and have patched the scenario in v1.0.4.
Note, this fix will only apply to new posts imported and posts updated from more recent fetches.
To prevent older posts in the database from being an issue, you may need to clear the post descriptions. If using our docker-compose setup this can be done like so:
docker-compose exec rss /usr/bin/php -r "(new SQLite3('/app/storage/database/database.sqlite'))->exec('UPDATE posts set description=\"\"');"

Post descriptions will be refreshed/updated for newer posts upon future imports.


Extend CI testing with more "smoke tests"

Inspired by: https://mattsegal.dev/devops-academic-research.html
Until we write unit tests for critical modules, our CI testing should include at least 2 or three different usage scenarios as "smoke test" (see Smoke Test section) to ensure nothing breaks.
Some scenarios to test:

Use visualization features
Use deeplabv3 with concatenation point for NIR band
Use debug mode
Use at least one more loss (ex.: sgd)
Use at least one model architecture from segmentation models pytorch (ex.: Unet pretrained)
etc.

Have a look at https://docs.docker.com/ci-cd/best-practices/  :how you can use tests in your Docker project and re-use them in the CI.


[BUG] Ubuntu version in deploy process

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?
Reviewing the repo
Version of CreatorsRegistry (for example "v0.19.0")
0.26.2
Description
I've noticed that the pipeline uses 'ubuntu-latest' when deploying, which can cause errors when the Ubuntu version changes.
It is safer to specify the specific system version you want to use.
Steps to Reproduce
Check the files in a directory: https://github.com/EddieHubCommunity/CreatorsRegistry/tree/main/.github/workflows
Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this
I would change the Ubuntu version to use the latest LTS.

Thanks for raising. The url provided doesn't work.
I am torn on this topic, because fixing it is better, but it often gets left behind also which creates other problems also.

URL fixed, don't worry, I just put information about what I found in my opinion. We can come back to this problem later :)


Stack name Generating invalid name

slsart configure created a stack with an underscore character and threw this error.
{ ServerlessError: 1 validation error detected: Value 'serverless-artillery-SkgG6R_yf-dev' at 'stackName' failed to satisfy constraint: Member must satisfy regular expression pattern: [a-zA-Z][-a-zA-Z0-9]|arn:[-a-zA-Z0-9:/._+]

As discussed, the short-term workaround is editing your serverless.yml and replacing any _ characters with an alphanumeric character.
Alternatively, you could run configure again but that's really not necessary.
We use shortid which base-64 encodes a random number to make a probably unique name to avoid stack collisions and the hilarity that can ensue.


Code for training llama pro?

Hi,
If I get it correctly, you have used code from https://github.com/allenai/open-instruct as base.
Would you release the full code of reproducing llama2 pro 8B?
Thanks!

Yes, of course. I will organize the code recently. Thanks for your interest.

Hey @hills-code ,could you also add code for converting a model by adding identity blocks for training ?
I am excited to use similar techniques for other open-source models !!
Thanks!:)

Hey @hills-code ,could you also add code for converting a model by adding identity blocks for training ? I am excited to use similar techniques for other open-source models like Qwen!! Thanks!:)

I have added the block expansion script under the folder scripts. You can check it for reference. Hope it will be helpful!

i will check that out,Thanks!

@hills-code  i was able to get it to work! Btw you only train the added layers and not even the lm head?

@raghavgarg97 No, I don't think you can skip the "post-training" for extra blocks on some new corpus (e.g. bigcode, as mentioned in the paper) before applying tuning

does anybody know how to continue pre-training? @yhyu13 @raghavgarg97 @yxgeee @hills-code

nah，code has not been released yet


Share API - user action requirement

I'm trying to implement the RN Share API on top of the Web Share API. The latter is not available inside workers and must be triggered by user activation, so from my tests neither the "JS" nor the "native" Share module can actually successfully invoke it.
The limitation seems to be inherent to the use of a worker+bridge to route user actions to API calls of this sort, which is interesting because this situation will in all likelihood repeat itself with other Web APIs (e.g. audio, probably?). Therefore I wonder if there is anything RNDom can do about this architecturally or whether we need new browser-level hooks to solve this. (Or a different kind of escape hatch in the short term - e.g. the ability to attach synchronous event handlers to certain views as needed, bypassing the bridge)

Ok, I was able to get this to work by creating a native component that extends View and attaches an onclick listener directly onto the HTML element. In my limited testing, it works quite well. I used RCTView since I couldn't find a native component for Touchable.
Sample code:
import { RCTView, type RCTBridge } from "react-native-dom";

class RCTFullscreenButton extends RCTView {
    constructor(bridge: RCTBridge) {
        super(bridge);

        this.classList = []; // remove pe-box-none class so we can set pointerEvents
        this.style.pointerEvents = "auto";
        this.addEventListener('click', this.onClick);
    }

    onClick() {
        const element = document.getElementById('video');
        if (element) {
            element.webkitRequestFullscreen();
        }
    }
}

customElements.define("rct-fullscreen-button", RCTFullscreenButton);

export default RCTFullscreenButton;

It might be good to write this up as a guide or formalize it so that this capability is built into react-native-dom. When dealing with video, you need to receive user gestures from a variety of sources: fullscreen via button, double click on the video & keypress plus being able to provide gestures to start the video when autoplay is disabled. I realize it's a bit hacky to have surfaces that are react-native-dom specific, but these are going to be fairly common scenarios. If we exposed an directOnClick prop (or whatever name is appropriate), that would probably be sufficient.
Slightly unrelated, but a capability to set the DOM id via a prop would be good as well. For example with video, you would want to fullscreen the encapsulating view that contains the video canvas and the controls. Making people create native components will be beyond the capabilities of a lot of React Native newcomers.


Auto height exmaple not working

I tried this example
https://github.com/eh3rrera/react-virtualized-examples/blob/master/04-dynamic-height-rows/src/App.js
But it's not working. Showing only empty data.

It works correctly for me. I cloned the repo, installed the dependencies for that folder, and run the app.
Can you tell me more about your environment or something else that might be relevant? For example, did you upgrade the versions of React or React-virtualized? Errors in the console?
Thanks!


Factor startup into cli runner method

A reasonable place to start writing scenarios...

Codecov Report

Merging #100 (dc92ab6) into dev (5fe9915) will increase coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev     #100      +/-   ##
==========================================
+ Coverage   54.05%   54.10%   +0.05%
==========================================
  Files          34       34
  Lines        6832     6825       -7
==========================================
  Hits         3693     3693
+ Misses       3139     3132       -7

Impacted Files
Coverage Δ

cli/src/lib.rs
0.00% <0.00%> (ø)

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


Added @escaping to setObject method

CacheBlockClosure and ErrorClosure are escaping closures, so they need explicit declaration of @escaping in Swift 3.

Can this be merged?

Thanks 👍


kn/fix skip then iterate bug

Pass _skipIndex when snapshotting results, and add missing test
Fix: #1409

Anyone?


Quartz+ Spring Boot + SQLite configuration

I'm trying to configure these three to work, but on startup quartz scheduler bean instantiation fails with sql error. The issue is that to obtain lock for QRTZ_LOCK table quartz is running SELECT * FROM QRTZ_LOCKS WHERE SCHED_NAME = 'quartzScheduler' AND LOCK_NAME = ? FOR UPDATE query. But sqlite does not support locks and throws SQLException
Is there any way to make it work?

@RamanShyrski Hello, i'm facing this issue now. Did you fixed it already? please share solution~ many thanks.


Update Release Notes For Important Change in triplet_semihard_loss

System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):
TensorFlow version and how it was installed (2.3 pip in docker):
TensorFlow-Addons version and how it was installed (pip in docker, tensorflow-addons==0.11.1):
Python version: 3.6
Is GPU used? (no):

Describe the bug
Not so much a bug. But this change could have been better documented in the release notes.
I had a unit test and it caught this when I upgraded from 0.10.0 to 0.11.1.
0.11.1. VERSION: https://github.com/tensorflow/addons/blob/v0.11.1/tensorflow_addons/losses/triplet.py#L123
0.10.0 VERSION: https://github.com/tensorflow/addons/blob/v0.10.0/tensorflow_addons/losses/triplet.py#L128
Basically the default value for argument squared was changed from True to False. To achieve the old behaviour you must use the new arg squared-L2.
Code to reproduce the issue
Not needed as clearly showed the git diff above between the two versions.
Other info / logs
Maybe this change could be documented in the change log, https://github.com/tensorflow/addons/releases/tag/v0.11.0, under tfa.losses section. It might help others if all of a sudden their model training is behaving differently.
Thanks!

Thanks for pointing out this! I've updated the release note. Please take a look at it to see if it works well :-) Thank you!

Perfect. That’s great. Thanks for the fast response.


The branch parameter is not defaulted to the branch that triggers a workflow

Based on the documentation, if you omit branch, it should default to the branch for the workflow.
I omitted the parameter, but it always reported deployments in the dashboard to master. Passing the branch using branch: ${{ github.ref_name }} fixed this, but the docs either need updating or the code fixing.

Addition to OP:
github.ref_name in a pull request refers to the pull request merge branch. Something like 42/merge. Where 42 is the PR number.
So instead, I've used:
branch:  ${{ github.head_ref || github.ref }}

github.head_ref refers to the base branch if being triggered in a pull request (i.e. feat-do-some-work). And if not a pull request, assume it's a push workflow and then github.ref is the branch that was pushed (i.e. main)

Btw I'm using reusable workflows.
I think that's the source of the issue. Given the action tries to get branch name from env vars GITHUB_HEAD_REF or GITHUB_REF_NAME. And when using reusable workflows, the called workflow doesn't have access to those unless you manually pass those env vars.
So guess another workaround could also be to pass those manually when calling the reusable workflow:
  deploy-preview:
    name: Deploy preview
    uses: ./.github/workflows/reusable-deploy-cloudflare-pages.yml
    env:
      GITHUB_HEAD_REF: {{ github.head_ref }}
      GITHUB_REF: {{ github. ref }}

I ran into the same issue (not using reusable workflows). Without setting the branch in config, actions run as the result of a PR were all named HEAD.
Setting to branch:  ${{ github.head_ref || github.ref_name }} gives me the source branch on pull_request and the branch name on push 🙌

Same problem for me, fixed by the previous comment. Here's my full config for future ref:
      - name: Build web
        run: |
          if [ "$GITHUB_REF_NAME" == "main" ]; then
            npm run build:web:prod
          elif [ "$GITHUB_REF_NAME" == "staging" ]; then
            npm run build:web:staging
          else
            npm run build:web
          fi

      - name: Publish site
        uses: cloudflare/pages-action@v1
        with:
          apiToken: ${{ secrets.CLOUDFLARE_API_TOKEN }}
          accountId: ${{ secrets.CLOUDFLARE_ACCOUNT_ID }}
          projectName: ${{ matrix.cf-project }}
          directory: www
          gitHubToken: ${{ secrets.GITHUB_TOKEN }}
          branch:  ${{ github.head_ref || github.ref_name }}

Also, when the event is triggered by pushing a tag, the Cloudflare deployments just show HEAD

Got this exact same issue

I have this issue as well.


Update README.md with azd init

Purpose
The azd team is planning on a few improvements to the workflow for azd for their GA launch. One of the consequences is that developers must first run "azd init" before running "azd up". This PR adjusts the README accordingly.
Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X] Documentation content changes
[ ] Other... Please describe:

Going to close for now, as it may not be needed.


entirely remove ring 0

because su is useless anyways + templeos supremacy

you mean ring 3?

yeah

we could try removing it temporarily until we make it bettter