Omnisharp "no output returned"

Hi !
I try to to work with Omnisharp without success.  I've tested with a project generated thanks dotnet new web && dotnet new sln, in the Program.cs file. There is non omnicompletion, even if I type CreateWebHostBuilder(args).<C-x><C-o>. There is no lint info (it should be have lint warnings at least about unused import).
I've tested like https://github.com/OmniSharp/omnisharp-vim/issues/416 from @Ja-rek .
$ dotnet --version
2.1.403

I'm on ArchLinux.
I've tested with the following init.vim
        call plug#begin('~/.vim/plugged')
        Plug 'w0rp/ale'
        Plug 'OmniSharp/omnisharp-vim'
        call plug#end()
        filetype plugin indent on
        let g:OmniSharp_timeout = 5
	let g:OmniSharp_proc_debug = 1
        let g:ale_linters = { 'cs': ['OmniSharp'] }

Thanks 	let g:OmniSharp_proc_debug = 1 we have the following output:
stdout: ['^[[40m^[[32minfo^[[39m^[[22m^[[49m: OmniSharp.Http.Startup[0]', '      Omnisha...n port ''42139'' at location ''/home/julioprayer/DCISS/dotNet/aspnet'' on host -1.', '']

I've tried several times to do OmniSharpInstall
ALE and omnisharp-vim are updated (I've deleted its folders, then install it with git clone).
I'm on ArchLinux
Name            : libuv
Version         : 1.23.1-1

( I've also tested with version 1.23.2)
$ nvim -v
NVIM v0.3.1
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/bin/cc -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wconversion -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Ws
trict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=auto -Wno-array-bounds -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_
FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/build/neovim/src/build/config -I/build/neovim/src/neovim-0.3.1/src -I/usr/include -I/build/neovim/src/build/src/nvim/auto -I/build/neovim/src/build/include
Compiled by builduser

Features: +acl +iconv +jemalloc +tui
See ":help feature-compile"

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

Run :checkhealth for more info

health#nvim#check
========================================================================
## Configuration
  - OK: no issues found

## Performance
  - OK: Build type: Release

## Remote Plugins
  - OK: Up to date

## terminal
  - INFO: key_backspace (kbs) terminfo entry: key_backspace=\177
  - INFO: key_dc (kdch1) terminfo entry: key_dc=\E[3~
  - INFO: $COLORTERM='rxvt-xpm'

health#provider#check
========================================================================
## Clipboard (optional)
  - OK: Clipboard tool found: xsel

## Python 2 provider (optional)
  - INFO: `g:python_host_prog` is not set.  Searching for python2 in the environment.
  - INFO: Multiple python2 executables found.  Set `g:python_host_prog` to avoid surprises.
  - INFO: Executable: /usr/bin/python2
  - INFO: Other python executable: /bin/python2
  - INFO: Python2 version: 2.7.15
  - INFO: python2-neovim version: 0.2.6
  - OK: Latest python2-neovim is installed: 0.2.6

## Python 3 provider (optional)
  - INFO: `g:python3_host_prog` is not set.  Searching for python3 in the environment.
  - INFO: Multiple python3 executables found.  Set `g:python3_host_prog` to avoid surprises.
  - INFO: Executable: /usr/bin/python3
  - INFO: Other python executable: /bin/python3
  - INFO: Python3 version: 3.7.1
  - INFO: python3-neovim version: 0.2.6
  - OK: Latest python3-neovim is installed: 0.2.6

 Current Filetype: cs
Available Linters: ['mcs', 'mcsc', 'omnisharp']
   Linter Aliases:
'omnisharp' -> ['Omnisharp', 'OmniSharp']
  Enabled Linters: ['omnisharp']
 Suggested Fixers:
  'remove_trailing_lines' - Remove all blank lines at the end of a file.
  'trim_whitespace' - Remove all trailing whitespace characters at the end of every line.
  'uncrustify' - Fix C, C++, C#, ObjectiveC, ObjectiveC++, D, Java, Pawn, and VALA files with uncrustify.
 Linter Variables:
 Global Variables:
let g:ale_cache_executable_check_failures = v:null
let g:ale_change_sign_column_color = 0
let g:ale_command_wrapper = ''
let g:ale_completion_delay = v:null
let g:ale_completion_enabled = 0
let g:ale_completion_max_suggestions = v:null
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%code: %%s'
let g:ale_echo_msg_info_str = 'Info'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_history_enabled = 1
let g:ale_history_log_output = 1
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_filetype_changed = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_lint_on_insert_leave = 0
let g:ale_linter_aliases = {}
let g:ale_linters = {'cs': ['OmniSharp']}
let g:ale_linters_explicit = 0
let g:ale_list_window_size = 10
let g:ale_list_vertical = 0
let g:ale_loclist_msg_format = '%code: %%s'
let g:ale_max_buffer_history_size = 20
let g:ale_max_signs = -1
let g:ale_maximum_file_size = v:null
let g:ale_open_list = 0
let g:ale_pattern_options = v:null
let g:ale_pattern_options_enabled = v:null
let g:ale_set_balloons = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 0
let g:ale_sign_error = '>>'
let g:ale_sign_info = '--'
let g:ale_sign_offset = 1000000
let g:ale_sign_style_error = '>>'
let g:ale_sign_style_warning = '--'
let g:ale_sign_warning = '--'
let g:ale_statusline_format = v:null
let g:ale_type_map = {}
let g:ale_use_global_executables = v:null
let g:ale_warn_about_trailing_blank_lines = 1
let g:ale_warn_about_trailing_whitespace = 1
  Command History:
(executable check - success) python
(finished - exit code 0) ['/bin/zsh', '-c', '''python'' ''/home/julioprayer/.vim/plugged/omnisharp-vim/python/ale_lint.py'' --filename ''/home/julioprayer/DCISS/dotNet/aspnet/Program.cs'' --h
ost ''http://localhost:42139'' --level ''warning'' --cwd ''/home/julioprayer/DCISS/dotNet/aspnet'' --delimiter ''@@@'' < ''/tmp/nvimDSJ6Ke/1/Program.cs''']
<<<NO OUTPUT RETURNED>>>
(finished - exit code 0) ['/bin/zsh', '-c', '''python'' ''/home/julioprayer/.vim/plugged/omnisharp-vim/python/ale_lint.py'' --filename ''/home/julioprayer/DCISS/dotNet/aspnet/Program.cs'' --h
ost ''http://localhost:42139'' --level ''warning'' --cwd ''/home/julioprayer/DCISS/dotNet/aspnet'' --delimiter ''@@@'' < ''/tmp/nvimDSJ6Ke/2/Program.cs''']
<<<NO OUTPUT RETURNED>>>

Thanks a lot in advance for your answer :-)

Did you add your project to your solution?
dotnet sln add [filename].csproj

@nickspoons
Thanks a lot !
I've forgotten to do that.
After this problem, I has error OmniSharp.MSBuild.ProjectManager Failed to load project file. I've resolved this problem thanks this comment https://github.com/OmniSharp/omnisharp-vscode/issues/2604#issuecomment-433167067 . On ArchLinux, It's done by sudo pacman -Rsn mono, and all work well. See also https://github.com/OmniSharp/omnisharp-vscode/issues/2234 .
For .Net Core projects and Linux users, maybe it could be interesting to add advise to uninstall the Mono package installed by the distro ?
Thanks a lot again, and thanks for this project ! Now I could start to discover .Net Core with my NeoVim ;-).

Today, I've experienced the same problem with $ dotnet --version : 2.1.500.
But all works well with $ dotnet --version : 2.1.500, no problems with 2.1.403, only with 2.1.500. Is it normal ?
I've tested on a complete new project, with all steps reproduced ($ dotnet new web && dotnet new sln && dotnet sln add Aspnet.csproj), and with the vimrc of the first comment.                                                             .
Thanks command :message in Neovim we have the output:
stdout: OmniSharp.Http.Startup[0]', '      Starting OmniSharp on arch 0.0 (x64)', '']
stdout: OmniSharp.Cake.CakeProjectSystem[0]', '      Detecting Cake files in ''/tmp/Aspnet''.', '']
stdout: OmniSharp.Cake.CakeProjectSystem[0]', '      Could not find any Cake files', '']
stdout: OmniSharp.Http.Startup[0]', '      Project system ''OmniSharp.DotNet.DotNetProjectSystem'' is disabled in the con
figurat
stdout: OmniSharp.MSBuild.ProjectSystem[0]', '      Detecting projects in ''/tmp/Aspnet/Aspnet.sln''.', '']
stdout: OmniSharp.MSBuild.ProjectManager[0]', '      Queue project update for ''/tmp/Aspnet/Aspnet.csproj''', '']
stdout: OmniSharp.Script.ScriptProjectSystem[0]', '      Detecting CSX files in ''/tmp/Aspnet''.', '']
stdout: OmniSharp.Script.ScriptProjectSystem[0]', '      Could not find any CSX files', '']
stdout: OmniSharp.Http.Startup[0]', '      Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpWo
rkspace
stdout: OmniSharp.Http.Startup[0]', '      Configuration finished.', '']
stdout: OmniSharp.Http.Startup[0]', '      Omnisharp server running on port ''43887'' at location ''/tmp/Aspnet'' on host
 -1.',
stdout: OmniSharp.MSBuild.ProjectManager[0]', '      Loading project: /tmp/Aspnet/Aspnet.csproj', '']
stdout:[49m: OmniSharp.MSBuild.ProjectManager[0]', '      Failed to load project file ''/tmp/Aspnet/Aspnet.csproj''.', ''
]
stdout: OmniSharp.MSBuild.ProjectManager[0]', '      Attemped to update project that is not loaded: /tmp/Aspnet/Aspnet.cs
proj', '']


Codeunit 12104 - LocalApplicationManagement: change scope to procedure CheckDigit

Dynamics 365 Business Central IT Localization.
To validate the "Fiscal Code"field, the Base App does the following:
trigger OnValidate()
var
LocalAppMgt: Codeunit LocalApplicationManagement;
begin
if "Fiscal Code" <> '' then
LocalAppMgt.CheckDigit("Fiscal Code");
end;
But the CheckDigit procedure declared in the LocalApplicationManagement codeunit has the OnPrem scope:

and we cannot use the validation in SaaS extension. Can you please change the scope to be available also for SaaS?
Thanks.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Angular material table - sort direction not reading in screen reader

🐞 bug report
Affected Package

 The issue is caused by package @angular/....
@angular-material
Is this a regression?

 Yes, the previous version in which this bug was not present was: ....
Yes it used to work in angular V5.
Description
 A clear and concise description of the problem...
Angular material table - sort direction not reading in screen reader.
MatSortHeaderIntl.sortDescriptionLabel is available in Angular V5 and its not available from V6. sortDescriptionLabel has an option to read the direction of the sort order(asc or dsc). Could someone please help fixing this issue or suggest a work around. Thanks in advance.
🔬 Minimal Reproduction

 https://stackblitz.com/...

🔥 Exception or Error

🌍  Your Environment
Angular Version:

Angular V9

Anything else relevant?

@chakravarthypasam which screen reader (and version)? Which browser (and version)? How did you interact with the sort-header: through screen-reader navigation or tab navigation?

I'm using JAWS screen reader Version 2020.2012.13 ILM. The browser is IE 11 version 11.1139.18362.0. I used tab navigation to  interact with the header. When I tab over to the header JAWS announces the "column name" followed by "To change sorting hit Enter". When I click on enter it Just announces "Enter" but not the sorting order something like "Sorted Ascending or Descending".

Closing this issue because Angular is dropping support for IE11 in the upcoming version 13. We don't plan to fix any further IE11 bugs going forward.


How i can add scrollable for mouse in Tabs component?

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
How i can add scrollable for mouse in Tabs component?
export function ScrollableTabsButtonAuto() {
const [value, setValue] = React.useState(0);
const handleChange = (_event: React.SyntheticEvent, newValue: number) => {
setValue(newValue);
};
return (
<Box sx={{ bgcolor: "background.paper" }}>

);
}
Examples 🌈
export function ScrollableTabsButtonAuto() {
const [value, setValue] = React.useState(0);
const handleChange = (_event: React.SyntheticEvent, newValue: number) => {
setValue(newValue);
};
return (
<Box sx={{ bgcolor: "background.paper" }}>

);
}
Motivation 🔦
No response

Hello @ethever, can you please explain what are you trying to achieve? Do you want to change the tabs component onScroll?

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Non-reference Names or Labels

The name module is very reference heavy. Any chance you could add a not-reference version of name/label, so that they can be stored without introducing lifetime parameters everywhere? I'm doing sub-domain checks against a fixed set of domains, and right now I have to re-parse domains into Name<'_> to do this.
Some simple options that I can think of:

Make the data field of Label public.
Add an unchecked From<[u8]> impl for Label.

Thanks, and I really appreciate all the work you've put into this crate.

Hi @ifazk
In the current implementation, it is possible to create a label from a Vec<u8> (or anything that implements Into<Cow<'a, [u8]>>. I don't if that solves your issue, since you still need to annotate the Name/Label with a lifetime.
I would prefer to not make the data field public, since that could result in unintentional misuse.
As for making owned versions of the Name/Label, I could try to implement it, but as of today, I don´t know a simple way to have both, borrowed and owned, implementations that would not lead to code duplication or a major rewrite of the whole crate.
Could you give a code example of how you want to use this? If I see a code example, I can have a better understanding of how to approach the issue

Thinking about it a bit more, what I'm looking for from the above suggestions is this:
impl<'a> Label<'a> {
    /// Returns the bytes of the label
    pub fn as_bytes(&self) -> &[u8] {
        self.data.as_ref()
    }
}

Like you said, I can use create a label from a Vec<u8>, but if all I already have a Label<'a>, I can't create a Vec<u8> from it, because the data field is private. Here's an example of how the above as_bytes() can be used.
pub mod example {
    use simple_dns::{Label, Name, SimpleDnsError};

    type StoredLabel = Vec<u8>;
    type StoredName = Vec<StoredLabel>;

    /// User configuration without any lifetime labels
    pub struct Config {
        blocked_domains: Vec<StoredName>,
    }

    impl Config {
        /// Create a Config from a list of blocked domains
        pub fn new(input_names: Vec<String>) -> Result<Config, SimpleDnsError> {
            let mut blocked = Vec::new();
            for input_name in input_names {
                let mut name = Vec::new();
                for label in Name::new(&input_name)?.iter() {
                    name.push(label.as_bytes().to_vec()); // Using as_bytes() here
                }
                blocked.push(name);
            }
            Ok(Config {
                blocked_domains: blocked,
            })
        }

        /// Checks if a name from a dns packet should be blocked
        pub fn is_blocked(&self, name: Name<'_>) -> bool {
            self.blocked_domains.iter().any(|blocked| {
                let blocked_labels = blocked
                    .iter()
                    .map(|v| Label::new_unchecked(v))
                    .collect::<Vec<_>>(); // first intermediate vector
                let blocked_name = Name::new_with_labels(&blocked_labels); // second intermediate vector
                name.is_subdomain_of(&blocked_name)
            })
        }
    }
}

As for having a simple way to have both borrowed and owned implementations without code duplication, this only works if there's a cheap way to convert the owned type into a borrowed type. For example, Vec<u8> is an owned type, and Label<'_>, is a borrowed type, and there's a cheap way to covert one to the other. You don't have to duplicate code that's for Label<'_> for Vec<u8> since a Vec<u8> can always be converted into a Label<'_>. Having something like that for Name would be nice, since I had to allocated two intermediate vectors to create a Name<'a> from a Vec<Vec<u8>>.

you could use the existing into_owned method to store the Name in the blocked_domains vec if you change the lifetime to 'static.
This example seems to compile
/// User configuration without any lifetime labels
    pub struct Config {
        blocked_domains: Vec<Name<'static>>,
    }

    impl Config {
        /// Create a Config from a list of blocked domains
        pub fn new(input_names: &[&str]) -> Result<Config, SimpleDnsError> {
            let mut blocked = Vec::new();
            for input_name in input_names {
                blocked.push(Name::new(input_name)?.into_owned());
            }
            Ok(Config {
                blocked_domains: blocked,
            })
        }

        /// Checks if a name from a dns packet should be blocked
        pub fn is_blocked(&self, name: Name<'_>) -> bool {
            self.blocked_domains
                .iter()
                .any(|blocked| name.is_subdomain_of(blocked))
        }
    }

In any case, I will add the as_bytes to Label, that is a good suggestion.
If I add a LabelOwned and NameOwned that doesn't have life times notation, all the other structs in the crate would need to know how to handle them.
For instance, RouteThrough has a Name<'a>, I would need to make an owned version of it as well or make it generic, something like RouteThrough<'a, T: AsRef<Name<'a>>>.
The same would apply to every other struct in the crate. This change would propagate all the way up to the Packet struct.

I think the 'static hack solves all of my current issues, so that probably just works. Thanks.
I'll close the issue, and you can add the as_bytes() whenever.


fix: fixed small typo

I only fixed a small typo error in README.md

Hi @0xflotus , why do you close this PR?

The typo errors were fixed

The typo errors were fixed

It's been fixed yet in master branch.


[Sema] Fix inappropriate diagnostics of access control for member declared in private extensions.

When a fileprivate method is declared in a private extension, a warning is raised since access level fileprivate is literally higher than private. This is not appropriate because extensions are top level declarations, for which private and fileprivate are equivalent.
This patch stops such warnings, and improves fix-it messages. Consider the case below. The original fix-it suggests overly downgrading to private. This patch makes it fileprivate, which makes more sense.
private extension Foo {
    public func foo() {}
    ^~~~~~
-   private
+   fileprivate
}

Resolves SR-8306.

@swift-ci Please test

@swift-ci Please test source compatibility


Black miniatures on gallery

hello, when i open the gallery i cant see  some miniatures from the pictures these are black, how can i fix that.

@tonydjh can you still reproduce it?


Feature request - Update to new update management solution and remove region dependency for LA

Current issue:
The Update management solution are not part of each Azure region, meaning that your deployment for log analytics and the automation account with update management will need to reside in supported regions due to linkage between Automation Account and Log analytics. (Example, Azure Automation account cannot be linked to Log analytics that resides in Sweden Central). In addition, this also effects Change tracking and Inventory solutions.
This gives several drawbacks as you do not have a self-contained Landing Zone in one Azure region and you will need to send logs to a separate region or have multiple log analytics.
Proposed solution
When GA, move to Update Center solution (public preview as of this date) as this removes dependency of the linkage between automation account and log analytics.

Hey @JimmyKarlsson112,
Great minds think alike, thanks for logging this request. This is something we already have on our internal backlog to look at when Update Center goes GA.

Trigger ADO Sync 1

Trigger ADO Sync 2


Revert changes made in previous PR

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ x] Have the files been linted and formatted?

What docs page needs to be fixed?

Redux Essentials:
Page 4:

What is the problem?
My previous contribution didn't work with subsequent use of nanoid for ids.
What changes does this PR make to fix the problem?
Reverts back to the previous correct version (error on my part)

I'll take care of it. Thanks!


fix spelling

Thanks!

great

also, i had made a pr to fix certain errors and sm64js' spelling, can other people contribute to the newsletter besides those assigned?

can other people contribute to the newsletter besides those assigned?

@Cooliokid956 sure, though a PR must be sent to the main repo - https://github.com/rust-gamedev/rust-gamedev.github.io

You can change the target of your PR to this repo. I also wanted to say that I’m ok with the changes


Small performance fix : unnecessary unboxing operations removed

Motivation :
Unboxing operations allocate unnecessary objects when it could be avoided.
Modifications:
Replaced Float.valueOf with Number.parseFloat where possible.
Result:
Less unnecessary objects allocations.

@doom369 thanks!
Cherry-picked into 4.0 (7a4a27574f21b2d13346864c976372f765e64226) and 4.1 (a4d8f930af1f17a6c48a9abd94f49e5fd719fc31)


set the condor job submission limit

don't submit more jobs than condor limit 25000 (consult Burt).
get the threshold from config and check along side with pending_job slot.
check JobSubmitter and WMBSHelper class

fixed


[docs] Update link to old Yahoo! page due to site redesign.

Yahoo! redesigned their website and our link to them is dead.

:+1:


Update 3-retrieving-data-as-lists.md

To be consistent with the Firebase api i suggest:
We reorder the update(newValue, key) method parameters to update(key, newValue)

@redian Thanks for all the PRs! Small PRs aren't an issue, but it is easier for us if you could bunch them up if you're doing a similar topic like the docs. No need to re-do anything right now, but in the future can you do a PR after your sweep? Thanks :)

Hi @davideast, will totally do that going forwards.
Thanks
R

Thanks @redian! I really appreciate these PRs so keep them coming 😄


Please disable pprof HTTP listener of vendored TiDB

/cc @tboerger @lunny

Gitea version (or commit ref):
v1.3.2 (and still in master)
Git version:
git version 2.15.1 (not relevant)
Operating system:
linux/amd64
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:
Not relevant

Description
The vendored github.com/pingcap/tidb package sets up a own HTTP listener at localhost:8888. (see https://github.com/go-gitea/gitea/blob/bdf18560110ac642c196054be38729760a4bbc39/vendor/github.com/pingcap/tidb/tidb.go#L293-L295).
Mind you, the variable in the quoted code lines is a different EnablePprof than github.com/go-gitea/gitea/modules/settings.EnablePprof!
I'd suggest to either combine the PPROF instances of TiDB and Gitea, or disable the TiDB one completely by setting github.com/pingcap/tidb.EnablePprof to false.
Screenshots
(no screenshots)

For other Gitea users: For the currently vendored version of TiDB, set TIDB_PPROF=0 in your gitea environment to prevent the additional HTTP listener on localhost:8888 from being opened.

This should be handled by TiDB, it's pretty crappy that they enable it by default...

It was handled by TiDB. The version vendored in Gitea does not have those changes, yet.

@tboerger: TiDB handled it in this PR: https://github.com/pingcap/tidb/pull/1541
That PR is older than the commit that added TiDB to Gitea (via go-xorm, I guess.)

Maybe we need to update tidb vendor

Than some maintainer should update the dependencies.

tidb now removed. Closing this issue.


[Doc] Add descriptions for bookieID

Add doc for https://github.com/apache/pulsar/issues/9174 and https://github.com/apache/pulsar/pull/9019
Update master doc firstly and then update 2.7.x.

@eolivelli could you please help review? Thanks
Is the option bookieId or BookieId?

@eolivelli could you please help review? Thanks

/pulsarbot run-failure-checks

@eolivelli I've merged master and re-run the test many times but it still fails, I do not know how to deal with test errors, could you please take a look? Many thanks!

/pulsarbot run-failure-checks

This change is only about docs, tests shouldn't even run

@eolivelli thanks. I'm wondering that since in this PR, I also modify the bookkeeper.conf and standalone.conf files, does this (red box) affect the test?

If so, how about changing it to # bookieId=  (add a hash mark)?

good catch !
yes you are right!
add the '#' at the beginning of the line


Fix flaky test preventing windows build

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

The windows zpoller self test is flaky and only passes ~50% of the time, usually preventing the package from being uploaded.

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


Provided instructions how to build live version locally

It would be nice to provide instructions how do build the live version (same as hosted on www.gnosis-safe.io/app) locally.
Currently you need to change homepagePathnamethe webpack.config.prod.js from /app/ to / (since we host it in a subfolder) and run the following commands:
export REACT_APP_ENV=production
export REACT_APP_NETWORK=mainnet
yarn build

then use a local server (e.g. server) to host it:
npm install -g serve
serve -s build_webpack -l 9000

Alternatively you can comment out line 2 in the start.js script (process.env.NODE_ENV = 'development') so that you can specify the NODE_ENV for yarn start-mainnet
Currently recommended solution

Create .env file in project root with following content:

REACT_APP_ENV=production
REACT_APP_NETWORK=mainnet
APP_PATH=/

Change homepagePathname in webpack.config.prod.js from "/app/" to process.env.APP_PATH || "/app/"
Call yarn build
Serve locally

npm install -g serve
serve -s build_webpack -l 9000

Change homepagePathname in webpack.config.prod.js from "/app/" to process.env.APP_PATH || "/app/"

@mikheevm would it make sense to adjust this in our repo to make it possible to build a live version without having to touch files that are checked into git?

maybe we could also add an .env.local_mainnet that contains
REACT_APP_ENV=production
REACT_APP_NETWORK=mainnet
APP_PATH=/

So that users just would have to copy it as .env file and call yarn build

well there's a different way I use actually 😄

Run yarn build
In build_webpack directory create a folder named app
Move img and static folder to app
serve -s build_webpack

Idk what's better. This one doesn't require any project changes

It would be nice if we do not have to move the assets, else you have to do that after each yarn build, but yeah already an improvement to adjusting the config :)

also if you want to run prod/mainnet version the user has to obtain some credentials (blocknative, portis, fortmatic, infura token). Should we also include this to instructions?

or should we register some throwaway ones ourselves?

No I would not do that ... we can expect developers to provide these, as long as it is mentioned in the readme/ docs.

Solved in https://github.com/gnosis/safe-react/commit/fc87c669cf047e368da0539f0463d22cb52c73e6.


Prepend table names to the generated enum names

Otherwise we'll get a collision if a column name is shared for an enum column across two tables

This change is

I was going to say that this change is backwards incompatible, but this is the new MySQL implementation, so I'm okay making a breaking change. Not sure why you're test is failing, but I think this is overall a good change.

I was going to say that this change is backwards incompatible, but this is the new MySQL implementation, so I'm okay making a breaking change.

Oh, hm. Well I'm all for avoiding backwards incompatible changes, but I think in this case it's more or less unavoidable? Right now the generated code will cause name collisions on the generated "enum" Go types; not sure  how else to resolve that.
That said, it might be worth spending more time deciding how the names should best be formed? E.g., if we want to get the de-pluralization to work, might be better to do it now than introduce another incompatible change in the future?

@kyleconroy Any thoughts on ^ ?
If not, can fix the tests/go forward with this change.

I think this is fine as-is! Will merge once the tests have been updated and are pasing.

Sounds good -- tests green now

Thank you!

Thanks for this one. Drove me crazy with databases where many tables had type being an enum.
I was just overriding the type to a string in the config file and it felt wrong.


OSD on GCP can be deployed into Shared VPC (XPN)

Shared VPC enables resource sharing and connectivity between projects within a common network while still maintaining isolation and control

Customers can now deploy OpenShift Dedicated (OSD) into a Shared VPC (xPN) on Google Cloud using OpenShift Cluster Manager (OCM) in the Red Hat Hybrid Cloud Console.
For more details, refer to OSD documentation


Problem of Best model

Since your Top1 and Top5 is acc rate not err rate, the condition of testTop1 < bestTop1 should be  testTop1 > bestTop1 isn't is right?

Are you referring to line 58-61 in main.lua? I don't see an issue.

Yes, I think your code is forked from fb.resnet.torch right? And in trian.lua, I find that, fb's version gives Top5 and Top1 err rates, your version gives acc rate, so in main.lua, I guess you will fail to save the model which is actually the best one.


DatabaseMetaData.getPrimaryKeys() returns garbage

Try the following table with <IP_ADDRESS>:
CREATE TABLE T (
  c1 int, c2 int,
  primary key (c1),
  unique (c1)
)
```

Thanks for the report.  Would you consider submitting a PR with your unit test and proposed solution?  Closing this for now.


"clang_ast.atd" does not work with adtj for Java generation

This is does not seem to be an issue with "facebook-clang-plugins" itself, but "clang_ast.atd" does not work with adtj for Java generation.
I've opened a ticket with ATD (https://github.com/mjambon/atd/issues/179). I am using atdj 2.0.0.
Is there a workaround for Java generation such as using an older version of clang_ast.atd? Does Facebook only use atd with ocaml?
Command used: atdj clang_ast.atd
clang_ast.atd.txt

Closing this issue as the issue opened on the atd repo is more actionable.


ADS1115 value from one channel is being copied to the other

Hi, I'm developing a project using the ADS1115. I'm facing an issue with copying values between two channels; it seems that the value from one channel is being copied to the other. I'm using a sample rate of 860 fps. Below is the acquireData function for reference:
void acquireData(data_t* data) {
   read_data = true;
   while(read_data){ //wait until the data is read and convReady is true which means that the value is ready.
      if(convReady) //convReady is connected to alertReady PIN
      {
        data->adc[0]  = ADS.getValue();
        ADS.requestADC(1);
        convReady = false;
        read_data = false;
      }
  }

  read_data = true;
  while(read_data){
      if(convReady)
      {
         data->adc[1] = ADS.getValue();
         ADS.requestADC(0);
        convReady = false;
        read_data = false;
      }
  }

}

Please note that the first time the program is executed I'm calling ADS.requestADC(0), do you have any suggestions or ideas about this issue.
Thank you in advance.

Hi,
Thanks for the question, as I am rather busy on other libraries/ issues it might take a few days. I will try to find some time today and come first suggestions.
Do you run in continuous mode?

All right, thank you, yes I'm running in continuous mode:
void userSetup() {
  ADS.begin(21,22); // SDA, SCL
  ADS.setGain(1);
  ADS.setDataRate(7);  // DATA RATE
  ADS.setMode(0);
  // set the thresholds to Trigger RDY pin
  ADS.setComparatorThresholdLow(0x0000);
  ADS.setComparatorThresholdHigh(0x8000);
  ADS.setComparatorQueConvert(0);             // enable RDY pin !!
  pinMode(4, INPUT_PULLUP);
  attachInterrupt(digitalPinToInterrupt(4), convReadyAlert, RISING);
  ADS.requestADC(0);
}

Hi, yes convReady its a volatile flag:
volatile bool convReady = false;
For a temporary fix we added delayMicroseconds(500); is there any alternatives solutions please to this problem ?

Hi, thank you for the alternative solutions 👍

Hi, thank you for the alternative solutions 👍


Add minimum height option to table

Fixes #115.
Adds a WithMinimumHeight() option to Model. When set, we add "padding" rows when rendering the view to meet the specified minimum height.
Caveats:

If the data size or page size is large, the table ends up being taller than the minimum height. The issue mentions "fixed" height (from me too), so I just want to call out that this solution is slightly different, but the use case is to fill the screen so I think this PR still fits that criteria.
There is an edge case where there are zero rows of data and a small minimum height, where we are one line height away from the requested minimum. Because we can't add only a border, we add an empty row with border anyway, leading to a table that is one line taller than the requested minimum.

Would be good to slip this into one of the examples as well. Maybe the flex example since it's dealing with stretching horizontally, we can also stretch it vertically.

Sounds good, I've added it to the flex example!

Ah we're going to need to be careful with coverage as well.  You've added a bunch of checks in the options, but looks like they aren't covered: https://coveralls.io/builds/63298625/source?filename=table%2Foptions.go
I know it's a bit verbose, but it would be helpful to have tests to make sure the expected internal height values are correct after adding these options when a minimum height is set.

Actually, looking at your tests, I think we'd just have to change the order of things.  Basically, apply the minimum height first, then apply the options after.  This would call your checks directly and should help considerably.  Maybe pass in a function that applies extra options rather than a pre-set model with the options already applied?

Maybe pass in a function that applies extra options rather than a pre-set model with the options already applied?

Do you mean in the view tests?
What do you think about adding a test in options_test.go that creates a model with a minimum height, then applies each of the changed options and re-checks the internal values?

Maybe pass in a function that applies extra options rather than a pre-set model with the options already applied?

Do you mean in the view tests?
What do you think about adding a test in options_test.go that creates a model with a minimum height, then applies each of the changed options and re-checks the internal values?

I think that would work fine too!

Thanks for the contribution!  Will release this along with your other PR once it's in.


Tolerant edge cases in LosslessJsonParser

When there is a edge case in the input variable payload such as array of null elements, the recursive method convertLosslessNumbersToNumberOrThrow underneath the createWorker throws cannot read properties of null error.
SDK Component
Zeebe gRPC client
Expected Behavior
The null should be ignored and the JSON string should be serialized successfully
Current Behavior
The convertLosslessNumbersToNumberOrThrow method in LosslessJsonParser.ts throws out exception on the 1st line
debug(Parsing LosslessNumbers to numbers for ${obj.constructor.name}) when the obj is null.
Error:
Cannot read properties of null (reading 'constructor')
Possible Solution
Add the null check in the convertLosslessNumbersToNumberOrThrow method in LosslessJsonParser.ts.
Add safe navigation operation on line 223:
obj?.constructor?.name
Add a null check before 224:
if (!obj) return obj;
Steps to Reproduce

Added a new unit test in LosslessJsonParser.unit.spec.ts

test('LosslessJsonParser correctly handles null objects', () => {
    const json = `{"abc": [null, null, null] }`;

    const parsedDto = losslessParse(json);

    expect(parsedDto.siuTaskComments).toMatchObject([null, null, null]); // 3 (string)
});

Run the unit test
The new added unit test will be failed with below error:

An unsafe number value was received for "abc" and no Dto mapping was specified.
    Cannot read properties of null (reading 'constructor')

      248 |     } catch (e) {
      249 |             const message = (e as Error).message
    > 250 |             throw new Error(
          |                   ^
      251 |                     `An unsafe number value was received for "${currentKey}" and no Dto mapping was specified.\n` +
      252 |                             message
      253 |             )

Context (Environment)
The error will stop Zeebe gRPC client from polling activate tasks.

Thanks for finding this and suggesting a fix. I've implemented it.


[BUG] Parser Error on structs.scad when trying to run: struct_set

Describe the bug
When trying to include the screws.scad and create a screw() or a standard struct get the following error:
Parsing design (AST generation)...
[ERROR: Parser error: syntax error in file structs.scad, line 48](.../OpenSCAD/libraries/BOSL2/structs.scad)
Execution aborted

Code To Reproduce Bug
#include <BOSL2/std.scad>
#include <BOSL2/structs.scad>

$fn=32;

str = struct_set(struct, "answer", 42);

Expected behavior
It should not be throwing an error.
Screenshots
N/A
Additional context
Add any other context about the problem here.

OpenSCAD Version: OpenSCAD version 2021.01
Other libraries used: None.

The parse error is that you put in those '#' characters.  OpenSCAD isn't C.
Without then you get the error that "struct" is unknown, which is what should happen.

Thanks! It worked with just one include, so not sure why it wasn't throwing the error earlier. Closing as not a bug.


Extending context with Koa 2.0

I may be missing this in the source code, but it doesn't appear that extending the context works anymore in Koa 2.0. Here's the code I've tried.
app.context.render = myRenderFn;

When I try to use ctx.render() in a route, it turns up as undefined. Any thoughts?

notice that context.render is a getter, not a variable.
2015-11-26 5:26 GMT+08:00 Derek Duncan<EMAIL_ADDRESS>
I may be missing this in the source code, but it doesn't appear that
extending the context works anymore in Koa 2.0. Here's the code I've tried.
app.context.render = myRenderFn;
When I try to use ctx.render() in a route, it turns up as undefined. Any
thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/koajs/koa/issues/609.

can you provide a short example for this issue?

@derek-duncan Did @alsotang's comment help? If not, I'll take a peek if you provide a small isolated yet runnable example. Thanks!

Closing. Please reopen with a test case


Added Iceland Holiday Support

Added Iceland holiday support and ran Black formatter on holidays.py

Hi @justinasfour04, thank you for your contribution.
I've made a review on this new work of yours but, even if I consider it very good, and fully understand the pros of code auto-formatting (it makes some sense), I have to point out that:

even if the new code often looks cleaner and more standard, we've lost some code format "intention" and readability on the way, in some places
the huge number of not "crucial" changes would over-complicate future analysis ( = "broken" history)
finally, to get the most out of your effort, we should probably adopt black formatter as a standard step for all future developments / commits (like we do for flake8 and automatic tests, to explain), in order to carry on the same code format. Unfortunately, black is available under Python 3 envs only, and this makes the resulting process trickier / less obvious (holidays is 100% Py2 compatible too, so its whole development / testing process should be too IMHO)

For these considerations, I think we have to (with some regret, I assure) leave the re-formatting task in standby for now, until at least some of these issues are removed / mitigated.
Do you think you'll be able to re-pack your modifications into a new PR excluding the format refactoring (some git cherry-picking should suffice)?
If needed, I can help you out with it.
Please let me know, thanks again
dr-p

@dr-prodigy I created a new pull request and I completely understand your point of view


Mixin conflict

https://github.com/Wilyicaro/Legacy-Minecraft/blob/1.20.4/common/src/main/java/wily/legacy/mixin/CreateWorldScreenMixin.java#L99
Here you are overriding init completely, which makes any mods that inject here fail hard. An easy solution is to just inject at HEAD and always cancel.
Related issue isXander/Controlify#310

Why would you want controlify tho?

Why would you want controlify tho?

I bet this isn't exclusive to Controlify, and to answer your question: Controlify is a significantly more powerful controller mod than what is included in this mod, such as vibration.

Fixed in version 1.6.3, thanks for reporting!


[clkmgr] block is not connected correctly to life cycle manager

https://github.com/lowRISC/opentitan/blob/3cde80a583a4c48d7ac49a678c62fb51bef3c97a/hw/top_earlgrey/rtl/autogen/top_earlgrey.sv#L1688
The block is connected to a const lc_ctrl_pkg::LC_TX_DEFAULT

See PR #6557

Thanks, this has been fixed now.


CTDNE implementation should handle cases when nodes do not appear in any of the random walks at all

Description
The authors mentioned keeping track of which nodes have appeared, and at the end performing a few walks from these nodes in order to guarantee that every node appeared at least once.
Since this approach isn't explicitly mentioned in the paper, we could decide to handle it differently especially if it turns out to negatively affect performance (speed) of the code too much.
User Story
As a data scientist
I want to use temporal walks that guarantees that all the nodes in my graph appear in the walks at least once.
So that I can reliably generate embeddings for all nodes from them.
Done Checklist

[ ] Produced code for required functionality
[ ] Tests written and coverage checked
[ ] Code review performed
[ ] Documentation on Google Docs  (if applicable)
[ ] Documentation in repo
[ ] Version number reflects new status
[ ] CHANGELOG.md updated
[ ] Team demo

A simple alternative approach: After running the word2vec model on the random walks, just assign zeros as embeddings to nodes that do not appear. This basically communicates that these nodes are uninformative, which doesn't sound unreasonable.

Another approach is to remove the positive/negative examples that reference nodes that don't have embeddings

This is now handled in the demo notebook as part of #1056 by assigning zeros as embeddings for those nodes that do not appear in any temporal walks.


Provide option for configuring offset number in kafka source

Problem
Currently the kafka source starts from the latest offset. There might be scenarios where the user might need to specify earliest offset or a specific offset number.
Persona:
Developer
Exit Criteria
User is able to specify offset number while creating the kafka source

/assign

Could be relevant: https://github.com/knative/eventing-contrib/issues/655

/remove-lifecycle stale

@itsmurugappan
What is the specific use case here? We (@slinkydeveloper and I) are trying to understand if this is a rare case that perhaps we should handle differently.
The problematic situation happens when using the offset when consumer group is defined vs not defined.

There are cases where people would want to retry specific transactions that may already have been processed.

There are cases where people would want to retry specific transactions that may already have been processed.

Are you trying to handle kafka transactions with kafka source? Perhaps KafkaSource is not the right tool for that.

In those scenarios it will be really handy if we could choose the starting point.

I'm particularly concerned about this feature, both on the spec side and the implementation side:

What happens if a consumer group already exists, say with offset 100, but the user configures in kafka source an offset, say 200?
What happens if, similar to above, the user doesn't configure the offset first, then updates the kafka source cr and configures the offset with a value higher than the actual processing status?
Should the KafkaSource continue to commit the offset when the initial offset is already configured?
KafkaSource spec fields ultimately are "templates" of the resource pods, what happens if we implement a scalable kafka source? Every instance should restart from that offset? Also note that this one might even create loops, because every time it would reset the consumer group state
Similar to above, what happens if the kafkasource crashes and then restarts? Should it restart from the configured offset?
Should the offset and consumer group be exclusive fields, to avoid any of the above problems? But what if the user wants to set both the consumer group and the offset (which is reasonable if the consumer group does not exists)?
How do we communicate to the user such edge cases?

I wonder, would it makes sense for you to create the consumer group by hand, with the fixed offset you want, and then create the kafka source specifying your custom consumer group name? There are different CLI tools out there to do so.

I wasnt talking about kafka transaction, i meant regular kafka messages, sorry for the wording there. We have faced scenarios where we had to reprocess some messages. That was my usecase.
I acknowledge your concerns. Writing a one of consumer for this purpose would make more sense. Thank you!

I have few other use cases for this request, since i closed it here, creating a issue in the new repo -> https://github.com/knative-sandbox/eventing-kafka/issues/289


TCVP-2791: Citizen Portal: Don't Allow PDF Uploads to OCR / Update Text

Description
This PR includes the following proposed change(s):

TCVP-2791
Added file type formats and validation

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes

Do we need to additionally update the API? The C# still permits the filetype upload of type pdf.
https://github.com/bcgov/jag-traffic-courts-online/blob/main/src/backend/TrafficCourts/Citizen.Service/Controllers/TicketsController.cs#L120

It's a temporary fix for now. We will be anyway working on allowing PDFs asap.


Failed to build the application in windows10

Please do not delete the issue template text before reading it because it tells you how to solve this problem.


Highly Critical issue on MAC Safari

Latest build on https://edumeet.geant.org/ fails to load on Safari on MAC - same case with latest build of my Dev instance - gives white screen with error.
I have an older build of the app on a client url ( not allowed to share publicly) - it works well in same safari.
Please help!
Safari Version:

Error on letsmeet.no

Don't mention me again, please.

Don't mention me again, please.

Edited.
Is it working well on your Mac Safari's?

It works on Version 14.1 (166<IP_ADDRESS>.6)

It works on Version 14.1 (166<IP_ADDRESS>.6)

Thanks Alex, what is the MAC OS version? I am trying on Mojave.

It works on Version 14.1 (166<IP_ADDRESS>.6)

Thanks Alex, what is the MAC OS version? I am trying on Mojave.

11.3.1 (20E241) Big Sur

seems to be solved.


Fix build.

Codecov Report

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

@@          Coverage Diff          @@
##           master     #3   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines         416    416
  Branches       24     24
=====================================
  Hits          416    416

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 c44b145...5190b43. Read the comment docs.


Fix Long tab lengths fail, and default back to 4 Issue #9147

Implement fix for issue #9147. Valid tab lengths are any number between 1 and 100.

Closing in favor of https://github.com/atom/atom/pull/9198


Premature dropping of asset handles stored in components

Bevy version
0.11.0, 0.11.1, and 0.11.2
What you did
I'm loading a labeled asset from a custom asset loader, and storing the handle in a component. The handle is generated via load() so I'm assuming that's a strong handle.
Actually, the scenario is a bit more complex: the asset is an XML file that contains both templates and styles, where each template and each style is a labeled asset. (There is no default asset). There's a system that loads a template and creates a view element tree; there's another system that loads the styles, and stores the resulting asset handles in components that are associated with the view element entities.
There are additional systems that process the loaded styles and update the scene graph.
What went wrong
The system as I have described is highly non-deterministic, and frequently fails to load styles. I can see this in a number of ways:

Polling the status of a style handle often returns "Unloaded".
Listening to asset events, I sometimes get "Removed" events for the stylesheets.

Neither of these things should be happening, since I never release or drop any of the asset handles. That is, the assets should never be in a state of "unloaded".
One thing I have observed is that putting "apply_deferred" immediately after the system that creates the asset handles reduces the occurrence of the problems (by a large amount), but does not entirely make them go away.
My conclusion is that there is some kind of race condition caused by the fact that the components containing the asset handles are not immediately inserted into the world. Unfortunately, I don't know enough about the internal details of strong handles to speculate. If strong handles were based on reference-counting, then I can't think of how this could possibly happen. OTOH, if they are based on mark-and-sweep, then it's possible that the command queue is not being considered as a GC root.
It's possible that I am doing something wrong, or that I don't completely understand the "rules of handles". I should note that when I store the handle into the component I am explicitly calling "handle.clone()".

A reproduction, whether minimal or not, would be really helpful for further investigation.

My initial hypothesis is that:

Assets are initialized into loading.
The strong handle is created by cloning.
The system ends.
The asset finishes loading.
The commands are applied.

If asset handles aren't being kept alive in commands then it would match the behavior you're seeing. Can you reproduce this if you swap to an exclusive system?

A reproduction, whether minimal or not, would be really helpful for further investigation.

All right, I'll spend some time on this. Note that even a minimal example is likely to be non-deterministic, so you may need to run it multiple times.

OK so here's what I have - it's not minimal, but it can point you to where the problems are happening. (I haven't tried the exclusive system thing yet).
Code is on a branch: https://github.com/viridia/panoply/tree/asset-loading-bug
You should only need to cargo run.
The main action happens in src/guise/view.rs. The system that loads the templates is create_views.
There are two systems that poll for style resources, one which uses asset events (update_view_styles), and one which uses change events on the component containing the style handle: (update_view_styles_poll). The reason for having two listeners is that I don't know which one will fire: the event-based one only fires the first time the asset is loaded, so any additional components that depend on that asset which are created after the asset is loaded need to use the other one. The second listener will poll until it finds the asset, but can't detect a hot replacement. (I wish there was a simpler approach - for example, if the asset event was per-handle rather than per-asset, so that even if an asset is already in memory it generates a load event for a newly-created handle.)
The log output will vary from one run to the next, but you should see one of the following two messages:
Updating node 'test-button' style handle status: 'Unloaded'

This is produced by the code that is polling based on a marker component.
2023-08-20T00:48:22.308844Z  WARN panoply::guise::view: Asset Removed: Template AssetPath { path: "editor/ui/test.guise.xml", label: Some("button") }

This line is produced by the listener that is checking for asset events, it means that it saw a "removed" event.

I tried making the system exclusive, and while it did run, it introduced other anomalies which makes it unsuitable as a reproducible example. I'm going to try and come up with a simpler example case, expect something in a day or two.

Actually, I have two additional questions:

Are we sure that AssetServer.load() produces a strong handle? The documentation doesn't say.

Would it make sense to hold off on debugging this until Asset V2 lands? I know handles are being significantly reworked. Once V2 stabilizes a bit more I can migrate my work to it.

I'm reasonably confident it does, but not 100% sure.
If this isn't urgent for you, I'd definitely encourage waiting until after Assets v2 to see if you can still reproduce.

Urgent? Well, I don't have a deadline, but it's definitely a blocker for the work I am doing. I might try and see if I can come up with a temporary workaround - like storing a copy of all the handles in a resource or something.

I've whittled down the reproduction case by about 80%. More importantly, I've removed a lot of the complexity and indirection in the code that polls asset handles, so that part should be quite a bit easier to read.
The code is now in a GH repo of its own:
https://github.com/viridia/bevy-asset-unload
Running the program produces one of two different log streams, both of which are wrong, depending on a race condition. It depends on which of the two polling systems gets run first - the one that listens for asset events, or the one that just checks the load state directly. In either case, you'll get a log output saying that the resource is in an unloaded state, which should not happen since we never drop the handles.

I also want to mention a number of potential red herrings. By this, I mean there are some alternate hypotheses as to why this might be happening, and I want to enumerate these up front so as to avoid having you waste your time.
The first has to do with the fact that the asset loader is fairly complex and doesn't resemble the typical asset loader. It's reading an XML file, and it's using an event-based reader (not serde, for reasons I won't go into), and it is inserting the labeled assets during the traversal. Also, there's no default asset, only labeled ones. So the hypothesis is that some quirk of the asset loader is causing this behavior.
However, I don't think any of this is relevant, because asset loaders don't "commit" their results until processing is finished - in other words, the loading of a complex asset is still "atomic" as far as the rest of the world is concerned.
A counter-argument to this is the fact that the asset loader is being invoked twice. That is, if you place a println() call in visit, you'll see two printed log lines. This should not happen, because the asset should be cached the first time it is loaded, there should be no need to deserialize it a second time.
The second red herring has to do with the way that my handles are stored: there's a "root" component which contains a handle to a "root" asset (the template) - this entity is the root of a tree which has children, each of which also contains an asset handle that points to a different labeled asset (the stylesheet) loaded from the same file. The tree is not built until the root asset has finished loading, so there are multiple phases of construction, each managed by a different system. So the alternate hypothesis is that there's some bug in the way that assets handles are being managed.
Again, I don't think this is the case. The fact that a handle even exists provides some strong guarantees, regardless of how or where it is stored, and it seems like those guarantees are being violated. But, I could be wrong...

with a very quick look at your code, it seems you're trying to load the asset with label main, but not such label is created. so it loads everything, and then unloads because you're holding an handle to an asset that doesn't exists

Correct. The template handle seems to work correctly, the style handles get unloaded. But neither should be unloaded, as I am keeping around both kinds of handles, stored within components.
If I understand correctly, a strong handle should never report a status of "unloaded", because the fact that the strong handle even exists should prevent the asset from becoming unloaded. (This invariant does not apply to the handle you get from an asset event reader.)
One thing you can play around with is commenting out the calls to apply_deferred within src/guise/plugin.rs. This was an attempt to work around the problem, but should not be needed.

It looks like you're hitting a known bug https://github.com/bevyengine/bevy/issues/9337
For Style, you're actually not holding strong handles from the first time the asset is loaded, but reloading them when you receive an handle for a Template. So you load the file, the templates are kept around because you have a strong handle but the styles are unloaded as you don't, then you try to reload the styles after receiving the event for the template, and that's where you're hitting that bug. You can either keep the handles from the first load, or wait for https://github.com/bevyengine/bevy/pull/8624

Oh, interesting. That does explain it. It also explains why the asset loader is being called twice.
I kind of assumed that the caching was happening on a per-file, not a per-asset level: that is, when I load a catalog of assets, I want to keep around the entire catalog in memory because I plan on accessing multiple assets within the next few frames.
So it sounds like what I need to do is figure out how to iterate through all of the assets in the file and store those in the ViewRoot.
I wonder if it would make sense to add a feature to the asset server to optionally indicate that a bunch of assets from the same file should be cached as a group? I don't know how complicated that would be, but it might make sense for other kinds of "atlas" assets.

This was really helpful investigation, thank you both. Let's close this as a duplicate.

@mockersf @alice-i-cecile I just have one follow up question, which is - what can I use as a temporary workaround? All of the approaches I have tried do not work.
For example, it has been suggested that I load all of the stylesheet assets at the same time as the template - but there's no way to do this. I don't know what the style asset paths are until I load the template, and this cannot be done synchronously. Similarly, there's no way to iterate through all of the asset paths until at least one asset is loaded.
I also tried to modify the asset loader code to add all of the stylesheets as "dependent assets" to the template asset while parsing the file. I figured that this would force them to stay in memory and not be unloaded. Unfortunately, this doesn't get past the bug - the stylesheets still have a status of "Unloaded" on the next frame.
I've also played around with storing the asset handle in a resource. No dice. The problem is not about how I am storing the handle, it's the sequence of loading a bunch of assets coming from the same file but spread out in time.
I'm out of ideas at this point.

When dealing with similar "staged asset loading" problems in the past, I've found using States and a "poll until ready, then advance" quite effective, if tedious. I'm not entirely sure the best way to do that if they share a file though: you may need to construct a dependency graph by parsing the file, and then block until all dependents are loaded.

Have your asset loader create a "meta" asset, something like
struct Meta {
    templates: HashMap<String, Handle<Template>>,
    styles: HashMap<String, Handle<Style>>,
}

And load it and keep a reference to it in something like a resource. Then either load your labelled assets from file, they should stay alive as you will keep a strong handle to everything, or get the handles from that resource.


ISPN-14553 Upgrades all Spring and Spring Boot dependencies

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

Merged, thanks


Upgrade part of ApplicationPlugin to properties

Blocked by:

https://github.com/gradle/gradle/issues/24713

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test AST

I've triggered the following builds for you. Click here to see all build failures.

AllSmokeTestsPullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


Setting up bottom navigation with proper pages with Flux

You can see the tabbar in action here.
The reducer which dispatches the necessary actions are here.


Broken depth ranges on Metal

All tests here are failing: "dEQP-VK.draw.inverted_depth_ranges". It doesn't look like Metal respects the depth viewport range as it should. I submitted a bug report, will keep this open for tracking.

Guess what, I hit a confirmed driver bug! A generic disability structure for Metal having this one bit is now a part of #2096, and it works great on my very specific macOS 10.13 + Intel GPU.


Cover generation seems broken

Even with the default configuration, it's impossible to get an acceptable result.
{
    "title": "Book Title",
    "author": "Author Name",
    "plugins": ["autocover"],
    "pluginsConfig": {
        "autocover": {
            "font": {
                "family": "Impact",
                "color": "#000"
            },
            "size": {
                "w": 1800,
                "h": 2360
            },
            "background": {
                "color": "#FFF"
            }
        }
    }
}

Result:

We probably won't fix this, because managing the dependencies of this plugin is a pain, and covers are losing importance. People who need a cover will just have to make one using simple tools such as Photoshop or Inkscape, and it might as well be simpler.

It's loosing importance because it got unmaintained. I was having some
ideas as moving all the rendering code to the SVG engine to make it simpler
and more flexible and only require to define some metadata inside
book.json, but I've not done it yet because it lasted so much time to
accept and publish my other pull-requests on npm. I don't need it anymore,
but if I get sure it will be reviewed in a razonable time, I'll more than
happy to do TheCorrectThing(tm) and implement it.
El 14/6/2016 12:06, "Nicolas Gaborit"<EMAIL_ADDRESS>escribió:

We probably won't fix this, because managing the dependencies of this
plugin is a pain, and covers are losing importance. People who need a cover
will just have to make one using simple tools such as Photoshop or
Inkscape, and it might as well be simpler.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GitbookIO/plugin-autocover/issues/14#issuecomment-225837177,
or mute the thread
https://github.com/notifications/unsubscribe/AAgfvtfJMCBnFLMVPNDpIfWIH-mM9YxZks5qLn00gaJpZM4I1J5k
.

@piranna, your past contributions were really appreciated. I can simply add you as collaborator on the repo, so you can freely merge contributions if needed (and also as NPM package publisher). Of course, we would keep a Pull Request based workflow, to allow reviews and discussions. What do you think ?

@piranna The autocover plugin is no longer enabled by default and we stopped rendering covers a long time ago.
However I did start coding something you might be interested in: https://github.com/AaronO/magic-covers. It's basically a library to generate covers based on generative art, it could be turned into a small single page JS app (with React) so people can enter their title other info and get a decent looking cover.
Here are a few samples based on the oreally cover style: https://github.com/AaronO/magic-covers

@piranna, your past contributions were really appreciated

Thank you :-)

I can simply add you as collaborator on the repo, so you can freely merge contributions if needed (and also as NPM package publisher). Of course, we would keep a Pull Request based workflow, to allow reviews and discussions. What do you think ?

Ok, I agree on that. My concerns was mostly because it seemed the repo got unmaintained in benefict of core gitbook one, and it was a bit sad...

@piranna The autocover plugin is no longer enabled by default and we stopped rendering covers a long time ago.

I know it's not enabled by default now, I think it's the correct thing here.

However I did start coding something you might be interested in: https://github.com/AaronO/magic-covers. It's basically a library to generate covers based on generative art, it could be turned into a small single page JS app (with React) so people can enter their title other info and get a decent looking cover.
Here are a few samples based on the oreally cover style: https://github.com/AaronO/magic-covers

They looks nice! :-D Although I don't know if it makes to have it only as a webapp, I'm more into automatically build everything from sources instead of adding some "blobs" to my code or books, and in this context a pregenerated image is blob if you could be able to generate the same image providing the raw source data instead...
I remember another code that was generating the cover backgrounds randomly using a system similar to GitHub identicons, but I've never seen it again, do you know what would have happened to it?
Oh, and by the way: please generate SVG files instead of PNG when possible, they are better for printing ;-)


'aresriot.aws-aps1-prod.ap-gp-mumbai-1'

This error pops up as soon as the agent select starts.

please download the temp build in #66

This is fixed with release 1.26.


HazyChild in scrollable won't move with the object when the overscroll effect gets triggered

Information

Haze version: 0.5.0
Platform: Android, Jetpack compose

Hi, I've just found your library and this is something we have been looking for in multiple projects but were never able to implement for Android. Thanks for building this. I did find a bug or maybe something that has not been implemented yet and wanted to let you know.
Expected Behavior
When using hazyChild in a scrollable it should move with the item when overscroll effect gets triggered.
Actual Behavior
When adding hazyChild to lazy column items and scrolling everything works as expected.
However when you pull a bit further and the overscroll effect gets triggered the blur stays in the same place while the item moves, showing the blur next to the item.
Steps to Reproduce the Problem
@Composable
private fun ListExample() {
    val numbers = listOf(1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14)
    val hazeState = remember { HazeState() }

    Box(modifier = Modifier.fillMaxSize()) {
        Text(
            text = "Background text",
            style = MaterialTheme.typography.headlineLarge,
            modifier = Modifier
                .align(Alignment.Center)
                .haze(hazeState)
        )

        LazyColumn(verticalArrangement = Arrangement.spacedBy(24.dp)) {
            items(numbers) {
                Card(
                    colors = CardDefaults.cardColors(containerColor = Color.Blue.copy(alpha = 0.4f)),
                    modifier = Modifier
                        .padding(24.dp)
                        .hazeChild(hazeState, shape = RoundedCornerShape(16.dp))
                ) {
                    Box(
                        modifier = Modifier
                            .fillMaxSize()
                            .padding(vertical = 48.dp)
                    ) {
                        Text(text = "item $it", modifier = Modifier.align(Alignment.Center))
                    }
                }
            }
        }
    }
}

https://github.com/chrisbanes/haze/assets/16354873/93097fdb-ccf7-4e31-818a-2a1381bbe29f

Duplicate of #66


This is way too basic. Another tool does the job way better

https://github.com/dotnetwise/Snap.js/
All the credits goes to jakiestfu we have only merged all the pull requests

Looks great! This is example for someone who wants to make something super custom and is looking for a hint towards a starting point.
Thanks for pointing out that lib though.

Looks great! This is example for someone who wants to make something super custom and is looking for a hint towards a starting point.
Thanks for pointing out that lib though.


Not returning https URL

Is it possible to force this to return an https://... URL?
If I run lt --port 8000 or lt --port 8000 -h https://localtunnel.me it will return something like http://wnorewfygi.localtunnel.me instead of https://wnorewfygi.localtunnel.me.

hm, wonder if this is a recent regression or if it always did this. Probably best to return https by default and have a flag for --insecure or some such.

I got around this by using https://github.com/rase-/zuul-localtunnel/pull/1 in Zuul


mpunits with conan 2.0

Hi, I don't know if this happened before because I used it only a couple of times and only once successfully.
Today, in my Fedora 37, I started from scratch and pip install conan seem to have installed Conan 2.0 (instead of 1.5x as I remember doing).
Then, when I try to use mpunits (defined as a dependency in a conanfile.txt) I get the following message, which seems to say mp-units' recipe is not compatible with Conan 2.
$ conan --version
Conan version 2.0.0
[correaa@PROART build]$ conan install .. -pr:b=default -s compiler.cppstd=20 -b=missing

======== Input profiles ========
Profile host:
[settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.cppstd=20
compiler.libcxx=libstdc++11
compiler.version=12
os=Linux

Profile build:
[settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.cppstd=gnu17
compiler.libcxx=libstdc++11
compiler.version=12
os=Linux

======== Computing dependency graph ========
Graph root
    conanfile.txt: /home/correaa/boron_hydride/ground_state/conanfile.txt
*********************************************************
Recipe 'mp-units/0.7.0' seems broken.
It is possible that this recipe is not Conan 2.0 ready
If the recipe comes from ConanCenter check: https://conan.io/cci-v2.html
If it is your recipe, check it is updated to 2.0
*********************************************************

ERROR: Package 'mp-units/0.7.0' not resolved: mp-units/0.7.0: Cannot load recipe.
Error loading conanfile at '/home/correaa/.conan2/p/mp-unb5b48022c2a49/e/conanfile.py': Unable to load conanfile in /home/correaa/.conan2/p/mp-unb5b48022c2a49/e/conanfile.py
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/correaa/.conan2/p/mp-unb5b48022c2a49/e/conanfile.py", line 1, in <module>
    from conans import ConanFile, CMake, tools
ImportError: cannot import name 'ConanFile' from 'conans' (/home/correaa/.local/lib/python3.11/site-packages/conans/__init__.py)

Is there a workaround for this?

Hi, unfortunately, it seems that while moving to Conan 2.0, the Conan team did not upgrade the mp-units recipe for now. See more at: https://github.com/conan-io/conan-center-index/discussions/16196. Filing an issue on ConanCenter could help. I will try to look into it also ASAP but I am now busy with other work 😢

I provided fixes to ConanCenter here: https://github.com/conan-io/conan-center-index/pull/16252.

What are the chances, I  started using conan and the same week they upgrade to 2.0 :rofl:
I worked around this by doing pip install conan==1.56.

It seems that it might take a while for ConanCenter. In the meantime, you can use the same package from our's artifactory. The only difference is that I named it mp-unit/0.7.0@mpusz/stable to make sure it does not collide with the ConanCenter's one. In order to download it, you will need to add our artifactory to your local Conan client with:
conan remote add conan-mpusz https://mpusz.jfrog.io/artifactory/api/conan/conan-oss

See more in https://mpusz.github.io/units/usage.html#conan-cmake-live-at-head.
I hope it helps and that ConanCenter will soon catch up with all broken packages.

@NAThompson, yes, that is expected. Visual Studio is not a valid compiler for Conan 2.0. Conan 2.0 will support only msvc as listed above.

Yes, 0.8.0 is still broken and I am not sure how to fix it until the dependencies catch up as well (i.e. linear_algebra is not 2.0 friendly until now). I will ask them today what is the progress on the migration.

Even after that, the package will not be fully functional as some of our dependencies (doxygen and linear_algebra) are not Conan 2.0 ready.

we need a package manager for the package manager 😁

After the release of Conan 2.0.1 and some more changes to our scripts, it seems that mp-units/0.8.0 is ready to be used with the following limitations:

Doxygen package on CCI is still not 2.0-friendly (this also means we can not update the official docs 😢  )
linear-algebra dependency also does not work with Conan 2.0 as of now, so we can't enable building those examples for now.

mp-units/0.7.0 PR on CCI was accepted but is still not merged. It seems that the Conan team has some significant issues refactoring all scripts to work properly in their CI for 2.0 as well. I will keep you updated. Until then you can use mp-units/0.7.0@mpusz/stable from our local Artifactory server as described above.

I used these commands:

apt-get update && apt-get install --no-install-recommends -qq -y pipx
- pipx ensurepath && source ~/.bashrc
- pipx install conan
- cd ground_state
- mkdir build && cd build
- conan profile detect
- conan remote add conan-mpusz https://mpusz.jfrog.io/artifactory/api/conan/conan-oss
- conan install .. -pr:b=default -s compiler.cppstd=20 -b=missing

for a project with:
$ cat conanfile.txt
[requires]
mp-unit/0.7.0@mpusz/stable

[generators]
CMakeToolchain
CMakeDeps

The output of the last command in the recipe above is this Unable to find 'mp-unit/0.7.0@mpusz/stable' in remotes this below.
Am I missing something? (I tried with mpunit0.8.0 also but it doesn't work, so I went for the artifactory workaround).
$ conan install .. -pr:b=default -s compiler.cppstd=20 -b=missing

======== Input profiles ========
Profile host:
[settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.cppstd=20
compiler.libcxx=libstdc++11
compiler.version=12
os=Linux

Profile build:
[settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.cppstd=gnu17
compiler.libcxx=libstdc++11
compiler.version=12
os=Linux

======== Computing dependency graph ========
mp-unit/0.7.0@mpusz/stable: Not found in local cache, looking in remotes...
mp-unit/0.7.0@mpusz/stable: Checking remote: conancenter
mp-unit/0.7.0@mpusz/stable: Checking remote: conan-mpusz
Graph root
    conanfile.txt: /builds/project-0/ground_state/conanfile.txt
ERROR: Package 'mp-unit/0.7.0@mpusz/stable' not resolved: Unable to find 'mp-unit/0.7.0@mpusz/stable' in remotes

Yes ;-) The package name is mp-units/0.7.0@mpusz/stable and not mp-unit/0.7.0@mpusz/stable.

As of today, my artifactory hosts the following packages:
$ conan list mp-units -r conan-mpusz
conan-mpusz
  mp-units
    mp-units/0.7.0@mpusz/stable
    mp-units/0.7.0@mpusz/testing
    mp-units/0.8.0@mpusz/testing

Yes ;-) The package name is mp-units/0.7.0@mpusz/stable and not mp-unit/0.7.0@mpusz/stable.

That was embarrassing. Sorry. Working now.

$ conan list mp-units -r conancenter
conancenter
  mp-units
    mp-units/0.6.0
    mp-units/0.7.0
    mp-units/0.8.0

"mp-units/0.8.0" was added to ConanCenter today. It is the only revision compatible with the Conan 2.0.


build fail with latest git

Linux Fedora 23
Rack 0.6.1
Is a dependency missing ?
g++ -Wsuggest-override -std=c++11 -Idep/include -DSLUG=mscHack -fPIC -I../../include -I../../dep/include -DVERSION=0.6.2 -MMD -MP -g -O3 -march=nocona -ffast-math -fno-finite-math-only -Wall -Wextra -Wno-unused-parameter -DARCH_LIN -c -o build/src/Windz.cpp.o src/Windz.cpp
src/Windz.cpp: In member function ‘void Windz::BuildWave(int)’:
src/Windz.cpp:447:2: error: ‘m_osc’ was not declared in this scope
m_osc[ ch ].semi = semis[ srand() & 0x7 ];
^
src/Windz.cpp:450:2: error: ‘m_wav’ was not declared in this scope
m_wav[ ch ][ 0 ].Init( EnvelopeData::MODE_LOOP, EnvelopeData::RANGE_Audio, false, 1.0f );
^
src/Windz.cpp: In member function ‘void Windz::BuildDrone()’:
src/Windz.cpp:496:2: error: ‘m_global’ was not declared in this scope
m_global[ GOSC_FILTER ].Init( EnvelopeData::MODE_LOOP, EnvelopeData::RANGE_0to1, false, 1.0f );
^
src/Windz.cpp:496:12: error: ‘GOSC_FILTER’ was not declared in this scope
m_global[ GOSC_FILTER ].Init( EnvelopeData::MODE_LOOP, EnvelopeData::RANGE_0to1, false, 1.0f );
^
src/Windz.cpp:497:2: error: ‘m_GlobalOsc’ was not declared in this scope
m_GlobalOsc[ GOSC_FILTER ].finc = 1.0f / frand_mm( 14.5f, 38.0f );
^
src/Windz.cpp:502:12: error: ‘GOSC_NOISE’ was not declared in this scope
m_global[ GOSC_NOISE ].Init( EnvelopeData::MODE_LOOP, EnvelopeData::RANGE_0to1, false, 1.0f );
^
src/Windz.cpp: In member function ‘virtual void Windz::step()’:
src/Windz.cpp:713:4: error: ‘m_wav’ was not declared in this scope
m_wav[ ch ][ wv ].m_Clock.syncInc = 32.7032f * clamp( powf( 2.0f, clamp( inputs[ IN_VOCT ].normalize( 3.0f ) + m_osc[ ch ].semi, 0.0f, VOCT_MAX ) ), 0.0f, 4186.01f );
^
src/Windz.cpp:713:115: error: ‘m_osc’ was not declared in this scope
m_wav[ ch ][ wv ].m_Clock.syncInc = 32.7032f * clamp( powf( 2.0f, clamp( inputs[ IN_VOCT ].normalize( 3.0f ) + m_osc[ ch ].semi, 0.0f, VOCT_MAX ) ), 0.0f, 4186.01f );
^
src/Windz.cpp:724:55: error: ‘fmorph’ was not declared in this scope
In += m_wav[ ch ][ wv ].procStep( false, false ) * fmorph[ wv ];
^
src/Windz.cpp:728:16: error: ‘m_osc’ was not declared in this scope
out += In * m_osc[ ch ].fval[ MOD_LEVEL ];
^
src/Windz.cpp:734:36: error: ‘m_GlobalOsc’ was not declared in this scope
out += frand_mm( -1.0f, 1.0f ) * m_GlobalOsc[ GOSC_NOISE ].fval;
^
src/Windz.cpp:734:49: error: ‘GOSC_NOISE’ was not declared in this scope
out += frand_mm( -1.0f, 1.0f ) * m_GlobalOsc[ GOSC_NOISE ].fval;
^
src/Windz.cpp:737:45: error: ‘m_GlobalOsc’ was not declared in this scope
ChangeFilterCutoff( &m_filter, m_cuttoff * m_GlobalOsc[ GOSC_FILTER ].fval );
^
src/Windz.cpp:737:58: error: ‘GOSC_FILTER’ was not declared in this scope
ChangeFilterCutoff( &m_filter, m_cuttoff * m_GlobalOsc[ GOSC_FILTER ].fval );
^
src/Windz.cpp:738:33: error: no matching function for call to ‘Windz::processFilter(FILTER_STRUCT ()[3], float)’
processFilter( &m_filter, &out );
^
src/Windz.cpp:573:6: note: candidate: void Windz::processFilter(FILTER_STRUCT*, float*)
void Windz::processFilter( FILTER_STRUCT pf, float pIn )
^
src/Windz.cpp:573:6: note:   no known conversion for argument 1 from ‘FILTER_STRUCT ()[3]’ to ‘FILTER_STRUCT’
../../compile.mk:64: recipe for target 'build/src/Windz.cpp.o' failed
make: *** [build/src/Windz.cpp.o] Error 1

My apologies, I had to do a quick commit and forgot I started making changes...

for now if you remove the Windz references it will build

Thanks ! I commented out the Windz references, everything is good now.
Btw, I ran into an earlier error but it's likely just due to the age of my system and compiler:
g++ -Wsuggest-override -std=c++11 -Idep/include -DSLUG=mscHack -fPIC -I../../include -I../../dep/include -DVERSION=0.6.2 -MMD -MP -g -O3 -march=nocona -ffast-math -fno-finite-math-only -Wall -Wextra -Wno-unused-parameter -DARCH_LIN -c -o build/src/mscHack.cpp.o src/mscHack.cpp
src/mscHack.cpp:45:14: error: ‘unsigned int index’ redeclared as different kind of symbol
unsigned int index = 4095;
^
In file included from ../../include/util/common.hpp:7:0,
from ../../include/rack.hpp:4,
from src/mscHack.hpp:1,
from src/mscHack.cpp:1:
/usr/include/string.h:477:1: note: previous declaration ‘const char* index(const char*, int)’
index (const char *__s, int __c) __THROW
^
src/mscHack.cpp: In function ‘void init_rand(unsigned int)’:
src/mscHack.cpp:59:11: error: overloaded function with no contextual type information
index = 4095;
^
src/mscHack.cpp: In function ‘short unsigned int srand()’:
src/mscHack.cpp:67:22: error: invalid operands of types ‘’ and ‘int’ to binary ‘operator+’
index = (index + 1) & 4095;
^
src/mscHack.cpp:68:20: error: invalid types ‘unsigned int [4096][]’ for array subscript
t = a * Q[index] + c;
^
src/mscHack.cpp:78:39: error: invalid types ‘unsigned int [4096][]’ for array subscript
return (unsigned short)( ( Q[index] = r - x ) & 0xFFFF );
^
../../compile.mk:61: recipe for target 'build/src/mscHack.cpp.o' failed
make: *** [build/src/mscHack.cpp.o] Error 1
This error was easily resolved by changing index to Index in mscHack.cpp


-AccessToken does not decrypt supplied PSCredential

VenafiTppPS version: 20.4
PowerShell version: 5.1

# Steps to reproduce

$AccessToken = New-Object System.Management.Automation.PSCredential('AccessToken', ($token.AccessToken | ConvertTo-SecureString -AsPlainText -Force))
Revoke-TppToken -AuthServer $AuthUrl -AccessToken $AccessToken

# Expected behavior

Token is revoked in TPP

# Actual behavior

StatusCode        : 202
StatusDescription : Invalid token format
Content           : {}
RawContent        : HTTP/1.1 202 Invalid token format
                    Access-Control-Allow-Origin: *
                    X-Frame-Options: SAMEORIGIN
                    X-Content-Type-Options: nosniff
                    Strict-Transport-Security: max-age=31536000; includeSubDomains
                    Content...
Headers           : {[Access-Control-Allow-Origin, *], [X-Frame-Options, SAMEORIGIN], [X-Content-Type-Options, nosniff], [Strict-Transport-Security,
                    max-age=31536000; includeSubDomains]...}
RawContentLength  : 0

# Fix

Revoke-TppToken, line 98
-$params.Header = @{'Authorization' = 'Bearer {0}' -f $AccessToken }
+$params.Header = @{'Authorization' = 'Bearer {0}' -f $AccessToken.GetNetworkCredential().password }

@wilddev65, since it's not being stored and security isn't an issue, would it make more sense for -AccessToken to accept a string instead of a credential?  I think the original idea was to have a mechanism to revoke a token if it was obtained from another source and converting it to a credential in this case is probably not needed.  Thoughts?
Out of curiosity, what is $token.AccessToken in your steps to reproduce, not from New-TppToken?

Sorry, $token.AccessToken is just a token string to feed Revoke-Token a PSCredential object. Should have been more clear about that.


stage can't be customized with Platform API

we can customize stage with oscm-portal from "Customize stage" menu.
but we can't customize stage with Platform API.
could you make it possible to do it?

@toshihiro-shiino,
what is the business motivation behind this request, what is the expected benefit?

@mdehn,
We are offering marketplace called Management Portal in our service.
We have tried to automate the setup of this Management Portal by calling Platform API based on strong requests from customers to shorten the delivery time after receiving the request for setup.
However, there are some items that can be configured only via Portal and not via Platform API, so we are still not be able to implement the automated setup to the fullest extent.
This request is issued for the purpose of realizing complete automation of the setup and with business motivation of reduced delivery time and manpower.
In addition, other than the intention to resolve this issue, we believe the abilities offered by Web interface and by API should be identical.

This is a valid request, but currently there are no plans to extend the API with setup functionality.
If this functionality is urgent for you, you may consider a contribution to OSCM.

@mdehn,
This functionality is not urgent for us.
I appreciate it if you remember this issue when planning to extend the API.


How to add announcement banner and announcements on page

Hey guys, someone can explain me how to add the components to integrate announcement banner and announcements on page? Thanks!

Hey!
Have you seen https://github.com/K-Phoen/backstage-plugin-announcements/blob/main/docs/latest-announcements-on-page.md & https://github.com/K-Phoen/backstage-plugin-announcements/blob/main/docs/latest-announcement-banner.md? :)


chore: upgrade typescript, frontend dependencies and set type module

Typescript upgraded to the latest version and framework and docs are made
into es modules. When this happened a lot of type errors occured. Upgrading
all frontend dependencies solved all but two who needed manual correction.

@olchyk98 @SebCodesTheWeb Can you look into the error in framework/lib/components/form/form.tsx?
I am not able to solve it myself.

Tests should pass now.


Case-insensitive searching

Original report by Kristopher Baehr (Bitbucket: krisbaehr, GitHub: krisbaehr).

Is there an equivalent to "exec sql SET OPTION SRTSEQ=*LANGIDSHR" as in an RPGLE program? This tells the program to perform case-insensitive searches when executing SQL. Could we run this before running a statement, or by setting a statement option? If this was a setting on a statement, not a connection that would be great. We can always use the upper() and lower() sql functions if necessary. Any help is appreciated.

Consulted some Db2 experts. This feature is not supported.
It depends on the user to handle the case-insensitive searching like where A=‘a' or A=upper('a')


Add Copies of the Visual Studio DX12 templates to GitHub

Suggested in the conversation of issue #7.  Splitting it out as a separate issue since I do not intend to close issue #7.  I expect that question to come up again otherwise.
This issue covers adding both the CoreWindow DX12 template, and a preview of the XAML DX12 template.  These will go into a folder called "Templates", but note that they will not be in the format Visual Studio recognizes as a "template".  They will be in a consumable "sample" format, essentially what you get when you create a new project in Visual Studio using the template.

Submitted a preview of the DX12+XAML template.
https://github.com/Microsoft/DirectX-Graphics-Samples/tree/feature_xaml_template/Templates/DirectX12XamlApp


Issue due to username case sensitivity in consent management

When logging into an SP, providing a different upper-lower case version of the username prompts the user the set consent page. Each upper-lowercase variation of one particular username has it’s own consent settings.
The case of the username should have no influence on the consent configuration and it should be bound to the user, regardless of the variation of the usernames’ case.

For the public fix the issue is by default resolved. So system will consider user name case insensitive by default. in the public fix.


Add screenshots

Add screenshots to the readme. It is helpful to understand what it is.

CTest result
Source code debug
Gcov

Done.


TASK: Adjust Coding style to recent CGL

Just a cosmetic fix to make the (PHP) code adhere to
our latest coding guidelines
Note: This would probably make #6 obsolete

Merging as no-brainer


Typo

The abstract is terrific. Is there a word missing here, "It also clarifies mandatory to implement features for the optionalities mentioned in the referenced specifications." What is mandatory? Are the implementation features mandatory? Or is there a word missing after mandatory? Thank you.

I think what is meant is "in places where the referenced specification says "optional", this interop profile will mandate one of the options for the sake of interoperability, and if the implementers want to be compliant with this profile, they need to follow those requirements.


Use Service Reference for Token

Instead of including the token as a variable, I'd love it if we could use it from a Connected Service.

Checked with VSTS team and they say this may make it into the work required for TFS 2017.1... That would be nice... But it still breaks every TFS server that's older... :(

Would be nice if we could detect TFS version from task and behave accordingly

Can't in the UI. one task.json to rule them all. But a minimum agent version would work. Or provide a variable workaroud, like I do with the Snyk task now.


Facebook Graph now v3.0 - tokens no longer working

Facebook has updated the graph API to v3.0, and now tokens are no longer working. However, it appears that tokens created before v2.12 are still functioning.
I switched the version in socialfeed.js, but it did not work.
I also scanned Facebook's Broken Changelog and came up empty-handed.
Any help you can offer would be much appreciated.
Thanks for the great plugin!

Oh yes, I'm not alone... welcome to issue #330 .

Possible fix, tested and currently working: go to
https://developers.facebook.com/tools/explorer/145634995501895/?method=GET&path=me%2Ffeed&version=v3.0
and grab the token it generates from here.

I generated new access token using my App id and app secret but when i put it into the social-feed.js it still dont show facebook page feed, in example (https://pavelk2.github.io/social-feed-example/) facebook is working, please help me too so i can make it work too.
im also not getting any errors in concole

@mutiullah7, don't bother with that. Most probably new FB API won't work with this plugin.
As an alternative I suggest you to try free option at juicer.io.
Other questions: issue #330.

@mutiullah7, don't bother with that. Most probably new FB API won't work with this plugin.
As an alternative I suggest you to try free option at juicer.io.
Other questions: issue #330.

Great to know, @mutiullah7. Thanks for feedback!

thank you @mutiullah7  and @vushe your discussions has really helped me fix the plugin.
As said in other posts, update your socialfeed.js to v2.12

request_url = Feed.facebook.graph + 'v2.12/' + userdata.id + '/posts'+ fields + limit + query_extention;

request_url = Feed.facebook.graph + 'v2.12/' + page + '/feed'+ fields + limit + query_extention;

After you've changed the above, go to this link and generate your new access code -> https://developers.facebook.com/tools/explorer/145634995501895/?method=GET&path=me%2Ffeed&version=v3.0
Add that into your js file and BOOM - fixed.
Thanks again!!

I believe the access codes which are generated by that link are time restriction.


fix usages filtering by region

@schfeca75 , please take a look!

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1028/


Typings for event functions requires HTMLElement, but querySelector (et. al) returns Element

dom-testing-library version: 3.12.0
react version: 16.7.0-alpha.0
node version: 0.13.0
npm (or yarn) version: 1.12.1

Relevant code or config:
fireEvent.click(container.querySelector('.backButton')!);

What you did:
Tried to trigger a click on a result of querySelector.
What happened:
Typescript complained (did not build).
Reproduction:

Any typescript project with the line above should produce the issue.
Problem description:

It disallows code that is perfectly valid.
Suggested solution:

Allow events to accept Element.

:tada: This issue has been resolved in version 3.12.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


JPMS - split package io.dropwizard.validation

io.dropwizard.validation is a split package between dropwizard-core and dropwizard-validation.
Because of that I wasn't able to compile my app as a JPMS module.
[using 2.0.1] I tested moving io/dropwizard/validation/InjectValidatorFeature.class from dropwizard-core.jar into dropwizard-validation.jar and my app can build using JPMS and seems to run ok.
That's probably not the right fix for this but it got rid of the split package issue.  Could io.dropwizard.validation have a different package name in dropwizard-core or should that class move into dropwizard-validation?

Refs #3233

Closed via #3233.