Cannot create a conda environment with ':' in the prefix.

Currently keep getting this error when fine-tuning:
Creating or resetting Conda environment from /mnt/c/Users/name/anaconda3/envs/DAFineTune2/setup/conda-environment.yml...
Error: generate-project[0]
   03:52:51.80 0 RunProcess error:
Error: generate-project[0]
   03:52:51.80 0 RunProcess error:CondaValueError: Cannot create a conda environment with ':' in the prefix. Aborting.
Error: generate-project[0]

@PostArchitekt check your conda-environment.yml for errors, especially the prefix line.

Looks fine. Am I missing something?
name: phi-2-env
channels:
  - pytorch
  - nvidia
  - defaults
dependencies:
  - pip=23.2.1
  - python=3.9.18
  - cuda=11.8
  - pip:
    - -r requirements.txt

So you're using phi-2 model. The output you posted is from the first-time-setup.sh in the setup directory. The script is run during project setup:
---cut---
# Todo make the name be dynamic with the project.
# Create or reset environment from conda.yaml located in the script's directory
echo "Creating or resetting Conda environment from $SCRIPT_DIR/conda-environment.yml..."
/opt/miniconda/bin/conda env create -f "$SCRIPT_DIR/conda-environment.yml"
---cut---

After step 3, when a new workspace is open, is the environment listed by conda? It should be at this starge:
conda env list

# conda environments:
#
base                     /opt/miniconda
phi-2-env             *  /opt/miniconda/envs/phi-2-env

Try running line 28 from a bash prompt to (re)create the environment:
/opt/miniconda/bin/conda env create -f "$SCRIPT_DIR/conda-environment.yml"
and use --verbose option for debug output to see what's going on.
I have an active phi-2 model (notice the * in conda env list above) and did not encounter such error.

STEP 2
Configure the model
Conda environment name matches Project name DAFineTune3. changed dataset to training dataset.  Generate Project
Everything goes well until:
Information: WorkspaceAutomation.Catalog.HuggingFaceClientProvider[0]
   10:06:19.77 0 Download for model microsoft/phi-2 completed.
Debug: generate-project[0]
   10:06:19.77 0 RunProcess fileName:wsl arguments:[ --exec bash /mnt/c/Users/Dx/anaconda3/envs/DAFineTune3/setup/first_time_setup.sh] elevated:False
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/condabin/conda
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/bin/conda
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/bin/conda-env
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/bin/activate
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/bin/deactivate
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/etc/profile.d/conda.sh
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/etc/fish/conf.d/conda.fish
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/shell/condabin/Conda.psm1
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/shell/condabin/conda-hook.ps1
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/lib/python3.11/site-packages/xontrib/conda.xsh
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /opt/miniconda/etc/profile.d/conda.csh
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:no change     /home/dx/.bashrc
Debug: generate-project[0]
   10:06:45.35 0 RunProcess data:No action taken.
Debug: generate-project[0]
   10:06:45.39 0 RunProcess data:Creating or resetting Conda environment from /mnt/c/Users/Dx/anaconda3/envs/DAFineTune3/setup/conda-environment.yml...
Debug: generate-project[0]
   10:06:45.99 0 RunProcess data:Retrieving notices: ...working... done
Error: generate-project[0]
   10:06:46.01 0 RunProcess error:
**Error: generate-project[0]
   10:06:46.01 0 RunProcess error:CondaValueError: Cannot create a conda environment with ':' in the prefix. Aborting.**
Error: generate-project[0]
   10:06:46.01 0 RunProcess error:
Debug: generate-project[0]
   10:06:46.05 0 RunProcess data:Script execution completed.
Debug: generate-project[0]
   10:06:46.06 0 RunProcess exit code:0 elapsed:00:00:26.2828113
Debug: generate-project[0]
   10:06:46.06 0 ExecuteAsync Completed Elapsed:00:00:57.0817591
[2023-12-26T15:06:46.064Z] [INFO] Command executed to generate the project

Launched window into (WSL) workspace and received this notification in VSCode: This workspace is on the Windows file system (/mnt/). For best performance, we recommend moving the workspace to the Linux file system (~/home).
/mnt/c/Users/Dx/anaconda3/envs/DAFineTune3
conda env list
# conda environments:
#
                         /home/dx/anaconda3
                         /home/dx/anaconda3/envs/Seamless
base                  *  /opt/miniconda

cd into setup directory:
/opt/miniconda/bin/conda env create -f "conda-environment.yml"
returns the same error:
CondaValueError: Cannot create a conda environment with ':' in the prefix. Aborting.
/opt/miniconda/bin/conda env create -f "conda-environment.yml" --verbose
CondaValueError: Cannot create a conda environment with ':' in the prefix. Aborting.
With nothing in the debug output

I just tried the same procedure to create the environment manually and it worked fine for me. As a workaround you can manually create a conda environment and install each of the packages, followed by 'pip -r requirements.txt'


Use Stock Images during Transformation

Category
[ ] Bug
[X] Enhancement
Expected or Desired Behavior
New function to allow a SharePoint stock image to be used on transformation pages.  The idea that pages can be enriched during transform with stock images.

Wouldn't this not already be possible today by creating a hard coded reference to the needed stock image in your page layout mapping file? Wondering how you would see this being more dynamic?

I hadn't considered that, so use a page/web part to load the file into the site before transform and then static reference.
I started to look at how this service works e.g. https://hubblecontent.osi.office.net/contentsvc/api/pivots?filter=pivotNames/any(p: p eq 'Stock Images')&search=Golf&top=1&skip=0
The search param on the above URL.
Then load the image e.g. https://cdn.hubblecontent.osi.office.net/m365content/publish/5052b1ec-5916-4fa3-8b84-ee4f089238ed/153753583.jpg based on the path of the result above.
Couldn't figure out yet how the UI the filename saved to site assets is constructed from this above and may reference another source.
Your solution is simpler, as the user could use the UI to find the file.

Lets close this, think we settled not to enhance this area.


ignore nil containers when saving inventory

Fixes #7915
Since we dont do partial refreshes for containers providers, theres no reason not to follow suit with all the other save_inventory methods. containers are deleted when their pod(container group) is deleted.
@simon3z

@Fryguy are you aware of any case where it make sense to call a save_* with nil hashes?.
Any way to handle this more elegantly than checking it in each method?
Maybe pushing the check in the save_* caller?

LGTM :+1:

LGTM too

@miq-bot add_label darga/yes

Travis failure seems unrelated


[Upgrading problem] Fail to upgrade from 2021.4.7 to 2021.4.17

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

System environment: Mac OS X 10.14.6
So in Terminal I have just run "brew update && brew upgrade" for a general upgrade of all brew components/extensions. After that I ran "youtube-dl --version" to double-check and surprisingly got back "2021.4.07". When I checked the parser it clearly appears that youtube-dl was indeed successfully upgraded:
==> Upgrading youtube-dl 2021.4.7 -> 2021.4.17
==> Downloading https://ghcr.io/v2/homebrew/core/youtube-dl/manifests/2021.4.17
######################################################################## 100.0%
==> Downloading https://ghcr.io/v2/homebrew/core/youtube-dl/blobs/sha256:5bb2b85c53a5519077df1a9d2efd2bed1a4b52259e5ffc4aa6054de2f8383b73
==> Downloading from https://pkg-containers-az.githubusercontent.com/ghcr1/blobs/sha256:5bb2b85c53a5519077df1a9d2efd2bed1a4b52259e5ffc4aa6054de2f8383b73?se=2021-04-24T12%3A55%3A00Z&sig=BhxH
######################################################################## 100.0%
==> Pouring youtube-dl--2021.4.17.mojave.bottle.tar.gz
==> Caveats
Bash completion has been installed to:
/usr/local/etc/bash_completion.d
==> Summary
🍺  /usr/local/Cellar/youtube-dl/2021.4.17: 1,395 files, 13.5MB
Removing: /usr/local/Cellar/youtube-dl/2021.4.7... (1,394 files, 15.2MB)
Removing: /Users/myname/Library/Caches/Homebrew/youtube-dl--2021.4.7... (3.5MB)
I therefore tried again with "brew upgrade youtube-dl" and then got back "Warning: youtube-dl 2021.4.17 already installed". Another "youtube-dl --version" gave the same result: "2021.04.07"
Hence my question: What is wrong in my brew/youtube-dl configuration, and how do I go about to sort this out via Terminal commands..?

Address this to homebrew support.


Azure - Give user a clear error if they attempt to use Azure Functions while running Python 2

I have seen several people stuck on this.

#3023

Fixed with https://github.com/capitalone/cloud-custodian/pull/3084


Descriptions in Nearby are in the wrong language

Summary:
Descriptions in Nearby are in random languages for me. I've seen Arabic and German so far (and also English, which is correct for my phone language).
Steps to reproduce:
The easiest way is to go to Nearby list and look at all the descriptions. But the bug carries over into the details shown in the individual places, and also the autofill when uploading for a Nearby place.
Device and Android version:
Samsung Galaxy S20FE, Android 11
Commons app version:
3.1-release
Screen-shots:

Would you like to work on the issue?
No

Hey @madhurgupta10 , thank you for the attempt but unfortunately the issue still recurs for me on 3.1-release.

Surfers Paradise Transit Centre has no description in English, and taking the only description available is the right thing to do:

Same for the other one:

@nicolas-raoul Interesting! I actually did not notice the drop down menu in Wikidata for languages, thanks for that. It looks like someone recently added a LOT of German descriptions for various points in Australia. I guess this is not a bug, then, and it is working as intended (even though it looks strange to the user)?

The best thing to do is to add descriptions in your phone's language too :-)

"it looks strange to the user"

Here are my thoughts on this:

Indeed, English speakers are not used to seeing other languages show up on their screens.
On the other hand, the 95% of humans whose native language(s) is not English are used to seeing some English show up on their screens.
Since English is the major language on Wikimedia projects by far, I believe that the latter will happen far more often than the former.

To mitigate the "issue":

I added a FAQ: https://commons-app.github.io/docs.html#language-fallback
We could consider sorting the fallback list differently, for instance by showing Latin-based languages before Arabic-based languages, based on the assumption that more people understand Latin scripts than Arabic scripts. It could be hard-coding systemic bias, though.

Yeah, I don't think I would be a fan of treating different languages differently either. :) Personally my preference would probably be to only see the descriptions in my phone's language or not at all, but it's not a huge issue for me except that I wonder if other users will mistakenly consider this a bug as well.
Thanks for the FAQ, I can point people to that if they ask me! ;)


lfs_file_open(O_CREAT), Error from lfs_dir_commit is hidden

