Emit buffer for fields

Feature request #323
When dealing with a third-party tool I noticed that some of the part of the multipart were with a particular charset, and others were with utf-8.
Since busboy assumes that all fields are from the same charset, this leads to an issue when parsing.
I added a raw buffer to the data emitted for the fields to be able to directly work with the Buffer and parse it with the correct charset.
bb.on('field', (name, val, buffer, info) => {
      console.log(`Field [${name}]: value: %j`, val);
      console.log(`Field [${name}] raw buffer`, buffer)
    });

I think a better and more backwards compatible solution would be to add a 'buffer' charset that returns the Buffer instead of converting to string.

I think a better and more backwards compatible solution would be to add a 'buffer' charset that returns the Buffer instead of converting to string.

Uhm, I do not follow you.
This was the fast solution I implemented in our scenario
Maybe a config to switch from converted string to Buffer in the val parameter?

Additionally regarding the backwards compatibility, while this solution may work for your use case, breaking everyone else's code by adding another parameter to event handlers is not a good idea. If this PR's changes were added as-is, everyone would suddenly be getting a Buffer instead of the info object containing the various pieces of information about the part.

Additionally regarding the backwards compatibility, while this solution may work for your use case, breaking everyone else's code by adding another parameter to event handlers is not a good idea. If this PR's changes were added as-is, everyone would suddenly be getting a Buffer instead of the info object containing the various pieces of information about the part.

Yeah, you are absolutely right, I will rewrite this to ensure that the code will not break for everyone. Thanks for the suggestion

Owever, there should be a config to force the buffer charset. The defCharset config is overwritten if in the headers we have the charset specified. something like forceFieldBuffer or similar in order to always output the buffer in the val parameter instead of the parsed text

@mscdex in the end I added a new config.
If I would like to specify a charset this would be however overwritten if another charset is found inside the multipart body.
By adding a config field I will overwrite this behaviour and emit field as buffer only in that situation. By default the field is false to avoid breaking changes

Hey @mscdex, I was just wondering if you had the time to review the latest changes I applied


test(hooks): add hooks testing library too

The type of this PR is: Enhancement

This PR resolves []
Description
After adding the react native testing library, it's easy to add the hooks one too.
I added the lib, and moved two hook tests we have to it. One of the tests is a bit funky where it checks the rerenders too, so we still need to use the regular testing inside a component, which is totally fine.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Add hooks testing library - pavlos

This PR contains the following changes:

Dev changes (Add hooks testing library - pavlos)

  Generated by :no_entry_sign: dangerJS against b2625bd491a927afd98809d432776475c72e6e26

There is a weird error on circle ci on detect-secrets check 🤔 :
#!/bin/bash -eo pipefail
detect-secrets-hook --baseline .secrets.baseline --exclude-secrets '[a-fA-F0-9]{24}' --exclude-lines 'W/"[!#-\x7E]*"' $(git ls-files)

The baseline file was updated.
Probably to keep line numbers of secrets up-to-date.
Please git add .secrets.baseline, thank you.
Exited with code exit status 3
CircleCI received exit code 3

This is great! But can you rename it to chore(testing): add hooks testing library too please? enhancement is not in the spec.

@admbtlr I changed it to test(hooks) since it's about tests. I thought we are using the format but not the names? Am I remembering wrong?

Fixing the secrets thing now

@admbtlr I changed it to test(hooks) since it's about tests.

test usually means that you're adding tests. chore is for infrastructure, such as adding libraries - which I think is more relevant in this case.

I thought we are using the format but not the names? Am I remembering wrong?

I don't remember this ever being decided, but maybe I missed something?

I'll add this to the next KS or mobile practice to talk about this 👍.

What is happening right now?! We need to make this step be in CI. CI can update this file, we shouldn't need to bother.

#squashongreen


AP-3158 add JSON schema validation to irregular incomes

What
Link to story
Describe what you did and why.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

We could also remove references to irregular_incomes from apipie.rb

@stephenrichards some minor comments/suggestions otherwise looks good


Add shields to README

See https://github.com/open-horizon-services/service-homeassistant for an example of shields to add

Also, consider adding the "Contributor" shield - example in https://github.com/open-horizon-services/service-node-red-hello-world

This README needs better information:

Front matter
Shields - license should match LICENSE file
Description
Prerequisites
Installation
Usage
Advanced Details
Authors - should match MAINTAINERS.md

See examples of good README.md files:

https://github.com/open-horizon-services/service-node-red-hello-world/blob/main/README.md
https://github.com/open-horizon-services/service-fledge/blob/main/README.md
https://github.com/open-horizon-services/service-homeassistant/blob/main/README.md


Initial Vue-Doxen throws: TypeError: crypto.randomUUID is not a function

I started implementing Vue-Doxen on a project.
I implemented the code I found here: https://thejaredwilcurt.com/vue-doxen/getting-started#getting-started then changed the ComponentA and ComponentB with actual components from my app.
<template>
    <div style="min-height: 1000px">
        <DoxenSidebar
            v-model="selectedDemo"
            :demos="demos"
        />
        <VueDoxen
            v-model="selectedDemo"
            :demos="demos"
        />
        <link
            href="https://unpkg.com/highlightjs/styles/ir-black.css"
            rel="stylesheet"
            type="text/css"
        >
    </div>
</template>
<script setup>
import 'vue-doxen/vue-doxen.css';

import Button from<EMAIL_ADDRESS>import PrimaryButton from<EMAIL_ADDRESS>import { computed, ref } from 'vue';
import { DoxenSidebar, VueDoxen } from 'vue-doxen';

defineOptions({
    name: 'DocumentationPage',
});

const selectedDemo = ref('PrimaryButton');

const demos = computed(() => ({
    Button,
    PrimaryButton: {
        component: PrimaryButton,
    },
}));
</script>

On the console, I get the following error:
vue-doxen.js?v=fe0c8af0:5924 Uncaught (in promise)

TypeError: crypto.randomUUID is not a function
    at Proxy.uniqueId (vue-doxen.js?v=fe0c8af0:5924:21)
    at ReactiveEffect.fn (chunk-SZCTCOGP.js?v=fe0c8af0:1236:13)
    at ReactiveEffect.run (chunk-SZCTCOGP.js?v=fe0c8af0:435:19)
    at get value (chunk-SZCTCOGP.js?v=fe0c8af0:1248:107)
    at Object.get [as uniqueId] (chunk-SZCTCOGP.js?v=fe0c8af0:4970:22)
    at Object.get (chunk-SZCTCOGP.js?v=fe0c8af0:4530:19)
    at Proxy.idFor (vue-doxen.js?v=fe0c8af0:5929:24)
    at ReactiveEffect.fn (chunk-SZCTCOGP.js?v=fe0c8af0:1236:13)
    at ReactiveEffect.run (chunk-SZCTCOGP.js?v=fe0c8af0:435:19)
    at get value (chunk-SZCTCOGP.js?v=fe0c8af0:1248:107)

   App is Laravel/Vue. Details:
   vue: 3.4.25
   vite: 5.2.8
  inertiajs/vue3: 1.0.15

Thank you

What version of Node.js are you using?
What browser are you using?
I didn't realize that support for crypto.randomUUID() was only added in mid 2021. So it requires Node 19+.

https://developer.mozilla.org/en-US/docs/Web/API/Crypto/randomUUID#browser_compatibility
https://caniuse.com/?search=crypto.randomUUID

Since Node 18 is still supported until next year, I'll swap out the native random UUID generator for a custom implementation.

https://nodejs.org/en/about/previous-releases

Thank you for your fast response!
I was on 18.19.1 version and I did an update to 20.14.0, but the issue insists.

All uses of crypto.randomUUID are removed from Vue-Doxen as of v0.1.5.
Update to the latest version. Let me know if you run into any other issues.

Good morning! Works like a charm! Thank you @TheJaredWilcurt !


How to Dynamic newElements : false

Hi zellerda,
thank for this great plugin.
I am Using three multi select drop downs in a form by tokenize, i  want one field can accept new elements adding , but the remain two shouldn't accept - newElements : false is needed dynamically.
Configuring in js file reverts all. can u pull me out from this .... Thank u in Advance..

Hi can you send me your code so I can check out ? You can send it to<EMAIL_ADDRESS>

Sent you the code of my form

Sorry ! this is the wrong address (not used anymore) send it to<EMAIL_ADDRESS>

ok, Sent to the new email address... thank u.

I'm not sure to understand your problem ! Just add
$('#work_status').tokenize({ newElements: false });

If you want to change this value after loading, try this but I'm not sure that will work ?
$('#work_status').data('tokenize').options.newElements = false;

ok thank u,
$('#work_status').tokenize({ newElements: false }); is suits for me.
thank u...


Fixed the Let's encrypt issues.

Initially, I was unable to add Let's encrypt SSL. Because of the nginx configuration, it was failing to start as there were no certificates available in the directory. So, with the fix, it will add dummy certificates in the dir and will update it with the Let's encrypts SSL

The repo has been significantly refactored, and as such conflicts with this PR.
The refactored work should resolve the issue that this PR was addressing.
I am closing this PR as it stands, but do feel free to open a new one based on the updated repository if your issue still persists.
Thank you,


Make Ingoing and Outgoing Transactions Easier to Distinguish

Description
Make ingoing and outgoing transactions easier to distinguish.
Motivation
It's rather difficult to differentiate incoming and outgoing transactions.
Requirements
I propose to go with the rest of the world on this, and make outgoing transactions red.
Are you planning to do it yourself in a pull request?
No.

Hey @Oskar-F, thanks for the suggestion. I've moved this issue to a discussion here 👍


React warning in console when passing lottieRef to Player component

Hi, I'm using the LottiePlayer component like so:
import type { AnimationItem } from 'lottie-web';
import { Player } from '@lottiefiles/react-lottie-player';
...
const lottieRef = useRef<AnimationItem>();
...
<Player
    lottieRef={(ref) => {
        lottieRef.current = ref;
    }}
    loop
    src="..."
/>

Everything works perfectly, except I'm getting the following error in the chrome dev tools console:

Is there any way I can get rid of that warning ?
Thanks!

If you want to set the ref for the component the syntax is different.
const lottieRef = useRef<AnimationItem>();
...
<Player
    ref={lottieRef}
    loop
    src="..."
/>

This syntax here returns the instance of the Lottie via the lottieRef prop through the callback method
 lottieRef={instance => {
          this.setState({ lottie: instance }); // the lottie instance is returned in the argument of this prop. set it to your local state
          }}

Hi @jon301 . The prop can be renamed lottieref on our side to fix this but that would be a breaking change to everyone using this prop. since we ve already named the prop as lottieRef, could you put the lottieRef along with all the other props into an object. and use the spread operator to pass in all the props into the player?

