Incompatibility with IntelliJ IDEA 2023.3.2

Hello @epm-dev-priporov,
the plugin is not compatible with the latest Intellij IDEA.
Could you please upgrade it?
Error message
Plugin 'Format-converter' (version '0.8.2') is not compatible
with the current version of the IDE, because it requires build 232.* or older
but the current build is IU-233.13135.103

Intellij IDEA information
IntelliJ IDEA 2023.3.2 (Ultimate Edition)
Build #IU-233.13135.103, built on December 20, 2023
Runtime version: 17.0.9+7-b1087.9 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 11.0
Kotlin: 233.13135.103-IJ

Thank you in advance,

@aerfus hello, thank you for the report. It's done and will be available in the upcoming update, now it's reviweing by JB, usually it takes 1-2 business days

@aerfus done.
Available fir downloading


Add support for Thymeleaf

Thymeleaf is a popular rendering engine which would be useful to support.

Thymeleaf relies on an event-based approach which prevents the usage of directives unfortunately.
More information can be found in issue 68 in https://github.com/thymeleaf/thymeleaf.
There therefore doesn't seem to be any value in adding Thymeleaf to Tiles Autotag.


Support for EF Core

Any idea if you are going to add support for Entity Framework Core?

I will be supporting EF Core. Actually, before they renamed it to EF Core, when it was EF7, I did some work converting this project to work on EF7 on this branch: https://github.com/NickStrupat/EntityFramework.Triggers/tree/ef7
I never made a NuGet package for it. Also it's decently behind the master branch now.
:D

That's so awesome 👍 Doesn't look like it was much work.
Yeah unfortunately they changed the name after they had already started calling it EF7 😞
Are you looking to support EF6 and EF Core?

Yes, I'm planning to support EF6 and EF Core for all of my EF projects :)

Just wanted to give you an update...
I'm a couple of days away from having a new package called EntityFrameworkCore.Triggers which is for .NET 4.5 and .NET Core (EntityFrameworkCore runs on both of those platforms).
EntityFramework.Triggers will be the same old package for .NET 4.0 and .NET 4.5.
All the code will stay in this repository.


KCL-10481 - Reconstruct data and selection from DOM to reliably resol…

Thanks for the review. Updated based on the comments. If you need anything to explain, we can have a call.


Settings seem to be wiped ios

So my app been published for more than a year now , and recently started to hear complainants from some iphone users that the app is kicking them out to the login page .
What  I do in my code is saving user ID in Settings.UID which is 0 by default , then it sets to user ID when they log in , and the only way to set it back to 0 if they log out . I do the following in App.OnStart()
if (Settings.UID != 0)
MainPage = new HomePage();
else
MainPage = new LoginPage();
Pretty simply , and straightforward . No possible mistakes.
I started doing my research and found an old ios bug
https://forums.developer.apple.com/message/207807#207807
https://forums.developer.apple.com/thread/44264
https://forums.developer.apple.com/message/45849#45849
https://forums.developer.apple.com/message/206691#206691
https://disqus.com/home/discussion/teamloom/how_we_fixed_the_ios7_forced_logout_bug_thats_been_plaguing_so_many_apps/
https://forums.developer.apple.com/message/205161#205161
briefly ios clears NSUserDefaults  , and I know that this plugin saves directly to NSUserDefaults , so I was wondering if this happened to anybody before and if so any suggestions on how to solve it .
Thank you

Not that I know of at all.
Reading the docs should be fine: https://developer.apple.com/documentation/foundation/nsuserdefaults
https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/UserDefaults/AccessingPreferenceValues/AccessingPreferenceValues.html
https://stackoverflow.com/questions/1638766/do-nsuserdefaults-persist-through-an-update-to-an-app-in-the-appstore

@kamelmojawaz1 I also started hearing this issue. But I've never had it on my test devices.
Did you manage to find the source of the problem?


Is it possible to use shorter and thus better readable methodnames in JSON?

Right now we sometimes have quite long and hard to read method names in the JSON-Files. This becomes tedious if the method names are slightly refactored, so that the names in the JSON files need to be changed too.  If the change is not done together with the original refactoring but instead as part of a bugfix a few days later, or by another person that doesn't now anything about the refactoring, it becomes hard to find , or even notice the difference.
The names are this extensive, because they must be distinct. Maybe this can be circumnavigated by checking if the name already exists in the SampleRepository. If this is not the case a short name can be used. Otherwise the extensive distinct name must be used.
Is this possible?

When I remember correctly, there is an option to overwrite the method name with a custom string. I guess nobody will consistently implement it like this, when there is no way to force it, so this feature might be useless. Perhaps we should just add some kind of strict mode? This mode will check a. that all given names are unique and b. that the name was set. Back then i did intend that it should be the default mode to name the method/pointcut/whatever always yourself to avoid this refactoring problem.


update toki pona

toki pona translations have been added for all new strings + some minor tweaks were made to existing strings

Thank you, merging now


Add support for sharp-duotone-solid icons

This PR adds support for the new sharp-duotone-solid variation, released with FontAwesome 6.6.0.
Thanks!

Worth noting, I set the prefix in the config file to fasds. This seems to differ from the convention used by the other prefixes.
I don't know how the other sharp set prefixes were determined, but fasds is the only way I could get these to load with Blade Icons. In my project I've had to override the config to change the prefixes for all the other Pro sets too (e.g. sharp-solid should be fass, sharp-thin should be fast, etc.), otherwise the Blade Icons package refused to load them.
Not sure if that's specific to my setup or just incorrect historically, but FYI @owenvoke in case that needs changing.

Thank you, @JackWH! Appreciate this. I'll fix the Pint changes separately, and then release.


Fix issues around new get_catalog_by_relations macro

resolves #8846
Problem
An optional parameter was added to BaseAdapter.get_catalog() in 1.7.0rc1, which was unfortunately overridden by databricks, and may also be overridden in other adapter implementations.
Solution
This changeset reverts the function to its original signature, and adds a new function called BaseAdapter.get_filtered_catalog(). It also reworks the docs generate code which had relied previously relied on the new parameter.
Significantly, the filtering is now performed against a set of BaseRelation objects passed to the adapter, instead of a set of nodes. This reduces the number of concepts that leak into the adapter, which should not need to "know" about the dag or nodes.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[x] This PR includes type annotations for new and modified functions

I think we might have introduced a pre-regression with this PR. See the details here: https://github.com/dbt-labs/dbt-snowflake/pull/816.


The add URL button for adding social network profiles in the integrated theme (orange) does not function

What happened?
The add URL button for adding social network profiles in themes does not function “TypeError: bootstrap.Popover.getInstance(…).getTipElement is not a function”! Previous versions of Tastyignter posed no issues.
What did you expect to happen?
The add URL button previously allowed for the adding of social network profiles to the footer
Version
3.5.x
What browser are you seeing the problem on?
All Browsers (Chrome, Firfox, etc)
Relevant log output
“TypeError: bootstrap.Popover.getInstance(…).getTipElement is not a function”!

Fixed


ASSET DRAINED - FP

Attacker: 0x0d745ad687f2b1e1941569f09f612f60ad4ad5bc
AD Alert: https://app.forta.network/alerts/0x996bd3802d5fd41322b9eaa4e0cddcce24a1eafa18eec64c8c5604295187d28e
Base bots:  ASSET-DRAINED Bot.Ownership Transfer Bot
Potential cause: base alert is FP for Txn

Something to look at when we start working on the exploitation AD


VS2017 error CS1061: “Task” does not contain a definition for  “ConfigureAwait”

I use vs2017  compile NjsonSchema，then tips
error CS1061: “Task” does not contain a definition for “ConfigureAwait”
Is something I config error ？

This is when you want to build the solution?

