Bug in Markdown support in acitivity feed

Affected module
UI
Describe the bug
The markdown support is not working for activity feed, it has it "reversed".
To Reproduce
Activity Feed

Table Page

Expected behavior
proper markdown support to be displayed in activity feed
Version:

OS: Chrome
OpenMetadata version: 0.12.1

@chirag-madlani can you check if its UI or backend issue?

On the activity feed, we are getting the diff from the backed in an inline code block format and markdown does not parse the link inside the code block.
Example :
[link](https://link.com)

On the other hand in the description, we directly get the markdown string and that's why we can see the parsed link.
Example :
link

@mohityadav766 can you please check and see if we can land the fix for 0.13.0

Fixed


Fix of inventory building and refactoring

ManageIQ::Providers::Inventory.build() creates persister and collector classes in bad order, which causes problem with non-collected data in persister.

Fix of PR: https://github.com/ManageIQ/manageiq/pull/17907

Next provides helpers collector_class_for, parser_class_for and persister_class_for.
Existing ones are using class_for method now.

@agrare pls look&merge

@slemrmartin looks like a legitimate spec failure

@agrare I forget to test against ansible and it doesn't have namespacing like others, so this err is causes by this PR


Boosters remaining after permission is removed

Detailed description of the issue:
So I've played around with adding temporary money and exp boosters to players, using LuckPerms temporary permissions.
For example, giving the player jobs.boost.all.money.3 for 300% money gain.
However, when the permission is removed, the player still has the booster.
Typing /jobs bonus miner on the players account

Checking if they have the permission:

Jobs version:
<IP_ADDRESS>
Server Type (Spigot/Paper/etc):
Paper
Server Version (using /ver):

#1368 I can confirm a lot of people is having this bug. It's still an issue on my server and the boost will not go away until the server restarts.

Same issue here, bug began in <IP_ADDRESS> and persists currently.

I can also confirm this issue.

I am having the same issue.


Russian localization has Y / N commands translated to Russian while expecting user input in English

A brief description
Russian localization incorrectly translates Y / N commands to Russian while expecting user input in English.

Tools ask for user input using localized format (Russian), and wait for the input in non-localized format (English)
For example:
'Продолжить загрузку: [д / н]:'
I have corrected the initially wrong observed result
Used input is accepted either in Russian (then the input is incorrectly displayed) or in English (them you must use 'y' or 'n' as usual).
##Expected results
System commands such as confirmations awaited from used input must be left in English despite the program output. For example, if an utility is awaiting user confirmation in YES / NO format, the utility must output to stdout messages that follow the convention:
<QUERY_LOCALIZED> [Y / N]:
where <QUERY_LOCALIZED> is a query for user input localized for the target language.
For example:
'Продолжить загрузку: [y / n]:'

Actual results (with terminal output if applicable)

Everything including system commands is now localized, so the currently used convention is
[[QUERY_LOCALIZED]] [[LOCALIZED_CONFIRM / LOCALIZED_DECLINE]]:
for example:
'Продолжить загрузку: [д / н]:'

Your Windows build number

10.0.14393.105

Steps / commands required to reproduce the error
Run any utility that requests some user confirmation.

Strace of the failing command

Required packages and commands to install

See our contributing instructions for assistance.

@bitcrazed, @miniksa might be able to provide more information on localization.

"Run any utility" isn't enough data for me to reproduce this. Please provide an exact example of what command you are running and what the output is. Screenshots are also appreciated.
cc: @paulcam206

Here's the sample output screen:

@gudenuff this is an upstream issue with Debian/Ubuntu. It totally depends on translations done by translation teams for respective utilities and packages. It is definitely NOT A BUG for WSL.
To sum up, the translation strings were translated incorrectly. That's it.

@wizardist Meaning that MSFT has no ties with whoever is localizing Ubuntu builds they host on WSL? So the quality and assurance here is made on Cannonical's side, did I get this right?
In Windows you generally expect to respond using original commands (Y or N), despite the localization:

(courtesy of computia.ru)
As you see from the above example, the strings within the chkdsk tool were translated into Russian, however, the tool expects the user to input Y or N.
Okay, I am closing the issue as per Pavel's note.

@gudenuff , "Canonical" -- that matches my understanding.
WSL is aiming to provide bug-for-bug compatibility with Linux-Ubuntu.  As a general heuristic, if an application behaves incorrectly on regular Linux-Ubuntu 14.04 and it also behaves incorrectly on WSL, then Canonical must first fix it for Linux-Ubuntu; WSL can then pull in the fix for Windows.
Though, if you're looking for quality and assurance, you may be in for a bit of a rude awakening :-)  WSL aims to provide bug-for-bug compatibility with Ubuntu Linux.  Which means the goal for now is to work as well, or as poorly, as Linux itself.  Linux in general is community-driven; there are distributions that aim to provide commercial support, but in my opinion/experience only a very few of them actually have the staffing and experience to do that in a rigorous way.  Most of them depend on the open-source community for support.


Merge asyncEach and forEach?

The functions forEach and asyncEach are almost exactly the same,
forEach(fn: Function) {
	var i = 0,
		len = this.length;
	for (; i < len; ) {
		fn(this[i], i, this);
		i++;
	}
}

async asyncEach(fn: Function) {
	var i = 0,
		len = this.length;
	for (; i < len; ) {
		await fn(this[i], i, this);
		i++;
	}
}

I'm wondering if it is possible to merge the two? Couldn't something like this do essentially the same thing?
async asyncEach(fn: Function, isAsync:? boolean) {
	var i = 0,
		len = this.length,
		wait = this.is('async'); // or check instance for a property descriptor labeling it as async?
	for (; i < len; ) {
		if (wait) {
			await fn(this[i], i, this);
		} else {
			fn(this[i], i, this);
		}
		i++;
	}
}

Or is this not possible due to how async functions work? I know that async/await is basically sugar over Promises, but if an await falls in a forest and no one is around to hear it...

addressed in most recent commit


Add support to django-meta 2.0 replacing g+ attributes with schema.org

Description
django-meta 2.0 drops g+ in favour of schema.org
appconfig fields names are kept unchanged to avoid a data migration
References
Fix #627
Checklist

[x] I have read the contribution guide
[x] Code lint checked via inv lint
[x] changes file included (see docs)
[x] Usage documentation added in case of new features
[x] Tests added

Coverage decreased (-0.1%) to 96.633% when pulling 94df9d017090ebda0e52e30efe7c22c076de0a1d on feature/issue-637-django-meta into 5d01a8668cc1733c956c130f61e69da9856e953e on develop.


Enable style tag on codeview

Its better to have style tag on codeview so that we can avoid inline css
Editor like summernote as this option.
Thanks in advance

https://github.com/JiHong88/suneditor/issues/1185
Added this issue to the 3.0.0 milestone.

Is there any progress on 3.0.0 milestone? What do you expect in terms of release?


Technical planning

[x] Tools - React? Vite?
[x] Deploy - Firebase?
[ ] Code architecture - files, etc. database?

React, TypeScript, Vite, Firebase...


pop_named() sometimes generates an exception

This simple code block generates an exception when I try to pop "Columbus Day" (last line):
from datetime import date

import holidays

us_holidays = holidays.CountryHoliday('US', prov=None, state='CA', years=2022)

print(us_holidays.get('2022-10-10').format("%s"))

us_holidays.pop_named("Columbus Day")

The exception is dictionary changed size during iteration, and it seems to be related to popping elements out of a dictionary while you're iterating through it: https://stackoverflow.com/questions/11941817/how-to-avoid-runtimeerror-dictionary-changed-size-during-iteration-error.  The element to be popped isn't actually removed.
I'm far from a competent Python programmer, but iterating over list(self) seems to solve the problem for me - no exception, and the item is removed properly:
    def get_named(self, name):
        # find all dates matching provided name (accepting partial
        # strings too, case insensitive), returning them in a list
        return [key for key in list(self) if name.lower() in self[key].lower()]

Thank you @rccoleman + @mborsetti ! 👍 Merged on beta.


fix(bug): fix firefox input bug

What this PR does 📖

Fixes https://github.com/Satellite-im/UplinkWeb/issues/367

Firefox
Before
https://github.com/user-attachments/assets/91144b55-648e-450e-b99e-2ae60184ce3a
After
https://github.com/user-attachments/assets/c7030ef1-74d7-41e2-9127-1fd28dfb3c1e
How to test:

This needs to be tested on all input fields, both chrome and firefox both input manually and copy paste.

Seems there is a bug with the status input when creating account, playwright is complaining about it:
https://satellite-im.github.io/test-reports/739/#suites/d00f6d21a65e7aed7382e811176e8822/b1ad9f27b839e71d/

thank you 😂
will fix it

Just needs re-test after new code, otherwise should be good to go

thank you 🎉


Doctrine\DBAL\Driver\PDO\Exception could not find driver

Hi Chris, hope all is well.  I seem to have this error message when using the pkg.  I installed the app on a linux server and run composer update which comes up with the above error message. Can you  please advise as to the solution if you have encountered something similar
composer.json require
php : ^7.3 | ^8.0
doctrine/dbal:  ^2.1
flar3/lodata: 61.11
framework : ^8.4

@kristalis Hi I assume you resolved this? Could you say what the problem was? Thanks!

Yes sorry, I enabled these in the php.ini and it worked
[PHP_PDO]
extension=php_pdo.dll
[PHP_PDO_MYSQL]
extension=php_pdo_mysql.dll


Some features use underscores for feature disabling

I tried disabling some features (btw great feature! pun intended), but some weren't working for me. When turning on the debug log I found that the one I wanted to disable was using underscores instead of dashes: linkify_branch_refs. Neither the overview (https://github.com/sindresorhus/refined-github/tree/master/source/features) nor the file (https://github.com/sindresorhus/refined-github/blob/master/source/features/linkify-branch-refs.js) reveal that.
It would be great if this (seems like an edge case) is fixed, or it becomes more discoverable.

Thanks! It was a mistake of https://github.com/sindresorhus/refined-github/pull/877/commits/1d31a5d6cc3137bbb02226c65df286fbecd0d734

Thanks for the fix and the migration!

I wonder now why you were trying to disable linkify_branch_refs

Because I usually double click the branch name on top of a PR to copy it to a CLI. I almost never go to the branch itself, so the link is also not that useful to me.
Maybe a copy button next to them would be helpful to support both cases.

Because I usually double click the branch name on top of a PR to copy it to a CLI. I almost never go to the branch itself, so the link is also not that useful to me.
Maybe a copy button next to them would be helpful to support both cases.

BTW related discussions were held in #77 and #78.

For reference, you can also select text in link with by holding alt (at least on Chrome/macOS)

Ha thanks for the links to the long discussions :) For my and my teams workflow it is very useful to have the branch name. But it's okay. I'm very happy being able to disable the linkify as I never visit those links anyway.
The alt is working for me too, although a bit buggy. I'll see if I can get it into my workflow.


