Bugs and Suggestions

Hi its me again hope you are doing great, I have a new report. Thanks.
Moving too fast in select mode
If you abruptly drag a feature, you can get out of it.
And if you move close to a vertex, you can move it.
All of this without releasing the left mouse button.

This happens because terradraw validate if the mouse is inside the figure each time we drag before moving it, we could change that validation to do it only when dragging starts, not every time 'drag' event is fired. And once we know the user started with a valid position we can move it wherever the event is saying.
Proposed Solution:
Preserve the user choice to drag the whole feature or just a vertex when start dragging (onDragStart), and just keep updating that option (onDrag), this way no matter how fast the user moves, as long as it does not release the button terradraw will only update the initial choice.
Check if the user wants to drag a vertex or the whole feature
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-66b8b33e810bc4e27d8af887cbf5f6415b2ad6156d33ed947972cb459ca94e3bR475-R497
Then on drag validate which type of drag we are doing and ONLY do that one.
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-66b8b33e810bc4e27d8af887cbf5f6415b2ad6156d33ed947972cb459ca94e3bR586
Mapbox/Maplibre Performance in select mode
Although now terra draw has a great performance in maplibre/mapbox there is still a lot of lag when in select mode.
This is likely because it has to update 3 layers (polygon, outline and points) each time we drag anything.
So first of all I know that terra draw wants to keep the same design and style across all adapters so maybe this is not the best solution.
But here it is, basically an approach to this could be to avoid updating the points layer when on select mode.
This could be done by checking if the user want to drag the whole feature, then onDragStart delete all selection and mid points, and restore them onDragEnd so that when the user drags would only see the polygon moving without any points.
In the other hand if the user want to drag only a vertex then delete all midpoints (because I discovered that all of them are sent to render method no matter if they are actually updated or not) and also delete the single selection point that the user selected, this way we wont update the points layer since the remaining selection points are not moving anywhere and could serve as reference.
Delete selection and midpoints onDragStart
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-66b8b33e810bc4e27d8af887cbf5f6415b2ad6156d33ed947972cb459ca94e3bR499-R533
Re-construct points onDragEnd
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-66b8b33e810bc4e27d8af887cbf5f6415b2ad6156d33ed947972cb459ca94e3bR621-R624
Features are not updated when there's a change in style
This happens in maplibre/mapbox adapter since style changes passes an array of all features as unchanged to render method.
Which now for performance reasons wont do anything if everything is unchanged.
Proposed Solution:
Just as we have a deletionOccured in render mode of, we could also have somthing like
const styleChangeOccured = Object.values(this.changedIds).every(change=>!change) && changes.unchanged.length > 0;
To force the update of the features if so.
For example
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-b0a667c9d571e4bfe0433d3e4fb6fbe181bec821b260b5250e1b6dffd073de99R366-R373
Error when quitting terradraw while have something selected
To reproduce it:
Using maplibre adpater, select a feature, then call the stop method of terra draw (which deletes everything).
You will see all features being deleted for a fraction of second and then the selected feature will appear again but without any points and not being able to modify it.
This happens because when we call the stop method of the main terradraw class then it correctly call the cleanUp method of the mode which will delete all selection and midpoints of the figure, but because the render method is waiting 16 milliseconds to do the update and as it waits we delete all features then at the end we would see only the selected feature again on the map.
Proposed Solution:
One way it can be solved its just by removing the debounce of the render method of the mapbox adapter
Removing debounce rendering
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-b0a667c9d571e4bfe0433d3e4fb6fbe181bec821b260b5250e1b6dffd073de99L295-L303
Avoid error when moving figure to the edges of the map
Currently there's an error when you move a figure outside the current map in maplibre adapter (when you have setRenderWorldCopies ON).
Also it would be nice to prevent figures going outside the latitude range (-90,90).
For example validating -85 , 85 in lat
https://github.com/JamesLMilner/terra-draw/compare/main...alanOracle:terra-draw:studio-approach?expand=1#diff-ace904c8e4070082cedc13adb31cfeb6a24851b1aea7ac5bc6304a6787903b27R68-R86
Prevent right click
It would be nice to have an option to ignore all right clicks so that it doesn't mix with other actions to be performed outside terradraw

@alanOracle thanks for raising all these - I have been looking into some of them. In future if possible can I kindly ask you to raise the issues individually so I can better keep track of them? No need to do it for these as I will create seperate tickets where necessary but just a polite request for the future!

Hey @alanOracle thanks again for raising these. I think they are either resolved (use the latest version 0.0.1-alpha.38) or there is a separate open issue for them now, except your last point about preventing right click. Could you create a separate issue with more information with what you mean?
Also I noticed the typings for setModeStyling were wrong, you need to pass the styling interface that is provided to the initialise the mode, so for example for polygon you need to pass a partial PolygonStyling object:
https://github.com/JamesLMilner/terra-draw/blob/be0f8f07f7d1043cd3bd6fa65c17edd81e1f9bdf/src/modes/polygon/polygon.mode.ts#L26


Pkgdown site

add pkgdown-based site!

going to merge this so I can setup GitHub Pages (needs to have a /docs folder in master to select that option)

we live! https://data-edu.github.io/dataedu/

whoops, turns out we have to use the master branch, not just /docs

okay, last word on this is you do have to use /docs - it was just taking a little while to load!


Use ES6

Would be great to use ES6 in the project and let Babel and rollup handle all the transpilation complexity.

addressed https://github.com/ApoorvSaxena/lozad.js/pull/9


improve amazon aws-java-sdk dependencies

for now carina-aws-s3 include whole aws-java-sdk. We have to try to minimize number of dependencies using exact modules we need.
Among all dependencies in carina aws dependencies have too huge list of jars

6.1.25+
https://github.com/qaprosoft/carina/pull/786

will there be any bad effects about the huge list dependencies of carina aws ? @vdelendik


Some important feedforward models to be supported at first stage

We are going to support  the conversion of models as follows at first stage. They all can be found in our models bank, some maybe only need to be verified after #19 merged, and some lack of necessary operators.