https://github.com/ARMmbed/littlefs/blob/494dd6673d008c03e3199b24af7bff0fe82017f1/lfs.c#L2366
lfs_dir_commit() returns an error, (LFS_ERR_IO in my case, because block write failed) and this is hidden behind a NAMETOOLONG which is completle wrong in this case.

Well, I hit on this wall at the same time . who can helps 8-(


bug: Dot-repeat blocks standard usage

Did you check docs and existing issues?

[X] I have read all the flash.nvim docs
[X] I have searched the existing issues of flash.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.9.1
Operating system/version
kubuntu
Describe the bug
Flash.nvim dot-repeat blocks standard dot-repeat usage (see reproducer).
Steps To Reproduce

Let's have a file with simple content:

"one.lua"
"two.lua"
"three.lua"

you are at line 1 and column 1
start / search: :/\.lua<cr>
you are on first line on the character .
remove extension .lua by pressing dt"
press n to get to next search result on line 2
press . to repeat extension remove - ups, it is not removed as expected.

Expected Behavior
In the step 7, the extension .lua on the line 2 should be removed - just as in the case without this plugin.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { "folke/flash.nvim", opts = {} },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Should be fixed now!

@vogler what happens in your buffer doesn't look like flash at all. Or at least not with the default settings.
The highlights should stay visible and for dt you should see two kinds of highlights.
Your statusline also doesn't seem to change to operator pending mode.
Are you sure there's no other plugin doing this?

If I disable flash, it also works as it should.
Just tried it again with a config that just loads flash (NVIM v0.9.0):

@vogler can you open a new issue with a proper repro as instructed in the issue template, with the exact steps to reproduce the issue?
Since it's working as it should for me, I need something reproducible to see what's happening


Logo proposal for bPage

Hello mate, I made a log for your bPage project. If you want to use it please comment below and I'll send you logotype, monochrome version and original files.

thanks  for design logo, i think the color maybe too dark

Which colors do you want ? If you like it I can change the color which you want


prax + vagrant

Not sure if that's an issue.. rather I just don't know how to set it up. So I installed prax, run:
root@myapp-dev:/srv/myapp# prax start
Starting prax in the background.
root@myapp-dev:/srv/myapp# prax open

xdg-open http://myapp.dev
/opt/prax/libexec/prax-open: line 26: xdg-open: command not found

Can anyone guide me as to how connect prax and vagrant?

xdg-open is usually available with a Desktop Environment (eg: GNOME), which certainly wasn't installed inside the Vagrant box.
I'm using Prax on the host computer using port forwarding, not inside the Vagrant box. Prax won't start the application by itself, but I still benefit from the port redirections and domains.


Adding 'status' field to the 'Connection' model - DO NOT MERGE BEFORE #92

Adding the status field as a custom choiceField.
Creating new methods for the Connection model, regarding the status field.
Creating tests for the field and methods.
[THIS PR SHOULD BE MERGED AFTER #92 AND WILL NEED A SMALL REBASE]
fix #111
NOTE: New migration file was left untouched.

@DanielMy777 we have merged #92. Please rebase and make sure the only necessary files are included in the PRs changed files.

@DanielMy777 we have merged #92. Please rebase and make sure the only necessary files are included in the PRs changed files.

@1Oreo @simzacks @Omeramsc
I've rebased and made sure it still works properly, please continue with the merging process =]


Update site content

Update site content when new content is available.

[ ] Show the last five episodes
[ ] Pull and parse the RSS feed

@dtoki I plan to just pull our info from Anchor for right now :)


Dead code removal pass removes valid operations

During test for tt-xla, it has been noticed that the dead code removal pass in the stablehlo to ttir pipeline will remove even valid operations, if they are referenced before there are defined. As an example mlit that currently fails on main with command
./build/bin/ttmlir-opt --debug --stablehlo-to-ttir-pipeline <code_path>:
module @jit_module_sqrt attributes {mhlo.num_partitions = 1 : i32, mhlo.num_replicas = 1 : i32} {

  func.func public @main(%arg0: tensor<4x4xf32> {mhlo.sharding = "{replicated}"}) -> (tensor<4x4xf32> {jax.result_info = ""}) {
    %0 = call @shmap_body(%arg0) : (tensor<4x4xf32>) -> tensor<4x4xf32>
    return %0 : tensor<4x4xf32>
  }

  func.func private @shmap_body(%arg0: tensor<4x4xf32>) -> (tensor<4x4xf32> {jax.result_info = "[('x',), None]"}) {
    %0 = stablehlo.sqrt %arg0 : tensor<4x4xf32>
    return %0 : tensor<4x4xf32>
  }
}

The error looks like this:
Propagating update to mlir::dataflow::Lattice<mlir::dataflow::ConstantValue> of %0 = func.call @shmap_body(%arg0) : (tensor<4x4xf32>) -> tensor<4x4xf32>
Value: <UNKNOWN>
helper.mlir:10:5: error: null operand found
    return %0 : tensor<4x4xf32>

Which suggests that the shmap_body function was deleted in the Dead Value pass (checked by removing the pass, and the pipeline worked).
In collaboration with @azecevicTT, we have managed to bisect the issue to the commit: https://github.com/tenstorrent/tt-mlir/pull/1481, which sets all of our ops to Pure, having an influence on the dead value pass. We should have a discussion on how to tackle this best, because it will necessitate changing the order of the ops.

FYI: @azecevicTT


Update defaultSort.json

Update requestBody according to the spec

Hi @shpoont,
Thanks for the PR.
Would you mind sharing the spec where this is described so I can update the readme?

Hi, this is according to https://spec.openapis.org/oas/v3.0.0#request-body-object

@shpoont Thanks again for taking the time to create the PR.

@shpoont just fyi: This will be part of the next release, if you would need this earlier you can pass along your custom sorting settings using the "--sortFile" CLI option.

No worries, I am using --sortFile already :)
Thanks!


[metric] Use utf-8 in MetricsRowKeySerializerTest

@udoprog

Codecov Report

Merging #210 into master will not impact coverage.

@@            Coverage Diff            @@
##             master     #210   +/-   ##
=========================================
  Coverage     46.66%   46.66%
  Complexity     2517     2517
=========================================
  Files           623      623
  Lines         16324    16324
  Branches       1258     1258
=========================================
  Hits           7618     7618
  Misses         8221     8221
  Partials        485      485

Continue to review full report at Codecov.

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


[First Timers Only] Display link to subscription transactions

In some cases we won't display all the native data associated with a subscription because it requires additional API calls. Instead we can provide a link to the resource within the Chargify account itself.
An example of this treatment already exists in the code.
 It would be great to display a link that opens a given subscription's transactions in the Chargify account itself, in a new tab.

I would like to contribute. Any more details ?

I'm also interested in contributing. Kindly provide further information.

Please review the contributing guide. Fork this repository, pull your fork down locally, then get the application up and running locally. From there you can implement the feature requested here and then submit a pull request.

Hey,
I would like to work on this issue.

Ok! Please review the contributing guide. Fork this repository, pull your fork down locally, then get the application up and running locally. From there you can implement the feature requested here and then submit a pull request.

Ok. Please add "hacktoberfest" label to this issue, and assign it to me.

I will add the label. I cannot assign you because you are not in the delivered organization. You are welcome to submit a Pull Request with your solution, as is anyone else for that matter.

@stevenmaguire I've forked the repo. Now how do i run it locally on my machine ? What will be the subdomain ?

To run the project locally, you will need to follow these instructions from the contributing guide:
Getting started:

Install the zat gem - $ gem install zendesk_apps_tools
Run the server - $ zat server
Preview the app - bring up zendesk, find a user or ticket, and add zat=true query string parameters to the url, like this:
  https://{your-subdomain}.zendesk.com/agent/users/{user-id}/requested_tickets?zat=true
    NOTE: if a shield icon appears in the browser, click on on and allow the script to run
  reload the app to make changes active

After $ zat server and before Preview the app will be prompted for an api_key and chargify subdomain. These need to be for a chargify account that you have access to.

@stevenmaguire So do I have to create a chargify account then ?

Yes. Or gain access to an existing one with sample data. You will also need to have access to a Zendesk account as well. Do you have those?

No. Neither I have access to any chargify account with sample data nor to a Zendesk account.

Ah, that is definitely a roadblock for this particular project. You will need those things before beginning.

@stevenmaguire do you have a preferred position where the link to subscription transactions should appear? Is it okay if it is placed just below the ID ?
Also what should be the label and the link text for the field ? For now, I'm setting the label as "Transactions" and link text as "Click Here". Below is the preview of how it appears -

The link will take the user to a new tab and opens the subscription's transactions in the Chargify account itself.
Please provide your feedback.

@udayapitchika Thanks for taking putting some energy into this. Because this link does not contain any practical information about the subscription, I feel like it does not belong in such a prominent location.
What do you think about a full width row at the bottom of the table with a centered link that says View Transactions?

        View Transactions

The above example does not have the handlerbars code to place the url and also includes inline CSS. Perhaps you can improve upon that a bit?

Thanks for the feedback @stevenmaguire. I really appreciate it.
I agree with you on the point that it doesn't belong on the top. I can update the handlebars code and improve the css.

https://github.com/delivered/chargify-zendesk/releases/tag/1.4.0 has been cut which addresses this issue. Thanks @udayapitchika!


[SidechainBlocks] use edd25519::Public in blocks + minor improvents

When creating Aura I had to realize that the public key is much more usable than the AccountId32 we had before.
Todo:

[ ] Merge: #396
[ ] Merge: #395
[ ] rebase on master

It is only seemingly less generic. At the same time, I am making the enclave functions generic over the Block's trait in all the functions I am touching for aura.
In this trait, the public key is an associated type. All the functions are abstracted over the trait. Only in top-level enclave or worker functions, we insert the block type.
I agree with your concern, but it is the entire sidechain block that needs to be more generic, and I would like to do this a different time.


gnused: 4.8 -> 4.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/gnused/versions.
meta.description for gnused is: "GNU sed, a batch stream editor"
meta.homepage for gnused is: "https://www.gnu.org/software/sed/"
meta.changelog for gnused is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.9 with grep in /nix/store/dj7qgzljjpkbc674c5jpdi56gzsgyzln-gnused-4.9
found 4.9 in filename of file in /nix/store/dj7qgzljjpkbc674c5jpdi56gzsgyzln-gnused-4.9

Rebuild report (if merged into master) (click to expand)

62438 total rebuild path(s)

62438 package rebuild(s)

First fifty rebuilds by attrpath

