Type incoherent in comparison of String type with a string literal

As I was implementing #53, I found an error that also occurred in String type.
After executing the following SQL,
create table test (s String) engine = BaseStorage
insert into test values ('test')
select s from test -- returned a single row 'test'
select s from test where s = 'test' -- error occurs here!

I received this error,
Received exception from server (version 2021.5.0):
Code: 7. DB::Exception: Received from localhost:9528, ::1. WrappingEngineError(WrappingDFError(Plan("'LargeUtf8 = Utf8' can't be evaluated because there isn't a common type to coerce the types to"))). Error during planning: 'LargeUtf8 = Utf8' can't be evaluated because there isn't a common type to coerce the types to.

After some analysis, I found TB's query is executed in DataFusion, and accidentally, String type in TB is translated to Arrow's LargeUtf8, but the string literal seems to be parsed as the type Utf8, which is incoherent with LargeUtf8.
Yet I have few ideas of how to fix this issue. Maybe we should modify the implementations in DataFusion?

@whjpji thanks for your reporting. Yes. This is DF's problem. The best way is to make utf8 castable to largetuft8 or similar.

In fact, this problem is tricky. The largerutf8 and utf8 is castable, but not enable in the planner. And more TB' string is a little different to the Arrow's string. So, to solve this problem, we need two fixs in one...


support for PDPKA login and signup

include v4 and v5 auth attempts
boost versions of kbpgp

r? @malgorithms

👍  from @malgorithms on this.


Add support for meta data for fonts

As of now, we are doing hard lookup for fonts by their exact name and also there are no way to specify font meta data like human readable name or different styles.
But in the long term plan - moving forward with font formulas we need to specify font style as well. So, this commit adds those necessary changes so we can specify font name, and the styles for font. This also adopt this changes for the lookup

We still need to adjust each font formulas to match within the downloaded fonts, but I will add those in a separate PR


Disable wait_pop_timed timer in unbounded case to reduce log spam.

Disable wait_pop_timed timer in unbounded case to reduce log spam.
Motivation

Silences logspam that happens every 250ms from typechecker coord thread. Unlike bounded usages of wait_pop_timed, the typechecker coord thread uses a unbounded queue that sets the expected number of queue entries to be MAX INT to create an effectively unbounded queue.
Test plan

N/A

cc @mkillianey-stripe who noticed this issue (thanks!)

@DarkDimius The problem is that, while sorbet is in LSP mode, it continuously emits a log line for wait_pop_timed every 250 ms. I wanted to prevent that from happening.
I see value in the log line for bounded cases for traces, so I figured it was important to keep in for that use case.
Would it be better to pass a silent arg to wait_pop_timed that silences the timer?

Would it be better to pass a silent arg to wait_pop_timed that silences the timer?

sgtm

Made adjustments; waiting for build to pass.


refactor: use internal execGit to list staged files

This pull request removes the staged-git-files and pify dependencies and uses the internal execGit to list staged files. This is done via
git diff --staged --diff-filter=ACM --name-only

