Question about resolving web addresses between local web containers

Hi there, I'm hoping you can provide some insights as to why this isn't working for me.
I have this repo running, and 2x web containers that have a nginx web server on port 80, and wordpress-php-fpm on 9000.
For all intents and purposes, consider these web containers a slight adaptation of your docker-wordpress setup.
My project structure looks like this:
|- webproxy
|   -- this repo
|- sub1.domain.com
|   -- nginx on 80
|   -- wordpress-php-fpm on 9000
|- sub2.domain.com
|   -- nginx on 80
|   -- wordpress-php-fpm on 9000

Everything works wonderfully, until I try to run some PHP scripts that reference the other subdomain, such as a database sync / migration plugin on wordpress.
For example, a PHP script on sub2.domain.com might want to curl https://sub1.domain.com... but it cannot resolve the address (Operation timed out).
If the PHP script attempts to curl any outside address, e.g. https://google.com, it resolves perfectly fine.
I tried curling from the nginx containers in the web containers as well, with the same result, so I don't think it's because the wordpress container is behind another nginx proxy.
Any idea why this doesn't work? Thanks in advance.

I have seen this before... the work around would be to use the container name for internal purpose in the same docker network, but this wouldn't solve the issue.

I have seen this before... the work around would be to use the container name for internal purpose in the same docker network, but this wouldn't solve the issue.

@evertramos FYI - i fixed the cause of this.
For anyone that runs into this in the future, the problem in my case is the docker host had a firewall such as ufw active.
The problem was ufw is configured to accept lan requests, but the docker services were running on a different subnet. Turning ufw off, or creating an exception rule can solve this issue.