Heres a stack overflow guide on the issue

Closing issue due to inactivity.


HADOOP-16878. FileUtil.copy() to throw IOException if the source and destination are the same

Backport to branch-3.3
https://issues.apache.org/jira/browse/HADOOP-16878

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
29m 35s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 2 new or modified test files.

_ branch-3.3 Compile Tests _

+0 :ok:
mvndep
3m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
26m 23s
branch-3.3 passed

+1 :green_heart:
compile
15m 26s
branch-3.3 passed

+1 :green_heart:
checkstyle
2m 41s
branch-3.3 passed

+1 :green_heart:
mvnsite
2m 53s
branch-3.3 passed

+1 :green_heart:
shadedclient
20m 59s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
2m  0s
branch-3.3 passed

+0 :ok:
spotbugs
3m  5s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m 11s
branch-3.3 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 25s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 58s
the patch passed

+1 :green_heart:
compile
14m 52s
the patch passed

+1 :green_heart:
javac
14m 52s
the patch passed

+1 :green_heart:
checkstyle
2m 40s
the patch passed

+1 :green_heart:
mvnsite
2m 51s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedclient
14m 34s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
2m  1s
the patch passed

+1 :green_heart:
findbugs
5m 23s
the patch passed

_ Other Tests _

-1 :x:
unit
10m  0s
hadoop-common in the patch passed.

-1 :x:
unit
118m 59s
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m  2s
The patch does not generate ASF License warnings.

282m 50s

Reason
Tests

Failed junit tests
hadoop.security.TestGroupsCaching

hadoop.hdfs.web.TestWebHDFS

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2383/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2383

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 14d21fe55d04 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 351b149

Default Java
Private Build-1.8.0_265-8u265-b01-0ubuntu2~16.04-b01

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2383/1/artifact/out/patch-unit-hadoop-common-project_hadoop-common.txt

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2383/1/artifact/out/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2383/1/testReport/

Max. process+thread count
3680 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2383/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

LGTM, +1


How to use auto-wired instance/key configuration from receiver?

Hello i've tried the following requests to see this work but continue to get the error "Could not find a valid configuration for WebHook receiver 'Pipedrive' and instance '#'. The setting must be set to a value between 32 and 128 characters long."
I've tried looking at all the different samples and tracing the code down and I can't really figure out why it won't match up under any configuration.
Requests:

POST /api/webhooks/incoming/pipedrive/1?code=bbbbaaaaaaaabbbbbb94583d6cf4cf1c0c7f767
HTTP/1.1
Host: localhost:30617
Content-Type: application/json

POST /api/webhooks/incoming/pipedrive/1 HTTP/1.1
Host: localhost:30617
Content-Type: application/json

POST /api/webhooks/incoming/pipedrive/bbbbaaaaaaaabbbbbb94583d6cf4cf1c0c7f767
HTTP/1.1
Host: localhost:30617
Content-Type: application/json

Configuration:
<add key="MS_WebHookReceiverSecret_Pipedrive" value="aaa, 1=bbbbaaaaaaaabbbbbb94583d6cf4cf1c0c7f767, 2=3bbbbbbbbbbbb33333333333fffffffffd" />
and also tried just value = key (third POST request).
code:
var receiverConfig = await this.GetReceiverConfig(pRequest, Name, pID, 32, 128);
and await EnsureValidCode(pRequest, pID);

The immediate question is what would I loose by accessing Configuration setting directly in the receiver using ConfigurationManager.AppSettings and just parsing this out myself to validate the instance and code if I wanted to enforce ?

Diego,
When you start up the server, you can see what is registered in the log output. That should hopefully tell you what is going on.
Let me know whether that helps!
Henrik


Enable Graphql in Kong

Graphql routes are missing from Kong Config
Feature request
Graphql should be enabled in Kong just like it is in the master supabase repo self-hosted example
Is your feature request related to a problem? Please describe.
Yes, currently we cannot query graphql in the version of supabase produced by the CLI, though #232 fixes the database-level concern there.
Describe the solution you'd like
I think all that needs to happen is the addition of the new route to Kong... but we may also have to update the postgrest container version. Not sure yet.

Made a small attempt here https://github.com/supabase/cli/pull/238, but I'm not familiar with pg_graphql. Are able to verify if that works?


Decide which element should move

When drag and drop, the whole element is moving. Instead of moving all the elements, We can decide which element need to move.

The only way you could implement this is by using a custom template and changing the element with the mwl-draggable directive.


Adding a custom variable to the release name

The section "How do I manage the names for new releases?" says that you can use a "custom variable" in your release name, but this doesn't seem to be working for me.  Could you clarify in the docs whether this is supposed to work, or help me understand what I'm doing wrong?
I added a "Pipeline Variable" in "Variables" tab and then tried to reference it in the "Options" -> "Release name format". When I created a new release, it showed up in the name as "$(Variable)" instead of substituting the value of that variable.

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

ID: 63234ade-28cc-02fc-a319-42e3491da601
Version Independent ID: 7ccbfe96-d61e-8bf2-de22-61150d648abd
Content: Release pipelines - Azure Pipelines
Content Source: docs/pipelines/release/index.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