Implement stack covariance matrix as in Wilcox et al.

Calculate the covariance matrix as described in Harry Wilcox's thesis.

Done, I used the numpy corrcoef for normalised covariance, and cov for covariance matrix. I think they're the same as what Wilcox et al. did?


is any licence needed to use the code?

Actual Behaviour

Expected Behaviour

Steps to reproduce it

LogCat for the issue

Screenshots of the issue

Would you like to work on the issue?

There is a MIT license, it was added two years ago.


Any Future Plan to support @monaco-editor/loader

Hi team,
Your Readme clearly state that this library is not working with @monaco-editor/loader or @monaco-editor/react in the FAQ section.
But I just need to ask that, do you have any feature plan to improve monaco-yaml to work with above two?.
My real problem is, I was working on a next-js project with monaco editor and I created the editor by using above mentioned @monaco-editor/loader library. Now I got a requirement to support YAML editing feature and I'm block due to this reason.
Appreciate it if someone can point me out a working example in next-js which use YAML in monaco editor.

I happen to be working with Monaco editor inside a Next.js project right now.
First, here’s a solution to working with monaco-editor without having to use @monaco-editor/react:
next.config.js
export default {
  webpack(config, options) {
    // https://dev.to/swyx/how-to-add-monaco-editor-to-a-next-js-app-ha3
    if (!options.isServer) {
      const specificRules = config.module.rules.find((rule) => rule.oneOf).oneOf
      const cssRule = specificRules.find(
        (rule) =>
          rule.test &&
          Array.isArray(rule.use) &&
          [].concat(rule.test).some((test) => test.test('anything.css'))
      )
      specificRules.unshift({
        sideEffects: true,
        test: /[\\/]node_modules[\\/]monaco-editor[\\/].+\.css$/,
        use: cssRule.use,
      })
    }
  }
}

