lib: os: cbprintf: Add support for packed packages.

Added support for packages with packed (without alignment) arguments.
Such packages are not using va_arg to get argument but copy data
directly from the buffer. This approach saves space and allow
to move message between different platforms (with same
endianess).
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

I'm not convinced by this.
My reaction to @pabigot's proposal from a couple months ago was about
the fact that the cbprintf code got way more complex and harder to
maintain. This is why I suggested the va_list-based alternative to allow
for reusing the print formatting code unchanged.
This PR is crossing the same line as @pabigot's original proposal, even
though the code is abstracted inside a macro. As soon as
CONFIG_CBPRINTF_PACKAGE_PACKED is turned on, the runtime printf code
complexity explodes for all users. What is the benefit in that case to
have static log records created at compile time with super clever macros
for space efficiency if the runtime code to consume them doubles in size?
One of the stated reasons is:
    Such packages are not using va_arg to get argument but copy data
    directly from the buffer. This approach saves space and allow
    to move message between different platforms (with same
    endianess).

This is broken not only across different endianess, but also across
word sizes. In particular, "%ld" (long) does not have the same size
on 32-bits vs 64-bits architectures, and similarly for pointers, size_t,
etc. And when is the alignment most likely to change? Yes, when moving
from 32 to 64 bits and the other way around. So... do you really need
to support this case?
As for saving space... I think there is a much simpler and more efficient
way to actually save space. The idea is to look at the va_list packaged
buffer and remove all the zero integers from it. That includes ints
representing placeholders for string pointers for those strings that
have been copied into the package, as well as actual zero values, or
even larger integers like long longs whose magnitude is way smaller
than the available value range (pretty common in practice), and of course
actual alignment padding. I did start to write some code a while ago to
do just that which used the unused portion of the header as a bitmap to
remember where to insert zeroes back in cbpprintf(). I just didn't get
around to complete it. But this has the potential to save much more space
than simply getting rid of alignment constraints, with very little new
code. And the printf code still remains unchanged/optimal.

@npitre

cbprintf code got way more complex and harder to maintain.

Change is limited only to argument getting method so i can't see how it increases complexity and thus cost of maintenance.

As soon as CONFIG_CBPRINTF_PACKAGE_PACKED is turned on, the runtime printf code complexity explodes for all users.

It is now foreseen that It will be enabled only in specific cases (some versions of multi-domain logging).

if the runtime code to consume them doubles in size

Enabling this option adds ~270 bytes of ROM (cortex-m3).

This is broken not only across different endianess, but also across word sizes.

Yes, you are right, it's not only the endianess but also discrepancy in standard type sizes. However, I've compared sizes and alignment for all platforms supported by zephyr and results are following. If we look at standard type sizes (excluding long doubles) then all 32 bit platforms has the same sizes, same goes for 64 bit platforms. As for alignment, there are more discrepancy between 32 bit platforms (while all 3 64 bit platforms has the same alignment). It means, that packed packages can be shared between 32 bit cores in same SoC (if long doubles are not used), same for 64 bit SoC.
The idea for using packed packages is multi-domain logger in multicore SoC. If you have a helper CPU that is relatively slow but you would like to get logging data from it to the master core. Since helper CPU is slow (or loaded) you would like to avoid string formatting there, using packed packages you can pass log message (packages) even when alignment does not match (but sizes do). It is assumed that if alignement+size does not match then the only option to pass package is to format a string locally and create a package that contains formatted string. That will be the default behavior, enabling packed version will be done on SoC level configuration.

As for saving space...

That is not the reason i'm doing it. If there is a gain that's good but i am aware that it is rather insignificant.

Yes, you are right, it's not only the endianess but also discrepancy in standard type sizes. However, I've compared sizes and alignment for all platforms supported by zephyr and results are following. If we look at standard type sizes (excluding long doubles) then all 32 bit platforms has the same sizes, same goes for 64 bit platforms. As for alignment, there are more discrepancy between 32 bit platforms (while all 3 64 bit platforms has the same alignment). It means, that packed packages can be shared between 32 bit cores in same SoC (if long doubles are not used), same for 64 bit SoC.

We agree that this doesn't work across architecture widths nor
endianness. And on 64-bit architectures the alignment tend to stay the
same already.
So what is your usage scenario where this would be needed?
If this is only one particular case then it is probably wiser to have
one particular solution rather than having a large solution that has the
looks of being generic but isn't in practice.

Ok, will close for now.


Use system type for button when blur effects is disabled

One of my projects uses custom tint color for buttons.
I would like to use this customization on this awesome control as well. :)
Therefore to achieve this when the disableBlurEffects is set to YES the button is instantiated with UIButtonTypeSystem.

Hi!
Thanks for the pull request. I'll review it this evening and let you know what I think.
But in general it sounds like a good idea :)
Best regards,
Roland

I will open a new pull request using an updated fork.


Add option to not display unread messages

Summary
I find the unread notifier (red circle with a number inside) a little noisy based on its current configuration -- I'd prefer to turn it off of change the number to only count my private messages and @-mentions.
Steps to reproduce
Running OS X, I receive messages in a channel I don't really care about and get an unread notifier.

@joelburget would you like that behavior in the webapp as well?  If so, we could probably just make this as a setting in the webapp (and the electron app would inherit that functionality).

I don't think I'm shown the number of unread messages in the webapp, but I
imagine if I did see a count there I'd like it to go away.
On Mon, Aug 15, 2016 at 11:22 AM Tim Abbott<EMAIL_ADDRESS>wrote:

@joelburget https://github.com/joelburget would you like that behavior
in the webapp as well? If so, we could probably just make this as a setting
in the webapp (and the electron app would inherit that functionality).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip-electron/issues/54#issuecomment-239831041,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAS-xdW6egQNNm7kFb18XhIVO7teeYsEks5qgIQjgaJpZM4JkeSD
.

Yeah, the webapp shows the total unread count in the favicon.

@joelburget yes it's bit noisy. @timabbott Could we implement this in web app settings (receive notifications only for mentions/private messages etc)?

