ESP32-S2 USB SD MSC not working well

Hi, @chegewara, I'm trying to use an ESP32S2 USB stick(https://www.hackster.io/news/create-custom-usb-devices-with-the-esp32s2stick-458bf370a522) to make an SD to USB adapter, however it seems to be taking too long to read the whole 8GB SD card for the USB, when I enable verbose core debug level, it just shows "[V][sdcard.cpp:65] onRead(): default onread". I saw on this post: https://www.esp32.com/viewtopic.php?t=15908 that you used a 16GB SD card and got it working. The comunication with the SD card is working, because the test code created the directory /mydir, wrote and read the hello.txt file, so to me it seems like something on the USB side.
My code is this:
USB_stick.zip
Obs.: I tried putting the code in the post, but I couldn't get right, this is my first post on github.
I'm using ESP32 2.0.1 RC1 and windows 10.
Edit: I tested it on linux and it worked, but on any windows machine it seems to not work.

Hi,
i dont know what is going on on windows, because i tested it only linux, but you cant expect it to be very fast, especially when you turn on verbose logs. This is only full speed USB, 12 mbit, and even this speed is not used, and  logs will slow it down 4 times or even more.

I also tested It without logs, and it's still too slow. I gonna test it on some other other OS' and try to see what might be causing the problem on windows 10.

For testing you can also try examples from adafruit. Ive read that adafruit library is working now.


C++ URI validation in Envoy

Looks like Envoy is using C++ URI validations in StsService:
https://github.com/envoyproxy/envoy/blob/062c895f499382ae61dead16db2a7e78b9146525/api/envoy/api/v2/core/grpc_service.proto#L94
Config validations throw an unimplemented exception when initializing the server (rather than a controlled a ProtoValidation error).
https://oss-fuzz.com/testcase-detail/5665272556158976
It might be a good time to implement this constraint. I think less new logic in PGV is better, and I wonder if it's best to use a regex with RE2 to implement the constraint, or a small C++ library (but not ideal, as it's a dependency)
@JimmyCYJ

@asraa Thanks for letting me know. I just hit the same issue when I start to set up integration test today.

Thanks for posting this issue @asraa, we also ran into the same issue. Would love to see this implemented, even if it's a regex check. Would be very helpful for config validation.


Initialize database with test data.

Seed database with a user account to allow for testing since that functionality will not be coded yet.

Added logic to seed if necessary when started. The following data was seeded to localdb.


Evaluator for Task Dispatcher should not be specified in SubmitTask.srv

Ref: https://github.com/osrf/rmf_core/pull/217#discussion_r545594614

TODO: Remove redundant start_time field as it is captured in TaskDescription.

Should be resolved in an upcoming PR


now can accept STF sha in test pipeline

related to issue #2680

-Grinder STF_OWNER_BRANCH example and its output

adoptium:master
output from grinder:

adoptium:2129b191b1111b665758fb755851025e35888a1b
output from grinder:

adoptium:invalid
output from grinder:

this code also has the change related to the PR #2882

Thanks @xius666 . The PR looks good except for the minor issue above. Also, Please remove the change that is related to PR #2882

https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1485/


Bro obfuscated his code and released it. Its free why would he obfuscated the code unless hes hiding a malicious Code init

It's so skids don't copy it and either add malware or sell it as their own.


Try again

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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

Issue with compiling Wannier90 - it makes more sense to release it as separate package:
/bin/sh ../libtool  --tag=FC   --mode=compile mpif90 -DHAVE_CONFIG_H -I. -I..   -DNDEBUG -D_FORTIFY_SOURCE=2 -O2 -isystem $PREFIX/include    -ffree-form -g -ffree-line-length-none   -O2 -mtune=native -march=native -g -O2  -c -o constants.lo constants.F90
libtool: compile:  mpif90 -DHAVE_CONFIG_H -I. -I.. -DNDEBUG -D_FORTIFY_SOURCE=2 -O2 -isystem $PREFIX/include -ffree-form -g -ffree-line-length-none -O2 -mtune=native -march=native -g -O2 -c constants.F90
/bin/sh ../libtool  --tag=FC   --mode=compile mpif90 -DHAVE_CONFIG_H -I. -I..   -DNDEBUG -D_FORTIFY_SOURCE=2 -O2 -isystem $PREFIX/include    -ffree-form -g -ffree-line-length-none   -O2 -mtune=native -march=native -g -O2  -c -o io.lo io.F90
libtool: compile:  mpif90 -DHAVE_CONFIG_H -I. -I.. -DNDEBUG -D_FORTIFY_SOURCE=2 -O2 -isystem $PREFIX/include -ffree-form -g -ffree-line-length-none -O2 -mtune=native -march=native -g -O2 -c io.F90
/tmp/ccVaYR7v.s: Assembler messages:
/tmp/ccVaYR7v.s:14: Error: no such instruction: `vmovdqu64 (%rdi),%xmm0'
/tmp/ccVaYR7v.s:17: Error: no such instruction: `vmovdqu64 16(%rdi),%xmm0'
/tmp/ccVaYR7v.s:20: Error: no such instruction: `vmovdqu64 32(%rdi),%xmm0'
/tmp/ccVaYR7v.s:22: Error: no such instruction: `vmovdqu64 48(%rdi),%xmm0'
/tmp/ccVaYR7v.s:24: Error: no such instruction: `vmovdqu64 64(%rdi),%xmm0'
/tmp/ccVaYR7v.s:146: Error: no such instruction: `vpxord %zmm0,%zmm0,%zmm0'
/tmp/ccVaYR7v.s:735: Error: no such instruction: `vmovdqa64 .LC18(%rip),%xmm0'
/tmp/ccVaYR7v.s:739: Error: no such instruction: `vmovdqa64 .LC19(%rip),%xmm0'
make[7]: *** [Makefile:630: io.lo] Error 1


API build is using Npm, instead of Yarn

Describe the bug
Oryx is not picking up Yarn from package.json while building API.
To Reproduce
Steps to reproduce the behavior:

I am trying to deploy my open-source project on SWA platform.
The project is a Yarn workspaces based monorepo and the app is a Next.js app.
The workflow is listed here: https://github.com/codelitdev/courselit/blob/workspace-based-resolution/.github/workflows/azure-static-web-apps-gray-meadow-012ce1110.yml
The App gets build correctly but the API is not picking up Yarn as the package manager. It is using NPM only.

GitHub Action Workflow Pipeline: https://github.com/codelitdev/courselit/blob/workspace-based-resolution/.github/workflows/azure-static-web-apps-gray-meadow-012ce1110.yml
GitHub Action run result: https://github.com/codelitdev/courselit/actions/runs/3952346891/jobs/6767308358
Expected behavior
The API should use Yarn instead of NPM.

Hi @rajat1saxena,
In context to nextjs, we're using api build to install production dependencies only, as the nextjs app is already built as part of the app build.
So to force using yarn during api build , can you try passing api_build_command argument and use it to only do something like yarn install --production.
Sorry for the inconvenience, and please let me know if it helps.


probe libappindicator or libayatana-appindicator at runtime

closes tauri-apps/tauri#3859

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[x] Yes, and the changes were approved in issue #___
[ ] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

The pull request removes features gtk-tray and ayatana-tray and should be considered as a breaking change

My intent when I added both feature flags is that we only probe at runtime if both are enabled, or we add a third feature flag (runtime-tray idk).

@FabianLars What about we move this to libappindicator-rs?
The reason it will panic is because both crate check in build script. Maybe we can change how libappindicator-rs checks it.
Probing on runtime don't need the lib actually installed.

That one is during compile time. After searching a bit, I found a crate similar to dlopen.
We might use it to bind both old and ayatana one.
I'll close this in favor of https://github.com/tauri-apps/libappindicator-rs/issues/31


Malformed packet crashes server

Your environment.

Version: 1.0.3 from https://github.com/pion/turn/releases/download/1.0.3/simple-turn-linux-amd64
Browser: n/a
Other Information

panic: runtime error: index out of range

goroutine 1 [running]:
encoding/binary.bigEndian.Uint16(...)
        /usr/local/go-d2c7dec183f0da628abf16848e9e92987feebe32/src/encoding/binary/binary.go:100
github.com/pions/pkg/stun.getChannelLength(...)
        /go/src/github.com/pions/pkg/stun/channel_data.go:40
github.com/pions/pkg/stun.NewChannelData(0xc000012600, 0x2, 0x5dc, 0x1, 0x0, 0x0)
        /go/src/github.com/pions/pkg/stun/channel_data.go:23 +0x161
github.com/pions/turn/internal/server.(*Server).handleUDPPacket(0xc000056100, 0xc00000c1c0, 0xc00000c1e0, 0xc000012600, 0x5dc, 0x5dc, 0x2, 0xc00006c870, 0x0)
        /go/src/github.com/pions/turn/internal/server/server.go:76 +0xc0
github.com/pions/turn/internal/server.(*Server).Listen(0xc000056100, 0x5670ad, 0x7, 0xd96, 0x5dc, 0x5dc)
        /go/src/github.com/pions/turn/internal/server/server.go:65 +0x2e3
github.com/pions/turn.Start(0x582220, 0x6506e8, 0xc00001a0e6, 0x11, 0xd96)
        /go/src/github.com/pions/turn/turn.go:24 +0x178
main.main()
        /go/src/github.com/pions/turn/cmd/simple-turn/main.go:48 +0x2ec

What did you do?
I was testing if the port is opened and crashed the TURN server. Oops. Very easy to repro
Start the server, in a different shell run:
echo a | nc -u localhost 3478

What did you expect?
Server gracefully handling malformed packets.
What happened?
Server crashed, see stacktrace above.

Try again,  I test the latest version,  don't crash yet.

This looks to have been fixed, if you are still having this issue please re-open. Thanks for using Pion TURN!
$ go run main.go -public-ip <IP_ADDRESS> -users foo=bar
turn ERROR: 2020/01/09 00:41:37 error when handling datagram: failed to create stun message from packet: unexpected EOF: not enough bytes to read header
turn ERROR: 2020/01/09 00:41:39 error when handling datagram: failed to create stun message from packet: unexpected EOF: not enough bytes to read header
turn ERROR: 2020/01/09 00:41:40 error when handling datagram: failed to create stun message from packet: unexpected EOF: not enough bytes to read header
turn ERROR: 2020/01/09 00:41:41 error when handling datagram: failed to create stun message from packet: unexpected EOF: not enough bytes to read header
turn ERROR: 2020/01/09 00:41:42 error when handling datagram: failed to create stun message from packet: unexpected EOF: not enough bytes to read header

