Create ImageAcquirer interface and package

This will add the ImageAcquirer interface to factor out the acquiring of the image from the main package. This encapsulation will enable easier testing and simpler code, and also will make it easier to add more ImageAcquirer in the future if we will need to.
The code for acquiring an Image from Docker's daemon was moved to docker-image-acquirer.go and to get an image from a running Docker container is in docker-container-acquirer.go.
I also creates the simpler ScannerFactory interface that ImageInspector implementation defaultImageInspector will implement. It is added for easier testing of ImageInspector as we already have the Scanner interface.

cc @nimrodshn I will value your review as well

@simon3z This is my suggestion to refactor the acquireImage function [1]. It makes the addition of new runtimes much easier.
[1] https://github.com/openshift/image-inspector/pull/82#discussion_r163207801
cc @jhernand


Backwards installer paths?

[Enter feedback here]
Isn't this backwards? Program Files = 64-bit apps and Program Files (x86) = 32-bit apps
The Teams MSI places an installer in %SystemDrive%\Program Files\Teams Installer on 32-bit Windows and %SystemDrive%\Program Files (x86)\Teams Installer on 64-bit Windows.

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

ID: b2f71d9c-6b35-72c9-dd94-7f62319c1656
Version Independent ID: 86172986-6cc8-eee5-1309-704d9de0dcd0
Content: Bulk install Teams using Windows Installer (MSI) - Microsoft Teams
Content Source: Teams/msi-deployment.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

Hi @SchoolITPro, thank you for your feedback.

On Windows 10 32 bits you only have %SystemDrive%\Program Files
On Windows 10 64 bits or Windows 11 you have %SystemDrive%\Program Files and %SystemDrive%\Program Files (x86). It doesn't matter if you install Teams 32 or 64 bits, it will be installed on %SystemDrive%\Program Files (x86).

Hi @SchoolITPro Could you please confirm this answers your question? Thanks.

I understand the comment but the practice described doesn't seem correct either:

It doesn't matter if you install Teams 32 or 64 bits, it will be installed on %SystemDrive%\Program Files (x86).

Microsoft (in my experience) has historically directed its users to install 64-bit applications in Program Files and 32-bit applications in Program Files (x86). To have a Microsoft application that ignores this guidance seems to be antithetical to the Microsoft ethos.
On a 64-bit operating system, 64-bit Teams should install in Program Files while 32-bit Teams should install to Program Files (x86); this would help keep the lines of (versioning) distinction clear when troubleshooting issues.
On 32-bit systems, you only have one standard or choice, so it would obviously be the one to use.

Thanks again for your comment @SchoolITPro. Documentation reflects how the product currently works. I was about to update it by mistake, but a colleague helped me realize my mistake. No need to update the article.
I know the product teams are always looking for user feedback and how to make Microsoft products better. They keep a close tab on user feedback and it would be awesome if you could add your voice to the Microsoft Feedback Portal: https://feedbackportal.microsoft.com/feedback/.
You might find others have made a similar request that you can add to, or you can open a new one. Let us know here if you do so we can up-vote your request.
Thank you for your contribution to making the docs better! Much appreciated!

How right you are that the documentation is the primary source of the application's current state, and that it shouldn't be updated until the application is changed.
Now that I have verified the documentation is accurate to the operation I will offer my feedback in the Portal :-).


Activates for the riffmaster but the wammy controls the mouse cursor

Activates for the riffmaster (xbox version) but the wammy controls the mouse cursor which goes right to the far left of the screen. You can move the mouse by using tilt of the controller and the wammy sort of but its not usable.

This is not a capability RB4InstrumentMapper has; if you're running Steam, make sure you have Steam Input disabled, as this is what its default desktop configuration does.

Thank you!


[Bug] Powershell scripts do not works in beeftext

Expected behavior
powershell script works as described
Actual behavior
example snippet and script:
Hello #{powershell:C:\Temp\script.ps1}!
Write-Host -NoNewLine ([System.Environment]::UserName)   -   works fine in powershell
but do not work in beeftext. It only return string "Hello !"
second script - for beeftext version -  works the same way.
Doing I something wrong?
Thanks for Help.
Steps to reproduce the behavior
Beeftext version (e.g. Beeftext v7.0 Portable Edition).

Operating system version (e.g Windows 10)
win 10 22H2   |  build 19045.2486

Hi.
Can you please try again, then see if there is an error message in the log file (Advanced ➜ Open Log File) in the main menu.

LOG:   2023-02-19 21:44:50.979 - WARNING - Evaluation of #{powershell:} variable failed: execution of d:\Utility\BeeftextPortableEdition\scripts\script.ps1


Bump repo to forc v0.37.0

Motivation
Description
There has been a new release of forc.
Motivation
Bug fixes and blockers

Blocked by https://github.com/FuelLabs/sway/issues/4408


fix: Fixes Padding Space

Fixes Issue
Closes #10150
Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Unfortunately I will be closing this PR for the reasons mentioned by another contributor here.
I would suggest reading our Contributing Guide ahead of making any further contributions so that you do not waste time and work which is not used, and also to avoid extra notifications for maintainers and confusion to other contributors.


[Models] Support Qwen model with PP

Adds support for QWen 1 model. Tested locally using test_pipeline_parallel.py with 1.8B

/ready

@youkaichao PTAL!


Lib/fontmake/main.py tweak help strings

Prevent user confusion like #526
Please squash merge

Thanks :)
I think the confusion came from fontmake not following the usual conventions of taking loose arguments as input/output paths, but maybe it helps.


Encapsulate Data Store Summarization and Context

related #4420 , #4448 , #4460 , #4477
The final PR to encapsulate Data Stores, and not expose contexts to the container runtime. Again this is just copy paste of functionality to encapsulate Data Stores. There is more work necessary on Data Stores to finalize the api.

@arinwt and @agarwal-navin looks like you both have open PR that might affect this code. happy to wait for those changes to land first. this change is pretty trivial, as it just copies methods from container runtime to data stores

I don't have any open PRs that conflict with this change.

@arinwt and @agarwal-navin looks like you both have open PR that might affect this code. happy to wait for those changes to land first. this change is pretty trivial, as it just copies methods from container runtime to data stores

Merged mine in, thanks for checking!


Adds undo-redo feature to cb-whiteboard

Issue that this pull request solves
Closes: #260
Proposed changes
Undo Redo feature
Brief description of what is fixed or changed
Added Undo Redo Feature to cb-whiteboard.ts using another class StateManager that keeps a stack of states, I have also added buttons for the same and a document listener for ctrl + z (undo) and ctrl + y (redo)
Types of changes
Added Buttons
Added New Class within cb-whiteboard.ts
Made some changes in code.
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request

@Kajol-Kumari I have made necessary changes, kindly review them.


Add Angular bindings

Very much Work in Progress

Added the bindings under client/typescript/frameworks with own package.json and tsconfig.json to comply with the Angular Package Format: https://angular.io/guide/angular-package-format
Excluded the angular sources from tsup build because they require a different compilation - using the angular compiler and ng-packagr
Added example project under examples/web-wa-sqlite-angular
The goal is to have signal-based bindings for the electricClient and to have maybe a similar structure with the TanStack Query Angular adapter: https://tanstack.com/query/v5/docs/framework/angular/overview, https://github.com/TanStack/query/tree/main/packages/angular-query-experimental/src - although is not quite idiomatic for the Angular ecosystem (yet) to have custom inject functions (like injectLiveQuery or injectConnectivityState) they seem to fit well in the current structure and I think they would improve the developer experience. But please let me know what you think

👋 we've been working the last month on a rebuild of the Electric server over at a temporary repo https://github.com/electric-sql/electric-next/
You can read more about why we made the decision at https://next.electric-sql.com/about
We're really excited about all the new possibilities the new server brings and we hope you'll check it out soon and give us your feedback.
We're now moving the temporary repo back here. As part of that migration we're closing all the old issues and PRs. We really appreciate you taking the time to investigate and create this improvement!

thank you for the info, this looks very interesting. Unfortunately I didn't manage to progress more with this PR in the last months.. The new structure seems very clean and will definitely be more easy to add new client libraries, can't wait to dive into it


Inference on Point Clouds

Would there be a possibility to make this work on point cloud ? with inputs as x, y, z or even retrain it so that it works on point clouds to detect lines on it?
I tried to rasterize the point cloud data and try to detect lines, but that doesn't work that well

You might need to fine-tune it on point cloud data to make it really work, because the current data is probably to different from what it was trained on. But this might be possible with some effort, yes.
Otherwise, fitting lines to your point cloud by casting this as a multi-model fitting should also work.

thanks :)


Linker command failed with exit code 1 (use -v to see invocation)

Expected Behaviour
Build success
Actual Behaviour
Build fails
Reproduce Scenario (including but not limited to)
Cordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0
Ionic App Lib Version: 2.0.0
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: macOS Sierra
Node Version: v6.6.0
Xcode version: Xcode 8.0 Build version 8A218a

Steps to Reproduce

ionic build ios
Open "yourAppFolder"/platforms/ios/yourAppName.xcodeproj
Build

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
iOS 10.0.2
Cordova CLI version and cordova platform version
cordova 6.3.1
ios 4.2.1
Plugin version
phonegap-plugin-push 1.8.2
Logs taken while reproducing problem
ld: warning: object file (/Users/nunoarruda/repos/zee-p3-app/platforms/ios/zee-p3-app/Plugins/phonegap-plugin-push/GCM/Libraries/libProtocolBuffers.a(Descriptor.pb.o)) was built for newer iOS version (8.3) than being linked (8.0)

ld: warning: object file (/Users/nunoarruda/repos/zee-p3-app/platforms/ios/zee-p3-app/Plugins/phonegap-plugin-push/GCM/Libraries/libProtocolBuffers.a(ObjectivecDescriptor.pb.o)) was built for newer iOS version (8.3) than being linked (8.0)

ld: warning: object file (/Users/nunoarruda/repos/zee-p3-app/platforms/ios/zee-p3-app/Plugins/phonegap-plugin-push/GCM/Libraries/libProtocolBuffers.a(PBArray.o)) was built for newer iOS version (8.3) than being linked (8.0)

...

ld: 270 duplicate symbols for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@nunoarruda this seems like a duplicate of #1025.
Did you try removing and adding the ios platform?