Yeah, you can open an issue in the webapp for having the favicon count only include @-mentions/PMs (if I'm not mistaken, this is a different setting from whether to receive desktop notifications, which we already have settings for).

@timabbott yes, this is different from desktop notifications settings. Just FYI I am fetching the badge counts from the title of page.

Merging it here https://github.com/zulip/zulip/issues/1736.


[Feat]: Add Eightify

What feature?
I would like to add Eightify - Youtube Video Summarizer tool.
Eightify automatically analyzes and extracts key insights from lengthy videos, condensing them into concise summaries. By providing a quick overview of video content, Eightify saves time and helps users get the main points without having to watch the entire video, making information consumption more efficient and convenient.
Add screenshots

Code of Conduct

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

Hello @JasonDsouza212 @VenomFate-619 , I have added my work still my issue has been closed. Please look into the matter.


Decouple GraphQL from engine module

To improve the code design (more generic, testability, consistency, etc). the use of the graphql-java dependency should be limited to the gateway module. Therefore, a new interface must be defined for the communication between gateway and engine modules.


gomod: Bumping a dependency that sets "go 1.21" in their go.mod sets "toolchain 1.21" in mine

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
gomod
Package manager version
1.20
Language version
1.20
Manifest location and content before the Dependabot update
go.mod. Link to original module file: https://github.com/boinkor-net/tsnsrv/blob/469a5104d7914816d185fec37a031cc47b67f032/go.mod
dependabot.yml content
https://github.com/boinkor-net/tsnsrv/blob/469a5104d7914816d185fec37a031cc47b67f032/.github/dependabot.yml
Updated dependency
"tailscale" from v1.46.1 to v1.48.1
What you expected to see, versus what you actually saw
I was expecting this PR to only contain the version bump; however, it also added this line: https://github.com/boinkor-net/tsnsrv/pull/25/files#diff-33ef32bf6c23acb95f5902d7097b7a1d5128ca061167ec0716715b0b9eeaa5f6R4
in diff form:
diff --git a/go.mod b/go.mod
index a6d61d9..73b1321 100644
--- a/go.mod
+++ b/go.mod
@@ -1,13 +1,14 @@
 module github.com/antifuchs/tsnsrv

 go 1.20
+toolchain go1.21.0

 require (
 	github.com/peterbourgon/ff/v3 v3.4.0
 	github.com/prometheus/client_golang v1.16.0
 	github.com/stretchr/testify v1.8.4
-	golang.org/x/exp v0.0.0-20230425010034-47ecfdc1ba53
-	tailscale.com v1.46.1
+	golang.org/x/exp v0.0.0-20230725093048-515e97ebf090
+	tailscale.com v1.48.1
 )

 require (
@@ -32,6 +33,7 @@ require (
 	github.com/cespare/xxhash/v2 v2.2.0 // indirect
 	github.com/coreos/go-iptables v0.6.0 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
+	github.com/dblohm7/wingoes v0.0.0-20230803162905-5c6286bb8c6e // indirect
 	github.com/fxamacker/cbor/v2 v2.4.0 // indirect
 	github.com/go-ole/go-ole v1.2.6 // indirect
 	github.com/godbus/dbus/v5 v5.1.0 // indirect

note the "toolchain" line that directly contradicts the go line above it. Go 1.20 does not understand the "toolchain" keyword, and so the resulting PR build fails.
(aside, yep, apparently it's fine to link the tailscale library into a go 1.20 library, but yeah, I should upgrade that version as soon as my toolchain supports it.)
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
Find the PR at https://github.com/boinkor-net/tsnsrv/pull/25
Smallest manifest that reproduces the issue
No response

@antifuchs should be working now if you want to give that PR a @dependabot recreate comment and try it out.

OK, that looks far more reasonable! Confirmed that it's the expected result, thank you (:

I'm seeing this toolchain insertion behavior happening again, this time with 1.22:
https://github.com/ossf/scorecard/pull/3928
go 1.21.5
+toolchain go1.22.1

I can open a separate issue if preferred, but was trying to avoid duplicates

I created a PR about toolchain removal https://github.com/dependabot/dependabot-core/issues/9422


Fine-tuning time estimation

Hi! thanks for a very interesting model released. Could you share how many GPU hours (+GPU used) you've spent to get the current model?
Also what data did you use for training? Modelscope or something without watermarks?

Hi! Thanks
Don't know exactly how many GPU hours but it was definitely a lot + lots of GPUs.
Modelscope is awesome, but Hotshot-XL is a totally different architecture/unrelated.


added faster-whisper support (recognize_faster_whisper() method on Recognizer class)

I just replicate existing recognize_whisper() method and adapt to work with faster-whisper.
Doing this will improve inference performance a lot.
Didn't removed "legacy" method, you decide if it worth.

Duplicate #693


Deploying container app with private docker.io image fails

Please provide us with the following information:

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
When trying to deploy a container app with a container image i a private docker registry you receive the following error:
The following field(s) are either invalid or missing. Invalid value: "<registry>/<image>:<tag>": GET https:: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:<registry>/<image> Type:repository]]: template.containers.<containername>.image.
Steps to reproduct

Open Cloud Shell in portal
upload yml-file

kind: containerapp
location: westeurope
name: \<name>
resourceGroup: \<resourcegroup\>
type: Microsoft.App/containerApps
tags:
 tagname: value
properties:
 managedEnvironmentId: /subscriptions/\<subscriptionId\>/resourceGroups/DefaultResourceGroup-WEU/providers/Microsoft.App/managedEnvironments/\<environmentname\>
 configuration:
   activeRevisionsMode: Multiple
   secrets:
     - name: mysecret
       value: thisismysecret
     - name: myregistrypassword
       value: \<registrypassword\>
   ingress:
     external: true
     allowInsecure: false
     targetPort: 80
     traffic:
       - latestRevision: true
         weight: 100
     transport: Auto
   registries:
     - passwordSecretRef: myregistrypassword
       server: docker.io
       username: \<username\>
 template:
   revisionSuffix: r
   containers:
     - image: \<username\>/\<image\>:\<tag\>
       name: <containername>
       env:
         - name: HTTP_PORT
           value: 80
         - name: secret_name
           secretRef: mysecret
       resources:
         cpu: 0.25
         memory: 0.5Gi
   scale:
     minReplicas: 1
     maxReplicas: 1

az containerapp create -n <name> -g <resourcegroup> --environment <environment> --yaml "<yaml-file>"

Expected behavior [What you expected to happen.]
Successfully deployed container app
Actual behavior [What actually happened.]
Deployment fails
Screenshots

Additional context
Ex. Did this issue occur in the CLI or the Portal?
Same issue occurs in both CLI and Portal

Same problem happening here. Adding more information:

The Portal doesn't save the Registry (possibly since deployment fails?). Manually creating the registry using az containerapp registry set does not help.
Using az containerapp up and passing in registry server/username/password fails with the exact same error.

Using --debug flag, this is what fails:
urllib3.connectionpool: https://management.azure.com:443 "PATCH /subscriptions/**omitted**/resourceGroups/**omitted**/providers/Microsoft.App/containerApps/**omitted**?api-version=2022-03-01 HTTP/1.1" 400 350
cli.azure.cli.core.util: Response status: 400
cli.azure.cli.core.util: Response headers:
cli.azure.cli.core.util:     'Cache-Control': 'no-cache'
cli.azure.cli.core.util:     'Pragma': 'no-cache'
cli.azure.cli.core.util:     'Content-Length': '350'
cli.azure.cli.core.util:     'Content-Type': 'application/json; charset=utf-8'
cli.azure.cli.core.util:     'Expires': '-1'
cli.azure.cli.core.util:     'x-ms-ratelimit-remaining-subscription-resource-requests': '499'
cli.azure.cli.core.util:     'api-supported-versions': '2022-01-01-preview, 2022-03-01, 2022-05-01, 2022-06-01-preview'
cli.azure.cli.core.util:     'Server': 'Microsoft-IIS/10.0'
cli.azure.cli.core.util:     'X-Powered-By': 'ASP.NET'
cli.azure.cli.core.util:     'x-ms-request-id': 'f041a2a8-4058-49b0-a469-8edaeb125e1e'
cli.azure.cli.core.util:     'x-ms-correlation-request-id': 'f041a2a8-4058-49b0-a469-8edaeb125e1e'
cli.azure.cli.core.util:     'x-ms-routing-request-id': 'UKSOUTH:20220810T085332Z:f041a2a8-4058-49b0-a469-8edaeb125e1e'
cli.azure.cli.core.util:     'Strict-Transport-Security': 'max-age=31536000; includeSubDomains'
cli.azure.cli.core.util:     'X-Content-Type-Options': 'nosniff'
cli.azure.cli.core.util:     'Date': 'Wed, 10 Aug 2022 08:53:31 GMT'
cli.azure.cli.core.util: Response content:
cli.azure.cli.core.util: {"code":"WebhookInvalidParameterValue","message":"The following field(s) are either invalid or missing. Invalid value: \"docker.io/**omitted**/**omitted**:159\": GET https:: UNAUTHORIZED: authentication required; [map[Action:pull Class: Name:**omitted**/**omitted** Type:repository]]: template.containers.**omitted**.image."}

The image string should be <server>/<image>:<tag>, not <username>/<image>:<tag>. For example:
..
registries:
     - passwordSecretRef: myregistrypassword
       server: docker.io
       username: myuser
 template:
   revisionSuffix: r
   containers:
     - image: docker.io/myimage:tag
..

We match the server part of the value in the image field to the server field in one of the entries in registries. If we don't find a match, then we try an anonymous pull, which fails with unauthorized as you see here.

I got the following CLI command to work with my private DockerHub registry.
az containerapp create -n mycontainerapp -g my-rg --image registry.hub.docker.com/myhubusername/imagename:latest --environment my-env --ingress external --target-port 80 --registry-server registry.hub.docker.com --registry-username myhubusername --registry-password myhubpassword

docker.io/myimage:tag

The following field(s) are either invalid or missing. Field 'template.containers.mywebsite-web-server.image' is invalid with details: 'Invalid value: "docker.io/docker.io/myimage:latest": GET https:: UNAUTHORIZED: authentication required';. (Code: InvalidParameterValueInContainerTemplate)

Naming image to image: docker.io/myimage:tag is resulting in error too

In my case changing registry from docker.io to registry.hub.docker.com worked

@anthonychu - Can you help with this?

Just had this issue while creating my first Container App through the portal. What fixed it was ats1999's suggestion of using registry.hub.docker.com instead of docker.io as the registry login url when creating the Container App for the first time.
This was pretty annoying as cpellensap is right, both the docks and the default placeholder urls are docker.io but cause this whole issue.
It's doubly annoying because I am not able to edit the registry url after failing to create the Container App for the first time. I can't create a new revision to change the image because there is no revision to base it off of since the Container App failed to create. I instead have to delete the whole container app and recreate it with different registry settings.
I don't understand why this issue is closed. The problem still exists.


Class 'mixin' not found when called from another command

Hi
I wrote a "helper" command to make it easy to copy information between projects. This is the handle method.
    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        if (\App::environment() === 'local' && class_exists('Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider')) {
            $this->call('ide-helper:generate');
            $this->call('ide-helper:meta');
            $this->call(
                'ide-helper:models',
                [
                    '--write' => true,
                    '--reset' => true,
                    '--no-interaction' => true,
                ]
            );
        } else {
            $this->info('Helper files not generated. They will only generate when developing locally.');
        }
    }

When I run this, the following exception is shown.
Exception: Class 'mixin' not found.
Could not analyze class App\User.

If I run the command manually, php artisan ide-helper:models --write --reset --no-interaction everything works as I'd expect it to and I get the output written to the models PHPDoc and I get the clean auto complete I'd expect.
This worked on a previous project, but not on the latest project which I'm using Vessel for.
What else can I try to get this working? The obvious solution to me is to run each of these commands individually instead of as one command (in the composer post-autoload-dump script) but I was wondering what it is that I'm misunderstanding to get this error when attempting to run all 3 at one time.
Edit: Generate and Meta work without any issues, either directly or as part of this command

Any update on this?

There's a similar issue where I did test this and any order worked => https://github.com/barryvdh/laravel-ide-helper/issues/216
Does this problem still exists?


Synopsis is printed less

synopsis is not fully printed. is there a way to do it.

on which endpoint?


Session Management

Question
We have an interesting bug in which we have cookie sessions for both OP and RP. We are using backchannel logout which works correctly when initiated, but say for instance we are on an RP and open a tab for the OP. We then go and clear all of the cookies for the OP.
What happens now is that the normal OP .net cookie auth flow kicks in and we've landed on the login page. The user then signs in with different credentials and goes to the RP. The RP still has the old session active and assumes thats the active user so there is a mismatch between RP and OP users.
We cant implement check session currently since that is all front channel and as far as i can tell, we cannot send along the RP cookie in the iframe to determine if the session is still active and valid due to 3rd party cookie restrictions in the browser.
It would seem that this scenario also would happen in the BFF implementation correct, or am i not seeing an equivalent to backchannel check session in the spec? Is there a solution here? Something similar to backend session management and somehow supply the OP's session id to the RP which they can send along via backchannel?
Minimal working example

user logs in and is on the RP
OP session dies
user refreshes the page in OP and gets redirected to login
logs in as someone else
gets back to RP and the old user is still there

Or am i over complicating things? If anyone has any advice or examples that'd be great.

Server side OP session mgmnt that keeps track of OP session per user (just like what it does in the cookie scheme).
Some kinda internal backend check session endpoint that can be called by RP. They send along some claims (subject and client id maybe) to check if an OP session exists for that user. If not, 401 response to RP which kicks off OIDC stuff.

I assume there is a way to do this with a new auth scheme other than cookies. Same interface but underlying impl using some data store on the backend.


upgrade v8 engine

current version in master  branch is v8 in node 0.10.32
target:

move to v8 in node 0.12.3
move to v8 in io.js

numerous breaking changes!
see
part 1/3 : https://strongloop.com/strongblog/node-js-v0-12-new-apis/
part 2/3: https://strongloop.com/strongblog/node-js-v0-12-apis-breaking/
part 3/3:  https://strongloop.com/strongblog/node-js-v0-12-c-apis-breaking/

current master branch compat with 0.10.38

last stable v8 from node 0.10.38 is
946641c2d8f2f2b9265e9eb194fa9187682b6a52

v8 from node 4.2.4 is
558daac3b09329ef8a00fde9f00b4932d14fb5b0

new version need VS2015


Executables with RPATH are not relocatable

The sailfish formula is not cellar :any because cmake sets the RPATH of the executable. I believe keg_relocate does not modify the RPATH. Could someone please confirm that? The RPATH should be given the same relocation treatment as install_name.

See https://github.com/Homebrew/homebrew-science/pull/3753#issuecomment-226634919

I'm not sure if it does and if it doesn't: we'd welcome a PR to do so.

Should the RPATH simply be deleted? I believe it's redundant with the absolute path install_name mechanism of finding shared libraries on Mac.

sailfish includes an RPATH, which makes the bottle non-relocatable. Removing the RPATH makes it cellar :any.
❯❯❯ brew install sailfish
❯❯❯ otool -l /usr/local/bin/sailfish |grep RPATH -A2
          cmd LC_RPATH
      cmdsize 56
         path /usr/local/Cellar/sailfish/0.10.1/lib (offset 12)
❯❯❯ brew bottle -v sailfish
Warning: String '/usr/local/Cellar' still exists in these files:
/usr/local/Cellar/sailfish/0.10.1/bin/sailfish
 --> match '/usr/local/Cellar/sailfish/0.10.1/lib' at offset 0x9e4
❯❯❯ chmod u+w /usr/local/bin/sailfish
❯❯❯ install_name_tool -delete_rpath /usr/local/Cellar/sailfish/0.10.1/lib /usr/local/bin/sailfish
❯❯❯ brew bottle -v sailfish
    cellar :any

If it can be: yes, that makes sense.

There's a relevant discussion on RPATH in Conda at https://github.com/conda/conda-build/issues/279

After some reading, it looks as though RPATH aren't used on Mac OS unless explicitly requested by @rpath in the install_name. If none of the dependent libraries use @rpath in the install_name, then RPATH is not used and can be safely deleted.
e.g.
install_name = /usr/lib/libncurses.5.4.dylib
is equivalent to
install_name = @rpath/libncurses.5.4.dylib RPATH=/usr/lib

Indeed, none of our Mach-O handling reads or manipulates the LC_RPATH load commands. That's probably because RPATHs haven't been a thing on OS X for too long and were very rarely used. The only thing stopping this from happening is that otool -l output is slightly more annoying to parse than otool -L and there's no more convenient way (I'm aware of) to list the embedded RPATHs.
Depending on the urgency of this:

We implement a solution now that uses otool and install_name_tool to perform the task.
We wait until later this summer ruby-macho learns to manipulate RPATHs and Homebrew switches to it for its Mach-O inspection and manipulation needs. At this point this feature will be trivial to implement as we'd be coding against a Ruby API. (ruby-macho is one of this year's Google Summer of Code projects.)

Should the RPATH simply be deleted? I believe it's redundant with the absolute path install_name mechanism of finding shared libraries on Mac.

Probably, which is easy enough to do with install_name_tool once you know which path to remove. It would be good to collect a few formulae that use RPATHs and to check how they use them (also taking into account the state before our manipulation of LC_*_DYLIB load commands).

If none of the dependent libraries use @rpath in the install_name, then RPATH is not used and can be safely deleted.

That's not entirely correct. You'd have to inspect the full tree of dynamic dependencies, including libraries loaded via dlopen to make sure none of them depend on that particular RPATH in their @rpath/-prefixed dylib load commands. That's not even academic, as a frequent setup (at least in my experience) is that only the main executable contains the LC_RPATH load command and then all other dependencies rely on that and express their inter-dependencies using @rpath/-prefixed dylib load commands.

Good clarification. Particularly the dlopen issue means that it's impossibly to determine statically whether the RPATH is used or not. So it's never safe to delete the RPATH without manual inspection, in which case it should be relocated using @@HOMEBREW_CELLAR@@ like install_name.

Of the 1959 Mach-O binaries in my Homebrew installation, only one has embedded RPATHs. Though that one is truly beautiful (the bottled shellcheck binary, it needs none of those):
/opt/homebrew/Cellar/shellcheck/0.4.4/bin/shellcheck:
  LC_LOAD_DYLIB=/usr/lib/libiconv.2.dylib
  LC_LOAD_DYLIB=/usr/lib/libSystem.B.dylib
  LC_RPATH=/usr/local/Cellar/shellcheck/0.4.4/lib/x86_64-osx-ghc-7.10.3/ShellCheck-0.4.4-Di0vBeJ5G22HeAaHhOLnuj
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/QuickCheck-2.8.2-8v0RiFeBskC7UIoPaBQj4x
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/array_67iodizgJQIIxYVTp4emlA
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/base_HQfYBxpPvuw8OunzQu6JGM
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/binar_3uXFWMoAGBg0xKP9MHKRwi
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/rts
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/bytes_6VWy06pWzJq9evDvK2d4w6
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/conta_2C3ZI8RgPO2LBMidXKTvIU
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/deeps_6vMKxt5sPFR0XsbRWvvq59
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/direc_0hFG6ZxK1nk4zsyOqbNHfm
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/filep_Ey7a1in9roBAE8bUFJ5R9m
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/ghcpr_8TmvWUcS1U1IKHT0levwg3
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/integ_2aU3IZNMF9a7mQ0OzsZ0dS
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/libexec/integer-gmp/lib
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/json-0.9.1-7kMKfrkn0Qy3BxYPUrdvN9
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/mtl-2.2.1-1XgQYjia7VJ5otlr6LA5f9
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/parsec-3.1.11-3g02YRopkV0859dUlfXjDZ
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/prett_JItwetRppk1H5Uq3xbjDGC
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/primitive-<IP_ADDRESS>-CanAn0ZmSDe1O8lK6kBSP2
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/proce_52AgREEfSrnJLlkGV9YZZJ
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/random-1.1-8PKo7QZOLuqJhOvzBKhJvd
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/regex-base-0.93.2-5g5YXZiScrKLs2R8SL8lg0
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/regex-tdfa-1.2.2-8NbBLGNW5Pf3YobyTLo8PR
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/syb-0.6-8Ynz83QzMazFTSpu5qYzrN
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/templ_GJPvtLC64aA4c1Jl10o2qt
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/text-<IP_ADDRESS>-8AnM1c81gDP9ebLwfSU223
  LC_RPATH=/private/tmp/shellcheck-20160515-52824-1fxs4fo/shellcheck-0.4.4/.cabal-sandbox/lib/x86_64-osx-ghc-7.10.3/tf-random-0.5-FWUyVDgIgj9L0dx0uEV4dc
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/time_FTheb6LSxyX1UABIbBXRfn
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/trans_GZTjP9K5WFq01xC9BAGQpF
  LC_RPATH=/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/unix_KZL8h98IqDM57kQSPo1mKx

I agree, we should probably clean them up and adjust the ones pointing into our prefix.

Does shellcheck use dlopen? Otherwise I see only two LC_LOAD_DYLIB neither of which use @rpath.
sailfish, which has an RPATH but no @rpath uses, is built using cmake. Is the default behaviour to set RPATH, or is something amiss with sailfish?

Does shellcheck use dlopen? Otherwise I see only two LC_LOAD_DYLIB neither of which use @rpath.

No, not that I'm aware of. The libraries might exist in the given paths while shellcheck is built by the Glasgow Haskell compiler, but the end result is a statically linked binary that (as you correctly observed) only links to two system libraries. In any case, the binary is working fine though none of those paths specified in the LC_RPATH commands exist on my system.

sailfish, which has an RPATH but no @rpath uses, is built using cmake. Is the default behaviour of cmake to set RPATH, or is something amiss with sailfish?

Sorry, I'm not familiar with sailfish, but I presume they decided to use RPATHs in their OS X builds to make relocating the binaries and/or libraries easier. Assuming it's a set of libraries, adding a new RPATH in a client binary is easier than adjusting all dylib load commands.

I'm wondering whether all cmake-compiled executables are non-relocatable due to RPATH.

Certainly not. A non-trivial amount of my binaries was built by cmake and none of them contain any RPATHs. That's totally up to the upstream authors and what they do with their build systems …

sailfish does not use RPATH on Mac OS to locate any of its libraries, even though RPATH is set. Delete its RPATH and it works just fine. Perhaps its cmake configuration is incorrect.

In any case, the binary is working fine though none of those paths specified in the LC_RPATH commands exist on my system.

shellcheck may be cellar :any if you delete all its RPATH entries, like sailfish.

Perhaps its cmake configuration is incorrect.

I vaguely recall from my CMake days there may be a way to globally disable such things. If not globally, there will be a way my either passing in a config variable or patching an option in.

sailfish does not use RPATH on Mac OS to locate any of its libraries, even though RPATH is set.

So in this regard it is similar to my shellcheck example …

shellcheck may be cellar :any if you delete all its RPATH entries, like sailfish.

Quite likely. I guess many of the formulae that are built with ghc might fall in that category, so that would be quite a win for relocatable bottles with a comparatively simple change.

pandoc, also compiled by ghc, does not have any RPATH.

This has been improved as much as it will be for now. If this is still broken for you: we'll accept PRs for this but we're not actively working on it at this time.

Some additional information: Once #656 ships, the ruby-macho backend for Mach-O manipulation will have full support for enumerating, adding, modifying, and deleting RPATHs. Once it becomes the default backend and the cctools-based implementation has been removed, we can start using these new features, e.g. remove bogus RPATHs that point to temporary locations or adjust those that point into the Homebrew prefix to further improve relocatability of bottles.

Thanks, Martin!

@woodruffw 👍


Hotel Reservation not working for Kubernetes HTTP workload generator

When I try to run the HTTP workload generator according to the README file in https://github.com/delimitrou/DeathStarBench/tree/master/hotelReservation/kubernetes,
cd /root/DeathStarBench/hotelReservation
./wrk -D exp -t 2 -c 2 -d 30 -L -s ./wrk2_lua_scripts/mixed-workload_type_1.lua http://frontend.hotel-res.svc.cluster.local:5000 -R 2

It shows that ./wrk file is not found. After reading through the file, I found that there is only wrk2 file without any executable scripts inside. I am wondering whether I do something wrong on my side or I should follow another instruction?
Many thanks.

@przmk0 Thanks for your suggestion. I am trying to make all in wrk2 directory and it shows me the error in the figure below. I am using MacOS version 12.5.1 with Intel chip. I think I have already installed all the dependencies. Do I need to migrate to other OS platforms to make everything?

@XiangyuG @lepidopterane-atsmith Not sure if you both are still stuck with getting a wrk2 client for Kubernetes, but there is now a working Docker image that can be leveraged to use the workload generator.


Revamp home page and remove Info page

Checklist before merging

[ ] I added an appropriate entry to CHANGELOG.md

[ ] As soon as a user is logged-in, there should be a redirect towards the Projects page.
[ ]  Fill the home page with the contents described in https://github.com/fractal-analytics-platform/fractal-web/issues/243#issuecomment-1690088260.
[ ] Re-center footer contents appropriately.


reuse kite instance

Is there a way as of now to reuse kite session(created outside this package) instead of passing api_key and generating and new session

@singla96
Maybe you can consider directly assigning access_token from the current active session to KiteTicker and Kite connect instance.

@ranjanrak  Yes we are doing the same.  Instead of passing request_token we modified it to accept access_token so that we don't need to generate new session everytime we use this.

