feature request: configurable disk space

Hi,
Currently, under resources, we can only configure vcpu and memory. I found that by default the disk space allocated is 10G. Do your team have any plan to make disk space also configurable?
Is it setting here? https://github.com/hyperhq/hyperd/blob/master/storage/defaults.go#L5
https://docs.hypercontainer.io/reference/podfile.html

@gnawux Hi, can you help with the issue or point me to someone who knows about it? Thanks!

@minz1027 the DEFAULT_DM_VOL_SIZE is the default value in case:

devicemapper is chosen as the storage driver
need create a volume by the daemon

And in most cases, the volumes are created by an external dedicated volume management system, and passed to the daemon. This is why it is not an configurable item now.
However, it is welcome to submit a PR for adding an size field for the pod spec.

@gnawux Hi Xu, thanks for your reply! I have some questions about this,

Does it mean the configurable vol size thing will only work for devicemapper?
Do you want me to just add a size field to the pod spec or pass it all the way to construct a dm?

@minz1027 currently, only the block devices support a "size" property. Once you have a size field, you may apply it to all storage drivers. For block devices, such as devicemapper and rawblock, they may create a volume with specified size. And for directories, the might be implemented based on quota, which depends on the feature of the underlying filesystem.

@gnawux Hi Xu, I have the code to pass size through cmd line and pod spec. But the thing is I found that the size property or storage for the daemon is set when creating a new daemon. https://github.com/hyperhq/hyperd/blob/20ab5d6fc8c38259f6d481667027c4cc381d0e2c/daemon/daemon.go#L136
By the time we tried to create a pod, the daemon should be already created and running. How can we override the storage property in that case?

I think you may change the interface of create volume,
https://github.com/hyperhq/hyperd/blob/bb3ac60ffd5b7c5dfe03f1da56b77b24a5683db6/daemon/storage.go#L204
here we create the volume with default size, and you may change it to employ the size specified in pod spec.

@gnawux Thanks for pointing that out! Should send in a PR soon :)

@gnawux Hmm.. I am not sure if that's the thing I want to change. My problem is when I try to hyperctl run with a very large image(~10G), I will get no space left on the disk error when pulling the image.
It doesn't seem to be related to the volume.

@gnawux Hi Xu, my PR got merge yay! I wonder when would you guys do release with that change? Thanks!

@minz1027 thank you, and it would be released in this week when #664 get merged


SW-2607 Updates to plants dashboard for sites without zones/plots

change layout and data being shown

I will get this in and follow up separately if design has more feedback.


Issues with window sizing in Windows

There's an offset in Windows at the top (in blue) because Windows puts its menu bar in the frame. BrowserView is absolutely positioned in the frame, which is why the  bottom is cut off. I still don't know why the right is cut off.

Investigate shenanigans with electron-window-state in windows.

Same behavior on Linux too, although it's just the Drive page

Same behavior on Windows on Gmail:

Same behavior on Linux too, although it's just the Drive page

Can confirm, on Linux its even worse.. BrowserWindow doesn't seem to fill the whole desktop window, and then the BrowserWindow keeps the same (incorrect) aspect ratio with which it was launched whenever your resize the desktop window.
Using arch / i3-gaps here.

Also darkmode seems to mess up the profile picture on the Gmail Hangouts chat widget in the left sidebar. Compare my profile picture in the above screenshot to this:

A simple fix for this could be to invert everything but the images. Something like this inverting the images back could work:
html {
    filter: invert(100%);
}
img {
    filter: invert(100%);
}

After some hair pulling, it looks like using BrowserWindow.getContentBounds() instead of getBounds() gets the true dimensions of the screen that lie within the window.
This should fix the awkward window sizing issue that was happening in linux and windows but not macos. These changes also don't seem to be breaking the sizing on macs.


Error on login screen

After following the instructions, I was able to launch to the login screen.   Would you know the appropriate credentials?  It does not recognize my Salesforce login and will not let me register a new login/password.
My attempt to register did throw an error to the screen.  Please see image and attached file with error text (username and password masked).
Thanks

ErrorOnRegister.txt

Rechecked my setup.  I was getting an error when I attempted to install electron into the app directory.  I am running the app on Windows 10 (on VSCode) and did not have the Microsoft Build tools.  I had installed them last night and retried - still hit the error.  However,  this morning all ran fine (guess I needed a reboot).  I re-ran steps 9, 10 & 11.  The app launched and I was able to successfully register and login.
Thanks again.


aot does not recognize the use of ctx.body within a try catch

What version of Elysia.JS is running?
1.0.22
What platform is your computer?
WSL Ubuntu
What steps can reproduce the bug?
// test.test.ts
import { describe, it, expect } from "bun:test";
import { Elysia, type Context } from "elysia";

describe("AOT", () => {
  it("???", async () => {
    class Controller {
      static async handle(ctx: Context) {
        try {
          // @ts-ignore
          const { token } = ctx.body;
          return token;
        } catch {
          return "nope";
        }
      }
    }

    const app = new Elysia({ aot: true }).post("/", Controller.handle);

    const response = await app.handle(
      new Request("http://localhost/", {
        method: "POST",
        body: JSON.stringify({ token: "yay" }),
        headers: { "Content-Type": "application/json" },
      })
    );

    expect(await response.text()).toEqual("yay");
  });
});

What is the expected behavior?
Pass all tests
What do you see instead?
Expected: "yay"
Received: "nope"

Additional information
workaround:
new Elysia({ aot: false })

reported on discord

Looks like its related to this: https://github.com/elysiajs/elysia/issues/656


Link of repository is the old one

In documentation,  link of repository is still https://github.com/TeamHeka/medkit

Indeed, there is an ongoing rework of the documentation for the next version. It will be corrected by then.


Update Feed UI styling for Desktop

[ ]  Update current Feed page styling for desktop.

Refer to the latest design in FIGMA under Desktop App MVP.
https://www.figma.com/file/4cV84H1jyMo0rJEAl3HNnJ/FightPandemics-landing-page?node-id=1377%3A8027

[ ] Designs need to be pixel perfect

@silenceway- any updates on this issue?


Table service returning AuthenticationFailed

Which service(blob, file, queue, table) does this issue concern?
table
Which version of the Azurite was used?
3.12.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
DockerHub, image id ca65efb4c19d
What problem was encountered?
getting a AzureHttpError: Server failed to authenticate the request. when listing the tables on a brand new container.
Also, not important for this issue but probably related, getting the following when trying to look at the tables in AzureStorageExplorer when trying to look at the tables present:
Unable to retrieve child resources.

Details:
Network timeout error: socket hang up

Steps to reproduce the issue?
The error can be tested using the following python script:
  from azure.cosmosdb.table import TableService

  AZURITE_ACCOUNT = "devstoreaccount1"
  AZURITE_DEFAULT_KEY = "Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw=="

  AZURITE_ENDPOINT = f"BlobEndpoint=http://<IP_ADDRESS>:10002/{AZURITE_ACCOUNT}"
  AZURITE_TABLE_CONNECTION = f"DefaultEndpointsProtocol=http;" \
                             f"AccountName={AZURITE_ACCOUNT};" \
                             f"AccountKey={AZURITE_DEFAULT_KEY};" \
                             f"{AZURITE_ENDPOINT};"

  service = TableService(connection_string=AZURITE_TABLE_CONNECTION, is_emulated=True)
  service.list_tables()

Which returns:
azure.common.AzureHttpError: Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature. ErrorCode: AuthenticationFailed
{"odata.error":{"code":"AuthenticationFailed","message":{"lang":"en-US","value":"Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.\nRequestId:671b875c-a002-00cb-6453-52a4e6000000\nTime:2021-05-26T17:19:55.7517331Z"}}}

Have you found a mitigation/solution?
No

Never mind.. was passing the blob configuration. This works:
from azure.cosmosdb.table import TableService

AZURITE_ACCOUNT = "devstoreaccount1"
AZURITE_DEFAULT_KEY = "Eby8vdM02xNOcqFlqUwJPLlmEtlCDXJ1OUzFT50uSRZ6IFsuFq2UVErCz4I6tq/K1SZFPTOtr/KBHBeksoGMGw=="

AZURITE_ENDPOINT = f"TableEndpoint=http://<IP_ADDRESS>:10002/{AZURITE_ACCOUNT}"
AZURITE_TABLE_CONNECTION = f"DefaultEndpointsProtocol=http;" \
                           f"AccountName={AZURITE_ACCOUNT};" \
                           f"AccountKey={AZURITE_DEFAULT_KEY};" \
                           f"{AZURITE_ENDPOINT};"

service = TableService(connection_string=AZURITE_TABLE_CONNECTION, is_emulated=True)
print(service.list_tables())


fix(versions): update Activiti/activiti-cloud-modeling-dependencies versions into develop

UpdateBot pushed maven dependency: org.activiti.cloud.modeling:activiti-cloud-org-dependencies to: 7.1.72

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.modeling:activiti-cloud-org-dependencies 7.1.72

 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.


Is it possible to make a better controller ?

Currently, my controller looks like that :

Note that there is no distinct way to

see current
change current

At this time of the screenshot, I was on "programmed - nightmode".
I understand the difficulties are elsewhere (API) but I think we need a much better controller !

Okay looking at it, it just seems it is a HA native UI which is just quite bad. I'll search for better ones I guess.

Heatzy convectors do not have a temperature sensor. It is therefore necessary to make a generic Thermostat and to associate a temperature sensor and the Heatzy switch in configuration.yaml
climate:
  - platform: generic_thermostat
    name: Thermostat Salon
    heater: climate.salon #Heatzy climate sensor
    target_sensor: sensor.lumi_rdc_temperature #Your temperature sensor (for me , it's Xiaomi Aqara Lumi)
    away_temp: 14
    comfort_temp: 19.5
    sleep_temp: 17.5
    hot_tolerance: 0.2
    cold_tolerance: 0.2
    keep_alive:
      minutes: 5


Injection FBI, Part 4, Section II, can't select FBI_inject_with_banner.app.

Hi,
I'm having a problem with the last part of the FBI's injection.
I was on a n3ds E 9.0.8 successfully downgraded, with an updated RedNand 11.XX
In the Section 1 I did all that was asked, i didn't have a Decrypt9 folder on the root of my sd card, just a Decrypt9WIP on the 3ds folder (so didn't touch it)
my Wi-Fi is on.
I've then dumped to hs.app (the file is dated from 29/02/2016, has a size of 0,99Mo and localized in files9).
I've extracted the Universal Inject Generator and copy the hs.app previously mentioned and the fbi.cia in the updates folder on my computer.
Launched the go.bat which created some files.
Copied FBI_inject_with_banner.app in the root of my sd card.
I've gone to the Health & Safety Inject option, but I can't press down to select the FBI_inject_with_banner.app.
Instead I have to press left, right, down, up, A then I can only choose hs.app, the D-pad doesn't change this option.
I'm using FBI 2.4.2 and Decrypt9WIP-20160814-201618.zip.
Any idea of what went wrong?