$ echo a | nc -u localhost 3478
$ echo a | nc -u localhost 3478
$ echo a | nc -u localhost 3478
$ echo a | nc -u localhost 3478
$ echo a | nc -u localhost 3478


[Firebase/Core][I-COR000003] The default Firebase app has not yet been configured.

[REQUIRED] Step 1: Describe your environment

Platform: Macbook Pro M1
Xcode version: 12.0.1
Firebase SDK version: 7.11.0
Installation method: Carthage
Firebase Component: DynamicLinks, Crashlytics, Auth, Database, Remote Config, Analytics

[REQUIRED] Step 2: Describe the problem
My app was crashed at "FirebaseApp.configure()" when I start to run it.
Here is the console log:
[Firebase/Core][I-COR000003] The default Firebase app has not yet been configured. Add `[FIRApp configure];` (`FirebaseApp.configure()` in Swift) to your application initialization.
Especially It was crashed when I run it on iOS 12.0, 12.4, and iOS 13.7 but it works on iOS 14.0. However, It's not crashed for all the above cases when I test on another laptop (is not M1) with Xcode 12.0.0.
How can I find the problem, please?
It's crashed at
FirebaseApp.configure()

@taihnguyen1210 Sorry for the trouble. It looks like a symptom of Firebase SDK symbols being included twice(or multiple) times in you app binary. Can you see console messages like the following?
objc[40943]: Class FIRApp is implemented in both
~/Library/Developer/Xcode/DerivedData/FrameworkTest-apqjxlyrxvkbhhafhaypsbdquref/Build/Products/Debug-iphonesimulator/DynamicFramework.framework/DynamicFramework
(0x10b2a87f8) and
~/Library/Developer/CoreSimulator/Devices/4821F959-24A6-4D78-A102-4C5703103D99/data/Containers/Bundle/Application/F017D210-113A-4DAF-9E17-BDE455E71E06/FrameworkTest.app/FrameworkTest
(0x10ad2d348). One of the two will be used. Which one is undefined.

See more details on this type of issues and solutions here.

@taihnguyen1210 Sorry for the trouble. It looks like a symptom of Firebase SDK symbols being included twice(or multiple times) in you app binary. Can you see console messages like the following?
objc[40943]: Class FIRApp is implemented in both
~/Library/Developer/Xcode/DerivedData/FrameworkTest-apqjxlyrxvkbhhafhaypsbdquref/Build/Products/Debug-iphonesimulator/DynamicFramework.framework/DynamicFramework
(0x10b2a87f8) and
~/Library/Developer/CoreSimulator/Devices/4821F959-24A6-4D78-A102-4C5703103D99/data/Containers/Bundle/Application/F017D210-113A-4DAF-9E17-BDE455E71E06/FrameworkTest.app/FrameworkTest
(0x10ad2d348). One of the two will be used. Which one is undefined.

See more details on this type of issues and solutions here.

Thanks for your reply. Sorry, I don't see it in the console messages. It just show "[Firebase/Core][I-COR000003] The default Firebase app has not yet been configured. Add [FIRApp configure]; (FirebaseApp.configure() in Swift) to your application initialization."
Btw, Should I clear the DerivedData folder?

@taihnguyen1210 You can try clearing the DerivedData folder. If it doesn't solve your issue we most likely will need a sample project where you issue is reproducible to make any progress on the it.

I cleared the DerivedData folder but it does not work. I am trying to create a sample project to reproduce it. Thanks

I am facing the same issue +1

I try to run on an iOS 12 real device then It works. I think there are some problems with simulators.

@taihnguyen1210 Interesting. There may be a race condition in the app so the timing works differently. Can you share a backtrace of the simulator crash?


feat: use display names for entities instead of ID (FC-0024)

There are two significant changes in this PR:

The existing video_plays and transcript_usage datasets are replaced with templated virtual datasets that allow us to join display names into models in a relatively performant way
Enrollment charts are added to the instructor dashboard

Here is a screenshot of how the dashboard looks (it only has recent local data, hence the very limited visualizations):

Thanks for the pull request, @SoryRawyer!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

@SoryRawyer In Open edX there are courses that are duplicates of others with a difference in the run, this should be available in the selector

And I think we can make the org field required and the other ones optional, the information will be filtered by using the row level security filters

@Ian2012 should the order of filters be: org, course name, then course run?

Also, I was thinking of pulling out the queries into a separate folder and including them using the jinja include macro. I think that'd make the queries easier to edit outside of Superset. What do you think?

I was thinking in just keeping an additional calculated field with that information and people just choose the course they need:
Open Demo Course - 2022
Open Demo Course - 2023
I like that approach, btw the root folder is template folder is templates

@Ian2012 I updated the dashboard configuration to include "run" (and updated screenshot in the PR description).

Okay I just deleted the dashboard/charts/datasets and was able to successfully create everything from scratch.

@SoryRawyer 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


We no longer build or test on OSX 10.10

Signed-off-by: Scott Hain<EMAIL_ADDRESS>Description
OSX 10.10 is deprecated and has been for a while.
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

But this happened in the 14.0 timescale so it's the correct place for them.

@thommay Fair, but we should probably call it out as an update or somesuch then? Adding things right in the middle of published release notes seems bad :)


Threadripper (TRX40) database support

First of all thank you for your efforts in this useful app-
I would like to ask how you have achieved EFi available in your database section for thread ripper CPU
They are quite wrong
if you have time take a look here:
[(https://www.macos86.it/topic/3307-trx40-bare-metal-vanilla-patches-yes-it-worksbutis-proxmox-better/#comments)]
In this thread are available different EFI for different motherboards and CPU
If I may, I would prioritize explaining that for example for threadripper CPUs there is no need to use kernel patches 14/15 (FIX patches) and also the kext section is quite wrong, also quirks could be better and a particular attention should be on MMIO part and initial 3 CPU kernel patches (they need some adjustment because are CPU and motherboard related)
AMD is quite unique as a platform, and not having direct support from the OpenCore developers, it becomes difficult to get it up and running if incorrect information is received.
thanks again for your wonderful work

Thank you very much for pointing out the problem.
Please author @5T33Z0 to answer your question, thank you.

@fabiosun Hi. Thanks for the info. The config used for AMD Ryzen/Threadripper is based on this: https://github.com/luchina-gabriel/BASE-EFI-AMD-RYZEN-THREADRIPPER
Since I don't have an AMD system, I cannot test AMD configs so I have to rely on info that's available. Can you attach an AMD config which is known working so I can adjust the settings? I can't download the ones listed in the thread, since I am not a member of this forum.

@5T33Z0 the site you referred to is very generic and not particularly calibrated on CPU threadripper
Threadripper cpu refer to two different types of sockets and chipsets (x399 and TRX40 the latest ones)
And it is not possible to have a generic (and above all working) EFI
if we use patches, kext and quirks also valid for Ryzen CPU model as X570 or lower chipset compatible
As regards in particular the platform that I have known for some time, namely TRX40, it still needs different functionalities.
The calculation and activation of the MMIO whitelist (which is functional and dependent on various parameters such as Bios version, manufacturers and occupied slots) is fundamental, to then arrive at the definition of specific kext for network cards or patches to be applied for models supported like the Aquantia
If possible, I recommend that you subscribe to the recommended forum, which is the best resource you can find on the web
just one message is enough to download everything
Having said that, I am attaching my config
valid for MSI Trx40 pro, CPU 3970x, Monterey 12.x with this BIOS configuration:
Above 4g on
Resize bar on
Uefi Hon
6900xt GPU
first, third and fourth slot occupied with:
GPU
PCIE Nvme card
Ethernet fenvi wifi / bt card
config.plist.zip
Pay attention.
MMIO whitelist is for this motherboard and configuration
Unfortunately the speech is too long (not complex but long) to be addressed in a github issue :)
attached config for trx40 has only 2 kexts mandatory to have audio or GPU working (Lilu and VirtualSMC)
Ethernet is seen out of box (wifi bt pciexpress one)
Aquantia need a patch and i21x ethernet is problematic in Monterey with available kexts )

Thanks for sharing your config and the explanations. If it is not possible to create a working EFI for AMD based on a config.plist template, my suggestions are the following:

Remove the AMD templates from OCAT (makes no sense to have it there if it generates unusable EFIs.
I think you Fabiosun should create a Report with the necessary changes for the Dortania Guide fro AMD Threadripper: https://dortania.github.io/OpenCore-Install-Guide/CONTRIBUTING.html#contributing-via-issues

Greetings 5T33Z0


Catch some floating point edge cases in clock

This fixes some edge cases where the clock would return an ugly decimal.
It should be a patch bump to 1.0.1.

Coverage increased (+0.005%) to 98.964% when pulling 7e755aaa16c9656540a0ccc6ba622b32fcf42eab on mhkeller:better-numbers into 9d1fe7456ff8577ecd9a3375002832c3a21c27f6 on bloomberg:master.


Pass CA when upgrading cluster through Ansible

When CA is available for provider, use it for the upgrade_cluster role.
Depends on: https://github.com/ManageIQ/manageiq-providers-ovirt/pull/313
This is part of implementing: https://bugzilla.redhat.com/show_bug.cgi?id=1644605

@agrare will you please review?

@miq-bot add-label hammer/yes

Hammer backport details:
$ git log -1
commit 8fb49aa604bfd052bfce9f4e44ed78bf21d95a32
Author: Adam Grare<EMAIL_ADDRESS>Date:   Thu Nov 22 11:51:16 2018 -0500

    Merge pull request #18229 from borod108/rfe/add_ca_ansible

    Pass CA when upgrading cluster through Ansible

    (cherry picked from commit 0598dc16c7e3db8e7ee32a0724e53dd9831bc9cc)

    https://bugzilla.redhat.com/show_bug.cgi?id=1644605


About the pretrained models

Hi Mr. Voigtlaender,
Thank you for your kindness in sharing this great work!
As for the pre-trained models provided on your website (https://omnomnom.vision.rwth-aachen.de/data/siamrcnn/):
hard_mining3: The best model
hard_mining3_onlygot: The model trained only with GOT10k
hard_mining3_resnet50: With a lighter backbone
hard_mining37: Can you help explain this?
Thank you very much!
Sincerely,
Merlin

Hi Merlin,
the model hard_mining37 is trained with different hyerparameters. I forgot the details, but this model is not important, I might actually delete it.
The other 3 models you described correctly.

Understood. Thank you very much, Mr. Voigtlaender!

Hello, is the above pre-trained model of the entire network (including Deep faeture extractor\region proposal network\re-detection head\update tracklets\dynamic programming\box2seg)?
Thank you very much!
Sincerely,
Jiajia

Yes, the models have all necessary weights. TDPA doesn't have learnable weights, just hyperparameters, and they are set by default to the values we used in the paper.

Yes, the models have all necessary weights. TDPA doesn't have learnable weights, just hyperparameters, and they are set by default to the values we used in the paper.

thank you for your reply.

I also want to know where the data set from finding general hard examples for detection in 3.2 is found? Can you provide a download link?

Where can I find the code for hard examples?

In the dashed box of Proposal Generation in Figure 2, is this part of hard examples mining?


Question on the parameters representation in the example of Bayesian NNs

Best greetings, dear authors:
I truly appreciate this package and I am playing with one of the examples (Bayesian NNs).
Honestly speaking, I am not working in this specific field and have limited background on this topic. But I would really appreciate that if you could tell the physical meaning of some evaluation parameters in your provided code (bayesian_nn.py).
Starts from line 86 and ends at line 95, I am wondering what's the role of "lb_samples, ll_samples, epochs, test_freq, anneal_lr_freq, anneal_lr_rate".
Hope it's not much work and thanks for your time!

@Xihaier Hi, Thanks for the question. lb_samples represents how many samples are used to estimate the lower bound, ll_samples represents how many samples are used for importance sampling to estimate the log likelihood. epochs is the number of passes through the training set, anneal_lr_freq and anneal_lr_rate is for anneal the learning rate.

Hi Jiaxin,
It's very helpful!
If I may, could you give me some suggestions on the selection of a set of appropriate setting of the aforementioned parameters? Say if my input data set X is an N by M matrix, with N denoting the number of samples and M denoting the number of features. Y is an N by 1 vector. I am wondering if there are any tricks in the initialization of these parameters in accordance with my dataset X and Y. If not, I am more than willing to go with trial and error method.

@Xihaier In fact for Bayesian NNs we often normalize the input features to be zero mean and unit standard deviation. And for lb_samples, the principle is that more samples you use, you will get a more accurate estimation of the lower_bound. For ll_samples, it relates to the evaluation for test log likelihood, you will typically need hundreds or thousands of them to obtain a reasonable estimate. The other two (epochs, anneal..) are just parameters for every machine learning program with gradient descent. You can tune it just like you do elsewhere.

Thanks a lot, I really appreciate your kind patience and helpful instruction.

You are welcome :)


feat: optional tether prop for DropdownTrigger

Introduces prop tether with value of 'right' or 'left' (default 'left') to DropdownTrigger.
Value of 'right' will tether the dropdown menu to the right of the trigger.

I'm not crazy about the property name "tether." What about "position" or "alignment"?

i would rename the tether prop to placement.

I could go for alignment - it would read better against the concept of Tether in the component, the Tether instance, etc.

placement is what the popover uses

I worry 'placement' of right might convey that the menu should be placed to the right of the button (implying the tether point is on the left).

alignment works for me, makes sense in this case. the dropdown is aligned to the right or left of the trigger

LGTM!

LGTM2
🌸 🍶


[JS][Authoring] Custom fields based on Image not being serialised

Target Application
Twist.com (our own renderer)
Problem Description
I have created a custom component that extends the Image class, the class itself is fairly simple, it's literally just:
export class Imageist extends Image {
    static readonly JsonTypeName = 'Image'
    static readonly aspectRatioProperty = new NumProperty(Versions.v1_0, 'aspectRatio')

    @property(Imageist.aspectRatioProperty)
    aspectRatio?: number

    getJsonTypeName(): string {
        return Imageist.JsonTypeName
    }
}

When I try and use this new field in my application, it doesn't get serialised. Calling getSchema() on it doesn't show the new field at all and I'm unsure why.
I have created a few custom components with additional fields in exactly the same way and all of them have worked without an issue. An example is one based on TextInput
export class TextInputist extends TextInput {
    static readonly JsonTypeName = 'Input.Text'
    static readonly rowsProperty = new NumProperty(Versions.v1_0, 'rows')
    static readonly inputStyleProperty = new StringProperty(Versions.v1_0, 'inputStyle')

    @property(TextInputist.rowsProperty)
    rows?: number

    @property(TextInputist.inputStyleProperty)
    inputStyle?: InputStyle

    getJsonTypeName(): string {
        return TextInputist.JsonTypeName
    }
}

The new fields here (rows and inputStyle) get serialised just fine without having to do anything except use them.  I've actually created custom versions of AdaptiveCard, TextInput, and ChoiceSetInput all with additional fields, and none of them have had an issue when it comes to being serialised.
I wonder whether it has anything to do with the fact none of the others I've done had a base class that overrode the populateSchema function? I actually did try this, and added my own override that explictly adds the aspectRatioProperty to the schema, but it still gets ignored.
Expected Outcome
I was expecting the new fields to be serialised in the same way as other new components I've written.
Actual Outcome
The new fields don't appear in the schema and so aren't serialised.

I forgot to mention, this is using 2.9.0 of the node package.

The plot thickens a little. It turns out that one of the properties that was working before is now no longer working in the TextInputist component (shown above). That property is the rows one. The inputStyle one still works just fine.
So what do these two failing properties have in common? They are both using the NumProperty to create the property static backing field. I have checked at runtime and the isSerializationEnabled property on the static property is set to true.
So because this property is now no longer working, I've gone back through our commits to where it did work, gone back to that specific commit, and it does again work. I've managed to trace it forward from there to a commit I made where the integration's version of adaptivecards package did change (context: we have all our integrations in a monorepo using npm workspaces), so effectively, this integration has gone from using 2.7.0 of the AC package, to 2.9.0 of the package.
So were there any changes between these versions that could have had an adverse affect on the NumProperty class?

@dclaux Do you have any insight into this one?

@scottlovegrove I'm surprised any of your properties would serialize.
Try overriding getSchemaKey and have it return a value specific to your class. For instance, in Imageist:
protected getSchemaKey(): string {
    return "Imageist";
}

Without the override, if for whatever reason your code ends up instantiating the out-of-box Image class, its constructor will build its schema and cache it under its "schema key". Subsequently, when your Imageist class is instantiated, given it inherits Image's schema key, it will use Image's already built schema, which does not include your custom properties. With the override, your class will have its own schema.
As I initially coded this, I wanted to dynamically create a unique schema key for each class extending SerializableObject, but eventually decided to leave it to every class to specify their own schema key. Why? I have no recollection. Room for improvement.
Please let me know if this solves your problem.

THanks @dclaux that fixed it 👍🏻 Is that something I missed somewhere in the docs? Looking at this I can't see any mention of that function. This is basically where I started with our new components.

@scottlovegrove technically, you shouldn't have to override getSchemaKey, that's why I didn't document it explicitly. By default, getSchemaKey returns the same value as getJsonTypeName. But the assumption is that most of the time, any given JSON type name will be used by a single class. Your (perfectly valid) implementation of Imageist uses the same JSON type name as the built-in Image, and so it has the same schema key, which is why in that "special" case you need to override getSchemaKey.


[balsa] Fix 304WithBody test and add new one.

Add //test/common/http/http1:codec_impl_test test case for 304 response with body for both BalsaParser and http-parser.  In this test, the body is dispatched to the parser in the same Parser::execute() call as the headers.  In this case, the parser treats the response as invalid due to the presence of the body.  Note that multiple requests cannot be parsed in a single Parser::execute() call.
Modify ProtocolIntegrationTest.304WithBody by appending '\r\n' to the end of the "body" of the 304 response, necessary for BalsaParser to signal an error.  This is because the "body" is fed into the parser via a separate Parser::execute() call, so it is in fact treated as a second response.
Tracking issue: https://github.com/envoyproxy/envoy/issues/21245
Signed-off-by: Bence Béky<EMAIL_ADDRESS>Commit Message: [balsa] Fix 304WithBody test and add new one.
Additional Description:
Risk Level: low, test-only change
Testing: //test/integration:protocol_integration_test
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/assign @KBaichoo


Self-send amount still "0.00000BTC" in PSBTs panel

We should probably do something about the icon as well. I'll open another issue for that


Don't consume etcd link

Once a new etcd-release is cut with links, we don't want to consume it within cf-release.
Thanks,
Christian and Nick

Hi @nmahoney-pivotal && @christianang, thanks for the PR! The unit tests scripts/test has failed after merging the PR, because our fixture manifests could no longer match generated manifests. We have pushed the fix to those fixture files, so everything is good now :)
In the future, we would appreciate if you could run the unit test first.
The Relint Team

Ah missed that, will remember in the future. Thanks.


fix: Add LegalBench datasets - 16

Checklist for adding MMTEB dataset
Reason for dataset addition: Showcases the model's performance on the LegalBench datasets, enabling evaluation on domain-specific legal texts.
Follow up to #682
Changes Made:
The following 10 datasets have been added in this PR:

Supply Chain Disclosure Best Practice Accountability
Supply Chain Disclosure Best Practice Audits
Supply Chain Disclosure Best Practice Certification
Supply Chain Disclosure Best Practice Training
Supply Chain Disclosure Best Practice Verification
Supply Chain Disclosure Disclosed Accountability
Supply Chain Disclosure Disclosed Audits
Supply Chain Disclosure Disclosed Certification
Supply Chain Disclosure Disclosed Training
Supply Chain Disclosure Disclosed Verification

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[x] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

@awinml already added in #689. However would love your review of that PR.

@KennethEnevoldsen I see, I will review that PR and close this in favour of that.

Closed in favour of #690.


the project file or web cannot be found

hi,
when i load the file #pack my revit plug in#, it doesnt read the file

Hi, For me it's the same "error".Thanks if someone can help us.
Good day Vincentz


Where to get help

Hello,
Posting here cause i do not know where else.
I'me trying to dive in fp-ts (with basic functional understanding), and some questions arose from time to time on the pattern "How can i do this' - i've browse receipe but most of the time I do not find my answer.
Si they re a gitter or a discord where i could ask those ?

@zedryas https://fpchat-invite.herokuapp.com/

Thank  you


no executable permission of Louvain/linux-* files