components/MonacoEditor/index.ts
import dynamic from 'next/dynamic'

export const MonacoEditor = dynamic(() => import('./MonacoEditor'), { ssr: false })

components/MonacoEditor/MonacoEditor.tsx
import * as monaco from 'monaco-editor'
import { setDiagnosticsOptions } from 'monaco-yaml';

export default function MonacoEditor() {
  return (
    <div
      ref={(node) => {
        const editor = monaco.editor.create(node, options)
      }}
    />
  )
}

Just make sure everything related to Monaco is handled by this next/dynamic import.

As for support for @monaco-react/loader:I currently don’t have plans to actively work on it, but I am open to a pull request. I have recently created monaco-tailwindcss and monaco-unified which support passing in the monaco-editor module when configuring. monaco-yaml could work the same way.

@remcohaszing I highly appreciate your quick response and the support. My requirement is working now.
Note: for those who are referring to this in future, you will have to return the webpack config in next.config.js file.
export default {
  webpack(config, options) {
    // https://dev.to/swyx/how-to-add-monaco-editor-to-a-next-js-app-ha3
    if (!options.isServer) {
      const specificRules = config.module.rules.find((rule) => rule.oneOf).oneOf
      const cssRule = specificRules.find(
        (rule) =>
          rule.test &&
          Array.isArray(rule.use) &&
          [].concat(rule.test).some((test) => test.test('anything.css'))
      )
      specificRules.unshift({
        sideEffects: true,
        test: /[\\/]node_modules[\\/]monaco-editor[\\/].+\.css$/,
        use: cssRule.use,
      })
    }
    return config;
  }
}