It seems the original question (How do I reference custom pipeline variables) was not answered here. If I create a pipeline variable, which the user enters when they create a new release - how do I use this variable in the "Release name format". I created ReleaseNo but found no way of referencing it Parts-$(ReleaseNo) does not work and neither does `Parts-$(Release.ReleaseNo).

This does not seem to cover 'custom' variables.
For example, a variable value is stored in a variable group. How do we add it to the 'release name format'?
In my case I'd like to have it this format: Data-Restore-$(sourceEnv)-to-$(targetEnv)-Release-$(rev:r)
sourceEnv and targetEnv are variables in a group.


How Loggable for extended entity?

I have User entity which extends Fos User Bundle entity, how to log field on super class?
I'm using yml

Sorry, this issue solved by using attribute overrides


Feat/pg semicolon support

Implementing support for finishing postgres queries with a semicolon per this issue.
Changes include:
- replace statements to remove semicolon from query in connectorx-python/connectorx/init.py (read_sql)
- Tests to confirm that command terminating semicolons are being removed.

@wangxiaoying I think this one is ready for review. Let me know if/what changes are needed. Thanks in advance!

Hi @Jordan-M-Young , thanks for the PR!
I think my concern is that ';' may also appears in other parts of the query, such as string matching. For example: select s from t where s like '%abc;'. In this case, simply replace ";" with "" will change the semantic of the sql query.
I would suggest to only replace the ";" if it is the last valid character in the sql query. Such as select s from t; or select s from t;[spaces]  but not select s from t where s like '%abc;'

@wangxiaoying Thanks for the suggestion. I believe my newest commit should allay that concern. If you wouldn't mind taking another look I'd appreciate it.

Thank you @Jordan-M-Young , I have merged the PR


Gzip flag overrides output file flag

Hi,
Using a combination of -g and -b flags produces an unexpected result.
.\ThermoRawFileParser.exe -N -f 1 -g -i="test.raw" -b="test.mzML.gz" creates a file named "test.mzML.gz.gzip"
I would also like to request, perhaps, that the default extension for using the -g flag be changed to .gz, which is the standard extension from gzipped files (for instance, gunzip doesn't recognize .gzip....)

Hi @lazear,
thank you for reporting this issue. Do I understand correctly, that "less surprising" result will be to use provided filename as it is (in layman terms - "user knows better")?
For example, .\ThermoRawFileParser.exe -g -i="test.raw" -b="test.mzML" should generate test.mzML that is in fact a gzipped one.

Sorry to barge in on the convo, but I think the least surprising is if:
.\ThermoRawFileParser.exe -g -i="test.raw" -b="test.mzML"
generates test.mzML.gz
and:
.\ThermoRawFileParser.exe -g -i="test.raw" -b="test.mzML.gz"
generates test.mzML.gz
(like "Save As...." applications with format selection do)

thank you for reporting this issue. Do I understand correctly, that "less surprising" result will be to use provided filename as it is (in layman terms - "user knows better")? For example, .\ThermoRawFileParser.exe -g -i="test.raw" -b="test.mzML" should generate test.mzML that is in fact a gzipped one.

I would expect that the generated mzML file name to exactly match the file name passed by the -b file ("user knows better"), but I think @edeutsch's suggestion (where the gz file extension is appended if not present) is probably the better approach for most users/applications (which generally rely on file extensions)

Thank you both for your input. I will stick to "add the extension, if it is not present". I guess, for unification, the same behavior should be added to other file formats, i.e. add .mzML to all mzML files, if it is not present and so on.

I have implemented the changes and added a test with the following table of test cases (with as strange user inputs as I can come with). Please, let me know, if there is any case that I might forgot to include. (I guess the table is rather self-explanatory)

Format
Gzip
UserInput
FileOutput

MGF
FALSE
rawfile
rawfile.mgf

MGF
FALSE
rawfile.
rawfile.mgf

MGF
FALSE
rawfile.mgf
rawfile.mgf

MGF
FALSE
rawfile.MGF
rawfile.mgf

MGF
FALSE
rawfile..
rawfile.mgf

MGF
TRUE
rawfile.mgf.gz
rawfile.mgf.gz

MGF
TRUE
rawfile
rawfile.mgf.gz

MGF
TRUE
rawfile..
rawfile.mgf.gz

MGF
TRUE
rawfile.mgf.GZ
rawfile.mgf.gz

MGF
TRUE
rawfile.mgf
rawfile.mgf.gz

MGF
TRUE
rawfile.MGF.
rawfile.mgf.gz

mzML
FALSE
rawfile
rawfile.mzML

mzML
FALSE
rawfile.
rawfile.mzML

mzML
FALSE
rawfile.mgf
rawfile.mgf.mzML

mzML
FALSE
rawfile.MzML
rawfile.mzML

mzML
FALSE
rawfile..
rawfile.mzML

mzML
TRUE
rawfile.mzML.gz
rawfile.mzML.gz

mzML
TRUE
rawfile
rawfile.mzML.gz

mzML
TRUE
rawfile..
rawfile.mzML.gz

mzML
TRUE
rawfile.mzml.GZ
rawfile.mzML.gz

mzML
TRUE
rawfile.mzML
rawfile.mzML.gz

mzML
TRUE
rawfile.MZml.
rawfile.mzML.gz

That looks good to me, thank you for tackling this @caetera! I think it would also be good to update the documentation & CLI flag help to clarify that the flag only effects the filename, and not the full file path.

Closed in version 1.4.2


Not working with ROS2 Dashing under Ubuntu 18.04 for PS4 gamepad

Hi,
I am able to view the PS4 gamepad under /dev/input/js0 (and /dev/input/eventX) using the ds4drv --hidraw (connected via USB). However when I run the joystick_ros2 node, it does not detect the device:
ros2 run joystick_ros2 joystick_ros2
/opt/ros/dashing/lib/python3.6/site-packages/rclpy/node.py:1053: UserWarning: Pass an explicit 'qos_profile' argument
  warnings.warn("Pass an explicit 'qos_profile' argument")
Joystick not found. Will retry every second.
Joystick not found. Will retry every second.
Joystick not found. Will retry every second.
Joystick not found. Will retry every second.
Joystick not found. Will retry every second.

This is because the inputs Python package is not detecting the device. This could be because ds4drv uses the evdev interface. Probably there is an incompatibility there with the Python inputs module not supporting my platform (I'm using NVIDIA Jetson). So I tried using the evdev module in joystick_ros2.py but then that changes the whole way of accessing the input events. I'd appreciate if you have any tips on getting this to work.
Thanks.

run input_example.py test

The gamepad is not detected:
Traceback (most recent call last):
  File "input_example.py", line 24, in <module>
    main()
  File "input_example.py", line 13, in main
    gamepad = devices.gamepads[0]
IndexError: list index out of range

Sony updated their company name, hence difference description strings of their controllers.
My fix:
in joystick_ros2.py
FROM:
JOYSTICK_CODE_VALUE_MAP = {
'Microsoft X-Box 360 pad': (XINPUT_CODE_MAP, XINPUT_VALUE_MAP),
'Sony Computer Entertainment Wireless Controller': (PS4_CODE_MAP, PS4_VALUE_MAP),
'Logitech Gamepad F710': (F710_CODE_MAP, F710_VALUE_MAP),
'Microsoft X-Box One pad': (XONE_CODE_MAP, XONE_VALUE_MAP)
}
TO:
JOYSTICK_CODE_VALUE_MAP = {
'Microsoft X-Box 360 pad': (XINPUT_CODE_MAP, XINPUT_VALUE_MAP),
'Sony Computer Entertainment Wireless Controller': (PS4_CODE_MAP, PS4_VALUE_MAP),
'Sony Interactive Entertainment Wireless Controller': (PS4_CODE_MAP, PS4_VALUE_MAP),
'Logitech Gamepad F710': (F710_CODE_MAP, F710_VALUE_MAP),
'Microsoft X-Box One pad': (XONE_CODE_MAP, XONE_VALUE_MAP)
}


servers not appearing on sidebar

no servers visible.

here's a screenshot of the empty serverlist.

it appears that it cannot see the discord construct of server folders presently, but it otherwise works as intended.

This should be fixed in this commit, I leave this issue open for a week if the bug is still present


Multicluster: kind setup

Extend the kind setup to allow for two clusters. One control cluster and one second that simulates a remote cluster.
The registry should be shared with the two clusters so that both clusters can access it.

@froesef this should have been closed by https://github.com/IBM/the-mesh-for-data/pull/142 right?

Assuming yes, @froesef please reopen if not.


import LzEditor from 'react-lz-editor' 报错！

ERROR in ./~/react-lz-editor/global/supports/resources/default/iconfont.woff
Module parse failed: /Users/qingdou/dev/team/qunju_business/node_modules/react-lz-editor/global/supports/resources/default/iconfont.woff Unexpected character '' (1:4)
You may need an appropriate loader to handle this file type.
上传不了图片，按钮点不了

上传后获取图片列表暂时还有问题，正在解决……


[RFE] check container help file inside image in roff format

Nowadays, MTF uses help.md (Markdown) format for testing some sanity checks.
https://github.com/fedora-modularity/meta-test-family/blob/devel/moduleframework/tools/helpmd_lint.py#L30
https://github.com/fedora-modularity/meta-test-family/blob/devel/moduleframework/helpfile_linter.py
It would be nice to add also possibility for roff format.
MTF will not check roff format in container/image. This is case for conu. Therefore reassigning.
MTF issue is here: https://github.com/fedora-modularity/meta-test-family/issues/138

I'm +1 on having code in conu which checks whether a help file is present in a container (image).
I'm -1 on the code which parses the help file, be it roff or markdown. Maintaining such code is completely infeasible from my PoV. If you have some good arguments here, I would like to hear them.

@TomasTomecek Yes, checking only presence of help file is fine. And I guess only roff file has to be used in container(image). I would check only sections like:
.TH "MEMCACHED"
.SH NAME
.SH DESCRIPTION
.SH USAGE
.SH ENVIRONMENT VARIABLES
.SH SECURITY IMPLICATIONS

Some of them are invalid if e.g. environment variables does not exist in Dockerfile or even they are not specified.

AC: implement basic method for checking existence of help file and for getting help file. Those methods then will be used in external linter or mtf.

Agreed that this may no longer be needed.

This will be done in colin https://github.com/user-cont/colin/issues/83


Side note for limit of Logic Apps with Managed Identities

I read on a third-party's page that there's a limit of 10 logic apps with system-assigned managed identities. I couldn't find any confirmation of this in the document. Does that limit exist?
If so, it would be useful to add that as a side note in the document.
Thank you for your time 😊

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

ID: cdfb9148-5d81-f65e-3cb3-aa7c5eb5867b
Version Independent ID: 5051db4d-83d1-a230-6fcb-9238a384961d
Content: Authenticate with managed identities - Azure Logic Apps
Content Source: articles/logic-apps/create-managed-service-identity.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@nestor-qhr
Thank you for reaching out. At this time we are reviewing the ask and will provide an update as appropriate

@nestor-qhr Service limits are published in a dedicated doc with a section for managed identity that is linked at the start of this doc as well.
Also, based on the linked doc, the limit is now 1000 logics apps with managed identity in a subscription.

@nestor-qhr Service limits are published in a dedicated doc with a section for managed identity that is linked at the start of this doc as well.
Also, based on the linked doc, the limit is now 1000 logics apps with managed identity in a subscription.

That's awesome. Thank you so much for getting back to me so quickly. Have great week!

@nestor-qhr: I'll make the link a bit more visible by tweaking the link text, thanks!


Unable to extend layers from private Bitbucket repository

Environment

Operating System: Windows_NT
Node Version:     v18.15.0
Nuxt Version:     3.7.0
CLI Version:      3.7.2
Nitro Version:    2.6.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      extends, devtools
Runtime Modules:  -
Build Modules:    -

Reproduction

Setup a private Bitbucket repository
Initialize a new Nuxt project
Try to extend from the Bitbucket repository (see code below)
Nuxt is unable to fetch the remote repository

Describe the bug
The concept of Nuxt Layers seems very promising. So I decided to establish Layers in my company as  way to share common components between various Nuxt projects. We generally store our code on Bitbucket. So I made a private Bitbucket repository for our shared code basis. However Nuxt is unable to fetch from this repository.
My setup:

.env file:
GIGET_AUTH=XXXXXXXXXXXXXXXXXXXXXXX

nuxt.config.ts:

export default defineNuxtConfig({
 extends: [
   'bitbucket:user_name/project_name#branch',
 ],
 // ...
})

Internally unjs/giget converts this to https://bitbucket.org/user_name/project_name/get/branch.tar.gz. However in my opinion something like https://api.bitbucket.org/2.0/repositories/user_name/project_name/src/branch/ would be more correct. I found a similar issue regarding private github repositories which is not released yet.
Is this  bug by Nuxt (/giget) or am I doing something wrong? Any help is much appreciated.
Additional context
No response
Logs
No response

It sounds like this would need to be fixed upstream in giget. Would you raise an issue there instead?


proposal: go/types: adopt "defined type" terminology consistently

The spec uses the term "defined type" for a type created by a declaration such as type T int; a defined type is capable of bearing declared methods. Historically, this was called a Named type, but the advent of type aliases made this confusing and ambiguous.
Though I was initially lukewarm about the term "defined type" (aren't all types defined?), recent work auditing dense type-checking code in x/tools as made me appreciate it, because there are now three kinds of "named" types:

defined types (*Named)
type parameters (*TypeParam, go1.18), and
type aliases (*Alias in go1.22).

In some cases, their namedness is a meaningful commonality that can be used by tooling. For example, they all have an associated TypeName symbol that provides their location and logical name for use in a tool's UI.
I propose that we follow through with the renaming, using the name "Named" as little as possible. Obviously we can't get rid of Named, or make it an alias for Defined (as its reflect.Type.String would change), but we can make Defined an alias for it:
// A Defined type is a type created by a declaration such as `type T underlying`,
// capable of bearing method declarations.
//
// Along with TypeParam and Alias, it is one of three types that are named
// and have an associated TypeName symbol.
// Named is a historical and deprecated term for Defined.
type Defined = types.Named
func NewDefined(obj *TypeName, underlying Type, methods []*Func) *Defined

// Named is the historical term for a Defined type.
// Use Defined in new code.
type Named struct { ... }

// Deprecated: use NewDefined.
func NewNamed( ... ) { return NewDefined ... }

It looks the latest spec has almost cleared the confusions. Named types include predeclared types, defined types and instantiated types. An alias to an unnamed composite type is not a named type.

As an aside, in the past, we did consider introducing
type Defined = Named

but we also concluded that it would only add more confusion (I'm pretty sure @findleyr and I discussed this). Maybe that's not the case anymore, now that "named types" are essentially types with names (but for alias types that denote type literals).
On the other hand, as you say, Defined type is also not a great term (we don't have "undefined" types). In retrospect, the correct term probably should have been "nominal type" from the start because they behave like in a nominal type system (https://en.wikipedia.org/wiki/Nominal_type_system). And, "nominal" nicely refers to the name of the type but also implies a certain behavior. In fact we have nominal (defined) and structural types (type literals) in Go. That said, I don't know that we/the community can stomach another terminology change just for the sake of being clearer in the spec.

Perhaps that we could even go a bit further.
Even nominal seem to imply that the name is part of the set of constraints wrt type identity.
It's not really true for interfaces, which behave structurally. Perhaps that it could even be reconciliated with aliases?
An interface type definition would define one of many aliases for that type.
Only non-interface type definitions would be able to define a name for a type.
A type name is almost a unique identifier for that type, at least within a compilation unit, if we don't take into account types defined within function scopes.
Obviously, interface types are identified differently, disregarding where they are defined.
A type thus being a "kind" of interface.
Just thinking.

@atdiar A defined interface type (a "nominal interface type") still behaves differently from a interface literal (a "structural interface"): two otherwise identical signatures are different types if one of them contains a defined interface type and the other contains a interface literal (link).
In other words, "nominal" is in fact exactly the right word because the name in fact does matter wrt to type identity.
Even non-interface types allow assignment of a value of structural type to a variable of nominal type if the underlying types are identical.
There's a (different) proposal that would make that difference go away for interfaces, but that's somewhat orthogonal. What you are suggesting is interesting but I am not sure we can make this change in a backward-compatible way. Rather than say that a type definition of an interface produces one of many aliases, I think it would be cleaner to say that the RHS of a type definition cannot be an interface. One can only define alias types for interfaces.

Oh, you're right. That wouldn't be backward compatible because type assertions/switches. That would require the version-based migration process.
Perhaps is it semi-nominal only still? Because variable assignability seem to not mind about the interface name?
https://go.dev/play/p/GYnd66YcmWz
The composite non interface types seem to be strict about the type definition of each of their arguments indeed.
In return position, doesn't seem that the name matters.
Interesting.


Enhance Ledger View

Lots of transactions makes it hard to navigate. Improve this by making the months collapsible and having the current month, with maybe the previous and next months, expanded.

[x] Collapsible sections for months

[x] Auto collapse all - expand current

[x] Frozen header row
[x] Create Settings panel for all account types

[x] Move balances into settings panel

Removed requirement for auto-scroll. This can be implemented later if desired but collapsible months is a big improvement already.


Clean up test command in docker config files

Only set the test command on the base config file and pass in the sanitizer argument for the 5.1 image (as 5.0 doesn't support the thread sanitizer)
Motivation:
We want to run the thread sanitizer to find threading issues
Modifications:
Remove the test command from the distro specific config files and only have it on the base config. Set the SANITIZER_ARG variable on the 5.1 specific config file.
Result:
Thread sanitizer is run for the Swift 5.1 docker image tests

@swift-nio-bot test this please

@swift-nio-bot test this please


Example for tomcat logs

Hi,
The Apache example works like a charm!  I also want to know if you can provide an example for Tomcat logs (localhost_access_log). By analysing the log, I can get the download statistics based on IP or country over a period os time.
Much appreciated.
Cheers,
Kayne.

Hi @spahgnum. The tricky part of creating some of these example is getting hold of a good sample dataset. Would you have a sample Tomcat logs dataset handy that we could build out an example for?
Thanks!

G'Day @spahgnum, if you get hold of some logs we'd be happy to add these!
In the meantime we will close this one off :)


troubleshooting bad-txns-premature-spend-of-coinbase

My guess: the newly-started group of dashmate regtest nodes isn't ready yet.
{
  code: -26,
  message: 'bad-txns-premature-spend-of-coinbase, tried to spend coinbase at depth 10'
}

I think coinbase transactions requires 100 confirmations before you can spend it. Just do more block generations - pshenmic

[What is a coinbase transaction?] block reward transaction basically. and other stuff. the first transaction in the block - pshenmic - pshenmic

https://learnmeabitcoin.com/technical/mining/coinbase-transaction/

Solution
Just gonna run generatetoaddress a bunch more times to ramp up the block count and unlock the funds for spending.


feat(www): Fixed scroll-area orientation & added width property

I'm proposing changes to the scroll-area component within the shadcn-ui library to introduce enhanced flexibility in handling various sizes and orientations. This modification aims to streamline the user experience and eliminate the need for repetitive code adjustments.
Demo
https://github.com/shadcn-ui/ui/assets/73414242/34fdd712-7dad-453a-b6f7-cd2362ffe862

what's status on this PR ? Component is still broken


Multiple Issues with Caesars Cipher Guide

Describe your problem and how to reproduce it:
First Issue: The Basic solution uses call() function but doesn't include any reference to it.
Second Issue: The Basic solution doesn't work when you use it as a solution for the challenge. You get the following error on the beta site:

rot13("SERR PBQR PNZC") should decode to FREE CODE CAMP
rot13("SERR CVMMN!") should decode to FREE PIZZA!
rot13("SERR YBIR?") should decode to FREE LOVE?
rot13("GUR DHVPX OEBJA SBK WHZCF BIRE GUR YNML QBT.") should decode to THE QUICK BROWN FOX JUMPS OVER THE LAZY DOG.

And the following error on the old Site

assignment to undeclared variable x
assignment to undeclared variable x
assignment to undeclared variable x
assignment to undeclared variable x

I guess setting
x = char.charCodeAt(0);
to
let x = char.charCodeAt(0);
fixes this issue.
Add a Link to the page with the problem:
https://www.freecodecamp.rocks/learn/javascript-algorithms-and-data-structures/javascript-algorithms-and-data-structures-projects/caesars-cipher
https://www.freecodecamp.rocks/guide/certifications/javascript-algorithms-and-data-structures/javascript-algorithms-and-data-structures-projects/caesars-cipher/
Tell us about your browser and operating system:

Browser Name: Mozilla Firefox
Browser Version: 66.0.5
Operating System: Windows 10

One more thing. The error generated on the old site is helpful whereas the one on the new site isn't. Probably something to look at.

@navjotjsingh That is a valid point regarding the error message, but that is a different issue related to how the challenge tests are ran.  I have gone ahead and created a PR (#36046) to address the solution in the Guide, but the last issue you mentioned will have to be discussed in a separate issue (feel free to create it if you have the time).
@ValeraS I could not find any outstanding issues regarding why this does not show the error "assignment to undeclared variable x" which does show in the actual browser console.  Are errors like this being suppressed on showing in the fake console for any reason?


Wrong katex rendering

Initial checklist

[X] I agree to follow the code of conduct
[X] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected packages and versions
7.30
Link to runnable example
No response
Steps to reproduce
This is the formula used \ce{SO4^2- + Ba^2+ -> BaSO4 v}
When we try this in katex demo, it renders just fine. but we try it in milkdown playground, it shows empty.
I looked at katex options and tried to modify katexOptionsCtx, It will never render successfully
https://github.com/Milkdown/milkdown/assets/105030733/f295dc4e-2939-4118-8536-f9aa5c473200
Expected behavior
renders successfully
Actual behavior
shows empty
Runtime
Chrome
OS
macOS
Build and bundle tools
Create React App

we need to import mhchem
import 'katex/contrib/mhchem/mhchem.js';


"is not a type" error on variable/type name conflicts

What version of Go are you using (go version)?
Current playground version.
Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
Unknown (plaground).
What did you do?
The following code compiles with fancyMap is not a type errors in both foo and foo2 (playground):
package main

type fancyMap = map[string]string

func foo(fancyMap fancyMap) {
	var _ fancyMap
}

func foo2(fancyMap fancyMap) {
	_ = make(fancyMap)
}

func main() {
}

What did you expect to see?
Successful compilation or a more informative error message.
What did you see instead?
Poor/incorrect error message. Simply renaming the fancyMap variable fixes the problem, but the error message is incorrect: fancyMap is a type.
Apologies if there's an existing issue for this.

The error message is correct: at the point of the erroneous declaration, fancyMap refers to the function parameter, and that is not a type.
Any suggestions on what a more informative error message would look like?

@ianlancetaylor Ah, I wasn't aware that variable bindings and types were intended to share the same namespace. In that case, perhaps issue a warning when a variable name shadows a type name or vice versa? If not that, then perhaps this specific error could be modified to look at the shadowed names and see if any of them is a type, and give an appropriate hint? I haven't looked at how Go's name resolution works, so I'm not sure how hard that would be in practice, but I'd be happy to try and put a PR together if one of these options sounds good to you.

perhaps issue a warning

Go doesn't have any warnings, intentionally.
As stated in the FAQ:

There are two reasons for having no warnings. First, if it's worth complaining about, it's worth fixing in the code. (And if it's not worth fixing, it's not worth mentioning.) Second, having the compiler generate warnings encourages the implementation to warn about weak cases that can make compilation noisy, masking real errors that should be fixed.

@bradfitz I didn't mean to suggest some greater internal policy change-- do you have an alternative that you would prefer to see? Erroring on bindings-shadowing-types is obviously a non-option. I suppose that leaves my second suggestion:

If not that, then perhaps this specific error could be modified to look at the shadowed names and see if any of them is a type, and give an appropriate hint?

I'd also be fine closing this as "WONTFIX" if you think such a change is unwanted.

A better error sounds fine. Go does strive to have readable and helpful errors.
I've retitled and relabeled this bug. It's not targeted towards any release, meaning anybody can send a fix whenever.


fix(twig) - Make language selector optional in nav/ local nav

Fixes :- https://github.com/international-labour-organization/designsystem/issues/740
Notes :-

Currently nav and local nav will always display the language selector this needs to be made optional.

Screenshots :-
Nav
Before Big screen

After Big screen

Before Small screen

After Small screen

Local Nav
Before Big screen

After Big screen

Before Small screen

After Small screen

@justintemps Did the changes to not show the language selector when it is empty. Like you said we will need @inesdgomes to have a look at it to see if the designs are okay for the state where it is empty.

@inesdgomes for you to check when you come back. CC @dali-ilo

Thanks @Shashika6, as discussed with @inesdgomes the dark blue background behind the language selector should also be removed when the language selector isn't there:

Hey @inesdgomes , @Shashika6 pushed up a fix but the height of the header changes a little. Is that ok?
https://github.com/international-labour-organization/designsystem/assets/12401179/c75bd7fb-ad33-408a-9090-5a1bb214a412

@justintemps @inesdgomes If required I could change the spacing to be the exact same way which is 26px. I changed it to 24px so that we are sticking with multiples of 4.

@Shashika6 We tested on the website and this looks fine as is. Thanks!


Indent/Outdent with tab key does not honor editor.autoIndent=false

Issue Type: Bug
When I have:
    /*
     * Comment
     */

If I select it and hit tab, I get:
        /*
        * Comment
        */

If instead I had hit shift-tab, I get:
/*
    * Comment
    */

Same happens with Ctrl-] and Ctrl-[ (if those are supposed to make a difference)
I hoped turning off autoIndent would stop this, but no dice.  I also turned off C++ formatting in the JSON config:
{
    "editor.autoIndent": false,
    "editor.detectIndentation": false,
    "C_Cpp.formatting": "Disabled"
}

I have done due diligence by asking (and bountying, +100) a question on StackOverflow to get an answer on what to do about this, but no one seems to know:
https://stackoverflow.com/q/53326134/preserve-spacing-on-indent-or-outdent-with-tab-in-vscode
If someone can provide a resolution here, be sure to go there and get the bounty points.
VS Code version: Code 1.29.1 (bc24f98b5f70467bc689abf41cc5550ca637088e, 2018-11-15T19:13:36.375Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2808)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.86GB (1.40GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (3)

Extension
Author (truncated)
Version

vscode-github
Kni
0.30.0

python
ms-
2018.10.1

cpptools
ms-
0.20.1

Guys Any update on this?
I'm having the same issue!

Come on, I just tried it in Vim, Sublime, Emacs... no other editor has a bug like this!
For the time being, a workaround is to use the indent-one-space extension, but this should be fixed properly.

This issue also affects indentation of XML severely. For example, with indentation set at 4 spaces, starting with this text:
<Component
 class='something'
 SomeProp='blah'/>

increasing indentation by one step using the "Indent Line" command produces this:
    <Component
    class='something'
    SomeProp='blah'/>

Starting with this text
    <Component
     class='something'
     SomeProp='blah'/>

and decreasing indentation by one step using the "Outdent Line" command produces this:
<Component
    class='something'
    SomeProp='blah'/>

This style of indentation may feel strange to some people, but that's part of the whole issue — who are people, and who is VSCode, to tell others how to format their code? If I wanted my text editor to mangle my formatting, I would opt in to automated formatting.
Increasing and decreasing indentation should be a simple operation of adding or removing one fixed string (in my example, a string consisting of four spaces) to/from the beginning of each line. Yes, there are some important edge cases to take care of (e.g. what happens when one of the selected lines doesn't have enough whitespace to take away on an outdent; or e.g. ignoring the last empty line of a block of lines selection), but VSCode gets them right already (and most plugins that implement things naively do not). Why confound the simple, deterministic operation of changing the indentation level with something as complex (and often buggy, which is why people turn it off) automatic formatting?
If changing the default behaviour of these actions is somehow controversial, can you at least provide "Indent Line Trivially" and "Outdent Line Trivially" pair of actions that people who don't want their code mangled can use?

👯

The work around does not appear to work everywhere:

Hi I am trying to reproduce this issue and am currently not able to on the latest insiders. I have no extensions installed and I see the following. I assume the issue has been fixed, and therefore will close this issue. If you encounter the issues in this thread again, please leave a comment here so I can reopen the issue.
https://github.com/microsoft/vscode/assets/61460952/ef39af38-d53e-4b6b-a894-4e5a0bac8776

@aiday-mar could you understand me

Aha I see thanks for the clarification. I reopened this issue.

Hi @hostilefork I am investigating the second example in your last comment, and I noticed the following. The issue you mentioned appears to happen specifically in the case when you use indentation with two spaces in your code but the document has detected a general indentation with 4 spaces. If you run the command Detect Indentation from Content beforehand, the document general indentation level will be 2 and upon indentation, the correct alignment will be preserved.
I suppose in your case you would like the indentation level of 4 to be applied on the full document even if locally you were using an indentation level of 2, and for this to preserve the alignment.
https://github.com/microsoft/vscode/assets/61460952/12731eb4-a1c1-4d34-9650-4a9d5b10116b

I suppose in your case you would like the indentation level of 4 to be applied on the full document even if locally you were using an indentation level of 2, and for this to preserve the alignment.

Yes, that's correct--generally I want hitting the tab key to do 4 spaces.
If I want two spaces, then hitting the space bar twice is all right.

Any update on this? This is a bit of a pain point switching between editors every now and then. Vscode is acting out of the ordinary here in comparison.

I came across this issue when struggling with something that I think is similar to https://github.com/microsoft/vscode/issues/63388#issuecomment-2025072130 and the initial comment in https://github.com/microsoft/vscode/issues/124943. I thought I'd add a slightly different example / use case to make sure this will be covered with any fixes:
I work a lot in C / C++ / Fortran code that uses different indentation amounts in different constructs. For example, the body of a function / subroutine might be indented by 2 spaces for the top-level indentation, but other blocks (bodies of conditionals, loops, etc.) are indented by 4 spaces. I'd like to have indentation set at 4 spaces so it works right in most places, and then have VSCode always indent by an additional 4 spaces relative to the previous line, no matter what column the previous line was on (ideally, this would happen both with the automatic indentation when hitting enter after starting a block, and with use of the tab key). Instead, VSCode seems to always want to indent to a "tab stop" - i.e., a multiple of 4 spaces in an absolute sense.
Here is a screen shot showing a simple example:
https://github.com/user-attachments/assets/3cf6df92-51ac-40c7-946e-fc7034c3215d
Note that indentation is set to 4 spaces here. What I want is for the x = 5; line to be indented to column 7; instead, VSCode indents it to column 5 (presumably because this is the first "tab stop" that is more indented than the line above it). In this example, the indentation ends up being fixed when typing the semicolon if I turn on "Format on type", but ideally I'd like for this to work correctly without needing to rely on a formatter.
i.e., I want indentation to work relative to the previous line, not based on tab stops that essentially assume that the previous line was itself indented to a tab stop.


Fix donation link for the Trump campaign

It's what'll make the Internet Great Again, just like it always is. (?)

I don't know how I feel about getting Rick Rolled by a pull request.


DCOS-12769: Make paging in log view a bit snappier

This PR fixes a few issues related to the same thing - making the paging become smoother:

It initializes the logs no matter what response comes from the server, so we do not see 'Empty Log' when there is an error.
It sets scrollTop directly for the log view in componentDidUpdate instead of using handleGoToBottom
It removes log window so the position calculation in componentDidUpdate continues to work
It lets the container handle when to retrieve a previous page, instead of assuming an ongoing request when at the top.

Initially, the paging goes pretty fast, but eventually the journald bug manifests and loading becomes increasingly slower. This is not handled by this PR.

Run unit tests

Run unit tests

Works well!


Could not find any cuda.h matching version '' in any subdirectory

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
v2.8.0
Custom Code
Yes
OS Platform and Distribution
linux ubuntu 2204
Mobile device
linux ubuntu 2204
Python version
3.10
Bazel version
6.1.1
GCC/Compiler version
11.3.0
CUDA/cuDNN version
ananconda3  2023.03
GPU model and memory
11.3
Current Behaviour?
I wanted to configure tensorflow,then after running . /configure command, I get the error "Could not find any cudn.h matching version '10' in any subdirectory", and I don't know how to handle this error.

Standalone code to reproduce the issue
(base) emma@emma-virtual-machine:~/tensorflow$ ./configure
You have bazel 3.1.0 installed.
Please specify the location of python. [Default is /usr/local/anaconda3/bin/python3]:

Found possible Python library paths:
  /usr/local/anaconda3/lib/python3.10/site-packages
Please input the desired Python library path to use.  Default is [/usr/local/anaconda3/lib/python3.10/site-packages]

Do you wish to build TensorFlow with ROCm support? [y/N]:
No ROCm support will be enabled for TensorFlow.

Do you wish to build TensorFlow with CUDA support? [y/N]: y
CUDA support will be enabled for TensorFlow.

Do you wish to build TensorFlow with TensorRT support? [y/N]:
No TensorRT support will be enabled for TensorFlow.

Could not find any cuda.h matching version '' in any subdirectory:
        ''
        'include'
        'include/cuda'
        'include/*-linux-gnu'
        'extras/CUPTI/include'
        'include/cuda/CUPTI'
        'local/cuda/extras/CUPTI/include'
of:
        '/lib/x86_64-linux-gnu'
        '/usr'
        '/usr/lib/x86_64-linux-gnu/libfakeroot'
        '/usr/local/lib'
Asking for detailed CUDA configuration...

Please specify the CUDA SDK version you want to use. [Leave empty to default to CUDA 10]:

Please specify the cuDNN version you want to use. [Leave empty to default to cuDNN 7]:

Relevant log output
No response

@tracerxia
Sorry for the late reply. Could you please make sure to follow the instructions mentioned here and check the tested build configuration as well. Please let us know if it helps?
Thank you!


Silicon Labs Support: Build OpenThread for Silicon Labs as a library and/or import into Simplicity Studio

Silicon Labs has apparently dropped support for their proprietary Thread stack in favor of OpenThread. However, the Silicon Labs Eco System revolves around Windows development with their flavor of Eclipse called Simplicity Studio. OpenThread development is geared towards Linux development and building/debugging without an IDE. I don't see any documentation or migration guides for how to either use OpenThread in Simplicity Studio or how to migrate existing applications to OpenThread.
It would be great is Silicon Labs would provide instructions on how to build OpenThread for Silicon Labs as a library and/or import into Simplicity Studio

Hi ,
Integration into Simplicity Studio is currently being planned.
In the mean-time, please use instructions provided below to build open thread.
You may find the Readme for mg12 at link:
https://github.com/openthread/openthread/tree/master/examples/platforms/efr32mg12
and for mg21 at link:
https://github.com/openthread/openthread/tree/master/examples/platforms/efr32mg21
mg21 is slightly different as a change has to be made to core_cm33.h in our SDK as is mentioned on that page.
Best Regards
Azam

updated link:
https://github.com/openthread/ot-efr32


(WIP) Initial work on nimble_parsec generated lexer

Initial work on a nimble parsec based lexer. Tried to design for nice errors and to avoid back tracking on happy path.

@jparise I'm curious why those reserved words are reserved. What was the idea behind them?

@jparise I'm curious why those reserved words are reserved. What was the idea behind them?

This is borrowed from Apache Thrift. The idea is that Thrift IDL shouldn't contain words that would clash with an output language's keywords when code generated. There are two possible approaches:

Prevent the IDL from ever containing languages' keywords (across a large set of supported languages).
Let the IDL contain close to anything, and deal with reserved words on a per-generator basis. The generator would have to recognize its own reserved words and renamed them accordingly.

Apache Thrift generally does (1) (at least for C++'s reserved words), so we copied that behavior in our lexer for compatibility. In practice, we should only be concerned with Elixir's reserved words (which aren't much of a problem) and could take approach (2).
In this project, reserved word rejection was added in #295 by @thecodeboss.

Makes sense, it was a bit perplexing to see keywords for other languages in here.


ZEPPELIN-3196. Plugin framework for Zeppelin Engine

What is this PR for?
More details is in the design doc. https://docs.google.com/document/d/1QRgznfDKsi6XxR9FTJtUuiGAvgQvmc12FdIbHrmOZzc/edit?usp=sharing
What type of PR is it?
[ Improvement | Refactoring]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3196

How should this be tested?

CI pass

Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@zjffdu
a strange description of PR )

I haven't updated the description, will do that after the PR is ready, It is still WIP

@Leemoonsoo @jongyoul @prabhjyotsingh @felixcheung Could you help review it ? Thanks, it is for 0.9.

Took a look changes and it looks good to me

@zjffdu I have a simple question. You introduced IS_ZEPPELIN_TEST but how can we use it? do we need to all jars into classpath manually to use IS_ZEPPELIN_TEST?

IS_ZEPPELIN_TEST is used for unit test for PluginManager. e.g. In NotebookRepoSyncInitializationTest, VFSNotebookRepoMock is just for testing, so could be constructed from classpath directly.

Thanks for review, will merge soon

@zjffdu, just updated to master, and noticed versioning toolbar is missing...
I think it's because of this PR.
We have zeppelin.notebook.storage=org.apache.zeppelin.notebook.repo.GitNotebookRepo in zeppelin-site.xml .. is there is some additional setting required now,
or perhaps Zeppelin now has to be build differently?
Thanks!

I wonder if https://github.com/apache/zeppelin/pull/2945 fixes issue I described in my earlier comment.
cc @cvaliente @myuwono

@Tagar yep! I have it in my private fork for the ServiceLoader fix. Eagerly waiting for the PR to get merged to master.


Inline AttachmentVisibility methods

This PR is the next stage in the larger refactoring which was started in #3850.
This inlines all the methods in AttachmentVisibility to use the methods in AttachmentData directly.
It also changes the various attachment-related updaters (and one a worker) to (a) all operate on an instance of AttachmentData rather than Attachment; and (b) to use the relevant methods on AttachmentData to highlight how they are equivalent to logic in the attachment-related controllers, particularly that in BaseAttachmentsController#fail.
Finally it removes the AttachmentVisibility class altogether.
The (hopefully) final stage will be to refactor the attachment-related controllers to make the logic more similar to that in Asset Manager's media controller download actions and to use lower-level (more understandable) methods on AttachmentData.
This should make it clearer how we are replacing Whitehall attachment functionality with a combination of code in Whitehall to update Asset Manager and new behaviour in Asset Manager to act on the data supplied by Whitehall.

@chrislo @chrisroos Thanks for reviewing! I'm going to rebase against master and force-push in preparation for merging.


problem loading stored file

Using the access groups option I stored a series of files and saved the encrypted uri's. Loading those uris using kcl.load_file doesn't work for me, nor does loading them after decryption.
This is for access group
aqidbrogrt
The encrypted uri is:
sha1-enc://bf03a31646b25c56bdaedc8fa9d4137dac9c783bf2787b8fcf571e19d3aee48f9a6b48f1304240031d339264ee00c7a8.ag_aqidbrogrt
When I decrypt using kcl.decrypt I get
sha1://89d8ba74d54d3be6f0e24a182e4f8e21d4e25524
I've added both @magland and Jeff's google ids to the access group to facilitate testing

Strange.

Could you just do a simple test with one of the files? Try to store without encrypting and then see if you can load.
If that works, try encrypting the uri, and then see if that loads.
If either of those don't work, try this simpler test:

import kachery_cloud as kcl
uri = kcl.store_text('random-h887001')
uri2 = kcl.encrypt_uri(uri, access_group='aqidbrogrt')
txt1 = kcl.load_text(uri2)
print(txt1)

Also, it doesn't seem like I'm in the access group. Could you double check that for jmagland [at] flatironinstitute [dot] org?

I’ve added you (again, I believe) to the access group:
[cid:64126BCF-63F9-48FB-883C-8653C96E2B0D]
so hopefully that will help.
Separately, the text saves and retrieves work both locally on the UCSF machine and on my laptop.
The file saving and retrieval works when I link a file on the UCSF machine and load based on that link on the same machine.
The file retrieval does that work when I use that same (unencrypted) uri on my laptop at home.
For reference, that uri is
'sha1://9004ce7af2db4e5861fad7e0143d6be656501d33'
thoughts?
On Jul 4, 2022, at 8:31 AM, Jeremy Magland<EMAIL_ADDRESS>wrote:
This Message Is From an External Sender
This message came from outside your organization.
Strange.

Could you just do a simple test with one of the files? Try to store without encrypting and then see if you can load.
If that works, try encrypting the uri, and then see if that loads.
If either of those don't work, try this simpler test:

import kachery_cloud as kcl
uri = kcl.store_text('random-h887001')
uri2 = kcl.encrypt_uri(uri, access_group='aqidbrogrt')
txt1 = kcl.load_text(uri2)
print(txt1)
Also, it doesn't seem like I'm in the access group. Could you double check that for jmagland [at] flatironinstitute [dot] org?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/flatironinstitute/kachery-cloud/issues/7*issuecomment-1173942630__;Iw!!LQC6Cpwp!uHfbT8lULEKMOb8O3j7mGwTrsjpgA7BtIYUTL1D3ac1oWKOqoSAxj2eapii2r61gx5S4-3feu35Oud1I5UlGgGL3JA$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/ABV4PSMGXUKGPUH6W4VO66TVSL7W7ANCNFSM52P6EK4A__;!!LQC6Cpwp!uHfbT8lULEKMOb8O3j7mGwTrsjpgA7BtIYUTL1D3ac1oWKOqoSAxj2eapii2r61gx5S4-3feu35Oud1I5UkMAc0BxA$.
You are receiving this because you authored the thread.

I think the issue is that you are using link_file(...) which only stores the file locally, rather than store_file(...) which uploads immediately to the cloud. This is related to our slack discussion. I'll close this issue for now, but feel free to reopen if appropriate.


[FLINK-22689][doc] Table API Documentation Row-Based Operations Examp…

…le Fails
What is the purpose of the change
Correct error sample codes in chapter map and flatmap in docs/content/docs/dev/table/tableApi.md.
Brief change log

docs/content/docs/dev/table/tableApi

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

This issue could be fixed by changing the "as" expression in org/apache/flink/table/api/internal/BaseExpressions.java to as "in" org/apache/flink/table/api/Table.java. I have tested those samples and now work smoothly. But I still feel confused about those two "as" expressions.
I have also tried using select to test the functionality of the as expression in org/apache/flink/table/api/internal/BaseExpressions.java with the sample code below:
Table table = input.select(call("func", $("c")).as("a", "b"));
The same example code can work well in select while it fails in map or flatmap. However, the as clause still fails to expand the result of that ScalarFunction to two independent columns.
So I guess the as expression in org/apache/flink/table/api/Table.java can both rename and expand fields while that in org/apache/flink/table/api/internal/BaseExpressions is only for naeming fields. Am I correct?

This issue could be fixed by changing the "as" expression in org/apache/flink/table/api/internal/BaseExpressions.java to as "in" org/apache/flink/table/api/Table.java. I have tested those samples and now work smoothly. But I still feel confused about those two "as" expressions.
I have also tried using select to test the functionality of the as expression in org/apache/flink/table/api/internal/BaseExpressions.java with the sample code below:
Table table = input.select(call("func", $("c")).as("a", "b"));
The same example code can work well in select while it fails in map or flatmap. However, the as clause still fails to expand the result of that ScalarFunction to two independent columns.
So I guess the as expression in org/apache/flink/table/api/Table.java can both rename and expand fields while that in org/apache/flink/table/api/internal/BaseExpressions is only for naeming fields. Am I correct?


Please update this to Rails 5!

I'm using this project (on this fork) successfully with Rails 5 today. What particular problems are you having?

Bundler could not find compatible versions for gem "rails":
In Gemfile:
rails (= 5.0.2)
devise_sms_activable was resolved to 0.0.9, which depends on
  rails (~> 3.0.0)


cover art causes corrupts player state on future3

Describe your problem
After copying some sound files into the shared folder (fresh installation) with embeded covers and access the library in the webui the covers are correctly shown, but no playback is possible.
Only after a restart of the service (or pi), the playback is working and cover art is correctly shown.
This behavior can be recreated if the cover-cache folder is emptied.
Also the initial call of the library after a service restart takes pretty long in Firefox. Chrome runs much faster.
But i'm unsure if its caused by the cover art.
What's your hardware set up?
Pi 3 A+, Rasberry Pi OS Bookworm 32 bit lite, RC522 RFID Reader
If possible, try to attach logs from ... (paths from RPi)
04.01.2024 00:37:05 -  247:__init__.py        - jb.PlayerMPD         - mpd.timer_status - ERROR    - UnicodeDecodeError: 'utf-8' codec can't decode byte 0xac in position 1: invalid start byte
04.01.2024 00:38:06 -  526:__init__.py        - jb.PlayerMPD         - MainThread      - ERROR    - ConnectionError: Connection lost while reading binary data: expected 8192 bytes, got 180 at uri Electric Callboy - 2022 - Tekkno/Electric Callboy - Tekkno - 01 - Pump It.mp3

Cheap fix would be a refresh button to restart the service, but not very user friendly

I am aware of it. It's a problem with MPD and occurs randomly. I have an idea, but it will take some time to be resolved.

#2325 fixed this on future3/develop.
Closing. Feel free to reopen, if your issue still persists.


format('Do') will not produce '31st'

Describe the bug
I want to provide moment with values 1-31 and format it into 1st, 2nd, ..., 31st.
I use this code: moment( i, 'D' ).format('Do') where i is an integer from 1-31.
Strangely, it correctly formats every value except for 31.
moment(31, 'D').format('Do') => 'Invalid date'
Even though these work:
moment(1, 'D').format('Do') => '1st'
moment(15, 'D').format('Do') => '15th'
To Reproduce
Steps to reproduce the behavior:

type moment(31, 'D').format('Do')
See that gives Invalid date

Expected behavior
It should produce the value '31st'
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version "moment": "^2.29.1"

Smartphone (please complete the following information if applicable):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. firefox, safari]
Version [e.g. 22]

Moment-specific environment

Eastern Daylight Time
11/2/2021 2:30 PM

Please run the following code in your environment and include the output:
console.log((new Date()).toString()) // Tue Nov 02 2021 14:33:15 GMT-0400 (Eastern Daylight Time)
console.log((new Date()).toLocaleString()) // 11/2/2021, 2:33:29 PM
console.log((new Date()).getTimezoneOffset()) // 240
console.log(navigator.userAgent) // Uncaught ReferenceError: navigator is not defined
console.log(moment.version) // 2.29.1

Additional context
Add any other context about the problem here.

Looks like you tried this code in November, which only has 30 days.
Without any other fields to work with moment(31, 'D') tries to create a moment set to the 31st of the current month.  As November doesn't have a 31st you ended up with an invalid date.
Make sure you create your moments for a month with 31 days - e.g. January:
moment({ month: 0, day: 31 }).format('Do') // "31st"

I see. That makes a lot of sense now thank you!


Export Typescript Client in npm package

Is there a reason the Client is not exported in the types?
It would be great to not have to reference a static path like this:

Actually, this is a solution for now (I still think the Mailgun Client type should be export from the package):

Hello @crobinson42
After the call of mailgun.client(options) you should receive the instance of properly typed Client.
Can you share a bit more details on why do you need this?
I am not sure that I understood the goal of exporting Client type.

@olexandr-mazepa it sounds like you're not familiar with typed language patterns.
Here's an OOP example/use-case for having the type available as an export:
class MyEmailService {
  private mailgunClient: MailgunClientType // declared, uninitialized, we must have the type here so it's known to all other class members

  constructor(mailgunConfigs: MailgunClientConfigType) {
    this.mailgunClient = new Mailgun(mailgunConfigs)
  }

  ...
}

Okay =)
SDK exports an appropriate .d.ts file for each type.
For example Client type you can take from 'mailgun.js/dist/lib/client'.
Can you explain to me what is the issue with this approach and what do you suggest instead?

@olexandr-mazepa it is a bad practice to import from a publish package from a static path.
The reason this is a bad pattern is because the directory structure of a package is an implementation detail that should only be known to the publisher of the package, thus, it can change at any time. The published directory structure is not part of the public API of a package, that is why the main and types properties exist in the package.json, to point the resolution engine (common.js, esm, etc.) to the file that should be imported.

It would be indeed nice if the Client type could be exported.
Instead of private client: ReturnType<Mailgun['client']> it would become private client: Client.

Hi @olexandr-mazepa
This is exactly how I imagined the fix (being able to import the client as you mentioned). Thanks for taking your time to address this.

I've just merged the PR and published a new version on npm.
So that you can use version 5.0.0 to be able to use the new approach for importing types.


Bring Alpine to 3.9

@zhouzhuojie please review

Any specific reason for the upgrade? From https://github.com/checkr/flagr/issues/105, I think some companies may need to stick with 3.6

No specific reason, just proposing to keeping it current. As for https://github.com/checkr/flagr/issues/105, that issue was opened a year ago, so needs might have changed since ...


Android photo taken in portrait gets rotated when saved to camera roll

Steps to reproduce
1.Capture an image on a Samsung device in portrait mode and save it to camera roll
Expected behaviour
The image should be saved as portrait
Actual behaviour
The image is saved as landscape
This could be fixed by rotating the image if device orientation is 0 and device model is samsung but it´s a bad solution. I don´t know if the issue exists on all samsung devices or just the ones I have tested it on. Also, rotating a bitmap makes the module slow
Environment

Node.js version:
React Native version: 0.26.2
React Native platform + platform version: Android 5.1

react-native-camera
Version: master

Same issue here on a Samasung Galaxy S5. Same RN version.
I don't have another device to try this on though :/

Forgot to mention what models I´ve tested it on. I´m experiencing this on Galaxy S4 and S6. I think this ultimately comes down to a hardware issue. I don´t think rotating the images on these devices is the right solution since it is very slow and the maintainers would not accept a PR of that kind.

Same issue on LG H340 Leon

I found a solution that suit my needs at least. I changed the captureTarget to temp and then saving it in my own Java code. This way i can show the temporary image while I´m rotating and what not. The bad part is that I have to maintain some sort of blacklist of models this problem effects.

What I ended up doing was to retrieve camera orientation and expose it via a React module like this :
    static {
        for (int i = 0; i < Camera.getNumberOfCameras(); i++) {
            Camera.CameraInfo info = new Camera.CameraInfo();
            Camera.getCameraInfo(i, info);
            if (info.facing == Camera.CameraInfo.CAMERA_FACING_FRONT) {
                FRONT_CAMERA_ORIENTATION = info.orientation;
            } else if (info.facing == Camera.CameraInfo.CAMERA_FACING_BACK) {
                BACK_CAMERA_ORIENTATION = info.orientation;
            }
        }
    }

    @Nullable
    @Override
    public Map<String, Object> getConstants() {
        return ImmutableMap.<String, Object>of(
                "BACK_CAMERA_ORIENTATION", BACK_CAMERA_ORIENTATION,
                "FRONT_CAMERA_ORIENTATION", FRONT_CAMERA_ORIENTATION
        );
    }

Then in my JS code I could rotate the image :
render() {
        const orientation = MyNativeModule.BACK_CAMERA_ORIENTATION;
        return (
            <View style={container}>
                <Image style={{
                    transform: [
                        width: screenWidth,
                        height: screenHeight,
                        {scale: screenHeight/screenWidth,}, //for 90, needs to depend on orientation
                        {rotate: orientation,},
                    ],
                }} source={{uri : this.props.image.path,}} resizeMode='contain' />
            </View>
        );
    }

Same issue on s5

Same issue on OnePlus X

I have this issue on iPhone 4 (8.4.1). Saving temp

This probably will be fixed when the Camera2 API isused (http://www.theverge.com/2015/11/9/9696774/google-nexus-5x-upside-down-camera). This however is quite a lot of work since both API's need to be implemented to keep this module compatible with older devices, and this will still not solve all problems. Some devices won't work well with the new API as seen in the release notes of this repo: https://github.com/commonsguy/cwac-cam2.

https://github.com/lwansbrough/react-native-camera/issues/336

Is this really a bug? The orientation is set correct in the exif orientation tag. If you need to save the image with a neutral rotation a generic solution would be to check the exif rotation tag, rotate the image according to the tag and then set the tag to 1 (normal).

@tobiasjunsten It is. have you worked with this lib before? If you have corrected it kindly share. 👍

I've fixed this problem (thanks @tobiasjunsten for the idea).
You can check my PR here: https://github.com/lwansbrough/react-native-camera/pull/493

@daavidjobe  are you able to test my PR?  It may help with this https://github.com/lwansbrough/react-native-camera/pull/682.

I see this is not closed despite the PR's listed above. I am running 0.9.4. Our app sets portrait orientation, but  Samsung devices (galaxy S4, S5 and S7) all capture in landscape. I the deprecated fixOrientation to true, and that fixed the problem. Is there a build in the pipeline that fixes this without using fixOrientation? Thanks!

I've got the same issue with 0.9.0 (the release went to production about 10 minutes before 0.9.4 was released, d'oh) on several Samsung devices, most notably the Galaxy Tab A.
Chrome on my laptop displays the images as expected once uploaded to my application server, but passing them through any kind of processing library (GD2, et al) gives the aforementioned rotation result.

@adjustit That is probably something else. A photo's EXIF data contains the orientation, and opening the photo in most photo viewers it will automatically rotate the image. See for example: https://stackoverflow.com/questions/7489742/php-read-exif-data-and-adjust-orientation

@cornedor we have narrowed it down to an issue that only seems extant in Samsung devices, apparently it's a relatively common issue on certain classes of Galaxy device. The issue manifests itself by always setting orientation to 0 in the EXIF metadata, so as far as the code is concerned it's interpreting the directionality correctly.

Looks very related to this issue I reported by the cordova-camera-plugin developers: https://issues.apache.org/jira/browse/CB-12958

I can CONFIRM this issue exists.. when I use the default built in camera app
that comes with my Samsung Galaxy S7..
only one pic I took, that being the first DID NOT  contain any EXIF data OR the orientation FLAG..
I am pretty sure I used I guess there is a camera app built into Facebook's MESSENGER APP lol...
the other two photos I took used.. the default camera.. X(
UUGHH SAMSUNG!!!!!!!!!!!!
IDIOT DEVELOPERS!!!!!!!!!!!!!!!

Is there any way (either fix/hack) to get the orientation fixed in Samsung devices? I'm facing this issue for a long time.

@DaniAkash
I copy the images to a PC folder on Windows..
then right-click and choose "Rotate Clockwise"

@MelchiorGaspar
Is there a way to do it in the Native Java side?

not that I am aware of.. my method is just a work around...
I can't find any functionality in my Galaxy S7 that rotates the images in a folder...

Another workaround I discovered was letting my customers make a photo with their native cam app and then in my app pick this image from the gallery. It looks like native app sets the correct EXIF orientation.
A think that could speedup things is to vote for the issue I've created for the Cordova plugin at https://issues.apache.org/jira/browse/CB-12958

A hot fix is
allowEdit: true,
correctOrientation: true,
But not working withouy allowEdit: true
:(

Capture an image on an iPhone device in portrait mode and save it to camera roll Landscape wrong orientation. Did anyone solve this issue?

We fixt it to offer the user to rotate the image after sending it to the server. I don't know if the bug is fixed in version 3.0.0 of the camera plugin

I'm still have an issue with that!

I have the same kind of issue with face detection with camera2, normally face detection only work in portrait and with a samsung device, it only work in landscape...

Hi, still seeing this issue on a Samsung Galaxy Note 3, Android 5.0

closing in favor of https://github.com/react-native-community/react-native-camera/issues/1807


2017 chevy volt pscm not seen.

Describe the bug
I’m getting a PSCM not seen on a bug reported. i’m getting it on, yeah I official download 0.9.6 even Forks the same issue. There’s no codes on my car. And you can see my steering wheel. Rotate the torque being registered.

Which car does this affect?
2017 Chevy volt
Provide a route where the issue occurs
Car parameters
openpilot version
0.9.6
Additional info
Also it says to update my panda firmware can someone please help me I’ve tired all docs online and python commands. I’ve tried to reflash panda.

That picture isn't stock openpilot. Check out our Discord for help getting started.


More lines in the notification screen (unlocked)

The notifications are to short. In some rooms you only see who has written a message and not the content.
Some apps allows to adjust the amount of line which are shown in the notification.
Seen on iPhoneX and iPhone 6plus.

Can you attach a screenshot of this behavior? The notifications seem to get as big as any other notification on iOS and they expand to show the full message when you 3D Touch on them.

Notification:

Inside of riot-ios

3D touched

Oh I see, you want multiple messages in one notification.

The reason why you don't see the content I assume is because you don't have "Show decrypted content" turned on in the Riot iOS settings.

I have seen this switch, but I’m not able to switch it on.

As far as I know the only way you wouldn't be able to switch it on is because "Notifications on this device" is disabled. Possibly a bug with the state of that.

Both switches are on now, but for this channel I get only the sender.

Here one more example, where one room has the full text ("Test") and the other room only show the sender.

solved:
msgtype was not set to m.text.
This was the reason of only getting the notifier without the text.


make run-console-local dont exists anymore

Bug Report
The README.md page of this project mentions that we can launch this make rule make run-console-local to run a local OpenShift console but this command dont exist anymore
Questions:

Is the rule run-console-local deprecated` or replaced by another one ?
If the rule run-console-local is deprecated, then the doc should be updated and the section user-interface removed ?

