Typescript fails to compile if I have methods on both the component and its mixin(s)

Version
2.5.9
Reproduction link
https://github.com/jiayexie/vue-typescript-declaration-repro
Steps to reproduce
The component file test.tsx doesn't build. It illustrates the problem with three scenarios:

If I define my component with mixin's method interface, and there isn't any method in my component, the build passes. I can call the mixin's method from my component render() function.
I define a method interface for my component, and use that to define my component. It cannot call the mixin's method even the mixins field references that mixin.
I make my own method interface extend the mixin's method interface. Now my component complains that I'm not implementing the mixin's methods.

What is expected?
I should be able to have the mixin and the component define different methods and use all of them from the component with type safety.
What is actually happening?
I cannot get a passing build with type declaration in this scenario. Please advise.

If a component defines a method with the same name as defined in the mixin, component's method overrides the method defined in the mixin. This could lead to unexpected bugs. Typescript implementation loudly tells you about method collision.
I think we should warn about collisions between mixins and component using them.
Ideally, you should follow some mixin namespacing rules to avoid collisions.

Hi @znck, I did not override the mixin's method. I am defining another method, and the compiler is complaining I did NOT override the mixin's method.

@octref @ktsn please look into this.

Type parameters on Vue.extend does not mean additional properties but property types of the component itself. In your example, you provide a mixin type as the type parameter but the component options does not follow the type, then the type check is failed.
Also, current Vue.extend is not expected to be provided some types manually. It's used for type inference from component options object.
We may support such use case in the future but TypeScript compiler cannot handle the use case currently.

@ktsn Thanks. I understand that the types should be inferred, but I am trying to use the methods from a mixin and not on my component. That's why I'm trying to work around the issue by manual typing.
I solved most of my issues by doing this:
const component = exampleMixin.extend({
    methods: {
        myOwnMethods() {
            this.mixinMethod();
        }
    }
});

However, if I want to mixin two different mixins this won't work. It is basically a problem of single vs multi-inheritance. Is it possible to support inferring types from the mixins array?

It's impossible to infer a component type from mixins array. We need to use Vue.extend or some wrapper function to do that.

Closing as a wontfix for now.


Get quote by number

Linked to https://github.com/XeroAPI/xero-node/issues/461

@RettBehrens - having more than one of the same methods defined on a path is invalid for OpenAPI specs.
'/Quotes/{QuoteID}':  can only have one "get"
I'll merge this PR and add a new path for  '/Quotes/{QuoteNumber}':


Research How Terraform is Handling JSON Policies

We can check how Terraform or other IaC apps are detecting the changes in JSON Policies and how are they checking if two policies are the same thing.
We can read their codes to gain an understanding of their logic.

OK seems this issue with the AWS API has been known. Terraform has these checks for it:

This one which itself has a call to:

This one

The overall logic can be translated to TypeScript and I think this can help us find a way to handle it properly.

Examples of our current ad-hoc handling of this issue are:

ECR
IAM
RDS (sort of, not a policy doc per se, but a similar issue)
S3 (not actually dealing with the issue, the module and its tests are just too simple to run into problems, yet)
VPC (also just haven't run into the issues, yet, because we're not stretching this in the test suite for this module)


System Messages blocked

Hello!
I think this might have happened after an update, but I can no longer see system messages.
"/roll" or when my friends come online is no longer shown.

happens to me too

Thanks, I've updated this and changed the functionality to work with the new /who limit.
Please let me know if this has resolved the issue or if you have any other feedback.


Xelah USFM editor integration

Usfm editor using Proskomma, Epitelete-Perf-Html and Xelah. issue #555

phosphor-react is already present in package.json.
gotochapter is fixed.
sessionData is not an error. a clean install and creating a new user should fix that.

Thanks @samueljd! Does merging this PR mean that an existing user in Ag cannot be used further and a new user needs to be created?

Not essentially. It works for me even with old usernames. I was suggesting a quick fix for testing.
I'll  test it  tomorrow and see if I can replicate it.
Moreover this PR has commits pertaining to the editor alone and it doesn't involve any of login or userprofile code. So if the error persists, it's either from a previous merge, or I've to take a deep look into the code.

@samueljd while creating the new year


Use archvied? instead of ems_id.nil?

We don't nullify ems_id for archived containers entities anymore, so instead of checking ems_id.nil? we should use the archived? method that checks whether deleted_on is nil.

@miq-bot add_label providers/containers

@simon3z @cben @zeari @Ladas Please review


DTFS2 :  GitHub Runners

self-hosted Git hub runners

shall we make a few more changes to it to include new app service plans? - this since it's going to rebuild the infrastructure anyway (we can discuss it on slack)

Sounds good


adds contacts.getByIdBatch method

@brainflake tests failed but I think its unrelated?

Yep, the failures are unrelated. Looks good. Thanks @masonator


set up clientPublicRepository to /client/dist when using no front gen…

…erator

@tcheymol why client/dist instead of client/build ?

I don't see why public directory should change


[doc] Update SIL.rst to reflect debug_value_addr deprecation

And restructure the "Debug Information" section to make it flow better.

@swift-ci please test


Produce project ZIP file as part of CI

So we don't have to do this manually when publishing to the Epic Marketplace.

I also tried to set Travis up to do a build/cook/package, cause this would be a good thing to test in an automated fashion. But the cook always fails on Travis, and I'm not entirely sure why. Cooking requires compiling shaders, so my guess is there's something missing that prevents that from working. Could be as simple to solve as a missing DirectX SDK, or as difficult to solve as missing GPU hardware.


can't uncompress tar.xz files in debian:jessie image

I can't uncompress tar.xz files in your debian jessie image.
root@cf8b1803ada8:/# tar -xJf node-v4.4.3-linux-x64.tar.xz
tar (child): xz: Cannot exec: No such file or directory
tar (child): Error is not recoverable: exiting now
tar: Child returned status 2
tar: Error is not recoverable: exiting now
root@cf8b1803ada8:/# curl -L https://nodejs.org/dist/v4.4.3/node-v4.4.3-linux-x64.tar.xz | tar -xJ
tar (grandchild): xz: Cannot exec: No such file or directory
tar (grandchild): Error is not recoverable: exiting now
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 8070k  100 8070k    0     0  1688k      0  0:00:04  0:00:04 --:--:-- 1989k
tar: Child returned status 2
tar: Error is not recoverable: exiting now

That's because the xz utility is not included in the default minbase set of packages for Debian; can you try again after installing the xz-utils package?

You are right, sorry for disturbing you with such a simple issue. Thanks!

No problem, glad you got it solved. :+1:

@tianon @chrpinhanedo
can you please explain this as i don't understand what changes you did here? sorry i am new to linux but please explain this as i am also facing same error?

You need to install the "xz-utils" package.


Public document out of date