Hi there!
Just upgraded my PhenoGraph with pip install -U PhenoGraph, and after that I got an error when running PhenoGraph, stating that permission was denied to my [path_to_env]lib/python3.7/site-packages/phenograph/louvain/linux-convert file. Same for the other two linux- files in that folder.
After changing permissions to those three files with chmod +x, the error was solved.
Not sure if it was a glitch in my system or a problem with the PhenoGraph package, but maybe you guys could check :)

I had the exact same issue. You can indeed fix the error easily with chmod +x.


New alpha items

Added 4 items from the metal detector room but im missing one of them
Added 2 new pets i dont think they are 100% right and we are missing uncommon and leg

Mark it as a draft


SPARK-5500. Document that feeding hadoopFile into a shuffle operation wi...

...ll cause problems

Test build #26422 has started for   PR 4293 at commit 78ba008.

This patch merges cleanly.

Test build #26422 has finished for   PR 4293 at commit 78ba008.

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

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

This is good to put. One idea that just came to my mind is ... why don't the downstream operators inspect whether they need to do copys or not?

Even if we can't fix this problem, I wonder if we could override some of the HadoopRDD methods to log warnings when they're called (e.g. cache).

Test build #26446 has started for   PR 4293 at commit 6e1932a.

This patch merges cleanly.

Test build #26446 has finished for   PR 4293 at commit 6e1932a.

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

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

@rxin @JoshRosen I like both of those ideas.  Updated patch implements Josh's .  Reynold's is a little more involved, but would be good to implement down the line as well.

Test build #26522 has started for   PR 4293 at commit cc46e52.

This patch merges cleanly.

Updated patch adds instructions on how to avoid the exception and extends behavior to NewHadoopRDD.
My opinion is still that this deserves an Exception rather than a warning.  It's not a groupByKey kind of situation where there is a more performant choice - users that cache RDDs in this way face fundamental correctness issues.  If I'm missing legitimate uses for this of course I take it back.

Test build #26522 has finished for   PR 4293 at commit cc46e52.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

retest this please

Test build #26525 has started for   PR 4293 at commit cc46e52.

This patch merges cleanly.

Test build #26525 has finished for   PR 4293 at commit cc46e52.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

@sryza the fact that we have a test case failing I think is enough of a reason to not do something too drastic here. How about just changing it to logWarning for now?

Looks like the test failures are legit

Test build #26538 has started for   PR 4293 at commit e9ce742.

This patch merges cleanly.

Ah, so the failing tests expose the possibility that someone could write their own InputFormat that doesn't reuse a single object, and caching that would be fine.  So changing it to logWarning sounds fine to me.
It might also be worthwhile to suppress the warning for BinaryFilesRDD and WholeTextFileRDD, though it would look fairly weird, because persist would need to be overridden in these methods to call the grandparent class's version of the method.  So I can add that if y'all prefer but will leave it out otherwise.

Test build #26538 has finished for   PR 4293 at commit e9ce742.

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

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

I'm merging this in master. Thanks Sandy.


Spurious "[street name] has no tags" error when saving new features

Adding streets and houses from a gpx trace I took, I tried to save it and got an error saying "Wilkshire Road has no tags" but it has 4 tags :-/
Attached:

screenshot of the error showing the tags

changeset file (with a txt extension because github):
changes.txt

This looks like a bug, but I can't reproduce it on my side. Can you tell the exact steps you took to get to this error message?

Sometimes on slow computer the warn may remain for few seconds, I think this is a kind of validator caused dalay with my experience.
( And looks like you are using RapiD, is this also occur in original iD ?

This happens to me regularly in iD. I tend to create quite large changesets with long ways, and after a number of changes iD gets generally slower. It's just a minor annoyance, it tends to disappear after a second or two.


Failed to allocate manager object, freezing

docker pull onlyoffice/communityserver:<IP_ADDRESS>0
docker start with error, would you please help check and give some clue to solve. many thanks !

systemctl enable monoserveApiSystem.service
++ seq 1 1
for serverID in $(seq 1 ${APP_MONOSERVE_COUNT})
index=1
'[' 1 == 1 ']'
index=
systemctl stop monoserve.service
System has not been booted with systemd as init system (PID 1). Can't operate.
systemctl enable monoserve.service
'[' false == true ']'
systemctl enable onlyofficeRadicale.service
systemctl enable onlyofficeTelegram.service
systemctl enable onlyofficeSocketIO.service
systemctl enable onlyofficeThumb.service
systemctl enable onlyofficeFeed.service
systemctl enable onlyofficeIndex.service
systemctl enable onlyofficeJabber.service
systemctl enable onlyofficeMailAggregator.service
systemctl enable onlyofficeMailWatchdog.service
systemctl enable onlyofficeMailCleaner.service
systemctl enable onlyofficeMailImap.service
systemctl enable onlyofficeNotify.service
systemctl enable onlyofficeBackup.service
systemctl enable onlyofficeStorageMigrate.service
systemctl enable onlyofficeStorageEncryption.service
systemctl enable onlyofficeUrlShortener.service
systemctl enable onlyofficeThumbnailBuilder.service
systemctl enable onlyofficeAutoCleanUp.service
'[' SERVER == SERVER ']'
mv /app/config/nginx/prepare-onlyoffice /etc/nginx/sites-enabled/onlyoffice
service nginx stop
systemctl enable nginx.service
Synchronizing state of nginx.service with SysV service script with /lib/systemd/systemd-sysv-install.
Executing: /lib/systemd/systemd-sysv-install enable nginx
++ grep cron
++ grep -v grep
++ awk '{print $2}'
++ ps auxf
PID=
'[' onlyoffice-elasticsearch ']'
service elasticsearch stop

Stopping Elasticsearch Server                                         [ OK ]

systemctl disable elasticsearch.service
Synchronizing state of elasticsearch.service with SysV service script with /lib/systemd/systemd-sysv-install.
Executing: /lib/systemd/systemd-sysv-install disable elasticsearch
rm -f /usr/lib/systemd/system/elasticsearch.service
rm -f /etc/init.d/elasticsearch
'[' -n '' ']'
rm -dfr '/tmp/onlyoffice*'
rm -dfr '/var/run/onlyoffice/*'
rm -dfr '/var/cache/nginx/onlyoffice/*'
'[' true == true ']'
exec /lib/systemd/systemd
[!!!!!!] Failed to allocate manager object, freezing.

Hello @jiayouchongmingniao you need add this parameter to docker run command.
-v /sys/fs/cgroup:/sys/fs/cgroup:rw

solved！thanks ！
modified from relative path to absolute path
-v ./sys/fs/cgroup:/sys/fs/cgroup:rw
to
-v /sys/fs/cgroup:/sys/fs/cgroup:rw


Log official steps from the documentation during deployment

Is your feature request related to a problem? Please describe.
When deploying Cloud Pak for Data using the deployer, it essentially runs the steps that can also be found in the documentation. Make sure the cpd-cli commands are logged in a special "support" file and point to the log files that capture the errors if they occur. This will make it easier for customers reaching out to product support if there is an issue.

[ ] Log the steps, sequentially
[ ] Create a "support statement" for Cloud Pak Deployer

Also for some teams that needs to deliver to clients the installation guide, the commands executed in a list... or something like that


"User Guide"

Added the instructions page + link in the footer.
Decided to call it "User Guide", since neither "Quick Start" nor "Documentation" quite work :)
From google doc:
https://docs.google.com/document/d/1dlRjrHNNFkFHeKjOhM3kzYT9pZrwl3iyyBVFziRxoNs/edit#heading=h.mnarhi6uncjm

Correct link to Google Doc:
https://docs.google.com/document/d/1sNRKZLjJ_KixQV3Bt1rkHxTJVb9KhWx24jMZWroRgrQ/edit?usp=sharing
On Fri, Apr 6, 2018 at 7:52 PM, Pat Shiu<EMAIL_ADDRESS>wrote:

Added the instructions page + link in the footer.
Decided to call it "User Guide", since neither "Quick Start" nor
"Documentation" quite work :)
From google doc:
https://docs.google.com/document/d/1dlRjrHNNFkFHeKjOhM3kzYT9pZrwl
3iyyBVFziRxoNs/edit#heading=h.mnarhi6uncjm
You can view, comment on, or merge this pull request online at:
https://github.com/webrecorder/webrecorder/pull/473
Commit Summary

Adding Quick Start page
Updating footer to User Guide

File Changes

M webrecorder/webrecorder/appcontroller.py
https://github.com/webrecorder/webrecorder/pull/473/files#diff-0 (5)
M webrecorder/webrecorder/static/stylesheets/styles.css
https://github.com/webrecorder/webrecorder/pull/473/files#diff-1
(12)
M webrecorder/webrecorder/templates/footer.html
https://github.com/webrecorder/webrecorder/pull/473/files#diff-2 (1)
A webrecorder/webrecorder/templates/howtoguide.html
https://github.com/webrecorder/webrecorder/pull/473/files#diff-3
(114)

Patch Links:

https://github.com/webrecorder/webrecorder/pull/473.patch
https://github.com/webrecorder/webrecorder/pull/473.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/webrecorder/webrecorder/pull/473, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJRlzv5h_GfLn9CcBgQEAZvMZ1LHitjRks5tl_-kgaJpZM4TK1td
.

--
Anna Perricci
Associate Director, Strategic Partnerships, Webrecorder at Rhizome
Rhizome http://rhizome.org/ at the New Museum
webrecorder.io | oldweb.today

@m4rk3r I've updated to clean up comments


Support Amazon API Gateway monitoring

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

Maybe we could close https://github.com/apache/skywalking/issues/9883 ?

Maybe we could close https://github.com/apache/skywalking/issues/9883 ?

Yes, we should.

In your screenshot, there is an endpoint called GET /students. Is this your route rule?

The CI would be blocked until https://github.com/apache/skywalking/pull/10643 is done.


if ''/dist'' folder has some folders inside, this plugin shows some error

Issue description or question
this is my dist folder

when I npm run build, this plugin first show some errors like this

and my css folder disappeared.

then I npm run build again, same error and my js folder disappeared,
then I run it again, the process succeed.
I wonder if I shouldn't have folders in my dist folder.
Webpack Config
const HtmlWebpackPlugin = require("html-webpack-plugin");
const MiniCssExtractPlugin = require("mini-css-extract-plugin");
const TerserPlugin = require('terser-webpack-plugin');
const CleanWebpackPlugin = require("clean-webpack-plugin");
const OptimizeCSSAssetsPlugin = require("optimize-css-assets-webpack-plugin");
const webpack = require('webpack');
const WorkboxPlugin = require('workbox-webpack-plugin');