[x]  recognize_digits ( #19 @kuke)
[ ]  ResNet50
[ ]  VGG16
[ ]  MobileNet
[ ]  SE-ResNeXt
[ ]  Inception-v4 (first add the Fluid config in models/fluid/image_classification)

Even all the models above are supported, only a small subset of operators  are used and verified. So for the rest operators, we may need another task:

[ ] Design the unit test framework to test every operator's conversion (@kuke)

Hints:

Choose one task every time and assign yourself when you are really ready for working on it;
Please append the model to the supported models section in README like #19 after its conversion being validated.

@kuke I think we can collect all the ops needed to be converted included by the listed models.
It's better to do  Design the unit test framework to test every operator's conversion first and we can implement converter for fluid ops parallel. When all operator converter finished, I think it would easy to validate the models.

I also agree with that  unit test is very important, so I assign it to myself.  But if you are interested in verifying the correctness of models' conversion, you can work on one now. From my experience, a runnable model will help you test the related operator's conversion and the unit test is not necessary.

I'll take READMEs/installation, etc. Thanks @kuke and @pkuyym for your work on the models.

@varunarora Great! It is the time to have a friendly README and installation. I am going to continue to work on models' conversion. I think rest of us can pick up a model or general operators' conversion. You see we have a lot of operators waiting to be implemented the conversion.


docs(cim): adds code comments rest of cim packages

:tada: This PR is included in version 2.2.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Implementers need to consider validity before and after set_oracle() is called, rather than being able to make a decision about which one to use at construction time (which would in turn allow the entire thing to be hidden behind a type).
Users need to ensure that the sequence of indices in the fetch() calls matches the sequence given by the oracle.
Users don't get to see the oracle predictions, which makes life difficult if the sequence is easily reproduced (e.g., streamed from an RNG). This would require some careful workarounds to expose the current and old predictions.

I propose a second suite of dense/sparse_row/column virtual Matrix overloads that accept a (probably unique) pointer to an oracle and returns OracleAwareExtractor objects, which have the following methods:

fetch() is modified so that it mutates the first row/column index argument with the next draw from the oracle stream. We also provide an overload that omits the first argument altogether if it's not needed.
finish() indicates whether the oracle stream is completed.

Then users can just do:
int pos;
while (!(ext->finish())) {
     auto current = ext->fetch(pos);
     // Do something at 'pos'.
}

Similarly, implementers can more tightly control the use of oracles at construction time in their own virtual Matrix methods. We'll provide a macro to automatically fill in methods when oracles are not actually needed by a class, so implementers aren't too burdened by this new requirement. (Note that Matrix implementations that do respond to oracles are already pretty complicated, so this shouldn't really add a lot of extra complication on top of those.)

Some more thoughts:
Oracles should have a total() method that specifies the number of predictions that they're going to make. It is very unusual to get a real stream of unlimited predictions, and most of the time, this can be accommodated by just breaking the stream into chunks and creating a new Extractor anyway. By providing the total, Matrix classes have the option of performing at least one initial pass through the set of predictions to decide on an extraction strategy (see below).
Replace the fill method with a simple get(i) to get the i-th prediction. This is more intuitive and the cost of a virtual function is not relevant. It also frees various implementations from having to store intermediate buffers, which incur their own cost; for example, we can get rid of the crazy caching that DelayedCombine is forced to do.
An implication of the random-access get() signature above is that the oracle references can be treated as const. There is no need for an Oracle to store the state, and allows us to re-use the same oracle pointer without cloning. This is again useful for DelayedCombine and other multi-matrix delayed operations (e.g., BinaryIsometricOp).
This should all be helpful for various oracle-aware matrices. For example, the HDF5 subclasses can run through the set of predictions, determine if they are (near-)consecutive, and then allocate a big buffer for contiguous extraction via the minimum number of calls to the HDF5 library. Then they could do a second pass through the set of predictions and actually perform the extraction. Right now, this wouldn't be possible because the oracles can't go back.


Issue with the script when exporting to bmp

Hello and thanks for your work. That's a pretty awesome tool that can save a lot of time.
I did encounter a bit of an issue : when exporting multiple layers to bmp format, 16 bit, trimmed by layer, the script opens a dialog to save the actual PSD file. It won't go any further than this dialog. And if you save, well, you save your PSD file and the dialog will reopen for every layer.
Probably BMP related because with the same settings in PNG works flawlessly.

Update : revived my old laptop running CS4 and the above setup works perfect. So the issue I described is CS6-related.


Change address from F4 to Ethereum address

When using get-balance function in line 72, code fails with "Error: network does not support ENS". Replace F4 address by Ethereum Address to resolve it.

Upvoting @nandit123 suggestion. Ran into the same error until I used the eth address instead of the f4 address.

Bump this, users have started to complain even in the slack :smiling_face_with_tear:
https://filecoinproject.slack.com/archives/C03KGBVJCKG/p1668858711498259


Drop support for Go version 1.11

What is the purpose of this change? What does it change?
Drop support for Go version 1.11. As of now, that Go version no longer receives security updates and it starts to block dependency updates.
Was the change discussed in an issue or in the forum before?
For example #2733 requires at least Go version 1.12.
Checklist

[x] I have read the Contribution Guidelines
[x] I have enabled maintainer edits for this PR
[ ] I have added tests for all changes in this PR
[x] I have added documentation for the changes (in the manual)
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

Since restic is a single program rather than a library, wouldn't moving to 1.14 be better?
Otherwise, LGTM. The patch is complete AFAICT.


change route to topic Names

corresponding PR on SC-Client: https://github.com/hpi-schul-cloud/schulcloud-client/pull/2137
link to ticket: https://ticketsystem.schul-cloud.org/browse/SC-6442

a div was changed to  a button in the PR mentioned above, requiring the path to be updated

Please use data-test-Ida to avoid fixed patches

Ids. :)


Minimal support for JSON-LD files

#56334 and #63270 already have requested better built-in support for JSON-LD by improving the JSON language features extension to include hover and completion for JSON-LD. That is not this issue.
I propose minimal support for JSON-LD, which basically just means when I open a .jsonld file or a file with the application/ld+json MIME type, it gets treated like normal JSON.
Right now when I do this, it doesn't detect the language at all and provides no syntax highlighting.

This should be very easy to implement, just by modifying the language contribution settings in the builtin JSON extension.

I pushed a change that .jsonld is now associated to JSON


vagrant up failing with mismatched guest editions

I recently upgraded my virtualbox installation to version <IP_ADDRESS>7 and my vagrant version to 2.2.7.  I installed the extpack associated with this version of virtualbox.
I then cloned the vagrant-boxes repository and tried to run vagrant up in the oraclelinux/7 folder.
All seems well until I am informed that the guest editions do not match and when vagrant/vbox try to upgrade the guest editions there is a failure which stops the provisioning of my vagrant box.
A partial log of the session follows:
Microsoft Windows [Version 10.0.18362.657]
(c) 2019 Microsoft Corporation. All rights reserved.
c:>v:
V:>cd 7
V:\7>vagrant up
getting Proxy Configuration from Host...
Bringing machine 'ol7-vagrant' up with 'virtualbox' provider...
==> ol7-vagrant: Importing base box 'ol7-latest'...
==> ol7-vagrant: Matching MAC address for NAT networking...
==> ol7-vagrant: Setting the name of the VM: ol7-vagrant
==> ol7-vagrant: Clearing any previously set network interfaces...
==> ol7-vagrant: Preparing network interfaces based on configuration...
ol7-vagrant: Adapter 1: nat
==> ol7-vagrant: Forwarding ports...
ol7-vagrant: 22 (guest) => 2222 (host) (adapter 1)
==> ol7-vagrant: Running 'pre-boot' VM customizations...
==> ol7-vagrant: Booting VM...
==> ol7-vagrant: Waiting for machine to boot. This may take a few minutes...
ol7-vagrant: SSH address: <IP_ADDRESS>:2222
ol7-vagrant: SSH username: vagrant
ol7-vagrant: SSH auth method: private key
ol7-vagrant:
ol7-vagrant: Vagrant insecure key detected. Vagrant will automatically replace
ol7-vagrant: this with a newly generated keypair for better security.
ol7-vagrant:
ol7-vagrant: Inserting generated public key within guest...
ol7-vagrant: Removing insecure key from the guest if it's present...
ol7-vagrant: Key inserted! Disconnecting and reconnecting using new SSH key...
==> ol7-vagrant: Machine booted and ready!
==> ol7-vagrant: Configuring proxy environment variables...
==> ol7-vagrant: Configuring proxy for Yum...
[ol7-vagrant] GuestAdditions versions on your host (6.1.4) and guest (6.0.12) do not match.
...
Copy iso file D:\vbox\VBoxGuestAdditions.iso into the box /tmp/VBoxGuestAdditions.iso
Mounting Virtualbox Guest Additions ISO to: /mnt
mount: /dev/loop0 is write-protected, mounting read-only
Installing Virtualbox Guest Additions 6.1.4 - guest version is 6.0.12
Verifying archive integrity... All good.
Uncompressing VirtualBox 6.1.4 Guest Additions for Linux........
VirtualBox Guest Additions installer
Removing installed version 6.0.12 of VirtualBox Guest Additions...
Copying additional installer modules ...
Installing additional modules ...
VirtualBox Guest Additions: Starting.
VirtualBox Guest Additions: Building the VirtualBox Guest Additions kernel
modules.  This may take a while.
VirtualBox Guest Additions: To build modules for other installed kernels, run
VirtualBox Guest Additions:   /sbin/rcvboxadd quicksetup
VirtualBox Guest Additions: or
VirtualBox Guest Additions:   /sbin/rcvboxadd quicksetup all
VirtualBox Guest Additions: Building the modules for kernel
4.14.35-19<IP_ADDRESS>.el7uek.x86_64.
VirtualBox Guest Additions: Running kernel modules will not be replaced until
the system is restarted
An error occurred during installation of VirtualBox Guest Additions 6.1.4. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.
Redirecting to /bin/systemctl start vboxadd.service
Redirecting to /bin/systemctl start vboxadd-service.service
Unmounting Virtualbox Guest Additions ISO from: /mnt
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.12
VBoxService inside the vm claims: 6.1.4
Going on, assuming VBoxService is correct...
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.12
VBoxService inside the vm claims: 6.1.4
Going on, assuming VBoxService is correct...
==> ol7-vagrant: Checking for guest additions in VM...
ol7-vagrant: The guest additions on this VM do not match the installed version of
ol7-vagrant: VirtualBox! In most cases this is fine, but in rare cases it can
ol7-vagrant: prevent things such as shared folders from working properly. If you see
ol7-vagrant: shared folder errors, please make sure the guest additions within the
ol7-vagrant: virtual machine match the version of VirtualBox you have installed on
ol7-vagrant: your host and reload your VM.
ol7-vagrant:
ol7-vagrant: Guest Additions Version: 6.0.12
ol7-vagrant: VirtualBox Version: 6.1
==> ol7-vagrant: Setting hostname...
==> ol7-vagrant: Mounting shared folders...
ol7-vagrant: /vagrant => V:/7
Vagrant was unable to mount VirtualBox shared folders. This is usually
because the filesystem "vboxsf" is not available. This filesystem is
made available via the VirtualBox Guest Additions and kernel module.
Please verify that these guest additions are properly installed in the
guest. This is not a bug in Vagrant and is usually caused by a faulty
Vagrant box. For context, the command attempted was:
mount -t vboxsf -o uid=1000,gid=1000 vagrant /vagrant
The error output from the command was:
/sbin/mount.vboxsf: mounting failed with the error: No such device
V:\7>
Microsoft Windows [Version 10.0.18362.657]
(c) 2019 Microsoft Corporation. All rights reserved.
c:>v:
V:>cd 7
V:\7>vagrant up
getting Proxy Configuration from Host...
Bringing machine 'ol7-vagrant' up with 'virtualbox' provider...
==> ol7-vagrant: Importing base box 'ol7-latest'...
==> ol7-vagrant: Matching MAC address for NAT networking...
==> ol7-vagrant: Setting the name of the VM: ol7-vagrant
==> ol7-vagrant: Clearing any previously set network interfaces...
==> ol7-vagrant: Preparing network interfaces based on configuration...
ol7-vagrant: Adapter 1: nat
==> ol7-vagrant: Forwarding ports...
ol7-vagrant: 22 (guest) => 2222 (host) (adapter 1)
==> ol7-vagrant: Running 'pre-boot' VM customizations...
==> ol7-vagrant: Booting VM...
==> ol7-vagrant: Waiting for machine to boot. This may take a few minutes...
ol7-vagrant: SSH address: <IP_ADDRESS>:2222
ol7-vagrant: SSH username: vagrant
ol7-vagrant: SSH auth method: private key
ol7-vagrant:
ol7-vagrant: Vagrant insecure key detected. Vagrant will automatically replace
ol7-vagrant: this with a newly generated keypair for better security.
ol7-vagrant:
ol7-vagrant: Inserting generated public key within guest...
ol7-vagrant: Removing insecure key from the guest if it's present...
ol7-vagrant: Key inserted! Disconnecting and reconnecting using new SSH key...
==> ol7-vagrant: Machine booted and ready!
==> ol7-vagrant: Configuring proxy environment variables...
==> ol7-vagrant: Configuring proxy for Yum...
[ol7-vagrant] GuestAdditions versions on your host (6.1.4) and guest (6.0.12) do not match.
...
Copy iso file D:\vbox\VBoxGuestAdditions.iso into the box /tmp/VBoxGuestAdditions.iso
Mounting Virtualbox Guest Additions ISO to: /mnt
mount: /dev/loop0 is write-protected, mounting read-only
Installing Virtualbox Guest Additions 6.1.4 - guest version is 6.0.12
Verifying archive integrity... All good.
Uncompressing VirtualBox 6.1.4 Guest Additions for Linux........
VirtualBox Guest Additions installer
Removing installed version 6.0.12 of VirtualBox Guest Additions...
Copying additional installer modules ...
Installing additional modules ...
VirtualBox Guest Additions: Starting.
VirtualBox Guest Additions: Building the VirtualBox Guest Additions kernel
modules.  This may take a while.
VirtualBox Guest Additions: To build modules for other installed kernels, run
VirtualBox Guest Additions:   /sbin/rcvboxadd quicksetup
VirtualBox Guest Additions: or
VirtualBox Guest Additions:   /sbin/rcvboxadd quicksetup all
VirtualBox Guest Additions: Building the modules for kernel
4.14.35-19<IP_ADDRESS>.el7uek.x86_64.
VirtualBox Guest Additions: Running kernel modules will not be replaced until
the system is restarted
An error occurred during installation of VirtualBox Guest Additions 6.1.4. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.
Redirecting to /bin/systemctl start vboxadd.service
Redirecting to /bin/systemctl start vboxadd-service.service
Unmounting Virtualbox Guest Additions ISO from: /mnt
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.12
VBoxService inside the vm claims: 6.1.4
Going on, assuming VBoxService is correct...
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.12
VBoxService inside the vm claims: 6.1.4
Going on, assuming VBoxService is correct...
==> ol7-vagrant: Checking for guest additions in VM...
ol7-vagrant: The guest additions on this VM do not match the installed version of
ol7-vagrant: VirtualBox! In most cases this is fine, but in rare cases it can
ol7-vagrant: prevent things such as shared folders from working properly. If you see
ol7-vagrant: shared folder errors, please make sure the guest additions within the
ol7-vagrant: virtual machine match the version of VirtualBox you have installed on
ol7-vagrant: your host and reload your VM.
ol7-vagrant:
ol7-vagrant: Guest Additions Version: 6.0.12
ol7-vagrant: VirtualBox Version: 6.1
==> ol7-vagrant: Setting hostname...
==> ol7-vagrant: Mounting shared folders...
ol7-vagrant: /vagrant => V:/7
Vagrant was unable to mount VirtualBox shared folders. This is usually
because the filesystem "vboxsf" is not available. This filesystem is
made available via the VirtualBox Guest Additions and kernel module.
Please verify that these guest additions are properly installed in the
guest. This is not a bug in Vagrant and is usually caused by a faulty
Vagrant box. For context, the command attempted was:
mount -t vboxsf -o uid=1000,gid=1000 vagrant /vagrant
The error output from the command was:
/sbin/mount.vboxsf: mounting failed with the error: No such device
V:\7>

Can you please also update your ol7 box?
We have recently released new boxes that should fix this issue.
vagrant box remove ol7-latest
vagrant up

I removed the vagrant box as you suggested and when I ran vagrant up I got the message that the guest editions were mismatched but that they had been updated so I ran

Vagrant halt

Vagrant up

This time thte shared folders were mounted successfully and the provisioning is proceeding.

Thank you!

--
Thanks, Donald
Quote of The Week: “Everyone is a genius. But if you judge a fish by its ability to climb a tree, it will live its whole life believing that it is stupid” –Albert Einstein

“As a leader, to be successful, is to help the people around you to be successful.” – Kent Boucher

Accessibility, like security, is better when built-in from the beginning rather than bolted on at the end.
http://www.oracle.com/
Donald Raikes | Accessibility Specialist
Mobile: HYPERLINK "tel:+15202717608"+15202717608 | VOIP: HYPERLINK "tel:+15205744033"+15205744033
Oracle Accessibility Program Office
| Tucson, Arizona
http://www.oracle.com/commitment
Oracle is committed to developing practices and products that help protect the environment

From: Philippe Vanhaesendonck<EMAIL_ADDRESS>Sent: Tuesday, March 3, 2020 1:31 PM
To: oracle/vagrant-boxes<EMAIL_ADDRESS>Cc: donraikes<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [oracle/vagrant-boxes] vagrant up failing with mismatched guest editions (#210)

Can you please also update your ol7 box?
We have recently released new boxes that should fix this issue.
vagrant box remove ol7-latest
vagrant up
—
You are receiving this because you authored the thread.
Reply to this email directly, HYPERLINK "https://urldefense.com/v3/https:/github.com/oracle/vagrant-boxes/issues/210?email_source=notifications&email_token=AJQS6NZ5YTUUYWAHJZHCIJLRFVSKBA5CNFSM4LASEGC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENVBAJI*issuecomment-594153509;Iw!!GqivPVa7Brio!LjgOExtoTsG0cZTwhYESRXw4Acy94TBX4WWrcpd6B262K4PX8s04vHTCNgh89un_$"view it on GitHub, or HYPERLINK "https://urldefense.com/v3/https:/github.com/notifications/unsubscribe-auth/AJQS6N4HKSHLZULYYXQT72LRFVSKBANCNFSM4LASEGCQ;!!GqivPVa7Brio!LjgOExtoTsG0cZTwhYESRXw4Acy94TBX4WWrcpd6B262K4PX8s04vHTCNktQZhFR$"unsubscribe. https://github.com/notifications/beacon/AJQS6NZECIMDPDO6NT5CTK3RFVSKBA5CNFSM4LASEGC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENVBAJI.gif


[vault-secrets-webhook] Create ClusterRole and ClusterRoleBindings when .Values.serviceAccount.create is false

Hi everyone,
I'm working on instrument the kubernetes auth method on Vault using an existing ServiceAccount, using its default Secret. Due so, I've set .Values.serviceAccount.create to false, which incurs on not creating the following manifests:

ClusterRole with name: {{ template "vault-secrets-webhook.fullname" . }}
ClusterRoleBinding with name: {{ template "vault-secrets-webhook.fullname" . }}-limited
ClusterRoleBinding with name: {{ template "vault-secrets-webhook.fullname" . }}-auth-delegator

Is there any reason to not be able to create those resources when an existing ServiceAccount is provided? I'm happy to open a pull request if this feature makes sense!

ATM upon deploying:

SA creation is optional.
ClusterRole is always created.
CluserRoleBinding limited is always created.
CluserRoleBinding auth-delegator is optional.

ref: https://github.com/bank-vaults/vault-secrets-webhook/blob/main/deploy/charts/vault-secrets-webhook/templates/webhook-rbac.yaml


TensorFlow verbose logging is too verbose even at level 1

Hello,
This is more of a feature request than an issue. Currently there is no convention on verbose logging of TensorFlow with VLOG() levels. This is making it pretty useless for development purposes. For example at  TF_CPP_MIN_VLOG_LEVEL=1, tensorflow/core/framework/log_memory.cc is printing all allocations and de-allocations which are spamming the log with hundreds of messages if not thousands per second. on the other hand tensorflow/core/common_runtime/graph_execution_state.cc is dumping nodedefs to the log file which sometimes contain the input data, creating hundreds of megabytes of strings as outputs . Similarly extremely verbose output is printed even at the verbosity level 1 making levels useless and logs unusable other than a few tiny example cases probably.
It would be nice to establish some conventions for logging levels on number of messages per second or amount of data printed to the log. Something in the lines of
0 < level 1 <= 0.1 msg/s
0.1 msg/s <= level 2 <= ~1-2 msg/s
1-2msg/s <= level 3 <= ~5-10 msg/s
.... and similarly for amount of information printed in the logs.
But the best solution would be to have a more advanced logging structure that would allow users to selectively enable or disable different logging levels for each module but I don't know how hard would it be to implement.
I am creating this issue to bring this feature/improvement request into attention.
Thanks,
Sami

@asimshankar do you want to assign this to someone or leave as community support?

I believe glog does support vmodule.
That said, I don't think we'll be working on switching the logging implementation/adding a vmodule feature in the near future, so marking this as "Contributions Welcome".
We'll be happy to see pull requests bumping up the log level of existing lines.
Deferring to @martinwicke about whether switching to glog is an option.

Forgive my ignorance -- what's the relationship between glog and absl logging? Ideally we wouldn't have both.
Independently, making TF more conservative about logs is a fine thing to do. I agree high volume logging levels are currently useless.

@martinwicke : Haven't looked in detail enough to be sure. Let's chat with the absl/glog folks off this thread.

vmodule is implemented now


Synapse Pipelines Web Activity cannot reach sites when using Managed Virtual Network with data exfiltration protection?

It appears as though the Web activity is unable to contact the management endpoint when the Synapse workspace is created using a Managed Virtual Network and data exfiltration protection. The first Web activity fails with an "unknown error".
This problem does not occur if i create the pipeline for a Synapse workspace that does not use a Managed Virtual Network with data exfiltration protection
Is there a workaround? The documentation should perhaps be updated to mention the restriction if it is not possible using a Managed Virtual Network.

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

ID: 239e657e-15d8-05a3-466d-4159b178307e
Version Independent ID: 866f6f93-0302-324b-668e-b41f67581d6d
Content: How to pause and resume dedicated SQL pools with Synapse Pipelines - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql/how-to-pause-resume-pipelines.md
Service: synapse-analytics
Sub-service: sql
GitHub Login: @kromerm
Microsoft Alias: makromer

Asked on StackOverflow

There is a section of "known limitations" under "Create a workspace with data exfiltration protection enabled" that is perhaps relevant. This says:

In data exfiltration protected workspaces, connections to outbound repositories are blocked. As a result, Python library installed from public repositories like PyPI are not supported.

I suppose this applies to pipelines too. If this is the case then I suggest adding a "known limitations" section to this "How to" page as wel.

Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

I have a similar Github-Issue about the same doc (https://learn.microsoft.com/en-us/azure/synapse-analytics/sql/how-to-pause-resume-pipelines) and the same scenario/environment (pipeline execution in Synapse Workspace with Managed VNet and data exfiltration protection ENABLED)
You can find the issue here:  https://github.com/MicrosoftDocs/azure-docs/issues/99174

@lucabovo my personal conclusion was that this was the root cause of the problem for us

I noticed that it is possible to trigger REST APIs using a web hook in Synapse Workspace with Managed VNet and data exfiltration protection ENABLED. I assume this because in a web hook no IR has to be specified.
Is this desired behavior? I mean, I can now just use a web hook to exfiltrate data...

When and how can this be addressed? Facing a similar issue .  WebHook also does not work .

@Cedz Use an integration runtime that is not contained within the managed vnet (e.g. a self-hosted IR).
The issue i raised here is regarding the documentation, not the functionality itself.

@gdubya and @Cedz  How can you leverage the self-hosted integration runtime (SHIR) when you are working with webhooks? I could not find a way to specify that. I created a SHIR on a private VNet and I validated I could connect to the webhook from that machine. The Synapse Workspace shows the SHIR status as running and the SHIR can communicate to the Webhook, but the pipeline calling the webhook fails as documented in this conversation.
Am I missing something? Is it really possible to access webhooks from Synapse with exfiltration turned on?
Thanks!

See my GitHub what I did to solve this: https://github.com/rebremer/securely-connect-synapse-to-azure-functions

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


PowerToys Run | Folder Plugin: ~ for home directory doesn't handle relocated folders

Microsoft PowerToys version
0.77.0
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
PowerToys Run
Steps to reproduce
Relocate Desktop to another drive/location.  Use ~\Desktop in PowerToys Run with Folder plugin enabled.
✔ Expected Behavior
I expect ~\Desktop to work
❌ Actual Behavior
It doesn't; wants to do a web search
Other Software
No response

Hi @Hossy
This is the expected behavior that can be reproduced in Windows Explorer/PowerShell.
PowerShell interpret ~ as %USERPROFILE%.
Are you able to access these folders from Explorer/PowerShell?
/needinfo

Are you able to access these folders from Explorer/PowerShell? /needinfo

I can in Explorer, but not in PowerShell.  Trying using C:\Users<user>\Desktop and Desktop was moved to D:\Users<user>\Desktop.  When doing it in Explorer, I enter C:\Users<user>\Desktop and Explorer changes it to D:\Users<user>\Desktop.

Woow! Didn't know this is worked in Explorer.
This is something that has always bothered me while working in PowerShell.


Update contact details for pjbgf

@pjbgf can you please force push, CI is stuck...

Closing in favour of https://github.com/fluxcd/source-controller/pull/1012.


[CLI]: wand.save() does not accept pathlib.Path

Describe the bug
This is contrary to how most python libraries work. For instance, it is perfectly fine to pass a pathlib.Path to

open()
torch.save()
numpy.save()
xarray.to_netcdf()
To give just a few examples. Explicitly requiring a string is unexpected and counter-intuitive.

Additional Files
No response
Environment
WandB version:
OS:
Python version:
Versions of relevant libraries:
Additional Context
No response

Hi @sehoffmann, thanks for reporting this! I'll create a new feature request to allow passing a pathlib.Path to wandb.save(), thanks for the feedback!

Hey @luisbergua,
I would like to work on this, can I pick this up ?

Hey @techytushar, great to hear it! Here you have all the steps to contribute, thanks a lot!


Fix erros when cancelling helsinki authentication

🛠 Changelog

Fixes errors when returning from Helsinki authentication by setting social auth middleware and appropriate error return URL.

🗒 Other notes
Additional notes if needed
🚚 Deployment reminder

[ ] PR requires deployment changes
[ ] Pipeline configuration change PR [link here]
[ ] Azure devops library updated if needed
[ ] Pipeline configuration change PR merged

🎫 Tickets
This pull request resolves all or part of the following ticket(s):

...

Note that the error return URL is by default to the backend admin panel. This should be configured with the env variable. There might be a way to set this from frontend via some query parameter, but didn't look into it yet. Still, this can be used as a quick fix.


Add CI pipeline notification for Travis

To prevent issues like #3 happen again

Don't know how you registratered yourself at travis but I also receive email notifications from travis for failed builds. This might be an intermediate solution until you have your pipeline in place.

Hi @discordier, I guess I didn't receive the email due to some organization setting on Github. Working with my admin on that now. Thanks for the suggestion!

Travis CI and Github notification has been configured


Can i use this for landscape only app?

"Take Photo" is fine but "Choose from Library" seems to require portrait mode. I wonder if there is a way to make it work with landscape mode only. Thanks.

This is a limitation of iOS. See #35.
Maybe react-native-image-crop-picker would better suite your needs?

Thank you for recommendation, i'm using react-native-image-crop-picker now, it's great.


Feature/heritage 231

Ticket URL: https://national-archives.atlassian.net/browse/HERITAGE-231
About these changes
Timeline / Histogram views being added in
How to check these changes
Pull branch and run locally, check the timeline views
http://<IP_ADDRESS>:8000/search/catalogue/?group=community&vis_view=timeline&timeline_type=century
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes
[x] Ensured that PR includes only commits relevant to the ticket
[x] Waited for all CI jobs to pass before requesting a review
[x] Added/updated tests and documentation where relevant

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

Got some errors on page-inspect using Chrome.

http://<IP_ADDRESS>:8000/search/catalogue/?group=community&vis_view=timeline&timeline_type=century

Invalid scale configuration for scale: xAxes chart.js@3.9.1:13

This was due to the version of chartJS we are using, I've adapted the code so the error does not occur.

http://<IP_ADDRESS>:8000/search/catalogue/?vis_view=timeline&timeline_type=decade&creation_date_from=1800#decade-counts-chart

Refused to apply style from 'http://<IP_ADDRESS>:8000/search/catalogue/css/styles.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

http://<IP_ADDRESS>:8000/search/catalogue/?vis_view=timeline&timeline_type=year&creation_date_from=1860#decade-counts-chart

Refused to apply style from 'http://<IP_ADDRESS>:8000/search/catalogue/css/styles.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

I've removed these style rules as the css is now handled by the sass stack


Speed up from-list

It looks like it might be possible to speed up the from-list function:
MAGICL> (let* ((n 50000)
                     (xs (loop :for i :below n :collect (coerce i 'double-float))))
                (sb-ext:gc :full t)
                (time (magicl:from-list xs (list n) :type 'double-float))
                (sb-ext:gc :full t)
                (time (magicl:from-array (coerce xs '(array double-float (*))) (list n)))
                (values))
Evaluation took:
  1.544 seconds of real time
  1.544000 seconds of total run time (1.544000 user, 0.000000 system)
  100.00% CPU
  4,188,179,784 processor cycles
  3,578,512 bytes consed

Evaluation took:
  0.000 seconds of real time
  0.004000 seconds of total run time (0.004000 user, 0.000000 system)
  100.00% CPU
  1,477,558 processor cycles
  400,016 bytes consed

Easy solution: delete from-list. 😆

It would make sense to replace both by a from-sequence that appropriately coerces its argument.
However, I wonder if the slowness of from-list is a symptom of some other issue that could be improved and have positive effects in other parts of the code.

Maybe related: https://github.com/rigetti/magicl/pull/63#discussion_r341383765
Also maybe this comment at the top of from-array
https://github.com/rigetti/magicl/pull/63/files#diff-4bbae3945a97f437320d2829a528e786R97

You were spot on, @appleby. The culprit is the call to nth within into!.

I can definitely fix the implementation of from-list as a band-aid for now but I think @jmbr is right about possibly shifting to a from-sequence constructor instead. Maybe from-array and from-list can be kept as aliases to from-sequence...

What's the benchmark now @colescott ? Should this be closed?

What's the benchmark now @colescott ? Should this be closed?


Connect lidar to esp32

Hi Being24
I get this when run code.
how to right connect to arduino board, thanks
get data
checksum error
start scan
start scan checksum error
scan times: 0 distance: 0 intensity: 0 disturb: 0
scan times: 1 distance: 0 intensity: 0 disturb: 0

I've encountered the same error myself.
I have not thoroughly verified the cause, but sometimes restarting SDM15 fixes it.
Can you please edit the library and show us the contents of the recv so I can try to determine the cause?


Setup fails to run on pybuilder 0.12.2 or later

Our company has a pybuilder package for an internal utility library that we use for our python development. Since pybuilder 0.12.2 was released, this library fails to install using pip. The following errors occur:
(venv) $ pip install<EMAIL_ADDRESS>Collecting<EMAIL_ADDRESS>  Cloning<EMAIL_ADDRESS>to /private/var/folders/7_/pvz_xscd2p72nknxpw8wvw400000gn/T/pip-hdwola22-build
    Complete output from command python setup.py egg_info:
    pyb 0.12.2
    PyBuilder version 0.12.2
    Build started at 2020-04-23 16:51:39
    ------------------------------------------------------------
    [INFO]  Installing or updating plugin "pypi:pybuilder_pytest, module name 'pybuilder_pytest'"
    [INFO]  Processing plugin packages 'pybuilder_pytest' to be installed with {}
    ------------------------------------------------------------
    BUILD FAILED - Missing plugin plugin 'pypi:pybuilder_pytest', module 'pybuilder_pytest': Traceback (most recent call last):
      File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pybuilder/pluginloader.py", line 175, in _load_plugin
        __import__(plugin_module_name)
      File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/pybuilder/reactor.py", line 532, in __import_with_plugins
        return self._old_import(*args, **kwargs)
      File "/private/var/folders/7_/pvz_xscd2p72nknxpw8wvw400000gn/T/pip-hdwola22-build/.pybuilder/plugins/cpython-3.6.1.final.0/lib/python3.6/site-packages/pybuilder_pytest/__init__.py", line 22, in <module>
        from pybuilder.plugins.python.unittest_plugin \
    ImportError: cannot import name '_register_test_and_source_path_and_return_test_dir'
     (site-packages/pybuilder/pluginloader.py:181)
    ------------------------------------------------------------
    Build finished at 2020-04-23 16:51:47
    Build took 8 seconds (8416 ms)

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/7_/pvz_xscd2p72nknxpw8wvw400000gn/T/pip-hdwola22-build/

This install was run in a virtual environment. If I use pybuilder 0.12.1, the pip install completes without error.
Is there something that I need to change in build.py or somewhere else in the packaging code for this library so it will work with the latest version of pybuilder?
Thanks in advance for your help.

This is a duplicate of #704 : _register_test_and_source_path_and_return_test_dir is an internal API that was (incorrectly) used by one of the plugins you are using: pybuilder_pytest:
File "/private/var/folders/7_/pvz_xscd2p72nknxpw8wvw400000gn/T/pip-hdwola22-build/.pybuilder/plugins/cpython-3.6.1.final.0/lib/python3.6/site-packages/pybuilder_pytest/__init__.py", line 22, in <module>

Since @AlexeySanko no longer maintains this plugin I'll have to return this function back just to make sure it's not breaking stuff for so many people.

Umm... no I can't do this - plugin loads code into PyBuilder core. I'll readd this function TEMPORARILY to make sure you guys can use this, but it'll be obsoleted in future versions once PyTest plugin is supported.

fixed in 0.12.4

Thank you @arcivanov. How long will this fix be active? I need to know how long I have to refactor our library to no longer use pybuilder_pytest.

Fix will be active until I build a replacement for pybuilder_pytest in the core. The removed method will not be used by any PyB code and as such will not have any side effects until PyTest plugin replacement is built.


Onboarding: "Better Than Free Transacations" Spelling Error

Is your feature request related to a problem? Please describe.
Currently it says:
Better Than Free Transacations
Transactions is spelled wrong. It should say:
"Better Than Free Transactions"

This was fixed in https://github.com/JoinSEEDS/seeds_light_wallet/pull/968


After adding minimum date scrolling is not getting selected on the very first time.

After adding minimum date on scrolling date it is not getting stop for the first time on another date it comes back to current date. Please help to solve the problem .
Please have a look on video it will help you to understand the issue

Hey @Akshaykapadi , thanks for reporting but we need all the information required in the issue template (with a reproducible minimum example).

Hi @mmazzarolo i will share the code and example


Visualize Attention Map

Dear authors,
Wonderful works! Wondering if you could kindly offer the code to visualize Figure 5 in your paper.

Could you point me to where I can find the code for generating Figure 5? Thanks!

Were you able to do it? I'm interested in it too

Yes, I ended up writing a script myself, but I didn't reproduce the exact same attention maps as the paper.


Make docs easier to read offline

It would be great if the Envoy docs were easier to read offline. Please consider supporting a function like Kubernetes Print entire section

hi @matt-declarativecloud if you can point to a compatible sphinx plugin that would enable this feature, we can  consider adding


esm

Do think this is really all that you need.
pure esm, no coffeescript, no complex dev setup like mocha, linting, git hooks yada yada, this pkg is so small anyway.
friendlier deno + browser import. (\w upcoming http loader in nodejs)

As I've mentioned in a few other issues, these are legacy packages you're commenting on. Many of them reflect the trends and the tools used at the time that, of course, have since died off.


Adding of support for vouchers

@BlackSmith Thank you for the pull request!  Life is busy at the moment with school starting, however I will attempt to merge in the coming 7 days.

You are welcome. :)
Can I ask you, I want to use unifi vouchers on my external authentication portal. I found out that the Unifi controller API does not support guest authentication via voucher. I can get all vouchers and check it in my script, after that I can use authorize_guest and enable of guest.
Unfortunately in this case, the voucher is still unused and expiration time does not start counting. Voucher never expire. :(
Do you have any idea how to solve this problem?
Thank you in advance.

Could you script run the delete_voucher method after a period of time?

Further, the README contribution is great!  I definitely think a solid API library needs good documentation so thank you.  I do not use voucher functionality in my build, nor know much about it.  Therefore, I am going to have faith the functionality works as the code looks good.


Changes Required

We noticed the folowing issues during the peer to peer code review:

The hamburger menu was displayed on the desktop view
The mobile nav was not properly hidden and was causing an overflow.

The issues have been fixed


Test 1M EDMF (diag and prog) on GPU

This PR makes EDMFX (prognostic and diagnostic) work with 1M microphysics on the GPU. Thank you @charleskawczynski !
@szy21 - I wasn't sure if you would like to also keep the 0M edmf aquaplanet runs in the CI?

You mean you want to keep the 0M in the GPU CI? Not the 1M? I think it would be good to have 1M tests on the GPU. Maybe we can add both then?
For diagnostic edmf we had to flatten the detrainment function signature to pass less things in. For prognostic edmf it's a bug in ClimaCore. Once it's fixed I should be able to use the normal fill (i.e. @.  blah = FT(0) instead of @. blah = FT(0) * blah2)

You mean you want to keep the 0M in the GPU CI? Not the 1M? I think it would be good to have 1M tests on the GPU. Maybe we can add both then?
For diagnostic edmf we had to flatten the detrainment function signature to pass less things in. For prognostic edmf it's a bug in ClimaCore. Once it's fixed I should be able to use the normal fill (i.e. @. blah = FT(0) instead of @. blah = FT(0) * blah2)

Sorry that was a typo (I was replying while explaining edmf equations to others lol). I meant we don't need to keep the 0M one.


Not working on IE7, also demo on website

Hi,
Demos on website are broken width IE7.
http://owlcarousel.owlgraphic.com/demos/basic.html
:(

Closing for #476


split Birt-Hogg-Dube syndrome

close #6518
close https://github.com/monarch-initiative/mondo/issues/6546
(the two tickets were related so I addressed them on one branch)

@nicolevasilevsky Apologies, I just saw something to change


Don't expose lifecycleAwareLazy in viewModel return type

https://github.com/airbnb/mavericks/issues/597

Codecov Report

Merging #603 (4b1faf3) into main (f44e85b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #603   +/-   ##
=======================================
  Coverage   55.06%   55.06%
=======================================
  Files          53       53
  Lines        2508     2508
  Branches      326      326
=======================================
  Hits         1381     1381
  Misses        976      976
  Partials      151      151

Impacted Files
Coverage Δ

...main/kotlin/com/airbnb/mvrx/MavericksExtensions.kt
8.18% <0.00%> (ø)

Continue to review full report at Codecov.

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


Bookie-server picks invalid empty config of journalDirectories

Issue
We have config management tool which manages bookie-server configuration and sets empty value of journalDirectories if bookie-server is still wants to use single journal dir: journalDirectory. In this case, bookie-server doesn't consider journalDirectory value and keeps empty list of journalDirectories which stats bookie with invalid configuration.
Expected behavior
Bookie-server should parse empty configuration journalDirectories= properly and avoid picking up empty value of journalDirectories and in that case, bookie-server should fallback to journalDirectory config.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Ability to extend static Event interface

This is discussion question about extending builtin d.ts in projects.
In my particular case I would like to extend Event static intreface to add some static fields to Event(for special runtime that extends browser runtime).
I have found no way to do it. But for Array there is a way to use declaration merging because ArrayConstructor is separate named interface.
Here is example on playground
Tried it in https://github.com/Microsoft/TypeScript/pull/22946
@mhegazy thanks for answers in PR.
Question in general:
Find a way how to make any definition extendable without forking builtin d.ts files and changeing them manually.

The answer to the general question is to make them classes (https://github.com/Microsoft/TypeScript/issues/574). there are some wrinkles here about what that entails though, callable constructors, and un-instantiate-able classes..

Is it required to be on the global Event? can you declare a wrapper?

In my case there already redefined Event(and some another browser api classes) in global by outer js code. This event does everything that original event does and some more. I cant simply extend Event, I should replace global Event defenition to more complicated defenition (declaration merging resolves issue as I showed in PR). I dont know any way to declare additional static fields to something that declared as 'declare Class'. Of cource I can rename extended Event to another name, but code quality became awful. Mutch better is just add special defenitions to project and define extended functionality for already available types.


Feature request : add in code 'debugger;' to tsserver for .ts and .js files

Hello,
I don't think tsserver provides a way to list all the debugger; written in ts or js files.
This feature is related to vscode: https://github.com/Microsoft/vscode/issues/4492
I hope you will take my request into consideration and this could help to enhance tsserver.
Thank you in advance for your feedback.

This looks like a linter rule rather than a part of the core pipeline. tslint already has a no-debugger rule that does this.

@mhegazy I believe the intent here is that a host could ask tsserver what all the debugger; statements are so it could use that information to populate a breakpoint list, not to disallow them

ooh. i see. sorry did not read the linked item. my bad.

Linked issue closed out of scope


fix semicolon errors

Signed-off-by: Shatakshi<EMAIL_ADDRESS>Description
Fixed semicolon errors as shown by sonatype lift
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4401 (31ff377) into master (d943ac3) will decrease coverage by 7.70%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4401      +/-   ##
==========================================
- Coverage   31.85%   24.15%   -7.71%
==========================================
  Files          59       59
  Lines        5063     5063
==========================================
- Hits         1613     1223     -390
- Misses       2989     3501     +512
+ Partials      461      339     -122

Flag
Coverage Δ

gointegrationtests
?

unittests
24.15% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-62.83%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
0.99% <0.00%> (-53.98%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.50% <0.00%> (-47.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.85% <0.00%> (-36.76%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-20.88%)
:arrow_down:

Continue to review full report at Codecov.

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

Thank you, @Shatakshi0805 👍


(SERVER-915) Enable trapperkeeper-authorization for empty whitelists

This commit changes the 'puppet-admin' and 'certificate-status'
endpoints to use trapperkeeper-authorization instead of the client
whitelists for authorization when the whitelist is present in
configuration but empty.
This commit also has a couple of somewhat unrelated changes:

Adds an explicit ring-core 1.4.0 conflict resolution dependency to
the project.clj.  comidi 0.1.1 pulls in ring-core 1.3.2, which
conflicts with the ring-core version that
trapperkeeper-authorization pulls in, 1.4.0.

Allows the X509Certificate for an ssl-client-cert to be optional
in a Ring request map.  This is needed because Ring will populate
this key with 'nil' when no SSL client certificate was provided for
the request.  Found this after adding a test for calling
'puppet-admin' with 'authorization-required: false'.

:+1:

Thanks, @fpringvaldsen.  I updated the branch with the two s/authorized-required/authorization-required changes in the log warnings.  Since it was just these two small changes, I just squashed it into the original commit - now as 171d9b3.

:+1: LGTM


Remote Extension host terminated unexpectedly 3 times within the last 5 minutes.

Type: Bug
I have a Windows machine and I try to open a project in my WSL2 environment with VS Code. However, I get the error: "Remote Extension host terminated unexpectedly 3 times within the last 5 minutes." with the latest release of VS Code server installed automatically on the WSL2 environment: 74f6148eb9ea00507ec113ec51c489d6ffb4b771 . It cannot see any git repository in the current folder, even though the root folder is a GitHub repository, thus I cannot see the changes made to my files. Please fix this as soon as possible.
  I disabled all of my extensions and it's still failing.
Here is what is showing up in the debugger as the error:
ERR No registered selector for ID: ms-vscode.npm-command: Error: No registered selector for ID: ms-vscode.npm-command
at vscode-file://vscode-app/Applications/Visual%20Studio%20Code.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:1515:26878

Please diagnose the root cause of the issue by running the command F1 > Help: Troubleshoot Issue and following the instructions. Once you have done that, please update the issue with the results.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

I fixed this bug by remove the vscode-remote-containers and vscode-server in WSL2, that is
rm -fr ~/.vscode-remote-containers rm -fr ~/.vscode-server rm -fr ~/.vscode.
Then, reconnect to WSL2 in vscode and re-install the extensions you need.


python3Packages.aqualogic: 2.5 -> 2.6

Motivation for this change
Update to latest upstream release 2.6
Change log: https://github.com/swilson/aqualogic/releases/tag/2.6
The Home Assistant's tests don't pass (python-jose issue).
Things done

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

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

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

Result of nixpkgs-review pr 118433 at 8d0b81ca run on x86_64-linux 1

2 packages built successfully:

python38Packages.aqualogic
python39Packages.aqualogic

1 suggestion:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/development/python-modules/aqualogic/default.nix:21:5:
   |
21 |     (fetchpatch {
   |     ^


IMPLEMENT I2D_TO_HOST FUNCTION

In src/unknown_cuda.cu implement i2d_to_host function which is currently a placeholder.

This feature was already implemented.


Relationship rework of Version 3 broke support for data classes that use custom implementations of GetHashCode

When I upgraded Dapper.FastCrud to the latest Version from a previous 2.x Version, I encountered a new issue in the context of relations.
Consider the following simplified scenario:
I have a data type Customer that has a set of Orders and a set of Payments. As soon as a single Customer has more than one Order and at least one Payment, whenever I query for that Customer and also ask to include both the related Orders and Payments, the result of that query will have duplicate Payments and/or Orders (the exact amount of duplicates depends on the number of unique Payments and Orders).
I could trace back this issue to the fact that I use protobuf messages as data types. The c# code generator for protobuf messages automatically adds a custom implementation of GetHashCode() that will return a hash value that depends on all values that are part of the object, especially including the contents of the collections that are navigation properties to the child sets (in my case Orders and Payments).
This is a problem because the implementation of LocalEntityCollectionKey uses the result of _instance.GetHashCode() in order to generate its own lazy _hashCode - as far as I can tell, _instance is the real object instance and not an EntityInstanceWrapper, therefore the implementation of GetHashCode() of the real data class is used.
Now, whenever the TypedEntityContainer is called to GetOrAddToLocalCollection, a new LocalEntityCollectionKey will be generated for the single Customer and that key will never match any of previous ones because even though all previous ones were constructed using the same Customer object, that Customer object had fewer child Order/Payment attached to it at the time the hash was evaluated. Therefore _localEntityInstances.TryGetValue using that key will never find the already existing entry for that Customer and in consequence all repetitions of child Order/Payment objects are also added to the navigation collections.
I think this problem can be averted if LocalEntityCollectionKey is changed to use  System.Runtime.CompilerServices.RuntimeHelpers.GetHashCode(_instance) instead of this._instance.GetHashCode().
Maybe passing the EntityInstanceWrapper around the real data object to the LocalEntityCollectionKey in order to generate the hash value from the instance wrapper would be another solution?

Yikes, that's quite a mess. Don't think that avoiding the call to GetHashCode on the entity instance is the right way to fix this though. I can totally see us forgetting about this edge scenario in the future and breaking it again.
I'm inclined to recommend not using these entities for more than what they are. They should only live in the DAL and mapped to business entities and DTOs for data transmission.

Thanks for the acknowledgement.
I actually think that using the EntityInstanceWrapper instead of the Entity itself could solve that issue, especially since the EntityInstanceWrapper implements its own hashcode function that seems to work upon the property values actually read from database - that means the hashcode of the wrapper looks to be very suitable for questions concerning the identity of the wrapped entity. Also since that class is part of Dapper.Fastcrud, you also have complete control over the hash implementation.
To implement that change, the ctor of LocalEntityCollectionKey (as well as the method GetOrAddToLocalCollection of the class TypedEntityContainer) need to be altered to ask for an EntityInstanceWrapper instead of the object instance (or actualInstance).
However there is one case where an instance of MainEntityResultSetParser<TMainEntity> is passed to the GetOrAddToLocalCollection, which obviously is not an EntityInstanceWrapper and it is probably not feasible to just wrap it at that place. That means there has to be an overload or alternative method for this particular case. (Also note that the parser class does not override the hash function provided by the base object type - that implementation is safe to use in the given context)

Don't think anyone will take a look at this. As I said, overriding GetHashCode for DB entities is a really bad idea and can cause other issues, especially since these guys could only be partially filled with data at various points. For example, you insert an entity without providing a primary key, which is a great candidate for identifying the entity via Equals and GetHashCode, just to come back later with the identifier filled in by the DB. It's the same entity, however the custom GetHashCode could say otherwise.
I strongly recommend creating a proper set of DTOs for data transmissions and have a set of mappers between the DB entities and your DTOs. That way you won't have any kind of problems and the two layers can evolve independently.


config.sh: check for ncursesw6

Actually https://github.com/akrennmair/newsbeuter/pull/201 looks way better than this, so closing.


Upgrades Envoy Image to v1.16.1

https://github.com/projectcontour/contour/pull/3151 updated the envoy image to 1.16.1. This PR does the same for the operator.
Signed-off-by: Daneyon Hansen<EMAIL_ADDRESS>

@jpeach commit e3cd5f0 addresses your comments above, PTAL.


support streaming AGG and some fix in UDAF

What type of PR is this：

[x] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：
for #2742
Problem Summary(Required) ：

return error when exception happened in UDF
fix wrong result in UDAF group by
fix crash in scalar UDF when args has const column
support streaming aggregate when using UDAF

run starrocks_be_unittest

run starrocks_fe_unittest

run starrocks_fe_unittest


replacing-if-else-chains-with-switch ???

Replacing if/else chains with switch
https://www.freecodecamp.com/challenges/replacing-if-else-chains-with-switch
Issue Description
I have worked this several different times, removing my code, refreshing the browser, clearing the cache, wiping my code, etc.
Browser Information

Browser Name, Version: Chrome Version 58.0.3029.110 (64-bit)
Operating System: Mac OS 10.12.2
Mobile, Desktop, or Tablet: Desktop

With this code below I have alternately gotten:
(1) the first three issues stating that I should not have if-else statements in my code, but all the rest of the switch-test issues correct.
To fix this, I deleted the original if-else code that I'd obscured (//).  Still got these issues.
(2) Cleared my code, refreshed the browser, cleared the cache, re-set the exercise code:  The "successful" passed switch items remained, but the if-else complaints persisted.
(3) I again cleared the original code, refreshed the browser, re-set--hit the "remove my code" button on the exercise refresh, as before.  Then I re-pasted my code that had achieved success with the actual switch issues, while not clearing the if-else complaints(despite having no if-elses).
I then obscured once again the original if-else code of the exercise.  I hit "Test" and this time I got "Maximum call overload" or something to that effect.
(4) I again clear, clear the cache, restart, etc.  Input my "good" code and removed the original exercise code.  This time, again, I passed all switch challenges, but the exercise continued to claim that I still had "if-else" statements in my code, and did not have "at least 3 break statements".  As you will see, I had quite a few break statements.
I'm going to move on anyway, but I wonder what if anything is wrong with my code.  Or the browser or browser version I'm using or...what?
Code
function chainToSwitch(val) {
var answer = "";
switch(val) {
case chainToSwitch("bob"):
answer = "Marley";
break;
case chainToSwitch(42):
answer = "The Answer";
break;
case chainToSwitch(1):
answer = "There is no #1";
break;
case chainToSwitch(99):
answer = "Missed me by this much!";
break;
case chainToSwitch(7):
answer = "Ate Nine";
break;
case chainToSwitch("John"):
answer ="";
break;
case chainToSwitch(156):
answer = "";
}
/*  if (val === "bob") {
answer = "Marley";
} else if (val === 42) {
answer = "The Answer";
} else if (val === 1) {
answer = "There is no #1";
} else if (val === 99) {
answer = "Missed me by this much!";
} else if (val === 7) {
answer = "Ate Nine";
}*/
return answer;
}
// Change this value to test
chainToSwitch(7);

The Exact error message is:  "Maximum call stack size exceeded" - But in the meantime, I get all the switch challenges noted as "correct" and the three complaints about how I should not have any If-else statements in the editor, and that I should have "at least 3 break statements" are not satisfied.
To address the presence of the "If-else" statements in the editor, (even though they existed in the original exercise code, I had obscured all of the original exercise code in the editor with */ - */), I tried again, deleting all the obscured original exercise code, leaving only what I thought would now be recognized as 100% correct.  But after refreshing, etc. I still got the same issues:
(1) the maximum call stack error message
(2) all passed but the first 3--still not reading the fact that I have more than 3 break statements in the code and that all the if-else statements are gone.
What am I missing?

hello bro!!!
this is my code....just now i have solved!!

You can post a screenshot if the problem still persists.
Cheers!!

@bneng Thank you for reporting this issue. You're creating infinite loops because your function never resolves itself. Please take a look at the example of how to implement switch statements. You don't need to call chainToSwitch within the switch statement. If you need a hint, you can take a look at the code someone has kindly shared above. In the future, please head to the Gitter chat rooms for coding help.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!


[Feature Request] 1.7.10?

Been getting into a really cool 1.7.10 mod with some friends, and I love to use this mod with it.
Could this happen?

Nop soory :(
1.7.10 is way to different from modern version to update it without too much works for the potential users.
Denis


[gpu] do not require signing metadata

wrapped signing code in conditional that checks for private key
replaced tabs with spaces
made use of get_metadata_attribute more frequently
using default value of "" unless otherwise supplied
using CUDA subversion of 12.4.1
building source in /opt/install-nvidia-driver

/gcbrun

/gcbrun


How to get URI of photo?

After
photoResult.saveToFile(someFile)

how can I retrieve the URI of the saved image?

The File java class has your answer:
File file = new File(
        getExternalFilesDir("photos"),
        "photo.jpg"
);
photoResult.saveToFile(file);
file.toURI(); <-- here you go

Also see: https://github.com/Fotoapparat/Fotoapparat/blob/master/sample/src/main/java/io/fotoapparat/sample/MainActivity.java#L87


Should 'fork' button say 'clone' instead?

I've just found this plugin, and when initially setup I wasn't entirely sure what the 'fork' button would do. It's clear that it clones the repository and optionally updates the repository.
This behaviour, while correct for the plugin, does not follow GitHub wording, where fork creates a new repository based on another.

Totally agree, it's so confusing!

I'm confused by the Update snippets options, is it pull or push, or both?


Issue with k8s.io/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm/

This is a Bug Report

When I try to login with exec/logs through kubectl then bellow error in terms of pod or container...
Error from server: error dialing backend: dial tcp XX.XX.XX.XX:10250: connect: connection refused

Problem:
Error from server: error dialing backend: dial tcp XX.XX.XX.XX:10250: connect: connection refused
Proposed Solution:
Page to Update:
https://kubernetes.io/...

clientVersion:
buildDate: "2020-02-11T18:12:12Z"
compiler: gc
gitCommit: 06ad960bfd03b39c8310aaf92d1e7c12ce618213
gitTreeState: clean
gitVersion: v1.17.3
goVersion: go1.13.6
major: "1"
minor: "17"
platform: linux/amd64
serverVersion:
buildDate: "2020-02-11T18:07:13Z"
compiler: gc
gitCommit: 06ad960bfd03b39c8310aaf92d1e7c12ce618213
gitTreeState: clean
gitVersion: v1.17.3
goVersion: go1.13.6
major: "1"
minor: "17"
platform: linux/amd64
And Docker version
Docker version 19.03.8, build afacb8b

I suffered because again and again modify in yaml and verify through describe command So please resolve it ASAP....

please use the support channels for questions:
https://github.com/kubernetes/kubernetes/blob/master/SUPPORT.md
thanks
/close
/triage support


Enhancement Request - Bulk URL Processing (Burp Piper)

Hi @xnl-h4ck3r,
I have a suggestion to make knoxnl Burp Piper even more user-friendly and efficient.
Currently, knoxnl Burp Piper allows us to send individual HTTP requests for analysis. However, this process can be slow and tedious when dealing with many URLs.
Could you consider adding a feature that lets us send multiple URLs at once for scanning? Here's how it could work:

Allow users to input multiple URLs or hostnames together, instead of one by one.
Send the host from sitemap into piper --> knoxnl

Adding this feature would save a lot of time and make knoxnl even more valuable for finding xss.

Hi @0xPugazh. Thanks for the suggestion. I will add it to the TODO list, but not sure when I'll get to that yet


AvoidReservedCharInCmdlet throws System.NullReferenceException sometimes due to CommandInfo.ResolveParameter issue

I report this error although I cannot reproduce the bug, moreover the analyzed code is private :-/
The exception is random during a build via PSake. I already had this error with the previous version 1.8.
The exception is most often raised when I open a powershell session and then execute my build script.
When, after the triggering of the error, I relaunch my error script, I no longer have an error
I have not yet had time to run only the PSSA command line.
Actual behavior
[STA] E:\Projets\xxxx> $error[0]|select *

PSMessageDetails      :
Exception             : System.NullReferenceException: Object reference not set to an instance of an object.
                           at System.Management.Automation.CommandInfo.ResolveParameter(String name)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.Helper.GetExportedFunction(Ast ast)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.AvoidReservedCharInCmdlet.<AnalyzeScript>d__0.MoveNext()
                           at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
                           at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.ScriptAnalyzer.<>c__DisplayClass83_1.<Analyze
                        SyntaxTree>b__2()
TargetObject          : E:\RELEASES\xxx\yyyy\Modules\Log4Posh\Log4Posh.psm1
CategoryInfo          : InvalidOperation: (E:\RELEASES\xxx…\Log4Posh.psm1:String) [Invoke-ScriptAnalyzer],
                        NullReferenceException
FullyQualifiedErrorId : RULE_ERROR,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at <ScriptBlock>, E:\Projets\xxx\build.psake.ps1: line 154
                        at Invoke-Task, E:\...\Modules\psake\4.9.0\public\Invoke-Task.ps1: line 108
                        at Invoke-Task, E:\...\Modules\psake\4.9.0\public\Invoke-Task.ps1: line 84
                        at <ScriptBlock>, E:\...\Modules\psake\4.9.0\public\Invoke-psake.ps1: line 303
                        at ExecuteInBuildFileScope,
                        E:\...\Modules\psake\4.9.0\private\ExecuteInBuildFileScope.ps1: line 57
                        at Invoke-psake, E:\...\Modules\psake\4.9.0\public\Invoke-psake.ps1: line 258
                        at <ScriptBlock>, E:\Projets\xxx\build.ps1: line 14
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}

--OR

PSMessageDetails      :
Exception             : System.NullReferenceException: Object reference not set to an instance of an object.
                           at System.Management.Automation.CommandInfo.ResolveParameter(String name)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.Helper.GetExportedFunction(Ast ast)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.AvoidReservedCharInCmdlet.<AnalyzeScript>d__0.MoveNext()
                           at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
                           at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.ScriptAnalyzer.<>c__DisplayClass83_1.<Analyze
                        SyntaxTree>b__2()
TargetObject          : E:\RELEASES\XXX\Modules\YYY\YYY.psm1
CategoryInfo          : InvalidOperation: (E:\RELEASES\xxx...yyyy.psm1:String) [Invoke-ScriptAnalyzer],
                        NullReferenceException
FullyQualifiedErrorId : RULE_ERROR,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeScriptAnalyzerCommand
ErrorDetails          :
InvocationInfo        : System.Management.Automation.InvocationInfo
ScriptStackTrace      : at <ScriptBlock>, E:\Projets\xxx\build.psake.ps1: line 154
                        at Invoke-Task, E:\SYSTOOLS\XXX\Modules\psake\4.9.0\public\Invoke-Task.ps1: line 108
                        at Invoke-Task, E:\SYSTOOLS\XXX\Modules\psake\4.9.0\public\Invoke-Task.ps1: line 84
                        at <ScriptBlock>, E:\SYSTOOLS\XXX\Modules\psake\4.9.0\public\Invoke-psake.ps1: line 303
                        at ExecuteInBuildFileScope,
                        E:\SYSTOOLS\XXX\Modules\psake\4.9.0\private\ExecuteInBuildFileScope.ps1: line 57
                        at Invoke-psake, E:\SYSTOOLS\XXX\Modules\psake\4.9.0\public\Invoke-psake.ps1: line 258
                        at <ScriptBlock>, E:\Projets\xxxs\build.ps1: line 14
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo : {}

The first three lines are always the same :

Exception             : System.NullReferenceException: Object reference not set to an instance of an object.
                           at System.Management.Automation.CommandInfo.ResolveParameter(String name)
                           at Microsoft.Windows.PowerShell.ScriptAnalyzer.Helper.GetExportedFunction(Ast ast)

The fourth concerns a rule name which can be different :
  at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.AvoidReservedCharInCmdlet.<AnalyzeScript>d__0.MoveNext()

The TargetObject property points to a module of the project. The module may not have been modified for several days and may not trigger an error during the previous analysis (10 minutes ago)
Environment data
v5.1  Windows server 2012

> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
Get-Module

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Management     {Add-Computer, Add-Content, Checkpoint-Computer, Clear-Con...
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Object, Con...
Script     <IP_ADDRESS>    posh-git                            {Add-PoshGitToProfile, Expand-GitCommand, Format-GitBranch...
Script     4.9.0      psake                               {Assert, Exec, FormatTaskName, Framework, Get-PSakeScriptT...
Script     2.0.2      PSReadline                          {Get-PSReadLineKeyHandler, Get-PSReadLineOption, Remove-PS...
Script     1.19.0     psscriptanalyzer                    {Get-ScriptAnalyzerRule, Invoke-Formatter, Invoke-ScriptAn...

Test without any error, performed after the error is triggered :
[STA] E:\Projets\XXXs> $error.Clear(); cls;1..20|% {.\build.ps1}

Command line :
$analysisResult = Invoke-ScriptAnalyzer -Path $ModuleOutDir -Settings $ScriptAnalyzerSettingsPath -Recurse -Verbose:($VerbosePreference -eq 'Continue')

#Setting file
@{
    ExcludeRules = @(
        'PSAvoidGlobalVars', #Legacy
        'PSAvoidUsingWMICmdlet', #Legacy
        'PSAvoidUsingDeprecatedManifestFields',
        'PSMissingModuleManifestField',
        'PSUseToExportFieldsInManifest',
        'PSUseUsingScopeModifierInNewRunspaces',
        'PSReviewUnusedParameter'
    )

    # Check if the scripts uses cmdlets that are compatible PS 3.0,
    Rules = @{
        PSUseCompatibleCommands = @{
            # Turns the rule on
            Enable = $true

            # Lists the PowerShell platforms we want to check compatibility with
            TargetProfiles = @(
                #PowerShell Version 3.0 Operating System Windows Server 2012
                'win-8_x64_6.2.9200.0_3.0_x64_4.0.30319.42000_framework'
            )
        }
    }
}

Looks like the old CommandInfo off the pipeline thread race condition

We could apply something similar to PR #1523

The problem I see is that taking a CommandInfo object and accessing it off the original pipeline thread is always going to have this race condition. The only safe way to manipulate these objects is to leave the pipeline on a dedicated thread and run delegates on it that copy the needed values over. I'm currently building something like this for PSES for other reasons, but it's not infeasible. Honestly though, this is a big reason why I think depending on PowerShell for linting here is an issue

Thanks @LaurentDardenne could you please to run only the PSSA command line when you have the chance and let us know if you are able to reproduce this?
Looks like the issue may be coming from PowerShell here, still, PSSA should protect itself from these types of errors/better handle parameter binding especially since we expect PSSA to be backwards compatible...we may want to check for a throw as well here, alternatively this rule could not use the helper function at all

Hi,
I think I am having a very similar issue.
When analyzing PSShouldProcess, I get the Object reference not set to an instance of an object.
Exception : ```
@{Exception=System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.UseShouldProcessCorrectly.TryGetShouldProcessValueFromAst(FunctionInfo functionInfo, Boolean& hasShouldProcessSet)
at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.UseShouldProcessCorrectly.SupportsShouldProcess(String cmdName)
at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.UseShouldProcessCorrectly.CheckForSupportShouldProcess()
at Microsoft.Windows.PowerShell.ScriptAnalyzer.BuiltinRules.UseShouldProcessCorrectly.d__7.MoveNext()
at System.Collections.Generic.List1..ctor(IEnumerable1 collection)
at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
at Microsoft.Windows.PowerShell.ScriptAnalyzer.ScriptAnalyzer.<>c__DisplayClass83_1.b__2()}

I had no problem yesterday, but today, analyzing doesn't work.

I found that it is specifically the PSShouldProcess rule that makes it not work. If excluded, it pass.

By the tag 'Consider 2.0', do you mean I should try 2.0 ? Since it is not on the PSGallery yet, how should I install it ?
Thanks

@avezinaATastus that looks like a different issue in this method:
https://github.com/PowerShell/PSScriptAnalyzer/blob/d04c03907b4a4b675bcc3cec3a32cbf3c4ee2d64/Rules/UseShouldProcessCorrectly.cs#L344-L398
Please open a new issue

@SydneyhSmith

Thanks @LaurentDardenne could you please to run only the PSSA command line when you have the chance and let us know if you are able to reproduce this?

Yes  I can, but not before a few weeks.

By the tag 'Consider 2.0', do you mean I should try 2.0 ? Since it is not on the PSGallery yet, how should I install it ?

That tag means we're considering addressing this issue in the 2.0 version, which is still being worked on.

@SydneyhSmith
My code having evolved, it no longer causes an error via psake or on the command line.
The only thing that differs on the first run is the scan time, but that's unrelated.
First run
--------------------------------------------------
Build Time Report
--------------------------------------------------
Name               Duration
----               --------
Init               00:00:00.033
Clean              00:00:00.073
BeforeBuild        00:00:00.001
RemoveConditionnal 00:00:00.000
BeforeStageFiles   00:00:00.001
CoreStageFiles     00:00:00.292
TestBOM            00:00:05.819
TestLocalizedData  00:00:00.000
AfterStageFiles    00:00:00.001
StageFiles         00:00:00.001
Analyze            00:00:40.536        <-------
TestBOMAfterAll    00:00:09.232
AfterBuild         00:00:00.007
Build              00:00:00.001
Total:             00:00:56.163

Next :
-----------------------------------
Build Time Report
-----------------------------------
Name               Duration
----               --------
Init               00:00:00.003
Clean              00:00:00.159
BeforeBuild        00:00:00.002
RemoveConditionnal 00:00:00.000
BeforeStageFiles   00:00:00.001
CoreStageFiles     00:00:00.137
TestBOM            00:00:07.524
TestLocalizedData  00:00:00.000
AfterStageFiles    00:00:00.001
StageFiles         00:00:00.001
Analyze            00:00:01.004      <------
TestBOMAfterAll    00:00:14.883
AfterBuild         00:00:00.004
Build              00:00:00.001
Total:             00:00:23.749

The module was not installed via Install-Module but downloaded from the PS Gallery + 'XCopy'.

I have this bug in 1.22.0.  Does anybody know what magical trick can be done to avoid it?  It may be that, like OP, I have installed PSScriptAnalyzer in a local directory instead of just install-module and letting module-search handle it.


4.0.0 stable release?

Hey, are you planning on releasing the preview.9 as a stable Version, so that we can get rid of the yellow "Preivew Packages in Use" label in Unity IDE HUD?
Thanks!

I'd also like to get rid of that annoying yellow label in Unity
Thanks

Sorry for my late reply.
UIEffect v5 is now released.
Please try it. :+1:
🔄 Upgrading from v4 to v5


Support maxOutputSize in native notebooks

Not sure if this will be handled by Jupyter or core side. But we should still support the maxOutputSize option to prevent massive input from eating up all the space. I know that I've run into this before myself. Also reported by a customer on Stack Overflow here:
https://stackoverflow.com/questions/68207737/jupyter-change-output-cell-size

Either support maxOutputSize or provide a better solution for dealing with large outputs.

Related to: https://github.com/microsoft/vscode/issues/126863
Sorry if in wrong repo


Getting started tutorial

Adding the notebook  with the chat_with_video example that will be used for getting started video.

@movchan74  @ashwinnair14  @evanderiel  Can you review this tutorial I wrote for the getting started video?

!pip install git+https://github.com/mobiusml/aana_sdk.git -qqqU

Shall we use pypi version instead?

I would suggest to add a link to integrations https://github.com/mobiusml/aana_sdk/blob/main/docs/integrations.md when you talk about Deployments.

from aana.projects.chat_with_video.const import (
    asr_model_name,
    captioning_model_name,
    max_video_len,
)
from aana.projects.chat_with_video.utils import (
    generate_dialog,
)

consts and generate_dialog util function are coming from the chat_with_video project.

from aana.projects.chat_with_video.const import (
    asr_model_name,
    captioning_model_name,
    max_video_len,
)
from aana.projects.chat_with_video.utils import (
    generate_dialog,
)

consts and generate_dialog util function are imported from the chat_with_video project.

I will take it into the main notebook body.
Question: I find the models being defined at two places and potentially can cause issues. One in the .const and other in deploys configs
asr_model_name = "whisper_medium"
captioning_model_name = "hf_blip2_opt_2_7b"
max_video_len = 60 * 20  # 20 minutes

deployments = [
    {
        "name": "asr_deployment",
        "instance": whisper_medium_deployment,
    },
    {
        "name": "captioning_deployment",
        "instance": hf_blip2_opt_2_7b_deployment,
    },
    {
        "name": "llm_deployment",
        "instance": internlm2_5_7b_chat_deployment,
    },
]

for deployment in deployments:
    aana_app.register_deployment(
        name=deployment["name"],
        instance=deployment["instance"],
    )

What might be a better way of handling this?

from aana.projects.chat_with_video.const import (
    asr_model_name,
    captioning_model_name,
    max_video_len,
)
from aana.projects.chat_with_video.utils import (
    generate_dialog,
)

consts and generate_dialog util function are imported from the chat_with_video project.

I will take it into the main notebook body.
Question: I find the models being defined at two places and potentially can cause issues. One in the .const and other in deploys configs
asr_model_name = "whisper_medium"
captioning_model_name = "hf_blip2_opt_2_7b"
max_video_len = 60 * 20  # 20 minutes

deployments = [
    {
        "name": "asr_deployment",
        "instance": whisper_medium_deployment,
    },
    {
        "name": "captioning_deployment",
        "instance": hf_blip2_opt_2_7b_deployment,
    },
    {
        "name": "llm_deployment",
        "instance": internlm2_5_7b_chat_deployment,
    },
]

for deployment in deployments:
    aana_app.register_deployment(
        name=deployment["name"],
        instance=deployment["instance"],
    )

What might be a better way of handling this?

Model names from const are only used for the database ops. Maybe we can call deployments and get the model name from it in the future.

Okay, then I will keep as it is and update when we have facility to call deployments and get models names.

On 12. Jul 2024, at 12:32, Aleksandr Movchan @.***> wrote:
from aana.projects.chat_with_video.const import (
asr_model_name,
captioning_model_name,
max_video_len,
)
from aana.projects.chat_with_video.utils import (
generate_dialog,
)
consts and generate_dialog util function are imported from the chat_with_video project.
I will take it into the main notebook body.
Question: I find the models being defined at two places and potentially can cause issues. One in the .const and other in deploys configs
asr_model_name = "whisper_medium"
captioning_model_name = "hf_blip2_opt_2_7b"
max_video_len = 60 * 20  # 20 minutes
deployments = [
{
"name": "asr_deployment",
"instance": whisper_medium_deployment,
},
{
"name": "captioning_deployment",
"instance": hf_blip2_opt_2_7b_deployment,
},
{
"name": "llm_deployment",
"instance": internlm2_5_7b_chat_deployment,
},
]
for deployment in deployments:
aana_app.register_deployment(
name=deployment["name"],
instance=deployment["instance"],
)
What might be a better way of handling this?
Model names from const are only used for the database ops. Maybe we can call deployments and get the model name from it in the future.
—
Reply to this email directly, view it on GitHub https://github.com/mobiusml/aana_sdk/pull/133#issuecomment-2225292602, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJL5SKZWVRGXDTGKWWCJZDZL6WFHAVCNFSM6AAAAABKXGMDW2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGI4TENRQGI.
You are receiving this because you authored the thread.


Mapping of runtime errors to line numbers

It would be nice if a runtime error occurred that the opcode could be traced back to a line number and source file.  Right now, it just aborts with the error, with no source/line/char information.  Kind of annoying.

https://github.com/voidqk/sink/commit/ce22a42becc479cf6741490c131cc37f1eb4eb12
https://github.com/voidqk/sink/commit/0b7284519a7b0b0fa0b7341f0429fe7e54417281


Asynchronous persistence

Asynchronous persistence， avoid IO block

You can leave a single pull request open, while you push up additional fixes, and later you can use an "interactive rebase" to squash your fixup commits into original, and then "force push" up the branch to make only the clean updated commits present in the old existing pull request.

OK，thank you very much


How to verify if local data is synchronized?

I'm using AngularFire2 Offline in my Ionic application. I want to verify whether local data is already synchronized with Firebase.
Does it have a function to check that?

Hi @thiagocarvp, you can just use the regular Firebase promise like this:
const promise = this.afoDatabase.object('car').update({maxSpeed: 100});
promise.then(() => console.log('data saved to Firebase!'));

This promise tells you when your local data is synchronized with Firebase.
You might also like reading working with promises.

Thanks for response!
Actually, I was looking for a function that verifies if whole data is already synchronized in my application.

@thiagocarvp can you give me an example of what you'd like to do? I think I need a little bit more context. Thanks!

It would be like to verify if every promise in AngularFire2 Offline is settled, like this:
myclass {
  counter: number = 0;

  update() {
    this.counter++;
    const promise = this.afoDatabase.object('car').update({maxSpeed: 100});
    promise.then(() => {
      console.log('data saved to Firebase!');
      this.counter--;
    });
    promise.catch(() => {
      this.counter--;
    }
  }

  isSynchronized() {
    return this.counter == 0;
  }
}

But I think this approach will lead to concurrency problems, and in case of app be closed, promises will be lost.

If I am able to get the promises of emulated updates which are made when app starts, I can count remaining promises, like I said in the comment above.
Is it possible to get these promises?

Not sure I follow your use case exactly, but you could consider using Promise.all()
Promise.all() will only resolve when ALL promises in the array resolves.
If a single promise fails, it will throw a catchable error.
const promiseArr: Promise<any>[];
Promise.all(promiseArr)
   .then(success => {})
   .catch(fail => {});

Thanks @larssn !
But in case of app would be killed, these promises (which are not resolved yet) would be lost. Then the app is open again, how could I get the promises for the updates which will be synchronized with Firebase when a connection is available?


Added register page

Register page added , fully functional with data storing into mongodb database, and signup with google

register logic isnt working and page design is too simple make it better

use nextauth for register part dont make post request as need to take care of jwt and cookie part also there make a callback call for rest of the things


Incompatible with Gambio Version: v<IP_ADDRESS>

Hi,
After installing the newest mollie module for my customer, the whole store crashed. It only shows the HTTP ERROR 500.
We cannot debug this further, the webhost does not allow to access the error log :/
Could you check?

Please note that we did not manage to reproduce the reported issue. After installing the latest module version 2.0.11 on Gambio <IP_ADDRESS> the module works as expected and there are no fatal errors.
Can you please give us more information about this environment so that we can manage to reproduce the behavior and continue with the investigation?

Please note that we are closing this issue from our side since it has been more than a month without any response from your side. Please feel free to reopen this ticket if you continue experiencing this issue and please provide us with more information.


Optimize tree() method of Stream class to reduce time complexity

Modified the tree() method to build the list in reverse order and then reverse it at the end, making the operation more efficient.
Earlier version did an insert(0, ...) operation on the list for each parent, which can have a worst-case time complexity of O(n**2) due to shifting of list elements for each call.
Modified implementation has an O(n) time complexity, with no additional space complexity, and maintains the same node ordering and behavior as the original tree() method.
The function docstring also needed a correction which has been incorporated into this fix.
Related Issue #399

Thank you, Daniil!
On Wed, Jul 17, 2024 at 3:54 PM Daniil Gusev @.***>
wrote:

@.**** approved this pull request.
Hi Aparna!
It's a nice catch 👍 .
About the optimization itself:
Please note that the tree() method is always called before any processing
happens, so the overall impact of this optimization on speed may be limited.
The docstring fix is correct ✅
Thanks for the contribution!
—
Reply to this email directly, view it on GitHub
https://github.com/quixio/quix-streams/pull/400#pullrequestreview-2182515027,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGHHTQWSSQCPETEPFVWGNLLZMZA5ZAVCNFSM6AAAAABK4QP3NOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCOBSGUYTKMBSG4
.
You are receiving this because you authored the thread.Message ID:
@.***>