Ali/simple planning simulator with disturbance generator

Note: Confirm our contribution guidelines before submitting a pull request.
Click the Preview tab and select a PR template:

Small change
We implemented disturbance generator classes for the simple_planning_simulator.

@mitsudome-r -san, @TakaHoribe -san will decide.


cdn.form.io/flatpickr/flatpickr.min.js and min.css not found

One of our projects is hosted internally and does not have access to internet. When we run the project, calls to flatpickr.mins.js and min.css fail because internet access is not allowed. I have searched the entire project but unable to find the reference to this file. Do I need to install any component so that these two files are available locally and cdn call is not performed?
Thanks

If you want this to work offline, then you will need to include Flatpickr as part of your application like so..
import { Flatpickr } from 'flatpickr';
window.Flatpickr = Flatpickr;

Our renderer will first check to see if there is a globally available Flatpickr and will not lazy load the CSS or JS if it is available.

Thanks @travist for your suggestions.
I added the suggested code but I get Module has no exported member 'flatpickr' error.

So after searching web a lot, I used the below statements, and now flatpickr.mins.js error does not appear however it is still unable to find min.css file, which means that clicking on Date Picker button does not display calendar. Do you know what is still missing?
import * as _flatpickr from 'flatpickr';
import { FlatpickrFn } from 'flatpickr/dist/types/instance';
const flatpickr: FlatpickrFn = _flatpickr as any;

You just need to import the flatpickr.css file into your SASS file of your application.

You just need to import the flatpickr.css file into your SASS file of your application.

looks like implicit npm dependency on "flatpickr" has been removed. we need to install "flatpickr" npm package explicity to make it work with latest versions.


Add test to validate _update_filter_form_initial

Not adding any new tests, but the number of skipped tests is trending downwards falling:
$ ./manage.py test
[ . . . ]
Ran 110 tests in 5.041s

OK (skipped=34)


develop->master (TASK-740)

Merge to master for changing next branch deploy to master.

@kkellerlbl What's running where now?
@rsutormin you might want to look at this, most of the changes are yours

OMG this is a year's worth of commits.

develop is currently on next (and therefore appdev and prod).
--
Keith Keller
Environmental Genomics and Systems Biology Division
(sent from mobile)
On Jun 16, 2017 17:13, "MrCreosote"<EMAIL_ADDRESS>wrote:

@kkellerlbl https://github.com/kkellerlbl What's running where now?
@rsutormin https://github.com/rsutormin you might want to look at this,
most of the changes are yours
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kbase/njs_wrapper/pull/89#issuecomment-309171107, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AECGTNjK-xxqx6SUdf4eujChTmSrVOigks5sExpDgaJpZM4N9B-B
.

OK, so this is not actually making any changes to any environments, so seems fine.

Yeah, it looks like a lot because we haven't been keeping master up to date (since staging was used for a while for next and appdev/prod).
All of these changes are going to go up on next before going anywhere else.  So we'll be able to figure out if we broke something before we put something up in production.


added snowplow analytics

Added snowplow analytics to site
Signed-off-by: wadeking98<EMAIL_ADDRESS>

My editor auto formatted the index file. The only parts I changed was the <script> block


no module named jupynium

Describe the bug
after installation with packer, when I run 'JupyniumStartAndAttachToServer', nothing happens.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Attaching to server
Logs in /tmp/jupynium/logs/
The tmp directory path can be different in a different OS.
Alternatively, run :JupyniumStartAndAttachToServerInTerminal to see the output directly in neovim.
Paste the output here
/usr/bin/python3: No module named jupynium

Output of jupynium --version
Jupynium v0.2.2

Output of nvim --version
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1702233742

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Additional context
I am running arch, so I had to change the comand in packer from run="pip3 install --user jupyium"  to
run=pipx install jupynium
I updated the $PATH variable, so I'm not sure, why it doesn't find it. Where would I change the search path for jupyium?

Hi, check if the build has failed because I think the command has a typo. run="pip3 install --user jupyium" should be "jupynium" not "jupyium".

JupyniumStartSync simply doesn't exist as a command (even right after installing & with filename 'test.ju.py')

You need to first start the server with :JupyniumStartAndAttachToServer which will open Firefox, and then the sync command will appear.
I believe installing with pipx wouldn't be a problem. Make sure the command jupynium is available from the environment you're running neovim. If you don't see any commands like the start and attach command above, it's probably because of the lazy loading not because of the python package.

I don't know why, but when I installed it over pipx nvim couldn't start the server via jupynium although it was available as a command in the shell. I did not lazy load the package, so at least for me this was not the issue.
The way I got it working in the end was installing it system wide with pip: $ pip install --break-system-packages jupynium
This has the disadvantage that you need to reinstall it every time you update your system, but so far that has been ok.

I can't thank you both enough for responding that quickly!
Got it to work finally, thanks!
Seemed to mostly be an issue of lazy loading and the global installation of python packages on Arch.
Got it to work by installing it in a venv and only loading the package manually if I really need it.
Great plugin @kiyoon, thanks a lot for your work!


[Snyk] Security upgrade lighthouse from 3.2.1 to 6.2.0

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

scripts/bench/package.json
scripts/bench/yarn.lock

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-CROSSSPAWN-8303230
  170

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.


Metadata Key suffixes

Closes #63
The rational is that backends may be explored by non zarr-aware tools,
like a filesytem and a GUI browser like Finder/Explorer/Nautilus, and
the .json extension will help to view/edit json in those.
While it may seem like this shoudl be store specific, once again the
fact that a zarr-hierarchy can be copied from one storage to another by
non-zarr-aware tool indicate that this shoudl be handled above the sotre
level.

Thanks @Carreau.


[v6] Windows not initializing driver (MMI Modular USB DAC)

Using USB Dac from MMI Modular
Describe the bug:
USB Dac not initializing correctly on Windows 11, keeps connecting/disconnecting. No audio and driver keeps trying to reinstall. Windows device manager shows an error with device.

Steps to Reproduce:

Using the USB DAC, enter bootloader mode by holding down button and plugging into windows. Windows will pop up in file explorer with a new device.

Download v6 firmware and copy to new device in said bootloader mode. Device will reset and go into audio dac mode.

Expected behavior:
Windows should load device as new audio device, play audio.
Real behavior:
Windows see's the device but is unable to initialize device. Seems to connect/reconnect constantly.
Media:
https://github.com/user-attachments/assets/2db46f4b-a70e-40d6-ab56-a95b6da250ee
Device Specs:
Windows 11 Pro v10.0.22631 (Build 22631) x64
ROG STRIX Z590-E GAMING WIFI motherboard
USB C - C cable and USB C - A cable were both tested, some USB2.0 and some 3.0
RTX 4070 ti
intel i9-10900k
64GB Ram
Additional context:
Running with iOS on an iPhone 15 Pro Max, the device struggles to connect the first time but eventually connects and works fine.
Pico Dac Error Log:
Pico Dac Error Log.zip

Did some more testing:
Running on a seperate computer, laptop with Windows 11, resulted in the same connecting/disconnecting bug. Seems like a driver issue off some sort?
Went through all the firmware versions from v1-v6, doesn't seem like v1-v3 dev even had windows detect anything was plugged in after updating to firmware. v5&v6 both seem to see the device, then 5-10 seconds later disconnect as seen in the video posted above.
Only firmware that seemed to do anything was one provided from MMI Modular, [pico_amp(2).zip](https://github.com/user-attachments/files/16787037/pico_amp.2.zip  but that just resulted in a, "Device cannot start [Code 10]" error in device manager.

Can report the same behavior on Windows10
On Manjaro Linux and android 12 it behaves like it should

[v7] Fixed issue with constant connect/disconnect, but after disconnecting headphones W11 drivers break for Pico Amp.
Current workaround: Go to device manager, uninstall Pico Amp driver under sound devices. After device is removed, go to the top and select, “Scan for Hardware Changes”. Device settings will return to default but will work again.
Process must be repeated after every disconnect, possibly reset of PC (later untested).

thanks for the report, i actually somehow did not see this issue until today!
i still have not figured out why this behavior is happening, and there may be some related issue where data does not get received properly either. unfortunately i am not very familiar with the sdk, it could be a bug on their usb implementation, i do not know.

What IDE are you using? I don't know much but wanna play around a bit

hello, i develop in vscode
it is more or less the default configuration for developing with rp2040 and pi picoo
for some reason my environment variables or include paths are messed up, so i have a local cmakelist. if there are some issues, maybe it will help:
CMakeLists.txt


Update Pipelines standalone deployment to describe generic standalone (as well as GCP)

https://www.kubeflow.org/docs/pipelines/standalone-deployment-gcp/

/assign

From a discussion today: The standalone deployment guide currently describes a GCP-specific deployment. But only one part of the guide is GCP-specific (the inverting proxy) and the rest can be generic to allow deployment on another clouds e.g. AWS.
We should update the page so that it describes the generic deployment of Pipelines Standalone, with a GCP-specific section and guidelines for how other clouds/on-prem/local should handle the proxy.
@jingzhang36 assigning this to you with thanks!

Note: PR https://github.com/kubeflow/website/pull/1472 moves the doc to a new location (without changing the content). The new location is at https://www.kubeflow.org/docs/pipelines/installation/standalone-deployment/

I think there is more in that doc which is sort of mis-placed.
For e.g.  section Before you get started  has sub section setup your cluster. I think in that sub section we can add reference to creation of cluster from AWS, Azure and GCP pages or just assume that user has cluster.

This section is also very specific to gcp.
Get the public URL for the Kubeflow Pipelines UI and use it to access the Kubeflow Pipelines UI:

We can modify the instruction to be generic.

/reopen
We still want this open.
Also FYI, we have already provided a platform agnostic env: https://github.com/kubeflow/pipelines/tree/master/manifests/kustomize/env/platform-agnostic. It should be good for general installation.

/lifecycle frozen


Found bugs/issues in README.md and CONTRIBUTING.md

Describe the bug/issue
-> Unnecessary whitespaces (&nbsp)
-> Information related to Augmentation.py file not present
-> Links to helper files not working because of hhttps in links
To Reproduce
NA, except for links which do not appear as links because of extra h
Expected behavior
Clean and complete files with any linking errors
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [Windows 10]
Browser [Chrome]
Version [80.0.3987.132 (Official Build)]

Additional context
Add any other context about the problem here.

Pull request created - https://github.com/AMLResearchProject/ALL-Detection-System-2020/pulls

Hi Rishabh, thank you for contributing, these would need to be pushed to the current development branch which is 0.0.2.

Automatic close did not work :) Issue is closed with attached pull request. Thank you @rishabhbanga !


