Conditional triggering updates when using patch

Thank you for your awesome library!
I stubled across the use case, that I want to debounce updates to the storage such that on_update is not fired if the component is not updated through patch, or a specified condition is not satisfied.
This has the benefit over get (and update) followed by an eventual patch that we dont have to double look-up the storage and entity. Something like the following snippet where we return true or false from the callback that updates the component to indicate a change:
registry.patch<MyComponent>(id, [](MyComponent& comp) {
  return true; // Or false if the component was not updated, then the on_update event should not be fired.
});

A more expensive workaround implementation of the behaviour described above would be something like this:
template <typename Component, typename Callback>
void maybe_patch(entt::registry& reg, entt::entity id, Callback&& callback) {
  Component& comp = reg.get<Component>(id);

  if (std::forward<Callback>(callback)(comp)) {
    reg.patch<Component>(id, [](Component&) {});
  }
}

Sadly it is not possible to implement the optimized solution because sigh_storage_mixin<basic_storage<Entity, Type>> only exposes the sigh as an attachable sink without making it possible to trigger the signal manually (the signal itself is private).
Making it possible to trigger the on_update signal would be advantageous for other use-cases as well where we inform the registry through an update made via mutable view or group iteration.
Do you know of any other possibility to implement this behaviour in a straight forward way?

I my experience using an empty patch reg.patch(e) is sufficiently performant, because the callback should get optimized away...
If you want to know about more optimized data structures of tracking change, I suggest you read https://ourmachinery.com/post/syncing-a-data-oriented-ecs/ , it has some examples.

Thank you for your quick response, calling patch with no callback is probably the intended way of solving my issue.
The article seems really interesting and I will definitly get deeper into it.


[Feature Request] Date Range picker

Would be great to have a date range component similar in functionality to the one here: http://www.daterangepicker.com/

@piehouserat Hey, this is something that we'd recommend to be developed in userland (in an app or an add-on library for material-ui). Something like this isn't defined in the spec and is definitely impacted by case-by-case requirements.
Generally we try to keep our components down to the building blocks you need to put such a widget together, rather than providing these complex widgets out of the box.

Ref #5082


Rewrite the micromamba Feature

Hi @eitsupi, I'm really sorry it took me so long to review this.
Could you please take a look at my comments when you get the chance? (I made them inline in the script.)
Thanks so much for your patience.

@eitsupi, what do you think of the changes I'm proposing here? Do they conform with the standard? Are they a good idea? Thanks!

Also, could you do package installs using micromamba in the tests?

Yes, I'll start working on this.

And, could you increment the version number to 0.1.0?

I could, but I'm confused about the release process. How does one know which commit corresponds to a release? I feel like we should create a tag in the repository. Would it make sense to postpone the version increment until we are ready for release? Then on the final commit, we increment the number, tag it, and execute the build workflow? Or is there a better way?

I added tests for the addCondaForge option and a test for installing the python package via micromamba. I look forward to your responses regarding the remaining open points.
Regarding the soft version matching, I opened https://github.com/mamba-org/mamba/issues/2164. I think it would be easy to implement it there, if they make it open.

I could, but I'm confused about the release process. How does one know which commit corresponds to a release? I feel like we should create a tag in the repository. Would it make sense to postpone the version increment until we are ready for release? Then on the final commit, we increment the number, tag it, and execute the build workflow? Or is there a better way?

Good point. Unfortunately, at the moment there is no way to do it!
See devcontainers/features#113.
As far as I know, there are currently no Features that be managed releases by tags, and once devcontainers/action#112 is merged and GitHub Actions does this, management by tags will become commonplace.......
If this is wanted now, the maintainer can manually push the tags after the release, since the release workflow is executed manually.
https://github.com/eitsupi/mamba-devcontainer-features/blob/12349ea7a31cba62e9a7b28392e51a0b4acc688d/.github/workflows/release.yaml#L2-L3

once https://github.com/devcontainers/action/pull/112 is merged and GitHub Actions does this, management by tags will become commonplace

That looks great! Can we anticipate the format of the tags? From this line
    const tag = `${type}_${id}_v${version}`;

it's not clear to me what are type and id.

That looks great! Can we anticipate the format of the tags? From this line
    const tag = `${type}_${id}_v${version}`;

it's not clear to me what are type and id.

Based on the following line, it would be something like feature_micromamba_v0.1.0.
https://github.com/devcontainers/action/blob/09709bc562da6615f791c204928d1415294c647e/src/main.ts#L66
However, this is not finalized, so it is better to use this for now and push the tag again with the new format if the format changes.

When you're satisfied with all the changes, could we merge this PR and then do the release in a separate commit?

I think your excellent insight and lots of work made this Feature great!

Yes, thank you so much for your patience and for teaching me so many things! I have learned a lot from you in this PR.

Thanks for the PR this has been really great, so let me know if you want to transfer to this repo to mamba-org.

transfer to this repo to mamba-org

I think this is a good idea, so I will propose this on Gitter in mamba-org. But in order that people can evaluate it, could you make a package under this repo?

Got it! Shall I merge and release right now?

I have proposed this in Gitter: https://gitter.im/mamba-org/Lobby?at=6399afca8bdea01368b61508


DNS resolution fails because of having a space appended to the service link names.

Rancher compose version - V0.1.1
docker-compose.yml used:
sangeethahariharan1@sangee01:~$ cat dns-rancher-compose.yml
client:
  scale: 1
web1:
  scale: 3
web2:
  scale: 3
sangeethahariharan1@sangee01:~$ cat dns-docker-compose.yml
client:
  restart: 'no'
  tty: true
  image: sangeetha/testclient
  links:
  - "dns123: mydns"
  stdin_open: true
web1:
  tty: true
  image: sangeetha/testlbsd
  stdin_open: true
web2:
  tty: true
  image: sangeetha/testlbsd
  stdin_open: true
dns123:
  image: rancher/dns-service
  links:
  - "web1: web1"
  - "web2: web2"

sangeethahariharan1@sangee01:~
The link name is stored as " mydns"  and  dns resolution for "mydns" from the containers in client service  fails.
mysql> select name from service_consume_map where name is not null;
+---------+
| name    |
+---------+
|  mydns  |
|  web1   |
|  web2   |
/var/lib/cattle/etc/cattle/dns/answers.json
"<IP_ADDRESS>": {
"client.": ["<IP_ADDRESS>"],
"client.rancher.internal.": ["<IP_ADDRESS>"],
" mydns.": ["<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>"],
" mydns.rancher.internal.": ["<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>", "<IP_ADDRESS>"]
},

Tested with Rancher compose version - V0.1.2
Using the below generated rancher and docker compose yml from rancher server for a environment that had "Service Alias" , I was able to successfully create a "Service Alias" using rancher compose CLI. DNS resolution on the dns name - "mydns1" resolves correctly.
sangeethahariharan1@sangee01:~$ cat dns-rc.yml
testweb1:
scale: 3
testclient:
scale: 2
testweb2:
scale: 3
sangeethahariharan1@sangee01:~$ cat dns-dc.yml
mydnsrancher:
image: rancher/dns-service
links:
testweb1: testweb1
testweb2: testweb2
testweb1:
tty: true
image: sangeetha/testlbsd
stdin_open: true
testclient:
tty: true
image: sangeetha/testclient
links:
mydnsrancher: mydns1
stdin_open: true
testweb2:
tty: true
image: sangeetha/testlbsd
stdin_open: true
sangeethahariharan1@sangee01:~$


Esify should not delete sprocket directives

Manifest files are being ignored by Esify.
Steps to reproduce:

esify app/assets/javascripts/admin/admin_vendors.coffee
See app/assets/javascripts/admin/admin_vendors.js

admin_vendors.js is empty because admin_vendors.coffee is commented. Decaf treats it as an empty file and thus isn't compiling it.
Main culprits

app/assets/javascripts/admin/admin_vendors.coffee
app/assets/javascripts/admin/merchant_checkout.coffee

cc @lemonmade

Good catch. I've manually converted these files in this PR: https://github.com/Shopify/shopify/pull/71460, so I think that should resolve this issue.

@fandy this is unfortunately a limitation of decafjs https://github.com/juliankrispel/decaf/issues/1


Optional removal of self-dependence for recurring jobs

I have a recurring job whose function can sometimes take longer to finish than the actual period of recurrence. The subsequent call of function would be independent of the previous one, however due to the implicit dependence, it's not started until the previous one is finished. Would it be possible to add some option that removes this dependence?
As an MWE, this is what I have now (the effective function call period becomes 6 seconds):
julia> using JobSchedulers
julia> function f()  # function runs within 4 seconds
           sleep(4)
           println(Dates.format(now(), "HH:MM:SS"))
       end;
julia> submit!(f, cron=Cron("*/2", "*", "*", "*", "*", "*"));  # function call at every 2 second, starting at 15:19:16
15:19:20
15:19:26
15:19:32
15:19:38
...

What would be great to have is something like this (the function call is 2 seconds as specified by cron):
 julia> submit!(f, cron=Cron("*/2", "*", "*", "*", "*", "*"), remove_self_dependence=true);  # function call at every 2 second, starting at 15:19:16
15:19:20
15:19:22
15:19:24
15:19:26
...

I mean the subsequent data wouldn't be even attempted to be obtained. Let's say that each time my_scraping_function is called it tries to download the data of the current time. Now if the function is called at 9:35 but there is a delay and that particular data is finally published at 9:42, then we would miss the function call (and so obtaining the data) at 9:40, and the next call will attempt to get data only at 9:45.

If I were you, I probably will obtain all data I can get through a single request, and add new data to my existing database. The time they publish does not matter, right?

Well, actually it does: besides the actual data, the (approximate and potentially delayed) publish date would also be recorded.

So the problem solved! Have a good one!


Add information on port to the connection error message

Or default to 5444. Users may be confused when they just enter https://machine since we default to 443.

If they explicitly type "https://", the schema implies the port number to be 443 if it's not specified - if we want to change that, we should change the schema.
Alternatively, we can prohibit omitting the port number - i.e. if they type "https://machine", we autocorrect it to include the port, just like we handle schemaless names already. That way it's never implicit. And autocorrection doesn't have to follow the rules for https schema, it can insert whatever port number we deem appropriate.

We already autocorrect "https://machine" to "https://machine:443":
[InlineData("https://host", "https://host:443")]
public void CompletePath(string original, string expected)

As for the bug, I can't find an error message that contains connection path. What are exact repro steps?

We already auto-correct


Put a testnet on scaleway

Create a 2-5 node testnet on scaleway so that other users can join.
Failing this, create a 1-node testnet on Scaleway, since that's even easier.

@faddat can i close this issue out?

Yup.