Hello team,
I'm a Microsoft Support engineer, who works on the ABRS team.
Looking at this MS document (https://docs.microsoft.com/en-us/azure/site-recovery/azure-to-azure-how-to-enable-zone-to-zone-disaster-recovery#faqs) that is out of date, the date listed is from 04/08/2020 and there is a newer MS document that is 5-month-old, and it shows Brazil South been added
https://azure.microsoft.com/en-us/updates/brazil-south-availability-zones-now-generally-available/
Neither me nor the customer I'm working with found any newer document showing Brazil South been supported for Zone-to-Zone replication.

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

ID: e4bd4d9f-e7a7-782c-53fb-e5023ef39640
Version Independent ID: f6488b68-615b-ce4f-188e-392099e93272
Content: Enable Zone to Zone Disaster Recovery for Azure Virtual Machines - Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-how-to-enable-zone-to-zone-disaster-recovery.md
Service: site-recovery
GitHub Login: @sideeksh
Microsoft Alias: sideeksh

@Ticozolano74 -
Thanks for brining this to our attention. We will investigate it further and update you shortly.

@Ticozolano74 - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Add an explicit scala-library dep.

As mentioned in the BUILD file comment: this is necessary because we do not currently implicitly add this dependency. See https://github.com/pantsbuild/pants/issues/14171.
Fixes #5.

Out of curiosity, where does this repo specify its scala version as 2.13? I thought Pants would use 2.12 by default.

Out of curiosity, where does this repo specify its scala version as 2.13? I thought Pants would use 2.12 by default.

https://www.pantsbuild.org/docs/reference-scala


Update opensearch-2.9.0-test.yml

Description
Added new datasources master key config required for datasource APIs.
Issues Resolved
Fix for Integ test failure https://github.com/opensearch-project/sql/issues/1866#issuecomment-1634820114
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peterzhuamazon  This is dummy one for integ test cluster domain. For every domain, user has to set this up on their own. Since integ test domains are temporary ones, it won't be an issue.


Constrain table constraint to record

Purpose
$subject.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/13709
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18299 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18299   +/-   ##
=======================================
  Coverage   15.06%   15.06%
=======================================
  Files          48       48
  Lines        1248     1248
  Branches      192      192
=======================================
  Hits          188      188
  Misses       1046     1046
  Partials       14       14

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 8936b87...8bbe00e. Read the comment docs.


router-perf: deploy only the objects for the configured TERMINATIONS

Current behavior: No matter what we configured for TERMINATIONS, all 4 namespaces' objects are created. https://github.com/cloud-bulldozer/e2e-benchmarking/blob/master/workloads/router-perf-v2/http-perf.yml.tmpl
Expected: If I set TERMINATIONS to http, only namespace of http and the objects under it are created. Same for other TERMINATIONS combinations.
Why: A customer case needs to evaluate the performance while pod number increases for http request. Because currently we installs all 4 namespaces, the pod number is 4 times of the pods actually visited by mb. Workaround can be using a customized http-perf.yml.tmpl with only http, but I think it's good to have this built in.
Please kindly review this enhancement request, if it is ok, I can open a pr to do it.

Hi @qiliRedHat , I was already aware of this issue, (however I didn't fix it yet as you noted here). This could be implemented with golang-templates included in kube-burner. I already have some WIP code to fix it, let me wrap it up it now.


Android QUERY_ALL_PACKAGES permission

The app requires use of the QUERY_ALL_PACKAGES permission, Google play console require to provide a description and short video of the core feature in the app that requires this permission.
What feature of thingboard require this permission?

Hello @luiscubilla,
the QUERY_ALL_PACKAGES is not used anymore.
Best Regards.


HDDS-4393. Test CI failures after force push.

What changes were proposed in this pull request?
Test CI failures after force push
(Please fill in changes proposed in this fix)
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4393
(Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HDDS-XXXX. Fix a typo in YYY.)
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

Hi, just curious? What kind of force push was necessary and why? (I have no problem with that, just interested to learn.)


Don't ask for an invalid group in an HRR

If the client sends us a group in a key_share that is in our
supported_groups list but is otherwise not suitable (e.g. not compatible
with TLSv1.3) we reject it. We should not ask for that same group again
in a subsequent HRR.
Fixes https://github.com/openssl/openssl/issues/21157

This pull request is ready to merge

Fix up pushed to address the feedback. @t8m and @tmshort - please can you reconfirm?

Still good, though the 1) on it's own line is ugly!

Pushed to master/3.1/3.0. Thanks.


admin route not found

Trying to make it run with OC2.3
Works fine for the hello world message on catalog. For standard admin however, I am always getting a "Route not found" message no matter what route I am writing (including the standard hello message).
Suggestions, how I could debug to find the problem?

Alright, my bad! Sorry! Somehow, I forgot to transfer the router.php to the admin folder! No wonder, it didn't work out.


Issues with callbacks (infinite loops, record status...)

There are various issues when dealing with callbacks when destroying a record:

A record is dirty after being destroyed. and Rails therefore considers that there are still changes to be saved to the database. Thus, if an after_destroy callback modifies the deleted record again, this may trigger new unwanted callbacks. Moreover, this prevents a record to be recovered after being destroyed without reloading it.
There is no way to know if a record was soft deleted or hard deleted in a destroy callback.
The after_destroy_commit callbacks are not called when hard destroying a record.
If a record is destroyed and then modified, the after_destroy_commit callbacks are called twice. This is what triggers issue #103.

This PR solves all these problems with the following changes:

The changes on the paranoid_value are removed when destroying a record.
The instance variable @destroyed is set to true only when the record is removed from the database, ie in destroy_fully!. This is how Rails manages destruction and knows that a record is no longer persisted. Thus, deleted? indicates if a record is deleted (soft or hard) and deleted_fully? indicates if a record is hard deleted.
The instance variable @_trigger_destroy_callback is now also defined when hard destroying a record.
The @_trigger_destroy_callback variable is set back to nil after soft destroying a record at the end of a transaction or when a new transaction starts. This is how Rails solved a similar issue when creating a record. This means that if a record is destroyed and updated in the same transaction, the after_destroy_callbacks are correctly called once only with the very last versions of Rails (5.2.1.rc1 as of now). Note that for Rails versions older than 5.1, the callbacks are still incorrectly called twice even with two separate transactions.

This PR also adds a test with the destruction of a record with ActiveStorage attachments. The loading of ActiveStorage is kinda complicated and there might be room for improvement. The previous change fixes the SystemStackError described by issue #103 but does not prevents the destruction of attachments when soft destroying a record.
There are still open issues. For example, there is still no way to know if a record is expected to be soft or hard deleted in a before_destroy callback. There is also an undefined behavior when destroying and recovering a record in the same transaction. Should after_destroy_commit be called? It currently depends on the Rails version...

It would be really great to get this merged.

Since this a rather large set of changes, I'll be merging them in one by one.

Sorry I did not respond earlier but I am quite busy for now.
I rebased my branch on master and the only real issue I had was with some change with ActiveRecord 6.0 where the state

I just noticed that the test about callbacks that I added failed with newer Rails versions. Actually, it failed only with Rails > 6.0.0 since Rails fixed issues about commit callbacks starting from version 6.0.1 (PR 37251). This PR fixes a lot of issues and therefore makes most of my changes completely useless with newer Rails versions. However, they are still needed with older versions.
So I updated my code to take theses changes into account. The only change still needed for Rails 6 is the setting of @_trigger_destroy_callback in destroy_fully! This means that the dirty hack that I added to the deleted? method with its optional parameter is no longer needed. Note that I decided to not support Rails 6.0.0, since this is the only subversion which fails the test.
As for the ActiveStorage test, Rails changed ActiveStorage loading so I had to add a "fake" Rails module to prevent a crash. But I also noticed that it will change again in future Rails version so I prefer to wait for the stabilisation of the Rails 6.1 branch before fixing it for the active_record_edge gemfile.


chore: update packages with minor and patches updates

Update packages whit minors and patches updates.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


have any plans to upgrade to javafx11?

@linxu At the moment unfortunately I don't have any time at all...
But as always: Every PR is welcome ;-)

It would be great if this project could upgrade to JavaFX11 and support JDK11 LTS.


Refactor signer to work for Reader

This was inspired by reading issue #81.
The following test program works after these changes (before it paniced):
func main() {
	bucket := "MY_BUCKET"
	file, _ := os.Open("/tmp/large_file")
	fi, _ := file.Stat()
	length := fi.Size()
	largeKey := "/test/golang/large_file"
	largeFile := s3.PutObjectInput{
		Bucket:        &bucket,
		Key:           &largeKey,
		Body:          file,
		ContentLength: &length,
	}

	simpleKey := "/test/golang/simple_key"
	simpleLen := int64(len([]byte("Simple test")))
	simpleUpload := s3.PutObjectInput{
		Bucket:        &bucket,
		Key:           &simpleKey,
		Body:          ioutil.NopCloser(bytes.NewReader([]byte("Simple test"))),
		ContentLength: &simpleLen,
	}

	client := s3.New(&s3.S3Config{aws.DefaultConfig})

	_, err := client.PutObject(&simpleUpload)
	if err != nil {
		log.Fatal(err)
	}

	_, err = client.PutObject(&largeFile)
	if err != nil {
		log.Fatal(err)
	}
}

I manually verified that the case of using a the file from os.Open it does go through the Seek codepath (also, it was a 1.5G file and my ram usage did not noticeably increase).
Note: running go test ./... passed, but make gives me an error both before and after this change (missing package: protocol/ec2).

As per the discussion in #81 I think it might be dangerous to silently buffer into memory when a Reader is not also Seekable. I would rather see an explicit wrapper like ioutil.NopCloser that converts a non-seekable Reader into something that responds to Seek() (and does nothing) in order to meet the protocol. That way, if you really want to pass something in that is not seekable, you would do:
Body: aws.NonSeekable(reader)

Otherwise it seems like it might be too easy for a user to unknowingly pass Body: reader of an arbitrary reader without realizing the memory implications.
Otherwise I think this is a good change.
Thoughts?

I understand the concern, but I feel having to know about a helper method to use a reader is slightly less usable.
I think if someone's uploading large objects that aren't seekable (e.g. reading a file through gzip) they should be using multipart uploads anyways and providing an easier path to using those transparently will go a lot of the way to assuage your concern.
I'll give an implementation that is more explicit about that tradeoff a go when I get the chance and see how it looks.

I feel having to know about a helper method to use a reader is slightly less usable.

I'm not sure usability would really be affected, since it would be very similar to the already-accepted practice of wrapping readers in ioutil.NopCloser(reader) for those methods that require it (like in the HTTP client itself). Concretely, it would look something like this:
s3.PutObject(&s3.PutObjectInput{Body: awsutil.NopSeeker(gzippedFile), ...});

Instead of:
s3.PutObject(&s3.PutObjectInput{Body: gzippedFile, ...});

That doesn't seem any less usable, just more explicit, and as far as I understand, explicit seems to be the preference.

I think if someone's uploading large objects that aren't seekable (e.g. reading a file through gzip) they should be using multipart uploads anyways

Ideally, yes, but realistically not all users are aware of this best practice. IMO this is why the io.ReadSeeker declaration is important, because it is documentative of the expected interface.

and providing an easier path to using those transparently will go a lot of the way to assuage your concern.

Sure, but work on managed upload support is not yet planned. By the same token, the managed upload component, when built, can be the thing that provides said "easier path" to transparent usage, whereby the low-level interface can maintain its clear expectation.

This is no longer really relavent with the current develop branch.
Perhaps we should revisit this in the future, but for now it doesn't make sense to leave this open I think.
I did do a bit more work around this, expressed in this commit.
Best,
Euan


CudaText: micro suggestion

For this patch https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/editors/cudatext/dont-check-update.patch
I added to CudaText the global variable which you can change to False.
to disable "Help / Check for updates".
https://github.com/Alexey-T/CudaText/commit/d21fedd9a3f608220938b6103421ebf3cdb4e0e4
@sikmir

OK, I will use it with next release.

OK, I will use it with next release.


recording each task's input in state server

Current behavior:
Today we never record input for state server, we should record it for the debugger.
Expected behavior:
Recorder the activity's input as well
What is the motivation / use case for changing the behavior?
Additional information you deem important (e.g. I need this tomorrow):

We introduce debugger recorder mode which will recorder input as well


Spatial Coverage of Resource shows ambiguous labels "North Longitude", "East Latitude"

Using www.hydroshare.org
I tried to create a generic resource (csv file with some snow reports).
When I edit the details of my resource, I can edit the coverage.
The form asks me for "North Longitude", "East Latitude", "South Longitude", "West Latitude". This is ambiguous. The correct labels should be "North Latitude", "East Longitude", "South Latitude", "West Longitude".
Please see the screenshot.

I just checked dev.hydroshare.org, and this one still exists.  It doesn't appear to have been fixed.

You are right. I forgot to change it for the page on edit form as well. I'll take care of it.

Checked on alpha.hydroshare.org


Add field translation functions