const path = require("path");
module.exports = {
    mode: "production",
    entry: {
        main:["@babel/polyfill", "./src/index.js"],
    },
    output: {
        path: path.resolve(__dirname, "dist"),
        filename: "js/[name].-[contenthash].js",
        publicPath: "/" /*打包完成时当前目录下的，一般情况下publicPath应该以'/'结尾，而其他loader或插件的配置不要以'/'开头*/
    },
    module: {
        rules: [
            {
                test: /\.css$/,
                use: [
                    {loader: MiniCssExtractPlugin.loader,},
                    {
                        loader: "css-loader",
                        options: {
                            modules: true,
                            localIdentName: "[path][name]__[local]--[hash:base64:5]",
                            sourceMap: true, /*因为打包后的文件都是被hash过和压缩过,所以当生产环境出问题了,可以方便的定位到原位置上*/
                        }
                    },
                    {loader: "postcss-loader"}
                ],
                exclude: /node_modules/,
            },
            {
                test: /\.css$/,
                use: [
                    {loader: MiniCssExtractPlugin.loader,},
                    {loader: "css-loader",},
                ],
                include: /node_modules/,
            },

            {
                test: /\.less$/,
                use: [
                    {loader: MiniCssExtractPlugin.loader,},
                    {
                        loader: "css-loader",
                        options: {
                            modules: true,
                            localIdentName: "[path][name]__[local]--[hash:base64:5]",
                            sourceMap: true,
                        }
                    },
                    {loader: "postcss-loader"},
                    {
                        loader: "less-loader",
                        options: {
                            sourceMap: true
                        }
                    }],
                exclude: /node_modules/,
            },
            {
                test: /\.less$/,
                use: [
                    {loader: MiniCssExtractPlugin.loader,},
                    {loader: "css-loader",},
                    {
                        loader: "less-loader",
                        options: {
                            sourceMap: true
                        }
                    }],
                include: /node_modules/,
            },

            {
                test: /\.m?js$/,
                exclude: /(node_modules|bower_components)/,
                use: {loader: "babel-loader",}
            },
            {
                test: /\.html$/,
                use: [
                    {
                        loader: "html-loader",
                        options: {
                            minimize: true
                        }
                    }
                ]
            },
            {
                test: /\.(png|jpe?g|gif|svg)(\?.*)?$/,
                use: [
                    {
                        loader: 'url-loader',
                        options: {
                            limit: 8192,
                            name: '[hash].[ext]',// 可以在这里改为images/[hash].[ext]新建一个文件夹,这样就不需要outputPath: 'images/'了
                            outputPath: 'images/'
                        }
                    }
                ]
            }
        ]
    },
    optimization: {
        minimizer: [
            new TerserPlugin({
                cache: true,
                parallel: true, /*启用cpu加速打包速度*/
                sourceMap: true, // set to true if you want JS source maps/*帮助快速查找问题所在的地方*/
                exclude: /node_modules/
            }),
            new OptimizeCSSAssetsPlugin({})/*这个插件是用来压缩css代码的,和postcss里面的cssnano是一样的,都能去除注释, 这个比用cssnano好*/
        ],
        namedChunks: true,
        runtimeChunk: true,
        splitChunks: {
            chunks: 'all',
            maxInitialRequests: Infinity,
            minSize: 0,
            cacheGroups: {
                vendor: {
                    test: /[\\/]node_modules[\\/]/,
                    name(module) {
                        // get the name. E.g. node_modules/packageName/not/this/part.js
                        // or node_modules/packageName
                        const packageName = module.context.match(/[\\/]node_modules[\\/](.*?)([\\/]|$)/)[1];

                        // npm package names are URL-safe, but some servers don't like @ symbols
                        return `npm.${packageName.replace('@', '')}`;
                    },
                },
            },
        },
    },

    plugins: [
        new CleanWebpackPlugin(),
        new HtmlWebpackPlugin({
            template: "src/index.html",
            // favicon:'./src/component/20180806050252871.ico',
        }),
        new MiniCssExtractPlugin({
            filename: "css/[name].[contenthash].css",// filename代表的是没有异步加载的css和从node_modules里面分离出来的css那部分css, 简单来说就是初始打包成的那块css
            chunkFilename: "css/chunk.[id].[contenthash].css",// chunkFilename代表的是从node_modules里面分离出来的css. 还有react-loadable异步加载的css
            // allChunks: true,
        }),
        new webpack.HashedModuleIdsPlugin(),
        new WorkboxPlugin.GenerateSW({
            // these options encourage the ServiceWorkers to get in there fast
            // and not allow any straggling "old" SWs to hang around
            clientsClaim: true,
            skipWaiting: true
        }),
    ]
};

Environment
win10,     "webpack": "^4.30.0",
"webpack-cli": "^3.3.0",,     "clean-webpack-plugin": "^2.0.1",
Run: npx envinfo --system --binaries --npmPackages clean-webpack-plugin,webpack
# Paste the result here

this problem  has been bothering me for many days, please help me out

You can definitely have folders in your clean target. This looks like a permissions issue at first glance.

How to define it? I can't  find any clue in the docs

The issue is most likely with windows filesystem. There is a long history of EPERM errors with rimraf.
It is also possible you are having legitimate permission errors like @johnagan suggested, but if subsequent cleans are successful, this seems unlikely. How are you starting/stopping the build? Are you doing it inside vscode?
Is it possible for you to create a minimal reproducible repository with exact instructions how to reproduce?

It's really weird, I come back to my project after a few days, this problem seems solved automatically, but I'll keep watching, once this problem shows up again, I'll create a repository and let you know, thanks for the help

Thanks for the update @LastStranger. Reopen this issue later if you need to.


Fixing bugs with groundwater, irrigation, soil bunds, canopy and root development

I was comparing AquaCrop-OSPy based on AquaCrop v6.1 (2022) vs v7.0 (2024) and found several bugs including:

check_groundwater_table.py doesn't return NewCond_zGW variable leading to constant groundwater levels (see #99)
HIref_current_day.py doesn't check for NewCond_CC_prev <= (Crop.CCmin*Crop.CCx) condition as it used to do in AquaCrop-OSPy v6.1 (perhaps it's how new AquaCrop v7 works, please confirm)
irrigation.py has hardcoded 15mm irrigation for IrrMngt_IrrMethod == 1 (see #132)
root_development.py doesn't return NewCond_rCor which is supposed to be updated daily according to AquaCrop-OSPy v6.1
infiltration.py has soil bunds multiplied by 1000, although they are provided in mm already

The changes to run_single_timestep make sense and will be approved.
The changes to infiltration are incorrect, because the zBund is supplied in metres so should be converted to millimetres (*1000) during these calculations. Please either remove these changes or provide evidence to support the removal of the '*1000' before resubmitting this commit.

Thanks :)
zBund in m would be fine if its use was consistent through the code. In some cases, it's compared to other variables in mm which leads to a mismatch in units. E.g:

FieldMngtTmp.z_bund in read_model_initial_conditions.py
FieldMngt.z_bund in reset_initial_conditions.py

Since the whole water balance is done in mm, we can make it consistent with zBund too so the users are less confused. Also, it's a bit strange that zBund is in m but bund_water (initial storage) in mm. Alternatively, you can search through all instances of zBund and z_bund to make sure it's everywhere multiplied by 1000. But again, this adds unnecessary operations to the code.

First of all, thank you for spotting a number of discrepancies in the code! Here are our comments:
In HIref_current_day, we purposefully made and tested the removal of L106-108 in this commit as part of the update from v6 to v7. Please remove this change before resubmitting.
The return of zGW in check_groundwater_table is correct and will be accepted.
The removal of hard-coded value in irrigation is correct and will be accepted.
The return of rCor in root_development is correct and will be accepted.
In run_single_timestep, all changes are correct apart from L273 which you have already fixed in the subsequent bunds-related commit.

I removed lines in HIref_current_day, thanks.


Add usage example with luasocket or luv (libuv)?

Hi, I like how lightweight mbedtls is and want to use it in my projects but unfortunately bindings seem to be missing some examples. Would be great to see the usage of it with either luasocket or luv.

Using lua-mbedtls with any I/O transport (both blocking and non-blocking) is pretty easy, but depends on the underlying I/O API.
In case of LuaSocket, a simple example might look like this:
local socket = require 'socket'
local ssl = require 'mbedtls.ssl'

local function read(h, n)
	return assert(h:receive(n))
end

local function write(h, d)
	return assert(h:send(d))
end

local tcp = assert(socket.connect('github.com', 443))
local cfg = ssl.newconfig('tls-client')
local ctx = ssl.newcontext(cfg, read, write, tcp)

ctx:write('GET / HTTP/1.0\r\n\r\n')
print(ctx:read(1000))
ctx:close()
tcp:close()

Output:
HTTP/1.1 301 Moved Permanently
Content-Length: 0
Location: https://github.com/
connection: close

Please note that the above code is primitive for simplicity, but I hope you'll get the idea.
As for luv, making a similar example is not that easy due to the nature of libuv's data reception strategy - it receives as much as currently available whereas Mbed TLS works with chunks of size N (more like the usual recv(2)). But with certain skills, one can implement a simple receive queue in order to wrap around Mbed TLS API. I've done that in my Lua projects that use libuv under the hood.

I added this example code to the README: https://github.com/neoxic/lua-mbedtls#usage-example-ssltls-over-luasocket


Puppeteer?

Just curious, was this library initiated before or after puppeteer?
Do you see the two libraries living alongside each other or do they serve the same purpose?

Hi!
To be honest, I don't know. I heard of puppeteer about a month ago but I started this on May.
I think you shouldn't have any problem with both libraries living alongside, but they do the same thing.
Which you may use depends on which of them you like more (maybe the syntax or the abstracted methods) .
Cheers!
Lucho


use correct get process user function

see #846

Hmm... throwing an exception is worse than the problems this is trying to solve. If get_current_user returns nothing, we can md5 the empty string.
Did you run into an error where you used sudo and then later couldn't access the files as a web user?

Test Program(test.php) owner 'develop'.
require_once '/path/to/your-project/vendor/autoload.php';

umask(0);
$cache = new Google_Cache_File(__DIR__ . '/test');
$cache->set('test', '123');

run script use original Google_Cache_File, raise error.
same name directory create, because get_current_user() returns same value 'develop'.
develop% sudo php test.php
develop% php test.php
PHP Warning:  mkdir(): Permission denied in /path/to/your-project/src/Google/Cache/File.php on line 176