Feature Request: allow to use storage

Using storage (selecting StorageClass to create PV) would allow to provision statefull apps via Rig dev. (similarly to Kubero as it has such feature.

With the plugins, any resources can now be created - including replacing the "deployment" step with e.g. a StatefulSet varient.


Coherent approach to running the examples

I'm not sure running say number-insight from a webpage via the browser makes much sense. Maybe a single command-line endpoint for running the samples?
Needs some thought.

The counter-argument to the command-line is that we should use web apps since it's likely that's the context in which our libraries are mostly used.
In general, I think we should just be consistent for all examples.


Issue 1835 handle auto-restore on twice

Fix for running auto-restore a second time on a project.
Integration tests for one and two runs.

cool. thanks


retrieve UTRs without coding sequence

We've had a user ask about retrieving just the UTR sequences for a gene. They've said the colour highlighting (#541) was useful for this but maybe we should also offer the option of de-selecting the coding sequence if one UTR option is selected, so they just retrieve UTR sequence in the first place.
Is that (a) technically straightforward and (b) useful enough to do?

technically straightforward

Yep.  That's what is did initially so it would be an easy change.

I think the colouring is  more generally useful so I'll make the colouring the HP ticket and this one
can happen whenever...

This is low priority with the colouring, but can be done whenever  if quick

This is low priority with the colouring, but can be done whenever if quick

It's quick if you don't mind an extra checkbox in the sequence section:
 [X] include exons

(Toggled on by default)

I've made that change locally but I've just noticed that it works poorly with RNA genes (you can't add up or down-stream bases).  I'll check in the change once I fix that.
By the way, for protein coding genes should I change the label on the checkbox to?:
[X] include coding exons

That seems more accurate since there are UTR exons?
I've also noticed that you can't get the sequence of a pseudogene at all and you can't in the main site either:
https://www.pombase.org/gene/SPCC576.16c
It says:
This gene has no transcripts available

No one has complained yet but I'd like to fix it.

No one has complained yet but I'd like to fix it.

I wouldn't worry about this yet. Some of the "predictions" for pseudo are a bit fanciful so the transcript may not exist, or be meaningless.
It might be better to provide nothing...
At least until https://github.com/pombase/curation/issues/21

I've fixed the problem that was holding things up: you couldn't get the up and down stream sequence of RNA genes.  So the "Include exon" checkbox will be back tomorrow and so you'll be able to select just the UTR sequences again.

So the "Include exon" checkbox will be back tomorrow

It's actually there now.  I updated the site we new code and last night's data.


Switch react-select for material-ui autocomplete

This lets us remove react-select library from the bundle, and unifies the look and feel of dropdowns.

@fedde-s just FYI, this changed your FilteringOtTableRF component slightly. Specifically, the options array no longer contains objects with label and value fields, just values.

From a brief look at the code I see no problem with that, as (unless I'm misreading) that options array was just an internal implementation detail of FilteringOtTableRF to match the API of the library that it was using before, and not one of FilteringOtTableRF's props. Thanks for the heads-up though!

This is breaking ot-ui, I'm pulling it until I can solve the problem.

This PR removes all dependencies to react-select from the code, but it is still required to build ot-ui as a peer dependency. To remove the library, its dependency has to be removed from ot-ui also.


Support ReactiveCocoa with Swift 2.0

Right now Moya compiles and works on its swift-2.0 branch including RxSwift but ReactiveCocoa is lagging behind due to a non-official podspec.
We need to support ReactiveCocoa as a dependency working with Swift 2.0

Curious what's involved in this? Seems like it's the last piece in the puzzle in getting Moya working under Swift 2/Xcode 7..
Great work, btw!

Sadly ReactiveCocoa dropped support for CocoaPods some versions ago.
This means that the podspec is not maintained by them and there's currently no Swift 2.0 podspec.
I forked ReactiveCocoa and added a podspec to their 2.0 branch but I'm having some issues when trying to compile the pod.
TL;DR
I need to figure out a correct podspec for Swift 2.0 :smile:

I maintain the podspec for ReactiveCocoa; is it out of date?

I think is up to date but if I try to point the pod to the Swift 2.0 branch it fails telling me there's no pod at that location or something along those lines (can't remember the exact error right now).

Hmm. You should be

Fetching podspec for `Moya` from `../`
[!] Unknown external source parameters for `ReactiveCocoa`: `{:branch=>"swift2"}`

Let me check because I just changed something and I might be writing the Podfile wrong

What's the podfile look like?

Ok; here's the Podfile:
platform :ios, '8.0'

source 'https://github.com/CocoaPods/Specs.git'

use_frameworks!

pod 'ReactiveCocoa', :git => 'https://github.com/ReactiveCocoa/ReactiveCocoa', :branch => "swift2"
pod 'RxSwift', :git => 'https://github.com/ReactiveX/RxSwift', :branch => 'rxswift-2.0'
pod 'Alamofire', :git => "https://github.com/Alamofire/Alamofire", :branch => "swift-2.0"

pod 'Moya/RxSwift', :path => "../"
pod 'Moya/ReactiveCocoa', :path => "../"
pod 'Moya', :path => "../"

target 'DemoTests' do

  pod 'Quick', :git => 'https://github.com/Quick/Quick', :branch => 'swift-2.0'
  pod 'Nimble', :git => 'https://github.com/Quick/Nimble', :branch => 'swift-2.0'
  pod 'OHHTTPStubs'

end

And here's the error:
Pre-downloading: `ReactiveCocoa` from `https://github.com/ReactiveCocoa/ReactiveCocoa`, branch `swift2`
[!] Unable to find a specification for 'ReactiveCocoa'.

Tried pointing to your fork just in case; this is what I'm getting:
Pre-downloading: `ReactiveCocoa` from `https://github.com/ashfurrow/ReactiveCocoa`, branch `swift2`

[!] Error installing ReactiveCocoa
[!] /usr/local/bin/git clone https://github.com/ashfurrow/ReactiveCocoa /var/folders/xk/378q_x1x4b3289cbr_kjyhzr0000gn/T/d20150821-25846-1w2akmk --single-branch --depth 1 --branch swift2

Cloning into '/var/folders/xk/378q_x1x4b3289cbr_kjyhzr0000gn/T/d20150821-25846-1w2akmk'...
warning: Could not find remote branch swift2 to clone.
fatal: Remote branch swift2 not found in upstream origin

My fork is way out of date.
Interesting – I would've thought it would use the spec in trunk. You should be able to point to your work and copy&paste this podspec into it: https://github.com/CocoaPods/Specs/blob/master/Specs/ReactiveCocoa/3.0-RC.1/ReactiveCocoa.podspec.json

I'm using this: https://github.com/esttorhe/ReactiveCocoa/blob/swift2/ReactiveCocoa.podspec
I'll take a look at the one you provided and see what differs from mine

I'm spending time this week to update our app to Swift 2 – I'll try to get this working.

Initial work is complete. I haven't finished yet, so I can't test, but this should help anyone currently stuck. I'm using my fork of ReactiveCocoa, and added a podspec to my branch.

I'ved pushed a version of ReactiveCocoa 3 with Swift 2 (Xcode 7 GM) support. Its version number is "3.0.0-swift2" and can be used in Podfiles and in podspecs.
Subsequently, I've updated the Moya podspec to use this new ReactiveCocoa version. There's a problem with the Carthage compatibility, but I'm hoping to have it wrapped up soon, and then I'll submit a PR.

OK, now all that's required for ReactiveCocoa extensions is the following in your podfile
pod 'Moya/ReactiveCocoa'

And that's it! :tada:


is the project still actively maintained?

Hello,
This is a very nice project and an initiative wrapper around the testrail api. We do utilise it in my current organisation and we think it is awesome. based on the issues and PR activity, it is starting to feel that the project is not active. Can you kindly clarify this point?
thanks in advance,

@AbdelrhmanHamouda Hello, I think this project is dead and no longer maintained. You can take part or use a new, actively developing project: TestRail Java API Client

thanks @Zahorodnii for the nice pointer,  will check it out for sure!

Hi @AbdelrhmanHamouda, have you had the opportunity to look at the TestRail Java API Client project? It would be great if you could share your feedback, thanks)

Hello,
Unfortunately no!
I did check the project but didn't get a chance to play with it.
The library is part of a wide spread used internal tooling in the organization so changing things require planning.

Hi @AbdelrhmanHamouda : Maintainer did not change repository for a year, you can use it 'as is'. Some of the forks have addressed shortfalls of the existing code , you can use & cherry pick some of the developments to satisfy your requirements.

Hello, @onurozardic-pointr. I think it would be more promising to use a TestRail Java API Client. This is not a fork, this is a completely new library with a different architecture and a more convenient interface.


Update font-charis-sil to 5.000

After making all changes to the cask:

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

Merged as 6a7c29e721b584027443c1224754ba5618171846.


vs code reveretd when SAVED file!!! Lost Full day of important work!

VSCode Version: 1.29.1
OS Version: MacOs Mojave

@nityansuman how exactly did that happen and is it reproducible?

@nityansuman is it possible that an external tool changed the file? Was the file dirty?

No not possible. The file was totally fine. It was running with correct output. I didn't make any changes after saturday evening and then when opened sunday night, it got reverted back after it got hanged and it restarted.

So to recap:

the file was not dirty
the file was opened in VSCode (visible)
the file suddenly turned into its previous contents
Code crashed and restart?

Yes code crashed and restarted. And code was open in vs code. When i opened i could only see the reverted file on sunday night, when the last time i saw was on saturday evening and it was working fine. No changes made after that.

Ok, please report back if you found steps to reproduce. Closing until then.


created shadowrun2e sheet

The styling for your new sheet is broken, the sheet is still usable and new so it won't mess up any games it progress, so I'm still going to add it to the approved list. You'll want to get your fixes done soon though so it will make it on the list in time for next week's pull.


The "HelloworldXamarin" demo link error too when building with Xamarin.iOS

/Users/liuzhen/Downloads/grpc-master/examples/csharp/HelloworldX

amarin/HelloworldXamarin.iOS/MTOUCH: Error MT5213: Duplicate symbol in: /Users/liuzhen/.nuget/packages/grpc.core.xamarin/2.44.0/build/Xamarin.iOS10/../../native/ios/universal/libgrpc.a(status.upb.o) (Location related to previous error) (MT5213) (HelloworldXamarin.iOS)

maybe not set gcc linkerFlags?

I'm going to close issue this since we're not going to pursuit Unity/Xamarin support in Grpc.Core anymore.

The Grpc.Core implementation is now in maintenance mode and we're only going to focus on critical and security fixes.
Supporting Unity/Xamarin simply presents too much maintenance overhead and it's also difficult to test with the test infrastructure available to us
The Unity/Xamarin support has always been classified as "experimental", so discontinuing the support shouldn't be too surprising.

See the recent update on gRPC C# for more details.
Note that in practice, running Grpc.Core on Unity/Xamarin might still be possible (there is nothing in principle that would prevent it from working), but it will be completely at your own risk and it's simply not one of the use cases the grpc team will help you with as Grpc.Core on Unity/Xamarin is now considered as "not supported" by us (but you might still be able to get help from the community).

What does this mean? Is there any ready-made plan? Your example is unavailable.

Hey @liuzhenLz
Do you find a solution ?


Add method of adding to preload

There is currently no way to add code to the preload without creating a plugin. Changing the preloader.js file works, but the changes are overwritten on every cap sync.
It would be nice to be able to add code, such as contextBridge.exposeInMainWorld calls, to the preloader without the large boilerplate associated with making a plugin.

As of 1.3.4, a file user-preload-script.js is located in the electron directory, this will get added into the regular preload. Version 3 will also make this easier on users


TypeError: init() got an unexpected keyword argument 'qk_scale'

I encountered the following error while running the program you provided. Can you help me resolve it.
TypeError: init() got an unexpected keyword argument 'qk_scale'

can you check your timm version? The required timm version is timm==0.3.2. This repo is based on timm==0.3.2(https://github.com/rwightman/pytorch-image-models), for which a fix (https://github.com/rwightman/pytorch-image-models/issues/420#issuecomment-776459842) is needed to work with PyTorch 1.8.1+.

I encountered the following error while running the program you provided. Can you help me resolve it. TypeError: init() got an unexpected keyword argument 'qk_scale'

above

Thank you for your help


aks update with "--pod-cidr" argument to a existing cluster still return with "Succeeded"

Describe the bug
There should be some warning or error notice to avoid the false idea of az aks update allowing --pod-cidr post cluster creation or after CNI upgrade[0].
Related command
az aks update
Errors
No error. (becasue it should provide some warnings or notices instead.)
Issue script & Debug output
For example, running az aks update  against a cluster that is already using CNI in overlay mode, the operation will still return as ok;
The command ran as : az aks update --name on-test --resource-group ae-test --network-plugin-mode overlay --pod-cidr <IP_ADDRESS>/28  -o yamlc
And the podCidr won't change as it should. Here is the partial output:
  networkPlugin: azure
  networkPluginMode: overlay
  networkPolicy: null
  outboundType: loadBalancer
  podCidr: <IP_ADDRESS>/16
  podCidrs:
  - <IP_ADDRESS>/16
  serviceCidr: <IP_ADDRESS>/16
  serviceCidrs:
  - <IP_ADDRESS>/16

Expected behavior
Given the --pod-cidr argument is assigned, it should be hinted the parameter won't pass through.
But the return of operation is stated as success in green(debug).
This might give a false expectation that POD CIDR is updatable.
Environment Summary
az --version
azure-cli                         2.57.0

core                              2.57.0
telemetry                          1.1.0

Extensions:
aks-preview                      0.5.156
azure-firewall                    0.14.8
bastion                            0.2.5
costmanagement                     0.2.1
fleet                              0.2.2
interactive                        0.5.1
k8s-extension                      1.5.1
resource-graph                     2.1.0
ssh                                2.0.1

Dependencies:
msal                              1.26.0
azure-mgmt-resource             23.1.0b2

Python location '/home/user/.asdf/installs/azure-cli/2.57.0/bin/venv/bin/python3'
Extensions directory '/home/user/.azure/cliextensions'

Python (Linux) 3.11.3 (main, May  5 2023, 11:38:52) [GCC 11.3.0]

Legal docs and information: aka.ms/AzureCliLegal

Your CLI is up-to-date.

Additional context
References:

[0] Upgrade CNI mode: https://learn.microsoft.com/en-us/azure/aks/azure-cni-overlay?tabs=kubectl#azure-cni-cluster-upgrade

Thank you for opening this issue, we will look into it.


[v2] Page callbacks don't work (at least for pages already in dom)

Bug, Version: v2 beta 11
For page-callbacks to work I have to change code in framework7
Router.prototype.pageCallback = function pageCallback (callback, pageEl, navbarEl, from, to, options, pageFromEl) {
...
From
var on = options.on; if ( on === void 0 ) on = {};
to
var on = route.route.on; if ( on === void 0 ) on = {};
Also the "options.pageEvents" references should be "route.route.pageEvents".
I don't know, if this is the right place for a fix, or if there is a general structural problem elsewhere.
The options-parameter in my case is, it is the page in dom on first page:
{
"route": {
"query": {},
"params": {},
"url": "/",
"path": "/",
"route": {
"path": "/",
"name": "index",
"url": "index.html",
"on": {"pageInit": ...}
}
}
}

This is still not working in beta and i'm not sure yet about implementation and correct place for this, but is on my roadmap. At the moment it can be used as is:

within page components' on
using usual app callback: app.on('pageInit', ...)
using DOM event: $(document).on('page:init', ...)

Fixed in latest beta


Bug Fixes

Fixed two bugs:

Joke API not working
Changed description and example text for Location API

Dev deployed at - http://team01-theanimated01-dev.dokku-07.cs.ucsb.edu/swagger-ui/index.html

I didn't change the text for Joke API. It was for the Location API which had Berlin as its example instead of Isla Vista


Pull me mom

Refactoring // fixes

NGL, I kinda prefer the older style where it's over multiple lines. It's about the same file size but it's much easier to read but now it's harder to figure out if there is a spelling mistake

imma merge it,  but after i need to get a bulk json formatter, because i dont feel like going through every file and formatting it 1 by 1 for all the jigsaw bits


fix: sp1 is faster when including recommended compilation flags

Fixes #1.

@jtguibas Having build issues with these flags https://github.com/succinctlabs/sp1/issues/1644 . Could you, please, take a look?

Responded in https://github.com/succinctlabs/sp1/issues/1644.


Cannot deploy: symfony_cache_path undefined

I was trying to use capistrano symfony, therefore I followed your docs here on github but when I try to deploy, I receive the following error:
"NameError: undefined local variable or method 'symfony_cache_path' for #SSHKIT:Backend::Printer:0x2f8b360.
This error occurs within symfony.rake, when trying to run the task "symfony:create_cache_dir".
I inspected the file "symfony.rake" and could not find from where the variable "symfony_cache_path" should be filled.
Did I miss something? I think, that I did not understand well on how to use a git repository within capistrano symfony.
I am developing on a dev machine lets say with ip .67, and the code is being versionised on another virtual machine with ip .68 by using Git and Bitbucket. In deploy.rb, I tried to connect to the git repo. In development, I am using an ssh key to push changes to the bitbucket - so I thought when using the same user from the dev machine in the deploy.rb file, I should also be able to connec to git repo:
set :repo_url, 'ssh://git@xxx.xxx.xx.68:7999/nav/myrepo.git'
set :ssh_user, 'jenkins'
server 'http://xxx.xxx.xx.68', user: fetch(:ssh_user), roles: %w{web app db}
When running this, I get the error described above..

Hi there,
I have got the same error... :(

Hello
I encounter the same bug
my deployment is successful without capistrano/symfony
as soon as I add capistrano/symfony, it no longer works
I use gitlab and symfony3 and everything is on ubuntu servers

It looks like this is a duplicate of #63.
Also, did anyone find a workaround?

Closing this due inactivity. Feel free to ping me if this is still an issue.


Able to publish on maven central

see https://maven.apache.org/guides/mini/guide-central-repository-upload.html

see https://support.sonatype.com/entries/24431666


Version 2 BGD 2015 2016 and V01 2022

Dear Mario,
I have updated the CSD with a migration page as we discussed and also de GLD files are updated in the branch. Please let me know if this ok.
I look forward to comments,
Alexandra

Hi Mario,
See the new changes.
Best regards,
Alexandra


Added runtime check for supported bar code types

During my testing, I could not can data matrices. So I set a breakpoint in RCTCameraManager during the metadata capture, and realized the data matrix was not being properly set.
So I changed it to do a runtime check, and now it works.

Is that better than #412?

Experienced the same issue with the data matrix, and manually integrating the change fixed the issue. Would be really useful if this could be merged into the top-level. Would be happy to help but not sure what steps are required?

Can someone rebase this? Seems useful to pull in.

I believe #586 is the rebase for this

Thanks @micksabox Then lets close this!


Macro highlighting failing after update

I'm unsure when this happened in the commits, I have treesitter update everyday, so I believe it has been since last night. I'm also unsure if this has to do with this repo or the main treesitter repo. However, attribute-macros no longer highlight with my code. #[derive(...)] does, but #[serde(...)] does not.
The highlight group under #[derive(...)] is the following:
# Treesitter
* **@variable** -> **rustTSVariable** -> **TSVariable**
*<EMAIL_ADDRESS>-> **rustTSFuncMacro** -> **TSFuncMacro**
* **@_name** -> **rust0** -> **0**
* **@_name** -> **rust0** -> **0**
* **@_name** -> **rust0** -> **0**
* **@_name** -> **rust0** -> **0**
* **@_name** -> **rust0** -> **0**

The highlight group under #[serde(...)] is the following:
# Treesitter
* **@variable** -> **rustTSVariable** -> **TSVariable**

Picture to show an example:

@aryx This looks to me like the intended effect of #93? #[serde()] is not a built-in macro, so #93 causes tree-sitter to parse it as an attr_item rather than a meta_item, which, being a new node type, is presumably not highlighted (I'm not at all familiar with how syntax highlighting with tree-sitter works, sorry).
Avoiding this kind of breaking change while supporting non-meta-item attributes would require a different approach than #93, probably using injections as @resolritter suggested in https://github.com/tree-sitter/tree-sitter-rust/pull/93#issuecomment-747863610.

It's not highlighted because highlights.scm needs to be updated to take into account the new attr_item construct?

Maybe? I don't see anything in highlights.scm (nor in the other queries) that looks like it handles the internal structure of attributes at all. There's just:
(attribute_item) @attribute
(inner_attribute_item) @attribute

Are there any tests for syntax highlighting, or an example of how best to write such tests?


ValueError: node 'Placeholder' in input_map does not exist in graph (input_map entry: input_image:0->Placeholder:0)

System information

Have I written custom code:
OS Platform and Distribution (Windows 10):
TensorFlow installed from (binary):
TensorFlow version (1.15):
Python version(3.7.6):
CUDA/cuDNN version(10.0):
GPU model and memory(GTX 1660ti and 6GB):

Description of problem
I was running file.py but still, it is not running, I did some changes in the source file. Still, the placeholder error is the same. The code is also updated with this question. Kindly review and Help.
Source code / logs
minimal code
with tf.io.gfile.GFile(args.model, "rb") as f:
    graph_def = tf.compat.v1.GraphDef()
    graph_def.ParseFromString(f.read())

    with tf.Graph().as_default() as graph:
        generated_image_1, generated_image_2, generated_image_3, = tf.import_graph_def(
                graph_def,
                input_map={'input_image' : input_tensor, 'short_edge_1' : short_edge_1, 'short_edge_2' : short_edge_2, 'short_edge_3' : short_edge_3},
                return_elements=['style_subnet/conv-block/resize_conv_1/output:0', 'enhance_subnet/resize_conv_1/output:0', 'refine_subnet/resize_conv_1/output:0'],
                name=None,
                op_dict=None,
                producer_op_list=None  *#line number 55*
            )

short_edges = [int(e) for e in args.hierarchical_short_edges.split(',')]

error
(myenv) D:\subbu>python file.py --model=models/model.pb --input_image=test_images/image.jpg
2021-12-25 00:54:11.765889: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library cudart64_100.dll
input image -  test_images/image.jpg (1200, 630, 3)
WARNING:tensorflow:From file.py:55: calling import_graph_def (from tensorflow.python.framework.importer) with op_dict is deprecated and will be removed in a future version.
Instructions for updating:
Please file an issue at https://github.com/tensorflow/tensorflow/issues if you depend on this feature.
Traceback (most recent call last):
  File "C:\Users\subbu\myenv\lib\site-packages\tensorflow_core\python\framework\importer.py", line 501, in _import_graph_def_internal
    graph._c_graph, serialized, options)  # pylint: disable=protected-access
tensorflow.python.framework.errors_impl.InvalidArgumentError: node 'Placeholder' in input_map does not exist in graph (input_map entry: input_image:0->Placeholder:0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "file.py", line 87, in <module>
    main()
  File "file.py", line 55, in main
    producer_op_list=None
  File "C:\Users\subbu\myenv\lib\site-packages\tensorflow_core\python\util\deprecation.py", line 507, in new_func
    return func(*args, **kwargs)
  File "C:\Users\subbu\myenv\lib\site-packages\tensorflow_core\python\framework\importer.py", line 405, in import_graph_def
    producer_op_list=producer_op_list)
  File "C:\Users\subbu\myenv\lib\site-packages\tensorflow_core\python\framework\importer.py", line 505, in _import_graph_def_internal
    raise ValueError(str(e))
ValueError: node 'Placeholder' in input_map does not exist in graph (input_map entry: input_image:0->Placeholder:0)

Hi @shubhambagwari !
It seems you are using older versions(1.x versions) of Tensorflow which is not supported any more. Attaching relevant threads for reference. Ref 1, 2.   Please upgrade your code  to 2.x versions and let us know!Thanks!

I am using tensorflow 1.15 version
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: mohantym @.>
Sent: Friday, December 31, 2021 6:14:09 PM
To: tensorflow/tensorflow @.>
Cc: Shubham Bagwari @.>; Mention @.>
Subject: Re: [tensorflow/tensorflow] ValueError: node 'Placeholder' in input_map does not exist in graph (input_map entry: input_image:0->Placeholder:0) (Issue #53593)
Hi @shubhambagwarihttps://github.com/shubhambagwari !
It seems you are using older versions(1.x versions) of Tensorflow which is not supported any more. Attaching relevant threads for reference. Ref 1https://github.com/tensorflow/tensorflow/issues/18352, 2https://forums.developer.nvidia.com/t/set-input-to-tensorflow-frozen-model-before-converting-to-trt-fails/110098. Please upgrade your code to 2.x versions and let us know!Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/tensorflow/tensorflow/issues/53593#issuecomment-1003367927, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGBQG4ESPCKLREYS7BRXJ3DUTWQRTANCNFSM5LA36P6Q.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @shubhambagwari Please post this on Forum or Stackoverflow for further assistance ? Thanks!


[gvar] we should perform IUP on composite glyphs

In some cases fonttools is doing IUP on composites, which is causing us to generate different output in some cases.
According to @behdad, in chat: we can just pass the contour points to IUP, with each point also added to the 'contour ends' vec.
alternatively we can avoid calling the iup function and just mark each delta where the values are (0, 0) as being interpolatable. (this feels like the more principled approach)

Duplicate of https://github.com/googlefonts/fontc/issues/398 ?

yes :)


Add link to GitHub in bottom navigation

like in activate footer
https://activate.mozilla.community/

@LucyeoH What text do you want this link (https://github.com/mozilla/activate.mozilla.community/issues) to have?

I think my recommendation was to have a link to THIS GitHub repo (https://github.com/ossn/ossn-frontend) linked from the OSSN website footer, in the same way that the activate repo is linked from the footer on the activate website.

Ok, that makes more sense. I must have read 'like' as 'link'

PR #296


dpkg-deb on mac OS 12.4

The supplied dpkg-deb from procurces results in this error:
[*] Packaging the deb file...
dyld[4327]: Library not loaded: @rpath/libmd.0.dylib
  Referenced from: /Users/syto203/Documents/Repositories/permasigner/dpkg-deb
  Reason: tried: '/opt/procursus/lib/libmd.0.dylib' (no such file), '/opt/procursus/lib/libmd.0.dylib' (no such file), '/usr/local/lib/libmd.0.dylib' (no such file), '/usr/lib/libmd.0.dylib' (no such file)

linking the one already installed (brew or macports) to the path of permasigner instead of downloading from procurses mitigates this error.

That’s interesting, it’s expecting for procursus
@verygenericname is there another macOS dpkg-deb? or will we have to use brew’s

Leaving this open for now, but I have made it require brew dpkg on macOS.

Macports has it too but I didn’t try it. Better to mention both in case a user has either.

I'd rather not have people use MacPorts. Brew isn't great but it's way better than MacPorts.
If people use MacPorts they probably know they can use that too.


Fix issue when compliance POST doesn't have modified time

Coverage decreased (-0.04%) to 26.266% when pulling 57074bb37033996ececaa2bef376d391cf0904e6 on alyiwang:master into 3cde8a55979bbc152e964c408e725cc7c061b83a on linkedin:master.


rolled back the caching mechanism of cargo

now that travis has got the latest version of rustfmt; rolling back the caching mechanism of cargo so that builds are bit faster

Coverage remained the same at 94.946% when pulling f157adfa5a6d71b114415de16dacae50e6571b53 on harrydevnull:master into d50399d099764816dc8b70857e28e44b630dda9f on AlexPikalov:master.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #127   +/-   ##
=======================================
  Coverage   95.56%   95.56%
=======================================
  Files          18       18
  Lines        1870     1870
=======================================
  Hits         1787     1787
  Misses         83       83

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 d50399d...f157adf. Read the comment docs.


Question about spikes in https://pypistats.org/packages/mlflow

I'm not sure if this is the right place to ask this. This is not a bug report, but a question. At https://pypistats.org/packages/mlflow (btw I'm an MLflow maintainer), we see spikes for several Python versions as shown below. What could cause those spikes?
Python 3.12 (date: 10-28 ~ 11-18)

Python 3.9 (date: 07-22 ~ 08-12)

I'll analyze the raw data.


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on<PHONE_NUMBER> for 17365f349f89f901235d19690fa0b1e8c26ca6b7
https://gitlab.com/MatrixAI/open-source/js-timer/-/pipelines/1065790050

Pipeline Succeeded on<PHONE_NUMBER> for 17365f349f89f901235d19690fa0b1e8c26ca6b7
https://gitlab.com/MatrixAI/open-source/js-timer/-/pipelines/1065790050


add support for passing custom environment variables to docker

Some crates need the use of specific environment variables (for example prost-build). Currently, only the ones starting with CARGO_, RUST, MATURIN_ and PYO3_ are passed to the Docker image.

The prost-build example isn't really useful, because we don't have protoc installed in ours docker images, passing PROTOC won't work.
If you are using a customized Dockerfile, you can just set env vars in Dockerfile.

I was building protoc and moving the binary to the work directory (automatically mounted into the container).
In the end, I added a job to build the proto files beforehand. Thanks anyway!


Mecha Kingdom Jax lacks detail <3

Flame on back thruster when increasing attack-speed;
Wings when levels up to 6.
Please check it,
Thank you very much!

Read before you open an issue with broken skins #1


Use cls instead of class name in Traceback.from_exception()

Type of changes

[ ] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[x] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Nothing too special, just changes Traceback.from_exception() to not rely on the specific class name and rather refer to cls. I'm guessing this probably wasn't meant to be subclassed, but it doesn't hurt to change this.
Haven't added any tests, but I could add a test looking somewhat like this if you think it makes sense to do so:
class SubclassTraceback(Traceback):
    pass

try:
    1 / 0
except Exception:
    assert isinstance(SubclassTraceback.from_exception(*sys.exc_info())

Seems reasonable 👍

Seems reasonable 👍


simp doesn't always synthesize instances depending on outParams

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Check that your issue is not already filed.
Reduce the issue to a minimal, self-contained, reproducible test case. Avoid dependencies to mathlib4 or std4.

Description
In simp calls, finding an instance that depends on an outParam of an instance later in the parameter list is not reliable. The following minimized example comes from leanprover-community/mathlib4#8386:
class Semiring (R : Type)

class Module (R M : Type) [Semiring R] :=
  (foo : M)

class FunLike (F : Type) (α β : outParam Type) :=
  (toFun : F → α → β)

class SemilinearMapClass (F : Type) (R S M N : outParam Type) [Semiring R] [Semiring S]
    [Module R M] [Module S N] [FunLike F M N] :=
  (map_smul : ∀ (f : F) (x : M), FunLike.toFun f x = Module.foo S)

set_option trace.Meta.synthInstance true in
set_option trace.Meta.Tactic.simp true in
example (F R S M N : Type) [Semiring R] [Semiring S] [Module R M] [Module S N] [FunLike F M N]
    [SemilinearMapClass F R S M N] (f : F) (x : M) :
    FunLike.toFun f x = Module.foo S :=
  by simp only [SemilinearMapClass.map_smul] -- simp made no progress
/-
Trace output:
[Meta.synthInstance] 💥 Semiring ?R ▶

[Meta.Tactic.simp.discharge] @SemilinearMapClass.map_smul, failed to synthesize instance
      Semiring ?R

[Meta.Tactic.simp.unify] @eq_self:1000, failed to unify
      ?a = ?a
    with
      FunLike.toFun f x = Module.foo S

-/

The equivalent Lean 3 code succeeds:
class Semiring (R : Type)

class Module (R M : Type) [Semiring R] :=
  (foo : M)

class FunLike (F : Type) (α β : out_param Type) :=
  (toFun : F → α → β)

class SemilinearMapClass (F : Type) (R S M N : out_param Type) [Semiring R] [Semiring S]
    [Module R M] [Module S N] [FunLike F M N] :=
  (map_smul : ∀ (f : F) (x : M), FunLike.toFun f x = Module.foo S)

example (F R S M N : Type) [Semiring R] [Semiring S] [Module R M] [Module S N] [FunLike F M N]
    [SemilinearMapClass F R S M N] (f : F) (x : M) :
    FunLike.toFun f x = Module.foo S :=
  by simp only [SemilinearMapClass.map_smul] -- simp succeeds

Context
For performance reasons I would like to change the definition of "morphism classes", including SemilinearMapClass in Mathlib: previously we wrote extends FunLike F α β, which we want to change to a parameter [FunLike F α β]. All morphism classes can succesfully be changed, apart from SemilinearMapClass, because currently simp does not reliably apply the map_smul lemma.
Steps to Reproduce
See minimized example code. The crucial part appears to be a simp lemma taking an instance parameter, which depends on an outParam of a class that does not appear in the simp goal.
Expected behavior: simp finds the SemilinearMapClass instance, assigns R and S using outParams, and uses this to synthesize Semiring R Semiring S instances, like in Lean 3.
Actual behavior: simp fails when it cannot synthesize Semiring instances, even though it could make progress.
Versions
#eval Lean.versionString
-- "4.5.0, commit 5ac244a932e52a14ec69ac20d41c5e268ffa1d62"

$ uname -a
Linux McChouffe 6.1.62-1-lts #1 SMP PREEMPT_DYNAMIC Thu, 09 Nov 2023 17:21:17 +0000 x86_64 GNU/Linux

Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

For some more context, @Vierkantor implemented in https://github.com/leanprover-community/lean/pull/659 an instance synthesis loop for simp lemma arguments, which is similar in spirit to the Lean 4 main elaborator's instance synthesis loop.
An analogous change to Lean 4 would be to have Lean.Meta.Simp.synthesizeArgs maintain a work list of instance metavariables, and make it so it fails only if no progress can be made or if there is a true synthesis failure. One design complication is deciding at what stage simp should discharge Prop arguments (maybe they could also be on the work list, where, like tactic metavariables, they are deferred if their type still has metavariables?)
It would be nice to make it so that, however this algorithm is changed, the algorithm for how rw synthesizes its arguments follows suit.

I should also point out that by rw [SemilinearMapClass.h] and SemilinearMapClass.h _ _ successfully prove the goal, but by apply SemilinearMapClass.h does not.

Has this already been resolved?

This does seem to be fix on v4.8.0-rc1!


test: change double type syntax

What problem does this PR solve?
close #473
What is changed and how it works?
change DOUBLE(16) to DOUBLE
Check List
Tests

Integration test

Side effects
Related changes

Lgtm


Fix search field styling in Safari

Not sure when this broke, but this disables the default search field style that has shown up. On linght mode it’s ugly but usable, but in dark mode it’s impossible to read your search terms.
Before:

After:

Weird, thanks!


Exemptions Breaking with latest code release

Hello,
We previously had some JSON exemptions that worked fine, but now that we took the latest version of code [v7.2.12] they no longer work -- we were on version 7.15, now our ADO pipeline is throwing the following error:
Set-AzPolicyExemption: /home/vsts/work/1/s/Scripts/Deploy/Deploy-PolicyPlan.ps1:374
Line |
 374 |              $null = Set-AzPolicyExemption @filteredExemption
     |                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | InvalidPolicyDefinitionReference : The policy exemption 'Exempting Redis
     | Policy until pipeline upgrade happens' create request is invalid. The
     | associated policy assignment is an assignment of a policy definition.
     | The policy definition reference is only valid for policy assignments of
     | policy set definitions. CorrelationId:
     | <snip>

##[error]PowerShell exited with code '1'.

And here is the existing exemption JSON that no longer works: (sanitized):
{
    "exemptions": [
        {
            "name": "Exempting Redis Policy",
            "displayName": "Exempting Redis Policy until pipeline upgrade happens",
            "description": "Exempting Redis Policy until pipeline upgrade happens ",
            "exemptionCategory": "Waiver",
            "expiresOn": "2023-06-25T21:00:00.000Z",
            "scope": "/providers/Microsoft.Management/managementGroups/my-mgmt-grp",
            "policyAssignmentId": "/providers/Microsoft.Management/managementGroups/azure-redco/providers/Microsoft.Authorization/policyAssignments/my-custom-policy-name",
            "metadata": {
                "stuff": "listed,here"
            }
        }
    ]
  }

Any thoughts?  I am confused by exceptions documentation because the CSV and JSON have different field names for policy assignment ID, assignmentId for CSV, and policyAssignmentId for JSON -- Maybe I'm the only one doing JSON exceptions still?

Investigating

FYI - we have backed out the upgrade of the latest EPAC Scripts, and we are still having the issue, so there's a good chance my issue is not relevant to the upgrade.  We are investigating as well (maybe our policy  assignment ID changed, or the backend API changed recently?)

Thx for update, was just about to pull your earlier version for repro.  still investigating...

We thought we had the issue. However, after building a test bed for your scenario, we cannot replicate the problem. Therefore, we need more information from your own investigation. If it yields nothing useful, we can do a debugging session. Please contact me at heinrich  dot gantenbein _at-signy thing microsoft _dottie com to set up a debugging session over Teams.

Thank you for the debugging session. It appears that this is a bug in the underlying Az Module with Set-AzPolicyExemption. We will rewrite this with Invoke-AzRestmethod instead.


2024 Fall Cohort Process - Call 5 (Oct 2)

To do list

[x] 1 week prior to call: Finalize lesson plan/slides
[x] By Fri after call: Download, format, post call recording
[ ] By Fri after call: Draft Digest, send for review @madiboynton
[ ] By Tues following call: Review digest
[ ] By  Wed following call: Send digest

Hi-ya @madiboynton ! Let me and @tures23 know when you've got the draft digest (doc in our Mentor Materials Folder) ready for our review. If you want to jump on a call to co-work before posting the GitHub Issue, we can make time for that too! :)

@AHolder1 whoops -- thank you for the reminder! I will draft the digest and put it in the folder today or tomorrow.

@AHolder1 @tures23 Done drafting! https://cawaterboards-my.sharepoint.com/:w:/r/personal/anna_holder_waterboards_ca_gov/_layouts/15/Doc.aspx?sourcedoc=%7BC8EC2749-38E8-4849-A84D-C16423878FB4%7D&file=DRAFT%20Digests.docx&action=default&mobileredirect=true

@madiboynton marvelous digest! added the Teams link for the 2 month check in, not sure the link makes sense so take a look :)  tag to you @AHolder1

Everything looks great @madiboynton ! Go ahead and create the Issue and @ the team to send whenever you're ready this week. If you want to jump on a call and co-work to get it done, let me know. :)


Moving the markdown converter solution to the src/docs folder

Since we have other utilities for managing docs that are totally separate from the actual source figured they should be together. Possibly will create a single sln for these two.
Its just moving
src/dotnet/tools/JavaDocToMarkdownConverter -> src/docs/JavaDocToMarkdownConverter

@NightOwl888 you ok with this? it's just moving a proj

I don't see an issue. It would be good to have all of the doc build stuff in the same folder.

great, will merge (trying to get some of this docs work done this week)


Update s3, sdk-core, sqs to 2.16.10

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.15.82 to 2.16.10.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, semver-minor

Superseded by #99.


Warp data as many as possible in SSLTranmission#write(ByteBuffer)

Write to the socket until no more space instead of requiring an additional selector loop.
They can give us performance gain when PUT blobs.

Codecov Report

Merging #1105 into master will decrease coverage by 0.06%.
The diff coverage is 41.58%.

@@             Coverage Diff              @@
##             master    #1105      +/-   ##
============================================
- Coverage     68.53%   68.47%   -0.07%
- Complexity     4982     4989       +7
============================================
  Files           389      389
  Lines         31270    31285      +15
  Branches       3998     3999       +1
============================================
- Hits          21432    21422      -10
- Misses         8778     8800      +22
- Partials       1060     1063       +3

Impacted Files
Coverage Δ
Complexity Δ

...java/com.github.ambry.network/SSLTransmission.java
62.57% <41.58%> (-0.46%)
58 <13> (+8)

...java/com.github.ambry.store/CompactionManager.java
86.75% <0%> (-2.65%)
19% <0%> (ø)

...b.ambry.network/BlockingChannelConnectionPool.java
63.47% <0%> (-2.18%)
8% <0%> (ø)

...in/java/com.github.ambry.store/BlobStoreStats.java
70.56% <0%> (-1.52%)
100% <0%> (-1%)

...ain/java/com.github.ambry.router/PutOperation.java
90.86% <0%> (-0.54%)
108% <0%> (-1%)

...om.github.ambry/network/BoundedByteBufferSend.java
31.57% <0%> (+5.26%)
6% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

minor typo with commit message: warp instead of wrap.
We can merge this after #1104 is merged


frontend failed to build. win10

docker-compose up --build
Building frontend
Step 1/3 : FROM node:10-alpine
---> 947b2561f403
Step 2/3 : RUN apk add --no-cache git
---> Running in fa89055619cb
fetch http://dl-cdn.alpinelinux.org/alpine/v3.11/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.11/community/x86_64/APKINDEX.tar.gz
(1/6) Installing ca-certificates (20191127-r1)
(2/6) Installing nghttp2-libs (1.40.0-r0)
(3/6) Installing libcurl (7.67.0-r0)
(4/6) Installing expat (2.2.9-r1)
(5/6) Installing pcre2 (10.34-r1)
(6/6) Installing git (2.24.1-r0)
Executing busybox-1.31.1-r9.trigger
Executing ca-certificates-20191127-r1.trigger
OK: 23 MiB in 22 packages
ERROR: Service 'frontend' failed to build: The command '/bin/sh -c apk add --no-cache git' returned a non-zero code:<PHONE_NUMBER>: failed to shutdown container: container fa89055619cb7496f11bcb83d8660c5bd82f4233535cd9a3b7559623a8963109 encountered an error during hcsshim::System::waitBackground: failure in a Windows system call: The virtual machine or container with the specified identifier is not running. (0xc0370110): subsequent terminate failed container fa89055619cb7496f11bcb83d8660c5bd82f4233535cd9a3b7559623a8963109 encountered an error during hcsshim::System::waitBackground: failure in a Windows system call: The virtual machine or container with the specified identifier is not running. (0xc0370110)
running docker run hello-world runs as expected.  other containers/compose apps working.

To be honest I've never tried to run the cookiecutter on Windows. 😅 Hopefully someone can submit a PR to fix and I'd be happy to merge.


how to prevent the icons from being highlighted on getting selected

how to prevent the icons from being highlighted on getting selected , when i click and hold on the items they get a highlighted circular select in and in my case the icons for floaty items are not circular so it looks ugly, Thanks in advance

i solved that putting the ImageButton again when fab clicked.
self.FAB.buttonImage = #imageLiteral(resourceName: "FABIcon")

Closing too old issue.


SqlServerDsc: Improvement to how Import-SQLPSModule import module

Pull Request (PR) description

Changes to SqlServerDsc

Updated helper function Import-SQLPSModule

To only import module if the
module does not exist in the session.
To always import the latest version of 'SqlServer' or 'SQLPS' module, if
more than one version exist on the target node. It will still prefer to
use 'SqlServer' module.

Changes to SqlAG

Removed excess Import-SQLPSModule call.

Changes to SqlSetup

Now after a successful install the "SQL PowerShell module" is reevaluated and
forced to be reimported into the session. This is to support that a never
version of SQL Server was installed side-by-side so that SQLPS module should
be used instead.

This Pull Request (PR) fixes the following issues
Helps with issue #1151
Task list

[x] Added an entry under the Unreleased section in the CHANGELOG.md? Entry
should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md?
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help?
[x] Comment-based help added/updated?
[x] Localization strings added/updated in all localization files as appropriate?
[ ] Examples appropriately added/updated?
[x] Unit tests added/updated?
See DSC Resource Testing Guidelines.
[x] Integration tests added/updated (where possible)?
See DSC Resource Testing Guidelines.
[x] New/changed code adheres to
DSC Resource Style Guidelines
and Best Practices?

This change is

Codecov Report

Merging #1175 into dev will increase coverage by <1%.
The diff coverage is 93%.

@@          Coverage Diff          @@
##            dev   #1175    +/-   ##
=====================================
+ Coverage    97%     97%   +<1%
=====================================
  Files        33      33
  Lines      3994    4012    +18
=====================================
+ Hits       3907    3925    +18
  Misses       87      87

Closing and reopening to kick off the tests igen (resolved bug in the test framework that cause the test from failing)


Manually install and configure MongoDB on a VM

I've created a vm using the following command:
az vm create
--resource-group myResourceGroup
--name myVM
--image CentOS
--admin-username azureuser
--generate-ssh-keys
Then I try to ssh into the public id address that was displayed in the output.
ssh matt<EMAIL_ADDRESS>And I got the following message:
Permission denied (publickey,gssapi-keyex,gssapi-with-mic).
I'm already logged into my Azure account so I'm wondering if that has something to do with my error.

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

ID: a7b52e21-3187-43de-e393-f9c2fe8daafa
Version Independent ID: 740a3793-60ce-af96-7704-5ff01a676664
Content: Install MongoDB on a Linux VM with the Azure CLI
Content Source: articles/virtual-machines/linux/install-mongodb.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

@mtschafe  Can you try to ssh using: ssh<EMAIL_ADDRESS>and see if that fixes it?

@mtschafe as @cynthn mentions, you need to use the username you created the VM with. It has nothing to do with the account you login to azure with.

Yes, that fixes it. Thank you.

I'm also having a problem writing in the repository file. I can open it with:
vi /etc/yum.repos.d/mongodb-org-3.6.repo
But I can't write in it:
W10: Warning: Changing a readonly file

@mtschafe thanks for the update. Glad to hear that fixed the issue.
As per writing to the DB, I am not overly familiar with MongoDB. However, if you are having issues writing, you might need to elevate to Sudo before you can write things to that directory. So I would try using the command sudo vi /etc/yum.repos.d/mongodb-org-3.6.repo
If that doesn't work, I suggest reaching out to MSDN, Stack Overflow or to MongoDB directly for further assistance.


VM not starting on vagrant up

I think the issue is occuring when it is trying to setup the shared folders. It just hangs.
This is on Windows7, VBox 5.0.2, vagrant 1.7.4.
Trying to use the example config and make files, with one small change. This is my
vagrant_synced_folders config:
vagrant_synced_folders:
  # The first synced folder will be used for the default Drupal installation, if
  # build_from_makefile is 'true'.
  - local_path: D:/Sites/drupalvm
    destination: /var/www
    id: drupal
    type: smb
    create: true

Last few lines of output when running vagrant up --debug are:
DEBUG host: Searching for cap: nfs_prune
DEBUG host: Checking in: windows
INFO nfs: Host doesn't support pruning NFS. Skipping.
INFO warden: Calling IN action: #<Vagrant::Action::Builtin::SyncedFolderCleanup
:0x3df9c20>
INFO subprocess: Starting process: ["C:\Windows\System32\WindowsPowerShell\
v1.0\/powershell.EXE", "-NoProfile", "-ExecutionPolicy", "Bypass", "$PSVersionT
able.PSVersion.Major"]
INFO subprocess: Command not in installer, restoring original environment...
DEBUG subprocess: Selecting on IO
DEBUG subprocess: stdout: 2

I think I found the issue. I found this is the vagrant issue queue: https://github.com/mitchellh/vagrant/issues/3139. Apparently, one needs PowerShell version 3.0 or higher. Win7 is at 2.0. I downloaded the latest Windows Management Framework, which comes with 4.0 and that got me much further into the process. I'll fork, update the README, and submit a pull request.

I'll definitely merge in that PR once the update has been made (putting the notes in docs instead of the main README). Thanks!


Data rendering

The MarchingCubes implementation for rendering vtkImageData objects is sufficient; I thought it was making shapes appear blocky, but that's how the shape actually is. This is the shape rendered in the
ShapeWorks desktop app:

versus in the Vue app:


Attribute correctness testcase to domain-name

Domain name is an attribute which carries information about the domain name of the application hosted. This has a pattern "^([a-z0-9]+(-[a-z0-9]+)*.)+[a-z]{2,}$"
For this pattern, we can check for correctness in following scenarios which is expected to fail
include special characters other than "-" and "."
string without character "." (it is expected to have a top-level domain at end)
string with space
Top-level-domain TLD containing a number
The about correctness should be checked for following services

[x] /v1/bequeath-your-data-and-die
[x]  /v1/redirect-info-about-approval-status-changes

Fixed as part of pull #191


performance issue 性能不好，有卡顿

At first, thanks for the effort into creating this plugin.

I feel like my vscode pauses every now and then as I type anything. Performance is much worse than using MMA frontend directly. My machine is Ubuntu 18.04, MMA 11.3, on a 6 core i5-8600k CPU, 16 GB DDR4 mem with SSD storage.

In particular, vscode pauses whenever an auto-completion menu is about to show up.

Thanks for your issue. Sorry about my plugin with such a serious problem.
This problem is generally caused by a bad regular expression that seriously affects text parsing. So try editing other files to see if there are performance issues. If the performance issue does only occur in files with specific paragraphs, please provide the relevant code.
Auto-completion is a problem that seems difficult to solve. The extension provides thousands of built-in symbols and their documentations (and there may be more in future versions), which may indeed cause the Caton. I'll try to solve it. If you have any good ideas, please let me know.

It is probably caused by a bug from vscode-textmate as well. There are some issues similar to my problem.


TextField misbehaves when the on-screen keyboard is closed

Steps to Reproduce

Execute flutter run on the code sample (be sure to run it inside an android emulator and not another platform)
when the app builds and launches successfully click on the floating action button to show the input modal
click inside the textField, this will bring out the android  on-screen keyboard, write anything inside the text field(for example a single letter "O") then click the add button(without closing the keyboard) then you will see in the console this
I/flutter ( 8443): Submitted taskTitle is: o which is expected and normal.
now repeat the previous step but close the keyboard before clicking the "Add" button: write anything inside the text field(for example a single letter "O") then close the keyboard either by clicking the enter button  on your host's physical keyboard, clicking the done button on the on-screen keyboard or  clicking the down arrow in the bottom left corner (in the navigation bar) then click the add button then you will see in the console this:
Submitted taskTitle is: "note there is nothing after the colon so in other words the $taskTitle variable was set to an empty string"

Expected results:
I would expect the result to be the same regardless if the keyboard is closed or not
Actual results:
The variable is getting emptied when the keyboard is closed

Code Sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  String taskTitle = '';
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text("test app"),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: () {
          showModalBottomSheet(
              isScrollControlled: true,
              context: context,
              builder: (context) {
                return AddItem();
              });
        },
        child: const Icon(Icons.add),
      ),
    );
  }
}

class AddItem extends StatelessWidget {
  AddItem({Key? key}) : super(key: key);

  String taskTitle = '';

  @override
  Widget build(BuildContext context) {
    return Container(
      padding: const EdgeInsets.all(20),
      child: Column(children: [
        const Center(
          child: Text(
            "Add An Item",
            style: TextStyle(color: Colors.lightBlueAccent, fontSize: 25),
          ),
        ),
        TextField(
          onChanged: (newText) {
            print("newText is $newText");
            taskTitle = newText;
            print("new taskTitle is $taskTitle");
          },
        ),
        const SizedBox(
          height: 20,
        ),
        TextButton(
          child: const Text("Add"),
          onPressed: () {
            print("Submitted taskTitle is: $taskTitle");
          },
        )
      ]),
    );
  }
}

Logs

Analyzing testing...

   info • Prefer const literals as parameters of constructors on @immutable classes • lib/main.dart:33:21 •
          prefer_const_literals_to_create_immutables
   info • Prefer const with constant constructors • lib/main.dart:34:13 • prefer_const_constructors
   info • This class (or a class that this class inherits from) is marked as '@immutable', but one or more of
          its instance fields aren't final: AddItem.taskTitle • lib/main.dart:53:7 • must_be_immutable
   info • Avoid `print` calls in production code • lib/main.dart:71:13 • avoid_print
   info • Avoid `print` calls in production code • lib/main.dart:73:13 • avoid_print
   info • Avoid `print` calls in production code • lib/main.dart:82:13 • avoid_print
   info • This class (or a class that this class inherits from) is marked as '@immutable', but one or more of
          its instance fields aren't final: AddItem.taskTitle • lib/models/add_model.dart/add_model.dart:3:7 •
          must_be_immutable
   info • Avoid `print` calls in production code • lib/models/add_model.dart/add_model.dart:21:13 •
          avoid_print
   info • Avoid `print` calls in production code • lib/models/add_model.dart/add_model.dart:23:13 •
          avoid_print
   info • Avoid `print` calls in production code • lib/models/add_model.dart/add_model.dart:32:13 •
          avoid_print
   info • Avoid `print` calls in production code • lib/models/add_model.dart/add_model.dart:33:13 •
          avoid_print

11 issues found. (ran in 4.4s)

[✓] Flutter (Channel stable, 2.10.5, on macOS 12.3.1 21E258 darwin-arm, locale en-IQ)
    • Flutter version 2.10.5 at /opt/homebrew/Caskroom/flutter/2.10.1/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5464c5bac7 (2 weeks ago), 2022-04-18 09:55:37 -0700
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/rhx/Library/Android/sdk
    • Platform android-32, build-tools 32.1.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] VS Code (version 1.66.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.40.0

[✓] Connected device (3 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 12 (API 32) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64   • macOS 12.3.1 21E258 darwin-arm
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 100.0.4896.127

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @moham96, Thanks for filing the issue. This seems to be happening because when you show/hide the Keyboard the AddItem widget is getting rebuilt. Since it is stateless the state of the widget is lost. One way to prevent this is to make AddItem as StatefulWidget.


Certain files not being included in bundle

Describe the bug

While bunldling a Vanilla JS project with image assets, using npm run build, there was one image that was not being bundled.

The image that was not being bundled was the last image to be pasted into my project

I have imported all images in the same way, using import ... from "filename.png" syntax in one file.

This is the output of npm run build
NOTE Clearly, ash.png is not included, whereas player.png in the same folder is included. Both are imported in the same file in the same way as shown above.

❯  npm run build

><EMAIL_ADDRESS>build D:\Sattwik\code\web\js\game-dev\clean-rpg
> tsc && vite build

vite v2.5.6 building for production...
✓ 22 modules transformed.
dist/assets/favicon.17e50649.svg        1.49 KiB
dist/assets/room-builder.325bbcfc.png   20.98 KiB
dist/assets/things.5fe02e3d.png         27.07 KiB
dist/assets/player.c5281145.png         7.15 KiB
dist/assets/ground.8651476d.png         69.11 KiB
dist/assets/interior.294e2488.png       203.12 KiB
dist/index.html                         0.78 KiB
dist/assets/index.cc5b3aa7.css          2.35 KiB / brotli: 1.40 KiB
dist/assets/index.f94427a3.js           137.68 KiB / brotli: 8.73 KiB
dist/assets/vendor.354f1e58.js          1305.08 KiB / brotli: skipped (large chunk)

(!) Some chunks are larger than 500 KiB after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/guide/en/#outputmanualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.

On starting the dev server with npm run dev, all the images load and everything works
But after building and previewing with npm run serve, ash.png does not load and a warning vendor.354f1e58.js:105 Local data URIs are not supported: ash-spritesheet is displayed in the console

Reproduction
Please refer to the repository here, containing all files and assets as is

Run the dev server with npm run dev and the project should work (the game should load with both images - player.png and ash.png)
Preview with npm run build then npm run serve and reproduce the bug

System Info
❯  npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers
npx: installed 1 in 6.14s

  System:
    OS: Windows 7 6.1.7601
    CPU: (2) x64 Pentium(R) Dual-Core  CPU      E5300  @ 2.60GHz
    Memory: 1.10 GB / 2.99 GB
  Binaries:
    Node: 12.21.0 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.11 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Internet Explorer: 8.0.7601.17514
  npmPackages:
    vite: ^2.5.4 => 2.5.6

Used Package Manager
npm
Logs
❯  npx vite build --debug
  vite:config no config file found. +0ms
  vite:config using resolved config: {
  vite:config   root: 'D:/Sattwik/code/web/js/game-dev/clean-rpg',
  vite:config   base: '/',
  vite:config   mode: 'production',
  vite:config   configFile: undefined,
  vite:config   logLevel: undefined,
  vite:config   clearScreen: undefined,
  vite:config   build: {
  vite:config     target: [ 'es2019', 'edge88', 'firefox78', 'chrome87', 'safari13.1' ],
  vite:config     polyfillModulePreload: true,
  vite:config     outDir: 'dist',
  vite:config     assetsDir: 'assets',
  vite:config     assetsInlineLimit: 4096,
  vite:config     cssCodeSplit: true,
  vite:config     sourcemap: false,
  vite:config     rollupOptions: {},
  vite:config     commonjsOptions: { include: [Array], extensions: [Array] },
  vite:config     dynamicImportVarsOptions: { warnOnError: true, exclude: [Array] },
  vite:config     minify: 'terser',
  vite:config     terserOptions: {},
  vite:config     write: true,
  vite:config     emptyOutDir: null,
  vite:config     manifest: false,
  vite:config     lib: false,
  vite:config     ssr: false,
  vite:config     ssrManifest: false,
  vite:config     brotliSize: true,
  vite:config     chunkSizeWarningLimit: 500,
  vite:config     watch: null
  vite:config   },
  vite:config   configFileDependencies: [],
  vite:config   inlineConfig: {
  vite:config     root: undefined,
  vite:config     base: undefined,
  vite:config     mode: undefined,
  vite:config     configFile: undefined,
  vite:config     logLevel: undefined,
  vite:config     clearScreen: undefined,
  vite:config     build: {}
  vite:config   },
  vite:config   resolve: { dedupe: undefined, alias: [ [Object], [Object] ] },
  vite:config   publicDir: 'D:\\Sattwik\\code\\web\\js\\game-dev\\clean-rpg\\public',
  vite:config   cacheDir: 'D:\\Sattwik\\code\\web\\js\\game-dev\\clean-rpg\\node_modules\\.vite',
  vite:config   command: 'build',
  vite:config   isProduction: true,
  vite:config   plugins: [
  vite:config     'alias',
  vite:config     'vite:modulepreload-polyfill',
  vite:config     'vite:resolve',
  vite:config     'vite:html',
  vite:config     'vite:css',
  vite:config     'vite:esbuild',
  vite:config     'vite:json',
  vite:config     'vite:wasm',
  vite:config     'vite:worker',
  vite:config     'vite:asset',
  vite:config     'vite:define',
  vite:config     'vite:css-post',
  vite:config     'vite:build-html',
  vite:config     'commonjs',
  vite:config     'vite:data-uri',
  vite:config     'rollup-plugin-dynamic-import-variables',
  vite:config     'asset-import-meta-url',
  vite:config     'vite:import-analysis',
  vite:config     'vite:esbuild-transpile',
  vite:config     'vite:terser',
  vite:config     'vite:reporter',
  vite:config     'load-fallback'
  vite:config   ],
  vite:config   server: { fs: { strict: undefined, allow: [Array] } },
  vite:config   env: { BASE_URL: '/', MODE: 'production', DEV: false, PROD: true },
  vite:config   assetsInclude: [Function: assetsInclude],
  vite:config   logger: {
  vite:config     hasWarned: false,
  vite:config     info: [Function: info],
  vite:config     warn: [Function: warn],
  vite:config     warnOnce: [Function: warnOnce],
  vite:config     error: [Function: error],
  vite:config     clearScreen: [Function: clearScreen],
  vite:config     hasErrorLogged: [Function: hasErrorLogged]
  vite:config   },
  vite:config   createResolver: [Function: createResolver],
  vite:config   optimizeDeps: { esbuildOptions: { keepNames: undefined } }
  vite:config } +46ms
vite v2.5.6 building for production...
transforming (1) index.html  vite:resolve 0ms   ./assets/ui/msg-box.png -> D:/Sattwik/code/web/js/game-dev/clean-rpg/src/assets/ui/msg-box.png +0ms
✓ 22 modules transformed.
dist/assets/favicon.17e50649.svg        1.49 KiB
dist/assets/player.c5281145.png         7.15 KiB
dist/assets/room-builder.325bbcfc.png   20.98 KiB
dist/assets/things.5fe02e3d.png         27.07 KiB
dist/assets/ground.8651476d.png         69.11 KiB
dist/assets/interior.294e2488.png       203.12 KiB
dist/index.html                         0.78 KiB
dist/assets/index.cc5b3aa7.css          2.35 KiB / brotli: 1.40 KiB
dist/assets/index.f94427a3.js           137.68 KiB / brotli: 8.73 KiB
dist/assets/vendor.354f1e58.js          1305.08 KiB / brotli: skipped (large chunk)

(!) Some chunks are larger than 500 KiB after minification. Consider:
- Using dynamic import() to code-split the application
- Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/guide/en/#outputmanualchunks
- Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I saw that there's a commit with the message "Solved THE PROBLAAAHM!" so I guess you have solved the issue?
And what you initially needed might be this option: https://vitejs.dev/config/#build-assetsinlinelimit

Yes, I solved it a different way. Thank you for informing about the option to change to solve the problem. :+1:


alpha of v1 flowkit

Closes #???
Description

For contributor use:

[ ] Targeted PR against main branch
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

114 of 139 (82.01%)  changed or added relevant lines in 8 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.5%) to 80.835%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

script.go
11
12
91.67%

setup.go
16
19
84.21%

event_fetcher.go
8
12
66.67%

interaction_builder.go
20
24
83.33%

transaction.go
0
6
0.0%

state.go
57
64
89.06%

Files with Coverage Reduction
New Missed Lines
%

script.go
1
79.61%

state.go
1
84.42%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
2286

Relevant Lines:
2828

💛  - Coveralls


Other type of ws message

When trying to authenticate using json message, received back:
<< {"event":"Error authenticating"}
Other type of ws message>
>

Should be fixed by commit e1e4dbf0dec11b8be15cbeb4f9aa38e65d1e3004 @hwchen hwchen committed 6 minutes ago


Created checklist directory and shipped tier 3 checklist

Outbound Checklists: Created checklist directory and shipped tier 3 checklist
Problem
We would like to start sharing our outbound checklists for others to use and take inspiration from.
Solution

Created outbound-checklists directory
Shipped tier 3 checklist! Other tiers coming soon!

Considerations:

Do we want a separate directory to store our checklists? Or should it live in its respective tier directory?
Would we like this converted to markdown at some point?

Considerations:
* Do we want a separate directory to store our checklists? Or should it live in its respective tier directory?

* Would we like this converted to markdown at some point?

Ship it in the Tier folder it corresponds to for now.
Plz open a ticket to convert to Tier3 checklist to .md file.

Issues have been created to convert to markdown file: #68 + #69
Waiting on input from Elizabeth and Dave before shipping

LGTM from Elizabeth!


Gene neighborhood summary on top of gene summary page could be more helpful

Are we planing on replacing that with something else?  If not...when you mouse over, it should show the RefSeq gene locus and the name.  Having the PATRIC id is not that helpful, and not having the name makes the whole thing more frustrating than useful.

I thing the gene neighborhood  cartoon will stay. Agree, that it needs to have RefSeq locus tags in the mouse over.
Sent from my iPhone

On Sep 25, 2016, at 8:56 AM, ARWattam<EMAIL_ADDRESS>wrote:
Are we planing on replacing that with something else? If not...when you mouse over, it should show the RefSeq gene locus and the name. Having the PATRIC id is not that helpful, and not having the name makes the whole thing more frustrating than useful.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

fixed and will be available on alpha soon.

fixed, verified on beta.

It should also have the gene function, and clicking on a gene should recenter the display. Otherwise you can't walk all the way down the genome using it.

now mouseover displays product and gene is clickable (move to the corresponding gene page)

all fixes verified on beta.


Border Radius on Square

Would be really nice if it was possible to add backgrounds to each individual item in the square view or any other alternative for adding border radius to the square view.
Really amazing work.

Currently you can increase the line thickness by using the parameter lineThickness. You can also set background to the whole view by using background attribute with isCustomBackground = true.
If you are asking for a background for just the Feild and not the entire View. Then I will take it into consideration and work on it.
Thanks.

I meant for individual fields.
Either a way to set background radius because in my case, I needed something more curved at the edges or to set a custom background per field.
I am really loving you work.

Great. Thanks.
Meant border radius not background radius :D

I have finished your feature in CORNER_RADIUS_MASK_PASSWORD. I will upload it as soon as I finish the mask password feature and let you know. It will be ready by the weekend

Hi,
You can find the feature in below version of the library

compile 'com.poovam:pin-edittext-field:1.1.0'

Use it like below example on SquarePinField

app:cornerRadius="10dp"

Let me know if you need any help. Thank for your patience.


Upgrade default Kubernetes version to 1.12.8

Upgrade default Kubernetes version to 1.12.8

/cherrypick release-0.6


Improve diagnostic for attempted initialization of non-nominal types

Resolves SR-4662.

@swift-ci please smoke test and merge

is void consider a non-nominal type? because i thought void type is a an empty struct

Void is considered non-nominal but is not subject to this diagnostic.


Appstudio update test-component-pac-znkv

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-znkv-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
51 seconds
build-container

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
48 seconds
clamav-scan

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
19 seconds
sanity-inspect-image

✅ Succeeded
15 seconds
sbom-json-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/test-component-pac-znkv-on-pull-request has successfully validated your commit.


Fix transaction restarts

The Problem

Fixes issue where the A in ACID for prisma actually gets more of a D
The Solution

Still TODO:

potentially fix tests/method
fix restore saving call


Podman support in cli

What content needs to be created or modified?

dapr int and dapr uninstall can now have additional parameter to pass a container runtime. Earlier there was hard dependency on docker.

    dapr init --container-runtime podman
    dapr uninstall --container-runtime podman

The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos
https://github.com/dapr/cli/pull/1041

Resolved


Modal refactor

Refactor Modals

one big thing I noticed is if we open two modals and the second modal is smaller than the first one, this looks very weird


Repeated warnings about $USER being unset

When running via launchd (through brew services), yabai sometimes prints messages about being unable to get $USER. Running env -v shows that it is set, though. The socket files are also created properly with my username in them, so I'm really not sure what's happening. yabai does not abort when printing the message.
The screenshot below shows my testing setup. Note that this is with almost no other configuration, so there's no custom settings for launchd at all.

That is likely an error when trying to send a message, not from the running instance of yabai. See https://github.com/koekeishiya/yabai/blob/master/src/yabai.c#L53

For what it's worth, the message being printed is still prefixed with yabai: and not yabai-msg:. It also happens without any user interaction (no keypresses, no mouse movement, no applications open other than iTerm2 with tail -f /path/to/log), so I'm really unsure what's happening here.

The message you get is only printed during startup of the actual persistent instance, so it would appear that something on your system tries to launch an instance of yabai at least.
Sent with GitHawk

The issue still occured after I ran brew services stop yabai, so you were definitely right about something on my system trying to launch yabai.
I think brew services or launchd went wrong somewhere and duplicated the launch services entry (or at least loaded it in a faulty state, which persisted reboots). Running sudo launchctl remove homebrew.mxcl.yabai fixed the issue for me.


Create meta-analysis.md

🚀 Deployed on https://6667310b38c6f316d1afc550--cozy-fox-0defec.netlify.app


Demo page not working

On the demo page, all the maps show  Oops! Something went wrong

API limit
I need to generate a new API ID for the demo
You have included the Google Maps API multiple times on this page. This may cause unexpected errors.
js:98:51
Google Maps API error: DeletedApiProjectMapError https://developers.google.com/maps/documentation/javascript/error-messages#deleted-api-project-map-error


[MINOR] fix(spark): Fix NPE for ShuffleWriteClientImpl.unregisterShuffle

What changes were proposed in this pull request?
Fix NPE for ShuffleWriteClientImpl.unregisterShuffle.
Why are the changes needed?
Since the id of RssShuffleManager in Executor is assigned in getWriter, this means that the id may be null before getWriter, so this may cause NPE to occur in ShuffleWriteClientImpl.unregisterShuffle.
error log:
23/12/12 14:25:47 ERROR RssShuffleManager: Errors on closing shuffle write client
java.lang.NullPointerException
	at java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:936)
	at org.apache.uniffle.client.impl.ShuffleWriteClientImpl.unregisterShuffle(ShuffleWriteClientImpl.java:751)
	at org.apache.spark.shuffle.RssShuffleManager.stop(RssShuffleManager.java:591)
	at org.apache.spark.SparkEnv.stop(SparkEnv.scala:90)
	at org.apache.spark.executor.Executor.stop(Executor.scala:335)
	at org.apache.spark.executor.CoarseGrainedExecutorBackend$$anonfun$receive$1$$anon$1.run(CoarseGrainedExecutorBackend.scala:203)

Does this PR introduce any user-facing change?
No.
How was this patch tested?
(Please test your changes, and provide instructions on how to test it:

If you add a feature or fix a bug, add a test to cover your changes.
If you fix a flaky test, repeat it for many times to prove it works.)

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (ecbf2e7) 53.22% compared to head (41af3ad) 53.37%.

Files
Patch %
Lines

...he/uniffle/client/impl/ShuffleWriteClientImpl.java
0.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1367      +/-   ##
============================================
+ Coverage     53.22%   53.37%   +0.14%
+ Complexity     2715     2183     -532
============================================
  Files           418      341      -77
  Lines         23932    15381    -8551
  Branches       2043     1426     -617
============================================
- Hits          12738     8209    -4529
+ Misses        10408     6705    -3703
+ Partials        786      467     -319

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


The link to hl-PopupPreviewBorder is different from the documentation

The link to hl-PopupPreviewBorder in the documentation is “FloatBorder”, but it is now “NormalFloat”.
This pull request will correct the actual link destination to “FloatBorder”.
If this change is not a problem for Matsui-san, I will send a similar PR to denops-signature_help later.
Thanks,

Thanks!


puppet: Set a long timeout on wal-g wal-push, to prevent stalls.

wal-g wal-push has a known bug with occasionally hanging after file
upload to S3[1]; set a rather long timeout on the upload process, so
that we don't simply stall forever when archiving WAL segments.
[1] https://github.com/wal-g/wal-g/issues/656

Merged, thanks @alexmv!


Missing environment variables in devel Docker container

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
After running development container, environment variables set via ansible script doesn't exist.
Expected behavior
All the environment variables set via ansible script are reflected for user while running Docker image with --devel flag.
Actual behavior
/home/${USER}/.bashrc doesn't contain environment variables. The missing variables are:
export PATH="$PATH:/root/.local/bin"
export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp
export CCACHE_DIR="/var/tmp/ccache"
export CC="/usr/lib/ccache/gcc"
export CXX="/usr/lib/ccache/g++"
export PATH="/usr/local/cuda/bin:$PATH"
export LD_LIBRARY_PATH="/usr/local/cuda/lib64:$LD_LIBRARY_PATH"

It causes multiple issues, e.g. running ROS 2 daemon with rmw_fastrtps_cpp DDS.
Steps to reproduce
./docker/run.sh --devel
cat $HOME/.bashrc
sudo su
cat $HOME/.bashrc

Versions
No response
Possible causes
Command exec /usr/sbin/gosu "$USER_NAME" "$@" from docker/etc/ros_entrypoint.sh run bash as user with default .bashrc file. We can fix this two ways:

Handle /etc/skel/.bashrc in ansible scripts. That means push all environment variables to two .bashrc files at the same time. In that case we will need to change export PATH="$PATH:/root/.local/bin" to export PATH="$PATH:\$HOME/.local/bin". I'm not sure if \ will keep $ in scripts as a special sign here. It has to be validated.
Add variables to $HOME/.bashrc via ros_entrypoint.sh. Copying /root/.bashrc is also possible if we modify export PATH="$PATH:/root/.local/bin" as described previously.

Additional context
No response

Hi @amadeuszsz could you resolve the failed checks on the PR ?

Hi @amadeuszsz could you resolve the failed checks on the PR ?

Hi @oguzkaganozt
Done!


fix(askscript): fix whitespace issue in control flows (#390)

The issue is not only in if but also in while, forOf, and forIn control flows. Hence, modified the code parser for all the required control flows. Passes all required tests.

@czerwinskilukasz1 I have added the .ask files in https://github.com/xFAANG/askql/tree/master/src/askscript/tests/04-control-flow in the following commit in PR. Please review and let me know if anything needs to be added, removed or modified.

LGTM, thanks, Shubham :)


Remove duplicate MultiSelectProperty interface

The interface is already defined right above.

 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.

Oops! Just noticed there's an identical PR already https://github.com/makenotion/notion-sdk-js/pull/102/files


Fix build breaks for master-next against llvm.org.

StringRef conversion to std::string needs to be explicit now.

@gribozavr @compnerd
What is the right way to submit changes like this? This is needed to fix a build breakage of master-next, but the changes apply to master as well. I'm submitting this pull request to master, in expectation that the changes will be automerged into master-next.
Is that appropriate?

There is one failing test, which is already broken at head as well:
Failing Tests (1):
    Swift(linux-x86_64) :: Incremental/fail/main.swift

This is targeting master. Should this be against master-next?

For now, let's take this on master-next with the other string conversion fixes. We should cherry-pick them all to master in one go.

Moved to here.


chore(manifest): add keywords to package.json

adds the keywords property to package.json with relevant keywords, where it was previously omitted.
noticed there weren't any keywords in the npm page for the package, figured it'd only take me a minute to add some relevant ones :)

Thank you so much for this PR! I'm sorry we didn't get around to reviewing it. I'm adding keywords in https://github.com/material-components/material-web/pull/4764 since it's faster than importing a contribution PR.
Hope we can get more PRs from you in the future!