Hi, copy FBI_inject_with_banner.app to files9 folder.

Thanks, it worked.

Hello, I'm following this instruction https://3ds.guide/installing-arm9loaderhax and got the same issue.
I did put the file FBI_v2.4.2_O3DS_EUR_injectable.app inside files9 folder but seems didn't help.
Somebody please help!
Many thank!


Present solutions deployed per app catalog. Closes #215

🎯 Aim

Present solutions deployed per app catalog and add additional app-related actions (deploy, retract, remove, enable, disable etc.)

📷 Result

✅ What was done

[X] Dynamic loading of children for ActionTreeItem
[X] Present solutions deployed per app catalog
[X] Allow to disable and enable a solution
[X] Allow to deploy and retract a solution
[X] Allow to remove a solution
[X] Clicking on app link should go to the solution appStore.aspx page
[] Allow to move solutions between app catalogs (from site level to tenant level and the other way around)
[] Allow to add a webpart to all sites

🔗 Related issue

Closes #215

But still I think it was pretty nice to go to the app catalog by clicking on the item

Oh, I missed that! I'll include it today. Should I just make the node itself a link, or would you prefer an action for it? The App Catalog node already looks like a link, so as a user I would think it should be clickable. Users can click on the node to navigate directly to the App Catalog, and click the arrow on the left to expand and see the list of apps.

or, I can provide additional action, if you prefer that?

@Saurabh7019 I'll leave it up to you. For me any of the options you mentioned are fine 👍 as long as we keep this functionality as I think it is just helpful.
I will leave the decision up to you 👍 this is an open-source and community-driven tool and I am confident you will pick the best option

@Saurabh7019 I also not just realized we should update the guidance in the readme and screenshots that present this part of toolkit to keep this up to date 👍

Awesome work 👏👏👏. You Rock 🤩


Fix portals in PWs. Copy more pixelscenes to PWs for authenticity.

Mainly the biome starts are needed, to block the player in while the collapse happens.

FWIW boss arena is one of the few that aren't copied. It is now properly evil.


crystal: init at 0.30.1

Motivation for this change
Latest crystal: https://crystal-lang.org/2019/08/12/crystal-0.30.1-released.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Cc @bcardiff


Remove rollup

The rollup module isn't currently being used in the dev/OSS version of the product - so removing it since right now it just fails to compile.
@kburtram - Do you know anything about what this is doing? Any issues with removing this here? It seems to be for web mode only from looking at how it's used in the distro repo so there doesn't seem to be a reason to keep it in this repo that I can see.

Pull Request Test Coverage Report for Build 807944738

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 43.536%

Totals

Change from base Build 807744838:
-0.002%

Covered Lines:
26464

Relevant Lines:
55320

💛  - Coveralls

@Charles-Gagnon what's the issue you're trying to fix with this PR?

It's just that right now you can't compile the build directory locally. Not normally an issue for most folks but it's something I ran in to when I was debugging something with the build gulp tasks.

@Charles-Gagnon if this isn't causing an immediate problem I'd rather wait until the webmode builds are working again before making changes here.  As part of that work we can move this file there is that makes sense.  At the moment I don't have context on the file, but I know there were module load issues around angular, etc.  I'd recommend keeping this open for the next couple weeks and we can revisit once I've had a chance to look at the builds.

shrug Both me and Alex hit this within the past couple of days. It's definitely annoying if you want to do anything with the build scripts.
Note that this doesn't affect the product builds at all. Those still merge in the distro repo which still contains this file and associated dependencies.
That being said - if it'd make you feel better we can also just add back in the missing package (rollup). I just didn't see the point in having this stuff for dev builds since all it's doing is add overhead.

Yes, I think it makes sense to hold off updating web related artifacts until I have a chance to look at them.  If this isn't blocking then let's hold off updating web-related content.

Alright, I'll open another PR for adding back in the dependency though since that's a much safer change to make


Problem with Metamask

index.ts:225 Uncaught (in promise) Error: call revert exception (method="balanceOf(address)", errorArgs=null, errorName=null, errorSignature=null, reason=null, code=CALL_EXCEPTION, version=abi/5.5.0)
at Logger.makeError (index.ts:225:1)
at Logger.throwError (index.ts:237:1)
at Interface.decodeFunctionResult (interface.ts:425:1)
at Contract. (index.ts:392:1)
at Generator.next ()
at fulfilled (index.ts:2:1)
Hi, I Clone your project.
This is the message I have when i connect my Metamask, do you have any idea ?
Thanks for your help

@seemo92 looks like perhaps you are not connected to Cronos testnet in your wallet? otherwise can you recap the steps that you took?


Text color change

PR Description 📜
Text color changing to black when enabling dark mode, and again changing to white when disabling dark mode.
Fixes #4362

Mark the task you have completed ✅