Internal load balancer does not allow internet access  to servers running as clustered SQL servers

I followed the document on how to Configure a SQL Server failover cluster instance on Azure virtual machines
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/sql/virtual-machines-windows-portal-sql-create-failover-cluster#step-2-configure-the-windows-server-failover-cluster-wsfc-with-s2d
I set the 2 servers in azure up with out a public IP address and after completing all of the steps in the document the SQL failover cluster works perfectly
The only issue is that the internal load balancer does not allow internet access so the servers are not capable of activating windows..
Windows was activated when I set up the server but because the servers can not access the Azure KMS server to re-activate they now show as not activated
==========================================

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

ID: af26cb8a-f7a0-291b-7ac3-e38f679df4ba
Version Independent ID: dd04d12d-05cb-6fb4-4a2c-ef85ebb15d0e
Content: SQL Server FCI - Azure Virtual Machines - Azure SQL Server VM
Content Source: articles/virtual-machines/windows/sql/virtual-machines-windows-portal-sql-create-failover-cluster.md
Service: virtual-machines-sql
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

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

@MikeRayMSFT Can you please add your comments.

@ted185 I'm working with our engineers to review your request. Thank you for the submission.

After adding a public ip address to the 2 sql server I am able to now access the internet
I had to

remove the load balancing rules
delete the 2 servers from the back end pool of the load balancer
shut down the server and associate the public ip addresses to the 2 sql servers
add the servers back in to the back end pool for the load balancer
boot the servers back up