The README.md page of this project mentions that we can launch this make rule make run-console-local to run a local OpenShift console but this command dont exist anymore

Could you check again? The command still works for me on master, and the relevant part of the Makefile hasn’t changed since May.

I just ran make run-console-local and verified that it works. Please reopen if you're still having problems.


Dependency override websocket broken

Describe the bug
Dependency override does not work for websockets.
The function add_api_websocket_route does not add dependency_overrides_provider to APIWebSocketRoute.
To Reproduce
Create a simple app with websockets and test it with override.
Expected behavior
The overrides should be taken into account, but the test uses the original dependency.
Environment

OS: Windows
FastAPI version: 0.49.0
Python version: 3.6.8

APIRouter.add_api_websocket_route should be modified to construct the APIWebSocketRoute like this:
route = APIWebSocketRoute(
            path,
            endpoint=endpoint,
            name=name,
            dependency_overrides_provider=self.dependency_overrides_provider,
        )

Thanks for the report @teucer !
This should be fixed in https://github.com/tiangolo/fastapi/pull/1122 by @amitlissack , in the latest version.
I'm gonna re-open this issue to let you confirm and close it.

Thank you for the quick turn-around!


Revert "Code for running motors"

Reverts AUV-IITK/fourtran#29

Send in a PR again, after fixing the directory structure.