[✅] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✅] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[✅] My changes generates no new warnings.
[ ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[✅] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Assign it to me and add necessary level labels under GSSoC.
Thank you soo much for contributing to our repository 💗

Thank you @dishathakurata ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @dishathakurata , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


adding a bower support to the library

I would like to use your library via bower, so this is a rudimentary setup of what the bower.json file should look like.  Please go over it and change anything that I got wrong.
Thanks.

Thanks!


GDP Icon

The GDP Icon Issue
Here is the snapshot of the banner of the gdp webpage

The GDP icon file is fine, but with the black background, the letter "G" and "P" are partially cut.
Possible solutions:

Change the black background to a lighter one, or
Modify the color of the text GDP in the GDP icon file so that it shows over the black background.

e2fd280b811ccf16d64e7e4e9ac269d65f1327bd


Ministry user processes request for a PSD grant

[placeholder ticket]
Context: PD related
$1 of need program

No longer a requirement. Absorbed by #1053.


Error with Quickstart demo

Successfully installed via cocoapods. Added import AnimatedGradientView (might be worth putting this step to your Quickstart instructions?).
Not able to test your Quickstart code, as I'm receiving the following error:
Value of type 'AnimatedGradientView' has no member 'animationValues'
Am I missing something obvious? Thanks.

Consider it added 🙂As for the issue with the animationValues property not being visible, I'm not certain what the issue would be here. I just created a brand new Swift 4.2 project, added the framework using Cocoapods and pasted the sample code which means to run fine. Is there anything more you could tell me about your setup that might shed some light on this? Are you able to compile and run the Example app included in this repo OK?

I too managed to get the Example appworking, and also built a new project simply adding the code snippet - which also worked... (Last line changed to this: self.view?.addSubview(animatedGradient))
At the mo the only difference I can see is that in newly created project snippet is in:
override func viewDidLoad() {
        //// here
        let animatedGradient = AnimatedGradientView(frame: view.bounds)
        animatedGradient.direction = .up
        animatedGradient.animationValues = [(colors: ["#2BC0E4", "#EAECC6"], .up, .axial),
                                            (colors: ["#833ab4", "#fd1d1d", "#fcb045"], .right, .axial),
                                            (colors: ["#003973", "#E5E5BE"], .down, .axial),
                                            (colors: ["#1E9600", "#FFF200", "#FF0000"], .left, .axial)]
        self.view?.addSubview(animatedGradient)
}

In my main project it's being called from:
override func touchesBegan(_ touches: Set<UITouch>, with event: UIEvent?) {
       /// here
}

But I can't see why this would make a difference.
Perhaps there is a conflict in my podfile?
platform :ios, '9.2'

target 'Kylypso' do
  use_frameworks!
  pod "AudioKit", '~> 4.4'
  pod "TouchVisualizer", '~>3.0.2'
  pod "AnimatedGradientView"

  pod 'FacebookCore'

  pod 'Firebase/Core'
  pod 'Fabric', '~> 1.9.0'
  pod 'Crashlytics', '~> 3.12.0'

end

post_install do |installer|
    installer.pods_project.targets.each do |target|
        target.build_configurations.each do |config|
            config.build_settings['ALWAYS_EMBED_SWIFT_STANDARD_LIBRARIES'] = 'NO'
        end
    end
end

Thanks for the information. My best guess at the moment is that the compiler is seeing the type of your view in your main project as UIView rather than as of type AnimatedGradientView which would explain why the compiler doesn't see the animationValues property.
You should be able to determine the type by adding a line such as print(type(of: myView)) before the line which reads:
myView.animationValues = ...

Could you run this line and see what is printed?

So, in order to discover the View type, I commented out the animationValues line. Then I saw the gradient (on a default colour?) - and type was indeed returned as AnimatedGradientView... The mystery deepens 🧐

That is a mystery - I created a new project with your Podfile and again everything worked fine. Are you using an older version of Swift in your project or is there anything else notable about your setup you can think of?

Solved - simple when you know how. I'd changed Swift from 4.0 - 4.2 in Pods project, but not in main... D'oh!
Thanks so much for your input on this - it did help me find the issue. Aaaand, those gradients are beautiful! 😎

Glad you solved it! 😃


Method to know which server was selected for a key

Hey all-
First off, excellent job on Twemproxy.  Our company is looking for an easy-to-use, consistent hashing solution for Redis, and this tool meets all our needs.
Question: Is there a way to know which server was selected for a particular key?
The reason I'm asking is because we might want to map several virtual servers to the same redis instance to establish our initial hash-ring and simultaneously reduce costs.  Knowing which server was initially selected will help us with data migration in the future.
Thanks again for releasing this to the community...it's a big help for start-ups looking for an out-of-the-box solution before redis cluster.
Cheers,
J

EVAL "return redis.call('info', 'server');" 1 keytolookup can be used to fetch information about the server, potentially including the port or the config file name (if started with a config file). Redis will send EVAL commands to a server based on the first key in an EVAL command
(keytolookup is the literal name to lookup)
It should contain something like config_file:/etc/redis/db009.conf, which would be useful if the config name is globally unique and doesn't change

I'm assuming https://github.com/twitter/twemproxy/issues/118#issuecomment-830876378 answers the original question about redis.
I'm not sure of the plans for the key_to_server branch, but from it having never been merged I assume it wasn't planned?


xworkspaces with i3 shows active workspace as "empty"

I'm trying to use xworkspaces plugin with i3 so I can keep my polybar config consistent when I switch to other WMs.
I've noticed a bug where an active workspace shows up as empty after switching to it from an empty workspace directly preceding it. I have attached a screen recording below:

To clarify what's going on:

switch from workspace 2 to 4 to show how it should behave
switch from workspace 2 to 3 to 4
once we end up on 4, it should be active, but instead it is displayed as empty, and none of the workspaces are highlighted.

Relevant part of my config:
[module/ewmh]
type = internal/xworkspaces
label-active = %name%(active)
label-active-background =#FFFFFF
label-empty = %name%(empty)

I speculate this is caused by i3 removing the empty workspace (3 in my example), causing the index of the current workspace to not change. I've tried to look through the module's source code, but was unable to pinpoint the exact cause.

I've got something similar looking happening in xmonad.


[FLINK-28193][runtime] Enable to identify whether a job vertex contains source/sink operators

What is the purpose of the change
Speculative execution does not support sources/sinks in the first version. Therefore, it will not create speculation instances for vertices which contains source/sink operators.
Note that a job vertex with no input/output does not mean it is a true source/sink vertex:

Multi-input sources can have input
It's possible that the vertex with no output edge does not contain any sink operator
A new sink with topology can spread the sink logic into multiple job vertices connected with job edges.

In this pr, we introduce methods to check whether a job vertex contains source/sink operators.
Verifying this change

Added unit test StreamingJobGraphGeneratorSourceSinkTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Could you not derive this from the size of inputs/results in the JobVertex?

Could you not infer this from the size of inputs/results in the JobVertex?

A job vertex with no input/output does not mean it is a true source/sink vertex.

Multi-input sources can have input
It's possible that the vertex with no output edge does not contain any sink operator
A new sink with topology can spread the sink logic into multiple job vertices connected with job edges.

Thanks for the reviewing! @wanglijie95
Merging.


rename gnome.adwaita-icon-theme

In nixos-unstable, adwait-icon-theme has been moved to the top-level.
trace: warning: The ‘gnome.adwaita-icon-theme’ was moved to top-level. Please use ‘pkgs.adwaita-icon-theme’ directly.

I think to get CI passing, and to keep support for stable nixpkgs, you'd need to do something like is done in this commit:
https://github.com/lilyinstarlight/nixos-cosmic/commit/66da31c8443a6336fc9e321e369381659684a1f7

FWIW, this means I can't rebuild currently if I have cosmic enabled. Unstable, 24.11

I think to get CI passing, and to keep support for stable nixpkgs, you'd need to do something like is done in this commit:
66da31c

https://github.com/lilyinstarlight/nixos-cosmic/pull/208 drops support for Stable as the Rust toolchain is no longer new enough to compile COSMIC.

I think to get CI passing, and to keep support for stable nixpkgs, you'd need to do something like is done in this commit:
66da31c

Stable CI is currently broken anyway due to the minimum Rust toolchain that COSMIC uses being raised, and that version isn't going to be backported to Stable unless another packages requires it for a security update, see https://github.com/NixOS/nixpkgs/pull/320250#issuecomment-2209425145.
https://github.com/lilyinstarlight/nixos-cosmic/pull/208 removes Stable as a build requirement for PR's to be merged and also fixes another CI issue.

I think to get CI passing, and to keep support for stable nixpkgs, you'd need to do something like is done in this commit:

66da31c

Stable CI is currently broken anyway due to the minimum Rust toolchain that COSMIC uses being raised, and that version isn't going to be backported to Stable unless another package requires it for a security update, see https://github.com/NixOS/nixpkgs/pull/320250#issuecomment-2209425145.
https://github.com/lilyinstarlight/nixos-cosmic/pull/208 removes Stable as a build requirement for PR's to be merged and also fixes another CI issue.
I do however agree that we shouldn't be breaking Stable just to remove non-fatal warnings from Unstable, especially when it's entirely possible that Stable could end up being a suitable build target again.

FWIW, this currently breaks my system builds if I want to use cosmic. I think it's because I don't have stable available, but rn I can't rebuild if I'm using the flake.

FWIW, this currently breaks my system builds if I want to use cosmic. I think it's because I don't have stable available, but rn I can't rebuild if I'm using the flake.

I'm not really sure what you're saying, is the branch in the PR breaking flake evaluation?

Sorry, that was unclear!
If I just use the main branch, I can't rebuild anymore. I'm using a fork instead with this PR implemented in the meantime.

What fork are you using? I've thought about forking this myself, I have like 500 build failure notifications from GitHub because I'm overly aggressive with my CI. I've been trying to ignore it but... the emails keep coming. :laughing:

@colemickens Addy419's! The person who submitted the PR.
nixos-cosmic = {
      url = "github:addy419/nixos-cosmic/main";
      inputs.nixpkgs.follows = "nixpkgs";
    };

Also may try a couple things on a personal fork to see if can get more juicy updates, cosmic is so darn nice.

Closing since this has been resolved for a while. Feel free to open a new PR if you are still having issues!


Verdictd: add make package cmd

Fixes: #1468
Signed-off-by: Zhou Liang<EMAIL_ADDRESS>

@liangzhou121
See this build failure in CICD:

pushd verdictd
1471
make
1472
make[4]: Entering directory '/root/rpmbuild/BUILD/inclavare-containers-0.6.4/verdictd'
1473
go<EMAIL_ADDRESS>requires
1474
<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>504 Gateway Timeout
1475
make[4]: *** [Makefile:28: opa] Error 1
1476
make[4]: Leaving directory '/root/rpmbuild/BUILD/inclavare-containers-0.6.4/verdictd'
1477
error: Bad exit status from /var/tmp/rpm-tmp.cvVlJl (%build)
1478
Bad exit status from /var/tmp/rpm-tmp.cvVlJl (%build)
1479

1480
1481
RPM build errors:
1482
make[3]: *** [Makefile:21: rpm] Error 1
1483
make[3]: Leaving directory '/root/inclavare-containers-0.6.4/verdictd/dist'
1484
make[2]: *** [Makefile:36: package] Error 2
1485
make[1]: *** [Makefile:54: package] Error 2
1486
make[2]: Leaving directory '/root/inclavare-containers-0.6.4/verdictd/dist'
1487
make[1]: Leaving directory '/root/inclavare-containers-0.6.4/verdictd'
1488
make: *** [Makefile:29: package] Error 1
1489
Cleaning up with exit code 2 ...

Please fix the build error in link:
error: Failed build dependencies:
	cmake is needed by verdictd-0.6.4-1.al7.x86_64


Add gen2-face-detection example

Added gen2-face-detection example, NN has input shape 160x120. Camera preview size is set to 640 x 480 and is resized with ImageManip to input shape. Result is shown on the camera preview, so accuracy might be a bit worse, but will add NN with bigger input shapes in the future.

The GIF makes me think "How you'll feel when you back the OAK-D-Lite KickStarter"


BUG: Silent Python crash with itk.MeshToPolyDataFilter

Summary
Attempting to convert an itk.Mesh to vtk.vtkPolyData results in a silent crash.
Steps to Reproduce
Running these steps in a Python shell causes the program to stall and crash.
import itk
import itkwidgets
mesh = itk.meshread('path/to/mesh',itk.F)
pd = itk.MeshToPolyDataFilter.New(mesh)

No error message is logged and the program exits silently.
I am not able to reproduce when I build locally with ITK and ITKMeshToPolyData, only when using the itk-meshtopolydata PyPi distribution.
Version
itk                           5.3rc4
itk-core                      5.3rc4
itk-filtering                 5.3rc4
itk-io                        5.3rc4
itk-meshtopolydata            0.8.1
itk-numerics                  5.3rc4
itk-registration              5.3rc4
itk-segmentation              5.3rc4
itkwidgets                    0.32.1

Platform
Windows 10
Python 3.8.5
Other notes
xref https://github.com/InsightSoftwareConsortium/itkwidgets/issues/448

Appears to be resolved in the v0.8.2 release. Unable to reproduce the issue with most up-to-date packages. Perhaps an issue somewhere in previous release packaging?


Get system version base just on first system module

This should solve Electron firmware with three system modules to be detected correctly

Coverage decreased (-0.1%) to 92.328% when pulling e18e7d2b658733f65769fc3ce25e650a52e1b5f2 on feature/less-restrictive-version-detection into 56609528b20cef1ce5a492798f4d48126b019bf0 on master.


⚠ Bofrost.lu has degraded performance

In 9ca5643, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1984 ms

Resolved: Bofrost.lu performance has improved in 4cbc4c1.


Combining forces?

Hi,
I am the author of the dicomParser project which you can find here:
https://github.com/chafey/dicomParser
Before writing this library I considered yours but decided not to use it because of my focus on the browser at the time (I wasn't sure how or if I could repurpose your library into the browser).  Anyway, I am finding myself needing to do more DICOM server side and am thinking now would be a good time to adopt ES6.   I am thinking that starting a new project would be best to avoid breaking the API for the users of my current library and wanted to see if you would be interested in collaborating on this effort.  Please drop me an email -<EMAIL_ADDRESS>

Hi Chris and Hi Juergen -
I'm also very interested in javascript dicom parsing and have been looking at Chris' dicomParser and at this dicom module.
Some of my colleagues and I have been experimenting with a different approach of using emscripten to cross-compile dcmtk.  We call it dcmjs and you can find it at http://dcmjs.org.
There are many tradeoffs, of course, like the fact that dcmjs.js is huge for a javascript file and the project is still in proof-of-concept stage.  But on the other hand it is based on very solid dicom infrastructure (dcmtk) that can handle a lot of the oddball special cases found in the real world.  New code that we're working on in dcmtk, like multiframe management and segmentation object support also becomes instantly available in C++ and javascript with compatible APIs.
What may be a good solution is to use dicomParser (or this dicom?) as a first choice but delegate to dcmjs for things that require more heavy-duty operations (i.e. load it in the background so pages can load quickly and parse most dicom files).
@chafey I'm also interested in cornerstone and maybe we should start emailing about our mutual interests.
Best,
Steve<EMAIL_ADDRESS>

Hi Steve,
I got an email response from Juergen stating that he has no plans to move this library forward.  Please drop me an email<EMAIL_ADDRESS>to continue a discussion about how we can work together

I do not consider any major rewrites at this point in time, I simply don't have the time.
For the time being, I am not working  on new features like dicom networking.
I will fix issues  ... when I get around to it.
If I would need some feature for a paying project, I would implement it.  At the moment I don't need anything but DICOM parsing in Javascript, and I don't expect that to change.

Thanks for the update - makes perfect sense.
On Thu, May 21, 2015 at 7:22 AM, grmble<EMAIL_ADDRESS>wrote:

I do not consider any major rewrites at this point in time, I simply don't
have the time.
For the time being, I am not working on new features like dicom networking.
I will fix issues ... when I get around to it.
If I would need some feature for a paying project, I would implement it.
At the moment I don't need anything but DICOM parsing in Javascript, and I
don't expect that to change.
—
Reply to this email directly or view it on GitHub
https://github.com/grmble/node-dicom/issues/5#issuecomment-104233783.


[ MB-9033 ] Update react-scripts v5.0.1

Jira ticket for this change
Summary
This patch updates react-scripts to version v5.0.1 in order to update the
build tool chain for this repository. This include various changes to our build
and test system and updates Webpack to version 5 among other changes.
There will be more description to come in the future.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

Messages

:book:
:link: MB-9033

  Generated by :no_entry_sign: dangerJS against 3fbd3f2926b7d3c4a45f119be93a4747c553c31a

The client test failures happening here seem related to the new regression tests around code coverage introduced in #12600. I'll have to investigate a bit more tomorrow I may be wrong here.

On second thought it doesn't seem like that's the case. Something else is happening here.

Now that I've pulled in master, I've realized that now the Client Test Coverage is causing me to be stuck. Will tackle this more tomorrow, but this is now ready to merge in after #8713 but we'll need to address the test coverage issue 😄
https://app.circleci.com/pipelines/github/transcom/mymove/42713/workflows/25ffe84a-2935-4055-96ba-a5ce0f6449e7/jobs/668667

@juliemoon good news. I don't believe there are any more Happo diffs beyond the ones in #8713 even though we are on USWDS 2.13.1. This means less Happo diffs to focus on. 😌

Had to rebase and force push due to conflicts in the yarn.lock file

@juliemoon there aren't any actual design changes in this PR. The files that I modified here are importing other libraries due to how webpack 5 works. Take a look at the Happo diff on this but otherwise, you should be able to give us the Approval without needing to dig too deep. These updates caused me to update a large number of files and our CODEOWNERS file had those listened under Design Review.


Release 0.10.1

Closes: #375

Codecov Report
Patch and project coverage have no change.

Comparison is base (c5235a0) 69.55% compared to head (00de490) 69.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #376   +/-   ##
=======================================
  Coverage   69.55%   69.55%
=======================================
  Files          20       20
  Lines        2989     2989
=======================================
  Hits         2079     2079
  Misses        910      910

Impacted Files
Coverage Δ

src/ingest.rs
83.57% <ø> (ø)

src/ingest/tests.rs
91.37% <ø> (ø)

src/publish.rs
66.66% <ø> (ø)

src/publish/tests.rs
97.68% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@kimhanbeom CI 실패하였는데 확인해 주세요.


npm install --global --production windows-build-tools breaks intermediately

OS Version:- Windows Server 2012 R2
NPM Version :-  5.6.0
Node Version:- v8.11.4
Logs:-

<EMAIL_ADDRESS>postinstall C:\Users\Administrator\AppData\Roaming\npm\node_modules\windows-build-tools
node ./dist/index.js
module.js:549
throw err;

^

Error: Cannot find module 'chalk'
at Function.Module._resolveFilename (module.js:547:15)
at Function.Module._load (module.js:474:25)
at Module.require (module.js:596:17)
at require (internal/module.js:11:18)
at Object. (C:\Users\Administrator\AppData\Roaming\npm\node_modules\windows-build-tools\dist\start.js:3:17)
at Module._compile (module.js:652:30)
at Object.Module._extensions..js (module.js:663:10)
at Module.load (module.js:565:32)
at tryModuleLoad (module.js:505:12)
at Function.Module._load (module.js:497:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: node ./dist/index.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Administrator\AppData\Roaming\npm-cache_logs\2018-09-14T04_53_30_151Z-debug.log
And this won't happen every time. Sometime it goes well and build tool install successfully.

Are you able to verify that npm actually didn't install chalk? Chalk is a dependency and should always be available - if it isn't, it appears that npm was unable to actually install the module properly... 🤔

This is what I have found on package.json on "windows-build-tools"
devDependencies": {
    "@types/chalk": "^2.2.0",
    "@types/fs-extra": "^5.0.4",
    "@types/jest": "^23.3.1",
    "@types/node": "^10.9.4",
    "jest": "^23.5.0",
    "ts-jest": "^22.4.6",
    "tslint": "^5.11.0",
    "tslint-microsoft-contrib": "^5.2.1",
    "typescript": "^3.0.3"
  }

@Aamir010 package.json just tells npm (or yarn) what it should install (and you'll see chalk in the dependencies section). The installed modules are placed inside node_modules, so you can look inside there to see what's installed. The problem you are seeing is very likely caused by intermittent npm problems (perhaps caused by network conditions).

You may also want to consider upgrading to the latest npm  (6.4.1 at the time of writing), which fixes a number of problems similar to what you experienced. See https://github.com/npm/cli#windows-install-or-upgrade

@jacobq Thanks let me give it a try by upgrading npm.  Do you think is there related to OS version as such ?

@Aamir010 I'm not sure I understand your question. Refer to the link above for information about how to upgrade npm. Personally, I use nvm-windows to manage NodeJS installations on Windows so usually just upgrade NodeJS to get the latest npm rather than upgrading it in place.


Fix details for Cass Ballenger's first term

Ballenger was elected mid-term (see http://bioguide.congress.gov/scripts/biodisplay.pl?index=B000104, https://en.wikipedia.org/wiki/Cass_Ballenger etc)

Thanks!


xplad cannot find Ledger device

Related to this issue:
https://github.com/cosmos/cosmos-sdk/issues/18423
We got the error below while adding key from Ledger in Mac
$ xplad keys add ledgerkey --ledger --coin-type=118
Error: failed to generate ledger key: failed to retrieve device: ledger nano S: hidapi: failed to open device

If we update go.mod to use zondax/ledger-go v0.14.3 instead of the v0.14.1, it is resolved. Please consider updating the package.

Please git checkout release/v1.3.x, then compile and run as root.


[ERROR] thread 'main' panicked at database/src/db/conn_builder.rs:117:114:

Describe the bug
/home/admin/.rusty-kaspa/kaspa-mainnet/logs/rusty-kaspa_err.log:
 `Result::unwrap()` on an `Err` value: Error { message: "IO error: While lock file: /home/admin/.rusty-kaspa/kaspa-mainnet/datadir/meta/LOCK: Resource temporarily unavailable" }

/home/admin/.rusty-kaspa/kaspa-mainnet/logs/rusty-kaspa.log:
2024-05-14 14:43:09.526+01:00 [ERROR] thread 'main' panicked at database/src/db/conn_builder.rs:117:114: called `Result::unwrap()` on an `Err` value: Error { message: "IO error: While lock file: /home/admin/.rusty-kaspa/kaspa-mainnet/datadir/meta/LOCK: Resource temporarily unavailable" }

My kaspad is launched by systemd, this is my service file:
[Unit]
Description=kaspad
After=network.service

[Service]
User=admin

ExecStart=/usr/local/bin/kaspad --utxoindex -d=warn --disable-upnp
Restart=on-failure
RestartSec=100s

[Install]
WantedBy=multi-user.target

systemctl status kaspad give me :
kaspad.service - kaspad
     Loaded: loaded (/etc/systemd/system/kaspad.service; enabled; preset: enabled)
     Active: active (running) since Tue 2024-05-14 14:26:29 BST; 25min ago
   Main PID: 2962 (kaspad)
      Tasks: 53 (limit: 9259)
        CPU: 27min 45.675s
     CGroup: /system.slice/kaspad.service
             └─2962 /usr/local/bin/kaspad --utxoindex -d=warn --disable-upnp

May 14 14:26:29 raspberrypi systemd[1]: Started kaspad.service - kaspad.
May 14 14:26:36 raspberrypi kaspad[2962]: 2024-05-14 14:26:36.779+01:00 [WARN ] P2P, got reject message: P2P pr>
~
~

and journalctl - kaspad returns:
May 14 14:26:29 raspberrypi systemd[1]: Started kaspad.service - kaspad.
May 14 14:26:36 raspberrypi kaspad[2962]: 2024-05-14 14:26:36.779+01:00 [WARN ] P2P, got reject message: P2P pr>

To Reproduce
Steps to reproduce the behavior:

launch kaspad --utxoindex -d=warn --disable-upnp as a systemd daemon
take a look at the logs
systemctl status kaspad
joornalctl -u kaspad

Expected behavior
I expected no errors in the logs.
Desktop (please complete the following information):

OS:  RaspberryPiOS lite
Kaspad version: 0.14.1
ARM architecture (Raspberry Pi5)

Additional context
The node seems to sync despite the error I get in the logs as evidenced by the growing from minute to minute size of datadir

No idea what the cause can be right now, but

can you please try the following? (this is more of a shot in the dark, but could be somehow PI-latency related..)

[Service]
Type=simple

your systemd service declares User=j but the logs are failing to access /home/admin/.rusty-kaspa ...  how can this be?    ...are you sure another instance is not already running?  (ps -ax | grep kaspad to check) ...

So far this looks environmental....

No idea what the cause can be right now, but

can you please try the following? (this is more of a shot in the dark, but could be somehow PI-latency related..)

[Service]
Type=simple

your systemd service declares User=j but the logs are failing to access /home/admin/.rusty-kaspa ...  how can this be?    ...are you sure another instance is not already running?  (ps -ax | grep kaspad to check) ...

So far this looks environmental....

user is admin not j.  I didn't want to leak my username here. I am positive, there is only one instance running.

No idea what the cause can be right now, but

can you please try the following? (this is more of a shot in the dark, but could be somehow PI-latency related..)

[Service]
Type=simple

I get this:
May 14 15:20:29 raspberrypi systemd[1]: /etc/systemd/system/kaspad.service:4: Assignment outside of section. Ignoring.

Are you able to run it manually, outside of systemd?

Are you able to run it manually, outside of systemd?

No. kaspad --utxoindex -d=warn --disable-upnp
gives me:
2024-05-14 17:30:29.152+01:00 [ERROR] thread 'main' panicked at database/src/db/conn_builder.rs:117:114: called `Result::unwrap()` on an `Err` value: Error { message: "IO error: While lock file: /home/admin/.rusty-kaspa/kaspa-mainnet/datadir/meta/LOCK: Resource temporarily unavailable" }
thread 'main' panicked at database/src/db/conn_builder.rs:117:114:
called `Result::unwrap()` on an `Err` value: Error { message: "IO error: While lock file: /home/admin/.rusty-kaspa/kaspa-mainnet/datadir/meta/LOCK: Resource temporarily unavailable" }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Exiting...

Can you delete the ~/.rusty-kaspa folder and try again? Do you get the same error?

I just realized that I did have multiple instance running, I had two separate service files of two different names for the same kaspad daemon. ps -ax | grep kaspad was not detecting it because they were restarting infrequently, every 100s. Thank you so much for your help and apologies for wasting your precious time.


Glint doesn't always recognize component

I have noticed recently that sometimes I will have red squigglys over a component that I know I have typed. Hovering over provides the following
Element implicitly has an 'any' type because expression of type '"FlSelect::WithTranslationKey"' can't be used to index type 'Globals'.
  Property 'FlSelect::WithTranslationKey' does not exist on type 'Globals'.glint:ts(7053)

Sometimes, I see the red squigglys and sometimes it works correctly and can find the component.
Here is what I see in the logs.
 Could not find source file: '/Users/adambaker/Development/gavant/spin-web/spin/app/pods/components/fl-select/with-translation-key/component.ts'.
    at getValidSourceFile (/Users/adambaker/Development/gavant/spin-web/spin/node_modules/typescript/lib/typescript.js:153646:29)
    at Object.getSyntacticDiagnostics (/Users/adambaker/Development/gavant/spin-web/spin/node_modules/typescript/lib/typescript.js:153896:52)
    at GlintLanguageServer.getDiagnostics (/Users/adambaker/Development/gavant/spin-web/spin/node_modules/@glint/core/lib/language-server/glint-language-server.js:83:29)
    at Timeout._onTimeout (/Users/adambaker/Development/gavant/spin-web/spin/node_modules/@glint/core/lib/language-server/binding.js:72:56)
    at listOnTimeout (internal/timers.js:554:17)
    at processTimers (internal/timers.js:497:7)

This is particularly frustrating as it makes it near impossible to figure out what you have actually typed correctly and what still needs work

We chatted a bit in Discord about strategies for tracking down a reproduction here, but an additional thought I had is that it might be useful to emit a synthetic diagnostic from the language server in this situation to let the user know that we aren't properly checking the file. That's obviously a bandaid for the real issue, but might at least be useful to make it clear that something's wrong.

@bakerac4 #250 updated the language server so that if an exception is raised trying to get diagnostics for a file, we emit an error for that file alerting you to the fact that that's happening. I know it's not as good as actually tracking down the underlying issue, but at least that should hopefully make it more obvious when diagnostics might be missing.

@bakerac4 are you still running into this issue?

Haven't paid attention to it in awhile as its just been something I dealt with. You can close for now if you want, and I'll re-open if I see it again?

👍  Sounds good. Hopefully if you start running into it again we'll be able to track down a way to actually reproduce the issue

Currently running into this...


Update stable version in repo from 3.0.1 to 3.1.0

Think about update stable version to 3.1.0
Version: stable=3.0.1 (204 days old) => latest=3.1.0 (15 days old)
Installs: stable=4884 (66.12%), latest=410 (5.55%), total=7387
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 3.1.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 3.1.0 but the current stable version is already 3.1.0.
So this issue should be closed.
@mcm1957 for evidence


Add quotes around asg_name variable

Looks like we might need to change https://github.com/tsupertramp/etcd-aws-cluster/blob/fix-autoscaling-groups-with-spaces/etcd-aws-cluster#L156 as well.

Thanks - Updated the Pull Request. If you want I can create a new one for "clean" git commit message history.

Sure might as well squash it down. I spun up my test cloudformation template and everything looked fine.  I wasn't able to replicate the spaces due to limitations imposed by CloudFormation but I'm confident your change should work just fine.

Even being obvious just by reading the diff, I confirm that it does work with spaced asg names.

@tsupertramp any reason you closed this?

Yeah i made a new one :) fixed everything in one commit (as it is cleaner)
https://github.com/MonsantoCo/etcd-aws-cluster/pull/21

ah gotcha


add new parameter LIGHT_EN_BLINKM to start blinkm lights

This suppose to make start-up cleaner for some boards that don't uses some sensors but try do start them because they are on the common rc.sensors.
some disscution is here #12409
This pr focus the blinkm leds.

After this pr I noticed a reoccuring error in the startup console output (seen when running dmesg on the MAVLink shell). There's always the error

Because in
https://github.com/PX4/Firmware/blob/22e4e85eeb1d2688a05f5b74cfbf936539264b1e/ROMFS/px4fmu_common/init.d/rcS#L205
the parameter gets accessed even though it might very well be that blinkm is not compiled for the platform you're running e.g. mine.


Run xcodebuild from ShellScript task?

Issue Description
I have a bash script in my project root that prepares my build artifacts with a call to xcodebuild, followed by moving some files around and pkgbuild commands. When run locally, this works great. When run on the hosted macOS agent, my xcodebuild fails immediately. Is it possible to run xcodebuild in this way on the hosted macOS agent? My config file is as follows:
jobs:
- job: OSX
  pool:
    vmImage: 'xcode9-macos10.13'
  steps:
  - script: sleep 15
  - task: ShellScript@2
    inputs:
      scriptPath: ./scripts/run.sh
      args: build wov

Note– the sleep 15 was a sort of last ditch attempt after reading various blogs that suggested that the vm environment was not all the way setup by the time xcodebuild was running. It didn't work :)
Task logs
log_13_6.zip
Error logs
2018-09-19T12:44:48.8872400Z ##[error]/bin/bash failed with return code: 65
2018-09-19T12:44:48.8974600Z ##[error]Bash failed with error: /bin/bash failed with return code: 65

My bad... of course this was an error on my part. One of my header include paths got confused by being cloned into an arbitrarily named directory. Since correcting that, I have been able to build via xcodebuild on the command line in Azure Pipelines successfully.


Evaluate a VMLA replacement

VMLA has always forked off quite early in the compilation flow from codegen (HLO) but with linalg-on-tensors landing and HLO being moved higher up in the stack (eventually outside of IREE entirely) it becomes an even bigger issue. This issue is tracking how to preserve the benefits we get from having VMLA as a reference backend while aligning closer with our other codegen targets (SPIRV, PTX, LLVM) to unwind the divergence in paths.
Key benefits of a VM-based reference backend (currently VMLA):

built on our VM, so no external dependencies
runs anywhere C and the rest of IREE runs
extremely low overhead (KB of RAM and ROM) so usable for bringup on embedded devices
1:1 with the MLIR, no additional toolchain to debug if there are issues
easy to add new methods as no external frameworks/libraries/compilers/runtimes need to be modified

VMLA is really two parts: the use of the IREE VM for calling functions and managing values and the VMLA module exporting a set of HLO-like operations. That the IREE VM is used is orthogonal to what the VMLA set of functions are: it's possible to use the VMLA functions with the VM running via emitc or via a completely different route (VM->LLVM AOT, wasm, etc). What's important to this issue is what the set of functions is and how we get to that set via the common codegen path in IREE. How those functions are called may change over time (or have multiple routes to get to it).
One route we can go is to simplify things and ignore the vector dialect: instead, we would lower to std+scf+math by scalarizing all of the vector ops (I believe there is a pass that does this). We can then use the VM builtins for control flow and basic arithmetic (loops, etc), add the planned but not yet implemented f32 extension to the VM for scalar math (~10-15 simple instructions here like the I64 above it), and then dump any helpers we need for fancier math into a math module we can reuse. This won't be fast, but it could be a good baseline to bring up and would always provide a fallback. It would flesh out the compiler path for a sibling target to SPIRV & co that takes in the vector dialect.
A follow-on to that - or maybe something even easier to do together - is picking some of the vector ops and mapping those to functions on the new module. In this way we would be treating the VM+module as a virtual variable-length vector ISA. While still in the vector dialect with nice vector<> types we have simple patterns that turn them into foo.vector.* ops, similar to how we do HLO->VMLA today. The set of ops is rather small, and we can use similar ones to LLVM to allow us to rip off those patterns. There's also some standard (or now "math"?) ops that operate on vectors, but the set is reasonable (add, div, etc).
Here's some relevant tests showing the vector ops:

vector.reduction -> llvm.intr.vector.reduce.* (llvm intrinsics)
vector.matrix_multiply -> llvm.intr.matrix.multiply (llvm intrinsics)

A lot of the ops in vector->llvm above are optional; for example, all the masked stuff is something we could punt on and legalize away in the future (the current SPIRV path doesn't support these, for example). The lowerings to LLVM for a lot of the operations are also lower level than we need. Instead, things like vector.transfer_read may map directly to our module ops just like above. We could also rip off patterns from the SPIRV world, where the load/store semantics are much higher level than LLVM and more of what we want:
https://github.com/google/iree/blob/416891602125f4cd244e226338793150bfc46f4f/iree/compiler/Conversion/LinalgToSPIRV/test/vector_to_gpu.mlir#L3-L20
One of the few open questions is how to represent the vectors in the VM. The easiest thing is to have a !foo.vector<...> type that is backed by stack memory. We'd then take the memrefs that float around, bufferize them using the upstream bufferization (or whatever ntv@ is doing now in his sandbox), and then load/store memref<->vector is something we externalize.
I think it may be useful to stand up a skeleton of the required components and then work bottoms-up to start eating away at the incoming IR:

[x] pick a name
[x] add a new top-level IREE dialect with that name - which would also have the vm imports.mlir file
[x] add a new IREE HAL target backend with a dummy pass pipeline
[x] add a new iree/modules/ native module
[x] add a new iree/hal/local/ loader (like the vmla loader) that wires the two together

From there, we can start fleshing out the pass pipeline to go from linalg-on-tensors down to vector+scf, scf->std, std->vm, and producing the final VM binary. We should be able to get a no-op executable running end-to-end.
From there we solve some of the first issues:

[x] hal.interface.* goo (like vmla)
[x] memref -> !foo.buffer, add type at runtime
[ ] vector -> !foo.vector, add type at runtime
[ ] vector load/store ops
[ ] pick a vector op to prototype (matrix multiply, reduce, etc) and plumb it through

The long tail after that is finding out the ~10 other vector ops we need, the ~10-15 std/math ops on vectors, and plumbing those through to runtime. Most will fall into the unary/binary/ternary template category and be fast to plumb.

+1 for the value of having a simple backend that doesn't have any toolchain/driver/whatever deps. I've really appreciated having a reference backend. Not sure if this would be prohibitively slow, but do we need the vector part of this? Could we just unroll linalg generics to loops of std/math/whatever scalar ops and then implement the handful of scalar ops?

Yep! That's the base approach with the vector stuff being something we sprinkle in where required. Many cases probably won't matter too much, but matmul and conv in particular are dealbreakers (like 1000x slower, minutes+ to run kind of bad). Those are the easiest vector ops to snoop on, though, as they'll end up being a 1:1 replaceOpWithNewOp. The more complex vector stuff (masked operations, reductions, etc) are all just optimizations we can do as needed. One benefit we'll have here is that we'll be able to run VMLA multithreaded, so even if it's 4x slower than the old C++ kernels if you run it on 4 threads you'll end up about the same :P

Thanks for the great (as always :)) write up Ben, I think we can very quickly bring up scalar pass then we add the vector intrinsic support later once we solve the hal.interface, memref -> !our_new_dialect.buffer and have the scalar part running smoothly (You know it feels good getting the X- perf :p)
Its going to be very interesting looking at performance of the VM executing STD level IR specially the good performance we have on other backends relays on the good and very heavy optimization done at LLVM and later codegen for that particular backends (e.g aarch64 backends eliminates  transposes in some cases by vertical loads registers + broadcast). which might build a case for MLIR level optimizations (VM-O3 !?).

The VM will never be fast as it's just a simple interpreter and WASM with a full JIT will be much faster but still not native (1-5x slower depending on the instruction mix) - so I can confidently say we will need a bulk of the larger operations to be the variable-length vector style. We'll want to plan less for how to perform optimizations on the VM and more for how to represent them as variable-length vector intrinsics (which should hopefully be easy to get out of the vector dialect). We lose a lot of the value here if we have to build out a big code path/significant custom optimizations/etc that's specific to this target or the VM - the goal is to slice off the common path and add as little as possible to it :)
Knowing what those intrinsic-style calls are like is important as it determines how we represent data (like !vmla.buffer) and we want to have that in mind as we setup all the stuff. We don't need to know the exact set, but having at least one will ensure we are building something that scales out to the rest instead of something that only works for the ultra-slow path that we know we can't use (full scalars running raw on VM). Like, "how do we get elementwise add + mul + tanh into something that uses add/mul/tanh intrinsics instead of a scalar loop" is the real hard/fun question we need to stitch together - everything after that is just plumbing :)

I've done a first pass and think I have a decent idea of where the dragons lie and have a good portion of the plumbing done. There's some larger specific issues to work out that I'll file dedicated issues for and I'll work to land the skeleton ASAP so we have something to iterate on.

Filed some enhancements that are generally good things (both for normal host VM and VMVX use):

#5544: Add fused VM ops for common sequences of addressing arithmetic.
#5545: Add a few dedicated bytecode->module shims for faster calls.
#5546: Implement the F32/F64 VM instruction extensions.

Created a project to track: https://github.com/google/iree/projects/30
And filed the major work items that'll follow after the skeleton:

#5547: Upstream improvements for scf.for folding on loop ranges.
#5548: Get work distribution working properly for VMVX (and others).
#5549: Lower vectors into VMVX.

These are things we'll want to brainstorm on and some may just resolve themselves with work already in-flight (like #5548 that ahmed may already be fixing for the LLVM path).

VMVX is landing; going to close this and focus on the required improvements in #5880.


my volley image view doesnot scale itself according to density in WCViewPAger

my volley image view doesnot scale itself according to screen density in WCViewPAger but when i use v4.viewpager it does(But wrapcontent doesnt work)

I think this might happen if pages view height isn't "fixed" in dips... Can you add an example?


Move user's guide information from README.md to wiki

The README.md file should provide just the following:

Overview
Installation instructions

The rest of README.md should be migrated to the github wiki.

Done.


KNOX-3077 - Add pac4j.cookie.max.age param

What changes were proposed in this pull request?
This PR introduces a new parameter pac4j.cookie.max.age for the Pac4J provider that enforces cookie age on the cookies created by the pac4j provider.
e.g.
         <provider>
                  <role>federation</role>
                  <name>pac4j</name>
                  <enabled>true</enabled>
                  <param>
                      <name>pac4j.callbackUrl</name>
                      <value>https://www.local.com:8443/gateway/knoxsso/api/v1/websso</value>
                  </param>

                 <!-- Unit is seconds -->
                  <param>
                      <name>pac4j.cookie.max.age</name>
                      <value>180</value>
                  </param>
            .........

Note that these value needs to be same as knoxsso.cookie.max.age if  you want Knox session to expire at the same time.
How was this patch tested?
This patch was tested locally.

Please don't forget to file a Doc Jira with this new config.


[EFRS-1091] Specific Image + Invalid API Key = API Hanging

Fixed bug

I have read the CLA Document and I hereby sign the CLA

recheckcla


Add an Overview page to the Optimize section

The Optimize section should have an overview page explaining what the "optimize" phase is, similar to what we already have for the build section

Done in https://github.ibm.com/IBM-Q-Software/iqp-channel-docs/pull/807


ide support js language comment ?

ide support js language comment ?

If you mean that whether the editor supports js or java style comment，it doesn't. Use lua style comment instead.@qssq

I found a lot of versions of textwarrior from the Internet, and I didn't find the answer. I can't display comments using the lua version.
//https://github.com/luoyesiqiu/simpleC/tree/master/app/src/main/java/com/myopicmobile/textwarrior/common
//https://github.com/LingSaTuo/CreateJS
//https://github.com/brnogz/TextWarriorLibrary
//https://github.com/qtiuto/lua-for-android english
//https://github.com/XsJIONG/VIDE
//https://github.com/fengdeyingzi/CodeEditor
//https://github.com/nirenr/AndroLua_pro

There is a part of the code inside.
Can't understand, don't know how to modify it to be compatible with js, realize // turn green

/**

Translates characters to character classes
*/
Private static final String ZZ_CMAP_PACKED =
"\11\0\1\1\1\2\1\66\1\66\1\3\22\0\1\1\1\44\1\41" +
"\1\43\1\0\1\51\1\0\1\42\1\53\1\54\1\50\1\10\1\60" +
"\1\16\1\14\1\52\1\11\11\6\1\62\1\61\1\46\1\15\1\45" +
"\2\0\4\13\1\7\1\13\21\5\1\12\2\5\1\4\1\64\1\55" +
"\1\63\1\5\1\0\1\17\1\22\1\34\1\21\1\24\1\32\1\5" +
"\1\37\1\31\1\5\1\25\1\27\1\5\1\20\1\26\1\36\1\5" +
"\1\23\1\30\1\35\1\33\1\65\1\40\1\12\2\5\1\56\1\0" +
"\1\57\1\47\6\0\1\66\u1fa2\0\1\66\1\66\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\ Uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\uffff\0\udfe6\0";

/**

Translates characters to character classes
*/
Private static final char [] ZZ_CMAP = zzUnpackCMap(ZZ_CMAP_PACKED);

/**

Translates DFA states to action switch labels.
*/
Private static final int [] ZZ_ACTION = zzUnpackAction();

can you hellp me?  😔

Long comment in lua is this style --[[  this comment can be multi-line ]], and short comment starts with --coment. @qssq

I know, I mean, I want to change it to js ide, then support // and /**/, I am having problems and can’t start.

It's not an easy task to convert the lua lexer to js lexer cause its definition is hard to change and many attributes is hard coded. I suggest you finding out another proper lexer to do so. Actually, I just copy the editor from another project for short. @qssq


POLIO-951: Add a number field "Percentage of the target population to be covered" that will be encoded manually

Explain what problem this PR is resolving

Add in risk assessment a number field "Percentage of the target population to be covered" that will be encoded manually

Related JIRA tickets : POLIO-951
Self proofreading checklist

[x] Did I use eslint and black formatters
[x] Is my code clear enough and well documented
[ ] Are my typescript files well typed
[x] New translations have been added or updated if new strings have been introduced in the frontend
[x] My migrations file are included
[ ] Are there enough tests
[ ] Documentation has been included (for new feature)

Changes

Add percentage_covered_target_population in round model
Add migration to create the field in the DB
Add text input field in risk assessment form
Define the field as a number in yup
Add label translation

How to test

Go in campaigns
Edit or create a campaign
Open risk assessment tab
Check if the percentage covered target population

Print screen / video

@cypress


Formatting inside pre elements

For a project, I was using axohtml, but I wanted to try out this crate because @yoshuawuyts has been tooting about it :)
Anyway, I ran into an issue. I'm trying to generate a lot of pre-rendered text. However, to_string seems to very neatly format the html generated. That's nice and all, but when preformatted text is reformatted, it loses all its preformatting! In my case, I end up with a ton of extra spaces and newlines which basically mean html is unusable for me. How we should solve this? I don't know. Probably don't do autoformatting inside
s

Hey there! Oh yeah, that doesn't sound great. Can you share a minimal repro so we can fix it?
As a hot fix we could probably revert the inline formatting.  But I'm thinking perhaps a better solution could be to disable indentation for all text_content elements instead? Would that have fixed your issue?

Or actually, now that I think about it —
 elements are probably the main element whose content we don't want to auto-indent I think?

Sure, I'll make an example (maybe even a PR if you'd like :). In any case, this is hard because you can always set the css white-space: pre and then have any element act like a pre. So ideally, we'd format nothing. But that's not nice either

