"Forward" Google Geo Coder

so I read over how google set up the calls for the forward geo coding and it seems that the reverse geo coder classes could be use with a different parser and a update to the url builder to allow for address parameter rather than lat and log.
Would you like to have the two types of geo coders completely separate or should the forward geo coder only have a new parser and maybe a new url builder?

What is common between the two should be shared and what is different should be different. What I shouldn't see is two classes that are 90% the same with the exception of the word "forward" or "reverse". Does that make sense?
Two parsers could be fine, if they have the same interface then I would make them inherit or implement that interface to reduce code duplication.

alright, should i have the url be one class then... the only difference is  address= rather than latlng= in the url


Deprecating TASK.SUCCESS_DISTANCE since TASK.SUCESS.SUCCESS_DISTANCE replaces it

Motivation and Context

In our config files, there are two separate SUCCESS_DISTANCE keys, one under TASK.SUCCESS_DISTANCE and one under TASK.SUCCESS.SUCESS_DISTANCE. It seems only the TASK.SUCCESS.SUCESS_DISTANCE is used since SUCESS is a measure on its own.
How Has This Been Tested

Removed the TASK.SUCCESS_DISTANCE key and ran a few configs and the unit tests.
Types of changes

Docs change / refactoring / dependency upgrade

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Shouldn't we be using YACS deprecated keys? Or throwing an error if the key is found? Loading old configs could silently break otherwise.

Shouldn't we be using YACS deprecated keys? Or throwing an error if the key is found? Loading old configs could silently break otherwise.

You are right, I should do this. I find there is some trickiness here because of this line. YACS will only check if a new key is deprecated if adding new keys is not allowed. (I think it is a bug.) One option would be to check ourselves for deprecated keys (which will increase code complexity) or add to YACS deprecated keys and not have meaningful warnings when the user uses a deprecated key.


Update fetch ADR regarding msw

Turns out that msw 1.x (which has been deprecated for a long time but is in extensive use) is not compatible with native fetch.

Might be worth trying https://codemod.com/registry/msw-2-upgrade-recipe and see how much that can automate for us in all of our repos to get us there


feat: Tabs UI refinements

Closes #515.
Small adjustments to the tabs UI based on conversation in #515.

@bprusinowski I had a small question on the spinner so I did not approve so as not to trigger the auto merge, but this is LGTM if you think we should keep the spinner.


Updated to match current ArcGIS Online UI

people have complained that our instructions here weren't sufficiently detailed. thanks!

I think ArcGIS Online used to show a list of the actual URLs. Now we have hidden them a bit. So this doc was probably current at some point.

for sure, and yours will be out of date in another month or so when they push the new and improved item details pages.
nothing to do but keep trudging along :smile:


Refactoring: implement arrow as a type constructor

Currently, arrow type construction (a -> b) is treated as a special one.
This is making the source codes related to type system (traits.rs, types.rs) a bit complicated than necessary.
Change "->" to a type constructor of kind * -> * -> * defined in Std.

Resolved at b3a0584ad6f97b734893259bb1d871bd7b0fca3e


Update Master from Develop

Users are ready for the list view and AVG stuff to go live, so off we go!

:D


⚠ Cioos National CKAN has degraded performance

In b46093a, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 18714 ms

Resolved: Cioos National CKAN performance has improved in a326005 after 8 minutes.


Change paths and config from js to ts

Pull Request Test Coverage Report for Build 1297

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 1296:
0.0%

Covered Lines:
41

Relevant Lines:
41

💛  - Coveralls


Add C#/WinRT Projection dlls for AppLifecycle and DynamicDependency

Add C#/WinRT Projection dlls for Microsoft.Windows.AppLifecycle and Microsoft.Windows.ApplicationModel.DynamicDependency
Modeled after MRT's C#/WinRT projection dll, but accounting for MRT vs SharedProjects/ProjectReunion_DLL deltas.
As previously discussed, structured as 1 projection dll per namespace (which also matches how MRT's structured)

/azp

/azp run

The intellisense data might be an issue in its own right - consider opening an IOU and dropping comment pointers to it in each xml file.

I created an issue for 1.0 DynamicDependencies: Flesh out Intellisense .xml with doccomments
#906
We can use that as the stalking horse to figure it out as that's already got all the doccomment/content, just a matter of working out process/mechanism to pass that along to Intellisense.
AppLifecycle and MRTCore should add doccomments. Then whatever the process we work out for DynDep we can apply to them too for full coverage

Abandoning due to too old and stale to merge correctly. Replaced by Added C# WinRT projections for DynamicDependency and AppLifecycle WinRT APIs #1208


Apply Import Validation Checks (DTO) for Qualification LEE Data to Import API

QUALIFICATION LEE Data Format Checks

The value for QualificationTestDate must be formatted as a date yyyy-mm-dd
The value for PotentialAnnualMassEmissions must be numeric, 5 digits max within the range of -9999.9 and 9999.9, only 1 decimal
The value for ApplicableEmissionStandard must be numeric, 9 digits max within the range of -99999.9999 and 99999.9999, only 4 decimal
The value for PercentageOfEmissionStandard must be numeric, 5 digits max within the range of -9999.9 and 9999.9, only 1 decimal
If any of these fields do not contain valid values, it's an error. Error Message:
[QUALLEE-FATAL-A] The value for [fieldname] in the Qualification LEE record #[index of record] is not formatted properly

QUALIFICATION LEE Data Value Checks

The value for ParameterCode must be string and limited to the results from  (SELECT distinct parameter_code FROM camdecmpsmd.vw_quallee_master_data_relationships)
The value for QualificationTestType must be string and limited to the results from  (SELECT distinct qual_lee_test_type_code FROM camdecmpsmd.vw_quallee_master_data_relationships)
The value for UnitsofStandard must be string and limited to the results from  (SELECT distinct unit_of_standard_code FROM camdecmpsmd.vw_quallee_master_data_relationships)
If any of these fields do not contain valid values, it's an error. Error Message:
[QUALLEE-FATAL-B] The value for [fieldname] in the Qualification LEE record #[index of record] is invalid

Testing
Test Case: TC657
Tester @mosesdeeCVP
Clean Monitoring Plan

Test Passed


Images Orientation Problems

Web page: http://www.morphmarket.com/c/reptiles/pythons/ball-pythons/36079
Click on image to bring up ekko-lightbox, and notice the orientation is different.  All other tools (browser, Facebook, and Mac Preview) render this image in portrait, but lightbox shows it in landscape.  This happens on a number of images.
Verified on Chrome and Safari.
Raw image: https://morphmarket-media.s3.amazonaws.com/media/raw_images/repterra/2016/08/20160822143729-image.jpeg

Interesting, opening the image shows the correct orientation but inspector shows it landscape:

I think you're running into this: https://bugs.chromium.org/p/chromium/issues/detail?id=56845
http://www.daveperrett.com/articles/2012/07/28/exif-orientation-handling-is-a-ghetto/

The reason I believed it was a lightbox error is because, middle-click open image in new window (without lightbox) produces correct portrait orientation.  Left-click opening the image in lightbox produces the (wrong) landscaped orientation.
However I understand from your links that the handling for this by the browser is very flaky and that this can be explained by its weirdness.
Thanks for the links.


Setting plugin specific configuration in vernemq.conf crashes the system

Environment

VerneMQ Version:1.0.0rc2
OS:Ubuntu 16.04
Erlang/OTP version (if building from source):

Expected behavior

Should be able to set Plugin specific settings in vernemq.conf

Actual behaviour

Setting plugin specific configuration in vernemq.conf crashes the system
As per below release notes, I will be able to set plugin specific settings in /etc/vernemq/vernemq.conf, However it crashes when tried to set the setting

https://github.com/erlio/vernemq/blob/b7cb18779733d73ad9769dce83ce3694c54dc732/changelog.md
Entries in /etc/vernemq/vernemq.conf
plugins.myplugin = on
plugins.myplugin.path = set to local path
myplugin.testsetting = Test
###Error as below
root# vernemq console
15:41:06.807 [error] You've tried to set myplugin.testsetting, but there is no setting with that name.
15:41:06.807 [error]   Did you mean one of these?
15:41:06.838 [error]     plugins.$name
15:41:06.838 [error]     plugins.$name.path
15:41:06.838 [error]     plugins.vmq_acl
15:41:06.842 [error] Error generating configuration in phase transform_datatypes
15:41:06.842 [error] Conf file attempted to set unknown variable: myplugin.testsetting
Error generating config with cuttlefish
run vernemq config generate -l debug for more information.
###Here is my schema file in priv/myplugin.schema
{mapping, "myplugin.testsetting", "myplugin.testsetting", [
{default, "Test"},
{datatype, string}
]}.

Hi, thanks for reporting. I think this might be a chicken-and-egg problem we didn't think about where vernemq doesn't know about the schema file of your plugin when booting. As a workaround, could you try and copy your schema file into the share/schema folder of vernemq and see if it works then?

Thanks, Iarshesel, It worked! Will this be fixed in future so this workaround can be avoided? Also, can it be updated in the documentation too for plugins?

I don't think this can be fixed easily as VerneMQ would have to know where to look for the schema file somehow (before it parses the vernemq.conf file). Documenting this might be the best option - but maybe we'll come up with an elegant solution ;)
Thanks again for bringing this to our attention!

We haven't come up with a better solution, so the workaround described is the solution we have to live with for the time being.


docs: v0.2.0

Description
Document change since v0.1.0
Related Tickets & Documents
Completes v0.2.0
Steps to Verify
New tables have been tested manually; the examples in the documentation provide a useful starting point

Summary by CodeRabbit

New Features

Introduced new tables to enhance data management and reporting capabilities.
Implemented pre-commit hooks to streamline local development processes.
Enhanced security measures with the addition of CodeQL and OSSF Scorecard scanning in GitHub Actions.

Refactor

Optimised client functionality with alternate search field names and keyed collections.

Documentation

Updated documentation to include comprehensive contributing guidelines, templates, support documents, and examples.

Chores

Updated the Go programming language to version 1.21.9.

@coderabbitai review


Update version format for Konflux builds

For example, in release branch release-0.2, use v0.2.<patch> for the version in Konflux builds, instead of 0.2-<build-number>-<sha>.
Also bump version from 0.1 to 0.2.
Ref: EC-438

Let's merge.


Add standard settings

The following settings have been added.

Set parallel execution to true
Change format to progress

Documentation

parallel passes down to Rubocop, and it does this:

Use available CPUs to execute inspection in parallel.

progress also passes down to Rubocop, and prints a . for each clean file.

Both of these are the Rubocop defaults. Standard overrides the defaults:

parallel interacts poorly with stdin mode. As such, it is forcibly overridden in stdin mode, and the default is not to use it.
progress doesn't mention standard, which seems to be why the Standard::Formatter was introduced.