Kegg

https://www.genome.jp/kegg/ko.html
https://www.kegg.jp/blastkoala/
https://academic.oup.com/bioinformatics/article/36/7/2251/5631907

script was written to allow the fetch of information from this database.
can be compared with collection from orthodb which is also working


Generalize console control

Sometimes a plugin needs control of the console for a prompt it doesn't make directly. Get control before calling all primitives that take a context (as a proxy for potential network activity or execs).
This isn't entirely safe. If it's backgrounded and prompts for input it will cause Wash to exit. Same if you Ctrl-C the input prompt. Using plugin.Prompt guards against both of these issues (guarding against Ctrl-C exiting the shell is new). I'm not sure how to fix this without making the daemon part of or a child of the shell process.
Fixes #297.
Signed-off-by: Michael Smith<EMAIL_ADDRESS>

So this handles the case when the plugin's API prompts for something, right?

Yes.

Ok, I think this works better now.


Parameter 'streets' not found in 'ADDRESS' category

Affected localizations: en-SG and ar

This is resolved by #142


Disign Pattern & Explanation over multiple source codes

I'm now using vscode-openai to help me to optimize a project. But It seems to transfer the current file in the editor only and does not keep an eye on the project structure and other context. This has reduced the usability of this plugin.
There are some ways to enhance these features:

Send a tree view of workspace (or project, or files user given) to the model with current source code.
Based on the tree view, model could tell us which file is worth to look on. For more one step, summary each file (and cached if no changes), hold function signatures, important comments, etc. then send summaries of key files to the model with current source code.
Or for simplicity and completeness, just ask users which source files they want to transfer, and do it directly, or append to the tree view.

Great suggestion, I had already started working on the feature. I hope to release it this month.


dfve - Improvement/expanding test coverage

WIP - dfve

Implementing functionality to deploy + use FREs
Next: Implement functionality to randomly lock veOCEAN, consume data, etc...

Went to a place w/ no internet. Was able to work w/ everything local. Great.
WIP, but pretty close:

createDataNFTWithFREs
randomlyCreatelocks
randomlyAllocate
randomlyConsume

Working through a few, final kinks w/ passing arrays to contracts while using specific data types.

I'm stuck at this point. I can't figure out why the exchangeId is invalid.

Were you able to work through the FRE readme first?  https://github.com/oceanprotocol/ocean.py/blob/main/READMEs/marketplace-flow.md


Add dependencies in conda to avoid CondaPackError

Add dependencies in conda to avoid:
CondaPackError:
Files managed by conda were found to have been deleted/overwritten in the
following packages:

- numpy 1.24.0:
    lib/python3.11/site-packages/numpy-1.24.0.dist-info/INSTALLER
    lib/python3.11/site-packages/numpy-1.24.0.dist-info/LICENSE.txt
    lib/python3.11/site-packages/numpy-1.24.0.dist-info/METADATA
    + 7 others

This is usually due to `pip` uninstalling or clobbering conda managed files,
resulting in an inconsistent environment. Please check your environment for
conda/pip conflicts using `conda list`, and fix the environment by ensuring
only one version of each package is installed (conda preferred).

This seems to happen when we install a package via conda and then install it again via pip

We'll use https://github.com/QCDIS/NaaVRE-conf to map names.
Also if the packages is not available in conda


➕🔊: add logr for use by client

Description
With go-anxcloud#50 merged, the client will issue a warning when configured to use a io.Writer for logging, telling the user to pass a logr.Logger instead.
This PR adds logr.Loggers for our log levels and passes the debugLogger to the client with the new option.
Acceptance tests

[ ] Have you added an acceptance test for the functionality being added?

Output from acceptance testing, while having a replace rule to use my local go-anxcloud

$ make testacc
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test $(go list ./... | grep -v 'vendor') -v  -timeout 120m -parallel=4
?   	github.com/anexia-it/terraform-provider-anxcloud	[no test files]
=== RUN   TestAccAnxCloudCoreLocationsDataSource
--- PASS: TestAccAnxCloudCoreLocationsDataSource (9.24s)
=== RUN   TestAccAnxCloudCPUPerformanceTypesDataSource
--- PASS: TestAccAnxCloudCPUPerformanceTypesDataSource (6.98s)
=== RUN   TestAccAnxCloudDiskTypeDataSource
--- PASS: TestAccAnxCloudDiskTypeDataSource (7.75s)
=== RUN   TestAccAnxCloudIPAddressesDataSource
--- PASS: TestAccAnxCloudIPAddressesDataSource (9.47s)
=== RUN   TestAccAnxCloudNICTypesDataSource
--- PASS: TestAccAnxCloudNICTypesDataSource (7.48s)
=== RUN   TestAccAnxCloudTagsDataSource
--- PASS: TestAccAnxCloudTagsDataSource (7.62s)
=== RUN   TestAccAnxCloudTemplateDataSource
--- PASS: TestAccAnxCloudTemplateDataSource (10.17s)
=== RUN   TestAccAnxCloudVLANsDataSource
--- PASS: TestAccAnxCloudVLANsDataSource (8.65s)
=== RUN   TestAccAnxCloudVSphereLocationsDataSource
--- PASS: TestAccAnxCloudVSphereLocationsDataSource (9.29s)
=== RUN   TestProvider
--- PASS: TestProvider (0.00s)
=== RUN   TestProvider_impl
--- PASS: TestProvider_impl (0.00s)
=== RUN   TestAccAnxCloudIPAddress
--- PASS: TestAccAnxCloudIPAddress (39.48s)
=== RUN   TestAccAnxCloudIPAddressReserved
--- PASS: TestAccAnxCloudIPAddressReserved (28.81s)
=== RUN   TestAccAnxCloudNetworkPrefix
--- PASS: TestAccAnxCloudNetworkPrefix (105.75s)
=== RUN   TestAccAnxCloudTag
--- PASS: TestAccAnxCloudTag (8.21s)
=== RUN   TestAccAnxCloudVirtualServer
--- PASS: TestAccAnxCloudVirtualServer (941.90s)
=== RUN   TestAccAnxCloudVirtualServerMultiDiskScaling
=== RUN   TestAccAnxCloudVirtualServerMultiDiskScaling/AddDisk
=== RUN   TestAccAnxCloudVirtualServerMultiDiskScaling/ChangeAddDisk
=== RUN   TestAccAnxCloudVirtualServerMultiDiskScaling/MultiDiskTemplateChange
--- PASS: TestAccAnxCloudVirtualServerMultiDiskScaling (1250.07s)
    --- PASS: TestAccAnxCloudVirtualServerMultiDiskScaling/AddDisk (405.25s)
    --- PASS: TestAccAnxCloudVirtualServerMultiDiskScaling/ChangeAddDisk (420.50s)
    --- PASS: TestAccAnxCloudVirtualServerMultiDiskScaling/MultiDiskTemplateChange (424.33s)
=== RUN   TestAccAnxCloudVLAN
--- PASS: TestAccAnxCloudVLAN (100.50s)
=== RUN   TestFlattenCPUPerfrormanceTypes
--- PASS: TestFlattenCPUPerfrormanceTypes (0.00s)
=== RUN   TestFlattenDiskTypes
--- PASS: TestFlattenDiskTypes (0.00s)
=== RUN   TestFlattenIPAddresses
--- PASS: TestFlattenIPAddresses (0.00s)
=== RUN   TestFlattenCoreLocations
--- PASS: TestFlattenCoreLocations (0.00s)
=== RUN   TestFlattenNetworkPrefixLocations
--- PASS: TestFlattenNetworkPrefixLocations (0.00s)
=== RUN   TestFlattenVLANLocations
--- PASS: TestFlattenVLANLocations (0.00s)
=== RUN   TestFlattenLocations
--- PASS: TestFlattenLocations (0.00s)
=== RUN   TestFlattenTags
--- PASS: TestFlattenTags (0.00s)
=== RUN   TestFlattenTemplates
--- PASS: TestFlattenTemplates (0.00s)
=== RUN   TestExpanderVirtualServerNetworks
--- PASS: TestExpanderVirtualServerNetworks (0.00s)
=== RUN   TestExpanderVirtualServerDNS
--- PASS: TestExpanderVirtualServerDNS (0.00s)
=== RUN   TestExpanderVirtualServerDisks
--- PASS: TestExpanderVirtualServerDisks (0.00s)
=== RUN   TestExpanderVirtualServerInfo
--- PASS: TestExpanderVirtualServerInfo (0.00s)
=== RUN   TestFlattenVirtualServerNetwork
--- PASS: TestFlattenVirtualServerNetwork (0.00s)
=== RUN   TestFlattenVirtualServerInfo
--- PASS: TestFlattenVirtualServerInfo (0.00s)
=== RUN   TestFlattenVirtualServerDisks
--- PASS: TestFlattenVirtualServerDisks (0.00s)
=== RUN   TestRoundDiskSize
--- PASS: TestRoundDiskSize (0.00s)
=== RUN   TestFlattenVLANs
--- PASS: TestFlattenVLANs (0.00s)
PASS
ok  	github.com/anexia-it/terraform-provider-anxcloud/anxcloud	2551.398s

Release Note
Release note for CHANGELOG:
NONE (I think)

References
Depends on go-anxcloud#50 merged and released, will need a version bump in go.mod when that PR is merged and included in a new release.
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

LGTM when checks are green

Uhm, this is a draft for a reason ^^'
But I'm currently working on it


get_many_mut with variable number of keys