[type:feat]  ZookeeperInstanceRegisterRepository and HttpClientShenyuHttpClient. (#3930)

Make sure that:

[x] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3965 (0016491) into master (a849593) will decrease coverage by 0.04%.
The diff coverage is 55.35%.

@@             Coverage Diff              @@
##             master    #3965      +/-   ##
============================================
- Coverage     70.11%   70.07%   -0.05%
- Complexity     6691     6699       +8
============================================
  Files           924      925       +1
  Lines         25174    25230      +56
  Branches       2275     2283       +8
============================================
+ Hits          17652    17679      +27
- Misses         6147     6170      +23
- Partials       1375     1381       +6

Impacted Files
Coverage Δ

...u/register/instance/zookeeper/ZookeeperClient.java
58.02% <33.33%> (-0.95%)
:arrow_down:

...nyu/sdk/httpclient/HttpClientShenyuHttpClient.java
50.00% <50.00%> (ø)

...zookeeper/ZookeeperInstanceRegisterRepository.java
75.86% <73.33%> (+1.44%)
:arrow_up:

...che/shenyu/sync/data/http/HttpSyncDataService.java
85.71% <0.00%> (-4.09%)
:arrow_down:

...shenyu/sync/data/consul/ConsulSyncDataService.java
78.33% <0.00%> (-1.67%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yunlongn  can you see it?

please resolve conflicts

@yunlongn  I did't change anythings about RegisterUtilsTest but it check fail in ci / build (8, ubuntu-latest) (pull_request)


Move around test folder

Fixes #

Proposed Changes

moving things around - again

Release Note

Docs

/lgtm

/test pull-knative-sandbox-eventing-kafka-integration-tests

/lgtm
Nice job!


Chore: add book keeper project

Add the book keeper app that uses Room for the database foundations

issue has already been resolved


"-[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application"

Whenever my keyboard (a fork of the demo keyboard) reads KeyboardViewController#needsInputModeSwitchKey, the following error is logged to the console:

-[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.

Do you see this too? Any ideas about how to suppress? The weird thing is that this occurs even when called after the view controller has been clearly initialized for quite some time e.g. when repeatedly switching between the emoji and alphabetic keyboards. It also occurs on both the simulator and device, fwiw.

At least in my own keyboard, I'm strongly considering suppressing this by just assuming that the keyboard always needs a switch button, that is editing https://github.com/danielsaidi/KeyboardKit/blob/63a3cc608ea6ba3884a3dffd31506f37413481c0/KeyboardKitDemoKeyboard/Keyboards/DemoKeyboard.swift#L32 to look like:
    static func switchAction(for vc: KeyboardViewController) -> KeyboardAction {
        return .nextKeyboard
    }

    static func imageAction(for vc: KeyboardViewController) -> KeyboardAction {
        return .keyboardType(.emojis)
    }

(which eliminates all calls to needsInputModeSwitchKey from the demo application).
I feel comfortable doing this because my custom keyboard is not a replacement for the system alphabetic keyboard. I bet that's true of many custom keyboards, that they'll accompany other keyboards?
Another nice side effect of removing the calls to needsInputModeSwitchKey is that then I can set up the keyboard in KeyboardViewController#viewDidLoad, since this comment no longer applies.

I have seen this before, but can't reproduce it now in 2.9.0 or 3.0.0 on simulator or real device.
However, I don't think you can do what you suggest, since the switch key is needed on devices with a home button and must not be used on face-id based devices.
Perhaps you could do one viewDidLoad setup where you assume that the switch is not needed, then refresh later if that value has changed?

Huh curious that you can't repro, I still can in 2.9.0 in the iPhone 11 Pro simulator, running a version of the demo app. (Perhaps my changes therein affect this, not exactly sure how atm. Anyhow.)

the switch key is needed on devices with a home button and must not be used on face-id based devices

Oh interesting. I have wondered about the need for a switch key given that my iPhone 11 Pro always shows the globe at bottom right, yeah.
A curious thing is that Gboard always shows a switch key: https://share.getcloudapp.com/ApujDy8x. Now, that's not just a switch key for them, if you tap it the keyboard switches but if you tap-and-hold it shows this menu https://share.getcloudapp.com/DOuQrG8z, but perhaps I could do similar.

Perhaps you could do one viewDidLoad setup where you assume that the switch is not needed, then refresh later if that value has changed?

I would like to continue to load the keyboard in viewDidLoad, seems to reduce "popping in" effects. I'm loathe to toggle the switch button thereafter, seems like it might be jarring, so that's another advantage of Gboard's approach.

Interesting, but that isn't the native behavior tough. Is it the "next" keyboard of a Gboard multi keyboard setup or the next system keyboard?
I will probably adjust the demo to setup the keyboard in viewDidLoad as well, but refresh it on viewDidAppear if the state changes.

Is it the "next" keyboard of a Gboard multi keyboard setup or the next system keyboard?

It's the next system keyboard. Video: https://share.getcloudapp.com/WnuGDYvm
Btw, I can see how this might be a violation of Apple's guidelines around custom keyboards? But if Gboard got away with it I might try to too heh.

Btw I switched over to master and pulled latest and can reproduce this warning in the demo app / with the demo keyboard, in the iPhone 11 Pro simulator. Shows up when I first click the Globe button to switch to the keyboard, then again when switching from emojis to alphanumerics within the keyboard.

I get no logs whatsoever from the demo keyboard, even if I attach to process. I have tried iPhone 11, iPhone 11 Pro and iPhone 8...no logs.
Can you try the latest master? I have cleaned up the demo keyboard a bit. Do you still see this warning? Can you attach a screenshot?

Yeah, I still see the warning on the latest master. Here's a video showing the warning appearing when I first switch from the system keyboard to the demo keyboard and then when switching between the alphabetic and emoji keyboards within the demo keyboard: https://share.getcloudapp.com/RBuvreqj.

I will have a look into this when I get some more time on my hands. I get no warnings whatsoever.

needsInputModeSwitchKey does return the right value for me btw (true if and only if the phone has a home button), even when it logs the warning.
I think I'll go through with the Gboard approach described above, though, by showing a settings key (instead of a switch key) with a long-press menu like shown / set up here.
Then, when that button is tapped, I can do:
        if needsInputModeSwitchKey {
            advanceToNextInputMode()
        } else {
            print("Open Settings")
        }

This will avoid the warning until the taps on the key, and if needsInputModeSwitchKey is broken, the user will still be able to long-press to switch.

How do you want to proceed with this issue?

I have a new wrinkle: a beta user on an iPhone 7 is reporting that, when he taps my settings-or-next-keyboard button (described above), it sometimes goes to the next keyboard and sometimes to settings—whereas, on a phone with a home button, it always ought to go to the next keyboard. This implies that needsInputModeSwitchKey may be unreliable after all.
I’m going to give him a build tomorrow that will make it easier to see what the value of needsInputModeSwitchKey is at any given time. If it seems that it is unreliable, I may stop relying on it in favor of other means of detecting whether the phone has a home button. Unfortunately, this simple way of checking doesn’t work in an extension, I’d guess because the window is already inset. So I may have to go with checking the device type.
TL;DR I should have an update in a few days.

Sorry to hear that :/

I gave my testers two new builds. The first build determined the functionality of the settings-or-next-keyboard button in KeyboardViewController#viewDidLoad, i.e. so that it would always be a settings button or a next-keyboard button rather than having the tap gesture look that up later.
The response to this build was that the button always showed as a settings button. This confirms that on the iPhone 7 and 8 (my testers' devices), needsInputModeSwitchKey is not reliable within viewDidLoad. That's kind of what the original error message hints at anyway.
But the previous build, where needsInputModeSwitchKey would be checked when the button was tapped, showed that needsInputModeSwitchKey could return a false negative well after viewDidLoad too.
For this reason, the final solution I've settled on looks like
import DeviceKit // https://github.com/devicekit/DeviceKit

fileprivate extension Keyboard {
    static func switchAction(for vc: KeyboardViewController) -> KeyboardAction {
        var needsSwitch = vc.needsInputModeSwitchKey

        if !needsSwitch {
            // In my testing, `needsInputModeSwitchKey` always returns the
            // right result (`true` iff the phone has a home button), but
            // reports from iPhone 7 and 8 say otherwise. Thus, we override
            // to require a switch key if the phone has a home button,
            // as measured by supporting TouchID (the devices here
            // https://github.com/devicekit/DeviceKit/blob/446e3ab022af7bb0058be74c1298e1f7e1615607/Source/Device.generated.swift#L702
            // seem so old that there will be no phones with home buttons that
            // _don't_ support TouchID); and doesn't support FaceID (a measure
            // to guard against Apple reintroducing TouchID on FaceID-capable
            // phones
            // https://www.cnbc.com/2019/09/05/apple-iphone-in-screen-fingerprint-reader-coming-in-2020-report.html ).
            //
            // Why not always do away with `needsInputModeSwitchKey` entirely
            // and just check whether the phone has a home button? Because the
            // switch key may be required on other phones in certain cases,
            // for instance if an app is running in compatibility mode
            // https://forums.developer.apple.com/message/344179#344179
            let currentDevice = Device.current
            let phoneHasHomeButton = currentDevice.isTouchIDCapable && !currentDevice.isFaceIDCapable
            if phoneHasHomeButton {
                needsSwitch = true
            }
        }

        return needsSwitch ? .nextKeyboard : .settings
    }
}

Testers confirm this fixes the button for them, they always see a next-keyboard button if they are on a phone with a home button, otherwise a settings button. I'll close this issue as a result.
Let me know if you'd like me to open a PR with the above code. I can understand if you don't feel confident enough in that override, it is a bit hacky.

I am changing how this can be accessed in 3.0. Hopefully, it works better then.

Cool. I suspect that there is some logic to when the correct value can be accessed, maybe when we receive viewWillSyncWithTextDocumentProxy or something like that. I don't think that the value not being available in viewDidLoad is a showstopper although it is nice to be able to lay out the final UI as early as possible.

I have added a new context concept in 3.0, which moves logic from the controller to the context. I think it will be easier to use.

That sounds cool. Maybe we could move textDocumentProxy.keyboardAppearance into the context as well https://github.com/danielsaidi/KeyboardKit/issues/90#issuecomment-645618267

The entire proxy is in the context, so it's really easy to access. The context approach requires some syncing from the input controller, but that is handled in the library's base class, so it's pretty stable.

2021-07-01 10:49:57.517452+0530 unifieldFlexKeyboard[38757:1161315] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-01 10:49:57.562920+0530 unifieldFlexKeyboard[38757:1161315] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-01 10:49:57.571722+0530 unifieldFlexKeyboard[38757:1161315] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
CoreSimulator 757.5 - Device: iPhone SE (2nd generation) (A988DE2B-1CF6-466E-96AA-4AE544016F01) - Runtime: iOS 14.5 (18E182) - DeviceType: iPhone SE (2nd generation)
(lldb)

on the start while trying to switch the keyboard to custom keyboard getting the above error

Hi! Thanks for letting me know! I’m away for a week but will look at this when I get back.

I never see these printouts 🤔
I probably need to set a flag that tells the setup whether or not it’s safe to make this check. I want much of the setup to be done as early as possible, but this has to wait until the extension gets the correct info.

any update on this issue ,keyboard work fine with some devices but its crashing on  some simulators, devices

I’m still away, but you could try overriding view functions in the demo vc and setup the keyboard later to see if it solves the issue.

@ibayramli2001 @shweta-poonia Do you get this with the demo app? If so, where is it printed? Do you need to attach the debugger to the keyboard extension to see it?
I run the demo app on an SE, just like @shweta-poonia and have attached the keyboard to the debugger, but I see no printouts whatsoever.

Ok, I can see these prints in the system logs:

Same issue with the XCode custom keyboard template

Ok, I’m closing this then. 😀 Just lmk if there are any serious side-effects and I’ll open it again. 👍

4:20
Wo pehle set karo shared container mein and then pull karo extension mein
shweta poonia  4:21 PM
yeah
New
Gursharan Singh  4:23 PM
2021-07-21 16:23:23.787385+0530 unifieldFlexKeyboard[5221:114027] Failed to inherit CoreMedia permissions from 5219: (null)
2021-07-21 16:23:23.787385+0530 unifieldFlexKeyboard[5221:114042] Failed to inherit CoreMedia permissions from 5219: (null)
2021-07-21 16:23:24.446215+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:24.492298+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:24.985896+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:25.075556+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:25.184139+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:25.227699+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:25.755903+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:30.973566+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:36.019391+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.
2021-07-21 16:23:36.029623+0530 unifieldFlexKeyboard[5221:113758] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.

I am still stuck with this issue, if you can help sometimes the extension crashes or doesn't run on some devices, I only find this issue.I couldn't find ant fix for this

Have you tried debugging the extension?

I have , I am getting these errror but extension is working on my phone but on other person’s phone the keyboard is not showing up

There are two issues from start , one which I have mentioned above and other is ended with exit code -0

I am able to see and run it on my device and Xcode but other people are not able to see it in demo or Dev builds

I have filed a feedback to Apple. The ID is FB9589279 if you want to upvote.

I am trying this out in a demo app, trying to find a place where I can just return false until the controller is in a good state. I still don't know what that state is 🤷♂
I am creating a UIViewControllerRepresentable in a demo app (not in the keyboard extension, for easier debugging, printing etc.) but no matter where or when I call needsInputModeSwitchKey, I get the warning.

@wearhere @shweta-poonia and everyone else. There is a new controller property that can be used to hard set the value of this, to avoid the vc calling the super property. It's not a fix by any means, but you can use it to suppress any warnings when you don't need the real behavior, which is the case for me in my new app.

@danielsaidi I also see intermittent crashes, and the keyboard is removed from the input switcher requiring me to jump to settings and enabling it again.
import UIKit
import KeyboardKit

class KeyboardViewController: KeyboardInputViewController {

    override func viewDidLoad() {
        super.viewDidLoad()
        let keyboardView = KeyboardView(viewModel: .test, recent: [])
        setup(with: keyboardView)
        needsInputModeSwitchKeyOverride = true
    }
}

(running the keyboard scheme)
2021-10-16 21:44:03.233187-0400 Keyboard[12325:2702071] [External] -[UIInputViewController needsInputModeSwitchKey] was called before a connection was established to the host application. This will produce an inaccurate result. Please make sure to call this after your primary view controller has been initialized.

I noticed this yesterday, but it’s not related to this. Can you create a separate ticket and see if you can reproduce it in 4.9.3?

I think I managed to solve it now. By not checking the real value until didMove(toParent:) has been called, the warning seems to have disappeared. I will add this to the 6.0 release.

The fix can be tested in the v6 feature branch.


2 small things, sample config and excludes

Just found this project today, very cool.
A couple of things,

An example .yaml config with examples of all options would be super. I know the read me contains most (all?) of the settings but a config to work off of is super helpful.

And way to setup excludes for the repository?

Thanks
Edit: I guess using the "quote:" to exclude is possible based on the documentation, I think I answered my own question.

Hi @abclution Thanks for your suggestions!

I have a working sample configuration that gives an "involved" configuration with multiple cloud providers, which I think is easy to pair down to a single cloud provider. Note that that configuration does contain ALL of the settings that I use. When you say "all of the settings", there's a long list, given the quote: parameter, but I think most people don't need that. If most people do, I should know about that!If you had something else in mind for an example configuration, please do let me know, and I'll look at adding it.

By "any way to set up excludes for a repository", I think by this you mean some way to exclude files in a repository? If that's what you mean, this is actually a duplicacy question, as the duplicacy program controls this, see Filters/Include exclude patterns. I'll include a sample filters file that I have for one of my repositories here:

e:Archive/Distributions/
e:Archive/Software/
e:CloudBackup-NO/
e:CloudBackup-YES/

# Don't back up temporary Mac OS/X files
e:\.DS_Store
e:\.DocumentRevisions-V100/
e:\.apdisk
e:\.fseventsd/
e:\.TemporaryItems/
e:\.Trashes/
e:\.VolumeIcon.icns

Hope this helps! If you have further questions, feel free to re-open this issue!


Implemented get_group for DataFrameGroupBy & SeriesGroupBy.

This proposes get_group for DataFrameGroupBy & SeriesGroupBy.
>>> kdf = ks.DataFrame([('falcon', 'bird', 389.0),
...                     ('parrot', 'bird', 24.0),
...                     ('lion', 'mammal', 80.5),
...                     ('monkey', 'mammal', np.nan)],
...                    columns=['name', 'class', 'max_speed'],
...                    index=[0, 2, 3, 1])
>>> kdf
     name   class  max_speed
0  falcon    bird      389.0
2  parrot    bird       24.0
3    lion  mammal       80.5
1  monkey  mammal        NaN

>>> kdf.groupby("class").get_group("bird").sort_index()
     name class  max_speed
0  falcon  bird      389.0
2  parrot  bird       24.0

>>> kdf.groupby("class")["name"].get_group("bird").sort_index()
0    falcon
2    parrot
Name: name, dtype: object

Thanks for the reviews, @ueshin

Thanks, @ueshin
Just addressed your comments except https://github.com/databricks/koalas/pull/1783#discussion_r493991145

Sorry for the mistake about missing co-author at #1793 - I overwrote the Co-authored-by session by PR descriptions by mistake. 😿
I'd really like to add @ueshin to co-author for this work.
Thanks!


Issue 434 - set dbh driver data to NULL as well in destructor

This change is

Coverage decreased (-4.08%) to 71.063% when pulling 83d7e595d1249374be8967cd23ee11829e98895f on yitam:issue434 into ba9579b504c29438f0aa231ce8219a8aa342ccfc on Microsoft:dev.

Codecov Report

Merging #829 into dev will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #829      +/-   ##
==========================================
+ Coverage   80.06%   80.06%   +<.01%
==========================================
  Files          25       25
  Lines        7323     7324       +1
==========================================
+ Hits         5863     5864       +1
  Misses       1460     1460

Impacted Files
Coverage Δ

...c14/x86/php-7.1.20-src/ext/pdo_sqlsrv/pdo_stmt.cpp
86.18% <0%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


yum install packages-microsoft-com-prod - out of memory error

I am trying to install powershell on CentoOS 8. I follow the official Microsoft instructions.
I had some previous issue.

I rented a nanode from Linode which is the smallest model. I get memory error:
[codingsafari@centosbox ~]$ sudo yum update
CentOS-8 - AppStream                                                                140 kB/s | 4.3 kB     00:00
CentOS-8 - Base                                                                     155 kB/s | 3.8 kB     00:00
CentOS-8 - Extras                                                                    43 kB/s | 1.5 kB     00:00
Extra Packages for Enterprise Linux 8 - x86_64                                       33 kB/s |  14 kB     00:00
packages-microsoft-com-prod                                                          28 kB/s | 2.9 kB     00:00
Out of memory allocating 771751936 bytes!
Killed

[codingsafari@centosbox ~]$ free -m
              total        used        free      shared  buff/cache   available
Mem:            821          77         648           1          95         633
Swap:           511          75         436

It is a fairly small node so I resized the node to the next bigger tier and the same error happens
[codingsafari@centosbox nginx]$ sudo yum install -y powershell
packages-microsoft-com-prod                                                                               25 kB/s | 2.9 kB     00:00
Out of memory allocating 419430400 bytes!

[codingsafari@centosbox nginx]$ free -m
              total        used        free      shared  buff/cache   available
Mem:           1829          83        1663           1          82        1631
Swap:           511          85         426

At this point, I highly doubt it has something to do with the size of the node. Does anyone know why this could happen?

Well, on your first snapshot you are getting this error during yum update, so I don't see any link to powershell here. Try  install something else, like java, to see if you getting this error. Also, try to reproduce it  installing centos container or VM on your local machine. Probably centos8 is just too heavy for 1GB machine. Keep increasing memory.


feat: read_gbq creates order deterministically without table copy

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

doctest failures:
FAILED third_party/bigframes_vendored/pandas/io/gbq.py::third_party.bigframes_vendored.pandas.io.gbq.GBQIOMixin.read_gbq
FAILED third_party/bigframes_vendored/pandas/io/parquet.py::third_party.bigframes_vendored.pandas.io.parquet.ParquetIOMixin.read_parquet
FAILED bigframes/pandas/__init__.py::bigframes.pandas.read_gbq
FAILED bigframes/pandas/__init__.py::bigframes.pandas.read_parquet
FAILED bigframes/session/__init__.py::bigframes.session.Session.read_gbq_table
FAILED bigframes/pandas/__init__.py::bigframes.pandas.read_gbq_query
FAILED bigframes/session/__init__.py::bigframes.session.Session.read_gbq_query
FAILED bigframes/pandas/__init__.py::bigframes.pandas.read_gbq_table

Let's remove the df.head(2) calls from samples where the data source doesn't have a guaranteed ordering.


Delete manifests/t/Tableau/Desktop/2023.3.0 directory

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Deleting version 2023.3.0 to keep with convention of other versions (should be 23.3.345).


Add next commands to cli output

This PR adds suggestions to the user on what command to run next after both the profile and analyze commands
i.e.

after profile command is run, the analyze command is pre-created and output to the user in the shell for them to copy and paste/run
after analyze command is run, the report command is given

will add testing soon


Fix #581 once min/max intensities in 3D V4.0

Fix #581 [Tests] 3.1.1 and master -> once min/max intensities in 3D, it's kept
For 4.0

Easy, sounds good to me :) merging, thanks


error with op amax

It is the problem that the max memory offset used to index the tensor exceeds the maximum value of int32. I am fixing it with int64 indexing when needed.

The new code no longer reports any errors, so close this issue.


Update to new api version 2023-06-30

Description
Updating sdk to support the new api version 2023-06-30
PR for the api specs can be found here
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Remove jQuery usage

Describe the solution you'd like
In a couple of places, we use jQuery in components to handle some events or browser incompatibility issues.
It's a net negative for the codebase and in some places like the PipelineView component, it was "abused" in order to get good performance. I think we can get rid of jQuery entirely to clean up code to be more "pure" React. It partially stems from my initial unfamiliarity with advanced React event handling when I first implemented some of Orchest, so it's not always used for good reason.
What does your solution aim to solve?
jQuery tech debt.
Describe alternatives you've considered
Leaving it in.

Fixed in #1143 :tada:


Installed files refer to the stage directory

The FreeBSD ports framework complains:
====> Running Q/A tests (stage-qa)
Error: 'bin/uninstall-mirtk' is referring to /usr/ports/graphics/mirtk/work/stage
Error: 'lib/cmake/mirtk/MIRTKInstallManifest.txt' is referring to /usr/ports/graphics/mirtk/work/stage

The stage directory should never be mentioned in the installed files.

When MIRTK is packaged for distribution as binary package, the uninstall-mirtk script and corresponding manifest file should probably not be included as distributions offer their own package managers that enable deinstallation of the files.
Probably this line should be optional.


How to add a common behavior to multiple components?

I want to build a resizable component which acts as a container for other components. Take for example any popular editor, it has various panes / widgets (File explorer, terminal, etc.) which can be resized. The purpose of each widget is different but the resizable behavior is common to all. There are 2 ways I can think of, to achieve this:

Have a common container component which is resizable and which can be wrapped around other elements / components. Not sure if this possible in Yew or not. Something like:

// Component x
html! {
    <Resizable>

child component x

    </Resizable>
}

// Component y
html! {
    <Resizable>

child component y

    </Resizable>
}

Add a CSS class to all the components that need to be resizable. Fetch the element using the class selector and the resizable behavior to them. I think that this is doable but I am not sure where the logic should go. Should it be kept in a service?

Hey @krankur, great question. I think you could try giving your Resizable component a children property which is of type Html<Self> and then do this:
// Component x
let children = html! {

{ /*child component x*/

};

html! {
  <Resizable children=children />
}

I'm also in favour of supporting wrapping html elements with component tags but it's not implemented yet. I'll see about adding it to the next release

New issue here: https://github.com/DenisKolodin/yew/issues/537

Thanks for logging the issue.

I think you could try giving your Resizable component a children property which is of type Html<Self> and then do this:

So, after I have the passed the children (of type Html<Self>) to the children property of the Resizable component, how can I append the children to the component's content (innerHTML)? Is it possible currently, or will it be supported only when #537 is resolved?

Then, in the Resizable component you would have something like:
html! {

    { self.props.children }

}

@jstarry  So, I tried this:
pub struct Props {
    children: Option<Html<Self>>,
}

But the compiler complains, saying:
the trait bound `components::navigation::resizable::Props: yew::Component` is not satisfied

the trait `yew::Component` is not implemented for `components::navigation::resizable::Props`

note: required by `yew::virtual_dom::VNode`rustc(E0277)
resizable.rs(37, 5): the trait `yew::Component` is not implemented for `components::navigation::resizable::Props`
the trait bound `yew::virtual_dom::VNode<components::navigation::resizable::Props>: std::clone::Clone` is not satisfied

the trait `std::clone::Clone` is not implemented for `yew::virtual_dom::VNode<components::navigation::resizable::Props>`

note: required because of the requirements on the impl of `std::clone::Clone` for `std::option::Option<yew::virtual_dom::VNode<components::navigation::resizable::Props>>`
note: required by `std::clone::Clone::clone`rustc(E0277)
resizable.rs(37, 5): the trait `std::clone::Clone` is not implemented for `yew::virtual_dom::VNode<components::navigation::resizable::Props>`
the trait bound `components::navigation::resizable::Props: yew::Component` is not satisfied

the trait `yew::Component` is not implemented for `components::navigation::resizable::Props`

note: required by `yew::virtual_dom::VNode`rustc(E0277)
resizable.rs(37, 5): the trait `yew::Component` is not implemented for `components::navigation::resizable::Props`
binary operation `==` cannot be applied to type `std::option::Option<yew::virtual_dom::VNode<components::navigation::resizable::Props>>`

note: an implementation of `std::cmp::PartialEq` might be missing for `std::option::Option<yew::virtual_dom::VNode<components::navigation::resizable::Props>>`rustc(E0369)
binary operation `!=` cannot be applied to type `std::option::Option<yew::virtual_dom::VNode<components::navigation::resizable::Props>>`

note: an implementation of `std::cmp::PartialEq` might be missing for `std::option::Option<yew::virtual_dom::VNode<components::navigation::resizable::Props>>`rustc(E0369)

Am I missing something here?

@krankur Looks like you're trying to use Props as a Component somewhere. I would need to see more of your code to know where the issue is happening. Also, I would recommend that you try using latest master which no longer requires that Component::Properties implement Clone and PartialEq. You can change your Cargo.toml from yew = "0.7" to:
yew = { git = "https://github.com/DenisKolodin/yew" }

This will require that you add this attribute to your Props:
use yew::Properties;

#[derive(Properties)]
pub struct Props {
    #[props(required)]
    children: Html<Self>,
}

@jstarry Here is a link to my code https://github.com/krankur/atelier-editor/blob/resizable-layout/editor/src/components/navigation/resizable.rs
I am on yew version 0.6. Do I need to bump the version?

Yes try using master which is 0.8.
A few suggestions on your code:
Your Resizable struct should have a props field
You should destructure the optional innerTemplate or make it not an Option type. Putting an Option inside the html! block will not work

Hey thanks Justin for all the help and patience. Sorry about the messy code. I was still trying to figure out the the trait yew::Component is not implemented for components::navigation::resizable::Props error, so didn't get to those parts.
So, I decided to upgrade to 0.8, as you suggested. But I am still getting the same compilation error: the trait yew::Component is not implemented for components::navigation::resizable::Props. If I remove the innerTemplate field from here, then the error goes away.
Also, I am getting the following error after bumping the yew version, on https://github.com/krankur/atelier-editor/blob/resizable-layout/editor/src/core/model.rs#L243, which I am not able to figure out. Not sure if I should continue this discussion here or move to another issue, or gitter.
the trait bound `yew::virtual_dom::VComp<_>: yew::virtual_dom::vcomp::Transformer<_, [closure@editor/src/core/model.rs:245:44: 245:94], std::option::Option<yew::Callback<std::string::String>>>` is not satisfied

the trait `yew::virtual_dom::vcomp::Transformer<_, [closure@editor/src/core/model.rs:245:44: 245:94], std::option::Option<yew::Callback<std::string::String>>>` is not implemented for `yew::virtual_dom::VComp<_>`

Let me know if I should move this discussion to somewhere else.

Hey no worries, we will figure this out :)
I think the Component trait not implemented error is happening because the html macro is assuming it is used inside a component method. So your approach of initializing to a default value inside Props is breaking things. I will have to look into making this work in the macro to fix the issue.
To get around this for now, can you try creating the innerTemplate value inside the render method of a component which is using your Resizable component?
In regards to your new issue. This is a breaking change in 0.8 which has an easy fix. Simply change onsignal inside PrefabNewModal to not be an Option type. Then add the #[props(required)] attribute to onsignal.
This is really new stuff so I appreciate you trying it out!

The new issue, one related to onsignal, got fixed by doing the changes that you suggested. Thanks :)
About the workaround for the original issue, I am assuming that by render method you mean the view method of the Renderable implementation for the component which is using the Resizable component. But I am not sure how would I pass the innerTemplate to the Resizable component. I need to append the innerTemplate inside the Resizable component. Are you suggesting something like:
impl Renderable<Model> for Model {
    fn view(&self) -> Html<Self> {
        let innerTemplate: Html<Self> = html! {

{"I am inner template"}

        };
        html! {
            <Resizable>{innerTemplate}</Resizable>
        }
    }
}

I tried the above, but this also doesn't compile.

Yes, sorry I meant the view method. Does it work if you try:
impl Renderable<Model> for Model {
    fn view(&self) -> Html<Self> {
        let innerTemplate: Html<Self> = html! {

{"I am inner template"}

        };
        html! {
            <Resizable innerTemplate=innerTemplate />
        }
    }
}

If I do the above, the compiler complains that no field innerTemplate on type components::navigation::resizable::Props and when I add the innerTemplate field to components::navigation::resizable::Props like below:
#[derive(Properties)]
pub struct Props {
    innerTemplate: Option<Html<Self>>
}

we again go back to where we started from, i.e. the compiler complains saying:
the trait bound `components::navigation::resizable::Props: yew::Component` is not satisfied

the trait `yew::Component` is not implemented for `components::navigation::resizable::Props`

note: required by `yew::virtual_dom::VNode`rustc(E0277)

It shouldn't be Html<Self> you need to do Html<Resizable>

Oops! corrected. And that fixes the original compilation error. Yay! Getting another error now here:
`yew::virtual_dom::VNode<components::navigation::resizable::Resizable>` doesn't implement `std::fmt::Display`

`yew::virtual_dom::VNode<components::navigation::resizable::Resizable>` cannot be formatted with the default formatter

Hmm this is because self.innerTemplate.as_ref().unwrap() is a &VNode<Resizable> which cannot be used in that way. Looks like the only way to get this working without changes in yew is for you to have innerTemplate be a type that implements Renderable. Try that :)

But wouldn't fixing the innerTemplate to a specific type (that implements Renderable) restrict the innerTemplate to always be of that type? I want the content of Resizable to not be fixed.

@krankur how about if you set innerTemplate to Box<impl Renderable<Resizable> does that fix your problem?

Is that allowed? I tried this:
#[derive(Properties)]
pub struct Props {
    // pub innerTemplate: Option<Html<Resizable>>
    pub innerTemplate: Box<impl Renderable<Resizable>>
}

and got the following error:
`impl Trait` not allowed outside of function and inherent method return types

So I tried this as well:
#[derive(Properties)]
pub struct Props {
    // pub innerTemplate: Option<Html<Resizable>>
    pub innerTemplate: Box<Resizable>
}

And I got the components::navigation::resizable::Resizable doesn't implement std::fmt::Display error.
I am new to Rust, which you might already have figured out, hence all these questions.

Sorry, I meant to say Box<dyn Renderable<Resizable>> 😄 This will allow you to use a different innerTemplate each time you use Resizable.
So, putting it all together (and a few minor changes) you would have:
// model.rs

struct ModelInner;
impl Renderable<Resizable> for ModelInner {
    fn view(&self) -> Html<Resizable> {
        html! {
{"I am inner template"}
}
    }
}

impl Renderable<Model> for Model {
    fn view(&self) -> Html<Self> {
        html! {
            <Resizable inner_template=Box::new(ModelInner) />
        }
    }
}

// resizable.rs

pub struct Resizable {
    console: ConsoleService,
    state: ResizableState,
    props: Props,
}

#[derive(Properties)]
pub struct Props {
    #[props(required)]
    pub inner_template: Box<dyn Renderable<Resizable>>,
}

impl Renderable<Resizable> for Resizable {
    fn view(&self) -> Html<Self> {
        html! {

                { self.props.inner_template.deref() }

        }
    }
}

It worked! Finally! I would never have been able to do this without your help. I learnt a lot. Thanks you so much for all your time and patience.

Great!! Happy to help. This isn't the most ergonomic code but cool to see that it can work! This solution will be sure to inspire the implementation of wrapper component tags in the future


[Outlining] Exception in log

[VFPT][Error 04:06:04 ] Exception Message: Cannot access a disposed object.
Object name: 'OutliningManager'.
Stack Trace:    at Microsoft.VisualStudio.Text.Outlining.OutliningManager.EnsureValid()
   at Microsoft.VisualStudio.Text.Outlining.OutliningManager.EnsureValid(SnapshotSpan span)
   at Microsoft.VisualStudio.Text.Outlining.OutliningManager.CollapseAll(SnapshotSpan span, Predicate`1 match)
   at FSharpVSPowerTools.OutliningTaggerProvider.<>c__DisplayClass7.<TextViewCreated>b__4(Object sender, SnapshotSpanEventArgs e) in d:\github\FSharpVSPowerTools\src\FSharpVSPowerTools\Commands\OutliningTaggerProvider.cs:line 90
   at Microsoft.FSharp.Control.FSharpEvent`2.Trigger(Object sender, TArgs args)
   at FSharpVSPowerTools.Outlining.OutliningTagger.triggerUpdate(ScopeSpan[] newSnapshotSpans) in D:\github\FSharpVSPowerTools\src\FSharpVSPowerTools.Logic\Outlining\OutliningTagger.fs:line 90
   at FSharpVSPowerTools.Outlining.clo@178-495.Invoke(Unit _arg1) in D:\github\FSharpVSPowerTools\src\FSharpVSPowerTools.Logic\Outlining\OutliningTagger.fs:line 178
   at FSharpVSPowerTools.ProjectSystem.VSUtils.protectOrDefault[a](FSharpFunc`2 f, a defaultVal) in D:\github\FSharpVSPowerTools\src\FSharpVSPowerTools.Logic\VSUtils.fs:line 431

Is this with current master? Do you have a reliable way to reproduce this?

yes, it's with current master. I happened on just after a file was opened. I'll try to make minimal repro.

Does this still occur? It might not happen any more due to #1260.

I have not checked but it's definetely fixed.

Master, VS 2015 Update 1:
[VFPT][Error 11:24:41 AM] Exception Message: Cannot access a disposed object.
Object name: 'OutliningManager'.
Stack Trace:    at Microsoft.VisualStudio.Text.Outlining.OutliningManager.EnsureValid()
   at Microsoft.VisualStudio.Text.Outlining.OutliningManager.EnsureValid(SnapshotSpan span)
   at Microsoft.VisualStudio.Text.Outlining.OutliningManager.InternalCollapseAll(SnapshotSpan span, Predicate`1 match, Nullable`1 cancel)
   at Microsoft.VisualStudio.Text.Outlining.OutliningManager.CollapseAll(SnapshotSpan span, Predicate`1 match)
   at FSharpVSPowerTools.OutliningTaggerProvider.<>c__DisplayClass10_0.<TextViewCreated>b__0(Object sender, SnapshotSpanEventArgs e)
   at Microsoft.FSharp.Control.FSharpEvent`2.Trigger(Object sender, TArgs args)
   at FSharpVSPowerTools.Outlining.clo@178-459.Invoke(Unit _arg1)
   at FSharpVSPowerTools.ProjectSystem.VSUtils.protectOrDefault[a](FSharpFunc`2 f, a defaultVal)

Is this still happening?

I think it does not matter even though it's still happening, everything work ok.


Ants model

An example I attempted to replicate from NetLogo. It shows ants gathering food and leaving pheromone trails for other ants to follow.

Can you please post here the image / video output?

https://github.com/JuliaDynamics/AgentsExampleZoo.jl/assets/7633732/ab4dd405-0743-4483-9e57-cb37c4181a22

looks very good, thank you!


Building model for cell2location

Good day!
I am trying out this nice tool to analyze data generated using Visium. I have employed cell2location for cell-type deconvolution and have successfully prepared the data according to the tutorials you shared in this repository. However, I want to continue with the receiver/sender analysis but did not find a tutorial explaining how to create the model in detail.
I assume it has to follow this tutorial (https://github.com/theislab/ncem_tutorials/blob/main/tutorials/model_tutorial_interactions.ipynb), but I am not sure whether the parameters suggested there are applicable for Visium as well and not sure how to input the h5ad file (from the data preparation section) for the model training. Could you please share how to generate the model?
Thanks in advance!

I am using the scripts to train the model in SLURM (deconvolution_hvg.sh and deconvolution_baseline_hvg.sh) . So far seems to be working. However, I am not sure I am doing it right. The scripts for the cell2location dataset were designed to run only one sample from the lymph node dataset. I have run cell2location to estimate cell abundances on 10 different samples. I have seen the code on data.py and it assumes it is only one sample, right? I am using the DataLoaderCell2locationLymphnode to read my dataset but it does not account for multiple samples.
I prepared the data as suggested in the tutorial but there is no metadata or additional info regarding each individual sample. Now all estimated cells have been placed in one unique matrix. Will that influence the outcome? How would you recommend I should proceed in this case?
Now is running like this:
Loading data from raw files
registering celldata
AnnData object with n_obs × n_vars = 671874 × 2000
    obs: 'index', 'target_cell'
    var: 'highly_variable', 'means', 'dispersions', 'dispersions_norm'
    uns: 'hvg', 'log1p', 'node_type_names', 'metadata', 'img_to_patient_dict'
    obsm: 'node_types', 'proportions', 'spatial'
collecting image-wise celldata
adding graph-level covariates
Loaded 1 images with complete data from 1 patients over 671874 cells with 2000 cell features and 21 distinct celltypes.
Mean of mean node degree per images across images: 20.010502
Using split method: node.
 Train-test-validation split is based on total number of nodes per patients over all images.

Excluded 0 cells with the following unannotated cell type: [None]

Whole dataset: 671874 cells out of 1 images from 1 patients.
Test dataset: 67187 cells out of 1 images from 1 patients.
Training dataset: 547009 cells out of 1 images from 1 patients.
Validation dataset: 60469 cells out of 1 images from 1 patients.

Hope I am being clear about my question. Thanks in advance!

Hi @ccruizm,
thanks for raising an issue and your interest in using ncem. I will push a tutorial on how to use ncem for deconvoluted visium beginning of next week so showcase the usage.
Maybe additional question: are you planning to identify potential type couplings in your data or do you want to inspect convergence of the model?
Running the model itself is only required once to identify if ncem can infer communication, afterwards you can just simply use the trained model so obtain insights from your data.

Thanks for your reply!  I want to use it mainly to check type couplings. I am particularly seeing specific cell associations based on the estimated cell-type abundance and would like to check how the transcriptome changes in the different cellular compartments we have identified.
I am not sure which of the models will be more suitable for my data. I am now running the linear one but do not know whether the others could fit better. Particularly for Visium data.
Looking forward to the upcoming tutorial!

Hi @AnnaChristina !
Just wanted to add that I am also looking into running NCEM for my Visium samples and looking forward and appreciate a lot your new tutorial and any advice! :) I also have a question on how to adapt grid search script to the LSF cluster in case you or anyone from your team has experience in that.
Many thanks and with best wishes,
Veronika

Hello @AnnaChristina!
Do you have any news where we can find the tutorial on how to use ncem for deconvoluted visium data, please?
Thanks!

Hi @ccruizm,
sorry for the delay, I added a tutorial for using ncem for deconvoluted Visium data here: https://github.com/theislab/ncem_tutorials/blob/main/tutorials/type_coupling_visium.ipynb
We added this to a new release and it is now available in ncem==0.1.4
Please let me know if you have additional questions.

No problem! thank you very much for this.

Dear @AnnaChristina ,
First of all, thank you so much for a great tool!
I am reopening this thread because I successfully have used your tutorial on deconvolved visium data but wanted to reiterate on the comment you made earlier, distinguishing the type coupling analysis from convergence.
I first prepared the data as described in this tutorial (https://github.com/theislab/ncem_benchmarks/blob/main/notebooks/data_preparation/deconvolution/cell2location_human_lymphnode.ipynb), then used this tutorial to find type coupling (https://github.com/theislab/ncem_tutorials/blob/main/tutorials/type_coupling_visium.ipynb). In the latter, the type coupling step is actually run before training a model. I am not sure I understand how these functions work. Am I supposed to have trained the model somewhere on my data in between these two tutorials? Or can I just obtain the type coupling from the InterpreterDeconvolution, which I understand fits a standard OLS?
Sorry if this has been explained elsewhere, but I couldn't find the information.
Thank you!


super-navigate does not work consistently between AdHoc and Scanner builds

From https://forums.appgyver.com/#!/steroids:view-id-not-working-adhoc

Judging from the error message, this issue is likely caused by trying to push the same preloaded view twice. Closing until proper repro.


MVP Recoil app, time jump needed to be fixed

Types of changes

[X] Bugfix (change which fixes an issue)
[X] New feature (change which adds functionality)
[X] Refactor (change which changes the codebase without affecting its external behavior)
[ ] Non-breaking change (fix or feature that would causes existing functionality to work as expected)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Purpose

There are 3 issues with our app currently.

The state data do not render properly in a Recoil application.
There is no way to know which state management is being used in the application.
Currently, there is no documentation to explain how the app works as a whole. There should be a document to describe the architecture of the app.

Approach

We changed the logic when adding state to the tree to render the data properly in the app.
We added a variable and initialized it to false and only set it to true if $RecoilDebugStates are available
We added a file named readme_2.md to explain how the app works and how the data flow in the app.

Learning

There two articles went over React Fiber extensively and aided us in understanding Recoil better.
https://dev.to/afairlie/to-understand-react-fiber-you-need-to-know-about-threads-3dof
https://medium.com/react-in-depth/inside-fiber-in-depth-overview-of-the-new-reconciliation-algorithm-in-react-e1c04700ef6e
Screenshot(s)

The image below showed the code that added the state of each node to the tree when our app is an Recoil App.

lol this branch name


Add Windows Reproducible Build From SBOM Scripts

Fixes #3426
This is the PR for the initial version of the windows reproducible build script, it covers these 5 tasks which were required to make this function correctly.

Add initial version of win repro script
Make reproducible scripts less noisy
Add reproducible patch functions to main scripts
Add Win SDK Version To SBOM
Restructure tooling scripts for clarity.

The script itself has been developed and tested extensively on an Adoptium Windows build machine, however it should work on any windows machine, that has been built and configured with the necessary (specific!) compiler versions.

Final output from script..

freetype/freetype.h
Checking and getting Tool Summary info:
CycloneDX SBOM has been created in C:\cygwin64\cmp20240301152251\temurin-build\workspace\target\OpenJDK21U-sbom_x64_windows_hotspot_21.0.2_13.json
build.sh : 17:00:30 : All done!
---------------------------------------------
Comparing JDKs
Unzip Source
Unzip Target
Pre-processing src_jdk
Expanding the 'modules' Image to compare extracted files
Expanding the 'src.zip' to normalize file permissions
Expanding jmods to process binaries within
Expanding the 'jrt-fs.jar' to remove signatures from within..
Removing all Signatures from src_jdk
Adding temp Signatures for src_jdk
Generating a RSA private key
...........................++++
...............................................................................................................++++
writing new private key to 'test.key'
-----
Removing all Signatures from src_jdk
Removing jrt-fs.jar MANIFEST.MF BootJDK vendor string lines
***********
SUCCESS :-)
***********
Cleaning any Temurin build-scripts release file BUILD_INFO from src_jdk
Successfully removed all SystemModules jdk.jpackage hash differences from src_jdk
Normalizing ModuleAttributes order in module-info.class, converting to javap
Pre-processing tar_jdk
Expanding the 'modules' Image to compare extracted files
Expanding the 'src.zip' to normalize file permissions
Expanding jmods to process binaries within
Expanding the 'jrt-fs.jar' to remove signatures from within..
Removing all Signatures from tar_jdk
Adding temp Signatures for tar_jdk
Generating a RSA private key
.....................++++
..........................................++++
writing new private key to 'test.key'
-----
Removing all Signatures from tar_jdk
Removing jrt-fs.jar MANIFEST.MF BootJDK vendor string lines
***********
SUCCESS :-)
***********
Cleaning any Temurin build-scripts release file BUILD_INFO from tar_jdk
Successfully removed all SystemModules jdk.jpackage hash differences from tar_jdk
Normalizing ModuleAttributes order in module-info.class, converting to javap
Number of files: 73271
Comparing src_jdk with tar_jdk ... output to file: repro_diff.out
Number of differences: 0
ReproduciblePercent = 100 %
---------------------------------------------
Removing jq
Package jq removed
Removing libjq1
Package libjq1 removed
Removing libonig5
Package libonig5 removed