Add publication credit guidelines and make visible

As discussedn here: https://nfcore.slack.com/archives/C02D7SW8LR3/p1688102269174659

Can we had a link to that when we use the nf-core create command?


Wiki not updated after 2.5.0

I was trying to switch from 2.5.0 to 4.0.0; there are a lot of behavior changes that are either improperly documented or regressions.
For instance, I haven't yet gotten the new point of createSrc: since it includes now everything by default, it seems you need in the opposite scenarios.
Then there are #293 and #296 — they aren't documented, and it's not clear whether they're intended.

Probably that is why I am having huge issues in multiprojects builds, crossproject builds and builds with macroses. I simply cannot make them compile from eclipse, probably something important in docs is  missing=(

@antonkulaga Maybe, but hand-waving like this isn't helpful. Could you make a proper bug report? Are the projects open source?
I have an working multi-project build available at https://github.com/inc-lc/ilc-scala, with version 3.0.0 of the plugin, and have used Eclipse on macro projects. (No experience with crosscompiled projects).
Some other things aren't supported AFAIK but can be avoided.

Look at the latest commit in development branch of my scala-js-binding library There I have 500+ errors

the installation page is also outdated, it should be:
addSbtPlugin("com.typesafe.sbteclipse" % "sbteclipse-plugin" % "4.0.0")


Add cargo bloat GHA action

Needs to revisit this, but since we get PRs from repositories outside of rtic-rs organisation this way is not available.


i18n(zh-tw): update tutorial/0-introduction

What kind of changes does this PR include?

Translated content

Description

Add tutorial/0-introduction in zh-TW.
Translate tutorial navigation into zh-TW.

conflicts resolved 🙏

LGTM! should be able to merge.


[UI-Components][Storybook] General page which documents automatically the juno color system. Also added a hidden how-to-mdx example page

Added automated general documentation section which automatically displays allowed listed colors from the juno color system. A hidden how-to-mdx page is added. fixes #433

No classes cannot be concatenated at runtime. Not if you want to be sure that they always work. It's random coincidence that the same class was maybe used elsewhere and hence already exists. What if the other place stops using that classname and then suddenly yours stops working as well. It's a bad idea to concatenate classes at runtime and if you do that elsewhere please change it :)