get_many_mut on slices (https://github.com/rust-lang/rust/pull/83608) does not need to handle a variable number of keys (although it might be nice) because, worse case scenario, you can sort your input array of indexes and iterate over it with successive split_at_mut.
In this case however, that's not an option, as HashMaps don't have the equivalent of split_at_mut.
So it may be useful to be able to lookup a variable (somewhat large) number of distinct keys at the same time in a HashMap, and get mutable references back.
That is in particular needed when we have a storage of many entries, and we want to run a round of updates on distinct subsets of keys, and want to run these in parallel.
Having get_many_mut follow a similar model to what is described at https://github.com/rust-lang/rust/pull/83608#pullrequestreview-966536956 and providing an interface that allows for this would consequently be useful - in fact, that is something I currently would use.

Keep me posted, m8.

I'd also really like to have this, I'm in a situation right now where I need to have an arbitrary number of mutable references and the way I'm doing it for now isn't great

The problem with returning a variable number of mutable references is that it requires returning a Vec<&mut T> which requires a dynamic allocation. It might end up being faster to just keep the keys around and doing a hash table lookup every time the value is needed.

It might end up being faster to just keep the keys around and doing a hash table lookup every time the value is needed.

It seems that this would be for the end user to decide, no?

Keep in mind that get_many_mut has a complexity of O(n^2) where n is the number of keys to look up. This is because each returned value must be checked against every other returned value to ensure that none of them point to the same entry, which is necessary to satisfy Rust's safety guarantees (mutable references are unique). This can quickly add up when n becomes large.

each returned value must be checked against every other returned value

That seems reasonable on the const size version because those are unlikely to ever be very large, but here indeed that would seem a bit too expensive, so probably flagging entry as currently being borrowed would fix this?
By the way, on the const size version, how do you deal with Eq implementations that are inconsistent? ("It's ok I'm equal to nobody - oh wait in fact I'm the same key" -> yields several mutable borrows to the same entry)
It looks like the flag at the entry level would be the solution for this as well, no?

We compare the pointers to the entries that are actually returned by the lookups.

Ah of course! ^^
Then I guess just sorting the pointers and checking for no dupes works.

Ah of course! ^^ Then I guess if n is too large just sorting the pointers and checking for no consecutive dupes works in O(n* log(n)).

Unlikely, since you still need to take into account the sort itself

Currently the implementation is optimized for small n (the most common case is expected to be n = 2).

Unlikely, since you still need to take into account the sort itself

The sort itself is O(n* log(n)).

Currently the implementation is optimized for small n

For the const-size version we probably want to keep the current version. It's also very practical to be able to pattern-match directly on the result.
However it looks like we could provide different APIs through the same function via a GetManyMut trait, as was considered for the std's version on slices.
Then we could have different versions based on the slice size:
let ptrs: Vec<*mut ??> = /* ... */;
if n < 10 { // 10 is the limit for insertion sort in std
    // do n^2
} else {
    let mut ptrs_sorted = ptrs.clone();
    ptrs_sorted.sort_unstable(); // n * log(n)
    assert!(ptrs_sorted.windows(2).all(|w| w[0] != w[1]); // n
}

With pointers, the problem becomes essentially exactly the same as they have there with indexes.
https://github.com/rust-lang/rust/pull/83608#pullrequestreview-966536956

Unlikely, since you still need to take into account the sort itself

The sort itself is O(n* log(n)) where n is the number of queried entries.

Well, as you can see from your own example, you first need to sort, and then go through the array again, looking for the equal pointers. In addition, you do not take into account that sorting will violate the order of the values. That is, the corresponding key will not point to the corresponding value.

Well, as you can see from your own example, you first need to sort, and then go through the array again, looking for the equal pointers

I'm not sure why that is an issue.

In addition, you do not take into account that sorting will violate the order of the values. That is, the corresponding key will not point to the corresponding value.

I do, that's what the ptrs.clone() is for. I'm returning ptrs at the end.

I do, that's what the ptrs.clone() is for. I'm returning ptrs at the end. If we want to have a single alloc we could also with_capacity(n*2) and duplicate the contents at the end of the vec, sort the subslice for the check, then truncate it.

Hmm... it might work.  It looks like the method you suggested will improve the existing implementation as well.
As for variable length.  Why not consider using something like https://doc.rust-lang.org/std/array/struct.IntoIter.html.  For example, create a separate search array and a separate output array.  Then push only unique pointers into the output array and return it as IntoIter?

Why not consider...

I'm afraid I don't understand what you have in mind. Perhaps that would be easier with a code sample?

Why not consider...

I'm afraid I don't understand what you have in mind. Perhaps that would be easier with a code sample?

So I came up with the function below. The only thing is that maybe it can be used for some other version of get_many_key_value_mut, since it is not guaranteed that the length of the iterator is equal to the length of the array of keys and that all the keys are in the map.
use core::ops::Range;

impl<T, A: Allocator + Clone> RawTable<T, A> {
    pub fn get_many_mut_two<const N: usize>(
        &mut self,
        hashes: [u64; N],
        mut eq: impl FnMut(usize, &T) -> bool,
    ) -> IntoIter<&mut T, N> {
        struct Guard<'a, T, const N: usize> {
            /// The array to be initialized.
            array_mut: &'a mut [MaybeUninit<T>; N],
            /// The number of items that have been initialized so far.
            initialized: usize,
        }

        impl<T, const N: usize> Guard<'_, T, N> {
            #[inline(always)]
            unsafe fn push_unchecked(&mut self, item: T) {
                // SAFETY: If `initialized` was correct before and the caller does not
                // invoke this method more than N times then writes will be in-bounds
                // and slots will not be initialized more than once.
                self.array_mut
                    .get_unchecked_mut(self.initialized)
                    .write(item);
                self.initialized += 1;
            }
        }

        // SAFETY: The `assume_init` is safe because the type we are claiming to have
        // initialized here is a bunch of `MaybeUninit`s, which do not require initialization.
        let mut array: [MaybeUninit<*mut T>; N] = unsafe { MaybeUninit::uninit().assume_init() };

        let mut guard = Guard {
            array_mut: &mut array,
            initialized: 0,
        };

        for (i, &hash) in hashes.iter().enumerate() {
            if let Some(cur) = self.find(hash, |k| eq(i, k)) {
                // SAFETY: hashes length is equal to `array` length
                unsafe { guard.push_unchecked(cur.as_ptr()) }
            }
        }

        let Guard { initialized, .. } = guard;
        // SAFETY: We provide the number of elements that are guaranteed to be initialized in the
        // loop above. We also know for sure that initialized is in the range of the array's length.
        let into_iter = unsafe { IntoIter::new_unchecked(array, 0..initialized) };

        let slice = into_iter.as_slice();

        // SAFETY: The `assume_init` is safe because the type we are claiming to have
        // initialized here is a bunch of `MaybeUninit`s, which do not require initialization.
        let mut out_array: [MaybeUninit<&mut T>; N] =
            unsafe { MaybeUninit::uninit().assume_init() };
        let mut guard = Guard {
            array_mut: &mut out_array,
            initialized: 0,
        };

        for (i, &cur) in slice.iter().enumerate() {
            if slice[i + 1..].iter().all(|&next| !ptr::eq::<T>(next, cur)) {
                // SAFETY:
                // 1. The `slice` length is equal or less than `out_array` length
                // 2. We have just verified that this is a unique pointer that does not repeat within the array.
                // 3. We got all the pointers from the `find` function, so they are valid.
                unsafe { guard.push_unchecked(&mut *cur) };
            }
        }

        let initialized = 0..guard.initialized;
        // SAFETY: We provide the number of elements that are guaranteed to be initialized in the
        // loop above. We also know for sure that initialized is in the range of the array's length.
        unsafe { IntoIter::new_unchecked(out_array, initialized) }
    }
}

/// A by-value [array] iterator.
pub struct IntoIter<T, const N: usize> {
    data: [MaybeUninit<T>; N],
    alive: Range<usize>,
}

impl<T, const N: usize> IntoIter<T, N> {
    /// Creates an iterator over the elements in a partially-initialized buffer.
    ///
    /// # Safety
    ///
    /// - The `buffer[initialized]` elements must all be initialized.
    /// - The range must be canonical, with `initialized.start <= initialized.end`.
    /// - The range must be in-bounds for the buffer, with `initialized.end <= N`.
    ///   (Like how indexing `[0][100..100]` fails despite the range being empty.)
    ///
    /// It's sound to have more elements initialized than mentioned, though that
    /// will most likely result in them being leaked.
    #[inline]
    const unsafe fn new_unchecked(buffer: [MaybeUninit<T>; N], initialized: Range<usize>) -> Self {
        // SAFETY: one of our safety conditions is that the range is canonical.
        Self {
            data: buffer,
            alive: initialized,
        }
    }

    #[inline]
    pub fn as_slice(&self) -> &[T] {
        unsafe {
            // SAFETY: We know that all elements within `alive` are properly initialized.
            let slice = self.data.get_unchecked(self.alive.clone());
            // SAFETY: casting `slice` to a `*const [T]` is safe since the `slice` is initialized,
            // and `MaybeUninit` is guaranteed to have the same layout as `T`.
            // The pointer obtained is valid since it refers to memory owned by `slice` which is a
            // reference and thus guaranteed to be valid for reads.
            &*(slice as *const [MaybeUninit<T>] as *const [T])
        }
    }
}

impl<T, const N: usize> Iterator for IntoIter<T, N> {
    type Item = T;

    #[inline]
    fn next(&mut self) -> Option<Self::Item> {
        // Get the next index from the front.
        //
        // Increasing `alive.start` by 1 maintains the invariant regarding
        // `alive`. However, due to this change, for a short time, the alive
        // zone is not `data[alive]` anymore, but `data[idx..alive.end]`.
        self.alive.next().map(|idx| {
            // Read the element from the array.
            // SAFETY: `idx` is an index into the former "alive" region of the
            // array. Reading this element means that `data[idx]` is regarded as
            // dead now (i.e. do not touch). As `idx` was the start of the
            // alive-zone, the alive zone is now `data[alive]` again, restoring
            // all invariants.
            unsafe { self.data.get_unchecked(idx).assume_init_read() }
        })
    }

    #[inline]
    fn size_hint(&self) -> (usize, Option<usize>) {
        let len = self.len();
        (len, Some(len))
    }

    #[inline]
    fn fold<Acc, Fold>(mut self, init: Acc, mut fold: Fold) -> Acc
    where
        Fold: FnMut(Acc, Self::Item) -> Acc,
    {
        let data = &mut self.data;
        self.alive.by_ref().fold(init, |acc, idx| {
            // SAFETY: idx is obtained by folding over the `alive` range, which implies the
            // value is currently considered alive but as the range is being consumed each value
            // we read here will only be read once and then considered dead.
            fold(acc, unsafe { data.get_unchecked(idx).assume_init_read() })
        })
    }