@singla96 @sriraj1122
Have added this feature to reuse the previous active session's access_token. You can read more about the usage here. You can upgrade to the latest version by: pip install optionchain_stream --upgrade


SPI: Queue up multiple transactions in interrupt mode

I shall leave spi_transmit_async queueing for a separate PR. The implementation is growing too complex to finish in a timely fashion.

Nvm, I managed to simplify the async implementation.


chore(go.mod): Bump celestia-app to v2.2.0 release in main

Porting @rootulp 's bump of celestia-app to main so we don't forget it.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 46.33%. Comparing base (2469e7a) to head (35a7342).
Report is 335 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3826      +/-   ##
==========================================
+ Coverage   44.83%   46.33%   +1.49%
==========================================
  Files         265      314      +49
  Lines       14620    18137    +3517
==========================================
+ Hits         6555     8403    +1848
- Misses       7313     8718    +1405
- Partials      752     1016     +264

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cannot merge automatic pull requests as a codeowner

Examples of this issue in https://github.com/protacon/once-oncenet2-backend/pull/62 and https://github.com/protacon/once-oncenet2-frontend/pull/154

I will leave these pull requests open so that you can find out where the problem is much easier. You can also merge those pull requests whenever you like if you get it to work.

The https://github.com/protacon/once-oncenet2-frontend/pull/154 worked now 🤔

The https://github.com/protacon/once-oncenet2-backend/pull/62 also worked. Seems that this might have been an issue in the GitHub side and is now solved. Closing this issue for now.

It's nice when bug fixes itself.  :D


Generic Git sources?

Any plans to allow integration from self-hosted or other Git source?
My company uses a self-hosted Gitlab instance which is not accessible from the outside, so we would only want to set up an outgoing webhook from our side.
Anyone can add a webhook to appcenter manually, if that's what is required, so I'm not quite clear why appcenter is not serving this very common use case.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 12fa9a24-2575-3fa4-ffab-cafaeab496c1
Version Independent ID: bbb99ead-eecc-222a-bc16-7509be5e8ec9
Content: Connect to a repository to start building your mobile app - Visual Studio App Center
Content Source: docs/build/connect.md
Service: vs-appcenter
GitHub Login: @siminapasat
Microsoft Alias: siminap