The existing field_map() function was able to translate human-readable names into field IDs. Renamed that to field_to_id() and added similar functions to similarly translate to full names and to aliases as well.
This will make it easier to work with fields when we read user input or write output that humans as supposed to parse easily.

We discussed this, but one thing to note is that not all issue fields are available from the /field endpoint. Because of this, a field may appear in an issue which is not present, leading to valid fields being rejected. One example is the parent field.

Tests I did (manual):
jirate ls --fields issuetype,parent,summary jirate ls --fields issuetype,summary jirate ls --fields issuetype,customfield_12319275
All worked correctly.

Found one fun quirk in the _fields_cache data we get from python-jira, let's not merge this while I investigate. It may need some tweaks.


Possible Bug: Investigate what happens if the url is changed while the hook is on the page

Likely, the hook will not re-fire, and the state will become stale.
This will be particulaly obvious if two parts of the ui use hook into the same query parameter. I expect when one updates the value, the other will not sync

Can confirm - the hook will not refire, I investigated slightly into the idea of capturing the history event but the only URL change event that is created is popstate essentially for when the user clicks Back. Because the hook uses history.replaceState instead of history.pushState there's no meaningful purpose to listen to that or do anything with it.
As part of investigating I noticed that if the setQuery is using an updater function, the state it sets ends up being stale, so have raised a PR to resolve that bug, however further investigation is likely wanted for components becoming stale if they share a state key.


Fix "some chats seem missing" bug caused by pinned chats

In case there are a lot of chats with some of them being pinned  they can't be properly fetched ending up in error "some chats seem missing" with the counter of chats infinitely increasing.
The reason of this is the fact that pinned chats (and messages in those) returned out of normal order so OffsetDate can't be properly used.

Turns out, this was not the only bug in chats loading logic, so I have rewritten it almost completely.
There are some comments in updated code. In short:

offset_date can actually be used with pinned chats, but value must be taken from last response.Dialogs and not last response.Messages (since messages are sorted just by date and dialogs are sorted by pin+date);
offset_date alone is not enough, offset_id and offset_peer are also required otherwise some chats with the same message time and/or id may be skipped or duplicated;
even with correct offsets pinned chats may be returned twice in different dialogsSlices, so some map[chatID]bool is always required;
sometimes a second loading pass may be required (if some chat receives new message while the list is loading, this chat will "jump to top" and may not make it into any of dialogsSlices).

It also turned out that limit is not an "upper limit" but more likely a "desired count". Since TG may return more chats in response (for example, if 3 chats are pinned and limit: 1, the response will contain 7 chats: 3 pinned and 4 regular ones). It did not introduce any bugs but just "fun".
Have these changes fixed your problem?
In any case, thank you for PR.

@3bl3gamer I tried different scenarios which were broken before and they are working perfectly fine now. The pull request is not required anymore as even more cases are covered now. Thank you!


Dataset

Could you provide all the datatset used in paper?Thanks!

I got an error "TypeError: Expecting matrix here" where I run the demo.py. Is there anything wrong with the dataset "Caltech101-20.mat"? Has anyone encountered this problem？
Traceback (most recent call last):
File "E:/experiment/2019-ICML-COMIC/COMIC-release/code/demo.py", line 9, in
X_list, Y = load_data(data_name)
File "E:\experiment\2019-ICML-COMIC\COMIC-release\code\data.py", line 11, in load_data
mat = sio.loadmat(data_dir)
File "E:\experiment\2019-ICML-COMIC\venv\lib\site-packages\scipy\io\matlab\mio.py", line 208, in loadmat
matfile_dict = MR.get_variables(variable_names)
File "E:\experiment\2019-ICML-COMIC\venv\lib\site-packages\scipy\io\matlab\mio5.py", line 292, in get_variables
res = self.read_var_array(hdr, process)
File "E:\experiment\2019-ICML-COMIC\venv\lib\site-packages\scipy\io\matlab\mio5.py", line 252, in read_var_array
return self._matrix_reader.array_from_header(header, process)
File "mio5_utils.pyx", line 675, in scipy.io.matlab.mio5_utils.VarReader5.array_from_header
File "mio5_utils.pyx", line 721, in scipy.io.matlab.mio5_utils.VarReader5.array_from_header
File "mio5_utils.pyx", line 894, in scipy.io.matlab.mio5_utils.VarReader5.read_cells
File "mio5_utils.pyx", line 666, in scipy.io.matlab.mio5_utils.VarReader5.read_mi_matrix
TypeError: Expecting matrix here
Process finished with exit code 1

There are some unknown errors in the dataset "Caltech101-20" provided by this depository. You need to look at datasets from other places, such as "https://github.com/youweiliang/Multi-view_Graph_Learning".


Outlook view

Here we are.  Removed calls to the original SceneList and replaces them with OutlineView since they do the same thing.  So now ctrl-atrl-o calls Outline View

The outline looks good, but markdown (italic, bold, and underline) don't seem to be working.  Did you pull in the latest grammar from master?


Core documentation link broken

I'm submitting a ...  (check one with "x")
[*] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
Core documentation link (http://core.ngx-translate.com/) on home page results in a "not found".  Perhaps it should be linked to https://github.com/ngx-translate/core/blob/master/README.md instead

Thanks you're right, do you want to make a PR on this file https://github.com/ngx-translate/ngx-translate-website/blob/master/src/app/components/app.component.ts ? Or I can make the change otherwise.


Mocha 6.2.2 binary does not exist when installed with yarn

Prerequisites

[x] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
When installing<EMAIL_ADDRESS>the "mocha" command does not exist inside of node_modules/.bin
Steps to Reproduce
> yarn --version
1.19.1
> node --version
v12.13.0

mkdir test
cd test
yarn init -y
yarn add -D mocha
ls -la node_modules/.bin/mocha # ls: cannot access 'node_modules/.bin/mocha': No such file or directory

If done with<EMAIL_ADDRESS>it exists as expected
Expected behavior: [What you expect to happen]
That the mocha binary would exist
Actual behavior: [What actually happens]
 ls: cannot access 'node_modules/.bin/mocha': No such file or directory

Reproduces how often: [What percentage of the time does it reproduce?]
Every time
Versions
only fails for version 6.2.2 (latest at time of writing)

It wasn't clear in my original description, but using npm, it works as expected on every version.

There are a few issues in Yarn about this, I don't think this has anything to do with Mocha.
Please try: yarn --check-files or install Mocha a second time (without uninstall).

A subsequent install, nor the yarn --check-files worked. The thing that's add is that is works every time with<EMAIL_ADDRESS>but never with<EMAIL_ADDRESS>Additional info, upon installing, it appears the node_modules/mocha/bin directory have some differences between the two versions:
6.2.1:
> ls -la node_modules/mocha/bin
total 24
drwxr-xr-x 2 [redacted] [redacted] 4096 Nov 20 16:05 .
drwxr-xr-x 6 [redacted] [redacted] 4096 Nov 20 16:05 ..
-rwxr-xr-x 1 [redacted] [redacted] 4565 Oct  8 08:46 mocha
-rwxr-xr-x 1 [redacted] [redacted]  185 Oct  8 08:46 _mocha
-rw-r--r-- 1 [redacted] [redacted]  306 Oct  8 08:46 options.js

6.2.2
ls -la node_modules/mocha/bin
total 12
drwxr-xr-x 2 [redacted] [redacted] 4096 Nov 20 16:06 .
drwxr-xr-x 5 [redacted] [redacted] 4096 Nov 20 16:06 ..
-rw-r--r-- 1 [redacted] [redacted]  306 Nov  1 16:18 options.js

Perhaps a dependency updated from 6.2.1 to 6.2.2 that skips some preinstall/postinstall script that creates those binaries? I've tried a few of the other solutions on the yarn issues to no avail. Below are the ones I found, but feel free to post others. Also feel free to close this issue because it seems to be a yarn issue.

@ksmithut thank you.
I tested on my Windows10 and the "node_modules/.bin" binaries are also missing. Additionally existing files in "node_modules/.bin" get deleted, so this folder remains empty at the end. There is an error message "EPERM: operation not permitted, unlink D:......bin\yarn".
The same result for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I played around with the read-only attribute of "node_modules" folder, but without success.
I will keep open this issue for two weeks, then it will be closed. Having a look at yarn's issue queue tells me to do this.


Enable-WriteCore-feature-for-operationalinsights

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.OperationalInsights


Android keeping the app state alive

When a user presses back at the root it completely closes the app (state is removed), my app needs to stay alive.
When the user presses the center button to go back to the android home screen the app is kept alive (state is in tact).
It seems the android Activity is terminated onDestroy() when the user presses back.
I already use a native background service (which stays alive) but the UI state is gone.
What is the right way if any to keep the app state alive when the user presses back at the root?
Other native apps seem to be able to do it but I'm not as experienced in android.

probably wasn't the place to ask this, but I had to override the onBackPressed in android to
  @Override
  public void onBackPressed() {
    moveTaskToBack(true);
  }

Bump

@abacaj, would be nice of you to actually share the solution after you so victoriously claimed you solved it..


Add client properties for secure mds

Description
This PR adds required properties for all CP components to talk to secured Metadata Server.  The detailed doc is here
Fixes # (FF-6440)
Type of change

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

How Has This Been Tested?
 molecule --debug converge -s rbac-mds-plain-custom-rhel
PLAY RECAP *********************************************************************
control-center1            : ok=71   changed=30   unreachable=0    failed=0    skipped=40   rescued=0    ignored=0
kafka-broker1              : ok=86   changed=33   unreachable=0    failed=0    skipped=64   rescued=0    ignored=0
kafka-broker2              : ok=79   changed=33   unreachable=0    failed=0    skipped=46   rescued=0    ignored=0
kafka-broker3              : ok=79   changed=33   unreachable=0    failed=0    skipped=46   rescued=0    ignored=0
kafka-connect1             : ok=75   changed=31   unreachable=0    failed=0    skipped=48   rescued=0    ignored=0
kafka-rest1                : ok=71   changed=30   unreachable=0    failed=0    skipped=40   rescued=0    ignored=0
ksql1                      : ok=72   changed=30   unreachable=0    failed=0    skipped=44   rescued=0    ignored=0
mds-zookeeper1             : ok=2    changed=2    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0
schema-registry1           : ok=67   changed=29   unreachable=0    failed=0    skipped=42   rescued=0    ignored=0
zookeeper1                 : ok=61   changed=28   unreachable=0    failed=0    skipped=38   rescued=0    ignored=0

MDS runs on secure mode
Verified properties from all components
Keystore location and path would get added if ssl_mutual_auth_enabled is set to true
Kafka broker
confluent.metadata.sasl.mechanism=PLAIN
confluent.metadata.security.protocol=SASL_SSL
confluent.metadata.ssl.key.password=xxx
confluent.metadata.ssl.keystore.location=/xxx/kafka_broker.keystore.jks
confluent.metadata.ssl.keystore.password=xxx
confluent.metadata.ssl.truststore.location=/xxx/kafka_broker.truststore.jks
confluent.metadata.ssl.truststore.password=xxx

Control Center
confluent.metadata.sasl.login.callback.handler.class=io.confluent.kafka.clients.plugins.auth.token.TokenUserLoginCallbackHandler
confluent.metadata.sasl.mechanism=OAUTHBEARER
confluent.metadata.security.protocol=SASL_SSL
confluent.metadata.ssl.truststore.location=/xxxx/control_center.truststore.jks
confluent.metadata.ssl.truststore.password=xxxxxxxxx

No impact with usecase which doesn't have either mds_tls_enabled or ssl_mutual_auth_enabled set to true

molecule --debug converge -s plain-rhel
confluent.metadata.sasl.jaas.config=org.apache.kafka.common.security.plain.PlainLoginModule required username="control_center" password="control_center-secret";
confluent.metadata.sasl.mechanism=PLAIN
confluent.metadata.security.protocol=SASL_PLAINTEXT

Test Configuration:
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules
[ ] Any variable changes have been validated to be backwards compatible

@nsharma-git Can you also confirm the bug number for this? Field Feedback should be triaged by Product and a bug should be created for us to track our actual work.

Thanks! I have added a corresponding ANSIENG JIRA and updated the in this PR


Add example with Hikari

add an example of usage with Hikari

from https://github.com/ClickHouse/ClickHouse/discussions/56127#discussioncomment-7433272
In JDBC driver .getConnection is generating an under the hood a single connection to the DB the heavy lifting of the connection pool is done outside the diver, by using.  Hikari externally solves the concurrency issue.
HikariConfig configuration = new HikariConfig();
configuration.setMaximumPoolSize(1);
configuration.setJdbcUrl("jdbc:ch:{hostname}:{port}/default?insert_quorum=auto&server_time_zone&server_version=<IP_ADDRESS>95");
configuration.setUsername(username);
configuration.setPassword(password);
configuration.setValidationTimeout(6000000);
HikariDataSource ds = new HikariDataSource(configuration);

Use Connection conn = ds.getConnection(); to get connection and you get it from hikari pool
I will provide a sample with full mvn example for using

hi @mzitnik and @mshustov  thanks for your response.  by the way, I can integrate hikari by myself. but , I also hope the community can provide a best practice example.

@mshustov Checked the code today and found an example how to use it just making sure that we want it in the ClickHouse docs is it correct.
@Young0516 is the example above is good enough.

@mzitnik thanks

@mzitnik let's add it to the docs


Make ShowToolchainsTask work with configuration cache

Fixes #20296

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


AuthClient.setCredentials method removed in TypeScript conversion

A user reported getting an error when upgrading to v23 of the googleapis npm module.  I tracked it down to the TypeScript conversion, where it looks like I accidentally removed the function
https://github.com/google/google-auth-library-nodejs/commit/5735a351269363320288ae9a43dc88937d83323a
We should probably add that back :)