optionally allow node_exporter access to shared /tmp

by default node_exporter.service sets PrivateTmp=yes which prevents
node_exporter from monitoring the actual /tmp, e.g. for children mount
points.

Thanks :+1:


Support Auto-Discovery

This will make the package work with auto-discovery in Laravel 5.5

Thanks. As per our contibution guidelines you need to:

Agree to assign the copyright of your contribution to Dev7studios (in your commit message or pull request).
Declare that the contribution is in the public domain (in your commit message or pull request).

@gilbitron Added!

Thanks 👍


EKS Clusters in an AWS shared VPC

What kind of request is this (question/bug/enhancement/feature request):
I think this is a bug
Steps to reproduce (least amount of steps as possible):
Create a rancher-managed EKS cluster in a AWS shared VPC by selecting the shared subnets in the Networking tab.
Result:
The cluster is stuck in updating state with the following error:
 Waiting for API to be available; InvalidRequestException: You do not have access to a default security group in VPC vpc-<shared-vpc-id>. Specify a security group, and try again. { RespMetadata: { StatusCode: 400, RequestID: "8c9f1d02-5036-49a4-9400-3fc7bf755728" }, Message_: "You do not have access to a default security group in VPC vpc-<shared-vpc-id>. Specify a security group, and try again." }

Other details that may be helpful:
The rancher role has the permissions as specified in minimum eks permissions. I'm pretty sure this is not a permission issue, though.
The vpc sharing docs explicitly state: Participants cannot launch resources using the default security group for the VPC because it belongs to the owner.
So the error makes sense in a way, because I indeed do not have access to the default security group of the shared vpc.
What I fail to understand is: Why does rancher need the VPCs default security group?
I tried some things, to no avail:

use pre-defined launch configuration
select an arbitrary pre-defined security group in the "Select security groups" step

Note that rancher DOES create the cluster-specific security group. The EKS cluster on AWS' side is active and has no issues.
Environment information

Rancher version: 2.5.7
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): EKS
Machine type (cloud/VM/metal) and specifications (CPU/memory): cloud
Kubernetes version (use kubectl version): 1.18

Also having this problem.

Any solution yet?  I'm using 2.5.8 now and having the same problem.
PS- If you delete the eks cluster, there's a laggard object called an "ekscc" that sticks around in the Rancher-hosting cluster's k8s system - and it won't delete.
Doing "kubectl -n cattle-global-data delete ekscc c-xyzab" or similar says it's deleted, but the command hangs and never returns.
I had to go under the hood onto one of the RKE nodes hosting rancher and manually delete the object from Etcd:
etcdctl get --prefix=true /registry/eks.cattle.io (find the path this way)
etcdctl del /registry/eks.cattle.io/eksclusterconfigs/cattle-global-data/c-cfmf6
After that I could attempt re-creating an EKS cluster with the same name.
This is definitely a Rancher bug!  The original issue with the security group and shared VPC, I'm still looking into.

For us, this was an issue with the configured node groups.
If running without a predetermined launch template, rancher tries to give the managed nodes the default security group of the VPC (which is not available in the shared VPC). It works when we create a launch template for the managed nodes and give the nodes a specific security group there.
Now we're running straight into https://github.com/rancher/terraform-provider-rancher2/issues/693 :sweat:
I will close this issue.

