Proposal: Update NavigationView to use ItemsRepeater

Proposal: Update NavigationView to use ItemsRepeater
Summary

Instead of ListView, perhaps NavigationView should use ItemsRepeater as an implementation detail. This shouldn't have any direct impact on users but would help clean up the code.
Rationale

Currently NavigationView has a dependency on ListView in order to display the MenuItems. This becomes a drawback in places where we have to circumvent some of ListView's existing behaviors which results in complicated codepaths and is a way for bugs to be introduced.
Pros to moving to ItemsRepeater:

Greater control over selection behavior which will result in cleaner code. Currently item selection logic is tangled up with the ListView selection logic

Have to keep track of selection state in two different locations. We have to manage and sync selection when it is changed using NavigationView API or when it is changed in the ListView.
Have to deal with callbacks for scenarios where we don’t want selection on invocation
Moving items in and out of a listview also sometimes influences selection

Item-repeater is less policy-heavy. Less restrictions can potentially mean less problems when implementing other future functionality. This is an advantage for Top and Hierarchical Navigation as these features are not based on a single listview and require movement of items. This flexibility also gives us a greater control over functionality such as animations.
Circumvents ListView pre-rs5 bug which manifests in databinding scenarios when adding a container as the root of the datatemplate.

Cons to moving to ItemsRepeater:

Have to re-implement behavior that is handled by listview

Selection
Item Invocation
Container - Item mapping
Markup VS Databinding scenarios

since ItemsRepeater only takes an ItemsSource, we would have to handle the scenarios where either the menuitems are declared in MarkUp or the ItemsSource requires default container wrapping (NavigationViewItem)

AutomationPeer
ItemContainerStyle/ItemContainerStyleSelector

Scenarios where previous app implementations are dependent on ListView functionality (such as SelectorItem.IsSelected) would get broken. Would have to write supporting code for these issues.

Functional Requirements

Important Notes

Open Questions

@ojhad Can you add some details about the pro/cons of this? We talked about it offline and it seems righteous but is it worth the cost?

I think this proposal was addressed with #1683 and can be closed.


expose row-group flush in public api

Which issue does this PR close?
Closes #1626
Rationale for this change
more power for the users to let them decide when to flush row group
Are there any user-facing changes?
not much, just one more method with single line body

Codecov Report

Merging #1634 (3e0dc99) into master (7d00e3c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1634   +/-   ##
=======================================
  Coverage   83.02%   83.03%
=======================================
  Files         193      193
  Lines       55577    55578    +1
=======================================
+ Hits        46145    46147    +2
+ Misses       9432     9431    -1

Impacted Files
Coverage Δ

parquet/src/arrow/arrow_writer.rs
97.66% <100.00%> (+<0.01%)
:arrow_up:

arrow/src/array/transform/mod.rs
86.68% <0.00%> (-0.12%)
:arrow_down:

parquet_derive/src/parquet_field.rs
66.21% <0.00%> (ø)

parquet/src/encodings/encoding.rs
93.56% <0.00%> (+0.18%)
:arrow_up:

arrow/src/datatypes/datatype.rs
66.80% <0.00%> (+0.39%)
: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 7d00e3c...3e0dc99. Read the comment docs.

Thanks again :+1:


Rename PushComplexInstruction to CreateComplexInstruction

This instruction does not push anything on the stack, it converts the last two elements on the stack in a new element. This is similar to instructions as CreateArrayInstruction.

🤷♂ I struggle to know which ones to call what. They almost all push something.

My interpretation is that the push operations push something completely new on top of the existing stack. As soon as something is popped in the operation, it is some kind of conversion. That the result of the conversion is pushed back onto the stack is a side effect of a stack architecture.


Start tagging with version numbers

We haven't been including fermitools-version tags in this repository and should start so that builds are reproducible.

Done as of the patch release.  Closing.


updating for testing

SUMMARY

ISSUE TYPE

Breaking Change
New or Enhanced Feature
Bug, Docs Fix or other nominal change

ADDITIONAL INFORMATION

Image succesfully built and deployed!


Reduce requested cpu and memory resource in FOM openshift deployment config

Describe the Task
In the openshift deployment config, we don't need that much cpu and memory resource, reduce those numbers in the deployment config could save the platform service team some resources.

Acceptance

In each the openshift deployment config yaml file, update the request cpu and memory to a lower number, CPU request could change to 30m, Memory request could change to 0.3Gi

DEV/TEST/DEMO to low request cpu and memory for not wasting platform resource. PROD can set to higher (if needed).

FOM Database deployment config looks like it is entirely missing request/limit settings (or just using defaults) - the limit of 0.25 CPU is most likely far too low.
API/Public/Admin can be sized down, especially public/admin as they are using caddy now.


Quick fix to decklist autocomplete and editing a deck description.

Add pack names into the deck description auto-completion.

Add preview to the edit published deck description form.
tired:

wired:

Deployed to production.


Download tasks_v2.pbtxt first, then update UI with tests to run

The recent change to decouple the front end code from the specific tasks being run can leave the UI looking empty until all the assets are loaded. If the UI can be updated when the tasks_v2.pbtxt is loaded, and if that gets loaded first, it should reduce the time to update the UI.

@mcharleb: I want to clarify how the UI should be updated.
Currently:
When the app is installed and opened the first time. The main screen shows the progress while loading assets, then after sometime (about 30s on my device) it shows the GO button.

Desired:
The app should show the screen with the GO button while loading the assets in the background? Is this correct?

@anhappdev  The reason the screen is blank to start is that the loaded tasks_v2.pbtxt make come from the sdcard and may specify only one task (e.g. Language Understanding) so that is the only task that should be displayed. The app doesn't know what to display until tasks_v2.pbtxt has been loaded since it gets the text to display from that file now (as it should). If we want to show the list of tasks and white background sooner, we need to load tasks_v2.pbtxt first, then update the screen, then download the other files. It seems to take several seconds to establish the download and get the first file.


🌐 Fix code examples in Japanese translation for docs/ja/docs/tutorial/testing.md

Some codes cannot be copied and pasted.
and then, I copied the codes from the English turorial pages;
target pages
https://fastapi.tiangolo.com/ja/tutorial/testing/
copied from pages

https://fastapi.tiangolo.com/tutorial/testing/
https://github.com/tiangolo/fastapi/blob/master/docs/en/docs/tutorial/testing.md

Before View

After View

Awesome, thank you @hirotoKirimaru! :cookie:
And thanks for the review @wakabame :bow:


libvmi: 0.12.0 -> 0.14.0-unstable-2024-08-06, libkvmi: init at 1.1.0-unstable-2023-12-13

Description of changes

Updates libvmi. It was previously broken due to #324693.
Adds libkvmi, a new dependency to libvmi's KVM support.

Things done

Built on platform(s)

[x] x86_64-linux

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review pr 328873".

This will fail until #324693 is merged.