    #[inline]
    fn count(self) -> usize {
        self.len()
    }

    #[inline]
    fn last(mut self) -> Option<Self::Item> {
        self.next_back()
    }
}

impl<T, const N: usize> DoubleEndedIterator for IntoIter<T, N> {
    #[inline]
    fn next_back(&mut self) -> Option<Self::Item> {
        // Get the next index from the back.
        //
        // Decreasing `alive.end` by 1 maintains the invariant regarding
        // `alive`. However, due to this change, for a short time, the alive
        // zone is not `data[alive]` anymore, but `data[alive.start..=idx]`.
        self.alive.next_back().map(|idx| {
            // Read the element from the array.
            // SAFETY: `idx` is an index into the former "alive" region of the
            // array. Reading this element means that `data[idx]` is regarded as
            // dead now (i.e. do not touch). As `idx` was the end of the
            // alive-zone, the alive zone is now `data[alive]` again, restoring
            // all invariants.
            unsafe { self.data.get_unchecked(idx).assume_init_read() }
        })
    }

    #[inline]
    fn rfold<Acc, Fold>(mut self, init: Acc, mut rfold: Fold) -> Acc
    where
        Fold: FnMut(Acc, Self::Item) -> Acc,
    {
        let data = &mut self.data;
        self.alive.by_ref().rfold(init, |acc, idx| {
            // SAFETY: idx is obtained by folding over the `alive` range, which implies the
            // value is currently considered alive but as the range is being consumed each value
            // we read here will only be read once and then considered dead.
            rfold(acc, unsafe { data.get_unchecked(idx).assume_init_read() })
        })
    }
}

impl<T, const N: usize> ExactSizeIterator for IntoIter<T, N> {
    #[inline]
    fn len(&self) -> usize {
        self.alive.len()
    }
}

impl<T, const N: usize> FusedIterator for IntoIter<T, N> {}

get_many_mut on slices (rust-lang/rust#83608) does not need to handle a variable number of keys (although it might be nice) because, worse case scenario, you can sort your input array of indexes and iterate over it with successive split_at_mut.

Actually I figure out how it can be done. I try to implement it in next two days.

@Amanieu, @Ten0 What if we provide the following API (draft pull request #408):
pub fn try_get_many_key_value_mut<'a, Q, I, const N: usize>(
    &mut self,
    iter: &mut I,
) -> ArrayIter<(&K, &mut V), N>
where
    I: Iterator<Item = &'a Q>,
    Q: ?Sized + Hash + Equivalent<K> + 'a,
{
    /* implementation */
}

pub unsafe fn try_get_many_key_value_unchecked_mut<'a, Q, I, const N: usize>(
    &mut self,
    iter: &mut I,
) -> ArrayIter<(&K, &mut V), N>
where
    I: Iterator<Item = &'a Q>,
    Q: ?Sized + Hash + Equivalent<K> + 'a,
{
    /* implementation */
}

Where ArrayIter practically repeats the implementation of IntoIter from https://doc.rust-lang.org/std/array/struct.IntoIter.html. Unfortunately, using IntoIter directly does not work due to the instability of new_unchecked. The ArrayIter itself is not only an iterator, it has a few more methods:
impl<T, const N: usize> ArrayIter<T, N> {
    /// Returns an immutable slice of all elements that have not been yielded yet.
    #[inline]
    pub fn as_slice(&self) -> &[T] {
         /* implementation */
    }

    /// Returns a mutable slice of all elements that have not been yielded yet.
    #[inline]
    pub fn as_mut_slice(&mut self) -> &mut [T] {
         /* implementation */
    }

    /// Returns an [`ArrayIter`] of the same size as `self`, with function `f`
    /// applied to each element in order.
    pub fn convert<F, U>(self, mut f: F) -> ArrayIter<U, N>
    where
        F: FnMut(T) -> U,
    {
         /* implementation */
    }
}

While your implementation is very clever, I don't think this is an API I actually want in hashbrown. It just ends up being very hard to use correctly (due to the interactions between the N constant and the iterator size, for example).
As I've said before, you really should just do separate lookups when you actually need the values instead of keeping an arbitrary number of mutable references around.


[cuda] Build error

Host Environment

OS: [e.g. Windows/Linux etc...]
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
./vcpkg install xxxx
Failure logs
-Cut and paste the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

@naveenmarni8 Thanks for posting this issue, In the following path file, we provide the download method of cuda。And can you download it and try again? :)

scripts/azure-pipelines/linux/provision-image.sh
https://github.com/microsoft/vcpkg/blob/a9ff505c5b8677af4c09a6e60860b1dde932fccc/scripts/azure-pipelines/linux/provision-image.sh

I am using windows os, can you suggest me other link? By the way, I already downloaded two different versions of cuda from nvidia but no use.

@naveenmarni8 Can you explain which port you installed? Because the re-step download you provided is ‘xxxx ’

@naveenmarni8 Is still work for this issue? Is the problem still there?

@naveenmarni8 Is still work for this issue? Please reply me if the problem still exist and I will reopen it.

Confirm

Yes you can close it.
On Wed, 16 Feb 2022, 2:38 pm Frank, @.***> wrote:

Confirm
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/22508#issuecomment-1041268572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALVB4EM5KJAQ655OS2EX7O3U3NSRRANCNFSM5LZWTDXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


MNT: Re-rendered with conda-smithy 2.4.2

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


Investigate how to get new notebooks from gists — and maybe any other site outside of GitHub

I think Github doesn't have an API to search gists. Please let me know if I'm wrong.
We could use gists' HTML search [1] and parse it.
But maybe we could explore ways to find notebooks everywhere.
For example via:

Github Archive (Bigquery). Is this possible? If so, at what cost?
Google Search

By the way, we could also create an input box #5 and an API endpoint where the community could manually add notebooks that are not in repos or not on Github.
[1] https://gist.github.com/search?q=extension%3Alivemd&ref=simplesearch

I think we could postpone this. Our current daily fetch for new notebooks plus https://github.com/notesclub/notesclub/issues/40 might be engough.


FormBuilderChipsInput Error using required validator

It works fine if I don't use validator: FormBuilderValidators.compose([                     FormBuilderValidators.required(context),                   ]),

But as soon als I do I get this Error Message

flutter_form_builder: ^4.0.2

Do u really use ^4.0.2 ?

Yes my Version is até the bottom of my post.
flutter_form_builder: ^4.0.2

It's not only in chips input. Same issue with text field input.

BTW...  You don't need to use compose when you only have 1 validator.

Is it possible that your problem is line 65:
return [];

Maybe that needs a type, like:
return <String>[];

And actually, adding const might be smart too:
return const <String>[];

Related to my last comment, I noticed that there was a missing type declaration here:
https://github.com/danvick/flutter_form_builder/blob/c2b63433778187cd9258499f1bf5f53781fa3ffa/lib/src/fields/form_builder_chips_input.dart#L9
It should say:
 final ChipsInputSuggestions<T> findSuggestions;

I submitted PR #621 to address that.  That should add the necessary type constraint, making my earlier suggestion irrelevant.  However, in the meantime (until the PR is merged into a Release), adding the type declaration should do the trick.


allow content to be pulled for specific platform(s), all platforms

Modified FilterPlatforms handler and RemoteContext so that either one specific platforms, various platforms, or all platforms can be pulled. Additionally, it is possible to push the content for all platforms to a remote registry.
This is the second step in allowing for multi-arch images on multi-arch platforms #2029
This closes #1772. Please run ctr images pull with the flag --all-platforms
Signed-off-by: Jess Valarezo<EMAIL_ADDRESS>

Codecov Report

Merging #2200 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #2200   +/-   ##
=======================================
  Coverage   41.11%   41.11%
=======================================
  Files          66       66
  Lines        7544     7544
=======================================
  Hits         3102     3102
  Misses       3961     3961
  Partials      481      481

Flag
Coverage Δ

#windows
41.11% <0%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

images/oci/exporter.go
0% <0%> (ø)
:arrow_up:

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 edf7f41...d855ea9. Read the comment docs.

LGTM


Get a map of all CenturyLink DSLAM locations

Surely this is public information.
If we know where those are, and we know the distance to properties via the right-of-way, then we can guess who may be able to get DSL, and at what speed.

http://www.sandman.com/cosearch.asp


load scheduler from config

With this PR the anna-molly scheduler can be dynamically created from the config file.

close until the tests are updated ...


feat: Added --logger option to the commands.

fix: logger code improvements. No more console.out, loggers are provided and not statically created giving the user, devs, and apps more options
fix: unit tests logs are hidden by default.
This is a PR from a list of improvements I need to do in order to support the new symbol-network app better.
To be merged after 1.1.1 release.

@Jaguar0625 , feedback fixes pushed. Ready to review. More PRs comming


Seems to have issues with VenusOS 3.00

Hey,
Any idea if this is working with VEnusOS 3.0?
On my VRM Console it does switch beteween the following two screens within a second from this:

to this:

Is this a v3.00 issue or maybe a setting issue?

Hello,
I haven't had time to update my Venus OS to version 3 yet. So I can't test it or reproduce it.
Question: On which device do you have it installed?
best Regards Rikkert-RS

Thanks for your quick reply!

Another question: How are the sensors connected? Which ones are installed?
Oh yes that often gives problems when Setuphelper packages are installed that interfere with each other.
Which are installed?

My 1 Wire temp sensor is connected to this board and from the board it connects to the RPi:

Ok connection looks ok for me. Pull up resistor is on the extra little Board right?
Sensor is DS18B20?
It seems that the Service is restarting again and again.
Is SSH access possible for you? Then we can get more out through the logfile

Yes, that's the sensor: https://amzn.to/3P2SsNb
This is what I've installed from setuphelper:

This is your package:

Oh i think is see the problem.
The problem at the moment is that the GPIO pin 26 collides with the "RpiGpioSetup"
You can use this both but only with mods in "RpiGpioSetup"
For this have a look at: https://github.com/Rikkert-RS/VenusOS-TemperatureService/issues/5#issuecomment-1556277827
For testing deinstall "RpiGpioSetup" and reboot you device

That make sense! I thought 26 sounded somehow familiar. But since the Relay Board needs to use Pin 26 - do I have the chance to change your package data pin to for example Pin4?