I believe that when creating the sql servers that will be put behind an internal standard load balancer you must create them with a public ip address and you are all set
this is not necessary for any other server I have on my internal network because they are not behind as internal standard load balancer

@ted185 Thanks for confirming that the issue is resolved now.
Your comments will remain on this page and content team will enhance the document calling out the Public IP address part you have mentioned in your comment.
We will now close this issue.

I believe the Public IP address solutions is incomplete and/or should not be recommended.  An external load balancer with a Public IP and an outbound NAT rule is how we resolved this and that is much more palatable than putting Public IP Addresses on production SQL Servers.


fix: endpoint, not base IRI

Could we get an update here? This should be easy to merge & the current text is not correct.


openshift: improve compatibility

This pull request includes updates for Openshift compatibility.
Tested:

Openshift 3.11
Kubernetes v1.10.11

Includes:

update container image permissions and user handling
update helm chart deployment docs
add security context related docs to Helm chart values

Container images: https://hub.docker.com/r/blazdivjak/docker-vernemq

@blazdivjak Sorry for the delay, but thanks for the contribution. To be honest we don't know a lot about Openshift, maybe others from the community could step in to review this.
cc @drf @francois-travais @codeadict

I like the changes you did. I did not use helm or the deployments but your changes to the Dockerfiles work perfectly and deploy on Openshift without root or any other extra-permissions.
I will not add a review and leave this to the contributers mentioned above.