Related: https://github.com/aws/containers-roadmap/issues/1602


fn dav1d_msac_init: Make safe

I'm not sure if this works correctly, but I tried to make this safe by adding a phantom lifetime to MsacContext for the buf's lifetime, which originally was a slice, but is made into pointers to be #[repr(C)] and stay layout-compatible with asm.  I'm particularly not sure about the 'static I stuck in the MsacContext in its containing struct.  Is that any less safe/checked than before?

I'm not sure it makes sense to add the lifetime to MsacContext. If we were threading that lifetime all the way from the source slice to MsacContext then it would work, but since we're using slice::from_raw_parts to construct the slice from a pointer, it's going to pull a lifetime from thin air that will always match. So adding the lifetime isn't going to correctly track the lifetime of the slice.

I plan on, after this, to thread the lifetime up until at least Dav1dData, which seems like it might be a ref counted slice/buf.  I want to do it incrementally, though.

Can you do threading the lifetime in a single PR? I think any incremental work there is going to leave us with lifetimes that don't make sense, so it'd probably be best to do a single PR where we can see how the lifetimes connect.

Can you do threading the lifetime in a single PR? I think any incremental work there is going to leave us with lifetimes that don't make sense, so it'd probably be best to do a single PR where we can see how the lifetimes connect.