great! thank you for that. If you remove the exception, I will merge this.

remove the exception, change log-level.


UIKit: Make rootView autoResize its subviews

This fixes the view size updates on screen rotation
Tested in compose/mpp/demo:

I'll think about an automated test for this. It doesn't seem to be easy to create such a test now, so I don't want to block this PR waiting for the test.


Update Twitter logo that is 'X'.

Feature Request
Short Description
Update Twitter logo with new one marked in screenshot below.

Is your feature request related to a problem? Please describe the use case.
UI/UX related.
Describe the solution you'd like
Update with new one

Heyy @Ayush9026  I would like to work on this issue. Will you please guide me to the repo link for the website? I tried but couldn't find one.

Thanks for noticing this!
Accuknox Website is not managed by the KubeArmor community. But I have informed the team to take note of it.


fix: light changes for halloween

 light improvements halloween
What?
light improvemnt
Why?
interiors look very dark

Closing because commit list got messed up by feat/halloween-2020 merge. Reopening it as #1500


Cannot use a plugin in the titleTemplate function

Environment

Operating System: Windows_NT
Node Version:     v20.1.0
Nuxt Version:     3.5.2
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      modules, app, typescript, vite, experimental
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
I have a plugin for i18n which uses "i18next": "^22.5.0","i18next-vue": "^2.1.1",
export default defineNuxtPlugin(({ vueApp }) => {
	vueApp.use(I18NextVue, { i18next });
	i18next.init({ fallbackLng: defaultLocale, resources });

	function translate(key: TranslationKeys) {
		const { t } = useTranslation();
		return t(key);
	}

	return { provide: { i18n: i18next, translate } };
});

Describe the bug
But when I use my plugin like this
const { $translate } = useNuxtApp();
useHead({ titleTemplate: (t) => `${t} / ${$translate('meta.home.title')}` });

I get the following error.

This works fine tho
const { $translate } = useNuxtApp();
useHead({ titleTemplate: `%s / ${$translate('meta.home.title')}` });

Additional context
No response
Logs
[frontend] [nuxt] [request error] [unhandled] [500] i18next-vue: No Vue instance in context. Make sure to register the i18next-vue plugin using app.use(...).
[frontend]   at Module.useTranslation (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/i18next-vue/dist/index.mjs:118:11)
[frontend]   at translate (C:\Users\ckuka\VS Code Projects\ows-events\frontend\plugins\i18n.ts:23:41)
[frontend]   at titleTemplate (C:\Users\ckuka\VS Code Projects\ows-events\frontend\pages\index.vue:43:39)
[frontend]   at resolveTitleTemplate (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/@unhead/shared/dist/index.mjs:79:12)
[frontend]   at tags:resolve (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/unhead/dist/index.mjs:59:28)
[frontend]   at /C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/hookable/dist/index.mjs:48:66
[frontend]   at /C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/hookable/dist/index.mjs:48:56
[frontend]   at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
[frontend]   at async Object.resolveTags (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/unhead/dist/index.mjs:995:7)
[frontend]   at async Module.renderSSRHead (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/@unhead/ssr/dist/index.mjs:77:23)
[frontend] [nuxt] [request error] [unhandled] [500] i18next-vue: No Vue instance in context. Make sure to register the i18next-vue plugin using app.use(...).
[frontend]   at Module.useTranslation (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/i18next-vue/dist/index.mjs:118:11)
[frontend]   at translate (C:\Users\ckuka\VS Code Projects\ows-events\frontend\plugins\i18n.ts:23:41)
[frontend]   at titleTemplate (C:\Users\ckuka\VS Code Projects\ows-events\frontend\pages\index.vue:43:39)
[frontend]   at resolveTitleTemplate (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/@unhead/shared/dist/index.mjs:79:12)
[frontend]   at tags:resolve (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/unhead/dist/index.mjs:59:28)
[frontend]   at /C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/hookable/dist/index.mjs:48:66
[frontend]   at /C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/hookable/dist/index.mjs:48:56
[frontend]   at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
[frontend]   at async Object.resolveTags (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/unhead/dist/index.mjs:995:7)
[frontend]   at async Module.renderSSRHead (/C:/Users/ckuka/VS%20Code%20Projects/ows-events/frontend/node_modules/@unhead/ssr/dist/index.mjs:77:23)

The issue here is that we don't have access to the runtime Nuxt context within renderSSRHead as it is an async method.
cc: @harlan-zw

Would you be so kind to offer a work-around? It's not a deal-breaker, %s works fine for now but would be nice to be able to use the function-way.

Hi, I don't really understand the problem if you already have a workaround with %s.
Template params are the expanded way to work around this problem.
If you want to use a function can't you just reference the $t function off of the existing i18n instance instead of re-instantiation with useTranslation() each time? (my i18n module knowledge is limited). I personally don't really see the value of using a function if the data isn't reactive though.

The data is reactive because it depends on the currently selected language. I'll look into suggested solutions, thanks!


Prometheus metrics endpoint

Would be cool the have  Prometheus metrics served via /metrics to provide insights, e.g. number of webhooks processed etc.

@bufferoverflow out of curiosity, what does your ideal deployment look like? I went this route early on, but when deploying into an environment with varying #'s of transient buz instances (gcp cloud run, lambda, etc) it provided limited value (since it was a large # of small instances).

Right now we plan to use buz as webhook receiver for GitLab System Hooks, see also https://gitlab.com/gitlab-org/gitlab/-/merge_requests/118616 to make it work smooth in combination with buz. To monitor /health is helpful but as we use  Prometheus all over it would be cool to also get insights on buz running on the machine.

Sweet, makes perfect sense! Thanks for the insight.


feat: dominikh/go-tools/staticcheck

Added package from:

dominikh/go-tools

See also: https://staticcheck.io/docs/getting-started/#installation

Thank you for your contribution.

@takumin Will you create pull requests soon?
If so, I'll release v2.7.0 after merging those pull requests.
If you won't create pull requests soon, I'll release v2.7.0 soon.

@suzuki-shunsuke
Sorry, we need to debug it, so you can release it :bow:

I see. No problem. Thank you for your contribution.

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v2.7.0


bug(checkbox, radio, toggle): inhomogenous ripple effects and other styles

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
14
Description
In v15 the ripple effect for radio differs from checkbox/toggle etc. - the radio has a short transition - would be nice to have same styling on every material elements (e.g. remove the transition - because then it is matching mat-button / mat-icon button as well as all other elements).
In v15 the radio and the slide toggle have no pointer cursor on hover the label (in v14 there was a pointer cursor)
In v15 the focus indicator/ripple on hover of checkbox and radio are 40x40px and not matching the new touch element wich is 48x48px. It would be more homogenous if the focus indicator of checkbox and radio match 48x48px - other elements like toggle, Slider, of course all mat-icon-buttons etc are 48x48px. 40x40px looks a bit weird if all elements are near together like in https://material.angular.io/components/tree/examples.
the new "more contrast" on hover for checkboxes and radio is not homogenous with all other controls (e.g. icon-buttons, menus etc.) as well - how to deactivate this new behaviour?
 On the later one the ripple is activated on clickDown and clickUp - if it was choosen before. Would be nice to have the same focus/ripple experience (only on click up) as on checkbox/toggle
Reproduction
Steps to reproduce:

just head to the docu and try the different elements in coparison:
e.g.

https://material.angular.io/components/radio/overview
https://material.angular.io/components/checkbox/overview
https://material.angular.io/components/tree/examples (tree with checkbox)

Expected Behavior
homogenous styling / behaviour
Actual Behavior
different styling / behaviour
Environment

Angular: 15
CDK/Material: 15

I have overwritten some css styles in my projects to circumvent the most (for me annoying) regressions / changes (mainly cursors for radio labels, suppress the new "more contrast style" on hover/checked stylings, bigger 48px ripples and hovers for radio and checkbox etc.).(mainly because i made the switch to v15 and the effort to )
However the radio ripple behaves still very different from the checkbox ripple.
The radio ripple is animated on enter and on focus leave and double ripples if chosen and clicked on label.
In the process of monkey patching the new mdc component styles i found other bugs in v15:

if you click right on the edges of a radio or checkbox (so that you click on the 48x48px touch div, but not on the 40x40px ripple) - the remaining focus indicator is different (the opacity is lesser - no reaminig focus is triggered), than if you click on the 40x40px ripple/input div).
the checkbox ripple is not triggerd, if you click on the label  - than there is just a background but no ripple animation.
on the slide toggle the remainig focus is buggy, too: if you click the label, the focus persists. if you click on the toggle itself, there is no remaining focus indication.

For anyone running into the same issue, I used this CSS workaround to hide the radio ripple:
.mat-mdc-radio-button.mat-accent {     --mat-mdc-radio-ripple-color: transparent;     --mat-mdc-radio-checked-ripple-color: transparent; }


Widespread Android Exception: Cannot read property 'push' of undefined

Problem
@Expensify/mobile-deployers
There seems to be a new crash in NewDot a recent release. The email mentions version<PHONE_NUMBER>, but this actually started occuring in version<PHONE_NUMBER>  1.4.29-0, which was a couple of releases ago... was the Firebase checkist step missed somehow?

Here's the Firebase exception & stack trace:
Fatal Exception: com.facebook.react.common.JavascriptException: TypeError: Cannot read property 'push' of undefined, js engine: hermes, stack:
use@1:2023797
anonymous@1:2064016
loadModuleImplementation@1:162235
guardedLoadModule@1:161468
metroRequire@1:160751
anonymous@1:2061618

I'm sick today and just completing chores, so can't spend any more time here. I didn't add the blocker label as this has existed in two prior versions, but we may want to block the next release until this is resolved as it seems pretty bad.
Solution
Locate the issue that introduced this.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0131326c989b3bc942
Upwork Job ID:<PHONE_NUMBER>789500416
Last Price Increase: 2024-01-24

@Julesssss this is deploy blocker, came from https://github.com/Expensify/App/pull/32326#issuecomment-1907639153

reverting here

Ah thank you

Okay, so we're all good here. We need to improve our Firebase exception though, as this major regression is currently on prod. I'll leave the issue open for a bit in case anyone has any suggestions.
If we had time, showing all newly occurring exceptions in the checklist would be a nice improvement. We could then require manual review and comment for each new exception.

We need to improve our Firebase exception though, as this major regression is currently on prod