AMB-plugins
ArchiSteamFarm
AusweisApp2
BeatSaberModManager
CHOWTapeModel
ChowCentaur
ChowKick
ChowPhaser
CoinMP
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
EBTKS
EmptyEpsilon
FIL-plugins
Fabric
HentaiAtHome
LAStools
LASzip
LASzip2
LibreArp
LibreArp-lv2
Literate
MACS2
MIDIVisualizer
MMA
NSPlist
OSCAR
OVMF

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dj7qgzljjpkbc674c5jpdi56gzsgyzln-gnused-4.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gnused https://github.com/r-ryantm/nixpkgs/archive/02ffcc0ba31ed9d216b8c8ca0790b48573050ce8.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dj7qgzljjpkbc674c5jpdi56gzsgyzln-gnused-4.9
ls -la /nix/store/dj7qgzljjpkbc674c5jpdi56gzsgyzln-gnused-4.9/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @Mic92 for testing.

╰─ % ./result/bin/sed --version
./result/bin/sed (GNU sed) 4.9
Copyright (C) 2022 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by Jay Fenlason, Tom Lord, Ken Pizzini,
Paolo Bonzini, Jim Meyering, and Assaf Gordon.

This sed program was built without SELinux support.

GNU sed home page: <https://www.gnu.org/software/sed/>.
General help using GNU software: <https://www.gnu.org/gethelp/>.
E-mail bug reports to<EMAIL_ADDRESS>

https://lists.gnu.org/archive/html/info-gnu/2022-11/msg00001.html


[Bug] Contributors of the MIDI-B challenge are not visible

Is there an existing issue for this?

[X] I have searched the existing issues

What product(s) are you seeing the problem on?
OpenChallenges
Current behavior
The contributors of the MIDI-B challenge are not visible on main and in prod. Other challenges successfully show their contributors.
http://localhost:8000/challenge/517/midi-b?tab=contributors
The console shows that the page is trying to make the following request, which fails:
http://localhost:8082/api/v1/organizations/475

Http failure response for http://localhost:8082/api/v1/organizations/475: 404 Not Found

http://localhost:8000/api/v1/organizations/475
{
  "title": "Entity not found",
  "status": 404,
  "detail": "The organization with the ID or login 475 does not exist.",
  "type": "ORGANIZATION-SERVICE-1000"
}

Expected behavior
The contributors should be visible.
Anything else?
This challenge has multiple contributors, but none are displayed. Wondering whether one contributor breaking is preventing all the other contributors for this challenge to be displayed.
Commit ID
40bcf76224dbc6918d2cf8baca596791a455b3cb
Are you developing inside the dev container?

[X] I am using the dev container

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contribution roles for this challenge (ID 517):
"1215","517","15","challenge_organizer"
"1216","517","171","challenge_organizer"
"1217","517","473","challenge_organizer"
"1218","517","474","challenge_organizer"
"1219","517","475","challenge_organizer"
"1220","517","476","challenge_organizer"
"1221","517","477","challenge_organizer"
"1222","517","131","challenge_organizer"
"1224","517","15","data_contributor"

@gaiaandreoletti @vpchung The data of the org with ID 475 is breaking the CSV file because of the comma in the name of the org.
"475","Ellumen, Inc.","ellumen,-inc","logo/ellumen.jpg",...

I just triggered the GH workflow.


Fix algorithm to avoid small directions in gradient_check

Use weighted root mean square to fix small values.  https://github.com/chainer/chainer/pull/8236#discussion_r331945995
Test the sampling routine.

Distribution before and after:

(scatter plot of samples of first 2 dims of 5-dim directions.)
Code:
import matplotlib.pyplot as plt
import numpy as np

N = 5

def f():
    x = np.random.randn(N)
    x /= np.sqrt(np.square(x).sum())
    return x

def f_clip():
    xp = np
    d = f()
    min_d = 0.1 / np.sqrt(d.size)
    is_small = min_d > xp.abs(d)
    is_large = xp.logical_not(is_small)
    n_small = is_small.sum()
    sq_large = xp.square(d[is_large]).sum()
    scale = xp.sqrt((1 - n_small * min_d ** 2) / sq_large)
    d[is_small] = xp.sign(d[is_small]) * min_d
    d[is_large] *= scale
    return d

def f_rms():
    d = f()
    return np.copysign(np.sqrt(0.99 * np.square(d) + 0.01 / d.size), d)

def main():
    plt.subplot(1, 2, 1)
    xs = np.array([f_clip() for _ in range(5000)])
    plt.scatter(xs[:, 0], xs[:, 1], s=1)

    plt.subplot(1, 2, 2)
    xs = np.array([f_rms() for _ in range(5000)])
    plt.scatter(xs[:, 0], xs[:, 1], s=1)

    plt.show()

main()

Jenkins, test this please.

Jenkins CI test (for commit d02772a31ca01d562d7d0e481e7ec2831decab4b, target branch master) succeeded!

LGTM!


Stroked Text

How can I created stroked text using NanoVG?
void RenderText(string font, string text, float x, float y, NVGcolor fill, NVGcolor outline, float thickness,              int text_align = NVG_ALIGN_LEFT | NVG_ALIGN_TOP) {
        nvgFontFace(nvg_ctx, font.c_str());
        nvgFontSize(nvg_ctx, size);
        nvgTextAlign(nvg_ctx, text_align);
        nvgFillColor(nvg_ctx, fill);
        nvgStrokeWidth(nvg_ctx, thickness);
        nvgStrokeColor(nvg_ctx, outline);
        nvgText(nvg_ctx, x, y, text.c_str(), text.c_str() + text.size());
}
I've been trying to do it like this but it doesn't work. SDL_ttf supports font outlines, and I need to see if NanoVG provides a similar feature, but so far it doesn't seem to be.

Nanovg does not support stroked text.
su 10. maalisk. 2019 klo 10.45 Simon NE Struthers<EMAIL_ADDRESS>kirjoitti:

How can I created stroked text using NanoVG?
void RenderText(string font, string text, float x, float y, NVGcolor fill,
NVGcolor outline, float thickness, int text_align = NVG_ALIGN_LEFT |
NVG_ALIGN_TOP) {
nvgFontFace(nvg_ctx, font.c_str());
nvgFontSize(nvg_ctx, size);
nvgTextAlign(nvg_ctx, text_align);
nvgFillColor(nvg_ctx, fill);
nvgStrokeWidth(nvg_ctx, thickness);
nvgStrokeColor(nvg_ctx, outline);
nvgText(nvg_ctx, x, y, text.c_str(), text.c_str() + text.size());
}
I've been trying to do it like this but it doesn't work. SDL_ttf supports
font outlines, and I need to see if NanoVG provides a similar feature, but
so far it doesn't seem to be.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/memononen/nanovg/issues/533, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFAbyCNpQOM9vmUXTBDmE0DqjWgBl9zQks5vVMYygaJpZM4bnJHX
.


BetterTwitter: TG Fork of Clutter

We are instead working on integrating TrustGraph features directly into MewsFeed (new name for clutter) here:
https://github.com/GeekGene/MewsFeed/pull/104


adding kit/datastore 'reading-list' example to mimic marvin

To steal from the new example's README.md:
The 'Reading List' Example
This example implements a clone of NYT's 'saved articles API' that allows users to save, delete and retrieve nytimes.com article URLs.
This service utilizes Google Cloud Datastore and is set up to be built and published to Google Container Registry, deployed to Google Container Engine and monitored by Google Cloud Tracing.
Instead of utilizing NYT's auth, this example leans on Google OAuth and Google Cloud Endpoints for user identity.
To run locally, have the latest version of gcloud installed and execute the ./server/run_local.sh script to start up the Datastore emulater and the reading list server.
A few highlights of this service worth calling out:

service.yaml

An Open API specification that describes the endpoints in this service.

gen-proto.sh

A script that relies on github.com/NYTimes/openapi2proto to generate a gRPC service spec with HTTP annotations from the Open API spec along with the Go/Cloud Endpoint stubs via protoc.

service.go

The actual kit.Service implementation.

http_client.go

A go-kit client for programmatically accessing the API via HTTP/JSON.

cmd/cli/main.go

A CLI wrapper around the gRPC client.

.drone.yaml

An example configuration file for Drone CI using the NYTimes/drone-gke plugin for managing automated deployments to Google Container Engine.

cloud-endpoints/service-ce-prd.yaml

A service configuration for Google Cloud Endpoints.

This example mirrors an example in gizmo's sibling server for Google App Engine, marvin.

Coverage increased (+0.3%) to 47.175% when pulling ce4c4c977f82ecee8280e96f8208e78779b64948 on reading-list into 1c0e1e9ca7a78cccba0be9bbb930543a186591f7 on master.

Coverage decreased (-0.07%) to 46.767% when pulling 89739bd46d4313d0a10f15f0b9934931afc56445 on reading-list into 1c0e1e9ca7a78cccba0be9bbb930543a186591f7 on master.

Coverage remained the same at 46.835% when pulling d0627ecea3b8cd7ad544fe941eabfdd42d6d4c75 on reading-list into 1c0e1e9ca7a78cccba0be9bbb930543a186591f7 on master.


How to distribute a framework without the Weaver dependency?

I can create an umbrella framework and include Weaver inside but that is not a recommended pattern.
Is there a way to include the weaver sources instead of import Weaver in release targets?

Hi @ferologics,
I'm not sure to understand what you're trying to do exactly.
If you're trying to use weaver in your framework without any need to import it, then the only way I can think of would be to import weaver's sources manually into your project. Maybe with a git submodule?
I don't see why having weaver as a dependency of your framework would be problematic though.


Update Serbian translation

Description
I've used the main branch as the base initially so I've missed some strings in the dev branch. Rebased on dev and updated the translation.

Changelog Entry
Changed

Updated the Serbian translation

Thanks!


DO NOT MERGE - simple GUI test case

DO NOT MERGE - test case for https://github.com/computational-cell-analytics/micro-sam/pull/223

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@8af8aa0). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev       #2   +/-   ##
======================================
  Coverage       ?   16.17%
======================================
  Files          ?       30
  Lines          ?     3964
  Branches       ?        0
======================================
  Hits           ?      641
  Misses         ?     3323
  Partials       ?        0

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

Hello @GenevieveBuckley! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file test/test_gui.py:

Line 16:25: E231 missing whitespace after ','
Line 17:13: E231 missing whitespace after ','
Line 18:13: E231 missing whitespace after ','
Line 21:80: E501 line too long (86 > 79 characters)
Line 22:80: E501 line too long (84 > 79 characters)
Line 32:80: E501 line too long (122 > 79 characters)

Now that https://github.com/computational-cell-analytics/micro-sam/pull/223 is merged, we can close this test branch.


"Be the first to comment" link doesn't work

"Be the first to comment" link on updates on the front page of codeforphilly.org doesn't lead to a comment page, just the update page itself.