Is .NET Core installed?
I havent tried with VS 2017 (i'm using VS 2019)
Have you tried to build on the cmd line? => dotnet build

Do you have something else in your solution named "Task" that could conflict with your definition?
You didn't say which framework version you are using, but ConfigureAwait has been there for a long time : it has been introduced in .net 4.5 : https://docs.microsoft.com/en-us/dotnet/api/system.threading.tasks.task.configureawait?view=netframework-4.8

@jeremyVignelles I think NET40 uses the AsyncBridge (or similar) which provides that. It works on my machine and in the CI build. So this must be a problem with @NeroBringer 's machine setup.


Examples on how to use connection string from code is missing

The page is all about the benefits of using a connection string with Application Insight rather than only the instrumentation key.
But examples on how to use the connection string from a .NET Core console application is missing. When clicking on the links from this page all the examples are using just an instrumentation key.
There should be more examples on how to use Application Insight with just a connection string
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 180c6ca8-2bdc-4685-ebc5-aaea2236937f
Version Independent ID: 74f63340-7428-a3a7-6d6c-701d3be4348d
Content: Connection strings in Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/sdk-connection-string.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @TimothyMothra
Microsoft Alias: tilee

@olavt - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

#assign: @rishabjolly

@olavt We are looking into this as part of a larger effort to update all Application Insights articles from instrumentation key to connection string examples.
I have created an internal work item to continue work on this issue.
https://dev.azure.com/mseng/TechnicalContent/_workitems/edit/1928114

#please-close


events: Add documentation and tests for typing: stop event.

The typing: stop event did not have any tests in test_events
hence its documentation wasn't added. So add tests and relevant
documentation for the typing stop event. Also edit the documentation
of typing: start to include the fact that servers should use
their own timeout incase stop event event isn't received.
Fixes #16122 .

Merged as the series ending with c81b9cb51627caa6e2627acf557aa3e18f607e97 after adding a bit more prose; thanks @orientor!


Please add a different key for fire, not everyone has Z next to X

Could you please add a different key for fire?
Not everyone has Z next to X, i for example have a German keyboard and have Y next to it, making it awkward to play.

Done. I took the opportunity to add support for other keyboard layouts. Thank you! :smile:


Use utxo in TestBlockInfo

See https://github.com/mintlayer/mintlayer-core/pull/299#discussion_r927465928 for the details.

Keep in mind that this already exists in the utxo crate but needs some maintenance and needs to be used.

@azarovh I think it is useful to reference the pull request that closes the issue.

The issue is not closed actually. I renamed it properly and reopened.

TestBlockInfo doesn't exist anymore in the chainstate test suite. This is considered over.


Fix the order of indices when merging

This PR should fix the problem on id mismatching. Ids is mismatched because merging order is wrong.

Would be great to have a test reproducing the problem and checking that now it works

I didn't have a good idea on how to reproduce the problem by a TDD (by adding a unit test).
But If we print out the log, i.e. print out local_file_paths, it is pretty obvious that we had an error:
Before:
2022-02-22 01:50:32,008 [INFO]: ['/tmp/tmp2msoal_r/index_11', '/tmp/tmp2msoal_r/index_12', '/tmp/tmp2msoal_r/index_10', '/tmp/tmp2msoal_r/index_0', '/tmp/tmp2msoal_r/index_9', '/tmp/tmp2msoal_r/index_6', '/tmp/tmp2msoal_r/index_8', '/tmp/tmp2msoal_r/index_5', '/tmp/tmp2msoal_r/index_7', '/tmp/tmp2msoal_r/index_3', '/tmp/tmp2msoal_r/index_4', '/tmp/tmp2msoal_r/index_2', '/tmp/tmp2msoal_r/index_1']

After:
2022-02-22 02:00:45,206 [INFO]: ['/tmp/tmpyduy7mav/index_00', '/tmp/tmpyduy7mav/index_01', '/tmp/tmpyduy7mav/index_02', '/tmp/tmpyduy7mav/index_03', '/tmp/tmpyduy7mav/index_04', '/tmp/tmpyduy7mav/index_05', '/tmp/tmpyduy7mav/index_06', '/tmp/tmpyduy7mav/index_07', '/tmp/tmpyduy7mav/index_08', '/tmp/tmpyduy7mav/index_09', '/tmp/tmpyduy7mav/index_10', '/tmp/tmpyduy7mav/index_11', '/tmp/tmpyduy7mav/index_12']

Ok merged
A test idea would be to use autofaiss in distributed mode that would generate index + several ids files (force ivf flat)
Then in the test manually generate a flat index and manually build an id mapping
Then do search on them and check the results (after using the id mapping) is the same

It could be useful to do it to make sure there is no additional bug and that we don't introduce any in the future


Cursor not resetting to normal color

Hi there! Nice plugin!
I like the plugin, and It works pretty well, except for the fact that the cursor color is not resetting to the default when I come from another mode (visual or yank)
Any clues why this might be happening?
THanks!

Can you provide your configuration along with the reproduction steps?

Hi, this is my config for this plugin:
return {
  "mvllow/modes.nvim",
  config = function()
    require("which-key").setup({
      plugins = {
        presets = {
          operators = false,
        },
      },
    })
    require("modes").setup({
      colors = {
        copy = "#f5c359",
        delete = "#c75c6a",
        insert = "#78ccc5",
        visual = "#9745be",
      },

      -- Set opacity for cursorline and number background
      line_opacity = 0.15,

      -- Enable cursor highlights
      set_cursor = true,

      -- Enable cursorline initially, and disable cursorline for inactive windows
      -- or ignored filetypes
      set_cursorline = false,

      -- Enable line number highlights to match cursorline
      set_number = false,

      -- Disable modes highlights in specified filetypes
      -- Please PR commonly ignored filetypes
      ignore_filetypes = { "NvimTree", "TelescopePrompt" },
    })
  end,
}

This is the only plugin I have installed with a fresh lazyvim installation. I am not able to figure out which causes the problem. I will dig deeper when I get time

Does your issue persist after removing which-key?

Hi, I just tried out. The issue is still there

im having the same issue. installed this along with a fresh lazyvim installation

I just tried lazyvim and still can not reproduce the issue, what terminal emulator are you using?

im using wezterm on windows
the wezterm cursor colors are set like this:
[colors]
background = '#363646'
foreground = '#dcd7ba'
cursor_bg = '#c8c093'
cursor_fg = '#1f1f28'
cursor_border = '#c8c093'

plugin settings are pretty standard:
function M.config()
  require("modes").setup({
    colors = {
      copy = "red",
      delete = "yellow",
      insert = "green",
      visual = "blue",
    },
    -- Set opacity for cursorline and number background
    line_opacity = 0.20,

    -- Enable cursor highlights
    set_cursor = true,

    -- Enable cursorline initially, and disable cursorline for inactive windows
    -- or ignored filetypes
    set_cursorline = true,

    -- Enable line number highlights to match cursorline
    set_number = true,

    -- Disable modes highlights in specified filetypes
    ignore_filetypes = { "neo-tree", "NvimTree", "aerial", "OverseerList", "TelescopePrompt" },
  })
end

and for lazyvim, its a fresh install and the only files in lua/plugins is the settings for this plugin and a theme
https://github.com/mvllow/modes.nvim/assets/5199715/4e8d20d2-d5ca-4168-a234-1f386e5cfdb9

I just tried wezterm + lazyvim on linux and macOS, and still can not reproduce the issue, unfortunately I don't have a windows machine at the moment to test it, maybe this is windows related issue

@franroa are you also using windows?

just tested this in neovide and it works ok. not sure what that might suggest...

@franroa are you also using windows? What terminal emulator are you using?
Also windows, with wsl

The problem appears on Powershell, Weztern and kitty (all that I use)

Having the same issue.
Env: Window terminal, and even exit neovim the color keeps. I am guessing the neovim cursor color is somehow bind with windows terminal cursor color.

I cannot reproduce the issue on the following platforms

Linux: alacritty, st, kitty, foot
macOS: alacritty, wezterm

I was able to reproduce the issue with macOS + iTerm, but this is a known limitation of iTerm regarding cursor shape manipulation
Can you guys reproduce the issue with the followning config @franroa @sho-87 @RayGuo-ergou
-- save as: guicursor.lua
local api = vim.api
local hi = api.nvim_set_hl

hi(0, "CursorRed", { bg = "Red" })

local i_cur = "i:ver20-CursorRed"

api.nvim_create_autocmd("InsertEnter", {
  callback = function()
    vim.opt.guicursor:append(i_cur)
  end,
})

api.nvim_create_autocmd("InsertLeave", {
  callback = function()
    vim.opt.guicursor:remove(i_cur)
  end,
})

Then open nvim with
nvim --clean -S guicursor.lua -- guicursor.lua

The expected behavior would be to set the cursor to red vertical line in insert mode and reset to default after leaving insert mode
If you can reproduce the issue with the steps above, I would assume it is an upstream (OS and/or terminal emulator) limitation/issue
Also let me know if you are able to reproduce the issue on windows with alacritty, I don't have a windows machine at the moment to test it

hmm when I try that my cursor doesnt change colour to red at all. Ive tried this in wezterm, powershell and cmd

wsl (ubuntu) + windows terminal cannot see red cursor.

neovim version:
nvim -V1 -v
NVIM v0.10.0-dev-2508+ga8131aee9
Build type: RelWithDebInfo
LuaJIT 2.1.1707061634
Compilation: /usr/bin/cc -O2 -g -Og -g -flto -fno-fat-lto-objects -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wvla -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -fsigned-char -fstack-protector-strong -Wno-conversion -fno-common -Wno-unused-result -Wimplicit-fallthrough -fdiagnostics-color=auto  -DUNIT_TESTING -DHAVE_UNIBILIUM -D_GNU_SOURCE -DINCLUDE_GENERATED_DECLARATIONS -I/home/raydev/git/neovim/.deps/usr/include/luajit-2.1 -I/home/raydev/git/neovim/.deps/usr/include -I/home/raydev/git/neovim/build/src/nvim/auto -I/home/raydev/git/neovim/build/include -I/home/raydev/git/neovim/build/cmake.config -I/home/raydev/git/neovim/src -I/usr/include

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/share/nvim"

Run :checkhealth for more info

Found tworelative issues in neovim:
https://github.com/neovim/neovim/issues/18053#issuecomment-1094302350
https://github.com/neovim/neovim/issues/23094
Thinking this is an upstream issue.
For me I can live without cursor color change.
And I have to say this is a very amazing plugin and thank you so much for your work and help @fitrh

Found two relative issues in neovim: neovim/neovim#18053 (comment) neovim/neovim#23094
Thinking this is an upstream issue.
For me I can live without cursor color change.
And I have to say this is a very amazing plugin and thank you so much for your work and help @fitrh

Same here

So i have this problem too and could replicate it with guicursor.lua
Howether i found a workaround if i know what color i want the cursor to be, then i can always set it.
`-- save as: guicursor.lua
local api = vim.api
local hi = api.nvim_set_hl
hi(0, "CursorRed", { bg = "Red" })
hi(0, "CursorWhite", {bg= "White"})
local i_cur = "i:ver20-CursorRed"
local n_cur = "i:ver20-CursorWhite"
api.nvim_create_autocmd("InsertEnter", {
callback = function()
vim.opt.guicursor:remove(n_cur)
vim.opt.guicursor:append(i_cur)
end,
})
api.nvim_create_autocmd("InsertLeave", {
callback = function()
vim.opt.guicursor:remove(i_cur)
vim.opt.guicursor:append(n_cur)
end,
})
`

require("modes").setup()
vim.opt.guicursor:append("n:block-Cursor") -- you can have it before or after setup, doesn't matter

the only issue with this solution is that if you leave the window without being in normal mode,
the cursor will remain the same. for example, you are in insert mode, but you switch to a different tmux panel.
using Windows Terminal WSL2

After looking deeper, i found that the fix from @zahimeen works on tokyo-night theme, but not on gruvbox.

After looking deeper, i found that the fix from @zahimeen works on tokyo-night theme, but not on gruvbox.

For my solution, the Cursor highlight bg must be defined.
The reason it works in the tokyonight.nvim theme is because of this line.
https://github.com/folke/tokyonight.nvim/blob/b9b494fa7f7bbf2fe0747b47fa290fb7a4eddcc7/lua/tokyonight/theme.lua#L34
There are many gruvbox ports so I can't check, but as far as I am concerned, they do not explicitly overwrite the Cursor highlight, instead just inheriting from the terminal.

Happy to accept a PR for a note under our known issues section :)


