Mitigate dependency cxf-rt-bindings-soap-3.4.10.jar

Mitigated cxf-rt-bindings-soap-3.4.10.jar vulnerability

Tested by running omnibus on local
Also ran script of omnibus-integration-test-stage

Adding suppression for this vulnerability because we are not using RPCRouterServlet of Apache SOAP in  a2d2, https://nvd.nist.gov/vuln/detail/CVE-2022-40705

@vdeshkar the cxf dependencies affect the workbench endpoint in omnibus mostly. After you start the test in omnibus local, check if the following path works on your local: http(s)://LOCAL_SERVER_NAME/omnibus-api/kie-services/server/containers
if that works, this PR has my blessing

@vdeshkar the cxf dependencies affect the workbench endpoint in omnibus mostly. After you start the test in omnibus local, check if the following path works on your local: http(s)://LOCAL_SERVER_NAME/omnibus-api/kie-services/server/containers
if that works, this PR has my blessing

@vdeshkar the cxf dependencies affect the workbench endpoint in omnibus mostly. After you start the test in omnibus local, check if the following path works on your local: http(s)://LOCAL_SERVER_NAME/omnibus-api/kie-services/server/containers
if that works, this PR has my blessing

@vdeshkar the CXF components are used when rendering the HTTP response as XML, so the result we need to check is from the browser for your particular local running. AnalyzeEnvironmentUtility uses JSON and I'm not sure it works specifically with the SOAP dependency here.

Okay @marianbuenosayres I will check again on web browser, Actually I tried with web browser by running omnibus, But it give me response not found on browser.

@marianbuenosayres I able to get the html response of kie containers on web-browser,


Integrate Bitly for lengthy links

For links that are lengthy and breaking the design, integrate bitly and shorten them

Example Project
https://helpwithcovid.com/projects/784-raise-a-million-for-unemployed-mothers-on-mother-s-day

I don't like that we have to pay for bit.ly. How about we try another approach and if the link is super big we should only the domain part? eg http://docs.google.com/... instead of that long google docs strirng.


Template all the things

Can you add some more enviroment vars to customize all the text strings? I like to translate the text to German ;)

Sure!


Prepend Return-Path header to inbound mail

Prepend a Return-Path header to the received email containing the 821.From address. This will then be available as part of the mail summary via the API and the web interface.
Closes #149

I'm closing this as this has fix been added via https://github.com/axllent/mailpit/commit/149bfa80c2e5984ae70920183e75a810399d4f75


GBFS carsharing extension

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Miryad3108:x: thbarYou have signed the CLA already but the status is still pending? Let us recheck it.

Sorry - wrong repository! We are working on a fork with improvements for car sharing.


Missing documentation

I've been searching in Issues and the docs but I can't seem to find documentation on the following features:

Password Protected Uploads
URL Formats (uuid, dates, random alphanumeric, original name, zws)

do you guys have any idea on how i may use these?

Yep, sorry about the documentation I haven't really bothered updating it as much.
Password protected uploads work like this:
Add the password header when uploading a file with your desired password, when people use the link returned by the request it will ask them to enter a password and the rest is self explanitory
URL Formats also work the same way, you add the format header which can be UUID, DATE, RANDOM (default), or NAME. Date just formats the file like this: YYYY-MM-DD_HH:mm:ss, RANDOM just creates a string of alphanumeric characters with the length specified in your config, uploader.length or UPLOADER_LENGTH (environment var). Name just uses the original name returned in the request.

It's okay, don't stress too much on it :)
Thanks for the help! I've been stumped on it for quite a while now.


Wrong code implementation in forest/binary_trees/traversal.py

@shunsvineyard
Running the following code:
from forest.binary_trees import binary_search_tree
tree = binary_search_tree.BinarySearchTree()
tree.insert(key=5, data="")
tree.insert(key=6, data="4")
tree.insert(key=3, data="30")
tree.insert(key=4, data="11")
tree.insert(key=2, data="7")
tree.insert(key=1, data="34")
print([item for item in inorder_traverse(tree, recursive=True)])
print([item for item in inorder_traverse(tree, recursive=False)])
print([item for item in postorder_traverse(tree, recursive=True)])
print([item for item in postorder_traverse(tree, recursive=False)])
print()
tree = binary_search_tree.BinarySearchTree()
tree.insert(key=2, data="")
tree.insert(key=1, data="4")
tree.insert(key=4, data="30")
tree.insert(key=3, data="11")
tree.insert(key=5, data="7")
tree.insert(key=6, data="34")
print([item for item in reverse_inorder_traverse(tree, recursive=True)])
print([item for item in reverse_inorder_traverse(tree, recursive=False)])
print()
Will give the following output:
[(1, '34'), (2, '7'), (3, '30'), (4, '11'), (5, ''), (6, '4')]
[(2, '7'), (3, '30'), (4, '11'), (5, ''), (6, '4')]
[(1, '34'), (2, '7'), (4, '11'), (3, '30'), (6, '4'), (5, '')]
[(2, '7'), (4, '11'), (3, '30'), (6, '4'), (5, '')]
[(6, '34'), (5, '7'), (4, '30'), (3, '11'), (2, ''), (1, '4')]
[(5, '7'), (4, '30'), (3, '11'), (2, ''), (1, '4')]
From which it is clear that the code implementation of the non-recursive internal functions of in-order, post-order and reverse in-order traversals are all incorrect.
My guess is that, taking _inorder_traverse_non_recursive() as an example, the code does not take into consideration the case that a left grandchild of a certain node may not have a right child, even though its parent has one. In other words, it is incorrect to just continue to the next iteration at the FIRST left grandchild who has no right child (as in Line 277 of traversal.py), because there may well be a lot of left grandchildren who has one or no right child interspersed along the way down left.
Please identify the issue and make corrections. Thanks!

Thank you for pointing this out. I will take a look.

The bug has been fixed and merged. https://github.com/shunsvineyard/forest-python/pull/27


OHM Tasking Manager ID editor not accessing OHM data base

Bug description
The 2.18.5 ID editor built into the tasking manger is not accessing the OHM server and does not show any lines or information.
The way this would look normally is to edit the data that is within OHM.
Repro Steps
Steps to reproduce the behavior:

Go to a task and edit in ID.
You can choose my task 1936/1937 aerial imagery
Select the slim yellow in Puget Sound that includes the Fort Lawton West Point Lighthouse and
Compare data from OHM ID to the tasking manager ID.

What it is doing
Right now ID in the tasking manger is trying to access the OSM server without success and any save is being saved locally according the error and not the OHM server

Yes this is true. The embedded iD in Tasking Manager is using the OSM version. Until recently we couldn't embed OHM iD in TM because it was an old version. But this will be now possible.

Yes this is true. The embedded iD in Tasking Manager is using the OSM version. Until recently we couldn't embed OHM iD in TM because it was an old version. But this will be now possible.

@geohacker, now that we have a working iD with date filters, what's involved in getting that iD working in Tasking manager?

We just discussed having @Rub21 take a pass at this to fork our fork of iD and adapt to have this version present in Tasking Manager. The version we're using on Production is here: https://github.com/OpenHistoricalMap/iD/tree/release

Revised plan -- @geohacker is going to add notes here on what was done by HOT to make iD work on their TM, and then we'll take a look at GreenInfo

The way I see HOT does it is this:

They have a fork of iD which is published on NPM https://www.npmjs.com/package/@hotosm/id
The fork has these changes to make it work for embedding in tasking manager https://github.com/hotosm/iD/commit/886ee5f38ce51b2b9f4ec4b56fefb41c14a0cd11
Then this npm package is added to tasking manager frontend's dependencies https://github.com/hotosm/tasking-manager/blob/develop/frontend/package.json#L11

In our case, we would fork the fork of iD and make these changes, push to NPM and add it here https://github.com/OpenHistoricalMap/tasking-manager/blob/staging/frontend/package.json
This should be tested locally first, but then merged to staging branch to deploy to staging.

Seems like we could do this in a taskingmanager branch of iD rather than a fork, and then we can pull future changes into that branch to keep Tasking Manager iD in sync with OHM.org version

@danrademacher I think we should keep this fork separate and keep the main fork clean from feature branches. It's also easier to manage NPM modules as separate github repos.

OK. I actually thought it wasn't even possible to fork one's own fork in the same organization. Which seems confirmed here:
https://github.community/t/creating-a-second-fork-on-a-repository-from-same-user/2729

Yes, sorry I should have been more clear — we will have to create a new repo and push a copy of our fork there. And use this copy solely for publishing the NPM module.

For step 1 (new, detached repo), I have created the new repository at https://github.com/OpenHistoricalMap/iD-TaskingManager
For step 2 (the changes) many of the diffs no longer apply to this version of iD, as that diff was a year ago.

The only diffs to be applied are to package.json, to make this distinct from OpenHistoricalMap/iD-TaskingManager.
The CSS changes are non-applicable as .id-editor has replaced #id-editor and there are no styles on body, html
The CSS build script already has corresponding changes in place.

For step 3, Dan created the account and I will put code here.

Per chat: Given that the newer iD codebase at https://github.com/OpenHistoricalMap/iD works with today's Tasking Manager without modification, we have elected NOT to use a detached repo over at https://github.com/OpenHistoricalMap/iD-TaskingManager but to simply publish OpenHistoricalMap/iD to NPM and use it.

[ ] delete repo at https://github.com/OpenHistoricalMap/iD-TaskingManager

[x] publish OHM iD as https://www.npmjs.com/package/@openhistoricalmap/id

[x] test installation into TM via yarn & npm

[x] test basic functionality

[ ] make up PR for Tasking Manager, to be merged into staging branch for additional testing

Example: Occidental Hotel
Location 19.00/47.60188/-122.33304
Existed from 1861 to 1880, replaced by a triangular parking garage 1881
1860: no Occidental Hotel

**1861: O.H. **

1850 - 1895, both existed within this window

1890 - 2020, within this window the O.H. was gone, but the Sinking Ship parking garage remains

@geohacker I tested locally and merged into staging but then realized I am not sure what the deployment pipeline is for Tasking Manager. Looking into the ohm-deploy repo, the only mentions of "task" I see in the template vars:
https://github.com/OpenHistoricalMap/ohm-deploy/blob/main/values.staging.template.yaml#L411-L456
But I'm not sure where we kick off a GH Action base don the latest version of the staging branch of the Tasking Manager.

