Bloated project size

What can be improved?
Project size is up to 250M. .git/ represents the vast majority of this (224M).
I took a look at the commits using this script, and can see some particularly egregious files lurking in history that we don't need...
642749c02fc3   16MiB examples/data/example_data/hk_zones.geojson
f262b44596a4   17MiB examples/data/example_data/hk_facilities_epsg_4326.geojson
8f019e221b4c   20MiB examples/data/londinium_facilities_epsg_27700.geojson
515bfe0fbc4b   20MiB examples/data/londinium_facilities_epsg_27700.geojson

These are no longer in use. So i think safe to remove from history via BFG.
However, you will need to then need all contributing users to make a fresh clone to prevent pushing them back into history. I am not sure how feasible this is now. But it will only get harder...
Version
v0.3.0

I would extend this to:
7ca30ee07cc6 3694558 examples/AK_test.ipynb
9599fc0cadcd 3826931 examples/AK_test.ipynb
9e529d1a79ac 4149741 examples/04_Example-Create-Population-Londinium.ipynb
d52d29205006 4149748 examples/04_Example-Create-Population-Londinium.ipynb
39965de28c9a 6676924 notebooks/Example-Create-Population-Londinium.ipynb
ca60e0ee8975 6679080 notebooks/Example-Create-Population-Londinium.ipynb
2a93286cbb82 6682550 examples/04_Example-Create-Population-Londinium.ipynb
642749c02fc3 17017366 examples/data/example_data/hk_zones.geojson
f262b44596a4 17455220 examples/data/example_data/hk_facilities_epsg_4326.geojson
8f019e221b4c 20524282 examples/data/londinium_facilities_epsg_27700.geojson
515bfe0fbc4b 20546490 examples/data/londinium_facilities_epsg_27700.geojson

That's ~105MB.
The gh-pages branch is going to continue to bloat the repo. Ideally we could move our documentation to readthedocs.org, but we have had issues in the past with that domain being randomly blocked by the Arup firewall...


Adding Mock Test

Purpose
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the change log
[x] Added tests
[ ] Updated the spec
[ ] Checked native-image compatibility

@PasinduDilmin, shall we close these two PRs(#3, #5) if these are redundant with this #6

https://github.com/ballerina-platform/module-ballerinax-intercom/pull/5
https://github.com/ballerina-platform/module-ballerinax-intercom/pull/3

@PasinduDilmin, I'm closing this PR as it has been inactive for the past 14 days, which exceeds the overdue period for PRs. Please don't hesitate to reopen it when work resumes.


fix: restic removal message

Why the changes were made
Change restic removal message. It wont be remove in OADP 1.4 anymore.
How to test the changes made
Check that all necessary places where modified (I ran grep -Iinr '1\.4' . --exclude=\*.{sum,mod,svg,out} --exclude-dir=developer to help myself).

/hold
1.4 general hold

/LGTM

/retest

/lgtm

/lgtm

/retest

/override ci/prow/4.14-e2e-test-aws
restore succeeds, but application not working flake

/chery-pick oadp-1.4

/cherry-pick oadp-1.4


Added more terms and one correction

Added more definition for terms
Suggested correction to groups to avoid mixing AD and SharePoint group terminology

Nice additions and corrections.


Add VB sample for CA1024

Fixes #22160

Docs Build status updates of commit 6acbbbd:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-UserAccountBuildPermissionInsufficientError] Owner BillWagner no longer has Read permission of the repository https://github.com/dotnet/docs. It might be due to recent GitHub changes on SSO. Please ask the owner BillWagner to logon to Docs Portal https://ops.microsoft.com once again. If it is not the case, please ask anyone of Admin permission of the repo to assign a new owner via Docs Portal https://ops.microsoft.com
[Warning-RepositoryOwnerPermissionInsufficient] Owner BillWagner of repository https://github.com/dotnet/docs has lost Admin permission. Please use Transfer Owner via Docs portal https://ops.microsoft.com to assign a new owner.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@Youssef1313 Thank you for adding the VB example!


add heap header file in dijkstra-fibonacchi-heap

Hey, I think we need to include fibonacchi-heap.cpp here, like https://ei1333.github.io/library/graph/mst/prim-fibonacchi-heap.hpp . Right?

LGTM, Thanks!


Add OrderdProduct module