Oh yes we can test something I'll build a pack with pin 4 and I'll create a tag for it. need some time for it 30min or so.
Alternatively it would also work if you connect via SSH and change the pin in the Config.txt
Config file is also possible but then you must always have SSH access.

Looks good but i do not quite understand what has been changed now?
I think the best convenience is to build it into the Venus menu. I am working on it but unfortunately it is a bit more time consuming

I did not change anything else - at least I'm not aware of ;-) . I just tried to re-install the package and change to Pin 4 again - even now the custom names are lost ;-)

But for now this helps a lot and I understand the dependency ... Will you be able to adjust this to make it a little more customizable?

yes as I wrote above it is possible

I think the best convenience is to build it into the Venus menu. I am working on it but unfortunately it is a bit more time consuming

is "RpiTemperature" still installed?

No, the "RpiTemperature" is not installed anymore.
Before restarting (I did change the names):

After Restart, looks good:

OK looks good for the moment,
i try to update my venus OS the next days and make the pin configurable in V0.6
You can see this in the setup helper when a new version is available
have a nice evening
Rikkert-RS

Thank you so much! :-)


Official Progress Bar Component

Feature Request
Ionic Info
cli packages: (C:\Users\<Username>\AppData\Roaming\npm\node_modules)

    @ionic/cli-utils  : 1.19.2
    ionic (Ionic CLI) : 3.20.0

global packages:

    cordova (Cordova CLI) : 8.0.0

local packages:

    @ionic/app-scripts : 3.1.9
    Cordova Platforms  : android 7.0.0
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.1.1
    Node              : v8.9.1
    npm               : 5.5.1
    OS                : Windows 10

Environment Variables:

    ANDROID_HOME : C:\Android\sdk

Misc:

    backend : legacy

Describe the Feature Request
In my opinion Ionic needs an official progress bar. Like all other UI frameworks have.
Describe Preferred Solution
A progress bar aka bar-graph or loading-bar with several options like

horizontally or vertically oriented
with or without display the value (e.g. percentage)
easy to set color (color attribute)
responsive / ionic like.
An indeterminate mode. (optional: buffer, query)
(optional) Clickable on the bar to set the value by user input.
Easy change the width of the bar graph.

Use cases:

Display an installation or download progress.
Video or audio position.
User input.

Describe Alternatives
There are tutorials on the net, how to create you own progress bar.
This is a basic component for an UI framework. It's time to finally implement it.
Additional Context
What is the best progress bar for ionic? I can't say it. There are so many different features and styles. I'm sure we all know how a basic progress bar looks like. Maybe let you inspire by the angular material progress bar.

Duplicate of https://github.com/ionic-team/ionic/issues/6669
(also see https://github.com/ionic-team/ionic/pull/12010)

Thanks for the issue! Closing as a duplicate of #6669. This is definitely something we want to get added but we're currently working hard to get v4 out. Let me know if it's something you'd like to submit a PR for and I can help out as much as possible. 🙂


Feature: Selective Modules

Closing as rollout model of Badger has changed from the original planning of this.


Revert "internal/distro/rhel8: un-exclude subman from edge"

We needed this patch to demonstrate RHC capabilities but we don't want to ship such a huge dependencies tree with subman/dnf - for 8.4 we'd revert this and revisit the dep tree.
@nullr0ute @gicmo ptal

The 8.4 beta freeze is passed, so any changes we merge to master will no longer land in 8.4. If you still need this, feel free to reach out to me directly and I can walk you through the process.

Based on side-channel discussion: I don't think this is needed any more? Should we close it?


Fix readme

Yes.Its need. =)

This is just undoing what 0589807 commit did?

This read is a bit wrong... where is the main alias?

Fixed!


ESP32S3: Slow connection to Mesh WPA2 WiFi network (IDFGH-11520)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[x] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hi All,
I recently upgraded a project from the ESP32-WROOM-32D to the ESP32-S3-WROOM-1U-N16R2 module and I am experiencing a slower WiFi connection times.
I am using platformIO and everything is up to date: framework-espidf @ 3.50101.230828 (5.1.1)
In my code I save the last BSSID and channel and use it for the next connection, this is my code:
uint8_t bssid[6] = {
	readNVSIntValue(TAG_NVS, "bssid1"),
	readNVSIntValue(TAG_NVS, "bssid2"),
	readNVSIntValue(TAG_NVS, "bssid3"),
	readNVSIntValue(TAG_NVS, "bssid4"),
	readNVSIntValue(TAG_NVS, "bssid5"),
	readNVSIntValue(TAG_NVS, "bssid6")
};
uint8_t channel = readNVSIntValue(TAG_NVS, "wifi_channel");

wifi_config_t config;
ESP_ERROR_CHECK(esp_wifi_get_config(WIFI_IF_STA, &config));

memcpy(config.sta.bssid, bssid, sizeof(config.sta.bssid));
config.sta.bssid_set = 1;
config.sta.channel = channel;
config.sta.scan_method = 0;

ESP_ERROR_CHECK(esp_wifi_set_config(WIFI_IF_STA, &config) );

This is my log:
I (1663) WIFI: WIFI_EVENT_STA_START
I (1667) wpa: BSS: Add new id 0 BSSID xx:xx:xx:xx:xx:xx SSID 'SSIDName' chan 1
I (1674) wifi:new:<1,0>, old:<1,0>, ap:<255,255>, sta:<1,0>, prof:1
I (2403) wifi:state: init -> auth (b0)
I (2406) wifi:state: auth -> assoc (0)
I (2410) wifi:state: assoc -> run (10)
I (2425) wifi:connected with SSIDName, aid = 1, channel 1, BW20, bssid = xx:xx:xx:xx:xx:xx
I (2426) wifi:security: WPA2-PSK, phy: bgn, rssi: -45
I (2429) wifi:pm start, type: 0
I (2431) wifi:set rx beacon pti, rx_bcn_pti: 14, bcn_timeout: 25000, mt_pti: 14, mt_time: 10000
I (2441) WIFI: WIFI_EVENT_STA_CONNECTED

Is the ~730ms added related to PMK generation?
Could it be related to: #3784?

@andreywis what's the IDF commit you are using?

Hi @kapilkedawat, I am using platformio 6.4.0 - https://github.com/platformio/platform-espressif32/releases/tag/v6.4.0
It uses esp-idf release 5.1.1 - https://github.com/espressif/esp-idf/releases/tag/v5.1.1
I believe the commit is https://github.com/espressif/esp-idf/commit/e088c3766ba440e72268b458a68f27b6e7d63986

Just upgraded to 5.1.2 and it's a lot faster now to connect.
Since I am using platformIO I added this to the platformio.ini:
platform_packages =
framework-espidf @ https://github.com/tasmota/esp-idf/releases/download/v5.1.2-org/esp-idf-v5.1.2-org.zip
My current consumption in deep sleep now is 94uA, it was 18uA only. But it's probably not related to this issue.


Rating and toast

Rating bar with 5 Stars
4 types of Toast (success, info, warning and error) with JS
Added component css file after documentation css for every element.

Your ui looks very clean and easy to use. Components are also relevant to use case and easy to place.

Thankyou :)


⚡  adding support for ain bipolar mod

This change allows the use of bipolar signals in the Analogue input (ain)
The main change is in the read_voltage method of the AnalogueInput class in europi.py. It considers the Min and Max Voltage provided in the constructor.
In order to test this, the test_AnalogueInput.py was created and the mock_hardware.py modified to mock analog inputs.
Now, the EuroPi ain circuit has a non-inverting input with 1.22 gain. This means that the wide range of the ain is 0 to 12.22V. To make the pytest doable with mock hardware, the default value of INPUT_CALIBRATION_VALUES in the europi.py needs to change to  [0, 64354] (roughly the range of 0 to 12V). Likewise, the test needs to specify an input of min/max of 0 to 12 and -6 to 6 V.
I did not test this on an original EuroPi, instead I used a remix that allows bipolar input. The Calibration script should work as long as the minimum voltage is used instead of Ground (or disconnected).
Not sure if the implementation of the test is ok, please make remarks if needed

Hey @luisgongod thanks for this submission. I know that there was some discussion of this change in the discord, but I'm trying to remember the full context. Specifically the details around this:

I did not test this on an original EuroPi, instead I used a remix that allows bipolar input.

I want to make sure that we have all the details recorded in this issue so that we don't have to dig through discord history.
As to the code changes themselves, they largely look ok. The changes on the tests side look good to me. I'm a little wary of the calibration changes. These will need more testing.
Finally, I ran the automated checks and the formatting check failed. We require that the core library is formatted using the black code formatter. If you don't have this set up, let me know and I can help you out, or make the formatting changes for you.

This is now very out of date with the firmware and calibration process, so I'm closing it for now. Bipolar input is planned for the EuroPi X so we can re-open this or use its findings when appropriate!


URL isn't highlighted (selected) when clicking into the address bar

Most browsers make it so that when you click/tap the address bar, the entire URL is automatically selected so you can just start typing and replace what's in there.
As it is now, you'd have to hold the backspace down for a long time, or manually select it all (requiring extra steps).

related: https://github.com/god-s-perfect-idiot/metro-browser-native/issues/13

fixed Thanks! @TheOnlineUser


AllJoyn Explorer: slow to show info after clicking an adapter or a device

I am using Win 10 Pro Build 10074 64 bit and AllJoyn Explorer <IP_ADDRESS>.
I have AllJoyn DSB (dummy virtual device and/or ZWave sample) running, on the same Win 10 Pro build. When I open AllJoyn Explorer, it finds the adapters and devices pretty fast. However, when I click the adapter or the device, it takes "A LONG TIME" for the AllJoyn Explorer to respond to get to the target information page.
Is this normal? This is too slow to be usable in real scenario.
Thanks,
Chih-fan

The behavior is not consistent.  Sometimes, I am getting this. Sometimes, I am not. So I am closing my own issue now.


react native document library then this problme showing

Invariant Violation: requireNativeComponent: "RNPdfScanner" was not found in the UIManager.

I think you meant to open this issue in a different repo.


FileNotFoundException thrown when LiteDatabase use GetCollection