I believe this only made it to staging, the PR that introduced the crashes were added to staging - crashing was found and then reverted in this same deploy https://github.com/Expensify/App/issues/35037
I also handled the deploy yesterday, and this specific crash did not show up on firebase

Okay yeah, we did release this to production. Thankfully only 6 users experienced the crash. I still think we need to prevent this from happening as if it was released on a weekend or post-reunification it would have been much worse.

I think thats my bad, when it comes to the first occurrence, sorry! I saw it happening in staging but it was isolated to only OnePlus devices so that one seemed as device specific and i did not want to block deploy for it.
Since it then disappeared i did not create an issue for it.
The second occurrence was obviously much more critical so thanks for flagging

There's no blame, I would have done the same if it only occurred on a single device.
We'll need to be careful once we deploy to 1000* more users. The 7-day rollout will help, but it would still be better to catch these before we release to prod.


Accelerometer bias correction using vision in position_estimator_inav

I think the way the mocap based accel bias correction was added causes vision based bias correction to have no effect, whether or not mocap is being used. The problem is here:
https://github.com/PX4/Firmware/blob/master/src/modules/position_estimator_inav/position_estimator_inav_main.cpp#L1030-L1032
Bias correction values are being reset before they were actually incorporated into the accel bias estimate further down in code.
Please note that I haven't been able to verify this through experiments yet. But this seems a very straightforward issue.

It would be fantastic if you could send a PR with a fix in case this turns out to be required. You certainly seem to have a point based on the high-level analysis.

Outdated.


Closing tooltip on click outside body

Currently only clicks on elements other than the tooltip inside the body are closing it.
I have a page where I display the tooltip at the bottom of the page and instinctively everybody clicks on the bottom, outside the body to close it and nothing happens.
The code responsible for this behavior seems to be the following:
$body.on(eventNames, function(event) {

        if (self._touchIsMeaningfulEvent(event)) {

            self._touchRecordEvent(event);

            if (!self.__options.interactive || !$.contains(self._$tooltip[0], event.target)) {
                self._close(event);
            }
        }
    });

Could you please handle clicks outside document body ?

Hi, right, we're using the body element to catch events, using document could do the trick.


It will crash if the response data is empty

it will crash if the response data is empty :-
the fix  go to line 110 and replace:
$data = $response->getData(true);
by
$data = $response->getData(true) ?: [];
that should do it..

Thank you! I'll fix it in a while.

Fixed


what is the best configuration to train a style

people train on people and objects but I haven't seen tutorials to train on a certain style.
I would like to train an "X" style, for example Salvador Dali
is this configuration ok?
dataset: images of salvador dali
SUBJECT_NAME: style
INSTANCE_NAME: salvador dali style