Add OrderdProduct module with all the features and tests.
Fixes #59
All the classes in models file are temporary except OrderedProduct (These classes didn't merge yet).
Please let me know about missing features and missing tests

since im not exactly sure which basic tests the mentors/maintainers wants for module, the code looks good to me.
also assuming all the base class (user supplier client etc) will be deleted once those classes will be merged.

Why does this PR also creates other modules? is it a WIP? or depends on another PR?

link this PR to an issue
check out :
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

Why does this PR also creates other modules? is it a WIP? or depends on another PR?

Its depends on another PR.

Please rebase your commit on top of upstream/main and fix your migrations because supplier is already merged and deliverylocation is about to be merged also and fix your paths in order look at my review on other PRs,Rawan and Lior

there are methods that do the same as built in methods, no point warping those methods as i see it.

Which methods?

Do note that when the other 2 open PRs will merge, you'll have to rebase and remove the other classes' declarations

@MeitarRizner #38 is merged, solve the conflicts and we'll be able to merge it :)

@MeitarRizner #38 is merged, solve the conflicts and we'll be able to merge it :)

@Omeramsc Done.


[Chip][material] Error when overriding colors

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://codesandbox.io/s/cranky-chatterjee-ls6jm4
Steps:

Extend the @mui/material/Chip module and the ChipPropsColorOverrides interface with an additional color option, tertiary.
Create a custom theme with the createTheme function. Set the primary property of the tertiary color in the theme palette.
Create an App component and wrap it with a ThemeProvider that provides the custom theme created in the previous step.
Inside the App component, use the Chip component with the color prop set to tertiary.
However, the attempt to use the Chip component with the tertiary color fails, resulting in an error. On the other hand, when the same steps are followed with a Button component, there are no errors and it displays correctly.

It seems that there is a problem specific to using the Chip component with the custom color.
Current behavior 😯
The page displays the following error message:
TypeError
Cannot read properties of undefined (reading 'type')

 at decomposeColor (https://ls6jm4.csb.app/node_modules/
mui/system/esm/colorManipulator.js:46:13
    at alpha (https://ls6jm4.csb.app/node_modules/
mui/system/esm/colorManipulator.js:155:11
    at ChipRoot.theme (https://ls6jm4.csb.app/node_modules/
mui/material/Chip/Chip.js:150:131
    at transformedStyleArg (https://ls6jm4.csb.app/node_modules/
mui/system/esm/createStyled.js:159:40
    at handleInterpolation (https://ls6jm4.csb.app/node_modules/
emotion/serialize/dist/emotion-serialize.cjs.dev.js:149:24
    at Object.serializeStyles (https://ls6jm4.csb.app/node_modules/
emotion/serialize/dist/emotion-serialize.cjs.dev.js:274:15
    at eval (https://ls6jm4.csb.app/node_modules/
emotion/styled/base/dist/emotion-styled-base.cjs.dev.js:167:34
    at eval (https://ls6jm4.csb.app/node_modules/
emotion/react/dist/emotion-element-48d2c2e4.cjs.dev.js:63:12
renderWithHooks
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:16305:18
updateForwardRef
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:19226:20
beginWork
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:21636:16
HTMLUnknownElement.callCallback
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:4164:14
Object.invokeGuardedCallbackDev
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:4213:16
invokeGuardedCallback
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:4277:31
beginWork$1
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:27451:7
performUnitOfWork
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:26557:12
workLoopSync
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:26466:5
renderRootSync
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:26434:7
performSyncWorkOnRoot
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:26085:20
flushSyncCallbacks
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:12042:22
flushSync
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:26201:7
Object.scheduleRoot
https://ls6jm4.csb.app/node_modules/react-dom/cjs/react-dom.development.js:27810:5
eval
https://ls6jm4.csb.app/node_modules/react-refresh/cjs/react-refresh-runtime.development.js:280:17
Object.performReactRefresh
https://ls6jm4.csb.app/node_modules/react-refresh/cjs/react-refresh-runtime.development.js:259:25
eval
https://ls6jm4.csb.app/node_modules/csbbust/refresh-helper.js:13:20

Expected behavior 🤔
The chip display with the custom color (here, the tertiary color).
Context 🔦
I have a wide range of colors in my site's palette, and I intend to use them in various components.
Your environment 🌎

npx @mui/envinfo
  Browser: Arc
    Version 0.108.1 (39609)
    Chromium Engine Version 114.0.5735.133
  System:
    OS: Linux 5.15 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
  Binaries:
    Node: 16.17.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 8.15.0 - /usr/local/bin/npm
  Browsers:
    Chrome: Not Found
  npmPackages:
    @emotion/react: latest => 11.11.1
    @emotion/styled: latest => 11.11.0
    @mui/base:  5.0.0-beta.4
    @mui/core-downloads-tracker:  5.13.4
    @mui/material: latest => 5.13.5
    @mui/private-theming:  5.13.1
    @mui/styled-engine:  5.13.2
    @mui/system:  5.13.5
    @mui/types:  7.2.4
    @mui/utils:  5.13.1
    @types/react: latest => 18.2.13
    react: latest => 18.2.0
    react-dom: latest => 18.2.0
    typescript: latest => 5.1.3

Hey @csouchet, thanks for the report!
The root cause for this issue is https://github.com/mui/material-ui/issues/29595. When extending any color that's not primary or secondary, contrastText is required, if not provided this error is thrown. As you can see in this codesandbox, commenting out the contrastText definition reproduces the issue. Providing contrastText should fix your issue.
This is expected but we can improve some things about it:

Improve documentation to make clear that contrastText is required (https://github.com/mui/material-ui/issues/29595)
Improve the error message to make it clear that the missing contrastText is the issue
Provide the internal createPalette util so contrastText can be calculated automatically if the users wishes to, as @siriwatknp suggested in this comment.

I'll start working on these

Hi @DiegoAndai,
Thank you very much for your answer :pray:
Indeed, I didn't see that contrastText is required in the documentation.
It works fine after I add it :slightly_smiling_face:

PR is up: https://github.com/mui/material-ui/pull/37743
Closing as duplicate of #29595, updates will be posted in that issue 😊


Add ImageContentSourcePolicy to operator/v1alpha1

Adding a new operator/v1alpha1 definition caled ImageContentSourcePolicy
which will be used when using a CR to set the registry mirror rules
for resiliency and offline installs/upgrades.
A new CRD will be created to encompass this.
Signed-off-by: Urvashi Mohnani<EMAIL_ADDRESS>

@mrunalp @abhinavdahiya @mtrmac PTAL

Talking with @smarterclayton it seemed like we don't want to add this API object to config/v1 as that has very high guarantee ..
Why not it to mirror.openshift.io v1alpha1 or v1beta1 ?

@abhinavdahiya should I rename the CRD to mirror then as well? Also where is v1alpha1 or v1beta1 in openshift/api? I see a v1alpha1 for operator, so should I create a new directory mirror/v1alpha1 and add this there?

@abhinavdahiya should I rename the CRD to mirror then as well? Also where is v1alpha1 or v1beta1 in openshift/api? I see a v1alpha1 for operator, so should I create a new directory mirror/v1alpha1 and add this there?

I think just moving this type to a new directorymirror/v1beta1 should be sufficient.
also add mirror:v1beta1 to https://github.com/openshift/api/blob/master/hack/update-deepcopy.sh#L13

Or operators.openshift.io (which is our private api) in v1alpha1?
On Jun 18, 2019, at 10:59 AM, Abhinav Dahiya<EMAIL_ADDRESS>wrote:
Talking with @smarterclayton https://github.com/smarterclayton it seemed
like we don't want to add this API object to config/v1 as that has very
high guarantee ..
Why not it to mirror.openshift.io v1alpha1 or v1beta1 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/api/pull/354?email_source=notifications&email_token=AAI37JYM66JDDLWY434UNJLP3DZ5DA5CNFSM4HYZIPIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX65PIA#issuecomment-503175072,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI37J3XJEKXSVRC5SNQZ7DP3DZ5DANCNFSM4HYZIPIA
.

Or operators.openshift.io (which is our private api) in v1alpha1?

+1 for operators/v1apha1

Will the openshift apiserver need to consume this for image pullthrough?
Similarly will builds and imagestreams need to consume this for image references?
fyi @dmage

How it will be coordinated with https://github.com/openshift/api/blob/a77179bc5896d5391a200f8a4578f092f8828178/image/v1/types.go#L241-L253 ?

@abhinavdahiya @smarterclayton moved it operator/v1alpha1

/lgtm

@abhinavdahiya @smarterclayton fixed up based on reviews, PTAL

/lgtm

Re: coordinated with pull policy: we shouldn't have to in the near term,
the local registry is assumed to be HA.  This approach mitigates unreliable
external components.
In the long term, we may want to mitigate the failure modes for local
registry by allowing it to fail "open" for a set of images in the event of
a control plane failure.  Previously this was the "public images" concept
where there is a repository that you can pull any publicly available image
from (i.e. registry.blah.svc/_public) - that would then be listed as a
mirror for all registries.  Nodes would try to pull those first (which
could then bypass auth in the presence of master failures) and so even in
the scenario where control plane is down, we could pull images.
On Thu, Jun 20, 2019 at 9:30 AM Oleg Bulatov<EMAIL_ADDRESS>wrote:

How it will be coordinated with
https://github.com/openshift/api/blob/a77179bc5896d5391a200f8a4578f092f8828178/image/v1/types.go#L241-L253
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/api/pull/354?email_source=notifications&email_token=AAI37J76QU3XMFPHVGXCZTTP3OA7ZA5CNFSM4HYZIPIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYFNKZY#issuecomment-504026471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI37J5W7MMGCX5RUVNZNLDP3OA7ZANCNFSM4HYZIPIA
.

No expectation this is consumed from API level in short term.  Based on
what we learn here, we will design a v2 api in the public config group that
may or may not support additional features.
On Mon, Jun 24, 2019 at 11:02 AM Clayton Coleman<EMAIL_ADDRESS>wrote:

Re: coordinated with pull policy: we shouldn't have to in the near term,
the local registry is assumed to be HA.  This approach mitigates unreliable
external components.
In the long term, we may want to mitigate the failure modes for local
registry by allowing it to fail "open" for a set of images in the event of
a control plane failure.  Previously this was the "public images" concept
where there is a repository that you can pull any publicly available image
from (i.e. registry.blah.svc/_public) - that would then be listed as a
mirror for all registries.  Nodes would try to pull those first (which
could then bypass auth in the presence of master failures) and so even in
the scenario where control plane is down, we could pull images.
On Thu, Jun 20, 2019 at 9:30 AM Oleg Bulatov<EMAIL_ADDRESS>wrote:

How it will be coordinated with
https://github.com/openshift/api/blob/a77179bc5896d5391a200f8a4578f092f8828178/image/v1/types.go#L241-L253
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/api/pull/354?email_source=notifications&email_token=AAI37J76QU3XMFPHVGXCZTTP3OA7ZA5CNFSM4HYZIPIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYFNKZY#issuecomment-504026471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI37J5W7MMGCX5RUVNZNLDP3OA7ZANCNFSM4HYZIPIA
.

@smarterclayton @abhinavdahiya comments addressed, PTAL.

@smarterclayton @abhinavdahiya comments addressed, PTAL.

/lgtm :)

@abhinavdahiya @smarterclayton @mrunalp addressed the comments. If we are good with this, can we get it in.

@abhinavdahiya @smarterclayton can we please get this in so we can continue the work that is relying on this. We can figure out the merging criteria later if that is okay? It would just be a PR to update the comments here.

Sufficient for first round of prototyping (with the known topics still to be resolved for 4.2)
/lgtm
/approve


feat: adds new test users

adds new users to the cluster
makes pwds configurable through env variables

Hi Bartosz, this PR is good to merge.
I edited the branch from master to development. After my local test, I will merge this change from development branch to master.
If you need to use latest changes, you can check out development branch when master hasn't got updated.
Thanks,

@yxun I think we might need cluster-admin for those users - where can I set it up in the scripts?

@bartoszmajsak we haven't wrote cluster-admin user creation step. We can check OpenShift 4 Doc. Either there is a way to create cluster-admin user or there is a way to update a regular user's privilege to a cluser-admin user.

oc adm policy add-cluster-role-to-user cluster-admin "user-name"

This PR has been merged into development and master branches.


Does the pretrained model apply for both GridKNNDownsample and GridDownsample?

Hello, I have a question about the pretrained model.
When I set knn_down to False for Swin3DUNet, the IoU appeared to be pretty low even after loading the pretrained model. Does it apply for both GridKNNDownsample and GridDownsample or only GridKNNDownsample?
If it's the latter, may I ask for a GridDownsample version? Thanks.

Our pretrained model uses only GridKNNDownsample for better segmentation result.
Sorry that we do not have the GridDownsample version, you can pretrain it on Structured3D.

Please feel free to contact us for any further issues.


how to add ipsrv client url image in zoomify

Hello ,
i am doing thing in which i was setup ipsrv server and with the help of this i was get http://localhost/fcgi-bin/iipsrv.fcgi?IIIF=/var/www/html/sample1.tif/info.json  this url but when i adding this url in my openlayer project  so its does not working
please help me to resolve this or suggest any other way so i can add  my custom image in open layer zoomify.
const zoomifyUrl = 'http://localhost/fcgi-bin/iipsrv.fcgi?IIIF=/var/www/html/sample1.tif/';

I cannot see the content of your localhost, but that looks like an IIIF source as in https://openlayers.org/en/main/examples/iiif.html

Hello milke Thanks for your feedback   ,
i am doing same thing mention by you in given IIIF source but in which
my link don't working please once look screen shot
in this screen shot when  i adding my ipsrv link then it  giving error
"can not read data from url"

this screen shot include my data from ipsrv link

please correct me where i  wrong your feeback will be appreciate

CORS access is needed to read data on your localhost from another domain.  You must either enable CORS on your localhost or copy the example to your localhost.

Thanks mike ,
issue resolved


<EMAIL_ADDRESS>does not have BABEL_CONFIG_JS_FILENAME = 'babel.config.js'

🐛 Bug Report
module<EMAIL_ADDRESS>detail: BABEL_CONFIG_JS_FILENAME = 'babel.config.js' is not in build/index.js, so when babel.config.js is modified, but the cache does not update.

See https://github.com/facebook/jest/pull/6911

@thymikee thanks a lot.


Support batch remove when using softDelete() hook

Is there a way to use the softDelete hook for batch requests?
I found out deleting multiple records at once via batch remove does nothing, the workaround is simply making multiple (individual) remove calls.
Steps to reproduce
// DOES NOT WORK: batch/multi-remove by softDelete()
app.service('files').remove(null, params)

// WORKS: less-performant workaround
const files = await app.service('files').find({
  ...params,
  paginate: false
})
await Promise.all(files.map(({ _id ) => app.service('files').remove(_id)))

The softDelete hook is configured as follows:
softDelete({
  // returns query object to spread in    query
  deletedQuery: async (ctx: HookContext) => {
    const param = ctx.params.query?.deletedAt
    const isTruthy = ['true', true].includes(param)
    return { deletedAt: truthy ? { $ne: null } : null}
  },
  // returns patch data to spread in data for remove
  removeData: async (ctx: HookContext) => ({               deletedAt: new Date().toISOString() })
  })

Expected behavior
Files should be set to new Date().toISOString() when deleting one, or multiple in batch.
Actual behavior
Only files deleted individually are set to new Date().toISOString().

I never used the softDelete hook myself. On my setups, feathers-sequelize handles this out of the box.
So I'm not planning to work on that but a PR is highly appreciated.
Please let me know, if you're up to work on this and also contact me, if you need guidance. 😊


About consembleEX

Dear @richardmwaba
Thank you for your work in consemble!
I found out consembleEX from the latest progress of your thesis. It seems a new effective tools!
Can you share the software here?
Best
Johnson

Thanks for reaching out, @Johnsonzcode.
We will make ConSemblEX public soon. Just doing housekeeping 😄.

Thank you so much!


[Replay]: cannot change model for replay mode

EPAM AI DIAL chat version
0.14.0-rc
What happened?
Steps:
1 Create a new chat
2 Create replay chat based on it
3 Click 'Start replay' button
4 Stop replay when part of content is received
5 Open chat settings
6 Try to change model -> model cannot be selected
Expected result: any model/app is selected, settings are displayed

Confidential information

[X] I confirm that do not share any confidential information

The ticket as a duplicate of https://github.com/epam/ai-dial-chat/issues/138
Verified on staging env successfully.


fix insert-key

@nnasaki : Thanks for your contribution to the Azure documentation! The author, @dsk-2015, has been notified to review your proposed change.


Correct the position of the footer

If you have a hint where to look to fix this I can do it

outertheme, the footline template


C1_W2_Lab_1_beyond_hello_world.ipynb Error in tf.keras.activation.softmax

input to softmax function: [1. 3. 4. 2.]
ValueError                                Traceback (most recent call last)
~\AppData\Local\Temp\ipykernel_21880\3343928682.py in
5
6 #feed the  inputs to a softmax activation function
----> 7 outputs = tf.keras.activations.softmax(inputs)
8 print(f'output of softmax function: {outputs.numpy()}')
9
[d:\resoluteai\anaconda\lib\site-packages\tensorflow\python\util\traceback_utils.py](file:///D:/resoluteai/anaconda/lib/site-packages/tensorflow/python/util/traceback_utils.py) in error_handler(*args, **kwargs)
151     except Exception as e:
152       filtered_tb = _process_traceback_frames(e.traceback)
--> 153       raise e.with_traceback(filtered_tb) from None
154     finally:
155       del filtered_tb
[d:\resoluteai\anaconda\lib\site-packages\keras\activations.py](file:///D:/resoluteai/anaconda/lib/site-packages/keras/activations.py) in softmax(x, axis)
90             output = e / s
91     else:
---> 92         raise ValueError(
93             f"Cannot apply softmax to a tensor that is 1D. Received input: {x}"
94         )
ValueError: Cannot apply softmax to a tensor that is 1D. Received input: [1. 3. 4. 2.]

Hi. Unfortunately, I can't reproduce the issue. Next time, you can also report this in our community for better visibility and quicker escalation. The link to join is in Week 1 of your classroom. Thanks!


python-flask for xenial

Could you add utopic, vivid, and wily in the middle too to fill out the definitions? otherwise it looks good.
This packages looks to be on all platforms: http://packages.ubuntu.com/xenial/python-flask

@tfoote updated with your comments. Do not have any installation of those ubuntu versions to test it though :(
Thanks!


Minus not being accepted from keyboard when inputting into a number field :-(

In vki-core.js, line 764, ie.
} else this.VKI_target.value += text;
input of "-", as text, is not being set to input value. In browser console appears following warning:
The specified value "-" is not a valid number. The value must match to the following regular expression: -?(\d+|\d+.\d+|.\d+)([eE][-+]?\d+)?
As a result this.VKI_target.value remains "" and the minus sign has been rejected by the number type input field. So I am not able for example to input a value of -9500.
This regular expression appears to miss at the start [-+] which allows for explicit positive and negative numbers to be entered. I take it that the regular expression and therefore the source of the problem comes from Google as I am using the Chrome browser version 51.0.2704.103. Do you agree?
The problem only seems to happen when i press the angular virtual keyboard "-" key - it just gets ignored - when i use a real keyboard the minus is accepted in the field.

Is anyone there?

Little interest with this issue.. so closing.


Add connection properties for specifying custom TrustManager

This PR adds two new connection properties:

trustManagerClass - The fully qualified class name of a custom javax.net.ssl.TrustManager
trustManagerConstructorArg - An optional argument to pass to the constructor of the TrustManager.
If trustManagerClass is specified and an encrypted connection is requested, the custom TrustManager will be used rather than the default system JVM keystore based TrustManager.

The primary use case for this is to dynamically specify the specific set of certificates that are trusted on a per connection basis without modifying the global settings for the JVM environment (i.e. no need to use a shared / global JKS).
For the error message localization code on line 1803, I've added handling for the case where the caught exception does not provide a localized message. That situation can arise when a custom TrustManager implementation is used as the Driver has no control over that code (it's use specified).
The new code tries to get a localized message, if one is not available then it tries the non-localized message, and if that is not available it defaults to an empty string. The final empty string default is required as subsequent code assumes that all caught exceptions will have a localized error message and would otherwise throw a NullPointerException from within the error handling code.
There's no additional tests for it in the test suite but I've tested it externally using both the specific custom TrustManager we'll be using as well as custom implementation of PermissiveX509TrustManager (which effectively mimics the behavior of trustServerCertificate=true).
See the prior PR submitted to the master branch, #70, for additional commentary and an example use case.

Codecov Report

Merging #74 into dev will decrease coverage by -0.07%.

@@             Coverage Diff             @@
##               dev      #74      +/-   ##
===========================================
- Coverage     27.3%   27.23%   -0.07%
  Complexity    1130     1130
===========================================
  Files           95       95
  Lines        23268    23293      +25
  Branches      3870     3875       +5
===========================================
- Hits          6353     6345       -8
- Misses       15627    15663      +36
+ Partials      1288     1285       -3

Flag
Coverage Δ
Complexity Δ

#JDBC41
27.14% <32%> (-0.05%)
1126 <ø> (+2)

#JDBC42
27.18% <32%> (+0.01%)
1130 <ø> (ø)
:x:

Impacted Files
Coverage Δ
Complexity Δ

...om/microsoft/sqlserver/jdbc/SQLServerResource.java
100% <ø> (ø)
4 <ø> (ø)
:x:

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
30.77% <ø> (-0.84%)
0 <ø> (ø)

.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java
71.33% <100%> (+0.36%)
23 <ø> (ø)
:x:

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
37.57% <50%> (+0.04%)
179 <ø> (ø)
:x:

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
46.03% <ø> (+0.14%)
183% <ø> (ø)
:x:

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
22.99% <ø> (+0.44%)
36% <ø> (ø)
:x:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a58c732...d2c6476. Read the comment docs.

I've rebased this atop the formatted dev branch so it applies cleanly.
Any idea when / if this is going to get merged? It's been in the "Under review" tag for two months.

@sehrope - we are currently considering the PR and trying to understand if there are potential security implications of providing support for third party trust managers.

Why was this closed? I'd still like to get something that provides this type of extensibility added to the driver At the moment it's not possible as you'd have to muck with the local JVM wide keystore.

Sorry @sehrope , I missed your last comment. We removed our previous Dev branch and all PRs created for Dev were closed automatically. We have re-opened the PR. I am sorry for the trouble.
We are looking at this PR, we will get back to you soon. Thank you.

Great. It's been a while since this was put together. I think it still applies cleanly but if not let me know and I'll rebase it.

Hi @sehrope ,
I have been doing some internal testing and your changes look good so far.
Please also make sure that, you add the new connection properties to the SQLServerDataSource too. You can have a look at #422 for guidance.

@sehrope,
I created a PR to your repository to add a few basic verification tests. You should be able to merge it to custom-trust-manager branch. It would also be great if you could add more test cases.

Yes I saw and submitted some feedback on it. Overall looks great!
I'll get that merged, update the naming of clazz/class, and then push a new version. Should have it out some time tomorrow.

To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

Pushed a bunch of formatting and test cleanup related updates.
Ideally I'd like to add a more complicated integration test that validates the server's specific certificate (or CA) but that will have to wait as I don't think the CI infrastructure can support that yet. It'd require changing the startup of the test server to either use an explicit cert or retrieve it prior to the test being run so both the Travis and AppVeyor config would be modified.

To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com. Once you sign the agreement, the cla-required label will automatically be removed.

I signed the CLA a while back but I think this PR predates the setup of the bot so it's likely confused. You may need to manually remove the tag.

Hi @sehrope,
Please reformat your changes to use Microsoft formatter. It can be found here.
I did some more testing using actual certificates and your changes look good. The PR will be merged once we finalize testing. Great contribution 👍

Formatting is complete. To minimize the diff I only ran the formatter on the changes themselves. Surrounding code was not reformatted.
Also, the enum declarations in SQLServerDriver don't completely align anymore because the name of the new one is longer than the rest (line 281). I'd suggest having that corrected in a separate PR as otherwise the diff would clobber any other PRs that are adding new properties.

Awesome. Thanks for merging this in!


Terminal Generator - 31 October (2)

Terminal Generator - 31 October (2)

Works independently but doesn't work as part of the platform. Need to add as a tool within the platform.


Dialog x with resize fix

Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[ ] Caught up with develop branch as of last change.
[ ] Added CHANGELOG entry, or determined not required.
[ ] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[ ] Updated doc comments / prop-types, or determined not required.
[ ] Reviewed and tested on Mobile, or determined not required.
[ ] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

closing this -- when we get the resize change in to develop, will pull down in to the dialogX branch.  Thanks for this colin!


fix(progress-spinner): default strokeWidth to 10% of the diameter

Currently the default stroke of a spinner is always 10px which doesn't look great on anything that's smaller than the default. These changes switch it to be 10% of the circle's diameter.
Fixes #7708.

Caretaker note: Please use extra care when presubmitting, due to recent spinner issues.
CSS: @jelbourn


Improve Nuxt auto-imports to ignore undefined name properties

Environment

Operating System: Linux
Node Version:     v20.12.0
Nuxt Version:     3.10.1
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, imports
Runtime Modules:  -
Build Modules:    -

Reproduction
Reproduction
Describe the bug
When configuring imports in nuxt.config.ts, if the name property is not specified, it should be ignored instead of being set to undefined.
configuration:
imports: {
  imports: [
    { as: '_', from: 'remeda' },
  ],
},

Current behavior:
const _: (typeof import('../../node_modules/remeda'))['undefined'];

Expected behavior:
const _: (typeof import('../../node_modules/remeda'));

By implementing this change, it will be possible to import all exports from modules that do not have a default export.
Additional context
No response
Logs
No response

This sounds more like an issue of https://github.com/unjs/unimport itself. Could you check it with only unimport and raise it there if it occurs with just the lib?

I found that setting the name property to '*' resolves this issue:
imports: {
  imports: [
    { as: '_', from: 'remeda', name: '*' },
  ],
},

result:
const _: typeof import('../../node_modules/remeda');

This solution was not documented in unimport's README, but it effectively addresses the issue.
Thank you for your help.

@jinyongp please feel free to send a PR to the README to add the case 👍🏻


fix final stabilization and optimization + also save the current grap…

…h into local storage
Description

Related Backlog Item

Issue: #

Context

Checklist:

[ ] I have documented my changes
[ ] I have tested the changes and they work as expected
[ ] I have assigned this PR to someone
[ ] I have run make format to format my code

Additional references

Works with stabilization only happeningf once. But still have errors in the console.


panic in crypto test unit tests

@adecaro .. assigning this to you but do move it to the right person if incorrect please.
Running
go test -timeout=20m $(go list github.com/openblockchain/obc-peer/... | grep -v /vendor/)
I see the following panic every time. Not sure if this is something in my environment (but I cannot think of why that should be...).
--- FAIL: TestSPI (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x38 pc=0x50be8e]
goroutine 6 [running]:
panic(0x703c60, 0xc82000e0d0)
/opt/go/src/runtime/panic.go:464 +0x3e6
testing.tRunner.func1(0xc82007c090)
/opt/go/src/testing/testing.go:467 +0x192
panic(0x703c60, 0xc82000e0d0)
/opt/go/src/runtime/panic.go:426 +0x4e9
crypto/ecdsa.randFieldElement(0x0, 0x0, 0x7f0a5af794f0, 0xc820018330, 0x0, 0x0, 0x0)
/opt/go/src/crypto/ecdsa/ecdsa.go:84 +0x4e
crypto/ecdsa.GenerateKey(0x0, 0x0, 0x7f0a5af794f0, 0xc820018330, 0x0, 0x0, 0x0)
/opt/go/src/crypto/ecdsa/ecdsa.go:100 +0x66
github.com/openblockchain/obc-peer/openchain/crypto/ecies/generic.TestSPI(0xc82007c090)
/opt/gopath/src/github.com/openblockchain/obc-peer/openchain/crypto/ecies/generic/spi_test.go:19 +0xe1
testing.tRunner(0xc82007c090, 0x90f4a0)
/opt/go/src/testing/testing.go:473 +0x98
created by testing.RunTests
/opt/go/src/testing/testing.go:582 +0x892
FAIL    github.com/openblockchain/obc-peer/openchain/crypto/ecies/generic       0.014s

@muralisrini, I guess, it happens when running the tests for my PR, right? If this is the case, I have just updated the PR with a fix. Thanks.

@muralisrini Oh, just realized that my previous PR has been merged. I have issued another one to fix the bug. Sorry for the inconvenient.

fixed by #730


HTTP payload compression

Is your feature request related to a problem? Please describe.
As a developer I would like to offload API request payload compression from the application to the gateway. Optional.
Describe the solution you'd like
When the clients headers support the compression and the response from the upstream is not compressed - compress the payload and return to the client for the text and json content types. Must be configurable (true/false, possibly preferred compressor - Envoy supports gzip and brottly) and overridable per route.
Describe alternatives you've considered
The alternative is to compress in the application and this has its own pros. Compressing on the gateway can create the significant CPU usage when there is high traffic load.

AFAIK, Envoy compresses automatically every request, unless a Vary header is available.
Correct me if I'm wrong @mbana @jasmingacic @kylehodgetts


WIP: Add cookie banner

What
Implement a configurable cookie banner based on the GOV.UK Design system cookie banner component
How to test

run locally
experiment with configuration options (example config)

To do:

[x] conditionally show different parts of the cookie banner based on config
[x]  document configuration
[ ] add tests

Build is failing.See https://github.com/alphagov/tech-docs-gem/issues/230 for detail

Hi @kr8n3r  - thanks for working on this! Was it ready for another review or did you need to do more work on it? (I've added @lfdebrux as a reviewer for when it's ready.)
@jonathanglassman Are you ok to work with @kr8n3r on the related documentation PR? I'm happy to pre-i or 2i it.

Hi @kr8n3r - thanks for working on this! Was it ready for another review or did you need to do more work on it? (I've added @lfdebrux as a reviewer for when it's ready.)
@jonathanglassman Are you ok to work with @kr8n3r on the related documentation PR? I'm happy to pre-i or 2i it.

it does require writing tests for it, but that would mean setting up a new test framework.
It worked on this when I had some time (which was a while ago) but it's not of any priority for us at the moment


Added layer for creating and updating the workflow

Issue #, if available:
Description of changes:
Added transport layer for creating/updating of the workflows and appropriate integration tests
CheckList:
[ ] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Can you share the integ test runs in here to add confidence on things working?

Can you share the integ test runs in here to add confidence on things working?

I think I added. Here is the integ-test file

Can you share the integ test runs in here to add confidence on things working?

Sorry didn't understood you correctly :) will attach the test results

TEST-org.opensearch.alerting.WorkflowMonitorIT.txt

did you add integ tests to verify  user permissions on workflow creation?

did you add integ tests to verify user permissions on workflow creation?

No i didn't. I checked a code (how we are doing the Secure*** tests in SecureMonitorRestApiIT -> we are dependent on the restClient while we have only transportLayer for the time being (when we are talking about the Workflows).
Do you know if we have some base test class or some class in alerting that enables us to add a roles to a client() object used in OpenSearchSingleNodeTestCase? Something like we have SecureRestClientBuilder? If we have that I can see how to add a roles and do a proper security check. I didn't manage to find it in a code.

https://github.com/opensearch-project/alerting/issues/834


be able to order transform rules

Current Terraform and Cloudflare provider version
Terraform v1.3.5
on linux_amd64

provider registry.terraform.io/cloudflare/cloudflare v3.30.0 (the option does not exist in version v4.2.0 either.)

Description
Within the interface, "transform rules" can be ordered. The ability to order the rules is mandatory for some treatments.
With this API endpoint: https://developers.cloudflare.com/api/operations/zone-rulesets-update-a-zone-ruleset-rule
I've found (with the dev tools on the interface) an undocumented attribute :
    "position": {
        "index": 3
    }

that can set the position of the transform rules in the list.
I would like to have this attribute available in the provider. For now i can't really use Terraform for the transform rules as i can't know in which order the rules will be processed.
Use cases

A rule where you delete some headers to avoid spoofing
A rule where you set the same headers the way you want if needed.

Potential Terraform configuration
resource "cloudflare_ruleset" "request_headers" {
...
  rules {
    action = "rewrite"
    action_parameters {
      headers {
        name      = "X-Security-Header"
        operation = "remove"
      }
    }
    description = "test rule"
    enabled     = true
    expression  = "(whatever!)"
    position {
        index = 3
    }
  }
}

References
No response

I managed to order the transform rules using a list (which is ordered), the provider respects the order in which you create the list. It should be documented as other resources does not behave this way.

the resource itself uses TypeList for the rules blocks and maintains that ordering. unless you are abstracting this with a module or dynamics, it will work as expected.
we don't support updating individual rules for exactly this reason where folks need ordering that they provide.


[code-freeze-week] must be scheduled the week-ends too

Coverage decreased (-0.3%) to 25.35% when pulling 91b4decf73391f197c4cee9af20f63374abfea15 on calixteman:freeze_week into 741a6c8c182b7bba5501b8e3b30086d3c58783c1 on mozilla:master.


dart fix command changed my code but not correct

The change is here
- if (tokens != null && tokens.isNotEmpty) {
+ if (tokens.isNotEmpty) {

dart info
Dart SDK version: 3.3.0 (stable) (Tue Feb 13 10:25:19 2024 +0000) on "linux_x64"

@bwilkerson thanks for the reply, there is nothing error reported in my VScode
Computing fixes in lib...
Nothing to fix!

My workspace dart info Dart SDK version: 3.3.0 (stable) (Tue Feb 13 10:25:19 2024 +0000) on "windows_x64"
But if I push code to the repo and trigger workflow action to run dart fix then the change happened
  unnecessary_null_comparison - 1 fix

This is my yml config
- run: dart --version
- run: dart fix --apply ./lib
- name: Commit formatting changes
   uses: stefanzweifel/git-auto-commit-action@v5

... there is nothing error reported in my VScode ...

That's a bit surprising. The dart fix command is finding a diagnostic when the IDE isn't, which shouldn't happen if they're analyzing the same code.
Are the contents of the file on disk the same as the contents of the edit buffer in VS Code?
The unnecessary_null_comparison diagnostic is presumably being generated because the analyzer believes that tokens is non-null at the beginning of the if statement. I can't tell why that would be without seeing the rest of the uses of tokens in the method or function.

I found if run pub get first then dart fix will ok, sorry for this dumb question, tks.


added a job delete button

Seemed to be missing a button to delete jobs.

Haven't looked at the detail of this but let's make sure that only creators can delete a job.


VCF output not compatible with bcftools norm

I have run freebayes v1.0.2-6-g3ce827d-dirty on Linux Lubuntu 15.10, gcc 5.2.1:
'''
freebayes -f ../genome/hs37d5.chr_only.fa --region 2:62034413-62036413 test1.bam > test1.vcf
#normalize with bcftools
bcftools  norm -m-both -c w  -f ../genome/hs37d5.chr_only.fa  test1.vcf
#the error:
#bcftools: vcfnorm.c:610: split_format_numeric: Assertion `nvals==(src->n_allele-1)*nsmpl' failed.
'''
The test1.vcf contains line triggering it:
'''
2	62035409	.	ATTTTTTTTTTTTTTTTG	ATTATTTTTTTTTTTTG,ATTTTTTTTTTTTTG	556.155	.	AB=0.204545,0.340909;ABP=36.372,12.6832;AC=1,1;AF=0.5,0.5;AN=2;AO=9,15;CIGAR=3M1X1D13M,1M3D14M;DP=44;DPB=51.6111;DPRA=0,0;EPP=3.25157,3.15506;EPPR=3.0103;GTI=0;LEN=17,3;MEANALT=10,10;MQM=58.2222,60;MQMR=57.25;NS=1;NUMALT=2;ODDS=32.2525;PAIRED=1,0.933333;PAIREDR=1;PAO=11.6667,5.66667;PQA=406.667,174.667;PQR=174.667;PRO=5.66667;QA=277,444;QR=132;RO=4;RPL=7,11;RPP=9.04217,10.1038;RPPR=5.18177;RPR=2,4;RUN=1,1;SAF=4,8;SAP=3.25157,3.15506;SAR=5,7;SRF=3;SRP=5.18177;SRR=1;TYPE=complex,del;technology.Illumina=1,1	GT:DP:DPR:RO:QR:AO:QA:GL	1/2:44:44,9,15:4:132:9,15:277,444:-61.1446,-24.7609,-18.7955,-37.7655,0,-33.0759
'''
If I manually split this line into two preserving all values from columns >5:
'''
2 62035409 . ATTTTTTTTTTTTTTTTG ATTATTTTTTTTTTTTG
2 62035409 . ATTTTTTTTTTTTTTTTG ATTTTTTTTTTTTTG
'''
it makes bcftools running without the error. So my guess is that for such calls either freebayes VCF lacks some values needed by bcftools. Or there is some VCF versions mix up.
Many thanks for writing the freebayes.
Darek Kedra

I get a different error, but also with norm:
[E::vcf_parse_format] Invalid character '.' in 'QR' FORMAT field at 1:10001
The problematic line is
1	10001	.	A	<*>	0	.	DP=367;END=10176;MIN=41	GQ:DP:MIN:QR:QA	1.88567e+06:367:41:1.9267e+06:41025.2
This is the output of freebayes with --gvcf --genotype-qualities --gvcf-chunk 100000 --min-mapping-quality 50
Additionally, it would be nice to have freebayes output be valid with the more stricter bcftools since we currently cannot stream freebayes output into bcftools due to the contig names not being output in the header.

This seems to be fixing the problem:
#from latest vcflib:
vcfbreakmulti my_freebayes.vcf > my_freebayes.multibreak.vcf
#bgftools/htslib 1.3.1
bgzip -i my_freebayes.multibreak.vcf
creates my_freebayes.multibreak.vcf.gz &  my_freebayes.multibreak.vcf.gz.gzi
bcftools norm -m-both  -c w -O z -f genome.fasta -o  my_freebayes.multibreak.norm.vcf.gz my_freebayes.multibreak.vcf.gz
#index it for downstream processing
bcftools  index my_freebayes.multibreak.norm.vcf.gz
So far no errors reported during these steps, plus the output looks sane.
Hope it helps
DK


Feature/dashboard mvp

This branch is a bit experimental, so I'm putting it up for a PR just to get some feedback. This is a first attempt at making some kind of useful dashboard for mEditor. Would you check it out and let me know what concerns / comments you have?

[ ] handle logged out user (both in the sense that we currently throw an error for logged-out and that we need do decide what content to show)


This PR

s/append/merge in getType() for ClientMergeQuery
set JsonProperty for getAggregators() for ClientMergeQuery
added serde tests between xxxTask and ClientXXXQuery so that changes in xxxTask will propagate to ClientXXXQuery.

:+1:

:+1:


Feat (#485) : close tabs on middle mouse click

=> feat #485
=> onMouseUp event is added at tabs
=> tabs close on middle mouse click
=> warning is shown for drafted work (unsaved)

@helloanoop as your and @martinsefcik said i have done all the changes . thanks for the previous reviews !

I know that currently you are overwhelmed, but this seems like a low-complexity issue. Any chance to move this forward soon?
Thanks!

Please merge this 🙏  It is giving me a headache that I cannot close tabs with middle click 🤣

I would also really appreciate this, it's one of the main things I'm missing right now.

This pull request code is with an issue when user attemps to close a tab with draft changes

I adapted this feature to the current Bruno code and opened a new PR -> check here

Thanks @helloanoop and @Gabrielcefetzada for this one✨
This was my first contribution to Bruno and you guys really helped me through it....


Unable to get the quickstart to work with service account key file

Expected Behavior
To modify the NodeJs quickstart: https://developers.google.com/admin-sdk/directory/v1/quickstart/nodejs to work with a service account key file
Actual Behavior
Errors
Steps to Reproduce the Problem

1.Modify the Authorize to use a key file in the Oauth2 constructor
1.
Specifications

Node version (node -v) - 10.14
OS (Mac/Linux/Windows) Mac

These samples are designed for 3-legged user auth. Please refer to the Google APIs Node.js Client Library documentation for information on how to use service accounts:
https://github.com/googleapis/google-api-nodejs-client#service-to-service-authentication


Symantec VIP MFA

💡 Summary
To allow Symantec VIP MFA security codes
Motivation and context
Allow additional MFA options (Symantec VIP).
Implementation notes
Allow user(s) to enter security code obtained from app into the CLI when authenticating
Acceptance criteria
Able to use Symantec VIP MFA security code to authenticate

Closing this issue out as no response has been given.


Nonstandard spaces

Comes from https://github.com/poldracklab/fmriprep/pull/1596/files#r280788899
With the implementation of the --output-spaces command line argument, we opened the door to better specify how the user wants their output data in terms of spaces and gridding.
After discussion on #1588, an interface was defined to standard target templates. This issue will focus on specifying the keywords and associated semantics for non-standard spaces.
Currently, fMRIPrep only supports T1w space, which has been given an alias anat. This means that using --output-spaces anat (or --output-spaces T1w), will lead to generating BOLD runs aligned with the anatomical reference created by fMRIPrep. By default, these runs are resampled in their native resolution (i.e., not in the anatomical's resolution).
Here and in #1376, @a3sha2 proposed generating BOLD series in their native space. That would be a new keyword we would need to define (and so on, so forth).
Some ideas I'm currently entertaining is to have run, func and (bold/sb)ref as potential options for such native space. I would shy away from native as we are also using it as a modifier and one could see --output-spaces MNI152Lin:native native which seems weird.
Pinging @satra, @edickie for a BIDS perspective.

Related: #1647

Let's say I want to have my EPI data in the T1w space, with 1mm isotropic voxel size. Is there currently a way to do this? I tried the following and it produced 1mm-resolution data in the MNI152NLin2009cAsym space, but the T1w space output used the voxel sizes from the EPI/BOLD data, as usual. (It produced the outputs I described but subsequently crashed because I used 'fs6' instead of 'fsaverage')
singularity run --cleanenv -B /tmp:/tmp /images/fmriprep-1.4.1.simg /subdir/BIDS_SAMPLE_SUBJNAME-fmriprep /subdir/BIDS_SAMPLE_SUBJNAME-fmriprep/derivatives/1.4.1-final participant --participant-label SUBJNAME --work-dir /subdir/BIDS_SAMPLE_SUBJNAME-fmriprep/derivatives/1.4.1-final -vv --omp-nthreads 8 --nthreads 16 --fd-spike-threshold 0.5 --notrack --return-all-components --resource-monitor --output-spaces T1w:res-1 fs6 --template-resampling-grid 1mm --force-bbr --use-syn-sdc --cifti-output --fs-license-file /ncf/nrg/sw/apps/freesurfer/6.0.0/license.txt

Hi @hhoke, this is a great question. At this point you can't do that. What kind of language would you suggest for the --output-spaces in your use-case?

That's a shame! It's something we need for our lab's pipeline. What's your timeline for adding this feature? Is it a matter of tacking something on to existing abstractions or will it require serious reworking?
T1w:res-1 seems to make sense to me. I like the idea of being able to specify different resolutions per output space. It was somewhat frustrating that T1w:res-1 actually isn't an available option, but the program started running anyway! Off the top of my head T1w for the default behavior, T1w:res-T1w for matchingT1w and T1w:res-func as a redundant way to get the current default behavior. I'm also somewhat confused as to exactly what --template-resampling-grid 1mm is doing.

@hhoke The --template-resampling-grid 1mm option will disappear (see #1898)
I think it would be interesting to have more strict rules, with a wider portfolio of behaviors for this command-line argument. @effigies was not in favor of creating a whole new language for this, but I'm afraid we will need to.
I think one should be able to do the following:
--output-spaces \
    MNI152NLIn2009cAsym:res-1:res-2:res-iso1.2mm:res-bold \
    T1w:res-native:res-bold \
    func:res-native:res-iso2.0mm \
    session:res-iso2.0mm

where we have one output in standardized space, one subject space, and two scanner spaces.
Resolutions work as follow:

res-<label found in template>: uses the given template grid. In case of overlap (e.g., the unfortunate situation where the template has a _res-bold_ component, the template selector takes precedence.
res-native: the native resolution of this run. So for T1w:res-native this means the T1w-reference's resolution and for func:res-native means this particular BOLD-run's resolution.
res-IsoX.Xmm: allows generating an isotropic grid if X.X (units) size. Default units would be mm, but that can be changed to other units (I'm thinking of rodents here).

Then, users would be able to concatenate an arbitrary number of res- modifiers, to generate several resamplings of the same data. Then the command above will generate:

BOLD data in MNI152NLIn2009cAsym space, with 4 different resolutions. Two of them matching the resolution of the res-1 and res-2 variants of the template (direct matching of BIDS-like entity), one with 1.2mm isotropic resolution covering the FoV of the template reference and finally one with whatever it was the original resolution of this BOLD run.
BOLD data aligned to the same-subject T1w reference, with two resolutions: the T1w template resolution and the BOLD run's.
BOLD data in its original gridding (func:res-native) and with 2.0mm isotropic resolution (func:res-iso2.0mm).
BOLD data in a session-subject space. This would only admit the arbitrary isotropic resolution form (runs across the session might have different resolutions).

Regarding the allowed space identifiers, they could take:

Any template ID in TemplateFlow for standard spaces
anat or T1w for the T1w anatomical reference
func for this particular functional run
session an average space across all bold runs and tasks in the protocol
task an average space across all runs of each particular task being processed (works with task filtering of the bids query).

I'm just thinking out loud. Opinions to move forward?
/cc @effigies @mgxd @poldracklab/fmriprep-docusprint-leaders

After reading through this once or twice, the general idea seems understandable to me. Two thigns are unclear:

How would the resolution modifier interact with additional modifiers such as cohort for the pediatric templates? It seems like : is used as a delimiter for both, so would you be looking at MNIPediatricAsym:res-1:res-2:cohort-2 or MNIPediatricAsym:cohort-2:res-1:res-2? The latter makes more sense to me, but the current docs suggest the former.

For fMRIprep users who aren't versed in TemplateFlow and where to find the templates, additional explanation in the docs of how to figure out what res-1 and res-2 of a given template actually output might be useful. Also, I assume that res-01 and res-1 evaluate to the same thing (seeing that the pediatric template has res-1 while others have 01).

Thanks for the super-timely feedback!

How would the resolution modifier interact with additional modifiers such as cohort for the pediatric templates? It seems like : is used as a delimiter for both, so would you be looking at MNIPediatricAsym:res-1:res-2:cohort-2 or MNIPediatricAsym:cohort-2:res-1:res-2? The latter makes more sense to me, but the current docs suggest the former.

I think the ordering of the cohort selector does not impede a univocal interpretation. But, in relation to #1817, one might want to normalize to two cohorts and have varying sets of outputs:
--output-spaces \
    MNIPediatricAsym:cohort-5:cohort-6:res-2:res-iso1.6mm

Would produce res-2 and 1.6 isotropic outputs in both cohorts (5 and 6). However, if you want different outputs:
--output-spaces \
    MNIPediatricAsym:cohort-5:res-2 \
    MNIPediatricAsym:cohort-6:res-iso1.6mm

Then it will generate the res-2 for the cohort 5 and the 1.6mm isotropic for the cohort 6.
How does this idea sound? And, is there something I'm not seeing about the ordering of the cohort- particle?

For fMRIprep users who aren't versed in TemplateFlow and where to find the templates, additional explanation in the docs of how to figure out what res-1 and res-2 of a given template actually output might be useful.

Totally. We are working quite a bit on the documentation of TemplateFlow, which will help to describe this, and some updates on fMRIPrep's documentation will be necessary linking to TemplateFlow's.

Also, I assume that res-01 and res-1 evaluate to the same thing (seeing that the pediatric template has res-1 while others have 01).

This is a good point too. By the looks of this bids-standard/bids-specification#301 it seems that the resolution value will not be an index anymore - meaning, TemplateFlow will need to be updated.

I think the ordering of the cohort selector does not impede a univocal interpretation. But, in relation to #1817, one might want to normalize to two cohorts and have varying sets of outputs:
--output-spaces \
    MNIPediatricAsym:cohort-5:cohort-6:res-2:res-iso1.6mm

Would produce res-2 and 1.6 isotropic outputs in both cohorts (5 and 6). However, if you want different outputs:
--output-spaces \
    MNIPediatricAsym:cohort-5:res-2 \
    MNIPediatricAsym:cohort-6:res-iso1.6mm

Then it will generate the res-2 for the cohort 5 and the 1.6mm isotropic for the cohort 6.
How does this idea sound? And, is there something I'm not seeing about the ordering of the cohort- particle?

This sounds great, and is very clear! The place I saw the other order of cohort selector was in the docs, where it specifies that:

Therefore, in upcoming versions of fMRIPrep, it will be possible to run it with --output-spaces MNIPediatricAsym:res-2:cohort-2 where cohort-2 would select the template instance for the, say, 24-48 months old range.

This might lead people to think that you have to first specify res, and then cohort.
Re: TemplateFlow, I had seen that PR and the discussion of how res-<index> could get confusing makes sense to me!


Remove unused boolean variable first in mbtiles.cpp .

Two unused variables both named first declared at line 133 and 512 are removed.

Thanks! I just merged another PR that also removed this.


Remove browser tooltip from icons

There's a <title>icon name</title> inside each icon's svg and there's no way to remove it via <Icon>'s props:

It's especially annoying when there's a <Tooltip> for that icon and user ends up seeing 2 tooltips: one from evergreen, one from browser. I think we should remove it.

How about this?
It looks not good because of empty string for title but it's working.
<Tooltip content="cross">
    <Icon icon="cross" title="" />
</Tooltip>


wy return status is always success

The return status from wy build (and other commands) seems to always be 0 (success).
For example if you try building a source file with errors as follows:
(in the LZ.wy project for example):
$ echo "bad program" >src/lz77/Bad.whiley
$ wy build
./src/lz77/Bad.whiley:1: unexpected end-of-file
bad program
^
$ echo $?
0
This return status should be non-zero, since the compilation failed.

Finally fixed this I believe.


OADP-4308: Don't consider unschedulable pods unrecoverable

For unit tests to pass, need these 2 PRs:

https://github.com/openshift/release/pull/53237
https://github.com/openshift/velero/pull/316 (cp to oadp-1.4 branch)

CI issue - goimports, but it's known

Fix forthcoming for unit tests. Prow will still fail them, though.

/override ci/prow/unit-test

ok  	github.com/vmware-tanzu/velero/internal/volume	0.051s	coverage: 48.9% of statements
ok  	github.com/vmware-tanzu/velero/internal/volumehelper	0.057s	coverage: 37.4% of statements
Success!
make[1]: Leaving directory '/home/whayutin/OPENSHIFT/git/OADP/velero'

/LGTM


ModuleNotFoundError: No module named 'x-flux-comfyui.xflux'

Traceback (most recent call last):
File "D:\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1941, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in call_with_frames_removed
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\x-flux-comfyui_init.py", line 1, in
from .nodes import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
File "D:\ComfyUI_windows_portable\ComfyUI\custom_nodes\x-flux-comfyui\nodes.py", line 12, in
from .xflux.src.flux.util import (configs, load_ae, load_clip,
ModuleNotFoundError: No module named 'x-flux-comfyui.xflux'

You not install the x-flux
Go to  ComfyUI/custom_nodes/x-flux-comfyui/ and run python setup.py

thanks


Remove duplicate devices from testing

Description
oneMKL gtests use device name as an unique parameter, but in some cases we have several similar devises on the machine (e.g. host selector is not fully supported and it behaves as CPU device instead). This PR removes duplicate devices from testing as it doesn't make sense to test functionality twice on the same devices.
Fixes #57
Checklist
All Submissions

[x] Have you formatted the code using clang-format?
[x] Do all unit tests pass locally? Attach a log.

Log:
[ 98%] Building CXX object tests/unit_tests/blas/level2/CMakeFiles/blas_level2_ct.dir/sbmv_usm.cpp.o
[ 99%] Building CXX object tests/unit_tests/blas/level2/CMakeFiles/blas_level2_ct.dir/hpr_usm.cpp.o
[ 99%] Built target blas_level2_ct
Scanning dependencies of target test_main_blas_ct
[ 99%] Building CXX object tests/unit_tests/CMakeFiles/test_main_blas_ct.dir/main_test.cpp.o
[100%] Linking CXX executable ../../bin/test_main_blas_ct
[100%] Built target test_main_blas_ct

          Start    1: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Column_Major_Intel_R__Core_TM__i7_6770HQ_CPU___2_60GHz
   1/1408 Test    #1: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Column_Major_Intel_R__Core_TM__i7_6770HQ_CPU___2_60GHz ..................................   Passed    0.57 sec
          Start    2: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Row_Major_Intel_R__Core_TM__i7_6770HQ_CPU___2_60GHz
   2/1408 Test    #2: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Row_Major_Intel_R__Core_TM__i7_6770HQ_CPU___2_60GHz .....................................   Passed    0.51 sec
          Start    3: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Column_Major_Intel_R__Graphics_Gen9__0x193b_
   3/1408 Test    #3: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Column_Major_Intel_R__Graphics_Gen9__0x193b_ ............................................   Passed    0.91 sec
          Start    4: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Row_Major_Intel_R__Graphics_Gen9__0x193b_
   4/1408 Test    #4: BLAS/Nrm2TestSuite/Nrm2Tests.RealSinglePrecision/Row_Major_Intel_R__Graphics_Gen9__0x193b_ ...............................................   Passed    0.91 sec

@mmeterel In the other PR, https://github.com/oneapi-src/oneMKL/pull/62 you mentioned that the Ubuntu Gen9 failures are due to this PR. I can verify that ctest cannot find any tests to run for Ubuntu-Gen9-DPC++ configuration.


nfs boot option

I boot several of my pi's via nfs/pxeboot and currently extract the filesystem from an image file produced by sdm using this script https://github.com/1stcall/sdm/blob/mydev/extfilesys.sh . I don't  think this is possible with custom plugin. Could u add an option to produce just a boot and root directories?

Just to be clear, you're looking for a way to extract the boot and root directories (presumably customized) from an IMG and store them at a given location in the file system?

Yes. The boot dir is then shared via tftp and the root via nfs.  This is to facilitate network booting of the pi.  The extract script works fine so there is no rush for this.  It would just be nice to not require the extra step.  You have already done the disabling of expanding root on 1st boot etc. which is obviously not required.  Bonus would be to delete the image file after extraction.

From: Benn @.>
Sent: 28 November 2023 3:55 AM
To: gitbls/sdm @.>
Cc: Carl McAlwane @.>; Author @.>
Subject: Re: [gitbls/sdm] nfs boot option (Issue #139)
Just to be clear, you're looking for a way to extract the boot and root directories (presumably customized) from an IMG and store them at a given location in the file system?
—
Reply to this email directly, view it on GitHubhttps://github.com/gitbls/sdm/issues/139#issuecomment-1829030755, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCHHLKGQ6PHWJ7UCO6LOJ3YGVOCVAVCNFSM6AAAAAA747CBQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRZGAZTANZVGU.
You are receiving this because you authored the thread.Message ID: @.***>

Got it. Seems fairly simple. Finishing up this release first, then will toss this in.
What does lifecycle maintenance look like? Do you update the nfs-booted systems (and hence the extracted file systems), or do you rebuild the IMG, update, and recreate the nfs-shared directories? Or not update at all? 😵

Just a standard apt update && apt upgrade-y along with a few git pulls.
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Benn @.>
Sent: Tuesday, November 28, 2023 2:07:44 PM
To: gitbls/sdm @.>
Cc: Carl McAlwane @.>; Author @.>
Subject: Re: [gitbls/sdm] nfs boot option (Issue #139)
Got it. Seems fairly simple. Finishing up this release first, then will toss this in.
What does lifecycle maintenance look like? Do you update the nfs-booted systems (and hence the extracted file systems), or do you rebuild the IMG, update, and recreate the nfs-shared directories? Or not update at all? 😵
—
Reply to this email directly, view it on GitHubhttps://github.com/gitbls/sdm/issues/139#issuecomment-1829915437, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCHHLIPXEIVLWRWPBSNOY3YGXV3BAVCNFSM6AAAAAA747CBQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRZHEYTKNBTG4.
You are receiving this because you authored the thread.Message ID: @.***>

So, to be clear, you simply run on that NFS tree, updating it in place with apt. Then, when it's time to rebuild the tree for whatever reason, you shutdown the network-booted pi, wipe and replace the tree on the nfs server, and boot the network-booted pi on the new tree.
Correct?

Yes.  When expeiementing and something works well, I write a custom sdm-plugin so that it is there on the next build.  I must experiment with running plugin on a customised system.

From: Benn @.>
Sent: 29 November 2023 1:54 PM
To: gitbls/sdm @.>
Cc: Carl McAlwane @.>; Author @.>
Subject: Re: [gitbls/sdm] nfs boot option (Issue #139)
So, to be clear, you simply run on that NFS tree, updating it in place with apt. Then, when it's time to rebuild the tree for whatever reason, you shutdown the network-booted pi, wipe and replace the tree on the nfs server, and boot the network-booted pi on the new tree.
Correct?
—
Reply to this email directly, view it on GitHubhttps://github.com/gitbls/sdm/issues/139#issuecomment-1831940570, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCHHLOWXHLOJ7IVN4M6Z3TYG45CXAVCNFSM6AAAAAA747CBQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZRHE2DANJXGA.
You are receiving this because you authored the thread.Message ID: @.***>

The nfs host is a Raspberry cm4, built with sdm.

From: Benn @.>
Sent: 29 November 2023 1:54 PM
To: gitbls/sdm @.>
Cc: Carl McAlwane @.>; Author @.>
Subject: Re: [gitbls/sdm] nfs boot option (Issue #139)
So, to be clear, you simply run on that NFS tree, updating it in place with apt. Then, when it's time to rebuild the tree for whatever reason, you shutdown the network-booted pi, wipe and replace the tree on the nfs server, and boot the network-booted pi on the new tree.
Correct?
—
Reply to this email directly, view it on GitHubhttps://github.com/gitbls/sdm/issues/139#issuecomment-1831940570, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCHHLOWXHLOJ7IVN4M6Z3TYG45CXAVCNFSM6AAAAAA747CBQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZRHE2DANJXGA.
You are receiving this because you authored the thread.Message ID: @.***>

My fork of sdm is a thttps://github.com/1stcall/sdm/tree/mydev
[https://opengraph.githubassets.com/08aeda9a64ed5a3ba99cbf7e0d0e00b27a88c8b84dd28b3ff62c2029659d29f5/1stcall/sdm]https://github.com/1stcall/sdm/tree/mydev
GitHub - 1stcall/sdm: Raspberry Pi SD Card Image Managerhttps://github.com/1stcall/sdm/tree/mydev
Raspberry Pi SD Card Image Manager. Contribute to 1stcall/sdm development by creating an account on GitHub.
github.com
I haven't done much with it for a while.  I'm currently teaching myself sveltekit with a veiw to build a sdm as a web server that returns a customised image, build using a cluster of rasberry pis.

From: Benn @.>
Sent: 29 November 2023 1:54 PM
To: gitbls/sdm @.>
Cc: Carl McAlwane @.>; Author @.>
Subject: Re: [gitbls/sdm] nfs boot option (Issue #139)
So, to be clear, you simply run on that NFS tree, updating it in place with apt. Then, when it's time to rebuild the tree for whatever reason, you shutdown the network-booted pi, wipe and replace the tree on the nfs server, and boot the network-booted pi on the new tree.
Correct?
—
Reply to this email directly, view it on GitHubhttps://github.com/gitbls/sdm/issues/139#issuecomment-1831940570, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADCHHLOWXHLOJ7IVN4M6Z3TYG45CXAVCNFSM6AAAAAA747CBQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZRHE2DANJXGA.
You are receiving this because you authored the thread.Message ID: @.***>

https://github.com/1stcall/sdmui/tree/development
https://sdmui.1stcall.uk

I just pushed plugins/extractfs with V10.2. It's not included in EZsdmInstaller, you'll need to pull it down some other way.
Usage:
sdm --runonly plugins --directory / --oklive --plugin extractfs:"verbose|img=/path/to/img.img|bootfs=/path/to/bootfs|rootfs=/path/to/rootfs"

It's a bit clunky as a plugin, but I wasn't prepared to add it into the core sdm.
Let me know your thoughts.

Why did you suggest changing the declare -x? Was there an issue with it as it was?
Yes.  SDMPT is already exported.  Therefore, in bash, if you re export it within a function, it will only export it to sub processes or functions called from within that function.

extractfs is in sdm V11 as a burn plugin. Much simpler! Please take it for a spin.


Calendar crash if current month is Novemver or December

If i swipe calendar to November, or set current date on November or December, appication hangs and crushes.
Any ideas?

This is my bug. Sorry. Issue closed


Delete Trash.cs

This takes 6 additional seconds to download, it wastes internet ( or data ) and it is literally pointless apart from some "statistics"

No we still need the statistics

This takes 6 additional seconds to download, it wastes internet ( or data ) and it is literally pointless apart from some "statistics"

If you have a problem leave the server delete your bots and don’t use moos


Bybit spot the same order amount for both sides of the trading pair, amm_arb

Bybit spot uses the same order amount for both sides of the trading pair.
I.E. if order_amount is set to 100, it will attempt to sell 100 tokens for USDT but also tries to buy $100 worth of tokens instead of just buying 100 tokens.
It happens when using amm_arb strategy with dex(quickswap) as a second connector and attempting to sell on Bybit spot.
release version 1.9.0 as well as dev-1.10.0

logs_conf_amm_arb.log

Update:
The same issue happens when using arb strategy with Bybit paired with another CEX
Bybit side swaps amounts for sides when buying, executes the orders, buy order goes through on Bybit but in the logs it says "market order canceled" straight after and order is left in "pending orders list" when using "status" command
1.10.0
dev-1.11.0
`
2022-12-15 20:54:48,001 - 1 - hummingbot.strategy.arbitrage.arbitrage - INFO - Markets are ready. Trading started.
2022-12-15 20:54:48,005 - 1 - hummingbot.strategy.arbitrage.arbitrage - INFO - Executing limit order buy of TKN-USDT at bybit and sell of TKN-USDT at cex with amount 4.03, and profitability 1.003052570383030460042397139 [clock=2022-12-15 20:54:48+00:00]
2022-12-15 20:54:48,025 - 1 - hummingbot.strategy.arbitrage.arbitrage - INFO -
Markets:
Exchange    Market  Best Bid Price  Best Ask Price  Mid Price
0    bybit  TKN-USDT        0.489776        0.491319  0.4905475
1   cex  TKN-USDT          0.4945          0.4977     0.4961
Assets:
Exchange Asset  Total Balance  Available Balance
0    bybit   TKN  =====
1    bybit  USDT  =====
2   cex   TKN         4.0378             4.0378
3   cex  USDT  =====
Profitability(without fees):
take bid on bybit, take ask on cex: -1.5921 %
take ask on bybit, take bid on cex: 0.6474 %
Pending limit orders:
market trading_pair                                  order_id  quantity            timestamp
0   bybit     TKN-USDT          BYBIT-BCCUT5efe46965e94968cb82      4.03  2022-12-15 20:54:48
1  cex     TKN-USDT  SCCUT5efe41043ad2d4968cb82asd123a171aba      4.03  2022-12-15 20:54:48
2022-12-15 20:54:48,281 - 1 - hummingbot.connector.client_order_tracker - INFO - Created MARKET BUY order BYBIT-BCCUT5efe46965e94968cb82 for 4.03 TKN-USDT.
2022-12-15 20:54:48,289 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.MARKET", "trading_pair": "TKN-USDT", "amount": "4.03", "price": "0.491319", "order_id": "BYBIT-BCCUT5efe46965e94968cb82", "creation_timestamp":<PHONE_NUMBER>.0, "exchange_order_id": "1311992452623761920", "leverage": 1, "position": "NIL", "event_name": "BuyOrderCreatedEvent", "event_source": "bybit"}
2022-12-15 20:54:48,372 - 1 - hummingbot.connector.client_order_tracker - INFO - Created MARKET SELL order SCCUT5efe41043ad2d4968cb82asd123a171aba for 4.0300 TKN-USDT.
2022-12-15 20:54:48,381 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.MARKET", "trading_pair": "TKN-USDT", "amount": "4.0300", "price": "0.4938", "order_id": "SCCUT5efe41043ad2d4968cb82asd123a171aba", "creation_timestamp":<PHONE_NUMBER>.0, "exchange_order_id": "639b8998612933214001f3c245", "leverage": 1, "position": "NIL", "event_name": "SellOrderCreatedEvent", "event_source": "cex"}
2022-12-15 20:54:48,390 - 1 - hummingbot.strategy.arbitrage.arbitrage - INFO - Market order canceled on bybit: BYBIT-BCCUT5efe46965e94968cb82 [clock=2022-12-15 20:54:48+00:00]
2022-12-15 20:54:48,390 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "order_id": "BYBIT-BCCUT5efe46965e94968cb82", "exchange_order_id": "1311992452623761920", "event_name": "OrderCancelledEvent", "event_source": "bybit"}
2022-12-15 20:54:48,390 - 1 - hummingbot.connector.client_order_tracker - INFO - Successfully canceled order BYBIT-BCCUT5efe46965e94968cb82.
2022-12-15 20:54:48,411 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "order_id": "SCCUT5efe41043ad2d4968cb82asd123a171aba", "trading_pair": "TKN-USDT", "trade_type": "TradeType.SELL", "order_type": "OrderType.MARKET", "price": "0.4945", "amount": "1.1296", "trade_fee": {"percent": "0.001", "percent_token": null, "flat_fees": []}, "exchange_trade_id": "1889495020085249", "leverage": 1, "position": "NIL", "event_name": "OrderFilledEvent", "event_source": "cex"}
2022-12-15 20:54:48,412 - 1 - hummingbot.connector.client_order_tracker - INFO - The SELL order SCCUT5efe41043ad2d4968cb82asd123a171aba amounting to 2.8339/4.0300 TKN has been filled.
2022-12-15 20:54:48,535 - 1 - hummingbot.strategy.arbitrage.arbitrage - INFO - Limit order completed on cex: SCCUT5efe41043ad2d4968cb82asd123a171aba [clock=2022-12-15 20:54:48+00:00]
2022-12-15 20:54:48,544 - 1 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "order_id": "SCCUT5efe41043ad2d4968cb82asd123a171aba", "base_asset": "TKN", "quote_asset": "USDT", "base_asset_amount": "4.0300", "quote_asset_amount": "1.99181452", "order_type": "OrderType.MARKET", "exchange_order_id": "639b8998612933214001f3c245", "event_name": "SellOrderCompletedEvent", "event_source": "cex"}
2022-12-15 20:54:48,544 - 1 - hummingbot.connector.client_order_tracker - INFO - SELL order SCCUT5efe41043ad2d4968cb82asd123a171aba completely filled.
2022-12-15 20:55:41,476 - 1 - hummingbot.client.hummingbot_application - INFO - stop command initiated.`

Hi @Myobbb, could you please provide more info, did you use different or same base asset to trade?

same base asset in all instances
it's an issue with Bybit spot connector

Here is another hopefully more clean example, where Bybit spot uses wrong amount when buying, buying in the amount of USDT instead of token it's paired to
bybit_buy_fail_log_example.txt

hi @Myobbb could you please check rate --token YOUR_TOKEN rate --pair YOUR_TOKEN-USDT commands for YOUR_TOKEN?
Also needed to check rate_oracle_source you use in config. Issue you had could be related to your token is not available in rate oracle you use.

Issue persists with use_oracle_conversion_rate set to False
it's exact same pair on both sides of arbitrage, so I never needed to convert
so I guess it rules out rate oracle being the reason behind it?

Okay, nice.
That's should be another issue, cause  use_oracle_conversion_rate set to False not used in amm_arb strategy, it's part of XEMM (Cross exchange market making) strategy.

@nikspz I could not recreate the issue with version 1.13.0 amm_arb on Bybit yet, trades just would not trigger when target price was reached, nothing in the logs.
It is my understanding that the issue with order amount is related to Bybit and not a particular strategy, since I experienced it on both "arbitrage" and "amm_arb" in the previous version and no changes have been made to the Bybit CEX connector since.
will do further testing to confirm it.

update:
I just recreated the situation with the latest version(1.13.0), issue persists, orders go through (with the wrong order amount), HMB does not register them as successful, and the second side does not trigger (concurrent_order_submission is set to False).

logs_conf_amm_arb_CMP.log

@Myobbb Thanks, I'll create a ticket for concurrent_order_submission. Could you please also provide your global config?
especially rate oracle source that you use

@nikspz concurrent_order_submission fails because of the initial problem with the Bybit market buy side, I don't think there is anything wrong with concurrent_order_submission in particular.
rate_oracle_source is kucoin, but I tried different ones in the past, and I don't think it actively plays a role here, tokens are paired 1:1

@Myobbb You're using kucoin rate oracle source and global token is USD
Noticed that bot failed to find rate for CMP when global_token is USD

USDT should work better for kucoin rate_oracle_source
After config global_token.global_token_name USDT:

@nikspz Thanks for the tip, I recreated the situation with USDT as a global token, and it did not affect the issue.
Please note that in your case you were only selling on Bybit, not buying, buying order amount is the problem.
Here are some more examples:

I looked over Bybit v1 API docs
https://bybit-exchange.github.io/docs-legacy/spot/v1/#t-placeactive
Under "Place Active Order" quantity, it says
Order quantity (for market orders: when side is Buy, this is in the quote currency. Otherwise, qty is in the base currency. For example, on BTCUSDT a Buy order is in USDT, otherwise it's in BTC. For limit orders, the qty is always in the base currency.)
probably the root of the problem.

As a temporary solution, I replaced
OrderType.MARKET with OrderType.LIMIT
https://github.com/hummingbot/hummingbot/blob/master/hummingbot/connector/exchange/bybit/bybit_exchange.py#L114
Limit orders go through with the correct amounts.


script_translator.cc无法编译

librime-1.7.2.tar.gz
[ 62%] Building CXX object src/CMakeFiles/rime.dir/rime/gear/script_translator.cc.o
/home/xxxx/temp/rime-builds/librime-1.7.2/src/rime/gear/script_translator.cc: 在成员函数‘void rime::ScriptTranslation::PrepareCandidate()’中:
/home/xxxx/temp/rime-builds/librime-1.7.2/src/rime/gear/script_translator.cc:490:23: 错误：‘exp’在此作用域中尚未声明
490 |     cand->set_quality(exp(entry->weight) +
|                       ^~~
/home/xxxx/temp/rime-builds/librime-1.7.2/src/rime/gear/script_translator.cc:504:23: 错误：‘exp’在此作用域中尚未声明
504 |     cand->set_quality(exp(entry->weight) +
|                       ^~~
make[2]: *** [src/CMakeFiles/rime.dir/build.make:1238：src/CMakeFiles/rime.dir/rime/gear/script_translator.cc.o] 错误 1
make[1]: *** [CMakeFiles/Makefile2:233：src/CMakeFiles/rime.dir/all] 错误 2
make: *** [Makefile:146：all] 错误 2
搜索了之后发现似乎和编译时的条件 -lm 有关，但不知如何处理。

怎樣重現這個編譯錯誤？

The issue can be fixed by adding #include <cmath> in src/rime/gear/table_translator.cc and src/rime/gear/script_translator.cc. I  notice the same FTBFS issue after openSUSE tumbleweed upgraded boost from 1.75 to 1.76. Maybe the previous version of boost pulled the cmath header occasionally, but the new version does not. I am not sure, though.
My rime is broken and so that fcitx5-rime, so I have to type in English. Sorry for the inconvenience.

謝謝。
可能是以前通過其他第三方庫隱含地引用了該頭文件。

This fix looks wrong. You are including <cmath> and then calling exp unqualified, with no using std::exp; or using namespace std;. You should either include <math.h> or use std::exp.

This fix looks wrong. You are including <cmath> and then calling exp unqualified, with no using std::exp; or using namespace std;. You should either include <math.h> or use std::exp.

@jwakely You are right. I tested the fix on GCC with libstdc++ and the code compiled, but this behavior, however, is not reliable since the standard library can choose to declare exp in the global namespace or not (see https://stackoverflow.com/a/11086087), although most implementations do. Perhaps you can make a PR to fix this issue.

Perhaps you can make a PR to fix this issue.

OK, will do.


Adding ssd1306 (oled) driver with example

A simple driver that allows the user to set pixels, draw horizontal and
vertical lines, as well as rectangles.

Coverage remained the same at 91.26% when pulling ce8619d1fa0162b18c79524bacdc5a1688fb2657 on oled-hardware-validation into 93aa09063a3bf6f87900557727a11b59ab3ab9ec on master.


[FEATURE]: Automatically replace special characters in the tracks name

Which tool
tidal-gui
Describe the feature
Some standalone DJ systems can't read special carachter like "é, è, ê, à, ç, ñ, û, ô, ï..." in the track title.
For exemple, 50% of French music has one or more special characters in the track name...
It would be great to have an option to automatically replace special characters in music titles.
Add screenshots
No response

In my opinion, it is up to the manufacturers of these systems to make an improvement. Did you ask them?

I fully agree with you! I ask for Denon DJ to do an update but the problem has been known for a long time but obviously no updade fixes it... I had to rename one by one each of my library titles. It might be nice to solve this problem as soon as you download...

Yes I understand, manufacturers are always ready to sell but for the rest... If not already, have you tried a batch renaming software like Advanced Renamer? This could be a temporary solution.

I have tried with mp3tag but not Advanced Renamer. I will try it today thank you!

You're welcome. Hope this will be useful for you.


Import feature

Motivation
I recently changed my device and tried to import my wallet with which I have been building and trying out the testnet but was unable to import the wallet because the feature is still lacking for now.
Usage example
No response
Possible implementations
No response

You can import seed phrases on the wallet currently in the setup phase once is setup, you can remove the wallet and start again.


Update troubleshooting-sapo-load-failures.md

Fix a misspelling ('form' should be 'from').

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


tests: fifo_timeout : Dereference after null check

fifo data obtained is dereferenced after NULL check.
Coverity-CID:186190
Signed-off-by: Nagaraj Hegde<EMAIL_ADDRESS>

I had to upload the changes again due to rebase issue.

Codecov Report

Merging #8082 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #8082      +/-   ##
==========================================
- Coverage   64.55%   64.55%   -0.01%
==========================================
  Files         420      420
  Lines       40141    40143       +2
  Branches     6763     6765       +2
==========================================
  Hits        25913    25913
- Misses      11106    11108       +2
  Partials     3122     3122

Impacted Files
Coverage Δ

tests/kernel/fifo/fifo_timeout/src/main.c
92.2% <0%> (-1.22%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43553da...5622183. Read the comment docs.


asNavFor skipping certain items

So I'm having a problem that I can't seem to solve.
I'm using the asNavFor variable to make 2 sliders move together while sliding. the first slider is the navigation showing 3 items and the second slider is the content.
When I slide the slider left or right, it doesn't seem to react. When I slide again it will skip the first slide and goes straight to the second.
This is the way I set it up.
https://codesandbox.io/embed/ox9k97my0y
I'm not sure if I'm doing something wrong or that the slick slider is doing something wrong. Could someone help me with this?

I have exactly the same problem. Not too sure what causes the issue here

Hi all.
I have submitted a PR which should fix this issue.
It would be great if you could check if this works for you as well.
https://github.com/akiran/react-slick/pull/1423

hi
This is not working as expected, This commit  is reflecting in version 0.23.1 aswell, please take a pull from the below url and run, In asNaFor demo  alternate images are loading on carousel arrow click event.
Downloaded files from this url :https://github.com/akiran/react-slick

hi
This is not working as expected, This commit  is reflecting in version 0.23.1 aswell, please take a pull from the below url and run, In asNaFor demo  alternate images are loading on carousel arrow click event.
Downloaded files from this url :https://github.com/akiran/react-slick

i can reproduce this issue

I am also having this issue where it only activates next slide on every other slide. I removed my code and used the example, same issue.

For those that need a quick fix, I reverted back to 0.23.1 and it is working normally.

For those that need a quick fix, I reverted back to 0.23.1 and it is working normally.

Or you could use patch-package, I demo it in the PR.


Dependencies update to info security improvement

Dependencies update to info security improvement
Based on: https://david-dm.org/twelve-net-co/seanjs

It seems that you didn't update them all. Also you changed links in README to twelve-net-co instead of seanjs-stack
If we wanna do it, we must update all packages with full test.

@Massad, done!

Need to fix the build. Will get back to you on that.

Need to fix the build. Will get back to you on that.
@felipo thanks for the update and heads up!

Your welcome @Massad


Resolve error prone issues

What can be done on tests is completed. Still have some on source side.


Implementing , Testing and documenting SHA-1 algorithm

Description
This pull request adds a advance SHA-1 hashing algorithm implementation. SHA-1 is a widely used cryptographic hash function that generates a 160-bit hash value from input data.
Features

Implements the SHA-1 hashing function.
Supports string input and returns the hash in hexadecimal format.
Includes unit tests to verify correctness.
verified the correctness of the algorithm by comparing the results with the sha1 algorithm available in Go and Java

Usage
You can use the SHA-1 function in my code as follows:
import { sha1 } from 'path/to/sha1';

const hashValue = sha1.hash('your input data here');
console.log(hashValue); // Outputs the SHA-1 hash

const isSame = sha1.compare('your input data here' , 'you previous hash value here')
console.log(isSame); // Outputs true if hash value is the same hash value of input data else false

About the contribution guidelines

I have made sure to run test and lint to verify correctness and quality of the code
My testing covers all the 100% of my code
I have added the README file with reference for further details

My folder named "sha1" is available in the cryptography folder