Hi,
II upgraded to version 2.0 RC LiteDB but the problem with the journal.db FileNotFoundException still persists when I debug. Now the exception was throw when I try to retrieve the data with the function GetCollection(). I use the correct connection string found in the wiki page for skip the journal but does not change.
 ConnectionString = string.Format("filename={0}; journal=false", ConfigurationManager.AppSettings["dbpath"]);
Thx you

Hi, now rc2 was released. Please, update your datafile that must fix this.


Can NNI be used in TF Estimator?

Environment:

NNI version: latest
NNI mode (local|remote|pai): local
Client OS: Ubuntu
Server OS (for remote mode only):
Python version: 3.7
PyTorch/TensorFlow version: TensorFlow-gpu 1.14.0
Is conda/virtualenv/venv used?: yes
Is running in Docker?: no

Log message:

nnimanager.log:
dispatcher.log:
nnictl stdout and stderr:

I want to add the intermediate result to the Tensorflow training, but since the code is executed in estimator.train, I don't know how to get the intermediate result.
logging_hook = tf.train.LoggingTensorHook({"loss": total_loss}, every_n_iter=10)
I see that the code has a loss output like the above, but I don't know how to make nni get the loss

SessionRunHook can be inherited and you can implement some hook like LoggingTensorHook and put nni.report_intermediate_result into it. You can refer to the source code of LoggingTensorHook for reference.


Watch for changes to content

Implements and depends on:

[x] https://github.com/alphagov/digitalmarketplace-utils/pull/193

Adds https://pypi.python.org/pypi/watchdog as a requirement to make watching for file changes use system events, which is faster

I might be missing something but where is watchdog used here? I can see it included as a dependancy but can't see the module imported anywhere.
Also, if it's to be used outside of tests it should probably be in requirements.txt which the others include.

Ah, I see, the Werkzeug reloader will use it if it's installed:
http://werkzeug.pocoo.org/docs/0.10/serving/?highlight=run_simple#reloader
Another question: why remove Flask-script if we're still using Manager from it?

@tombye You're right about the dependency on Flask-script, although it does get imported by dmutils so nothing breaks. But I think https://github.com/alphagov/digitalmarketplace-utils/pull/195 cleans things up a bit.

Have flask init init the manager
Depends on and implements alphagov/digitalmarketplace-utils#195

I've added some updates to brings in the changes from alphagov/digitalmarketplace-utils#195 but with a version of init_manager what returns a Manager instance (7db91ec).

:+1:


Too many backslashes stripped in documentation generation

I've noticed the generated documentation for to_csv strips backslashes in the newline character

line_terminator : string, default ‘n’

I thought it was a case of slash stripping, but I checked the code and it's double backslashed there - so the doc generation is a bit aggressive. The IO doc is fine, its code is the same - double backslashed.