Analysis
I don't even use Standard, but it runs as part of CI. Sure, I'll merge these changes, and if someone complains I'll back them out.


bug(MatRadioGroup): paging options prevents keyboard focus

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Using a mat-paginator to page the options in a MatRadioGroup prevents the radio group from receiving focus using the tab key on pages other than the first page.
Reproduction
StackBlitz link: https://stackblitz.com/edit/q2wcmz
Steps to reproduce:

Click in the input box at the top to give it focus
Press tab; note that the radio group receives focus
Press tab again; note that this gives focus to the next-page button in the paginator
Press space or enter to change to the next page; note that the radio group options have changed
Press shift+tab; note that the paginator previous-page button receives focus
Press shift+tab again; note that the input box receives focus, skipping over the radio group
Press tab; note that the paginator receives focus, skipping over the radio group

Expected Behavior
After changing pages, the radio group should still be able to receive focus using the keyboard.
Actual Behavior
After changing pages, the radio group is unable to receive focus using the keyboard
Environment

Angular: 15.1.1
CDK/Material: 15.1.1
Browser(s): Chrome 114.0.5735.110, Firefox 115.0.2 (64-bit)
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

Hello,
At first glance, this looks like an accessibility anti-pattern to me. For accessibility, I don't suggest combining paginator and radio buttons in this way. Making dynamic updates without notifying the user can confuse users of assistive technology.
Although, I am able to reproduce without using paginator in this stackblitz: https://stackblitz.com/edit/q2wcmz-f8vccs?file=src%2Fapp%2Fradio-overview-example.ts
This seems to happen when the selected radio option is removed. After removing the selected radio option, I can no longer tab to the radio group.
Giving P4 priority and assigning to @mmalerba as owner of radio.
Best regards,
Zach

To add some clarification/detail...
You're correct, the radio group is unable to receive focus when the selected item is not on the current page. I should have been more clear about that.
Another detail is that this problem does not occur when no item is selected. The reason for this is that the control internally uses tabindex to change the keyboard navigation behavior when an item is selected vs. when no item is selected.


added .net core .csproj, .xml & nupkg

So I pulled the main .cs files over from the old standard to Core1.1 My current dev setup made running the unit tests weird so if you want to give them a run on your end and give me some feedback I can look into any problems.

@jamesmanning I think Travis CI is failing due to the new .net implamentation of the .csproj files. ill look into the appveyor now

You want to delete the binary nupkg too, btw; it's a compiled output.

Lots of small points, but overall: YAY!

@EamonNerbonne I'm still having Namespace problems, i'm 100% sure when i understand i'm going to be kicking myself but can you shead any light?

I tried to cleanup whatever else I noticed; I put that in #13 - can you check if that works for you?  Because everything except for the tests appears to work for me...


qiskit.synthesis.MatrixExponential breaks if evolution time is a bound Parameter

Environment

Qiskit Terra version: 0.19.0
Python version: 3.7.9
Operating system: mac OS

What is happening?
Using the PauliEvolutionGate with a parameterized time and the MatrixExponential as synthesis method breaks. The reason is that the evolution time, after being bound, is not cast to a float but remains a ParameterExpression with 0 free parameters. That type is incompatible with SciPy which is used to compute the matrix exponential.
How can we reproduce the issue?
from qiskit.circuit import QuantumCircuit, Parameter
from qiskit.circuit.library import PauliEvolutionGate
from qiskit.synthesis import SuzukiTrotter, MatrixExponential
from qiskit.opflow import I, X, Y, Z
from qiskit.quantum_info import Operator

# evolution time and operator we evolve
time = Parameter("t")
op = (I ^ X ^ Y) + (I ^ Y ^ X) - (Z ^ Z ^ I)

# evolution gate
synth = MatrixExponential()
evo = PauliEvolutionGate(op, time=time, synthesis=synth)

# plug into circuit
circuit = QuantumCircuit(op.num_qubits)
circuit.append(evo, range(op.num_qubits))
print(circuit.draw())

# bind time to some value and obtain matrix
value = 0.23
bound = circuit.bind_parameters([value])  # or {time: value}

print(bound.decompose())

produces
Traceback (most recent call last):
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/circuit/parameterexpression.py", line 444, in __float__
    return float(self._symbol_expr)
  File "symengine_wrapper.pyx", line 1143, in symengine.lib.symengine_wrapper.Basic.__float__
  File "symengine_wrapper.pyx", line 968, in symengine.lib.symengine_wrapper.Basic.n
  File "symengine_wrapper.pyx", line 4278, in symengine.lib.symengine_wrapper.evalf
RuntimeError: Not Implemented

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "evoop.py", line 24, in <module>
    print(bound.decompose())
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/circuit/quantumcircuit.py", line 1528, in decompose
    decomposed_dag = pass_.run(circuit_to_dag(self))
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/transpiler/passes/basis/decompose.py", line 90, in run
    if node.op.definition is None:
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/circuit/instruction.py", line 232, in definition
    self._define()
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/circuit/library/pauli_evolution.py", line 105, in _define
    self.definition = self.synthesis.synthesize(self)
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/synthesis/evolution/matrix_synthesis.py", line 43, in synthesize
    exp = expm(-1j * time * matrix)
  File "/Users/jul/opt/miniconda3/envs/py37/lib/python3.7/site-packages/scipy/linalg/matfuncs.py", line 255, in expm
    return scipy.sparse.linalg.expm(A)
  File "/Users/jul/opt/miniconda3/envs/py37/lib/python3.7/site-packages/scipy/sparse/linalg/matfuncs.py", line 591, in expm
    return _expm(A, use_exact_onenorm='auto')
  File "/Users/jul/opt/miniconda3/envs/py37/lib/python3.7/site-packages/scipy/sparse/linalg/matfuncs.py", line 626, in _expm
    A = A.astype(float)
  File "/Users/jul/Qiskit/qiskit-terra/qiskit/circuit/parameterexpression.py", line 450, in __float__
    ) from exc
TypeError: ParameterExpression with unbound parameters (set()) cannot be cast to a float.

What should happen?
Properly evolve.
Any suggestions?
See #7508

With slightly modified code to account for changes in Qiskit since this issue:
from qiskit.circuit import QuantumCircuit, Parameter
from qiskit.circuit.library import PauliEvolutionGate
from qiskit.synthesis import SuzukiTrotter, MatrixExponential
from qiskit.quantum_info import Operator, SparsePauliOp

# evolution time and operator we evolve
time = Parameter("t")
op = SparsePauliOp.from_list([("IXY", 1), ("IYX", 1), ("ZZI", -1)])

# evolution gate
synth = MatrixExponential()
evo = PauliEvolutionGate(op, time=time, synthesis=synth)

# plug into circuit
circuit = QuantumCircuit(op.num_qubits)
circuit.append(evo, range(op.num_qubits))
print(circuit.draw())

# bind time to some value and obtain matrix
value = 0.23
bound = circuit.assign_parameters([value])  # or {time: value}

print(bound.decompose())

this no longer seems to be reproducible with Qiskit 1.0, so I'll close the issue.


panic on use of cloned BuiltEntityClone

The following triggers a panic on current master (9d9951d)
use hecs::*;

#[derive(Clone)]
struct A;

fn main() {
    let mut world = World::new();

    let mut builder = EntityBuilderClone::new();
    builder.add(A);
    let built_entity = builder.build();

    let built_entity = built_entity.clone();

    world.spawn(&built_entity);
}

Changing impl Clone for Common<DynamicClone> @ https://github.com/Ralith/hecs/blob/9d9951dd663ff90f1b89d76e465009829565c2bc/src/entity_builder.rs#L406 to clone the ids field seems to prevent the panic.

Thanks for the excellent report! Fix at https://github.com/Ralith/hecs/pull/254.

Fix released in 0.7.6.


Build error in Sample App for OpenThread NCP