Okay, I'll try and see how it goes.


About the torch version issue

Hello,
after installing torch=1.7.0, cuda=10.2, and the corresponding latest torch-sparse，torch-scatter，torch-cluster and torch-spline-conv, the following error occurred in the running code. Could you tell me the correct version of torch.

thank you


Setup Amazon Managed Service for Prometheus (AMP) Workspace

@bion you are working on it or already did it, right?

This was implemented in https://github.com/civiform/civiform/pull/3321


fixed animated picking

Picking (raycasting) was occasionally not working for animated models. Turns out this is because the broad phase uses bounding boxes to cull meshes, but the bounding  boxes are wrong for animated models. If they're wrong by enough, then it  doesn't bother checking the triangles for intersection. This is fixed by setting both the bounding sphere  and  bounding box in such a  way as to circumvent the early-outs.
I also changed the editor to default to bounds="tight", which I'd simply forgotten to before since this is the more precise way to frame.

WOW 😳


bootloader dependencies fail to compile in 0.11.0 and higher

Transitive dependancies like serde, thiserror, etc. are not linking to the core versions after version 0.11.0 for a no_std environment.
For version 0.10.13 it still works correctly and compiles as expected.
Used rustc:
❯ rustc --version
rustc 1.77.0-nightly (bf3c6c5be 2024-02-01)
Example error:
error[E0463]: can't find crate for `std`
 --> /home/robert/.cargo/registry/src/index.crates.io-6f17d22bba15001f/thiserror-1.0.56/src/display.rs:1:5
  |
1 | use std::fmt::Display;
  |     ^^^ can't find crate
  |
  = note: the `x86_64-skuld` target may not support the standard library
  = help: consider building the standard library from source with `cargo build -Zbuild-std`

error[E0463]: can't find crate for `alloc`
   --> /home/robert/.cargo/registry/src/index.crates.io-6f17d22bba15001f/fastrand-2.0.1/src/lib.rs:113:1
    |
113 | extern crate alloc;
    | ^^^^^^^^^^^^^^^^^^^ can't find crate

error[E0463]: can't find crate for `std`
 --> /home/robert/.cargo/registry/src/index.crates.io-6f17d22bba15001f/thiserror-1.0.56/src/display.rs:2:5
  |
2 | use std::path::{self, Path, PathBuf};
  |     ^^^ can't find crate
  |
  = note: the `x86_64-skuld` target may not support the standard library
  = help: consider building the standard library from source with `cargo build -Zbuild-std`

error[E0463]: can't find crate for `std`
 --> /home/robert/.cargo/registry/src/index.crates.io-6f17d22bba15001f/fastrand-2.0.1/src/global_rng.rs:5:5
  |
5 | use std::cell::Cell;
  |     ^^^ can't find crate
  |
  = note: the `x86_64-skuld` target may not support the standard library
  = help: consider building the standard library from source with `cargo build -Zbuild-std`
  error[E0412]: cannot find type `Result` in this scope
  --> /home/robert/.cargo/registry/src/index.crates.io-6f17d22bba15001f/serde-1.0.196/src/de/seed.rs:13:49
   |
13 |     fn deserialize<D>(self, deserializer: D) -> Result<Self::Value, D::Error>
   |                                                 ^^^^^^ not found in this scope
   |
help: consider importing one of these items
   |
1  + use core::fmt::Result;
   |
1  + use core::result::Result;
   |

error: could not compile `serde` (lib) due to 5106 previous errors

Starting with v0.11, your kernel should depend on bootloader_api, not bootloader directly. See https://github.com/rust-osdev/bootloader/blob/main/docs/migration/v0.10.md . The bootloader crate can then be used from a build script or builder crate to turn your kernel into a bootable disk image.

I think this should be solved.


Convert permutation importance results to a proper importance score.

Going forward, the permutation importance results should be returned as proper importance scores (i.e., [0,  inf) where higher the value, higher importance). This is crucial as users may use a loss metric (where a lower value indicates higher accuracy) or a rank-based metric (where a higher value indicates higher performance). Having a uniform importance score will improve the generalizability of the feature ranking plotting.

https://github.com/monte-flora/py-mint/blob/a8e987ff0e37445b4a13d7ebabb586309209809c/pymint/common/utils.py#L87. Added compute_importance to ..common.utils.py. This also normalizes the scores so that single-pass and multi-pass can be compared. Additionally, the to_pymint_importance also normalizes the scores (max scaling so the range is [0,1]). This will allow for a comparison of importance between methods.


Downgrade level of NotFound error for missing element in GET HTTP request

Link to reproduction
No response
Describe the Bug
As discussed with @DanRibbens and @jmikrut, the error from a GET request for a missing element currently spits a message such as
ERROR (payload): NotFound: The requested resource was not found.
    at findByID (/Users/ben.routson/Repos/playmaker-cms/node_modules/.pnpm/payload@2.24.1_@types+react@18.2.55_typescript@4.9.5_webpack@5.90.1/node_modules/payload/src/collections/operations/findByID.ts:94:15)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at findByIDHandler (/Users/ben.routson/Repos/playmaker-cms/node_modules/.pnpm/payload@2.24.1_@types+react@18.2.55_typescript@4.9.5_webpack@5.90.1/node_modules/payload/src/collections/requestHandlers/findByID.ts:26:17)

James and I agreed that this should be downgraded to an info level message, as opposed to a system error, and can still be considered to return an HTTP 404 response.  Thus, this ticket is requested to log the work to ensure that this type of GET message doesn't spit out a system error message/status.
To Reproduce
Just create a HTTP GET request for an element that doesn't exist in the DB/system, and observe the system/console for this error message.
Payload Version
2.24.1
Adapters and Plugins
No response

This would be a breaking change so I think there needs to be a feature flag specific to this or possibly a way to set the level of error for other thrown errors as well. For example ValidationError is another that comes up all the time and doesn't need to pollute the logs typically.
We will discuss the API around how this will be configurable.

Makes sense then, I'll stay posted on this thread for any updates.


The $downloadTotal parameter can be zero in progress callback

It would be worth mentioning that this may be zero when the Content-Length header is not available.

Hello @BenMorel, i create a fork of this repo and release this marge request into the new version(6.4.0).
Namespace compatibility is 100%, and you can use my changes if you patch composer.json in your projects

replace "guzzlehttp/guzzle": "^6.3" to "someblackmagic/guzzle": "^6.4"
add "guzzlehttp/guzzle": "6.3.*" into replace block

Fork link: https://github.com/SomeBlackMagic/guzzle/

So many changes in the master branch and these changes not released in tag. In my project i use phpstan and i see an error about the throwable interface.  I fork this repo and add a tag to fix errors. In next version, i won't merge into my repo another MR  which have not yet been merged in this repo


Adding the files for the recommendation system

Fixes #
Describe the changes you have made in this PR -
I have added the code for the cleaning module, the model (LDA + KD Trees), and the recommendation scripts (both old and new projects) that create a URL for the projects.
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.751% when pulling 8660a61858549c097872d909512e8d5039af7c58 on DevanshD3:recommendation-system into 270ab87efe9408e831cae79e898f85bceb9dba5a on CircuitVerse:master.

I think it is better to keep this in separate repository for visibility


P1357R1 Traits for [Un]bounded Arrays

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2019/p1357r1.pdf

実装済み


fix bug for object copy

Signed-off-by: sunfch<EMAIL_ADDRESS>
What this PR does / why we need it:
There are 3 bugs:

The canned acl can not be added to target object in object copy request
Update the processing logic of "X-Amz-Metadata-Directive" header.  the object metadata cannot be replaced in object copy request at present.
The md5 cannot be returned in put object request
Return nil instead of err in s3 grpc server

The PR fix the bugs as above
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #917
Special notes for your reviewer:
Release note:

Hi @sunfch Can you please confirm the following with this fix:
a) As AWS doesn't support Copying ACL in COPY OBJECT, we will now get the ACL from Source and Update it on Target?
b) All possible metadata will now be re-stored to target object because of X-AMZ-METADATA-DIRECTIVE?
c) What is the behaviour for COPY and REPLACE when srcBucketname != DestBucketName?
d) As we are returning nil instead of err, there is no other handling required?

thanks for your review,

Hi @sunfch Can you please confirm the following with this fix:
a) As AWS doesn't support Copying ACL in COPY OBJECT, we will now get the ACL from Source and Update it on Target?
I dont get the ACL from Source in the PR, and only support a canned ACL in our COPY request using the x-amz-acl request header.
b) All possible metadata will now be re-stored to target object because of X-AMZ-METADATA-DIRECTIVE?
According my understanding, if X-AMZ-METADATA-DIRECTIVE is COPY,  metadata from source object will be re-stored to target object, and this is default behavior.
c) What is the behaviour for COPY and REPLACE when srcBucketname != DestBucketName?
According my understanding, if X-AMZ-METADATA-DIRECTIVE is COPY, metadata is copied to target object in COPY request, and if if X-AMZ-METADATA-DIRECTIVE is REPLACE, metadata is replaced with the metadata in the COPY request header.
d) As we are returning nil instead of err, there is no other handling required?
We define error member in response struct,  and the error is returned to grpc client, and gprc client handle with it

thanks for your review,

Hi @sunfch Can you please confirm the following with this fix:
a) As AWS doesn't support Copying ACL in COPY OBJECT, we will now get the ACL from Source and Update it on Target?

I dont get the ACL from Source in the PR, and only support a canned ACL in our COPY request using the x-amz-acl request header.

b) All possible metadata will now be re-stored to target object because of X-AMZ-METADATA-DIRECTIVE?

According my understanding, if X-AMZ-METADATA-DIRECTIVE is COPY, metadata from source object will be re-stored to target object, and this is default behavior.

c) What is the behaviour for COPY and REPLACE when srcBucketname != DestBucketName?

According my understanding, if X-AMZ-METADATA-DIRECTIVE is COPY, metadata is copied to target object in COPY request, and if if X-AMZ-METADATA-DIRECTIVE is REPLACE, metadata is replaced with the metadata in the COPY request header.

