How do I migrate a v6 Stack with 3 DBs to a v7 stack?

I tried, I failed.

Migration of the auth db in a v6 will not work as the flyway schema table has been renamed so it will never find the v6 one and thus will try to create tables afresh which will conflict with the old ones.  The plan is to change the migration to rename all the v6 table to old_auth_ (use old_auth_ rathet than old_ to avoid any clashes with stroom tbls) then create the new ones and copy data over as needed if old_tables exist.
If we want to bring the auth db into the stroom db then we will need a pre mig step to rename all the auth tables then exp/imp all the renamed tables into stroom.


crackmapexec with Empire + Deathstar

Hi Guys
Just wondering if anyone could lend a newbie a hand? I have installed Empire and Deathstar - Deathstar is listening fine through Empire.
I have tried Empires in built agent deployment / stagers but they don't seem to work.
I have read that I can use crackmapexec to stage an agent - does anyone have a spare moment to explain how this can be achieved?
Thanks
Phil

You can view the CrackMapExec modules currently avable using the --list-modules flag. One of them, you will notice, is empire_exec. You will need to modify the Empire REST API credentials in the cme.conf file, which can be located in a variety of places such as {install root}/cme/data/cme.conf. After creating a listener in Empire, you execute CME with the flag -M empire_exec and -o LISTENER=<listener name>, where  is the Name you set for the Empire listener.

If you have questions about CME i suggest joining the #crackmapexec slack channel on bloodhoundhq.slack.com and ask them there. Please only open an issue ticket to report bugs.


2cols pages

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--elixirsolutions--hlxsites.hlx.live/
After: https://2cols-pages--elixirsolutions--hlxsites.hlx.live/blog

closing this for now, will fix/re-open later


[Proposal] Enable Discussions

Discussion is currently disabled, are you interested in turning it on? @maronuu @kitagry
https://github.com/m3dev/kannon/discussions/landing
I've never managed a repository with it, so I don't know how good the experience would be, though...especially, I'm not sure whether issues is enough or not.