Add an OEP-2 compliant openedx.yaml file

This adds an openedx.yaml file, as described by OEP-2:
http://open-edx-proposals.readthedocs.io/en/latest/oeps/oep-0002.html
The data in this file was transformed from the contents of
edx/repo-tools-data:repos.yaml

@cpennington looks like this PR got clobbered.

A few comments.

Is this work stable now, that is, are you waiting for thumbs?

@nedbat: Yes.

Reviewed.

@shaunagm: This is the first of my PRs.

Only two new micro-nits. :+1:


Can not deploy the App on my Subscription

I am the owner of the subscription
It fails on the Step #4 Registering Azure AD App.
Have tried 2 times with 2 different regions
Log-07-05-2021-09-43-57.log
Log-07-05-2021-09-50-57.log

@Ivan-Ada
Hi did you manage to get the installation working? if so can we close this issue?

@leestott
yes it works, thank you so much!


bigquery: make queries faster

Simple benchmarks introduced by
https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/1775
showed that Ruby was much slower than other languages.
Investigations showed that we spent disproportionate amount of time
deserializing JSON into TableData, just to later convert it into
Data.
This commit circumvents this work by deserializing JSON into a hash
instead using JSON.parse, and later convert the hash into Data.
On my machine, this made the benchmarks run about 4x as fast.

Can you post some before and after benchmarks?

@geigerj Is a performance degradation like this something that should be addresses in the apiary client?

@blowmage We do own the google-api-ruby clients, so it's possible to make changes there if @pongad would rather address it at that level. That said, they are officially in "maintenance mode", so I think it is OK to make this change at the google-cloud-ruby level unless we would see significant cross-API impact at little additional implementation cost.

@blowmage

Can you post some before and after benchmarks?

I have the complete numbers in a spreadsheet, but I'm not sure how to publicly share it. I'll summarize in paragraph form here.
Previously running 5 queries take ~1,600s wall clock time and ~1400s CPU time. We spend most of the time deserializing JSON in Apiary client.
With this change, we run in ~400s wall clock time and ~140s CPU time. While the CPU usage is still significant, we spend most of the time waiting for RPC now. This is measured on my development machine on Google network.

@blowmage @quartzmo PTAL


Suggestion about the tool to generate mask image?

Thanks for your great work. I am trying to try "paint with words" using your code, but I can not find an easy way to get the mask image. I have tried "Microsoft Paint", but there are many small holes around the edge. Can you share the tool you use to generate the mask image?

Try using free software GIMP, Photoshop if available, or changing your brush settings within MS paint - all of these can create the mask images used in the examples depending on brush settings

@mittski Thanks for yours help, I find an online paint tool named jspaint which has the same function. And I also try MS paint again based on your suggestion, I find that if I use pencil instead of other brushes, the hole will be disappeared.


Not working after installation

Bug Report
Doe not work after installation.

Q
A

BC Break
no

Library Version
0.1.13

PHP version
8.0.0

Summary
Just installed in my machine using composer require phparkitect/phparkitect. Using command ./vendor/bin/phparkitect check I see the following error message: "PHP Warning:  Undefined array key "version" in /Users/simonegentili/Development/github.com/sensorario/prova-architect/vendor/phparkitect/phparkitect/src/CLI/Version.php on line 24". And also this error message: "PHP Fatal error:  Uncaught TypeError: Arkitect\CLI\Version::get(): Return value must be of type string, null returned in /Users/simonegentili/Development/github.com/sensorario/prova-architect/vendor/phparkitect/phparkitect/src/CLI/Version.php:24".
My composer.json is
{
    "require": {
        "phparkitect/phparkitect": "^0.1.13"
    }
}

Current behavior
It is simple not working.
How to reproduce
Just install phparkitect and then run './vendor/bin'/phparkitect'
Expected behavior
I dont know actually.

Trying to run test I can see that my php version is too recent.
1) Arkitect\Tests\Unit\Analyzer\FileParserTest::test_parse_file_with_name_match
PHP < 8.0 is required.

I think that it's not a PHP version problem but a problem when we try to get the version of phparkitect for the CLI.
I-m working on it here #157

We have released a new version 0.1.14 with the fix.
At the moment I close this issue but feel free @sensorario to write again here if you have the same problem.


Maven Checkstyle + format all code

Looks like some of the code will still get reformatted by the eclipse (mvn java-formatter:format shows quite a few).   We should reformat them in one bunch, make a checkstyle version of the eclipse formatter, and add that to the verify lifecycle.

#276
Passing on checkstyle for now;  formatter should be attached to the process-sources lifecycle, so should be applied on any and every build.  Still may be worth a checkstyle enforcement later, but we will try this for now.


Fix error markup locations in no-consecutive-blank-lines tests

☠ TSLint's time has come! ☠
TSLint is no longer accepting most feature requests per #4534. See typescript-eslint.io for the new, shiny way to lint your TypeScript code with ESLint. ✨
It was a pleasure open sourcing with you all!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


plexamp: 4.10.1 -> 4.11.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for plexamp is: Beautiful Plex music player for audiophiles, curators, and hipsters
meta.homepage for plexamp is: https://plexamp.com/
meta.changelog for plexamp is: https://forums.plex.tv/t/plexamp-release-notes/221280/74
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.11.0 with grep in /nix/store/h6hcjyw1b331yk34ln01hi0l02569s63-plexamp-4.11.0
found 4.11.0 in filename of file in /nix/store/h6hcjyw1b331yk34ln01hi0l02569s63-plexamp-4.11.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

plexamp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/h6hcjyw1b331yk34ln01hi0l02569s63-plexamp-4.11.0 \
  --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 plexamp https://github.com/r-ryantm/nixpkgs/archive/0cde7df13a5d2d6b06867a0b7e412e72fafaf34e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0cde7df13a5d2d6b06867a0b7e412e72fafaf34e#plexamp

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

plexamp

Maintainer pings
cc @killercup @Redhawk18 @Synthetica9 for testing.

@NixOS/nixpkgs-merge-bot merge

If someone could apply the backport label when merged for 24.05

Result of nixpkgs-review pr 328085 run on x86_64-linux 1

1 package built:

plexamp


Enable two-factor authentification

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Google Authentificator
SMS OTP
etc..

Dublicate of #3811


alternating cell fill?

is this possible given the current implementation? you can specify the cell fill but there doesn't seem to be an easy mechanism for filling on alternating rows

It seems like cell/header backgrounds don't work at all - using this attribute just paints a black rectangle, you can't specify the color. Looking at the code, the color option never gets passed through: https://github.com/voilab/voilab-pdf-table/blob/master/voilab-table.js#L42

It seems like cell/header backgrounds don't work at all - using this attribute just paints a black rectangle, you can't specify the color. Looking at the code, the color option never gets passed through: https://github.com/voilab/voilab-pdf-table/blob/master/voilab-table.js#L42

To set the background color you need to call doc.fillColor on the PDFDocument prior to calling addBody / addHeader - the problem is that doc.fillColor also defines the text color.  So if you ever set fill or fillHeader as far as I can tell you will never see the text in the cell just a colored cell.  It's not really useful in it's current form - we'd need to add some "color" properties to the column definition for both foreground and background colors.

I have managed to get alternate row shading working using onRowAdd and onPageAdded. I shade each row using a rectangle with the fill and stroke set to the shading colour. The majority of rows are shaded in onRowAdd this is triggered just before the row is added so the shading appears behind the text.
However I was getting an issue where a blank area was being shaded just before a page break. To fix this I had to detect when a row was about to trigger a page break and perform the shading inside the onPageAdded event.
The code could probably be made neater but here it is:
// The below section of code draws a background behind each row.
// rowShadingCounter initialised to 0 elsewhere
.onRowAdd((tb, row) => {
  currentRowHeight = row._renderedContent.height;
  const pageBreakPoint = doc.page.height - doc.page.margins.bottom - 5; // minus the row top padding of 5

  // check if a page break is about to occur when the row is added. If so leave onPageAdded to set the row background
  if (tb.pdf.y > (pageBreakPoint - currentRowHeight)) return;

  const shade = rowShadingCounter % 2 === 0 ? colours.rowShade1 : colours.rowShade2;
  // shade the row
  rowShading(tb.pdf, 70, tb.pdf.y, 449, currentRowHeight, shade)
    .fillAndStroke(colours.black, colours.black); // set back to black for text

  rowShadingCounter++;
})
.onPageAdded(tb => {
  const shade = rowShadingCounter % 2 === 0 ? colours.rowShade1 : colours.rowShade2;
  // shade the first row, add 10 to the y to avoid shading the table headers
  rowShading(tb.pdf, 70, (tb.pdf.y + 10), 449, currentRowHeight, shade)
    .fillAndStroke(colours.black, colours.black); // set back to black for text

  rowShadingCounter++;
  // add table headers
  tb.addHeader();
});

const rowShading = (doc, x, y, w, h, shade) => {
  return doc
    .rect(x, y, w, h)
    .fillAndStroke(shade, shade);
};

This results in:

I've tested it with rows of varying heights and it still handles it.

Nice job! What you can do is to pack your code in a plugin (check FitColumn Plugin for a base to start).
I can then reference it here if it fit the needs of many.

Nice job! What you can do is to pack your code in a plugin (check FitColumn Plugin for a base to start).
I can then reference it here if it fit the needs of many.

I've done as you've suggested. I wasn't sure how you wanted me to send the plug-in so I've opened a new PR, hope that is okay.


Failing to build link using llvm-3.6.2

chris@heimdall:~$ apt-cache polyc llvm-3.6
E: Invalid operation polyc
chris@heimdall:~$ apt-cache policy llvm-3.6
llvm-3.6:
  Installed: 1:3.6.2-1
  Candidate: 1:3.6.2-1
  Version table:
 *** 1:3.6.2-1 0
        500 http://ftp.nz.debian.org/debian/ stretch/main amd64 Packages
        100 /var/lib/dpkg/status