This was fixed in the next branch, and will be part of the 1.0 milestone release.

So...this seems to mean that there is no way to call APIs on behalf of logged in users. Isn't this pretty much half the use cases for this library? Seems like it might warrant a 0.x release to fix, no?

You can work around this today with 0.x by directly assigning to the credentials property:
client.credentials = credentials;
Hope this helps!


Session keeps getting destroyed after a couple of minutes

I cannot use this service, as my session and instances get automatically destroyed after a couple of minutes of tinkering.
It happened twice in a row, using latest Chrome on Windows 10 (my environment should not matter though).

what are you running in the environment? We have some checks that detect some not permitted workloads and we could be having some false positives there.

@marcosnils I only copy & pasted commands from the steps shown in the terminal:

Created two nodes from the web interface
Initialized one node as master by running:  kubeadm init --apiserver-advertise-address $(hostname -i) --pod-network-cidr <IP_ADDRESS>/16
Initialized cluster networking on the worker node by running: kubectl apply -f https://raw.githubusercontent.com/cloudnativelabs/kube-router/master/daemonset/kubeadm-kuberouter.yaml

Got error message: The connection to the server localhost:8080 was refused - did you specify the right host or port?

I couldn't get any further that that.

Initialized cluster networking on the worker node by running: kubectl apply -f https://raw.githubusercontent.com/cloudnativelabs/kube-router/master/daemonset/kubeadm-kuberouter.yaml

You need to run this on the master node as well..  not on the worker node.
On the worker node you only need to run the kubeadm join output that the first command produces.
Just tested this myself and my session doesn't get terminated

It could be elasticsearch is cpu intensive?

yes, that's the reason.

@marcosnils Would it be possible to have a banner that warns the users when they exceed the permitted workloads?

Not something that's a priority for the moment. The environment is designed to learn docker and run "basic" examples. Anything resource intensive will automatically terminate the session


Issues with displaying sidebar on Lepiter ToC Page

Version: v0.8.2692
Discord thread on the issue with video.
When opening the ToC page from Spotter the sidebar has issues. When I open any other page on same database from spotter the ToC sidebar displays correctly.
The video shows a brand new database that I added just for this so reproducible across Lepiter databases

Thanks. Even simpler to reproduce is to search for Glamorous Toolkit Book


Refreshing the web page causes the static file read to be missed

Hello,
When I run the following code and add a text file called test.txt into the directory fun, it gets read on every subsequent web page refresh. Here's the code I used:
require 'rack-static-if-present'

use Rack::StaticIfPresent, :urls => ["/",'hidden.txt'], :root => "fun"

hello_world = lambda do |env|
  [200, {"Content-Type" => "text/plain"}, ["Hello World!"]]
end

run hello_world

If I open the URL from the address bar more than once there isn't a problem. Which suggest there is some sort of caching request being sent from the browser that needs to be handled.
--
James Robertson

Hmm! I barely remember the rationale for this gem!
It's not immediately obvious to me how that would happen, although this isn't a lot of code so maybe I'll look at it later. Do you have any more specific ideas?


Can't get Facebook Integration to work.

What should I put in as my valid callback URL.
Currently I listed the Site URL as http://localhost:3000/ but what should I use for the Callback URL. I am building this on Cordova and Angular.JS
Thanks

Here are my Facebook Ionic settings.

If you look at Ionic example, I think redirectUri is set to localhost/ there as well.

Thanks


cadvisor fails to collect stats

version: kontena/cadvisor:0.19.3
host: ubuntu 15.10, kernel: 4.2.0-30-generic
docker: 1.10.2 (aufs)
Cadvisor do not collect any information about node :(
CONTAINER                      CPU %           MEM USAGE/LIMIT      MEM %           NET I/O
web-1                                   N/A%            N/A / N/A                           N/A             N/A/N/A
It seems related to ""journalctl": executable file not found in $PATH" error message.
cadvisor container logs:
0307 14:42:23.645334 07297 storagedriver.go:132] No backend storage selected
I0307 14:42:23.645473 07297 storagedriver.go:134] Caching stats in memory for 2m0s
I0307 14:42:23.645544 07297 manager.go:128] cAdvisor running in container: "/docker/76fc2ce7a1afd366aaf0e568567c302ce9ee4869a63401582eb08cbe955dac11"
I0307 14:42:23.878388 07297 fs.go:108] Filesystem partitions: map[/dev/vda1:{mountpoint:/rootfs major:253 minor:1 fsType: blockSize:0}]
I0307 14:42:23.885057 07297 machine.go:48] Couldn't collect info from any of the files in "/etc/machine-id,/var/lib/dbus/machine-id"
I0307 14:42:23.885208 07297 manager.go:163] Machine: {NumCores:2 CpuFrequency:1797917 MemoryCapacity:2097971200 MachineID: SystemUUID:6F267954-DC39-4449-A325-D13F22609045 BootID:85802b30-7e46-4425-a18b-bb8d0186e641 Filesystems:[{Device:/dev/vda1 Capacity:42143100928}] DiskMap:map[253:0:{Name:vda Major:253 Minor:0 Size:42949672960 Scheduler:none}] NetworkDevices:[{Name:datapath MacAddress:0a:8f:5c:d5:08:2b Speed:0 Mtu:1410} {Name:dummy0 MacAddress:96:ef:53:be:12:88 Speed:0 Mtu:1500} {Name:eth0 MacAddress:04:01:b3:6c:1c:01 Speed:0 Mtu:1500} {Name:eth1 MacAddress:04:01:b3:6c:1c:02 Speed:0 Mtu:1500} {Name:weave MacAddress:fe:0a:77:d5:a2:8b Speed:0 Mtu:1410}] Topology:[{Id:0 Memory:2097971200 Cores:[{Id:0 Threads:[0] Caches:[]}] Caches:[]} {Id:1 Memory:0 Cores:[{Id:0 Threads:[1] Caches:[]}] Caches:[]}] CloudProvider:Unknown InstanceType:Unknown}
I0307 14:42:23.887549 07297 manager.go:169] Version: {KernelVersion:4.2.0-30-generic ContainerOsVersion:Alpine Linux v3.2 DockerVersion:1.10.2 CadvisorVersion:0.19.3 CadvisorRevision:e6a64da}
I0307 14:42:23.889713 07297 factory.go:194] System is using systemd
I0307 14:42:24.101227 07297 factory.go:236] Registering Docker factory
I0307 14:42:24.111452 07297 factory.go:93] Registering Raw factory
I0307 14:42:24.305147 07297 manager.go:1006] Started watching for new ooms in manager
W0307 14:42:24.305385 07297 manager.go:237] Could not configure a source for OOM detection, disabling OOM events: exec: "journalctl": executable file not found in $PATH
I0307 14:42:24.306494 07297 manager.go:250] Starting recovery of all containers
I0307 14:42:24.349215 07297 manager.go:255] Recovery completed
I0307 14:42:24.389957 07297 cadvisor.go:96] Starting cAdvisor version: 0.19.3-e6a64da on port 8989