SUBJECT_NAME : Style
INSTANCE_NAME : (a unique identifier like a password that SD doesn't recognize)  eg: Saldalstyl

@TheLastBen Where can we set SUBJECT_NAME?
It's something different than session name?

the session_name is just a name for the working session, it has no relation to the training, with the new method, you don't have to specify the word style, just rename the pictures into a unique identifier eg: "Saldalstyl"


Update vagrant to 1.9.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Closing as dupe of #30455


HTML Home Page

USER STORY

AS A user I WANT to view a clean and polished UI SO THAT I can navigate easily
ACCEPTANCE CRITERIA

GIVEN I want an experience
WHEN I visit the home page
THEN I can see a clean and polished UI
WHEN I click on the nav links for user input or nutrition information
THEN I am redirected to the appropriate section
WHEN I view on different screen sizes
THEN the application is responsive

Adjusted font sizes, margins, and grid-template layout for HTML tables to respond to different screen sizes.


Every change in tile_buildings2 crash the game

Problem is in a subject. I don't see any log file so I will describe my actions one by one.

I've extracted images from tile_buildings2.gm1.
I've changed 2 pictures of granary in the "Images" folder on my custom ones.
I've imported images and created a new gm1 file.

After all that my game crashes while loading. What have i done wrong? ¯_(ツ)_/¯

Can you send me you edited picture?
Did you use the orginal Stronghold files, if you have gm1 files edited by imagetoolbox in your GM1 location the maybe wrong and to big and it will crash.(best is to reset all GM1 files and edited new with my tool, because files have issues if they get to big togheter or witb other edited files.
Send me your edited picture and i test it for you.
Log files not working right now it will be implement soon.

To test the program i have used original Stronghold 1 GM1 file and some building sprites from Stronghold Russian Story. Here's the file:
https://www.upload.ee/files/9980275/tile_buildings2.gm1.html

For me you File works with Stronghold Crusader Extreme, hm i need to check it with Stronghold 1 too, i will test it with Stronghold 1

or you modified some with Toolbox?

Nope, I've reinstalled fresh Stronghold especially for modding. :)

Ok i installed now Stronghold 1 and there it wont load the GM1 File(also not the orginal from Stronghold Crusader), but in Stronghold Crusader it works. I will look at the diffrents between the GM1 Files in Stronghold 1 and Stronghold Crusader. Thanks for reporting the issue, i look at the diffrents here so you can also edit Stronghold 1 GM1 Files.

OK, thank you. Also, I have made some testing and edited tile_churches.gm1 works just fine for me.

Can you send me the .png files you edit so i can add them and make the GM1 File? I just edit my orginal Stronghold 1 file and that worked? But your not working.
Maybe the Problem is you need to start the Converter as Administartor and than it will work?

Here they are:
https://www.upload.ee/files/9980978/Images.zip.html
I've tried to run the Converter as admin, but it didn't changed anything.

Hey Cybercorpse,
i think i found the issue in my Code, i dont know why but Stronghold 1 want some specific variabels to be set, so i do this now in my Programm(its just a hotfix solution, but works well, i will go in depth later for the issue).  Just Donwload from the new Release: https://github.com/Gaaammmler/Gm1KonverterCrossPlatform/releases


[Question] hipDeviceProp' major/minor for AMD GPUs

Hi,
I have 2 GPUs: GTX 1650 (Nvidia, Turing arch) and RX 5500 XT (AMD, gfx1012) + rocm 4.5.0.
If i start the program with Nvidia GPU and populate hipDeviceProp_t, i get major & minor compute capability = 7 and 5 accordingly (in total -> 7.5), and that's completely fine.
if start the same program with Radeon, i get 10 & 1 -> 10.1.
What do the values of major/minor attributes in hipDeviceProp_t mean for AMD GPUs?
RX 5500 XT has compute capability of about 10.1?

Hi @HardMax71, apologies for not following up.  Is this ticket still open?  Thanks.

Closing the ticket.  @HardMax71, please re-open if your query still not fully answered.  Thanks.


local changes present right after git lfs clone

This seems to be related to a known issue, but I can't seem to find good documentation as to what should be happening (or a link to the actual known issue).
I clone a repo with git lfs clone, I enter it and the status shows that all the lfs managed paths have local, uncommitted changes. There's a sentence in the documentation that says "this is known git issue and... well... deal with it by doing ______" so I assume it's safe to ignore. However, say I want to be working on a different branch. I cannot switch branches, because of my "local uncommitted changes", so I can't actually ignore it.
Should I be committing those changes? If so, won't that end up with one extra commit from each person who clones the repo, all of them committing the same changes to the same files? If I'm not supposed to commit them, what should I do?
Say I commit them, or stash them, or whatever... when I switch to the other branch (that I've been itching to switch to) I have to do git lfs fetch and git lfs checkout . in order to get my lfs files for that branch, but once that's done, yet again, there are "local uncommitted changes" that I have to deal with.
Am I doing something wrong or is this the way git-lfs is supposed to handle cloning repos?

Ideally, you won't see changes immediately after cloning. Can you see what the change is? Are these files that aren't using LFS, but match file paths in .gitattributes for LFS?

They are definitely all the files that match .gitattributes (images) but I don't think they're not using LFS, since they only appear after running git lfs fetch and git lfs checkout, so they are certainly coming form the LFS server.

Try creating a temporary branch and committing. What's in the commit?

Ok, I just did the following:
git checkout -b temp
git commit -am "test commit"

and the result was:
[temp 6e354b8] new commit
 998 files changed, 3122 insertions(+), 3 deletions(-)
 rewrite content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail.jpg (100%)
 rewrite content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail_big.jpg (100%)
 rewrite content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail_bw.jpg (100%)
...

Might be time to post your .gitattributes contents.

What does git show say?

@javabrett there is one .gitattributes file in every folder that contains images (I believe this is the doing of bfg -git-lfs) and they all have the same content:
*.{jpg,png,gif,jpeg,psd,xcf,pdf} filter=lfs diff=lfs merge=lfs -text

@technoweenie git show, after the commit (I expect that's what you meant), is of this form:
commit 6e354b8402e76ca3c656a016a3a967691b0b15ad
Author: Vlad Seghete<EMAIL_ADDRESS>Date:   Thu May 19 18:27:36 2016 -0500

    new commit

diff --git a/content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail.jpg b/content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail.jpg
index 0019587..1ac3fef 100644
Binary files a/content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail.jpg and b/content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail.jpg differ
diff --git a/content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail_big.jpg b/content/articles/blog/a-taxis-suspension-of-judgement/cab_thumbnail_big.jpg
index 89906fa..d3ca401 100644
...

Ah yes, the .gitattributes helps a lot (side note: this project sorely needs an issue template). That is an invalid file pattern that bfg writes. Here's an issue about it: https://github.com/rtyley/bfg-repo-cleaner/issues/116.
You may be better off trying https://github.com/bozaro/git-lfs-migrate to migrate your repository data.

Hmm, so the issue is that git and/or git-lfs does not always deal with bracket globbing in the .gitattributes file? If that's the case I think that closes this and open a PR with the documentation for bfg

Git LFS doesn't use the file, only Git. Though, the issue is that bfg is creating invalid .gitattributes files.


Make trend chart a running total

Trend chart was meant to be a running total not the value of each day.
Not sure what made me not realize this with the first pr.....

Deploy preview for koku ready!
Built with commit 735b27f16f79ac6023a0a17aee4e5148a0166f9e
https://deploy-preview-59--koku.netlify.com

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2fe00e3). Click here to learn what that means.
The diff coverage is 75%.

@@           Coverage Diff            @@
##             master     #59   +/-   ##
========================================
  Coverage          ?   88.5%
========================================
  Files             ?      86
  Lines             ?     722
  Branches          ?      61
========================================
  Hits              ?     639
  Misses            ?      78
  Partials          ?       5

Impacted Files
Coverage Δ

src/components/trendChart/trendChart.tsx
84.61% <ø> (ø)

src/components/trendChart/trendChart.styles.ts
100% <ø> (ø)

...rc/components/reportSummary/reportSummaryTrend.tsx
80% <0%> (ø)

src/components/trendChart/trendChartUtils.ts
100% <100%> (ø)

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 2fe00e3...735b27f. Read the comment docs.


Update README

Information about license and building tools is missed and some grammar issues.

Thanks for the revision of the readme file.


Statuses in timeline

I tried to play with localization and stuck with passing t() objects into VIS component as application fails. Have to find a way to resolve localization in runtime  and pass translated values. Also, we have not localized action and statuses names  regarding to game locale settings.

So, I have added ability to define Statuses module per job. In this case each status can be configured with additional options, at this moment it just provides possibility to map it to other status so they can stack at same row, but it can be good place to provide color settings for each status as well.

It looks like you might have some auto-formatting config running in your editor, potentially? (i.e. changes in Cooldowns.js) - try to avoid running autoformatting that isn't encoded in the eslint/tslint config in the repo, as it just generates diff kludge.
Yeah, resharper in VS 2017. makes life easier, but must be configured for this project separatelly.


added pr and issue template

fixes #18

This is an empty ISSUE & PR template

Also make sure to stay updated to our main branch.
Your branch ami is behind our main branch

Also make sure to stay updated to our main branch.
Your branch ami is behind our main branch


Support Raw textures in Babylon Native

Support raw textures in Babylon Native to enable RGBA buffers to be used as textures on shaders/materials.
Should currently work in Babylon already, work is to bring support to Babylon Native.

bjs PR : https://github.com/BabylonJS/Babylon.js/pull/9612
BN PR : https://github.com/BabylonJS/BabylonNative/pull/546


Go to changes link is broken

Describe the bug
From the Changes detection tab, if you click on Go to changes button it will redirect to the old /changes?test=... url which is not working anymore.
To Reproduce
Open change detection tab and client Go to changes:

Then you should see:

Version
What is the version of Horreum ?
From main branch
$ git rev-parse HEAD

a3ed1b101bb8e0de9ac7cdf390dd6391ed02e2a6

Given that the changes page has been moved under the corresponding Test (as a tab), we cannot simply redirect the location to /test/<testId>#changes as this won't work because the tabs selection is managed internally by FragmentTabs component and there is no way (at the moment) to programmatically change the selected tab from outside of the FragmentTabs itself.
On the other hand I don't even think we still need this "Go to changes" button because, at the moment, both pages reside in the same Test page.
So, I'd propose to completely get rid of the "Go to changes" button as users can simply select the "Changes" tab from that page, wdyt @johnaohara ?

@lampajr One of the main ideas behind the re-org of the UI was to remove the need for these "Go To" buttons that were in a lot of places of the UI, so I am +100 to remove it!


ChainedTokenCredential should skip credentials throwing CredentialUnavailableException.

Currently the DefaultAzureCredential handles the CredentialUnavailableException to continue on in the credential chain. ChainedTokenCredential needs the same behavior.

This is already the current behavior. Perhaps it should be updated to only iterate the list once like DefaultAzureCredential does, but this can be addressed with a different issue. Closing.


Update domains.json

Add vote.wefinance.club domain
Hi! What is your PR about?

[ ] Add or edit a skin
[ X] Add a custom domain for your space
[ ] Add an alias to migrate your space

thanks


Good place for missing release configuration information

Hi All!
This page is where people might end up after failing to be able to install Office applications together.
For example, I wound up on this page because I was running into an issue installing Visio 2019 VL along side O365 ProPlus.
Thanks to "Herb Gu" on this Technet thread
As you can see in his post, he provides two links, and then more data that we need related to changing the release configuration to "broad" instead of the standard "PerpetualVL2019"
This information he provides about the specific release channels is not provided on either of the support pages he links, nor this page I am commenting on, but it could definitely be helpful if that information was added and linked to on one or more of these pages.
Thanks!

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

ID: d40c96f3-1c97-b538-ef82-ffbf0eef204d
Version Independent ID: 2d6883b5-1b5c-dc16-9999-17d0652bfa48
Content: Supported scenarios for installing different versions of Office, Project, and Visio on the same computer - Deploy Office
Content Source: DeployOffice/install-different-office-visio-and-project-versions-on-the-same-computer.md
Service: o365-proplus-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

Hi. I stumbled upon this while looking for info about my company’s deployment of ProPlus vs Apps. (They’ve been having trouble getting me ProPlus, which I’d had before,) I’m not technical, but I read the MS Support article about deploying two versions at once. I think they tried that, but it didn’t work. I also read about deploying ProPlus to one group first and Apps to another group. I’m an editor who relies heavily on formatting. Any idea why they would decide to put me in the Apps group? Also: Any advice on how to politely escalate this (essential ProPlus features, or possible risks of Apps?) It’s been almost a month, and I’m afraid the clients will start to notice. My job could be at risk. Thank you in advance.

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


[Plugin generation] Kiota should throw error when api contains OAuth with implicit flow

What are you generating using Kiota, clients or plugins?
Kiota plugin
In what context or format are you using Kiota?
Windows executable
Client library/SDK language
None
Describe the bug
Copilot not support OAuth with implicit flow, currently it only support OAuth with authcode flow. Kiota should throw error if user selected API contains OAuth with implicit flow.
components:
  securitySchemes:
    oAuth2AuthImplicit:
      type: oauth2
      description: This API uses OAuth 2 with the implicit grant flow. [More info](https://api.example.com/docs/auth)
      flows:
        implicit:   # <---- OAuth flow(authorizationCode, implicit, password or clientCredentials)
          authorizationUrl: https://api.example.com/oauth2/authorize
          scopes:
            read_pets: read your pets
            write_pets: modify pets in your account

Sample schema:
https://raw.githubusercontent.com/SLdragon/example-openapi-spec/main/oauth-implicit.yaml
Related bug:
https://github.com/microsoft/kiota/issues/5288
https://github.com/microsoft/kiota/issues/5070
https://github.com/microsoft/kiota/issues/5162
Expected behavior
Kiota should throw error
How to reproduce
Open sample schema: https://raw.githubusercontent.com/SLdragon/example-openapi-spec/main/oauth-implicit.yaml -> click generate plugin
Open API description file
No response
Kiota Version
v1.18.100000002
Latest Kiota version known to work for scenario above?(Not required)
No response
Known Workarounds
No response
Configuration
No response
Debug output
Click to expand log
```

</details>

### Other information

_No response_

The original issue for checking the authentication type #5070  was underspecified, TTK really only supports oauth2 with authcode flow when looking into https://github.com/OfficeDev/teams-toolkit/blob/7422a1dffb7d54ca5e926e025fcdc72ae0380e17/packages/spec-parser/src/manifestUpdater.ts#L115
@sebastienlevert , @darrelmiller Please confirm Sydney does not support  OAuth with implicit flow for GA.

Implicit OAuth2 flow is not supported by Copilot.  However, just because an OpenAPI does not say the API supports OAuth2 AuthCode does not mean the API doesn't support it.  We should not fail to generate a Plugin because of this.

Thanks @darrelmiller , so closing as WON'T DO for GA now. We can come back later and revisit if needed.

@SLdragon You could, but you would be violating the spirit of the OpenAPI specification. consider a different example. If an operation declares that it returns 200 and a 400, should we assume that it will never return a 500?  Making that assumption would be a mistake.

Also there seem to be two different issues being discussed here:

Should we fail if implicit flow is supported?
Should we fail if some security schemes are described but Copilot does not support any of them?

The first is obviously not something we should do.
The second depends on whether you believe that when an author provides OpenAPI security information in an OpenAPI description that it is required to be exhaustive. The fact is, it is not. I have no issue emitting a warning that there appear to be no compatible security schemes but to emit and error would be wrong.

@maisarissi @darrelmiller is implicit flow the only one that's not supported?
How do I get the list of supported / unsupported security schemes?

This epic has the rules and supported auth flows: https://github.com/microsoft/kiota/issues/5162

I don't think we should throw an error here. I think we should warn, but not error out. Thoughts on this @maisarissi?

After some thought, yes, I do agree we should use warnings but let the user create the plugin. We shouldn't fail generation for "unsupported" auth, including OAuth implicit flow.
So, to summarize:

We should generate API Plugins and neither warn nor error when the OpenAPI description has no security definition.
We should generate API Plugins and warn users when the specified auth flow is different from OAuth code flow and HTTP bearer token (currently the supported auth scenarios).
We should generate API Plugins and warn users when there is a combination of two auth methods for an endpoint (AND scenario, e.g., API Key + OAuth for calling a #GET operation)."

The warning should be emitted for each selected endpoint that does not fill the requirements: <selected_endpoint>: "There appear to be no compatible security schemes for Copilot usage."
This is the same approach used by the manifest validator library.
cc @thewahome

Thank you for all your responses. Recently, we've been redesigning the validation flow in TTK to prevent users from being blocked when the OpenAPI specification includes authentication methods not supported by Copilot.

@maisarissi is the set of warnings also expected when using the CLI?

@maisarissi is the set of warnings also expected when using the CLI?

Yes

Hello @maisarissi
We have been throwing errors and failing plugin generation when the specified auth is unsupported / we have multiple schemes defined. But from the requirements, they are going to change to warnings

We should generate API Plugins and warn users when the specified auth flow is different from OAuth code flow and HTTP bearer token (current supported auth).
We should generate API Plugins and warn users when there is a combination of two auth methods for an endpoint (AND scenario, e.g., API Key + OAuth for calling a #GET operation)."

Introducing these changes means that we go ahead and generate the plugins but it's unclear to me what authorization we are going to set for the path once we give the warning. What should happen to the path? Should we skip it if unsupported?

Hey Charles,
if the auth is supported, continue to do what we are doing right now, if it's not supported, warn the user and create with auth none and let the dev handle that.

@maisarissi a plugin could have many paths with the wrong authentication set, I would propose that we log the warnings instead

We absolutely should log the warning so the user can filter and look at them. I like to idea of having the "Authentication warning". Not sure about the "Using anonymous auth" for each endpoint though. We would be repeating that every endpoint with unsupported auth.
@thewahome  could we do a single warn with the message: "Incompatible security schemes for Copilot usage detected in the selected endpoints." and maybe add a button "Logs" where the user can click to load the filtered log?

How would the logs look? Or should we just show the output tab?