@batjko I apologize for my late response, I was on vacation last week. The comments feature of our documentation is intended to enable customers to suggest changes to the docs or ask clarifying questions about the docs.
App Center support is provided directly from within the App Center portal. Any time you need help, just click the blue chat button in the lower-right corner of any page and our dedicated support team will respond. For additional information, see the [App Center Help Center] (https://intercom.help/appcenter/getting-started/welcome-to-app-center-support).


Not able to get value from input field when applying it in angular.

<input type="text" id="date-format" class="form-control ng-valid ng-touched ng-dirty ng-valid-parse" ng-model="campaignCrud.deadLine" tabindex="0" aria-invalid="false" data-dtp="dtp_fgiAo">
This is the content we get after we select date and time from the datepicker. So how to get the date value from the input field.?
Please help.

I need the value that is displayed in this field.

use js initialize and store that value in $scope.

I think it's stored in $scope.campaignCrid.deadLine

@T00rk  I am not getting the value in $scope.campaignCrud.deadLine. Is there any other way to get the value like getElementById or sort of.

$('#dtp').bootstrapMaterialDatePicker().on('change', function(e, time) {         $scope.time = e.target.value; });

That's strange. Have you tried to set a different ng-model name ?
I think if you set a property, it has to be initialized.
Have you thought of setting $scope.campaingCrud = { deadLine : null } in your controller ?

I have tried to get another solution to it. Added a 'value' attribute to the input field when the date nad time selected. Below is the idea.
setElementValue: function ()
              {
                 this.$element.trigger('beforeChange', this.currentDate);
                 if (typeof ($.material) !== 'undefined')
                 {
                    this.$element.removeClass('empty');
                 }
                 this.$element.val(moment(this.currentDate).locale(this.params.lang).format(this.params.format));
                 **this.$element.attr('value', moment(this.currentDate).locale(this.params.lang).format(this.params.format));**
                 this.$element.trigger('change', this.currentDate);

              },

Added this line in JS file. So i got the value in Input field and then it was easily bindable in the controller.
Thanks guys for your help.

@nasr18 Hello

@aaron170 Where you added this code?

In this file.
bootstrap-material-datetimepicker.js
Line# 767


Test only: Bug: Issue with Test Type Code value in Global View (Flow to Load Reference Data)

Issue:
In the Global view for Flow to Load Reference Data when a user views Test Data the Test Type Code value says Select a value which is incorrect
Steps to Reproduce:

In ECMPS, navigate to QA Test Data
Select and open a Configuration
Click on View for the Flow to Load Reference Data (Test Type Group dropdown)
In the view modal you will see that the Test Type Code displays 'Select a value'. This needs to be fixed

Bug fixed


[.\node_modules.bin\electron-rebuild.cmd ] window has a problem

.\node_modules.bin\electron-rebuild.cmd      ( electron version 0.36.7)
executing result:
Unable to find electron-prebuilt's version number, either install it or specify
an explicit version

This issue is with electron-rebuild so it's probably better if you open it there https://github.com/electronjs/electron-rebuild .

@code1031 This is node-pre-gyp continuing to not work

@paulcbetts thanks a lot ,how to solve the problem?

Run electron-rebuild with the pre-gyp-fix option.
Patch node-pre-gyp, you can do that by installing the branch from PR 187 locally.
Monkey-patch node-pre-gyp at runtime using https://github.com/enlight/electron-node-pre-gyp-fix

I am sorry,I couldn't understand，I can concretely how to do ?

tried many times ,only to find


Missing alt tags in templates

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
My PR QC checks failed with:
https://github.com/inventree/InvenTree/actions/runs/5072715583/jobs/9110794538
I have not touched those templates, so I do not know how did they failed for me now (as I see the latest master commits passed QA).
If this is really an issue I can fix it with a PR quite easily.
Steps to Reproduce

Expected behaviour

Deployment Method

[ ] Docker
[ ] Bare metal

Version Information
master
Relevant log output
https://github.com/inventree/InvenTree/actions/runs/5072715583/jobs/9110794538

@martonmiklos are you sure this was not caused by the version bump in the linting tool?

@martonmiklos are you sure this was not caused by the version bump in the linting tool?

Dah, you have the point, I reverted that accidental change and we will see if the QC will pass.

Bumping versions on the linting tools often leads to "interesting" behaviour. Those actions should always be a separate PR - we had a hard time diagnosing issues like that before.

Bumping versions on the linting tools often leads to "interesting" behaviour. Those actions should always be a separate PR - we had a hard time diagnosing issues like that before.

Yes I apologize for committing it "blindly". How should we deal with this lint upgrade? I could send a PR which bumps the version and fixes the issues if you would like to.

That would be a good approach. I normally need 3-4 full CI runs to get all issues with linking updates so don‘t be discouraged.

@matmair can this be closed out?


CPT-LX-001: Create a Basic Bash Script for System Monitoring #5

Hi,
Thank you for your fantastic contribution to the Codeku Cloud and DevOps micro tasks! We truly appreciate your effort. However, we regret to inform you that this particular task cannot be approved due to a few errors. But don’t worry—this is just the beginning! There will be plenty of opportunities in the future, and we’re confident that you’ll succeed next time.
We encourage you to keep trying, and we wish you the very best of luck!
Warm regards,
Team CoDeKu


added series.second docstring, and  adjusted inconsistent leading whitespace for hour and minute.

Nice @brentomagic, thank you!


Run docker build as a dependency, not a test step

Not overriding the dependency step can cause Circle to infer ruby and run a
bundle install. That can leave files in the source tree that end up copied into
the docker image and mess with bundler's operation within that image.
Also, the build step is (IMO) a dependency, not itself a test.
/cc @codeclimate/review -- open to opinions here

LGTM

:shipit:


Create a blog variant of the starter

Create a version of this starter that includes a blog.

Hi @taylorbryant,
I created a PR maybe useful I am new in React and not an Frontend-Dev.
Review:
#115 115

@larsroettig Thank you so much! I’m going to keep this barebones. Feel free to fork this repo and make a blog version!


dependencies

I've started working on an adapter for tsfeatures in sktime alan-turing-institute/sktime#968 but I'm running in some dependencies issues.
The main problem is the strict requirements
e.g. scikit-learn==0.23.1 instead of scikit-learn>=0.23.1 or just scikit-learn.
Also entropy is now deprecated and replaced by https://github.com/raphaelvallat/antropy

Hi! I've released the 0.3.0 version addressing these issues.
Thanks!

Legend! thanks


Documentation is not clear around the "public image"

Both the examples of user and system managed identity instruct to create the Azure Container App from public image initially.
It is not clear if this is for demonstration only or a hard requirement.
If this is a hard requirement then it greatly complicates IAC deployment and also deployments in a secured environments.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 87860070-60db-800a-f7d0-0fcb562f3b80
Version Independent ID: 0d8a5f29-4706-96df-0e49-82dbc477bf1f
Content: Azure Container Apps image pull from Azure Container Registry with managed identity
Content Source: articles/container-apps/managed-identity-image-pull.md
Service: container-apps
GitHub Login: @v-jaswel
Microsoft Alias: v-wellsjason

@nosalan:
First, I humbly apologize for the delay in resolving this.
It is not a hard requirement to use a public image.
We've updated the article in question (https://learn.microsoft.com/azure/container-apps/managed-identity-image-pull) to add the following:
"Create a container app with a private image
If you don't want to start by creating a container app with a public image, you can also do the following.

Create a user-assigned managed identity.
Add the acrpull role to the user-assigned managed identity.
Create a container app with a private image and the user-assigned managed identity.

This method is typical in Infrastructure as Code (IaC) scenarios."

Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.
#please-close


Packing Collection of Playbooks

Packaging playbook in collection is still under development and structure of this code is defined well for Galaxy import.
Although https://docs.ansible.com/ansible/latest/dev_guide/developing_collections.html#id13

Open issue in Ansible Galaxy pending
https://github.com/ansible/galaxy/issues/2163


HiDPI support

I have a laptop with a high resolution display. For GUI programs to be usable, they need to be scaled to about 150% or greater. I would love an option to zoom in or scale the interface (maybe there's a way of reading this from the GUI?).
In the meantime, there may be a way to scale with a custom CSS.

Nowadays is  only possible to increase the size of the fonts inside the query box. Any other kind of zoom is not available yet. But looking into the electron API seems it is simple to achieve https://github.com/electron/electron/blob/master/docs/api/web-frame.md#webframe.
I can't work on that. But I would glad help if you are interested in do a PR for that.

wierd ... what kind of HiDPi is it ?
here on my retina i don't experience this kind of issue O_o

I'm using Ubuntu.

mmh ok, but what is your resolution, pixel density ?
mmh maybe it's a linux related issue.. did not try on linux

My display is 2560x1440 on a 14in laptop. Not crazy, but I'm sure others have higher DPI screens. I use Unity and scale the screen to 1.25.
Screenshots aren't the best tool for showing issues regarding DPI, but compare the size of the text in this bug report, which by many measures would be considered small (smaller than my parents could possibly read!!) to the text in Sqlectron. The "Search" box for example is small enough to be unusable.
I imagine the fix to this would be the same as approach that browsers like Firefox and Chrome take: to scale everything up to account for HiDPI.

To use a different zoom factor. Include this configuration on your ~/.sqlectron.json:
"zoomFactor": 2,

PS: Still have to wait a new release to use this setting.

Thank you! I can confirm that this is fixed.


Reading individual resources from the database shouldn't use ORDER BY

Expected Behavior
Reading a single resource should be a simple select query.
For example: SELECT * FROM `<table>` WHERE key = "<resource name>" LIMIT 1;
Current Behavior
The query for reading includes ordering by primary key.
For example: SELECT * FROM `<table>` WHERE key = "<resource name>" ORDER BY `<table>`.`key` LIMIT 1;
Steps to Reproduce

Set the logging.level server config value to debug.

Call GetProject, GetApi, GetApiVersion, GetArtifact, or any other API method that should read a single row from the database. Note: GetApiSpec and GetApiDeployment use more complicated queries.

Check the server logs for the output from each database query.

Solution Ideas
Use Take instead of First in get.go and tag.go query builders.

I don't disagree with this, but for prioritization, can we estimate the performance impact?

Hard to say. The queries to create our tables already define the key column as unique by including PRIMARY KEY (`key`), so the DBMS should have enough information to know sorting is completely unnecessary to handle this query. If the DBMS is smart enough there should be effectively zero performance impact.
If it sorts the table, then performance might depend on available indexes. Practically though, unless the table is quite large I doubt the worst possible execution of this query would be noticeably different than the optimal one.
All said, the fix is just s/First/Take/g in two files.


Integrate PointRend with Mask RCNN

Dear sir,
Thanks for sharing
I am interested in integrating pointrend with mask rcnn
Could you kindly share the code with me?
Your work will be cited in my research paper
Thanks in advance

@FatmaMazen2021 did you try to integrate PointRend with Mask RCNN?


Connect with Facebook account

Hi,
Can someone help me or give me a tutorial to write code for "Connect with Facebook account".
I already write API side code with Passport-Facebook. And when I go to
http://local:9000/login/facebook, by browser it works fine : so when I'm already connected to Facebook, it connects me successfully to Grocesries app and show me groceries, if not connected it shows me facebook login web page and after entering Facebook credentials it connects me successfully to Grocesries app and show me groceries.
Now, I put a button in Groceries login.js named "Connect with Facebook account" and I got status=302 code with no result.
Please, if you need any details, don't hesitate to return back to me.
Thanks

Hi @mawoole,
Thank you for your interest in NativeScript.
For your case you could use nativescript-firebase plugin for Facebook authentication in your app. For further help you could review the sample project here  and the given description in the project readme.
Hope this helps

Hi @tsonevn,
Thank you.


Maintenance in reals

There are various things left to do in the reals, including embedding the rationals. This was done haphazardly before the proof of completeness; I've moved some of the relevant definitions and theorems in order to do it earlier.
This created some minor issues (rewrite was getting confused between +, * defined on R and +, * inferred from algebra), so I figured it was time to migrate theorems from algebra to R. This is not done yet, and I'll be traveling for the next couple weeks. This commit is so Jeremy can make changes to min/max in algebra without interfering with my changes. I'll clean things up when I get back to Pittsburgh!

subsumed by #697


Add targets for uploading to S3 + repo cleanup

Fixes #41. Also helps with #27 but doesn't complete it. Still need to figure out the dynamic-naming piece & how to inject SVG into Vue from a URL.
What this PR does do:

Push up all files needed by Vue to S3 (+ include .ind files for our S3 pushes)
Updated Vue code to match names used in S3
Updated README documentation
Bug fix for reshape2::dcast() deprecation
Delete unused 4_animate files
Add temporary fix for including the SVG file (already pushed to S3) in assets/ until @ceenell is able to inject SVG from a URL into Vue.

Results in the current state of the app from a local build to look like this:

Once this is merged, I plan to delete the now unused files in S3. I don't want to do this right away, though, because I believe that we will need to build beta and prod or they will break?


Eliminate dependence of qfvm, close #130

add license to simulator files
eliminate dependence of qfvm

close #130


Update command summary for find and shortcuts for help and upcoming meetings

Put find t/TAG with other find commands, reformat table accordingly
Update shortcuts for help and upcoming meetings

Codecov Report
Base: 60.70% // Head: 60.70% // No change to project coverage :thumbsup:

Coverage data is based on head (1c56729) compared to base (c2dd7de).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #216   +/-   ##
=========================================
  Coverage     60.70%   60.70%
  Complexity      554      554
=========================================
  Files           103      103
  Lines          2059     2059
  Branches        211      211
=========================================
  Hits           1250     1250
  Misses          737      737
  Partials         72       72

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Golang and Rust to Webassembly demo

Looks great, @jnyfah! Would you mind just renaming file names / sidebar titles to simply "Rust" and "Golang" please?

Okay, would do that

I got this now. Almost there:
Error: Image static/static/img/tutorial/golang-wasi.jpg used in docs/WebAssembly/Golang.md not found.     at async Promise.all (index 0) Error: Image static/static/img/tutorial/rust-wasi.jpg used in docs/WebAssembly/Rust.md not found.     at async Promise.all (index 0)

Hi Jennifer, still failing with image not found. Would you mind building locally before committing please?


Remove unnecessary null pointer checks

Extra null pointer checks are not needed in functions like the following.

FoliageItem
Mesh
TextureCubemap

hi, @elfring you can assign this to me

:thought_balloon: Would you become interested to use a development tool like “clang-tidy” for corresponding source code adjustments?

Yeah sure

@manav-06 Are you interested in fixing this up?

Yeah

@manav-06  Alright i am assigning this issue to you.

@Jaysmito101 is this issue completed? I want to work on this issue. Can you please assign this issue to me ?

@suyashmadhesia alright

Extra null pointer checks are not needed in functions like the following.

FoliageItem
Mesh
TextureCubemap

So I have to remove null pointer check in given those three files ?

@suyashmadhesia not just these but throughout the entire project. Those are just some examples.

So I have to remove null pointer check in given these three files ?

I proposed a change (at such source code places at least).
:thought_balloon: Would the linked information sources motivate you to check if more update candidates can be improved accordingly?

@elfring @Jaysmito101 Got it. I have to find and remove all unnecessary null pointer checks throughout the code base.

:thought_balloon: Are you going to select any tools which can be more helpful for such development task?

@elfring I don't know much but I will be using Visual Studio for debugging and development. If you have any suggestion please help me be I am a beginner to open source contribution with little knowledge of gdb and intermediate knowledge of C++.

If you have any suggestion …

:thought_balloon: Did you notice any previous links to the development tool “clang-tidy”?

It showing static member function.

:thought_balloon: I wonder about such an information.

Would you like to try clang-tidy out on the command line?
Did you configure this tool as a plug-in for any integrated development environments?

Maybe, let me try it by myself if I got stuck at any point I will ask it out

Hey @Jaysmito101, can you please assign this issue to me

hey @jaysmitho can you assign this issue to me if it is not solved yet

This issue will be fixed with later pushes.


AuthContext and Package.json updated

Changes:

AuthProvider implemented
 wrapped with AuthProvider
useAuth can be used to get an object with an object defined by the auth interface in AuthProvider.tsx and a function to update the auth state for sign in
Basic stub for AuthSelector made. Pages main purpose is to provide links to the different login pages (probably need to split the login component if wanted to work on separately)

also added a routes and route elements to the App.tsx


SV support is broken

It turns out that none of the SV functions (get_manta_sv, get_combined_sv, get_manta_sv_by_sample, get_manta_sv_by_samples) support capture seq type and there is currently no functionality to return all SVs for all capture samples in GAMBL
Slack Message

Follow this Slack thread.


Release 1.2.0

[x] Create a release ticket in github. Copy this checklist.
[x] Remove TODOs
[x] Make a release branch
[x] Document new state algorithm
[x] Add const docs
[x] Update Readme (supports draft-06)
[x] Add docs to npm ignore

VERSION="..."
VERSIONALPHA="$VERSION-alpha.0"
ISSUE="..."

git checkout -b release/$VERSION

[x] Update package.json version to $VERSIONALPHA and commit

git commit -m "release: Update package version to $VERSIONALPHA #ISSUE"

[x] Tag with the alpha version

git tag $VERSIONALPHA

[ ] Publish test npm version $VERSIONALPHA

npm publish

[ ] Check package internals - contains all required files (lib/, README.md, ./djv.js, package.json, ...), no extra files
[ ] Check with updated json-schema test suite - check generated report
[x] Check speed metrics
[ ] Update package.json version to $VERSION and commit

git commit -m "release: Update package version to $VERSION #ISSUE"

[ ] Create a changelog

npm run changelog

[ ] Tag $VERSION

git tag $VERSION

[ ] Merge release branch to master (via github)
[ ] Publish version $VERSION

npm publish

[ ] Send update to gitter
[ ] Update json-schema supports draft-06 description

Performance
Before

Validator
Relative speed
Number of test runs per second

djv
100%
24087 (± 2.15%)

After
performance

Validator
Relative speed
Number of test runs per second

djv
100%
23386 (± 2.38%)

Validation Errors
Before
All validators fail this test
some languages do not distinguish between different types of numeric value, a float is not an integer even without fractional part
djv failed tests
Some validators have deliberately chosen not to support parts of the spec. Go to the djv homepage to learn if
that is the case for these tests.

test failed
reason

ECMA 262 regex non-compliance, ECMA 262 has no support for \Z anchor from .NET
Expected result: false but validator returned: true

validation of URIs, an invalid protocol-relative URI Reference
Expected result: false but validator returned: true

ref overrides any sibling keywords, ref valid, maxItems ignored
Expected result: true but validator returned: false

base URI change - change folder in subschema, number is valid
Expected result: true but validator returned: false

root ref in remote ref, string is valid
Expected result: true but validator returned: "Maximum call stack size exceeded"

root ref in remote ref, object is invalid
Expected result: false but validator returned: "Maximum call stack size exceeded"

All other tests passed.
After
All validators fail this test
some languages do not distinguish between different types of numeric value, a float is not an integer even without fractional part
All other tests passed.

https://github.com/json-schema-org/json-schema-org.github.io/pull/158


Simplify cutscene detection, detect when player ismoving or inlobby and improve refresh splits

Note

Use bLocView pointer to detect cutscene (@traumvogel)
Use bIsMoving pointer to detect when the player is not moving, useful for reset and start (@traumvogel)
Use GObjects pointer to detect when GameObjects is not instanciated then the player is in lobby, useful for softreset (@traumvogel)
Rename Func TriggerRefreshSplits => IsSettingsChanged
Rename Action UpdateCurrSplit => InitCurrSplit
In UpdateEnabledSplits when all categories are unchecked refresh splits once
In IsSettingsChanged clear splits list even when no category is checked
Call InitCurrSplit only when reset or start and change InitCurrSplit behavior
Add more comments

Maybe InitCurrSplit is only useful in start section since :

After a reset there is always a start
During a run if the player reset manually reset section is not necessarly called before a start

If we worry about complexity and cost. Maybe we could enter in some update section logic every second instead of doing this each clock time. With somethine like :
	// Enter a function every second
	vars.delayIncr = 0;
	Func<bool> EnterHereEverySecond = () => {
		if (vars.delayIncr < refreshRate) {
			vars.delayIncr++;
			return false;
		}
		vars.delayIncr = 0;
		return true;
	};
	vars.EnterHereEverySecond = EnterHereEverySecond;

And in the following lines :
358		if (vars.EnterHereEverySecond() && vars.currSplit < vars.splits.Count)
360		else if (vars.EnterHereEverySecond())
364		if (vars.EnterHereEverySecond() && timer.CurrentPhase != TimerPhase.Ended)

If it's good idea I can commit this.


How get value from CheckBox?

and update current value
{                         "type": "cupertino_switch",                         "args": {                           "activeColor": "#2EB2FD",                           "value": true,                           "onChanged": "##myFunction(key:keyName, value:{{value}})##",                         }                       }

It looks like you are using the old templating syntax, so I'd first recommend moving to the new syntax as documented here.
For any built in Form based widget that has an id field, the framework automatically provides you with code to get the value.  So updating your example...
{
  "type": "row",
  "args": {},
  "children": [{
    "type": "cupertino_switch",
    "id": "my_switch_id",
    "args": {
      "activeColor": "#2EB2FD",
      "value": true
    }
  }, {
    "type": "conditional"
    "args": {
      "conditional": {
        "my_switch_id": true
      },
      "onFalse": {
        "type": "text",
        "args": {
          "text": "Not Checked"
        }
      },
      "child": {
        "type": "text",
        "args": {
          "text": "Checked!"
        }
      }
    }
  }]
}

You can see this in action in the cupertino_switch example and the respective asset file.


ChartContainer throws error when trying to create a chart with mixed chart types

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior
ChartContainer does not render multiple series as intended and throws an error that the children is not of the right type.
Expected behavior
We should be able to create a chart with multiple types of data series.
Minimal reproduction of the problem with instructions
When following the guide to create mixed chart types here:
https://reaviz.io/?path=/docs/docs-advanced-custom-charts--page
an error is thrown.
Demo
https://codesandbox.io/s/reaviz-chartcontainer-bug-k1cj4?file=/src/App.js
Environment

Libs:
- react version: 16.13.1
- reaviz version: 8.1.3

For Tooling issues:
- Node version: v10.16.0
- Platform:  Windows

There is a typo in docs. ChartContainer accepts children as a function:
https://github.com/reaviz/reaviz/blob/master/src/AreaChart/AreaChart.tsx#L302-L312

@SleepWalker Is there somewhere that you would recommend looking to help debug this?

Looks like an example from docs is incomplete/incorrect. Even if you'll properly use ChartContainer you will be required to manually setup all the dependencies for AreaSeries and BarSeries and this requires much code and efforts to complete.
If you'll decide to go with higher order implementations such as AreaChart and BarChart than you will have difficulties to setup axes and grid, tooltips and other user event related features will work only for the graph, that was plotted on top the other.
But, I may be wrong, because I'm new to reaviz, d3 and data visualization


vacuum: Xiaomi Mijia Ultra-thin (STYTJ04ZHM) - supported ?

Does this integration support the Xiaomi Mijia Ultra-thin vacuum cleaner (model: STYTJ04ZHM)
Manufacturer: Dreame Tech
I wont buy unless I know I can safely integrate it into home assistant.
Currently it's only available as a chinese model.
for reference: https://www.xiaomitoday.com/2021/06/02/xiaomi-mijia-ultra-thin-robot-vacuum-cleaner/

https://home.miot-spec.com/s/dreame.vacuum.p2148o
This component supports most
vacuum robot produced by dreame, I think this can also be well supported.


Lazy deserialized interface

DataArray and DataEntry introduced the concept of lazy de-serialization. We can design the whole API with such structures to provide lazy deserialization. E.g. when we do map.put it returns the old value and we de-serialize it by default, but the user may not be interested in the old value at all.

The new Mixed(Map, MultiMap, List, etc) api actually will provide this capability.


Inject script using next/script

Starting with Next.js v11, Vercel recommends to inject scripts using next/script as opposed to using a <script /> tag inside of next/head. See deprecation warning at the bottom of the next/head docs.
If you agree, I'm happy to open a PR addressing this.

Whoops, I just noticed there is already an open PR for this. Closing this. :)

Thanks anyway @garritfra! I am waiting for nextjs to fix an issue until I can merge the pull request.


Replace insert_assertion from devtools by inline_snapshot.snapshot

@alexmojaki Can you help me to fix the tests that are failing? They are failing because we customized those assertions.
If possible, guiding me on what is the supposed workflow instead of doing it yourself.

[ ] Regenerate the lockfile.

I'll open another PR to solve this.


Safari 16.4 shows ResizeObserver loop completed with undelivered notifications

Hi there,
While developing locally in Safari with webpack-dev-server I kept getting the error messages thrown when using <Balance> and resizing the window.  I tried to debug this for a while assuming I was doing something wrong.  Then I tried going to the Vercel homepage  in Safari and viewed the Web Inspector console while resizing the browser.  You can see the same error repeatedly output there: ResizeObserver loop completed with undelivered notifications.
Trying the same steps with Chrome, I did not see any of those messages.
My questions are:

Do I need to worry about this error message, is it indicative of a possible memory leak or other unstable condition?
Is it possible to fix this from happening from within the Balance component?

Any guidance or tips would be appreciated.
Thanks,
Matt

+1, encountering the same issue.

This is still an issue in Safari 17.2.1

Still an issue in Safari. Would love to clear out my Sentry 😂

Noting I also have this runtime issue in Chrome, can't use this until it's fixed


C# Log.Comment example doesn't work

In https://github.com/MicrosoftDocs/windows-driver-docs/blob/staging/windows-driver-docs-pr/taef/wexlogger.md, the C# example doesn't work without first using WEX.Logging.Interop;

@suncats - Yes that would a good idea to include the required using statement in the code sample, as was done in the C++ sample, right above it. So I updated the topic to do that. Thanks for seeing this gap and filing this issue.


Updated Tentacle Registration

Modified tentacle registration to allow config to specify the NIC adapter to bind to.

Resubmitting PR to staging, closing this one.


Would you like to explain step 8

I can run the web server, but  can see nothing in the categories page.
I guess this has to do with the following command
python manage.py loaddata /path/to/data.json
could you explain the argument '/path/to/data.json'
How do I load your demo data ?
thanks

Demo data was not uploaded to GitHub.
You may create categories and products on admin page.

On the admin page I added a category with a photo, but I can't browse the photo while I refresh that page. I   checked the critical item like "DOCUMENT_ROOT  and  STATIC_ROOT , and everything seems ok.   The only suspect is the warning messages when I started the webserver .
It said :
Warning :local configuration not found
Please see store/settings/config.sample.py for details.
I have made a copy of config.sample.py and modified some information according to my envirnment.
What might the reason be then ?
I 'm so confused.   look forward to your reply and thanks .

If you run it with manage.py runserver (for testing), you can leave all configs as default by place a blank config.py on store/settings/ folder.
Configs related to image uploading are MEDIA_ROOT and MEDIA_URL (default value are defined in here).
Sorry I can't remember many details here after two years. If you saw error messages or somethings, you may post them here so I can have a look.


Feature request: (Mirrormaker) External Kafka integration

What is currently missing?
When following the document below and configuring an external Apache Kafka Cluster (step 2)  we need a service integration resource to configure the external kafka.
https://docs.aiven.io/docs/products/kafka/kafka-mirrormaker/howto/integrate-external-kafka-cluster.html
How could this be improved?
This could be a new type of aiven_service_integration. Proposed resources:
resource "aiven_service_integration" "external-kafka-cluster" {
  project                             = aiven_project.myname.project
  destination_endpoint_id  = aiven_service_integration_endpoint.kafka_endpoint.id
  integration_type              = "kafka"
  source_service_name       = aiven_pg.myservice.service_name
}

resource "aiven_service_integration_endpoint" "kafka_endpoint" {
  project       = aiven_project.myname.project
  endpoint_name = "externalkafka"
  endpoint_type = "kafka"

  kafka_config {
   endppoint_name = "cluster"
   bootstrap_server = "kafka-demo-bootstrap.testing.com:9094"
    type = "SSL"
    ca_certificate = local.ca_certificate
    client_certificate = local.client_certificate
    client_key = local.client_key
  }
}

Is this a feature you would work on yourself?
Not at present.

Looks like now we have this integration type.


V2.0.2

In this version:

Add more currency info:

Decimal digits
Plural name
Symbol on left,
Decimal separator
Thousands separator

Add more currencies:

Albanian Lek
Armenian Dram
Azerbaijani Manat
Burundian Franc
Botswanan Pula
Belarussian Ruble

Remove Lithuanian Litas currency (was replaced by the Euro).

Before you merge this, could you add this to the Currency class so it's JSON enabled? I think I got all the fields there.
Map<String, dynamic> toJson() => {
  'name': name,
  'code': code,
  'symbol': symbol,
  'flag': flag,
  'name_plural': name_plural,
  'decimal_digits': decimal_digits,
  'thousands_separator': thousands_separator,
  'decimal_separator': decimal_separator,
  'space_between_amount_and_symbol': space_between_amount_and_symbol,
  'symbol_on_left': symbol_on_left,
  'number': number,
};

@voivoed  Sorry I am the only contributor, so I create the PR in order to merge the branch.
I will add toJson in next version. Is really important to you? If yes I can release a new version with toJson.

— Mauro

On Mar 18, 2021, at 3:55 PM, Daniel Ioannou @.***> wrote:

@voivoed Sorry I am the only contributor, so I create the PR in order to merge the branch.
I will add toJson in next version. Is really important to you? If yes I can release a new version with toJson.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hey,
I’m sorry to bug you so much about this. I do need to serialize the currency info to save it in my settings but I can do it myself in my app. Thank you so much for the work you’ve done.
— Mauro

On Mar 18, 2021, at 4:05 PM, Mauro Reis @.***> wrote:

— Mauro

On Mar 18, 2021, at 3:55 PM, Daniel Ioannou @.***> wrote:

@voivoed Sorry I am the only contributor, so I create the PR in order to merge the branch.
I will add toJson in next version. Is really important to you? If yes I can release a new version with toJson.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


[Designer] Add isRequired to inputs

Automatically generated task. Any updates to the title and body will be overwritten. Do NOT modify the labels.
Designer implementation of #1978

Implemented with #2547


Support for $geometry filter

This is good. Once the discussions are answered/resolved, will merge.

waiting for travis


Is timeserver not responding?

Is there some error with the server?
It doesn't respond to requests since around 10:30 Stockholm time?

Is there some error with the server?
It doesn't respond to requests since around 10:30 Stockholm time?

Correct, I'm upgrading... Sorry for the disruption...

Ok!
Thank you for the superquick response!
When do you yhink it will be up again?
Best regards, Anders

Thank you for the superquick response!
When do you yhink it will be up again?

It's up again.

Supermany thanks!
Best regards, Anders

https://github.com/ropg/ezTime/blob/7b3c8aa020be818ac149e0762543ac5e81ccfabe/src/ezTime.h#L153-L155

@ropg can't resolve timezones anymore, so did your upgrade break stuff?

Hello, The server retreive of TZ location fails

Hi!
Do you have any update of what has happened and when it will work again?
Best regards,
Anders

Hello, The server is currently not responding - do you know when it will be available again?
Thanks in advance.

FWIW https://github.com/ropg/ezTime/pull/123 makes setting a custom server easier.

Holy crap, the 2343 was a mistake on my end, and then I didn't check my computer again. I'm so, so sorry...
it should now be up at 2342. Please test.
Sorry...
(And yes, I think it would make sense to have a service like timezoned have more servers, but it's my own homebrew protocol instead of something standardized that has an RFC and all. Will work on putting it on the same box that hosts my backup nameserver so there's at least two.)

The port is still not open...

Now server is responding with timezone definition on port 2342 so it seems ok!
Thanks!

The port is still not open...

You can´t scan UDP! Because it´s a connectionless protocol.

To check if it's working, use:
$ nc -u timezoned.rop.nl 2342

And then type any identifier and follow it with a Ctrl-D (and no return). You'll then see the timezone info right after what you typed:
londonOK Europe/London GMT0BST,M3.5.0/1,M10.5.0

And yes, I think it would make sense to have a service like timezoned have more servers

Did a crude attempt here, putting your code in a container image.
https://github.com/TheGroundZero/ezTimeContainer


Schema reactivity

I have a quickForm rendered with the schema option and my schema is constructed from a reactive-var. Very much like the code in the autoform-demo package.
It is almost working but I am experiencing an error when a field is removed from the schema.
Adding fields works like a charm but removing it throws error for every autoform template related to the field being removed.
Exception in template helper: Error: Invalid field name: 7e36a84b8ceb597e95717ff9
    at Object.getDefs (http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:257:13)
    at Object.autoFormGetComponentContext [as getComponentContext] (http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:532:20)
    at Object.afQuickFieldIsGroup (http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:7238:30)
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:2786:16
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:1607:16
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:2834:66
    at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:3382:12)
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:2833:27
    at Object.Spacebars.call (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:172:18)
    at http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:7189:22