@macdonst Yes, I've tried removing and adding the iOS platform, I've tried cordova clean ios and also removing the platforms and plugins folders and installing everything again. The problem persists.

@nunoarruda
I had the same probelm. In my case phonegap-plugin-push conflict with the other plugin cordova-plugin-fcm.

@carlosrodriguesjunior thanks for the tip, it was helpful. I've tested phonegap-plugin-push in a blank Ionic project and it worked. Tested it again in my project and it failed. It looks like it's conflicting with cordova-plugin-googleplus:

I have no alternative to cordova-plugin-googleplus so I have to keep using it. Will try OneSignal-Cordova-SDK instead of phonegap-plugin-push.

@nunoarruda ah, I see. Well once PR #1183 is merged the plugin will be using CocoaPods to pull in those dependencies. Then @EddyVerbruggen and I will be able to resolve the issue.

Waiting for https://github.com/EddyVerbruggen/cordova-plugin-googleplus/issues/272 to be fixed.

Hi, I have the same problem but I don't have 'cordova-plugin-googleplus' plugin installed in my APP:
cordova-plugin-compat 1.1.0 "Compat"
cordova-plugin-console 1.0.5 "Console"
cordova-plugin-contacts 2.2.1 "Contacts"
cordova-plugin-device 1.1.4 "Device"
cordova-plugin-dialogs 1.3.1 "Notification"
cordova-plugin-facebook4 1.7.4 "Facebook Connect"
cordova-plugin-file 4.3.1 "File"
cordova-plugin-file-transfer 1.6.1 "File Transfer"
cordova-plugin-globalization 1.0.5 "Globalization"
cordova-plugin-inappbrowser 1.6.1 "InAppBrowser"
cordova-plugin-network-information 1.3.1 "Network Information"
cordova-plugin-settings-hook 0.2.4 "Cordova Settings Hooks"
cordova-plugin-splashscreen 4.0.1 "Splashscreen"
cordova-plugin-statusbar 2.2.1 "StatusBar"
cordova-plugin-whitelist 1.3.1 "Whitelist"
cordova-plugin-x-socialsharing 5.1.3 "SocialSharing"
ionic-plugin-deploy 0.6.5 "IonicDeploy"
ionic-plugin-keyboard 2.2.1 "Keyboard"
phonegap-plugin-push 1.9.2 "PushPlugin"

How can I know plugin what causing me this build error?

