Post jenkins build output (console output) to slack

Jenkins generate some outputs for each build, which is shown in "Console output" panel for each build.
Can I make the slack-plugin post that output to slack? Is this feature already available?
I tried checking the box "  Include Test Summary" in the slack notification configuration. However, what I got on slack is "No Tests found."

The Include Test Summary box is for including the results of JUnit tests.  To see what that is supposed to look like when properly configured see pull request #25 because it has a screenshot.
Regarding including actual console output being posted to Slack, there is not currently a way for this to happen.  I personally wouldn't find this useful but I don't use Jenkins nor the slack plugin (I only maintain pull requests and releases).
If this is a feature you truly want you're welcome to contribute it.  I don't actively develop this project.

I concur with Sam. Our builds generate a lot of output (175k+) and this would have to be attached. I think it's a bit  far from the notification purpose of the plugin and possibly a security issue too. The link in the slack post allows you to check the details of the build easily.

I have a related issue. When Jenkins runs my python unit tests, the Slack message on failure is always "No Tests found" even though the console output shows Jenkins did find and run tests:
....F.F.....................FF.......
======================================================================
FAIL: test_xxx (xxx.module)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "xxx", line 1201, in patched
    return func(*args, **keywargs)
  File "xxx", line 95, in test_xxx
    assert rv.status_code == 200
AssertionError:
-------------------- >> begin captured logging << --------------------
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (1): localhost
--------------------- >> end captured logging << ---------------------

----------------------------------------------------------------------
Ran 37 tests in 0.572s

FAILED (failures=4)
Build step 'Execute shell' marked build as failure
Setting commit status on GitHub for https://api.github.com/repos/xxx
Finished: FAILURE

What do I need to do for the Slack notifications to give me the correct output?

@rogthefrog that's a separate issue (better to be opened in a new issue).  That has to do with how Jenkins collects unit tests.  By default Jenkins only collects JUnit tests from Java.  It's part of the test collection.  I'm not sure if there's a workaround for surfacing python unit tests in Jenkins.

@samrocketman ok, will do. Thank you.

There's no reason a hyperlink to the latest console output cannot be included
http://[jenkins_server]/job/[job-name]/lastBuild/console

You can post the build log as a code snippet (that is actually the best option)!

fabulous idea dejlek, can you point to any instructions on doing so?
thanks!

+1

No idea how, but I am assuming Slack API must give this option! :)

So I know 0 about jenkins other than using it but it sounds like http://javadoc.jenkins-ci.org/hudson/model/Run.html#getLog(int) gives us what most of us want? and the text option for slacks api is https://api.slack.com/docs/attachments#text

Yes, problem is that slack plugin for jenkins does not have this feature.

+1

I was just experiencing this problem too. After seeing here that this option only supports JUnit, I used the following work around.
I'm using xUnit to write my test results to an HTML file. So I just added a formatted link to the test results in the custom message.
<http://jenkins-server:port/job/JobName/ws/path/to/xunit/test/results.html|Test Results>
This puts a Test Results link in the Slack notification which (duh) I can click to see the nicely formatted xUnit test results html content.

@mikesigs thanks. this was super helpful. Where do you see how to format messages like this?  I was trying a regular href tag and it was not working as expected.  Your method worked great.

@ezeeetm Glad it helped you! I found the info in the Slack API docs: https://api.slack.com/docs/formatting

@mikesigs oh brother, now we have Slack Message Markup Language (SMML).  The world already knew HTML!  I guess (now that I look at it) there are some slack specific things that are easier to do this way.  Thanks again!

Really want this feature!
@samrocketman Can I get an invite to slack channel? Thanks.

There's an invite link on the README.  Here's the link https://jenkins-slack-testing-signup.herokuapp.com

I was able to get Slack to include the test results after I added the Publish Junit Test Result report.

@jimthemighty ,  I was wondering if you could tell me how to set up slack to include the test results. Thank you

Including $BUILD_URLconsole in the custom message field does the job for me.

@jeetendra-choudhary the only problem with URL is when Jenkins located inside VPC and have no access without VPN. So you cannot use it on your mobile(you actualy can connect to vpn etc... but not workable at all)

Same here, not for slack but for gerrithub, would be better than link to console, the actual console output of failed tests (or even all the output)

please reopen if still an issue, cleaning up the repo and this looks like it can be worked around easily, feel free to refile with a specific ask

The request was to include the console log in notifications.  (I would add that limiting it to the last N lines of the log would be desirable.)
Calling currentBuild.rawBuild.getLog(int) triggers a sandbox access exception, so that is not an option.
Including a link to the log is a different use case, not a workaround.
Please reopen.  Thanks.

Why can't you just use the ${BUILD_LOG} jenkins macro in the custom message field?

Why can't you just use the ${BUILD_LOG} jenkins macro in the custom message field?

@dsteinmo
$BUILD_LOG is not a variable set by jenkins, it's only available in the context of the email-ext plugin

To get test summary on slack using protractor, you can use jasmine-reporters
var reporters = require('jasmine-reporters');
onPrepare: function () {
jasmine.getEnv().addReporter(new reporters.JUnitXmlReporter('outputdir/', true, true));
}
In jenkinsn, select "Publish JUnit test result report" from Post Build Actions and specify path to xml. In this case, *.xml
after test execution. you will see an informative test summary like this one

Also, don't forget to check "include test summary" one of the options from slack notifications plugin

To get test summary on slack using protractor, you can use jasmine-reporters
var reporters = require('jasmine-reporters');
onPrepare: function () {
jasmine.getEnv().addReporter(new reporters.JUnitXmlReporter('outputdir/', true, true));
}
In jenkinsn, select "Publish JUnit test result report" from Post Build Actions and specify path to xml. In this case, *.xml
after test execution. you will see an informative test summary like this one

Also, don't forget to check "include test summary" one of the options from slack notifications plugin

Hi All,
Same question
We have the pipeline that sending the Jenkins to build notification to slack that success or failure.
But now we need the whole console output in text format to send the slack channel. Is there any way? Please anyone can help me here.
Thank you.

Hi All,
Same question
We have the pipeline that sending the Jenkins to build notification to slack that success or failure.
But now we need the whole console output in text format to send the slack channel. Is there any way? Please anyone can help me here.
Thank you.

What test framework are you using? You should probably check JUnit XML reporter. It sends a test summary to the slack channel after test execution.


UICollectionViewController implementation

I'm porting some other code over to ProMotion, and I'm using a UICollectionViewController to display a grid of images. I was thinking about implementing something like PM::CollectionScreen in ProMotion, but want to make sure if that's something you're interested in?
If yes, I will start sketching out a solution in here and get it implemented once it looks ok to you :)

This is being addressed by #717

Will close when we merge.