chris@heimdall:~/devel/ponyc$ make config=release
genopt.cc
genfun.c
host.cc
genobj.c
genprim.c
genheader.c
gentype.c
gencall.c
program.c
platformfuns.c
package.c
use.c
reach.c
paint.c
symbols.cc
dwarf.c
docgen.c
traits.c
pass.c
expr.c
scope.c
finalisers.c
flatten.c
names.c
sugar.c
syntax.c
import.c
token.c
error.c
stringtab.c
printbuf.c
lexer.c
bnfprint.c
id.c
frame.c
parser.c
ast.c
source.c
symtab.c
parserapi.c
subtype.c
cap.c
alias.c
viewpoint.c
sanitise.c
lookup.c
reify.c
matchtype.c
assemble.c
call.c
reference.c
control.c
operator.c
array.c
ffi.c
match.c
lambda.c
literal.c
postfix.c
Linking libponyc
pagemap.c
pool.c
alloc.c
heap.c
threads.c
trace.c
cycle.c
gc.c
objectmap.c
delta.c
actormap.c
stat.c
socket.c
ssl.c
stdfd.c
posix_except.c
lsda.c
time.c
paths.c
directory.c
messageq.c
actor.c
fun.c
stack.c
list.c
hash.c
asio.c
event.c
epoll.c
mpmcq.c
cpu.c
scheduler.c
start.c
options.c
Linking libponyrt-pic
gtest_main.cc
gtest-all.cc
Linking libgtest
main.c
Linking ponyc
build/release/libponyc.a(genopt.o):(.rodata._ZTV11HeapToStack[_ZTV11HeapToStack]+0x40): undefined reference to `llvm::FunctionPass::createPrinterPass(llvm::raw_ostream&, std::string const&) const'
collect2: error: ld returned 1 exit status
Makefile:389: recipe for target 'build/release/ponyc' failed
make: *** [build/release/ponyc] Error 1

Any ideas on how I can proceed?

I have just found some traces of llvm-3.5 still on my system, so I suspect the linker is being confused.
Will report back after I have cleaned that up.

I had a bunch of trouble and wasn't able to get it to behave under debian testing, moved to arch and the problem is solved.


OpenOCD Error - CH32V003

I tried using the guide to flash a CH32V003F4P6 board and it does not seem to work. To ensure that it is not a problem with my setup, I have tested the wiring and hardware using Mounriver studio and it works fine.
My Setup
Ubuntu 22.04 (WSL2)
Wiring ->https://embeetle.com/figures/supported-hardware/wch/boards/ch32v003f4p6-evt-r0-1v1/connections_01.webp
udev rules are fine, and user is added to plugdev group
ls -l /dev/bus/usb/001/004
crw-rw---- 1 root plugdev 189, 3 Dec 31 17:35 /dev/bus/usb/001/004

WCH-LinkE-R0-1v3 was updated to the latest firmware my Mounriver studio.
Toochain setup
git clone --recursive https://github.com/karlp/openocd-hacks/
./bootstrap
./configure CFLAGS="-Wno-error" --prefix=/opt/wch-openocd --program-prefix=wch- --enable-wlink
sudo make install
export PATH=$PATH:/opt/wch-openocd/bin

Issue
Ran erase operation but gave me the following error
$wch-openocd -f wch-riscv.cfg -c init -c halt -c "flash erase_sector wch_riscv 0 last " -c exit
Open On-Chip Debugger 0.11.0+dev-g266d7c8d3 (2023-12-28-12:40)
Licensed under GNU GPL v2
For bug reports, read
        http://openocd.org/doc/doxygen/bugs.html
Info : only one transport option; autoselect 'jtag'
Ready for Remote Connections
Error: unknow WCH-LINK

Sorry for late reply.
Most likely it's WCH-LINKE version and firmware version issue, maybe the MRS studio update it to new firmware version that openocd-hacks not supported.
I have a WCH-LINKE R0 1V2 and tested with all resources mentioned in this repo, it worked as expected. according to the msg below, the firmware version of WCH-LINKE R0 1V2 is Info : WCH-LinkE-CH32V307  mod:RV version 2.8.
I will try later after get a R0 1V3 WCH-LINKE adapter.
Open On-Chip Debugger 0.11.0+dev-g266d7c8d3-dirty (2024-11-22-00:12)
Licensed under GNU GPL v2
For bug reports, read
        http://openocd.org/doc/doxygen/bugs.html
Info : only one transport option; autoselect 'jtag'
Ready for Remote Connections
Info : Listening on port 6666 for tcl connections
Info : Listening on port 4444 for telnet connections
Info : WCH-LinkE-CH32V307  mod:RV version 2.8
Info : wlink_init ok
Info : This adapter doesn't support configurable speed
Info : JTAG tap: riscv.cpu tap/device found: 0x00000001 (mfg: 0x000 (<invalid>), part: 0x0000, ver: 0x0)
Warn : Bypassing JTAG setup events due to errors
Info : [riscv.cpu.0] datacount=2 progbufsize=8
Info : Examined RISC-V core; found 1 harts
Info :  hart 0: XLEN=32, misa=0x40800014
[riscv.cpu.0] Target successfully examined.
Info : starting gdb server for riscv.cpu.0 on 3333
Info : Listening on port 3333 for gdb connections

I got a WCH-LinkE r0 1v3 today, the firmware version is 'v2.14', and make it work again.
Please use this updated and patched OpenOCD based on upstream OpenOCD 0.12 dev codebase: https://github.com/cjacker/wch-openocd.
More firmware version and future models should also get supported.

I got the almost latest source of WCH OpenOCD, also at: https://github.com/cjacker/wch-openocd


[SPARK-22696][SQL] objects functions should not use unneeded global variables

What changes were proposed in this pull request?
Some objects functions are using global variables which are not needed. This can generate some unneeded entries in the constant pool.
The PR replaces the unneeded global variables with local variables.
How was this patch tested?
added UTs

Test build #84548 has started for PR 19908 at commit a33bd99.

Test build #84548 has finished for PR 19908 at commit a33bd99.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
    case class ReqAndHandler(req: Request, handler: MemberHandler)
trait TypeCoercionRule extends Rule[LogicalPlan] with Logging
case class AnalysisBarrier(child: LogicalPlan) extends LeafNode

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84548/
Test PASSed.

@cloud-fan @kiszk @viirya may you please review this? Thanks

Test build #84571 has started for PR 19908 at commit e7cdbd7.

Test build #84571 has finished for PR 19908 at commit e7cdbd7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84571/
Test PASSed.

Merged build finished. Test PASSed.

LGTM

thanks, merging to master!


2 new NetX KBs and minor change to 3rd

Added 1 support summary KB, 1 feature requrest summary KB, updated / made typo & syntactical fixes to the CLC Network Exchange KB

@JasonEHolland please fix link...see my comment.


design initial archive page for FAQ

We need a initial design for archive page for the FAQs.

....


Improve README.md reading flow

The reading flow of the README.md could be improved by relocating some blocks ordered by their importance.
Would suggest the following order:

Running test
Licencse
Changelog
(Code of Conduct)
Contributing

Sounds good to me. Do it!


Improve support for 12 hour increments

This PR fixes a generalizesinterpolatedTable so that it will work with points of any spacing simultaneously fixing a bug where the interpolated point was being placed 12 hours ahead when the first table row was a UT 12 row.
This also adds 12hr increments on the time slider, but prevents WWT time from updating when the time is 12UT [cuz the sky flips] (there is probably a more elegant solution than a global variable and if statements 😆 ).
And the time slider label now shows the date for the current selected timezone.

Personally I think it would be better to have a good way to more generally update the date/time when the slider time is not a perfect UTC date. This same issue comes up when e.g. selecting one of the 12-hour UTC points.
I have an implementation of that idea that seems to be working which I can push onto here. The basic idea is to not connect selectedTime as a v-model and just handle all the updating in the change listener.


Audience mispelling

Modify this link to include the branch name, and possibly the page this PR modifies:
https://osc.github.io/ood-documentation-test/johrstrom-patch-1/
Add your description here

Would be handy if we had some kind of GH Action to spell check the documents.  Especially if I'm adding docs, as I can't spell and my IDE at this time doesn't check spelling.

Opened a ticket for the same - I just happened to spot this while I was chatting with someone.


Loop-hole in defining array of samplers ?

Say I have an array of samplers in shader code:
uniform sampler2D mySamplers[2];

It looks like it's perfectly fine to do the following
bgfx::UniformHandle hnd = bgfx::createUniform(UniformType::Enum::Sampler, 2);
uint32_t samplers[2] = { 0, 1 };
bgfx::setUniform(hnd, samplers, 2);

to assign texture stage 0 to mySamplers[0] and stage 1 to mySamplers[1].
But then to actually bind a texture to each stage, one would have to call
bgfx::setTexture(0, /* which handle ? */, texHandle1);
bgfx::setTexture(1, /* which handle ? */, texHandle2);

but there are no handle for the individual array elements.
So either setUniform() should complain about array of samplers not being supported or maybe there is a lack of a function to just bind a texture to a stage without setting the uniforms.

Ok my bad.
Looking at the code, it looks like passing BGFX_INVALID_HANDLE to setTexture() only binds the texture. Maybe the documentation could mention that ?

Alright it's an actual inconsistency then.

I see that @hugoam has a branch with a related fix (provides an overload of setTexture() that doesn't require passing a uniform)
https://github.com/hugoam/bgfx/commit/6b159c9d43cbbf11bfebb8afe2038024ccaec47a

That doesn't handle cases where it's needed to pass uniform.

Well in those cases you'd use the original overload of course

It goes against what bgfx is supposed to be... Idea is that you write code and it works everywhere. Your idea worked since D3D9, just GL required sampler always.


Bot crash when it gets spammed

As you can see, when someone spam =play with one music, an error occurs, it's because the bot try to delete the file while it's used.
On Linux the bot will crash, on Windows the bot will have some weird issues such as sound saturated etc...
Well in both cases, the bot stops to answer because it think that it's downloading

For the moment the only way to prevent from this issue is to set the SaveVideos = yes otherwise it will try to delete an used file.

doesn't happen to me

Did you try with a link instead of keywords ?
Btw i've only tested on the master branch

im on review branch. Tested with links too

With save_videos = False ?

no

tested with save videos false

I've understand the problem, when you send too fast the same command, it will add the same song twice at the same time.
It plays the two songs at the same time (weird) and after a couple of time it crash.
Maybe we should add a check to prevent this to happening


ISPN-15166 SIFS rebuild index cannot compact file

https://issues.redhat.com/browse/ISPN-15166

Merged! thanks @wburns !


fix for excluding last column in unpivot macro

Excluding the last column of a table in the unpivot macro would add an additional union all to the end.

Thanks @alexcampana - you can ignore the Circle failure -- i think that's unrelated to your PR.
I'm looking at the original logic though, and I'm confused how this even works!!
{%- for col in cols -%}
    {%- if col.column not in exclude -%}
      {% set _ = include_cols.append(col) %}
    {%- endif %}
{%- endfor %}

{%- for col in include_cols -%}

    select
      {%- for exclude_col in exclude %}
        {{ exclude_col }},
      {%- endfor %}
      cast('{{ col.column }}' as varchar) as field_name,
      {{ dbt_utils.safe_cast(field=col.column, type=cast_to) }} as value
    from {{ table }}
    {% if not loop.last -%}
      union all
    {% endif -%}
  {%- endfor -%}

@alexcampana just got tests passing over here: https://github.com/fishtown-analytics/dbt-utils/pull/82
I'm waiting on a PR. Once that's merged, if you can pull in master, then your tests will pass and we can merge this!

@alexcampana thanks for your patience! I think if you merge master, these tests should now pass!

@drewbanin no problem, looks like the tests are passing now!

@alexcampana I just released 0.1.16. Thanks for the fix!


Nuxt generate moves compiled css into

I want to use Bootsrap with my own colors, so I've made my ~assets/styles/app.scss:
$primary: yellow;
@import '~bootstrap/scss/bootstrap.scss';

Then I've tried to add this to my nuxt.config.js:
  build: {
    extractCss: true
  },
  css: [
    '~assets/styles/app.scss'
  ]

It's all works great, but when I'm trying to run nuxt generate it pushes all result styles into EACH html page. It's around 145 Kb of extra styles w/o caching.
So how to build css file and include it into head manually?

Depending on this comment it should be some vendor common css, isn't it?

Please try to use extractCSS as described here.
This should solve your problem.

I have the same issue, nuxt just embed all complied css into every html head, that makes html very big.
I have try extractCSS, but it doesn't work for me....
nuxt.config.js
module.exports = {
  /*
  ** Router config
  */
  router: {
    middleware: ['check-auth',]
  },
  /*
  ** Headers of the page
  */
  head: {
    title: 'test',
    meta: [
      { charset: 'utf-8' },
      { name: 'viewport', content: 'width=device-width, initial-scale=1' },
      { hid: 'description', name: 'description', content: 'Nuxt.js project' }
    ],
    link: [
      { rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }
    ]
  },
  build: {
    extractCSS: {
      allChunks: true
    },

        vendor: ['vuex','iview','axios','@tinymce/tinymce-vue','vue-amap','sockjs-client','webstomp-client']
  },
  /*
  ** Global CSS
  */
  // css: [
  //   'normalize.css',
  //   '~assets/main.css'
  // ],
  plugins: ['~/plugins/layout','~/plugins/vendor'], // https://nuxtjs.org/guide/plugins/
  css: [
    'normalize.css',
    '~assets/main.css',

    // 项目中的 CSS 文件
    '~/assets/styles/iconfont/iconfont.css',
    // 项目中的 Sass 文件
    { src: '~/assets/styles/iview-custom.less', lang: 'less' },
    { src: '~/assets/styles/common.less', lang: 'less' },
    { src: '~/assets/styles/app.less', lang: 'less' },
    { src: '~/assets/styles/animate.less', lang: 'less' },
    { src: '~/assets/styles/aurora.css'},
  ],
  /*
  ** Environement variables
  */
  env: {
    AUTH0_CLIENT_ID: '',
    AUTH0_CLIENT_DOMAIN: ''
  },
   /*
  ** Customize the progress bar color
  */
  loading: { color: '#3B8070' },
  /*
  ** Build configuration
  */
  build: {
    /*
    ** Run ESLint on save
    */
    extend (config, { isDev, isClient }) {
      if (isDev && isClient) {
        config.module.rules.push({
          enforce: 'pre',
          test: /\.(js|vue)$/,
          loader: 'eslint-loader',
          exclude: /(node_modules)/
        })
      }

    }
  }
};

@TonyLuo Which nuxt version are you using?

the latest one: "nuxt": "^1.4.2",

@TonyLuo As described in the docs:

CSS inside async components will remain inlined as JavaScript strings and handled by vue-style-loader.

If you upgrade to nuxt-edge (upcoming Nuxt 2 build), the default behavior is to extract all css in extra files with webpack4. I'd suggest to try it out.

I try nuxt-edge and set   build: {extractCSS: true}, but it is still not working.
nuxt still embed all compiled css into every html file header

the different files still have the same inline css in the header when using nuxt-edge to generate static files


Invalid scrubling algo

Hi.
In docs 'scrub_fields
Array of field names to scrub out of _POST and _SESSION. Values will be replaced with asterisks. If overridiing, make sure to list all fields you want to scrub, not just fields you want to add to the default. Param names are converted to lowercase before comparing against the scrub list.'
But actually comparing worked as case sensitivity in case target key is not defined in scrub_fields as regular expressions. And in session params not converting to lowercase.

For example scrub_fields = array('password') and $_SESSION['PASSWORD'] will not be scrubled because you did simple in_array('password', array('PASSWORD'));

Test passed because its covered by regular expressions /password/i

Thanks for the report, @dimkalinux . We'd be happy to accept a PR that fixes this if you are up for it; otherwise we'll get to it soon. As a workaround, it sounds like you should be able to use regexes (e.g. "/password/i") instead of strings.

https://github.com/rollbar/rollbar-php/pull/63

Fix released in version 0.15.0. Thanks again!


Add an endpoint to calculate Zero Coupon Bond Value

Add an endpoint to calculate zero coupon bond value.
Create a function in ./helpers/functions.py file for the calculation.
Map it with an endpoint GET /zero_coupon_bond_value in the main.py file with properly.

hey @nikhil25803 please assign this to me.

Go ahead


Audit for Monitoring Plan Submission - Global-View

Definition of Ready

[ ] Does this meet the INVEST criteria?
[ ] Does this have a clearly defined user?
[ ] Does this have an estimate?
[ ] Does this have acceptance criteria?
[ ] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[ ] Does this have a dev link been provided in the design?
[ ] Is this traceable to a feature, epic, or PI?

User Story
As an ECMPS user, I want to view the audit record on a Monitoring Plan in the Global-View, so that I know the user and time who last submitted a Monitoring Plan Record
Acceptance Criteria
Given Monitoring Plan data was submitted successfully
When a user accesses the respective facility in the Global-View
Then the system displays 'Last submitted by: UserID, 8/23/2021'
Development Notes
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[ ] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Testing
Test Case:
Tester:
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Is the user story ready to be deployed to the test/staging environments?

Test Passed


Getting rename or repackaging error.

Hi
I got jco version 12 and when I try to use it, it gives me "Illegal JCo archive "sapjco3-3.0.jar". It is not
allowed to rename or repackage the original archive "sapjco3.jar" error, I followed the example which is given in your web site, maven adds version to the file and jco doesnt like that, what should I do?
Thanks.

Hi, from JCo 3.11 on, the SAP library checks the name of its archive. You can either switch to 3.10 or handle the archive name in a special way.
Either configure Maven to exclude the version or don't handle JCo with Maven (which you usually don't want). See the discussion here: https://sourceforge.net/p/hibersap/discussion/813164/thread/52913577/


Bug 1170736 non ff dl btn collection fix

The only change between this PR and the previous one (https://github.com/mozilla/olympia/pull/588) is the change of the width of the item-info class to make room for a wider button. This button is only shown to non-firefox users.
Previous issue:

Current PR:

It is worthy noting that these buttons currently do not work on the collections page as part of an outstanding bug from 11 months ago (https://bugzilla.mozilla.org/show_bug.cgi?id=1038301). Once that bug is fixed, it should fix this button and the "Continue to Download" button.
@magopian r?

Seems you pushed to the same branch, which makes it difficult to review/rebase/merge. Please prefer creating a new branch next time, thanks!

Looks good to me, I'm just worried about what happens if we have translations that are longer than this text. We'd have the same issue again, right?
I don't know how difficult it would be to mimic what's done with the "standard" install buttons that resize and word-wrap? Mind taking a look at that @FrancescoSTL ?

@magopian Done. This should wrap-text on any concealed button if they would normally overflow :)

This fix was working locally. Looks like zamboni.css was the correct file to be editing. This should now wrap-text on the concealed button where they might overflow.

@FrancescoSTL what is the remaining change in the buttons.less file? As I explained in my previous comment, this file isn't loaded on the collections page, so I guess this also needs to be moved to the zamboni.css file?
I tried moving it to the zamboni.css file to check what it was for, but the end result doesn't look right:

The remaining code in the buttons.less file is for the add-on pages themselves. Sorry, this pull has gotten confusing because when I tried to fix the bug with the collections page, I pushed to the existing branch which contained code for the original CTA.

Superseded by #609


Implement Helpers.

See examples/helpers.js / .html for usage.
I'd appreciate a review :)
cc @lapidus @bcherry @bobthecow @khamidou

+1 (I have nothing intelligent to add)

looks useful. Maybe it can have a differet syntax though like:
{{name|ucase}}

So you could eventually pipe results.

I'd love to have this, but it seems to violate mustache's "no logic in templates" principle. Helpers are explicitly called out in the mustache readme ("Instead of views consisting of ERB or HAML with random helpers...").
Still, I think simple helpers would really help streamline some of my views' logic.
+1!

I agree with Benoît -- it would be nice to have a better-defined syntax.  How about something like this?
{{~helper varname}}

That would fit conceptually with blocks {{#varname}} and partials {{>partial name}}

@benoitc not a bad idea at all. See my latest commit :)
@pegli I don't like to introduce more magic symbols, so I went with @benoitc's suggestion.
@MartinodF the idea is directly stolen from handlebars :) c.f. http://writing.jan.io/mustache-2.0.html

You asked for a code review, so I did one :)
I generally really like this addition.  Great stuff!

@bcherry awesome review, thanks :)

-1 I'd rather not add this patch. It would be much cleaner to keep all logic in your view code, where you can document it properly and properly handle any errors that occur.

@mjijackson a -1 is a bit strong here for a feature you don't have to use. Why block a feature that others find useful but that doesn’t interfere with your work?

I think we can start an extras branch and implement these things there. What do you think?

Yes, let's do that! Which version you think we should fork? (as the latest seems to have been a considerable refactor)

yeah, base your work off master, awesome! :)


refact: namespace all labels in the helm charts

Describe what this PR does

Removes heritage and release label
Renames:

app > app.kubernetes.io/name
chart > helm.sh/chart:
component > app.kubernetes.io/component

Adds:

app.kubernetes.io/managed-by label
app.kubernetes.io/managed-by

This results in the following labels:
    labels:
      app.kubernetes.io/component: provisioner
      app.kubernetes.io/instance: ceph-csi-rbd
      app.kubernetes.io/managed-by: Helm
      app.kubernetes.io/name: ceph-csi-rbd
      app.kubernetes.io/version: canary
      helm.sh/chart: ceph-csi-rbd-1.3.0-canary

Is there anything that requires special attention
The change isn't backwards compatible, helm upgrade will fail since labels are immutable after an deployment:
# helm upgrade ceph-csi-rbd charts/ceph-csi-rbd --namespace storing
Error: UPGRADE FAILED: cannot patch "ceph-csi-rbd-nodeplugin" with kind DaemonSet: DaemonSet.apps "ceph-csi-rbd-nodeplugin" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app.kubernetes.io/component":"nodeplugin", "app.kubernetes.io/instance":"ceph-csi-rbd", "app.kubernetes.io/managed-by":"Helm", "app.kubernetes.io/name":"ceph-csi-rbd", "app.kubernetes.io/version":"canary"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable && cannot patch "ceph-csi-rbd-provisioner" with kind Deployment: Deployment.apps "ceph-csi-rbd-provisioner" is invalid: spec.selector: Invalid value: v1.LabelSelector{MatchLabels:map[string]string{"app.kubernetes.io/component":"provisioner", "app.kubernetes.io/instance":"ceph-csi-rbd", "app.kubernetes.io/managed-by":"Helm", "app.kubernetes.io/name":"ceph-csi-rbd", "app.kubernetes.io/version":"canary"}, MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable

Can be resolved by uninstalling the current release and redeploying with the same values but this should be mentioned and this must be a major Helm chart release.
Related issues
Fixes: #612

if you are using cephfs fuse client or rbd-nbd driver, This could be an issue as we need to delete the daemonset of both cephfs and rbd (all mounts inside the daemonset container will be gone  you will see transport endpoint not connected issues) https://github.com/ceph/ceph-csi/issues/703
we need to document the step on how to do upgrade for the same

@Madhu-1 Are the upgrade scenarios described in #770?

@wilmardo we need to add a document for the upgrade with helm charts

@wilmardo can you rebase this PR ?

@humblec Will do in the next couple days :)

@wilmardo I hope we are progressing on this :) .

@Madhu-1 Thanks for the reminder, working on it as we speak. Rebased to current master but found some missed values when reviewing myself, on it as we speak!

@wilmardo Thanks ! :) , iic the last comment was for me .

@wilmardo may be little more efforts to make CI happy ? :)

@humblec Done, didn't notice that the CI failed :+1:

@humblec Done, didn't notice that the CI failed +1

Cool. thanks . The PR looks good to me ! LGTM.

@Madhu-1 ptal .

This is a breaking change, we need upgrade doc for this one and we cannot track this for 2.1.0

this is a breaking change we need to see what we can do on this one. @nixpanic any suggestion from your side on this one?

@wilmardo could you write down your recommendation for upgrading a current deployment to the new label/format?

@wilmardo ping .. revisit?

Yeah busy with work and not that much time for open source at the moment.
Don't know if there is an casual upgrade path for this, Kubernetes sees these labels as immutable so it will error on trying to upgrade.
I don't know if it is feasible to remove and reinstall for user?

Yeah busy with work and not that much time for open source at the moment.
Don't know if there is an casual upgrade path for this, Kubernetes sees these labels as immutable so it will error on trying to upgrade.
I don't know if it is feasible to remove and reinstall for user?

Thanks for the reply @wilmardo even I have the same concern as it's not feasible to reinstall the whole cluster

Might be something for the 3.x.x release then?
I deleted and redeployed my ceph pods without problems in the past, since the PV and PVC keep existing for the current pods I had no downtime.
Needs testing though, it has been sometime since I have done that so my memory might be incorrect.

Might be something for the 3.x.x release then?
I deleted and redeployed my ceph pods without problems in the past, since the PV and PVC keep existing for the current pods I had no downtime.
Needs testing though, it has been sometime since I have done that so my memory might be incorrect.

looks like we can do it. but it will be a  downtime for the user. application pods cannot use PVC during the upgrade as we are deleting and recreating all the cephcsi pods. if we can add documentation for the upgrade we can do it.
as you said we need good testing for it for  both cephfs and rbd.

moved this out of release-v3.0.0 milestone

Might be something for the 3.x.x release then?
I deleted and redeployed my ceph pods without problems in the past, since the PV and PVC keep existing for the current pods I had no downtime.
Needs testing though, it has been sometime since I have done that so my memory might be incorrect.

looks like we can do it. but it will be a downtime for the user. application pods cannot use PVC during the upgrade as we are deleting and recreating all the cephcsi pods. if we can add documentation for the upgrade we can do it.
as you said we need good testing for it for both cephfs and rbd.

@wilmardo any comment on this one?


Update badges on Readme

This will be fixed once develop is merged into master.

Could this repo move away from the current git flow? That way features and bugfixes make their way in more regularly.

Once 6.0 is out they can be made more regular. I've been away from home so haven't been able to do much work. There are a couple issues (#248, #168) we haven't heard from in a while.
@WolfspiritM was going to test out 6.0 as well to see if #168 is still relevant but haven't heard back from them.


add Misaka Mikoto Tsun Lines

From To Aru Majutsu no Index S1 Last EP and To Aru Majutsu no Index S3 EP10

From To Aru Majutsu no Index S1 Last EP and To Aru Majutsu no Index S3 EP10
Btw nice trivia


If renewals fails with 404 we need to register again

Hi,
If the lease of the eureka-client somehow runs out, without the heartbeat reaching the Eureka server, we need to register the application again.
This is handled by the Eureka server by responding to the renewal request with the status code 404.
See the Netflix implementation to : https://github.com/Netflix/eureka/blob/master/eureka-client/src/main/java/com/netflix/discovery/DiscoveryClient.java#L825
Best,
/Frederik Mogensen

Good callout. This should be fairly easy to implement. We can just make renew() call register() on a 404. One thing we might want to do is move the startHeartbeats() and startRegistryFetches() calls out of register() and into start() so that we don't have double the intervals when this happens. Other than that, should be a small code change + adding a unit test.

Fixed in #50. Available in v2.4.0. Thanks again to @mogensen and @xanderv87.


add fitecvirtual.edu.co

@jferdi24 Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

@jferdi24 You might miss my previous comment. Please react.

@jferdi24 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for the understanding!


Arcata/Eureka, CA Police - Justice for Josiah Lawson

Hello, I'm from northern california and am very thankful for the template you helped me with for eureka, ca.  I saw the template which was made for Breonna Taylor and was hoping i could ask for assistance in creating one for Josiah Lawson who was murdered in 2017 by a white supremicist but his killer was never charged and the police botched the investigation.  There have been many documentaries and stories made demanding justice for his unsolved murder. His mother, Charmaine Lawson, is still fighting for the justice of her sons death and working very hard. She made a new donation page https://www.gofundme.com/f/justice-for-josiah-litigation-assistance?utm_medium=copy_link&utm_source=customer&utm_campaign=p_lico+share-sheet to start collecting for the discovery phase of the litigation process.
I have trouble bringing all the necessary components together at once but i want to do everything i can do give awareness to this case so she may get the justice and peace she deserves.
https://www.justiceforjosiahlawson.com/
https://www.youtube.com/watch?v=v6uzA83_y5c
if you need any more information to construct the template please let me know

Hey, we are not assisting in creating templates, only bringing ones in from the community. I'm going to close out this issue, because we are only accepting issues that contain all of the valid information:

A subject message for the email
A list of emails (at least one) for the government officials you’d like to receive the message
A detailed message concerning the budgeting decisions that are at stake in your community

Please read more about our email style guidelines here.
If this information is copied from an external source please provide a link to the original source.
See our current links for examples at https://defund12.org.
Thank you!!


Fix Config_heavyl.pl on mac

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)

Follow up to #33 and #34 to fix errors on osx (found in https://github.com/bioconda/bioconda-recipes/pull/13708 and https://github.com/bioconda/bioconda-recipes/pull/14149)

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

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Green! Sorry to bug you again @bgruening, but if you have a moment for this :) Maybe it's the last pr for this (:crossed_fingers:)


Example code error

I've been searching for a straightforward library that can generate bounding boxes around text using SWT, so I'm really excited to find this. It looks great! I just noticed a couple little issues in the readme that I thought I'd mention.
In your example code, acceptCC_aspectratio = 5 throws an error because it's expecting a float. Changing it to 5. works. It might be worth adding import numpy as np to the example also, since it's required.

Thanks for pointing it out @aazuspan, it has been updated.

Thanks for pointing it out @aazuspan, it has been updated.

Closing this issue!

Closing this issue!


onPurchasedProductsFetched onClick

Hello,
Is there any way I can use onPurchasedProductsFetched on a button click, such as "Restore purchases"? Or do I have to launch the entire BIllingConnector on the activity I want to restore purchases to?

Hi,
You'll need to launch the entire BillingConnector on the activity.


Automated Change Deployment workstream engagement model

Automated Change Deployment workstream engagement model
Our community meetings will follow a Lean Coffee pattern.
Because of potential tool restrictions across our community of organizations, we will use Github for all collaboration.
Over time, we will provide updates detailing community activity at each meeting of the FINOS SIG (see here for meeting announcements).
Meeting Announcements
Announcements for workstream meetings will be made as issues on the GitHub issue tracker.
We will follow the same conventions used for the FINOS DevOps Automation SIG meetings.
Meeting Cadence
To start, we will hold meetings every two weeks, starting the week of Feb 27, 2023.
Based on community input, we will seek to evolve the meeting cadence to maximize participation within the group as well as with other FINOS workstreams.
Meeting Notes
Meeting notes will be added as comments to the meeting announcement.
Meeting Agenda
We believe a regular cadence of real-time in-depth discussions amongst tech leaders will result in acceleration of constructive ideas and actions that advance our clients and the industry.
We will know we have succeeded when attendees contribute to deliverables and provide feedback to improve working group meetings.
Why
Provide an opportunity for generative discussion and ideas.
Leverage the hive mind of tech leaders.
Foster comradely through technical curiosity and geekdom.
Create artifacts and resources that organizations can leverage to accelerate their implementation and adoption of Backstage.
Who
Open to all practitioners.
How
Size: Unlimited, but will breakout into smaller groups to promote participation.
Format: Using Lean Coffee (link1, link2), we pitch and vote on which topics the group wants to discuss and time-box the discussion.
Intent: Prefix topics with your intention so the group is clear on the proposer's outcome.
What
Examples of intent (and to prefix your proposed topics):
[inform] You are simply seeking to inform the group of this item. You may field clarifying questions from the group on your inform, but not seeking further discussion at this time.
[ideation] You are seeking divergent and wide perspectives from this group. In this brainstorming mode, all ideas are accepted, without critical analysis. Clarify whether you’d like to ideate on the problem space or the solution space.
[analysis] You are asking the group to help you poke holes in your idea, plan, etc.
[quest] You are seeking information and responses to a question you have.
Meeting guidelines
We will start at :05 after, as attendees join.
enjoy the music
join the Lean Coffee meeting issue here
read today's proposed topics in the project board
add your own proposed topics to the list
Agenda
2mn - intro
10mn - pitch topics
35mn - voted topic discussion
5mn - silent readouts, comments / questions via comments in issue
Small group facilitation
Depending on attendance, we may split out into subgroups to ensure all voices are heard.
We will agree on the room's topic so that attendees may choose according to their interest.
Someone within the group will be responsible for creating an issue, and taking (and posting) notes from the discussion.
If the topic is completed before time is up, the group may vote on another topic to discuss.
Artifacts
Any documentation artifacts generated by the group will be organized in the ./docs/autochangemgmt/ folder of this repository.
Contributions are welcome.
See the ./README.md for licensing and contribution requirements.

#69


Is there a recursive feature?

Is there a feature to crawl a website looking for broken links? If not, could this be mentioned in the documentation?

there is not currently, but that is the subject of the next version

same problem but it works with {}

This is amazing! Thank you so much! :heart_eyes:


Adding the merices table

Adding the merices table to the readme file
@Trisha-tech , please assign me under this issue !!!

@Aman-G-upta not really needed, please create new issue if you have any other idea in mind


Merge from SBX to NPD - Practica_14

Merge from SBX to NPD - Practica_14

Merge from SBX to NPD - Practica_14


Enhancement: Allow for uploading of emails

Title.
Will be as email archives. Need to figure out details.

https://github.com/demisto/parse-emails


Need support for android map utils library

Please provide features of following library -
https://github.com/googlemaps/android-maps-utils

any update on this? @zoechi

there is this package https://pub.dev/packages/maps_toolkit
but it is incomplete, im creating a new and im going to provide it hear, as it going to be near to close this issue here soon

https://pub.dev/packages/google_maps_utils
I made these package with official converted code from OP link
needing more features, request I'll do it the quickest possible

can you add routeboxer?

Closing as dup of https://github.com/flutter/flutter/issues/50636


Create SafeString to bypass Sanitization for formatArgs

It might be nice to allow for bypassing the formatArg sanitization process for particular arguments, especially if you want a format arg to be a SRML tag.  I propose creating an extension of String called SafeString, and altering the sanitizer to skip over arguments of that type.

String is final, so we'll have to create a totally new class for this.


change order of speaker questions. move availability to speaker section

Hey I'm pretty new to eventyay. Just installed eventyay on my local machine but my interface is different from yours. Can you explain why?

Merging for testing on dev deployment.


Update README.rst

Add direct link to PyPi and change badges to be all from the same source (shields.io) for consistency.

Codecov Report

Merging #329 into master will decrease coverage by 89.68%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #329       +/-   ##
==========================================
- Coverage   89.68%   0.00%   -89.69%
==========================================
  Files           5       5
  Lines        1057    1057
==========================================
- Hits          948       0      -948
- Misses        109    1057      +948

Impacted Files
Coverage Δ

toml/ordered.py
0.00% <0.00%> (-100.00%)
:arrow_down:

toml/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

toml/decoder.py
0.00% <0.00%> (-89.75%)
:arrow_down:

toml/encoder.py
0.00% <0.00%> (-88.33%)
:arrow_down:

toml/tz.py
0.00% <0.00%> (-86.67%)
:arrow_down:

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 9be6458...2984788. Read the comment docs.

Thanks! Merged.


ModuleNotFoundError: No module named 'torch'

I've been able to go through the whole process without issues, but can't launch the script. I've checked and pytorch with other dependencies were installed without issues, but for some reason it can't be accessed it seems. Do you maybe know what I'm doing wrong? Thank you <3
(ml) phylyp@Phylyps-MacBook-Pro sd-gui % python gui.py
Traceback (most recent call last):
  File "/Users/phylyp/sd-gui/sd-gui/gui.py", line 2, in <module>
    import torch
ModuleNotFoundError: No module named 'torch'
(ml) phylyp@Phylyps-MacBook-Pro sd-gui %

Closing this since I already responded on Reddit 😄


Include cstdint in gz

On some setups, I get a bunch of errors that look like
In file included from /home/adamant/sshash-jermp/include/gz/zip_stream.cpp:1:
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:121:5: error: ‘uint32_t’ does not name a type
  121 |     uint32_t get_crc() const;
      |     ^~~~~~~~
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:67:1: note: ‘uint32_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
   66 | #include <vector>
  +++ |+#include <cstdint>
   67 |
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:124:5: error: ‘uint32_t’ does not name a type
  124 |     uint32_t get_in_size() const;
      |     ^~~~~~~~
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:124:5: note: ‘uint32_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:137:5: error: ‘uint32_t’ does not name a type
  137 |     uint32_t crc_;
      |     ^~~~~~~~
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:137:5: note: ‘uint32_t’ is defined in header ‘<cstdint>’; did you forget to ‘#include <cstdint>’?
/home/adamant/sshash-jermp/include/gz/zip_stream.hpp:180:5: error: ‘uint32_t’ does not name a type
  180 |     uint32_t get_crc() const;
      |     ^~~~~~~~

This include fixes it.

Thanks!


"Move while emotes" can't be used

https://github.com/user-attachments/assets/78df13e9-35e6-4e44-9faf-d81f4af36f53

Strange. It's updating the tips on the top right, but it's not doing what it's supposed to. Could you share with me a profile code for you modpack? I wonder if it could be from a mod compat issue. This was working fine for me the last time I tested it.

Strange. It's updating the tips on the top right, but it's not doing what it's supposed to. Could you share with me a profile code for you modpack? I wonder if it could be from a mod compat issue. This was working fine for me the last time I tested it.

0190feb4-5b3a-dae8-2837-9398f302f0df
This is my modpack code
Explain more, I used to play with my friends before and they worked fine
It seems to be a problem with my device only

By the way, what key do you use to open your emote menu?

It's working fine for me with this modpack.
By any chance is your alt key being held down while you emote? That may be what's causing your character to freeze in place.
Other than that, if it happens to you again, would you be able to send your log file with me so I can check it out?

I know the problem. It seems that if I connect the controller, it will make this function not work.
Thank you for taking the time to help.


execute multiple middlewares one by one?

Using multiple @UseBefore won't work. Only the last one will be executed. Is there a workaround way to execute middlewares one by one in @UseBefore?  I want to check the user's session before uploading image via multer-s3.
    @Post("/upload_image")
    @JsonResponse()
    @UseBefore(CheckUserSession)
    @UseBefore(MulterS3)
    async uploadImage(@Req()req:any) {

    }

My CheckUserSession is something like this:
@Middleware()
export class CheckUserSession {
    public use(req: express.Request, _res: express.Response, next?: express.NextFunction) {
        if (!req.session || !req.session.user_id) {
            res.direct('/login/fail')
        }
        if (next) {
            next();
        }
    }
}

multers3
export const MulterS3 = multer({
  storage: multerS3({
    s3: s3,
    bucket: 'some-bucket',
    metadata: function (req, file, cb) {
      cb(null, {fieldName: file.fieldname});
    },
    key: function (req, file, cb) {
      cb(null, Date.now().toString())
    }
  })
})

export function UseBefore(...middlewares: Array<Function>): Function;

So:
@Post("/upload_image")
@JsonResponse()
@UseBefore(CheckUserSession, MulterS3)
async uploadImage(@Req()req:any) {

}

@stonecold123 please use what @19majkel94 suggest. This solution should work. @19majkel94 but first approach also should work, if its not, then probably there is a bug.

Thank you! It works.

@pleerock
It is probably a bug. My multiple @UseBefore() approach doesn't work.


Build fails on Node v8.7 (node-gyp)

$ sudo npm i -g slap@latest
gyp WARN EACCES user "root" does not have permission to access the dev dir "/usr/lib/node_modules/slap/node_modules/marker-index/.node-gyp/8.7.0"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/usr/lib/node_modules/slap/node_modules/marker-index/.node-gyp"

Looks like a problem with SELinux. Execute setenforce 0 command and reinstall slap

Same here, node 9.9.0
gyp ERR! configure error  gyp ERR! stack Error: EACCES: permission denied, mkdir '/usr/lib/node_modules/slap/node_modules/marker-index/build' gyp ERR! System Linux 4.13.0-37-generic gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild" gyp ERR! cwd /usr/lib/node_modules/slap/node_modules/marker-index gyp ERR! node -v v9.9.0 gyp ERR! node-gyp -v v3.6.2 gyp ERR! not ok  npm ERR! code ELIFECYCLE npm ERR! errno 1 npm ERR<EMAIL_ADDRESS>install: node-gyp rebuildnpm ERR! Exit status 1 npm ERR!  npm ERR! Failed at the<EMAIL_ADDRESS>install script. npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

this repo is not maintained anymore.

See this issue, I have answered it, #414

See this issue, I have answered it, #414


Trace-X Release 23.12 Compliance Acceptance Criteria

Release Compliance 23.12
Source in Catena-X Confluence + Expert Contacts here. (Source only accessible for Catena-X Consortia members in current transition phase)

[x] Gaia-X compliance confirmed by dedicated expert.

Obtain approval from CX Gaia-X expert(s), prior to Gate review.
Provide info as early as possible; expect optimization loops based on feedback
=> check linked PDF summary for requirements (Source only accessible for Catena-X Consortia members in current transition phase).

Contact:

Release Mangement

https://jira.catena-x.net/browse/TRACEFOSS-2766

[x] GDPR compliance confirmed
(personal data; Data Protection & Privacy DPP)

assessment must be completed via Catena-X GDPR questionnaire (Source only accessible for Catena-X Consortia members in current transition phase.)
either the product/service doesn`t process any relevant data,
or - in case the assessment reveals potential issues - a more severe "Data Privacy Impact Assessment" shall be conducted (if applicable: robust mitigation actions must be completed)
Provide evidence for GDPR compliance prior to Gate review. Previous assessments remain valid, as long as approach to handling of personal data was not changed.