Sorry for being late to the party. However, the changes are sound to me, but I wonder whether this has been tested outside OpenShift given the various permission changes (apparently shouldn't be an issue, but better safe than sorry). So thumbs up, especially if somebody tested this on standard k8s :)

I did test it on k8s as well. Not extensively however.

@greggiles thanks a lot for your testing and vote to merge this.
@dergraf merge?
@ertanden given your experience, could you give a comment on this?

@ioolkos this looks good to me.
Maybe a small comment, instead of adding the new file uid_entrypoint, we could also put the content of it at the beginning of the start_vernemq script. It would simplify things I guess.


Support for DataSource

The mock and no-op drivers shall support the DataSource interface in addition to the Driver interface.
Other interfaces in javax.sql, as well as ConnectionBuilder, ShardingKey, and ShardingKeyBuilder in java.sql  shall be added to the wraps and the no-ops.

This seems to be much work with little benefit.
I might change my mind about this later, but for now I will close this issue.


Bootstrap italia

It would be great to build the theme also for bootstrap-italia.. Is it scheduled?

@danimog we should ask to @astagi 😉


variable t must be of type time_t

time_t can be defined as signed int on some platforms (e.g. my 32-bit ubuntu 14.04 with libc 2.19 and gcc 4.8.4), generating compilation error when being compared with uint32_t.
It would be more correct to define t the same type as tv.tv_sec, i.e. time_t.

Thanks!


Exponential unfolding behaviour in Meta.isDefEq and Meta.whnf reduction

Prerequisites
Please put an X between the brackets as you perform the following steps:

[X] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[X] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[X] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
In the following reduction with a definition of mergeWith that has been adapted from mergeWith from Batteries.RBSet onto List, there seems to an exponential number of reductions with respects to the size of the first or second list.
def _root_.List.mergeWith {α β} [BEq α] (t₁ t₂ : List (α × β)) : List (α × β) :=
  t₂.foldl (init := t₁) fun t₁ a₂ =>
    t₁.cons <| match t₁.find? (a₂.1 == ·.1) with | some a₁ => a₁ | none => a₂

/--
info: [reduction] unfolded declarations (max: 32752, num: 1):
  [reduction] Nat.rec ↦ 32752[reduction] unfolded reducible declarations (max: 32752, num: 1):
  [reduction] Nat.casesOn ↦ 32752use `set_option diagnostics.threshold <num>` to control threshold for reporting counters
-/
#guard_msgs(info) in
set_option diagnostics true in
set_option diagnostics.threshold 32000 in
example (T : Nat) : [(2, T)].mergeWith (List.replicate 13 (1, 0))
                    = (List.replicate 13 (1, 0)).concat (2, T) := by rfl

Increasing the size of the list to 14 in List.replicate will double the number of reductions of Nat.rec to ~64000 and it quickly becomes intractible.  I feel like it's due to the same expression being reduced multiple times and not being cached/shared properly during the reduction.
One thing to note is that this is using a free variable T : Nat to force use of Meta.isDefEq instead of Kernel.isDefEq, because the latter doesn't seem to suffer from the same issue.
Context
We are using merging of Batteries.RBMap to build up a circuit from a description, and we were having trouble proving definitional equality using eq_refl.  A link to a relevant discussion is here: Link to Zulip conversion.  We then took the advice and modified eq_refl to use Kernel.isDefEq even with the presence of free variables, which seems to work well.
However, we are facing more issues because we are typing terms using elements of the map, and this leads to unwanted (and slow) reduction everywhere (with Meta.whnf this time), including in elaboration and delaboration.
Steps to Reproduce
Link to example

Open link to example, and see that currently Nat.rec is unfolded ~32000 times.
Change 13 to 14 in List.replicate and see that the number of unfolds doubles.

Expected behavior: I would expect a polynomial number of unfolds as the size of the list increases.
Actual behavior: Instead, the number of unfolds seems to be exponential in the size of the lists.
Versions
4.12.0-nightly-2024-10-02
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

Not a complete analysis, but I got curious, and there is definitely a lack of sharing: The foldl is duplicating a bunch of term that never get forced, in particular the t1, and it seems that this causes repeated evaluation.
The problem disappears if one makes the foldl explicit, and then hoists out the match:
def _root_.List.mergeWith {α β} [BEq α] (t₁ t₂ : List (α × β)) : List (α × β) := go t₁ t₂
  where
    go t₁
    | [] => t₁
    | a₂::t₂ =>
      let k x := go (t₁.cons x) t₂
      match t₁.find? (a₂.1 == ·.1) with
      | some a₁ => k a₁
      | none => k a₂


No command 'vue' found - Ubuntu 16.04

Step by step
1.- Tech: node v7.0.0, npm v3.10.8.
2.- Installation $sudo npm i -g vue-cli
/usr/local/bin/bin/vue -> /usr/local/bin/lib/node_modules/vue-cli/bin/vue
/usr/local/bin/bin/vue-list -> /usr/local/bin/lib/node_modules/vue-cli/bin/vue-list
/usr/local/bin/bin/vue-init -> /usr/local/bin/lib/node_modules/vue-cli/bin/vue-init
/usr/local/bin/lib
└──<EMAIL_ADDRESS>3.- Create project
$vue init webpack-simple vue-cli
4.- Error:
No command 'vue' found, did you mean:
Command 'vpe' from package 'texlive-latex-extra' (universe)
vue: command not found
Any suggestions?
Regards.

In your enviroment, I seem that vue-cli don't through a path.
Did you check vue-cli with which command?

@kazupon thanks for replying, when I type which vue-cli does not display information on the terminal, just like with vue-cli.

I hope you will try to check other CLI tools (browserity etc ) which implemented with node.js

I was able to solve the problem configuring npm of global packages in my home folder.
https://docs.npmjs.com/getting-started/fixing-npm-permissions#option-2-change-npms-default-directory-to-another-directory
Please close.