@jamonholmgren - Looks like this was merged in (https://github.com/infinitered/ProMotion/pull/717) and can be closed.

thanks @jcarbo!


Windows special font gap between white space and other characters

VSCode Version: Any
OS Version: Windows

Steps to Reproduce:

Install ubuntu subsystem, zsh and using theme agnoster
Install powerline fonts to C:\Windows\Fonts
Configure VSCode to use zsh and powerline fonts:

"terminal.integrated.shell.windows": "C:\\Windows\\System32\\bash.exe",
"terminal.integrated.fontFamily": "'inconsolata for powerline'",

Ctrl + ` to see error

Does this issue occur when all extensions are disabled?: Yes

If I use powerlevel9k, things are worse, I will get this:

I think I have to reinstall Windows 😂 or switch to macOS/Linux

Any one could help, please? Thanks in advance.

Using cmder with special character λ, I will get this gap too. But using the external terminal, everything is okay.

Don't know why but updating to Windows Insider Program build solves my problem.
My current Windows version is 1903:

The terminal works perfectly now:


Russian slug  and Multibyte String

Generate russian slug not correct if using strtolower.
Better using mb_strtolower if that function exists.

@wowkaster Thanks for creating this pull request. I made a small change just to remove the check for mb_strtolower since mbstring is a required PHP extension for BookStack. Otherwise all good.


Cursor is not immediately at correct place after expanding snippet

Hi!
This is a result of my investigation into echasnovski/mini.nvim#484. In short my original problem: expanding snippet in a way which triggers scroll animation leads to some text change and exiting Select mode. I found it to be a two-part problem.

Right after snippet expansion cursor is not at the place it needs to be. I verified this by adding these lines here:
	_G.info = {
		cur_line = vim.fn.getline("."),
		cur_pos = vim.api.nvim_win_get_cursor(0),
	}

The core reason seems to be that any_select() utility function does not feeds keys immediately, as it uses default n flag. I tried using several variations with x flag, which did place cursor immediately but resulted into some weird Select mode behavior (for example, not exiting it right away after first <Esc> but only after second <Esc>).
My investigation stopped here.

Animating scroll when Select mode is active seems to reset it. I have addressed this in 'mini.animate'.

Hey!
Yeah, that's a bit of a rough area, since due to missing api feedkeys has to be used to set the visual selection.
I had tried to feedkeys to execute immediately in the past, but without success so far, so not that confident in getting that to work :sweat_smile:
I skimmed over the issue you linked, but couldn't really tell what sequence of events is causing the issues. Could you go into that a bit?

Hey! Yeah, that's a bit of a rough area, since due to missing api feedkeys has to be used to set the visual selection. I had tried to feedkeys to execute immediately in the past, but without success so far, so not that confident in getting that to work 😅

I feel you. This is indeed tricky.

I skimmed over the issue you linked, but couldn't really tell what sequence of events is causing the issues. Could you go into that a bit?

Sure. Here is the basic rundown of what seems to happen:

User expands snippet in such a way that triggers WinScrolled.
On WinScrolled 'mini.animate' does its dark magic of starting scroll animation. This involves caching current and target window views. At this point in time window view says that cursor is at the wrong column, because it is executed after snippet expansion but before keys from any_select() take effect. This also means that mode is not yet Select.
Keys from any_select() take effect.
'mini.animate' restores initial window view which moves cursor in Select mode and causes unwanted side effects.

Ah okay, that helped, ty!
Are you sure that expanding alone (not counting the subsequent jump into the snippet) can trigger WinScrolled? Only nvim_buf_set_text is used while expanding (at least I'm pretty sure that's all we do in terms of cursor/buffer-manipulations)
Maybe the successive cursor_set_keys in any_select are at fault? The first one causes one WinScrolled, and the second one causes another while animate is still animating the first one?
And if they are at fault, I don't think (or can't tell how, at least :D ) running feedkeys immediately would solve the issue

Are you sure that expanding alone (not counting the subsequent jump into the snippet) can trigger WinScrolled? Only nvim_buf_set_text is used while expanding (at least I'm pretty sure that's all we do in terms of cursor/buffer-manipulations)

Yes, pretty sure. A WinScrolled event is basically triggered when top window line has changed. There are situations when setting text does that.
Here is an init file to reproduce:
vim.o.scrolloff = 10

vim.api.nvim_buf_set_lines(0, 0, -1, false, vim.split(string.rep('a', 1000), ''))
vim.api.nvim_win_set_cursor(0, { vim.api.nvim_buf_line_count(0), 0 })
vim.cmd('normal! zb')

local add_text_after_cursor = function()
  local cur_pos = vim.api.nvim_win_get_cursor(0)

  local text = { 'This', 'is', 'big', 'snippet' }
  vim.api.nvim_buf_set_text(0, cur_pos[1] - 1, cur_pos[2] + 1, cur_pos[1] - 1, cur_pos[2] + 1, text)
end

vim.keymap.set('n', '<M-m>', add_text_after_cursor)

_G.n_winscrolled = 0
vim.cmd('au WinScrolled * lua _G.n_winscrolled = _G.n_winscrolled + 1')

Steps:

Run nvim -u <path-to-init-file>.
Execute :=n_winscrolled to ensure that it is 0.
Press <M-m>.
Execute :=n_winscrolled again to see that it is 1.

Ahh okay, yup.
And running all luasnip-cursor-manipulations at once would fix this because mini.animate uses defer (or something similar) to enque its scroll?

Ahh okay, yup. And running all luasnip-cursor-manipulations at once would fix this because mini.animate uses defer (or something similar) to enque its scroll?

Right now it would "fix" it because it would immediately enter Select mode, i.e. before 'mini.animate' decides to start scroll animation. If cursor is dealt with separately from starting Select mode, I'll see what I kind do further.

Okay I've been trying for some time now, but I can't get past the SELECT-but-not-SELECT-mode, where everything seems like SELECT, until you start typing and the text just.. moves :/
What if luasnip sets some kind of flag to indicate that there is currently an expansion going on, and animate can act accordingly?

Okay I've been trying for some time now, but I can't get past the SELECT-but-not-SELECT-mode, where everything seems like SELECT, until you start typing and the text just.. moves :/

Yeah, it does seem very strange indeed. I am borderline convinced that this is some kind of issue in Neovim (coming from Vim, probably) itself. Maybe some minimal reproducible example will be enough for an issue there?

What if luasnip sets some kind of flag to indicate that there is currently an expansion going on, and animate can act accordingly?

I'd have to think about it, but generally it needs to be a very strong case for adding special conditions for some external plugins (outside of 'mini.nvim'). The use case at hand doesn't look like it.

TBH, something general like animate.disable()/animate.reenable(), which just sets some flag that prevents the animation-black-magic from initializing in WinScrolled sounds reasonable, I'd be open to adding the corresponding calls in our selection-routine.
That said, the issue doesn't seem too bad, one jump forward/back should bring everything back into order.

Maybe some minimal reproducible example will be enough for an issue there?

Oh yeah, that would be great to have. Unfortunately, recording the keys and feeding them without the rest of luasnip in the background is working fine, so there's something more subtle going on.
I'm not all that motivated to get into that, if someone else wants to, be my guest :D

TBH, something general like animate.disable()/animate.reenable(), which just sets some flag that prevents the animation-black-magic from initializing in WinScrolled sounds reasonable, I'd be open to adding the corresponding calls in our selection-routine.

That is already possible using buffer-local vim.b.minianimate_disable variable.
I mean, adding these lines here works for this particular use case, but I wouldn't call this a robust solution:
		local buf_id = vim.api.nvim_get_current_buf()
		local minianimate_disable_cache = vim.b[buf_id].minianimate_disable
		vim.b[buf_id].minianimate_disable = true
		vim.schedule(function()
			vim.b[buf_id].minianimate_disable = minianimate_disable_cache
		end)

Oh yeah, that would be great to have. Unfortunately, recording the keys and feeding them without the rest of luasnip in the background is working fine, so there's something more subtle going on.

That is peculiar. My initial guess would be that it is related to some other feedkeys() usage.

That is already possible using buffer-local vim.b.minianimate_disable variable.

Ah, nice!
True, not robust, but maybe something to stick in ones personal config as a bandaid:
local ls = require("luasnip")
local orig_se = ls.snip_expand
ls.snip_expand = function(...)
    local buf_id = vim.api.nvim_get_current_buf()
    local minianimate_disable_cache = vim.b[buf_id].minianimate_disable
    vim.b[buf_id].minianimate_disable = true
    vim.schedule(function()
        vim.b[buf_id].minianimate_disable = minianimate_disable_cache
    end)

    orig_se(...)
end

Might work, could you try it?

Might work, could you try it?

I tried, and it doesn't universally work. I use expand_or_jump in my workflow which uses local snip_expand() function, so overriding exported one in the table won't have effect.


Bug: Files not opening for me

Description
Hi, I am unable to launch Files either from Start or via my Win + E shortcut. I tried repair option from app settings, no luck.
Please let me know how I can manually update me registry for Win + E combination to launch my default windows explorer.
Steps To Reproduce

Go to Windows Start, Files - Open
or press Win + E shortcut

Files Version
<IP_ADDRESS>
Windows Version
cannot access as Files not opening
User ID
cannot access as Files not opening
Log File
cannot access as Files not opening

Thanks for the report, can you type Event Viewer into the Windows start menu search > Click Windows logs > Click Application > Screenshot any errors that have Files.exe shown on the first line
You can revert by open this https://raw.githubusercontent.com/files-community/Files/main/src/Files.App/Assets/FilesOpenDialog/UnsetFilesAsDefault.reg > Right click anywhere > Save as > Set "Save as type" (Around the bottom) to all files > Remove .txt from the end of file name > Run the file

Thank you for sharing the .reg file.
I am not able to spot any such error in Event viewer, however I am able to get hold of debug.log file from below location
C:\Users\alpha\AppData\Local\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState
Will that help?

It probably won't have the error for this type of crash, but please uploaded it anyway.

please find the log
debug.log
One more thing from the log file: the excel file in below location 'Books.xlsx' has gone missing, I can't find it in recycle bin even, looks like Files app has deleted it, any way to recover it?
An error occurred trying to start process 'C:\Users\alpha\OneDrive_Troy\OneDrive\Personal\Books.xlsx' with working directory 'C:\WINDOWS\System32'. The system cannot find the file specified.

Has it actually been deleted? If it on OneDrive then you can check its recycle bin.

Yeah unfortunately, I don't keep my OneDrive running in the background, I just sync it once a quarter, so all recent changes made to the spreadsheet was saved locally on my laptop. I found the same file in the recycle bin of OneDrive but that was from the month of September so obviously missing any updates I made to the file after that month 😞

can you type Event Viewer into the Windows start menu search > Click Windows logs > Click Application > Screenshot any errors that have Files.exe shown on the first line

I tried this again today morning, before opening Event Viewer I tried launching Files which didn't open, in Event Viewer under today's date, I could only see Level type Information and no Error, still I manually scrolled through all Information line items but couldn't find anything related to Files, strange no?
Forgot to mention one thing yesterday, only the Books.xlsx file went missing from C:\Users\alpha\OneDrive_Troy\OneDrive\Personal
Other excel, word, text files were still present. In fact I tried opening Excel app to check if I can recover Books.xlsx from recent files but got message saying Excel.exe program couldn't be find when I clicked Excel under Windows Start menu apps, I had to then reinstall Excel again (Word, PowerPoint were fine), weird!

Do you see a splash screen for Files or does it not even start?

If it occurred before you were using Files, I'm not sure what could have caused the file to go missing.

Can you please share more details about what happened? Did the file go missing while you were using Files?

please find the log debug.log
One more thing from the log file: the excel file in below location 'Books.xlsx' has gone missing, I can't find it in recycle bin even, looks like Files app has deleted it, any way to recover it?
An error occurred trying to start process 'C:\Users\alpha\OneDrive_Troy\OneDrive\Personal\Books.xlsx' with working directory 'C:\WINDOWS\System32'. The system cannot find the file specified.

I don't keep my OneDrive running in the background, I just sync it once a quarter, so all recent changes made to the spreadsheet was saved locally on my laptop. I found the same file in the recycle bin of OneDrive but that was from the month of September so obviously missing any updates I made to the file after that month 😞

@yaira2 please see my previous messages, quoted here for ease of reference! The file went missing when I was unable to launch the Files app.

No the splash screen didn't appear, nothing happened when I tried to launch Files.

Did it work previously before you opened this issue?

Of course, I only logged this issue when it stopped working!

It seems the system had trouble finding the file when you tried to open it. Debugging this can be quite challenging because the task of opening files is handed over to Windows, with Files simply acting as the go-between. I know how frustrating it is not having a clear answer. However, with so many great minds out there, someone might have a clue or a way to replicate the issue.

I get that, feel free to close this ticket or keep open as you wish 🙂

I'll do that for now, but if anyone else has an idea, please share 🙂


Mutualize ESLint

Problem
We use Eslint in standalone_site, lib_classroom, lib_common, lib_components, lib_tests; they have almost the same configs and rules except for import/order.
When we want to add a new rule, we have to do it in every packages, it is redundant.
Solution
Mutualise all the redundant config in marsha/src/frontend by integrating Eslint cascading configuration, thanks to root: true: https://eslint.org/docs/latest/user-guide/configuring/configuration-files

I suggest you to take a look at the new configuration file they introduce to not doing something already deprecated:
https://eslint.org/docs/latest/user-guide/configuring/configuration-files-new
https://eslint.org/blog/2022/08/new-config-system-part-1/
https://eslint.org/blog/2022/08/new-config-system-part-2/
https://eslint.org/blog/2022/08/new-config-system-part-3/

This config doesn't seem well supported yet.
The other eslint plugin must have a special config as well (without extends).
In the exemple below  'eslint-plugin-prettier' generate an error because of the use of the extends keyword.
Error: Unexpected key "extends" found.
import pluginPrettier from 'eslint-plugin-prettier';
import typescriptParser from '@typescript-eslint/parser';

const config = [
  pluginPrettier.configs.recommended,
  {
    files: ['src/**/*.ts', 'src/**/*.tsx'],
    languageOptions: {
      parser: typescriptParser,
      parserOptions: {
        ecmaVersion: 'latest', // Allows for the parsing of modern ECMAScript features
        sourceType: 'module', // Allows for the use of imports
        ecmaFeatures: {
          jsx: true,
        },
        tsconfigRootDir: '.',
        project: ['./tsconfig.json'],
      },
    },
  },
];

export default config;


[BUG] Menu translation only works after tab change

txAdmin/FXServer versions:
v4.13.3
Describe the bug
Whenever you open the menu, it will always be english, until you press tab.
This is surely related to serverCtx sync that changed a few versions ago, since it is already in the dependencies of menu\src\components\MainPage\MainPageList.tsx.

Cannot reproduce bug on local environment. When updating Locale from settings page in Web UI, the Menu and subsequent locale phrases are immediately updated upon receival of "setServerCtx" message event.

Oh that's 100% a game thing, related to serverCtx updates.
Since forever reviewing the serverctx thing has been on my list - which will happen with the review of the two debug variables, and prints.

Oh that's 100% a game thing, related to serverCtx updates.

hm weird, serverCtx updates through both GlobalState bag and through Get-Set serverCtx event fallback was functioning fine when I was testing.

Confirmed that this has been fixed.
IDK which version fixed it though.


[SDK] Improve Vote contract tests

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on updating tests for the deploy-vote and vote functionalities, including renaming and reorganizing imports, and modifying test cases to ensure accurate contract interactions and assertions.
Detailed summary

Removed unused isAddress import.
Updated test case names for clarity.
Reorganized import paths for consistency.
Modified test assertions to use voteContract instead of contract.
Removed redundant test cases related to proposal existence.
Ensured accurate proposal creation and existence checks.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5156  👈
#5155
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kien-ngo and the rest of your teammates on  Graphite


Visualizer: option to select start rule has no effect

I'm noticing when using the visualizer that the option to select the start rule doesn't seem to have an affect.  For now I'm just manually moving the target rules to the top of the grammar --but this feature would be really nice to have work.
I've got a test suite with a bunch of test samples for constituent parts of my grammar -- I'd love to generate an index.html from my codebase such that one could easily view and visualize the parse for the different test samples!
Amazing project btw -- went from knowing nothing to solving my problem in less than a day with Ohm!

Duplicate of #45


Windows Speech Input Provider generates Assertion failed error with non XR SDK profiles

Describe the bug
When using the in-editor player to run example scenes, after the 3rd session the Windows Speech Input Provider generates an Assertion failed error every time the in-editor player is used. The error is generated on every example scene in the MRTK after it starts showing up. This issue happens without the XR SDK installed and with the XR SDK installed when trying to use a non XR SDK profile in the Unity 2020 LTS. The error will not generate for the first 2 in-editor player sessions after restarting the editor when the project has encountered the error, but will start generating again on the 3rd session.
To reproduce
Steps to reproduce the behavior:

Create a new project and import the MRTK
Open any example scene
Choose the 'DefaultMixedRealityToolkitConfigurationProfile" for No XR SDK installed/XR SDK installed or "DefaultOpenXRConfigurationProfile" for OpenXR
Press the Play button to start the in-editor player
See error

Screenshots

Your setup (please complete the following information)

Unity Version 2020.3.0f1
Unity Version 2020.3.1f1
MRTK Version 2.6.1
XR SDK
No XR SDK
OpenXR

Target platform (please complete the following information)

Standalone
UWP
Android

Additional information
This issue happens on OpenXR with the DefaultOpenXRConfigurationProfile.

Since this error is happening in Unity code (UnityEngine.Windows.Speech.PhraseRecognizer.Start()) can you file a Unity bug please? Sounds like they might not be cleaning something up properly.

We should test this on Unity 2019 to see if this is a newly introduced problem.

Potentially be solved by:
#9419

I can confirm that this issue is fixed by #9419 and is not present in 2.7 with XR SDK and OpenXR on Unity 2020.3.7f1


?

Whats the difference between this and https://github.com/AnIdiotsGuide/guidebot-class

Well, guidebot-class is a Class based version of guidebot, The main difference is the use of classes of course, not sure what else to say. If you're wondering what a class is read the MDN page that I linked, Have a good day/night.


created dynamic update feature w/Jquery

Created new ID's for the HTML element, selected them in Jquery and ammended the text content within them. Essentially allowing the user to see the api values.

Great work on this one!


Allow Dense to have any AbstractVector storage

Also, make TensorStorage <: AbstractVector to make the interface more minimal.

Closed by  #135.


Make clean target remove docker images

I only managed to get this working by removing the explicit tag names. This kind of made sense to me for use during the development process but probably is not what we want for release builds. I'll open an issue to discuss.

I'm closing this one for now, and creating a new PR for #102 that is cleaner in how it handles docker version tags and also does the removeImage during clean


libcoap: 4.3.0 -> 4.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libcoap/versions.
meta.description for libcoap is: "A CoAP (RFC 7252) implementation in C"
meta.homepage for libcoap is: "https://github.com/obgm/libcoap"
meta.changelog for libcoap is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.3.1 with grep in /nix/store/3ihm2vdf72qljkl2d06s3krhqp3951ps-libcoap-4.3.1
found 4.3.1 in filename of file in /nix/store/3ihm2vdf72qljkl2d06s3krhqp3951ps-libcoap-4.3.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
libcoap

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

libcoap

Maintainer pings
cc @kmein for testing.

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

1 package built:

libcoap


ORM and DBAL: Erroneous PDOException "There is no active transaction" thrown during commit

Overview
Assume that we rely on the implicit transaction handling of the Doctrine ORM EntityManager. Assume the following simple code
$em = $this->getDoctrine()->getManager();
$rep = $em->getRepository('MyBundle:MyEntity');
$entity = $rep->find( 42 );
$em->remove( $entity );
$em->flush();

and further assume that entity 42 cannot be removed due to violated foreign key constraints.
Expected result
The expected result is that $em->flush() throws some sort of exception that relates to the foreign key constraint, e.g. \Doctrine\DBAL\Exception\ForeignKeyConstraintViolationException.
Actual result
The actual result is a native PHP PDOException with error message "There is no active transaction". The latter is an astonishing exception, because the aforementioned code does not deal with transactions directly.
Investigations and Proposed Solutions
Below I will explain where the bug stems from. The story is a little bit longer, because it involves 1 bug inside of the ORM component and 3 bugs inside of the DBAL component.
Reason 1: ORM component
Snippet from class \Doctrine\ORM\UnitOfWork
331: public function commit($entity = null)
332: {
...
373:     $conn->beginTransaction();
374:
375:     try {
...
403:         // Entity deletions come last and need to be in reverse commit order
404:         if ($this->entityDeletions) {
405:             for ($count = count($commitOrder), $i = $count - 1; $i >= 0 && $this->entityDeletions; --$i) {
406:                 $this->executeDeletions($commitOrder[$i]);
407:             }
408:         }
409:
410:         $conn->commit();
411:     } catch (Exception $e) {
412:         $this->em->close();
413:         $conn->rollback();
414:
415:         $this->afterTransactionRolledBack();
416:
417:         throw $e;
418:     }
...
440: }

At line 410 while calling  $conn->commit() the following exception is thrown:
$e = {PDOException} [9]
  message = "SQLSTATE[23503]: Foreign key violation: 7 ERROR:  update or delete on table "..." violates foreign key constraint "..." on table "..." DETAIL:  Key (id)=(42) is still referenced from table "..."."
  *Exception*string = ""
  code = "23503"
  file = "/srv/www/matthias.nagel/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php"
  line = 1216
  *Exception*trace = {array} [9]
  *Exception*previous = null
  errorInfo = {array} [3]
    0 = "23503"
    1 = "7"
    2 = "ERROR:  update or delete on table "..." violates foreign key constraint "..." on table "..." DETAIL:  Key (id)=(42) is still referenced from table "..."."
  xdebug_message = "..."

This is correct, because the foreign key constraint is evaluated during the commitment. However, the type of the exception should probably be a \Doctrine\DBAL\Exception\ForeignKeyException. We come to this point later, because this must be fixed elsewhere.
The problem here is that at line 413 inside of the catch-block $conn->rollback() tries to rollback a transaction that has already been terminated by $conn->commit(). It seems that this code does not take into account that $conn->commit() can fail with an exception after the transaction has already been terminated. In this case the transaction is automatically rolled back at server side and must not be rolled back a second time.
At the moment at line line 413 $conn->rollback() throws an additional exception and the original exception is lost, because line 417 will never be executed:
$e = {PDOException} [9]
  message ="There is no active transaction"
  *Exception*string = ""
  code = "0"
  file = "/srv/www/matthias.nagel/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php"
  line = 1278
  *Exception*trace = {array} [9]
  *Exception*previous = null
  errorInfo = null
  xdebug_message = "... left out ..."

Proposed solution (part 1 of 4): Check inside of the exeption handler if transaction is still active and only roll back in this case. Replace line 413 by: if( $conn->isTransactionActive() ) $conn->rollback()
However, even at the moment the exception thrown by rollback should be \Doctrine\DBAL\ConnectionException::noActiveTransaction() and not a plain PDOException. This dicrepancy is due to another bug. We come to this point later, because this must be fixed elsewhere.
Reason 2: DBAL component
The both remaining bugs that have already been mentioned are located at class \Doctrine\DBAL\Connection
1164: public function beginTransaction()
1165: {
...
1168:     ++$this->_transactionNestingLevel;
...
1172:     if ($this->_transactionNestingLevel == 1) {
...
1176:         $this->_conn->beginTransaction();
...
1180:     } elseif ($this->_nestTransactionsWithSavepoints) {
...
1188:     }
1189: }
...
1199: public function commit()
1200: {
1201:     if ($this->_transactionNestingLevel == 0) {
1202:         throw ConnectionException::noActiveTransaction();
1203:     }
...
1212:     if ($this->_transactionNestingLevel == 1) {
...
1216:         $this->_conn->commit();
...
1220:     } elseif ($this->_nestTransactionsWithSavepoints) {
...
1228:     }
1229:
1230:     --$this->_transactionNestingLevel;
...
1235: }
...
1263: public function rollBack()
1264: {
1265:     if ($this->_transactionNestingLevel == 0) {
1266:         throw ConnectionException::noActiveTransaction();
1267:     }
...
1273:     if ($this->_transactionNestingLevel == 1) {
...
1277:         $this->_transactionNestingLevel = 0;
1278:         $this->_conn->rollback();
...
1287:     } elseif ($this->_nestTransactionsWithSavepoints) {
...
1296:     } else {
...
1299:     }
1300: }

At line 1216 inside the commit-method the method $this->_conn->commit() of the underlying native PHP PDO object is called. Here the original PDOException due to the violated foreign key constraint is thrown. However, the code does not take into account that $this->_conn->commit() might fail with an exception and still terminates the transaction nonetheless. The _transactionNestingLevel should be decremented to 0, because the transaction has been terminated.
Proposed solution (part 2 of 4): Nest $this->_conn->commit() into a try-block, catch the PDOException , inside the catch-block check if the transaction has been terminated and decrement _transactionNestingLevel if necessary.
Proposed solution (part 3 of 4): Moreover, re-throw the PDOException but wrap or convert it into an appropriate child object of \Doctrine\DBAL\DBALException (in this particular case: \Doctrine\DBAL\Exception\ForeignKeyConstraintViolationException).
At the moment the condition at line 1265 inside the rollback-method does not catch the aforementioned case. If the proposed solution is applied the _transactionNestingLevel would be zero and ConnectionException::noActiveTransaction() would be thrown. At the moment _transactionNestingLevel erroneously equals 1 and $this->_conn->rollback() throws a PDOException because there is no active transaction. This problem would be fixed with proposed solution part 2.
However, there are still one more bug in this code. (I am sorry.) In method beginTransaction() at line 1168 the _transactionNestingLevel is increased and the transaction is started at line 1176. The code does not take into account that $this->_conn->beginTransaction() might fail with an exception without starting a transaction. In this case _transactionNestingLevel erroneously equals 1 but should be 0.
Proposed solution (part 4 of 4): Nest $this->_conn->beginTransaction() into a try-block and decrement _transactionNestingLevel again, if the transaction could not be started.

@nagmat84 first of all thanks for the very detailed issue description and investigation. There seem to be some valid points here, although I want to point out two things:

It's not all about PDO so your expectations about PDOExceptions are only true for PDO drivers and other might most certainly react differently.
I think the root problem of the Doctrine code not catching exceptions from commit() and beginTransaction() cals is that the related DBAL connection interface does not define exceptions to be thrown but return true or false instead. This might have been a design mistake or by intention I don't know. The question here is then whether we can redefine the interface description to throw exceptions (which it currently partially does anyways) or if driver implementations have to be adjusted. I personally would prefer the first as error information might get lost otherwise.

That pointed out, I might take a look at the DBAL side issues as soon as I have time.
Thanks for reporting.


[crossport] Update image pull to use tags if specified over the digest

Description:
Update the image pull logic to use tags over digest. This change was
added to release-v0.11 and not on main.
Signed-off-by: Sneha Chhabria<EMAIL_ADDRESS>Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[X]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[X]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Codecov Report

Merging #4221 (6e40f0d) into main (0e3f746) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4221      +/-   ##
==========================================
+ Coverage   69.55%   69.58%   +0.02%
==========================================
  Files         211      211
  Lines       11427    11427
==========================================
+ Hits         7948     7951       +3
+ Misses       3430     3427       -3
  Partials       49       49

Flag
Coverage Δ

unittests
69.58% <ø> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/certificate/rotor/rotor.go
87.50% <0.00%> (+3.12%)
:arrow_up:

pkg/messaging/workqueue.go
100.00% <0.00%> (+10.52%)
:arrow_up:

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 0e3f746...6e40f0d. Read the comment docs.


Implement UI components library

Still in discussion here: #86
Also linked / duplicated by #90

@comoser @rbgoncalves Going with https://ant.design/  as decided in meeting.


Issue with calculate_AHC_transitions and sampling zero paths in dev branch

The following code raises a type error  when calculating the AHC transitions
TypeError: 'Fraction' object is not iterable
df = pd.read_excel("data/medical_dm_modified.xlsx")
falls_st = StagedTree(dataframe=df,sampling_zero_paths=[('Blast','Experienced', 'Easy','Blast', 'extra')])
falls_st.calculate_AHC_transitions()
falls_st.create_figure("st_fig_extra_paths.pdf")

Similarly when adding in missing paths that include all of the outcomes of a floret such as in the following example
Titanic.csv
import pandas as pd
from cegpy.src.cegpy.trees import staged
data = pd.read_csv("datasets/Titanic.csv")
missing_paths=[('Crew', 'Female', 'Child'), ('Crew', 'Female', 'Child', 'No'), ('Crew', 'Female', 'Child', 'Yes')]
st = staged.StagedTree(data,sampling_zero_paths=missing_paths)
st.calculate_AHC_transitions()

@peterrhysstrong I have applied a fix to dev. Could you test it has solved it for you?


RLOS: Distance parameter does not have a dedicated units control (ArcGIS Pro)

The distance parameter on the RLOS (radial line of sight) tool does not have it's own units control. The distance parameter uses the same units set in the Offsets portion of the interface. This is usability issue as I believe users will want to specify different units for the offset and distance.

@nfeuerstein Not sure if this is a bug or an enhancement.

@elinz even there was only one unit configuration that applied to all inputs, it is odd to have it placed in the Offsets section only when it applies to distance as well.  That said, the user can get around with the clunkiness until next release.

@nfeuerstein Sounds good. Thanks.

Verified. This doc has been covered by updating the workflows from issue #279


Bump opa/containerd to latest versions

Let us update to newer containerd v1.7.20:
https://github.com/containerd/containerd/releases/tag/v1.7.20
We will be newer than v1.7.19 where a new module containerd/containerd/api was introduced:
https://github.com/containerd/containerd/releases/tag/v1.7.19
Note that we need to update opa as well as it uses the newer containerd:
https://github.com/open-policy-agent/opa/releases/tag/v0.67.1
This will help with the situation we are in kubernetes where we are stuck with an older version of cadvisor as we have dependency clash that prevents us updating:
https://github.com/kubernetes/kubernetes/pull/126361#issuecomment-2264232155

cc @helsaawy @princepereira @kiashok

cc @princepereira @kiashok


[1st viz] beneficiary demographics facts dimensions structure

Follow-up for #893.
beneficiaryDemographics now returns a {"facts": [...]} data structure.
Definition in #888 is updated.

@MaikNeubert @HaGuesto let me know if this works for you


3d Clock and Polymaths Clock

2 Clocks,
3D Clock:
a simple perspective scaling demo, useful to get an idea about graphics
performance, or as an example for perspective scaling by script.
Uses the Vector font and a simple 3d projection
A Polymaths Clock
a nice every day clock design, efficient on CPU, only redraws parts of the screen
per minute, per hour, per 3 hours.
Features Hour, Minute, graphical representations of  Day, a Day in the Week,
Moon phase, next  day of Moon phase change in the week, and noon/midnight 24 clock line.

Great - thanks!


Fix storage error message to display details of error

When a storage error occurs, the error parameter passed to the
promise's then() method is an object, and therefore when attempted
to be output as a string via a template literal, the error is
displayed as [object Object].
This PR updates the error output to output the entire error object as
JSON. There is no single property that provides enough detail to
display on its own, so JSON was determined to be the best waym to
summarize.

The original issue can be seen in the conversation of https://github.com/microsoft/vscode/issues/6442#issuecomment-493794022

The underlying error object is
{
  "errorCode":"load",
  "moduleId":"vscode-sqlite3",
  "neededBy":["===anonymous3==="],
  "detail":{"code":"MODULE_NOT_FOUND"}
}

I believe this is another spot where cross-compilation to ARM was not considered, and will create a separate PR to update the offending module.

@stevedesmond-ca can you resolve the conflict?

I am not convinced this is a good fix, rather I wonder why the loader is returning a JSON structure and not a real Error. I am expecting an Error object when import fails. @alexandrudima maybe you could chime in when you are back.

@bpasero The loader does not give an Error instance when loading fails, it gives an object of the shape shown in the comment.

Thanks for closing, I forgot to close it... it was fixed upstream...


Add attributes to the time variable in netcdf files

Changed two lines in slocum.py to make sure that the attributes for the time variable are not getting lost or being specified twice. Attributes are from the metadata yaml file.

Thanks @hvdosser


Syntax error in chapter2/softmax.py

chapter2/softmax.py doesn't seem to work with Tensorflow 1.0:
cross_entropy = tf.reduce_mean(tf.nn.softmax_cross_entropy_with_logits(y_pred, y_true))

File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/ops/nn_ops.py", line 1578, in softmax_cross_entropy_with_logits
labels, logits)
File "/usr/local/lib/python3.4/dist-packages/tensorflow/python/ops/nn_ops.py", line 1533, in _ensure_xent_args
"named arguments (labels=..., logits=..., ...)" % name)
ValueError: Only call softmax_cross_entropy_with_logits with named arguments (labels=..., logits=..., ...)

Thank you for the catch! It is now adapted to tf 1.0 (also works with older versions).


Spike: investigate possibility to use version field in for incomming messages in SQS

Spike: investigate possibility to use version field in for incomming messages in SQS

Done, task created


Fixed KinematicBody sliding in bullet

Fixed the slope slide bug for kinematic bodies when using the bullet physics engine. This is my second submission, think I got it right this time. Let me know if I need to fix the submission, I'm new to git.

@JankMultiMediaStudio This PR needs to be updated for it to be merged. It fails the CI check, there are conflicting files, and it has bad git history (there shouldn't be a merge inside of your PR). For more information: https://docs.godotengine.org/en/latest/community/contributing/pr_workflow.html

I saw your comment on Gonkee's video and read your write up. I was wondering if you tested running at walls, because it seems as though your solution would stop kinematic bodies from sliding against walls.

Thanks for bothering with reading my long comment, its cool to know people
actually read that stuff. I hadn't actually tested that, because I was more
so worried about the sliding bug, but I just gave it a quick run and seems
to work well enough. I did notice some jitter in the slide still though,
and I think thats because the returned values that are calculated with the
move and collide function are still off. I couldn't actually figure out how
to correct that while I was working on it. My project notes just say that
it's good enough for now, but the bug is probably in the same area I was
working on if you're inclined to try and track it down.
That being said I'd recommend just rotating your movement vector parallel
to the colliding normal during the collision to be safe.
On Mon, Mar 9, 2020, 10:27 AM Kevin Fischer<EMAIL_ADDRESS>wrote:

I saw your comment on Gonkee's video and read your write up
https://jankmultimediastudio.github.io/godot3.1-kinematic-collision-response-write-up.html.
I was wondering if you tested running at walls, because it seems as though
your solution would stop kinematic bodies from sliding against walls.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/34442?email_source=notifications&email_token=AJ43MV7T6IT55XXQNVJJG2TRGUYJFA5CNFSM4J4SKGO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIHDSY#issuecomment-596668875,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ43MV5EFSXKDF7HFTIJAQLRGUYJFANCNFSM4J4SKGOQ
.

Do you have something to show the sliding problem?

@AndreaCatania , yes, it was shown in this video

Do you have something to show the sliding problem?
Hey Andrea, I had the same problem too:
https://drive.google.com/file/d/1qz65576vF6DGVk_inoPMSRrznvZKqvlQ/view?usp=sharing

@bemyak thanks for the link.
This PR changes how the de-penetration algorithm works and in a way that will create problems in other context; but let me explain better.
The algorithm performs the de-penetration iteratively; even if it knows the exact penetration distance it de-penetrates the body a little bit each iteration. This approach has proven to always de-penetrates in all conditions.
However, the sliding problem that I see in the above video is not related to how the algorithm works and should be fixed on your side.
I suppose that you are using the function move_and_slide https://docs.godotengine.org/en/3.2/classes/class_kinematicbody.html that has the argument stop_on_slope (false by default).
You have to set that parameter to true, and also make sure that the movement argument points perfectly toward down when the character is stationary on a slope.

@bemyak thanks for the link.
This PR changes how the de-penetration algorithm works and in a way that will create problems in other context; but let me explain better.
The algorithm performs the de-penetration iteratively; even if it knows the exact penetration distance it de-penetrates the body a little bit each iteration. This approach has proven to always de-penetrates in all conditions.
However, the sliding problem that I see in the above video is not related to how the algorithm works and should be fixed on your side.
I suppose that you are using the function move_and_slide https://docs.godotengine.org/en/3.2/classes/class_kinematicbody.html that has the argument stop_on_slope (false by default).
You have to set that parameter to true, and also make sure that the movement argument points perfectly toward down when the character is stationary on a slope.

thanks for the quick reply, actually what you see in the video is mad by using move_and_collide

@bemyak thanks for the link.
This PR changes how the de-penetration algorithm works and in a way that will create problems in other context; but let me explain better.
The algorithm performs the de-penetration iteratively; even if it knows the exact penetration distance it de-penetrates the body a little bit each iteration. This approach has proven to always de-penetrates in all conditions.
However, the sliding problem that I see in the above video is not related to how the algorithm works and should be fixed on your side.
I suppose that you are using the function move_and_slide https://docs.godotengine.org/en/3.2/classes/class_kinematicbody.html that has the argument stop_on_slope (false by default).
You have to set that parameter to true, and also make sure that the movement argument points perfectly toward down when the character is stationary on a slope.

It did fix the problem but unfortunately, there's a new problem I'm seeing that my player would just jump when I climb up the slope when I stop moving https://i.imgur.com/eLgJcZP.gif . Is there any fix of it?

Just following up on this. I understand that the core devs are probably a bit busy moving towards 4.0, but is there something that I've missed when it comes to submitting this? I understand a review taking a while, but its been over 3 months for a fix that I would argue that this engine isn't worth using without, which is why I fixed it in the first place (I really like this engine's tool chain).
I don't want to seem impatient, but I'm not familiar with github and how exactly things are done. If someone could get me up to speed, or let me know what further steps to take, I'd appreciate it.

@JankMultiMediaStudio Unfortunately this is just the nature of open source. Lots of people submitting code, not enough senior maintainers to review all the PRs. There are some pull requests that have been open for years, including my several of my own.
The best thing you can do in the meantime is keep your PR updated and working. Right now, it fails the Travis CI check due to style issues. Also, it needs to be rebased.

I already review this, here my answer: https://github.com/godotengine/godot/pull/34442#issuecomment-599034819

@AndreaCatania it doesn't look like you were right in your initial response about what function they were using.
Here's a video URL timestamped to the specific part showing the full understanding of the bug and showing it in action. https://youtu.be/NSNe91lMe_Q?t=276

The depenetration mechanism has to move along the Collison normal to always unstuck.
In the past we had some cases where the kinematic body was not able to slide because move back along the motion vector doesn't produce a depenetration. So the following motion attempt were all failing.
While the doc says that, the move and collide is not a raw query like move_shape, and its depenetration algorithm is tie to move and slide and can't follow what the doc says in order to make it to work.

Wouldn't it be better to not have move_and_slide's logic depend on move_and_collide's logic? I'm just throwing this out there, but couldn't there either be a flag to turn off the depenetration logic for move_and_collide or have the two functions use different code paths?
I'll admit that I haven't done a deep dive to really understand how things work but if the design of the two functions prevent one from meeting its requirements, doesn't that indicate some sort of code smell problem?

Also, purely out of curiosity, why can't a body use the motion vector to get unstuck? This is just me wanting to learn more about this, not trying to disagree in any way.
From a glance, if your xPos was fine at the start, increasing xPos by 1 caused it to get stuck, why wouldn't subtracting 1 from it not unstick it?

I think there is the intention to improve the Kinematic Stuff. I really would like to decouple the Kinematic Body into two separate classes:

KinematicBody (Only raw queries, like the one found in the DirectSpace [In this way is possible to have kinematic bullets or any other thing])
KinematicCharacter (The current Move and slide function, integrated using the Raw query of the KinematicBody)

You can find here more info:
https://github.com/godotengine/godot-proposals/issues/628

Also, purely out of curiosity, why can't a body use the motion vector to get unstuck? This is just me wanting to learn more about this, not trying to disagree in any way.

When the motion vector is perpendicular to the normal direction it wont unstuck. This happens when you hit a wall and the gravity makes you move along Y axis. Or when your motion vector is horizontal and the Normal of the ground is along Y.
In all those cases just move along the motion vector is not enough to unstuck the character.

This PR is not a good idea, because it completely removes the convex sweep test, which is critical for accurate motion collision detection.
Furthermore, instead of trying to fix the problem by changing the de-penetraton algorithm to de-penetrate in the opposite direction of motion, #35945 fixes the sliding issue by accurately detecting how far the body can move without colliding, which completely removes the need for de-penetration.

Closing since this PR has not been updated in a long time and there is a lot of feedback above that hasn't been addressed.

Closing since this PR has not been updated in a long time and there is a lot of feedback above that hasn't been addressed.


RFC: get_channel_types polysemy

I've stumbled on a not-so-nice aspect of the API when working on #7486:

mne.io.pick.get_channel_types (source) takes no input, gets all known channel types as dict keys (values are FIFF constants for kind and coil_type).
mne.io.pick._get_channel_types (source) takes an Info, picks, boolean unique, and boolean whether to restrict to data channels only. Returns a list or set depending on unique.
<Raw/Epochs/Evoked>.get_channel_types (source) method (via the ContainsMixin). Returns list of channel types.

Number 1 is called once in a test, and nowhere else in the codebase. I'd like to propose renaming it from get_channel_types to get_all_channel_types, show_all_channel_types, or get_channel_types_dict. I prefer the first option. This is a public function so it would be an API change w/ deprecation cycle. If acceptable, I could roll it into #7486 or do it in a separate PR.

Number 1 is called once in a test, and nowhere else in the codebase. I'd like to propose renaming it from get_channel_types to get_all_channel_types, show_all_channel_types, or get_channel_types_dict. I prefer the first option.

That was for @cbrnr in #4842 so perhaps he should comment. But I agree that inst.get_channel_types is probably more widely used and that the overlap is problematic, so +1 for renaming mne.io.pick.get_channel_types to get_all_channel_type_constants or something similar (a variant of names you suggested that makes it clearer it's giving a mapping between ch types and their constants, in this case via a dict)

+1 to rename to get_all_channel_type_constants
assuming @cbnr is fine with that

I'm fine with renaming the function, but I'd prefer a slightly shorter name if possible. Also, I haven't checked the return values of mne.io.pick.get_channel_types and mne.io.pick._get_channel_types, but can/should we de-duplicate code by calling the private function inside the public one?

this is now what the PR from Dan does. Except using a short function name

Also, I haven't checked the return values of mne.io.pick.get_channel_types and mne.io.pick._get_channel_types, but can/should we de-duplicate code by calling the private function inside the public one?

Not possible, they do completely different things, which is why I proposed to rename one of them.
What I have done is deduplicate the instance methods and the private function, which were doing (roughly) the same thing.


chore: align dev pg version to prod

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


Add first-timer message to PULL_REQUEST_TEMPLATE.md

Description (required)
This might help when trying to find first-time contributors on Discord for their first-time contributor alert!

Thanks all! We'll try out this new version and see how we like it!


SakulFlee/issue46

Exporting from CLI seems to be broken at the moment.
Might reattempt/reopen at some later point.


Users can inadvertently blow up psake with name conflicts

Psake currently dot-sources a lot of user code into its own internal scope (properties blocks, etc).  This can be dangerous, as users may wind up overwriting the same variable names used by Psake internally.  For example, a quick repro:
Properties {
    $psake = $null
}

task default -depends IBrokeIt

task IBrokeIt {
    Write-Host 'This will never output'
}

<#
Output:

Invoke-psake : The property 'build_success' cannot be found on this object. Verify that the property exists and can be set.
At line:1 char:1
+ Invoke-psake .\testing.ps1
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Invoke-psake], RuntimeException
    + FullyQualifiedErrorId : PropertyNotFound,Invoke-psake
#>

Pester had the same problem prior to v3.0.  It can be fixed, but involves a fair bit of jumping back and forth between script and module scopes (and keeping track of what's visible from each).

I had the same issue. What fixed my issue is closing the console and open it again

Are there any other reasons someone could run into this error: Invoke-psake : The property 'build_success' cannot be found on this object. Verify that the property exists and can be set.? I'm having some trouble figure out what variable I may have stomped on. I've commented whole swaths of code only to have the error continue. Has anyone created at least a list of variable names to stay away from?

Having variables like 'module' or 'key' is very likely in the build system is almost guaranteed here and there. Yet just setting 'module' variable will crash psake hard.
After some debugging time we had to quick fix psake (Invoke-psake.ps1:294) like:
 $_module = $module
 $_initialization = $initialization

 foreach ($_key in $properties.keys) {
     if (test-path "variable:\$_key") {
         set-item -path "variable:\$_key" -value $properties.$_key -WhatIf:$false -Confirm:$false | out-null
     }
 }

 . $_module $_initialization

Maybe it is not a full solution, but at least failproof for simple case


create DB error

I run mssql in docker
when i  create DB there is a  error(with SSMS2008):
Cannot write properly IsFileStream. This property is not available on SQL server 7.0 (Microsoft.SqlServer.smo)

@q409640976 -- thank you for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around the general task.
You may find answers here:

Transact-SQL - MSDN forum
DBA Stack Exchange
Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


[Bug] Incorrect task status after batch task execution is completed

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Description: When I run the demo task (batch job), I find that it runs successfully, but the end status is wrong

StreamX Version
dev
Java Version
1.8
Flink Version
1.14.5
Scala Version of Flink
2.12
Error Exception
None

Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Implementation logic.
We get the corresponding archive log file from the jobid [the log file is generated by the jobmanage at the end]
The archived log file is parsed by json to get the state of the file
{
"jobs":[
{
"jid":"002c63091ecc95bd395260dbbdc69d2a",
"name":"SQLJob-passport-api_kp_ziroom_com_dpLogin-1594315680",
"state":"FAILED",
"start-time":1594315681845,
"end-time":1594315691522,
"duration":9677,
"last-modification":1594315691522,
"tasks":{
"total":2,
"created":0,
"scheduled":0,
"deploying":0,
"running":0,
"finished":0,
"canceling":0,
"canceled":1,
"failed":1,
"reconciling":0
}
}
]
}


Update NNX installation instructions in mnist tutorial

What does this PR do?

Specifies that nnx is now part of flax, and just suggest to install flax with pip

Fixes # (issue)
Checklist

[x] This PR fixes a minor issue

Closing since upgrading flax breaks GPU install version of jax on Colab


RAM Calculation Error in git-pull.js

When I try to do a git pull, I get this error message:  Error while calculating ram usage for this script. Unexpected token (7:0)
[home /]> free
Total:     16.38TB
Used:       0.00GB (0.00%)
Available: 16.38TB

I found out why it was not running correctly, the script file was HTML no javascript.


[Event Hubs] Allow user to configure initial position in the subscribe method when receiving from all partitions

There needs to be a way for user to specify the initial position for each partition that the EventHubConsumerClient decides to process events from if checkpoints are not available
Current proposal is to use the options to pass a map between partitionId and EventPosition

@ramya-rao-a  - Is it important for them to configure it per-partition or simply to allow them to provide a fallback?
We have this option now that can be passed by subscribe()'rs:
/**

The event position to use when claiming a partition if not already
initialized.

Defaults to EventPosition.earliest()
*/
defaultEventPosition?: EventPosition;

After talking with the other team members it looks like what we're offering is functionality equivalent to what other teams are offering as well. Closing this issue.

@ramya-rao-a  brought up an interesting point that the other SDKs allow you to return the offset from your callbacks (which we don't) which gives the user the option to use other critiera (per partition) to figure out the start offset. I can extend our code to allow the same.

Okay, after talking with @chradek - the simple solution here is to allow the user some flexiblity.
They can pass a constant EventPosition (ie, EventPosition.latest(), EventPosition.earliest()) or they can pass us a callback (@chradek  calls it a provider) that can take a partition ID and return an EventPosition, giving them more freedom to pick where they want to start.

And the final final design is now in PR - which is to just provide a method on partition context that the user can call to set the position.
If no position is retrieved from a checkpoint and nothing is specified by the user we default to EventPosition.earliest().


Additional init options

It would probably be nice to be able to have the dependency list as an optional input parameter to spago init, especially when working with existing packages based on bower. I suspect this is somehow related to #24 but I wasn't sure about the main focus there (most links there refer to no longer existing pages, e.g. from-bower scripts). Alternatively, one could optionally, define dependencies list in separate file spago-dependencies.json (which, being a simple list of strings, would also be a valid dhall file imported by spago.dhall). This file could be generated or overwritten by a simple script (probably, as a temporary solution before deciding whether to integrate the dependency import functionality deeper into the spago tool or not - most likely, it should remain a separate command/script keeping in mind the principle of separation of concerns), e.g.
jq '(.dependencies? | keys? // []) + (.devDependencies? | keys? // []) | map(ltrimstr("purescript-"))' ./bower.json > spago-dependencies.json

cat spago.dhall

{ name =
    "package-name"
, dependencies =
    ./spago-dependencies.json
, packages =
    ./packages.dhall
}

Hi @antislava! This is a good analysis, and I think it'd be worth adding your script to a FAQ section "Migrating from bower"
The next step after documenting this would be to automatically migrate a bower project when finding one. We do already migrate psc-package projects automatically in spago init, and I think it would be good to also support bower.json migration in the same way (that is, I don't think we should be able to provide options to init, as it should just detect the bowerfile)
I probably won't manage to do this myself as I'm concentrating on other things on the road to 1.0 (because your script does the job pretty well already), so I'll mark this as "help wanted" and I'm available to provide support, answer questions and give pointers if someone wants to implement this 🙂 (this is valid also for #24, which is about importing bower dependencies as additions in your local package set)


[Idea] Add Wasabi Storage?

i have some Cloud storage that very cheap
i hope you can integrate on this amazing script
https://wasabi.com/

Hello @Dhanamerdeka ,
All integrations are custom and provided as a reference via our documentation
You're welcome to submit a PR to add Wasabi integration example/tutorial


Api gateway gave 404 each time and hystrix didnt work

Trying basic tutorials on Api Gateway but it doesnt seem to work.
Issue Logged here.
https://stackoverflow.com/questions/63482951/spring-cloud-gateway-simple-routing-or-fallback-doesnt-work
Issues:

Routes always gave 404
Hystrix setFallbackUri didnt work or forward to the respective restful api.

Please don't cross post. Closing in favor of stack overflow


[Doc] Limitations on having curly braces ({ }) characters in Admin Credentials

Description:
Add a note to the documentation specifying the limitations on using curly braces ({ }) in the Super Admin password credentials.
Affected Product Version:

WSO2 API Manager v3.2.0

@athiththan11 ,for example when we say not to add the % sign we say "This parsing exception happens because the percentage (%) sign acts as the escape character in URL parsing."
What can we specify as the reason for this?
[1] https://apim.docs.wso2.com/en/latest/install-and-setup/setup/security/logins-and-passwords/maintaining-logins-and-passwords/#change-the-super-admin-credentials

"${" and "}" are reserved characters as they are used to identify the properties that are specified as system properties.


bundler: revert 4dd40b15d8

To fix nixosTests.gitlab I reverted the bundler update that broke it.
I know this isn't the best way to solve it, but I don't know a better way
closes #352963

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

OfBorg is slow, so: rebuild estimates are pretty small:
    284 x86_64-darwin
    578 x86_64-linux

and most importantly, nixosTests.gitlab got fixed for me (aarch64.nixos.community machine).

Blocking the channel is bad, so let me merge this fast.  Anyone, feel free to find a better solution.


Milestone 2:  Add like button

Add like button
in this Milestone I:

Added like buttons functionality

Hi @mohisa302 ,
Status: Approved :heavy_check_mark: :heavy_check_mark: :heavy_check_mark:

To Highlight 💪🏻

Add & remove tested ✔
All tests are green ✔

Good job implementing the requirements  :+1: Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Optional Suggestions  :

N/A

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag  ichala in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


New version: KDE.ruqola version 1.9.1

Pull request has been created with Komac v1.6.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@superusercode Please don't use the urls from https://binary-factory.kde.org/. They are ephemeral and will disappear after some time. Only once the installer appears in https://download.kde.org/stable/ruqola/ the link will be stable


The Meeting Legends are bunching up together on the directory.pdf export

My printer pointed this issue last week. 😢 😄
https://capture.dropbox.com/dKpu02AtUISaenzO

oops thanks give it another try now

🕺 SUCCESS! Thanks!


Enable targeting .NET Standard 2.0

Enable targeting .NET Standard 2.0 as well as .NET Standard 2.1 to allow consumers on .NET Framework and downlevel .NET Core 2.0 runtimes to consume Dogged.
When the Target Framework is netstandard2.0 we pull in the System.Memory package which includes the Span<T> and Memory<T> types that are used in the library but missing from the standard < v2.1.

OH SHIT FIRST PULL REQUEST.


More easy and robust way to determine scroll offset for a header view.

Hi, thanks for the PR.
Unfortunatelly I cannot merge it to the repo in the current state. Although the approach using computeVerticalOffset is most certainly correct, your "simplification" of the code made the library not work correctly.

Removal of the mRecyclerWantsTouchEvent as well as everything tied to it reintroduced Issue #4
In your implementation you did not include proper implementation for RecyclerViews with a reversed LayoutManager (the current implementation fails on such Recyclers)


DRILL-8379: Update Calcite to 1.33.0

DRILL-8379: Update Calcite to 1.33.0
Description
Updated Caclite and Avatica versions. Had to update Jackson to avoid errors with ES Calcite connector.
Documentation
NA
Testing
All unit tests pass.

Thanks @vvysotskyi   This looked like it wasn't quite as painless as going from 1.31 to 1.32, but didn't look like it required any major surgery.

Thanks @vvysotskyi This looked like it wasn't quite as painless as going from 1.31 to 1.32, but didn't look like it required any major surgery.

Agree, this update wasn't trivial and forced us to sacrifice some temporary table functionality in https://github.com/apache/drill/pull/2733, but still, it was much simpler than jumping from 1.21.0 to 1.31.0.


Work around potential nil $?

Process.wait populates $? from which the exit call reads the exit status, but if the SIGINT trap above (added in d0fbc51) fires and the process terminates itself in reaction to that forwarded SIGINT, then $? will be nil as the kill call terminated the process and there is nothing left to wait for:
foreman-0.77.0/lib/foreman/cli.rb:104:in `exit': no implicit conversion from nil to integer (TypeError)
	from foreman-0.77.0/lib/foreman/cli.rb:104:in `run'
	from thor-0.19.1/lib/thor/command.rb:27:in `run'
	from thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from thor-0.19.1/lib/thor/base.rb:440:in `start'
	from foreman-0.77.0/bin/foreman:7:in `<top (required)>'
	from foreman-0.77.0/bin/foreman:23:in `load'
	from foreman-0.77.0/bin/foreman:23:in `<main>'

+1 this barfs all the time on node clusters.

Thanks!


fix: remove forceFocusVisible

This removes forceFocusVisible from the last usage in dialogs, and removes it entirely as something core supports. tryApplyFocus also goes away as it was using forceFocusVisible and was only used in 1 place in dialogs.
For dialogs specifically, we'd still like a focus ring when a dialog opens and focus gets placed on one of the buttons. To accomplish this, I've created a dialog-specific implementation in the dialog mixin, and used a .force-dialog-focus-visible CSS class to be very explicit about what we're doing.
Once this merges, I'll remove the .focus-visible CSS class from any other repos that were using it.

I am SHOCKED that there weren't visual-diff failures. What's even going on. 🤯

I'm not entirely sure why we still want to force focus visible upon opening the dialog. Wouldn't we just want to let :focus-visible do its thing, and if someone is using a mouse then they won't see focus indicator, just like everything else?

I see. I'll chime in there with my vote. 😄

After some further discussion we've decided to not even support this in dialogs, which I think is the best outcome. I've removed it there as well.


cockpit: 314 -> 315

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for cockpit is: Web-based graphical interface for servers
meta.homepage for cockpit is: https://cockpit-project.org/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 315 with grep in /nix/store/babqbkrf2abqd4l806g7g7dv6bfzf6a3-cockpit-315
found 315 in filename of file in /nix/store/babqbkrf2abqd4l806g7g7dv6bfzf6a3-cockpit-315

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cockpit

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/38e238915e9c5df62149f8d0a5a67e1748019452#cockpit

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

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

1 package built:

cockpit

Maintainer pings
cc @lucasew for testing.

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

1 package built:

cockpit

@NixOS/nixpkgs-merge-bot merge


TH12: Grudge Bow "The Bow of Genzanmi Yorimasa" only has 2 phases selectable, not all 4

@webbedspace
webbedspace commented on 12 Nov 2021
This spell has 4 phases, each separated by a cancel:

Sparse fast bullets from alternating sides
Dense slow bullets from both sides
Sparse fast bullets from both sides
Dense medium-speed bullets from both sides

Currently you can only select phases 1 and 4 (I think).

Grudge bow now has 4 phases and the timeout phase as https://github.com/touhouworldcup/thprac/pull/111#issue-1384134082


[material-ui][examples] className mismatch causing dark mode issues with Next.js App Router

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Steps:

Install the example app

curl https://codeload.github.com/mui/material-ui/tar.gz/master | tar -xz --strip=2  material-ui-master/examples/material-ui-nextjs-ts
cd material-ui-nextjs-ts
npm install
npm run dev

Update src/components/ThemeRegistry/theme.ts to

import { Roboto } from 'next/font/google';
import { createTheme } from '@mui/material/styles';

const roboto = Roboto({
  weight: ['300', '400', '500', '700'],
  subsets: ['latin'],
  display: 'swap',
});

const theme = createTheme({
  palette: {
    mode: 'dark',
  },
  typography: {
    fontFamily: roboto.style.fontFamily,
  },
  components: {
    MuiAlert: {
      styleOverrides: {
        root: ({ ownerState }) => ({
          ...(ownerState.severity === 'info' && {
            backgroundColor: '#60a5fa',
          }),
        }),
      },
    },
  },
});

export default theme;

Current behavior 😯
Dark mode kind of works, but it breaks on the MUI component that triggers the error Prop className did not match. This happens every time I refresh the browser.
Screenshot with the error:

Notice that element which produces the error (in the header) does not have dark mode applied.
Expected behavior 🤔
Dark mode works without error
Context 🔦
Run example application with dark mode
Your environment 🌎
Reproduced in Brave and Firefox browsers
System:
OS: Linux 6.2 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
Binaries:
Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
Yarn: Not Found
npm: 10.2.0 - ~/.nvm/versions/node/v18.18.0/bin/npm
Browsers:
Chrome: Not Found
npmPackages:
@emotion/react: latest => 11.11.1
@emotion/styled: latest => 11.11.0
@mui/base:  5.0.0-beta.19
@mui/core-downloads-tracker:  5.14.13
@mui/icons-material: latest => 5.14.13
@mui/material: latest => 5.14.13
@mui/private-theming:  5.14.13
@mui/styled-engine:  5.14.13
@mui/system:  5.14.13
@mui/types:  7.2.6
@mui/utils:  5.14.13
@types/react: latest => 18.2.28
react: latest => 18.2.0
react-dom: latest => 18.2.0
typescript: latest => 5.2.2

Hi @samuelsycamore,
Thanks for your reply!
I was actually talking about this part I highlighted in the screenshot below, and not the blue alert.

That seemed broken to me, because I expected the text to be white on dark mode.
Anyway, I have updated the npm packages of this example app and now with latest versions I do not get the "Prop className did not match" error anymore.
System:
OS: Linux 6.2 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
Binaries:
Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
Yarn: Not Found
npm: 10.2.0 - ~/.nvm/versions/node/v18.18.0/bin/npm
Browsers:
Chrome: Not Found
npmPackages:
@emotion/react: latest => 11.11.1
@emotion/styled: latest => 11.11.0
@mui/base:  5.0.0-beta.20
@mui/core-downloads-tracker:  5.14.14
@mui/icons-material: latest => 5.14.14
@mui/material: latest => 5.14.14
@mui/private-theming:  5.14.14
@mui/styled-engine:  5.14.14
@mui/system:  5.14.14
@mui/types:  7.2.6
@mui/utils:  5.14.14
@types/react: latest => 18.2.30
react: latest => 18.2.0
react-dom: latest => 18.2.0
typescript: latest => 5.2.2
I assume you tested with these versions as well since the packages versions are not locked in package.json and you tested a few days later when the new versions were already available.

I see, thanks for clarifying. I didn't notice that those dark mode styles weren't being applied because I couldn't see the black text on the dark background. 😅  I looked a little deeper and found the culprit in layout.tsx:
              <Typography variant="h6" noWrap component="div" color="black">
                Next.js App Router
              </Typography>

We should update this example to use a theme color (or none at all) instead of hardcoding in black. If you drop that prop it'll work as expected. 👍
I'm glad to hear that the console error seems to be resolved—I suspect it was Emotion's fault, and maybe the fresh installation cleared it up. I'll open a PR to fix that example that'll close out this issue.


Remove duplicate docs between platformatic db and platformatic service

This pr closes  #375.
Configuration file and Plugins file have been rremoved in favor of links to Platformatic Service.

@mcollina please review

Closing due to inactivity (and merge conflicts). If you want to resume work on this, please reopen.


Environment Variables logging to console but aren't able to be used

Environment

Operating System: Darwin
Node Version:     v20.15.1
Nuxt Version:     3.12.4
CLI Version:      3.12.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      compatibilityDate, devtools, modules, runtimeConfig
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-qvsvrt?file=app.vue
Describe the bug
I can run console.log(supabaseUrl, supabaseKey); and both print to the console without issue however when I try to createClient(supabaseUrl, supabaseKey); I receive supabaseUrl is required.
I have tried adding a check or throw for the variables like so:
if (!supabaseUrl || !supabaseKey) {
  throw new Error("Supabase URL and Key are required");
}

and this also throws. Any idea what could be going on?
Additional context

### Logs

```shell-script
Nuxt 3.12.4 with Nitro 2.9.7                                                                    4:16:05 PM
                                                                                4:16:05 PM
  ➜ Local:    http://localhost:3000/
  ➜ Network:  use --host to expose

ℹ Using default Tailwind CSS file                                             nuxt:tailwindcss 4:16:06 PM
ℹ Using local storage from .data/hub                                                  nuxt:hub 4:16:06 PM
  ➜ DevTools: press Shift + Option + D in the browser (v1.3.9)                                  4:16:06 PM

ℹ Tailwind Viewer: http://localhost:3000/_tailwind/                           nuxt:tailwindcss 4:16:06 PM
✔ Vite client built in 16ms                                                                    4:16:07 PM
✔ Vite server built in 908ms                                                                   4:16:08 PM
✔ Nuxt Nitro server built in 964 ms                                                      nitro 4:16:09 PM
ℹ Vite client warmed up in 1585ms                                                              4:16:11 PM
ℹ Vite server warmed up in 2310ms                                                              4:16:12 PM
✔ Vite server hmr 94 files in 718.437ms

createClient@http://localhost:3000/_nuxt/node_modules/.cache/vite/client/deps/@supabase_supabase-js.js:6624:28
setup@http://localhost:3000/_nuxt/app.js:37:34
callWithErrorHandling@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:200:21
setupStatefulComponent@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:7686:46
setupComponent@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:7647:58
mountComponent@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:4863:21
hydrateNode@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:3773:25
hydrateSuspense@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:7047:29
hydrateNode@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:3809:40
hydrateSubTree@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:4940:24
componentUpdateFn@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:4960:27
run@http://localhost:3000/_nuxt/node_modules/@vue/reactivity/dist/reactivity.esm-bundler.js:150:21
@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:5097:20
setupRenderEffect@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:5107:11
mountComponent@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:4875:24
hydrateNode@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:3773:25
hydrate@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:3647:16
mount@http://localhost:3000/_nuxt/node_modules/@vue/runtime-core/dist/runtime-core.esm-bundler.js:2634:20
@http://localhost:3000/_nuxt/Users/maclong/Developer/web/quantum/brooklyn-advocacy/node_modules/nuxt/dist/app/entry.js:55:19

runtimeConfig: { public: { key: ...., url: ... } } }

You have to explicitly expose it to the front-end.

runtimeConfig: { public: { key: ...., url: ... } }

You have to explicitly expose it to the front-end.

Doesn't this make the secret key visible though?

I tried setting it up again and did so via the Nuxt Module this time and it worked pretty much on install and definition of the variables in .env I'll make sure to do it that way next time.


add missing riskScore, checkNumber, balances attrs

This missing fields were causing tests to fail. some other assorted attr_* cleanup as well that removed warnings.

Thanks for submitting, but when reviewed the PR by @maschwenk covered a few more cases and that got merged.  Making this one duplicate.


call asset_exists function in pallet-asset-registry

Motivation
pallet-asset-registry is currently using T::Assets::minimum_balance(asset_id).is_zero() as a way to verify whether some Asset exists on Trappist's pallet-assets.
That is a bit hacky, and served as motivation for https://github.com/paritytech/substrate/issues/12444
Suggested Solution
Ever since https://github.com/paritytech/substrate/pull/12782 was merged, we can replace https://github.com/paritytech/trappist/blob/56369530620d4d88398fad00b8206070e1a2f9c2/pallets/asset-registry/src/lib.rs#L150 with a simpler call to asset_exists.

https://github.com/paritytech/substrate/pull/12782 only got into the release pipeline after polkadot-v0.9.35
so that's also a prerequisite: Trappist needs to be using substrate on polkadot-v0.9.35 or later

So glad that was finally added.

Depends on https://github.com/paritytech/trappist/issues/122

Closed by #131


Enhancement in UI of current text browser

In the current text browser, the scroll bar is hardly visible.
It will be better if we can make the scroll bar more visible.

Updated on commit number :- > df121603ec95fd407a59e30bd98f153c7aa0eab0

One issue still persists as scrollbar ui changes after a keyboard input ...

Same UI fix needs to be done for horizontal scroll.


[SPARK-45159][PYTHON] Handle named arguments only when necessary

What changes were proposed in this pull request?
Handles named arguments only when necessary.
Why are the changes needed?
Constructing kwargs as dict could be expensive. It should be done only when necessary.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @HyukjinKwon @zhengruifeng @allisonwang-db

Merged to master.


Unexpected EOF

I have some text files, containinig special characters, okay for Npp to view/edit/save
However, the gzip compressed file opened by this plugin is trimmed, not all content is displayed
The compression/decompression process is done by 7-zip, which is lossless
Perhaps some weird characters make the plugin think that this is the end of the stream to decode and display?
To make it easy to understand, I've made up a text file to reproduce my problem
The following attachment contains a text file of about 24 lines, however the *.txt.gz opened contains only 3 lines
bugtest20220623.txt.gz

Problem solved upon updating to v1.2.2.
Thanks a lot.
:)


One more mention of Connexion

Might be interesting to mention.

Coverage remained the same at 100.0% when pulling d0c16be86426338c158fef02c49281cc04ca6adb on rafaelcaricio:add-article-to-readme into b57ecbaa4ac88bcd8250a65dca1361b87090d1d8 on zalando:master.

:+1:


[MM-51514] Allow Calls widget to open user settings

Summary
As we are implementing persistent Calls preferences and settings, we'd like a way to open the modal from within a call. PR implements a new openCallsUserSettings method to allow triggering this functionality from the global widget window.
Example
https://github.com/user-attachments/assets/c3521aac-69de-4daf-8b06-d8d7d7bbdff1
Ticket Link
https://mattermost.atlassian.net/browse/MM-51514
Release Note
NONE

Adding @enzowritescode since this changes the Desktop API

@devinbinnie Do you think we could push the new package version?

@devinbinnie Do you think we could push the new package version?

Done


Define and develop smart contracts for agent interactions with the blockchain network.

Waiting to confirm through research if smart contracts are needed for the project


[GSSOC'23]Egg Dropping Puzzle

Hi,I am contributor of GSSOC'23, I want to add the egg dropping puzzle problem. Please assign me this issue.
@aaryahjolia
Thank you.

@aaryahjolia I want to create a egg droppin puzzle in cpp . Assign this to me. I would like to work on it.


Add error for overloaded Haskell constructors

Solves #125.
Unfortunately, it seems the Agda compiler only does its compiler magic for each definition independently, so there is no state carried over as to the previous definitions that have been compiled.
We check all the constructors generated for duplicates prior to writeOutput in postCompile and refuse to continue if there are duplicates. This does also add scaffolding for future post-compilation, pre-render verification steps.

Unfortunately, it seems the Agda compiler only does its compiler magic for each definition independently, so there is no state carried over as to the previous definitions that have been compiled.

Yes, we have discussed this limitation before, and then we came to the conclusion that the best way to have some shared state would be to use an IORef (which is possible since the C monad implements MonadIO).
However, for this issue the solution you propose also seems like a reasonable approach.

The test suite fails on the following:
== Comparing output ==
diff -r build/Issue125.err golden/Issue125.err
1c1
< Cannot generate multiple constructors with the same identifier: ACtr
---
> Found multiple declarations of constructor: ACtr

Please run make test and make golden and then commit the changes to the golden files.

I was looking for a cabal test equivalent and didn't think to check for a makefile. Oops.


Prepare publish

Codecov Report

Merging #20 (e3342d5) into master (5f8a04b) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #20   +/-   ##
======================================
  Coverage    7.97%   7.97%
======================================
  Files           6       6
  Lines         589     589
======================================
  Hits           47      47
  Misses        542     542

Impacted Files
Coverage Δ

src/bin/potcpu.rs
0.00% <ø> (ø)

src/bin/potnet.rs
19.42% <ø> (ø)

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 5f8a04b...e3342d5. Read the comment docs.