Contact:

Release Mangement

https://jira.catena-x.net/browse/TRACEFOSS-2767

[ ] Interoperability Check
Acceptance criteria are fulfilled as described in in the following area on confluence: Interoperability Quality Gate Requirements (Source only accessible for Catena-X Consortia members in current transition phase.)

Approval will be documented centrally by Expert(s)

Obtain approval from Interoperability experts prior to Gate review.

Therefore consult the regular office hours as early as possible; expect optimization loops based on feedback. (Source only accessible for Catena-X Consortia members in current transition phase.)
Contact:

SYSTEM ARCHITECT2

https://jira.catena-x.net/browse/TRACEFOSS-2768

[ ] Data Sovereignty Check
Acceptance criteria are fulfilled as described in in the following area on confluence:

Data Sovereignty Guardrails for Release 3.2 (Source only accessible for Catena-X Consortia members in current transition phase.)
Approval will be documented centrally by Expert(s)
Obtain approval from Data Sovereignty experts prior to Gate review: Experts will schedule a review with your team.

Expect optimization loops based on feedback.
Contact:

SYSTEM ARCHITECT3,
SYSTEM ARCHITECT4

https://jira.catena-x.net/browse/TRACEFOSS-2769

[x] Confirmation of published CX Standards
Product Owner confirms:

The FOSS product under review is compliant to all relevant Catena-X Standards which are already published on the Association website.
Failure to do so may ultimately make the reference implementation useless, as the mandatory CX Certification cannot be accomplished.

[x] Verification of foreseen CX Standards
Product Owner confirms:

The content of upcoming CX Standardization Requests (STAN) has been reviewed and feedback was provided to the responsible PCA/BDA. As per current status, the FOSS product under review will be compliant as soon as the CX Standard is published. (Info here) (Source only accessible for Catena-X Consortia members in current transition phase.)
Minimum: PCA/BDA have finalized all release-relevant Standardization Requests (STAN). Those have passed the CX Association`s technical committee (TC4S) and are pre-published on the website. Opt-out period in progress.
Best case: Association Board approval available and CX Standard is published on website.

@jzbmw & @SteDue
Confirmation of published CX Standards
Trace-X as a business app there are no standards foreseen. Please confirm.

@jzbmw & @SteDue Verification of foreseen CX Standards Trace-X as a business app there are no standards foreseen. Please confirm.

herby I confirm that no standard needs to be forseen.

@jzbmw & @SteDue Confirmation of published CX Standards Trace-X as a business app there are no standards foreseen. Please confirm.

herby I confirm that no standard needs be published for trace-X

https://confluence.catena-x.net/display/BDPQ/(TRF)+-+GDPR+Compliance+23.12

Felix Gerbig schrieb
Hallo Martin,
Ich denke wir können uns das Meeting sparen – an den Anforderungen zur Gaia-X Compliance hat sich bisher nichts geändert und ihr müsst als Anwendung nichts tun.
Viele Grüße
Felix

@kelaja
Zuständigkeit QGates noch in Klärung Releasemanagement- (Eskalation ist erfolgt)

Data Sovereignty Check was approved in last release R3.2.

No significant changes since release 23.9 regarding data Data Sovereignty.


config: support lable ok-to-test on test-prod

support lable ok-to-test on test-prod.

/lgtm
/approve


Feature Proposal: Custom Sources

As an Airy developer trying to connect source types that are not yet supported by Airy Core,
I want to be able to create custom sources (and effectively channels of the relevant custom source type),
so that I can ingest conversational data in my Airy cluster
via a webhook from a third party system that produces conversational data
Requirements

there needs to be a way to create, update and delete/deactivate the custom source
the custom source should have a distinguishable name, so that we can create multiple custom sources and differentiate between them
there needs to be a way to define the data model for events to be received from such custom source (e.g. message payloads) and a map so we can extract metadata from it
there needs to be a way to define how to send requests back to the custom source, e.g. how to send messages and authorize the relevant API requests (unless the integration was read only for such relevant custom source e.g. to extract conversational analytics, but no responses were required from Airy)
there needs to be a way to connect a channel to an Airy cluster of the source type of the relevant custom source including defining a unique identifier in order to map incoming events accordingly, e.g. channel_id, members.user_id, team_id, app_id, ALL_MESSAGES_FOR_SOURCE

splitting this ticket in a general request / feature proposal for how to do sources and the suggested getstream.io source

This is covered by this milestone and already possible using the Source API. Therefore I am closing this ticket.


fix: Change name and description, add scope for config

Looks good, but the build fails. When it's fixed, I will approve.


Making injection of clearfix element during iframe content initialization optional

Hey, thank you for putting this handy library together! I'm just trying it out for a project for the first time and works pretty smoothly.
One thing I noticed though was that - for some of the pages in my application - the injection of the clearfix element during the initialization of the iframe content window will lead to most of my iframe content being hidden behind this specific clearfix element.
Is there currently a way to opt out of this injection? If not, would it make sense to provide an option flag that defaults to true to make this a non-breaking change for current users but still allow other users to opt out of the automatic clearfix element injection?
If this seems feasible, I'm happy to send a PR. Looking forward to your feedback!

Not heard of anyone having that issue before, does using a different height
calculation method help?
On Wed, Jul 11, 2018 at 5:35 PM, Jessica Jordan<EMAIL_ADDRESS>wrote:

Hey, thank you for putting this handy library together! I'm just trying it
out for a project for the first time and works pretty smoothly.
One thing I noticed though was that - for some of the pages in my
application - the injection of the clearfix element during the
initialization
https://github.com/davidjbradshaw/iframe-resizer/blob/d8434af956f9d54981e956a47969bb26e84089bc/js/iframeResizer.contentWindow.js#L171
of the iframe content window will lead to most of my iframe content being
hidden behind this specific clearfix element.
Is there currently a way to opt out of this injection? If not, would it
make sense to provide an option flag that defaults to true to make this a
non-breaking change for current users but still allow other users to opt
out of the automatic clearfix element injection?
If this seems feasible, I'm happy to send a PR. Looking forward to your
feedback!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/davidjbradshaw/iframe-resizer/issues/606, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAkHEq_KKYQSdOv0ZJXkfE5ORyL5JQl-ks5uFhtcgaJpZM4VLYBt
.

I want to emphasize that this is not an issue with the library itself, but rather with the integration of the library with other applications: As far as I can see it, iframe-resizer adds the clearfix div element pre-assuming that there is no particular styling attached to itself as an element in the context of the application that it is put into.
E.g. in the app I'm working on there's  a styling used (which cannot be refactored away easily either), which assumes a particular element inside of the app to follow right after the body tag. The styling sets the height of this particular element  to 100%:
body > * {
    height: 100%;
}

In this scenario the clearfix div that is added to the DOM by iframe-resizer suddenly becomes the direct descendant of the body element and therefore inherits this styling, hiding all the other content.
The injection of the clearfix element is something that might also change the layout of other applications unexpectedly, in which the DOM change is not anticipated. I think it would be cool to leave the clearfix element as a default so everyone can still reap the benefits of it. Yet, I think it would be useful for users to be able to opt-out of it, in case they have particular styles added to their apps which rely on a certain DOM layout to be present and stay in place.
In my case, I wouldn't know how to circumvent this issue except for checking that my application is loaded as an iframe and then appending a particular class to the body tag which will update the aforementioned default styling.
What are your thoughts on this?

I think this is actually described in the issue https://github.com/davidjbradshaw/iframe-resizer/issues/427 as well:

Another edge case would be if for some (evil) reason there was a global style setting a margin or min-height to all divs, then the clearFix one may end up creating an extra space at the bottom.

I just tried to reproduce this by adding your css to the example file and it does not create an issue

Anyway I will add height:0 to it's inline style to guard against this.