At least, we can subscribe it on slack with official integration. (some old articles say it's not yet supported)
https://github.com/orgs/community/discussions/2844#discussioncomment-2061814

we can use discussion now? https://github.com/m3dev/kannon/discussions
I don't know what is different between issue and discussion. But I agree to make discussion enabled.

I don't know well on the discussion feature, but it seems good.

thanks! let's try to use.


Assert failed when changing the IAS Zone cluster zone status (0x0002) attribute value (TZ-933)

Answers checklist.

[X] I have read the documentation ESP Zigbee SDK Programming Guide and tried the debugging tips, the issue is not addressed there.
[X] I have updated ESP Zigbee libs (esp-zboss-lib and esp-zigbee-lib) to the latest version, with corresponding IDF version, and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.2.1
esp-zigbee-lib version.
1.3.2
esp-zboss-lib version.
1.3.2
Espressif SoC revision.
ESP32-H2
What is the expected behavior?
When i want to change the IAS Zone cluster zone status (0x0002) attribute, my program crash on an assert in the zcl/zcl_ias_zone_commands.c file. The expected behavior is that the attribut value change without crashing the program.
What is the actual behavior?
When i try to set the IAS Zone Status attribute value (0x0002) with the following function:
esp_zb_zcl_status_t ret = esp_zb_zcl_set_attribute_val(1,
ESP_ZB_ZCL_CLUSTER_ID_IAS_ZONE,
ESP_ZB_ZCL_CLUSTER_SERVER_ROLE,
ESP_ZB_ZCL_ATTR_IAS_ZONE_ZONESTATUS_ID,
(void*)&status,
false);
the program crash with the following message:

I im interrested to know what cause the assert to failed and what is related to this assert. Is there specific steps required to change the zone status attribute value?
Steps to reproduce.

Create an attribtue list for the IAS Zone cluster :

Add Zone status attribute to the list:

Add the IAS Zone cluster to the cluster list:

Add clusters list to the endpoint list and register the device:

Start the Zigbee stack :

Try to change the IAS Zone - zone status attribute value :

More Information.
No response

@Vprime007,
The issue is triggered by a missing attribute. You can call esp_zb_ias_zone_cluster_create() to create the IAS zone cluster, which includes the mandatory attributes of IAS zone.

Thank you for the advice. My issue is fixed.


Failed to parse workflow: jsonL: cannot unmarshal array into GO struct field DAGTask.spec.templates.dag.tasks.arguments of type v1alpha1.Arguments

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[X] I'd like to contribute the fix myself (see contributing guide)

What happened/what did you expect to happen?
Failed to parse workflow: jsonL: cannot unmarshal array into GO struct field DAGTask.spec.templates.dag.tasks.arguments of type v1alpha1.Arguments
Version
argo:v3.0.0-rc6
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  name: test-dag3  # workflow的名字
  #namespace: 10006--67275600
  labels:
    workflows.argoproj.io/creator: system-serviceaccount-argo-argo-server
spec:
  entrypoint: main  # 定义main为主要templates
  arguments:
    parameters:
    - name: recevie_task # 定义recevie_task
      value: task_msg
  templates:
    - name: main
      dag:
        tasks:
          - name: yewuxuqiuzonghefenxi  #业务需求综合分析
            template: yewuxuqiuzonghefenxi
            arguments:
              parameters:
              - name: param
                value: "{{workflow.parameters.recevie_task}}"
          - name: mubiaoquyufenxi  #目标区域分析(依赖业务需求综合分析)
            depends: yewuxuqiuzonghefenxi
            template: echo
            arguments:
              parameters:
              - name: param
                value: "{{tasks.yewuxuqiuzonghefenxi.outputs.parameters.result}}"

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

Please check your template spec. Also argo:v3.0.0-rc6 is too old.


Add driver for MySQL

I've followed the how-to-add-driver doc to add a driver for MySQL to VerdictDB. Could you please have a check? Thanks.

Thanks for the contribution! But, we actually have added a MySQL driver, as found in our quick start guide: http://docs.verdictdb.org/getting_started/quickstart/

Thanks for the information! This pull request is actually a part of Prof. Mozafari's research screening tasks, where we need to write a driver for MySQL or PostgreSQL. I guess you already have both in the master branch. BTW, your quick start guide is pretty good!


Add arrow-*-[to/from]-line icons

Multiple use cases for these…

arrow-right-to-line is also the tab ⇥ key icon

arrow-right-from-line is also export from codicons

arrow-down-to-line is a common download icon (similar to https://lucide.dev/icons?search=download)

arrow-down-to-line and arrow-up-from-line are also used as git push/pull in GitKraken, for example:

Can you try to design a variant with the length of the lines a bit extended?
The arrow looks visually a bit bigger than the line. So maybe extending the line it looks more balanced.

Can you try to design a variant with the length of the lines a bit extended?

@ericfennis pushed…

@danielbayley: Can we add the use cases from #980 as tags?
Other than that LGTM!

@danielbayley: Can we add the use cases from #980 as tags?

@karsa-mistmere Done.


Use diff instead of cmp in test/InterfaceHash/edited_method_body.swift

We're seeing some flakiness in this test, but aren't sure what it's complaining about. cmp only says where the difference was, but doesn't print what was different. diff prints what was difference so we have a chance to figure out what happened.
This should be fine since the test is not run on Windows.

@swift-ci please smoke test


Support None to unlimit 'display.max_rows'

This PR adds the support to unlimit the number of rows at 'display.max_rows'.
>>> import databricks.koalas as ks
>>> ks.set_option("display.max_rows", None)
>>> ks.range(1001)
        id
0        0
1        1
...
1018  1018
1019  1019

[1020 rows x 1 columns]

Codecov Report

Merging #742 into master will decrease coverage by 2.35%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #742      +/-   ##
==========================================
- Coverage   94.03%   91.67%   -2.36%
==========================================
  Files          32       32
  Lines        5650     5659       +9
==========================================
- Hits         5313     5188     -125
- Misses        337      471     +134

Impacted Files
Coverage Δ

databricks/koalas/config.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/series.py
95.02% <100%> (-0.17%)
:arrow_down:

databricks/koalas/indexes.py
97.29% <100%> (+0.07%)
:arrow_up:

databricks/koalas/frame.py
94.53% <100%> (-1.7%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
77.5% <0%> (-7.5%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/namespace.py
79.66% <0%> (-1.34%)
:arrow_down:

... and 5 more

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 625cea6...f7d979c. Read the comment docs.


Add tashkeel text

@ahmedre this is probably one of the most requested features these days...

Merge once you give it a +1

sounds good, jazakAllah khair.


Update composer.json

The bundle also works with symfony 2.7

Hi and thanks for this PR.
I don't know if we want to have 2.7 support because this bundle will evolve and we don't want to manage BC for this new project before SF 2.8.

Thanks for the quick answer. I have project that still uses 2.7.
Since symfony 2.7 has LTS until 2018 you might consider supporting it: http://symfony.com/doc/current/contributing/community/releases.html#schedule

Since symfony 2.7 has LTS until 2018 you might consider supporting it:

Yeah and 2.3 is security supported until 2017. IMO, this is not an obligation to support older versions for new projects.
But, I will accept for 2.7 for now. If we have BC break changes later, we'll push a new major version.
Could you please:

Resolve conflict?
Add SF 2.7 version on .travis.yml?

Thanks.

Also can you please update README.md file?

One last thing once finished: Please squash your commits and rename the comment for a proper title : "Allow Symfony 2.7".

ping @mathielen. Could please rebase so I can push a new release?
Thanks! :+1:

ping @mathielen ? :-)

Closed in favor of #9.


Add option to disable running integration tests (#18454)

Add option to disable running integration tests

(cherry picked from commit 932865ceeef51af743fabf8b6524030972455b32)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 48.292%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
14821

Relevant Lines:
29135

💛  - Coveralls


0091 - Decode Ways

Description of the Problem
A message containing letters from A-Z is being encoded to numbers using the following mapping:
'A' -> 1
'B' -> 2
...
'Z' -> 26

Given a non-empty string containing only digits, determine the total number of ways to decode it.
The answer is guaranteed to fit in a 32-bit integer.
Code
class Solution:
    def numDecodings(self, s: str) -> int:

Link To The LeetCode Problem
LeetCode

Can you assign this to me.


Enhancement Request: Ability to use other MPI Launchers in tests

All,
This is just a Feature Request for changes to the parallel tests for netCDF. Namely, all the tests in netCDF seem to assume that mpiexec is available and will work. Sometimes, this isn't the case (well, it's usually there; it's the will work that has issues).
For example, with Intel MPI 5.1, mpiexec requires mpd to be booted, but mpiexec.hydra (symlinked to mpirun) is by far the preferred launcher for Intel MPI. If you don't go into the tests and change to mpirun, the tests all fail if mpdboot isn't on. (I think in Intel MPI 2017, they just removed the mpd version, so the test might work again!). Likewise, SGI MPT prefers mpiexec_mpt, Open MPI prefers mpirun, and MVAPICH2 prefers mpirun_rsh.
So, I was wondering if it'd be possible to have the mpirun command be something that, say, configure could pass in (or environment variable, or whatever cmake does). I suppose defaulting to mpiexec is a good default since, I think, the MPI Standard says it is recommended (though not required).
Obviously, if netCDF could detect the type of MPI stack, that'd be ideal, but just allowing one to set it would be fine.
Thanks,
Matt
PS: Note that "netCDF" here means netCDF-C and netCDF-Fortran. I just started here because this is the Big Cheese™. If an acceptable solution can be created for netCDF-C, I could then file an issue over at Unidata/netcdf-fortran

I have nothing to say against the proposed enhancement; the issue comes from the lack of in-house resources.  I can wire something like this in as an option, e.g. a way to set the environmental variable for the mpi executable, but we do not have a broad selection of parallel systems to choose from, so testing would be limited.
I will mark this as under consideration and will add it to the best of our ability here, and will document it as such; I will implement whatever solution is made here in netCDF-Fortran as well.  I'll also add that a pull request with this functionality would be quite welcome :).  Otherwise, we'll address it when we're able to.
Thanks for the background information, however; I was not aware of any of these issues, so it is great to know.

I agree that this would be a useful enhancement.
FWIW a workaround is to change the contents of nc_test4/run_par_test.sh.

I know I could hack a way in using pure Make (fun with $(shell)!) but it's autotools...it's always autotools... Maybe it's time to learn!

FWIW a workaround is to change the contents of nc_test4/run_par_test.sh.

Indeed, my usual method is to convert all the mpiexec to mpirun. The odd one comes with MPT. It's mpirun is an odd one to say the least. Luckily netcdf-c/fortran don't do anything fancy and/or interact with resource managers

I also ran into this need when installing netcdf on a supercomputer.
I have fixed this so that the --with-mpiexec= argument to configure allows the user to set the command that will be used to launch MPI programs. The default value remains mpiexec.
This is on WDC branch ejh_change_mpiexec. I will put up a PR presently.


chore: gas estimates in sign bytes

Background
Include gas estimate and fee structure in turnstone messages hash.
Testing completed

[ ] test coverage exists or has been added/updated
[ ] tested in a private testnet

Breaking changes

[ ] I have checked my code for breaking changes
[ ] If there are breaking changes, there is a supporting migration.

Great work, only some small remarks. There is a panic as well where there shouldn't be.

Regarding the panics, we already have other panics in the same functions, I just continued the same style. I think we can refactor them out and return an error instead. However, I don't see these functions being used in Begin or End blockers. Is this an issue or a false positive?

Great work, only some small remarks. There is a panic as well where there shouldn't be.

Regarding the panics, we already have other panics in the same functions, I just continued the same style. I think we can refactor them out and return an error instead. However, I don't see these functions being used in Begin or End blockers. Is this an issue or a false positive?

Unless I'm mistaken, the panic statements are in hot code paths. Did a short scan and found one in the keccak256 for submit logic call and valset update for example, which are currently both used to calculate the bytes to sign.
It's interesting the CodeQL plugin didn't find these yet, but we definitely want them out. In theory, failing to pack the arguments should never happen, but better be safe then risk an app hash.

Great work, only some small remarks. There is a panic as well where there shouldn't be.

Regarding the panics, we already have other panics in the same functions, I just continued the same style. I think we can refactor them out and return an error instead. However, I don't see these functions being used in Begin or End blockers. Is this an issue or a false positive?

Unless I'm mistaken, the panic statements are in hot code paths. Did a short scan and found one in the keccak256 for submit logic call and valset update for example, which are currently both used to calculate the bytes to sign.
It's interesting the CodeQL plugin didn't find these yet, but we definitely want them out. In theory, failing to pack the arguments should never happen, but better be safe then risk an app hash.

 I'll refactor that out and return an error instead.


Policies CRUD

Basic Policy [CRU]D support.

New routes for policy view/create/edit
Added new GraphQL Query for the list view to show all current internal policies
routing

/policies-and-procedures/policies/create
/policies-and-procedures/policies/01JEY6A2YGFKWVAXY76QGDSHX0 - view
/policies-and-procedures/policies/01JEY6A2YGFKWVAXY76QGDSHX0/edit

Looks like the new new input styling is effecting the login screen. I'll need to work on that separately.


Reducer should not throw MaterialstoreException when the store and store-base dire is not present

derived from https://github.com/kazurayam/inspectus4katalon-sample-project/issues/8
2022-12-12 16:53:47.870 ERROR c.k.katalon.core.main.TestCaseExecutor   - ❌ Test Cases/CURA/main FAILED.
Reason:
com.kazurayam.inspectus.core.InspectusException: com.kazurayam.materialstore.core.filesystem.MaterialstoreException: store.reflect(currentMaterialList) returned previousMaterialList of size == 0. currentMaterialList = CURA/20221212_165311
	at com.kazurayam.inspectus.core.internal.ChronosDiff.step3_reduceChronos(ChronosDiff.java:86)
	at com.kazurayam.inspectus.core.internal.ChronosDiff.process(ChronosDiff.java:28)
	at com.kazurayam.inspectus.core.Inspectus.execute(Inspectus.java:8)
	at com.kazurayam.inspectus.core.Inspectus$execute.call(Unknown Source)
	at main.run(main:38)
	at com.kms.katalon.core.main.ScriptEngine.run(ScriptEngine.java:194)
	at com.kms.katalon.core.main.ScriptEngine.runScriptAsRawText(ScriptEngine.java:119)
	at com.kms.katalon.core.main.TestCaseExecutor.runScript(TestCaseExecutor.java:445)
	at com.kms.katalon.core.main.TestCaseExecutor.doExecute(TestCaseExecutor.java:436)
	at com.kms.katalon.core.main.TestCaseExecutor.processExecutionPhase(TestCaseExecutor.java:415)
	at com.kms.katalon.core.main.TestCaseExecutor.accessMainPhase(TestCaseExecutor.java:407)
	at com.kms.katalon.core.main.TestCaseExecutor.execute(TestCaseExecutor.java:284)
	at com.kms.katalon.core.main.TestCaseMain.runTestCase(TestCaseMain.java:142)
	at com.kms.katalon.core.main.TestCaseMain.runTestCase(TestCaseMain.java:133)
	at com.kms.katalon.core.main.TestCaseMain$runTestCase$0.call(Unknown Source)
	at TempTestCase1670831587859.run(TempTestCase1670831587859.groovy:25)
Caused by: com.kazurayam.materialstore.core.filesystem.MaterialstoreException: store.reflect(currentMaterialList) returned previousMaterialList of size == 0. currentMaterialList = CURA/20221212_165311
	at com.kazurayam.materialstore.base.reduce.Reducer.chronos(Reducer.java:72)
	at com.kazurayam.materialstore.base.reduce.Reducer.chronos(Reducer.java:44)
	at com.kazurayam.materialstore.base.reduce.Reducer.chronos(Reducer.java:24)
	at com.kazurayam.inspectus.core.internal.ChronosDiff.step3_reduceChronos(ChronosDiff.java:68)
	... 15 more

2022-12-12 16:53:47.879 INFO  c.k.katalon.core.main.TestCaseExecutor   - END Test Cases/CURA/main

When the store and store-backup directory is not present and ChronosDiff ran,  an MaterialstoreException will be raised. Because of this code:
Reducer Line No.70
        if (previousMaterialList.size() == 0) {
            throw new MaterialstoreException(
                    String.format("store.reflect(currentMaterialList) returned previousMaterialList of size == 0. " +
                            "currentMaterialList = %s/%s", currentMaterialList.getJobName().toString(),
                            currentMaterialList.getJobTimestamp().toString()));
        }

Is this check necessary?
It is better not to perform this check = do not throw Exception, and the Reducer silently performs comparison a real material against a void materia, resulting 100% diff.

I found that there isn't a JUnit test for the com.kazurayam.materialstore.base.reduce.Reducer class. At first, I should add a test for the class.

Reducer#chronos() method implementation had a code:
        if (previousMaterialList.size() == 0) {
            throw new MaterialstoreException(
                    String.format("store.reflect(currentMaterialList) returned previousMaterialList of size == 0. " +
                            "currentMaterialList = %s/%s", currentMaterialList.getJobName().toString(),
                            currentMaterialList.getJobTimestamp().toString()));
        }

This was just unnecessary.
I removed it.


OpenWhisk Plugin Error using deprecated kong.tools.responses

The error module 'kong.tools.responses' not found occurs on initialization
(See trace attached)

It appears that the relevant line may be in kong-plugin-openwhisk/kong/plugins/openwhisk/handler.lua... with the line local responses     = require "kong.tools.responses".  I'm not a lua developer... so, not in the best position to make a fix and submit a PR...
This Changelog shows the below statement regarding plugins (which is an indication of the issue)
The kong.tools.responses module was removed. Please use kong.response.exit from the PDK instead. You might want to use kong.log.err to log internal server errors as well.

@treyau21 the fix has been merged in, but have yet to push out a new release (will go out shortly)


Unintended Full Recompilation When Integrating Cargo Chef into Graph Node Dockerfile

I'm trying to add cargo chef to this open source project. I've modified the Dockerfile like so. When I try to build this, I see a full recompilation happening during the main build command. I also see a warning emitted during the cooking phase
Step 18/53 : RUN RUSTFLAGS="-g" cargo chef cook --release --recipe-path recipe.json
 ---> Running in 7b1bb385cb88
warning: virtual workspace defaulting to `resolver = "1"` despite one or more workspace members being on edition 2021 which implies `resolver = "2"`
note: to keep the current resolver, specify `workspace.resolver = "1"` in the workspace root's manifest
note: to use the edition 2021 resolver, specify `workspace.resolver = "2"` in the workspace root's manifest
note: for more details see https://doc.rust-lang.org/cargo/reference/resolver.html#resolver-versions

Here's the recipe.json file that gets generated.
The docker build command I use is
docker build --target graph-node \
            --build-arg "COMMIT_SHA=$(git rev-parse --short HEAD)" \
            --build-arg "REPO_NAME=$(git remote get-url origin | cut -d ":" -f 2 | cut -d "."

I checked one of the *.d files under target/release/deps and I see the hardcoded paths starting with /graph-node which seems correct to me.

On the surface, it looks correct.
Do you have build logs to share?

(The resolver warning has nothing to do with cargo-chef, that's a cargo warning that you need to handle)

Yup, here are some build logs: https://gist.github.com/paymog/b71cc9d87add2f8087f97f70ad257f4c
FWIW I don't see that resolver warning when building without cargo-chef

Yup, here are some build logs: https://gist.github.com/paymog/b71cc9d87add2f8087f97f70ad257f4c
FWIW I don't see that resolver warning when building without cargo-chef

These are the build logs from a fresh run (i.e. this line would come from the cache otherwise). I need the build logs for a second run that has a warm cache to understand if things are working or not.

Whoops, sorry about this. These logs should have that stage cached.

From the logs I can infer that you have a rust-toolchain.toml file and that it points to an older version of Rust than the one bundled in cargo-chef's Docker image.
The best course of action, in that case, is to "sync" the Rust version ahead of time—see this as an example.

Hmm, I'm not using the cargo-chef docker image - I'm using the chef planner image as shown here

It's the same, you're getting the latest Rust toolchain but your toolchain file wants 1.68.

That seems to have fixed it, thank you for your help!


Visual Studio for Mac EOL

Visual Studio for Mac is now EOL^1, but the instructions in your README.md still call for it.^2

i am aware; however i currently don't have the motivation to essentially fully rewrite the project.
you can get the last VS Mac Version here: https://aka.ms/installer/preview

i should maybe pin this.


Update NuGet Packages

As far as I could tell, these changes don't cause any build issues. But, knowing my luck, I'm probably missing something. Lets just see where this lands.
This doesn't update all NuGet packages, just the ones that seemed like they were close in version and were unlikely to have breaking changes.
Microsoft Reviewers: Open in CodeFlow

Usually, we update package reference versions only when we need to (i.e. vulnerability detected, compliance issue, need new version for new feature, out of support, etc). And we "try" to make all projects point to same package version to minimize potential issues.
Since this one mostly only affects tests and projects that are not used in other places(except the wingetutilinterop which is used in winget service), I think it's probably ok if the wingetutilinterop change is reverted.

/azp run


docker: split images to idf-rust and idf-rust-examples

Closes https://github.com/ivmarkov/rust-esp32-std-hello/pull/8
The idea behind this PR is to have two images:

espressif/idf-rust - which contains Rust compiler and can be used in CI or base image for others
espressif/idf-rust-examples - images based on idf-rust which contains additional tools and examples

Right now we have two examples:

"idf.py-first" approach in /opt/rust-esp32-example
"cargo-first" approach which dependes on cargo-pio in /opt/rust-esp32-std-hello - repo: https://github.com/ivmarkov/rust-esp32-std-hello.git

@rgcottrell @JurajSadel @mabezdev @igrr @jessebraham @ivmarkov Please review the change.

I'm not really knowledgable in this stuff as I've yet to use these docker images... so take this with a grain of salt, but... my understanding was, that we need Docker images for the tooling only. Meaning: the Rust compiler, clang, Espressif toolchains, whatever else necessary for the "cmake-first" example, and whatever necessary for the "cargo-first" example.
Why are we pushing the example projects themselves inside the docker images?

Why are we pushing the example projects themselves inside the docker images?

The advantage here is that we are showing an example of how to build off of the tools-only docker image to add additional project-specific tooling and environmental setup that could serve as the basis of local developer builds or CI jobs. I agree, however, that checking out the project again might be a bit weird.
Another option would be to mount the root of the repo as the image's working directory. That would allow changes and build artefacts to be more easily shared between the host and the image.

Another option would be to mount the root of the repo as the image's working directory. That would allow changes and build artefacts to be more easily shared between the host and the image.

That would be my preference as well, as this makes moving on from "running the example" stage to the "developing your project" stage more natural. It might be as simple as having a list of steps on "how to build this example using Docker" which includes cloning or downloading this repo.

Great discussion. I see that more explanation should be included and that developer workflow might not be obvious.
Docker is really great for having a stable reference environment. Something like a "snapshot" of working state. Dockerfile provides a way how to describe the environment in repeatable fashion.
Some IDEs like Visual Studio Code provides even plugins which allows work in the container to look-like local work.
The major advantage for developer is quick and clean bootstrap. The developer can then discard the environment quickly and cleanly. It's even possible to make a snapshot of the environment and ship it to other machine using docker commit and docker save.
Bootstrapping process to have working environment with ESP-IDF + Rust is just one command:
docker run -it espressif/idf-rust-examples

E.g. I was trying to configure environment for std example for an hour on macOS and still something was wrong. I was not able to build a component, because something was not correct. With the Docker I was able to build the example within few minutes. Then it was necessary just to copy result from the container and flash it:
docker cp CONTAINER_ID:/opt/rust-esp32-std-hello/target/xtensa-esp32-espidf/release/rust-esp32-std-hello .


adding detection of update available and mode on pluging

Hi @stas-demydiuk ,
I made some first changes after some experiments with OTA to implement enhancement #282 . My idea is to have a mode (like the 'permit join') that allows zigbee2mqtt to run an update when a device reports that it has firmware updates available. This status is added to the normal status updates that are published to topic zigbee2mqtt\<friendly_name> in the payload as element update_available with values true and false.
So I added detection (now it only logs it) in the devices/device.py class but am not sure how to continue from there, as I cannot get the value of self.OTA_enabled from the BasePlugin class to the device base class. Hope you can provide some advice there.
I wanted to stay away from a design where the self.OTA_enabled would trigger a query on all devices whether they have an update available. The downside of the above is that it will take a state change on the device so that it reports its status so you are never sure you have had them all. On the other hand, querying devices may not work for battery powered devies that go to sleep.
I ran a small test on my network of 10 devices, updating 1 of them did not influence performance what so ever. But I have only 3 devices left that actually can update, so I'm saving them for the final test......

@JanJaapKo I would suggest to implement required feature via UI that plugin will provide soon. I've already added couple examples like renaming device and setting its state. See code in network-map branch

Hi @stas-demydiuk the custom pages look brilliant! Networkmap nicely displayed, nice table (hint: add manufacturer to model column). Would be great to add to the devices table a column with the update status with a button to trigger sending the actual update command. I'll see if I get rund to that (js ain't my best language though.....).

Hi @stas-demydiuk
I've got it working from the frontend, with the following remarks:

I haven't figured out how to hide the 'download' button for devices that do not have an update available
I want to have a report back to the modal form if the update started or not (would at least notify user of no update available or not supported) but not sure how to do this
I added also the 'vendor' to the table but it gives an error (for the coordinator I presume) but it did not do that for model info, which in my case is also missing for the coordinator. How did you work around  that?

Thanks, will include this change to #290


Add support for RFC 8360

Add module and test for RFC 8630

Codecov Report

Merging #72 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   99.31%   99.31%   +<.01%
==========================================
  Files          68       69       +1
  Lines        5400     5409       +9
==========================================
+ Hits         5363     5372       +9
  Misses         37       37

Impacted Files
Coverage Δ

pyasn1_modules/rfc8360.py
100% <100%> (ø)

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 c8f7dce...a667720. Read the comment docs.

Thank you!


fixed widgets are not shown with after_content

Hi All,
I had a problem when I was using Widgets.
(A) what you did
I added widgets to after_content section using Widgets API in my admin dashboard using blank.blade.php.
Like:
Widget::add($widget_definition_array)->to('after_content');
Widget::add($widget_definition_array)->section('after_content');
Widget::add($widget_definition_array)->group('after_content');

(B) what you expected to happen
I expected they are shown after content section.
(C) what happened
They are not appeared anywhere.
(D) what have you already tried to fix it?
Widgets are now stored into Fluent class before rendering.
But, after_content_widgets section in blank.blade.php, $widgets variable is still checked.
I think it is no longer necessary.
Thanks,

A new inspection was created.

The inspection completed: No new issues

Hello @onkbear
From code I kind agree with you. But not sure if Fluent is the only way Widgets are used, or if default array is still supported. I will ping @tabacitu so he can have a look at it.
Thanks for reporting.

Right! I can confirm this, looks like I've forgotten to change the way they're loaded for after_fields, thanks a lot for putting together the PR @onkbear ! Merging to master now - it'll probably get tagged and released on Monday, the latest.
Cheers!


SPI data register needs option for 8-bit access

The STM32F3 SPI data register DR can respond to reads and writes differently, based on whether the access is 8 bits wide, or 16 (or more) bits wide. For example, when the configured SPI data frame size is 8 bits or less, a single 16-bit write to DR will be interpreted as two data frames. This is called data packing, and is described in the reference manual (see STM32F303x/328x Reference Manual, RM0316, DM00043574, p.965 "Data Packing").
The stm32f30x crate currently has no way of performing a DR register read or write that is not 32 bits, so e.g. a write using
let data_out: u8 = 0xa5;
(*SPI2.get()).dr.write(|w| w.dr().bits(data_out as u16));

results in two data frames being loaded into the transmit FIFO. As a work-around, one may write
core::ptr::write_volatile(&(*SPI2.get()).dr as *const _ as *mut u8, data_out);

to write a single data frame, but at that point, one may as well be using C. :)
I don't recall, nor could I find in a quick search, any other instances of STM32 peripherals behaving this way, but there may be other circumstances where the access width is important. Since the stm32f30x crate is auto-generated from SVD, maybe support for SPI belongs in a higher-level HAL or BSP. But at minimum, it would be nice if the SVD patch added a bit of documentation on the DR register making note of this.

to write a single data frame, but at that point, one may as well be using C. :)

Or you could write an extension trait implement it on the register type and then be able to write: spi.dr.write_u8(0xaa).
The proper way to fix this in the SVD file is to use overloaded registers. Basically you declare two registers at the same address in the SVD file and then svd2rust gives you an API to use the register as one variant or the other so would be able to write spi.dr8.write(0xaa) or spi.dr16.write(0xaaaa). Or least that how it should work; svd2rust doesn't currently know how to handle "overloaded registers" so it just emits API for one of the registers.
Yet another way would be to extend the register API generated by svd2rust to provide methods to directly write to the upper / lower half-word / byte. But that method would be unsafe by default as the register may have "meaning" (configuration register) instead of just being a data register. Perhaps there can be some mechanism to opt into safety; for example the bits method becomes safe when <writeConstraint> in the SVD file says that the whole range is safe to use.

any solution to this yet?  AFAIK this is still a problem and in recent rust nightlies the workaround above results in compilation failures:
error: assigning to `&T` is undefined behavior, consider using an `UnsafeCell`


bug

bug report.txt
main.txt
i introduced a bug to my main.rs (main.txt), and used cargo valgrind run to detect it, but it crashed instead.
Note: I used valgrind in a docker, so it might the issue, idk.
thanks

Hey, I'm really sorry for the huge delay. I wasn't active in this repository for a long time. Your issue should now be fixed at last (on the master branch, a release is yet to come in the next few days).


TypeError: selection.interrupt is not a function

Hello David
i got an error in source code after npm install and npm run dev.
thanks you
(unknown) TypeError: selection.interrupt is not a function
at zoom.transform (eval at  (bundle.js:1598), :159:17)
at Selection.selection_call [as call] (eval at  (bundle.js:1550), :338:12)
at VueComponent.mounted (eval at  (bundle.js:1164), :167:13)
at callHook (eval at  (bundle.js:744), :2917:21)
at Object.insert (eval at  (bundle.js:744), :4154:7)
at invokeInsertHook (eval at  (bundle.js:744), :5956:28)
at Vue.patch [as patch] (eval at  (bundle.js:744), :6175:5)
at Vue._update (eval at  (bundle.js:744), :2656:19)
at Vue.updateComponent (eval at  (bundle.js:744), :2784:10)
at Watcher.get (eval at  (bundle.js:744), :3138:25)

and a warning here
[Vue warn]: Error in mounted hook: "TypeError: selection.interrupt is not a function"
found in
--->  at E:\vue_workspace\Vue.D3.tree\src\Tree.vue
 at E:\vue_workspace\Vue.D3.tree\example\tree\App.vue

What is the scenario? Are trying to clone the repo? If so try to run npm install again.

it worked. i run npm install again.
thank you

the problem is in the npm mirror depository.
when i download dependencies from the npm mirror , the error occur again.
then i run npm install again and download from npm center depository, it correctly worked.
thanks for your help. @David-Desmaisons


Check for embassy-12qw34er.local address on the Setup Complete page

Making the Setup Complete page even more simple it would be cool to use os-level health check on the Setup Complete page
and display LAN Instructions only if embassy-12qw34er.local address is available for the local network.

The reason we display the Initializing Embassy page with loading indicator is to buy time for both the Tor and LAN services to broadcast. If the final success page is reached, and the LAN service is not yet broadcasting, perhaps we need to remain on the Init page longer. If this is an intractable problem, meaning there is no predictable amount of time we can wait to display success, then another solution is needed. Possibly something like what @k0gen suggests, but I'm also not convinced this will be clear enough to the user. At minimum, we would also need to display messaging alongside the spinner: ie "waiting for LAN connection...", but it does feel a little weird to go from the "Initializing Embassy..." page to a success page that is clearly still initializing something. Might it be better to leave them on the "Initializing Embassy" page until we detect the LAN service, then send them to the static success page?

Might it be better to leave them on the "Initializing Embassy" page until we detect the LAN service, then send them to the static success page?

If by this you mean waiting for the LAN service at the "Initializing Embassy" page (spinner and timeout timer would be cool) and if LAN service is available redirect to embassy-12qw34er.local address and display "Setup Complete!" page?
Then yes that would be the most desirable and user friendly solution.

We cannot redirect because of certificate constraints. The browser will not trust start9-abcdefgh.local. and we can't redirect to the .onion site either because most likely their browser does not support Tor. For 0.3.0, I think we'll stick with the current flow but perhaps spend more time on the "initializing Embassy" page in order to ensure the .local site is accessible.

Assigning to @dr-bonez and @MattDHill for future scoping

In recent builds, the time until the .local is initialized is almost instant. It may be best to poll the .local (on port 80 without following redirects to avoid cert errors) to see if it's up.

closing for reasons above


Improve scratch url

Thank you for the contribution. It was merged directly as b307067f0919b29d5175da76fd4ceef9fa0e3e01 to keep commit history cleaner. Your contribution is still credited to you.


Sdk

@ReddSnow closing this since there's no description. Feel free to add a description and re-open the issue, if needed.


[SPARK-44451][BUILD] Make built document downloadable

What changes were proposed in this pull request?
Make built document downloadable
new steps:

built-docs:

(actually 55M to download)

Why are the changes needed?
1, to make it much easier for reviewers to verify the document PRs;
2, to make it much easier for maintainers to check the built document in branches;
Does this PR introduce any user-facing change?
no
How was this patch tested?
CI and manually check the downloaded docs

https://github.com/zhengruifeng/spark/actions/runs/5566016597/jobs/10167115410
I also test directly upload the directory, but it is slower and the artifact size is bigger (~112M):

(actually 112M to download)

cc @HyukjinKwon @dongjoon-hyun @Yikun @LuciferYang

thank you all
merged to master (4.0.0)


TE-3.2: Traffic Balancing According to Weights - Adding interface type ethernetCsmacd

Adding interface type ethernetCsmacd in setDUTInterfaceWithState

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 57.219%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1181

Relevant Lines:
2064

💛  - Coveralls

/gcbrun

/gcbrun

/gcbrun


Remove use of unwraps

remove use of unwraps where it might lead to a panic.
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

there are a couple of ways to do this, I generally used matches.  If reviewers would prefer a different approach, feel free to suggest it.

merge me

/lgtm

@baude Merged. Thanks


chore(deps): update dependency google-auth to v2.23.3

This PR contains the following updates:

Package
Update
Change

google-auth
patch
==2.23.0 -> ==2.23.3

Release Notes

googleapis/google-auth-library-python (google-auth)
v2.23.3
Compare Source
Bug Fixes

Serialize signer keys on getstate for pickling (#1394) (8b783b9), closes #1383

v2.23.2
Compare Source
Bug Fixes

Support urllib3<2.0 versions (#1390) (07c464a)

v2.23.1
Compare Source
Bug Fixes

Less restrictive content-type header check for google authentication (ignores charset) (#1382) (7039beb)
Trust boundary meta header renaming and using the schema from backend team. (#1384) (2503d4a)
Update urllib3 to >= 2.0.5 (#1389) (a99f3bb)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: requirements.txt
installing v2 tool python v3.12.0
[23:00:21.285] INFO (9): Installing tool python v3.12.0...
linking tool python v3.12.0
Python 3.12.0
pip 23.2.1 from /opt/containerbase/tools/python/3.12.0/lib/python3.12/site-packages/pip (python 3.12)
[23:01:06.497] INFO (9): Installed tool python in 45.2s.
installing v2 tool hashin v0.17.0
[23:01:07.594] INFO (170): Installing tool hashin v0.17.0...
linking tool hashin v0.17.0
Traceback (most recent call last):
  File "/opt/containerbase/tools/hashin/0.17.0/3.12/bin/hashin", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/opt/containerbase/tools/hashin/0.17.0/3.12/lib/python3.12/site-packages/hashin.py", line 794, in main
    import pkg_resources
ModuleNotFoundError: No module named 'pkg_resources'
[23:01:09.455] FATAL (170): Command failed with exit code 1: /usr/local/containerbase/bin/install-tool hashin 0.17.0
    err: {
      "type": "Error",
      "message": "Command failed with exit code 1: /usr/local/containerbase/bin/install-tool hashin 0.17.0",
      "stack":
          Error: Command failed with exit code 1: /usr/local/containerbase/bin/install-tool hashin 0.17.0
              at makeError (/snapshot/dist/containerbase-cli.js:42911:13)
              at handlePromise (/snapshot/dist/containerbase-cli.js:43810:29)
              at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
              at async InstallLegacyToolService.execute (/snapshot/dist/containerbase-cli.js:46271:5)
              at async InstallToolService.execute (/snapshot/dist/containerbase-cli.js:46450:9)
              at async InstallToolShortCommand.execute (/snapshot/dist/containerbase-cli.js:46766:14)
              at async InstallToolShortCommand.validateAndExecute (/snapshot/dist/containerbase-cli.js:1344:26)
              at async _Cli.run (/snapshot/dist/containerbase-cli.js:2457:22)
              at async _Cli.runExit (/snapshot/dist/containerbase-cli.js:2465:28)
              at async main (/snapshot/dist/containerbase-cli.js:46909:3)
      "shortMessage": "Command failed with exit code 1: /usr/local/containerbase/bin/install-tool hashin 0.17.0",
      "command": "/usr/local/containerbase/bin/install-tool hashin 0.17.0",
      "escapedCommand": "\"/usr/local/containerbase/bin/install-tool\" hashin 0.17.0",
      "exitCode": 1,
      "cwd": "/tmp/renovate/repos/github/renovate-reproductions/25148",
      "failed": true,
      "timedOut": false,
      "isCanceled": false,
      "killed": false
    }
[23:01:09.681] INFO (170): Installed tool hashin with errors in 2s.


[BUG] Cannot install packages on fresh install

What happened?
When running pnpm install plasmo, I am experiencing an error, preventing using the library
node version: v18.20.3
pnpm version: 9.1.1
running on macOS 14.5
Please let me know if you need any more info!
Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
No response
Relevant log output
 ERR_PNPM_WORKSPACE_PKG_NOT_FOUND  In : "@swc/types@workspace:^" is in the dependencies but no package named "@swc/types" is present in the workspace

This error happened while installing the dependencies of<EMAIL_ADDRESS> at<EMAIL_ADDRESS> at<EMAIL_ADDRESS> at<EMAIL_ADDRESS> at<EMAIL_ADDRESS>
Packages found in the workspace:
Progress: resolved 469, reused 424, downloaded 0, added 0

(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Also tried yarn and npm, having the same symptoms, some error with the @swc/types package.
yarn asks you to choose an option for the version, but still fails in the end
[1/4] 🔍  Resolving packages...
Couldn't find any versions for "@swc/types" that matches "workspace:^"
? Please choose a version of "@swc/types" from this list: (Use arrow keys)
❯ 0.1.7
  0.1.6
  0.1.5
  0.1.4
  0.1.3
  0.1.2
  0.1.1
warning plasmo > @plasmohq/parcel-config > @parcel/config-default > @parcel/optimizer-htmlnano > svgo ><EMAIL_ADDRESS>Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility

After choosing, I get this error:
Error: Cannot find module '.../node_modules/<EMAIL_ADDRESS>    at Module._resolveFilename (node:internal/modules/cjs/loader:1140:15)
    at Module._load (node:internal/modules/cjs/loader:981:27)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:128:12)
    at node:internal/main/run_main_module:28:49 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

This issue occurs with @swc/core version 1.5.11. To resolve this issue:

Install @swc/core version 1.5.7:

npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS># or
pnpm i<EMAIL_ADDRESS>

If @swc/core is already installed, fix it to version 1.5.7. Remove any caret (^) if present.

"dependencies": {
  "@swc/core": "1.5.7"
}

For more details, refer to the https://github.com/swc-project/swc/issues/8988

Thanks @cckn, your fix resolved the issue

May I ask how you solved it?

@MarMomento I followed exactly what was said here above

This issue occurs with @swc/core version 1.5.11. To resolve this issue:

Install @swc/core version 1.5.7:

npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS># or
pnpm i<EMAIL_ADDRESS>

If @swc/core is already installed, fix it to version 1.5.7. Remove any caret (^) if present.

"dependencies": {
  "@swc/core": "1.5.7"
}

For more details, refer to the swc-project/swc#8988

This package is in plasmo's dependencies, installing<EMAIL_ADDRESS>directly in the project doesn't work.
I solved it by downgrading plasmo directly to v0.80


Upgrade cosmos sdk and wasmd to latest version

The repo is very outdated right now and every since 2 major updates have happened for wasmd and cosmos-sdk each.

The latest 0.7.1 contains Cosmos SDK 0.45.14 and wasmd 0.30 types. Thank you for bringing this up!


[New Feature]:  Add know more button when ever any match match happen while playing the game.

🎮 feature Request
feature logic and basic description:
For example when we add force and time it result in motion, and a pop up screen shows motion, we can add a know more option on that pop-up screen to give a detailed knowledge on how force and time together results in motion.
Point down the features
feature points:
Increase knowledge.
Make the site more attractive
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Kindly assign this task to me.


Station Mules trading for other stations

So I have a bunch of stations, ore refinery, methane refinery etc. I assigned a station mule to each of them to trade to a warehouse. I ticked the assign to station , two way trade, and make target a warehouse boxes. It started out working well, but now i see different traders trading with stations they are not assigned to and others just standing still while there is enough ware to trade.

station mules will work for other stations when the other stations have more urgent priorities than the home. This may be addressed with a "serve source only" param in the future.

Ah that explains it. I would love a serve source only param in the future!

One thing, I have a supply mule assigned to a player wharf on source only, yet it still takes stuff from other stations and takes it to a warehouse station. Not just supplying the wharf.

Yes we noticed a bug today with serve source only. The good news is, where it is in the code means that the home station does not need any wares for their immediate production. It happens once we start checking for trade wares. However, it's still annoying that the param doesn't work, sorry about that.

work will be attached to https://github.com/Misunderstood-Wookiee/Mules-and-Warehouses-Extended/issues/64 so closing this as duplicate.


Reflective tape detection with OpenCv

Determine whether or not to use the Pi, and figure out what types are retuned by common OpenCV functions in Java.
The end goal is to be able to incorporate error from the angled tape camera into a PID.

The camera stream issue is done, we have 2 cameras running on decent resolution, and 20 fps.

I have code for the GRIP reflective tape detection that returns a value indicating how much the robot should turn to line up with the tape, and follow the line.  I just need to test it tomorrow.

Nice work!

Anybody know if it is possible to transfer the GRIP program output to the SmartDashBoard to display it?

You'd have to use CameraServer to push the stream to the robot if I remember correctly, @sMyn42 may have a better idea

Where do you plan to execute your GRIP pipeline? On the roborio, on a
raspberry pi?
Once you answer that question we can discuss getting the data out of the
pipeline
On Sat, Feb 9, 2019 at 2:41 PM Roshan Ravi<EMAIL_ADDRESS>wrote:

You'd have to use CameraServer to push the stream to the robot if I
remember correctly, @sMyn42 https://github.com/sMyn42 may have a better
idea
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FRCteam4909/2019-Deep-Space/issues/66#issuecomment-462073241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKvARezIXTkItOGiWLXZB8U6-icsY_3ks5vLyRQgaJpZM4aiFTN
.

We decided to just stick to java and open cv, no Pi.
On Sat, Feb 9, 2019, 5:04 PM Blake Bourque<EMAIL_ADDRESS>wrote:

Where do you plan to execute your GRIP pipeline? On the roborio, on a
raspberry pi?
Once you answer that question we can discuss getting the data out of the
pipeline
On Sat, Feb 9, 2019 at 2:41 PM Roshan Ravi<EMAIL_ADDRESS>wrote:

You'd have to use CameraServer to push the stream to the robot if I
remember correctly, @sMyn42 https://github.com/sMyn42 may have a
better
idea
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/FRCteam4909/2019-Deep-Space/issues/66#issuecomment-462073241
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAKvARezIXTkItOGiWLXZB8U6-icsY_3ks5vLyRQgaJpZM4aiFTN
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FRCteam4909/2019-Deep-Space/issues/66#issuecomment-462083415,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ah4wPyPyFGsLF2JZwDVm6WLdHguXcyOIks5vL0XsgaJpZM4aiFTN
.

@sMyn42 can you work with @Vultimate1 to display on the SmartDashboard during Monday's meeting?

Is vision processing still a possibility?  If so, we could just use the grip file created for detecting the reflective tape, without any java code.  We can still see the reflective tape through the grip file.  I've made it so you see its outlined shape to detect the tape.

@Vultimate1 if you can get working offboard, then we can consider it. can you provide a more in depth explanation of your proposal?

decided against.


Fix #792 Date>>print:on:format:alphabet:formatters: not escaping $' c…

…haracter.
This change should fix the bug.

LocaleTest>>testPrintDateTimeOnFormat modified as suggested.
This test passes with this other solution thanks for the review.


WMAgent Monitoring (What to monitor)

need to gather requirement

Whole documentation and plan here: https://github.com/dmwm/WMCore/wiki/WMAgent-monitoring
I'd say we've implemented > 60% so far

This has been fully implemented since the end of 2018, I think.


[nix] Show nix profile install progress

Summary
This shows inline progress when installing new packages (both global and local). nix profile install does this automatically but only in tty mode. @gcurtis curious if we can somehow make nix believe we're in tty mode without having to specify stdin and stdout?
Removed 2 blockers that are no longer needed:

PackageInstallWriter is no longer needed I don't think. It could not see the lines it was meant to supress.
Returning package not found is a bit redundant because all callsites of ProfileInstall already defensivelly check if package exists. If the user somehow does end up trying to install a package that doesn't exist, we still show a decent error

Fixed 2 semi-related errors:

global add  would print success message even if nothing was installed because of errors
Removing a package from global devbox is impossible if the nix package was not installed

How was it tested?
devbox add curl hello bazel
devbox add global curl hello bazel

@savil it has to be a video or a gif because the output is dynamic inline. Which also means I don't think we need the indent to make it look good. Will share.

I'm a bit too lazy to make a video, this is what it looks like:

The lower line is dynamic and shows progress.
this is what it looks like after installing multiple packages

I went ahead and removed the redundant "package is now installed" and now it looks like

Two weird behavior nits:

If package is already installed, no output.
When installing local packages we also install the global ones which is weird.

Will address this in follow up


Add IconTextBox

specs to come, placeholder issue for now.
Basically, we would want some sort of TextBox control that can have some sort of iconbutton that will fire a command when clicked. Picture a Search Box where you can type into it and either Enter or tapping on the magnifying glass icon will fire the command.
Unclear at this point if this should be in Themes or Toolkit

I was brainstorming on what would be the best approach to implement the TextBox with a trailing icon + command and doing some tests. Ludwig/Steve by default the uwp/winui TextBox includes a delete button that appears when text is entered in the box. When a user clicks on it, the text in the TextBox is cleared. I don't think of the scenario where we will have a trailing icon + command (like a search icon) in addition/next to the default delete button with the default clear all action right ?
I was thinking Steve that by default it will be the default delete button (*except for specific cases described below), but if a control extensions or custom properties are added we can maybe collapse the default delete button (that depends on specific default code-behind of the TextBox) and display the other button with the icon and command that we need.

IF we go with a new TextBox Styles approach with new extensions to accommodate the additional icon and command I did a little test on this branch (dev/agzi/TestIconButtonTextBox-Styles) with the TextBox Filled Style, but basically, we could have something like this at the end I think: But we will have 4 Styles in total at the end (with the two new styles): MaterialFilledTextBoxStyle Filled style that has the default delete button behavior MaterialFilledWithTrailingIconTextBoxStyle (better name to be found) Filled style that has the extensions for the leading/trailing icon + command extensions but without the default delete button behavior MaterialOutlinedTextBoxStyle Outlined style that has the default delete button behavior MaterialOutlinedWithTrailingIconTextBoxStyle (better name to be found) Outlined style that has the extensions for the leading/trailing icon + command extensions but without the default delete button behavior

IF we go with another new custom control based on TextBox to be able to add the icon and command properties I'm wondering about the necessity of creating a control that facades the button properties for the trailing icon and command, but if we go that route maybe we can better manage with the code behind and have only one custom control based on TextBox that can have the default delete button behavior when no trailing icon/command are specified and in the opposite situation the button with the trailing icon and command will be available and the delete button collapsed. That way we can continue to only keep 2 styles (MaterialFilledTextBoxStyle and MaterialOutlinedTextBoxStyle), but we will not use the default TextBox control, just the new custom one based on it.

Steve let me know if you think of other possibilities that I didn't think about it and what approach you will like the best in your opinion.

*Specific cases to take into consideration: Note also that by default with the uwp/winui TextBox, the delete button is shown only for editable, single-line text boxes that contain text and have focus. The delete button is not shown in any of these cases:

IsReadOnly is true (not editable)
AcceptsReturn is true (multiline textbox)
TextWrapping is Wrap (text is wrapping inside the textbox)

I was brainstorming on what would be the best approach to implement the TextBox with a trailing icon + command and doing some tests. Ludwig/Steve by default the uwp/winui TextBox includes a delete button that appears when text is entered in the box. When a user clicks on it, the text in the TextBox is cleared. I don't think of the scenario where we will have a trailing icon + command (like a search icon) in addition/next to the default delete button with the default clear all action right ?
I was thinking Steve that by default it will be the default delete button (*except for specific cases described below), but if a control extensions or custom properties are added we can maybe collapse the default delete button (that depends on specific default code-behind of the TextBox) and display the other button with the icon and command that we need.

IF we go with a new TextBox Styles approach with new extensions to accommodate the additional icon and command I did a little test on this branch (dev/agzi/TestIconButtonTextBox-Styles) with the TextBox Filled Style, but basically, we could have something like this at the end I think: But we will have 4 Styles in total at the end (with the two new styles): MaterialFilledTextBoxStyle Filled style that has the default delete button behavior MaterialFilledWithTrailingIconTextBoxStyle (better name to be found) Filled style that has the extensions for the leading/trailing icon + command extensions but without the default delete button behavior MaterialOutlinedTextBoxStyle Outlined style that has the default delete button behavior MaterialOutlinedWithTrailingIconTextBoxStyle (better name to be found) Outlined style that has the extensions for the leading/trailing icon + command extensions but without the default delete button behavior
IF we go with another new custom control based on TextBox to be able to add the icon and command properties I'm wondering about the necessity of creating a control that facades the button properties for the trailing icon and command, but if we go that route maybe we can better manage with the code behind and have only one custom control based on TextBox that can have the default delete button behavior when no trailing icon/command are specified and in the opposite situation the button with the trailing icon and command will be available and the delete button collapsed. That way we can continue to only keep 2 styles (MaterialFilledTextBoxStyle and MaterialOutlinedTextBoxStyle), but we will not use the default TextBox control, just the new custom one based on it.

Steve let me know if you think of other possibilities that I didn't think about it and what approach you will like the best in your opinion.
*Specific cases to take into consideration: Note also that by default with the uwp/winui TextBox, the delete button is shown only for editable, single-line text boxes that contain text and have focus. The delete button is not shown in any of these cases:

IsReadOnly is true (not editable)
AcceptsReturn is true (multiline textbox)
TextWrapping is Wrap (text is wrapping inside the textbox)

@Xiaoy312 suggestions:
we probalby dont need to define a new control, just reuse the existing one and just alter the base material style to support them, or make an icon-supporting copy the contract would look something like:
<TextBox Text=...
    utu:InputExt.LeadingIcon=...
    utu:InputExt.LeadingIconCommand=...
    utu:InputExt.TrailingIcon=...
    utu:InputExt.TrailingIconCommand=... />

"visual":
filled:
+--------------+                               +---------------+
|{leading-icon}|{text/placeholder}   {X-delete}|{trailing-icon}|
+==============+===============================+===============+

IsReadOnly or TextWrapping shouldnt be of concern if we are leveraging TextBox; as multiline, so long as we v-center/bottom (design choice) the icons, it wouldnt be a problem either
optionally we can throw in another attached property to control if x-clear/delete should ever appear at all

Am ok with a new IconButtonTextBoxStyle approach. Let's start with Outlined and do Filled later once it works.
Not sure if there's the need to have both the new Button AND the Delete button present at the same time.
It would use ControlExtensions.TrailingIcon and ControlExtensions.Command Attached Properties.

@kazo0 not sure if this helps,
but it sounds like the functionality you described above is available in:
AutoSuggestBox Class
which seems a lot like Search in Material

@kazo0 not sure if this helps, but it sounds like the functionality you described above is available in: AutoSuggestBox Class which seems a lot like Search in Material

yes @NVLudwig but in this case, the implementation needs to be more general, not just for searching purposes in order to be able to have any trailing icon we want for the TextBox with any command related to this icon

MaterialFilledTextBoxStyle
MaterialFilledIconButtonTextBoxStyle
MaterialOutlinedTextBoxStyle
MaterialOutlinedIconButtonTextBoxStyle

We are going with these two new names for now after I synced with @kazo0 and @Xiaoy312

@agneszitte-nventive The Search Scenario requires both the clear and an extra button to confirm the operation

Error Handling could require an icon on the right (not actionable) instead of the clear button

@agneszitte-nventive The Search Scenario requires both the clear and an extra button to confirm the operation
Error Handling could require an icon on the right (not actionable) instead of the clear button

cc @francoistanguay, @kazo0

Yeah I'd say the delete button should be available on its own since it is built into the TextBox control itself. We wouldn't need to use the attached properties and a command for the delete. I think having both is fine, there's no exposed property to hide the clear button though I don't think.

Hum then let me think about a way of having only one style for Outlined (same with only one style for Filled later on).
So adjusting the current MaterialOutlinedTextBoxStyle to manage the wanted behavior:

Adding the trailing icon using ControlExtensions.TrailingIcon and ControlExtensions.Command Attached Properties
Updating ControlExtensions.Icon to ControlExtensions.LeadingIcon Attached property so it's more clearer for the leading icon
Finding a way to hide the delete button when it's not needed (maybe with another new attached property)?

Error Handling will be done with the ControlExtensions.TrailingIcon visible to show the error icon, no command (as not actionable), and hiding the delete button maybe?
But @NVLudwig is there a scenario where we will have the trailing icon + delete button and when there is an error the trailing icon will remain and only the delete button will become the error icon not actionable ?

But @NVLudwig is there a scenario where we will have the trailing icon + delete button and when there is an error the trailing icon will remain and only the delete button will become the error icon not actionable?

I would imagine that is an invalid scenario.
IF The clear button only exists when there is an input.
IF the input is proven invalid the error badge should prevent the user from launching the same request again, THEN the error badge should replace the action button OR it should replace both the input and clear buttons.
So the possible error scenario is error badge + clear button. OR  error badge alone and user has to manually clear input (this seems to be the case in the Auto complete and Search Component for WinUI and Material)

But @NVLudwig is there a scenario where we will have the trailing icon + delete button and when there is an error the trailing icon will remain and only the delete button will become the error icon not actionable?

I would imagine that is an invalid scenario. IF The clear button only exists when there is an input. IF the input is proven invalid the error badge should prevent the user from launching the same request again, THEN the error badge should replace the action button OR it should replace both the input and clear buttons.
So the possible error scenario are error badge + clear button. OR error badge alone and user has to manually clear input (this seems to be the case in the Auto complete and Search Component for WinUI and Material)

Thanks for the feedback, make sense to me also

we will need 3 icons:

Leading Icon,Command,Is{0}Visible + {0}}Command: customizable via attached-dp
delete button: not-customizable & hidden from developer
^ you will manage this with visual-states:"ButtonVisible|ButtonCollapsed" (https://github.com/unoplatform/uno/blob/master/src/Uno.UI/UI/Xaml/Controls/TextBox/TextBox.cs#L865-L881)
^ the binding is automagically done with x:Name="DeleteButton"
Trailing Icon,Command,Is{0}Visible: customizable via attached-dp

trailing and delete button are 2 separate concepts
in the context of "error badge + clear button", it fits into the above setup as "TrailingIcon + delete button"
we can have them both coexisting, or make error badge to go away by manipulating IsTrailingIconVisible based on focus

@kazo0 I will sync with you about this item before moving forward

Okay, @agneszitte / @Xiaoy312, let's decide on an official spec for this
Leading Icon / Command
Update ControlExtensions.Icon to ControlExtensions.LeadingIcon and add a ControlExtensions.LeadingIconCommand
For trailing icons, I believe we can have more than one icon if we want, right @NVLudwig / @SMarengere ? Or do we just want to say that we can only have one customizable trailing icon plus the already built-in delete icon button?
Trailing Icon / Command
Create ControlExtensions.TrailingIcon and ControlExtensions.TrailingIconCommand
Could we maybe support having a collection of Trailing Icons? like ControlExtensions.TrailingIcons and some sort of way of setting an icon and a command for each one?

Could we maybe support having a collection of Trailing Icons? like ControlExtensions.TrailingIcons and some sort of way of setting an icon and a command for each one?

it would have to be a DependencyObjectCollection where T contains all the properties (icon, command, is-enabled)
the problem is the built-in textbox's clear button wont be able to fit into this collection
as that would be owned+managed by the textbox, while this collection would be the our hand

i don't want to touch the clear button at all, that's its own thing that is owned by TextBox. Let's just add the Command for the leading icon and a new trailing icon and trailing icon command.
Do we need the IsVisible? can people just set the Icon when they want an icon and not set it when they don't need it?

that wont be bindable, without a custom converter

(In case some details here can help https://github.com/microsoft/microsoft-ui-xaml/issues/784)

So for now we will have:
Leading Icon / Command
Update ControlExtensions.Icon to ControlExtensions.LeadingIcon and add a ControlExtensions.LeadingIconCommand and ControlExtensions.IsLeadingIconVisible
Trailing Icon / Command
Create ControlExtensions.TrailingIcon and ControlExtensions.TrailingIconCommand and ControlExtensions.IsTrailingIconVisible

@kazo0 Should this issue be moved to Themes repo?


LanguageFrequency refactor

LanguageFrequency and LanguageNgrams is the same thing, why do I need 2. Need to refactor into the same bit of code but the nicest name that its understandable how It can be used.

As of 859e9b40cc07075f4a26bee20e90f344f1ef7aa8


feat(commands): implement RFC 7151 (File Transfer Protocol HOST Command for Virtual Hosts)

Fix #114
Todo:

[x] Add handler for command
[x] Add listener
[x] Provide option for checking hostname just at login time
[ ] Checking for hostname (domain name / ip literal) syntax according to section 3.1, including port number persistence
[ ] Checking if IP address matches server IP (in case supplied) (section 3.1)
[ ] When TLS used, checking if supplied hostname matches with one from TLS (section 3.2.2)
[ ] CLI options (specifying a map between virtualhosts and user/pass/root combos)
[ ] Tests

[BUG] It looks like passing motd and/or anonymous has no effect :(

Thanks for the start of this PR! I'll take a look into giving you a hand with in when I get a chance. I'll read up on the HOST command first.

TL;DR:
The parameter of a host command is the hostname, obviously.
The hostname SHOULD NOT contain a port.
If it's an IPv6, the hostname SHOULD be encapsulated in []s. If an IPv4, it MAY be enclosed between brackets or provided in the IPv6 form.
According to the specs, the server should:

Validate the syntax (I was afraid to implement this, because validating IPv6 addresses is hard)
Validate hostname validity if possible

If it's a bare IP (the IP should be our public IP)
If TLS used and hostname was in certificate

I think this project shouldn't be so strict about validation. If ever implemented, maybe it's a good idea to provide an option to turn validation off.
Okay, syntax check could come any time :)

Oh, and I was able to test my program using WinSCP - the only client implementing HOST I found so far


koji_promote: report all pushed images

When reporting pushed images, don't just look at the results of the pulp_push plugin.
For v1: report only pulp_push results
For v1+v2: report pulp_push and pulp_sync results
For v2: report only pulp_sync results
As osbs-client will configure pulp_push and pulp_sync according to whether v1 and/or v2 is in use, all we need to do is collect the results from both pulp plugins.
If pulp is not used, report tag_and_push results (now modifed to return ImageName instances).

Shouldn't the TestKojiPromote be now skipped also if PULP_SYNC_KEY is None ?
i.e.
@pytest.mark.skipif(PULP_PUSH_KEY is None or PULP_SYNC_KEY is None,
                    reason="requires pulp_push and pulp_sync")
class TestKojiPromote(object):

Oh, true. In practice they'll either both be or neither, but I'll change it anyway.

LGTM

what about metadata exit plugin?

Because pulp_sync adds to the PushConf for the workflow, the metadata plugin picks it up automatically.
...which suggests that's how koji_promote ought to be doing it as well.

@twaugh thanks Tim; LGTM then

I've updated the koji_promote plugin to use TagConf+PushConf instead of looking through plugin results.
I also added two other fixes in separate commits: I think the metadata "name" field needs to include the image's namespace as well, and the tests did not run correctly in Python 3.

LGTM

Thanks. Re-basing and squashing fix-fix commits.

LGTM


specialised traders

Current state of the branch:

Log file agent_hist_log.txt needs a "]" to be manually added at the end of the file after the program has been terminated.
The specialism cannot yet be switched on and off with a config param.

Agents with id < num_agents/2 are grey in colour, and are specialised to collect water at a higher rate, determined by SPECIAL_WATER_ACQUIRE_RATE. The other half of the agents are white in colour, and are specialised to collect food at the SPECIAL_FOOD_ACQUIRE_RATE.
A non-default implementation of find_nearest_trader() overwrites the default when the Routing trait is implemented for the Trader. This custom implementation routes traders to the nearest trader of the other specialism when they select the ToAgent action. This might not be the most appropriate way to do this.
When resources are collected by each agent in the .step() method on Forager, the id of the agent is matched on to determine whether SPECIAL_FOOD_ACQUIRE_RATE or FOOD_ACQUIRE_RATE is added to the inventory (in the case the resource is food).


New App: Line clock

Analog clock with improved readability.

Looks nice, I will be using this for a while. Thanks :)
One thing first is the app declares stuff in global scope. Especially the const storage makes it so e.g. the watch hangs when trying to start the settings app since that variable name is taken. You have two possibilities here:

delete the remove function of setUI so  that the watch will reset before loading the launcher/other apps. Or...
follow this tutorial to implement fast loading to the launcher properly: https://www.espruino.com/Bangle.js+Fast+Load

I'll look some more and be back if i find anything else :+1:

Thanks @thyttan for pointing this out. I disabled the remove function for now. I will optimise for fast loading in a future release.

Thanks!


Adds operationOutcome for type not present

Description
Adds specific operationOutcome diagnostic for type not present
Related issues
Addresses Bug 71574
Testing
Manual testing

Can we add a test for this?

Added e2e test


Services and packages

The addon is amazing, is it still in development?
Are services and packages planned to come on next updates? If not, do you think it's possible? (In case I decide to fork and attempt to work on it)

Hi, lucroch, thanks.
Updates are spontaneous or at the request of users.
There is no specific development plan.
If something specific needs to be added, then write here.
Perhaps there will be some development, while I myself do not know...


HandlesAuthorization deny method code parameter doesn't change

Laravel Version: 8.49.2
PHP Version: 8.0.8
Database Driver & Version: MySQL 8.0

Description:
I am using a Policy class and using the deny allow methods in the HandleAuthorization class as Trait, the deny method accepts the status code as the second parameter, even though I give the 403 code, I get a response code as 401 every time.
Steps To Reproduce:

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

Hi, I don't mind it's a question, it looks like a bug. Because given status code not reflect to response.


make command does not work : Makefile:2: *** missing separator.  Stop.

Hi,
If I copy your demo/make to a demo/Makefile file and run :
cd demo
make all
Makefile:2: *** missing separator.  Stop.

I would like to be able to use both ways to build, that is to say :
./Makefile and make
Can you help me ?

@sebma, hello! This demo does not support the use case you've described. As it's currently written, the top is interpreted by the shell and the bottom is interpreted by Make. If you wanted to add this functionality by building on the existing implementation you would need to:

Provide a means for exec make -f- "$@" << 'eof' to be interpreted functionally the same in shell but as a comment or no-op in Make. Offhandedly, I don't know a hack to do this.
Duplicate any top command line options in the bottom. Make supports setting many (but not all, I think) command line options in the makefile itself so hopefully this wouldn't be the blocker.

Some but not all of the demos do support whole-script interpretation by their interpreter, such as the Python demo, so that may be where the confusion came from. I've updated the readme to clarify that this isn't always the case.

@niedzielski Thanks for your explation :)


JavaScript sample code

Do you have sample code for mock server using JavaScript?

Please see the following links:
http://mock-server.com/mock_server/getting_started.html#javascript_example
and
http://mock-server.com/mock_server/creating_expectations.html#javascript_example


fix: 登录初始密码，跳转到修改密码页面

fix: 代码回滚

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.v_yutyi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add UPDATE query mapper

Here is the implementation of UPDATE query mapper. Especially it is very useful together with IfExists option.

is there any chance someone will review it?

Closing as we are not going to add any new features to the mapper in the driver 3.x series. A new mapper API will be published after the 4.0 release.
@vagifzeynalov I like the idea though so I went ahead and created JAVA-2042 so that we don't forget it when the new mapper API will arrive. Thanks for your time and effort!

@adutra no problem. Looking forward for 4.0 release!


V05 zarr v3

Work in progress...

Handle reading of OME-Zarr v0.5 groups & metadata (with Zarr v3)
Validate against RFC-2 schemas in https://github.com/ome/ngff/pull/242
Use zarrita.js to read Zarr v2 and v3 arrays

@will-moore: I'm going to close this so that we only have folks looking in one place. Unless we want to expend more energy in supporting the devX versions, I'd propose that we just leave the individual PRs (dev1=#35, dev2=..., etc.) open so that the URLs like https://deploy-preview-35--ome-ngff-validator.netlify.app become our first line of validation.


Compile target does not automatically change when Core devices are selected

When you select a Photon, Electron, or Raspberry Pi device in the device selector, the compile target indicated at the bottom status bar changes to the corresponding platform.
That is not the case when you select Cores from the device selector, originally submitted in the community thread and confirmed on my machine.
https://community.particle.io/t/new-desktop-ide/28800

Fixed with https://github.com/spark/particle-dev/commit/738e019228da2b50922d1d946c4b090b474d18e9


resolve.alias not applied

Hi,
I wanted to optimize my performance and therefore wanted to prefetch react:
new webpack.PrefetchPlugin('react'),
Problem with that, is that only a the index.js from react is prefetched and every other part of react 100+ files is still normally processed. I wanted to go around that by providing an alias for react to the distributed version with the following line:
    resolve: {
        alias: [
            { react: 'react/dist/react'}
        ],

This did not change anything, so I played around and also tried to completely screw it up with:
        alias: [
            { react: 'gulp'}
        ],

It is still normally processing my require('react') as there is no alias.
Any ideas?

I'm not familiar with the alias API, but from the examples I've seen, it does not appear to take an array as a parameter. Try:
resolve: {
  alias: {
    react: 'react/dist/react'
  }
}

Perfect that did the trick.


feat: add custom output file name support

Awesome thanks ! Will review asap ! 👏

Works well, thanks! Approved and merging


Add MSVC analysis to project

Signed-off-by: The MathWorks, Inc<EMAIL_ADDRESS>

Codecov Report

Merging #685 (2dc562f) into development (c4a8d62) will decrease coverage by 1.15%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #685      +/-   ##
===============================================
- Coverage        90.32%   89.16%   -1.16%
===============================================
  Files              249       91     -158
  Lines            13629     6288    -7341
===============================================
- Hits             12311     5607    -6704
+ Misses            1318      681     -637

Impacted Files
Coverage Δ

...rk/include/cppmicroservices/detail/WaitCondition.h
71.42% <0.00%> (-23.02%)
:arrow_down:

framework/src/bundle/BundleRegistry.cpp
79.29% <0.00%> (-12.68%)
:arrow_down:

framework/src/util/FrameworkPrivate.cpp
70.96% <0.00%> (-5.78%)
:arrow_down:

framework/src/service/ServiceEvent.cpp
71.79% <0.00%> (-5.76%)
:arrow_down:

...mework/src/service/ServiceReferenceBasePrivate.cpp
86.45% <0.00%> (-5.64%)
:arrow_down:

.../cppmicroservices/detail/BundleAbstractTracked.tpp
82.30% <0.00%> (-5.28%)
:arrow_down:

framework/src/service/ServiceRegistrationBase.cpp
77.31% <0.00%> (-4.58%)
:arrow_down:

framework/src/util/FrameworkEvent.cpp
61.81% <0.00%> (-3.94%)
:arrow_down:

framework/src/bundle/BundleResourceBuffer.cpp
67.34% <0.00%> (-3.86%)
:arrow_down:

framework/src/service/ServiceListeners.cpp
96.16% <0.00%> (-3.84%)
:arrow_down:

... and 228 more

could you provide a link to more information on MSVC Code Analysis?

https://github.com/microsoft/msvc-code-analysis-action


Popup: the positioning and aligning of the Popup around the trigger are limited

Feature Request
Problem description
The positioning and aligning of the Popup relative to the trigger are limited. For example, the following position is not possible to be defined with the current properties:

Proposed solution
We have currently prop align which indicates where should the popup be placed relatively to the trigger. We are missing property that will indicate with which part of the Popup should the trigger be aligned (top, center, start, end, bottom).

The Semantic Popup allows for positioning on any side.  It also also allows for adding vertical and horizontal offsets to align it perfectly.  It has had many iterations for a few years.
Let's start with this API:
https://react.semantic-ui.com/modules/popup/#variations-position


Multi-level menus (submenus)

How to create a multi-level menu (submenu)? Is there a ready-made solution?

I have this solved in a different project and I'm planning to refactor the menu so this is going to be really easy. It's planned, but not worthy to be done at the current dev branch.

Hello, I wanted to know if this feature is currently possible. Thank you!

It's not possible yet but it will be. It's blocked until the postcss version is released

Closed in favor of #1169


contribution

I would like to contribute to the project by adding new rules.
How can I get started?

Welcome in our community and thank you for your interest to contribute to K8sGPT!
Please take a look in our contribution guide (https://github.com/k8sgpt-ai/k8sgpt/blob/main/CONTRIBUTING.md) and join our slack workspace. Furthermore, we have some good first issues in our repos.


Example notebook how to discover datasets via API request

From my chat with @emaroon today. It would be very helpful to have an example how to query the API for these specific grid datasets. Ill try to work on that the week after Scipy.

Is the usefulness of this perhaps superseded by https://github.com/pangeo-forge/pangeo-forge.org/pull/93, following which, the datasets for any feedstock will be discoverable on the website?

Yes I believe it actually is. I ll close this for now.

Human readable access (website) vs. machine readable access (API) are distinct use cases.
We still eventually need a STAC catalog for Pangeo Forge Cloud datasets.


Custom blueprint not installing

After installing semantic-ui-ember (3.0.0), I try to run
ember generate semantic-ui-custom
and I'm returned:
Removing semantic-ui-css  Cannot read property 'then' of undefined
and the error log points to the .then in the /blueprints/index.js at:
afterInstall(options) {  return this.ui.write('Removing semantic-ui-css').then(function () {
Presumably the install fails, as there is no 'less' installation anywhere I can find, and I cannot get the semantic config file to recognize my theme.config file no matter where I place it.

= Update =
Using the 'index.js' from the 3.0.0-beta3 along with the ember generate semantic-ui-custom command seems to generate the blueprint and install the /semantic folder properly.

Same as https://github.com/Semantic-Org/Semantic-UI-Ember/issues/223 - sorry for the dupe.


Allow manual attest to complete all / any open attestation runs

This change allows user to attest at any time and will close of any existing attestation runs against that entity (for that user).


ImTui-ImGui built against ImGui v1.89.6

Tested this locally with my project (Cataclysm: Dark Days Ahead) and appears to work.

@ggerganov hi, i am using this branch locally for my work on Cataclysm and I thought you might like to integrate this for the purpose of getting the changes pushed upstream. We are able to use this code for SDL and ImTui builds.

No sense in keeping this open if there is no interest. Closing.


[Issues Migration] Migration of issues from an external

Is your feature request related to a problem? Please describe.
This Feature is needed to complete the migration from IBM repository to External Git repository.
Describe the solution you'd like
A script is needed to migrate the whole list of issues.
Describe alternatives you've considered
N/A
Additional context
N/A

Issues were migrated from the private repo to this one, each of the issues that have the Historical tag are the ones from the private repo


Remove stray whitespace in settings

This seems like an error in the default settings.

Can you also do this change?
diff --git a/sublime-package.json b/sublime-package.json
index 827e159..679202a 100644
--- a/sublime-package.json
+++ b/sublime-package.json
@@ -75,6 +75,7 @@
                     "python.analysis.diagnosticSeverityOverrides": {
                       "default": {},
                       "description": "Allows a user to override the severity levels for individual diagnostics.",
+                      "additionalProperties": false,
                       "properties": {
                         "reportAssertAlwaysTrue": {
                           "default": "warning",

It would catch those issues in the future.

Done

Thanks!


build: update to node 20

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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 32.759%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
245

Relevant Lines:
659

💛  - Coveralls


Grafana can't get the data of iotdb time series database, Why？

Describe the bug
I recently encountered a problem. When I successfully used “iotdb-grafana” to establish a connection with the iotdb database.But I can’t get the image on the page where the chart is made. Although I have searched for my keywords in the database, the icon does not draw a time series line chart.
My guess is that the database time is a millisecond timestamp, while grafana uses yy:mm:dd:hh:ss.
But I find Grafana  support the  millisecond timestamp.?
Maybe there may be other reasons, ？？
Expected behavior
Can not draw the time series line chart .
Screenshots
pic
Desktop (please complete the following information):

OS: Win10
Browser chrome,
Version iotdb-grafana-0.12.0-SNAPSHOT

Additional context
NO

Hi, You could see the log of iotdb-grafana.war to see the sql that the grafana sends to IoTDB. The sql could be executed in the CLI. Then you can put the sql here.

ok，thanks
I find the databases of date is different from the local computer date. When I deal with it. It's working.


Autocomplete for @-mentions should prioritize users I interact with

For example, if on chat.zulip.org I type @tim, I get 5 suggestions for users I've never heard of -- more than enough to fill the visible list -- before I see Tim Abbott. Similarly if I type @tab (the start of the localpart/username of his email address), or @abb.
We should follow the same logic the web app has for this. See typeahead_helper.js, particularly sort_for_at_mentioning and its subroutines.

Hmm, I have worked on this logic few months back. That seems be only useful only for smaller realm.
@zulipbot claim

Great, thanks!

That seems be only useful only for smaller realm.

I think this may be what you mean already, but just to be explicit: I think the logic we need and don't yet have, which the web app has, gets more and more necessary as a Zulip org (realm) grows. Our existing logic is potentially enough for small realms, but users of chat.zulip.org, and I think a lot of mid-size realms much smaller than that one, will be glad to see this implemented :smile:

One thing worth mentioning is that the logic of sort_for_at_mentioning is based on the recent_senders data structure, which comes from data for the most recent 1200 messages that we fetch in the webapp.
We don't do the same fetch in mobile, so while we can probably use a similar algorithm, I'm not sure we have the data stored effectively in the app currently to do this that same way.

Sure. There are a lot of elements of that algorithm that should be useful independently of each other -- so I'd like to first merge all the ones we can do without additional data.
After that, we can problem-solve how best to do the rest. (For example, there might be appropriate summaries the server could provide us -- potentially even some which would improve ranking quality for the web app beyond what it currently does.)

I believe this will be fixed with a fix for https://github.com/zulip/zulip-mobile/issues/3902.


fix: bump chart and hedera version,  re-enable node add

Description
This pull request changes the following:

bump chart and hedera version
re-enable node add

Related Issues

Closes #

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add azure resource data grid provider

Pull Request Test Coverage Report for Build 240290295

2 of 37 (5.41%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 39.072%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/azurecore/src/constants.ts
0
1
0.0%

extensions/azurecore/src/utils.ts
1
11
9.09%

extensions/azurecore/src/extension.ts
0
24
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/azurecore/src/extension.ts
1
0%

Totals

Change from base Build 240285798:
-0.006%

Covered Lines:
19015

Relevant Lines:
44452

💛  - Coveralls


doc(README): reorganization of the content structure

Closes #40

@rultor, release; tag is 0.0.6.

@rultor, release; tag is 0.0.6.

@rafamizes OK, I will release it now. Please check the progress here

@rultor, release; tag is 0.0.6.

@rafamizes Done! FYI, the full log is here (took me 3min)