Describe the bug
When attempting to build the OpenThread NCP sample app for Silicon Labs Thunderboard Sense board.
Building and flashing other sample apps like blinky and hello-world work fine. The build error refers to CONFIG_NET_CONFIG_IEEE802154_DEV_NAME being undeclared. Is this supposed to supplied as a build argument?
To Reproduce
Steps to reproduce the behavior:
Command:
west build -p -b efr32mg_sltb004a samples/net/openthread/ncp
Exits with the following error:
/home/blitz/zephyrproject/zephyr/subsys/net/lib/openthread/platform/radio.c:185:33: error: 'CONFIG_NET_CONFIG_IEEE802154_DEV_NAME' undeclared (first use in this function)
185 |  radio_dev = device_get_binding(CONFIG_NET_CONFIG_IEEE802154_DEV_NAME);
|                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/blitz/zephyrproject/zephyr/subsys/net/lib/openthread/platform/radio.c:185:33: note: each undeclared identifier is reported only once for each function it appears in
[562/617] Building C object modules/mb..._crypto__mbedtls.dir/library/rsa.c.obj
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1: /usr/bin/cmake --build /home/blitz/zephyrproject/zephyr/build
Expected behavior
A sample build of the OpenThread NCP would demonstrate capability of Zephyr running simple communications over the Thread stack.
Impact
As I am unable to run this quick check before we move out from Silabs stack, it is stopping me from convincing my team to move to Zephyr as our long term dev platform for our OpenThread based IoT devices.
Screenshots or console output
blitz@workdesk:~/zephyrproject/zephyr$ west build -p -b efr32mg_sltb004a samples/net/openthread/ncp
-- west build: making build dir /home/blitz/zephyrproject/zephyr/build pristine
-- west build: generating a build system
Including boilerplate (Zephyr base): /home/blitz/zephyrproject/zephyr/cmake/app/boilerplate.cmake
-- Application: /home/blitz/zephyrproject/zephyr/samples/net/openthread/ncp
-- Zephyr version: 2.2.99 (/home/blitz/zephyrproject/zephyr)
-- Found Python3: /usr/bin/python3.6 (found suitable exact version "3.6.9") found components: Interpreter
-- Board: efr32mg_sltb004a
-- Found west: /home/blitz/.local/bin/west (found suitable version "0.7.2", minimum required is "0.7.1")
ERROR: ld.so: object 'libgtk3-nocsd.so.0' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
-- Found dtc: /home/blitz/zephyr-sdk-0.11.2/sysroots/x86_64-pokysdk-linux/usr/bin/dtc (found suitable version "1.5.0", minimum required is "1.4.6")
-- Found toolchain: zephyr (/home/blitz/zephyr-sdk-0.11.2)
-- Found BOARD.dts: /home/blitz/zephyrproject/zephyr/boards/arm/efr32mg_sltb004a/efr32mg_sltb004a.dts
ERROR: ld.so: object 'libgtk3-nocsd.so.0' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
efr32mg_sltb004a.dts.pre.tmp:77.26-85.5: Warning (spi_bus_bridge): /soc/usart@40010800: node name for SPI buses should be 'spi'
also defined at efr32mg_sltb004a.dts.pre.tmp:302.9-320.3
: Warning (spi_bus_reg): Failed prerequisite 'spi_bus_bridge'
-- Generated zephyr.dts: /home/blitz/zephyrproject/zephyr/build/zephyr/zephyr.dts
-- Generated devicetree_unfixed.h: /home/blitz/zephyrproject/zephyr/build/zephyr/include/generated/devicetree_unfixed.h
Parsing /home/blitz/zephyrproject/zephyr/samples/net/openthread/ncp/Kconfig
Loaded configuration '/home/blitz/zephyrproject/zephyr/boards/arm/efr32mg_sltb004a/efr32mg_sltb004a_defconfig'
Merged configuration '/home/blitz/zephyrproject/zephyr/samples/net/openthread/ncp/prj.conf'
Configuration saved to '/home/blitz/zephyrproject/zephyr/build/zephyr/.config'
Kconfig header saved to '/home/blitz/zephyrproject/zephyr/build/zephyr/include/generated/autoconf.h'
warning: TEST_RANDOM_GENERATOR (defined at boards/shields/esp_8266/boards/sam4e_xpro.defconfig:17,
subsys/random/Kconfig:8) was assigned the value 'y' but got the value 'n'. Check these unsatisfied
dependencies: ((BOARD_SAM4E_XPRO && NETWORKING && SHIELD_ESP_8266) || !ENTROPY_HAS_DRIVER) (=n). See
http://docs.zephyrproject.org/latest/reference/kconfig/CONFIG_TEST_RANDOM_GENERATOR.html and/or look
up TEST_RANDOM_GENERATOR in the menuconfig/guiconfig interface. The Application Development Primer,
Setting Configuration Values, and Kconfig - Tips and Best Practices sections of the manual might be
helpful too.
warning: SHELL_STACK_SIZE (defined at subsys/shell/Kconfig:20) was assigned the value '3072' but got
the value ''. Check these unsatisfied dependencies: SHELL (=n). See
http://docs.zephyrproject.org/latest/reference/kconfig/CONFIG_SHELL_STACK_SIZE.html and/or look up
SHELL_STACK_SIZE in the menuconfig/guiconfig interface. The Application Development Primer, Setting
Configuration Values, and Kconfig - Tips and Best Practices sections of the manual might be helpful
too.
-- The C compiler identification is GNU 9.2.0
-- The CXX compiler identification is GNU 9.2.0
-- The ASM compiler identification is GNU
-- Found assembler: /home/blitz/zephyr-sdk-0.11.2/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc
-- Cache files will be written to: /home/blitz/.cache/zephyr
-- Version: g2ac83762
-- Configuring done
-- Generating done
-- Build files have been written to: /home/blitz/zephyrproject/zephyr/build
-- west build: building application
[1/617] Preparing syscall dependency handling
[460/617] Building C object zephyr/sub...p__l2__openthread.dir/openthread.c.obj
/home/blitz/zephyrproject/zephyr/subsys/net/l2/openthread/openthread.c:85:13: warning: 'ipv6_addr_event_handler' defined but not used [-Wunused-function]
85 | static void ipv6_addr_event_handler(struct net_mgmt_event_callback *cb,
|             ^~~~~~~~~~~~~~~~~~~~~~~
/home/blitz/zephyrproject/zephyr/subsys/net/l2/openthread/openthread.c:78:39: warning: 'ip6_addr_cb' defined but not used [-Wunused-variable]
78 | static struct net_mgmt_event_callback ip6_addr_cb;
|                                       ^~~~~~~~~~~
[555/617] Building C object zephyr/sub...es/openthread_platform.dir/radio.c.obj
FAILED: zephyr/subsys/net/lib/openthread/platform/CMakeFiles/openthread_platform.dir/radio.c.obj
ccache /home/blitz/zephyr-sdk-0.11.2/arm-zephyr-eabi/bin/arm-zephyr-eabi-gcc -DBUILD_VERSION=zephyr-v2.2.0-2377-gf95c349c36ab -DEFR32MG12P332F1024GL125 -DKERNEL -D_FORTIFY_SOURCE=2 -D__PROGRAM_START -D__ZEPHYR__=1 -I../include -Izephyr/include/generated -I../soc/arm/silabs_exx32/efr32mg12p -I../subsys/settings/include -I/home/blitz/zephyrproject/modules/hal/cmsis/CMSIS/Core/Include -I/home/blitz/zephyrproject/modules/hal/silabs/gecko/Device/SiliconLabs/EFR32MG12P/Include -I/home/blitz/zephyrproject/modules/hal/silabs/gecko/emlib/inc -Imodules/openthread/build/etc/cmake -I/home/blitz/zephyrproject/modules/lib/openthread/etc/cmake -I/home/blitz/zephyrproject/modules/lib/openthread/include -isystem ../lib/libc/minimal/include -isystem /home/blitz/zephyrproject/modules/lib/openthread/zephyr/../include -isystem /home/blitz/zephyrproject/modules/lib/openthread/zephyr/../examples/platforms -isystem /home/blitz/zephyr-sdk-0.11.2/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/9.2.0/include -isystem /home/blitz/zephyr-sdk-0.11.2/arm-zephyr-eabi/bin/../lib/gcc/arm-zephyr-eabi/9.2.0/include-fixed -Os -imacros/home/blitz/zephyrproject/zephyr/build/zephyr/include/generated/autoconf.h -ffreestanding -fno-common -g -mcpu=cortex-m4 -mthumb -mabi=aapcs -imacros/home/blitz/zephyrproject/zephyr/include/toolchain/zephyr_stdint.h -Wall -Wformat -Wformat-security -Wno-format-zero-length -Wno-main -Wno-address-of-packed-member -Wno-pointer-sign -Wpointer-arith -Wno-unused-but-set-variable -Werror=implicit-int -fno-asynchronous-unwind-tables -fno-pie -fno-pic -fno-strict-overflow -fno-reorder-functions -fno-defer-pop -fmacro-prefix-map=/home/blitz/zephyrproject/zephyr/samples/net/openthread/ncp=CMAKE_SOURCE_DIR -fmacro-prefix-map=/home/blitz/zephyrproject/zephyr=ZEPHYR_BASE -fmacro-prefix-map=/home/blitz/zephyrproject=WEST_TOPDIR -ffunction-sections -fdata-sections -std=c99 -nostdinc -MD -MT zephyr/subsys/net/lib/openthread/platform/CMakeFiles/openthread_platform.dir/radio.c.obj -MF zephyr/subsys/net/lib/openthread/platform/CMakeFiles/openthread_platform.dir/radio.c.obj.d -o zephyr/subsys/net/lib/openthread/platform/CMakeFiles/openthread_platform.dir/radio.c.obj   -c /home/blitz/zephyrproject/zephyr/subsys/net/lib/openthread/platform/radio.c
/home/blitz/zephyrproject/zephyr/subsys/net/lib/openthread/platform/radio.c: In function 'platformRadioInit':
/home/blitz/zephyrproject/zephyr/subsys/net/lib/openthread/platform/radio.c:185:33: error: 'CONFIG_NET_CONFIG_IEEE802154_DEV_NAME' undeclared (first use in this function)
185 |  radio_dev = device_get_binding(CONFIG_NET_CONFIG_IEEE802154_DEV_NAME);
|                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/blitz/zephyrproject/zephyr/subsys/net/lib/openthread/platform/radio.c:185:33: note: each undeclared identifier is reported only once for each function it appears in
[562/617] Building C object modules/mb..._crypto__mbedtls.dir/library/rsa.c.obj
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1: /usr/bin/cmake --build /home/blitz/zephyrproject/zephyr/build
Environment (please complete the following information):

OS: Linux Ubuntu 18.04.4 LTS
Toolchain Zephyr-SDK-0.11.2

@vijayrao1988 I am not sure 802.15.4 is supported on Silicon Labs devices due to the requirement for a binary module. @mnkp can I'm sure expand on that a bit.

The build error refers to CONFIG_NET_CONFIG_IEEE802154_DEV_NAME being undeclared. Is this supposed to supplied as a build argument?

No, this is a Kconfig option. Looking at the list of supported drivers I don't see anything related to Silicon Labs. Likely because of this issue

Closing since this is not supported on SiLabs boards. Please follow this issue to get updates on the binary blob discussion.


Debugger UI not split into four quadrants

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (ba9fa20 2024-06-24T00:20:52.822818467Z)
What command(s) is the bug in?
forge debug
Operating System
Linux
Describe the bug
The debugger layout in the document looks like:

But mine actually looks like:

My display resolution is 1920x1080, the debugger window is maximized.  I tried both Wezterm and Kitty terminal, no difference.
How to split it into four quadrants?
OS: Fedora 40, KDE

Hi @aj3423, have you tried reducing the font size of your terminal? The breakpoint for the four quadrants layout is a minimum width of 200 characters.

@zerosnacks I set the font size to smaller and it worked, now there is four quadrants. But the font looks a bit too small, is it possible add a feature to force the four quadrants layout? Or make the quadrant size adjustable by launch parameter? The part of the red rectangle is a bit wasted.


Update the resource list web page

In the United States: Call 911 for emergency services or call 988 for the Suicide and Crisis Lifeline
If outside the United States, call the emergency number or Suicide Hotline listed for your country of residence: https://blog.opencounseling.com/suicide-hotlines/
Will also need this translated to spanish

Where exactly does this need to go?
Do we need the spanish version before publishing?

Talked with Bethany, I have a screenshot below of what to put where (adding subsections for "In U.S." and "Outside U.S.", just changing the number to 988 is good no need to include entire first sentence from above, include second sentence under "outside US"


Make the privacy policy background scroll with the content

Currently the background stays in one place while you scroll down the page. Instead, the background should scroll as well.

Implemented in 0e29b75.


[Feature Request]: Nav-Bar for contributors page

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
nav bar for contributors page
Use Case
can easily visit other pages
Benefits
can easily visit other pages
Add ScreenShots
before  :

want to add this  :

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

@apu52 please assign me this issue

@ANKeshri nav bar not needed in that page , instead make a return to home button .I am assigning u this issue .Make the changes


Move the package to org.embulk.util.json from org.embulk.spi.json

To be independent from embulk-core's, moving their packages.

Thanks!


New interface to set user groups and permission (as matrix)

We need a better overview of all groups and their permissions at one(!) page with possibility to enable/disable special permissions via mouse click.
Suggestion: build overview as matrix table like permissions dialog in Gallery3:
         read   write  ...
role 1    X       -
role 2    X       X
role 3    -       -

We have to discuss the ui concept. Implement it in angular should be easy. It's a 2.1 feature...

Short note to everybody who concerns: After merge of this feature it is (more or less) necessary to reset the database because we do not have any migrations between the old and the new built-in groups.

Is it possible to create migrations? One gole of OpenSlides 2.0 was, that there is no need to recreate the Database. At least from 2.0 to 2.1 an Update should be possible.
If I would be still active in OpenSlides, I would Veto this pull request without a migration

I’m strongly with @ostcar - migrations should be of huge concern as having to reset your database is probably the worst thing you can do to a user without having a major release jump to argue for it.

See #2315.


Added for key, value in map

Fix ++/-- on arrays
Updated ast.AssocExpr
Bool true is now equal to 1/1.0 instead of 0

env is now at 100% coverage again!
Fixed Addr for external
Updated and added test to env external resolver
Added TestGetInvalid
Added tests to TestFunctions

Thank you


Is this working at all?

Hi, I want to avoid using libzmq as it means I can't create binaries with no dependencies for OSX. How far along was this project?

Hey there, I have not touched this code in a long time. It can speak part of the wire protocol. It might be a good jumping off point, if only to understand the binary framing. The library never got around to handling queue behavior, retries, etc. I would not consider this usable replacement for ZMQ.


luarocks make --pack-binary-rock must not require root privileges

Got error:
$ git clone https://github.com/lua-nucleo/lua-nucleo.git
$ cd lua-nucleo
$ luarocks make ./rockspec/lua-nucleo-scm-1.rockspec --pack-binary-rock

Error: Your user does not have write permissions in /usr/local/
-- you may want to run as a privileged user or use your local tree with --local.

while I thought that "luarocks make --pack-binary-rock" doesn't actually need to get anything from /usr/local.

You are right, it shouldn't perform this check when using --pack-binary-rock.

Thank you for the help!


Set StatusBar color

So far I have come up with this.
The issue is that .setStatusBarColor is only supported in SDK 21.
So You have to change android.platform to 21 in build.properties
Since Titanium 4.0  21 is the target SDK, but I'm not sure if it is ok to change since it would drop support for earlier versions of Titanium.
Maybe You can test these changes?

Tested on android 5.0 and 4.4. Works as expected on Titanium 4.0.
Not sure about earlier versions of Titanium.

Yes - thats true. Failed to remove that when updating

Hey man, thank you for this PR - great help!
Just one question, why did you replace some constants with static numbers?

It was while I was playing around while trying to figure out why it is not compiling. Turned out that you have to set Android SDK to newer version. Reverted back to constants.
Still there is question about bumping minimum supported SDK versions.

You can use the Android SDK 21 with earlier Versions of Titanium. So it is not necessary to change the minimum Titanium SDK. Since you restrict setting setStatusBarColor to be only executed if the API >= 21 this should be allright.

OK, I've tested this. It works just fine! No problems with Titanium 3.5.0 and Android 4.3.
Merging!


disable Toast and logger for production env

I'm was following AngularJS Patterns: Clean Code by pluralsight.
Is there any way to disable Toast and logger for production env using modular app?
Any idea?
Appreciate your help. Thanks

@barcelona23 Do you solved this issue?


Add a ns-export dataset command to generate colmap/ns-data datasets from a trained model.

Is your feature request related to a problem? Please describe.
Currently, it's possible to export (refined) camera poses from a trained model in json form.
Unfortunately, the exported json is not directly compatible with the transforms.json input (image paths are changed to absolute, image order is changed, train and eval datasets are split and parameters required for transforms.json are omitted), so a lot of manual conversion needs to be done.
Additionally, splatfacto relies on the colmap .bin files and pointcloud as well.
If you only export the refined poses, modify them into a valid transforms.json, then try to train a splatfacto with the resulting poses, the initial pointcloud is misaligned, only random init is usable.
Describe the solution you'd like
Ideally I would like a way to export a complete ns-data/colmap dataset, optionally including exporting an aligned pointcloud generated from our nerf/splat result (with configurable sparsity).
Additionally, being able to simply export transforms.json in a compatible format directly would also be nice to have, but limited due to above outlined alignment issues.
Describe alternatives you've considered
An external script could do it, of course, but it feels like it would be more useful to have it in Nerfstudio directly.
Additional context
Use case: If you have a dataset with poor poses, you can optimize them by running a training run with pose optimization enabled (i.e. running a nerfacto-huge), but the output quality suffers during that run. Also, not all models might be equally good at pose optimization (i.e. nerfacto-huge might be better at it than splatfacto-big).
"Pre-running" a training and exporting the poses/dataset can result in a significantly better dataset, and thus a significantly better result using other methods.
Plus having the "fixed" dataset means it can be used in other projects as-is too.
I've tried to look into implementing it myself, but I don't know the Nerfstudio codebase well enough to make it happen (yet) with the limited time I have.

This would be great! I would like to use nerfacto with the camera optimization on, and then train splatfacto on the original dataset with the adjusted poses. To do this currently, you would need to manually convert the output from "ns-export cameras" to align with the original input data.


Prepare comments in literate programming style in script input/produce.sh

Prepare comments in literate programming style in script input/produce.sh

Fixed via https://github.com/RedHatInsights/insights-results-aggregator-utils/commit/4d86831899328e60aff39516cefef97cb29de40a


Added redirecting functionality  to index.html on clicking login and Signup

Issue
Fixes #52
Description
Added redirecting functionality  to index.html on clicking login and Signup
Type of change
You can write manually also

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation change

Screenshot
Screenshot of the fix if required.
Checklist:

[x] My code follows the style guidelines of this project
[x] The app runs properly
[x] I have performed a self-review of my own code
[x] I have formatted and commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

A special thanks to @shmiitg  for helping to come out from chaos

A special thanks to @shmiitg bhaiya for helping to come out from chaos

sorry i was checking it out now, but i suddenly saw this. i was very busy today @ShivaRapolu01  bhaiya , please pardon me for being late. @ShivaRapolu01 @shmiitg i am giving points to both of you bhaiya. Both of you are senior to me. thank you

@ArchismanKarmakar no problem brother , only a small change was to be made but I took it upto creating another file

@ArchismanKarmakar no problem brother , only a small change was to be made but I took it upto creating another file

there is a small bug @ShivaRapolu01 bhaiya please fix that , the url should be https://localhost...... but there is http & it is not working.


WIP: test-careplan

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #[issue number]
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

@DebbieArita Kindly update this PR so we can have it merged.

@DebbieArita please add an appropriate title for this PR


ActionMenus are too small!

This line https://github.com/frontapp/front-ui-kit/blob/d8a8951a1ab9d882cb6dd9220366c052d3a1aad7/src/components/popover/reposition.tsx#L63 forces <ActionMenu> to have a maximum width of 160px, regardless of its width prop.

Actually I'm not really sure that the problem lies in that line, but the issue is there 😕

Actually, the problem is here:
https://github.com/frontapp/front-ui-kit/blob/main/src/elements/dropdown/dropdown.tsx#L201
the minWidth prop of the Dropdown component is not passed down

:wave: Thanks for raising this issue. This is by design, action menus are suppose to be a smaller menu to do quick actions from. If you are needing a larger menu, you should use the Dropdown component (Ref) and build out a custom dropdown.
But as you noted, it does seem like we are not truly supporting the minWidth parameter. I have fixed this in https://github.com/frontapp/front-ui-kit/pull/177 and now when changing the size of the ActionMenu it does grow bigger.


[16.04] no more sudo ?

I have a set of environment development docker image based on this image, and my build rely on sudo which seems to be no more existing since the update to 16.04. Is it on purpose?(as I can't see it in the changelog) if so could it be possible to know the rational and the workaround (i.e if it's because it can cause security issue, I will not try to install sudo myself for example :) )

Baseimage-docker is meant to be minimal. It only installs the bare minimum it needs. The fact that sudo is not installed by default means only that: it is not installed by default. It is not removed on purpose. It install it yourself i you want to. The fact that it was installed by default on previous versions was probably because some package dependend on it, not because it was consciously included.

ok, no problem, perfectly make sense. thanks for you answer :)