A comment-section-blocking chrome extension was to blame here


Avail zones

The main enhancement is to be able pull Availability Zones and VPCs data from Amazon Aws using Edda.
Secondarily, updated to use Java 1.8 with Scala 2.11 and newer Amazon Aws Java Sdk (1.9.6).
Along with Java 1.8 updated Gradle Plugin findbugs to use latest version 3.0.1.
Additionally, added ability to pull CloudWatch metrics: CPUUtilization, StatusFailedCheck, and EstimatedCharges.

edda-pull-requests #64 SUCCESS
This pull request looks good

edda-pull-requests #65 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » edda » edda-pull-requests #1 FAILURE
Looks like there's a problem with this pull request

Closing this for now as it has conflicts and includes commits for several unrelated issues. Please open specific PRs for items that are still needed. Thanks.


Fixed C++ inference tutorial

Description
Tutorial did not properly preprocess images and as such produce incorrect predictions. The example would set default values for mean but not for std_dev. So if the user does not provide the file that contains those values, the example would crash. The C++ example predicts now the same class as in the Python example. However, the probabilities differ (9.72 versus 10.411) and we should to look into that

@NRauschmayr Thank you for fixing the example. Can you please fix failing CI?
@leleamol Can you take a look?

I think we have to update the unit-test: it is using an outdated and not maintained model https://github.com/apache/incubator-mxnet/blob/a9458caec285d3c28d16588fef6aef5e219c7472/cpp-package/example/inference/unit_test_inception_inference.sh
The test should rather use a model from the official model zoo. According to https://mxnet.incubator.apache.org/api/python/gluon/model_zoo.html pre-processing has to be done the following way
image = image/255
normalized = mx.image.color_normalize(image,
                                      mean=mx.nd.array([0.485, 0.456, 0.406]),
                                      std=mx.nd.array([0.229, 0.224, 0.225]))

@NRauschmayr Could you please resolve the merge conflicts??
@nswamy @szha Could you please review it? Thanks!

@NRauschmayr Can you rebase with the latest master please ?
@leleamol Gentle ping for review.

Sorry for the late reply. I have not done the rebase yet, because I saw that the files I modified were deleted by the following commit https://github.com/apache/incubator-mxnet/pull/15164 I wanted to check first whether or not it makes sense to re-add those files, but I did not have the time yet to compile the latest c++ tutorial of the master branch.

@mxnet-bot run ci [all]

OK looks like this part is already refactored into the imagenet_inference.cpp example and it's handling the mean values correct. Still, thanks for proposing the patch.


[feature request]- torchvision.transforms.ColorJitter Enhancement

Currently, the ColorJitter function only takes in a single float value for the parameters brightness, contrast, saturation and hue. The value is chosen uniformly from [max(0, 1 - brightness), 1 + brightness] (minus hue). What would improve this function is to be able to set your min and max bounds by inputting a tuple for each parameter.

I'm working on it

I think this can be closed now.


Add support for pulling different DEPLOYMENTS envvar based on provided envPtr

This PR allows for trcsh to pull a different DEPLOYMENTS value depending on its provided env.

If env=dev-1, it would pull DEPLOYMENTS from DEPLOYMENTS-1
If env=QA it would pull from DEPLOYMENTS
If env=QA-2 it would pull from DEPLOYMENTS-2

This PR assumes at DriverConfig#EnvRaw will always be dev or QA (unsure if this is true; may run into unexpected behavior on prod/staging releases).
Might be better to only strictly replace the dev/QA keywords depending on intended behavior / scope.


Doc updates for mc RELEASE.2024-01-11T05-49-32Z

Doc updates for the following:

Document alias naming requirements, including new minimum length rules
mc tag set --exclude-folders modifies the behavior of --recursive

Staged:
http://<IP_ADDRESS>:9000/staging/DOCS-1106/linux/reference/minio-mc/mc-alias-set.html#mc.alias.set.ALIAS
http://<IP_ADDRESS>:9000/staging/DOCS-1106/linux/reference/minio-mc/mc-tag-set.html#mc.tag.set.-exclude-folders
Fixes https://github.com/minio/docs/issues/1106

👍🏻 @djwfyi for the suggestions, yes words are much better than regexes 😂


Bug in latest release

I'm using isomorphic routing with Grapnel and grapnel-server which was working great. After I upgraded to Grapnel v0.6.3 (from 0.6.2) my pages stopped to respond. At first, I thought there was a bug in my code, so I started to debug and found the real cause of the problem.
In you last commit https://github.com/bytecipher/grapnel/commit/5a838d4f03c681439b970ec57589513e41a4b5e1 you changed a lot of functionality in grapnel.js, that eventually broke grapnel-server.
The wareShouldRun function is now accepting wrong data https://github.com/bytecipher/grapnel/blob/server-router/index.js#L122. In previous version, the req attribute was holding the entire connect's request (a huge object), and here's how it looks now:
{ params: {},
keys: [],
matches: [],
match: [ '/admin/users', index: 0, input: '/admin/users' ] }

So the page now hangs, because req.method is always undefined. The res object is now also became quite a srinked.
Hope this info will be helpful to you.

@ilearnio This is because of the version mismatch in package.json. Try using<EMAIL_ADDRESS>it should have this fix.
Thanks! :+1:


Test case TestAmazonCart.test_cart_modify fails

Hi,
When I run tests, the test case TestAmazonCart.test_cart_modify always fails. Although this does not affect my product code, this gets in the way of supporting Python 3, ref. #77. In my environment, the test case fails both in Python 2 and 3. @yoavaviram, could you tell me does it succeed in your environment?
Expected Result
All the tests succeed.
Actual Result
Only one test case TestAmazonCart.test_cart_modify always fails. Other tests succeed.
(venv2)$ nosetests
...................................E
======================================================================
ERROR: test_cart_modify (tests.TestAmazonCart)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/orange/github-libs/python-amazon-simple-product-api/tests.py", line 459, in test_cart_modify
    assert_equals(new_cart[cart_item_id].quantity, '3')
  File "/Users/orange/github-libs/python-amazon-simple-product-api/amazon/api.py", line 1312, in __getitem__
    raise KeyError('no item found with CartItemId: {0}'.format(cart_item_id,))
KeyError: 'no item found with CartItemId: C8AIO6HTRFXA6'
-------------------- >> begin captured logging << --------------------
bottlenose.api: DEBUG: Amazon URL: http://webservices.amazon.com/onca/xml?...
--------------------- >> end captured logging << ---------------------

----------------------------------------------------------------------
Ran 36 tests in 36.346s

FAILED (errors=1)

Environment
(venv2)$ python -V
Python 2.7.11
(venv2)$ pip freeze
bottlenose==0.6.3
docutils==0.12
funcsigs==1.0.2
Jinja2==2.8
lxml==3.6.0
MarkupSafe==0.23
mock==2.0.0
nose==1.3.7
pbr==1.10.0
Pygments==2.1.3
python-dateutil==2.5.3
six==1.10.0
Sphinx==1.1.3
wheel==0.24.0
You are using pip version 7.1.2, however version 8.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

Fixed.


Rename page removes it from side nav

Package
filament/filament
Package Version
v3
Laravel Version
v10
Livewire Version
v3
PHP Version
PHP 8.3
Problem description
After having added pages to filament via artisan make:filament pages I found I needed to rename them. Having done so they have vanished from the Filament side nav. I have searched for a way to get the nav to 'see' them again without success so have had to rename them back. I have looked through web.php etc trying to find where Filament is storing their names - no success. Monumental waste of time.
Expected behavior
I expected them to be detected in the same way they were detected when I created them via artisan.
Steps to reproduce
Create a page
php artisan make:filament-page Settings
Then rename it in VS-Code or via your OS.
Reproduction repository
thereisntone
Relevant log output
No response

Really!? I really don't need to put a whole project into git for this. Maybe I should lodge it as a feature request instead.


Skip packages that are broken in master

For large reviews, it always takes a bit of extra time to weed through the broken builds and classify them. Some builds might be broken but they are also broken in master, and therefore merging the PR doesn't make things worse.
A decent strategy would be to collect all the failed attributes. And then try to run them on master with --dry-run. If any build is going to happen, then consider them as already broken and remove them from the list. This approach would also partially solve #138 and #63

The dry-run approach would work if hydra has not yet tried to built the package. It needs to be combined with checking if there are build logs in the binary cache.

Instead of remove from the list I'd suggest to list it separately.


IDETECT-3547 - Upgrade Guidance on transitive dependancies

Pull request for new "full-result" api objects.  This has been "locally tested" and successfully gets the transitive guidance.  It's pretty much all generated code.  Some of the objects have nothing to do with this issue (eg. Bom classes) however they may be needed in upcoming releases.
The classes that DO impact this ticket, are the ScanFullResult.... ones.
This PR must be completed (along with subsequent library version updates) before the synopsys-detect changes can be merged.

Changes seem reasonable to me. I think you'll need to change the build.gradle version = '2022.10.6-SNAPSHOT' line to match the definition you generated from.

Done


Chatflow for capa reservation doesn't work when node doesn't have a valid Stellar address

If capacity is reserved on a node that doesn't have a public Stellar wallet address defined, reservation fails. So there should be filtering put in place to exclude these nodes.
On mainnet, this will lead to substantial reduction of valid nodes until 3Bot connect app wallet is released (later this week).

Seems that even farms that have the 'TFT' address filled have bogus addresses...
https://explorer.grid.tf/explorer/farms/1
https://horizon.stellar.org/accounts/GD3DUMWG2I2AD2ZVZ3KA77NQP75TXNUXWQXLCROGC4ANZ657RZENC6OD
I'm not sure how to progress from here... should we really validate those aswell?

I think these 'bogus' addresses were created before Stellar network reset. Best is to have all Stellar addresses deleted at the same time as Stellar testnet reset.

There were address set there before we enabled validation of the address. Nothing to do with the stellar reset, we're talking mainnet here, there is no reset happening.
I asked @andhartl to update the addresses on freefarm to put valid ones.

Related: https://github.com/threefoldtech/tfexplorer/issues/51

https://github.com/threefoldtech/jumpscaleX_libs/pull/192

for TFT currency return the following error :

verified on branch  rc_10.5.3


OAS : /v1/bequeath-your-data-and-die : callback PromptForBequeathingDataCausesALTbeingRequestedToStopNotificationsToOldRelease can be removed