debug.js:41 Exception in template helper: Error: Invalid field name: 7e36a84b8ceb597e95717ff9
    at Object.getDefs (http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:257:13)
    at Object.autoFormGetComponentContext [as getComponentContext] (http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:532:20)
    at Object.afQuickFieldIsFieldArray (http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:7244:30)
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:2786:16
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:1607:16
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:2834:66
    at Function.Template._withTemplateInstanceFunc (http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:3382:12)
    at http://localhost:3000/packages/blaze.js?4e49999979a58da0e2265f7bd3f5910f9901b07b:2833:27
    at Object.Spacebars.call (http://localhost:3000/packages/spacebars.js?7bafbe05ec09b6bbb6a3b276537e4995ab298a2f:172:18)
    at http://localhost:3000/packages/aldeed_autoform.js?5078663143adf2fce70916f147f7080a0634563d:7194:24

And this goes on...
I thought my error was related to https://github.com/aldeed/meteor-autoform/issues/794 but I still have the error after the update this morning.
Should I use the workaround trick you used in the autoform-demo package ?

Yes, you probably still need the workaround. The error can sometimes appear in the demo pkg, too. At some point I can probably make it work properly, but it's difficult to debug the blaze reactivity stuff.

Hi @elGusto ,
I've been trying to get autoform to work with reactive schema using reactive var, but not getting it somehow:
schematics = new ReactiveVar({});
var field1 = {
  name: {
    type : String
  }
}
schematics.set(field1);
Schema.fieldBuilder = new SimpleSchema(schematics.get());

Template.newField.events({
  'click #option1' : function(tmpl, ev) {
    // add something to json.
    var currentStruc = schematics.get()
    _.extend(currentStruc, {
      phone : {
        type : String
      }
    });
    schematics.set(currentStruc);
  }
});

Console output shows that a new field has been added to schematics, but the code does not reactively update and my page stays the same, just wondering what I'm doing wrong here, also if there is a problem with actually making the schema completely reactive, I believe the workaround using destroyForm in issue 353, is that correct?

Hi @elGusto,
which workaround trick are you referring to?

@Gigacross I am talking about this https://github.com/aldeed/autoform-demo/blob/master/client/views/quickform/quickform.html#L57-L59
Based on the code that you provided, I am not able to tell what you are doing wrong. Are you giving the schema to autoform with a helper using your reactiveVar.get()?

@elGusto Thanks! I implemented that work around but I'm still getting errors, not entirely sure where I went wrong... basically the idea is to delete the whole form, and recreate it.
I managed to find out how it was implemented after looking at the example code provided in the demo, so I managed to successfully implement dynamic schema, hence I removed my reply here. But now I don't know how to change the key / rename the title of the key i.e. { name : { type : String }}, do you have a recommended way of renaming name reactively?
I think I have to make a full layer between user and making the schema, so basically user -> buildSchemaPkg -> generate schema -> feed into autoform, all my attempts to create a dynamic form properly has been futile... that way autoform don't really have to deal with all the reactivity also...

@Gigacross I am also working on the exact thing. I didn't get there yet but will later today if all is good.
I will update here when I can really give you an answer.

@elGusto Cool! In the meantime I'll also attempt to create that myself, may be not build into a full package, but I'll be able to put a gist together or something, if you happen to accomplish it before I do, let me know! Thanks in advance :)