Thanks for reporting. The read_csv docs seem to have the same problem (http://pandas-docs.github.io/pandas-docs-travis/generated/pandas.read_csv.html) for the sep and delim_whitespace keywords.
Probably easiest solved by making it raw strings of some of them (r"..."). Interested in trying to fix this?

I looked into this and made the changes to fix it. Just for reference, it also impacted read_table, as it was tied into the docs with read_csv.


Add deployment_review, merge_queue_entry, and repository_ruleset events

Resolves #53

Before the change?

After the change?

Pull request checklist

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

[ ] Yes
[x] No

I've finished typing deployment_review event, but the reviewer property requires #278 😢 It seems like more and more webhook definitions require a solution.

Hey @JamieMagee thank you for the PR, I see that the reviewers property is still marked with a todoand not typed (using dynamic) do you have any plans to update that?

@ilmax I do, but it will require fixing #278 to fully resolve. Please take a look and comment if you have any ideas or suggestions.


How to run PSScriptAnalyzer

I do not seem to get the code inspection using PSScriptAnalyzer to work.
PSSA is installed on my machine, but e g when choosing a file in IntelliJ and selecting the "Analyze -> Inspect Code" menu option, I don't get the same result as when running PSSA from the command line (in an example file, I get no errors in IntelliJ, but in PSSA, I get several remarks).
I have likely missed something obvious?

Hi, maintainer of PSScriptanalyzer here 👋🏻
The intellij extension seems to use PowerShellEditorServices, which has a default set of hard-coded rules here.
Therefore, when you run Invoke-ScriptAnalyzer, it runs all the available rules.
You can use a PSScriptAnalyzer settings file if you want to control which rules to run or just run all of them (in this case just put @{} as the content of the file). PowerShellEditorServices supports settings files as well, but I am not sure if the intellij extension provides a way to exposes this API to the user. cc @ant-druha

Yep, IntelliJ has the ability to fetch and run particular inspections by name. We should implement that.
Currently, you only can run all the PSScriptAnalyzer inspections at once: find action Run Inspection by Name and type PSScriptAnalyzer into the name box.
Interestingly, for me it seems to work properly: the Inspect Code runs the same set of inspections, including the PSScriptAnalyzer.

For now, I consider this as working, and will open a new issue on running a particular inspection.


MacOS kext binding int\uint issue.

Input C/C++ Header
Standard system mach/mach_types.h header
Bindgen Invocation
let bindings = bindgen::Builder::default()
        // The input header we would like to generate
        // bindings for.
        .header("kex_req.h")
        // Finish the builder and generate the bindings.
        .generate()
        // Unwrap the Result and panic on failure.
        .expect("Unable to generate bindings");

Actual Results
Running "cargo test":
   Compiling kext v0.1.0 (file:///Users/jony/kext)
error[E0308]: mismatched types
 --> src/lib.rs:9:16
  |
9 |         return KERN_SUCCESS;
  |                ^^^^^^^^^^^^ expected i32, found u32

error[E0308]: mismatched types
  --> src/lib.rs:12:16
   |
12 |         return KERN_SUCCESS;
   |                ^^^^^^^^^^^^ expected i32, found u32

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0308`.
error: Could not compile `kext`.
warning: build failed, waiting for other jobs to finish...
error[E0308]: mismatched types
 --> src/lib.rs:9:16
  |
9 |         return KERN_SUCCESS;
  |                ^^^^^^^^^^^^ expected i32, found u32

error[E0308]: mismatched types
  --> src/lib.rs:12:16
   |
12 |         return KERN_SUCCESS;
   |                ^^^^^^^^^^^^ expected i32, found u32

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0308`.
error: Could not compile `kext`.

To learn more, run the command again with --verbose.

"cargo test" completed with code 101
It took approximately 3.927 seconds

Expected Results
Original C code looks like this:
kern_return_t kext_start(kmod_info_t * ki, void *d)
{
    return KERN_SUCCESS;
}

kern_return_t kext_stop(kmod_info_t *ki, void *d)
{
    return KERN_SUCCESS;
}

When in rust writing same functions shows missmatched return types (i32 != u32):
#![allow(non_upper_case_globals)]
#![allow(non_camel_case_types)]
#![allow(non_snake_case)]
include!(concat!(env!("OUT_DIR"), "/bindings.rs"));

pub fn kext_start (ki: &kmod_info_t, d:&Fn()) -> kern_return_t {
    return KERN_SUCCESS;
}
pub fn kext_stop (ki: &kmod_info_t, d:&Fn()) -> kern_return_t {
    return KERN_SUCCESS;
}

Whow's KERN_SUCCESS defined? Is it a macro? If so, we try to guess the type, but you can override it with https://docs.rs/bindgen/0.38.0/bindgen/callbacks/trait.ParseCallbacks.html#method.int_macro

@emilio it is not a macro

/usr/include/mach/kern_return.h:72:#define KERN_SUCCESS                 0

/usr/include/mach/kern_return.h:72:#define KERN_SUCCESS 0

That is pretty much a C macro, thus with no type information.f
Bindgen can only guess here unfortunately. You can override the type bindgen will use with something like:
fn int_macro(&self, name: &str, _value: i64) -> Option<IntKind> {
    if name == "KERN_SUCCESS" {
        return Some(IntKind::Int);
    }
    None
}

In a ParseCallbacks implementation.


Fix FastAPI version on generated requirements.txt

Closes #58

Codecov Report

Merging #61 (1d6de2c) into master (b816c51) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #61   +/-   ##
=======================================
  Coverage   83.68%   83.68%
=======================================
  Files           7        7
  Lines         141      141
=======================================
  Hits          118      118
  Misses         23       23

Impacted Files
Coverage Δ

manage_fastapi/config.py
85.71% <100.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 b816c51...1d6de2c. Read the comment docs.


operator [N] curator-operator (0.0.1)

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

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

[ ] Have you considered whether you want to use semantic versioning order?

[x] Is your submission signed?

[ ] Is operator icon set?

Updates to existing Operators

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

Your submission should not

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

Operator Description must contain (in order)

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

Operator Metadata should contain

[x]  Human readable name and 1-liner description about your Operator

The curator retrieves infrastructure utilization based on time period, namespace, and infrastructure parameters.

[ ] Valid category name1
[ ] One of the pre-defined capability levels2

[x] Links to the maintainer, source code and documentation -

Curator operator documentation - https://curator-operator.readthedocs.io/en/latest/about.html

[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

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

@skanthed please fix icon

@mvalarh Yes working on the icon. We will have it soon.

@skanthed Value of 'metadata.annotations.containerImage' not found, please check. This field is required on Operatorhub.
The operator is using some image which must have exactly the same value as 'metadata.annotations.containerImage'.
Please check both values and make them the same.

/retest

/test all

@skanthed If you expand logs from ci/prow run, you get following:
Value of 'metadata.annotations.containerImage' not used in any container, please check.
The operator is using some image which must have exactly the same value as 'metadata.annotations.containerImage'.
Please check both values and make them the same.

/merge possible

@skanthed image tag v0.0.1 does not exist. There is only latest visible for us.

/retest

@J0zi pushed again. And also checked the docker hub. It exists now

/merge possible

@skanthed what's the status with DB secret? Are you going to remove it from operator startup as we already spoke offline?

@J0zi I was on leave for some time. I am back and,  Yes working on removing secret at the time of operator startup.

/test all


gh-1927 Files written by WriteData now use task attempt id in name

Codecov Report

Merging #1938 into develop will decrease coverage by 1%.
The diff coverage is 5%.

@@              Coverage Diff              @@
##             develop    #1938      +/-   ##
=============================================
- Coverage      64.73%   63.73%   -1.01%
+ Complexity      2228     2106     -122
=============================================
  Files            882      760     -122
  Lines          28950    24552    -4398
  Branches        3616     3128     -488
=============================================
- Hits           18741    15648    -3093
+ Misses          8738     7622    -1116
+ Partials        1471     1282     -189

Impacted Files
Coverage Δ
Complexity Δ

...ore/operation/handler/utilities/SortFullGroup.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...e/operation/handler/spark/utilities/WriteData.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...uetstore/operation/handler/AddElementsHandler.java
89.39% <100%> (ø)
0 <0> (ø)
:arrow_down:

...fer/parquetstore/partitioner/GraphPartitioner.java
72.97% <0%> (-10.82%)
0% <0%> (ø)

...fer/parquetstore/partitioner/GroupPartitioner.java
88% <0%> (-4%)
0% <0%> (ø)

...hq/gaffer/hbasestore/retriever/HBaseRetriever.java
78% <0%> (-2%)
0% <0%> (ø)

...r/accumulostore/operation/MultiEntityIdInputB.java

...er/accumulostore/utils/IteratorSettingBuilder.java

...er/accumulostore/utils/IteratorOptionsBuilder.java

...gov/gchq/gaffer/accumulostore/utils/ByteUtils.java

... and 118 more

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 e8a1616...d38b6bb. Read the comment docs.


Tried to initialize from a previous compilation

Running flutter test from Android Studio:
Warning: Tried to initialize from a previous compilation (file:///home/david/git/precept/precept_client/build/test_cache/build/c075001b96339384a97db4862b8ab8db.cache.dill.track.dill), but the file was not self-contained. This might be a bug.

The Dart team would greatly appreciate it if you would take a moment to report this problem at http://dartbug.com/new.
If you are comfortable with it, it would improve the chances of fixing any bug if you included the file /tmp/initialize_from.dillAOAZER/initialize_from.dill.gz in your error report, but be aware that this file includes your source code.
Either way, you should probably delete the file so it doesn't use unnecessary disk space.

Forgot the file:
initialize_from.dill.gz

My best guess for triage is... area-vm?

/cc @jensjoha @johnniwinther

This is easy to reproduce by having the compiler race with itself, e.g. run flutter test simultaneously on the same directory in multiple consoles at the same time.

Thanks for reporting this.
With Hixies comment in mind I'll assume this is a tooling error (as in the tool provides us a partly written dill --- the tool (flutter in this case) shouldn't do that) and I'll close the issue.


KAFKA-14461: Move StoreQueryIntegrationTest to junit5 and fixing logic in a couple of tests for finding active streams

StoreQueryIntegrationTest#shouldQuerySpecificActivePartitionStores and StoreQueryIntegrationTest#shouldQueryOnlyActivePartitionStoresByDefault has a logic to find active partitions by doing a modulo with 2 and comparing the remainder. This can break when a new test is added and since Junit chooses an arbitrary order to run the tests, modulo checks can fail. This PR tries to make it deterministic.
Also, this PR uses Junit5 annotations so that the cluster and input topic can be setup/destroyed once.

Looks like there was a checkstyle failure. Would push a fix for that.

Looks like there was a checkstyle failure. Would push a fix for that.

Fixed the checkstyle issue.

Test failures look unrelated..

``

Thanks for the KIP @vamossagar12 !
I have just one minor comment!

Thanks @cadonna . I made the requested change.

Build failures are not related:
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.mirror.integration.IdentityReplicationIntegrationTest.testReplication()
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationBaseTest.testReplicationWithEmptyPartition()
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.integration.BlockingConnectorTest.testBlockInConnectorConfig
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.integration.ConnectWorkerIntegrationTest.testBrokerCoordinator
Build / JDK 11 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[1] Type=Raft-CoReside, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV1, Security=PLAINTEXT
Build / JDK 11 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[2] Type=Raft-Distributed, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV1, Security=PLAINTEXT
Build / JDK 11 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[5] Type=Raft-CoReside, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV1, Security=PLAINTEXT
Build / JDK 11 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[6] Type=Raft-Distributed, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV1, Security=PLAINTEXT
Build / JDK 11 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[1] Type=Raft-CoReside, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV1, Security=PLAINTEXT
Build / JDK 11 and Scala 2.13 / org.apache.kafka.tools.MetadataQuorumCommandTest.[2] Type=Raft-Distributed, Name=testDescribeQuorumStatusSuccessful, MetadataVersion=3.4-IV1, Security=PLAINTEXT
Build / JDK 8 and Scala 2.12 / kafka.api.PlaintextConsumerTest.testSubsequentPatternSubscription()
Build / JDK 8 and Scala 2.12 / org.apache.kafka.streams.integration.SmokeTestDriverIntegrationTest.[2] false
Build / JDK 17 and Scala 2.13 / org.apache.kafka.connect.integration.ExactlyOnceSourceIntegrationTest.testSeparateOffsetsTopic
Build / JDK 17 and Scala 2.13 / kafka.api.TransactionsExpirationTest.testBumpTransactionalEpochAfterInvalidProducerIdMapping(String).quorum=zk
Build / JDK 17 and Scala 2.13 / kafka.server.DynamicBrokerReconfigurationTest.testTrustStoreAlter(String).quorum=kraft
Build / JDK 17 and Scala 2.13 / org.apache.kafka.streams.integration.SmokeTestDriverIntegrationTest.[1] true
Build / JDK 17 and Scala 2.13 / org.apache.kafka.streams.integration.SmokeTestDriverIntegrationTest.[1] true


WIP [guidelines] [writing_tests.md] Recommended tools

This is a work in progress PR that will contain the contents of the Recommended tools section of the writing_tests.md guideline.

This is a comment to save pictures uploaded to GitHub:

@HarshaNalluru I'll be taking your advice! Thought it might take me some time, but I will do it for sure! Thank you so much!

I'll merge this for now.


Implement design-time construction of the parsing trie

Originally this issue was about lazy initialization. However, we have a solution now that preserves the benefits of upfront initialization fast parsing, simpler code) and lazy initialization (fast construction time): we will create a binary file which describes the trie at design time which will be wrapped into the executable. At runtime, the trie will be constructed in one upfront allocation by reading from the binary file.

Instead of lazy initializing, why don't we: a) store the trie in an array of value type nodes, b) initialize the trie (i.e. the array) from data created at compile time, similarly to how FormattingData initializes the formatting data array (i.e. reads the data from .bin files).

Ah, compile-time, not design time. will adjust accordingly.


Code Error in Example 2

In Example 2 there's a missing $ in "Console.WriteLine(-here-"\t{student...). As is - the Console Output doesn't display the actual first and last name of the students in a Group.
it should be:
Console.WriteLine($"\t{student.LastName}, {student.FirstName}");

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

ID: ae769054-b1de-d528-baab-ea2a275d2a72
Version Independent ID: b5130725-9cfc-5424-53b6-a6b0860aa6fc
Content: Group query results
Content Source: docs/csharp/linq/group-query-results.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

it's also missing in Example 3

There is a "Console.WriteLine" missing in Example 6 before the perentheses "($"Name starts with {scoreGroup.Key.FirstLetter} who scored {s} 85");" just before the inner foreach-loop


Geosuggest is unable to catch google api error

Short description
I was trying to create UI which will rendered differently if geosugget input has an google api error.
Expected results
I expected that should be a call back function such as onGoogleAPIError, so I will know there is an api issue and then render my ui accordingly.
Actual results
There is no way to know if google api call is failed.

Do you have a test setup where there is an Error?

I was catching the google place api error from browser console when my google api call is reach the limit. At that moment, I think if I can redesign my ui to produce an alternative ui if google place api is reach the limit.

So basically, It is good if we output place api response status.

That could be nice to have. If you want a feature like that, feel free to implement! A unified onApiError handler would be nice I think. We use three APIs and there should be at most 1 handler I think.
What do you think?

@ro-ka sure will do.


Click Refresh button changes order of columns in Metadata panel

Description
I move column and click Refresh button.
After that my columns change the position but it is expected that they remain the same.
https://github.com/user-attachments/assets/45e6f410-c7ed-42e1-8f66-5d6916f19e08
DBeaver Version
Community Edition <IP_ADDRESS>407122322
Operating System
Windows 10

Thank you for report

It happened for me not constantly, looks like a floating bug.


[Question] Implementation of OT Changes

Sorry for spamming with issues these days, been starting to play with the current version but at some stage would love some clarifications on the roadmap for future versions.
I am working on a personal wiki system, with a touch of Etherpad in which each page persists automatically over time. In the ideal case, in setting up of a doc, the current text and the revision history are populated from server-side, and one can go back to some point in time and edit that version. If the edited version is not the latest one, a new branch is made. I am also planning to add a tree visualizer for all the branches.
With reference to the current Document class implementation, it makes use of two arrays (done and undone) for each transformations for Undo/Redo. If I am to implement that particular feature here, each transformation should be assigned a unique identifier so that when I do an Undo sequence I would know exactly which revision should I base my following transactions on.
I am not sure if I am trying to reinvent the wheel for this kind of mechanisms here, nor if anything is already covered in our roadmap here. Would appreciate any pointers to the OT mechanism here. Thanks.

It looks like, what you have in mind is similar to what Git does. There you definitely need SHAs to identify changes.
We are currently moving towards to a solution which is similar to what Google Docs does.
There you do not have branches, but every participant (including a server) has a working copy which it applies incoming changes to. The noticable difference to a Git like approach is, that an Undo is not a roll-back (moving backwards) but just applying an inverted operation (moving forwards).
With real-time collab there are some transformations (OT) necessary, so that your undo/redos can be applied to the local working copy correctly.
So, altogether, this is a complex topic :)
We will add a real-time collab implementation in the next months. A first running version will hopefully be ready by end of Nov.
HTH

Yes, I am aware that this is a very complicated topic :) Before I read how Substance works, I have been reading ShareJS. Making OT works right is difficult.
In my use-case though, multiuser session is more like a plus, so while it would be great to have multiple users working on the same document each with consistent local changes, it is not a priority here. However, if our plan here is to implement as a linear revision history then, I think I can change the design docs to reflect that as well. Linear revision history has its advantage of being simpler, too. :)
If I am to pursuit an Etherpad-like application based on current implementation of Substance, I think my best bet is to listen to document:change and persist every transformations that I got there, if we can neglect network availability and concurrency issues? Thanks.