It's because of docker 1.10.x .. cadvisor (and kontena) does not support it yet.

thanks for information.

Looks like cadvisor v0.21.0 supports Docker 1.10 + v0.22.0 added some crucial bugfixes

Yeah, latest cadvisor supports docker 1.10 and we are going to include it in Kontena 0.12. We might have done upgrade sooner but current way of running cadvisor in a container spikes cpu load through the roof when filesystem stats are collected. Kontena 0.12 will include "a great hack" that flips cadvisor process from container to host namespace (using nsenter). This allows us to use latest & greatest cadvisor without sucking all the resources from nodes.


Mbasiuk/chore fix salesforce it

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[x] Other... Please describe:
Fix Integration test

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:


Deleting a component in a BWA doesn't remove it from rendering when rerunning the app in VS

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Deleting a component in a BWA doesn't remove it from rendering when rerunning the app in VS.

This problem only occurs in VS, not VSC and not via the .NET CLI. Should this be closed in favor of a report using the Report a Problem process in VS? 👂
This problem occurs not only for stopping the debugger and restarting the app. It also occurs for closing the solution, re-opening the solution, and starting the app.
The problem is resolved by cleaning the solution.

Expected Behavior
Deleting the component and running the app again shouldn't still permit accessing/using the component.
Steps To Reproduce

Create a BWA from the RC1 template.
Add the following Components/Pages/Counter2.razor component to the server project:

@page "/counter-2"
@attribute [RenderModeServer]

<PageTitle>Counter 2</PageTitle>

Counter 2

Current count: @currentCount

<button class="btn btn-primary" @onclick="IncrementCount">Click me</button>

@code {
    private int currentCount = 0;

    private void IncrementCount()
    {
        currentCount++;
    }
}

Run the app. Navigate to /counter-2 to confirm it renders interactively.
Stop the debugger.
Delete the Counter2 component from the Components/Pages folder.
Rerun the app with the debugger.
Navigate to /counter-2 in the browser.

The Counter2 component still renders interactively.

Stop the debugger.
Clean the solution.
Rerun the app.
Navigate to /counter-2 in the browser.

The Counter2 component is gone.
Exceptions (if any)
None
.NET Version
8.0.100-rc.1.23455.8
Anything else?
.NET SDK:
Version:   8.0.100-rc.1.23455.8
Commit:    e14caf947f
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22621
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.100-rc.1.23455.8\
.NET workloads installed:
There are no installed workloads to display.
Host:
Version:      8.0.0-rc.1.23419.4
Architecture: x64
Commit:       92959931a3
RID:          win-x64
.NET SDKs installed:
3.1.426 [C:\Program Files\dotnet\sdk]
5.0.214 [C:\Program Files\dotnet\sdk]
6.0.317 [C:\Program Files\dotnet\sdk]
7.0.111 [C:\Program Files\dotnet\sdk]
8.0.100-preview.7.23376.3 [C:\Program Files\dotnet\sdk]
8.0.100-rc.1.23455.8 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.20 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.22 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.0-preview.7.23375.9 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.0-rc.1.23421.29 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.20 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.22 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.0-preview.7.23375.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.0-rc.1.23419.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.15 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.20 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.22 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.9 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.0-preview.7.23376.1 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.0-rc.1.23420.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found

@guardrex thanks for reporting this.
This likely looks like an up-to-date check issue in Razor, since likely VS thinks the assembly is up to date and doesn't need to rebuild it.

@jjonescz PTAL and see if you can repro, thanks!

Yes, can repro in VS 17.8 Preview 5 with RC2 and 8.0.100-rtm.23506.1. Looks like an issue for @dotnet/razor-tooling - the project is not built at all after the component is deleted. If you change something else apart from deleting the component (e.g., a line of code), the project is rebuilt and the component is removed properly.

Tooling is not involved in build. If its a fast up to date check issue, then its probably something missing from the Razor SDK props or targets files, or a change in the project template that violates some assumption they make. I believe those props and targets are owned by the compiler team.
That said, I do have a little experience with the FUTD so if you want me to take a look, or help out, let me know.