This should be the same behaviour as `staged-git-files, since only the filenames were used.

:tada: This PR is included in version 9.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Table alias is not reflected on fields

Describe the bug
When setting a table alias, the alias is not used on the fields and conditions, which returns errors, at least in postgres.
Environment (please complete the following information):

OS: windows
Database:  postgres
Database driver: pq
Jet version 2.9.0

Code snippet
  SELECT(t.Device.DeviceID, t.Device.CompanyID, t.Device.DeviceName, COUNT(STAR).OVER()).
  FROM(t.Device.AS("d"))

SELECT device.device_id AS "device.device_id",
     device.company_id AS "device.company_id",
     device.device_name AS "device.device_name",
     COUNT(*) OVER ()
FROM public.device AS d;

failed to query db. error: pq: invalid reference to FROM-clause entry for table "device"

Expected behavior
The table alias should be used on all references to it.

Hi, @RangelReale ,
You need to alias table before the query.

Device := t.Device.AS("d")

SELECT(Device.DeviceID, Device.CompanyID, Device.DeviceName, COUNT(STAR).OVER()).
FROM(Device)


batch_normalization layer argument name and documentation wrong in core and contrib layers

The batch normalization layers both in core and contrib take a center argument that controls whether a variable beta should be created that is then added after the normalization. The name of the argument (center) and even more so the documentation (center: If True, subtract beta. If False, beta is ignored.) indicate however that this argument controls whether the mean of the input is subtracted as part of the normalization.

@sguada


[Gatsby-Plugin] README is outdated

Summary
It seems the README is a bit out-of-date. https://rpm.newrelic.com is no longer since it now redirects to https://one.newrelic.com. If would be great to update the steps to include the new experience.
Acceptance Criteria

[ ] project readme accurately reflects current functionality of the plugin.

This can be done along with #31 , that will get to know the plugin


Intercom google analytics

Basically a bunch of javascript

The intercom js is supposed to go inside the body tag, otherwise :shipit:

fixed the intercom thing!


Recommended way of loading remote images?

Thank you for this project. I'm finding it very useful, but i'm running into an issue lately where the cropped doesn't seem to render properly (Chrome, Nuxt3) when I have multiple instances of a component on my page (an image uploading component), where that image uploading component contains a cropper.
I've narrowed it down to the fact that if I have just one instance of that component on my page, then it works, mostly.
Your guide on multiple croppers and the documentation  on it doesn't seem to clarify how to resolve this. Could you provide some clarity on how to correctly use the MultipleCropper?

Closing this as I figured I can just hide the cropper with a v-if and it solved the issue.


[Feature] : Design Join Us Page

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
@Counselllor
when user click on join us link from footer, we will redirect them to /join-us route
and it will look like

please assign it to me, i would like to work on that issue

please assign this issue to me. I would very much be pleased to work on this issue.


Rails/HttpPositionalArguments autofix doesn't preserve parenthesis

We specify our spec actions with parenthesis, but the Rails/HttpPositionalArguments autofix doesn't preserve them:
-    put(:update, valid_params)
+    put :update, params: valid_params

I think this can be closed via #3783


VIZ: bug in plot_sensors(kind='3d')

Describe the bug
When plotting MEG and EEG sensors at the same time, the EEG and MEG sensors appear in the wrong position relative to one another.  Presumably this is an issue of which coordinate frame is chosen.  An example is here, where the EEG sensors are higher up than the MEG sensors (implying that the subject's head was embedded in the MEG helmet).
Steps to reproduce
See the linked tutorial.
@agramfort WDYT?  @larsoner suggested (offline) that if ch_type='all' then everything should get transformed to head coords, but if only plotting a single sensor type we can plot in whatever coord sys they're in.  If the overhead is not too bad, I might suggest always plotting in head coords, even if only one sensor type is chosen.  As an aside, it might also be good to label the 3d plot axes with a unit (meters).

+1 with what @larsoner suggested.

If the overhead is not too bad

Transform overhead should be negligible.

I might suggest always plotting in head coords, even if only one sensor type is chosen.

I was actually starting to think the same thing. So I would say to always use head coords for now, and if people want MEG sensors in MEG coords we can add coord_frame later -- or just tell them to use plot_alignment which already has these transform options built in.

As an aside, it might also be good to label the 3d plot axes with a unit (meters).

Agreed, feel free to add this at the same time as the coord frame changes


release: Release v1.0.5

This PR updates the metadata of Pendulum and Foucoco and bumps the dependency version to a new patch version. Apparently nothing changed about the metadata on Amplitude, I tried updating that too.
Closes #492.

Oh I didn't notice that, thanks for pointing it out! Must have slipped in an automatic merge with the PR for vault integration tests. I changed it now to 1.0.5 as well.

I'll merge this now because no breaking changes can be introduced in this PR.


Update docs to tell how to run Packit on other distros

I tried setting up pakckit using the command:
pip3 install --user packitos
I am getting error:
Collecting rpm-py-installer
Using cached https://files.pythonhosted.org/packages/5d/c6/7be50a13948a744976cf8f5eca31deb7e73138101e142a6177132d05c21e/rpm-py-installer-1.0.0.tar.gz
Complete output from command python setup.py egg_info:
running egg_info
creating pip-egg-info/rpm_py_installer.egg-info
writing pip-egg-info/rpm_py_installer.egg-info/PKG-INFO
writing dependency_links to pip-egg-info/rpm_py_installer.egg-info/dependency_links.txt
writing top-level names to pip-egg-info/rpm_py_installer.egg-info/top_level.txt
writing manifest file 'pip-egg-info/rpm_py_installer.egg-info/SOURCES.txt'
reading manifest file 'pip-egg-info/rpm_py_installer.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: no previously-included files found matching 'test-requirements.txt'
no previously-included directories found matching 'tests'
writing manifest file 'pip-egg-info/rpm_py_installer.egg-info/SOURCES.txt'
[INFO] Installing...
Traceback (most recent call last):
File "install.py", line 2202, in
main()
File "install.py", line 2196, in main
app = Application()
File "install.py", line 26, in init
self._load_options_from_env()
File "install.py", line 80, in _load_options_from_env
raise InstallError('rpm command not found. Install rpm.')
main.InstallError: rpm command not found. Install rpm.
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-build-e72g6hm7/rpm-py-installer/setup.py", line 129, in
'bdist_wheel': BdistWheelCommand,
File "/home/work/env/lib/python3.6/site-packages/setuptools/init.py", line 144, in setup
return distutils.core.setup(**attrs)
File "/usr/lib/python3.6/distutils/core.py", line 148, in setup
dist.run_commands()
File "/usr/lib/python3.6/distutils/dist.py", line 955, in run_commands
self.run_command(cmd)
File "/usr/lib/python3.6/distutils/dist.py", line 974, in run_command
cmd_obj.run()
File "/tmp/pip-build-e72g6hm7/rpm-py-installer/setup.py", line 57, in run
install_rpm_py()
File "/tmp/pip-build-e72g6hm7/rpm-py-installer/setup.py", line 20, in install_rpm_py
raise Exception('Command failed: {0}'.format(cmd))
Exception: Command failed: /home/work/env/bin/python3 install.py
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-e72g6hm7/rpm-py-installer/

Thanks for the report.
You need to have rpm installed in your system -- it's a tool for manipulating with the packages on RPM-base distributions like Fedora, Centos, RHEL, ... I am not sure if it can be installed on other distributions, but you can try installing that. (You can also try installing yum or dnf that are built on top of it. Some people use it for installing packages to custom directories.)
It would be probably easier to run packit in the podman/docker container with fedora image.

+1 for a container
docker.io/usercont/packit

For this we should update our documentation an include instruction for how to run Packit on other distros.

Still wanted I think.

Still wanted I think.


Publish standalone Pyinstaller binaries

Is your feature request related to a problem? Please describe.
Running OpenAPI generators on networks disconnected from the internet is challenging. This usually involves moving runtimes and transitive dependencies from the Internet to the disconnected network manually, and then trying to install the packages in a non-standard way (e.g. from the filesystem).
openapi-generator helps this situation by distributing itself as a standalone JAR: https://openapi-generator.tech/docs/installation/#jar
Describe the solution you'd like
Use GitHub Actions and PyInstaller to build standalone openapi-python-client binaries for Windows, macOS, and Linux. This would make it possible for users who want to run openapi-python-client to only need to download a binary, and then be able to run that anywhere without having to worry about Python, pip, PyPI packages, etc.
Describe alternatives you've considered
There are many other Python binary building libraries, but in my experience, PyInstaller has the best support.
Others include:

https://github.com/indygreg/PyOxidizer
https://nuitka.net/

I have enough "fun" trying to support binaries built from Python applications, so I don't intend on ever doing it again 😬. If someone else wants to make and support such a project, they're more than welcome to, but I'm too bitter from past experiences to do it myself here.


Add more tests to runtime2/dispatcher

Add more tests to cover more scenarios
Improve the test infrastructure to capture more state for existing tests.
Switch from Empty{} to Struct{} for instances for exercising expressions.
Add fullLog method to tests, for helping with debugging tests. This prints out the full state log, allowing for better investigation experience.

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
20a5d0760b14c1f0a742fe94091e9c1259f97e84
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Any takers? Perfect chance to up your code reviewer creds for fixit.

Closing this in favor of, https://github.com/istio/istio/pull/3988, which also includes this change.


XMLLibXML: 2.0129 -> 2.0132 fixes failing tests & CVE-2017-10672

Motivation for this change
Issue is described in the cpan RT [1]. Patch was submitted via a GitHub PR
[2].
[1] https://rt.cpan.org/Public/Bug/Display.html?id=122958
[2] https://github.com/shlomif/perl-XML-LibXML/pull/8
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Backported in 3753929d7d7


refactor: Report usage-based warnings as user-facing messages

The other warnings are developer-facing and should indeed remain as DeprecationWarning.

So the rationale is that docs writer are not always developers and won't care about Python warnings? Or even have them disabled? I'm not sure they would care more about an INFO log... :thinking:

I think users would care more about an info log compared to literally nothing as it is now.
DeprecationWarning means very simply "don't call this function". People aren't calling any functions here. There's nothing to learn from a stack trace. Indeed, why is mkdocstrings calling its own deprecated function? That's what DeprecationWarning is saying

Yet another way to look at it is:
DeprecationWarning means - upgrade everything to latest and if the issue still persists, complain to the developer of the plugin. These cases that I'm changing are incorrectly suggesting this.

To me a deprecation warning is not exclusively used for functions. We deprecated using selection and rendering keys, so we inform the users with the standard deprecation mecanisms that they should stop using them, because they will disappear in the next versions and things will break. Users/devs who care about keeping up-to-date dependencies should enable warnings, as best-practice. If they don't and get caught by a breaking change later, that's on them :confused: Though I agree an additional log cannot hurt.

Do you know any application that informs its users about removing features by emitting a Python DeprecationWarning (which, again, almost nobody will see)?
The fact that this feature removal happens to be implemented by reorganizing some Python code is completely coincidental. Users don't care to see references to Python.
I don't know why you call it a "standard mechanism" in this context when its only purpose is to tell developers to change their code in response to some other change. It actually directly says so here https://docs.python.org/3/library/exceptions.html#DeprecationWarning

Hmm but maybe we could consider just integrating this kind of warning instead https://docs.python.org/3/library/exceptions.html#UserWarning - it even seems to be enabled by default

Well, it still talks about "user code" so still not a great fit

https://docs.python.org/3/howto/logging.html#when-to-use-logging also confirms that the expected action in response to Warnings is to change the code of the application. Which doesn't match in these cases that I'm suggesting to change.
I also plan to make this the official guidance of MkDocs.

And sorry that I sound so aggressive about this 😞

And it's not just me either.
https://github.com/mkdocs/mkdocs/blob/0437b7f85380a409244e3b4fabcfb84dccdf6efa/mkdocs/contrib/search/__init__.py#L69-L72

Thanks for the links, it seems the DeprecationWarning should not be used for that indeed. How do you feel about prefixing the log line with "DEPRECATION"? I worry users will simply miss/ignore them.

OK! Why not.
Here's how it will look (first warning is unrelated but nice to see in surrounding)
INFO     -  DeprecationWarning: 'UnescapePostprocessor' is deprecated. This class will be removed in the future; use 'treeprocessors.UnescapeTreeprocessor' instead.
              File "/home/blaxpirit/.local/lib/py/mkdocs_literate_nav/parser.py", line 22, in <module>
                _unescape = markdown.postprocessors.UnescapePostprocessor().run
              File "/home/blaxpirit/.local/lib/py/markdown/util.py", line 103, in deprecated_func
                warnings.warn(
INFO     -  mkdocstrings: DEPRECATION: mkdocstrings' watch feature is deprecated in favor of MkDocs' watch feature, see https://www.mkdocs.org/user-guide/configuration/#watch
INFO     -  Cleaning site directory
INFO     -  mkdocstrings: DEPRECATION: 'selection' and 'rendering' are deprecated and merged into a single 'options' YAML key
INFO     -  Documentation built in 0.41 seconds
INFO     -  [12:36:10] Watching paths for changes: 'docs', 'mkdocs.yml', 'mkdocs_gen_files'


Make String Match Map Deterministic For Iteration

The current string match map has non-deterministic behavior when iterating. With minimal added overhead we can make this map into a linked hash map which will make iteration consistent and allow order to matter for our string match map configurations.

handled under https://github.com/NationalSecurityAgency/emissary/pull/113


Question about Downloading cam_int.zip

Hi. Last time I asked a question about downloading the full dataset due to some damaged .zip files which cannot be unzip. And thanks for your effort on uploading the whole dataset. I successfully downloaded and unzipped all .zip files but cam_int.zip (39GB). Can you solve this problem? Looking forward to your reply and appreciating all your effort.

@dekunma can you check that file? Thank you.

My guess is that cam_int.zip is too big to be downloaded easily. I will split it into two files, each of which will be around 20GB, and upload them today.
If you are encountering some other problems besides this, please let me know @lianggroup.

@lianggroup I have split cam_int.zip into cam_int1.zip and cam_int2.zip and uploaded them to both Baidu Netdisk and Google Drive. Please try if you are able to download them now.

I‘ve successfully downloaded and unzipped the cam_int.zip. Thanks a lot!


HTML not being reset after upload (suggestion)

I was working with this package and noticed that after a successful upload, the progress bar stays there even if I leave the page and come back. To clear the bar you need a page refresh. I don't know if this is the best way to do this, but for a quick fix I use the template rendered block to clear it. I went through the package briefly and didn't know where you could add this to clear it up at the package level. Thought I'd throw it out there.
Template.editModal.rendered = function () {
	$('.progress').remove();
}

Remember we are using a client side collection to control our data. If you do not want your loader to show all you need to do is remove the data from the collection OR build a helper that checks whether the collection has been fully uploaded or not. I think the response object has this. Either way, inspect S3.collection and you'll figure it out quickly I'm sure.


Docs refer to old test-utils package

The Vue test utils documentation (testing SFC with Jest) refers to the following command:
$ npm install --save-dev jest vue-test-utils

which is the non-scoped package, resulting in the following warning.
<EMAIL_ADDRESS>VUE-TEST-UILS WARN: vue-test-utils has moved to a scoped package. The new repository is available @vue/test-utils. To install the new package, run npm install --save-dev @vue/test-utils

Thanks, but this is a vue-test-utils issue and should be reported there instead.


stepDefinitions in explicitConfiguration are ignored when using type-specific configuration

Current behavior
stepDefinitions defined in explicitConfiguration (e.g. .cypress-cucumber-preprocessorrc.json) seem to be ignored when using type-specific configuration, causing cypress-cucumber-preprocessor to fall back to the defaults.
Desired behavior
Projects with both Cypress e2e and ct can have different requirements for stepDefinitions for each, requiring an explicitConfiguration (e.g. .cypress-cucumber-preprocessorrc.json) as follows:
{
	"e2e": {
		"stepDefinitions": ["cypress/e2e/[filepath].js", "someFolder/cypress/support/step_definitions/*.js"]
	}
}

Preferably, these would override the defaults.
Test code to reproduce
Here's an isolated, minimal reproduction demonstrating the behaviour of combineIntoConfiguration:
https://github.com/fabkru/cypress-cucumber-preprocessor-config-bug-repro/blob/master/src/index.test.ts#L27
Test outputs the following:

Versions

Cypress version: -
Preprocessor version: 18.0.0
Node version: -

Checklist

[x] I've read the FAQ.
[x] I've read instructions for logging issues.
[x] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

Hi @fabkru, thanks for reporting this issue. I've fixed this with v18.0.3.


GIF Converter in Python

Describe the project you want to add with tech stack
an .mp4 to GIF converter in python and vice versa.
Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 @TusharKesarwani

from moviepy.editor import VideoFileClip
from PIL import Image
def mp4_to_gif(mp4_file, gif_file):
video = VideoFileClip(mp4_file)
video.write_gif(gif_file)
print("Conversion successful. GIF saved at:", gif_file)
def gif_to_mp4(gif_file, mp4_file):
gif = Image.open(gif_file)
gif_frames = [frame.copy() for frame in ImageSequence.Iterator(gif)]
gif_frames[0].save(mp4_file, save_all=True, append_images=gif_frames[1:], loop=0, duration=gif.info['duration'])
print("Conversion successful. MP4 saved at:", mp4_file)

if name == "main":
print("Select conversion type:")
print("1. MP4 to GIF")
print("2. GIF to MP4")
choice = int(input("Enter your choice (1 or 2): "))
if choice == 1:
    mp4_file = input("Enter the path of the MP4 file to convert: ")
    gif_file = input("Enter the path to save the GIF file: ")
    mp4_to_gif(mp4_file, gif_file)
elif choice == 2:
    gif_file = input("Enter the path of the GIF file to convert: ")
    mp4_file = input("Enter the path to save the MP4 file: ")
    gif_to_mp4(gif_file, mp4_file)
else:
    print("Invalid choice. Please select either 1 or 2.")


Can't immediatly open a new modal after closing one

I have two modals and closing the first one leads to opening the second one:
this.setState({
    isFirstModalVisible: false,
    isSecondModalVisible: true,
})

This works fine with react-native's modal, but with react-native-modal, the second one won't show up.
I have tried to use a callback function:
this.setState({isFirstModalVisible: false}, () => {
isSecondModalVisible: true,
})

But it's still not working with react-native-modal
My current solution is to have a timeout function with 0.5s delay to open the second modal, but this introduced an unwanted delay. Are there any other better workarounds without delays?

I have same issues........

This is happening because you can't have two modal on screen at the same time and while the modal is transitioning out it is still considered "on screen", which is the reason why you timeout solution works correctly.
A cleaner solution is showing the second modal when onModalHide of the first modal is triggered.
Let me know if it fixes your issues.

@mmazzarolo  Thanks it worked! I did not know about that. I try to use modal as the way i use it in React.js

this.setState({loadingModal : false});
setTimeout(function(){self.setState({loadModalConfirm : true})}, 500);
giving a delay to next modal worked for me


It shows zero amount of the variant product if the catalog rule is applied.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
It shows zero amount of the variant product if the catalog/cart rule is applied.
Preconditions

Admin should have at least one configurable product available.
Admin should have one catalog rule and should be applied to that product.

Steps to reproduce

Go to 'Shop'
Scroll down to 'variant product'
See the error

Expected Result
It should be shown the product discount amount instead of zero.
Actual Result
It shows zero amount. Here
Screenshots
No response
Additional context
No response

@ashishkumar-webkul  Issue not replicated. it is working fine.

Checked the same with the latest code pull and found that there is no such issue which is appearing. Please check the screenshot for the same here
Hence I am closing the issue from here.


Relative sourcemap base

This implements #319, by ensuring that the source maps paths match up to the input folder, using a custom sourceMapsBasePrefix option.
Ideally the CLI should know to configure this option based on the output directory location relative pathing, but by default it will likely usually be ../.
I compared the source maps here to the ones generated by TypeScript and the output file is pretty much identical now, but when I tried testing with node --inspect-brk in the Chrome dev tools, I still couldn't get the source map to apply.
I ran the source maps in the Webpack visualizer at https://sokra.github.io/source-map-visualization and can confirm they are all working 100%.
The only thing I can think here is either if my local setup isn't running file system source maps for some reason (so @styfle perhaps you can verify your side), or if the Webpack boostrap file is somehow making things invalid as it is the first source ("/webpack/bootstrap"). I tried changing this name to different forms but that didn't seem to make a difference.
Not sure what else to try, but source maps debugging is always like this the first time :)

There's a really odd CI failure on this one...

Ok I just tested in a TypeScript project and I'm also seeing the source maps not applying there so this may actually be something to do with my setup.

I had the same problem with the inspector and Chrome in the past couple weeks. Maybe it’s a mismatch between v8 in Node vs Chrome?
Try VS Code and see if that works because the tsc source maps work for me with Code.
I’ll restart CI.

@guybedford This is really close!
I found that the source map is observed by VS Code, however the line numbers look wrong.
Steps to reproduce
git clone https://github.com/styfle/ncc-bug-sourcemap
cd ncc-bug-sourcemap
git checkout pr327

Working with tsc
yarn build
code . # open with VS Code
node --inspect-brk dist
# Visit the Debugger and select "Attach"

Not working with ncc
# below is my copy of ncc built from your branch
/path/to/ncc/dist/ncc/cli.js build index.ts --source-map --no-source-map-register
node dist
code . # open with VS Code
node --inspect-brk dist
# Visit the Debugger and select "Attach"

Codecov Report

Merging #327 into master will increase coverage by 0.1%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #327     +/-   ##
=========================================
+ Coverage   69.45%   69.55%   +0.1%
=========================================
  Files          12       12
  Lines         383      381      -2
=========================================
- Hits          266      265      -1
+ Misses        117      116      -1

Impacted Files
Coverage Δ

src/index.js
71.51% <100%> (+0.26%)
: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 99e71ef...8117ad8. Read the comment docs.

I just tested this out and I can replicate the source maps issue. But in my own test, the source maps worked fine.
I ran the source maps through the checker visualization at https://sokra.github.io/source-map-visualization/#custom and get them lining up correctly, see below.

So the above actually seems to be a VSCode / debugger source maps issue. This might be due to Node.js issues with source maps, which are being fixed thanks to the new compileFunction approach which has been worked on by bcoe.


Execution of script closest.m errors

when using the code
m = f1.match(f2);
in the finding correspondence using surf features video the following error is produced
Execution of script closest as a function is not supported:
C:\Users*USERNAME*\AppData\Roaming\MathWorks\MATLAB Add-Ons\Toolboxes\Machine Vision Toolbox for MATLAB\vision\closest.m
Error in SurfPointFeature/match (line 122)
[cor2,err] = closest(D1, D2);
Error in rvc2 (line 18)
m = f1.match(f2);
This same issue for closest.m occurs in the advanced mathematical morphology code as well for the colorkmeans function
I have linked a text file for the contents of the closest.m file to help with this issue. If you follow the file directories shown earlier with your own username this should point to the matlab file as well.
closest.m.txt

It seems like this is the error reported here, plus suggested fix: https://github.com/petercorke/machinevision-toolbox-matlab/blob/master/mex/README
@Tjeard0 can you please check?

From the looks of it that should fix the issue, I'll test this when I get back home and update the files and GitHub issue respectively if it works

If it works, could you please add a link to this README in the relevant video scripts?

Sounds good, I'll make sure to add in the link!

After testing the make command as recommende in the readme all of the mex files compile succesfully but the apriltags.c file fails. It links to a website: https://april.eecs.umich.edu/wiki/index.php/AprilTags
which produces an error 404 when I try and access it. After the make command I also tried to run the file and it was still producing the error

I didn't have any issues compiling this, closing here.


Mopidy and aplay uses USB sound card but Shairport uses analog

I'm using an Enermax DreamBass AP001 USB sound card recognized by lsusb as follows:
Bus 001 Device 004: ID 040d:3400 VIA Technologies, Inc.

Mopidy radio and local/network files playback works fine on v0.5 without any changes (output left to blank/autodetect with: output = ): it uses the USB sound card.
But Airplay (from iOS) playback uses the analog output.
I've found some similar issue (not on a Pi Music Box setup tough): Cannot get Shairport working with USB soundcard and the advised command:
amixer cset numid=3 1

followed by a reboot seems to have done the trick: both Mopidy/Shairport now use the USB sound card.
I'm posting it here for future reference and in case it breaks tomorrow ;)
Here are some aplay outputs:
aplay -l:
**** List of PLAYBACK Hardware Devices ****
card 0: ALSA [bcm2835 ALSA], device 0: bcm2835 ALSA [bcm2835 ALSA]
  Subdevices: 8/8
  Subdevice #0: subdevice #0
  Subdevice #1: subdevice #1
  Subdevice #2: subdevice #2
  Subdevice #3: subdevice #3
  Subdevice #4: subdevice #4
  Subdevice #5: subdevice #5
  Subdevice #6: subdevice #6
  Subdevice #7: subdevice #7
card 0: ALSA [bcm2835 ALSA], device 1: bcm2835 ALSA [bcm2835 IEC958/HDMI]
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 1: Dongle [VIA USB Dongle], device 0: USB Audio [USB Audio]
  Subdevices: 0/1
  Subdevice #0: subdevice #0
card 2: sndrpihifiberry [snd_rpi_hifiberry_dac], device 0: HifiBerry DAC HiFi pcm5102a-hifi-0 []
  Subdevices: 1/1
  Subdevice #0: subdevice #0

aplay -L:
null
    Discard all samples (playback) or generate zero samples (capture)
sysdefault:CARD=ALSA
    bcm2835 ALSA, bcm2835 ALSA
    Default Audio Device
sysdefault:CARD=Dongle
    VIA USB Dongle, USB Audio
    Default Audio Device
front:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    Front speakers
surround40:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    4.0 Surround output to Front and Rear speakers
surround41:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    4.1 Surround output to Front, Rear and Subwoofer speakers
surround50:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    5.0 Surround output to Front, Center and Rear speakers
surround51:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    5.1 Surround output to Front, Center, Rear and Subwoofer speakers
surround71:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    7.1 Surround output to Front, Center, Side, Rear and Woofer speakers
iec958:CARD=Dongle,DEV=0
    VIA USB Dongle, USB Audio
    IEC958 (S/PDIF) Digital Audio Output
sysdefault:CARD=sndrpihifiberry
    snd_rpi_hifiberry_dac,
    Default Audio Device

Since no reply I'll close this.reopen if its still a problem.


AUTOPILOT_VERSION (148) wrong field decoding order

Generated AutopilotVersion class has wrong field decoding order. As result we have shuffled fields and no way to properly get some fields.
Field order in constructor is correct BTW.
AUTOPILOT_VERSION

@yaroslavkulinich, the field ordering for serialisation and deserialization is done according to the MAVLink field ordering scheme.
You can compare it with other MAVLink libraries, dart_mavlink for example. You can see that the ordering is same for AUTOPILOT_VERSON.
I had kept the ordering in the constructor according to their arrangement in the MAVLink XML definitions so that it is easy to correlate while using the class.

@yaroslavkulinich okay, thanks.
I will try dig deeper with the issue.
But without additional analysis for me it's clear that MissionPlanner correctly represents AUTOPILOT_VERSION message (described in mavlink inspector), but with mavlink-kotlin deserialization I get shuffled fields. For the same message.
I'm using ardupilot SITL. Version 4.6.0 dev.
This problem is something new for me, because I never faced it with other messages.

@yaroslavkulinich, I have checked the message with both PX4 and Ardupilot on real flight controllers. I am getting the correct data. When we print the message using the implicit toString(), the fields will get printed according to their order in the constructor.Can you please provide screenshots

Closing due to inactivity.

@yaroslavkulinich Hi, I got the same issue in version 1.2.6, here is the log and capture from wireshark
2024-07-10 18:15:34.268 25776-25817 FlyController           com.test.sample                 D  onMessageReceived: TerrainReport(lat=0, lon=0, spacing=0, terrainHeight=0.0, currentHeight=0.0, pending=112, loaded=0)
2024-07-10 18:15:34.268 25776-25817 FlyController           com.test.sample                 D  onMessageReceived: Vibration(timeUsec=11689756109, vibrationX=0.03799032, vibrationY=0.0223004, vibrationZ=0.02042753, clipping0=0, clipping1=0, clipping2=0)
2024-07-10 18:15:34.270 25776-25817 FlyController           com.test.sample                 D  onMessageReceived: BatteryStatus(id=0, batteryFunction=MavEnumValue(entry=UNKNOWN, value=0), type=MavEnumValue(entry=UNKNOWN, value=0), temperature=32767, voltages=[947, 65535, 65535, 65535, 65535, 65535, 65535, 65535, 65535, 65535], currentBattery=24, currentConsumed=809, energyConsumed=27, batteryRemaining=75, timeRemaining=0, chargeState=MavEnumValue(entry=OK, value=1), voltagesExt=[0, 0, 0, 0], mode=MavEnumValue(entry=UNKNOWN, value=0), faultBitmask=MavBitmaskValue(flags=[], value=0))
2024-07-10 18:15:34.270 25776-25817 FlyControlActivity      com.test.sample                 D  onBatteryStatus: BatteryStatus(id=0, batteryFunction=MavEnumValue(entry=UNKNOWN, value=0), type=MavEnumValue(entry=UNKNOWN, value=0), temperature=32767, voltages=[947, 65535, 65535, 65535, 65535, 65535, 65535, 65535, 65535, 65535], currentBattery=24, currentConsumed=809, energyConsumed=27, batteryRemaining=75, timeRemaining=0, chargeState=MavEnumValue(entry=OK, value=1), voltagesExt=[0, 0, 0, 0], mode=MavEnumValue(entry=UNKNOWN, value=0), faultBitmask=MavBitmaskValue(flags=[], value=0))
2024-07-10 18:15:34.270 25776-25817 FlyController           com.test.sample                 D  onMessageReceived: Attitude(timeBootMs=11689773, roll=-2.9905872, pitch=0.4307207, yaw=0.78559434, rollspeed=-0.0032989802, pitchspeed=0.0018957397, yawspeed=4.4002663E-6)
2024-07-10 18:15:34.270 25776-25817 FlyController           com.test.sample                 D  onMessageReceived: VfrHud(airspeed=0.0, groundspeed=0.0060461196, heading=45, throttle=0, alt=0.0, climb=0.005788166)
2024-07-10 18:15:34.270 25776-25818 MavFlyControllerWRer    com.test.sample                 D  sendMessage: Heartbeat(type=MavEnumValue(entry=GCS, value=6), autopilot=MavEnumValue(entry=INVALID, value=8), baseMode=MavBitmaskValue(flags=[], value=0), customMode=0, systemStatus=MavEnumValue(entry=UNINIT, value=0), mavlinkVersion=3)
2024-07-10 18:15:34.281 25776-25818 MavFlyControllerWRer    com.test.sample                 D  sendMessage: CommandLong(targetSystem=1, targetComponent=1, command=MavEnumValue(entry=null, value=512), confirmation=0, param1=148.0, param2=0.0, param3=0.0, param4=0.0, param5=0.0, param6=0.0, param7=0.0)
2024-07-10 18:15:34.300 25776-25818 FlyController           com.test.sample                 D  onMessageReceived: Heartbeat(type=MavEnumValue(entry=GENERIC, value=0), autopilot=MavEnumValue(entry=ARDUPILOTMEGA, value=3), baseMode=MavBitmaskValue(flags=[MANUAL_INPUT_ENABLED, STABILIZE_ENABLED, CUSTOM_MODE_ENABLED], value=81), customMode=0, systemStatus=MavEnumValue(entry=CRITICAL, value=5), mavlinkVersion=3)
2024-07-10 18:15:34.300 25776-25818 FlyControlActivity      com.test.sample                 D  onHeatBeat: HeartbeatInfo(type=GENERIC, autopilot=ARDUPILOTMEGA, customModeEnabled=true, testEnable=false, autoEnable=false, guidedEnabled=false, stabilizeEnabled=true, hilEnabled=false, manualInputEnabled=true, safetyArmed=false, customModeInfo=)
2024-07-10 18:15:34.381 25776-25818 FlyController           com.test.sample                 D  onMessageReceived: CommandAck(command=MavEnumValue(entry=REQUEST_MESSAGE, value=512), result=MavEnumValue(entry=ACCEPTED, value=0), progress=0, resultParam2=0, targetSystem=255, targetComponent=1)
2024-07-10 18:15:34.381 25776-25818 FlyController           com.test.sample                 D  onMessageReceived: AutopilotVersion(capabilities=MavBitmaskValue(flags=[], value=0), flightSwVersion=0, middlewareSwVersion=0, osSwVersion=67307519, boardVersion=0, flightCustomVersion=[0, 0, 99, 4, 9, 18, 64, 87], middlewareCustomVersion=[97, 97, 99, 97, 100, 56, 56, 102], osCustomVersion=[0, 0, 0, 0, 0, 0, 0, 0], vendorId=0, productId=0, uid=64495, uid2=[97, 57, 55, 99, 99, 98, 52, 49, 55, 0, 36, 0, 23, 81, 50, 51, 53, 56])
2024-07-10 18:15:34.466 25776-25818 FlyController           com.test.sample                 D  onMessageReceived: Attitude(timeBootMs=11690023, roll=-2.9905877, pitch=0.43072635, yaw=0.7856079, rollspeed=-0.0045892815, pitchspeed=0.001149722, yawspeed=2.5050272E-4)
2024-07-10 18:15:34.466 25776-25818 FlyController           com.test.sample                 D  onMessageReceived: VfrHud(airspeed=0.0, groundspeed=0.005734841, heading=45, throttle=0, alt=0.0, climb=0.004777441)
2024-07-10 18:15:34.667 25776-25818 FlyController           com.test.sample                 D  onMessageReceived: GlobalPositionInt(timeBootMs=11690253, lat=0, lon=0, alt=0, relativeAlt=5294, vx=0, vy=0, vz=0, hdg=4501)
2024-07-10 18:15:34.667 25776-25818 GpsController           com.test.sample                 D  calcDistance: homeLatLng is null

@ZhipengJuly @divyanshupundir
Sorry for delay. I can confirm it.
I captured AutopilotVersion message from the same SITL using MissionPlanner MavLink Inspector and using mavlink-kotlin through debug.

It looks like some shift is present.

@divyanshupundir
Okay. I found the problem. During decoding the logic skips capabilities field because safeDecodeBitmaskValue(8) used -> with value 8 it always returns 0.  Shift is not happening and all next decoding phases are wrong.  So it's the matter of fixing the MavDataDecoder.safeDecodeBitmaskValue(dataSize: Int): UInt extension function.

The problem affects only these 2 messages:

Thanks @yaroslavkulinich and @ZhipengJuly for the detailed analysis. I finally got the problem. I will fix this and push an update soon.

@yaroslavkulinich @ZhipengJuly I have fixed the issue in version 1.2.7-alpha. Can you please test it out? I don't have access to my test setup.
If it works fine, I'll create the stable release in version 1.2.7.

@divyanshupundir Thank you!
Just checked - everything ok!

Awesome! I'll create a release soon.
Thanks for the bug report!


Fix support for dynamic matchers

See https://github.com/rspec/rspec-collection_matchers/pull/48
If a matcher supports dynamic arbitrary methods called on it, e.g.  have_at_least(100).bucks, it would also consider a supports_value_expectations? call as a dynamic part.
Instead of calling supports_*? methods on a matcher, explicitly check if it responds to it.
Symmetrical change for main that suffers when rspec-rails 4.1.0.pre is used and rspec-expectations 3.11.0.pre is at https://github.com/rspec/rspec-expectations/pull/1294

Pretty weird, several builds failed with
Fetching gem metadata from https://rubygems.org/....
Retrying dependency api due to error (2/4): Bundler::HTTPError Net::HTTPBadGateway: I/O error

while https://status.rubygems.org/ reports no incidents. Re-running.


Update GraalVM docs for 2201.7.0 release

Description:

$Subject

Describe your task(s)

[x] Update Build a native executable learn page
[x] Add GraalVM official notice in the release blog
[x] Check the release notes with GraalVM-related changes

Closing this issue as all the tasks are complete now


Add password requirements to login page

closes #90
Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

closes #90
Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

Thanks for fixing the issue. Do you mind sharing the screenshot ?

closes #90
Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

Thanks for fixing the issue. Do you mind sharing the screenshot ?

Nice, will do. We'll see how I get along with the checking-as-satisfied section, but at the very least a static list.

Kind of a funky SVG but does the thing:

Not sure why the useEffect call is getting {password: 'something'} instead of a raw string.

New look with bootstrap SVGs:

Local version is failing with
./src/useConfirm.tsx:2:0
Module not found: Can't resolve 'jotai'
  1 | import * as React from 'react';
> 2 | import { atom, useAtomValue, useSetAtom } from 'jotai';
  3 | import Modal from 'react-bootstrap/Modal';
  4 | import Button from 'react-bootstrap/Button';
  5 |

Import trace for requested module:
./pages/_app.tsx

https://nextjs.org/docs/messages/module-not-found

Which is a weird one because it's installed and worked previously 🤷
Anyway, no screenshot from this round for that reason but the changes should be in place

@jaggederest you might need to run your setup with --build to rebuild the image to include the new packages/setup. Lmk if that doesn't work for you though


Highlighting on sub_path not working for namespaced routes

Basically I've got a namespace called admin and when viewing an entity the generated URLs are incorrect. They use the ID of the viewed entity as part of the path, which is wrong.  Here's some code to chew on in my routes:
namespace :admin do
    resources :bookings, :staff, :pages, :treatments, :clients, :reports, :vouchers

    match 'bookings/cancel/:id' => 'bookings#cancel', :as => :cancel_booking
    match 'treatments/cost/:id' => 'treatments#cost', :as => :treatment_cost, :via => :get
end

Then the navigation.rb file:
primary.item :bookings, 'Bookings', admin_bookings_path, :highlights_on => :subpath
primary.item :treatments, 'Treatments', admin_treatments_path, :highlights_on => :subpath
primary.item :clients, 'Clients', admin_client_path, :highlights_on => :subpath
primary.item :vouchers, 'Vouchers', admin_voucher_path, :highlights_on => :subpath
primary.item :reports, 'Reports', admin_report_path, :highlights_on => :subpath
primary.item :pages, 'Pages', admin_pages_path, :highlights_on => :subpath
primary.item :staff, 'Staff', admin_staff_path, :highlights_on => :subpath

If, for example, I'm viewing the following: http://localhost:3000/admin/clients/2 then the following URL's are generated for each of the links:
http://localhost:3000/admin/bookings
http://localhost:3000/admin/treatments
http://localhost:3000/admin/clients/2
http://localhost:3000/admin/vouchers/2
http://localhost:3000/admin/reports/2
http://localhost:3000/admin/pages
http://localhost:3000/admin/staff/2

Any with /2 on the end are wrong (obviously).
This is running on Ruby 1.9.2 and Rails 3.2.0

Shouldn't those paths be:
admin_clients_path, admin_vouchers_path, admin_reports_path, admin_staffs_path (I admit that last one is a little odd, and will depend on the inflections in use :-)).
Strange that the singular path form is passing the current :id as an argument though, I'll dig in to that further and get back to you...
Cheers,
Mark.

Also, could you elucidate on the version of Rails you're using please?

Based on my initial cursory testing, it looks like this is normal rails behaviour when specifying singular resourceful route helpers without an argument.  So I don't think this is a simple-navigation issue.  Going to close this as an issue, but happy to reopen it if you think otherwise! :-)

Quite right sir! Thanks for the awesome support.
admin_staff_index_path too, pluralization engine is quite clever, even when used for generating the routes.
Cheers!

Hehe, thanks for the thanks!
Glad to hear you've got it working... Have fun! :-D


ORACLE 与SQLSERVER 开启CDC 最低需要什么权限

@kolnick please see doucuments below
https://ververica.github.io/flink-cdc-connectors/master/content/connectors/oracle-cdc.html
https://ververica.github.io/flink-cdc-connectors/master/content/connectors/sqlserver-cdc.html
Can you close this issue if slove this question?

Considering collaboration with developers around the world, please re-create your issue in English on Apache Jira under project Flink with component tag Flink CDC. Thank you!
cc @GOODBOY008

你好你发的邮件我已收到谢谢！Your message has received，thank you for your mail.


Use govukInput for buyer signup form

https://trello.com/c/Ht0k2BEJ/31-1-replace-text-boxes-with-govuk-frontend-text-input-component-in-create-a-buyer-account-journey
The Create Buyer Account form is relatively simple, consisting of just one text input. This PR directly replaces the WTForm-generated widget (which uses the old FE toolkit template) with the govukInput macro, following the steps in the User Frontend (see commit messages).

Bumps govuk-frontend-jinja to fix the indent.njk errors
Use the correct static path for this app
Add in govuk-label override style, based on the User FE equivalent
Replace the form input, using autocomplete settings in line with User FE and attributes from the existing WTForm

Tested the validation banners locally - seems to work ok!
Functional tests are OK up until the usual local Notify failure for this journey.

Awesome!
I'm getting browser validation styles locally if I enter a malformed email address, instead of Error Messages.

@domoscargin that's what we currently have on our User FE login form (where I copied this from... ) what's the recommended behaviour here? Let the browser do its thing or ensure our own error messages get shown?

@domoscargin that's what we currently have on our User FE login form (where I copied this from... ) what's the recommended behaviour here? Let the browser do its thing or ensure our own error messages get shown?

We should turn off HTML5 validation and display error messages: https://design-system.service.gov.uk/patterns/validation/.
That's what we're doing for this form on prod at the mo.

@domoscargin that's what we currently have on our User FE login form (where I copied this from... ) what's the recommended behaviour here? Let the browser do its thing or ensure our own error messages get shown?

We should turn off HTML5 validation and display error messages: https://design-system.service.gov.uk/patterns/validation/.

Interesting, so add novalidate to the form element?
We should do the same for the User FE form as well in that case.

Yes, I think a novalidate will sort it - works for me locally for sure.


Use LRU caching for footers in ParquetInputFormat.

The caching approach needs to change because of issues that occur when the same ParquetInputFormat instance is reused to generate splits for different input directories. For example, it causes problems in Hive's FetchOperator when the FetchOperator is attempting to operate over more than one partition (sidenote: as far as I could tell, Hive has been reusing inputformat instances in this way for quite some time). The details of how this issue manifests itself with respect to Hive are described in more detail here: https://groups.google.com/d/msg/parquet-dev/0aXql-3z7vE/Gn5m094V7PMJ
The proposed patch can be found here: https://github.com/apache/incubator-parquet-mr/pull/2
Reporter: Matt Martin / @matt-martin
Assignee: Matt Martin / @matt-martin
Related issues:

Unnecessary getFileStatus() calls on all part-files in ParquetInputFormat.getSplits (is related to)

Note: This issue was originally created as PARQUET-4. Please see the migration documentation for further details.

Julien Le Dem / @julienledem:
Issue resolved by pull request 2
https://github.com/apache/incubator-parquet-mr/pull/2


Update recover and reset_state

Description of change
This PR makes the following changes:

Remove recover() from Subscriber
Add reset_state() to Author
Remove sync_state() from Author::recover()
Update examples and documents

Addresses #163
Type of change

Enhancement (a non-breaking change which adds functionality)
Documentation Fix

How the change has been tested

All examples updated to reflect need for sync_state() following recover()
Binding examples updated and tested as well

Change checklist

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

Also resolves #127


[CXF-9025] Increase unit test coverage on org.apache.cxf.catalog

@jgoodyear I am closing this one, we have pretty good coverage at the moment already


[BUG]: "make install" command is failing for csm-operator

Bug Description
For running operator in dev mode, it required to run following command in csm-operator folder
make install and make run.
While running make install it will produce following error
Error: load packages in root "/root/csm-operator/tests/e2e": err: exit status 1: stderr: go: updates to go.mod needed; to update it:
go mod ti
Logs
root@master-1-ro7wVbIvOtAmB:~/csm-operator# make install
test -s /root/csm-operator/bin/controller-gen && /root/csm-operator/bin/controller-gen --version | grep -q v0.15.0 ||
GOBIN=/root/csm-operator/bin go install<EMAIL_ADDRESS>/root/csm-operator/bin/controller-gen rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases
Error: load packages in root "/root/csm-operator/tests/e2e": err: exit status 1: stderr: go: updates to go.mod needed; to update it:
go mod tidy
Usage:
controller-gen [flags]
Examples:
# Generate RBAC manifests and crds for all types under apis/,
# outputting crds to /tmp/crds and everything else to stdout
controller-gen rbac:roleName= crd paths=./apis/... output:crd:dir=/tmp/crds output:stdout
    # Generate deepcopy/runtime.Object implementations for a particular file
    controller-gen object paths=./apis/v1beta1/some_types.go

    # Generate OpenAPI v3 schemas for API packages and merge them into existing CRD manifests
    controller-gen schemapatch:manifests=./manifests output:dir=./manifests paths=./pkg/apis/...

    # Run all the generators for a given project
    controller-gen paths=./apis/...

    # Explain the markers for generating CRDs, and their arguments
    controller-gen crd -ww

Flags:
-h, --detailed-help count   print out more detailed help
(up to -hhh for the most detailed output, or -hhhh for json output)
--help                  print out usage and a summary of options
--version               show version
-w, --which-markers count   print out all markers available with the requested generators
(up to -www for the most detailed output, or -wwww for json output)
Options
generators
+webhook[:headerFile=][,year=]                                                                                package  generates (partial) {Mutating,Validating}WebhookConfiguration objects.
+schemapatch[:generateEmbeddedObjectMeta=],manifests=[,maxDescLen=]                                                                                package  patches existing CRDs with new schemata.
+rbac[:headerFile=],roleName=[,year=]                                                                                package  generates ClusterRole objects.
+object[:headerFile=][,year=]                                                                                package  generates code containing DeepCopy, DeepCopyInto, and
+crd[:allowDangerousTypes=][,crdVersions=<[]string>][,deprecatedV1beta1CompatibilityPreserveUnknownFields=][,generateEmbeddedObjectMeta=][,headerFile=][,ignoreUnexportedFields=][,maxDescLen=][,year=]  package  generates CustomResourceDefinition objects.
generic
+paths=<[]string>  package  represents paths and go-style path patterns to use as package roots.
output rules (optionally as output::...)
+output:artifacts[:code=],config=  package  outputs artifacts to different locations, depending on
+output:dir=                               package  outputs each artifact to the given directory, regardless
+output:none                                       package  skips outputting anything.
+output:stdout                                     package  outputs everything to standard-out, with no separation.
run controller-gen rbac:roleName=manager-role crd webhook paths=./... output:crd:artifacts:config=config/crd/bases -w to see all available markers, or controller-gen rbac:roleName=manager-role crd webhook paths=./... output:crd:artifacts:config=config/crd/bases -h for usage
make: *** [Makefile:61: manifests] Error 1
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
clone csm-operator: https://github.com/dell/csm-operator.git
cd csm-operator
then run make install
This will produce the error
Expected Behavior
It should complete the step without any error.
CSM Driver(s)
NA
Installation Type
No response
Container Storage Modules Enabled
No response
Container Orchestrator
NA
Operating System
NA

/sync


can not run Julia2Nix successfully on MacBook M1

When I followed the instructions on my MacBook M1 Pro,  I got the error messages as below:
warning: input 'nix-filter' has an override for a non-existent input 'nixpkgs'
error: attribute 'julia-17-release-aarch64-darwin' missing
   at /nix/store/shndcj46cdfmb1kibdjrhvryc7mxksh4-nix/julia2nix/overlays/installApp.nix:19:12:

       18|   inherit version;
       19|   inherit (julia-sources."julia-${version}-${system}") pname src;
         |            ^
       20|
   Did you mean julia-18-release-aarch64-darwin?

(use '--show-trace' to show detailed location information)
/nix/store/19lf3x5r6z169wg98m6m40y8v8fy4bj9-source/direnv_lib.sh:58: /Users/alan/projects/Julia2Nix/.std/julia2nix/devshells/default/shell-profile/entrypoint: No such file or directory

Only  julia-18-release-aarch64-darwin available on aarch64-darwin;
please try https://github.com/JuliaCN/Julia2Nix.jl/blob/main/templates/dev/flake.nix#L27
package = pkgs.julia_bin-18 or  package = julia2nix.packages.${system}.julia_18-bin;

Furthermore, Could you share your flake config here?

Hi GuangTao,
There is only one line of flake configuration in the file ~/.config/nix/nix.conf
experimental-features = nix-command flakes

Only  julia-18-release-aarch64-darwin available on aarch64-darwin;
please try https://github.com/JuliaCN/Julia2Nix.jl/blob/main/templates/dev/flake.nix#L27
package = pkgs.julia_bin-18 or  package = julia2nix.packages.${system}.julia_18-bin;  instead.

I got the same error as before. It seems that the variable version of Julia is not correct.

Fixed by https://github.com/JuliaCN/Julia2Nix.jl/commit/00d692aed5d2411c234dda58a1dca3bccc30ce41#diff-0aa872778243091e122dbd95590392267ea6e0cf66890d6e1543ed2606e1ee57R27
you can try again.
Notice: Maybe the 1.8 version cannot be used to generate julia2nix.toml under darwin, and the relevant code needs to be updated. PR is welcome

I changed the Julia version in the area of line 14-18 of the file Julia2Nix/nix/julia2nix/devshellProfiles/packages.nix, and I can successfully run the commands you shared above. Before the changes, I got the same error as before.

I changed the Julia version in the area of line 14-18 of the file Julia2Nix/nix/julia2nix/devshellProfiles/packages.nix, and I can successfully run the commands you shared above. Before the changes, I got the same error as before.

Here is the running results:

Fixed by 00d692a#diff-0aa872778243091e122dbd95590392267ea6e0cf66890d6e1543ed2606e1ee57R27 you can try again.
 nix flake lock --update-input julia2nix
nix develop --command julia --version

Notice: Maybe the 1.8 version cannot be used to generate julia2nix.toml under darwin, and the relevant code needs to be updated. PR is welcome

Now I can run the Julia2Nix at the root folder level by following the commands you shared. I am curios how could I use the dev template? Is there a light way of using the template for creating a Julia enabled project? Could you please share the guideline of using the template with me?
Thanks!

you can create the template by the flake-init.
https://github.com/JuliaCN/Julia2Nix.jl#default

Thanks for your information.
Just test the template on MacBook Pro M1, and got the error message as below.

I didn't add the GR of aarch64-darwin. Please set it to false.
https://github.com/JuliaCN/Julia2Nix.jl/blob/main/templates/dev/flake.nix#L35

Thank you GuangTao,
Now I am able to run the template devshell  very well on aarch64-darwin.

@alan-zhang-22 @GTrunSec is this issue now closed?


Optional settings only reinstated in explicit sections

Observed behavior: enabling default settings to be emitted only does so for sections that are already present in the input.
Expected/assumed behavior: that doing this without explicitly mentioning all sections would result in the emission of all other sections along with their defaults to form a fully-populated configuration.
(There are two sections in the specification that are not present in the output below.)
*Data.Ini.Config.Bidir Data.Text> updateIniFile defaultConfig parseConfig (pack "[misc]\ndatabase = foo\n") (UpdatePolicy True False CommentPolicyNone)
Right "[misc]\ndatabase = janadb\nbackend = any\ntmux-session-name = jana\nescrow-path = spdz-keys\nproject-root = .\nlog-directory = logs\n"

This has been addressed with the new API on the bidir branch, and works in my tests now.


fix: quality gate

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

382 of 406 (94.09%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.04%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/TypealizR/CodeFirstSourceGenerator/CodeFirstSourceGenerator.cs
119
122
97.54%

src/TypealizR.CLI/Commands/CodeFirst/ExportCommand.cs
124
145
85.52%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1475

Relevant Lines:
1512

💛  - Coveralls


[1ES] Update extensions build template

Issue describing the changes in this PR
resolves #issue_for_this_pr
Pull request checklist

[x] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[x] My changes should not be added to the release notes for the next release

[ ] Otherwise: I've added my notes to release_notes.md

[x] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[x] I have added all required tests (Unit tests, E2E tests)

Additional information
Update extensions build template to build & pack all projects that start with the given project name

@jviau - I changed the approach here due to a number of limitations - mind re-reviewing?

@jviau - I changed the approach here due to a number of limitations - mind re-reviewing?

Which limitations where you encountering?

An alternative is instead of calling the template multiple times, you could provide multiple projects into the parameter and then for-each add tasks to build them.
parameters:
- name: projects
  type: object

.
.
.

    steps:
    - ${{ each project in parameters.projects }}:
      -  template: build-extension.yml
         parameters:
           - project: ${{ project ""

# build-extension.yml
parameters:
- name: project
  type: string

steps:
  - task: DotNetCoreCli@2
    ....

@jviau - I changed the approach here due to a number of limitations - mind re-reviewing?

Which limitations where you encountering?

The main issue is the NuGet Publish task: there is a conflict error when we publish twice, so I refactored the template to set a publish flag and so the CI would only publish on the second run (using dependsOn). But then I realized this doesn't work as the folders would only contain the one nuget package from the job it was published from. An easy solution would be to have a different name for the nuget package folder that we publish, but I don't like this idea. I would like to keep the naming the same and consistent across pipelines so that release pipelines only need to look for one folder instead of having it look through multiple folders in a given CI run.
After all the back and fourth, I landed on using extensions/${{ parameters.ExtensionProjectName }}*/**/*.csproj, this would allow the pipeline to pick up any extension folder that starts with the project name such as:

extensions/Worker.Extensions.Http.AspNetCore and
extensions/Worker.Extensions.Http.AspNetCore.Analyzers

However, I do like your proposed suggestion of using an object and looping through to run pack for each project via ${{ each project in parameters.projects }}:
Both solve the issue, but the latter lets you be more explicit about which projects are packaged and I prefer that a lot more :)


Laravel 10.13.0 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v10.13.0 branch.
Before merging, you need to:

Checkout the shift-ci-v10.13.0 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:x: Shift was unable to run composer update. Please be sure to run composer update before merging to ensure your dependency changes are fully compatible and reflected in your composer.lock file.


java.lang.Exception: Stack trace

I'm running:
MC 1.12.2 - CatServer (forge 2838) - https://github.com/CatServer/CatServer
TickDynamic: 1.0.2
No other mods installed
I get this spammed error message in the log:
https://pastebin.com/u8xv5RR1

not enough info


chore: update to 6.22.04

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

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

The issue is that clad is being passed the wrong values for LLVM_DIR and Clang_DIR (i.e. the builtin paths not the externals).

This was already fixed in ROOT master for the nightlies, I'll try backporting it https://github.com/root-project/root/pull/6503


Respect email privacy option in user search via API

Fixes #4502 ...
I was also thinking of something along the line of the code below where you use ****_ instead of leaving it as an empty string

if ctx.IsSigned {
        if users[i].KeepEmailPrivate {
              results[i].Email = "*****"
        } else {
             results[i].Email = users[i].Email
        }
 }

I think admins should still see even if user has selected to keep email private

I think leaving as empty string is ok

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4512   +/-   ##
=======================================
  Coverage   20.21%   20.21%
=======================================
  Files         156      156
  Lines       31125    31125
=======================================
  Hits         6292     6292
  Misses      23875    23875
  Partials      958      958

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 d0fef43...51835d9. Read the comment docs.


pf-avatar missing dark variant image implementation

Patternfly v4 has a dark variant avatar image:  https://github.com/patternfly/patternfly-elements/issues/2323
The current version of avatar does not implement this feature.  The dark avatar svg is available in the component, the attribute and image swap need to be added.

@anujsingla asked to grab this

I have made changes for this issue - https://github.com/patternfly/patternfly-elements/pull/2357
closing this issue.


[pulsar-broker] clean up topic that failed to unload from the cache

Motivation
Right now, if topic unloading times out while bundle-unloading then broker doesn't clean up the topic from the cache and broker-stats API report false ownership of the topic which leads to incorrect stats output.
Modification
Clean up topic from the cache if topic unloading fails while namespace-bundle unloading.

/pulsarbot run-failure-checks


Feature/gitlab auth provider

Hey, I just made a Pull Request!
Implement auth handler and sign-in resolvers for GitLab auth providers, as described in #6245
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Tests for new functionality and regression tests for bug fixes
[x] All your commits have a Signed-off-by line in the message. (more info)

Hey I've just added the after-vacation label here, as I want to get some extra eyes from the other maintainers when they're back off holiday, although I think it's on the right lines! 🎉

@Rugvip cleaned this pr up based on your feedback. Should be more in line with the github provider now (as opposed to the MS provider I based this on initially).

Looks like API reports are missing which causes the build to fail. Please run yarn build:api-reports and commit the files.


Classpath not correctly set up

Which version and edition of Flyway are you using?
6.3.1 Community Edition
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Gradle plugin
Which database are you using (type & version)?
Postgresql 10.5 from testcontainers
Which operating system are you using?
Linux, gradle-5.6.4
What did you do?
Executed grdale task flywayMigrate from https://github.com/mschirmacher/generateJooqFromTestContainers/blob/flywayBug/build.gradle.kts
What did you expect to see?
An error indicating there are no migrations (not yet ;-))
What did you see instead?
An error from testcontainers "Database name postgresql not supported" and is thrown here
https://github.com/testcontainers/testcontainers-java/blob/43a76abb5d27f322a6a7a30ffc148beb31cd5736/modules/jdbc/src/main/java/org/testcontainers/jdbc/ContainerDatabaseDriver.java#L101
TestContainers is trying to load an implementation of JdbcDatabaseContainerProviderfrom java's  ServiceLoader
here:
https://github.com/testcontainers/testcontainers-java/blob/43a76abb5d27f322a6a7a30ffc148beb31cd5736/modules/jdbc/src/main/java/org/testcontainers/jdbc/ContainerDatabaseDriver.java#L92
The debugger shows, that no implementation of the interface JdbcDatabaseContainerProvider can be found although the dependency
"org.testcontainers:postgresql:$testcontainersVersion" contains the correct mapping.
This seems to indicate, that the flyway gradle plugin sets up the classpath incorrectly

I can confirm that this issue arises, though with slightly different versions of flyway (6.4.2) and testcontainers (1.14.1).

Have you found a workaround to this?

Unfortunately i was unable to find a workaround

O tried it again with the state my experiment was in, but no success.
I created a minimal example to reproduce the issue: https://github.com/SanderSmee/flyway-issue-2736
When executing ./gradlew flywayMigrate the reported error is the unexpected result.

@Lyeeedar & @DoodleBobBuffPants, I saw this issue was closed in 6b8fb6a which was tagged for version 7.0.3. I tried version 7.0.3 in both @SanderSmee's example repo and my own project. In both cases, I still get the same error and cannot run migrations against a testcontainers database.
Should this issue be reopened? And can you help us resolve the error?

The error is because Testcontainers should be run within a testing framework (e.g. JUnit). If you use the Flyway API within a test against a Testcontainers database it should work, could you try that?

Thanks for @DoodleBobBuffPants for looking into this!
I do have some tests in my applications that run within a testing framework, calling Flyway directly in setup methods. Works great there, no issues. What I'm trying to do now is add additional end-to-end system tests where I start up my application locally and run tests against it. My application is happy to use a Testcontainers database so I just need the migrations run.
I don't know if you'd consider this a valid use case, but I'd like to use the Flyway Gradle plugin to run migrations against a Testcontainers database to support this type of testing. Is it possible to fix the classpath issue that @mschirmacher was pointing out? Or is there some other way to configure the Gradle plugin to achieve this?

The way to achieve this would be to create a test calling Flyway and then you could create a task for that specific test in order to work with Testcontainers.
There isn't a classpath issue, it's just that to run against Testcontainers you must call Flyway from within a test as Flyway doesn't handle the starting and stopping of the database containers.

Closing due to inactivity


Doesn't support Proxy in arguments.

I have
api.someMethod(1, 2, {
   prop: 1
});

I run it from inside VM2. VM2 replaces all external things with Proxy.
So, instead of {prop1: 1} worker-threads will receive instanceof Proxy, which is not transferable between ports (not serializable)
Any thougts?
As dirty hack that cna be fixed by JSON.parse(JSON.stringify())
But what if I want pass a function as callback?
Like
 {
   prop: 1,
  onSomething: () => {console.log("something happened");};
}

If Proxy cannot be passed between two ports, then serializing it is the only way to go.
The only thing I can do is provide a parameter to help you with the serialization.
For functions, I think probably no RPC framework would support using functions as arguments,
although serializing JavaScript functions is possible, but it almost certainly makes the code stink.

The newly added postMessage option allows you to perform a deep copy of the request/response before it is sent, which should solve the Proxy problem.
I tried adding serialization options, but in the end found them really unnecessary, postMessage should solve most of the problems.

Another option has been added: receiveMessage, now you have full control over the format of the intermediate data.

Indeed, adding generic callback is good solution which should work for many use cases.
const rpcDiscordApi = createClient<DiscordApi>(proxyData.discordApiPort, {
        postMessage: (port: MessagePort | Worker, request: IRequest<unknown>) => {
            for (let i = 0; i < request.params.length; i++) {
                if (NodeUtils.isProxy(request.params[i])) {
                    request.params[i] = JSON.parse(JSON.stringify(request.params[i]));
                }
            }
            port.postMessage(request);
        }
    });

Works fine, serializes proxies to plain object.
Now regarding callbacks.
Serializing them is evil, because callback must be executed in context (on the side) where it was created.
Obviously because that lamdba can hold local references to another functions and local state.
When I told about them I expected RPC-like behavior, where callback remains on the side where it was created and only triggered by backward event from RPC.
But this is IPC already, not RPC. Which sounds out of scope of current project.
But anyway... after additional thinking, I come to mind thta I don't need such callbacks at all.
Becuase my requires is - to survive service restarts.
Afte restart all existing lambdas in memory will be killed (no listener will be available after restart)
So I will go with plain events: user did action on one side --> event has sent to another side --> another side is responsible to wire that event and perform callback action.


Fix a panic when changing the max concurrent challenges

What this PR does / why we need it:
Found this one in testing, when you change the max-concurrent-challenges value to a lower one that challenges currently in progress you end up with a negative number for a slice index,
This fixes this preventing a panic() from stopping cert-manager
Release note:

Fix a panic when changing the max concurrent challenges to a lower value

/kind bug

/restest

/retest
now without typo

@wallrj indeed just forgot to push it...

/cherry-pick release-1.0


Attribute Mapping titles are the wrong way around

Under Zendesk - Provisioning - Attribute Mapping the title for each attribute table is incorrect.  The Azure AD Attribute title should be Zendesk and vice versa.  The same applies if you click the "Show Advanced Options".  The two links under this "Edit attribute list for Zendesk" and "Edit attribute list for Azure Active Directory" are titled incorrectly.

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

ID: 16ae6014-a5f6-03f3-4ad4-4425cfe5b6ca
Version Independent ID: 1d3d0543-89f7-180a-e52a-b5668a314e6a
Content: Tutorial: Azure Active Directory integration with Zendesk
Content Source: articles/active-directory/saas-apps/zendesk-tutorial.md
Service: active-directory
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@falccc Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@chetansriv Can you please verify this issue?

@falccc I don't see any issue with the titles. Can you please add screenshot and explain more on what exactly you want to be changed.

@falccc : Thank you for your feedback! This has been resolved as of December 2018.

@zchia thanks for the confirmation

#please-close


add config type

Cleans up daemonset and operator configuration by creating a module to load the necessary items.

/hold
I'll fix the issues

/hold cancel

This PR is ready for review.

/lgtm

/approve

/assign @harche

/lgtm
/approve
verified this change on MoC cluster with real GPUs.


Add isProjecting flag

Is there a reason why this has not been merged?
If there isn't: I would love to see this flag added, to be able to use this package in Laravel Nova, as discussed in #77 :)

Here you go!


When connecting using the websocket protocol, if the username or password is empty, it will crash.

Describe the bug
When connecting using the websocket protocol, if the username or password is empty, it will crash.
To Reproduce
Use any MQTT client at will (such as MQTTX), connect using the WebSocket protocol, set the username and password to empty, and then make the connection. The broker will crash.
** Environment Details **

NanoMQ 0.21.10
Windows 11

Just realized you also submit an issue here, sorry for the late reply


[nethost] Update to 7.0.0

Update the nethost port to v7.0.0 following the latest .NET 7 release announcement: https://devblogs.microsoft.com/dotnet/announcing-dotnet-7/

What does your PR fix?

N/A

Which triplets are supported/not supported? Have you updated the CI baseline?

Same as previous version

Does your PR follow the maintainer guide?

Yes, I've used vcpkg x-add-version nethost

@microsoft-github-policy-service agree

Please get failure logs here. Please ping me if you need any help.
CMake Error at CMakeLists.txt:63 (target_compile_options):
  Cannot specify compile options for target "libnethost" which is not built
  by this project.

Thanks for the hint on checking the error logs on the PR. I fixed all the windows related ones, but the ones on Linux and Mac fail, although I'm not able to understand why or what is going on. Any hint is appreciated. Thanks for reviewing :)

I will try to repro and fix the following issue:
config-x64-linux-out.log

Hi @Adela0814 any update on this PR?

Hi @Adela0814 any update on this PR?

I'm sorry I haven't had time to fix this recently. I'll find time to check it as soon as possible.

Sorry, I can't solve this problem at the moment, I suggest convert this PR into draft. You can also try to seek help from upstream.

Let's see if rebasing to master helps, thanks!

@Adela0814 I managed to find what the issue was, apparently the project has a custom way to provide a special file included only when the build is static ( hit only by Linux and OSX targets ). I fixed that and now it builds fine under all platforms.


Rename blocks folder to block

The folder 'blocks' should be 'block' otherwise the datapack wont work.

Updated, thanks!


prometheus-dnsmasq-exporter: init at 2018-05-05, add module

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build prometheus-dnsmasq-exporter

Success on x86_64-linux (full log)
Attempted: prometheus-dnsmasq-exporter
Partial log (click to expand)
shrinking /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin/bin/dnsmasq_exporter
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin/bin
patching script interpreter paths in /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin
checking for references to /build in /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/4fdacyi4ciyc0qgh8aqws5azz5kx75sc-dnsmasq_exporter-unstable-2018-05-05
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/4fdacyi4ciyc0qgh8aqws5azz5kx75sc-dnsmasq_exporter-unstable-2018-05-05
checking for references to /build in /nix/store/4fdacyi4ciyc0qgh8aqws5azz5kx75sc-dnsmasq_exporter-unstable-2018-05-05...
/nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin

Success on x86_64-darwin (full log)
Attempted: prometheus-dnsmasq-exporter
Partial log (click to expand)
installing
/private/tmp/nix-build-dnsmasq_exporter-unstable-2018-05-05.drv-0/go /private/tmp/nix-build-dnsmasq_exporter-unstable-2018-05-05.drv-0
/private/tmp/nix-build-dnsmasq_exporter-unstable-2018-05-05.drv-0
post-installation fixup
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/4lwv29g7x3b7g4xgy362a8fvzvp3554f-dnsmasq_exporter-unstable-2018-05-05-bin/bin
patching script interpreter paths in /nix/store/4lwv29g7x3b7g4xgy362a8fvzvp3554f-dnsmasq_exporter-unstable-2018-05-05-bin
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/avhnlp3rhacwqzlq2hgbq8ndvmm7n5q2-dnsmasq_exporter-unstable-2018-05-05
/nix/store/4lwv29g7x3b7g4xgy362a8fvzvp3554f-dnsmasq_exporter-unstable-2018-05-05-bin

Success on x86_64-darwin (full log)
Attempted: prometheus-dnsmasq-exporter
Partial log (click to expand)
/nix/store/4lwv29g7x3b7g4xgy362a8fvzvp3554f-dnsmasq_exporter-unstable-2018-05-05-bin

Success on aarch64-linux (full log)
Attempted: prometheus-dnsmasq-exporter
Partial log (click to expand)
shrinking /nix/store/xayj19sl19ra2j65k6yq9dbm8msh5llr-dnsmasq_exporter-unstable-2018-05-05-bin/bin/dnsmasq_exporter
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xayj19sl19ra2j65k6yq9dbm8msh5llr-dnsmasq_exporter-unstable-2018-05-05-bin/bin
patching script interpreter paths in /nix/store/xayj19sl19ra2j65k6yq9dbm8msh5llr-dnsmasq_exporter-unstable-2018-05-05-bin
checking for references to /build in /nix/store/xayj19sl19ra2j65k6yq9dbm8msh5llr-dnsmasq_exporter-unstable-2018-05-05-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/dx8jx9z0b0hrb7n3i8rznbia42b123h9-dnsmasq_exporter-unstable-2018-05-05
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/dx8jx9z0b0hrb7n3i8rznbia42b123h9-dnsmasq_exporter-unstable-2018-05-05
checking for references to /build in /nix/store/dx8jx9z0b0hrb7n3i8rznbia42b123h9-dnsmasq_exporter-unstable-2018-05-05...
/nix/store/xayj19sl19ra2j65k6yq9dbm8msh5llr-dnsmasq_exporter-unstable-2018-05-05-bin

Success on aarch64-linux (full log)
Attempted: prometheus-dnsmasq-exporter
Partial log (click to expand)
these derivations will be built:
  /nix/store/jf6yrzjb1ra8yfi52s2yv5mww27irhid-go-1.10.1.drv
  /nix/store/rjzk6xcgsql58q5jf4yv1h78s5arsfrf-govers-20150109-3b5f175.drv
  /nix/store/r0mknlryv4rqfsqx6vijcbnw4nysj06x-dnsmasq_exporter-unstable-2018-05-05.drv
waiting for locks or build slots...
/nix/store/xayj19sl19ra2j65k6yq9dbm8msh5llr-dnsmasq_exporter-unstable-2018-05-05-bin

Success on x86_64-linux (full log)
Attempted: prometheus-dnsmasq-exporter
Partial log (click to expand)
shrinking /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin/bin/dnsmasq_exporter
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin/bin
patching script interpreter paths in /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin
checking for references to /build in /nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/4fdacyi4ciyc0qgh8aqws5azz5kx75sc-dnsmasq_exporter-unstable-2018-05-05
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/4fdacyi4ciyc0qgh8aqws5azz5kx75sc-dnsmasq_exporter-unstable-2018-05-05
checking for references to /build in /nix/store/4fdacyi4ciyc0qgh8aqws5azz5kx75sc-dnsmasq_exporter-unstable-2018-05-05...
/nix/store/awdg2kiy42lcd87349xg9213rhqv121y-dnsmasq_exporter-unstable-2018-05-05-bin


Migrate data from ExperimentSeries

After #875 , data migration will be needed:
Create ExperimentSeriesGroup for first_experiments with highest order, for random_experiments with medium order, and for last_experiments with lowest order, and move the dashboard value from random_experiments to the associated ExperimentSeriesGroup. For every pk currently in the list, retrieve the associated experiment, create a GroupedExperiment and assign it to the ExperimentSeriesGroup in question.
This could be done through a Django migration, OR we might just recreate the currently existing ExperimentSeries manually through the admin interface, which at this moment would probably be overseeable.
After this, the following fields of ExperimentSeries can be deleted:

first_experiments
random_experiments
last_experiments
dashboard

Done on acceptance and production server.


Generates release assets without goreleaser on 'make dist'

This removes the dependency on goreleaser by implementing what it does
in make.
This will fix #352 once the release process uses it, which will be a
different PR.

± |make-dist {3} ✓| → file dist/*
dist/func-e_dev_checksums.txt:       ASCII text
dist/func-e_dev_darwin_amd64.tar.gz: gzip compressed data, last modified: Sun Aug 29 04:19:00 2021, from Unix, original size modulo 2^32 7503360
dist/func-e_dev_darwin_arm64.tar.gz: gzip compressed data, last modified: Sun Aug 29 04:19:01 2021, from Unix, original size modulo 2^32 7743488
dist/func-e_dev_linux_amd64.tar.gz:  gzip compressed data, last modified: Sun Aug 29 04:19:01 2021, from Unix, original size modulo 2^32 7005696
dist/func-e_dev_linux_arm64.tar.gz:  gzip compressed data, last modified: Sun Aug 29 04:19:02 2021, from Unix, original size modulo 2^32 6948352
dist/func-e_dev_windows_amd64.msi:   Composite Document File V2 Document, Little Endian, Os: Windows, Version 5.0, MSI Installer, Code page: 1252, Title: Installation Database, Subject: func-e installer, Author: Tetrate, Keywords: Installer, Comments: This installer database contains the logic and data required to install func-e., Template: x64;1033, Revision Number: {D3FEA555-CCBD-4DCF-9FE3-55E05B7C63ED}, Create Time/Date: Sun Aug 29 04:19:04 2021, Last Saved Time/Date: Sun Aug 29 04:19:04 2021, Number of Pages: 310, Number of Words: 2, Name of Creating Application: msitools 0.101, Security: 2
dist/func-e_dev_windows_amd64.zip:   Zip archive data, at least v2.0 to extract

ps the msi action uses the work here, but it will be used in the release workflow also, later once we refactor out the old makefile.

ok so looks like the windows image maybe doesn't have zip.. or maybe it needs the paths different.. I will try later on my windows host.
process_begin: CreateProcess(NULL, zip -qj dist/func-e_dev_windows_amd64.zip build/func-e_windows_amd64/func-e.exe, ...) failed.
make (e=2): The system cannot find the file specified.

yeah looks like zip isn't installed.. maybe we ask people to install it. otherwise, it gets complicated https://github.com/bamler-lab/constriction/blob/831ac417526d14de34977aa2665ce752d3a5a5fc/.github/workflows/release.yaml#L199-L210

7zip might do the trick easier https://github.com/ducaale/xh/blob/8e90649a535e82c07b0ec380f7e93d019236c890/.github/workflows/release.yaml#L98-L104
I will work on this later

ok hopefully I got it. I also removed the ✔ from the output as it wasn't formatting on windows
🤞

ok all tidied up.. if windows fails on github it is different than local and will report back before saying "review please"

heh next is shasum.. somehow it must be there on my machine but not on the github actions windows worker
/usr/bin/sh: shasum: command not found
mingw32-make: *** [Next.mk:104: dist/func-e_dev_checksums.txt] Error 127
rm build/func-e_linux_arm64/func-e build/func-e_darwin_arm64/func-e build/func-e_windows_amd64/func-e.exe.signed build/func-e_linux_amd64/func-e build/func-e_darwin_amd64/func-e
Error: Process completed with exit code 2.

ok all OS-specific things are sorted I think

thanks for the help folks!


packet: Consider removing tf variable setup_raid_*

Right now there are several variables: setup_raid_ssd_fs, setup_raid_ssd, setup_raid_hdd and setup_raid.
The code is messy and cause several regressions already (https://github.com/kinvolk/lokomotive-kubernetes/pull/72 and https://github.com/kinvolk/lokomotive-kubernetes/commit/85ef5fa2d88661be5904b8ddf1033d1c31635a2f to name two, I'm sure there are more).
We should re-consider why these variables were added and see if we can remove some to simplify things. Currently, also, they can't be used at the same time as it was expected, because of: https://github.com/kinvolk/lokomotive-kubernetes/issues/73.
The goal of this issue is to double check we need these variables or not, to make sure we remove all the unneeded code

We need atleast creating RAID of ssd devices if nothing else for the Rook Ceph to utilize the aggregated SSD device as a metadata device.


Microphone for speech to text in chat

Is your feature request related to a problem? Please describe.
Implement a microphone for speech to text in chat
Describe the solution you'd like
User taps on the microphone which starts listening for input which is then rendered on the input box once the user stops speaking
Additional context
We will leverage Web Speech API for this. Chrome and Mozilla have their implementations but for Safari we will have to see what can be used.

This has been implemented


Design and run fetcher test for networking incompatibility post-fast sync, including tarpit

The feat-fast-sync branch has had a feature added to recently to reject nodes running a legacy version of the node. An nctl test should be created to test this functionality, ideally also with the tarpitting feature enabled.

NCTL related. Need to figure out who is going to support this.
Note - Bundle all NCTL related issues into this category.


Downloading attachments do device doesn't seem to work

You can close this, I said that it started working afterwards. This is @intellijence on discord btw.

You can close this, I said that it started working afterwards. This is @intellijence on discord btw.

While this is similar to your issue, I still think there is room for improvement on our end

Yeah ig. Wait I don't remember making this discord account, whose account am I on?

lmao


Rescue and handle Octokit::Unauthorized when viewing user pages

It's possible for a user's OAuth token to get invalidated by whoever manages the connected OpenSourceFriday OAuth app on GitHub.  With the current behavior, the user will be shown a 500 error page which doesn't describe what went wrong or how to fix it.
Fixing the problem is pretty straightforward though, the user needs to sign out and sign back in to go through the OAuth flow again and generate a new token.  To give a better overall experience, I've updated the UsersController#show action to catch the error when it occurs and then

sign the user out
redirect them to the homepage
prompt them to sign in again with a flash error message

I reproduced the issue when running the app locally per the README instructions (🙇   very helpful!) and with the change in this PR the user will end up seeing the following after being redirected:

I added the token to the english localization file but I don't speak the other languages to supply non-english translations.

Thanks for the PR @jonabc but this site has now been ported to GitHub Pages so this is no longer needed!


[Snyk] Security upgrade ruby from 2.6.5 to 2.7.6

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

openshift-configs/Dockerfile

We recommend upgrading to ruby:2.7.6, as this image has only 370 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

714
Integer Overflow or Wraparound SNYK-DEBIAN10-GIT-3232718
No Known Exploit

714
Out-of-bounds Write SNYK-DEBIAN10-IMAGEMAGICK-539706
No Known Exploit

714
Out-of-bounds Read SNYK-DEBIAN10-IMAGEMAGICK-539710
No Known Exploit

714
Buffer Overflow SNYK-DEBIAN10-PYTHON37-1063182
No Known Exploit

714
Integer Overflow or Wraparound SNYK-DEBIAN10-PYTHON37-3090928
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Checkmarx One – Scan Summary & Details – 6b23edd8-e908-4333-984a-6d8c85eb254f
No New Or Fixed Issues Found


using local system time to sign AWS requests

AWS request signatures include the request time, and many AWS services will refuse a request if that signature indicates a time that is more than 5 minutes off from the service's time.  The AWS C++ SDK signs these requests using local system time and on some devices, particularly PCs, it is probably not a good idea to trust system time.  It would be nice to have an option that syncs with ntp or some other trusted source upon SDK initialization and uses that for request signing, or allow us to more easily provide our own time function (which can be done now, I believe, by implementing your own signer but you need to implement that entire class).

I've added an internal task for this.

This problem manifests when the system clock is incorrect. While this is an infrequent problem for online services using the SDK it is a problem for end-user applications where mis-configuration of personal computers is more common. Further, end-users may not be able to easily correct this problem, as for example when running as a limited user without permissions to re-synchronize the system clock.
I'd like to suggest that the SDK enable the caller to override std::chrono::system_clock::now so that the user can provide a different time value based on user-level time-synchronization with an external time source other than NTP.
Overriding the time function would allow the user-code to receive a timestamp from a server, compute the delta from current time, and provide modified time-stamps back to the AWS SDK for signature-creation purposes.
This change would not create a security hole because the user code already has the capability to effectuate a similar change using API hooking or similar code-modification techniques.

Instead of overriding  std::chrono::system_clock::now, the signing algorithm should accept a DateTime parameter (possibly with a defaulted value of DateTime::Now()). It would be the caller's responsibility to call DateTime::Now() and pass the result to the signer .

I would suggest instead of passing a DateTime to allow caller to provide an offset to be applied by SDK every time it queries std::chrono::system_clock::now. Similar behavior as in aws-sdk-js: https://github.com/aws/aws-sdk-js/commit/3b14ada908fa6d5515d956386b35b3865ac871c2
Even better would be to have an option to automatically correct failed requests due to clock skew within the SDK itself without the client having to calculate and specify an offset, again similar to aws-sdk-js: https://github.com/aws/aws-sdk-js/pull/706

In my observation of this failure, the PC clock may be correct but the Local & TimeZone settings cause the signature time to be off by an hour or two and induce the failure.

We shouldn't be concerned about time zones since we pull the gmt time. Do you think this is a case of a user adjusting the time to their time zone instead of setting their timezone?
Sent from my iPhone

On Sep 16, 2016, at 2:06 PM, mmustafa-hp<EMAIL_ADDRESS>wrote:
In my observation of this failure, the PC clock may be correct but the Local & TimeZone settings cause the signature time to be off by an hour or two and induce the failure.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Not really sure what is causing it.  We are seeing instances in many countries as well. So your guess is good. Some users in countries with multiple time zones may be setting the time zone and adjusting the clock.

I just pushed a fix for this. The way it works is as follows. If we recieve a signing error, we check the date header returned from the server. If it's off, we automatically adjust the clock skew and retry the request. Once it is adjusted the first time, the client will have the correct timestamp. This should happen invisibly to the user without any errors surfacing.
Feel free to reopen if you still have problems or have any further questions.

Thank you. We will integrate and test.

Does anyone know if the AWS Amplify handles clock skew like this, i.e. transparent adjustment?