The workaround is no longer working for me on version 5. I'll see if I can track down a fix.

Nevermind, the concept of the workaround still works for me. I ended up making the schema a ReactiveVar so I can control when it changes. First I set destroyForm to true, which will cause the template to remove the existing autoForm from the DOM, then I set the schema var to the NEW schema, and then the second computation fires to set destroyForm back to false.
It still feels horrible doing it this way, as the timing is ambiguous but this pattern seems to be working for me.
<template name="stepView_form">
    {{#if destroyForm}}
    {{! this is a wonky workaround for the fact that autoform doesn't handle reactively changing schema attribute, which should be fixed eventually}}
    {{else}}
        {{#autoForm id="stepViewForm" schema=schema doc=answers resetOnSuccess=false}}
            {{#each afFieldNames}}
                {{> afQuickField name=name options=afOptionsFromSchema}}
            {{/each}}
        {{/autoForm}}
    {{/if}}
</template>

var currentStep;
var destroyForm = new ReactiveVar(false);
var schema = new ReactiveVar(null);

Template.stepView_form.created = function(){
  schema.set(getSchema(this.data.step));

  this.autorun(function(){
    var step = Template.currentData().step;

    if(step.slug !== currentStep){
      destroyForm.set(true);
      currentStep = step.slug;
      schema.set(getSchema(step));
    }
  });
};

Template.stepView_form.rendered = function(){

  this.autorun(function () {
    if (destroyForm.get()) {
      destroyForm.set(false);
    }
  });
};

Template.stepView_form.helpers({

  destroyForm: function(){
    return destroyForm.get();
  },

  /**
   * Create a SimpleSchema to represent the step
   */
  schema: function(){
    return schema.get();
  }
});

function getSchema(step){
  // this function builds a schema object, I removed the code for simplicity
  return new SimpleSchema(schema);
}

Yep this is it but I still get the errors like before, the workaround didn't correct the bug which occurs only when a field is removed from the schema.
I made a package to create autoform schemas with the UI. The goal is to provide a simple form builder that then could be used to create some sort of CMS system. It is still in the (very) early stage but I will publish it soon enough when I have time to clean some stuff out.

Ive built a similar AF schema builder in a proprietary app I'm making.
Maybe when you are done we could tackle the schema reactivity bugs together.
On 31 Mar 2015 08:05, "Gus"<EMAIL_ADDRESS>wrote:

Yep this is it but I still get the errors like before, the workaround
didn't correct the bug which occurs only when a field is removed from the
schema.
I made a package to create autoform schemas with the UI. The goal is to
provide a simple form builder that then could be used to create some sort
of CMS system. It is still in the (very) early stage but I will publish it
soon enough when I have time to clean some stuff out.
—
Reply to this email directly or view it on GitHub
https://github.com/aldeed/meteor-autoform/issues/802#issuecomment-88123623
.

I have prepared a branch of autoform that allows for the schema to be reactive.
It's not an ideal solution, but this may work for our needs. I would appreciate any help you can provide in testing the branch. I wouldn't recommend it for production until it is tested further.
https://github.com/abecks/meteor-autoform/tree/reactive-schema
Using this branch, you can return a SimpleSchema object to the schema attribute of a form (via a helper), and the form will re-render without errors.
<template name="form">
    <button data-schema="schemaOne" class="btn btn-change-schema">One</button>
    <button data-schema="schemaTwo" class="btn btn-change-schema">Two</button>

    {{> quickForm schema=formSchema doc=this id="testForm"}}
</template>

Schema = {
  schemaOne: new SimpleSchema({
    name: {
      type: String
    },
    options: {
      type: String,
      autoform: {
        options: function () {
          return {
            'Test': 'test',
            'Test2': 'test2'
          }
        }
      }
    }
  }),
  schemaTwo: new SimpleSchema({
    phone: {
      type: String
    }
  })
};

if (Meteor.isClient) {
  Session.setDefault('formSchema', 'schemaOne');

  Template.form.helpers({
    formSchema: function () {
      return Schema[Session.get('formSchema')];
    }
  });

  Template.form.events({
    'click .btn-change-schema': function (e) {
      var $btn = $(e.target);
      var schema = $btn.data('schema');

      Session.set('formSchema', schema);
    }
  });
}

I've made as few changes as possible. The problem lies in how the AutoForm context is passed around to field template helpers. Field's grab their "AutoForm context" from calls to AutoForm.getSchemaForField or AutoForm.Utillity.getComponentContext, both of which rely on AutoForm.getCurrentDataForForm. getCurrentDataForForm retrieves the data using Blaze.getData. This means that all template helpers have a reactive dependency on the data context of the form itself.
A change in schema will cause all of the fields in the form to re-render. The problem is that the helper's reactive dependency on Blaze.getData triggers BEFORE the re-rendering of the form fields. This causes all sorts of errors as the field's try to look up their data context again, as they don't exist in the new schema.
My branch adds a few checks and try/catch enclosures to prevent fields from throwing errors when they no longer exist in the schema.

I've just ran across a similar issue when changing the schema reactively, I assume this fix hasn't made it into core yet @abecks?

Not yet. Sorry guys. Will try to resubmit my PR soon. You can always take
my reactive schema branch for yourself.
On Aug 5, 2015 11:52 AM, "Chris Waring"<EMAIL_ADDRESS>wrote:

I've just ran across a similar issue when changing the schema reactively,
I assume this fix hasn't made it into core yet @abecks
https://github.com/abecks?
—
Reply to this email directly or view it on GitHub
https://github.com/aldeed/meteor-autoform/issues/802#issuecomment-128106891
.

Resubmitted my PR, watch #1132 for updates.

@abecks :+1: thanks man!

@abecks any new with this PR?
Thanks

Update and revive @abecks PR(https://github.com/aldeed/meteor-autoform/pull/1132) here: https://github.com/aldeed/meteor-autoform/pull/1205

PR has been merged. Will release new version soon.


Extension refactoring ideas

This collects some notes and refactoring ideas for the extension.

/namada-interface/packages/storage/src/store/MemoryKVStoreProvider.ts is named by a class in it, but not the exported class. I think it should be MemoryKVStore.ts.

namada-interface/packages/storage/src/store/index.ts seem to be exporting export * from "./BaseKVStore"; but it seem not to be used outside of this module.

Add comments to explain the meaning of the key types, such as
export enum KeyStoreType {
  Mnemonic = "mnemonic",
  PrivateKey = "private-key",
}

export type DerivedAccount = {
  id: string;
  address: string;
  alias?: string;
  establishedAddress?: string;
  parentId?: string;
  path: Bip44Path;
  type: KeyStoreType;
};

Add comments & tests
// namada-interface/apps/extension/src/App/Accounts/AddAccount.tsx
const findNextIndex = (accounts: DerivedAccount[]): number => {
  let maxIndex = 0;

  accounts
    .filter((account) => account.type !== KeyStoreType.Mnemonic)
    .forEach((account) => {
      const { index = 0 } = account.path;
      maxIndex = index + 1;
    });

  return maxIndex;
};

/namada-interface/packages/storage/src/store/MemoryKVStoreProvider.ts is named by a class in it, but not the exported class. I think it should be MemoryKVStore.ts.

FYI - This has been deleted entirely in the next PR: #105, along with BaseKVStore

namada-interface/packages/storage/src/store/index.ts seem to be exporting export * from "./BaseKVStore"; but it seem not to be used outside of this module.

This file has been removed in #105

There is a jumpy UI element. This is as it gets a border of 1px only when pressed down.
https://user-images.githubusercontent.com/2773320/196744707-3fced2f0-9177-4768-b3c5-39db58745b00.mov

Change name to plural
// namada-interface/apps/extension/src/router/Router.ts

export abstract class Router {
  protected msgRegistry: MessageRegistry = new MessageRegistry();
  protected registeredHandler: Map<string, Handler> = new Map();

// to

export abstract class Router {
  protected msgRegistry: MessageRegistry = new MessageRegistry();
  protected registeredHandlers: Map<string, Handler> = new Map();

// apps/extension/src/App/Accounts/AddAccount.tsx

const validateAccount = (
  account: number,

to
// apps/extension/src/App/Accounts/AddAccount.tsx

// this validates for now that the account do not exist before with the given path
const validateAccountDoNotExist = (
 account: number,

I feel that it is more effortless to define Borsh serializing/deserializing in Rust and just expose all needed types/funcs from there. We have some at namada-interface/packages/types/src/tx/schemashielded.ts and a few others in the containing folder.
In case they are kept on TypeScript side, unified naming (class and it's schema definition) and keeping only the ones required outside of file public.

refactoring idea
// apps/extension/src/provider/Signer.ts

import {
  TransferMsgValue,
  TransferMsgSchema
} from "@anoma/types";

const transferMsgValue = new TransferMsgValue({
  source,
  target,
  token,
  amount,
  key,
  shielded,
});

const transferMessage = new Message<TransferMsgValue>();
const serializedTransfer = transferMessage.encode(
  TransferMsgSchema,
  transferMsgValue
);

to
// apps/extension/src/provider/Signer.ts

import {
  TransferMsgValue,
} from "@anoma/types";

const transferMsgValue = new TransferMsgValue({
  source,
  target,
  token,
  amount,
  key,
  shielded,
});

const aaa = transferMsgValue.toBorshEncoded();

Idea for better type safety and easier refactoring:
In apps/namada-interface/src/slices/transfers.ts submitTransferTransaction we get an integration and use signer from that. We get it by using a getter and then casing it to Signer. Further, that integration is of type Integration (apps/namada-interface/src/services/integrations.tsx).
Integration is type Integration = typeof Anoma | typeof Keplr | typeof Metamask;
All of these implement Integration so they contain integration(). But they all return a different thing from signer: signer () => S | undefined. Only Namada returns Signer so if there is another "Integration" returned by the signer(), type system does it but it will crash.

An idea for establishing more clear connection between types where Borsh schema is defined in TypeScript. I feel that it could lead to annoying debugging sessions, when Rust suddenly cannot deserialise something:
In the current state we pass some objects from TypeScript to Rust using borsh-js. In these cases we have defined a schema in TS and using that schema we turn objects to Uint8Array and pass them to Rust functions. On the Rust side the byte array is again deserialised to Rust objects. This works but we have to do the mapping based on mere strings and not any actual types that would warn us when we refactor or simply change the types.
Starting to do some inline types in JS might also not be good, as it might need some hacks and tricks. Maybe simply placing a URL to the schema definition next to the Rust types that we do like this.
Otherwise some Serde deserialization might be more robust to bring the objects over.


tables

working on it.

missing some tables seen on almayer apparently

i looked at the tables code at cmss13.
looks like missing tables are: prison table, almayer table, cloth table.


Add support for search:extracted to MatchDocumentSummary

Now that the search option extract-document-data can insert a search:extracted element into the search results, let's add support to get that using MatchDocumentSummary (via SearchHandle).

This is a duplicate of #262


RFC: BitReader.Reset() and BitReader.SetReader(io.Reader) to allow use with sync.Pool

I'd like to be able to reuse a BitReader from a sync.Pool but have no way to zero-value out its 3 encapsulated fields.
In order to use it in a sync.Pool I'd need to implement both a Reset() method (to set count to 0) as well as a SetReader(io.Reader) method to swap out r without allocating a new BitReader.
(To be symmetrical between the reader the writer Reset() and SetWriter() could also be added to BitWriter but I don't personally have a need for it.)
I'll happily implement both methods and tests but just wanted to introduce the idea first to see if you're open to the addition.
An alternative would be a single method Reset(io.Reader) which accomplishes both. While this is a more succinct interface the common pattern for me (with a thin wrapper around sync.Pool) is to prepare an object (i.e. SetReader()) when getting it from the sync.Pool and have the sync.Pool take care of resetting it (i.e. Reset()) implicitly after i've Put() it back into the pool.

Sure, I'll accept a patch for Reset(io.Reader.   Having one for the Writer probably makes sense too, even if it's not yet going to be used.

Implemented in https://github.com/dgryski/go-bitstream/pull/8


Updates Overwriting Rules?

Hi, I've noticed that at least one of my rules seems to occasionally get reset to the default. So far it's happened 2-4 times and I suspect it's related to either automatic updates for the extension or my Chromium updates. I only change the destination path in that rule to capitalise the directory name and add a sub-directory, nothing else. This is a minor nuisance but I thought I should mention it anyway. Perhaps this behaviour is intended and I'm doing something wrong?

please provide steps to reproduce this issue

I'm not sure what you mean. I installed the extension from the extension store website, opened up the Options page, then changed one of the Destination Path values from "torrents/" to "Torrents/Intake/". It works for a while (maybe 0.5 to 3 months) and then eventually goes back to "torrents/". I'll let you know if/when it happens again.

I can confirm. This has happened for me as well. 😭
Chrome has recently popped up the extensions rules page several times while I didn't ask for it, and other than feeling annoyed and dismissing it instantly (because the download did happen just fine!) I just ignored it. Highly annoying, though!

Every update just overwrites the rules. so troublesome.
Even when i save them in a file i still have to reimport everytime

It still hasn't happened to me since the last time I mentioned it. Not sure if there have been any new releases since then. Options page says version 0.3.2.

there is only one old code that updates rules on version update if version was less than 0.2.4 https://github.com/unintended/download-organizer-chrome-extension/blob/master/manager.js#L124
there is no more code that changes rules automatically

Actually the 0.2.4 version was a lot of time ago, so this migrations takes no effect anymore and I will remove it in the next release

Well specifically for me the Issue occurs whenever disk space gets full and i free up some space.
the extension always opened up options page like an update had occured.
looking at the code i think its because you use localStorage instead of chrome storage local.


When going off duty- ability to change on/off-duty time

shift review - before or after showing reports?

Design and flow as per https://mongodb.invisionapp.com/share/RWX43EEF567#/screens/415937302