Maybe a parameter on the builder allowing you to disable formatting on that element?

This is your minimal example:
use html::text_content::PreformattedText;

fn main() -> std::io::Result<()>{
    std::fs::write("test.html", PreformattedText::builder().text("test").build().to_string())
}

It outputs

    test

While it should output

test

As a test case:
#[cfg(test)]
mod tests {
    use html::text_content::PreformattedText;

    #[test]
    fn test_pre() {
        assert_eq!(PreformattedText::builder().text("test").build().to_string(), "
test
".to_string())
    }
}

What would you say to
PreformattedText::builder().text("test").autoformat(false).build().to_string()

(and maybe specifically for pre elements default autoformat to false)

The alternative is a .render_not_formatted or similar, but that wouldn't format the entire tree. Advantage is that we don't need to add a bool to every element.

thanks Yosh!


Fix #477 msbot secret -n outputs object

Fix for #477

Pull Request Test Coverage Report for Build 1052

0 of 3 (0.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 68.607%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/MSBot/src/msbot-connect-generic.ts
0
1
0.0%

packages/MSBot/src/msbot-secret.ts
0
1
0.0%

packages/MSBot/src/msbot-update-generic.ts
0
1
0.0%

Totals

Change from base Build 1048:
0.05%

Covered Lines:
3840

Relevant Lines:
5353

💛  - Coveralls


Validate the encrypted AMI id when updating

When updating an encrypted AMI, make sure that the AMI exists before
proceeding.

👍


Add link to blog post describing idempotence pattern

@udidahan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@MikeWasson: I'm sorry - only the author of this article, @dragon119, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@MikeWasson thanks very much 😃


Update Composer dependencies (2021-08-04-00-08)

Visual regression test passed!
View the visual regression test report


L5r ffg sheet

Include the name of the sheet you made changes to in the title.
Changes
Provide a few comments about what you have changed.
Additional comments
Anything else we should know?.
Roll20 Requests

If changes fix a bug or resolves a feature request, be sure to link to that issue.
For pull request that change multiple sheets please confirm these changes are intentional for all sheets. This will help us catch unintended submissions.
When updating existing sheets if you are changing attribute names please note what steps you have taken, if any, to assist players in keeping their data.

Tell me if I did that right. I'm self taught with github and I'm still flailing about a bit.

It looks good to go. :) Github has a learning curve so we all learn together.


Add support telemetry controller

What this PR does / why we need it:
Adds the internal gardener-shoot-telemetry project into the test-infra project
Originally authored by @dkistner
Special notes for your reviewer:
We also need to discuss codeowner ship.
Release note:

Add telemetry-controller to test-infra

code ownership will be taken by the test-infra maintainers as discussed offline.


Request Support for VILA

Thanks for open-sourcing the framework
One of the LMMs which has been doing great work and has been iterating their models is VILA : https://github.com/NVlabs/VILA
It would be great to add the support for the same

If its okay I can work on a the PR to integrate the same @junming-yang @kennymckormick ?

Hi, @amitbcp. Thanks for your work to support the VILA model. Please don't hesitate to contact us if you need any help.

Sure will try to wrap it within this week !!

Thanks for your contribution. VILA is supported now!


Cannot find module<EMAIL_ADDRESS>imported from /app/server/chunks/app/server.mjs

Hi. I am using Nuxt 3 for my application where I use this package. The problem is that after version 1.3.3. I started getting this error in production. From what I found, the proxy directory is missing from the build. I think this change is the cause: https://github.com/bufbuild/protobuf-es/pull/566
Do you have an idea how to resolve this?

Hey David, the directory is there in the published npm package. I assume that the error is raised by Node.js?
If you are using an older version of Node.js, it is possible that this is a bug in the module resolution that has been fixed in a later version. So I'd try updating Node.js to a latest v18 or v20.
If you set the environment variable NODE_DEBUG=module, Node will print debugging information for the module resolution. For a complex project, the debugging output alone might not show the smoking gun. If you can isolate the problem, we're happy to take a look if you can push it up 🙂

Hi. Thank you for looking into this. I made a small reproduction repo showcasing my problem.
https://github.com/DavidDomkar/bufbuild-protobuf-error-reproduction
If you clone and do yarn install && yarn build && yarn preview, you can see the webpage with the error on localhost:3000. The problem is, despite the proxy folder being in node_modules, when running yarn build, the build process creates another node_modules directory in .output/server/node_modules and there the directory is missing. I am using node 18. I think this is something related to how vite does treeshaking when compiling the server. But I do not know how to fix it properly.

Interesting, I see the same issue. I'm not familiar with Nuxt, but it looks like it is using Nitro for the server.
Nitro (or some dependency it uses) sees the "module" export we set, and deletes all other exports from our package. Node.js however will prefer the "import" export over "module", which is also set by our package. As a result, our package - as modified by Nitro - is broken on Node.js.
It looks like Nitro is tracking the issue in https://github.com/unjs/nitro/issues/1558. I think it would be helpful to provide them with your reproducible example.

Thank you for looking into the details, this was very helpful. I left the comment in the linked issue, hopefully, they will look into it as it is a Nitro issue.

Hi dear @timostamm and @DavidDomkar. Checking this repo/packages, it seems there is an issue with ordering of exports field conditions. Rollup prefers the module export condition where as you have to consider it is a nonstandard one but when a library is externalized, native ESM resolution in Node.js, prefers import condition which is a different one.
The fix is pretty straight forward you can see https://github.com/sanity-io/client/pull/318 in sentry package also https://github.com/sanity-io/client/pull/309 for more context. I will try to make a similar PR.
BTW just saw this lib, seems pretty awesome ❤

We can't really prefer "import" over "module", as described in https://github.com/bufbuild/protobuf-es/pull/611#discussion_r1386409260.
We might be able to modify our exports as follows:
  "type": "module",
  "exports": {
    ".": {
      "node": {
        "import": "./dist/proxy/index.js" // ESM wrapper specific to Node.js, re-exports CJS
      },
      "require": "./dist/cjs/index.js",
      "default": "./dist/esm/index.js" // pure ESM
    }

We will have to run more extensive tests before we can make this change (https://github.com/connectrpc/examples-es/pull/1002 for context), but it does look reasonable.
@DavidDomkar, to work around the issue, you can modify your package.json scripts as follows:
    "build": "nuxt build && yarn run postbuild",
    "postbuild": "cp -r node_modules/@bufbuild/protobuf/dist .output/server/node_modules/@bufbuild/protobuf",

This will restore the build artifacts of the package, until Nitro disregards the "module" condition for the node-server preset, or protobuf-es makes the exports change above (which will take some time to thoroughly test).

We can't really prefer "import" over "module",

I hope you revise this decision later. Because I have to emphasise regain, module export condition was never a standard and causes problems with ESM-native runtimes.

We might be able to modify our exports as follows:

This would also a good idea to at least support node.import export condition i suppose it will fix the bug. Only consider there are more ESM-native runtimes that support externalized native ESM dependencies. bun and deno. You might consider runtime-keys for the future.

until Nitro disregards the "module" condition for the node-server preset

Nitro does not disregards module condition. Node.js natively never regards it because standard condition is import for ESM. When trace used files. It is fooled with export condition as rollup resolver (for i guess traditional reasons) still support non-standard module condition and prefers it because your package sayis to prefer it. It will be extracted but in runtime Node.js looks for import condition which well does not works as file was not traced properly.
There are of course several solutions to suggest users fixing protobuf-es  dep issue but I'm afraid nothing is to fix particularly in Nitro other than patching incompatible/nonstandard packages or rollup behavior which I'm afraid is not a safe choice.

We can't really prefer "import" over "module", as described in #611 (comment).
We might be able to modify our exports as follows:
  "type": "module",
  "exports": {
    ".": {
      "node": {
        "import": "./dist/proxy/index.js" // ESM wrapper specific to Node.js, re-exports CJS
      },
      "require": "./dist/cjs/index.js",
      "default": "./dist/esm/index.js" // pure ESM
    }

We will have to run more extensive tests before we can make this change (connectrpc/examples-es#1002 for context), but it does look reasonable.
@DavidDomkar, to work around the issue, you can modify your package.json scripts as follows:
    "build": "nuxt build && yarn run postbuild",
    "postbuild": "cp -r node_modules/@bufbuild/protobuf/dist .output/server/node_modules/@bufbuild/protobuf",

This will restore the build artifacts of the package, until Nitro disregards the "module" condition for the node-server preset, or protobuf-es makes the exports change above (which will take some time to thoroughly test).

Thank you for providing the workaround, I will use it until this is resolved another way. I can confirm that the quick fix with the node export condition works for me when I patch the package in node modules.


November Infrastructure and Documentation Changes

This Epic is tracking several infrastructure/documentation issues we'd like to fix in November. Specifically:
#1260 and adding additional guides to the EvalML documentation (issues to come).

Dockerhub credentials added, closing epic


Update README

I have provided some changes in the README file . Kindly review it @sethshivam11
Fixes #37

@sethshivam11 Kindly assign me this and put label on this too.


fix(node): Log node p2p information to stderr as well so docker runners can see it

When the node starts up, we print p2p information of the node here.
We should also log out to stderr so docker container users can see it.

Would like to work on it.

Hi @renaynay ! Wondering if you had a chance to look at my PR!
Thx.

Closed until there will be complains


Click event for stacked Area chart

Hi,
I have a stacked area chart and i need each area should be clickable and based on the area clicked data1/data2/data3 respective route will be done.
If I set data.selection.grouped = true, I am able to click on the whole area but the click event is getting triggered for all data(data1, data2 and data3).
If I set data.selection.grouped = false, Click is available only on the data point, but I need the whole area should be clickable.
Please let me know how can i achieve this.

Hi,
The issue queue is to be used for reporting defects and problems with C3.js, in addition to feature requests and ideas.
It is not a catch-all support forum.
For general support enquiries, please use the Google Group at https://groups.google.com/forum/#!forum/c3js.
All questions involving the interplay between C3.js and any other library (such as AngularJS) should be posted there first!
Thank you !


Jesse: Watch (?-i:vRG4ZD)(?#fiverr)

Jesse requests the watch of the watch_keyword (?-i:vRG4ZD)(?#fiverr). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:vRG4ZD)(?#fiverr) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


[Question]: Usage of exclamation marks for ZodType readonly fields

First of all, thank you for this amazing package 🙏
I've been reading through your code and stumbled upon an unfamiliar syntax:

What do the exclamation marks do? What is this syntax / token called?

It's the non-null assertion operator, it's an operator that tells TypeScript that it's same to assume that an optional/nullable value exists. For example:
const asdf: string | null | undefined = "string";
const qwer = asdf!; // type: string

You should only use it if you're sure that the variable will be assigned to a real value (not undefined/null). Usually TypeScript is smart enough to figure this out automatically:
const asdf: string | null | undefined = "string";
if(asdf){
  asdf; // string
}

But in more complicated scenarios the typechecker may not be able to figure out what's going on.
I'm using it as a hack though. These properties (_type, etc) are never actually assigned to anything, they're just a convenient way to "store" the class's generic properties (e.g. Output, Input, and Def) so they can be easily retrieved from an instance.
const mySchema = z.string();
type schemaType = typeof mySchema['_output']; // string

// much easier than this...
type schemaType = typeof mySchema extends ZodType<infer Output, any, any> ? Output : never;

Because I never assign a value to _type, etc in the ZodType constructor, TypeScript complains, because by default it makes sure you assign values to every declared property. Adding the exclamation point tell TypeScript not to worry about it.


Security concern- Can't restrict api access while using AGM with WKWebview

I love AGM so much and I also use it in all my cordova(ionic)apps but I hadn't realized the security risk until today. AGM make uses of googlemaps browser API and there is no way to restrict the access as far as I understood. Could anyone please advice if I should use AGM only in web apps/websites and restrict google api access by HTTP referrers (web sites) ? If that's the case, I have to rewrite all my related code and use native maps/google places features.
Sorry that I know it's not the issue of AGM. I just need some guidance how to use AGM properly and securely.

@lolaswift glad that you enjoy using AGM but I don't understand your problem. This problem uses the google maps standard JS API. What do you mean by "restricting access"?

@lolaswift please reopen if you can provide the needed information


Improve the bug report template

Collapse the log output in our bug report template by wrapping it in <details>.
Sync the template with the Github issue template in .github/ISSUE_TEMPLATE/bug_report.md.
Fix some typos.

Codecov Report

Merging #2046 (9d5ab06) into main (09ff504) will increase coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head 9d5ab06 differs from pull request most recent head f04b798. Consider uploading reports for the commit f04b798 to get more accurate results

@@           Coverage Diff           @@
##             main    #2046   +/-   ##
=======================================
  Coverage   77.73%   77.74%
=======================================
  Files         420      420
  Lines       12846    12846
  Branches      588      588
=======================================
+ Hits         9986     9987    +1
+ Misses       2860     2859    -1

Impacted Files
Coverage Δ

...scala/at/forsyte/apalache/io/ReportGenerator.scala
0.00% <ø> (ø)

...a/at/forsyte/apalache/tla/lir/TlaLevelFinder.scala
100.00% <0.00%> (+2.63%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


chore: add inputs to demo app

:tada: This PR is included in version 7.22.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Threading not working as expected on Linux

Problem
When building on Linux the variable for threading gets set to 1 all through the CMake config, CMake headers and generated headers. No warnings or errors during compile but the audioManager does not get updated automatically as expected on a threaded version of the library.
Symptoms
All the examples, where the threaded library should make music play through to the end, requires an additional line of audioMgr->update() in the while(mysound->isPlaying()) block to work as expected.
Investigation
I followed the code to the point that I'm sure it's configured correctly and that pthreads gets included in the build. However I think the assistance of a main developer on this issue might get us to a resolution faster.
Solution
I'm prepared to spend time and effort on fixing this issue, seams logical since I'm building on Linux anyways, but would request the help of a developer to get into the code faster (no point in floundering around for a week ;o). Contact me if you can help. I'll get stuck in in the meanwhile...

I am not currently working on a linux machine however i will look into the issue you are having.

I had another, closer, look at the code today and I can see that in src/cThread.cpp lines 23 - 41:
bool cAudioThread::start() {
	cAudioMutexBasicLock lock(Mutex);

	if(IsInit)
		return IsInit;

        IsInit = true;

	#ifdef CAUDIO_PLATFORM_WIN
	ThreadHandle = reinterpret_cast<HANDLE>(_beginthreadex(0, 0, threadFunc, this, 0, &ThreadID));
	if(ThreadHandle == 0)
		CloseHandle( ThreadHandle );
	#else
	ThreadID = pthread_create( &ThreadHandle, 0, threadFunc, this );
	#endif
        IsInit = ThreadID != 0;
	return IsInit;
}

the function returns false on linux, the thread never gets created.
I'll look some more at the docs for this tomorrow and see if I can figure out why this would be. I have already made sure that cmake configures gcc to run with the -pthread option (as indicated it should by googling a bit ...) but that had no effect.
If you have any idea of what might be wrong here please dont hessitate to point me in the right drection ;o) I have never played with threads before, so this might be a long ride for me othervice ...

Installing a linux box so i can begin testing.

Found the problem. This section:
    #else
    ThreadID = pthread_create( &ThreadHandle, 0, threadFunc, this );
    #endif
        IsInit = ThreadID != 0;
    return IsInit;

should be:
    #else
    pthread_create( &ThreadHandle, 0, threadFunc, this );
    #endif
        IsInit = ThreadHandle != 0;
    return IsInit;

With that fix threadin works as expected on Linux as well ... I'll open a pull request for the fix in a sec ;o)

Awesome i got caught up yesterday and i wasn't able to look into it. glad you found it. Ill pull once you submit

Pull request issued ;o) I'll close this issue now.

Just tested on windows its working correctly. GJ

Thank you ;o)


Storage firewall, virtual networks and operations with Blobs

When restricting access to a storage account for specific virtual networks and subnets, it is not possible to access the data within the containers, as expected.
However, it seems that it is still possible to list existing blob containers, create them, delete them and even change the IAM settings for each of them, at least via the Azure Portal. Shouldn't all these operations be forbidden if performing them from outside the subnets defined in the firewall settings?

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

ID: 2632af75-7664-b79b-ac5b-c7c17e6a609a
Version Independent ID: 6ecf71b6-77c4-3ab6-8ec9-353ea407d70f
Content: Configure Azure Storage firewalls and virtual networks
Content Source: articles/storage/common/storage-network-security.md
Service: storage
Sub-service: common
GitHub Login: @cbrooksmsft
Microsoft Alias: cbrooks

@antgonmir Thanks for your question. We are checking on this and will respond to you soon.

@antgonmir Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

I am having a v similar issue, when restricting access via networks, you cannot view the contents via file browser on the azure portal page.
Seems a bit too restrictive and ridiculous.

I notice the Microsoft DevOps is missing from the list of Trusted Microsoft services. Is it actually considered one the Trusted Microsoft Services, and if not, is it on the Roadmap to be added? Wanting to be sure if turning on VNet and Firewalls to KeyVault that we don't break the DevOps access. Thanks.

@antgonmir - responding to the original query, Blob Storage containers are managed from the portal through the Azure Resource Manger API's, which are not controlled by the firewall.

DevOps is not included in the trusted services exception for storage.

@antgonmir @c240amg @gusgallows - Please let us know if you have further questions.
#please-close


JAX Error when run on CPU without connection to TPU

Description
I tried running the following very simple python code using JAX on my local device, without any connection to a GPU or TPU.
import jax
import jax.numpy as jnp
x = jnp.arange(10)
print(x)

However, I received the following error, which I have not been able to resolve by myself. Any ideas on how to fix it?
UserWarning: cloud_tpu_init failed: ConnectionError(MaxRetryError("HTTPConnectionPool(host='metadata.google.internal', port=80): Max retries exceeded with url: /computeMetadata/v1/instance/attributes/agent-worker-number (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7f621d36ead0>: Failed to establish a new connection: [Errno -2] Name or service not known'))"))
 This a JAX bug; please report an issue at https://github.com/google/jax/issues
  _warn(f"cloud_tpu_init failed: {repr(exc)}\n This a JAX bug; please report "
WARNING: Logging before InitGoogle() is written to STDERR
I0000 00:00:1686133010.112154  337255 common_lib.cc:156] Failed to fetch URL on try 1 out of 6: Couldn't connect to server

What jax/jaxlib version are you using?
v0.3.25
Which accelerator(s) are you using?
CPU
Additional system info
Ubuntu
NVIDIA GPU info
No response

Running it on 0.4.11 just causes the following message to repeat six times.
WARNING: Logging before InitGoogle() is written to STDERR
I0000 00:00:1686146493.777501  408669 common_lib.cc:156] Failed to fetch URL on try 1 out of 6: Couldn't connect to server

Then this comes up, after which the messaes from above repeat again.
Failed to get 'tpu-env' from instance metadata: Couldn't connect to server

This one also appears after a while. Any ideas?
WARNING: could not determine TPU accelerator type. Set env var TPU_ACCELERATOR_TYPE to set manually. TPU runtime may not be properly initialized.

How did you install jaxlib?

Also, can you share pip list's output? Do you have libtpu installed?

I just used
pip install jaxlib

I do have libtpu-nightly installed.

Ah! Uninstall libtpu-nightly if you don't have a TPU.
@skye Can we fail more gracefully in this case? It's not necessarily a JAX bug, merely "you don't have a TPU" here...


Remove version: Bitvise.SSH.Client version 9.35

:warning: Vanity URL
This package uses a vanity URL, which always points to the latest version. Since a newer version is released, outdated versions should be removed from the repository.

Resolves #198832

Microsoft Reviewers: Open in CodeFlow

URL: https://dl.bitvise.com/BvSshClient-Inst.exe
OK
(Automated message - build 923)


[MERGE] [FEATURE] Merge existing codes from GitHub Gists into NLPurify

Due to limitations and more flexibility, the old codes from GitHub Gists are to be merged and published as a collection utility library in PyPI. On successful transition, the gist will be gradually deprecated and information about the same will be made available here. The issue template tracks all the changes that are required for a successful transition and gradually makes the library available to an end user.

[x] #2
[x] #4

[x] #5

[ ] #8
[x] #6

Code documentation #8 failed. Reopening to track the progress.


Add Traffic Monitor CRConfig history endpoint, rejecting bad configs

This makes it possible to debug Traffic Ops sending the wrong CRConfig, an older CRConfig, or the Traffic Ops FQDN resolving to a wrong IP (for example, if a GSLB was misconfigured).
This also adds an additional safeguard, in addition to the one in the Router, to make the Monitor reject a CRConfig it receives older than the one it has, or with the wrong CDN.
This depends on #1468 and should be merged after it.

Fixed conflicts.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/570/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/614/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/622/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/579/


Using regex rather than directly check for indices in the sql args

The previous implementation will fail with empty strings, and other issues can be found. The current implementation is more robust.

Do we need a test for this?

@antgonza added

:+1: if tests pass.

Coverage increased (+0.01%) to 79.14% when pulling<PHONE_NUMBER>fba664dadbf8522948f04342d048af on josenavas:queue-issue into 6819e8ad625502551dcaf1c99de2f485b030bdfe on biocore:master.

regex whisperer @adamrp @squirrelo @wasade any of you available for a quick review?

One comment

Coverage increased (+0.05%) to 79.18% when pulling 9d3a86d6e41e841fc2e69a41e06854ee89796482 on josenavas:queue-issue into 6819e8ad625502551dcaf1c99de2f485b030bdfe on biocore:master.

Coverage increased (+0.05%) to 79.18% when pulling 9d3a86d6e41e841fc2e69a41e06854ee89796482 on josenavas:queue-issue into 6819e8ad625502551dcaf1c99de2f485b030bdfe on biocore:master.


Updating SQL MI failover group script

Description

Checklist

[ ] This pull request was tested on both of:

[x] PowerShell 5.1 (Windows)
[ ] PowerShell 6.x
[ ] PowerShell 7.x (Latest PowerShell)

[x] Scripts do not contain static passwords or other secret tokens.
[x] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform and PowerShell version: [pscustomobject]$PSVersionTable | Select-Object OS, BuildVersion, PSVersion
Az version: (Get-InstalledModule -Name Az).Version

Hey @mikefrobbins , when you have a chance, could you merge this? Thanks!


is this library compatible with catboost?

I hope you will implement support for catboost Yandex library

/label question

No, currently here is no catboost support.
I will have a look on catboost later and come back to you. But no particular plans for now.

Also look forward to catboost

same here

https://github.com/catboost/catboost/discussions/2299