[DISCUSSION] Make filter totally asynchronous and event-driven.

Recently, I am thinking of making the whole invoker chain inside Dubbo asynchronous, event-driven. And I believe that Filter can be changed to better adapt to the new model.
Before 2.7.0, you can find the definition of Filter was completely synchronous:
public interface Filter {
    Result invoke(Invoker<?> invoker, Invocation invocation) throws RpcException {
       // before
        Result result = invoker.invoke(invocation);
       // after
      return result;
    }
}

In 2.7.0, a callback onResponse was added to allow users to handle the async value when result returns.
public interface Filter {
   Result invoke(Invoker<?> invoker, Invocation invocation) throws RpcException {
   }

   void onResponse(Result result, Invoker<?> invoker, Invocation invocation) {
        // biz return
    }
}

Now, how about the following definition, totally asynchronous and event-driven?
public interface Filter {
   void onSend(Invocation invocation) {
     // before invoke, throw exception to terminate
    }

   void onResponse(Result result, Invoker<?> invoker, Invocation invocation) {
        // biz return successfully
    }

   void onError(Throwable e) throws RpcException{
        // biz throw exception
    }
}

good idea. But I want to remind that in onError we still need some information other than Throwable, such as invocation and result.

@chickenlj
If we can reach a consensus, can you let me try it?
:)