Investigated, but don't understand why FUTDC doesn't see the removed file.
When Counter2.razor is added, I see output:
Build started at 14:39...
1>FastUpToDate: Build acceleration is not enabled for this project. See https://aka.ms/vs-build-acceleration. (BlazorApp1)
1>FastUpToDate: Comparing timestamps of inputs and outputs: (BlazorApp1)
1>FastUpToDate:     Adding UpToDateCheckBuilt outputs: (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\BlazorApp1.dll (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\BlazorApp1.pdb (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.pdb (BlazorApp1)
1>FastUpToDate:     Adding newest import input: (BlazorApp1)
1>FastUpToDate:         C:\Program Files\dotnet\sdk\8.0.100-rtm.23506.1\Sdks\Microsoft.NET.Sdk.Razor\targets\Sdk.Razor.CurrentVersion.targets (BlazorApp1)
1>FastUpToDate:     Adding Compile inputs: (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Program.cs (BlazorApp1)
1>FastUpToDate:     Adding UpToDateCheckInput inputs: (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\App.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\MainLayout.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\NavMenu.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Counter.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Counter2.razor (BlazorApp1)
1>FastUpToDate: Input UpToDateCheckInput item 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Counter2.razor' is newer (2023-11-02 14:38:33.211) than earliest output 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll' (2023-11-02 14:34:28.129), not up-to-date. (BlazorApp1)
1>FastUpToDate: Up-to-date check completed in 0.7 ms (BlazorApp1)
1>------ Build started: Project: BlazorApp1, Configuration: Debug Any CPU ------
1>C:\Program Files\dotnet\sdk\8.0.100-rtm.23506.1\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(311,5): message NETSDK1057: You are using a preview version of .NET. See: https://aka.ms/dotnet-support-policy
1>BlazorApp1 -> C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll
========== Build: 1 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========
========== Build completed at 14:39 and took 00.612 seconds ==========

When Counter2.razor is removed, the output is:
Build started at 14:39...
1>FastUpToDate: Build acceleration is not enabled for this project. See https://aka.ms/vs-build-acceleration. (BlazorApp1)
1>FastUpToDate: Comparing timestamps of inputs and outputs: (BlazorApp1)
1>FastUpToDate:     Adding UpToDateCheckBuilt outputs: (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\BlazorApp1.dll (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\BlazorApp1.pdb (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.pdb (BlazorApp1)
1>FastUpToDate:     Adding newest import input: (BlazorApp1)
1>FastUpToDate:         C:\Program Files\dotnet\sdk\8.0.100-rtm.23506.1\Sdks\Microsoft.NET.Sdk.Razor\targets\Sdk.Razor.CurrentVersion.targets (BlazorApp1)
1>FastUpToDate:     Adding Compile inputs: (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Program.cs (BlazorApp1)
1>FastUpToDate:     Adding UpToDateCheckInput inputs: (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\App.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\MainLayout.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\NavMenu.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Counter.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Error.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Home.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Pages\Weather.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Routes.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\_Imports.razor (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\MainLayout.razor.css (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\NavMenu.razor.css (BlazorApp1)
1>FastUpToDate:         C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\BlazorApp1.csproj.BuildWithSkipAnalyzers (BlazorApp1)
1>FastUpToDate:     No inputs are newer than earliest output 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll' (2023-11-02 14:39:02.736). Newest input is 'C:\Program Files\dotnet\sdk\8.0.100-rtm.23506.1\Sdks\Microsoft.NET.Sdk.Razor\targets\Sdk.Razor.CurrentVersion.targets' (2023-11-02 14:33:11.979). (BlazorApp1)
1>FastUpToDate: Checking built output (UpToDateCheckBuilt with Original property) file: (BlazorApp1)
1>FastUpToDate:     Source      2023-11-01 11:36:26.984: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\MainLayout.razor.css' (BlazorApp1)
1>FastUpToDate:     Destination 2023-11-01 11:37:17.999: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\scopedcss\Components\Layout\MainLayout.razor.rz.scp.css' (BlazorApp1)
1>FastUpToDate: Checking built output (UpToDateCheckBuilt with Original property) file: (BlazorApp1)
1>FastUpToDate:     Source      2023-11-01 11:36:26.989: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\Components\Layout\NavMenu.razor.css' (BlazorApp1)
1>FastUpToDate:     Destination 2023-11-01 11:37:18.000: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\scopedcss\Components\Layout\NavMenu.razor.rz.scp.css' (BlazorApp1)
1>FastUpToDate: Checking items to copy to the output directory: (BlazorApp1)
1>FastUpToDate:     Checking copy items from project 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\BlazorApp1.csproj': (BlazorApp1)
1>FastUpToDate:         Checking PreserveNewest item (BlazorApp1)
1>FastUpToDate:             Source      2023-11-01 11:36:26.947: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\appsettings.json' (BlazorApp1)
1>FastUpToDate:             Destination 2023-11-01 11:36:26.947: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\appsettings.json' (BlazorApp1)
1>FastUpToDate:         Checking PreserveNewest item (BlazorApp1)
1>FastUpToDate:             Source      2023-11-01 11:43:26.620: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\global.json' (BlazorApp1)
1>FastUpToDate:             Destination 2023-11-01 11:43:26.620: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\global.json' (BlazorApp1)
1>FastUpToDate:         Checking PreserveNewest item (BlazorApp1)
1>FastUpToDate:             Source      2023-11-02 14:39:02.736: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\obj\Debug\net8.0\apphost.exe' (BlazorApp1)
1>FastUpToDate:             Destination 2023-11-02 14:39:02.736: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.exe' (BlazorApp1)
1>FastUpToDate:         Checking PreserveNewest item (BlazorApp1)
1>FastUpToDate:             Source      2023-11-01 11:36:26.945: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\appsettings.Development.json' (BlazorApp1)
1>FastUpToDate:             Destination 2023-11-01 11:36:26.945: 'C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\appsettings.Development.json' (BlazorApp1)
1>FastUpToDate: Project is up-to-date. (BlazorApp1)
1>FastUpToDate: Up-to-date check completed in 1.6 ms (BlazorApp1)
========== Build: 0 succeeded, 0 failed, 1 up-to-date, 0 skipped ==========
========== Build completed at 14:39 and took 00.042 seconds ==========

The input is not there, but FUTDC doesn't care. If I contrast that with adding and removing a .cs file, I get
Build started at 14:48...
1>FastUpToDate: Build acceleration is not enabled for this project. See https://aka.ms/vs-build-acceleration. (BlazorApp1)
1>FastUpToDate: The set of project items was changed more recently (2023-11-02 14:47:46.765) than the last successful build start time (2023-11-02 14:39:50.845), not up-to-date. (BlazorApp1)
1>FastUpToDate:     Compile item added 'Class1.cs' (BlazorApp1)
1>FastUpToDate: Up-to-date check completed in 0.5 ms (BlazorApp1)
1>------ Build started: Project: BlazorApp1, Configuration: Debug Any CPU ------
1>C:\Program Files\dotnet\sdk\8.0.100-rtm.23506.1\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(311,5): message NETSDK1057: You are using a preview version of .NET. See: https://aka.ms/dotnet-support-policy
1>BlazorApp1 -> C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll
========== Build: 1 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========
========== Build completed at 14:48 and took 00.643 seconds ==========

and then
Build started at 14:48...
1>FastUpToDate: Build acceleration is not enabled for this project. See https://aka.ms/vs-build-acceleration. (BlazorApp1)
1>FastUpToDate: The set of project items was changed more recently (2023-11-02 14:48:26.075) than the last successful build start time (2023-11-02 14:48:06.760), not up-to-date. (BlazorApp1)
1>FastUpToDate:     Compile item removed 'Class1.cs' (BlazorApp1)
1>FastUpToDate: Up-to-date check completed in 0.3 ms (BlazorApp1)
1>------ Build started: Project: BlazorApp1, Configuration: Debug Any CPU ------
1>C:\Program Files\dotnet\sdk\8.0.100-rtm.23506.1\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.RuntimeIdentifierInference.targets(311,5): message NETSDK1057: You are using a preview version of .NET. See: https://aka.ms/dotnet-support-policy
1>BlazorApp1 -> C:\Users\janjones\Code\issues\razor\9329-remove-component\BlazorApp1\bin\Debug\net8.0\BlazorApp1.dll
========== Build: 1 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========
========== Build completed at 14:48 and took 00.625 seconds ==========

Binlogs from VS don't even contain UpToDateCheckInput items for Counter2.razor (nor Class1.cs), so not sure how to debug this more.
Btw, the MSBuild items that make this work are defined here:
https://github.com/dotnet/sdk/blob/053416bc097e73bf24468818e0e35bce73580984/src/RazorSdk/Targets/Sdk.Razor.CurrentVersion.targets#L364

@drewnoakes is Razor missing something from our FUTD items that allows the project system to track items being removed from the project?

On a phone, but the fact that removing the razor file doesn't trigger the same "items changed" event for those items seems like it might be a bug on my side. We might not be including that item in the item hash. I'll transfer this to project system and investigate.

Transferred. Thanks for looking at it!


secret/configmap volume is failure beacuse of defining domain error.

Is this a BUG REPORT or FEATURE REQUEST?:

/kind bug

What happened:
create vmi using configmap/secret volume by the direct of document "https://kubevirt.io/user-guide/docs/latest/creating-virtual-machines/disks-and-volumes.html#containerDisk"， qemu process is not OK, check that there are some loop error logs in vmpod/compute container as follow and finally vmpod get a Error state:
{"component":"virt-launcher","level":"info","msg":"Driver cache mode for /var/run/kubevirt-private/secret-disks/app-secret-disk.iso set to none","pos":"converter.go:166","timestamp":"2019-04-03T01:59:48.657197Z"}
{"component":"virt-launcher","level":"error","msg":"missing target information for device /var/run/kubevirt-private/secret-disks/app-secret-disk.iso","pos":"virDomainDiskDefParseXML:9680","subcomponent":"libvirt","timestamp":"2019-04-03T01:59:48.657000Z"}
{"component":"virt-launcher","kind":"","level":"error","msg":"Defining the VirtualMachineInstance failed.","name":"vmi-secret","namespace":"default","pos":"libvirt_helper.go:96","reason":"virError(Code=20, Domain=20, Message='missing target information for device /var/run/kubevirt-private/secret-disks/app-secret-disk.iso')","timestamp":"2019-04-03T01:59:48.657841Z","uid":"f6320c6b-55b3-11e9-82d2-0c1262f5429b"}
{"component":"virt-launcher","kind":"","level":"error","msg":"Failed to sync vmi","name":"vmi-secret","namespace":"default","pos":"server.go:123","reason":"virError(Code=20, Domain=20, Message='missing target information for device /var/run/kubevirt-private/secret-disks/app-secret-disk.iso')","timestamp":"2019-04-03T01:59:48.657877Z","uid":"f6320c6b-55b3-11e9-82d2-0c1262f5429b"}

but I can find the app-secret-disk.iso device in vmpod/compute when the vmpod is Running as follow:
[root@vmi-secret secret-disks]# pwd
/var/run/kubevirt-private/secret-disks
[root@vmi-secret secret-disks]# ls -la
total 372
drwxr-xr-x 2 qemu qemu     33 Apr  3 01:58 .
drwxr-xr-x 8 root root    188 Apr  3 01:58 ..
-rw-r--r-- 1 root root 378880 Apr  3 01:59 app-secret-disk.iso

What you expected to happen:
vmpod Running, qemu process ok and virtctl console vmi is useful
How to reproduce it (as minimally and precisely as possible):
by the direct of document "https://kubevirt.io/user-guide/docs/latest/creating-virtual-machines/disks-and-volumes.html#containerDisk" configMap and secret section.
Anything else we need to know?:
my configMap:
kubectl create cm vmi-cm --from-literal=vmidisk=configmap --from-literal=app=configmap

option my secret:
kubectl create secret generic vmi-secret --from-literal=key1=supersecret --from-literal=key2=topsecret

my vmi specifications:
---
apiVersion: kubevirt.io/v1alpha3
kind: VirtualMachineInstance
metadata:
  labels:
    special: vmi-multus-multiple-net
  name: vmi-secret
spec:
  domain:
    devices:
      disks:
      - disk:
          bus: virtio
        name: containerdisk
      - disk:
          bus: virtio
        name: cloudinitnoscriptdisk
      - disk: {}
        name: app-secret-disk
        # set serial
        serial: D23YZ9W6WA5DJ487
      interfaces:
      - bridge: {}
        name: default
    machine:
      type: ""
    resources:
      requests:
        memory: 4096M
  networks:
  - name: default
    pod: {}
  terminationGracePeriodSeconds: 0
  volumes:
  - containerDisk:
      image: kubevirt/fedora-cloud-container-disk-demo:latest
    name: containerdisk
  - name: cloudinitnoscriptdisk
    cloudInitNoCloud:
      userData: |-
        #cloud-config
        password: fedora
        chpasswd: { expire: False }
        ssh_authorized_keys:
          - ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQCaJEcFDXEK2ZbX0ZLS1EIYFZRbDAcRfuVjpstSc0De8+sV1aiu+dePxdkuDRwqFtCyk6dEZkssjOkBXtri00MECLkir6FcH3kKOJtbJ6vy3uaJc9w1ERo+wyl6SkAh/+JTJkp7QRXj8oylW5E20LsbnA/dIwWzAF51PPwF7A7FtNg9DnwPqMkxFo1Th/buOMKbP5ZA1mmNNtmzbMpMfJATvVyiv3ccsSJKOiyQr6UG+j7sc/7jMVz5Xk34Vd0l8GwcB0334MchHckmqDB142h/NCWTr8oLakDNvkfC1YneAfAO41hDkUbxPtVBG5M/o7P4fxoqiHEX+ZLfRxDtHB53 me@localhost
        bootcmd:
          # mount the secret
          - "mkdir /mnt/app-secret"
          - "mount /dev/$(lsblk --nodeps -no name,serial | grep D23YZ9W6WA5DJ487 | cut -f1 -d' ') /mnt/secret"
  - name: app-secret-disk
    secret:
       secretName: vmi-secret

Environment:

KubeVirt version (use virtctl version): v0.15.0
Kubernetes version (use kubectl version): v1.10.6
VM or VMI specifications:  default/vmi-secret
Cloud provider or hardware configuration: None
OS (e.g. from /etc/os-release): centos7
Kernel (e.g. uname -a): 3.10.0
Install tools:
Others:

additional:
[root@volume-ji:/home/ubuntu/kubevirt/vm/volume/secret]$ kubectl get po | grep vmi-secret
virt-launcher-vmi-secret-lzbng                0/2       Error     0          16m
[root@volume-ji:/home/ubuntu/kubevirt/vm/volume/secret]$ ps -ef | grep vmi-secret
root      6025  5478  0 10:14 pts/5    00:00:00 grep --color=auto vmi-secret
[root@volume-ji:/home/ubuntu/kubevirt/vm/volume/secret]$

Hi @volume-ji, thanks for the detailed report! On a quick look I don't see an obvious reason for this error, neither in your manifests nor in the code. Need to dig deeper into it later on. Will post an update as soon as I know more.

I test the same scene in my other PC, it worked. I just want to make a report about kubevirt, it does not matter. close the issue


How Routing traffic depending on path not domain in docker

I have tried this and its not working
traefik:
image: traefik
command: --web --docker --docker.domain=docker.localhost --logLevel=DEBUG
ports:
- "80:80"
- "8080:8080"
- "443:443"
volumes:
- /var/run/docker.sock:/var/run/docker.sock
- /dev/null:/traefik.toml
machine:
image: katacoda/docker-http-server
labels:
- "traefik.backend=machine-echo"
- "traefik.frontend.rule=Path:machine-echo./machine"
echo:
image: katacoda/docker-http-server:v2
labels:
- "traefik.backend=echo"
- "traefik.frontend.rule=Path:echo-echo./echo"

very curious about this also, to route traffic to different api version ...

What is not working exactly? Have you tried PathPrefix or PathPrefixStrip rules instead?

@emilevauge I have tried using this - "traefik.frontend.rule=Path:echo-echo./echo"  I can't see the documentation of "PathPrefix" or "PathPrefixStrip"

http://docs.traefik.io/basics/#frontends

@emilevauge thanks
Finally got it, it wasn't so obvious from the documentation
traefik:
image: traefik
command: --web --docker --docker.domain=docker.localhost --logLevel=DEBUG
ports:
- "80:80"
- "8080:8080"
volumes:
- /var/run/docker.sock:/var/run/docker.sock
- /dev/null:/traefik.toml
whoami1:
image: emilevauge/whoami
labels:
- "traefik.backend=whoami"
- "traefik.frontend.rule=Host:whoami.docker.localhost"
whoami2:
image: emilevauge/whoami
labels:
- "traefik.backend=whoami2"
- "traefik.frontend.rule=Path:/test2/, /mwas/"

@emilevauge  quick question how does one do regex for path in traefik

Hi Guys,
I am trying to do the same but cannot make it works.
I am trying to have mydomain.dev/pma pointing on the phpmyadmin backend:
web:
    image: nginx
    links:
        - app
    volumes_from:
        - app
    volumes:
        - ./web/nginx.conf:/etc/nginx/nginx.conf
    labels:
        - "traefik.backend=web"
        - "traefik.frontend.rule=Host:mydomain.dev"
        - "traefik.port=80"

phpmyadmin:
    image: phpmyadmin/phpmyadmin
    links:
        - db
    labels:
        - "traefik.backend=phpmyadmin"
        - "traefik.frontend.rule=Host:mydomain.dev;Path:/pma"

Anyone having an idea how to solve this?
Thanks

@romainlaisne I use this label
"traefik.frontend.rule=PathPrefix:/static/"

Hi,
I haven't solved my issue yet, but I think that it is not Traefik causing me trouble in the end.
I use Symfony and the routing system is coming on top of everything which I think causes troubles.
Thanks for you help @mwaaas.

@romainlaisne Did you find a solution for you mydomain.dev/pma problem? I've got the same problem.

@romainlaisne Found the answer, use "traefik.frontend.rule=Host:mydomain.dev;PathPrefixStrip:/pma"

Solution for me:
I tried something similar to:
    [...]
    labels:
        - "traefik.frontend.rule=PathPrefixStrip:/pma"
        - "traefik.frontend.rule=Host:mydomain.dev"
        [...]

But figured out, this does not work. (Maybe line 1 does overwrite line2?)
For me worked:
    [...]
    labels:
        - traefik.frontend.rule=Host:mydomain.dev;PathPrefixStrip:/pma
        [...]

Thanks!  I was looking for path based routing as well and it works like following for me
docker run -dit --name xxx --label "traefik.frontend.rule=PathPrefixStrip:/xxx/" --label traefik.backend=xxx myapp


Click through not working like sarif file generated from credscan

when viewing the results in the scans tab( extension SARIF SAST Scans Tab) using the almost depricated credscan, I can click on the error line and it brings me to the specified line inside the repo.
Can this feature also be implemented in gitleaks?  I saw the format was a little bit different, maybe this is the reason?

Hi @VincentOspazi ,
I like the idea, however I don't think this is possible. The report generation is part of GitLeaks itself and not part of this extension. The only think that this extension does is upload the report to a valid location in Azure DevOps so that the SARIF SAST Scans Tab can pick this up. Basically it's pretty dumb ;)
Making your suggestion work requires that Zachary needs to include the correct URL with  line of code to the web based source control repository.  I think this approach is different depending on the tool. Maybe you can make a suggestion here: https://github.com/zricethezav/gitleaks but again, this is something I cannot control. Sorry for that.


(PUP-10387) Remove puppet module build, generate and search actions

CLA signed by all contributors.

I think we can delete Puppet::ModuleTool::Applications::Searcher and related tests too


Civ 5239 check your answer part2

JIRA link
https://tools.hmcts.net/jira/browse/CIV-5239
Change description
Check your answers Part2- Claim Amount
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

yes here is include part 1


Are there any plans to develop a LB based on CPP language?

As I see, you have now provided LB libraries in four languages. Are there any plans to develop a CPP language LB library?

Good question. We did the languages that we knew people were using. Are you using a C++ DynamoDB driver? Can you help us get started but telling us how you use this driver? Or even better, contribute a load balancer for the C++ load balancer? It should be fairly easy to do - all the other languages took less than a day of work each :-)

Preliminary analysis:
One way of achieving load balancing would be to decorate all request handlers (https://github.com/aws/aws-sdk-cpp/blob/48c3e1c675745b9baf9dde7f280b5e39fd2fb5bd/aws-cpp-sdk-dynamodb/include/aws/dynamodb/DynamoDBClient.h#L361-L3283) with a small wrapper:

Change the URI (m_uri field) by calling the OverrideEndpoint() helper function
Call the original function, which will send to the overridden URI
It would, however, be quite cumbersome, since every single request handler would need to be changed. Instead, it looks like it should be enough if we:
Create a new class, inheriting from DynamoDBClient
Override its BuildHttpRequest function, which thankfully is virtual. Each request goes through this function, so it looks like it can be used to intercept every request and overwrite its target address. If I'm right, overriding this single function should be enough to provide load balancing similar to what other languages have. Of course, there should also be a thread which regularly updates the node list from alternator's /nodelist endpoint.

Preliminary analysis:
One way of achieving load balancing would be to decorate all request handlers (https://github.com/aws/aws-sdk-cpp/blob/48c3e1c675745b9baf9dde7f280b5e39fd2fb5bd/aws-cpp-sdk-dynamodb/include/aws/dynamodb/DynamoDBClient.h#L361-L3283) with a small wrapper:

Change the URI (m_uri field) by calling the OverrideEndpoint() helper function
Call the original function, which will send to the overridden URI
It would, however, be quite cumbersome, since every single request handler would need to be changed. Instead, it looks like it should be enough if we:
Create a new class, inheriting from DynamoDBClient
Override its BuildHttpRequest function, which thankfully is virtual. Each request goes through this function, so it looks like it can be used to intercept every request and overwrite its target address. If I'm right, overriding this single function should be enough to provide load balancing similar to what other languages have. Of course, there should also be a thread which regularly updates the node list from alternator's /nodelist endpoint.

And indeed, here's a "AlternatorClient.h" snippet which successfully intercepts and rewrites URI of a request, tested with official DynamoDB examples from aws-doc-sdk-examples:
#include <aws/core/Aws.h>
#include <aws/dynamodb/DynamoDBClient.h>

class AlternatorClient : public Aws::DynamoDB::DynamoDBClient {
public:
        AlternatorClient(const Aws::Client::ClientConfiguration &clientConfiguration = Aws::Client::ClientConfiguration())
                : Aws::DynamoDB::DynamoDBClient(clientConfiguration) {}
        virtual void BuildHttpRequest(const Aws::AmazonWebServiceRequest &request, const std::shared_ptr< Aws::Http::HttpRequest > &httpRequest) const override {
                httpRequest->GetUri() = Aws::Http::URI("http://localhost:8000");
                std::cout << "URI set to " << httpRequest->GetUri().GetURIString() << std::endl;
                return Aws::DynamoDB::DynamoDBClient::BuildHttpRequest(request, httpRequest);
        }
};

So the matter boils down to implementing a simple fetcher for local nodes and using the list to balance the load.

And indeed, here's a "AlternatorClient.h" snippet which successfully intercepts and rewrites URI of a request, tested with official DynamoDB examples from aws-doc-sdk-examples:
#include <aws/core/Aws.h>
#include <aws/dynamodb/DynamoDBClient.h>

class AlternatorClient : public Aws::DynamoDB::DynamoDBClient {
public:
        AlternatorClient(const Aws::Client::ClientConfiguration &clientConfiguration = Aws::Client::ClientConfiguration())
                : Aws::DynamoDB::DynamoDBClient(clientConfiguration) {}
        virtual void BuildHttpRequest(const Aws::AmazonWebServiceRequest &request, const std::shared_ptr< Aws::Http::HttpRequest > &httpRequest) const override {
                httpRequest->GetUri() = Aws::Http::URI("http://localhost:8000");
                std::cout << "URI set to " << httpRequest->GetUri().GetURIString() << std::endl;
                return Aws::DynamoDB::DynamoDBClient::BuildHttpRequest(request, httpRequest);
        }
};

So the matter boils down to implementing a simple fetcher for local nodes and using the list to balance the load.


Support for TS 5.5 ${configDir} syntax

Hi,
I'm just setting up a new monorepo project based on TS 5.5. It seems eslint-import-resolver-typescript does not support its new syntax.
// This tsconfig.json works
{
    "compilerOptions": {
        "baseUrl": "./src",
        "paths": {
            "#/*": ["./*"]
        }
    },
    "include": ["src/**/*"]
}

// This tsconfig.json does not work
{
    "compilerOptions": {
        "baseUrl": "${configDir}/src",
        "paths": {
            "#/*": ["./*"]
        }
    },
    "include": ["src/**/*"]
}

Specifically in
// MyComponent.tsx
import { AttributeFilter } from "#/components/AttributeFilter";
...

Throws:
Unable to resolve path to module '#/components/AttributeFilter'. eslint (import/no-unresolved)

Using version "eslint-import-resolver-typescript": "^3.6.1".
It also seems this package does not support paths without baseUrl (introduced in TS 4.1)
Is it possible truth? Do you plan to add? May I help somehow?
Thanks!

See also https://github.com/privatenumber/get-tsconfig/issues/81
eslint-import-resolver-typescript's baseUrl and paths support is powered by get-tsconfig. I will bump the version of get-tsconfig ASAP once it is implemented and released.


Bug: Ekko goes invisible after he uses ult

Hi guys, when i set the ekko skin to default (i can't tell any of these new skins so i prefer to play vs default skin)
So when i set the ekko skin to default, it all works good until ekko uses ult, after that he goes invisible for the rest of the game (unless i change the skin)
just a small bug i thought you should know.

I'm using the latest version that was uploaded 4 days ago, v3.0.9
But i noticed this issue a couple of weeks ago, so I don't think it has anything to do with the current version.


Set cmdwintype or some other alternative

As suggested in the README, one of the most useful things this plugin provides is the ability to filter out the lines of the command window. However, right now you cannot tell which type of command window is open. I want to, for example, filter out the :q and :wq commands in a "vim/cmd" window, but filter out something else entirely in a "lua/cmd" window.
It would be nice to add a feature for this; some sort of argument to the autocmd, perhaps, or a buffer-local variable...

Thanks!
Added get_context().type.


[Feature] does bmtrain support torch 2.0+

Is your feature request related to a problem? Please describe.
torch 2.0 is stable
Describe the solution you'd like
torch 2.0 is stable
Describe alternatives you've considered
No response
Additional context
No response

Supported since BMTrain 0.2.3


AttributeError: 'Meteor' object has no attribute 'lock'

when trying to training from scratch, one iteration is finished, and it need to compute scores, and some issues occur as follows:
Traceback (most recent call last):
File "D:/AIBU/image_caption/showAttendTell/train.py", line 26, in
main()
File "D:/AIBU/image_caption/showAttendTell/train.py", line 22, in main
solver.train()
File "D:\AIBU\image_caption\showAttendTell\core\solver.py", line 172, in train
scores = evaluate(data_path='./data', split='val', get_scores=True)
File "D:\AIBU\image_caption\showAttendTell\core\bleu.py", line 47, in evaluate
final_scores = score(ref, hypo)
File "D:\AIBU\image_caption\showAttendTell\core\bleu.py", line 14, in score
(Meteor(),"METEOR"),
File "D:\AIBU\image_caption\showAttendTell\pycocoevalcap\meteor\meteor.py", line 24, in init
stderr=subprocess.PIPE)
File "D:\software\Anaconda\lib\subprocess.py", line 709, in init
restore_signals, start_new_session)
File "D:\software\Anaconda\lib\subprocess.py", line 997, in _execute_child
startupinfo)
FileNotFoundError: [WinError 2] 系统找不到指定的文件。
Exception ignored in: <bound method Meteor.del of <pycocoevalcap.meteor.meteor.Meteor object at 0x000001E3288BC6D8>>
Traceback (most recent call last):
File "D:\AIBU\image_caption\showAttendTell\pycocoevalcap\meteor\meteor.py", line 76, in del
self.lock.acquire()
AttributeError: 'Meteor' object has no attribute 'lock'
How to deal with it? Thank you!

I installed java8, and add shell=True in subprocess.Popen() function, then this problem is solved.


Update filterwarnings with new lib2to3 deprecation

While this was originally a PendingDeprecationWarning, it is now a DeprecationWarning and as such the filter needs to be updated.
I made it look like the entry used in colcon-core, which didn't need to be updated after the upstream class changed. If you'd instead like to update the class to match the pattern used in this package, I'm happy to change it.

I made it look like the entry used in colcon-core, which didn't need to be updated after the upstream class changed. If you'd instead like to update the class to match the pattern used in this package, I'm happy to change it.

I'm good with the consistency if that makes maintabilty simpler.


Small fixes

Some small fixes to take care of errors in the Vagrant build.

Thanks for the fixes !


Event Filter eventName & Event Data not working

Describe the Bug
When post event data like this  umami.track('signup-button', { name: 'newsletter', id: 123 }); umami returns 500 error
umami > PrismaClientKnownRequestError:                             │
│                            ││ umami > Invalid `prisma.eventData.createMany()` invocation:        │
│                            ││ umami > Null constraint violation on the fields: (`event_id`)      │
│                            ││ umami >     at zr.handleRequestError (/home/bluedot/web/alberta.b  │
│                            ││ umami >     at zr.handleAndLogRequestError (/home/bluedot/web/alb  │
│                            ││ umami >     at zr.request (/home/bluedot/web/alberta.bluedot.ltd/  │
│                            ││ umami >     at relationalQuery (/home/bluedot/web/alberta.bluedot  │
│                            ││ umami >     at __WEBPACK_DEFAULT_EXPORT__ (/home/bluedot/web/albe  │
│                            ││ umami >   code: 'P2011',                                           │
│                            ││ umami >   clientVersion: '5.0.0',                                  │
│                            ││ umami >   meta: { constraint: [ 'event_id' ] }                     │
│                            ││ umami > }

In event API eventName filter is not working, it returns all events in given time.
Database
MySQL
Relevant log output
No response
Which browser are you using? (if relevant)
No response
How are you deploying your application? (if relevant)
No response

After reviewing the source code and database found that if I use event_name it works.

What version are you using? I can't replicate this.

What version are you using? I can't replicate this.

It looks like they are on a super old version:

@saaiful Could you try updating to the latest version of Umami?

I'm using version 2.5.0, maybe this pull request has code from my old pull request.

What version are you using? I can't replicate this.

I'm using 2.5.0, basic event works without issue, but if I try to send any data, I get 500 error.
http://localhost:3000/api/websites/****/event?timezone=Asia/Dhaka&eventName=pr0m0-imp-25 this API endpint returns all event, it's eventName does not work, so logged what query is being used and found eventName is not allowed or listed in lib/constants.ts's FILTER_COLUMNS  object.

This is the error I'm getting in local and production server while sending data with event.
Invalid `prisma.eventData.createMany()` invocation:

Unique constraint failed on the constraint: `PRIMARY`
    at zr.handleRequestError (D:\wamp64\www\umami2\node_modules\@prisma\client\runtime\library.js:122:8308)
    at zr.handleAndLogRequestError (D:\wamp64\www\umami2\node_modules\@prisma\client\runtime\library.js:122:7697)
    at zr.request (D:\wamp64\www\umami2\node_modules\@prisma\client\runtime\library.js:122:7307)
    at async relationalQuery (webpack-internal:///(api)/./queries/analytics/events/saveEvent.ts:45:9)
    at async __WEBPACK_DEFAULT_EXPORT__ (webpack-internal:///(api)/./pages/api/send.ts:65:9) {
  digest: undefined
}

This is the error I'm getting in local and production server while sending data with event. I'm using MySQL database.
Invalid `prisma.eventData.createMany()` invocation:

Unique constraint failed on the constraint: `PRIMARY`
    at zr.handleRequestError (D:\wamp64\www\umami2\node_modules\@prisma\client\runtime\library.js:122:8308)
    at zr.handleAndLogRequestError (D:\wamp64\www\umami2\node_modules\@prisma\client\runtime\library.js:122:7697)
    at zr.request (D:\wamp64\www\umami2\node_modules\@prisma\client\runtime\library.js:122:7307)
    at async relationalQuery (webpack-internal:///(api)/./queries/analytics/events/saveEvent.ts:45:9)
    at async __WEBPACK_DEFAULT_EXPORT__ (webpack-internal:///(api)/./pages/api/send.ts:65:9) {
  digest: undefined
}

My database had some column missing, set database default engine to InnoDB, Now event with data is working. But the filter is not working.


Would you please include a code example in C#?

[Would you please include a code example in C#?
Thank you!]

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

ID: 58442835-d11e-2643-2de2-019934bd6ef8
Version Independent ID: 87390f29-835d-078b-4e8a-3f0ef5937e0f
Content: ForecastingModel.AutoArima Property (Azure.ResourceManager.MachineLearning.Models) - Azure for .NET Developers
Content Source: xml/Azure.ResourceManager.MachineLearning.Models/ForecastingModel.xml
Service: azure
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

This request is tracked in #35292.   Closing this as a duplicate.


Cleaner ReleaseType parsing from project properties

Motivation
This makes ReleaseType class accessible in build configuration, so setup like this is possible:
gradle.properties
# ...
release_type=stable

build.gradle
import me.modmuss50.mpp.ReleaseType
// ...
publishMods {
    type = ReleaseType.of(project.release_type)
    // ...
}

Current behaviour
Currently this can be replicated this way:
build.gradle
import me.modmuss50.mpp.PublishOptions
// ...
publishMods {
    type = PublishOptions.ReleaseType.valueOf(project.release_type.toUpperCase())
    // ...
}

This approach includes more boilerplate and has a less descriptive error message.

Looks good, make sure to run spotlessKotlin and spotlessKotlinApply Gradle tasks so the build passes 👍


Add ReadSnapshots2 API and mark ReadSnapshots as deprecated

According to the document(https://developers.mixin.one/docs/api/transfer/snapshots#get-snapshots), ReadSnapshots API is missing some optional parameters(opponent, destination and tag). I added ReadSnapshots2 to fill in the missing params and marked ReadSnapshots as out of date.

@yiplee Updated.