For /v1/bequeath-your-data-and-die, we have callback PromptForBequeathingDataCausesALTbeingRequestedToStopNotificationsToOldRelease to end notification subscription /v1/notify-link-updates of old OKM.
Problem statement:
As per issues #59 and #79, the forwarding-construct "LinkChangeNotification" could be an INVARIANT_PROCESS_SNIPPET for the subscription /v1/notify-link-updates , where the new subscription overwrites the OUTPUT fc-port of old subscription.
If so, while sending the callback, PromptForBequeathingDataCausesALTbeingRequestedToNotifyLinkUpdatesToNewRelease, the old-release subscription would be automatically replaced by the new one and an additional callback (PromptForBequeathingDataCausesALTbeingRequestedToStopNotificationsToOldRelease ) would not be required to end the previous subscription.
Proposal:
Can we remove PromptForBequeathingDataCausesALTbeingRequestedToStopNotificationsToOldRelease from the callbacks list of /v1/bequeath-your-data-and-die.

Change in underlying conditions:

In ApplicationLayerTopology issue#59, it has been decided, NOT to change from subscription to invariant process snipped (=> several applications are allowed to subscribe for notifications about link updates).

Assumed consequences:

PromptForBequeathingDataCausesALTbeingRequestedToNotifyLinkUpdatesToNewRelease adds a new subscription, but doesn't overwrite the existing subscription of the old release.
The subscription of the old release must still be deleted, and we continue to need the callback PromptForBequeathingDataCausesALTbeingRequestedToStopNotificationsToOldRelease.
No change would be required.

@IswaryaaS : If you would agree and also see no need for change, please close the issue.

Thanks for clarifying. Hence closing the issue


ibmcloud: Temporary fix for #1127 with restartPolicy

Description:

In #1127 , test cases are getting failed because container failed to reach completed status in first attempt
In order to resolve the issue, updating restartPolicy of the pods from v1.RestartPolicyNever to v1.RestartPolicyOnFailure

In order to check the stablitiy of test cases with v1.RestartPolicyNever  and v1.RestartPolicyOnFailure
Ran the test cases in loop to find out the ratio of PASS and FAIL status
with  v1.RestartPolicyNever

7 out of 10  got PASS Status
with v1.RestartPolicyOnFailure
10 out of 10 got PASS Status