@carryxyh Let's wait to hear more voices.

About compatibility issues.
The current version will enter onResponse due to all requests (correct or incorrect). If we modify the code, the correct response enters onResponse, the error response enters onError, and onResponse will only process the correct request if the user does not modify the Filter code.

Bringing Ian's question to here:

In your new proposed interface, how could we construct a filter chain? Say,
how could filter-a process further a value processed by filter-b?

I don't think we still need a nested Filter chain as before, the new Filter can simply work as follow:
Iterate(filters) {
    filter.onSend();
}

Result result = invoker.invoke();

result.whenComplete((value, t) {
  Iterate tail-to-head(filters) {
     filter.doResponse();
     filter.doError();
  }
});

Can we borrow the design of The Netty Project, say returning a org.apache.dubbo.filter.Future that is declared like below:
class Future<T> {
  T get(); // wait for the result synchronously
  void onComplete(SomeContext c); // asynchronously
  void onSend(SomeContext c); // asynchronously
  void onError(SomeContext c); // asynchronously
  // ... other callbacks
}

then users can do something like this:
Filter filter = ...;
Future f = filter.invoke(...);
// if synchronous call is needed
f.get();
// if asynchronous call is needed
f.onSend(context -> { /* lambda */ });
f.onError(context -> { /* lambda */ });
// ... others