same here but with this plugins:
"com.cmackay.plugins.googleanalytics"
"cordova-plugin-console"
"cordova-plugin-device"
"cordova-plugin-inappbrowser"
"cordova-plugin-ms-appinsights"
"cordoba-plugin-device"
"cordova-plugin-splashscreen"
"cordova-plugin-statusbar"
"cordova-plugin-whitelist"
"ionic-plugin-keyboard"
"phonegap-plugin-push"
`

Hi @adamgalmor ,
the solution for me was open the .xcworkspace file insted of .xcodeproj located inside my '/platforms/ios' folder.

@ramoncarreras thanks!!!

@nunoarruda I have the same problem.
@Mahluleli-Goodson XCode Workspaces were introduced in 2011. You are always supposed to use them over Projects when available but thanks to @ramoncarreras for commenting to remind folks.

@macdonst thanks for that

@ramoncarreras thanks!!!

Ey @ramoncarreras thanks for your help!!!

Thanks, a life saver! 👍

Thanks @ramoncarreras!! I can sleep now!

I used OneSignal-Cordova-SDK and had the same problem! Thanks @ramoncarreras, you saved my time! :)

@ramoncarreras Thanks! Perfect!

@ramoncarreras Awesome, thank you!

You're welcome @adamgalmor, @Mahluleli-Goodson, @russiann, @nailujpeloz, @wcustodio, @YaroslavG, @ibnclaudius and @speedtreammanga.

I just encountered this problem today and it was only fixed after removing cordova-plugin-console. Weird because everything was working fine last week.

@andrews05 Did you create your project recently? If so, can you confirm if you created it using<EMAIL_ADDRESS>If so, I'd recomend you read the to-be-released blog post on that new version here, to understand why a duplicate symbol error is related to the cordova-plugin-console plugin.

@andrews05  I see... Create a new issue with the full template report then, please.

Important (since cordova-ios 4.5.0):
Please note, that the cordova plugin cordova-plugin-console is now part of the cordova-ios platform. So if you're using<EMAIL_ADDRESS>or higher absolutely remove cordova-plugin-console!
cordova plugin rm cordova-plugin-console
This concerns plain cordova apps and ionic apps as well.
Further information: Console-Plugin on GitHub

thanks @mglmgl
it worked for me 👍

For me, I had to remove all plugins, then re-add them (and re-build the project) one-by-one to identify the conflicting library.

thanks @mglmgl
it worked for me 👍


onDraw

Thank you barteksc for the project.
if i want to click something that i have draw.
what do shall ?

If you are meaning can you click a url and open a page the answer is not supported yet.

thanks for your reply.
pdfView.onDraw(new OnDrawListener() {
public void onLayerDrawn(Canvas canvas, float pageWidth, float pageHeight, int displayedPage) {
if (displayedPage == 5) {
canvas.translate(pageWidth / 2 , pageHeight / 2);
Bitmap bitmap = BitmapFactory.decodeResource(getResources(), R.drawable.ic_launcher);
canvas.drawBitmap(bitmap, 0, 0, paint);
}
...        .load();
if i want to click this bitmap... i don't know how to listen this click event.

I would create a long click event as that event is not used where as click drag pinch are all used.

Are you trying to create multiple mini pages that you click to view the larger pdf?

If this is what you are trying to do simply put something over the top of pdfview and action on clicking that object personally i create a thumb nail and allow the clicking of the thumbnail which opens the main pdf, send page number to the pdfview object and it will open that specific page.

i think i got it now.
thank you, you've really helped me a lot.  :-)

If you need more detail get in touch


feat: create web pages with "About us" and "Cards"

Hey @vBrusentsov please improve your code and remove fixed width and height in css. You can use only max-width for the parent block to limit the width of content.

Thank you. I ended refactored code according to your advice

@vBrusentsov You have several unresolved issues. You can find them by scrolling to the top of this conversation or by clicking on the 'Files' tab. Look for the comment right under the line that needs fixing

@vBrusentsov please check unresolved conversations (comments) and update your homework


Add jakirkham and rerender (conda-smithy 0.9.0)

Add myself as a maintainer and re-render with the recent version of conda-smithy (0.9.0).

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

conda-build version issue on py34 Windows x64. :cry:

That's just this problem ( https://github.com/conda-forge/staged-recipes/issues/448 ). So, we can copy this line and add it here. Though we should probably add that to conda-smithy.

Thanks @ocefpaf for fixing this.

Rebased to clean out re-rendering changes as they are unneeded.


Make ingress/egress service names configurable

It currently looks like the service name that the ingress/engress proxies are looking for is derived from the arguments passed from to the container runtime. This yields the expectation that istio-ingress/istio-egress k8s services must exist. Can you either LMK how to configure these names or make them configurable?
https://github.com/istio/istio/blob/master/install/kubernetes/istio-auth-with-cluster-ca.yaml#L213
        args: ["proxy", "egress", "-v", "2"]

Those are just the arguments with which the proxy is started, to tell its role.
If you are asking about the istio-egress service name, it can be changed from the yaml files.
Is this an issue or a question? If question, I encourage you to post questions to istio-dev mailing list.

From the log files I see the following. I assume that the --service-cluster istio-proxy is the kubernetes service name it's expecting to see. Is that correct?
I0608 21:32:04.006228       1 watcher.go:154] Envoy command: [-c /etc/envoy/envoy-rev1.json --restart-epoch 1 --drain-time-s 2 --parent-shutdown-time-s 3 --service-cluster istio-proxy --service-node egress]

If I change the service name from isitio-egress to something else then I see this behavior when running through https://istio.io/docs/tasks/egress.html
root@sleep-1345314204-pq1r2:/# curl http://externalbin/headers
no healthy upstream

If I recreate the istio-egress service on kubernetes then things work again.
Happy to move this over to the istio-dev mailing list.

If you get no healthy upstream, it's a valid issue.
It is not related to "--service-cluster istio-proxy", this is an Envoy specific config. BTW, Envoy's cluster is a different concept than k8s cluster.
"--service-cluster
(optional) Defines the local service cluster name where Envoy is running"

Okay let me keep digging into my stack here to figure what's breaking

@lachie83: You can set the egress address via the egressProxyAddress field in the service mesh: https://istio.io/docs/reference/config/service-mesh.html
The name of the ingress service in Kubernetes should not matter for the rest of the Istio components.

closing this but please reopen (in https://github.com/istio/issues/issues ) if needed


.NET client port

I started porting the client to .NET. I mostly needed LogBuffers for IPC - it was too good to invent my own :)  LogBuffers already work in my Spreads library. I will eventually port the remaining parts of the client, but currently do not need it and won't touch the remaining parts this month. If someone is already working on open-source .NET client port, please let me know. Or please join or fork this project: https://github.com/Spreads/Aeron.NET.

Hi @buybackoff. Interesting timing. There is an official port of the Aeron client in .NET that will be released in the very near future. This will be a full client. Might be interesting to see if it works for your needs.

Cool! I wasn't going to proceed with the full client soon, so this is a good news! Since my LogBuffers implementation is binary compatible, it will be easy to switch. BTW, how much is "very near future"?

Weeks, possibly a month at the outside.

Timing indeed was interesting :) .NET version is now on par with Java in this PR https://github.com/AdaptiveConsulting/Aeron.NET/pull/37


Add dynamic prices

I am getting every day the price for my power for the next 24h.
This means a constant input for the price is not possible.
Would you update an input number evey hour or do you think that a direct integration would be possible.
I am using https://github.com/mampfes/ha_epex_spot/ for the prices.

Prices comes from input_number and are read at each cycle. So it is already available. Do I miss something?

Understood. I thought that you do planning for the future. Then you would need the future prices.
Thx for the clarification.

Hi @jmcollin78,
input_number is okay for static prices, I've a tibber integration that stores the actual value (per hour) in a sensor: sensor.electricity_price_street_name so would be great if I could use this directly. Or should I need to copy that value to a custom input_number helper?
Kind regards,
Andree

If I open sensor entity type, people will configure with wrong sensor type. I prefer input_number for which I'm sure to have a number.
But this can be challenged and tested with a PR.


Can't use static values in join condition

How do we write a query like below in json-sql:
select * from TableA as a inner join TableB as b on a.id = b.id and b.name = 'some value'.

var result = jsonSql.build({
  table: 'TableA',
  join: [{
    type: 'inner',
    table: 'TableB',
    on: {
      'TableA.id': 'TableB.id',
      'TableB.name': {$value: 'some value'}
    }
  }]
});
// select * from "TableA" inner join "TableB" on "TableA"."id" = "TableB"."id" and "TableB"."name" = $p1;

But I think it is better to use condition:
var result = jsonSql.build({
  table: 'TableA',
  join: [{
    type: 'inner',
    table: 'TableB',
    on: {
      'TableA.id': 'TableB.id'
    }
  }],
  condition: {
    'TableB.name': 'some value'
  }
});
// select * from "TableA" inner join "TableB" on "TableA"."id" = "TableB"."id" where "TableB"."name" = $p1;

I expected the first solution to work and was trying same, but it didn't work.
I later figured out the solution if anyone wan't to do the same:
var result = jsonSql.build({
  table: 'TableA',
  join: [{
    type: 'inner',
    table: 'TableB',
    on: {
      'TableA.id': 'TableB.id',
      'TableB.name': { $eq: { value: 'some value' } },
    }
  }],
  condition: {
    'TableB.name': 'some value'
  }
});

My examples work well, I checked them in nodejs shell, results was taken there.


pre-commit support

What?
Added a hooks.yaml file so ansible-lint can be easily used with pre-commit
Why?
Pre-commit is a great tool used to more easily define git pre-commit hooks.  There are many supported hooks, but ansible-lint is not currently one of them.  This PR will change that.
When this is merged, I will open a PR to add this to the pre-commit supported hooks page.

If you like hooks with ansible-lint, you'll love hooks with ansible-review
ansible-review is far better designed for this kind of thing, but I'm not averse to adding this useful configurable tool to ansible-lint


Update the --openjdk_testrepo arg name in runAqaArgParse.py

Describe the bug
In the runAqaArgParse.py file, the --openjdk_testrepo arg name should be updated to --aqa-testrepo
Expected behavior
The run-aqa should later take in --aqa-testrepo arg

Hi, I was investigating the effect of the one-liner in #298 via the  similar workflow

I checked again to make sure the checked out repo is the intended one. However, it still would not bring the correct output. As suggested from last meeting, shall we try merging this one-liner?


Service fabric down time

Is there any downtime associated with scaling down no of nodes from 10 to 5? Also what about changing the vm size?

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

ID: 71869d24-fe49-3b0f-b096-72d0bf515905
Version Independent ID: 9db6765c-b334-5ea5-13e7-aff0c59f4e58
Content: Common questions about Microsoft Azure Service Fabric - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-common-questions.md
Service: service-fabric
GitHub Login: @athinanthny
Microsoft Alias: pepogors

@anoopasokan Thanks for the feedback! We are currently investigating and will update you shortly.

I saw that you have support case open for your question. I would let the support engineer continue with the backend investigation on your follow up questions.
Posting the answer for other customers, who may have the same question:
In general for scale in / out operations like adding and deleting the node you will not observe the downtime. You have to make sure you have the minimum nodes / VM required by your cluster for your durability and reliability requirement.
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-cluster-scale-up-down
Additonally to change the size of the VMs or operationg system which is scale up /down operation you may observe a small downtime  : refer to https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-scale-up-node-type while load blancer is been configured.
For scale in operation to bring down the nodes you have to first change the Reliability of the clsuter from Platinum to Silver, once that is done you have 5 seed nodes running instead of 9 you can scale in.

We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


vale2 update

Pull Request Description

Update submodule and config to use new Vale2 guidelines checking tool.
fix errors raised by this new version

Fixed Issue(s)

see PegaSysEng/Misc#29

 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.


soc: arm: nordic_nrf: Remove duplicates from Kconfig.soc

Removing duplicates in selects for config SOC_NRF52832 and
SOC_NRF52840. Duplicates were unexpected merge artifacts.
Signed-off-by: Krzysztof Chruscinski<EMAIL_ADDRESS>

Codecov Report

Merging #12157 into topic-counters will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           topic-counters   #12157   +/-   ##
===============================================
  Coverage           48.31%   48.31%
===============================================
  Files                 280      280
  Lines               43348    43348
  Branches            10380    10380
===============================================
  Hits                20945    20945
  Misses              18251    18251
  Partials             4152     4152

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 215f872...7745043. Read the comment docs.


Use ebs as default ephemeral disk backing

adjust ephemeral_disk description
add use_instance_storage description

Signed-off-by: Felix Riegger<EMAIL_ADDRESS>

Also contains documentation for:
https://www.pivotaltracker.com/story/show/119545877


Correct dimmer lights not be interpreted as  an ON/OFF light

As described in https://github.com/gicamm/homeassistant-comelit/issues/86
my dimmers were interpreted as ON/OFF lights due to the fact that the variable passed is a value instead of a set. This makes supported_color_modes incorrect.

Thank you for your contribution


README.md: fix link to oci runtime-spec

Signed-off-by: Lei Jitang<EMAIL_ADDRESS>

Dup of #379.


PathNotFoundException: MaterialIcons-Regular.otf

Steps to reproduce

Create app
Run the app on web

Expected results
Expected to run the App
Actual results
Flutter crash report.
Please report a bug at https://github.com/flutter/flutter/issues.
command
flutter run
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
PathNotFoundException: PathNotFoundException: Cannot open file, path = '/Users/arditneziri/Developer/flutter/bin/cache/artifacts/material_fonts/MaterialIcons-Regular.otf' (OS Error: No such file or directory, errno = 2)

#0      _File.throwIfError (dart:io/file_impl.dart:675:7)
#1      _File.openSync (dart:io/file_impl.dart:490:5)
#2      ForwardingFile.openSync (package:file/src/forwarding/forwarding_file.dart:74:16)
#3      ErrorHandlingFile.copySync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:319:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:595:14)
#5      ErrorHandlingFile.copySync (package:flutter_tools/src/base/error_handling_io.dart:318:5)
#6      writeBundle.<anonymous closure> (package:flutter_tools/src/bundle_builder.dart:230:19)
<asynchronous suspension>
#7      Future.wait.<anonymous closure> (dart:async/future.dart:524:21)
<asynchronous suspension>
#8      writeBundle (package:flutter_tools/src/bundle_builder.dart:185:3)
<asynchronous suspension>
#9      WebDevFS.update (package:flutter_tools/src/isolated/devfs_web.dart:991:9)
<asynchronous suspension>
#10     ResidentWebRunner._updateDevFS (package:flutter_tools/src/isolated/resident_web_runner.dart:562:35)
<asynchronous suspension>
#11     ResidentWebRunner.run.<anonymous closure> (package:flutter_tools/src/isolated/resident_web_runner.dart:322:41)
<asynchronous suspension>
#12     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:24)
<asynchronous suspension>

Flutter Doctor output
Doctor output

[✓] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale de-DE)
    • Flutter version 3.22.3 on channel stable at
      /Users/arditneziri/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (5 weeks ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/arditneziri/Library/Android/sdk
    • Platform android-33, build-tools 30.0.3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

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

[✓] Android Studio (version 2021.2)
    • 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.12+0-b1504.28-7817840)

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

[✓] Connected device (3 available)
    • macOS (desktop)                 • macos                 • darwin-arm64   •
      macOS 14.5 23F79 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad • darwin         •
      macOS 14.5 23F79 darwin-arm64
    • Chrome (web)                    • chrome                • web-javascript •
      Google Chrome 127.0.6533.120

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @arditneziri93
Does the issue persist if you run app again? Can you please try upgrading Flutter SDK to the latest stable version 3.24.0 and retry?

Hi
I did it and it was still missing. I had to download the
Material-Icons-Regular.utf and place it manually in the folder so that it
worls again. This happened after I installed the latest flutter version.
Am Di., 20. Aug. 2024 um 07:25 Uhr schrieb Huy @.***>:

Hi @arditneziri93 https://github.com/arditneziri93
Does the issue persist if you run app again? Can you please try upgrading
Flutter SDK to the latest stable version 3.24.0 and retry?
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/153680#issuecomment-2297992529,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJP2IVHWKEBRFW5HR265KULZSLHLBAVCNFSM6AAAAABMXKAE5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJXHE4TENJSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think that file is somehow missing or corrupted while downloading Flutter SDK. I will mark this as solved. If you experience it again and can reproduce it consistently, please write in comments or file a new issue. Thanks!


Failed to publish to Firefox with some lint error

Error: There is a JavaScript syntax error in your code, which might be related to some experimental JavaScript features that aren't an official part of the language specification and therefore not supported yet. The validation cannot continue on this file.
It is verified by https://github.com/mozilla/addons-linter/ lint when you want to publish extension

I see from the issues in the Mozilla linter project that modules are not supported yet. I do have plans to concatenate the scripts and not use modules so it seems I have to bring forward this plan.

Thanks to your nice work, everything works well now.

I see from the issues in the Mozilla linter project that modules are not supported yet. I do have plans to concatenate the scripts and not use modules so it seems I have to bring forward this plan.

FWIW, the issue linked above is for internal addons-linter stuff, not for supported features in web extensions. I think modules are already supported by FF and the linter also allows to use modules.

But anyway, at the time, I actually received this error report when I was uploading the addons. It prevented me from making further submissions.
Currently, after I use the latest version of BlazorExtensions, these errors are gone.
I have now also successfully upload the addons.
https://addons.mozilla.org/zh-CN/firefox/addon/amazing-favorites/
Maybe there is a better way to fix the linter or this project, but I don`t know.


Where is the rest of the material struct bound?

I looked at the MiniEngine app. It provides a good introduction to DX12. Very cool ;-)
But it is not entirely clear to me where the rest of the material struct (vectors of ambient and emissive colors, floats like opacity, shininess etc.) is bound to the shaders. I can only see the textures as SRV's in the Model.h/ModelH3D.cpp.