v1.RestartPolicyNever
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test (31.37s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test/Peer_pod_with_work_directory_has_been_created (0.69s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#01 (31.40s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#01/Peer_pod_with_work_directory_has_been_created (0.65s)
    --- FAIL: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#02 (300.62s)
    --- FAIL: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#03 (300.56s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#04 (31.19s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#04/Peer_pod_with_work_directory_has_been_created (0.61s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#05 (30.95s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#05/Peer_pod_with_work_directory_has_been_created (0.61s)
    --- FAIL: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#06 (300.54s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#07 (31.35s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#07/Peer_pod_with_work_directory_has_been_created (0.77s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#08 (31.43s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#08/Peer_pod_with_work_directory_has_been_created (0.70s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#09 (31.52s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#09/Peer_pod_with_work_directory_has_been_created (0.94s)

v1.RestartPolicyOnFailure
--- PASS: TestCreatePeerPodAndCheckWorkDirLogs (350.93s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test (41.31s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test/Peer_pod_with_work_directory_has_been_created (0.70s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#01 (31.07s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#01/Peer_pod_with_work_directory_has_been_created (0.51s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#02 (31.04s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#02/Peer_pod_with_work_directory_has_been_created (0.51s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#03 (31.03s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#03/Peer_pod_with_work_directory_has_been_created (0.51s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#04 (31.03s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#04/Peer_pod_with_work_directory_has_been_created (0.50s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#05 (56.23s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#05/Peer_pod_with_work_directory_has_been_created (0.50s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#06 (36.03s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#06/Peer_pod_with_work_directory_has_been_created (0.51s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#07 (31.02s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#07/Peer_pod_with_work_directory_has_been_created (0.51s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#08 (31.13s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#08/Peer_pod_with_work_directory_has_been_created (0.50s)
    --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#09 (31.04s)
        --- PASS: TestCreatePeerPodAndCheckWorkDirLogs/WorkDirPeerPod_test#09/Peer_pod_with_work_directory_has_been_created (0.51s)
PASS


fix(select): left/right arrows should navigate input

Description
Left/right arrow keys were being captured by the menu component. See integration test to reproduction steps.
Scope

[x] Patch: Bug Fix
[ ] Minor: New Feature
[ ] Major: Breaking Change

Should this change be applied to https://5cf150903b0d0f00086aa9a5--baseui.netlify.com/components/datepicker/#timepicker too?

hmm, it works as expected if you were to type something into there, then keyboard nav. the initial text rendered in the box is not applied to the input element at all 😉, and therefore un-editable. I can look into this change, but is a subtly different requirement


Fix query option documentation

Hi,
I hope the behavior I documented is right, please validate :)

Thank you for this PR.
I think you have misunderstood the intent. The line should say:

"... overwrite a query string value supplied in the URI of a request."

When running the following:
$url = 'http://www.example.com?foo=123&test=456';
$client->request('GET', $url, ['query' => ['foo' => 'bar']]);

You will end up requesting the url http://www.example.com?foo=bar&test=456

@Nyholm strange, in my case it overwrites all parameters:
$url = 'http://www.example.com?foo=123&test=456';
$client->request('GET', $url, ['query' => ['foo' => 'bar']]);

results in GET /?foo=bar HTTP/1.1, an empty query array even discards all parameters.
I noticed this behavior change while migrating from guzzle 5 to guzzle 6, which is a pain.
I'am using guzzle 6.22.

You are correct @doalex, it is me who is living in the past.


New function data doctor

Updated set_title to "Scale Conversion" and enabled appending the scale_version value, to be appended to new dataframe, e.g., for dataframe by the name of df_test, and feature name of 'total_sales", and scale_conversion variable value of = "cutoff_0_1" will create the new column as df_test.total_sales_cutoff_0_1

Refactored data_doctor to Enhance Functionality and Introduce Plot Saving Feature
Description:
This PR introduces several enhancements and refinements to the data_doctor function. Below is a breakdown of all the key updates:
Updates:

Added Plot Saving Feature:

Introduced the ability to save plots as PNG or SVG images.
New parameters added:

save_plots (bool): Controls whether to save the plots. Defaults to False.
image_path_png (str, optional): Directory to save PNG images if save_plots=True.
image_path_svg (str, optional): Directory to save SVG images if save_plots=True.

If save_plots=True but neither image_path_png nor image_path_svg is provided, a ValueError is raised.
The default filename for the saved images is generated dynamically based on the feature_name and scale_conversion (or 'original' if no conversion is applied).

Added Filename Logic for Saving Plots:

The plot filenames are generated in the following format:default_filename = f"{feature_name}_{scale_conversion if scale_conversion else 'original'}"

These filenames are used for both PNG and SVG if the corresponding paths are provided.

Improved Error Handling:

Incorrect scale_conversion Strings: If an invalid string is provided for scale_conversion, the function now throws a ValueError. The valid options include:

'abs', 'log', 'sqrt', 'cbrt', 'invrs', 'stdrz', 'normlz', 'boxcox', or None.

Example:if scale_conversion not in valid_conversions:
    raise ValueError(f"Invalid scale_conversion '{scale_conversion}'. Valid options are: {valid_conversions[:-1]}")

Plot Saving Validation: Added validation to ensure that when save_plots=True, at least one of image_path_png or image_path_svg must be specified.
Raised a ValueError when this condition is not met:if save_plots and not image_path_png and not image_path_svg:
    raise ValueError("You must provide either 'image_path_png' or 'image_path_svg' when 'save_plots=True'.")

Labeling Enhancements for Plots:

Improved the labeling for both KDE and boxplot:

KDE Plot Title: A title was added to the KDE plot to indicate the feature name and the applied scale conversion:axes[0].set_title(f"KDE Plot: {feature_name} (Scale: {scale_conversion})")

The boxplot now has the lower and upper cutoff values displayed below the plot using set_xlabel() on the boxplot axes.
KDE plot was labeled with a title and X-axis label, showing the feature_name but without the cutoff values.
Boxplot X-axis labels now include both Lower cutoff and Upper cutoff underneath the graph.

Documentation (Docstring) Updates:

Updated the docstring to reflect the new parameters and behavior changes:

Added documentation for the new parameters: save_plots, image_path_png, and image_path_svg.
Clarified the conditions under which the plots are saved.
Specified the newly raised ValueError for missing image paths when save_plots=True.
Specified the newly raised ValueError when an incorrect string is provided for scale_conversion.
Updated details to mention the added KDE plot title.

Calling The Function
data_doctor(
    df=my_dataframe,
    feature_name="feature_column",
    scale_conversion="log",
    apply_cutoff=True,
    lower_cutoff=0.1,
    upper_cutoff=1.0,
    show_plot=True,
    save_plots=True,
    image_path_png="/path/to/save/plot.png",
    image_path_svg="/path/to/save/plot.svg"
)


Naming guidelines

Is your feature request related to a problem? Please describe.
Naming guidelines would be useful if documented if desired.
For example:

akri-agent-daemonset
akri-controller-deployment

Is this guidance akri-[THING]-[TYPE]?
It is a common pattern with Kubernetes but I find it overwrought since these objects all have Kinds (and Labels) and e.g.: kubectl get deployment/akri-controller-deployment is redundant and leads to lengthy names
Is your feature request related to a way you would like Akri extended? Please describe.
Describe the solution you'd like
Definitive naming guidelines
Describe alternatives you've considered
Additional context
Add any other context or screenshots about the feature request here.

The existing convention is certainly redundant.  I think it happened fairly organically, without much debate.
I would be totally open to debating/simplifying/documenting the naming convention.
akri-[THING]-[TYPE]

akri: this seems important to reduce the chances of naming collisions
[THING]: this seems important just for identifying components
[TYPE]: this seems purely redundant and I can't think of a benefit.  I suspect it came to be ONLY because it clarified the various deployment yaml names.  Can anyone think of a benefit for this?

I would be happy to simplify to akri-[THING].

The existing convention is certainly redundant.  I think it happened fairly organically, without much debate.
I would be totally open to debating/simplifying/documenting the naming convention.
akri-[THING]-[TYPE]

akri: this seems important to reduce the chances of naming collisions
[THING]: this seems important just for identifying components
[TYPE]: this seems purely redundant and I can't think of a benefit.  I suspect it came to be ONLY because it clarified the various deployment yaml names.  Can anyone think of a benefit for this?

I would be happy to simplify to akri-[THING].

There are namespaces of course and Akri could e.g. be deployed to a namespace akri and the daemonset could then just be called agent but, I think the prefixing akri- is a good compromise.
I will try today deploying Akri to the non-default namespace.
Since I highlighted the issue, I'll take the action to document the solution :-)

There are namespaces of course and Akri could e.g. be deployed to a namespace akri and the daemonset could then just be called agent but, I think the prefixing akri- is a good compromise.
I will try today deploying Akri to the non-default namespace.
Since I highlighted the issue, I'll take the action to document the solution :-)

Went with creating a proposal (https://github.com/deislabs/akri/pull/204)

Went with creating a proposal (https://github.com/deislabs/akri/pull/204)


upgrading to angularjs 1.3.x

Hi,
Is an upgrade is planned to use ng 1.3.x ?
Tx

+1

The problem is the dependency for angular-bootstrap.This lib supports only 1.2.x yet.

depending on what parts of uibootstrap you need it's "safe" to upgrade to 1.3.x. Some stuff like datepicker have quickfixes, others kinda borked, so ymmv.

angular-bootstrap.0.13.0 added support of angular 1.3
Is there are any problems with pull request from @adnasa?

The problem is the dependency for angular-bootstrap.This lib supports only 1.2.x yet.
Do you have resource supporting this statement?

0.13.0 has been out for a little while now and even the site mentioned the following:

AngularJS (requires AngularJS 1.3.x, tested with 1.3.13). 0.12.0 is the last version of this library that supports AngularJS 1.2.x.

I don't see any reason for waiting any longer
@terales Is PR #71 relevant/up to date to follow up on this issue?
of course, I can spend some amend my PR if needed ;)
Take care!

@adnasa, I've already used your PR as template to update my app dependencies. But I can't merge it to main generator repository, so I stated a question to maintainers.
I think they can merge it without any problems and with value for all who starts new app now.

okay. then we'll wait for @cgross after he's finished consulting his crystal ball of enlightenment :grinning:


300: Automated Athena CUR Query and E-mail Delivery : An error occurred (InvalidRequestException) when calling the StartQueryExecution operation: line 1:190

Hello,
I followed the instructions, however I got an issue when i try to execute my Lamdba Function step...

{
"errorMessage": "An error occurred (InvalidRequestException) when calling the StartQueryExecution operation: line 1:190:
mismatched input '.xxxxxxxxxxxx'. Expecting: ',', '.', 'AS', 'CROSS', 'EXCEPT', 'FOR', 'FULL', 'GROUP', 'HAVING', 'INNER', 'INTERSECT', 'JOIN', 'LEFT', 'LIMIT', 'NATURAL', 'OFFSET', 'ORDER', 'RIGHT', 'TABLESAMPLE', 'UNION', 'WHERE', <EOF>, <identifier>",

config.yml
#S3 location for athena cur query results temp storage
#Don't miss '/' at the end of path
#Make sure your Lambda role has write/read permission to this path
CUR_Output_Location: s3://athena-billing-xxxxxxxxxxxx/

#CUR database name and table
CUR_DB: "athenacurcfn_xxxxxxxxxxxx_c_u_r_hourly_athena.xxxxxxxxxxxx_cur_hourly_athena"

#CUR file name with suffix '.xlsx', which is sent as attachment via SES
CUR_Report_Name: 'cost_utilization_report.xlsx'

#SES configuration
Region: eu-west-1
Subject: 'Cost and Utilization Report'
#i.e<EMAIL_ADDRESS>Sender<EMAIL_ADDRESS>#If there are mulitple recipients, seperate them by commas. i.e. 'john<EMAIL_ADDRESS>Recipient<EMAIL_ADDRESS>#Mail body
Body_Text: "Hi,

Please find your Cost Utilization Report attached:
..........

"

I tried the Athena query by enabling the debug mode :

SELECT year, month(line_item_usage_start_date) month, product_product_name AS "Product_Name", sum("line_item_unblended_cost") AS "Cost($)" FROM athenacurcfn_xxxxxxxxxxxx_c_u_r_hourly_athena.xxxxxxxxxxxx_cur_hourly_athena WHERE (year='2022' AND month='4') OR (year='2022' AND month='3') OR (year='2022' AND month='2') GROUP BY  1,2,3 ORDER BY  1,2,37

The query above has been found in the CW Logs, which is not working. When i try to fix it directly in Athena query by adding "DATABASE"."TABLE"

SELECT year, month(line_item_usage_start_date) month, product_product_name AS "Product_Name", sum("line_item_unblended_cost") AS "Cost($)" FROM "athenacurcfn_xxxxxxxxxxxx_c_u_r_hourly_athena"."xxxxxxxxxxxx_cur_hourly_athena" WHERE (year='2022' AND month='4') OR (year='2022' AND month='3') OR (year='2022' AND month='2') GROUP BY  1,2,3 ORDER BY  1,2,3

The documentation should specify the variable formatting for CUR_DB, which is :
CUR_DB: '"ATHENA_BD"."ATHENA_Table"'

PR : https://github.com/awslabs/aws-well-architected-labs/pull/503

Thanks for the contribution! PR merged. I close the issue.


replace == with equals

if (step.getName() == AssertedConclusionInference.NAME)
is incorrect and works by luck

Can you elaborate a bit more? What is the problem?

http://stackoverflow.com/questions/513832/how-do-i-compare-strings-in-java

Anything does not work? You wrote that something works by luck.
Do you have an example when asserted inferences are not removed?
I am aware that .equals() is generally not the same as ==, regardless of the type of objects.

Take a look at SO comments. new String("name") == "name" is false, so you should not use such comparisons to avoid UB in future.

The purpose of this code is to check if the inference step originates from the AssertedConclusionInference, not from any other inference which happens to have the name "Asserted Conclusion".
Besides, String.equals() is much slower than == since to compare two strings one has to individually compare all characters of these string.

If strings are same objects, then loop is not executed, if they are different, then 2-3 iterations willl be enough to return false. I just wanted to point out typical bug in java code, if you don't think its a bug, well, its up to you to decide.

Thanks for your advice. Although it is a typical mistake for novice programmers, in this case it is not a bug as I explained. Further, with equals() there is a higher chance one could get a bug since somebody can accidentally introduces an inference with the name "Asserted Conclusion", which will then be removed.
If you use a static checker such as "find bugs" and it flags it as a potential problem, we can think about a workaround here.

Well, my IDE settings highlighted it :)
It will work unless someone decides to make deep copy or clone this string, you are right.
Take a look at other ==s if you are concerned :)


UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb0 in position 37: invalid start byte

Hello, I've run this same script to convert DICOMS to bids format a bunch of times for participants within this study, but this one appears to be erroring and I'm not sure why. They have all the correct files and I can't seem to find the issue.
I looked back at previous issues with similar errors and that fix didn't work for me unfortunately.
Thank you!
INFO: Running heudiconv version 0.5.4
INFO: Need to process 1 study sessions
INFO: PROCESSING STARTS: {'subject': 's4111', 'outdir': '/out/bids/', 'session': None}
INFO: Processing 1304 dicoms
INFO: Reloading existing filegroup.json because /out/bids/.heudiconv/s4111/info/s4111.edit.txt exists
INFO: Doing conversion using dcm2niix
INFO: Populating template files under /out/bids/
Traceback (most recent call last):
File "/opt/miniconda-latest/bin/heudiconv", line 11, in
load_entry_point('heudiconv', 'console_scripts', 'heudiconv')()
File "/src/heudiconv/heudiconv/cli/run.py", line 127, in main
process_args(args)
File "/src/heudiconv/heudiconv/cli/run.py", line 323, in process_args
dcmconfig=args.dcmconfig,)
File "/src/heudiconv/heudiconv/convert.py", line 212, in prep_conversion
getattr(heuristic, 'DEFAULT_FIELDS', {}))
File "/src/heudiconv/heudiconv/bids.py", line 77, in populate_bids_templates
populate_aggregated_jsons(path)
File "/src/heudiconv/heudiconv/bids.py", line 109, in populate_aggregated_jsons
json_ = load_json(fpath)
File "/src/heudiconv/heudiconv/utils.py", line 176, in load_json
data = json.load(fp)
File "/opt/miniconda-latest/lib/python3.6/json/init.py", line 296, in load
return loads(fp.read(),
File "/opt/miniconda-latest/lib/python3.6/codecs.py", line 321, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb0 in position 37: invalid start byte
chmod: cannot access '/data/projects/STUDIES/SEAT/fMRI/bids/sub-s4111': No such file or directory

This looks like it was meant for the nipy/heudiconv repo, please open an issue there


[storage-blob] fix uploadStream to support 4GB blockSize

/azp run js - storage-blob - tests

/azp run js - storage-blob - tests

/azp run js - storage-blob - tests

Test in windows Node 10 is failing due to no enough memory, tracking with https://github.com/Azure/azure-sdk-for-js/issues/10701.

/azp run js - storage-blob - tests

/azp run js - storage-blob - tests

/azp run js - storage-blob - tests

/azp run js - storage-blob - tests

I would like to hear more about the ArrayBuffer approach's performance issues! But, a pooled Buffer seems like a fine approach to me.
It is good general advice to avoid allocating many long-lived objects if you can get away with it, but the performance tradeoff between GC churn vs. extra buffer copies seems hard to theorize about without testing.
A question I have is why do we need to hold on to the entire file in memory? You could get away with with using chunks everywhere if the only chunks you are holding are waiting to be uploaded or in progress (either being uploaded). I imagine it has something to do with retry policy?

A question I have is why do we need to hold on to the entire file in memory? You could get away with with using chunks everywhere if the only chunks you are holding are waiting to be uploaded or in progress (either being uploaded). I imagine it has something to do with retry policy?

@bterlson We don't hold on to the entire file in memory. Just the current blocks that we are uploading. We need to keep them in memory while the block is being uploaded.
So in the non-copying way, number of chunks that's alive when uploading with 4GB blockSize is about concurrency (say 8) * 4000MB / readable.highWaterMark (probably = 64KB) ~= 8 * 64000 = 512000. Not sure if this could cause some GC pressure.

I think the approaches should probably be benchmarked. It might even be with the packing approach that you end up allocating all those small buffers anyway, under the covers.

I think the approaches should probably be benchmarked. It might even be with the packing approach that you end up allocating all those small buffers anyway, under the covers.

@bterlson
What do you mean by "allocating all those small buffers anyway, under the covers." With the packing approach, small chunks emitted from the stream will also be allocated, but won't be kept alive while that block is being uploaded.
Benchmarking is nice to have but with low priority. Single threaded Node.js is not promising for performance anyway.
Will merge this for now and do optimization if needed in the future.


add granulate gmaestro addon

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@shapirov103 thank for your quick response, Ive fixed the implementation to match the guidelines you provided (published new npm package, added 3 secrets, opened a new pr in cdk-eks-blueprints-patterns).

Hi @shapirov103, since @kirshenboim is OOO, I'll answer on his behalf.
Regarding the grafanaLogsSecretName and grafanaMetricsSecretName, currently, we pass those secret values hard coded. We are working to remove those values from the helm chart (we'll open a new pr in the aws-quickstart,cdk-eks-blueprints-patterns after the update).
Regarding the client_id, this value is already encrypted from our side.
Also, attaching the PR for aws-samples repository.
https://github.com/aws-samples/cdk-eks-blueprints-patterns/pull/57 (edited)

@ShaharYak
If you would like to move forward with the current implementation then docs should be updated to reflect that the approach has a security vulnerability so that customers are aware.
You mentioned client_id is already encrypted, can't this value be reused as is elsewhere? E.g. a malicious actor gains access to the encrypted client secret and uses it in a different blueprint for example?
Happy to discuss if needed.

Hi @shapirov this is understandable. We added an explanation in the docs as per your request. We will open a new PR in the following weeks that will include your suggested implementation and remove the explanation from the docs.

@shapirov103 updated Security issue text

@kirshenboim I will release 1.6 version this week and test out the pattern on it. Will merge, provided the pattern succeeds.

@kirshenboim @noamsalinger We dont see any traction on this addon and i also see conflicts which needs to be resolved. Could you please check on this, fix all that is required so we can move forward with this addon.?

Hi @elamaran11 @shapirov103 I've fixed the conflicts of this PR in https://github.com/aws-quickstart/cdk-eks-blueprints/pull/727.

Hi @marina-vrublevsky We have feedback on the Blueprints Pattern repo too, please fix those. We can dive in to this PR, once you fix the pattern and once we cover validation.
https://github.com/aws-samples/cdk-eks-blueprints-patterns/pull/57

@marina-vrublevsky Closing this PR as we have a new PR for Granulate.


SAP Fiori launchpad module in CAP project

Hi I'm using WebIDE for my development - SAP business application project in WebIDE.
I am having issues getting the fiori launchpad to work in the CAP project . Every other module is working fine . But when i add a FLP module and deploy the project - even though the project deployment completes successfully, the approuter will shown an internal server error when it redirects to the FLP module .
https://XXXXXXXX.cfapps.eu10.hana.ondemand.com/cp.portal/site -> Gives internal server error.
here is my project: https://github.com/shyamkumarc/CAP-POC
Im attaching the CF deploy logs here in case that helps.
CAP_POC1-db.log
CAP_POC1-srv.log
cap-poc1-approuter.log
flp1.log
OPERATION.log
CAP_POC1_ui_deployer.log

I would suggest that you checkout the mta.yaml of SHINE for Cloud Foundry.

Hi All,
I was able to fix the issue. Sharing the answer here so that it'd be of help for someone who encounters the same issue.
Am mentioned above, I was using WebIDE full stack for my development and the template for creating a CAP project is using the "SAP cloud platform business application" project. I noticed some flaws in this template which were causing the issue.

The SCP business application template doesn't seem to be defining the scope uaa.user in the XS-security file when a project is created .

2.The XS-security file doesn't get linked with the XSUAA resource when a project gets created.
I went ahead and fixed both the above points and now I'm able  to get the FLP instance working.
I'm not sure if the above 2 were conscious decisions based on some thought , or just a miss from the relevant team's end .
Fixes to be made:
XS-security file:
Add a scopes section , after the description section in the XS security file as follows.
"scopes":[
{
"name": "uaa.user",
"description": "UAA"
}],
MTA file:
Add the path to XS-security file on XSUAA resource in your MTA file
resources:
-name: <existing_UAA_instance>
type: org.cloudfoundry.managed-service
parameters:
path: ./xs-security.json
service-plan: application
Apart from that , if your deployment fails and the log shows error : "Upload application content failed { CODE: '1001' } validation error: Error while parsing request; Error: maximum file length exceeded" , you may refer to the following note regarding fixing the HTML application repo host memory :
https://launchpad.support.sap.com/#/notes/0002764058
https://help.sap.com/viewer/65de2977205c403bbc107264b8eccf4b/Cloud/en-US/b8343d9d8b984d73adbae5fe657f13db.html


Update partial.md to use correct syntax highlighting

aspx-csharp
Fixes #7138

Hello @Matthew-Bonner ... We can't accept this. The correct code lang for these is cshtml.


Mast: Watch vbucks

Mast requests the watch of the watch_keyword vbucks. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

vbucks has been seen in 4 true positives, 0 false positives, and 0 NAAs.

Approved by Daniil in Charcoal HQ


ND Functions communicator library

Communicator library packages for nondeterministic functions: date, random, bcrypt.
NPM publish: less cut-n-paste.

Is the purpose of having 1 package per wrapper to reduce dependencies?

Is the purpose of having 1 package per wrapper to reduce dependencies?

If they were complicated packages, I'd say yes... like an S3 communicator would have heavy dependencies.
As it sits, I was really on the fence about having one package per silly little communicator.  It's clear which package you want if they aren't globbed together, but date and random are really tiny.  Bcrypt does have a dependency you might be curious to know about.


fetch latest binaries before running CI tests

Signed-off-by: Shyam Jesal<EMAIL_ADDRESS>

what about knative containers, btw? are they pulled on every CRI test?


NON_PARMEC_ARGV()

Would it be possible to implement a NON_PARMEC_ARGV() utility, equivalent to the Solfec NON_SOLFEC_ARGV() utility?

Yes: see the ARGV command now:)

Thank you for this :)


Different type hints will result in TypingError when using overload

test code:
""""""

from numba import njit, types
from numba.extending import overload

def generic_add(a, b):
    pass

@overload(generic_add)
def ol_generic_add(a: types.Number, b: types.Number):
    if isinstance(a, types.Integer) and isinstance(b, types.Integer):

        def impl(a: types.Integer, b: types.Integer):
            return a + b

        return impl

    elif isinstance(a, types.Float) and isinstance(b, types.Float):

        def impl(a: types.Float, b: types.Float):
            return a + b

        return impl

@njit
def foo():
    return generic_add(1, 2), generic_add(1.0, 2.0)

print(foo())

got TypingError:
    Rejected as the implementation raised a specific error:
      InternalError: Typing and implementation arguments differ in argument names.
    Typing signature:         (a: numba.core.types.abstract.Number, b: numba.core.types.abstract.Number)
    Implementation signature: (a: numba.core.types.scalars.Integer, b: numba.core.types.scalars.Integer)
    Difference: {<Parameter "a: numba.core.types.abstract.Number">, <Parameter "b: numba.core.types.scalars.Integer">, <Parameter "a: numba.core.types.scalars.Integer">, <Parameter "b: numba.core.types.abstract.Number">}

Somehow, numba regards type hints as parts of arguments, causing this error.
Perhaps before comparing arguments, shall we remove type hints first, given numba basically doesn't care type hints for now?

I think type hints are used for jitclass members: https://github.com/numba/numba/pull/5609

I wonder why my PR didn't break the related tests for jitclass type annotation support.
The test cases:
    def test_type_annotations(self):
        spec = [("x", int32)]

        @jitclass(spec)
        class Test1(object):
            x: int
            y: pt.List[float]

            def __init__(self):
                pass

        self._check_spec(spec, Test1, spec + [("y", types.ListType(float64))])

    def test_type_annotation_inheritance(self):

        class Foo:
            x: int

        @jitclass
        class Bar(Foo):
            y: float

            def __init__(self, value: float) -> None:
                self.x = int(value)
                self.y = value

        self._check_spec(
            test_cls=Bar, all_expected=[("x", typeof(0)), ("y", typeof(0.0))]
        )

I guess the typing for class fields didn't go through the part I changed in OverloadTemplate?

I'd bet that these are different things and the check that is it failing only applies to overloads.


Improving API documentation. Adding CSS, numbering, sections, etc.

Coverage remained the same at 89.081% when pulling 04ec641055e949d2a972fe41fa28fe695c0ce4f2 on tgianos:master into 2130c9defd03a15c9480c511a6da724d74f63db4 on Netflix:master.


Angular ng repeat with filter issue.

This is the jsfiddle demo:
https://jsfiddle.net/gaojianzhuang/grr2rL8L/
When the page first loaded, the directive trigger is ok. But when we input a filter query just like "1", the directive not trigger, then we remove the query filter, the directive trigger again.
How to make the directive trigger every time when the filter result change ?
Thx.

You're only outputting the trigger when $scope.$last. It seems to be working under that condition just fine.
If you want to detect when the filter changes use ng-change on the filter input.

@jbedard is right. Closing since there is nothing actionable here.
BTW, for general support question such as this, it is better to use one of the appropriate support channels.
GitHub issues are reserved for bug reports and feature requests.

@jbedard Actually, I will use a jquery plugin after the ng repeat dom rendered. Because of the angular js does not have the event for listening the dom render. And now the directive not be detected when do the filter.
And the element dom updated, but the directive not execute by my expected, this will cause the jquery plugin error. If we detect the filter change, the jquery plugin still will not expected.
Thx.
This is the updated jsfiddle demo: https://jsfiddle.net/gaojianzhuang/grr2rL8L/


Reinstate outputs for census-merged LADs

so that City of London and Scilly Isles are in outputs. (Just adjust Westminster/Cornwall proportionately)

proportionately adjust migrations according to Scilly/Cornwall and City/Westminster population ratios, then round and convert to int (model obs must be integer)


IONIC_ENV is not set properly when running the build through the lib API

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
I made a simple script to run a build of the app, but it seems that IONIC_ENV is not set accordingly.
I use the script pasted below as such:
// make_a_package.js
const {buildApp} = require('./build_app.js');

buildApp()
    .subscribe(() => {
        // do something else after the app is build
    });

Even though I send {isProd: true} with the context and I can see that the context is properly created, IONIC_ENV is not exported with the proper value.
What behavior are you expecting?
IONIC_ENV should be set.
Steps to reproduce:

Clone the conf. app;
Add rxjs as a dep.
Add the two scripts in this issue description;
Run node ./make_a_package.js.

// build_app.js
const {isMaster} = require('cluster');
const {clean, build, generateContext} = require('@ionic/app-scripts');
const {Observable} = require('rxjs');

/**
 * Clean and build the app.
 * NOTE: This function will clean and build the app (in that order) using @ionic/app-scripts.
 * @param {BuildContext} context - The context of this build.
 * @returns {Observable} A stream that emits when it has completed the built.
 */
function buildApp(context = {
	isProd: true,
	runAot: true,
	runMinifyJs: true,
	runMinifyCss: true,
	optimizeJs: true
}) {
	const ctx = generateContext(null);
	Object.assign(ctx, context);
	return Observable.fromPromise(clean(ctx))
		.mergeMap(() => Observable.fromPromise(build(ctx)));
}

module.exports = {
	buildApp
};

if (isMaster) {
	process.on('message', (ctx) => {
		buildApp(ctx)
			.subscribe(() => {
				process.send(null);
			}, (err) => {
				process.send(err);
			});
	});
}

Which @ionic/app-scripts version are you using?
1.0.0
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

I think this should be resolved.
Thanks,
Dan


Provide a way to set example query string

Currently, we can only set example headers and requests in DocService for annotated services. A user might want to set an example query or example path as well.

/cc @matthew-bailey, @heowc

It seems to be a good idea. 👍
Could this be a replacement for a mock server?

@julnamoo I hope your next Armeria contribution 😉


Code coverage / source maps broken after v2.3.1

If I upgrade these dependencies to the latest versions, code coverage seem to lose the source maps integration, I can see only the coverage of the minified javascript.
Any idea on how to solve this issue?
Package                   Current Wanted Latest Package Type URL
monocart-coverage-reports 2.3.1   2.3.1  2.7.6  dependencies
monocart-reporter         2.2.4   2.2.4  2.4.5  dependencies

Actually I fixed a sourcemap issue for Node.js 18 in version 2.7.6.
What is your Node.js version?
I need more information from you to reproduce the issue. For example, creating a minimal repository for debug.


Add "Click to copy functionality" for commands on FAQ page

Describe the bug
A clear and concise description of what the bug is.
When the user goes to the FAQ page, the user doesn't have the ability to copy those commands by clicking on the commands.
Expected behavior
The command should be copied when the user clicks on those commands.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser : chrome

@unnati914  @joewxboy can I work on this issue?

@Abhijay007 Take a look at how we implemented it on this page: https://open-horizon.github.io/quick-start
When you commit, don't forget to add the sign-off flag.
When you create a PR, begin the title with Issue-139:

BTW, if there's a better way to do it than what is already done, let's update the component so it's improved everywhere.

BTW, if there's a better way to do it than what is already done, let's update the component so it's improved everywhere.

got it :)


add k8s support for ehsm-kms

test: passed the unittest
Signed-off-by: Liang1XZhao<EMAIL_ADDRESS>

change the folder name to k8s to align the name and the commit title to "add k8s support for ehsm-kms"

It will be changes.


Explain how to install dotnet-serve in README

Either as a global tool:
dotnet tool install --global dotnet-serve

or by adding a tool manifest.
I go stuck for a bit at this step, maybe other people will too.

PR welcome


fix: change DeviceType.MULTI_OSC value

In https://github.com/nrkno/sofie-timeline-state-resolver/pull/243, there was a conflict with both MULTI_OSC and TRICASTER wanting to have the id of 24.
To minimise impact, MULTI_OSC has been changed to 25, and we should do the same in R49 (the first release containing MULTI_OSC) to avoid MULTI_OSC: 24 ever reaching a production deployment

Other information:

Codecov Report
Patch coverage: 100.00% and no project coverage change

Comparison is base (7e92c11) 30.42% compared to head (8b3fc09) 30.42%.

: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             @@
##           release49     #250   +/-   ##
==========================================
  Coverage      30.42%   30.42%
==========================================
  Files             71       71
  Lines           8842     8842
  Branches        2238     2238
==========================================
  Hits            2690     2690
  Misses          5605     5605
  Partials         547      547

Impacted Files
Coverage Δ

...ackages/timeline-state-resolver-types/src/index.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


INSERT has more expressions than target column

That is because pg-format is flattening (expaning) your array. You'll want to do ARRAY[%L] to fix it.
Example:
INSERT INTO people (email_addresses) VALUES (ARRAY[%L])


Add Fetch API

Would you be interested in a pull request adding the Fetch API as a standard client?

Please add 👀

Hello guys, you can use a mutator to do that. Just like this.

Very happy with the mutator. Works great with within Nuxt 3!

There is actually an example of a custom client using fetch in the docs like shared above.
https://orval.dev/guides/custom-client

Agree with @hahanein.
The default options should require no additional packages, and so Fetch should be the default.

We could for sure add a fetch client directly but for the moment it cannot be the default client since it would be a breaking change


Import Job for Opportunity Data from beta.sam

Is your feature request related to a problem? Please describe.
In the system we need to have all current opportunities and past awards so that we can correlate the data
Describe the solution you'd like
Beta.sam posts CSV's daily and has historic postings as well. we will need to bulk import the historic data going back 5 years if possible. WE will also want award data.  https://sam.gov/data-services/Contract Opportunities?privacy=Public
Describe alternatives you've considered
could look at other ways to pull the data
Additional context
it would be great to find a way to correlate the previous award data to the current opportunities to come up with a list of companies with a high chance of pursuit.

Issues such as this will be tracked separately from now on in their own project.


Version 2.0.0 publishing inaccurate response times

We noticed that upgrading our Microsoft.ApplicationInsights.AspNetCore dependency in our services to version 2.0.0 introduced inaccurate response times logged in Application Insights portal. We are seeing response times in the seconds (1-5s), but when we performance test our application we get response times in the milliseconds. We also tried debugging the services and diagnosing the response times via browser network utility, and we see a large discrepancy in what is being published to application insight, and what is recorded in the browser.
Downgrading the version to 1.0.2 fixes the issue, so it seems like some bug is introduced in version 2.0.0. Unfortunately we want to be using .netcore 1.1, which doesn't support the version 1.0.2, so we really need this to get fixed.

Open issue in regards to this: https://github.com/Microsoft/ApplicationInsights-aspnetcore/issues/350

This should have been fixed in 2.1.0-betas and looks like a duplicate of #350.


github actions trigger changed to run on push and pull_request

Looks like status badge and last pipeline status of repository doesn't updates itself due to not trigger of pipeline when push on master.
Added configuration to run on push to trigger pipeline when we merge the request.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f73cd85). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #10   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         1
  Lines             ?         2
  Branches          ?         0