build: link SeaStar with POSIX threads library.

It looks SeaStar isn't linking with pthreads while making extensive use of its procedures (like pthread setname_np in src/core/reactor.cc which was causing linking errors on my machine). I would kindly ask for a local review before going with this patch to the upstream.

https://github.com/ceph/ceph/pull/25671


Feb end game plan

planning plan #3872
release manager @Binyang2014
test period: 2/10 ~ 2/17

Test plan item
Feature issue id | Feature Owner | Test Owner(s) | Doc ready for test (Y/N)

VS Code | @yiyione | @qfyin | Test case: #4186
#4024 new UI | @debuggy | @yangou1988 | Y, use Yang's design doc as test cases;
#4020 Grafana | @Binyang2014  | @abuccts | No document needed
#4131 #4125 priority class + pod creation error pattern | @abuccts | @yqwang-ms | No document needed
#3866 auto deletion leak | @Binyang2014 | @yqwang-ms | to do: binyang open up leak for test
#4027 remove yarn version | @ydye | @abuccts | No, @abuccts will write test cases
#4081 storage manager restart issue | @Binyang2014 | @abuccts | No document needed
2 P1 security issues (secret/ACS) | @Binyang2014 | @mydmdm |  No document needed
Job history dev log | @debuggy | @debuggy | for debug only  (Any doc needed?)
#4105, #3823 | @Binyang2014 | @yiyione | submit job with runtime plugin(storage, ssh, tensorboard, ...), review its yaml file, make sure the behavior is as expect. (Need to check new job, clone job, import job)
#4014 | @Binyang2014 | @mydmdm | plugin as an optional requests
Job retry page issue #4008 |@debuggy | @debuggy | bug fix  (finish)
NNI integration | @SparkSnail
#4022 | @ydye  |@Binyang2014 | deploy and precheck
tag and commit number | @ydye/@debuggy | @Binyang2014 (#4190)
Finalize protocol - Job UUID #3935 | @abuccts | @debuggy  | check annotation and md5 decode
K8S Dashboard https enabled #4025 | @Binyang2014 | TBD | Need doc first
Service page #3892  | @sunqinzheng | @debuggy

done.


additional webhook message

Describe the feature
have a box under the webhook logs to send an additional message along with the embed, such as when the panic button webhook or 911 goes off you can add a additional message of pinging the 911 role along with the embed
Additional Context
No response


work update

add annotation - more dev friendly
getEmptyView  - get the existing view from the empty view or null if
the view is not inflated
please release an update on grade. thank you. @cymcsg  @cymcsg

Thank you! You can use 0.3.18 now.


ContentLanguage Option

There's a constant for a ContentLanguage header, but there's no option for it on option.go.
Is this missing for some specific reason or was it just forgotten? Would a PR adding it be welcome?

@hangzws mind taking a look at this issue? :)

I just made a PR for it - #128 - do you mind taking a look and approving/merging or requesting changes? :)


Is this up to date?

is that information is up to date ?
if so , when it will be supported regarding the System-internal or on-chassis serial ports

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

ID: e79afb68-4107-5766-a8ee-da81b31c5c0d
Version Independent ID: e8fb40ec-0774-30a9-47bb-69fca4df39b0
Content: Windows.Devices.SerialCommunication Namespace - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.devices.serialcommunication.yml&version=GBlive&_a=contents
Service: unspecified
GitHub Login: @wdg-dev-content
Microsoft Alias: wdg-dev-content

For onboard and X86 there is a  System.IO.Ports.SerialPort() that you can use. Unfortunately this only looks for COMx ports and rejects any UARTx port name that forces to have two versions of the app depending on the target platform. I use a wrapper class that hides this dual personality of the serial port to the rest of the application.

Apologies for a long-delayed response here.
As this is API reference content, it's kept up to date with the APIs available in the Windows SDK. If there's still nothing in this area that provides the functionality you want, I'd suggest providing product feedback. Thank you.

Sorry i didn't see the alert for this issue, Yes the problem is still the same that "System-internal or on-chassis serial ports may be enumerated by DeviceInformation.FindAllAsync(), but cannot be opened by SerialDevice.FromIdAsync() because they currently are not supported. However, serial ports connected over USB, such as on USB-to-Serial cables are supported."


allow changing stages after incident is ended

Summary
For consistency with other fields on the RHS, allow changing state even after ending the incident.
Ticket Link
https://mattermost.atlassian.net/browse/MM-28030

Codecov Report

Merging #285 into master will increase coverage by 0.59%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #285      +/-   ##
==========================================
+ Coverage   45.96%   46.56%   +0.59%
==========================================
  Files          22       22
  Lines        2837     2835       -2
==========================================
+ Hits         1304     1320      +16
+ Misses       1360     1335      -25
- Partials      173      180       +7

Impacted Files
Coverage Δ

server/incident/service.go
18.58% <0.00%> (+3.48%)
: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 c1e1bb1...8dc3544. Read the comment docs.


Update the PR template

Use new PR template. And here is an autolink: JIRA-2160

yeah good shout. I think we need to do a git diff before running some workflows @jheng-hao-lin-cko