Hi, I'm glad you like MiniEngine!  I apologize if ModelViewer's name is a bit misleading.  It's not a very general purpose model viewer.  It might be better called the SponzaViewer, because it's tailored to that specific scene.  The shaders we render with are designed around the data we found in the Sponza OBJ file.  We don't have a more generalized scene representation or a variety of materials to go with it.  But that shouldn't stop you from being able to write your own shaders and constant buffers to define your own materials with variable parameters.  So far, we've spent most of our time creating widely applicable and generally reusable code.  When it comes to defining your particular rendering style and materials, that's where people often go with something custom.  Cheers!

Okie Dokie. Thanks for your explanation. I was just not sure whether I overlooked something when I have read the code.


Indicator Board with LEDs

Description
Adds indicator board with power, cycle controller red/yellow/green, UI computer red/yellow/green LEDs.  See #890 for dimensions: looks wide enough to fit next to screen in front acrylic panel but I don't have the hard drive space/Inventor experience to go in and check this myself.
Closes #890

Could you give me dimensions of the board in writing, so that I can confirm its fit?
You might as well export a Step file, also to the GDrive folder.

Oops forgot about the manufacturing files going to GDrive instead (hadn't thought of them as particularly big).  Will move.  STEP is a good idea once we settle on an outline.

Electrical Design looks good.
Layout: would it be better visually to put each color of LEDs next to one another?, i.e.
RR
YY
GG

instead of
R
Y
G
R
Y
G

I guess that would be very marginally better - something like this then:

Ah, you beat me, I just made a new branch with to do the same:

Yeah, @dcstraney that looks good, I wasn't able to push my branch anyways due to an LFS problem.  Let's go with what you updated.

I'll pull the latest from you and review

You've got a wire break just north of D3, check your DRCs., otherwise everything else passes.

Strange, I'd just hit the DRC - but that was after tweaking positions again here and there, so must've fixed that in the process of shifting things around a little.  Checks out fine now, will check it in.


Bypass not working on megaup.net (NSFW)

Domain
https://megaup.net
Link
Example link: https://megaup.net/1TAzt/The_bot.png but when i wait: https://download.megaup.net/?idurl=0lixLaV5L89JKamcHXUbeswkcyY8BAN%2BwO2I0DGIPkOSdArt%2Bnk0DNkgT1vgbt%2BolX%2FcKYgLNoasii3YQX0Oq4zGpDA0iZ%2BnmmQzQaGZYPk%3D&idfilename=The+bot.png&idfilesize=35.00+KB
Version
Last
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
Example link: https://megaup.net/1TAzt/The_bot.png
but when i wait: https://download.megaup.net/?idurl=0lixLaV5L89JKamcHXUbeswkcyY8BAN%2BwO2I0DGIPkOSdArt%2Bnk0DNkgT1vgbt%2BolX%2FcKYgLNoasii3YQX0Oq4zGpDA0iZ%2BnmmQzQaGZYPk%3D&idfilename=The+bot.png&idfilesize=35.00+KB

according to the bypass checker website, megaup is not supported
its safe to close the issue

Search existing issues before opening new ones
Duplicate of #965


about Updating an endorsement policy problems

I don’t know if it’s a bug to update the endorsement strategy. Personally, I think the execution logic is a bug.
peer lifecycle chaincode approveformyorg \
--signature-policy "AND ('Org1MSP.peer','Org2MSP.peer')" \
-o localhost:7050 \
--ordererTLSHostnameOverride orderer.example.com \
--channelID mychannel \
--name fabcar \
--version 1.0 \
--package-id $CC_PACKAGE_ID \
--sequence 1 \
--tls \
--cafile ${PWD}/organizations/ordererOrganizations/example.com/orderers/orderer.example.com/msp/tlscacerts/tlsca.example.com-cert.pem

Output log

No error
2021-12-27 06:14:22.983 UTC [chaincodeCmd] ClientWait -> INFO 001 txid [026dc39d7a11a849c0dd656f5bdb96e1900b2a8e6ea7306e4e334d13b5d64b92] committed with status (VALID) at

But when I checked the approval status, I found that the organization did not approve
peer lifecycle chaincode checkcommitreadiness  \
--channelID mychannel  \
--name fabcar  \
--version 1.0  \
--sequence 2  \
--tls  \
--cafile ${PWD}/organizations/ordererOrganizations/example.com/orderers/orderer.example.com/msp/tlscacerts/tlsca.example.com-cert.pem  \
--output json

However, when I see that I also need to add that option when executing the commit command, I have also had questions, is this not necessary? Setting chaincode-level endorsement policies，Isn't it okay to do it via the '—sequence' option?  I still think it's a logical bug！！！

I have solved the above problem, the solution is as above.

The approveformyorg and commit parameters must match exactly. Looks like this is working as intended.


cep-002: Revised Event Structure (Acceptance Review)

Please use this PR to comment on a  proposed ctapipe enhancement proposal  (CEP):  CEP-002: change the internal event structure.
Recall that ctapipe releases before 1.0.0 are subject to small and large changes in the API, and this is one such large change that has been discussed for several years now.  Note that this affects any code that uses ctapipe, and therefore adopting it will require some changes to local code and scripts.  This CEP has already been internally discussed in #2304, and is now open for public comment.
Please read the proposal for CEP-002 in the ctapipe docs and give constructive comments here if anything is ambiguous or missing.
Details about what is a CEP and how we incorporate them can be found in the documentation as CEP-001

@kosack, you should also change the status in the header of the file itself

Docs failure is fixed in main, please rebase

Can you please clarify how the events will be indexed and connected?
In the structure below, are the SubarrayEventIndex and TelescopeEventIndex independent? If yes, how the telescope-specific data shall be gathered to form a Subarray event, through tel:Map[...]?
SubarrayEvent
- index: SubarrayEventIndex
- simulation: SimulatedShower
- dl0: DL0Subarray
- dl1: DL1Subarray
- dl2: DL2Subarray
- dl3: DL3Subarray
- tel: Map[tel_id -> TelescopeEvent]

TelescopeEvent
- index: TelescopeEventIndex
- simulation: TelescopeSimulation
- r0: R0Telescope
- r1: R1Telescope
- dl0: DL0Telescope
- dl1: DL1Telescope
- dl2: DL2Telescope
- dl3: DL3Telescope

Also, with the new structure, will ctapipe-merge tool support merging of DL1 events (especially DL1a, required for Deep Learning models)?

In the structure below, are the SubarrayEventIndex and TelescopeEventIndex independent?

I don't understand whay you mean with "independent" here.
These two containers contain the "primary key"
identifying the subarray or telescope event respectively, i.e. obs_id / event_id for SubarrayIndexContainer and obs_id, event_id, tel_id for TelescopeIndexContainer.

If yes, how the telescope-specific data shall be gathered to form a Subarray event, through tel:Map[...]?

The key of .tel is the telescope id, as before and written in the snippet you posted.

Also, with the new structure, will ctapipe-merge tool support merging of DL1 events (especially DL1a, required for Deep Learning models)?

This CEP is concerned with the in-memory structure of the data (ctapipe containers), not the file structure.
So your question is independent of this CEP, but yes, we do plan to implement merging files per multiple telescopes into one file for the whole subarray, as this will also be how ACADA will take actual CTA data.

If we want to merge this now, @kosack needs to remove the "draft" version of it we only kept so that the diff here renders nicely and not just "file renamed without changes".

As discussed in today's meeting, I removed the CEP 3 from this PR and will open an individual PR for this.
So I think this is ready to be merged to formalize the acceptance.


Expose Navigator share method

Please list the related package(s)

checkout-ui-extensions
checkout-ui-extensions-react

If this related to specific APIs or components, please list them here
N/A

Is your feature request related to a problem? Please describe.
Our org is currently migrating to checkout extensibility and one of the features we're trying to replicate is the ability to share content using the Navigator interface built into web browsers. I understand extensions run on workers so we're very limited in what we can access. Still, it would be great to be able to have access to a component or API that allows us to open the share dialog on mobile devices. If not the Navigator share method, it would be nice to have a way to use share buttons that can open native apps on both iOS and Android.

Describe the changes you are looking for

I am looking for a way for users to share text and images via social media using browser-provided APIs such as the Navigator interface.
Describe alternatives you’ve considered

I considered using iOS universal links but that seems pretty limited as far as sharing content goes
Additional context

We would strongly benefit from this too - and would appreciate it.


[Add]: ISSUE TEMPLATE FOR BUG 🪲

Details

As part of our efforts to streamline the bug reporting process and ensure clear and concise bug reports, we propose adding a standardized "Bug Report" template for GitHub issues.
Currently the Issue template of this repository is not clear to manage these types of details in it.
This template will provide a structured format for users to report bugs they encounter while using our project.
The bug report template will include sections such as "Bug Description," "Steps to Reproduce," "Expected Behavior," "Screenshots (if applicable)," "Environment (if applicable)," and "Additional Context."
By using this template, we can gather consistent and comprehensive bug reports, enabling our team to identify and resolve issues more efficiently.
The template will encourage users to provide essential details, making it easier to reproduce bugs and find appropriate solutions.

Screenshots

Type of Contribution

[ ] Update to an existing Animated Button
[ ] Adding a new Button
[ ] Resolving a bug
[X] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Hey there @Souravvmishra 👋🏻
Please make sure to assign this issue to me to make the work done, Thank you!

@Spyware007


Volume Increase Or Decrease

There Is a Volume Button  By That Button We Just Mute🔇 Or Unmute🔊 Volume Can We Add Volume Increase🔈🔉🔊 Or Decrease🔊🔉🔈 Slider Button To It
I Think It Is Very Common Function That Users Can Use Or Users What That Button Present On Website
-Thank You🙏

I added it into my backlog to investigate. Personally, I have mixed feelings on per video volume adjustment beyond mute as users can get into a funny 'nesting' of volumes: system level, app level, per video level, but I'll look into how I could expose that in the UI.

@ashishsiot check out the latest v2.1 release. I added in some basic volume control. https://github.com/pseudosavant/player.html/releases/tag/2.1.0


zfs: strip debug symbols (closure size)

Since we enabled BTF the zfs module size became significantly larger due to included debug symbols.
Closure size changes on my system when upgrading:
initrd-linux: -12427.8 KiB
linux: -12419.3 KiB
nc2k9ym14spzz2pgq6hn84k8k9zgf686: ∅ → ε, +102451.2 KiB
nixos-system-turingmachine: 21.11.20211015.a2dcfa2 → 21.11.20211017.c6f77b7
visjwxsjqmh8c9lxnrxsw5w19zszflm6: ε → ∅, -102451.0 KiB
zfs-kernel: -12419.3 KiB

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

How come the normal strip hook wasn't catching this?

Oh, I guess because it gets compressed?


not load this fonts

Package
All packages
Existing issue?

[X] I checked the existing issues

What happened?
Expected vs actual result:
Steps to reproduce:

not load fonts in my code
Relevant log output
----------------FIREBASE CRASHLYTICS----------------
I/flutter (11931): Exception: Failed to load font with url https://fonts.gstatic.com/s/a/a05eb17c43309b14b916303c48995b19407a7cdcf47bc6d8085d464722627918.ttf: Failed host lookup: 'fonts.gstatic.com'
I/flutter (11931):
I/flutter (11931): #0      _httpFetchFontAndSaveToDevice (package:google_fonts/src/google_fonts_base.dart:256:5)
I/flutter (11931): <asynchronous suspension>
I/flutter (11931): #1      loadFontIfNecessary (package:google_fonts/src/google_fonts_base.dart:173:11)
I/flutter (11931): <asynchronous suspension>
I/flutter (11931): ----------------------------------------------------
I/flutter (11931): Error: google_fonts was unable to load font Poppins-SemiBold because the following exception occurred:
I/flutter (11931): Exception: Failed to load font with url https://fonts.gstatic.com/s/a/a24a61e9a408f85504dcdcd11edc4995adceb4ab585c0011f39cfbe193248b71.ttf: Failed host lookup: 'fonts.gstatic.com'
I/flutter (11931):
I/flutter (11931): See https://docs.flutter.dev/development/data-and-backend/networking#platform-notes.
I/flutter (11931): If troubleshooting doesn't solve the problem, please file an issue at https://github.com/material-foundation/flutter-packages/issues/new/choose.

I have the same problem, this don't doesn't load


Mark go.sum as generated

Description

Checklist:

[x] I am adding new or changing current functionality

[x] I have added or updated the tests for the new or changed functionality.

This has been attempted and rejected before. See https://github.com/github-linguist/linguist/pull/4660


Any Switch handling of empty text values

First, thank you for this amazing set of nodes. I'm not exaggerating when I say that the Any Switch is the backbone of my most powerful workflows. It's deceptively simple and intuitive, but it makes creating complex workflows feel like a breeze -- even more so when combined with nodes such as Anything Everywhere.
I noticed that the Any Switch currently considers empty strings to be non-empty values:

In this simple workflow, I would expect "pineapple" to be displayed, since it's the first non-empty value.
The current behavior seems to run counter to the spirit of the node. It's easy enough to change (by adding a custom comparator for strings here), but I wonder if that was an oversight, or if it was by design?

Oh, I also meant to mention: With the recent commit https://github.com/rgthree/rgthree-comfy/commit/45781c3dbbe921847992084acc196d09d664901a, Context Switch nodes now support a dynamic number of inputs. That's a nice upgrade!
This would also be a very useful feature for the Any Switch node. I think that's the only way the current node can still be improved, if it started with just two inputs and then added/removed inputs dynamically as needed.

Hmmm. I'm honestly a little conflicted. The Any Switch was really built to be used with muting. So, if you didn't want an empty string to be chosen, you would mute that node.
It is true that I added support for an empty Context being skipped if passed in (at least for the Context Switch, not sure if it works that way for the Any Switch if using Contexts). But I worry a bit about changing the behavior here, especially now that it's been out for so long...
While empty string may seem to make sense, I could see someone who uses an empty string as the last value so it always passes a valid string if others a muted. I could also think this opens the question up to empty lists being ignored, or a 0 number, etc.
For now, I think I'm erring on the side of not changing the existing behavior and having it rely on muting as the primary switching.

Separately, dynamic inputs for the any switch should be easy. Good idea

8c92a8a5d8352156b671f1c684b5d51a0b9f646a adds support for dynamic number of inputs to Any Switch!

Hmmm. I'm honestly a little conflicted. The Any Switch was really built to be used with muting. So, if you didn't want an empty string to be chosen, you would mute that node.
It is true that I added support for an empty Context being skipped if passed in (at least for the Context Switch, not sure if it works that way for the Any Switch if using Contexts). But I worry a bit about changing the behavior here, especially now that it's been out for so long...
While empty string may seem to make sense, I could see someone who uses an empty string as the last value so it always passes a valid string if others a muted. I could also think this opens the question up to empty lists being ignored, or a 0 number, etc.
For now, I think I'm erring on the side of not changing the existing behavior and having it rely on muting as the primary switching.

That makes sense and it's probably what I would do as well, considering the node has been out for a while and this change would break backwards compatibility.
Thank you for implementing dynamic number of inputs!

I think this is worth documenting here, in case it's useful to someone in the future:
With the dynamic number of inputs feature, the Any Switch node now prunes its inputs whenever any connections are changed. When determining whether an input should be pruned, the node checks if that input has a link: https://github.com/rgthree/rgthree-comfy/blob/8c92a8a5d8352156b671f1c684b5d51a0b9f646a/src_web/comfyui/utils_inputs_outputs.ts#L6
If like me, you use the Any Switch extensively with "virtual" links such as those created by Anything Everywhere, that will cause those inputs to be dropped. You'll need to add an extra condition or two at the line above.


fix(semantic): correctly resolve identifiers inside parameter initializers

close: #2644
This fixes function parameters. I think we need an extra AST node to fix catch parameters, which will be the next PR.

#3046  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite

Merge activity

Apr 21, 11:38 AM EDT: @Boshen merged this pull request with Graphite.


Added User-List

I'm leaving the button-color up to you :)
If you want anything changed, just address it :+1:

@JohnnyCrazy Please can you copy your css changes into /resouces/scss/main.scss so they can compile into the .min.css - thanks!

@JohnnyCrazy I've pulled out your css changes into scss now, don't worry about that last request. Will tweak the UI a little before this goes live - the profile buttons don't work for FB logins, and the large button is a bit big so will refine and push live in next few days.  Thanks again!

This looks awesome!!!

OK that's now live - without the profile button for now because it doesn't work with FB spotify logins.  https://db.tt/BHg3YEYO  Thanks agains Mr Crazy


lib/config/uglifyjs is missing in 3.1.0

Release of 3.1.0 broke our build at Swagger Editor. Apparently lib/config/uglifyjs' is missing.

+1
Having the same issue.

+1
Having same issue.

@sindresorhus @arthurvr we have a breaking changing here that we need to fix.
3.1.0 ships a breaking change where uglify flow step was renamed from uglifyjs to uglify for name consistency to fix an issue (#409), which give us :

lib/task/uglify
uglify flow step

which is the same as in grunt-contrib-uglify, which in fact depends on ... uglify-js
How guys do you think we should resolve this issue ? Breaking change is really bad but we might have others issues about naming consistency.
At least we can revert change from this commmit and publish a 3.2.0 version.
Another way is to support both name with a deprecation notice to encourage people to use the name uglify and not uglifyjs.
I would prefer a notice in README and a fix to support both name with a deprecation notice.

@sindresorhus @arthurvr I'm working on a fix that would support both naming. It should not be that hard.

Message added to README.

If you revert the commit and publish 3.2.0 will uglify still work? I've changed from uglifyjs to uglify as above. Will I have to change back again?!

As an additionally note, should this have been published as a major version update?

+1

This should have been published as 4.0.0. Putting this change in 3.1.0 violates semantic versioning.

@hristozov you're right but as we are missing maintainers this change was not seen as a breaking change (my bad).
I will publish a patch probably this evening (Europe) to support both 'uglify' and 'uglifyjs'. Need to fix some tests.
Would you agree with a 3.1.1 ? It would help anyone having this issue to update to latest.

@paulclarkin no you would not have to change anything with my patch. It would support both.
We need to help you guys and still be consistent with the name which should be uglify as grunt-contrib-uglify.

Look for uglifyjs in the Gruntfile.js and renaming it to uglify worked for me.

@rafialikhan you're right.
Just to let you know the patch will be released today.

@sindresorhus @arthurvr I don't have the right to publish the new version on npm.

@stephanebachelier can you re-open this issue until the new version has been published to npm?

Done.


Other programs except osu!

I really like to use my touchpad in absolute mode with other programs than osu!
Using it inside the browser for example with different drawing apps would be huge for me.
Are there any plans to implement such a thing in the future or are there other possible pathways to achieve something like that on windows or linux?

I tried this with two approaches: AbsoluteTouch and Pad2Screen. AbsoluteTouch had some problems with performance and didn't work well in games, but if you have a Synaptics touchpad, I think it does fit your usecase. Pad2Screen was a Windows kernel driver that would basically turn your touchpad into a touchscreen system-wide, the only problem is that I never actually got it to work since drivers are fickle things.
AbsoluteTouchEx technically works on things other than osu!, but it requires that the program use raw input, which is primarily used by games. I don't think browsers will ever support it, so chances are slim that this will ever happen with AbsoluteTouchEx :-(

AbsoluteTouch.exe worked (I have a Synaptics touchpad), Thank you Andrew.
I didn't realize how hard it is to draw in absolute mode, when I'm just hovering with the cursor there is no way to click&hold other than double tapping with the pen and losing the previous position, because it's hard to hit the same spot.
I'm using my second hand for other controls, otherwise I would use the seperate buttons on my touchpad for right and left click.

how do I use is on Osu! everytime i press shift + f6 my cursor just wont move.... I have synaptics touchpad though i installed different precision driver it works without issue its just that shift +f6 on osu! wont make my cusor move... I get it that i wont move when in calibration mode but when i exit calibration and turn on absolute touch i cant move it... help


[AutoPR cognitiveservices/data-plane/Personalizer] Add missing cognitive services personalizer readmes

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6132

(message created by the CI based on PR content)


new resume doesn't create anything

I've just installed RR on Ubuntu 20.04 running Nginx. I've built the app successfully, and can load the front page, but, whenever I try to create a new resume, it doesn't create anything and sends me back to the create new resume page.
Am I missing something
THank you

nevermind...found the issue


Hyperlinks

Applying a link to text in the article.
User Stories
Author

As an author, I want to be able to make text into a hyperlink so that I can link out to an external URL.
As an author, I want to be able to see if text is a hyperlink so that I can check this is correct.
As an author, I want to be able to follow a hyperlink out to the target so that I can check the link is going to the right place.
As an author, I want to be able to remove a hyperlink from the text in the article so that I can make any necessary adjustments before publishing.
As an author, I want to be able to edit the target for a hyperlink so that I can correct any errors.

Production staff

But what if . . . ?

Considerations

XML requirements
eLife uses the element ext-link
<p>... a freely available open-source software (<ext-link ext-link-type="uri" xlink:href="https://gitlab.com/tjian-darzacq-lab/spot-on-matlab">https://gitlab.com/tjian-darzacq-lab/spot-on-matlab</ext-link>) based on a model previously introduced in ... </p>

eLife allows ext-link as a child of the following (not including references):

license-p
p
source
attrib
td
th
title

In addition SciELO allows ext-link as a child of the following (not including references):

comment
product

Taken from: https://github.com/elifesciences/TextureRequirements/issues/86

This functionality is already present in Texture but may need some refinement based on usability testing

We may want to follow a more standard pattern where the url input is displayed upon clicking the link icon in the toolbar (see google docs as an example)

Links currently look exactly the same as underlined text and impossible to tell apart at present


"Unauthorized use of Copyrighted Content" fixed

Though, keep in mind this is a temporary work around, and therefore Musescore might break it instantly when they get the chance to.

Previously tried to do a pull request, but it messed up the line breaks (https://github.com/Xmader/musescore-downloader/pull/49)
So this is a pull request with the corrected line breaks, and my change to the script.

Also I crudely used an alert to address the issue, but it might be a better idea to make some sort of floating countdown above the download area (not inside of the div though, since Musescore is using mutation detections specifically in the area where the buttons are being dynamically added.

Even if this doesn't get merged, as I see a better fix being worked on by the dev, this still serves at least a temporary purpose for those that need a quick fix.

I'll go ahead and close this now, since the issue if fixed with the latest version.


Configure the STAC Browser to use bearer type authentication

part of https://github.com/unity-sds/unity-project-management/issues/117

Thanks @rtapella for creating this ticket.  @ngachung can you please assign this to a sprint so that we can track it on our project board?

Done. Associated PR: https://github.com/unity-sds/unity-data-services/pull/339


Calico failed to parse Node PodCIDR when using host-ipam

I updated Calico from 3.13.0 to 3.15.0 and now Felix is having problems parsing the Node PodCIDR. I'm using the host-ipam CNI plugin.
The logs show the following error:
2020-07-09 08:43:06.754 [INFO][8] startup/startup.go 299: Early log level set to info
2020-07-09 08:43:06.755 [INFO][8] startup/startup.go 315: Using NODENAME environment for node name
2020-07-09 08:43:06.755 [INFO][8] startup/startup.go 327: Determined node name: app-kubema-vc1.example.com
2020-07-09 08:43:06.756 [INFO][8] startup/startup.go 359: Checking datastore connection
2020-07-09 08:43:06.782 [INFO][8] startup/startup.go 383: Datastore connection verified
2020-07-09 08:43:06.782 [INFO][8] startup/startup.go 104: Datastore is ready
2020-07-09 08:43:06.865 [INFO][8] startup/startup.go 678: Using autodetected IPv4 address <IP_ADDRESS>/24 on matching interface eth1
2020-07-09 08:43:06.865 [INFO][8] startup/startup.go 727: No AS number configured on node resource, using global value
2020-07-09 08:43:06.865 [INFO][8] startup/startup.go 201: Setting NetworkUnavailable to False
2020-07-09 08:43:06.915 [INFO][8] startup/startup.go 610: FELIX_IPV6SUPPORT is false through environment variable
2020-07-09 08:43:06.932 [INFO][8] startup/startup.go 234: Using node name: app-kubema-vc1.example.com                                                                                Calico node started successfully
bird: Unable to open configuration file /etc/calico/confd/config/bird.cfg: No such file or directory                                                                                bird: Unable to open configuration file /etc/calico/confd/config/bird6.cfg: No such file or directory
2020-07-09 08:43:08.760 [INFO][37] confd/config.go 81: Skipping confd config file.                                                                                2020-07-09 08:43:08.760 [INFO][37] confd/run.go 17: Starting calico-confd
2020-07-09 08:43:08.948 [INFO][37] confd/client.go 827: Updated with new cluster IP CIDRs: []                                                                                2020-07-09 08:43:08.948 [INFO][37] confd/client.go 818: Updated with new external IP CIDRs: []
2020-07-09 08:43:08.948 [INFO][37] confd/watchersyncer.go 89: Start called                                                                                2020-07-09 08:43:08.948 [INFO][37] confd/client.go 315: Source SourceRouteGenerator readiness changed, ready=true
2020-07-09 08:43:08.949 [INFO][37] confd/watchersyncer.go 127: Sending status update Status=wait-for-ready                                                                                2020-07-09 08:43:08.949 [INFO][37] confd/watchersyncer.go 147: Starting main event processing loop
2020-07-09 08:43:08.975 [INFO][37] confd/watchercache.go 297: Sending synced update ListRoot="/calico/resources/v3/projectcalico.org/nodes"                                                                                2020-07-09 08:43:08.976 [INFO][37] confd/watchersyncer.go 127: Sending status update Status=resync
2020-07-09 08:43:08.976 [INFO][37] confd/watchersyncer.go 209: Received InSync event from one of the watcher caches                                                                                2020-07-09 08:43:08.976 [WARNING][37] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 08:43:08.976 [WARNING][37] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:                                                                                2020-07-09 08:43:08.976 [WARNING][37] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 08:43:08.976 [INFO][37] confd/client.go 766: Recompute BGP peerings: HostBGPConfig(node=app-kubema-vc1.example.com; name=ip_addr_v4) updated; HostBGPConfig(node=app-kubema-vc1.example.com; name=ip_addr_v6) updated; HostBGPConfig(node=app-kubema-vc1example.com; name=network_v4) updated; HostBGPConfig(node=app-kubema-vc1.example.com; name=rr_cluster_id) updated; BlockAffinityKey(cidr=<IP_ADDRESS>/24, host=app-kubema-vc1.example.com) updated; app-kubema-vc1.example.com updated; HostBGPConfig(node=app-kub
eno-vc1.example.com; name=ip_addr_v4) updated; HostBGPConfig(node=app-kubeno-vc1.example.com; name=ip_addr_v6) updated; HostBGPConfig(node=app-kubeno-vc1.example.com; name=network_v4) updated; HostBGPConfig(node=app-kubeno-vc1.example.com; name=rr_cluster_id) updated; BlockAffinityKey(cidr=<IP_ADDRESS>/24, host=app-kubeno-vc1.example.com) updated; app-kubeno-vc1.example.com updated; HostBGPConfig(node=app-kubeno-vc2.example.com; name=ip_addr_v4) updated; HostBGPConfig(node=app-kubeno-vc2.example.com; name=ip_addr_v6) updated; HostBGPConfig(node=app-kubeno-vc2.example.com; name=network_v4) updated; HostBGPConfig(node=app-kubeno-vc2.example.com; name=rr_cluster_id) updated; BlockAffinityKey(cidr=<IP_ADDRESS>/24, host=app-kubeno-vc2.example.com) updated; app-kubeno-vc2.example.com updated
2020-07-09 08:43:08.977 [INFO][37] confd/watchercache.go 297: Sending synced update ListRoot="/calico/resources/v3/projectcalico.org/bgppeers"                                                                                2020-07-09 08:43:08.977 [INFO][37] confd/watchersyncer.go 209: Received InSync event from one of the watcher caches
2020-07-09 08:43:08.982 [INFO][37] confd/watchercache.go 297: Sending synced update ListRoot="/calico/resources/v3/projectcalico.org/bgpconfigurations"                                                                                2020-07-09 08:43:08.982 [INFO][37] confd/watchersyncer.go 209: Received InSync event from one of the watcher caches
2020-07-09 08:43:08.989 [INFO][37] confd/watchercache.go 297: Sending synced update ListRoot="/calico/resources/v3/projectcalico.org/ippools"                                                                                2020-07-09 08:43:08.989 [INFO][37] confd/watchersyncer.go 209: Received InSync event from one of the watcher caches
2020-07-09 08:43:08.989 [INFO][37] confd/watchersyncer.go 221: All watchers have sync'd data - sending data and final sync                                                                                2020-07-09 08:43:08.989 [INFO][37] confd/watchersyncer.go 127: Sending status update Status=in-sync
2020-07-09 08:43:08.989 [INFO][37] confd/client.go 315: Source SourceSyncer readiness changed, ready=true                                                                                2020-07-09 08:43:08.989 [INFO][37] confd/client.go 335: Data is now syncd, can start rendering templates
2020-07-09 08:43:09.073 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:                                                                                2020-07-09 08:43:09.163 [ERROR][37] confd/resource.go 292: Error from checkcmd "bird6 -p -c /etc/calico/confd/config/.bird6.cfg223526888": "bird: /etc/calico/confd/config/.bird6.cfg223526888, line 2: Unable to open included file /etc/calico/co
nfd/config/bird6_aggr.cfg: No such file or directory\n"                                                                                2020-07-09 08:43:09.163 [INFO][37] confd/resource.go 226: Check failed, but file does not yet exist - create anyway
bird: Unable to open configuration file /etc/calico/confd/config/bird.cfg: No such file or directory                                                                                bird: /etc/calico/confd/config/bird6.cfg, line 3: Unable to open included file /etc/calico/confd/config/bird6_ipam.cfg: No such file or directory
2020-07-09 08:43:09.651 [INFO][37] confd/resource.go 264: Target config /etc/calico/confd/config/bird6.cfg has been updated                                                                                2020-07-09 08:43:09.666 [INFO][37] confd/resource.go 264: Target config /etc/calico/confd/config/bird_aggr.cfg has been updated
2020-07-09 08:43:09.666 [INFO][37] confd/resource.go 264: Target config /etc/calico/confd/config/bird_ipam.cfg has been updated                                                                                2020-07-09 08:43:09.666 [INFO][37] confd/resource.go 264: Target config /etc/calico/confd/config/bird6_aggr.cfg has been updated
2020-07-09 08:43:09.751 [INFO][37] confd/resource.go 264: Target config /etc/calico/confd/config/bird.cfg has been updated                                                                                2020-07-09 08:43:09.763 [INFO][37] confd/resource.go 264: Target config /etc/calico/confd/config/bird6_ipam.cfg has been updated
2020-07-09 08:43:09.848 [ERROR][36] felix/connecttime.go 58: BPF program list not json. error=unexpected end of JSON input output="\n"                                                                                2020-07-09 08:43:10.192 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 08:43:10.192 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:                                                                                2020-07-09 08:43:10.192 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 08:43:10.351 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:                                                                                2020-07-09 08:43:10.351 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 08:43:10.351 [WARNING][36] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
bird: device1: Initializing
bird: direct1: Initializing
bird: Mesh_10_100_198_211: Initializing
bird: Mesh_10_100_198_212: Initializing
bird: device1: Starting
bird: device1: Connected to table master
bird: device1: State changed to feed
bird: direct1: Starting
bird: direct1: Connected to table master
bird: direct1: State changed to feed
bird: Mesh_10_100_198_211: Starting
2020-07-09 06:46:20.582 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:20.677 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:20.677 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:20.677 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
bird: Mesh_10_100_198_211: Connected to table master
bird: Mesh_10_100_198_211: State changed to wait
2020-07-09 06:46:21.672 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:21.676 [WARNING][55] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
bird: Mesh_10_100_198_212: Connected to table master
bird: Mesh_10_100_198_212: State changed to wait
bird: Graceful restart done
bird: Mesh_10_100_198_211: State changed to feed
bird: Mesh_10_100_198_212: State changed to feed
2020-07-09 06:46:22.565 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:22.569 [WARNING][55] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
bird: Mesh_10_100_198_211: State changed to up
bird: Mesh_10_100_198_212: State changed to up
2020-07-09 06:46:27.721 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:27.721 [WARNING][55] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
bird: Mesh_10_100_198_212: State changed to start
2020-07-09 06:46:31.684 [WARNING][55] confd/bgpnodeprocessor.go 175: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:
2020-07-09 06:46:31.685 [WARNING][52] felix/felixnodeprocessor.go 232: Failed to parse Node PodCIDR CIDR="" error=invalid CIDR address:

The overlay network seems functional but the error message is spaming on all Pods of the daemonset.
Expected Behavior
I would expect that the Node PodCIDR is correctly parsed from the Kubernetes node metadata.
Current Behavior
Felix is spamming the message over and over.
I checked the nodes and the Pod CIDR ist set:
kubectl get nodes app-kubema-vc1.example.com -o yaml
...
spec:
  podCIDR: <IP_ADDRESS>/24
  podCIDRs:
  - <IP_ADDRESS>/24
...

I saw that there was an improvemnt for host-ipam in libcalico. Cloud this improvement be a problem?
See here.
Steps to Reproduce (for bugs)
Bootstrap a Kubernetes cluster with Calico as Network Plugin and use host-ipam.
Your Environment
Calico Version: 3.15.0
Kubernetes 1.17.8
OS: CentOS 7 with Kernel  5.7.6
CNI Config:
    {
      "name": "k8s-pod-network",
      "cniVersion": "0.3.1",
      "plugins": [
        {
          "type": "calico",
          "log_level": "info",
          "datastore_type": "kubernetes",
          "nodename": "__KUBERNETES_NODE_NAME__",
          "mtu": __CNI_MTU__,
          "ipam": {
            "type": "host-local",
            "subnet": "usePodCidr"
          },
          "policy": {
            "type": "k8s"
          },
          "kubernetes": {
            "kubeconfig": "__KUBECONFIG_FILEPATH__"
          }
        },
        {
          "type": "portmap",
          "snat": true,
          "capabilities": {"portMappings": true}
        },
        {
          "type": "bandwidth",
          "capabilities": {"bandwidth": true}
        }
      ]
    }

Even i do see the similar error in calico 3.15.0. Any clue what could be wrong

I see the similar issue. My investigations led me to a bug in calico/felix: https://github.com/projectcalico/felix/issues/2403

Thanks for digging into this @akolosov-a. Let's use the bug that you raised - https://github.com/projectcalico/felix/issues/2403 - to track this one, and move further conversation over there, since it has a detailed description of the root cause.


pixel-wise loss dominates generator training

As paper states, the lambda_pi = 10, so the pixel-wise loss dominates during the generator training process. Is it OK?

Besides， as in the codes:
        if args.pi == True:
            temp = args.lambda_pi*self.criterion_pixel_wise(self.preds_S, self.preds_T, is_target_scattered = True)
            self.pi_G_loss = temp.item()
            G_loss = G_loss + temp
        if args.pa == True:
            #for ind in range(len(args.lambda_pa)):
            #    if args.lambda_pa[ind] != 0.0:
            #        temp1 = self.criterion_pair_wise_for_interfeat[ind](self.preds_S, self.preds_T, is_target_scattered = True)
            #        self.pa_G_loss[ind] = temp1.item()
            #        G_loss = G_loss + args.lambda_pa[ind]*temp1
            #    elif args.lambda_pa[ind] == 0.0:
            #        self.pa_G_loss[ind] = 0.0
            temp1 = self.criterion_pair_wise_for_interfeat(self.preds_S, self.preds_T, is_target_scattered = True)
            self.pa_G_loss = temp1.item()
            G_loss = G_loss + args.lambda_pa*temp1

It seem that the weight of pixel-wise loss is args.lambda_pi(10) and the weight of pair-wise loss is args.lambda_pa(0.5), while in the paper they share the same weight lambda1(10) in Eq. 4 @irfanICMLL

Except what you mentioned for pixel distillation loss, I also find a strange setting for pixel distillation loss.
https://github.com/irfanICMLL/structure_knowledge_distillation/blob/ce208e1e5ba9177ecfc42519a2c64148d396fb71/utils/criterion.py#L225
Pixel distillation loss is not divided by the batch size N, while other losses have been divided by N.
Combined with what @ArsenLuca mentioned, pixel distillation loss is with a large loss weight: lambda_pi * N, which should be why the value of pixel distillation loss dominates the total loss value.

The pixel-wise loss does not demonstrate generator training. If you try to remove the GAN and the pa loss, the results will become worse. The value of it is very large due to the new reproduce version. However, the main idea of using structure knowledge in knowledge distillation still works in this setting. We vary the loss weight, it will affect the results, but not much.


Executor added to block key

Executor added to block key as a protection for blocking same fingerprints from different executors

Pull Request Test Coverage Report for Build 44

27 of 27 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 42:
0.0%

Covered Lines:
1202

Relevant Lines:
1202

💛  - Coveralls


[!] The plist file at path /ios/.symlinks/plugins/gr_zoom/ios/MobileRTC.xcframework/Info.plist doesn't exist.

getting this error while installing pods

The problem still going?

Resolved.
On Tue, Jun 25, 2024 at 6:21 PM Lucas Motta @.***>
wrote:

The problem still going?
—
Reply to this email directly, view it on GitHub
https://github.com/25LucasAnselmo/gr_zoom/issues/20#issuecomment-2188853236,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMGTDHAHMSRZDBZEOGCTL3DZJFRTZAVCNFSM6AAAAABHOK7QEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBYHA2TGMRTGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


hot reload cogs if owner

add owner only command to reload cogs, saves stopping bot and restarting it when event is happening

https://discordpy.readthedocs.io/en/latest/ext/commands/extensions.html


Jobcode: Overzealous validation

Should allow jobcode with length of 1 , and more than 12 characters too.
Should allow the use of symbols. Eg. Financial Analyst: FA-101
Ensure that jobcode is case insensitive, or give warning when there is potential error in input.

Eg. swe101 and sWe101 might refer to the same jobcode
Suggested solution: save jobcode in uppercase letters, like the name field.

Otherwise, the existing validation rules leads to overzealous validation

Must have:

save jobcode in uppercase letters, like the name field.
lower limit 1, upper limit 50

Nice to have:

not allowing consecutive dash


Unbound cannot be started after configuring dnscrypt

unbound -V

Version 1.14.0

Linked libs: libevent 2.1.12-stable (it uses epoll), OpenSSL 1.1.1d  10 Sep 2019
Linked modules: dns64 cachedb subnetcache ipset respip validator iterator
DNSCrypt feature available
TCP Fastopen feature available

The configuration is as follows:
interface: <IP_ADDRESS>@8443
interface: ::0@8443
dnscrypt:
dnscrypt-enable: yes
dnscrypt-port: 8443
dnscrypt-provider: 2.dnscrypt-cert.exp.domain.com.
dnscrypt-secret-key: /etc/dnscrypt/ssl.key
dnscrypt-provider-cert: /etc/dnscrypt/ssl.cert

What is the error from unbound, that it prints why it cannot start? It can be found in the logs.

What is the error from unbound, that it prints why it cannot start? It can be found in the logs.

How to output error log?

The error logs should be located in the syslog for the machine. By default this is where unbound logs contents. You can configure unbound to log somewhere else with the log-file directive with a filename of your choice. And set the amount of logging with the verbosity directive.
https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#term-logfile-filename
https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#term-use-syslog-yes-or-no
https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#term-verbosity-number
These links to the man page may help to configure logging, if needed. Or find the error in the syslog for the machine, by default.
If you have trouble finding out why unbound does not start, there is a commandline option to help with that. To use it, start unbound from the commandline, from the terminal, and briefly attempt to start it from there. If it starts running, you would have to kill the process. By giving unbound the -d option it stays attached to the terminal console, so it does not fork away in to the background. And with the -dd option logging is directed to the terminal as well. So that you can see the log output on the console of the terminal. Usually you may also want to set which config file to use, if it is not in the default location, and then start with something like unbound -c myconfig -dd, perhaps add -vvvv or so to increase logging verbosity if you want.
This is in the man page here.
https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.html#cmdoption-d

The error logs should be located in the syslog for the machine. By default this is where unbound logs contents. You can configure unbound to log somewhere else with the log-file directive with a filename of your choice. And set the amount of logging with the verbosity directive.
https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#term-logfile-filename https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#term-use-syslog-yes-or-no https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.conf.html#term-verbosity-number These links to the man page may help to configure logging, if needed. Or find the error in the syslog for the machine, by default.
If you have trouble finding out why unbound does not start, there is a commandline option to help with that. To use it, start unbound from the commandline, from the terminal, and briefly attempt to start it from there. If it starts running, you would have to kill the process. By giving unbound the -d option it stays attached to the terminal console, so it does not fork away in to the background. And with the -dd option logging is directed to the terminal as well. So that you can see the log output on the console of the terminal. Usually you may also want to set which config file to use, if it is not in the default location, and then start with something like unbound -c myconfig -dd, perhaps add -vvvv or so to increase logging verbosity if you want.
This is in the man page here. https://unbound.docs.nlnetlabs.nl/en/latest/manpages/unbound.html#cmdoption-d

/usr/local/etc/unbound/unbound.conf:92: error: syntax error
read /usr/local/etc/unbound/unbound.conf failed: 1 errors in configuration file
90    interface: <IP_ADDRESS>@8443
91    interface: ::0@8443
92    dnscrypt-enable: yes
93    dnscrypt-port: 8443
94    dnscrypt-provider: 2.dnscrypt-cert.exp.com.
95    dnscrypt-secret-key: /etc/dnscrypt/1.key
96    dnscrypt-provider-cert: /etc/dnscrypt/1.cert

The issue is a missing line between line 91 and 92, dnscrypt: that is the start of the dnscryp options section.
Then, once you are done with the dnscrypt options, you would need to start the server: section again if you have more server options below it.

The issue is a missing line between line 91 and 92, dnscrypt: that is the start of the dnscrypt options section. Then, once you are done with the dnscrypt options, you would need to start the server: section again if you have more server options below it.

fatal error: module conf 'subnetcache validator cachedb iterator' is not known to work

From what I can see, there is no listed combination with subnetcache and cachedb that is known to work. What you suggest seems reasonable, if that works, I can add it to the list of known combinations.

I have added the combination to the list of allows entries, in the commit above.

I have added the combination to the list of allowed entries, in the commit above.

thank you 😁


Update spring boot, liquibase, jgit

Description
Updating dependencies, and building the project now with Java 17 (LTS). Had to replace Cobertura by JaCoCo in order to get the build green. The test coverage reduced, but that's because Cobertura was reporting incorrectly (see comment below about it).
Motivation and Context
Proactively updating a few dependencies.
How Has This Been Tested?

[x] mvn clean test
[x] Running Docker Compose locally

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

https://github.com/common-workflow-language/cwlviewer/runs/6405459959?check_suite_focus=true#step:6:384
Time to upgrade to something newer than Java 8?

https://github.com/common-workflow-language/cwlviewer/runs/6405459959?check_suite_focus=true#step:6:384
Time to upgrade to something newer than Java 8?

Good idea. Probably latest LTS 17 if it works with our dependencies & deployment - https://www.oracle.com/java/technologies/java-se-support-roadmap.html

Upgraded to Java 17, then had to update the build tooling too. Upgraded the maven plug-ins, it broke coverage. Fixed coverage, now the report is complaining it was reduced in 7.1 % :sweat_smile: Almost there! More tomorrow.

There are way more files in the JaCoCo report than in the Cobertura one. And Cobertura appears to have been broken for a while. I confirmed GitDetails.java that was 100% for Cobertura is not actually fully covered by our tests. So the JaCoCo number looks correct :+1:

@mr-c updated the branch, but I cannot merge it, could you merge it if it's looking OK to you, please? Thanks!!! :pray:

Coverage will go down due to a problem at how Cobertura was counting covered code. But we will increase that later :muscle:


Should postal code include country identification

Problem description
Postal code is not unique: for example 31000 is Pamplona in Spain and Toulouse in France.
In most cases the country can be derived from the phone number, but there can be other more sophisticated cases.
Making postal code unique should help to detect error API calls.
Possible evolution
Geographical coordinates are unique, the country property can be added to the postal code to make it unique like in https://github.com/camaraproject/DeviceStatus/blob/main/code/API_definitions/device-roaming-status.yaml
  CountryName:
     description: The ISO 3166 ALPHA-2 country-code
     type: string

Alternative solution

Additional context

My point is: for me, as a Telco Operator in one country and exposing the API, my service endpoint will receive a call to this API only if the request is for me, and I just have to apply the parameters in the request to the country I am operating on. It is assumed that there is a previous mechanism to route the request to the corresponding operator.
So, from the other perspective, for me, as an API client providing a service in one country, I send a request regarding an MSISDN (or IP:port, ...) and I expect that the request will reach the Telco operator of that user in that country, so I should not need to specify the country. And I think it is similar even though the service provider is operating in several countries: me, as a service provider, I want to verify the postalCode 31000 in the country of the user. If I have to include the country name, I am forced to somehow gather more information that in fact I don't need.
On the other hand, In Roaming Status it makes sense because it is, effectively, considering the scenario where the user is out of the country, but the country name parameter is in the response, not in the request: in that case the client/service provider is asking whether the user is in a different country from the one that they are suposed to be, which is the home country, the country where the Operator is operating and received the request.

With roaming and federation it makes sense to add CountryName

Just to try to have content to document our conversation this morning:
We are trying to follow the principles of simplicity for the user and data minimization:

Simplicity: a Service Provider wanting to verify the MFL of a end user by using the postal code would be assuming that the postal code is for the country which the phone number belongs to. This will work because the request will reach the telco operator responsible for that phone number.
Data minimization: when a telco operator receives a request having a postal code and a country name, I see 3 scenarios:

postal code and country name make sense for the operator: everything is ok to proceed with the request (no error returned), but the country name probably wouldn't have been necessary.
The Service Provider included a wrong country name but a correct postal code (it may happen by accidental error or by wrong information holded by the Service Provider): the telco operator will return an error of mismatching. This is good if the cause was an accidental error, but goes against data minimization if the cause was wrong information holded by the SP).
The Service Provider included a right country name but a wrong postal code (again, either by accidental error or by wrong information of the SP): again, returning an error is good if the cause was an accidental error, and against data minimization if the cause was wrong information.

This said, it's true that there are scenarios where including de country name would help the Developer/Service Provider avoid mistakes. But this would be based on including a parameter to "double check" the information they are providing in the request, and it wouldn't make sense to add parameters in any API just to double check (data minimization). So, we agree that it is very necessary to have a good documentation to make the Developer aware that the postal code will be processed by the telco operator responsible for that user assuming that the postal code is for the country where the telco operator operates.
Anyway, this comment is probably missing other scenarios or causes of mismatches between both parameters...

Currently postal code is not included in the proposed common definition of Area type: https://github.com/camaraproject/Commonalities/pull/315 .


[chore] Add Dependabot auto-update-workflow

Changes
This PR adds a new workflow to allow Dependabot to auto update the generated proto files when the google.golang.org/grpc/cmd/protoc-gen-go-grpc package is bumped.
This will avoid manual tasks like we had to do in:

https://github.com/open-telemetry/opentelemetry-demo/pull/1807
https://github.com/open-telemetry/opentelemetry-demo/pull/1802

Merge Requirements
For new features contributions, please make sure you have completed the following
essential items:

[ ] CHANGELOG.md updated to document new feature additions
[ ] Appropriate documentation updates in the docs
[ ] Appropriate Helm chart updates in the helm-charts

@CharlieTLe would you be able to take a look?
Basically created the token with the permissions you have shared in #1802 and copied your workflow: https://github.com/CharlieTLe/opentelemetry-demo/blob/6a07a6e29a18c3d8dfe89e7596054f06439f6504/.github/workflows/dependabot-auto-update-protobuf-diff.yml
Only one thing I didn't quite get.
Is this hash coming from somewhere?
git config user.email<EMAIL_ADDRESS>Should I adapt it here?

Yeah, it should be adapted here. I got it from looking at the email on commit it makes.
For example, on https://github.com/open-telemetry/opentelemetry-demo/commit/40ff491f719137e37207e66aa03166438c843f8a.patch, we can see the line:
From: "dependabot[bot]"<EMAIL_ADDRESS>
I picked this username so that if we ask dependabot to do a rebase, it doesn't complain that someone else besides it did the commit.


optimize x is T when x is null

Class types implement is like this (see addTypeTests):
    if (obj != null && obj[isClass]) return true;
    return dart.is(obj, this);

instead it should be
    return obj != null && (obj[isClass] || dart.is(obj, this));

We can do this for almost all types.
Null, Object, dynamic, and FutureOr already have their own special is code (see _emitClassTypeTests), so as long as those ones allow null, we should be good.

https://dart-review.googlesource.com/c/sdk/+/9792


[RFE]  Utilize Azure Spot virtual instances for cluster nodes

Is your feature request related to a problem? Please describe.
Want to create nodes using the cheaper (by almost 90%) spot instances on Azure
Describe the solution you'd like
The ability to select the azure Spot Option
Select the eviction type and policy
Describe alternatives you've considered
Currently considering spinning up an Ubuntu instance and installing rancher within Azure rather than my local datacenter, then spinning up additional linux hosts, and  running the manual node registration process.

Bump...

+1

+1

Much needed.


avoid log channel closed caused endless loop

if log watcher meets error and closes logCh, log monitor will enter endless for loop

/assign @dchen1107

/ok-to-test

@wenjun93 Thanks for the fix!

/lgtm
/approve

@wenjun93 Can you sign CLA?

@wangzhen127 we may want to cherrypick this into release/1.0

@wenjun93 Can you sign CLA?

@Random-Liu I changed my email in the past, now corrected

/lgtm


Trying to integrate with Angular 2 Schema Form - Getting error

I am trying to integrate with https://github.com/makinacorpus/angular2-schema-form
import { disableDeprecatedForms, provideForms} from "@angular/forms"
import { WidgetRegistry } from "angular2-schema-form";
bootstrap: [AppComponent,[disableDeprecatedForms(),provideForms(),WidgetRegistry]]
My requirement is to have some pages dynamic and some pages static. So I tried both bootstrapping inside the the main.ts, about.module.ts and also inside the app.module.ts, but got the same error.
I am pretty new to Angular 2 and not able to find out why the integration is not working. Could you please help in integrating this?

For support requests you can use gitter or take a look at project's the wiki page in order to get more complete idea on how the it works.


Support Major.Minor.Build.Revision

Could you add support for this, it is assuming Major.Minor.Build is the latest versioin, however Major.Minor.Build.Revision is latest.
Also ensure this works with pre-release kind of versioning too.
Reference: https://learn.microsoft.com/en-us/dotnet/standard/library-guidance/versioning

Done, thank you for the suggestion. It's available in v0.3.3. Let me know if you spot any issues please.