==========================================
  Hits              ?         2
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

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


Fix typos

Fix typos

Thank you very much!
There's no need to change the comment of hardProtos, but I've made it more clear and merged.
Other changes are great!


[UI] use connection metadata to get context

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

@aabidsofi19


Reply message feature.

took 1 week to make this, but this feature is so easy to make
kill me plz
EDIT: also including #698, that pr is getting old

Attachments need to be formatted correctly

Is there a way to add this without copying most of sendMessage?
In the past, this sort of duplication has been a sign that maybe it should just be integrated as a different type of message in sendMessage.

sorry for late response

bump

Additional bump :)

@Schmavery

@Schmavery please, check this out
I had to create a forked version of the repo just to get the new stuff because this one is way too old

@Schmavery bump

ok, fixed that


nxdomain responses include huge timeouts

There doesn't appear to be a way to set minTTL for nxdomain replies at this time. This would be very helpful to prevent upstream dns servers from breaking how acme-dns works.  I'm running into this issue currently with an infoblox based dhcp/dns server upstream. It holds onto the nxdomain reply for an hour+ and never checks back with acme-dns till it's too late and the letsencrypt request has expired.

I believe that the last field in the SOA record should be set to a much lower value (eg, 1 or 60) to prevent NXDOMAIN responses from being cached for a long term. It's currently 86400, which may lead to NXDOMAIN responses being cached for up to a day.
https://github.com/joohoi/acme-dns/blob/master/dns.go#L65