d) As we are returning nil instead of err, there is no other handling required?

We define error member in response struct, and the error is returned to grpc client, and gprc client handle with it

Hi @sunfch , does that mean, if X-Amz-Metadata-Directive is set to be COPY, then all metadata of target object will be set the same as source object, except ACL, and if X-Amz-Metadata-Directive is set to be REPLACE, then all of targe object will be set according to the request?

thanks for your review,

Hi @sunfch Can you please confirm the following with this fix:
a) As AWS doesn't support Copying ACL in COPY OBJECT, we will now get the ACL from Source and Update it on Target?

I dont get the ACL from Source in the PR, and only support a canned ACL in our COPY request using the x-amz-acl request header.

b) All possible metadata will now be re-stored to target object because of X-AMZ-METADATA-DIRECTIVE?

According my understanding, if X-AMZ-METADATA-DIRECTIVE is COPY, metadata from source object will be re-stored to target object, and this is default behavior.

c) What is the behaviour for COPY and REPLACE when srcBucketname != DestBucketName?

According my understanding, if X-AMZ-METADATA-DIRECTIVE is COPY, metadata is copied to target object in COPY request, and if if X-AMZ-METADATA-DIRECTIVE is REPLACE, metadata is replaced with the metadata in the COPY request header.

d) As we are returning nil instead of err, there is no other handling required?

We define error member in response struct, and the error is returned to grpc client, and gprc client handle with it

Hi @sunfch , does that mean, if X-Amz-Metadata-Directive is set to be COPY, then all metadata of target object will be set the same as source object, except ACL, and if X-Amz-Metadata-Directive is set to be REPLACE, then all of targe object will be set according to the request?

@sfzeng ,  yes, this is my understand too

Thanks for the explanation. Yes, when i asked about X-AMZ-Metadata-Directive for copy, i meant all except ACL.


Bus.plug type ergonomics

Hi,
I think Bus.plug could be more ergonomic. plug seems to require type parameter for the stream being plugged into the Bus in some cases. It seems that Bus<A>.plug requires the plugged stream type B to be assignable to A and A to be assignable to B while I believe that the former would be enough. (Note: strict mode TS is required for the compile error)
import * as Bacon from "baconjs";

type TestObject = { a?: number; b?: string };
const testBus = new Bacon.Bus<TestObject>();

// these work
testBus.push({ a: 242124 });
testBus.push({ b: "jee" });

// this works:
testBus.plug(Bacon.interval(1000, 0).map<TestObject>(() => ({ a: 999 })));

// this does not compile
testBus.plug(Bacon.interval(1000, 0).map(() => ({ a: 999 })));

// type testing  done by @milankinen
interface BBus<V> {
  plug(x: Bacon.Observable<V>): void;
  plug2<OV extends V>(x: Bacon.Observable<OV>): void;
}

const b = (testBus as any) as BBus<TestObject>;

// this compiles without errors
b.plug2(Bacon.interval(1000, 0).map(() => ({ a: 999 })));

This is a likely simple fix per se, but to have the whole project compile with —strict will take a lot of effort. I’d like to have a failing test first and then improve typings.

On 15 Jun 2019, at 15:21 , Antti Risteli<EMAIL_ADDRESS>wrote:
Hi,
I think Bus.plug could be more ergonomic. plug seems to require type parameter for the stream being plugged into the Bus in some cases. It seems that Bus.plug requires the plugged stream type B to be assignable to A and A to be assignable to B while I believe that the former would be enough. (Note: strict mode TS is required for the compile error)
import * as Bacon from "baconjs";
type TestObject = { a?: number; b?: string };
const testBus = new Bacon.Bus();
// these work
testBus.push({ a: 242124 });
testBus.push({ b: "jee" });
// this works:
testBus.plug(Bacon.interval(1000, 0).map(() => ({ a: 999 })));
// this does not compile
testBus.plug(Bacon.interval(1000, 0).map(() => ({ a: 999 })));
// type testing  done by @milankinen
interface BBus {
plug(x: Bacon.Observable): void;
plug2(x: Bacon.Observable): void;
}
const b = (testBus as any) as BBus;
// this compiles without errors
b.plug2(Bacon.interval(1000, 0).map(() => ({ a: 999 })));
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/baconjs/bacon.js/issues/738?email_source=notifications&email_token=AACHQZERL7MIBM47CN2QBCLP2TNFDA5CNFSM4HYO5V5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GZWEKFQ, or mute the thread https://github.com/notifications/unsubscribe-auth/AACHQZCCXZH4TQISFSU4HETP2TNFDANCNFSM4HYO5V5A.

How about this: https://github.com/baconjs/bacon.js/commit/02c28fd2c4acea6a3c1eb1620c42fda763fceca4

Included in 3.0.4


Fix coding style in CLI error templates

Fix coding style exactly the same in two error templates.
--- error_exception.php	2015-07-21 08:21:16.000000000 +0900
+++ error_php.php	2015-07-21 08:13:20.000000000 +0900
@@ -1,16 +1,16 @@
 <?php defined('BASEPATH') OR exit('No direct script access allowed'); ?>

-An uncaught Exception was encountered
+A PHP Error was encountered

-Type:     <?php echo get_class($exception), "\n"; ?>
+Severity: <?php echo $severity, "\n"; ?>
 Message:  <?php echo $message, "\n"; ?>
-Filename: <?php echo $exception->getFile(), "\n"; ?>
-Line Number: <?php echo $exception->getLine(); ?>
+Filename: <?php echo $filepath, "\n"; ?>
+Line Number: <?php echo $line; ?>

 <?php if (defined('SHOW_DEBUG_BACKTRACE') && SHOW_DEBUG_BACKTRACE === TRUE): ?>

 Backtrace:
-<?php	foreach ($exception->getTrace() as $error): ?>
+<?php	foreach (debug_backtrace() as $error): ?>
 <?php		if (isset($error['file']) && strpos($error['file'], realpath(BASEPATH)) !== 0): ?>
 	File: <?php echo $error['file'], "\n"; ?>
 	Line: <?php echo $error['line'], "\n"; ?>

Related: #3983

We don't have an official style to follow for templates, they only adhere to common sense and in that regard, there's nothing broken and therefore nothing to fix.
That being said, I see you're aiming for some kind of consistency, but make it all consistent if that's the case - align to "Line Number" instead of "Filename", and inside the backtrace too.

I don't know what is the problem.
One outputs two newline characters, the other outputs single newline character.
I hope it will be fixed.
Do you say I must fix this PR like below?
Severity:    Parsing Error
Message:     syntax error, unexpected '$this' (T_VARIABLE)
Filename:    /Users/kenji/work/codeigniter/CodeIgniter/application/controllers/Welcome.php
Line Number: 24

Bar the word "fix", yes - that's what I'm saying. :)

Do you intend to do that or ... ?

I'll fix this PR. Please wait.

Is this okay?

Yep, thanks.


[BUG] Built in rule TA-000001 false-positive on site config object

Describe the bug
The builtin rule TA-000001 has the following rule:
"anyOf": [
        {
          "resourceType": "Microsoft.Web/sites/config",
          "allOf": [
            {
              "path": "properties.detailedErrorLoggingEnabled",
              "equals": true
            },
            {
              "path": "properties.httpLoggingEnabled",
              "equals": true
            },
            {
              "path": "properties.requestTracingEnabled",
              "equals": true
            }
          ]
        },

But, whilst those are the property names for properties.siteConfig in the Web/Sites app, they are not for the Microsoft.Web/sites/config resource.
This is a valid site config, but the rule is still triggering on this template--
resource configLogs 'Microsoft.Web/sites/config@2022-03-01' = {
  name: 'logs'
  parent: appService
  properties: {
    applicationLogs: { fileSystem: { level: 'Verbose' } }
    detailedErrorMessages: { enabled: true }
    failedRequestsTracing: { enabled: true }
    httpLogs: { fileSystem: { enabled: true, retentionInDays: 1, retentionInMb: 35 } }
  }
  dependsOn: [configAppSettings]
}

This means the rule will always trigger on a Web/sites/config object, because properties.detailedErrorLoggingEnabled doesn't exist.

Hi @tonybaloney,
Thank you for reaching out with this issue.
We are going to triage it and update back on this thread.
Thanks,
MSDO team

Hey @tonybaloney, it looks like the gap here is between the type of site config (controlled by the name property). The properties in the rule are correct for a Web config (where name is "web"), but the schema is different for the Logs config (where name is "logs").
As @EfratMalkaMS mentioned, we still need to triage this, but I wanted to give a quick update on initial investigation.

Hi @tonybaloney,
We're still investigating the issue, is this a blocking issue?
Thakns,
Efrat

@EfratMalkaMS yes I have this false-positive on 23 separate projects.

Hi @tonybaloney ,
We're still working on triaging the issue and wanted to check if you could possibly mitigate the issue by suppressing the specific finding in question that would be great.
Thanks,
Efrat


Cannot return the value of randomWords without console.log

I would love for it to output a word without me using console.log in a react app i am building. If possible, how do i go about it?

randomWords returns a word to your code, which is then responsible for doing something with it. It does not write things to React, or the console, or anywhere else. It just generates words.
The simplest thing you can do with it is pass it to console.log, but there is no restriction on what you do with it. For instance, you could write something like this in vanilla JavaScript:
document.querySelector('#some-input-element').val(randomWords());

And now the input element with that id would contain the word.
In a React component, you could use random-words to initialize a property in this.state, but this isn't a feature of random-words itself, because random-words is only responsible for its own job (returning a word to your code).


array index error in quickstart 1

While printing the target label, it prints the label at the index 0.
raw_test.target[0]
But shouldn't it actually be raw_test.target[9] since our instance is at index 9 here:
# Generate explanations

exp = explainer.explain_instance(
    labels=[0, 1, 2], # which classes to produce explanations for?
    instance=raw_test.data[9],
    num_features=5 # how many words to show?
)

I run the code with index 0 first and my label was shown as
rec.sport.baseball => 0 , (which is wrong classification)
but the rest of the explanation, the words and confidence scores were the same as your tutorial result. Then I changed the index to 9, and it showed the right label:
soc.religion.christian => 2

Hi @nilaycicekli  Thank you for pointing out the error. We will update the tutorial accordingly.


Docker: Really slow DVD ripping

Is it too be expected to rip six 30 minute episodes off of a DVD to take 4+ hours in docker?
I noticed the log says my hardware is:
[03:19:05]  - Intel microarchitecture Skylake
[03:19:05]  - logical processor count: 4
[03:19:05] Intel Quick Sync Video support: no

Is it because QSV isn't working? My CPU has QSV, and it works in another Docker container I am running.
log: E1616_164247586768.log

The docker image on dockerhub doesn't include Quicksync support. You can rebuild the image yourself if you want to include QS support.
As for the time it takes, you could try changing ripmethod: backup to ripmethod: mkv this helps for me personally, but it may not help.


Docker error, missing joblist lib. How to fix this????

Docker error, missing joblist lib. How to fix this????

I have made some modifications to the Dockerfile, you can download Dockerfile and run it again

thank you so much


[BUG] Removing an item from an ObservableList with TrnasformMany and a projection

Describe the bug
As shown in RolandPheasant/DynamicData.Snippets#5, I have set up an ObservableList with TransFormMany and a projection inside. When I remove an item from the ObservableList, it is not removed from the output. It works with an ObservableCache.
As Roland guessed, the projected item is not removed from the list because it is not found. I believe the culprit is here: https://github.com/RolandPheasant/DynamicData/blob/67367d0068531a94c9ec8a3e197a0b0055f5eca4/DynamicData/List/ChangeAwareList.cs#L519. The call to IndexOf is not using the equalityComparer passed when using TransformMany. I haven't found a way to pass it properly (yet).
Steps To Reproduce
Run the unit tests in https://github.com/RolandPheasant/DynamicData.Snippets.
I'm also happy to send a PR with unit tests here.
Expected behavior
The items should be removed from the output.
Screenshots
n/a
Environment

OS: Windows
Version: tested in 6.8 (the issue is still present in the master branch today)
Device: n/a

Additional context

could use https://docs.microsoft.com/en-us/dotnet/api/system.collections.generic.list-1.findindex?redirectedfrom=MSDN&view=netframework-4.8#System_Collections_Generic_List_1_FindIndex_System_Predicate__0__ and use the equality comparer

The issue is more how to pass the predicate to ChangeAwareList than using it ;-)