You are right, that would be the way right now.
Be aware that along with the wave implementation we will compress changes so that not every single key-stroke makes a new version. This will be a preparational step, see #17, which I'll implement soon.

BTW, ShareJS has google wave implemented already, however, the source code is not very ...ehm... maintainable. Substance has an equivalent implementation on OT side, but not yet the wave prototocol.

...and that is precisely one of the reasons that I am reluctant in incorporating ShareJS in my own projects. The source code still tends to float around with unclear milestones set. The docs being pretty unclear and obsolete also scares me out.
I think I have one more question too: if I am to persist the undo/redo history, just like what we do for the contents, any existing APIs that I can follow or do we need to change parts of the source code for that? Thanks again.

Not yet, I am sorry. I want to could add DocumentChange.toJSON and fromJSON in a branch if you want.

Would it be part of the Beta 2 milestone? I think it could be useful for others, too.

Totally!

#55

Cool! :heart:

@netsgnut just to let u know, we are rolling out beta4 with real-time support. Will be continued in beta5.

Great! Can't wait to try! :)


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 816304c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-04-15-03-49-52 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-04-15-03-49-52

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add a POLICIES document

This isn't intended to be authoritative, but it's a start!

Fixed, thanks @deivid-rodriguez!

Comments addressed. 🎉

This looks good to merge! @bundlerbot r+


add a manpage

Please describe the problem you're trying to solve
➜ man yazi
No manual entry for yazi

is sad. Yazi has so many features an offline doc is helpful
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
generate a manpage
Additional context
No response

Hi, you can download the Yazi documentation to your local machine and use any tool you like to view them. Let me know if this can meet your needs!
Perhaps we can explore converting these Markdown files into manpage format, but I'm not sure if the result would be ideal or if it's worth it.

you can download the Yazi documentation to your local machine and use any tool you like to view them. Let me know if this can meet your needs!

it's true for much software, yet software usually have their manpage because it's standardized and convenient for the users and the package maintainers. It's not critical but I thought I would raise the point as yazi --help can't cover all the features in yazi.
I try a lot of software for different reasons, and I've grown attached to manpages xD

I second @teto point of view
when I type man whatever I expect to see the help
each app implements things differently
whatever --help
whatever -help
whatever -h
whatever help

having a man page is a decent thing, even if the man page simply mentions the app name and the argument to launch to get the help

having a man page is a decent thing, even if the man page simply mentions the app name and the argument to launch to get the help

It's already there, when you type yazi --help, you will be able to see the application name and the list of launchable parameters:
❯ yazi --help
Usage: yazi [OPTIONS] [ENTRY]

Arguments:
  [ENTRY]  Set the current working entry

Options:
      --cwd-file <CWD_FILE>            Write the cwd on exit to this file
      --chooser-file <CHOOSER_FILE>    Write the selected files to this file on open fired
      --clear-cache                    Clear the cache directory
      --local-events <LOCAL_EVENTS>    Report the specified local events to stdout
      --remote-events <REMOTE_EVENTS>  Report the specified remote events to stdout
      --debug                          Print debug information
  -V, --version                        Print version
  -h, --help                           Print help

I played around with it a bit, and converted these markdown files to manpage format using pandoc. Although a lot of information was lost (anchors, images, videos, etc.), it works. Here is the converted file:
yazi.1.zip
You can preview it by running:
man ./yazi.0

You closed the ticket, but I don't see any issue or pull request about this.
When will it be added ?


Add dmd and ldc to path after installation

@Manuzor Could you check this out?

@Manuzor Could you check this out?

You are right dmd's source is in this repo: https://github.com/PxlBuzzard/ChocoPackages
Please just publish ldc then

You are right dmd's source is in this repo: https://github.com/PxlBuzzard/ChocoPackages
Please just publish ldc then

I added some changes after I merged yours. Let me know if you see a problem with that. Thank you for your submission!

I added some changes after I merged yours. Let me know if you see a problem with that. Thank you for your submission!

By the way, I pushed a new package version <IP_ADDRESS>10118 which is currently awaiting moderation. Should be online soon. Please test it and see if it works as expected.

By the way, I pushed a new package version <IP_ADDRESS>10118 which is currently awaiting moderation. Should be online soon. Please test it and see if it works as expected.


[Request] Set relations on model property

You are able to request model relations off it's properties. For example, if a Post belongs to a User you can do the following:
$user = $post->user;

I'd like to see the reverse made possible - setting relations on a model property. This would make it easier to set relations rather than using something like $post->user()->associate(User::first()) or $user()->posts()->attach(Post::first()) - where the API is different depending on the relationship type.
// Single relations
$post->user = User::first();

// Plural relations
$user->posts = [Post::find(1), Post::find(2)];

This would provide a more consistent (and more readable) way to set the relations of a model.

Pull request please?


Formatter removes line break between implicitly concatenated strings

Couldn't find an exact issue, but I'm almost certain this has been reported already.
Maybe related to #6936
Also see #5893
Line-length: 88
Black (23.7.0):
print(
    "aaaaaaa {:.2f}% "
    "aaaaaaaaaaaaaaaa".format(
        (
            ((df_aaaaaaaaaaa["fak"].isna()) | (df_aaaaaaaaaaa["fak"] == 0)).sum()
            / len(df_aaaaaaaaaaa)
        )
        * 100
    )
)

Ruff (0.0.287):
print(
    "aaaaaaa {:.2f}% " "aaaaaaaaaaaaaaaa".format(
        (
            ((df_aaaaaaaaaaa["fak"].isna()) | (df_aaaaaaaaaaa["fak"] == 0)).sum()
            / len(df_aaaaaaaaaaa)
        )
        * 100
    )
)

Playground: https://play.ruff.rs/126e4315-3c75-4268-b43a-12586c5da779

Black applies special formatting for strings inside binary expressions. We have a "cheap" implementation that works for the most simple cases but it falls short in more complicated settings. I hope to have a fix for this by next week.

We're marking this as an intentional deviation, since the resulting code doesn't exceed line width and reduces vertical spacing. (If you make the strings longer, we do split and format identically to Black.)
Can be closed once documented.


Open folder doesn't work on new window.

Issue Type: Bug
When opening a new window and trying to open a folder in the new window it doesn't work no error is display but the folder is not display either. It's like vscode freezes trying to open the folder in the new window.

Azure Data Studio version: azuredatastudio-insiders 1.42.0-insider (c5fc37c373f965d77d3d860e7756fd9c4c4f0ca0, 2023-02-17T06:29:22.173Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No
Preview Features: Enabled

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz (8 x 2995)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.69GB (0.79GB free)

Process Argv
--ms-enable-electron-run-as-node .

Screen Reader
no

VM
0%

Extensions: none

@octaviordz Thanks for reporting this! I can confirm the repro with both insiders build and official build.

Version: 1.41.2
Commit: 0f0960d0e774e1ce903b79378addff96117c27a6
Date: 2023-02-08T20:15:24.215Z
VS Code: 1.67.0
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 21.6.0

Thank for your response @caohai. I have been seeing this behavior for a while now. As I recall it started as sometimes it would work and other times it won't but now it pretty much fails every time.
The work around that I have been using is in a console/terminal go to the folder and run "azuredatastudio ." . Hope this helps.

It looks like in some cases the workbench files are loading prior to running the sql/setup.js loader.  This causing all the model-view UI components to pull in angular prior to reflect-metadata loading.  I'll look for a way to get reflect-metadata loaded early in the bootstrap sequence.
bootstrap-window.js:259 [uncaught exception]: reflect-metadata shim is required when using class decorators
b	@	bootstrap-window.js:259
(anonymous)	@	bootstrap-window.js:52
emit	@	node:events:526
window.onerror	@	node:electron/js2c/renderer_init:73
error (async)
(anonymous)	@	node:electron/js2c/renderer_init:73
./lib/renderer/init.ts	@	node:electron/js2c/renderer_init:73
__webpack_require__	@	node:electron/js2c/renderer_init:1
(anonymous)	@	node:electron/js2c/renderer_init:1
___electron_webpack_init__	@	node:electron/js2c/renderer_init:1
(anonymous)	@	node:electron/js2c/renderer_init:141
compileForInternalLoader	@	node:internal/bootstrap/loaders:312
compileForPublicLoader	@	node:internal/bootstrap/loaders:252
loadNativeModule	@	node:internal/modules/cjs/helpers:49
Module._load	@	node:internal/modules/cjs/loader:811
c._load	@	node:electron/js2c/asar_bundle:5
executeUserEntryPoint	@	node:internal/modules/run_main:77
(anonymous)	@	node:internal/main/run_main_module:17

While the problem is not resolved, I chose to add a key to the Windows registry, allowing me to open Azure Data Studio in a more practical way. View issue #22225 .

Here's the workaround I use for this issue, which is still occurring in 1.48.0:

open new window from existing window (I use Ctrl+Shift+N)
open devtools by clicking Help > Toggle Developer Tools
now open the folder within ADS

Not sure if related, but I occasionally see these errors
GET vscode-file://vscode-app/c:/Program%20Files/Azure%20Data%20Studio/resources/app/out/vs/code/electron-sandbox/workbench/css/common.css net::ERR_FILE_NOT_FOUND
WARN Via 'product.json#extensionEnabledApiProposals' extension 'github.copilot-chat' wants API proposal 'semanticSimilarity' but that proposal DOES NOT EXIST. Likely, the proposal has been finalized (check 'vscode.d.ts') or was abandoned.
ERR Extension 'github.copilot' appears in product.json but enables LESS API proposals than the extension wants. package.json (LOSES): inlineCompletionsAdditions, terminalDataWriteEvent product.json (WINS): inlineCompletionsAdditions

Update to above:  seems my workaround is no longer working for me.
I'm on Windows 10, but others reported it on Windows 11.

It worked once for me today, then not again.  Console error:
Uncaught reflect-metadata shim is required when using class decorators
DecoratorFactory @ decorators.ts:300


Bug fix, refactor day header code

Bug fix for when you want to show 1 day and display 1 day on screen. Also refactored header configuration by creating a protocol called DayContainer. Now as long as you have UICollectionReusableView that conforms to the protocol it will get configured correctly.

I believe you closed it and I can't merge it.. can you reopen it?