Good point, 60sec should be sufficient.


Where are the configuration options?

Playing with dev-rome a bit the documentation seems to only go one level deep; listing the commands that are available on ./scripts/dev-rome --help. Specific help for subcommands was not present and so I ran into 👇🏻 quite quickly.
Couldn't find a rome.json or rome.rjson or .rome.json or .rome.rjson for
    /my/cool/project/path

The following would be really helpful (to me, a totally random new user 😅):

A way to get docs for sub-commands
Some kind of indication of how to interact commands (./scripts/dev-rome config --help)
A pointer to config options and how to hydrate a new project

Seems relevant to https://github.com/facebookexperimental/rome/issues/7

hi,
Rome project config exist as rome.json or rome.rjson files. It can also in the  rome property, in package.json.

#55 will add some documentation.

It looks like this PR https://github.com/facebookexperimental/rome/pull/165 will improve CI output include subcommand help and examples


⚠ Bofrost.lu has degraded performance

In 4122a79, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1166 ms

Resolved: Bofrost.lu performance has improved in 3aa0324 after 6 minutes.


match/switch keyword not swapped for a lifted object type

From BuckleScript (but not specific to it)
let asd : Js.t {. match : int} = {"match": 1};
let foo = asd##match;

Error:
Error: This expression has type { match : int }
       It has no method switch: /Users/chenglou/Github/reason-react-example/src/simple/page.re:31

This is because the second line is translated to let asd: < match :int  > Js.t = .... The match keyword didn't get translated over.

Fixed


Adjust to small screen sizes

We noticed that some parts of the redesign don't work on phone screens yet - we didn't really look at these during the offsite, purposefully.

e.g. categories are off

Footer also looks weird

https://www.figma.com/file/aPmSe86hz49J8n1daelWUBDp/Speakerinnen?node-id=3007%3A130213


feat: add new lambda feature

For interacting with lambda_http, part of the official awslabs toolkit.
Needs more real testing. TBD.
This is easier to do in Tide directly because otherwise you have to make a wrapper, which seems unnecessary (though I could be convinced that perhaps this is still better in a separate crate).

This looks good and works completely fine for me. Only gripe I had was that binary is returned for all content types, but I couldn't find a better solution than just switching it out for lambda_http::Body::Text(body.into_string().await?).
Maybe we could pass the body type as a generic or something? I don't like the idea of checking the headers to find out what type should be returned.
lambda-http 0.4.0 adds explicit lifetimes: 72111315fc58ae60e3d1ccd0e48fb64ab6e4e044

I ended up doing this as the "lambda-http" for the Tide-wrapping framework I maintain for my employer: https://github.com/eaze/preroll#list-of-optional-add-on-features

I've just had a quick look over tide-lambda-listener; by comparison to this solution, it seems unnecessarily complicated? I don't see any difference functionality wise - how come you chose that solution over this?

Oh right. This is also available standalone as https://github.com/Fishrock123/tide-lambda-listener, which I completely forgot about.


Still busy with this?

Hey David, good work I'm busy with a particle system and trying to learn about post processing, I wonder if you were still busy with this project.

Would you mind if we can have a call on discord/or some other app one day and you can explain a bit of your code? I would write it down and make a wiki on my repository and explain that + some other concepts I learned, I think it can help a lot of people.

You can ask here if you have questions.
This is what the ssao effect in this repository is based on: https://learnopengl.com/Advanced-Lighting/SSAO
It's just a translation from opengl to wgpu.