I may have found something. If that's OK, I'll submit a PR tomorrow and we can see if I'm headed in the right direction.

A PR is welcome. Thanks

I can confirm that this is fixed in 6.13.1 by #265.


adding the source-map-explorer dep and putting an npm script to run b…

…uild and analyze

This is an awesome addition. It will give us some nice insights into our bundle cost breakdowns


/uuid and /name cmds + new util

/uuid and /name commands
new utility to turn trimmed uuid into uuid with dashes

here, i actually did not know you could do that

any updates?


[Snyk] Security upgrade lint-staged from 15.2.2 to 15.2.4

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json

package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Inefficient Regular Expression Complexity SNYK-JS-MICROMATCH-6838728
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: lint-staged
    The new version differs by 11 commits.

8be6c8e chore(changeset): release (#1419)
4f4537a build(husky): fix release issue with Husky
95d096d chore(changeset): release (#1411)
72483cb docs: fix typo (#1417)
86fba6f build(deps): update dependencies (#1418)
1353320 test: add e2e test for linting difficult filename with ESLint + Prettier (#1413)
28cc901 ci(release): disable Husky hooks in release action
d698162 build(deps): update dependencies
ab263ba docs: fix typo (#1408)
6a14c53 chore(release): add empty changeset
738021b docs: add example avoid globs overlap & tasks edit files (#1405)

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Note: This is a default PR template raised by Snyk. Find out more about how you can customise Snyk PRs in our documentation.
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Inefficient Regular Expression Complexity

Solved in #239.


Set the notebook name in the browser tab title

Fixes #99
https://user-images.githubusercontent.com/591645/106870931-fa643000-66d1-11eb-8d69-6608fdacf6fe.mp4
Also add favicons for notebooks, files and terminals:

cc @chmp for review if you want to try out this change on Binder:

Oh wow, that was fast. Thanks a lot. I spun up the binder and that was exactly what I thought of.

Nice, thanks for testing!


Validate external names

fixes #1409

This change is

Coverage decreased (-0.02%) to 43.601% when pulling f8b213cbea74b5a278febe7d32de658347cf6d95 on aledbf:resolve-external-names into 27d4e8b3d51c5f15cbbc23ecc79f9a30d04f9b10 on kubernetes:master.


Running with my own dataset input

Hi, I'm trying to run this code with my own dataset,
but my dataset's log file is less than 2000 timestamps and it couldn't make landmark & pose graph.
I get the results file but how can I build a landmark & pose graph with few datasets?
This is the error log I got.

Maybe there's something wrong with your dataset. Please send your dataset along with your config to my email<EMAIL_ADDRESS>and I can take a look.


Review poison report for .NET 7.0

The poison report need to be reviewed before .NET 7.0 can be released.  All difference should have individual issues opened to track and resolve.
The poison report can be viewed by running the poison test or viewing the results from internal CI

As part of this, we should pay close attention to System.Memory and Microsoft.Bcl.HashCode - related to https://github.com/dotnet/source-build/issues/2871

@crummel, is there anything else to do on this?  It seemed like you analyzed as part of https://github.com/dotnet/installer/pull/14345 and opened issues for specific items to follow-up on.


feat: add and improve components, add new stores

Closes #75 #74
add new package: react-helmet
improve : App, Maps, Sidebar components
add new components: Error, Loading
improve  app store
add new stores : title, maps, links
delete unused OtherComponent
add new styles to theme.tsx
chore: refractor code with prettier

@olegnowikow the error store stores only one error. Using notistack is impractical


Add a trending product card on homepage

Is there any API written already to get the trending product?


How image generation is works?

How image generation is works? will it reply markdown response while using openai library? or how does it work?

Image generation works identically to the OpenAI image generation. It will generate an image, respond with either a url or a base64 encoded image. It uses Cloudflare R2 to serve the images to you if you use URL generation.


docs(rcon): default password is not "minecraft"

Describe the problem
The documentation states:

The default password is "minecraft" but change the password before deploying into production by setting RCON_PASSWORD.

It looks like this was changed in https://github.com/itzg/docker-minecraft-server/pull/2071.
Container definition
N/A
Container logs
N/A

Thanks. A PR would be great.


It looks like the 0.10.0 Jul 18, 2020 has a Cargo.lock

I'm rather new to Rust, so this may be a bogus report, but:
It looks like the 0.10.0 Jul 18, 2020 crate is being distributed with a Cargo.lock.
If I understand correctly, libraries are discouraged from distributing a Cargo.lock.

Thanks for bringing this up. I was not aware of this but I think this is the default behavior of cargo when it publish a package including examples (which is the case of tui). My understanding is that the lock file is ignored during the build process though.


Update all examples with new plotting methods from #1011

Once #1011 is merged, we should update our general examples to use the new plotting methods where appropriate since they remove a lot of boiler plate, in particular for simple cases.
Once it is released, we can also update sub-packages.

[ ] Refs #1089 -- include annular CoM in CoM example drive-by


Fix pagination when using an object with dir-paginate

When using an object with dir-paginate the pagination was not getting a valid length, so no numbers where shown within the pagination template.
The new code now checks to see if the collection is an array or object, and correctly returns the length.
I've also tested this in a clean Angular JS project to confirm the issue and the fix.
Thanks!

Whoa, can't believe I forgot to test for this. Good catch, thanks!

No problem, thanks for the quick merge and 0.9.4 release!


Bug 2213451:[release-4.11] makefile: block OCP upgrades beyond the X+1 version

This is an automated cherry-pick of https://github.com/red-hat-storage/odf-operator/pull/316

/bugzilla refresh

/lgtm


Nominate Meenal as recurrent contributor

I'd like to nominate @mjhajharia to be a recurrent contributor at ArviZ.
Following our governance doc, the next step is for you  to accept the nomination which you can do commenting on this issue. If you accept the nomination, there will be a council vote (in private) and we'll publish the result here.
In case it helps or you are still wondering, arviz.org has a description of the recurrent contributor role.

Meenal has already done a couple contributions to arviz repo and we also discussed the scope and potential api of xarray-einstats before I got started with it. I think it will be great to have her on slack so she can participate and better coordinate going further.

thanks, I'd love to be more involved and help out however I can, I accept!

Very excited to announce that the RV council voted and accepted the nomination! Welcome!
Can you reach out to arvizdevs (at) gmail (dot) com with the email you'd like us to use when adding you to the ArviZ slack?

done!

@mjhajharia there might have been a typo on the email. I can't find your message on arviz's email account

is this is the right email?

Invite sent, as expected, the email ended up on spam somehow, sorry.


What do I do if the Report a problem methods are missing from Visual Studio?

I cannot follow the instructions on this page because both the icon and the Help menu item are missing.
Looks like this:

If there is a setting that could be flipped to create this state we need to know how to restore the feature when needed. (I'm not sure what I did to get here)

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

ID: 2386e67e-a67b-de55-db57-e29a83dcb410
Version Independent ID: 6247fba9-4255-9a59-c206-0e8547e12335
Content: Report a problem with Visual Studio - Visual Studio (Windows)
Content Source: docs/ide/how-to-report-a-problem-with-visual-studio.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @madskristensen
Microsoft Alias: madsk

That is very strange. Somehow, your Help menu had the Send Feedback submenu removed. You can report a problem from the Visual Studio Installer instead.

That did get me there, albeit without the stuff you want to pass in from the main application. I still need to know how to add my report a problem functions back in to the main application.

@A9G-Data-Droid -- String.Empty (""), this repo is solely for documentation issues, with limited support options.  Please consider these resources:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.

@WilliamAntonRohm, I don't think pointing users to external sites is appropriate for something that should go to an internal Microsoft support team. The irony is that the only way to contact visual studio support is the thing that is broken. Thus, there needs to be a troubleshooting resource that doesn't depend on the broken workflow. Many documentation pages have troubleshooting sections at the bottom. The requirements of "Report a problem" should be listed as well as any information about how to disable or enable this functionality, if that's even possible. There needs to be a workaround to report a problem with report a problem.

@A9G-Data-Droid -- for issues with the Feedback Tool in Visual Studio, please email us at<EMAIL_ADDRESS> Apologies for not mentioning that sooner!

Thanks @WilliamAntonRohm, that is exactly what needs to be at the bottom of this article. You could just paste that line right in there.