[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[x] (Package updates) No release notes: change is not major.

[x] Fits CONTRIBUTING.md.

Resolves this upstream request.
Add a :+1: reaction to pull requests you find important.

@matthiasbeyer: Hi! I understand if you no longer wish to have anything to do with libvmi, but if you are interested, I would appreciate any comments you may have on this update. Use it with the first three commits from #324693.

Dropping patch: Build failure was fixed upstream.

Hi! Thanks for working on this!
Two options:
The simple one would be, you add a patch (as first patch of the PR) removing me as maintainer (feel free to add my Signed-off-by) and add yourself. After that, do what you want :laughing:
The not-so-simple one, if you don't want to take maintainership is: Split your libvmi: 0.12.0 -> 0.14.0... patch into three patches: One that moves the package to pkgs/by-name, one that formats it and one that updates it.
It is really hard to review the patchset this way, that's why I'd like to see this patch splitted.
Hope that helps! :wink:

The not-so-simple one, if you don't want to take maintainership is: Split your libvmi: 0.12.0 -> 0.14.0... patch into three patches: One that moves the package to pkgs/by-name, one that formats it and one that updates it. It is really hard to review the patchset this way, that's why I'd like to see this patch splitted.

@matthiasbeyer: Thanks for hanging in there! The main Xen PR was merged, so I can finish up with this PR.
As requested, the libvmi update commit has been split in three separate patches. I do plan on maintaining this package, and you haven't been a maintainer since e9304d7, I just requested one final review from you as the sole LibVMI maintainer before the package was orphaned.

Passes nixpkgs-hammering libvmi -e attribute-typo. (libVersion is not a typo.)

Result of nixpkgs-review pr 328873 run on x86_64-linux 1

6 packages built:

libkvmi
libkvmi.dev
libkvmi.lib
libvmi
libvmi.dev
libvmi.lib

Looks like LibVMI doesn't support AArch64 at all. Dropping the platform.

Result of nixpkgs-review pr 328873 run on x86_64-linux 1

6 packages built:

libkvmi
libkvmi.dev
libkvmi.lib
libvmi
libvmi.dev
libvmi.lib


Trivial app segfaults when there are more than two routes

This trivial served app below segfaults immediately when there are more than two routes. It works fine as long as only /route1 and /route2 are present, but as soon as I add /route3, it segfaults.
This does not happen if served is built with build type set to Debug. I have also tried building served with build type set to RelWithDebInfo and running the app in a debugger. The debugger pointed out multiplexer.cpp:139 as the culprit, which I find hard to believe since it's just std::vector::push_back:
_handler_candidates.push_back(
	path_handler_candidate(get_segments(path), served::methods_handler(_base_path + path, info), path));

The app:
#include <served/plugins.hpp>
#include <served/served.hpp>

int main(int, char **) {
    served::multiplexer mux;
    mux.use_after(served::plugin::access_log);
    mux.handle("/route1").get([](served::response &response, const served::request &) {
        response.set_body("1");
    });
    mux.handle("/route2").get([](served::response &response, const served::request &) {
        response.set_body("version2");
    });
    mux.handle("/route3").get([](served::response &response, const served::request &) {
        response.set_body("3");
    });
    served::net::server server("<IP_ADDRESS>", "8080", mux);
    server.run();
    return 0;
}

If I comment out
    mux.handle("/route3").get([](served::response &response, const served::request &) {
        response.set_body("3");
    });

it works fine, but with three or more routes, boom!

Actually this is enough to reproduce:
#include <served/served.hpp>

int main(int, char **) {
    served::multiplexer mux;
    mux.handle("/version/1").get([](served::response &, const served::request &) {
    });
    mux.handle("/version/2").get([](served::response &, const served::request &) {
    });
    mux.handle("/version/3").get([](served::response &, const served::request &) {
    });
    return 0;
}

This looks like a stdlib bug on macOS. Compiling compilers other than system default works just fine.

Hey @krzysztofwos, I can't reproduce this myself and the issue is pretty old now, I'm closing but feel free to reopen if this issue still impacts you.


Enhanced Typescript for error checking, expanded tests to cover changes

Typescript updated to be more modular, which also helps code readability.
Areas checked by automated tests is significantly expanded
All automated tests pass:
https://travis-ci.org/jkbits1/backend/builds/244686232

merged as passes automated tests


Net Billing results do not show $ credits

For the Net Billing option, monthly bill is calculated correctly based on credits earned, but dollar credit amount not reported correctly. See https://sam.nrel.gov/forum/forum-general/3122.

It looks like it could be more of a naming issue than a code issue. Here's what I see running the SAM file from that issue with a different weather file (Blythe, CA):

The credits are being applied immediately during the month when they are earned. "Excess generation $ credit earned" is reserved for months where the credit rolls over to the next month (Net billing with carryover to next month):

Thoughts on name changes that might make this more clear? Or am I missing something about the bug?

I agree there is a labeling problem, and also think it would be helpful to modify some of the monthly output variables to help people understand how SAM calculates the monthly bill for he different billing options:
Excess generation (kWh) for both net energy metering options:
* sum(generation[h]) - sum(load[h]) for net energy metering options

* sum(generation[h] - load[h]) for net billing options

* set to zero for buy all / sell all

Net metering credit (kWh), for net metering only, kWh credit that applies to current month (includes this month's excess generation and any credits rolled over from previous months).
Net metering credit ($), for net metering with $ credits only, shows $ credit that applies in current month.
Net billing credit ($), for two net billing options, shows $ credit that applies in current month, regardless of when it was earned. Or, use that for net billing only, with a separate Net billing carryover credit ($) for net billing with carryover to next month.

Finally had some time to dig into this. I like these categories, are these generally additions or replacements? Some notes are below:
With net metering Excess generation only fits that calculation when there is one TOU period. For example, for the SAM default rates, if sum(generation[period_3]) > sum(load[period_3]) but sum(generation[period_1]) < sum(load[period_1]) at the end of the year, the customer would still receive true-up credits for the excess generation in period 3. I would favor reporting those credits in the Excess generation (kWh) column.
Are there any differences (besides the name) between the Net metering credit column and the existing Excess generation cumulative kWh credit earned column? Or is this closer to Excess generation kWh credit applied?
With Net metering credit ($) I think it makes sense to include the end of year true-up credits in that column for the net metering option, similar to the existing behavior of Excess generation $ credit earned.

@brtietz I agree with your suggestion for Excess generation -- my examples show equations for a simple flat rate.
Excess generation cumulative kWh credit earned, Excess generation kWh credit applied, and Excess generation kWh credit earned were an attempt to make the excess generation credit calculation transparent, but are confusing. What would be useful is to replace those with one Net metering credit column that shows the kWh that are applied in a given month, and perhaps an Excess generation column that shows the excess generation in each month (although that can easily be calculated by subtracting Electricity load from Monthly energy, so maybe not necessary).
I agree with Net metering credit ($) and true-up credit.

Some clarifications on the above for net metering based on Paul and my meeting this morning:

Excess generation kWh applied will be removed. The numbers currently in this category have some errors, and determining how many credits are "applied" in one month can be ambigious
Rename Excess generation cumulative kWh credit earned to more clearly indicate that this shows the annual credits used in the true-up (exact name TBD)
Add a variable showing the cumulative monthly rollover credits, the variable in ur_month.ec_energy_surplus. Users can go to the existing "electricity rate data by tier and period" variables to see more about how these numbers are computed.

Net billing will be updated as outlined above.

@brtietz, are the ur_month.ec_energy_surplus restricted to rollovers in the same periods?

Within a month, yes. Between months, if the period times change, the credits can be shifted between periods using the code in https://github.com/NREL/ssc/blob/c1a40c53f91bca6949dee63df9ed6aae672fbe71/shared/lib_utility_rate_equations.cpp#L837
I was thinking that having a sum of the hours available in ur_month.ec_energy_surplus would be a useful indicator to tell the user whether they needed to dig into the "electricity rate data by tier and period" outputs for a given month.

@cpaulgilman For net billing, does it make sense to make Excess generation
sum((generation[h] - load[h]) > 0) ?
This equation above ensures that:
(excess generation) * (sell rate) = (excess $ earned)
for a flat sell rate. (I still need to make the updates for net billing $, just reporting the odd hybrid in front of me)
SAM 2020.2.29 reports net monthly kWh in this category for both net metering and net billing.  It's worth noting that your equation above has the same result for net metering and net billing due to the associative property, and I want to make sure that's what we want.
Changing reporting the sum of the surplus credits would mean excess generation changes based on the utility rate option, so I'm open to adding a new variable for the above output instead.

Additional bug found while developing these changes: In the help, "net billing with carryover to the next month" says "For months with excess generation, the dollar value of the excess is credited to next month's bill." and "Any credit remaining at the end of December is credited to the December electricity bill."
However, in develop and SAM 2020.2.29 credits earned in December are never applied. Should these be applied in December or January?

https://github.com/NREL/SAM-documentation/blob/master/UtilityRates/NetBilling/Updates_2016.7.28/Workbooks/Tiered TOU single meter Net Billing %24 rollover.xlsx row 8764 column AG suggests credits earned in December should be applied to the "December year end payout"

@brtietz I think it makes sense to show the same excess generation (kWh) value regardless of the billing option.
I think the December credits should be applied to January now that we can do that, although that would mean January in Year 1 starts with a credit.
Was the "December year end payout" required before we could apply the December credits to January?
Finally, should the "Month for credit rollover" option be available for the Net billing with carryover to next month option?

@cpaulgilman Do you have an example of a utility that does continuous rollover like that? The only one I could find on this topic was Arizona Public Service: https://www.sunrun.com/utility/az/aps/how-to-read-your-aps-nem-bill
https://www.aps.com/en/Residential/Service-Plans/Compare-Service-Plans/Renewable-Energy-Riders#RCPExportRider
It looks like they'll roll over Dec to Jan if the credit is < $25, and cut a check if it's >= $25.
I think the "Month for credit rollover" option should be available if we go with the code in #483, but we should take it away if we decide to do continuous rollover.


Updated cordova section to include asset install

The Cordova section in the Overview did not thoroughly explain where to get the CSS and JS files for manual install.
Resolves driftyco/ionic#4433

Thanks!


Asset hugo_0.102.0_Linux-64bit.deb has files in wrong location

Thanks for creating and releasing Hugo!
The latest release asset for Debian, hugo_0.102.0_Linux-64bit.deb, has its built files in the wrong location:
$ dpkg-deb -c hugo_0.102.0_Linux-64bit.deb
-rwxr-xr-x root/root     11347 2022-08-29 02:28 ./LICENSE
-rwxr-xr-x root/root     10927 2022-08-29 02:28 ./README.md
-rwxr-xr-x root/root  55280808 2022-08-29 02:35 ./hugo

If one were to install this DEB file, it would place the hugo binary in the root directory.  It should be something like /usr/local/bin/hugo instead.  Similarly, LICENSE and README.md should be non-executable and placed in something like /usr/local/share/doc/hugo.
Previous releases using the old build process did exactly that.

I haven't investigated any other build assets: similar issues may exist with the new build setup.

I'm pretty sure this is a "Deb thing" only, thanks for the report, I will fix this tomorrow.

Thanks for the quick response!  Yes, it seems that all DEB versions (64bit, ARM64, _extended 64bit, ARM64) have this particular issue.

Ah. Am assuming this is why my usual GitHub Action for building the site (with Embedded Dart Sass) failed, with a hugo: command not found error, after it installed hugo_extended_0.102.0_Linux-64bit.deb:
check_suite_focus=true#step:8:8).sh: line 1: hugo: command not found
Error: Process completed with exit code [12](https://github.com/brycewray/hugo_site/runs/8060357563?

But, as Scarlett O’Hara would remind us, “Tomorrow is another day.” 😄

@brycewray yea, thinking about it I should have deleted those Debs when this issue was raised, that would have produced less noise ...

Just a note, I checked one of the deb archives in v0.101.0;
root@ubuntu:~/hugo/debtest# dpkg -c hugo_0.101.0_Linux-ARM.deb
drwxr-xr-x 0/0               0 2022-06-16 07:51 ./usr/
drwxr-xr-x 0/0               0 2022-06-16 07:51 ./usr/local/
drwxr-xr-x 0/0               0 2022-06-16 07:51 ./usr/local/bin/
-rwxr-xr-x 0/0        47710208 2022-06-16 07:26 ./usr/local/bin/hugo

Just the binary, no "extra files", which is what I'm going to do here.


Solver leaving a lot of CP when more Quality can be done.

Required Information
Class: Alchemist
Level: 83
Craftsmanship: 2606
Control: 2457
CP: 507
Recipe Name: Panaloaf Yeast
Recipe Level: 83
Solver Seed:<PHONE_NUMBER>188
Expected Behaviour
100% HQ and Progress
Actual Behaviour
Stops at 98% and leaves a huge chunk of CP left. 367 CP out of 507.
Steps To Reproduce
Start Quality set to 1860 due to using HQ Palm Sugar.

There's a strange rounding error right now where it assumes this to be 100% HQ for some reason, I'll get a fix in soon.

There's a strange rounding error right now where it assumes this to be 100% HQ for some reason, I'll get a fix in soon. For now, you can work around this by adding some HQ% margin on the right.

Thanks! I'm not sure how the HQ margin works. I'll give that a go though :)


[BUG] Iterating over memoryview in pure python mode is not optimized

Describe your issue
Pure python version of the code example from cython documentation https://cython.readthedocs.io/en/latest/src/userguide/memoryviews.html#indexing
seems to yield not-optimized C code. Following code
import numpy as np

def add_one(buf: cython.int[:,:]):
    for x in range(buf.shape[0]):
        for y in range(buf.shape[1]):
            buf[x, y] += 1

# exporting_object must be a Python object
# implementing the buffer interface, e.g. a numpy array.
exporting_object = np.zeros((10, 20), dtype=np.intc)

add_one(exporting_object)

Yields following annotated code (for brevity not showing actual C code, but the color gives the hint):

For comparison the current Cython example
import numpy as np

def add_one(int[:,:] buf):
    for x in range(buf.shape[0]):
        for y in range(buf.shape[1]):
            buf[x, y] += 1

# exporting_object must be a Python object
# implementing the buffer interface, e.g. a numpy array.
exporting_object = np.zeros((10, 20), dtype=np.intc)

add_one(exporting_object)

Yields following annotated code:

OS
MacOS
Python version
3.9
Cython version
master

My bad. I forgot to import cython :-/.


Installation failed on cloud service.

Hi,
I am using Aliyun cloud service to install environment for HyperEnclave. I followed the installation guide step by step, from reinstalling the kernel to 5.10.0, to the last step starting the service. I ran dmesg and got the following result:
command: ~/hyperenclave/scripts# dmesg | grep hyper
[   91.378659] hyper_enclave: loading out-of-tree module taints kernel.
[   91.378717] hyper_enclave: module verification failed: signature and/or required key missing - tainting kernel
[  254.543904] Modules linked in: iptable_nat nf_nat nf_conntrack nf_defrag_ipv6 nf_defrag_ipv4 libcrc32c xt_addrtype iptable_filter bpfilter br_netfilter bridge stp llc overlay hyper_enclave(OE) intel_rapl_msr intel_rapl_common intel_powerclamp nls_iso8859_1 crct10dif_pclmul ghash_clmulni_intel cirrus aesni_intel crypto_simd drm_kms_helper cryptd cec glue_helper rapl rc_core fb_sys_fops syscopyarea input_leds sysfillrect sysimgblt serio_raw pvpanic qemu_fw_cfg mac_hid sch_fq_codel drm ramoops reed_solomon pstore_blk pstore_zone efi_pstore ip_tables x_tables autofs4 nvme_tcp nvme_rdma rdma_cm iw_cm ib_cm ib_core nvme_fc nvme_fabrics virtio_net nvme net_failover i2c_piix4 crc32_pclmul failover psmouse nvme_core floppy [last unloaded: hyper_enclave]
the boot_image macro in dmesg is this (not sure if this is right?):
Command line: BOOT_IMAGE=/boot/vmlinuz-5.10.0-051000-generic root=UUID=05b5f359-2108-4208-9ecd-8eff3357a7cb ro vga=792 console=tty0 console=ttyS0,115200n8 net.ifnames=0 noibrs iommu=pt nvme_core.io_timeout=4294967295 nvme_core.admin_timeout=4294967295 memmap=8G$0x100000000 iommu=off intremap=off no5lvl quiet
I can not understand why this happens? I guess it might be related to the fact that I was using a cloud service, which may have a very different boot up measurement for TPM. I wonder if you authors have tried installing it on commercial cloud service before?

Hi @pyz-creeper , thanks for your interest on HyperEnclave.
The message shown in the log you provide:

[ 91.378717] hyper_enclave: module verification failed: signature and/or required key missing - tainting kernel

indicates that the OS vendor has enabled the kernel module signature verification](https://www.kernel.org/doc/html/v5.1/admin-guide/module-signing.html) on your kernel, and it won't load any module that hasn't been singed.

I wonder if you authors have tried installing it on commercial cloud service before?

We are now working on deploying HyperEnclave in virtualized environment.
For now, suggest do not try HyperEnclave on cloud environment. Currently HyperEnclave needs to be deployed on bare-mental hardware. You can try it on your the PC or server.

Thank you @Bonjourz . I will follow your suggestions and run my experiment on a local PC.


Revert "fix(deps): update dependency @freecodecamp/loop-protect to v3"

Reverts Guidetteliv11/freeCodeCamp#1

Kka

Heyy.


build: install TensorRT directly in dockerfiles

@angersson I see TRT available in https://developer.download.nvidia.cn/compute/machine-learning/repos/ubuntu1804/x86_64/ so there is no need to install a local repo.
Unfortunately, the case is not true for ppc64le. Ping @tjakob to double-check.

It is my understanding that NVIDIA has no current plans to upload TensorRT packages for ppc64le.
So the ppc64le check will need to stay.

It is my understanding that NVIDIA has no current plans to upload TensorRT packages for ppc64le.
So the ppc64le check will need to stay.

@tjakob TRT5.1 GA has ppc64 packages: https://developer.nvidia.com/tensorrt-5136ubuntu-18042ppc64le-gnucuda-101cudnn75.tar.gz

Ping @aaroey; I see 220e8e059a2514567b33cc2614fea0935bd3f631 and I presume this PR is obsolete.

@pooyadavoodi They only have .tar.gz packages for ppc64le (no rpms or debs). So additional work would need to be done for TensorRT and ppc64le (pulling that tar file, and extracting it to the correct location).

@byronyi that commit was rolled back and I'll redo that later.

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!


Fix changelog entry and move to 2.x section

Signed-off-by: Anshu Agarwal<EMAIL_ADDRESS>
Description
This PR fixes changelog entry. The task that this changelog entry tracks needs to in 2.x. release.
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #6057 (b75dffe) into main (ebb5813) will decrease coverage by 0.03%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6057      +/-   ##
============================================
- Coverage     70.75%   70.73%   -0.03%
+ Complexity    58721    58692      -29
============================================
  Files          4774     4774
  Lines        280903   280903
  Branches      40582    40582
============================================
- Hits         198760   198699      -61
- Misses        65831    65877      +46
- Partials      16312    16327      +15

Impacted Files
Coverage Δ

...adonly/AddIndexBlockClusterStateUpdateRequest.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...ava/org/opensearch/action/NoSuchNodeException.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...rch/client/transport/NoNodeAvailableException.java
28.57% <0.00%> (-42.86%)
:arrow_down:

...readonly/TransportVerifyShardIndexBlockAction.java
9.75% <0.00%> (-41.47%)
:arrow_down:

...opensearch/persistent/PersistentTasksExecutor.java
22.22% <0.00%> (-40.75%)
:arrow_down:

...indices/readonly/TransportAddIndexBlockAction.java
20.68% <0.00%> (-37.94%)
:arrow_down:

...regations/metrics/AbstractHyperLogLogPlusPlus.java
51.72% <0.00%> (-37.94%)
:arrow_down:

.../indices/readonly/AddIndexBlockRequestBuilder.java
0.00% <0.00%> (-33.34%)
:arrow_down:

...min/cluster/snapshots/get/GetSnapshotsRequest.java
52.63% <0.00%> (-31.58%)
:arrow_down:

...search/aggregations/metrics/ParsedPercentiles.java
48.42% <0.00%> (-31.58%)
:arrow_down:

... and 476 more

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


Differential loading is not working with ng serve / ng build --watch command

Differential loading is not working with ng serve/ng build --watch command. I am using angular 8.

First one is generated when I build the project without watch mode.

This one's with watch mode and it does not create differential bundles. Is this expected behavior or bug?

Since we want to speed up the development round trip time, it's intentional to not have differential loading on ng serve.
@alan-agius4 can comment about ng build --watch.
Is there any particular reason you need differential loading on ng serve?

Hi, yeah it is expected that —watch will not trigger differential loading, the main reason for this that we are running builds sequentially which would make the watch experience slower.

Alerts
I don't have a very good reason for asking this.
I am in process of migrating angularjs project over to CLI. Our Frontend is served using Rails assets pipeline and Frontend has very tight coupling with Rails. Reason to use watch is so that I can still get reasonable updates when I change Angular code but at the same time don't have to modify my hand-written script tags in Rails ERB template. I understand why this should not be the case but it would be nice if I can provide a flag knowing I will have slower experience to get the same bundles during this process.

Just to make sure I understand, you use this process for development, right? Why is differential loading crucial in the process? Why you can't use only the ES2015 output?

Purely development.
On Thu, Jun 27, 2019 at 4:11 PM Minko Gechev<EMAIL_ADDRESS>wrote:

Just to make sure I understand, you use this process for development,
right? Why is differential loading crucial in the process? Why you can't
use only the ES2015 output?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/angular-cli/issues/14911?email_source=notifications&email_token=AAET7V4WFB7VP3HHRS4MIOTP4VCIZA5CNFSM4H3VREHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYYTWLY#issuecomment-506542895,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAET7V2SLATIPZ4UUS3V3OTP4VCIZANCNFSM4H3VREHA
.

--
Software Engineer | Mode |<EMAIL_ADDRESS>Front-End scripting

Hi @mgechev, the main reason at least for me is that we test locally ie11 before we deploy via browserstack. Is there a way to do this? I tried ng serve --prod but it still doesn't work

@cafesanu you can build the app (ng build --prod) and serve it with any static web server (for example serve, serve -s dist/[project-name])

Thanks @mgechev . Yeah that's is what we've been doing. The only issue is that the dev cycle gets really slow this way as every single change needs the app to be built from scratch, so I end up wasting a lot of time (plus no sourcemaps, although I thing --source-maps would do the trick).
I totally understand your comment "we  want to speed up the development round trip time", but the ironic part is that when we are testing ie11 (not all the time) then the dev cycle becomes painfully slow. I think it should of course not be the default with ng serve, but a flag, something like ng serve --diff-loading would be super nice


S3 Bucket KMSKey validation is missing

if you want to create a s3 bucket via crossplane ex.:
apiVersion: s3.aws.crossplane.io/v1beta1
kind: Bucket
metadata:
  name: xxxx
spec:
  forProvider:
    accelerateConfiguration:
      status: Enabled
    acl: private
    locationConstraint: eu-central-1
    objectLockEnabledForBucket: false
    serverSideEncryptionConfiguration:
      rules:
      - applyServerSideEncryptionByDefault:
          kmsMasterKeyId: hioufhsdfjkdsfkjsdhfkksdjhkjds
          sseAlgorithm: aws:kms

crossplane will create the bucket with an kmsMasterKeyId but this is not correct - the "KeyId" is in AWS the KeyId in full ARN at the end the bucket is not usable because of missing kmsMasterKeyId reference - also via aws console you see following validation error:

did we need to change the kmsMasterKeyId to kmsMasterKeyArn in crossplane crds ?

The documentation of the field:
// AWS Key Management Service (KMS) customer master key ID to use for the default
// encryption. This parameter is allowed if and only if SSEAlgorithm is set
// to aws:kms.
//
// You can specify the key ID or the Amazon Resource Name (ARN) of the CMK.
// However, if you are using encryption with cross-account operations, you must
// use a fully qualified CMK ARN. For more information, see Using encryption
// for cross-account operations (https://docs.aws.amazon.com/AmazonS3/latest/dev/bucket-encryption.html#bucket-encryption-update-bucket-policy).
//
// For example:
//
//    * Key ID: 1234abcd-12ab-34cd-56ef-1234567890ab
//
//    * Key ARN: arn:aws:kms:us-east-2:111122223333:key/1234abcd-12ab-34cd-56ef-1234567890ab
//
// Amazon S3 only supports symmetric CMKs and not asymmetric CMKs. For more
// information, see Using Symmetric and Asymmetric Keys (https://docs.aws.amazon.com/kms/latest/developerguide/symmetric-asymmetric.html)
// in the AWS Key Management Service Developer Guide.

Its name is also the same in SDK. So, I think this is the expected behavior. Feel free to reopen if you think that's not the case.


Add quantity selection to order form

Rather than having quantity selection on details page, we can put an option in the order form for more than one of the same item. Similar to how Amazon does this on their checkout page.

This is under discussion. I have mockups of the details page with the quantity dropdown and without. We will need to come to a group consensus on this.

We will keep the quantity selection on the stock details page, but we will also allow for the user to change the quantity during the ordering process.


How should branch filtering work in combination with reltables?

Let's say I have a main DITA Map with content like:
<topichead navtitle="Windows Installer" keyscope="windows-installation">
  <ditavalref href="topics/getting_started/install_hwu_windows.ditaval">
    <ditavalmeta>
      <dvrResourceSuffix>-hwuwindows</dvrResourceSuffix>
    </ditavalmeta>
  </ditavalref>
  <topicref href="topics/getting_started/install_select_parallel.dita" keys="install_paralel"/>
  <topicref href="topics/getting_started/install_specify.dita" keys="install_specific"/>
  <mapref href="container_reltable.ditamap" processing-role="resource-only" type="map"/>
</topichead>

The "container_reltable.ditamap" contains a reltable which links two of the topics together:
<reltable>
  <relrow>
    <relcell linking="sourceonly">
      <topicref
        href="topics/getting_started/install_select_parallel.dita"/>
    </relcell>
    <relcell linking="targetonly">
      <topicref href="topics/getting_started/install_specify.dita">
        <topicmeta>
          <linktext>Run on cluster</linktext>
        </topicmeta>
      </topicref>
    </relcell>
  </relrow>
</reltable>

But because of the branch filtering the two topics get renamed so there will be no related links between them in the published output.
How could we make this work? Is this a problem with the specification not covering this aspect or with the DITA OT branch filtering implementation?
I also tried in the reltable to use keyrefs to the keys defined for the topics (having a keyscope defined on the entire "Windows Installer" chapter. But this does not seem to work, the reltables seemed to be globally processed.
Link to a sample DITA project.

I would say that related links that use direct URI references are basically undefined--because it is impossible to know the use context intended there is no way for a processor to do the right thing because it is impossible to know what the right thing is in that case.
That is--before DITA 1.2 relationship tables were broken for this reason (that is, they never worked and never could work with direct URI references to topics).
Thus in this case whatever the OT does is equally correct and incorrect.
Topicrefs from relationship tables should only use key references and those references must be to normal-role topicrefs, otherwise the same lack of use context condition holds. The only improvement with references to resource-only keys is that those keys can at least be in a scope so the system has the ability to generate correct scope-augmented key references.
So it's not that this case is not addressed by the DITA spec, it's that it can't be addressed because there is no single behavior that would be correct. That is, it's a "don't do that" condition.
That said, the OT could apply a default behavior of rewriting URI references within a filtered branch to reflect any scope-defined resource augmentation. That would probably produce the intended effect most of the time (because people making direct URI references have already come to depend on the HTML generation behavior which results in some links working "by accident" but in the way that authors expect (or don't realize is only working by accident).

I agree, unfortunately using keyrefs (which is the most elegant and non-ambiguous approach) in the reltable still does not work.

Adding my vote. For me (and I suspect others) branch processing of reltables would be very useful. As an example, I use branch processing to generate branches of help topics for several different LMSes. Most of the content in the branch is common among all the LMSes (thus, branch processing is useful). Yet with the current limitation, I need to maintain separate reltables for each of the LMSes, with largely duplicated content.

When you say "branch processing of reltables" what do you mean precisely?
That is, do you mean having ditavalrefs within topicrefs within reltable cells or you do you mean having references to the base scopes for topicrefs within filtered branches that also define keyscope augmentations?
Consider this map:

      {markup to make the effective scope "scope01-brA" and add "brA" to resource URIs}

     {markup to make the effective scope "scope01-brB" and add "brB" to resource URIs}

  ...

The reference as authored is clear and resolvable (key "somekey" exists in scope "scope01").
However, after branch filtering is applied, there will be no "scope01", only "scope01-brA" and scope01-brB". (So one could argue that the key reference is actually incorrect as authored but I don't think we ever considered this case when thinking through key scopes and branch filtering.)
What should happen to the keyref in the reltable?
Seems like a reasonable solution would be to duplicate the topicref for each filtered branch.
Alternatively the author could author the reltable topicrefs to reflect the scopes as filtered:

Here reflecting the author's desire to relate topics in one branch to topics in another branch.
The key references are perfectly resolvable given the map as authored because the ditavalref markup provides all the information necessary to compute the effective key scopes, so there should be no reason authors can't reliably create this kind of key reference.

I don't see your map, but I can at least partly clarify: what would be useful to me as a writer would be the ability for a relationship table to be processed as part of the branch contents, that is:
<topicref...>
  <ditavalref...>
  <ditavalref...>
  <mapref...>
</topicref>

where the ditamap referenced by the mapref includes one or more relationship tables.
I hadn't considered ditavalrefs within topicrefs within reltable cells; I don't think that would be useful to me, as the principal value of branch processing is to reuse a significant set of content with variations that can be specified via filtering and scoped keys.
I don't see a situation where defining branches within a relationship table would solve a problem that couldn't be addressed by referencing a specific scope or by including a relationship table within a branch. So, to me that use case adds complexity without value.
That said, because the spec does not prohibit ditavalrefs within topicrefs within reltable cells, if we make an implementation choice not to support this, we should document it and recommend the supported implementation.

By "processed as part of the branch contents" I assume you mean that:

Any filtering applied to the branch is also applied to the relationship table (e.g., if there are two filters I get two effective relationship tables)
Any key references are resolved in the context of the branch's key scopes, rather than in the context of the root map's key scopes, meaning that you could have unqualified (or partially-qualified key references in the relationship table that would be resolved in the context of the containing scopes).

If that's what you mean that seems like a reasonable expectation and useful behavior, if not the only useful behavior (the other being to treat the relationship table as though it had occurred as a literal child of the root map, but that seems not at all useful).
However, I'm not sure the DITA spec allows it or explicitly disallows it. Here's what I can find that's relevant:
From the  reference entry:
"Within a map tree, the effective relationship table is the union of all relationship tables in the map."
That doesn't really address the issue of effective key scopes for relationship tables in maps included in scopes.
I can't find any other discussion of relationship tables and key scopes.
I'm not sure we ever considered the case of a filtered branch including reference to a submap that then contained a relationship table (maybe Robert did)? If we had I'm pretty sure we would have realized the need to say something about it.
This looks like a hole in the specification to me in that we never considered the case and therefore never provided any language to address it.
I think the only way it would make practical and useful sense to have reltables referenced from filtered branches would be if the both:

The branch filtering is applied to the reltable
Any key references within such a reltable are treated as though they originated in the containing scope.

Note that this is really more about references to maps that contain reltables where the map reference is within a key scope--the use or non-use of branch filtering doesn't add anything here.
So maybe the TC needs to consider this case and offer some guidance on how we think processors should handle the case in the absence of any clear normative language.
I certainly wouldn't object if the OT decided to implement reltable processing in this way. I can't think of any reason why this wouldn't be the correct behavior.

Yes, Eliot, that's precisely what I mean. 😄

Adding an example with using reltables with keyscopes and branch filtering:
ditavalRefWithLinkingReltable.zip

👍 Another one of our end users expected branch filtering to work in relationship with related links (direct links, not keyrefs).

Hi,
I have a situation similar to what Radu posted above. I have a ditavalref and a reltable that uses direct links. As others have noted, this does not work - but only for webhelp. For eclipse output, this seems to be working fine.
`

...
`

Fixed by #2393


Make PostVisitBinaryOpNodeAddition more functional

A big one, this changes PostVisitBinaryOpNodeAddition to be a mapping of inputs to outputs. This change also makes BinderCheckTypeResult a class instead of a readonly struct. This allows the Coercions property to default to an empty list instead of defaulting to null, removing the need to check for null when consuming the coercions list.

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:x: 1 Binary Compatibility issues for Microsoft.PowerFx.Interpreter compared against version 0.2.2-preview.20220830-319453.

MembersMustExist : Member 'Microsoft.PowerFx.PowerFxConfigExtensions.EnableCollectFunction(Microsoft.PowerFx.SymbolTable)' does not exist in the implementation but it does exist in the contract.

:heavy_check_mark: No Binary Compatibility issues for **Microsoft.PowerFx.LanguageServerProtocol.dll**
:heavy_check_mark: No Binary Compatibility issues for **Microsoft.PowerFx.Transport.Attributes.dll**


Is there a high res version of this image?

https://creativecommons.org/about/global-affiliate-network/
Anyone have a high res version of this?

Hello Jennie,
Yes, Jenn has provided larger version and vector: reisthereahighresversionofthisimage589.zip

Thank you @ageeroms


[BUG] No bundle type check during crc setup

General information

OS: macOS (should affect all OSes the same)
Did you run crc setup before starting it (Yes/No)?
Running CRC on: Laptop

CRC version
# Put `crc version` output here
Build from git, commit 2351a01dd

Steps to reproduce

crc config set preset openshift
crc setup --bundle ~/crc_podman_vfkit_4.4.2_amd64.crcbundle

Your system is correctly setup for using CRC. Use 'crc start' to start the instance

crc start --bundle ~/crc_podman_vfkit_4.4.2_amd64.crcbundle

Preset openshift is used but bundle is provided for podman preset

Expected
crc setup detects the bundle mismatch
Actual
crc setup is successful, and tells me to run crc start

The execution of crc setup should fail with an error message that the bundle does not match the configured preset type.

There's that warning:
$ crc config set preset microshift
Changes to configuration property 'preset' are only applied when the CRC instance is created.
If you already have a running CRC instance with different preset, then for this configuration change to take effect, delete the CRC instance with 'crc delete', setup it with `crc setup` and start it with 'crc start'.

$ crc setup -b ~/Downloads/crc_libvirt_4.12.9_amd64.crcbundle
WARN Using crc_libvirt_4.12.9_amd64.crcbundle bundle, but crc_microshift_libvirt_4.12.9_amd64.crcbundle is expected for this release
.
.
.

I'm not suggesting that it's good as it is :), but for comparison crc start is successful even if some operators aren't running yet (exit code 0).

Should the warning also be displayed together with the reason of failure? It would look like this:
$ crc setup --bundle crc_microshift_libvirt_4.16.7_amd64.crcbundle
WARN Using crc_microshift_libvirt_4.16.7_amd64.crcbundle bundle, but crc_libvirt_4.16.7_amd64.crcbundle is expected for this release
FATA Preset openshift is used but bundle is provided for microshift preset

Warning for custom bundle is non-fatal and we carry on but I think in case of different preset it should be fatal and we should error out.

Merged


Promises don't resolve when using SCSS

Hi all,
I copied code from the newsletter example template and when I call Promise.all built from the below code. If i comment out the report.render function the promises resolve fine but when I add it in they don't resolve.
return report.render(testResults).then(function (results) {

            // setup e-mail data
            var mailOptions = {
                from: 'Test<EMAIL_ADDRESS>// sender address
                to: recipients, // list of receivers
                subject: 'Test', // Subject line
                html: results.html
            };

            // send mail with defined transport object
            return transporter.sendMailAsync(mailOptions);
        }).catch(function(err) {
            console.log(err);
        });

I know this is an issue with SCSS because I have an css version that works properly.

+1 had same issue.
Converted scss to css and it started working.

Try changing it to:
report.render(Object.assign({}, testResults)).then(function (results) {

If that works then it is the same issue as #167

you may want to use v3.x now


Add fallback to termopen() for Windows

nvim_open_term() seems to not work for glow on Windows, falling back to termopen() for Windows should work for now.
above img is before, bottom is after:

oops, there was already discussion on this @ #107, closing for now.


Update dependencies for llama-index-callbacks-arize-phoenix

Description
Update dependencies for llama-index-callbacks-arize-phoenix
Fixes # (issue)
Type of Change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Closing in favor of #10685


adding blessed module

blessed/blessed.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/blessed
http://bower.io/search/?q=blessed
others?

[ ] has a test file? (blessed/blessed-tests.ts or others)
[ ] pass the Travic-CI test?


Updating css of sidebar and app.jsx

Closes: #277

Title : Updating the CSS of Project cart sections
Name: :  Astha Negi
Idenitfy yourself: : GSSOC Contributor

Describe the add-ons or changes you've made 📃
I have added styling to the cards so that when you open the sidebar menu, the width of the cards adjusts itself. In small devices, the sidebar overlaps the cards section just like before changes.
Screenshots 📷
https://github.com/Avdhesh-Varshney/WebMasterLog/assets/93486108/a7c8efff-b18d-4ab7-93e5-28dcecc22097
Type of change ☑

What sort of change have you made:

[x ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[ x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x ] This PR does not contain plagiarized content.
[ x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly wherever it was hard to understand.
[x ] My changes generate no new warnings.

Hey @Avdhesh-Varshney kindly review this pr.


Add Flask app sample that runs on Heroku

heroku login
heroku create

export SLACK_BOT_TOKEN=xxx
export SLACK_SIGNING_SECRET=xxx
heroku config:set SLACK_BOT_TOKEN=$SLACK_BOT_TOKEN
heroku config:set SLACK_SIGNING_SECRET=$SLACK_SIGNING_SECRET

git init
git remote add heroku https://git.heroku.com/xxx.git
git checkout -b main
git add .
git commit -m'Initial commit for my awesome Slack app'
git push heroku main

Category (place an x in each of the [ ])

[ ] slack_bolt.App and/or its core components
[ ] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/run_tests.sh after making the changes.

Congrats for your first-ever commit in this repo, Mike!


thin: Required device-mapper target(s) not detected in your kernel

I am hitting the following error when executing heketi-cli setup-openshift-heketi-storage:
Error: Unable to execute command on glusterfs1-1373000839-qq9jv:   /usr/sbin/modprobe failed: 1
  Cannot read thin-pool target version.
  thin: Required device-mapper target(s) not detected in your kernel.
  Run `lvcreate --help' for more information.

On the hosts there is Ubuntu 16.04.1 LTS installed.
What are the  prerequisites for Heketi/Glusterfs to create volumes?

Ok, in my case I needed to execute modprobe dm_thin_pool on all nodes to resolve this.

Closing the issue since it seems it worked itself out. :) Feel free to comment and re-open if I'm mistaken.

@jarrpa Could you reopen this? @webwurst suggestion indeed worked out, but this is a reproducible issue that needs to be automated somehow I guess.. Maybe this issue needs to be raised somewhere else, like kubelet or something

PR #155 should provide a mechanism for automating this, I believe.


Set up GPU CI

@avik-pal would you mind taking a look at this (clearly I don't know how to set things up properly)

I can't see the settings page on buildkite, can you give me access there? If you can't find my name, adding LuxDL or SciML team would also do

I added LuxDL,

can see it, thanks!


mise env still do not auto-load when using Run/Debug Configurations

Describe the bug
unfortunately, mise env still do not load when using Run/Debug Configurations
To Reproduce
Steps to reproduce the behavior:

Create a Gradle project
Paste println("HELLO: ${System.getenv("HELLO")}") at build.gradle.kts
$ mise set HELLO=WORLD
Enter ctrl + ctrl to call Run Anything window
Execute ./gradlew clean within Run Anything window
HELLO: null is output

Expected behavior

if open Edit Configurations > Environments > Edit environment variables, mise env variables shoud be registered
HELLO: WORLD should be printed

Screenshots

Can't find 'HELLO' env

HELLO is null but expected printing WORLD
Desktop (please complete the following information):

OS: MacOS
IntelliJ version: Ultimate 2<IP_ADDRESS>
Mise Plugin version: 3.2.2

Additional context
issue-project.zip

Hi @songkg7, thank you for your report!
The issue was resolved in #117 and it will be appeared in v3.2.3.


Add support for 3D Tiles with translucency

For reference, similar support was added to Cesium for Unreal in this PR:
https://github.com/CesiumGS/cesium-unreal/pull/898
Requested on the community forum:
https://community.cesium.com/t/alpha-textutre-support/22069

Any progress on this? The lack of alpha texture support in Cesium for Unity prevents the 3D Tiles export from our Remoscape cloud service from being used in Unity, since trees, power line pylons, masts, etc. cannot be visualized properly.

No progress yet. If you have the development capacity, we welcome a pull request. This feature was added by a member of the community in Cesium for Unreal, and it's probably reasonable for someone outside of Cesium to add it for Unity as well. Here's the Unreal PR:
https://github.com/CesiumGS/cesium-unreal/pull/898

Requested on the forum here.

Requested on the forum again.


Create team for Governance Committee and ensure TAC, GB have access to this repo

I have added the TAC as a group; we need to define groups for the GB and GB observers, then we can do away with individual memberships to the repo.

Has this issue been addressed?  Can it be closed now?


source/DialogueBox.hx:183: characters 2-6 : Expected } pls help

code on 183-193 is
    else if (PlayState.SONG.song.toLowerCase()=='waterfall' || PlayState.SONG.song.toLowerCase()=='showdown')
	{
		portraitRight = new FlxSprite(0, 40);
	    portraitRight.frames = Paths.getSparrowAtlas('portraits/boyfriendPortrait', 'shared');
	    portraitRight.animation.addByPrefix('enter', 'BF Portrait Enter instance', 24, false);
	    portraitRight.setGraphicSize(Std.int(portraitRight.width * PlayState.daPixelZoom * 0.9));
	    portraitRight.updateHitbox();
	    portraitRight.scrollFactor.set();
	    add(portraitRight);
	    portraitRight.visible = false;
	}

what does line 182 look like?

hello?
how am I supposed to help if you don't answer...
if this issue is still present, please respond
else close this issue.
thanks :3


Update all dependencies

Dependency updates: What will they break? Will they break things?? Let's find out!
Updated all dependencies, including some major versions which will probably break something.

@iansu #11583 relates to this PR.

For anyone using VScode

I tend to use this VS Code extension a lot called Version Lens

It adds a decoration (similar to Gitlens) next to the dependency name


Need to Work on Movies Page

The design of movie page doesn't looks cool, Maybe there's a problem with linking CSS file You can look at it here :

Here is the link of the following page  https://qazimaazarshad.github.io/Movie-Streaming-Website/movies.html

Yaa someone was working on this but she made the changes without updating the repo and messed it up.
Completely update this page like the other pages.

Please assign me this issue.

Please assign me this issue.

If the issue remains unresolved for a few more days then I will assign you.

Please assign me this issue under LGMSOC'21


Is possible to change the the days label on the datepicker to longer strings?

Hi,
Excelent library, really enjoyed. I have a question that I think it could be related to #11 but I am not sure.
When you are seeing the date picker in Catalan for example, you get this:

Of course all the days of the week in Catalan start with the letter "D", so this is the expected behavior. But Google is sorting this behavior using one more letter. For example in google calendar:

Or in the alarm app:

So, the question is, is it possible to achieve this in the date picker dialog? So the users have a better lucking picker
thansk very much for the picker ;)

Adding more information to this... the day’s labels in google calendar, in Spanish is the follow when using one letter:

The "Wednesday" day label is represented by "X" when the actual name is "Miercoles" that's because that's the correct representation of the day in Spanish.
In the date picker, because it use the first letter, the day is represented as "M" the same as the one before "Martes" that would be "Tuesday".

I'm aware that the current implementation doesn't work in all locales.
There are two ways to solve this:

find a way to have the java Calendar class return a locale adjust single letter for a day (or two in the cases of Catalan or Chinese). Unfortunately I haven't figured this out yet.
build exceptions for locales where the current implementation breaks down. This is what I've done so far. Unfortunately I don't speak all languages in the world, so I rely on people like you to report these and let me know how it should be done.

is cool, the library is very cool.
What about an option to load into de dialog a char array for the week days label per locale. I mean, like i pass as a parameter the way is going to render in catalan or spanish when i create the dialog. At least it could be a workaround :cry:

will you update the app with using the chars for catalan and spanish as described in the second option?

I'll add special cases for Catalan and Spanish.

@wdullaer sweet mate :+1: :dancer:

I figured out the proper way to do this :-)

thanks, thats a very good line of code...love the delete code. :+1:


[QUESTION] Metrics support? (eg. Grafana, statsd, etc.)

Description
Can FastAPI be configured to transmit API usage metrics to a metrics server (e.g. Grafana, statsd, etc.? For example, success and failures for API calls (api.api_call.error.404)

Thanks for the help here @dmontagu ! :clap: :bow:
Thanks for reporting back and closing the issue @jjwwiodyne  :+1:


Unable to debug C program

Type: Bug
I am unable to debug simple C program. Building works well using gdb.
However, when I start process with debugging, suddenly the VS Code is trying to read my main.c file at some weird path. It looks like duplicate path, the path is consisting of correct path, but somehow two times added.

VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-9300HF CPU @ 2.40GHz (8 x 2400)

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_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
7.92GB (1.93GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

cmake-tools
ms-
1.14.33

cpptools
ms-
1.16.3

cpptools-extension-pack
ms-
1.3.0

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263:30776702
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
a2ce3375:30757347
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
hideindicator:30766889
pythongtdpath:30769146
gsof1:30774496
e440d664:30776459
pythonidxptcf:30772540
pythondjangotscf:30772537
pythonnoceb:30776495

Debug console:
=thread-group-added,id="i1"
GNU gdb (GDB) (Cygwin 10.2-1) 10.2
Copyright (C) 2021 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
Type "show copying" and "show warranty" for details.
This GDB was configured as "x86_64-pc-cygwin".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
https://www.gnu.org/software/gdb/bugs/.
Find the GDB manual and other documentation resources online at:
http://www.gnu.org/software/gdb/documentation/.
For help, type "help".
Type "apropos word" to search for commands related to "word".
Warning: Debuggee TargetArchitecture not detected, assuming x86_64.
=cmd-param-changed,param="pagination",value="off"
[New Thread 11204.0x3048]
[New Thread 11204.0x2694]
[New Thread 11204.0x3420]
Thread 1 "main" hit Breakpoint 1, main () at C:\Users\victo\Desktop\ahojec\main.c:61
Loaded '/cygdrive/c/WINDOWS/SYSTEM32/ntdll.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/KERNEL32.DLL'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/KERNELBASE.dll'. Symbols loaded.
Loaded '/usr/bin/cygwin1.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/advapi32.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/msvcrt.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/sechost.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/RPCRT4.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/SYSTEM32/CRYPTBASE.DLL'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/bcryptPrimitives.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/system32/netapi32.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/SYSTEM32/SAMCLI.DLL'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/ucrtbase.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/SYSTEM32/SAMLIB.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/SYSTEM32/NETUTILS.DLL'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/System32/ws2_32.dll'. Symbols loaded.
Loaded '/cygdrive/c/WINDOWS/system32/mswsock.dll'. Symbols loaded.
[New Thread 11204.0x1a48]
Thread 5 received signal SIGINT, Interrupt.
[Switching to Thread 11204.0x1a48]
0x00007ff8196ae971 in TlsGetValue () from /cygdrive/c/WINDOWS/System32/KERNELBASE.dll
Execute debugger commands using "-exec ", for example "-exec info registers" will list registers in use (when GDB is the debugger)

Terminal:
Windows PowerShell
Copyright (C) Microsoft Corporation. All rights reserved.
Try the new cross-platform PowerShell https://aka.ms/pscore6
PS C:\Users\victo\Desktop\ahojec>  & 'c:\Users\victo.vscode\extensions\ms-vscode.cpptools-1.16.3-win32-x64\debugAdapters\bin\WindowsDebugLauncher.exe' '--stdin=Microsoft-MIEngine-In-uqdt15lu.2im' '--stdout=Microsoft-MIEngine-Out-e5uywp2l.iqo' '--stderr=Microsoft-MIEngine-Error-scum15qt.21p' '--pid=Microsoft-MIEngine-Pid-tcqetktj.tnd' '--dbgExe=C:\cygwin64\bin\gdb.exe' '--interpreter=mi'

Found a solution here: https://github.com/microsoft/vscode-cpptools/issues/10630
However, very frustrating...

/extC++

It looks like this is caused by the C++ extension. Please file the issue to the C++ extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Add backpressure mechanism for work queue to avoid overflow scenarios for storage operations

see https://phabricator.mesosphere.com/D500#15710

looks like a dup of #5152

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-1831. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Fixed initialization in base DV class

Purpose
I found another issue I introduced when merging DVs into one base class in #130. Some parameters (upper, lower, scale) were not being set when a DV is created with their values as None, so there was an error when addVarGroup was called.
Expected time until merged
Should be quick, just one line and it fixes a bug
Type of change

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing
Without this, some DVs will not run without bounds.
Checklist

[X] I have run flake8 and black to make sure the code adheres to PEP-8 and is consistently formatted
[ ] I have run unit and regression tests which pass locally with my changes
[ ] I have added new tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

We should consider testing the optimization related parts of the code.

Why didn't our MACH-Aero tests catch any of these?


Update sbt-example to 9.1.0

Updates com.thoughtworks.example:sbt-example from 9.0.0 to 9.1.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.thoughtworks.example", artifactId = "sbt-example" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #163.


Make Graphiql Link Dynamic

Sometimes People Work On Sub-Directory, So PW Installation not always on the root.

:tada: This PR is included in version 1.0.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ENH: printing for meta data object of vectors

closes #4368

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

As best I can tell: the print_helper operator is being instantiated for vector<vector<double>>, so it tries to do ostream << vector<double>, which is not defined.


Docker says time is mismatched

Recently setup Py-KMS in docker and noticed this when looking at logs. I have another py-kms LXC in proxmox and have no such warnings. I can provide more info if needed. I checked date and time and both are correct though maybe a minuet or so apart.
Thu, 07 Apr 2022 18:56:20 WARNING  Okay, something went horribly wrong while localizing the request time (proceeding anyways): Timezone offset does not match system offset: 0 != -14400. Please, check your config files.
Thu, 07 Apr 2022 18:56:20 WARNING  With count = 26, activated client could be detected as not genuine !

Try to set timezone of the docker container with TZ enviroment variable... add somewhere when creating docker -e TZ="Europe/Berlin" (or whatever your timezone is). If docker is run on linux you can also mount /etc/localtime to py-kms /etc/localtime with -v /etc/localtime:/etc/localtime:ro (dont forget ro, it makes it read only).

@dudefoxlive Did it work? If yes, please close this issue...

Try to set timezone of the docker container with TZ enviroment variable... add somewhere when creating docker -e TZ="Europe/Berlin" (or whatever your timezone is). If docker is run on linux you can also mount /etc/localtime to py-kms /etc/localtime and set timezone that way with -v /etc/localtime:/etc/localtime:ro (dont forget ro, it makes it read only).

I did this originally when creating it and no. same issue.

Well its definatly some kind of time inconsistencys.... if its not timezones are you using NTP to set time on server/clients? If you dont, you should... also one other thing... its just a warning anyway :)

-> Caused by: https://github.com/regebro/tzlocal/issues/79

I've triggered a rebuild of the Docker images... Maybe that helps? Otherwise I have to dive deeper what's going on here...

I have noticed that the Docker VM is about 10 or so seconds ahead

Huh? This smells like a classic problem, where you do not use the same NTP servers / the docker host has no ntpd active... Is this even related?

ok so update. I eventually found out that my Domain Controller was about 10 seconds behind and my docker was actually correct. I have fixed the time issue and now my DC is syncing with an NTP server. Same servers as my Docker VM. still after fixing that issue I am still getting the same message as before.

if no tz is defined in the container, utc is set. do you have a diff of -2h with utc ?

OK so new update. Setting the ENV TZ=US/Eastern has fixed the issue of saying the localization time being wrong. Now the only warning I am getting is "Wed, 20 Apr 2022 12:52:01 WARNING  With count = 26, activated client could be detected as not genuine !" Any ideas on that?

Glad to hear, that you "fixed" the problem. I sadly have no idea why your timezones were such a mess (maybe even the time of the Docker host is not localized?)...
Regarding the second warning... Try to change the activation count (see params) - but it should be fine...


[Storage] az storage file upload takes too many required params

In practical use, the old CLI allows:
azure storage file upload MyShare MyDirectory ./thisFile.txt
while we require
az storage file upload -s MyShare -d MyDirectory -n thisFile.txt --local-file-name ./thisFile.txt

The --name param should be optional
This --local-file-name should allow a -p alias

$ az storage file upload --help

Command
    az storage file upload

Arguments
    --local-file-name [Required]: The path and file name to upload.
    --name -n         [Required]: The primary resource name.
    --share-name -s   [Required]: The name of the file share.
    --account-key
    --account-name
    --connection-string
    --directory-name -d         : The destination directory to upload to.
    --sas-token

Global Arguments
    --debug                     : Increase logging verbosity to show all debug logs.
    --help -h                   : Show this help message and exit.
    --output -o                 : Output format.  Allowed values: json, tsv, list.  Default: json.
    --query                     : JMESPath query string. See http://jmespath.org/ for more
                                  information and examples.
    --verbose                   : Increase logging verbosity. Use --debug for full debug logs.

$ azure storage file upload --help
help:    Upload the specified local file to storage
help:
help:    Usage: storage file upload [options] [source] [share] [path]
help:
help:    Options:
help:      -h, --help                                   output usage information
help:      -v, --verbose                                use verbose output
help:      --json                                       use json output
help:      -s, --source <source>                        the local file path
help:      --share <share>                              the storage share name
help:      -p, --path <path>                            the path to the storage file
help:      --concurrenttaskcount <concurrenttaskcount>  the maximum number of concurrent upload requests
help:      -q, --quiet                                  overwrite the specified storage file without confirmation
help:      --sas <sas>                                  the shared access signature of the storage file share
help:      -a, --account-name <accountName>             the storage account name
help:      -k, --account-key <accountKey>               the storage account key
help:      -c, --connection-string <connectionString>   the storage connection string
help:      -vv                                          run storage command in debug mode
help:
help:    Current Mode: arm (Azure Resource Management)

@JasonRShaver comparing this to XPlat, it looks like they use --source/-s as an alias for --local-file-path. They combine the name/directory parameter with their --path/-p argument.  We already use -s for --share-name/-s, which I'd recommend keeping for consistency within the CLI itself. My recommendation:

Alias --local-file-path to --source (no flag) as a balance for consistency between Xplat and our own CLI.
Remove --name and replace it with --path\-p, which will consist of the dir/filename (like --source). If the filename portion isn't supplied, it will use the same filename as the --source parameter.

Thoughts?

I agree, great change!


[BUG] Adobe loader conflict with openai URL

Description
when changing the loader to adobe with good credentials I see this error:
Invalid URL '/openai/deployments/gpt-4o/chat/completions?api-version=2024-02-15-preview': No scheme supplied. Perhaps you meant https:///openai/deployments/gpt-4o/chat/completions?api-version=2024-02-15-preview?
Reproduction steps
change the loader to adobe
upload a file

Screenshots
![DESCRIPTION](LINK.png)

Logs
The error is this

Traceback (most recent call last):
  File "/app/libs/ktem/ktem/index/file/pipelines.py", line 726, in stream
    file_id, docs = yield from pipeline.stream(
  File "/app/libs/ktem/ktem/index/file/pipelines.py", line 588, in stream
    docs = self.loader.load_data(file_path, extra_info=extra_info)
  File "/app/libs/kotaemon/kotaemon/loaders/adobe_loader.py", line 128, in load_data
    figure_captions = generate_figure_captions(
  File "/app/libs/kotaemon/kotaemon/loaders/utils/adobe.py", line 245, in generate_figure_captions
    results = [future.result() for future in futures]
  File "/app/libs/kotaemon/kotaemon/loaders/utils/adobe.py", line 245, in <listcomp>
    results = [future.result() for future in futures]
  File "/usr/local/lib/python3.10/concurrent/futures/_base.py", line 451, in result
    return self.__get_result()
  File "/usr/local/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/app/libs/kotaemon/kotaemon/loaders/utils/adobe.py", line 240, in <lambda>
    lambda: generate_single_figure_caption(vlm_endpoint, figure)
  File "/app/libs/kotaemon/kotaemon/loaders/utils/adobe.py", line 208, in generate_single_figure_caption
    output = generate_gpt4v(
  File "/app/libs/kotaemon/kotaemon/loaders/utils/gpt4v.py", line 48, in generate_gpt4v
    response = requests.post(endpoint, headers=headers, json=payload)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 115, in post
    return request("post", url, data=data, json=json, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 575, in request
    prep = self.prepare_request(req)
  File "/usr/local/lib/python3.10/site-packages/requests/sessions.py", line 484, in prepare_request
    p.prepare(
  File "/usr/local/lib/python3.10/site-packages/requests/models.py", line 367, in prepare
    self.prepare_url(url, params)
  File "/usr/local/lib/python3.10/site-packages/requests/models.py", line 438, in prepare_url
    raise MissingSchema(
requests.exceptions.MissingSchema: Invalid URL '/openai/deployments/gpt-4o/chat/completions?api-version=2024-02-15-preview': No scheme supplied. Perhaps you meant https:///openai/deployments/gpt-4o/chat/completions?api-version=2024-02-15-preview?

Browsers
No response
OS
No response
Additional information
No response

need to specify the value of 'AZURE_OPENAI_ENDPOINT' and 'AZURE_OPENAI_API_KEY'  in the .env file.


Limit resource access

Some endpoints were only verifying against the project ID passed as argument in order to determine whether the resource is forbidden or not.
Context
For instance, let's say we have a project A with env 1 and 2 and a project B with env 3.

If a user of the project A tries to get /projects/A/environments/1/flags, they get the flag of the env 1
If a user of the project A tries to get /projects/B/environments/13flags, they get a 403
HOWEVER, - If a user of project A tries to get /projects/A/environments/**3**/flags, they get the flags of the env 3 without having access to it.

Take care of "remove member"


Fix crash on Android release build

Resolve https://github.com/CovidShield/mobile/issues/75

Verified on Android release build!


Create a store using current AWS role attached to process

👋🏽 I am trying to create an Icechunk store using the current role of the ec2 instance I am running the following notebook in (which is a 2i2c JupyterHub for VEDA), and finding that using s3_from_env may not work with IAM-role based access, as I am getting an expired token error (when no token is being passed, the role itself should be used for access, not temporary tokens). But I could be doing something wrong.
See https://gist.github.com/abarciauskas-bgse/e4d27a8d41dd887657d04a544122c64a for a MRP. Note: this assumes you are using some role which has access to nasa-veda-scratch. Also note that it does work when you use S3 credentials generated using boto3 from the current session.
Any ideas @paraseba @sharkinsspatial @maxrjones

Thanks for sharing Aimee! I don't have a solution for you right away, but I can say that that THIS:

using the current role of the ec2 instance I am running the following notebook in (which is a 2i2c JupyterHub for VEDA) ... using s3_from_env

is exactly how we use Icechunk internally at Earthmover (except it's our own JupyterHub rather than a 2i2c one). It just works and I don't have to use the .get_frozen_credentials() workaround from your gist.
@mpiannucci may have ideas...

@abarciauskas-bgse the following works for me on the VEDA Hub. I wonder if this is actually an issue with configuring the bucket through both the storage and config parameters rather than directly related to auth.
import icechunk
import zarr

storage_config = icechunk.StorageConfig.s3_from_env(
    bucket="nasa-veda-scratch",
    prefix="icechunk/test-mursst",
    region="us-west-2"
)
store = icechunk.IcechunkStore.create(storage_config)
group = zarr.group(store)
array = group.create("my_array", shape=10, dtype=int)
array[:] = 1
store.commit("first commit")

I wonder if this instance has the AWS_SESSION_TOKEN environment variable set, or some other similar one, and somehow the aws cli manages to ignore it.

Thank you all! I did have a mistaken leftover token in my environment. Apologies for the rookie mistake!


⚠ Fortnite has degraded performance

In 73a6b40, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1777 ms

Resolved: Fortnite performance has improved in 0cb046c.


Same etag everytime, empty dict

I am trying to use this library but I am having a problem with ETags. The Etag I get is always the same, upon further inspection I have noticed that the request dict is always empty. Any suggestion? Here is my code:
def my_etag(request, *args, **kwargs):
    return hashlib.md5(':'.join(request.GET.dict().values()).encode('utf-8')).hexdigest()

class DeviceMachines(generics.ListAPIView):
    queryset = Device.objects.all()
    serializer_class = DeviceMachinesSerializer

    def get_queryset(self):
        queryset = super(DeviceMachines, self).get_queryset()
        device = queryset.filter(id=self.kwargs['device_id'])
        store = Store.objects.filter(name=device[0].store)
        machines = Machine.objects.filter(store__in=store)
        return machines

    @etag(my_etag)
    def list(self, request, *args, **kwargs):
        queryset=super(DeviceMachines, self).list(request, *args, **kwargs)
        queryset.data = {
            'machines': queryset.data
        }
        return Response(queryset.data)

Thank you!

Function my_etag is just for the example. You need to write your own etag function based on your needs. You can find more information here: https://docs.djangoproject.com/en/dev/topics/conditional-view-processing/

Of course the etag will be same if requests.GET is always empty. That's correct behaviour. Etag should be same if you provide same input data.
You have to create your own etag function (which probably won't use requests.GET). The correct way how to do it is to think about what will define your objects. Maybe you can use some data from your models which will be used to generate etag?

But my question is, why is the request always empty? It's not supposed to... My queryset.data is actually not empty and I get the results I want on my website. But when I try to parse the request on the etag function it's empty.
Anyway, I'll try your suggestion, I just wanted to know why I can't access the data inside the etag func

It's empty because you didn't provide HTTP GET parameters. Read docs here: https://docs.djangoproject.com/en/dev/ref/request-response/#django.http.HttpRequest.GET
Anyway, this discussion is not about this lib anymore => closing the issue.


not working properly on macOS

the video is like this

My platform is macOS 12.4, with an m1 pro chip
I confirm this issue is not because of my own code, as I test the same code on a windows machine successfully.

https://github.com/valbok/QtAVPlayer/issues/220
the same?

My platform is macOS 12.4, with an m1 pro chip
I confirm this issue is not because of my own code, as I test the same code on a windows machine successfully.

yes
Sorry for not checking issue list
I am trying to figure out what happened


(./examples/pointnet2_classification.py) FileExistError in Microsoft Windows when creating a new ModelNet 10 Dataset.

🐛 Bug

The Chinese means:
When the folder exists, you cannot create the folder with same name.
To Reproduce
Steps to reproduce the behavior:
run ./examples/pointnet2_classification.py on Microsoft Windows.
Expected behavior
As the error shows, just delete the empty ./raw dir, and rename the ./ModelNet10 dir to ./raw dir, re-run the code, problem solved. This is a way of dealing with the problem.
Environment

OS: Microsoft Windows 10, build 17134
Python version: 3.6.4
PyTorch version: 1.1.0(+cu90)
CUDA/cuDNN version: 9.0.176/7.4
Compiler version: (Visual Studio 2015, with MSVC 14.0)
Any other relevant information: No

Additional context
P.S. is num_workers really counts?

👆, After modifying the code below, I run the code, and it throws an error on multiprocessing which is related to DataLoader.

👆, My notebook's CPU is Intel Core i7-8750H, with an NVIDIA GeForce GTX 1060 (6GB) GPU. I am running some webpages on Microsoft Edge when I want to run the code.

👆, This time, after removing num_workers=6 (Sorry to tell you that the previous issue, i also modified this part), it is running.
I know it should be a PyTorch issue. But...
yours sincerely,
@wmf1997

Does the first error occur for other datasets as well? There seems to be a Windows specific bug in makedirs. Maybe you are able to fix this.

Perhaps No. I re-downloaded ShapeNet and extracting and processing goes fine.

So when exactly does this error occur? If you want to download a dataset to a directory where raw/ already exists? Then makedirs should skip the creation of this directory.

When I re-downloaded the ModelNet in a new folder. I just run the ./examples/pointnet2_classificatoin.py (without any modification). After downloading the new ModelNet10, perhaps in the process of extracting? (sorry i forgot several hour's ago's process)
It is not a big problem. Manually remove the folder and rename it is okay~


Vue-Mobx integration broken with PR #7828

Version
2.5.18
Reproduction link
https://codesandbox.io/s/k90k8kwn2r
Steps to reproduce

Use the mobxjs/mobx-vue package (or a custom integration with Mobx)
Everything worked fine on Vue 2.5.17 and is broken on 2.5.18 and above
See that TypeErrors are thrown

What is expected?
Vue would not attempt to to set the __proto__ property of a Proxy.
What is actually happening?
Reactivity with Mobx is broken:
[Vue warn]: Error in event handler for "input": "TypeError: 'set' on proxy: trap returned falsish for property 'proto'"
We attempted to upgrade Vue from 2.5.17 to latest version for our applications - thought it would be smooth because upgrade target was still 2.5.x, but ever version after 2.5.18 is broken with Mobx integration.
We have a custom integration with Mobx-state-tree, but some of the code is very similar to mobxjs/mobx-vue integration code.  We are not using that library, and I'm not a Vue internals expert,  but I think I  tracked the problem down to this PR: https://github.com/vuejs/vue/pull/7828.
Since then I have searched around and found this issue reported to that package also: https://github.com/mobxjs/mobx-vue/issues/15#issuecomment-453570397.  @Nemikolh wrote the codesandbox example I referenced above and it appears to be the same conclusion.

Hi @posva, to be honest, probably nothing; but neither is anything broken in Mobx unless you try to use Vue - so I'm a bit at a loss as to how to proceed 😞 With my personal apps (using Vue and Vuex) nothing is wrong.  When I'm working with my employers app (Vue, Mobx, MST) this function:
function protoAugment (target, src: Object) {
  /* eslint-disable no-proto */
  target.__proto__ = src
  /* eslint-enable no-proto */

in vue/src/core/observer/index.js invokes the Mobx set on observable arrays and returns false (causing the vue warning). https://github.com/mobxjs/mobx/blob/daf6ac0ac8dd369fb6179ec6a7fbbb231f383f9f/src/types/observablearray.ts#L96-L109
I forked the Mobx repo and added a check to the end of the set function:
        if (name === "__proto__") {
            return true
        }
        return false

But I have a really bad feeling about that as I have no idea what I might be breaking in Vue.  It looks like Vue is trying to hook into the prototype chain to observe changes, but Mobx is already doing that.  So, I don't know what the answer is.


Hot reload for python files

Description of the Bug
Hot reload is not working on other python files. It watches main.py by default ig.
But with home_page.py file which is imported in main.py is not working properly.
But If I make changes... terminal gets updated but not reflecting in window.
And I'm using Python Declarative style instead .kv
Code and Logs
# `main.py`

from kivy import Config

# config window size
Config.set("graphics", "width", 360)
Config.set("graphics", "height", 640)

from libs.screens.home_page import HomePage
from kivymd.tools.hotreload.app import MDApp

class MainApp(MDApp):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)

    def build_app(self):
        return HomePage()

if __name__ == "__main__":
    MainApp().run()

# `home_page.py`

from kivy.utils import get_color_from_hex
from kivymd.uix.screen import MDScreen
from kivymd.uix.relativelayout import MDRelativeLayout

class HomePage(MDScreen):
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)

        self.md_bg_color = get_color_from_hex("#03020c")
        self.add_widget(self.app_bar())

    # AppBar layout
    def app_bar(self) -> MDRelativeLayout:
        # Appbar elements
        app_bar_layout = MDRelativeLayout(
            size_hint_y=0.1,
            pos_hint={"top": 1},
        )
        # ...rest codes

        return app_bar_layout

Versions

OS: Linux
Python: 3.10.10
Kivy: 2.2.1
KivyMD: 1.1.1

Hot reload is not working well yet. We will work on improving it in version 1.3.0

Hi, Will support for kv file be added too?

How ever the hot reload for kivy works properly in windows/linux
You can refer for more info:  https://github.com/ramayanbindas/Kivymd_hotreload_demo


Nested autodoc (autodoc in docstrings) breaks nav

class GriffeLoader(_BaseGriffeLoader):
    """The Griffe loader, allowing to load data from modules.

    ::: griffe.loader._BaseGriffeLoader.__init__
    """

Every item rendered before ::: griffe.loader._BaseGriffeLoader.__init__ (including it) is lost: they do not appear in the nav (secondary nav? the right sidebar by default on Material).

Without nested autodoc
With nested autodoc

Information (please complete the following information):

OS: arch
Browser: firefox
mkdocstrings version: 0.16.2

Additional context
Probably related to autorefs.

I know this issue is quite old, but I'm seeing the same behavior even in today's version 0.24.0. I guess that feature is not commonly used.
In the content area, the nested autodoc is rendered correctly (including the correct heading levels), but in the Nav is broken, just as explained.
Any idea what could be causing this issue?

Indeed, the issue is not fixed yet, and I believe very few try to nest autodoc in docstrings.
For a possible explanation: it's possible that the nested rendering pass breaks the context/state of the upper pass, notably the headings recorded by the toc extension. This issue might be magically fixed if we move to a pymdownx.blocks implementation of the extension instead of ::: markup.

Might be able to fix this by setting a nested variable in the handler to true in the try block of do_convert_markdown, and restore its previous value in finally. Then by updating AutodocProcessor.run to reassign headings (handler._headings) when in "nested" mode (to propagate them back to the upper layer) and optionally drop the headings logic when "nested".


Linter can warning if you use on-click="foo" instead of on-click="{{foo}}"

Recent polymer 0.8.7 changed the syntax. Help from the linter in tracking these changes and/or typos down would be really useful.

This issue has been moved to dart-lang/polymer-dart#367.


struct.error: unpack str size too short for format

I get the following error while using enjarify with pypy3:

struct.error: unpack str size too short for format

Using python3 fixes the issue. Also, the script will not use python because python -c "print(range)" returns "<built-in function range>", not "<class 'range'>".

Given how slow pypy3 is, I should probably just make the launcher invoke python3 directly.

@Storyyeller hi
I got this results running latest enjarify with latest PyPy3 with Python 3.5 support (hashtests.py using range 10 ):
pypy3 -m enjarify.hashtests  21,44s user 0,11s system 99% cpu 21,552 total

python3.5 -m enjarify.hashtests  73,71s user 0,37s system 99% cpu 1:14,09 total

Is this issue solved ?

Well the issue of pypy3 being slow is certainly fixed. I haven't seen any problems running with pypy, but I'm not the OP.


GEOMESA-1120 Tests for unparameterized Lists/Maps in SFTs

Signed-off-by: Andrew Hulbert<EMAIL_ADDRESS>

I believe that untyped get treated as List[String] - so you can't just put anything in there... I could be wrong.

Cool, yeah I tested it out I put a java.util.jap inside of it...declared it as a plain List in the SFT though (keeping the lie alive)...wasn't a geomesa datastore just using the SFT spec. stuff

163392ceb423118b514107b681b2d59e2ca335da