Done on Saturday while procrastinating on nonwork chores: Hmm, so this should have deployed automagically, and the Action ran but the new iD is not present on staging.
I had tested the ohm-id branch locally, then merged to staging and thought that would be enough. But then when I checked out staging locally and ran yarn install and yarn start`, I still didn't see the new iD. What's up?
I suspected some caching/not downloading new iD issue, so I then ran yarn upgrade @openhistoricalmap/id and presto, working from staging locally:

I changed our build command from cd frontend && yarn install to cd frontend && yarn cache clean && yarn install and that forced the update.
Next up, why are we missing locales on staging but not locally? That we will tackle next week!
This is on https://tasks-staging.openhistoricalmap.org/

The locales are installed by running a specific script in package.json,
yarn build locales I think it was.
On Sat, Apr 10, 2021, 3:54 PM Dan Rademacher @.***>
wrote:

Done on Saturday while procrastinating on nonwork chores: Hmm, so this
should have deployed automagically, and the Action ran but the new iD is
not present on staging.
I had tested the ohm-id branch locally, then merged to staging and
thought that would be enough. But then when I checked out staging locally
and ran yarn install and yarn start`, I still didn't see the new iD.
What's up?
I suspected some caching/not downloading new iD issue, so I then ran yarn
upgrade @openhistoricalmap/id and presto, working from staging locally:
[image: image]
https://user-images.githubusercontent.com/1423200/114286298-4c973980-9a12-11eb-8303-7e89cb078e9d.png
I changed our build command from cd frontend && yarn install to cd
frontend && yarn cache clean && yarn install and that forced the update.
Next up, why are we missing locales on staging but not locally? That we
will tackle next week!
This is on https://tasks-staging.openhistoricalmap.org/
[image: image]
https://user-images.githubusercontent.com/1423200/114286579-ea8c0380-9a14-11eb-855a-106fc02bc58b.png
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/OpenHistoricalMap/issues/issues/151#issuecomment-817212891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXZEQPRNWN52XWUJJQVGD3TIDJJNANCNFSM4SCARE4A
.

I see that in Tasking Manager, we can do build locales though when I ran this locally, I got an unexpected result of rolling back a bunch of OHM-specific changes we want, and not doing anything obvious for iD localization:

Looks like this:

We definitely don't want to do that on staging. Not sure if that means the original locales changes were made in the wrong place or what. Let's call that a task for another day. But it also doesn't seem to touch iD at all.
Looking at the code that is working locally, in the git-ignored build folder we have frontend/build/static/locales/en.json that appears to be a locales file only for iD:
       "date_ranges": {
            "title": "Date Range",
            "start_date": {
                "description": "Start Year",
                "tooltip": "The start year, as an integer, negatives OK",
                "placeholder": "e.g. 1487"
                 ...

That file is identical to iD/dist/locales and totally different from the TM locales file at /frontend/src/locales/en.json
So why is it that I have that file locally but it is not created when this action runs?
https://github.com/OpenHistoricalMap/tasking-manager/blob/staging/.github/workflows/staging.yml

Digging deeper, worried I'm just mixed up here.
This is the locales file in our S3 deployment:
https://s3.amazonaws.com/tasks-staging.openhistoricalmap.org/static/locales/en.json
this includes:
  "date_ranges": {
            "title": "Date Range",
            "start_date": {
                "description": "Start Year",
                "tooltip": "The start year, as an integer, negatives OK",
                "placeholder": "e.g. 1487"
            },
            "end_date": {
                "description": "End Year",
                "tooltip": "The end year, as an integer, negatives OK",
                "placeholder": "e.g. 1976"
            }

aha, and this appears to be all about aggressive browser caching, dang it. disabled cache:

Deployed and working!


chore(main): release 0.8.1

:robot: I have created a release beep boop
0.8.1 (2024-02-16)
Bug Fixes

ci: install rye without prompting (668a7c1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/engeir/volcano-base/releases/tag/v0.8.1 :sunflower:


How to modify bds if my SGE cluster using vf=memory,p=n_cpu?

Hi
I succeed in submitting my jobs using this pipeline. However, I found the resource claimed in bds.config does not work so the pipeline would run even if my cluster is full of jobs.  My SGE cluster used vf=memory,p=n_cpu instead of h_vmem and slots. We used to submit jobs like:
qsub -cwd -l vf=10g,p=8 -V run.sh

In my bds.config, I used:
sge.pe = make
sge.mem = vf
sge.timeout = h_rt
sge.timeout2 = s_rt
clusterRunAdditionalArgs = -V

The question is how could I modify my config file to adding something like vf=10g,p=8 in qsub command? Thank you.

BDS only works with SGE PE (parallel environment). You may need to ask your
admin to make a PE for the pipeline. Please see "SGE cluster options" in
https://pcingola.github.io/BigDataScript/bigDataScript_manual.html for
details.
Jin
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Tue, Aug 1, 2017 at 6:10 AM, huboqiang<EMAIL_ADDRESS>wrote:

Hi
I succeed in submitting my jobs using this pipeline. However, I found the
resource claimed in bds.config does not work so the pipeline would run even
if my cluster is full of jobs. My SGE cluster used vf=memory,p=n_cpu
instead of h_vmem and slots. We used to submit jobs like:
qsub -cwd -l vf=10g,p=8 -V run.sh
In my bds.config, I used:
sge.pe = make
sge.mem = vf
sge.timeout = h_rt
sge.timeout2 = s_rt
clusterRunAdditionalArgs = -V
The question is how could I modify my config file to adding something like
vf=10g,p=8 in qsub command? Thank you.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kundajelab/chipseq_pipeline/issues/16, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIOd_D2WMr0J-tPnD1uUVrxEmEHaFEAjks5sTyO6gaJpZM4OptER
.


nCAT_main.c does not compile

I tried to run the XCAT example and found that those nCAT-related functions were not included in the DLL. I checked clib_build\MakeWindows64 and other related files, nCAT_main.c is not compiled at all. None of the MakeVariables_* files includes it. It works fine after I added nCAT_main.c to the end of MakeVariables_1.
I have recompiled the DLL with some changes in the C code so I'm not sure if the original DLL in the repo has the same problem.

I have fixed the compiling script issue, thanks!
I have double checked the dll and so file, the ncat functions are there.
Maybe it was because your local file was not updated?

I recompiled the dll file yesterday with the scripts in the repo, that's the reason why those ncat functions are not in my dll.
Thanks for the quick fix.


using weaveworks/net-plugin in swarm mode

I am using docker in swarm  mode. I use weaveworks/net-plugin and enable multicast as follows
I did the following step to deply my service in swarm mode with cote and weaveworks/net-plugin.

step one: i run the following commands in each node

docker plugin install weaveworks/net-plugin:latest_release &&
docker plugin disable weaveworks/net-plugin:latest_release &&
docker plugin set weaveworks/net-plugin:latest_release WEAVE_MULTICAST=1 &&
docker plugin enable weaveworks/net-plugin:latest_release

step two: I ran my compose file this my file

version: '3'

networks:
  cote:
    # driver: "overlay"
    driver: "weaveworks/net-plugin:latest_release"
services:
  api-service:
    image: aaami1ster/taxi:base
    deploy:
      replicas: 1
      restart_policy:
        condition: on-failure
    environment:
      - NODE_ENV=production
      - TZ=Africa/Cairo
    networks:
      - cote
    command: node api-service/bin/www
    ports:
      - "3100:3100"
my-service:
    image: aaami1ster/taxi:base
    deploy:
      replicas: 1
      restart_policy:
        condition: on-failure
    environment:
      - NODE_ENV=production
      - TZ=Africa/Cairo
    networks:
      - cote
    command: node my-service/app.js

I enable multicast, but still not work. what am I missing?????

Have you tried to include
driver_opts:
    WEAVE_MULTICAST: 1

https://docs.docker.com/compose/compose-file/#network-configuration-reference
It might work to explicitly set the driver opts params on the stack.


Write protection error when using devmode installer.

Describe the bug
When using the Devmode installer, there will be an error complaining about having write protect on somewhere along the installation.
For whatever reason, rebooting the Chromebook will make it act as a successfully murkmodded Chromebook, but some things are missing.
However, if you run the Installer command again, it will work correctly without error.
To Reproduce
Steps to reproduce the behavior:

Open the terminal with CTRL+ALT+ ->
Log in as root and run the installer command
Do everything it says and wait.
See error

Expected behavior
The Chromebook should install murkmod successfully the first time.
Screenshots

System (please complete the following information):

Board [Dedede]
Version [v117]
Installation method used [devmode installer]

Additional context
-When rebooting after error, murkmod will be missing plugins, emergency reverting, and various other things will be missing or broken.
-This does not actually mess up installation if you just run the installer again, but does mess it up if you only reboot.

During installation, the installer tries to set your GBB flags to 0x8091 - it should just continue regardless if write-protect is enabled on your flash chip, but clearly it is not. This is indeed not the intended behavior. You're right, though - most preliminary assets will be installed to your system, and the rest can be repaired by rerunning the installer (through the update shortcut in mush).


How to install it manually?

Hey, I have question, how to install this lib manually (without installing from Git)?
I have tried to move all the files to the Unity Assets folder but it show me bunch of errors.
Am i missing something?

clone, drag and drop into assets folder, this is the manual way


CNF-11236: Apply extra manifests in the order specified by the apply-wave annotation

Sort, group by and apply extra manifests with respect to apply-wave
annotation
Refactor sort, group by and load grouped manifests to use generics to
remove code duplications
Signed-off-by: Saeid Askari<EMAIL_ADDRESS>

/retest

/unhold

/test security

/retest

/lgtm

/approve

/override ci/prow/integration


Can't define method correctly

In my WAT it's like this:
(import "go" "debug" (func (;0;) (type 1)))
My Java code does this:
                var module = "go"; 		var func = WasmFunctions.wrap(_store, WasmValType.I32, i -> { System.out.println(i); }); 		var extern = Extern.fromFunc(func); 		_linker.define(module, "debug", extern);
My result is:
Exception in thread "main" io.github.kawamuray.wasmtime.WasmtimeException: incompatible import type for go::debug 	at io.github.kawamuray.wasmtime.Linker.nativeModule(Native Method) 	at io.github.kawamuray.wasmtime.Linker.module(Linker.java:21)
If I don't define it:
Exception in thread "main" io.github.kawamuray.wasmtime.WasmtimeException: unknown import: go::debug has not been defined 	at io.github.kawamuray.wasmtime.Linker.nativeModule(Native Method) 	at io.github.kawamuray.wasmtime.Linker.module(Linker.java:21)
How to solve?

Can you share the function signature declared in your WAT which is referred by (type 1)?

I'm not sure where to find it.
Maybe: (type (;1;) (func (param i32))) ?
I'll attach my hello world example.
main.wasm.zip

H-m. I've tried similar example as below, but it worked perfectly well. Could it possibly an issue of wasmtime itself? You may wanna confirm by writing simple rust code that directly uses wasmtime to execute your wat.
Also, providing your full java code might helps.
hello_test.wasm
(module
  (type (;0;) (func (param i32)))
  (type (;1;) (func))
  (import "go" "debug" (func $_ZN10hello_test5debug17hee89981947308813E (type 0)))
  (func $debug_entry (type 0) (param i32)
    local.get 0
    call $_ZN10hello_test5debug17hee89981947308813E
    return)
  (func $dummy (type 1))
  (func $__wasm_call_dtors (type 1)
    call $dummy
    call $dummy)
  (func $debug_entry.command_export (type 0) (param i32)
    local.get 0
    call $debug_entry
    call $__wasm_call_dtors)
  (table (;0;) 1 1 funcref)
  (memory (;0;) 16)
  (global (;0;) (mut i32) (i32.const 1048576))
  (global (;1;) i32 (i32.const 1048576))
  (global (;2;) i32 (i32.const 1048576))
  (export "memory" (memory 0))
  (export "__heap_base" (global 1))
  (export "__data_end" (global 2))
  (export "debug_entry" (func $debug_entry.command_export)))

public class HelloWorld {
    public static void main(String[] args) {
        try (Store<Void> store = Store.withoutData()) {
            try (Engine engine = store.engine();
                 Module module = Module.fromFile(engine, "./hello_test.wasm");
                 Func debugFunc = WasmFunctions.wrap(store, I32, n -> {
                     System.err.println("Debug: " + n);
                 });
                 Linker linker = new Linker(store.engine())) {
                linker.define("go", "debug", Extern.fromFunc(debugFunc));
                linker.module(store, "", module);
                try (Func f = linker.get(store, "", "debug_entry").get().func()) {
                    WasmFunctions.Consumer1<Integer> fn = WasmFunctions.consumer(store, f, I32);
                    fn.accept(123);
                }
            }
        }
    }
}

Initially, I used "new Instance" like in examples/src/main/java/examples/Gcd.java ...
Now I'm using only the Linker without Instance, like in your example now or in
examples/src/main/java/examples/MemoryInterop.java ...
And it works. 16 Go examples are running fine, so thanks for writing a Java wasmtime again.


Fix HTML streaming

This usually goes unnoticed, as

HTTP2 streaming usually requires HTTPS, which typically isn't enabled on localhost, and
the API requests in our examples are quite fast

But at some point I seem to have broken the out-of-order streaming, such that everything resolves at once.

Closed by #475.


[Flaky Test] tests/python/relay/opencl_texture/test_network.py::test_mobilenet_v1_fp32[opencl -device=adreno]

These tests were found to be flaky (intermittently failing on main or failed in a PR with unrelated changes). See the docs for details.
Tests(s)

tests/python/relay/opencl_texture/test_network.py::test_mobilenet_v1_fp32[opencl -device=adreno]

Jenkins Links

https://ci.tlcpack.ai/job/tvm/job/PR-13441/1/display/redirect

close for being a duplicate as #13443


Error: This method cannot be used with flat config. Add your entries directly into the config array.

I'm using eslint v9 and I got that error

Thank you. Closing in favor of #1589


Created pytest core for mac

Based on mloubout's test install branch

Depends on 1068 so merge that before this.

yask does not support OS X so remove that one, not sure about ops (@FabioLuporini)
You probably don't need a new yaml file but you can add cases to the existing pytest-core

not sure about ops

ops is fine for now as there is no actual offloading (yet)

You probably don't need a new yaml file but you can add cases to the existing pytest-core

agreed

@navjotk @mloubout @ggorman : do we actually need to run tests on the mac build ? isn't it not just enough to check the installation ? perhaps run test_operator only ?
I think running the entire test suite is overkill

I think running the entire test suite is overkill

I agree, only thing needed on OSX is to make sure it's installed and compiler works with it so a single test should be fine.


Unable to configure if you disable notifications.

The first thing I did was disable notifications to get rid of the persistent notification (I know, its necessary).  But after doing so you are unable to configure the app as when you launch it either turns it on or off.  The only way to configure is to press the notification...if the user chooses to hide the notification there is no way to configure after the first run.
If the pixel filter is on and you open pixel filter app, it should go to the configuration screen, not turn it off.

I've copied this behavior from the Screen Filter app. I guess I'll just add
an option to disable notification to the Pixel Filter itself, and then show
config as you describe.
On Jun 21, 2015 1:27 AM, "mercianary"<EMAIL_ADDRESS>wrote:

The first thing I did was disable notifications to get rid of the
persistent notification (I know, its necessary). But after doing so you are
unable to configure the app as when you launch it either turns it on or
off. The only way to configure is to press the notification...if the user
chooses to hide the notification there is no way to configure after the
first run.
If the pixel filter is on and you open pixel filter app, it should go to
the configuration screen, not turn it off.
—
Reply to this email directly or view it on GitHub
https://github.com/pelya/screen-dimmer-pixel-filter/issues/1.

Fixed.


Zkillboard overview in UI itself

Currently its just a number, no info about the ship or value
The zkillboard feed looks like this in pathfinder, something equivalent (and optional again) would be awesome

If you enter this, please take into account that these additional windows are optional (you can completely remove them). The current zipper tag is beautiful. Otherwise, it will overload the visual interface.

something equivalent (and optional again) would be awesome

Like I said myself, this should be an optional feature.


Use jboss parent 19 & newer checkstyle

Core - Full Integration Build 1113 is now running using a merge of aa4857b532678ec9342b38f590faefff3b420b23

Linux Build 1973 is now running using a merge of aa4857b532678ec9342b38f590faefff3b420b23

Windows Build 1523 is now running using a merge of aa4857b532678ec9342b38f590faefff3b420b23

Windows Build 1523 outcome was SUCCESS using a merge of aa4857b532678ec9342b38f590faefff3b420b23
Summary: Tests passed: 3207, ignored: 71 Build time: 0:22:28

Linux Build 1973 outcome was SUCCESS using a merge of aa4857b532678ec9342b38f590faefff3b420b23
Summary: Tests passed: 3207, ignored: 71 Build time: 0:25:15

Core - Full Integration Build 1113 outcome was FAILURE using a merge of aa4857b532678ec9342b38f590faefff3b420b23
Summary: Tests failed: 1 (1 new), passed: 2877, ignored: 377 Build time: 0:48:16
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.web.authentication.FormAuthenticationWebFailoverTestCase(SYNC-tcp).test: <no details avaliable>

retest this please

Windows Build 1529 is now running using a merge of aa4857b532678ec9342b38f590faefff3b420b23

Core - Full Integration Build 1119 is now running using a merge of aa4857b532678ec9342b38f590faefff3b420b23

Linux Build 1979 is now running using a merge of aa4857b532678ec9342b38f590faefff3b420b23

Windows Build 1529 outcome was SUCCESS using a merge of aa4857b532678ec9342b38f590faefff3b420b23
Summary: Tests passed: 3207, ignored: 71 Build time: 0:22:26

Linux Build 1979 outcome was SUCCESS using a merge of aa4857b532678ec9342b38f590faefff3b420b23
Summary: Tests passed: 3207, ignored: 71 Build time: 0:25:07

Core - Full Integration Build 1119 outcome was SUCCESS using a merge of aa4857b532678ec9342b38f590faefff3b420b23
Summary: Tests passed: 2877, ignored: 377, muted: 1 Build time: 0:48:43


allow users to configure basic authentication credentials

re #26
Sorry, I had to open a new PR because I wanted to change the branch on my fork.

Hi Joe, sorry for the delay, I will pick this back up as soon as possible.

@joemasilotti Here are the results of my research. Please have a look and let me know how you think we should proceed 🙇
UIWebView
Here is how I tested the UIWebView case:
In the SceneDelegate I prepared a view controller with a web view and pointed the web view to the basic auth protected website.
let vc = UIViewController()
let view = vc.view!
let webView = WKWebView()

webView.load(URLRequest(url: URL(string: "https://jigsaw.w3.org/HTTP/Basic/")!))
view.addSubview(webView)
webView.translatesAutoresizingMaskIntoConstraints = false
webView.topAnchor.constraint(equalTo: view.topAnchor).isActive = true
webView.bottomAnchor.constraint(equalTo: view.bottomAnchor).isActive = true
webView.leadingAnchor.constraint(equalTo: view.leadingAnchor).isActive = true
webView.trailingAnchor.constraint(equalTo: view.trailingAnchor).isActive = true

window.makeKeyAndVisible()
window.rootViewController = vc

Case 1
Default behavior (Before setting the webview's navigation delegate and implementing navigation challenge)
Results:
The web view loads the page, receives a basic auth 401 and shows the response body.

Case 2
After setting the webview's navigation delegate and implementing navigation challenge.
// code above +
webView.navigationDelegate = self

// and implement relevant method:
extension SceneDelegate: WKNavigationDelegate {
  func webView(_ webView: WKWebView, didReceive challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {
    completionHandler(.performDefaultHandling, nil)
  }
}

Results: Same as before.
Case 3
Leaving the implementation blank
extension SceneDelegate: WKNavigationDelegate {
  func webView(_ webView: WKWebView, didReceive challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {
    // nothing
  }
}

Results: App crashes with following error:
Thread 1: "Completion handler passed to -[sma_shin_ios___debug_dev.SceneDelegate webView:didReceiveAuthenticationChallenge:completionHandler:] was not called"

Case 4
Passing the correct credentials
extension SceneDelegate: WKNavigationDelegate {
  func webView(_ webView: WKWebView, didReceive challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {
    completionHandler(.performDefaultHandling, nil)
  }
}

Results: Auth challenge succeeds and http request resolves successfully

TurboNavigator
Case 1:
You implement the callback and don't do anything:
func session(
    _ session: Session, didReceiveAuthenticationChallenge challenge: URLAuthenticationChallenge,
    completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void
  ) {
      // do nothing..
  }

Result: App crashes with this error:
Thread 1: "Completion handler passed to -[Turbo.ColdBootVisit webView:didReceiveAuthenticationChallenge:completionHandler:] was not called"

Case 2.
You implement the callback and handle the callback with "performDefaultHandling" without providing credentials:
func session(
    _ session: Session, didReceiveAuthenticationChallenge challenge: URLAuthenticationChallenge,
    completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void
  ) {
      completionHandler(.performDefaultHandling, nil)
  }

Result:  The request is treated as 401 and enters error handling logic
func session(_ session: Session, didFailRequestForVisitable visitable: Visitable, error: Error) {
  if let turboError = error as? TurboError, case let .http(statusCode) = turboError {
    if statusCode == 401 { // ENTERS THIS CODE PATH
      let authURL = Application.url.appendingPathComponent("/users/sign_in")
      turboNavigator.route(authURL) // infinite loop
    }
  }
}

Based on the logic in my app, this just resulted in an infinite loop.
Note The exact same thing happens when using
completionHandler(.useCredential, nil)

Case 3.
You implement the callback and handle the callback with ".cancelAuthenticationChallenge" without providing credentials:
func session(
    _ session: Session, didReceiveAuthenticationChallenge challenge: URLAuthenticationChallenge,
    completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void
  ) {
      completionHandler(.cancelAuthenticationChallenge, nil)
  }

Black screen of death but at least there is no infinite loop 😄
Conclusion
When the authentication challenge fails, it gets treated as any other 401 and there is no way to detect that the 401 resulted from a failed authentication challenge once we are in didFailRequestForVisitable. This is problematic because if you send the user to ${baseURL}/user/sign_in which is also under basic auth, then it will result in an infinite loop.
Ideally, Turbo would detect that the www-authenticate:  is present, and either include that info in TurboError so we can treat it differently than other 401.
here -> https://github.com/hotwired/turbo-ios/blob/main/Source/Visit/ColdBootVisit.swift#L81
But until that is established, it might be better to throw an error to let the user know that we received an http challenge, and that they have to configure the credentials. (Quick feedback)

Here is another proposal: https://github.com/joemasilotti/TurboNavigator/pull/29/files
I am just trying out different approaches. Please let me know your thoughts @joemasilotti 🙇

Thank for the detailed write up!
I don't think that Turbo Navigator should make any assumptions about how someone might code their authentication logic. What works for you or me might break something for an existing app relying on basic HTTP auth.
I think the best way to handle this is replicate what turbo-ios is already doing: expose a similar method in the delegate and provide a sensible default implementation. In this scenario, calling the completion block performing default handling.
This means that if someone doesn't implement the TurboNavigationDelegate method then the default iOS behavior will occur. And if they implement it then they can do whatever they want.
One thing I'm not sure about yet is if the delegate should expose the web view, Turbo Session, or neither.
public protocol TurboNavigationDelegate {
    func didReceiveAuthenticationChallenge(_ challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void)
}

public extension TurboNavigationDelegate {
    func didReceiveAuthenticationChallenge(_ challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {
        completionHandler(.performDefaultHandling, nil)
    }
}

extension TurboNavigator: SessionDelegate {
    public func session(_ session: Session, didReceiveAuthenticationChallenge challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {
        delegate?.didReceiveAuthenticationChallenge(challenge, completionHandler: completionHandler)
    }
}

@joemasilotti Sorry for the delay,  I updated the PR following the implementation you suggested. I also confirmed that this works in my own iOS app. As a user of the library, my application code looks like this:
extension Tab: TurboNavigationDelegate {

  // ...

  func didReceiveAuthenticationChallenge(_ challenge: URLAuthenticationChallenge, completionHandler: @escaping (URLSession.AuthChallengeDisposition, URLCredential?) -> Void) {
    completionHandler(.useCredential, Application.stagingBasicAuthCredentials)
  }
}

Please let me know if this PR is good to go.

One thing I'm not sure about yet is if the delegate should expose the web view, Turbo Session, or neither.

I thought I needed access to the web view when I was implementing javascript messaging last week but in the end, I was able to get access to it through these:
self.turboNavigator.modalNavigationController.topViewController.visitableView.webView
self.turboNavigator.navigationController.topViewController.visitableView.webView


Update go-acme/lego to v4.3.1

What does this PR do?
Update go-acme/lego to v4.3.1
Motivation
Fixes:

[dnsprovider] exoscale: fix dependency.

related to https://github.com/exoscale/egoscale/issues/482
More

[ ] Added/updated tests
[ ] Added/updated documentation

Any chance of a release incorporating this change?  We could do with the fix from https://github.com/go-acme/lego/issues/1357 being available.


Fix nested union test expectations

I updated the expectations of the "union > nested" test, as I think they are incorrect.
My understanding is that unionAll is supposed to have inner join semantics, similar to forEach. This means that if an expression is not matched, the resource should be dropped from the result.

This change is actually not right, after taking another look at the processing model I think the original test case is fine.
See: https://chat.fhir.org/#narrow/stream/179219-analytics-on-FHIR/topic/Nested.20union.20test.20expectations
I've updated our implementation and am withdrawing this.


Publish and document the model and its codegen for coupling/decoupling ergoCub wrist.

Following the creation of the wrist controller and its integration in the ems fw, we want to publish and document the model and its generated fw for coupling/decoupling of eCub wrist.
Please see also this integration activity.

Here it is the PR:
https://github.com/robotology/icub-firmware-models/pull/22
The folder also contains a README.md about the mathematics of the spherical joint.

Thank you @ale-git!
I'll be reviewing the PR asap.

Done via:

https://github.com/robotology/icub-firmware-models/pull/26


client.publish make wdt reset with using topic in variable

Hello, thank you for really great library for mqtt protocol.
One of versions (sorry i don't remember which is) i use String variables for topics. In last update you removed using strings and require using char massive.
Ok, i rebuild my code to work with char* but one problem is critical.
I using command>answer logic system on my code (for special requests) and that means i send command to Sometopic/Device-1/System request like "?" and receive some device info in this topic.
That means function callback is called and receive three params. After i try to recognize which topic is used for triggering function and answer in callback using "topic" chars for publich, but when i use this .
void callback(const char* topic, byte* payload, unsigned int length) {
  aS.p("<i> Message arrived [").p(topic).p("] ");
  for (int i = 0; i < length; i++) {
    aS.p((char)payload[i]);
    aS.pln(i);
  }
  Serial.println();
  char exPayload = (char)payload[0];
  if (String(topic) == remotes){ //remotes var is Sometopic/Device-1/Executebles as char massive [30]
    if(exPayload == '?'){
      snprintf (msg, 50, "RELAY1=%d:%d", relayState[0], relayState[1]);
      aS.pln(topic);
      aS.pln(client.state());
      client.publish(topic, msg); //problem here if i use topic variable (pointer?!)
      // client.publish("Sometopic/Device-1/Executables", msg); //works fine
    }
}

aS.pln - its defined function for serial print in advancedSerial lib
I added for test Serial outputs after each command and program stops in client.publish line
if I use a clear indication of a topic like "Sometopic/Device-1/Executables" everything works fine
If i use topic variable where i receive it as function parameter, esp going to reboot by WDT reset.
I try to print contained data in "topic" and connection state as you see in code. Everything is fine, topic correct and connection is 0 (connected)
So many changes in build sketch with debug outputs of everything and enable exeptions do not provide any information why this happens. It's just wdt reset while try publish.
Can you help me? Or give me true direction how to make this work.

Bad idea to publish something inside callback. Better copy received data to global var buffer in callback, leave callback and process this data in loop. Also, you are publishing to topic on which you are subscribed - so you will receive sent msg (at first I thought that it`s will be recursion, but it's not - callback will be triggered only inside client.loop() function).
But I still don't know why you code crashes. Could you please provide minimum sketch to reproduce this?

In this library input and output data stored in same buffer, so it wouldn't work properly if you are use pointer to input topic as output topic. If you don't want move publishing outside of callback, you can copy topic content to local variable and use this variable in publishing.
//You can use arduino strings
String topicToPublish = String (topic);
...
client.publish(topicToPublish.c_str(), msg);

//or C strings (char array)
char topicToPublish [strlen(topic)+1];  //one extra place for string termination symbol
strcpy (topicToPublish, topic);  //copies string to topicToPublish from topic
...
client.publish(topicToPublish, msg);

Chose one of this ways

Thank you very much! I am really frustrated before, because this feeling everything works fine and after some updates it breaks now, maybe i need change my command answer logic as you told me (device  received own answer, and it's just ignore, but this is a parasite traffic).
I tried your advice's, and it's work perfect to me.
I was a little uncertainty, whether necessary to understand is why the device freezes. Since the problem was solved with your help.

You are welcome).

@saloid
I see how you help others by doing a good thing. Some people really appreciate it. I would like to thank you at least a little. Perhaps this is unacceptable, but if you are interested, let me know. The address is in the profile.


[ENH] Framework for train accuracy estimates in time series classifiers

Several time series classification algorithms can efficiently produce estimates of accuracy (or probabilities for other metrics) on the training data. A few examples include:

Out of bag error in interval-based forests.
Leave one out cross-validation leveraging the nearest neighbour classifiers and already transformed data in dictionary-based approaches.
Removing the redundancy of transforming time series multiple times for cross-validation in algorithms such as STC and ROCKET.

While these are generally useful for evaluation classifier performance, our main usage is generating weights for the HIVE-COTE classifier.
Currently there is no framework/structure for producing these, or determining if a classifier even can produce its own estimates. As far as I am aware the only classifiers which can do so currently are a few of the dictionary-based ones I have implemented such as BOSS (method below). However, as we improve the sktime HIVE-COTE implementation, each component should be able to do this ideally.
    def _get_train_probs(self, X):
        num_inst = X.shape[0]
        results = np.zeros((num_inst, self.n_classes))
        divisor = np.ones(self.n_classes) * self.n_estimators
        for i in range(num_inst):
            sums = np.zeros(self.n_classes)

            preds = Parallel(n_jobs=self.n_jobs)(
                delayed(clf._train_predict)(
                    i,
                )
                for clf in self.classifiers
            )

            for c in preds:
                sums[self.class_dictionary.get(c, -1)] += 1

            dists = sums / divisor
            for n in range(self.n_classes):
                results[i][n] = dists[n]

        return results

An algorithm may have multiple methods of producing an estimate also. For example, TDE can produce both out of bag and leave one out estimates. This one is more minor (and prehaps should be handled by individual classifiers), but I will include for discussion. Perhaps an inheritable class would be the way to go about this? The following follows the functionality of how we handle this in our Java package.
  class EstimateMethods(Enum):
      Default = 1
      OOB = 2
      CV = 3

  class TrainProbabilityEstimator():
      def __init__(self):
          self.estimate_method = EstimateMethods.Default

      def _get_train_probs(self, X):
          raise NotImplementedError("abstract method")

Though this is probably overkill, it coud also just be a standard name for such methods.

for context, in tsml this is controlled through the base class EnhancedAbstractClassifier
https://github.com/uea-machine-learning/tsml/blob/master/src/main/java/tsml/classifiers/EnhancedAbstractClassifier.java
not sure why we didnt just use an interface, but Im sure there were reasons.
For python, in the first instance, I suggest that we use the existence of the method
def _get_train_probs(self, X):
as proof that the classifier can produce the estimates, then in hive cote either use this method if it exists or cross validate if it does not. OOB is the method we should default to. Slightly less hacky way, this can be added to the Capabilities flags and tested in the unit testst, then HC can just check these. I think on reflection this is better

Very interesting thought! (somewhat late to the party here)
Personally, I would:

distinguish methods that produce in-sample predictions (e.g., CV-like procedures) from methods that produce some in-sample loss estimates (e.g., oob error estimates)
introduce specific methods for this, e.g., predict_CV, predict_proba_CV or score_oob, signature to be determined
if multiple options exist to produce either, this should be a hyperparamter set in constructor, e.g., predict_oob_strategy (which has a default and can be one of multiple strings)
if the algorithm allows arbitrary losses, this should be able to take arbitrary sklearn metrics for classification; these may be arguments to score_oob etc, or the constructor if already needed there.

Evaluation using in-sample predictions or out-of-bage estimates would be a "special" evaluation method.
What do you think, @mathhewmiddlehurst, @tonybagnall, @mloning?

@MatthewMiddlehurst this functionality is now present, can we close this issue?

I think the only thing that changed is the tag correct? The current setup is fine for my usage, but unsure how visible the functionality will be for others. Should probably change the function names to something better at least.

should this perhaps be documented with the base class then?


[Bug Report] 执行报错

登录后按照 Wiki 查询最新成绩，报错如下
>>> print(zxw.get_self_mark())
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.9/site-packages/zhixuewang/student/student.py", line 221, in get_self_mark
    exam = self.get_exam(exam_data)
  File "/usr/lib/python3.9/site-packages/zhixuewang/student/student.py", line 109, in get_exam
    return self.get_latest_exam()
  File "/usr/lib/python3.9/site-packages/zhixuewang/student/student.py", line 151, in get_latest_exam
    exams = self.__get_page_exam(1)
  File "/usr/lib/python3.9/site-packages/zhixuewang/student/student.py", line 134, in __get_page_exam
    json_data = r.json()
  File "/usr/lib/python3.9/site-packages/requests/models.py", line 900, in json
    return complexjson.loads(self.text, **kwargs)
  File "/usr/lib/python3.9/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.9/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.9/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

https://github.com/anwenhu/zhixuewang-python/pull/32
我已经对此修复

https://github.com/anwenhu/zhixuewang-python/pull/32
我已经对此修复


Fix: line pull request formatter

@abr2001 can you add more details on why this was required, preferably with a test or a description how to trigger the scenario you're trying to fix?

Any update, @abr2001?


Add EnvObserver and WritableFileObserver

These interface types can be plugged into an Env via NewObservedEnv().
A number of hooks are added to facilitate Go-side callbacks from
C++ call contexts.

This CR turned into a bit of a rabbit hole because my C++ comfort level is lacking.  Comments inline

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions, all commit checks successful.

env.go, line 1 [r1] (raw file):
It makes me a little uneasy that I don't see test coverage here.  I can't gauge the effort vs impact of putting something together given that this repo uses different testing methods, C/C++ crossover, and we probably won't be touching it a whole lot, but I'll leave "add testing" as a suggestion.
Alternate strategy: maybe add some low level unit tests in our workspace repo?

env.go, line 115 [r1] (raw file):
Would it not be cleaner to write this function (and all following) as func (env envObserserverHandle) gorocksdb_env_* and avoid having to pass state?

hooked_env.cpp, line 8 [r1] (raw file):
Is cgo_export.h supposed to be in the gorocksdb root?  Might be some syntax I'm missing here.

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 3 unresolved discussions, all commit checks successful.

env.go, line 1 [r1] (raw file):
This comment was prior to me seeing recorder_test.go... feel a little better :)

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 3 unresolved discussions, all commit checks successful.

env.go, line 1 [r1] (raw file):
Yep, this is a problem. The best I've come up with is basically black-box round trip testing: building a database with known inputs while recording it, and then playing it back & confirming the expected database state. That I absolutely intend to write as part of this overall work.
Unit-level testing here is tough because there's not much which can be asserted about individual operations of EnvObserver, except that they get called (sometimes). How and when is, by design, a black box. Suggestions welcome.

env.go, line 115 [r1] (raw file):
These functions are actually the hand-off from C to Go. The wonky signature is so that C code, which knows nothing about Go types or closures, can call them.

hooked_env.cpp, line 8 [r1] (raw file):
_cgo_export.h is automatically generated by cgo

Comments from the review on Reviewable.io

LGTM

Review status: all files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

env.go, line 1 [r1] (raw file):
I'm on board with black box style testing.  We can always revisit adding unit tests here if we decide to add more to this fork.

Comments from the review on Reviewable.io


wrong place to post

I know, this is the wrong place to post.
But I am looking for @mapero. Since Jochen Scheib is also the developer of "bashGroup / node-red-contrib-fritz" but no one reacts on my issue placed there.
I would apreciate someone could tick Jochen on his shoulder!

I will get in contact with you


Bindings with shift and numerals not working.

Steps to Reproduce:
Add a binding with Shift and some numeral to the config. For example:
bind=SHIFTSUPER,1,movetoworkspace,1
Expected Outcome
The binding should execute. And in this case move to workspace 1.
Noted Outcome
It doesn't work.

Does SUPERSHIFT work?

maybe read the wiki before posting an issue that has been described in it.
https://github.com/hyprwm/Hyprland/wiki/Advanced-config#binds

Oh sorry my bad.


form:input in foreach generates the same ID (as for 1st elem)

form:input in foreach generates the same ID (as for 1st elem)

correct, make sure to define ID manually via iterator


Allow autowiring for Symfony >= 3.3 and Symfony 4 migration

Since Symfony 3.3, services can be autowired. This is a huge (and welcome) change, but it requires some changes from bundles.
Also, some of the features used by this bundle have been removed from Symfony 4. So it can also be seen as an upgrade to Symfony 4.
Would it be possible to make this change? I'm not sure how to proceed because I don't know much about reusable bundles, but please tell me if I can help!

I think there are two things here:
a) autowiring - publicly published bundles aren't encouraged to use this, although we may want to enable it (by making it easier to locate and inject an address renderer, say, when performing autowiring elsewhere). Do you have opinions about how this should happen?
b) specifically which are the features you've identified here that are removed from Symfony 4? (I'm not in disagreement, but would just like to be clearer about what we are referring to here!)

I'm not sure, but I think that declaring services via yaml with an alias as a key has been removed and that keys must be class names.
But there are so many changes that I'm not sure what's mandatory and what's only best practice.

@shieldo I'm (finally) migrating my application to Symfony 4, but this bundle has a hard dependency to the symfony validator that is incompatible with Symfony 4.
I don't think the changes in this component affect your bundle. Would you be willing to go from
"symfony/validator": "~2.7|~3.0",

to
"symfony/validator": "~2.7|~3.0|~4.0",

?

@mcorteel-harel I'm in the process of getting this set up, and making some extra service declarations to facilitate autowiring.

Awesome, thanks!
I guess after that you can close this issue.

@mcorteel-harel I've made a PR though haven't had the opportunity to functionally test yet. https://github.com/usemarkup/AddressingBundle/pull/18


Check if any flavors are missing

I know for example IHO is already missing:

And we need to add them to the Software section:

And soon CCSDS.

@ronaldtse I can't understand how to correctly add new flavors to the site. For example, I've added relaton-callconect to the _config.yml:
    ...
    - id: relaton-calconnect
      label: CalConnect
      gem: relaton-calconnect
      sources:
        - { url: "https://www.calconnect.org", label: "CalConnect" }
    ...

Then I created _software/relaton-calconnect.adoc and copied _software/relaton-3gpp/assets/symbol.svg to _software/relaton-calconnect/assets/symbol.svg. After generating the site Jekyll creates '_software/relaton-calconnect/.git` dir. When I tried to stage changes I got:
$ git add .
warning: adding embedded git repository: _software/relaton-calconnect
hint: You've added another git repository inside your current repository.
hint: Clones of the outer repository will not contain the contents of
hint: the embedded repository and will not know how to obtain it.
hint: If you meant to add a submodule, use:
hint:
hint:   git submodule add <url> _software/relaton-calconnect
hint:
hint: If you added this path by mistake, you can remove it from the
hint: index with:
hint:
hint:   git rm --cached _software/relaton-calconnect
hint:
hint: See "git help submodule" for more information.
warning: adding embedded git repository: _software/relaton-ccsds

I didn't find any submodules for other flavors and decided not to add it for relaton-calconnect. I removed the _software/relaton-calconnect/.git dir and pushed the changes. But now the _software/relaton-calconnect/assets/symbol.svg isn't shown on the flavor page:

Can you give me a direction to address this issue?

@andrew2net perhaps you have committed some files that were generated during build. I recommend running make clean before committing changes to the source. This is due to some quirks of how Jekyll Open Project theme works.

Broken submodules should definitely  be removed, I see a few that return HTTP 404…

I've made the fix for adding flavor icons in #52.

I confirm all the new flavors have been added.


[Event Request]  InitLastDocDate COD12

I would like to request an event on function InitLastDocDate in COD12. We have a transaction no. that makes sense to be inited inside the standard function.

If you could provide the event signature and position, it would be helpful. Please refer to any other event request for examples.

Sorry Jesper,
Bellow you can find the standard Function with our new line:
LOCAL InitLastDocDate(GenJnlLine : Record "Gen. Journal Line") WITH GenJnlLine DO BEGIN   LastDocType := "Document Type";   LastDocNo := "Document No.";   LastDate := "Posting Date";   **LastTempTransNo := "Transaction No."; //New Line** END;
In order to suite us, we needed a onAfterInitLastDocDate or a OnBeforeLastInitLastDocDate event with (GenJnlLine, var LastTempTransNo) as parameters.

We already have global variable NextTransactionNo. I'd suggest you think how to rework you solution using existing COD12 and events instead of converting you legacy code 1:1 using events. You can finally get better solution.


TAP implementation changes expected offset

Different implementations of TAP have different expected offsets.
Examples:

TAP Name
Expected Result
Offsets observed
Travel observed
Travel-to-offset

CL CNC Tap
0.0-1mm
-0.15, -0.042
0.0740
* 2

TAP R6
0.5-2mm
-1.308

TAP R8
0.5-2mm
-1.260, -1.320
0.0270
* 10 + 1

Mellow Tap
0.5-2mm
-1.725

Vitalii MetalTap
0.5-2mm
-0.865
0.500

Thanks to the people below for providing these numbers

RedWish, Migo, Sanaa, EightFinger

Observed results
v2 Stock RC8 Results (Redwish), expected ~1.260
Samples: 5, Total Steps: 27
Probe Mean: 0.0038 / Min: 0.0013 / Max: 0.0050
Measure Mean: 0.0308 / Min: 0.0308 / Max: 0.0308
Travel Mean: 0.0270 / Min: 0.0250 / Max: 0.0300

v1 Vitalli MetalTap (EightFinger), expected ~0.865
Samples: 10, total Steps: 100
Probe Mean: -0.0235 / Min: -0.0356 / Max: -0.0069
Measure Mean: 0.0265 / Min: 0.0265 / Max: 0.0265
Travel Mean: 0.0500 / Min: 0.0500 / Max: 0.0500
Calculated STA Z-Offset: 0.052

I think what you might be measuring is the linear stickyness. What about if you 1) pulled the toolhead away quickly, and 2) added a slight pause for the toolhead to sink back down on the linear rail

@The-Original-Reth I'll give this a try. I think since I switched the probe_speed to 1 I've gotten lower travel distance - and a lower calculated offset.

Have run Auto_Tap a few times now, and the calculated Z-offset on my R8 Tap is always ~1.310.
This ends up with Z0 about .6mm too high. When I manually set the z_offset, the nozzle just touching the bed ends up at 0.611
18:19:12
Auto TAP performing 10 samples to calculate z-offset on R8 tap
Probe Min: -2.0, Stop: 2.0, Step: 0.0025
18:19:43
Auto TAP sample 1
Traveled: 0.0300 from z-0.5335 to -0.5035 on step 12
18:20:00
Auto TAP sample 2
Traveled: 0.0300 from z-0.5335 to -0.5035 on step 12
18:20:18
Auto TAP sample 3
Traveled: 0.0300 from z-0.5335 to -0.5035 on step 12
18:20:35
Auto TAP sample 4
Traveled: 0.0300 from z-0.5335 to -0.5035 on step 12
18:20:53
Auto TAP sample 5
Traveled: 0.0300 from z-0.5329 to -0.5029 on step 12
18:21:10
Auto TAP sample 6
Traveled: 0.0300 from z-0.5335 to -0.5035 on step 12
18:21:28
Auto TAP sample 7
Traveled: 0.0325 from z-0.5341 to -0.5016 on step 13
18:21:47
Auto TAP sample 8
Traveled: 0.0325 from z-0.5335 to -0.5010 on step 13
18:22:05
Auto TAP sample 9
Traveled: 0.0325 from z-0.5341 to -0.5016 on step 13
18:22:23
Auto TAP sample 10
Traveled: 0.0325 from z-0.5341 to -0.5016 on step 13
18:22:23
Auto TAP Results
Samples: 10, Total Steps: 124
Probe Mean: -0.5336 / Min: -0.5341 / Max: -0.5329
Measure Mean: -0.5026 / Min: -0.5026 / Max: -0.5026
Travel Mean: 0.0310 / Min: 0.0300 / Max: 0.0325
Calculated z-offset on R8 tap: 1.310

## [probe]
## z_offset = -0.611

@ieRev Could you try running it again on the newest version? Your results vs other ones I've seen are... frightening for this being able to work on R8. Thank you for posting!

Updated to newest version, then set Z_offset to 0.00 in printer.cfg, homed & QGL'ed before running:
18:01:32
Auto TAP performing 10 samples to calculate z-offset on R8 tap using 'rev hop' method
Probe Min: -2.0, Stop: 2.0, Step: 0.0025
18:02:01
Auto TAP sample 1
Traveled: 0.0275 from z0.0663 to 0.0938 on step 11
18:02:18
Auto TAP sample 2
Traveled: 0.0275 from z0.0675 to 0.0950 on step 11
18:02:34
Auto TAP sample 3
Traveled: 0.0275 from z0.0656 to 0.0931 on step 11
18:02:50
Auto TAP sample 4
Traveled: 0.0275 from z0.0650 to 0.0925 on step 11
18:03:07
Auto TAP sample 5
Traveled: 0.0275 from z0.0644 to 0.0919 on step 11
18:03:23
Auto TAP sample 6
Traveled: 0.0275 from z0.0644 to 0.0919 on step 11
18:03:40
Auto TAP sample 7
Traveled: 0.0275 from z0.0650 to 0.0925 on step 11
18:03:56
Auto TAP sample 8
Traveled: 0.0275 from z0.0644 to 0.0919 on step 11
18:04:12
Auto TAP sample 9
Traveled: 0.0275 from z0.0644 to 0.0919 on step 11
18:04:29
Auto TAP sample 10
Traveled: 0.0275 from z0.0644 to 0.0919 on step 11
18:04:29
Auto TAP Results on R8 using 'rev hop' method
Samples: 10, Total Steps: 110
Probe Mean: 0.0651 / Min: 0.0644 / Max: 0.0675
Measure Mean: 0.0926 / Min: 0.0926 / Max: 0.0926
Travel Mean: 0.0275 / Min: 0.0275 / Max: 0.0275
Calculated z-offset: 1.210

After running, Saved Config, rehomed/QGL and did G0 Z0.. Nozzle tip was 0.6mm from the bed
This was pretty much exactly the same distance it was high by yesterday. I will be starting a (short) print and will update with how much offset is needed once it starts

Hope this might help  :)
V2.4R2
Auto TAP Results on R8 using 'rev hop' method
Samples: 5, Total Steps: 25
Probe Mean: 0.0013 / Min: 0.0000 / Max: 0.0025
Measure Mean: 0.0263 / Min: 0.0263 / Max: 0.0263
Travel Mean: 0.0250 / Min: 0.0250 / Max: 0.0250
Calculated z-offset: 1.249
12:54
Auto TAP sample 5
Traveled: 0.0250 from z0.0025 to 0.0275 on step 5
12:54
Auto TAP sample 4
Traveled: 0.0250 from z0.0025 to 0.0275 on step 5
12:54
Auto TAP sample 3
Traveled: 0.0250 from z0.0000 to 0.0250 on step 5
12:54
Auto TAP sample 2
Traveled: 0.0250 from z0.0000 to 0.0250 on step 5
12:54
Auto TAP sample 1
Traveled: 0.0250 from z0.0013 to 0.0263 on step 5
12:53
Auto TAP performing 5 samples to calculate z-offset on R8 tap using 'rev hop' method
Probe Min: -1.0, Stop: 2.0, Step: 0.005
I ended up having to adjust .44 to get a good first layer.

Adding to the info. Voron 2.4R2 350mm running StealthBurner, CW2 and Tap R8 (with 4 magnets).
 Auto TAP Results on R8 using 'rev hop' method Samples: 3, Total Steps: 12 Probe Mean: -0.0033 / Min: -0.0038 / Max: -0.0025 Measure Mean: 0.0167 / Min: 0.0167 / Max: 0.0167 Travel Mean: 0.0200 / Min: 0.0200 / Max: 0.0200 Calculated z-offset: **1.203** Auto TAP sample 3 Traveled: 0.0200 from z-0.0025 to 0.0175 on step 4 Auto TAP sample 2 Traveled: 0.0200 from z-0.0038 to 0.0162 on step 4 Auto TAP sample 1 Traveled: 0.0200 from z-0.0038 to 0.0162 on step 4 Auto TAP performing 3 samples to calculate z-offset on R8 tap using 'rev hop' method Probe Min: -2, Stop: 2.0, Step: 0.005
Actual offset after babystepping in-print and saving using Z_APPLY_OFFSET_PROBE
#*# #*# [probe] #*# z_offset = -0.950

Voron 2.4R2 350mm, CW2 and SB R8 with 4 magnets, umbilical here:
19:24
Offset does not match expected result. Expected between 0.70-2.00, Got: 3.274
19:24
Auto TAP Results on R8 using 'rev hop' method
Samples: 3, Total Steps: 54
Probe Mean: -1.3742 / Min: -1.3750 / Max: -1.3737
Measure Mean: -1.2842 / Min: -1.2842 / Max: -1.2842
Travel Mean: 0.0900 / Min: 0.0900 / Max: 0.0900
Calculated z-offset: 3.274
19:24
Auto TAP sample 3
Traveled: 0.0900 from z-1.3737 to -1.2837 on step 18
19:24
Auto TAP sample 2
Traveled: 0.0900 from z-1.3750 to -1.2850 on step 18
19:24
Auto TAP sample 1
Traveled: 0.0900 from z-1.3737 to -1.2837 on step 18
19:23
Auto TAP performing 3 samples to calculate z-offset on R8 tap using 'rev hop' method
Probe Min: -2, Stop: 2.0, Step: 0.005

when setting manually I use z_offset = -1.380 (or close to that value)

Voron 2.4R2 300mm, CW2 & SB with Tap R8 using 4 magnets
22:42:03
$ AUTO_TAP FORCE=1
22:42:03
// Auto TAP performing 3 samples to calculate z-offset on R8 tap using 'rev hop' method
// Probe Min: -2, Stop: 2.0, Step: 0.005
22:42:16
// Auto TAP sample 1
// Traveled: 0.0300 from z0.0050 to 0.0350 on step 6
22:42:22
// Auto TAP sample 2
// Traveled: 0.0300 from z0.0038 to 0.0338 on step 6
22:42:29
// Auto TAP sample 3
// Traveled: 0.0300 from z0.0038 to 0.0338 on step 6
22:42:29
// Auto TAP Results on R8 using 'rev hop' method
// Samples: 3, Total Steps: 18
// Probe Mean: 0.0042 / Min: 0.0038 / Max: 0.0050
// Measure Mean: 0.0342 / Min: 0.0342 / Max: 0.0342
// Travel Mean: 0.0300 / Min: 0.0300 / Max: 0.0300
// Calculated z-offset: 1.296

Normal offset is around -1.09 so about 0.2mm difference

Voron 2.4R2 350mm; CW2 & Stealthburner. Using Tap R8 with 4 magnets and a Revo Micro
$ AUTO_TAP
14:59:52
// Auto TAP performing 3 samples to calculate z-offset on R8 tap using 'rev hop' method
// Probe Min: -2, Stop: 2.0, Step: 0.005
14:59:52
// Homing Z
15:00:41
// Auto TAP sample 1
// Traveled: 0.2150 from z0.0075 to 0.2225 on step 43
15:01:18
// Auto TAP sample 2
// Traveled: 0.2050 from z0.0075 to 0.2125 on step 41
15:01:55
// Auto TAP sample 3
// Traveled: 0.2000 from z0.0075 to 0.2075 on step 40
15:01:55
// Auto TAP Results on R8 using 'rev hop' method
// Samples: 3, Total Steps: 124
// Probe Mean: 0.0075 / Min: 0.0075 / Max: 0.0075
// Measure Mean: 0.2142 / Min: 0.2142 / Max: 0.2142
// Travel Mean: 0.2067 / Min: 0.2000 / Max: 0.2150
// Calculated z-offset: 3.059
15:01:55
!! Offset does not match expected result. Expected between 0.70-2.00, Got: 3.059```

Also using a kinematic bed mount with probe location at `310, 310`. Could be incompatible?

My usual offset is `-0.9645`

Voron 2.4R2 250mm; CW2 + Stealthburner on Clone Vitalli CNC TAP. z_offset set to 0

AUTO_TAP
Auto TAP performing 3 samples to calculate z-offset on VITALII_CNC tap using 'rev hop' method

Probe Min: -2, Stop: 2.0, Step: 0.005

Auto TAP sample 1

Traveled: 0.0200 from z-0.0013 to 0.0187 on step 4

Auto TAP sample 2

Traveled: 0.0200 from z-0.0013 to 0.0187 on step 4

Auto TAP sample 3

Traveled: 0.0200 from z-0.0025 to 0.0175 on step 4

Auto TAP Results on VITALII_CNC using 'rev hop' method

Samples: 3, Total Steps: 12
Probe Mean: -0.0017 / Min: -0.0025 / Max: -0.0013
Measure Mean: 0.0183 / Min: 0.0183 / Max: 0.0183
Travel Mean: 0.0200 / Min: 0.0200 / Max: 0.0200
Calculated z-offset: 0.432

Offset does not match expected result. Expected between 0.50-1.50, Got: 0.432

Auto TAP Results on CL_CNC using 'rev hop' method
Samples: 5, Total Steps: 168
Probe Mean: -0.3150 / Min: -0.3212 / Max: -0.3087
Measure Mean: -0.1470 / Min: -0.1470 / Max: -0.1470
Travel Mean: 0.1680 / Min: 0.1650 / Max: 0.1700
Calculated z-offset: 0.651
Auto TAP sample 5
Traveled: 0.1700 from z-0.3212 to -0.1512 on step 34
Auto TAP sample 4
Traveled: 0.1700 from z-0.3200 to -0.1500 on step 34
Auto TAP sample 3
Traveled: 0.1650 from z-0.3150 to -0.1500 on step 33
Auto TAP sample 2
Traveled: 0.1700 from z-0.3100 to -0.1400 on step 34
Auto TAP sample 1
Traveled: 0.1650 from z-0.3087 to -0.1437 on step 33
Auto TAP performing 5 samples to calculate z-offset on CL_CNC tap using 'rev hop' method
Probe Min: -2, Stop: 2.0, Step: 0.005

Chaotic Labs Tap (v1), Stealthburner and Galileo 2 Extruder, Voron 2.4, probed after Homing and QGL, the offset was way off, it's more like, -0.310 and not 0.651


Thread usage in AcquireStep

This is mainly about the Semaphore code of the project, I haven't tested the other tasks: The plug-in is using the common ForkJoinPool for background jobs, however the size of the pool is limited (by default) to approx. the number of CPUs , which leads (in case one has a more threads waiting for semaphores than CPU) to deadlocks or low throughput (if all threads are waiting to acquire semaphores, but no-one releases one resp. if most threads are waiting but just a few are free to trigger the actual Pipeline steps - see my PR). The limitation also leads to situations where a job acquiring semaphore A affects are totally different job acquiring semaphore B, just because there are no threads left actually waiting for the semaphore.
The limitation of the ForkJoinPool is fine for CPU-bound tasks, but the tasks executed by the plug-in are not CPU-bound, they're just waiting for semaphores resp. for the step to finish.
One option to solve this is to increase the size of the ForkJoinPool or use a dedicated pool per Jenkins job. Another option would be the following:
Do not acquire semaphores in dedicated Futures, but try to use a single thread that more or less polls a single or multiple semaphores each few hundred ms and launches (on success) dedicated executor threads (not bound to a pool or at least bound to a pool sufficiently sized) triggering the Pipeline steps. This will limit the number of threads waiting for semaphores to one and makes sure acquire steps always execute. The first part could be achieved by a custom thread, that simply takes requests (consisting out of semaphore, count, optional timeout and Runnable to start if successful) via a queue, iterates over the queue every few hundred ms, tries to acquire a semaphore with a zero or very short timeout and launches the task, or by scheduled CompletableFutures, that do the same but reschedule themselves to be executed again in a few hundred ms if needed. Concerning the second part I'd avoid in any case running the body invoker in the common ForkJoinPool to make sure they can always be executed and can release the semaphore again, no matter whether the pool is exhausted or not - I'd either use dedicated threads, a custom pool (if possible resizeable without the need to restart Jenkins) or check whether there is a better way to run the body invokers asynchronously without wasting a thread waiting here.

fixed #6 and #9


test: add integration test

Closes:

https://github.com/lucacasonato/fresh/issues/104

Notes:

https://github.com/lucacasonato/fresh/pull/140#pullrequestreview-915155162

Have a tests/cli_test.ts file
Have a Deno.test("fresh init", async () => { ... });
Create a temp dir with Deno.createTempDir
Run deno -A --no-check cli.ts init <tmp> using Deno.run and await the process finishing
Check all the necessary files exist
Run deno run -A main.ts on the generated output with tmp path as CWD
Wait until Listening on ... is printed by subprocess
fetch the address to check that the project is functional
Kill the subprocess & wait for it to exit

The test seems to be hanging.


Create max-tree-depth.rb (Max Depth of Binary Tree)

#hacktoberfest

Hey @CraigP17 !
Excellent.
You can star this repository for further updates.
Have fun  🙌🏼


Move awarding bounty into wizard flow

Progenitors see solutions on proposal page
Can select a number to "Award Bounty" for the solution
Can free enter a bounty amount to reward

https://docs.google.com/spreadsheets/d/1STKLjP9PIvBt7NwyN9s4wJF-2xPpvUEIxzWKU3u1LRc/edit#gid=0

Solution: Try stuff
http://swarmbot.io
1: Vote Up
2: Send Reward
x: Exit
To take an action, simply enter the number or letter at the beginning of the line.

2

Enter reward amount to send to Duke for the solution "Try Stuff":

1000

Sending reward!
PM to solution creator:
You have received a reward of amount 1000 for your solution LINK


samples: Remove CONFIG_CPLUSPLUS

This causes the following build errors:
/__w/gsoc-2022-arduino-core/gsoc-2022-arduino-core/Arduino-Zephyr-API/samples/i2cdemo/prj.conf:2: warning: attempt to assign the value 'y' to the undefined symbol CPLUSPLUS
Hence, remove it from all samples as it is no longer needed.

Merging this ASAP, because it's breaking CI. We can always drop later if any objections.


Puzzling result when modifying GeoSIR example

I aim to use Mesa, and potentially Mesa-geo, to model virus transmission within a building (specifically, a hospital). So I tried substituting a geojson file derived from a polygon Shapefile representing the internal layout of a building, for the file of Toronto neighbourhoods. I attach the zipped file below (the building is a university building in Chicago, the Edward D Anderson Building, as I don't yet have the hospital floor plans).
The model appears to run successfully, but the visualization does not work properly. A map appears, and I can zoom in on the building. No internal structure or agents show up - perhaps not surprising given the source of the "Leaflet" maps - but the steps counter does not appear, and the chart has the wrong Y-axis, and nothing appears on it as the model runs. I have (accidentally) discovered that running the original model within a Jupyter notebook, preceded by the command:
%matplotlib widget
reproduces the same faulty visualization.
I can't see any feature of the geojson file that differs in any obvious way from the original. Using a different alternative - a map of postal districts in part of London - does not cause the same problem. Using a geopandas GeoDataFrame, or the Shapefile itself, gives the same result as using the geojson file. Removing "map_element" from the list of visualization elements used to define "server" results in the steps counter and correct chart appearing.
Any ideas what is happening? Thanks.
Nick Gotts<EMAIL_ADDRESS>- new work email)
Rooms2_fixed_4326.zip

Clarification: I did of course change the MAP_COORDS and unique_id parameters appropriately.

Clarification: I did of course change the MAP_COORDS and unique_id parameters appropriately.

OK, resolved (not by me). The problem appears to have been that some of the properties in my geojson file had the value null; changing them to "null" worked.

OK, resolved (not by me). The problem appears to have been that some of the properties in my geojson file had the value null; changing them to "null" worked.

Glad you got it working! Feel free to open more issues if you run into more issues

Glad you got it working! Feel free to open more issues if you run into more issues


Supply typescript definitions

This makes the library usable out of the box in a typescript project.
Without these definitions, users need to handcraft their own definitions, at least this:
type DeviceToken = string

type Listener<T> = (data: T) => void

declare module "pushy-electron" {
    export function listen(): void
    export function register(opts: { appId: string }): Promise<DeviceToken>
    export function setNotificationListener<T>(listener: Listener<T>): void
}

@tadaskay Thanks a lot for this! Merged and published in 1.0.7 💯


Simplify by using viper.SetEnvKeyReplacer

Instead of handling the mapping of env keys to config keys explicitly, we can use vipers SetEnvKeyReplacer to do the mapping for us, simplifying bindFlags.
Passes the tests.

Thanks for the fix! This is much more straightforward 👍


Add note about post-test

Added note that do-while loop is considered a post-test loop!

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@ClarkTravis
Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality >standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


could not parse max_hits_to_report as int64

I am guessing this error is because the input nimble config omits this param, so it cant parse null into int?
thread 'main' panicked at 'Error -- could not parse max_hits_to_report as int64', src/reference_library.rs:53:10
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Can either python or rust be made more tolerant to missing params (maybe default values)?

fixed with https://github.com/BimberLab/nimble/commit/98a13abd81b9496a60d366c51d03a7b9f9bf7e90


Adding Longest Increasing Subsequence Algorithm in C,C++ and Java

Description
We have given an array arr[] of size N, we need to find the length of the Longest Increasing Subsequence (LIS) i.e., the longest possible subsequence in which the elements of the subsequence are sorted in increasing order.
Describe the solution you'd like
I would like to implement the algorithm in C,C++ and Java
It is one of the most popular algorithm which is missing in the repo. Kindly allow me to work on this issue under SSOC'23

Hey @dhruti26 please assign me this issue to work on and resolve it under ssoc'23 program.

Hey @Kumar-laxmi please assign this issue to me so i can work on this issue under ssoc'23 program.

@Kumar-laxmi please assign me this issue as it was raised by me and so that I can submit my PR

@Kumar-laxmi please assign this issue to me so that i can start working on this asap.

I can solve these issue in all three languages. Kindly assign me this issue.

@Kumar-laxmi Please assign me this issue under SSOC'23


Translation - French

Still in progress. I will let you know when it's done.

closing due to inactivity.
if you want to re open, please make against the develop branch


Failed to connect to the remote extension host server (Error: Unknown Error occured)

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

VS Code Version: 1.66.2
OS Version: Windows_NT x64 10.0.18363

Steps to Reproduce:
Note: This error message is the same for different Data Scientists / Data Engineers working in the same team but with possible different laptops and Windows versions, extensions, which makes us think the switch from 1.65.2 to 1.66.2 is causing this issue.
When updating VS Code from 1.65.2 to 1.66.2, VS Code is no longer able to connect to instances in Azure Machine Learning Studio.
Error messages are as follows:
Failed to connect to the remote extension host server (Error: Unknown Error occurred)

request to https://***********.francecentral.instances.azureml.ms/api/terminals?****** failed, reason: certificate signature failure

[Extension Host] Unhandled Error: ENOPRO: No file system provider found for resource 'vscode-remote://amlext***/home/azureuser/***'    at p.withProvider (vscode-file://vscode-app/c:/Users/*******/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:610:27776)    at processTicksAndRejections (node:internal/process/task_queues:96:5)    at async p.exists (vscode-file://vscode-app/c:/Users/********/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:610:30371)    at async Promise.all (index 0)    at async vscode-file://vscode-app/c:/Users/********/AppData/Local/Programs/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2110:2110
D @ console.ts:137

Please file this on https://github.com/microsoft/vscode-tools-for-ai/issues


[PE-D][Tester C] list treatment message ends with a comma

Labels: type.FunctionalityBug severity.VeryLow
original: lilozz2/ped#3

#169

This is classified as a cosmetic flaw and will be worked on in future implementations


Add mach_port_construct and the required types.

@JohnTitor pinging in case this got lost

Sorry, LGTM!


Add auto-publish to PyPi

Migrated from GitLab: https://gitlab.com/meltano/sdk/-/issues/89
Originally created by @aaronsteers on 2021-04-01 23:46:42

View 3 previous comments from the original issue on GitLab


[Bug]: When reading the content from the configuration file specified by the --config parameter, the parameter type was not considered.

Your current environment

This bug is environment-independent.

Model Input Dumps
No response
🐛 Describe the bug
The code in the method '_load_config_file' of vllm/vllm/utils.py which reads the config file from the parameter --config has a bug.
# only expecting a flat dictionary of atomic types
processed_args: List[str] = []

config: Dict[str, Union[int, str]] = {}
try:
    with open(file_path, 'r') as config_file:
        config = yaml.safe_load(config_file)
except Exception as ex:
    logger.error(
        "Unable to read the config file at %s. \
        Make sure path is correct", file_path)
    raise ex

for key, value in config.items():
    processed_args.append('--' + key)
    processed_args.append(str(value))

return processed_args

The code here simply spans the key-value pairs in the config file. So, if I want to store a 'store_true' parameter like '--trust-remote-code', I cannot put it in the config file.
For example, I had test 'trust-remote-code:',  'trust-remote-code: ""', 'trust-remote-code: yes', 'trust-remote-code: true'. When the key-value pairs were spanned, the cli parameters will be like 'vllm server model_name --trust-remote-code true'. Then an error be like saying 'there is no parameter named "true"' will happened.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Hi, I'd like to contribute to this project. Can I take up this issue?

Hey!! Happy to help this issue.

Sorry for the late reply. Just open a PR once you're ready, don't need to ask!


Problem of Maximizing

When I click the "maximize" button, the task bar disappeared.

On it.

This issue is fixed by @Tempz, merged in #15.
Thanks for reporting, closing this now.


Error with loading the saved quantized model

Describe the bug
I am using the following code to save the "TheBloke/WizardLM-13B-V1.1-GPTQ" locally and then reload it from the disk, but in loading time I am getting this error:  ValueError: QuantLinear() does not have a parameter or a buffer named weight.
Hardware details
RAM: 40GB
GPU: 2* RTX 3090
CPU: 24 * AMD
Software version
CUDA Version: 12.0
python: 3.10.12
pytorch: 2.0.1+cu117
transformers: 4.31.0
auto_gptq:  0.5.0.dev0
To Reproduce
Steps to reproduce the behavior:

Run the following code

import torch
import os
from transformers import AutoTokenizer
from auto_gptq import AutoGPTQForCausalLM

def load_gptq_model(model_name, trust_remote_code=True,
                     quantize_config=None,inject_fused_attention=True):
    name = model_name.split('/')[-1]
    if name in os.listdir('./models'):
         model = AutoGPTQForCausalLM.from_quantized('./models/'+name,
                                                    use_safetensors=True,
                                                    trust_remote_code=trust_remote_code,
                                                    use_triton=False,
                                                    use_cuda_fp16=False,
                                                    quantize_config=quantize_config,
                                                    inject_fused_attention=inject_fused_attention,
                                                    device ='cuda:0',
                                                    device_map='auto',
                                                    strict=False)

         tokenizer = AutoTokenizer.from_pretrained('./models/'+name,use_fast=True)
         return model, tokenizer
    model = AutoGPTQForCausalLM.from_quantized(
        model_name,
        torch_dtype=torch.float16,
        use_safetensors=True,
        trust_remote_code=trust_remote_code,
        use_triton=False,
        use_cuda_fp16=False,
        quantize_config=quantize_config,
        inject_fused_attention=inject_fused_attention,
        device ='cuda:0',
        device_map='auto'
    )
    tokenizer = AutoTokenizer.from_pretrained(model_name,use_fast=True)
    model.save_quantized('./models/'+name,use_safetensors=True)
    tokenizer.save_pretrained('./models/'+name)
    return model, tokenizer
model_name = "TheBloke/WizardLM-13B-V1.1-GPTQ"
model, tokenizer = load_gptq_model(model_name)

See the error

Traceback (most recent call last):
File "/home/student/Experiemnts/SharedTask2023/Notebooks/o.py", line 42, in
model, tokenizer = load_gptq_model(model_name)
File "/home/student/Experiemnts/SharedTask2023/Notebooks/o.py", line 10, in load_gptq_model
model = AutoGPTQForCausalLM.from_quantized('./models/'+name,
File "/home/student/Experiemnts/SharedTask2023/.venv/lib/python3.10/site-packages/auto_gptq/modeling/auto.py", line 108, in from_quantized
return quant_func(
File "/home/student/Experiemnts/SharedTask2023/.venv/lib/python3.10/site-packages/auto_gptq/modeling/_base.py", line 890, in from_quantized
accelerate.utils.modeling.load_checkpoint_in_model(
File "/home/student/Experiemnts/SharedTask2023/.venv/lib/python3.10/site-packages/accelerate/utils/modeling.py", line 1336, in load_checkpoint_in_model
set_module_tensor_to_device(
File "/home/student/Experiemnts/SharedTask2023/.venv/lib/python3.10/site-packages/accelerate/utils/modeling.py", line 262, in set_module_tensor_to_device
raise ValueError(f"{module} does not have a parameter or a buffer named {tensor_name}.")
ValueError: QuantLinear() does not have a parameter or a buffer named weight.
Expected behavior
I expect the model to be loaded without any error.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Some lines of code are out of the  code box!

Hi! Can you also downgrade to auto-gptq 0.4.2 to see if this problem still exists?

Same problem here. Reproduced in 0.4.2. @PanQiWei

Hi, I can not reproduce the issue on AutoGPTQ main with accelerate 0.22.0, 0.23.0, 0.24.0. Until the next release, please upgrade accelerate and build the main branch from source.
Using:
import torch
import os
from transformers import AutoTokenizer
from auto_gptq import AutoGPTQForCausalLM

model = AutoGPTQForCausalLM.from_quantized("/fsx/felix/WizardLM-13B-V1.1-GPTQ",
                                        use_safetensors=True,
                                        use_triton=True,  # no issue either with use_triton=False
                                        use_cuda_fp16=False,
                                        inject_fused_attention=True,
                                        device_map='auto',
                                        strict=False,
                                        disable_exllamav2=True,  # no issue either with disable_exllamav2=False
                                        disable_exllama=True)  # no issue either with disable_exllama=False

tokenizer = AutoTokenizer.from_pretrained("/fsx/felix/WizardLM-13B-V1.1-GPTQ", use_fast=True)

Feel free to open a new issue with a reproduction and your environment if the issue persists.

Installing auto-gptq compatible with cuda version (https://github.com/PanQiWei/AutoGPTQ#installation) solved this problem for me


Sidebar filters and sidebar tag badges don't co-exist

I had the same issue. I noticed that when removing my sidebar filters in manager.ts, it did show up.
I have a sidebar filter in manager.ts so it won't display my stories with tag 'chromatic-only'
manager.ts
import { addons } from '@storybook/manager-api';
addons.setConfig({
  sidebar: {
    filters: {
      patterns: (item) => {
        // in the sidebar exclude all items with the tag 'chromatic-only'
        return !item.tags?.includes('chromatic-only');
      },
    },
  },
});

Originally posted by @Fedduh in https://github.com/Sidnioulz/storybook-addon-tag-badges/issues/15#issuecomment-2482298338

Hi @Fedduh, sadly I could not reproduce this issue.
I've added the same sidebar filter config you're using on my local storybook instance on the main branch. Could you please run it and confirm you see the sidebar badges?
Could you please share a repo where sidebar filters trigger the issue so I can investigate it?

It was a bit of a hassle to setup a repo (my own repo is private).
I used some AI to set it up, but it does demostrate the issue. Perhaps it's cause by the Angular version of Storybook?
This is the stackblitz:
https://stackblitz.com/edit/sb1-kd55we?file=.storybook%2Fmanager.ts
If you run it (npm i && npm start) it will show the badge in the sidebar, and on top

When you

comment out the code in './storybook/manager.ts' (the sidebar property)
restart the application (stop current and npm start)

It will only show on top.

Hope this helps you debugging. I could indeed not reproduce the issue in your main branch.

I suspect that addons.setConfig({ sidebar }) erases the whole sidebar object, including renderLabel.
Could you try to keep your filters and add the following?
// .storybook/manager.ts
import { addons } from '@storybook/manager-api'
import { renderLabel } from 'storybook-addon-tag-badges'

const filters = { /* ... */ };

addons.setConfig({
  sidebar: {
    filters,
    renderLabel,
  }
})

That indeed fixes it! Thanks :)

Yay! Thanks for reporting back. I'll add a more explicit use case in the README so it's easier to spot.


.map files included in published wwwroot/dist/

I was quite stunned for a second that I found the map files included in my published ReactReduxSPA.
The publishOptions exclude entry in project.json seems to be ignored since the move to the new (old) .pubxml file.
  "publishOptions": {
    "include": [
      "appsettings.json",
      "ClientApp/dist",
      "Views",
      "web.config",
      "wwwroot"
    ],
    "exclude": [
      "wwwroot/dist/*.map"
    ]
  },

I tried to add something like the following to my CustomProfile.pubxml:
    <ExcludeFilesFromDeployment>
      wwwroot\dist\*.map
    </ExcludeFilesFromDeployment>

(or wwwroot/dist/*.map ...) but so far was out of luck.
The map files are older than the other files in dist but they are copied nonetheless.
For now I'm manually deleting the map files but an automatic solution would be preferred ;)
PS: BTW - thanks for a great start template! I've rebuilt my complete website as a universal ReactReduxSPA ;) (www.carecom.de)

The project.json tooling is being deprecated, so we probably won't add any further enhancements to those files, or even any noncritical fixes. Everything is moving to .csproj-based tooling.
Glad you've got a good solution for your own requirements though!


make test failed: "t.Cleanup undefined"

Output of helm version:
master branch, last commit is https://github.com/helm/helm/commit/467bd49bb0cb0d613e802c738a0e38225eec054a
make test output
WARN [runner] Can't run linter goanalysis_metalinter: deadcode: analysis skipped: errors in package: [/Users/davinli/go/src/helm.sh/helm/v3/cmd/helm/dependency_build_test.go:27:2: could not import helm.sh/helm/v3/pkg/repo/repotest (/Users/davinli/go/src/helm.sh/helm/v3/pkg/repo/repotest/server.go:42:4: t.Cleanup undefined (type *testing.T has no field or method Cleanup))] WARN [runner] Can't run linter unused: buildssa: analysis skipped: errors in package: [/Users/davinli/go/src/helm.sh/helm/v3/pkg/repo/repotest/server.go:42:4: t.Cleanup undefined (type *testing.T has no field or method Cleanup) -: could not load export data: no export data for "helm.sh/helm/v3/pkg/repo/repotest"] ERRO Running error: buildssa: analysis skipped: errors in package: [/Users/davinli/go/src/helm.sh/helm/v3/pkg/repo/repotest/server.go:42:4: t.Cleanup undefined (type *testing.T has no field or method Cleanup) -: could not load export data: no export data for "helm.sh/helm/v3/pkg/repo/repotest"] make: *** [test-style] Error 3
My Go version is 1.13
While T.CleanUp is new in Go v1.14 https://tip.golang.org/pkg/testing/#T.Cleanup,  https://github.com/helm/helm/blob/master/go.mod#L3 is outdated?

Please update to a more recent version of Go. t.Cleanup is a feature introduced in Go 1.14.
See https://github.com/helm/helm/blob/master/.circleci/config.yml#L8
Good catch on the go.mod file. I assume we can update that. But based on the outcome of CI, that field does not appear to affect anything.

@lemonli FYI, Go 1.13 is no longer a supported version of Go by the Go community. Helm should compile against all the supported versions of Go.


feat(prs): pr template 89

Ticket
closes #89
Checklist

[x] There are no warnings in the browser console
[x] I made sure mobile portrait view looks good
[x] I made sure mobile landscape view looks good
[ ] I have added unit test coverage

[x] Unit test coverage not applicable

Additional Notes

@cfernando415 feel free to pull down the repo and add to the template (it won't reflect in this PR's description though, that was copy and paste). it will apply to all future PRs after merge

@cfernando415 feel free to pull down the repo and add to the template (it won't reflect in this PR's description though, that was copy and paste). it will apply to all future PRs after merge


Update Bubblewrap to use the latest Android Build Tools version 30.0.3

The Build Tools version is specified at https://github.com/GoogleChromeLabs/bubblewrap/blob/master/packages/core/src/lib/androidSdk/AndroidSdkTools.ts#L26.
Updating this should be easy enough, but we should ensure everything works correctly on Linux, MacOS, Windows and ChromeOS.

Can you export this const as well as other versions that are used in internal version checks? If someone wants to make a derivation of the project you could then require("AndroidSdkTools").BUILD_TOOLS_VERSION

Yes, this shouldn't be a problem.


fails when asked for diff sample rate

I have a trace I'd like to simplify -- see here gist
When I specify a sample rate other than the input (1s), only two points are returned gist
If I use the 1s input sample rate, I get back something with 100 points that looks pretty good (input is 900+ points) gist
This is a problem because the map matching API seems to expect data samples at 5s intervals.
cc @planemad

@sbma44 saw your comment that the problem was in the exported trace. Is this still open?

Yeah, left this open because I'm still curious about the behavior. My problem was having multiple LineStrings in the submitted FeatureCollection. geojson-tidy produced okay results when minimumTime is set to 1 but not when set to 5.
When the traces are combined, however, the default value (5) works fine. So I guess this is okay to close, though perhaps it would be desirable to throw an error when there's more than one feature?


Makefile improvements

Allow for missing .bsv_tools
Set default for BRAM_SIM to avoid dangling _ in IP name

Thank you for your contribution.