Updated matcher description

Updated the matcher's documentation with the new description for collection.shouldContainAllInAnyOrder().
@Kerooker I believe this closes #933

Thanks, we'll merge this when 4.0 is released.


Tooltip

This adds tooltip when hovering the prompt type. This is for us who can't remember what each type does.
Feel free to modify tooltip that is more accurate, I don't have a good grasp on how exactly it behaves.

Thanks! That's a good idea. The descriptions are mostly good.


Created file es-DO.js

Added file translation for Dominican Republic

Please add it to demo index.html

You forgot to include your language file to the page. Also there is another index.html for Bootstrap 3 example.

Hi, sorry, i has not used the demo html files, i added the i18n js file and updated index.html and index-bs3.html

Thank you. Now everything is fine.


stdfork does not respect stderr

When creating a process with stdfork the stderr is not passed through.
This feature needs to be implemented in stdFork.ts and stdForkStart.js
Similar to what we already do for stdout, minus the waiting on the 'ready' signal
We also need to the cleanup similar to stdout

This is not a feature request but a serious bug: stdfork does respect stderr by sending it to stdout! In case of debug adapters this will corrupt the wire protocol and make VS Code abort the debug session.
Both php-debug and go-debug were hit by this:
https://github.com/felixfbecker/vscode-php-debug/issues/19
https://github.com/Microsoft/vscode-go/issues/206
https://github.com/Microsoft/vscode-go/issues/223
We were not seeing this problem before because VS Code was swallowing json parse exceptions silently and tried to continue running. However both debuggers had reports of mysterious session errors that we can now explain with this error.
We have two options for fixing this:

introduce a separate named pipe for stderr (real fix, higher risk)
disabled stderr output (low risk)

Since 'stderr' output is not really supported by VS Code and only shows up in the dev tools console, I think we should implement the low risk option for GA.

@weinand I think the issue you linked was not caused by this, but I got several reports at my Gitter channel:

I fixed them by eliminating every single occasion where logs could get output to stderr or stdout (some of which I wasn't even aware of, it was a dependency that logged stuff without my knowledge). But I constantly wondered why these issues only appeared since the Feb release. If VS Code swallowed JSON parse errors before, that is an explanation.


DatePicker: MoveToDate Feature

Why We Need This
We just fixed DatePicker date when dropdown opens in last release. But we have still issues in static picker etc. And i also think we can not cover all possible situations.
So the best way is adding a basic, simple method in API: MoveToDate
With this basic version, just called and date scrolls and shows the DatePicker's date (if it not null)
With the overrided version, user can set a date whatever they want and go to this date with or without submitting.
Usage Examples

Can add Today button in picker actions easily,
Navigate MinDate or MaxDate with one click.
Can use for possible situations when date changed but the visual did not. So solves all possible visual bugs. (Its not about StateHasChanged because visual is controlled by PickerMonth property)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

https://user-images.githubusercontent.com/78308169/159156612-3ed6899b-1148-4644-be3a-d687b7c0bff2.mp4
Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

GoToDate would be a better name

Does it select the day? If I read the code correctly it does not select the given day, it just goes to the month that contains that day and maybe keeps the previously selected day selected. Right?

Does it select the day? If I read the code correctly it does not select the given day, it just goes to the month that contains that day and maybe keeps the previously selected day selected. Right?

User have option there, by default submit is true, but user can set it false to not selecting date but navigating it.
And for basic version of MoveToDate, it already navigates to selected date, no need to change or submit.

GoToDate would be a better name

Im bad at naming 🙂

Thanks!


[release/2.8] Add code to handle pagination of parts. Fixes max layer size of 10GB bug

backport of https://github.com/distribution/distribution/pull/2815

@DavidSpek can you please make the CI pass?

@DavidSpek this should fix the CI: https://github.com/distribution/distribution/pull/3843/files

Thanks for the reviews. I'm traveling at the moment but I'll try and fix this PR later today or tomorrow.

@flavianmissi Should be good for reviewing now

Codecov Report
Patch and project coverage have no change.

Comparison is base (1b5f094) 58.72% compared to head (22a8050) 58.72%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##           release/2.8    #3893   +/-   ##
============================================
  Coverage        58.72%   58.72%
============================================
  Files              102      102
  Lines             7104     7104
============================================
  Hits              4172     4172
  Misses            2286     2286
  Partials           646      646

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

PTAL @thaJeztah @corhere @squizzi


Don't make messages loop outside back in

I noticed recently that messages being sent from verified domain to another inside the same postal instance will result in the message being sent outside to the internet and then back to the same server. I believe that it would be beneficial for postal to automatically determine if the address is handled by itself and simply send it internally without going to the MX records and then sending to the external IP.
So for example if I have a domain called example1.com and a domain called example2.com both verified and they have email routes<EMAIL_ADDRESS>and<EMAIL_ADDRESS>postal should automatically see when<EMAIL_ADDRESS>is sending to<EMAIL_ADDRESS>and route it internally. And then if<EMAIL_ADDRESS>tried to send to<EMAIL_ADDRESS>it would do the mx record lookup because it doesn't have a route specified to that address internally.

Maybe still check the MX records but if the MX record is the postal server ignore the IP address part and just do it internally? That way if they do change the MX record it can work as its supposed to but it doesn't do this weird loop thing?


View is translated to Ansehen not Ansicht

Description / Beschreibung
Pretty much what the title says. It is visible on the project page.

very nice bug!
Comes straight from the Django source:

https://github.com/search?q=repo%3Adjango%2Fdjango ansehen&type=code
https://github.com/django/django/blob/86561844ce66cda3e6a8c22d4ace4c2d1bc1f2e7/django/contrib/admin/locale/de/LC_MESSAGES/django.po#L359

https://stackoverflow.com/questions/7878028/override-default-django-translations


[OTHER] Add table of content in the README.md file

Title
Add table of contents in the README.md file
Description
Currently there is no table of contents section in the README.md file of this repo. Adding a table of contents section will improve the readability.
Screenshots (optional)
Attach screenshots here if applicable

kindly assign this issue to me

Hi @Petsamuel , I'm assigning this issue to you. Kindly raise a PR when you are done 👍🏻

Closing this issue as it is fixed in #15


KiCad freezes when pressing plugin icon

Describe the bug
When I press the JLCPCB tools icon in the PCB editor, KiCad freezes.
To Reproduce
I tried it with various projects, same result overall. I also tried the minimal case:
(0. Install plugin via plugin manager)

create new project
Add one resistor 0402 1k, connect both terminals
In PCB editor, import changes, draw one wire connecting both resistor pads
press JLCPCB icon

Expected behavior
Shouln't crash... should open plugin GUI
KiCad Version
Application: KiCad PCB Editor (64-bit)
Version: (6.0.1), release build
Libraries:
    wxWidgets 3.1.5
    libcurl/7.78.0-DEV Schannel zlib/1.2.11
Platform: Windows 10 (build 19043), 64-bit edition, 64 bit, Little endian, wxMSW
Build Info:
    Date: Jan 15 2022 13:36:27
    wxWidgets: 3.1.5 (wchar_t,wx containers)
    Boost: 1.76.0
    OCC: 7.5.0
    Curl: 7.78.0-DEV
    ngspice: 35
    Compiler: Visual C++ 1929 without C++ ABI
Build settings:
    KICAD_USE_OCC=ON
    KICAD_SPICE=ON

Please repeat what you described above, but before you press the JLCPCB Icon, open the python console.
That will spawn a stderr/stdout windows which gives a stacktrace that shows what error causes your issue.

I have the same issue. Just installed the plugin, I clicked the icon, Kicad crashes.

Hello. Same problem for me, nothing relevant I think in stderr, see attachment.
SO Windows 11, Kicad 6.0.1

e

I just did the exact same thing, deleted my plugin, installed it via PCM store.
Created a new project added two 0402 resistors with value 10k.

Created an outline connected te pads on the board:

Clicked the JLCPCB Icon and it worked without any error at all 😕

And I run the exact same version:
Application: KiCad PCB Editor (64-bit)

Version: (6.0.1), release build

Libraries:
	wxWidgets 3.1.5
	libcurl/7.78.0-DEV Schannel zlib/1.2.11

Platform: Windows 10 (build 19044), 64-bit edition, 64 bit, Little endian, wxMSW

Build Info:
	Date: Jan 15 2022 13:36:27
	wxWidgets: 3.1.5 (wchar_t,wx containers)
	Boost: 1.76.0
	OCC: 7.5.0
	Curl: 7.78.0-DEV
	ngspice: 35
	Compiler: Visual C++ 1929 without C++ ABI

Build settings:
	KICAD_USE_OCC=ON
	KICAD_SPICE=ON

Maybe some kind of permissen problem 🤔

Can one of you install an older version to see if its a problem with this specific version?

Uninstalled 2022.02.03 and installed 2022.01.06. Looks like it works now. Kicad no longer hangs, GUI window opens.

Please repeat what you described above, but before you press the JLCPCB Icon, open the python console. That will spawn a stderr/stdout windows which gives a stacktrace that shows what error causes your issue.

I just did that - no problem, GUI opens up. Tried also without opening the console, same result. Different project - same result, works!

@Martinnnnnnnnnnn you are still on 2022.02.03 and its working for you now, right?
@wowtah would you mind testing 2022.02.03 again and see if it works for you as well?

2022.02.01 works for me, but 2022.02.03 and 2022.02.02 hangs Kicad

I tested if it has to do with missing parts db and/or project db but I wasn't able to reproduce the error 😕

I've been having the same problem in 6.0.1 on Windows 11. I've had to go back to 2022.01.06 for it to work. It seems to be newer versions than that freeze KiCad. Or at least do for me.

@oofus Do you get no error at all even if you start the console before clicking the JLCPCB icon?

I've just pushed a small change to main. I would highly appreciate if one or more of you guys could check if that solves the issue!

@Bouni I get no error when the console is closed.
When the console is open the stdout/stderr window appears and I see similar output to what has already been posted here by others, ie the last output is about components already existing and will be ignored.

@Bouni I've just downloaded latest from Github, having removed the version from the Plugin and Content Manager.
You're fix seems successful so far. I'm going to open and close everything a few times to see if it's consistent, as you think it may be a race condition.
Can I suggest that you put the plugin version in the plugin window (an about button?). I'm 99% sure I'm running what I downloaded from Github but it would be good to know 100%.

@oofus I think for quite some time how to put the version into the header. See #15
I did not yet come up with a solution 😕

I've just pushed a small change to main. I would highly appreciate if one or more of you guys could check if that solves the issue!
Tried the plugin today (install via plugin manager): crash. Tried again, crash again.
Replaced mainwindow.py in some obscure user directory: crash again.
Opened console: crash again, no error shown in console. Just two messages DEBUG - update from board - Part R8 does not exist... and another for Q4. Nothing else.
Tried again with my test project (one resistor) Works. Debug message 2022.02.09 19:26:38 - DEBUG - download -  Download JLCPCB SMT Parts Library(20220208).csv with a size of 128.86MB
Going back to project that crashed before: Works! Messages
2022.02.09 19:28:19 - DEBUG - update_from_board -  Part Q4 does not exist in the database and will be created from the board.
2022.02.09 19:28:19 - DEBUG - update_from_board -  Part Q4 does not exist in the database and will be created from the board.
2022.02.09 19:28:19 - DEBUG - update_from_board -  Part Q3 does not exist in the database and will be created from the board.
2022.02.09 19:28:19 - DEBUG - update_from_board -  Part Q3 does not exist in the database and will be created from the board.
and so forth. Strange, R8 not shown again, every part twice.
If I had to guess my feeling would be it is something with loading the JLC database. Also the pattern is: After starting up Windows, it often fails. After it worked once, it works always.

Tried the plugin today (install via plugin manager)

You need to install it manually as the PCM only shows released versions, not the working branch (main)

Tried the plugin today (install via plugin manager)

You need to install it manually as the PCM only shows released versions, not the working branch (main)

Yes, I realized that. I copied in the modified mainwindow.py manually.

does it change the behavior?

As I wrote above: no, crashes either way: Original installed via plugin manager or with modified mainwindow.py.
Works a few tries later, reason unknown.


Build fails with GCC 14.1

Bok dečki,
I've been using the library with GCC 13.2 but after updating to 14.1 this error appears:
/root/prj/x86_build/_deps/async-mqtt5-src/include/async_mqtt5/detail/control_packet.hpp: In member function 'void async_mqtt5::packet_id_allocator::free(uint16_t)':
/root/prj/x86_build/_deps/async-mqtt5-src/include/async_mqtt5/detail/control_packet.hpp:143:32: error: 'upper_bound' is not a member of 'std'; did you mean 'lower_bound'?

  143 |                 auto it = std::upper_bound(
      |                                ^~~~~~~~~~~
      |                                lower_bound

Adding #include <algorithm> (which defines std::upper_bound) in include/async_mqtt5/detail/control_packet.hpp fixed the build for me.

Hello!
Thank you for reporting this issue!
I pushed the commit to fix the compilation error you described.


[QUESTION] How to set project name

It is possible to get project information
ConnectionHiddenCheckClient client = calcFactory.Create();
client.OpenProject(ideaProjectFileName);
projInfo = client.GetProjectInfo();

could you please add something like
client.SetProjectInfo(projInfo);

so it is also possible to update the project data?

HI @ZTThomasOberthanner, it is not currently possible.  I have changed to enhancement.


[BREAKING] Global app removal

This PR removes the inclusion of the global app reference and uses the static property on app base to store all application instance references including the active application.
API changes

Removes pc.app

This will likely to break a lot of people's projects.

Oh most definitely it will... counting the number of client and projects of ours we may need to check/update.

So we need to update 16 active projects of ours after this.

What do we gain after this?
Is it really necessary?
Is it possible to make this change but still people allow to use pc.app?

The change makes sense if you want to support multiple playcanvas instances on the same page. I would suggest to keep this backwards compatible by just always making pc.app point to the last created instance. That way it won't break all these projects that only use one instance.

There are so many references to pc.app in the forums, older tutorials and most likely in many user projects 😇. While it's possible to apply a patch just after the engine has loaded for projects that use pc.app, do consider @erikdubbelboer 's suggestion.

This will likely to break a lot of people's projects.

Yes but this will be included in a non forced update along with other breaking changes

So we need to update 16 active projects of ours after this.

What do we gain after this?
Is it really necessary?
Is it possible to make this change but still people allow to use pc.app?

Removes ambiguity when dealing with multiple applications. Having a global pc.app does not indicate which of these applications it refers to. Again this change will be included in a non-forced engine editor update which you will have to upgrade to by choice

Is non-forced updates a new thing that the editor is going to support? Cause right now we're always forced into all updates.

Currently, it is already possible to create multi-app apps, and devs wbo do that will have references to apps stored and used as required.
So I would say removing pc.app would not affect them at all, but will definitely affect everyone else, like 98% of single-app apps.
Convenience when debugging from console - is a massive.
So if the only reason of removing pc.app is to reduce confusion for multi-apps, then I would suggest to improve docs for multi-app cases, and not remove pc.app.
Is there any other reason for its removal?

I also raise concern on this breaking change. Unless it is needed for another feature, we really shouldn't break this for the reasons above

i see where you're coming from, importing app is very dangerous and error prone.  Might I suggest an assertion error/warning + deprecation when accessing pc.app when multiple apps are present.  You could always deprecate it then and rename it to make it clear what its for e.g. pc.currentApp and throw an error explaing what you need to do when accessing pc.app for old tutorials etc

I am probably an odd one, but I've never used pc.app. Was it an old way of getting an application instance? Anyhow, I am curious about "other breaking changes" that @kpal81xd mentioned.

@LeXXik
https://github.com/playcanvas/engine/pull/6407
https://github.com/playcanvas/engine/pull/6584
And actually thinking about it, I think what @kpal81xd means by non forces update is that the next minor/major release is the build with the WebGL 1 support removed
https://forum.playcanvas.com/t/phase-out-of-the-webgl1-support/35705
So the current version of the engine (1.71) will always be supported in the engine going forward

Currently, it is already possible to create multi-app apps, and devs who do that will have references to apps stored and used as required. So I would say removing pc.app would not affect them at all, but will definitely affect everyone else, like 98% of single-app apps.
Convenience when debugging from console - is a massive.
So if the only reason of removing pc.app is to reduce confusion for multi-apps, then I would suggest to improve docs for multi-app cases, and not remove pc.app.
Is there any other reason for its removal?

pc.app should not be relied on for access to the application instance it should be on the user to correctly manage their application reference as opposed to relying on a global definition of the application. For debugging yes it is useful that is the only reason I would consider leaving it in.
As for affecting other applications this change will be alongside other breaking changes in a non-forced update.

I'm not sure I see what this solves. We used to have a private pc.app that would return a last created app.
Here we have a public AppBase.getApplication(id) which returns the same value when no ID is passed in.
So it only changes the interface, but not solve the actual problem of having a global access to the last created app.
I think the goal here should be to remove any way for the user to obtain global/static reference to the current application, as that in any form causes the problem for multi-app users. Where the app is required by the engine (for example the Entity constructor, but also Asset I think and others, the app instance should be provided by the user, instead of the engine using last created application. But that is a lot larger breaking change unfortunately.

If the aim is discourage use of this common private API and not blocking a feature, please consider deprecating it with advice what to use instead. It will be much less pressure on user/customer support and less likely to negatively affect developer sentiment, especially those with long running projects.

non-forced update

Can you please provide a definition of it? There were no such updates before, so is this different version that users can migrate to, is it a separate version that users use one or the other, will it be forced after some time, what is it?
Many people and projects rely on pc.app, and it is up to docs to provide meaningful info on when it is not recommended.
The reason people use it - is because it is useful and convenient. So there should be a strong reason for its removal, otherwise, it just makes devs lives less convenient and introduces change for no technical reason.

With regards to the forced update I had assumed the community already knew about the split between engine v1 and v2. We will properly outline what this is and how it will work in a later post.
As for pc.app taking @mvaligursky's point into account technically all global access to the current application should be avoided since for multi applications its reference is ambiguous. For single applications however I see the usefulness of having global access to the current application despite it not being the best practise to use.

With regards to the forced update I had assumed the community already knew about the split between engine v1 and v2. We will properly outline what this is and how it will work in a later post.

I read every PR here on git, and am very up-to-date with the engine's development, and this is somewhat surprising to hear. I don't think there was any discussion on such, nor blog post. The only mention was about the LTS-webgl1 version to be supported by the editor.

Closed due to unnecessary breakage of projects


Improve run instances UX: avoid removing data and add a clear popup.

Improves the experience of using the "Run Instances" dialog by not aggressively resizing the internal structures, I was bitten by this multiple times when testing things that require multiple instances with different parameters: If I have the dialog set up with 4 instances with different parameters but I want to run two instances temporarily for a different test, lowering the number deletes all the data in the rows that are gone, so when I go back to the old number I'd have to set it up again.
To make it a bit easier to remove things if, for example, the user creates a ton of different params and wants to go back, I've added a right click menu with the option to clear a single line in the tree or to clear everything, I think that should cover the more common use cases.
Here's a video of it in action:
https://github.com/user-attachments/assets/4eca03e2-6880-47ff-9ca3-48e07b63cedf

Had to add a new project metadata entry to keep the value, because before we were just relying on the size of the list's size for the instance count field, but since now we're keeping all the data it's not really possible to do this without storing the number somewhere.

😅 silly mistake, fixed and rebased.

I think it's an improvement, but I find it a bit weird that some information is kept but not visible to the user. And the overall UX of having to decrease the number in the SpinBox which then removes lines with important configuration was IMO always subpar.
How about instead doing something like this:

Remove the SpinBox, and add a "Add Instance" button that adds a new entry to the tree (could be similar to the "Add Element" buttons in the Inspector for array/dictionary editors.
Add a first column with a checkbox to enable the instance.

So you can have e.g. 4 instances configured, and just check the ones you want to be active. This also makes it easier to have various presets for different configurations and test them separately without having to rewrite things in a specific order.
My only minor concern is that there's already two "[ ] Enabled" columns and having a 3rd one might not make clear at glance what each checkbox affects, but IMO that would still be a net improvement compared to both the current status, and the implementation in this PR.

Instance counter makes it easier to tell how many instances will start.
Maybe we can keep the SpinBox, but set its max value to number of added instances. The "Add Instance" button could work.

Thanks!

Oh that took me by surprise, I was already working on the improved version but I'll take my time and do a new PR 😄


Is there a way to handle errors without building a stack trace?

Building an error stack is expensive. Is there a way to handle a 404 status without getting a full exception error stack?

We have error objects. There is no way around it. But as long as you don’t print the stack or access it, it won’t be rendered. What exactly do you need?

No problem. Happy coding!


Julia/tenable release

Initial release of tenable

Should be added here as well for an initial release: https://github.com/DataDog/integrations-core/blob/master/requirements-agent-release.txt


Regex on field keys in the SELECT clause

Pages to update: InfluxQL Reference, Data Exploration, Functions.
Syntax:
SELECT /<regular_expression>/ FROM_clause

SELECT <function>(<regular_expression>)

Holding off until I can get https://github.com/influxdata/docs.influxdata.com/pull/817 merged.


Default MG Sandboxes - lifecycle of subscriptions not clear

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform:  v1.6.3
azure provider: v3.77.0
module: 4.2.0
Description
Not sure if bug or need for guidance.
We set our default Management Group to Sandboxes.
Running without any further configuration, subscriptions added to Sandboxes MG are tried to be removed by CAF on terraform plan:

Steps to Reproduce

Set Default Management Group to Sandboxes
Add subscription to Management Group Sandboxes, do not configure anything else
Result: CAF tries to remove subscription on terraform plan
Expected: CAF ignores the subscription in Default MG Sandboxes

Additional context

There was an old issue that has been closed due to an update on subscription and MG handling, but it doesn't seem to be fixed:
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/issues/24
Any help on this would be highly appreciated.

Hi!
Have you tried setting strict mode for subscriptions to be false?
This should fix your issue, though you will either have to run terraform import or move the subs into another MG before the next plan/apply.

Hey @matt-FFFFFF, thank you for your fast response on this issue/question.
I guess you are refering to strict_subscription_association.
I will try with the switch, wanted to migrate to v5.0.x anyways and this item came up for migration pre-tasks.
Anyway, with this set to false we still will run into issues if new Subscriptions are onboarded and put into our Default-MG Sandboxes.
For me, the lifecycle is not clear, since we probably block our deployments (plan/apply) because of new Subscriptions within a Default-MG that sits below root-MG of the caf-module. We always would have to import new subscriptions to the state first, even if they are not yet in our scope of management (and therefore are moved to a secured default-mg).
How could we handle this differently without breaking our lifecycle.
Currently, I worked around this by setting subscription_id_overrides for sandboxes (our default-MG) like this:
locals {
  # Define Subscriptions to be placed into Management Groups
  subscriptions = {
    sandboxes = [
      # List subscriptions that need to go into sanboxes-MG by definition
    ]
  }
}

# read all subscription within our sandboxes-MG (Default-MG)
data "azurerm_management_group" "sandboxes" {
  name = "${var.root_id}-sandboxes"
}

#...

module "enterprise_scale" {
  source  = "Azure/caf-enterprise-scale/azurerm"
  version = "4.2.0"

  # ...
  subscription_id_overrides = {
     # Set subscription for Sandboxes
     sandboxes = distinct(concat(local.subscriptions.sandboxes, data.azurerm_management_group.sandboxes.subscription_ids))
  }
}

With this setup, new subscriptions within the sandbox-MG are more or less ignored, since we do define them via a DataSource. But the code is not really clean, since we reference MGs in the DataSource that are created by the CAF module...whereas we reference the DataSource-Output in the CAF-Module again.
I will discuss with our team if we can set the strict_subscription_association to false and live with the downside of the blocker when new subscriptions are added to Sandboxes-MG via Default-MG assignment.

Hi! Yes I was referring to that variable.
I don't agree that you will have the same issue. With strict association set to false, any new subscriptions that are added to sandboxes will not be removed by the module.
You will have a one time task to deal with the conversion to management group subscription association resources - as per the v5 release notes.


Fix excessive method showAccountEstimate and zero number

[x] I have read the CONTRIBUTING.md and make ci passes on my machine.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #110   +/-   ##
=======================================
  Coverage   19.04%   19.04%
=======================================
  Files           3        3
  Lines         147      147
=======================================
  Hits           28       28
  Misses        117      117
  Partials        2        2

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 0758cc5...ff6b5e2. Read the comment docs.


Add support for additional alpha when the term-window is not focused

Solves #295
This is a somewhat adapted version of suckless_alpha_focus for this build of st

This is nice, but it sort of conflicts with the keybinding to make a window more or less transparent. This PR makes it so if you remove some alpha from a window, as soon as you defocus it, it returns to the standard transparency for unfocused windows and will darken again if you refocus, which is sort of strange.
I think it would be better to keep a the alpha variable, but also have a variable that represents the additional transparency that is subtracted from the focused window alpha value.
That way, if I make a window less transparent, that special level of transparency will be retained when I change focus (although it might slightly increase by that rate of difference).

That is a really good point. I have completely forgotten about this keybinding since I recently switched to your st build (form my custom build) and haven't used this keybinding ever.
Yes the alpha variable is kept anyway. I'm not certain whether the additional variable should be substracted from the original alpha or multiplicated with the alpha value.
The first option does decrease the opacity by a fixed value and therefore the rate of difference does change.
The second option decreases the opacity more for higher alpha (opacity) values which is probably the desired behaviour?
I think it is up to you to decide which of the options is best for this st build.

I have tested both possible implementations and came to the conclusion that Luke's proposed solution is better and yields more intuitive results.


Connection weight different on network load

Why is this https://github.com/vkaracic/JARVIS/blob/master/static/js/initd3.js#L21-L22

Explanation https://github.com/cazala/synaptic/issues/98
Fixed with https://github.com/vkaracic/JARVIS/commit/b23981dc7dfcfc43d37c5690f84034653daec9f7


Add Oliveira scheme

YAML file: https://github.com/protesilaos/oliveira/blob/master/lb/oliveira.yml

Hi 👋.
Thanks for the request.
I'm taking a break from the computer today (I'm writing to you from Android), but I'll get on this tomorrow.
In the mean time, I highly encourage you to submit a pull request. All you have to do is add the scheme you linked to the db/schemes folder and you're good to go 👍

Hello !
@protesilaos You can submit a pull request. In case of trouble, I will be around. Feel free to summon me today ;)

Hello! Thank you both!
I think I prepared the PR the right way. Apologies for doing this on a Sunday. Did not mean to disturb you.
Also congratulations for re-launching this project. Wish you best of luck.

You're too polite -- it's no bother at all :smile:
Thank you very much for the pull request.


Creature Support for Phaser 3

Hello all,
Creature dev will like to integrate Phaser 3 as one of the target runtimes for the Creature Animation Tool ( https://creature.kestrelmoon.com/ )
There are 2 types of runtimes Creature supports:

Standard runtime ( either JSON or Binary Flatbuffers ), contains all features, including the latest SkinSwapping, Gap Step Compression, Item Switching etc.

CreaturePack ( fully binary ), super compact and very performant, designed for mobile and web delivery platforms. Ideally we can integrate the latest WebAssembly version: https://medium.com/@kestrelm/creaturepack-high-performance-2d-webgl-character-animation-with-webassembly-72c436bec86c
If not, the dev is happy to backport some of the features like Gap Step compression back into the CreaturePack JS version. Here are 200 dinosaurs running in realtime with the latest WebAssembly accelerated CreaturePack runtimes:
https://creature.kestrelmoon.com/WebDemo/wasm/PixiJS-WASM-Pack-MultiDemo.html

The latest WebAssembly CreaturePack supports something called Delta Compression which allows for even greater levels of size compression with almost no quality or performance loss:
https://www.youtube.com/watch?v=vgHpt38q3oI
Integration into Phaser 3 does not require any compilation, the wasm binary along with the js loader just needs to be distributed with it.
So very complex animation with full on mesh deformations can be delivered for < 0.5 mb as seen in the video.
Both formats require the standard OpenGL/WebGL way of rendering meshes. That will mean a mesh has:

Points
Uvs + Colours ( Per Vertex )
Indices ( an array of ints defining the triangles of the mesh )

All of these attributes are time-varying in nature. That's really all that's required to get the Creature runtimes working with Phaser 3.
CreaturePack for example is very easy to use, here is a piece of sample code for running a single character in CreaturePack in PixiJS:
https://github.com/kestrelm/Creature_WebGL/blob/master/PixiJSRef-Pack-Demo.html

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Hello all,
I managed to implement preliminary Phaser 3 support for the latest CreaturePack runtimes:
https://github.com/kestrelm/Creature_WebGL/tree/master/Phaser3
Preview video:
https://twitter.com/KestrelmMoon/status/1112949221047459840
Cheers

@kestrelm going to close this issue off, as you've already released a great Phaser 3 plugin which Creature users can download and use for themselves :)


Migrate to Swift 5.0

Migrate project to Swift 5.0

@AlbertArredondoAlfaro Can you check why your PR fails?

@paweljankowski I have reverted my last commit. The issue is the update of the Xcode version on Travis. I thought it makes sense to apply this change, but honestly I don't know why then the CI fails to run.

@RajatJain4061 Can you merge this ? We really need Swift 5 support :-)

@jpotts18  Can you merge this PR ? Swift 5 support is really needed :-)

@AlbertArredondoAlfaro I'm using right now your master branch with Carthage but it's return
Task failed with exit code 65:
/usr/bin/xcrun xcodebuild -project .../Carthage/Checkouts/SwiftValidator/Validator.xcodeproj -scheme SwiftValidator -configuration Release -derivedDataPath .../Library/Caches/org.carthage.CarthageKit/DerivedData/10.2.1_10E1001/SwiftValidator/5.0.0 -sdk iphoneos ONLY_ACTIVE_ARCH=NO CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive -archivePath /var/folders/xv/7py__ckn12sbt71zvdbpw44m0000gn/T/SwiftValidator SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO STRIP_INSTALLED_PRODUCT=NO (launched in /...l/Carthage/Checkouts/SwiftValidator)

This usually indicates that project itself failed to compile. Please check the xcodebuild log for more details: /var/folders/xv/7py__ckn12sbt71zvdbpw44m0000gn/T/carthage-xcodebuild.MaJfz1.log```

@andresilvagomez You should use the "Swift-5.0" branch.

@AlbertArredondoAlfaro where is "Swift-5.0" branch? Carthage can't update SwiftValidator with 5.0 syntax from master branch :(

@AlbertArredondoAlfaro where is "Swift-5.0" branch? Carthage can't update SwiftValidator with 5.0 syntax from master branch :(

I found this - https://github.com/AlbertArredondoAlfaro/SwiftValidator/tree/swift-5.0 ;)

Hi @shift00, you are right, you should use the  https://github.com/AlbertArredondoAlfaro/SwiftValidator/tree/swift-5.0 branch.


Clarify Presentation Submission

Clarify that Presentation Submission is not mandatory for any target protocol

I don't understand how a protocol would know which items are being submitted against which criteria if they don't do this. If this is made optional, what will the instructions be in the spec? Is this what we really intend to say: "Presentation Submission objects are optional, and if not present, just kind of figure out what things they threw at you somehow, k thx bye!" ?

I don't understand how a protocol would know which items are being submitted against which criteria if they don't do this. If this is made optional, what will the instructions be in the spec? Is this what we really intend to say: "Presentation Submission objects are optional, and if not present, just kind of figure out what things they threw at you somehow, k thx bye!" ?

@csuwildcat Are you saying that a wallet that potentially has orders of magnitude more credentials than are in the set sent to the RP, and that does not have access to a presentation submission, and that normally has less processing power than the RP, is somehow able to determine which of these credentials match the presentation definition, but the RP is not able to do the same thing? If you expect the wallet to "just kind of figure it out", then why cannot you expect the RP to do the same thing?

In principle I agree with @David-Chadwick's opinion. It does raise a question though.
How would it work with a definition and VP that might be left open to interpretation because the definition might not be very well defined? On the wallet side a human could have been involved in the actual selection/mapping, which is not available on the verifier side. So the human has a direct impact on the submission data and thus the JSON path relationship between the definition and the VP path.
Let's say I have a VP with 3 VCs and let's say these have different purposes, but might have a field like for example bank account number that differs. The definition next to other fields, is looking for that bank account number, but now basically has 2 values. With submission data, the person associated with the wallet, could have said it was the bank account nr from VC 1. The verifier might be unable to pick the correct VC for it (really depends on the definition of course).
Of course it could be left up to the verifier to return an error if the result would be ambiguous

@nklomp I dont think your argument holds much water. Could you please give a specific example of where you scenario could exist rather than a vague suggestion. For example, if the RP required 3 identical VC types for different purposes, then it is likely to send 3 requests, not one. A real life example is getting an e-boarding card from Easy Jet on my smart phone. I can get boarding cards for my whole family on my phone. But I have to provide passenger details first (passport number etc.). Easy Jet asks me to fill in each passenger details separately. It does not send one request and expect me to return n different passport details. This real life use case can easily be migrated to a VC system by returning one passport VC for each passenger request. No presentation submission is needed. In fact, if the majority of RP requests only require one VC in response (which I suspect they will, such as age verification), then no presentation submission is needed. We should not complicate the majority of uses for the outlying use cases.

My point was about the 'asymmetry' between the holder where a person can actually associate data with the submission data, which a verifier cannot do.
Examples are basically everything where there can be 2 or more of similar data that do really belong to me.
Example: Phone numbers or e-mail addresses. You want my private and work phone numbers and/or e-mail addresses. The definition could be looking for my private e-mail address and work address, both could have similar constraints. There are many examples/forms that actually want that type of info from people. I might have multiple VCs that actually contain that information. As a holder I can associated the correct addresses with the correct definition fields. Something the verifier cannot do without the definition at hand.
Similar examples for PII of children etc.
Obviously you could create multiple interactions or restrict the definitions so that it might not happen, but reality is that there are many of these situations, so not sure I agree with your assertions. I believe at the minimum it should be pointed out that omitting submission data, means taking extra care to create definitions that do not leave room for ambiguity.

There's no ambiguity if you check for the right credentialSchema or type member, which PE allows for. I agree with @nklomp that omitting Presentation Submission objects is an unnecessary footgun that only serves to reduce interop, fragment the ecosystem, and introduce more needless failures.

@nklomp Good examples. Thankyou. These do not indicate where presentation submissions are always absolutely necessary (for example, the VCs could be of different types, or have properties of different types: home phone number, work phone number etc. allowing the RP to determine which VC matches which of its requirement), but they are examples of where a PS would always ensure that no ambiguity exists in the returned VP.
I suggest that the solution to this is that the transfer protocol has a field (PS required) that can be set to true or false by the RP, to indicate where a PS is required to help it determine which credential matches which requirement.

PR#309 proposes a resolution of this issue by including a field in the presentation definition that allows the RP to indicate whether it requires the presentation submission or not

@David-Chadwick is this stale given what happened with 309, or is there a new PR coming?

@David-Chadwick @dwaite do you have further input here?
we will close this item in the PE meeting May 19th

Closing PR per decisions of the WG surrounding this topic.


Build EBPF module for opensnitch, at present selecting the ebpf monitor method fails.

Describe the bug
The opensnitch nixos module allows the selection of the process monitor method, and currently the most preferred option ebpf is listed as valid. When configured via the module option, the service fails back to the "proc" process monitor method, since it cannot find the opensnitch.o ebpf module which is required for it to funciton.
Steps To Reproduce
Steps to reproduce the behavior:

set services.opensnitch.settings.ProcMonitorMethod = "ebpf";
check the journal logs for the message "open /etc/opensnitchd/opensnitch.o: no such file or directory"

Expected behavior
I would expect that if the "ebpf" ProcMonitorMethod is utilized as documented, that the module would be installed.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
I have taken a stab at building the module based on the opensnitch source and my current kernel source. The module builds successfully and properly identifies kworker threads which the other methods cannot accomplish. For example, kernel wireguard is silently dropped by opensnitch unless ebpf is used. Below is what I have successfully building:
{ lib
, pkgs
, stdenv
, fetchFromGitHub
, pkg-config
, clang-tools
, llvmPackages
, libllvm
, elfutils
, flex
, bison
, libressl
, bc
, rsync
, python3
, which
, zlib
}:

stdenv.mkDerivation rec {
  pname = "opensnitch_ebpf";
  version = "1.5.2";

  srcs = [
    (pkgs.opensnitch.src.overrideAttrs (old: { name = "opensnitch-src"; }))
    (pkgs.linuxPackages_zen.kernel.src.overrideAttrs (old: { name = "kernel-src"; }))
  ];

  sourceRoot = ".";

  nativeBuildInputs = [
    pkg-config
    clang-tools
    llvmPackages.clang
    libllvm # llvm-strip
    elfutils
    flex
    bison
    libressl
    bc
    rsync
    python3
    which
    zlib
  ];

  #https://aur.archlinux.org/cgit/aur.git/?h=opensnitch-ebpf-module-git
  #docs in the repo for patching/building aren't 100 percent up to date

  patchPhase = ''
    runHook prePatch
    substituteInPlace kernel-src/tools/lib/bpf/bpf_helpers.h \
      --replace 'struct bpf_map_def' 'struct bpf_map_defold'
    substituteInPlace opensnitch-src/ebpf_prog/Makefile \
      --replace '/bin/rm' 'rm'
      runHook postPatch
  '';

  configurePhase = ''
    runHook preConfigure
    ebpf="opensnitch-src/ebpf_prog"
    cp $ebpf/opensnitch*.c $ebpf/common*.h $ebpf/Makefile kernel-src/samples/bpf
    patchShebangs .
    cd kernel-src
    ( set +o pipefail; yes "" | make oldconfig )
    make prepare
    runHook postConfigure
  '';

  buildPhase = ''
    runHook preBuild
    make headers_install
    cd samples/bpf && make
    objdump -h opensnitch*.o
    llvm-strip -g opensnitch*.o
    runHook postBuild
  '';

  installPhase = ''
    runHook preInstall
    mkdir -p $out/etc/opensnitchd
    install -Dm644 opensnitch.o $out/etc/opensnitchd/
    runHook postInstall
  '';
}

Notify maintainers
@onny @raboof @zowoq @pennae
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.0.10-zen2, NixOS, 22.11 (Raccoon), 22.11.20230417.cd749f5`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`

Oh nice that you've managed to package the kernel module. Would you like to open a pull request for it?

Oh nice that you've managed to package the kernel module. Would you like to open a pull request for it?

@onny It isn't 100 percent ready to go, there are a few issues.

Should this even be a separate package? It seems to me that it could either be its own package, which could be conditionally included in the opensnitch nixos module if the ebpf method is chosen. The other way could be to integrate this code into the opensnitch package itself.
The package would ideally be compiled utilizing the specific sources of the users nixos configuration. Is it possible to add some code to the opensnitch nixos module which could automatically use whatever kernel src the user has configured?
The resulting opensnitch.o file has to be installed into /etc/opensnitchd, which would need to be done in the nixos module using environment.etc

Let me know what you all think about these thoughts. I am fairly new to packaging with nix, and feel a little over my head with how to implement this, so at this time I don't have a PR with a concrete suggestion of what to do.

@Slime90 thank you for your thoughts about this.
I would suggest packaging at in a new file inside the opensnitch directory, for example pkgs/tools/networking/opensnitch/module.nix.
Really don't know yet how these kernel modules are packaged in Nix but it would be nice if it would choose the correct kernel headers and sources.
We could install this module by default or only if its configured in the module. This might also depend on how other packages are doing this.
Maybe we don't have to place it in /etc and could configure OpenSnitch to look at an other directory for the module.

Offhand, the location in the opensnitch project dir structure makes perfect sense.
Looking around, I see I think a good representative example of a third party kernel module, and its inclusion in a nixpkg project. Take a look at:
nixos/modules/programs/droidcam.nix
pkgs/os-specific/linux/v4l2loopback/default.nix
Finally you will be able to see that kernel modules get added to: pkgs/top-level/linux-kernels.nix instead of pkgs/top-level/all-packages.nix
Of special note is the use of kernel.moduleBuildDependencies as a native build dependency.
My confusion with all of this is that even though we are buliding an ebpf module, we are not loading it into the kernel config using boot.kernelModules , this ebpf module is called by the opensnitch application at run time. Maybe you can take a look at some of the stuff above and let me know.

The below now builds with nix build -L .#linuxPackages_zen.opensnitch-ebpf after being added to pkgs/top-level/linux-kernels.nix.
This should mean that the config option in the nixos module for opensnitch should in theory be able to install the package with the boot.extraModulePackages = config.boot.kernelPackages.opensnitch-ebpf; option set in the configuration, although I have not yet tried importing this local nixpkgs source into my flake to actually test it.
At this point I think the ebpf procmon option in the service module can conditionally set config.boot.extraModulePackages, and perhaps use environment.etc to link it to the correct path
{ lib
, kernel
, pkgs
, stdenv
, fetchFromGitHub
, pkg-config
, clang-tools
, llvmPackages
, libllvm
, elfutils
, flex
, bison
, libressl
, bc
, rsync
, python3
, which
, zlib
, ...
}:

stdenv.mkDerivation rec {
  pname = "opensnitch_ebpf";
  version = "1.5.2-${kernel.version}";

  srcs = [
    kernel.src
    (pkgs.opensnitch.src.overrideAttrs (old: { name = "opensnitch-source"; }))
  ];

  sourceRoot = ".";

  nativeBuildInputs = [
    pkg-config
    clang-tools
    llvmPackages.clang
    libllvm # llvm-strip
    elfutils
    flex
    bison
    libressl
    bc
    rsync
    python3
    which
    zlib
  ];

  #https://aur.archlinux.org/cgit/aur.git/?h=opensnitch-ebpf-module-git
  #docs in the repo for patching/building aren't 100 percent up to date

  patchPhase = ''
    runHook prePatch
    substituteInPlace source/tools/lib/bpf/bpf_helpers.h \
      --replace 'struct bpf_map_def' 'struct bpf_map_defold'
    substituteInPlace opensnitch-source/ebpf_prog/Makefile \
      --replace '/bin/rm' 'rm'
      runHook postPatch
  '';

  configurePhase = ''
    runHook preConfigure
    ebpf="opensnitch-source/ebpf_prog"
    cp $ebpf/opensnitch*.c $ebpf/common*.h $ebpf/Makefile source/samples/bpf
    patchShebangs .
    cd source
    ( set +o pipefail; yes "" | make oldconfig )
    make prepare
    runHook postConfigure
  '';

  buildPhase = ''
    runHook preBuild
    make headers_install
    cd samples/bpf && make
    objdump -h opensnitch*.o
    llvm-strip -g opensnitch*.o
    runHook postBuild
  '';

  installPhase = ''
    runHook preInstall
    mkdir -p $out/etc/opensnitchd
    install -Dm644 opensnitch.o $out/etc/opensnitchd/
    runHook postInstall
  '';
}

Thank you @Slime90, i started creating a PR with your changes. I'll add you as co-author but also feel free to open a new PR if you want https://github.com/NixOS/nixpkgs/pull/229627
Further I opened an upstream ticket asking for a configuration option for the kernel module path https://github.com/evilsocket/opensnitch/issues/928

Should a new issue be created for actually modifying the opensnitch nixos module to conditionally include the extra module package, as well as link to etc?
I was thinking we could try
environment.etc."opensnitchd/opensnitch.o".source = "${config.boot.kernelPackages.opensnitch-ebpf.outPath}/etc/opensnitchd/opensnitch.o"

Is that valid?


Affinity Loss usage

Hi @yoshitomo-matsubara
I want to do semantic segmentation using distillation using "AffinityLoss" as in the paper "Knowledge Adaptation for Efficient Semantic Segmentation".
Its been shown in the line
class AffinityLoss(nn.Module):
I tried to use example section mentioned in
python3 examples/semantic_segmentation.py --config configs/sample/coco2017/multi_stage/ktaad/lraspp_mobilenet_v3_large_from_deeplabv3_resnet50.yaml --log log/coco2017/ktaad/lraspp_mobilenet_v3_large_from_deeplabv3_resnet50.txt
But while debugging it will never excute AffinityLoss module.
requesting you to help us how to go ahed. Seeking your help

Hi @ChidanandKumarKS
How did you confirm that it will never execute AffinityLoss module? If you change any part of code or yaml file, please share them here.
Note that the yaml config is for a multi-stage training, and AffiniyLoss will be used only in the 2nd stage after the 1st stage training (1 epoch).

@yoshitomo-matsubara
Sorry the training was quite long time. As you mentioned it is 2-stage training, iam able to debug for the section of the code based on Affinity loss. Its working.
Thanks
Kumarr