the location of the sticker

how can i change the placement of the sticker ?

@liyue0213
Please use the special script face_preparation.py for it:

You need to find parameters for the sticker position initialization. Use the next command to find these parameters:
python3 face_preparation.py PATH_TO_THE_IMAGE_WITH_HAT_ONLY --mask
It will show sticker placement with default parameters. Change parameters until the image looks like a prepared image with the sticker. You can see the parameters using --help flag.

But note that the new location could not be the good position to place the sticker because of the attention of the FaceID neural net.

@liyue0213
Please use the special script face_preparation.py for it:

You need to find parameters for the sticker position initialization. Use the next command to find these parameters:
python3 face_preparation.py PATH_TO_THE_IMAGE_WITH_HAT_ONLY --mask
It will show sticker placement with default parameters. Change parameters until the image looks like a prepared image with the sticker. You can see the parameters using --help flag.

But note that the new location could not be the good position to place the sticker because of the attention of the FaceID neural net.


Search workflow by name & status

API currently returns an error when trying to search by workflow name & workflow status. It would help users to be able to filter based on multiple filter types to reduce the search set.

This is working now.


ZWJ Sequence

I think it may be useful to include a recommended ZWJ sequence, especially one of the longer ones. For example, 👩👩👧👦 is ideally displayed as one 'family' character, but is actually 7 Unicode characters, and may fallback to display 4 characters for applications that don't directly support it. Inspired by: https://stackoverflow.com/questions/43618487/why-is-treated-so-strangely-in-swift-strings

I know this is nitpicking, but can we be a little more precise here?
It's a sequence of 7 unicode code points, not characters. Those will be more than 7 Bytes (depends on encoding). Ideally they should be displayed as a single glyph, but the fallback is to display them as a sequence of 4 glyphs.
My browser (chromium) actually shows them as four glyphs, but I cannot select them individually, only as a unit. So it seems to implement the fallback very well.
Which also shows the use of such a ZWJ sequence: implementations should treat them as a single unit for text manipulation purposes, even if they consist of multiple glyphs, code points and bytes.


chore: fix snyk issue

Also update some of the old dependencies

Actually this one is weird. It passes snyk locally but fails on CircleCi

Actually this one is weird. It passes snyk locally but fails on CircleCi

This starts failing as of snyk CLI 1.431.4 (note the brew tap is a bit behind and latest as of this moment is 1.431.2, which is why this is not reproducible locally).
I'd like to get the snyk version pinned in the orb configuration (don't currently see a way of doing that) and will also report this resolution issue to snyk. Need to dig into exactly what they were doing in that change to see what went wrong.
Ref: https://github.com/snyk/snyk/pull/1548

Actually this one is weird. It passes snyk locally but fails on CircleCi

This starts failing as of snyk CLI 1.431.4 (note the brew tap is a bit behind and latest as of this moment is 1.431.2, which is why this is not reproducible locally).
I'd like to get the snyk version pinned in the orb configuration (don't currently see a way of doing that) and will also report this resolution issue to snyk. Need to dig into exactly what they were doing in that change to see what went wrong.
Ref: snyk/snyk#1548

OK dug into this a bit more before filing with them, their fix is valid (at least the spirit of it, didn't dig into the implementation). Pretty significant issue is resolved here. Previously, all configs are combined to do version resolution - meaning if you have a newer version in your test classpath for example, a vulnerability in your runtime classpath won't be flagged. Now they isolate them (at least that's the goal!), which is a good thing.
The problem for us is that means vulnerabilities that are fixed in some configs are exposed in others. At the end of the day though, all we really care about (from a vulnerability perspective) is the runtimeClasspath. So we'll update the snyk invocation to filter the classpaths, which should resolve the issue and also make it a bit faster to run.


where does the endpoint showing in gif exist

as you can see , I can not click any of the element in this dashboard ,it seems freezing
but in the browser mode it seems works but still no endpoint like. examples/dashboard examples/cards

if I create a new one ,the freezing problem gone
(base) wenke@wenkedeMac-mini templates % pnpm create tauri-ui my-tauri-app --template vite
WARN  GET https://registry.npmjs.org/sisteransi error (ECONNRESET). Will retry in 10 seconds. 2 retries left.
.../Library/pnpm/store/v3/tmp/dlx-20381  |   +6 +
.../Library/pnpm/store/v3/tmp/dlx-20381  | Progress: resolved 6, reused 4, downloaded 2, added 6, done
✔ Select a framework: › ⚡Vite + SvelteKit
✔ Target operating systems for the Tauri Github Action › Windows (x64), macOS (x64), Linux (x64)
Scaffolding project in /Users/wenke/github/tiktoka-studio-app-tauri-ui/templates/my-tauri-app
Done. Now run:
cd my-tauri-app
pnpm i
pnpm tauri dev
(base) wenke@wenkedeMac-mini templates % cd my-tauri-app
(base) wenke@wenkedeMac-mini my-tauri-app % pnpm i


[Bug]: Uncaught exception while emptying recycle bin

Describe the Bug
When files are deleted outside of qBitManage it throws an uncaught exception while emptying the recycle bin.
It just keeps on looping after that throwing the same error over and over again.
I don't think it is that necessary to keep wanting to find the files it needs to delete. and keep looping until those files are found.
When your objective is empty the bin and the files are gone, why keep looping and searching for files that you thought where there? it is empty...
`
|================== Searching: /data/downloads/torrents/private/movies/.RecycleBin ==================|
| Deleted kings.of.mulberry.street.let.love.reign.2023.1080p.web.h264-edith.nfo from /data/downloads/torrents/private/movies/.RecycleBin (Last modified 2 days ago). |
| Uncaught Exception                                                                                |
Traceback (most recent call last):
File "/app/qbit_manage.py", line 563, in
start_loop()

File "/app/qbit_manage.py", line 322, in start_loop
start()

File "/app/qbit_manage.py", line 436, in start
stats["recycle_emptied"] += cfg.cleanup_dirs("Recycle Bin")

File "/app/modules/config.py", line 532, in cleanup_dirs
fileStats = os.stat(file)

FileNotFoundError: [Errno 2] No such file or directory: '/data/downloads/torrents/private/movies/.RecycleBin/torrents/private/movies/Kings.of.Mulberry.Street.Let.Love.Reign.2023.1080p.WEB.h264-EDITH/kings.of.mulberry.street.let.love.reign.2023.1080p.web.h264-edith.nfo'
`
Config
No response
Logs
https://gist.github.com/appiekap653/16d420d4d7edf9fcd4757734dc7ec509
Screenshots
No response
Installation
Docker
Version Number
3.6.1
What branch are you on?
master

It seems to be that qbit_manage is unable to delete those files and/or doesn't have the correct permissions to read those files. I've added additional logging and fixed the uncaught exception error so it shouldn't crash anymore.
Can you update to the develop branch and enable trace logging by adding the env variable QBT_TRACE=True

Well the reason is not permissions, the files are simply not there anymore. Like i said in the first post, the files where deleted outside of qbitmanage.
Are you keeping some kind of list of all the files in the bin? And delete them by going over that list?
If so, would it be an idea to just remove the items from your list of the physical files are not present anymore, instead of give an error?

Nope, qbit_manage gets the list of all the files in the bin at the time it's cleaning so it was seeing those files at the time of run. That's why I believe it's a permission issue since those files definitely do exist (or at least qbit_manage think they do exist) at the time of running.
Those trace logs would definitely help confirm that theory though.

well... to be honest, the trace logs doesn't say anything anymore about it, for two reasons:

The logs are showing only the last 2 hours while it happened yesterday.
those files are definitely not there since like i said in the beginning. I deleted them outside of qbitmanage, which started the looping error. but when I noticed yesterday that new torrents weren't getting tagged anymore I restarted the qbitmanage container and it kept looping on the error. hence my question if it stores the filelist of the bin...

What triggered the uncaught error is totally clear. I deleted some files that qbitmanage thought where there still.
What is wierd is that qbitmanage has an uncaught exception while he is emptying the bin. isn't an empty directory not the goal? why argue for files that should have been there and keep arguing and finally crash while if it has been there you would just deleted them...
Why not make it like: oh that file was supposed to be there, i need to delete it. someone else deleted it, i don't care, move on to the next...

Qbit manage sees that the files are there. Either way, this uncaught exception is now handled so itll be fixed in the next release

development_log


Z3 integer exception

Description
Getting this error using Mythril 0.20.2 for vulnerability detection:
$ myth -x Contract.sol
mythril.mythril [CRITICAL]: Exception occurred, aborting analysis. Please report this issue to the Mythril GitHub page.
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/mythril/mythril.py", line 592, in fire_lasers
    enable_iprof=enable_iprof,
  File "/usr/local/lib/python3.7/site-packages/mythril/analysis/symbolic.py", line 102, in __init__
    creation_code=contract.creation_code, contract_name=contract.name
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/ethereum/svm.py", line 151, in sym_exec
    self._execute_transactions(created_account.address)
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/ethereum/svm.py", line 190, in _execute_transactions
    execute_message_call(self, address)
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/ethereum/transaction/symbolic.py", line 58, in execute_message_call
    laser_evm.exec()
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/ethereum/svm.py", line 236, in exec
    new_states, op_code = self.execute_state(global_state)
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/ethereum/svm.py", line 285, in execute_state
    self._execute_pre_hook(op_code, global_state)
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/ethereum/svm.py", line 562, in _execute_pre_hook
    hook(global_state)
  File "/usr/local/lib/python3.7/site-packages/mythril/analysis/modules/integer.py", line 108, in execute
    func(state)
  File "/usr/local/lib/python3.7/site-packages/mythril/analysis/modules/integer.py", line 119, in _handle_add
    op0, op1 = self._get_args(state)
  File "/usr/local/lib/python3.7/site-packages/mythril/analysis/modules/integer.py", line 113, in _get_args
    self._make_bitvec_if_not(stack, -1),
  File "/usr/local/lib/python3.7/site-packages/mythril/analysis/modules/integer.py", line 186, in _make_bitvec_if_not
    stack[index] = symbol_factory.BitVecVal(value, 256)
  File "/usr/local/lib/python3.7/site-packages/mythril/laser/smt/__init__.py", line 128, in BitVecVal
    raw = z3.BitVecVal(value, size)
  File "/usr/local/lib/python3.7/site-packages/z3/z3.py", line 3741, in BitVecVal
    return BitVecNumRef(Z3_mk_numeral(ctx.ref(), _to_int_str(val), BitVecSort(bv, ctx).ast), ctx)
  File "/usr/local/lib/python3.7/site-packages/z3/z3.py", line 2925, in _to_int_str
    _z3_assert(False, "Python value cannot be used as a Z3 integer")
  File "/usr/local/lib/python3.7/site-packages/z3/z3.py", line 92, in _z3_assert
    raise Z3Exception(msg)
z3.z3types.Z3Exception: Python value cannot be used as a Z3 integer

Environment

Mac OS Mojave 10.14.3
Mythril 0.20.2
Solidity 0.4.25
Python 3.7.3

Hi @mahjiang , thanks for reporting the issue.By any chance can you post the contract code, or at least the minimal version of the code which might result in the error. That would be quite helpful

@norhh Can you try to reproduce this locally? Otherwise we'll have to close this issue

@mahjiang I can't unfortunately reproduce it locally, tried it with a lot of contracts, try running the contract with v0.20.4, If the problem still persists please feel free to reopen this issue.

@norhh I discovered multiple contracts that produce this error. Let me know if you would like other addresses for solving this issue.
To reproduce
Run myth -v4 analyze -1 --execution-timeout 3600 -a 0x126c473bf26360075e084555687dB4FF4B7C917e


Align label and icon of 'add certificate' button to spec (EXPOSUREAPP-8530)

Addresses EXPOSUREAPP-8530.
Not sure if the old behaviour was intended due to small device sizes ... If so, feel free to close the pr and please set the Jira issue to obsolute.

@UA text changed in the german and english translation. Please align this also with the other languages.

I'm afraid that this ticket should be obsolete, please refer to https://www.figma.com/file/KuDEcMSREg3VhZRhC3FXTY/COVID19_Master_2.5?node-id=24387%3A335

Ok. I set it to obsolete.


GO:0000976 transcription regulatory region sequence-specific DNA binding, align with SO

'GO:0000976 transcription regulatory region sequence-specific DNA binding' should be reformulated as
'transcription cis-regulatory region binding'
(The other rationale is that GO:0000987 cis-regulatory region sequence-specific DNA binding is essentially identical to GO:0000976 as things currently stand)
This will allow including chromatin binding, etc as children of GO:0000976.
@colinlog
Thanks, Pascale

GO:0001067 regulatory region nucleic acid binding
->  rename 'transcription regulatory region nucleic acid binding'

Also - what is the difference between

GO:0000976 transcription regulatory region sequence-specific DNA binding: Interacting selectively and non-covalently with a specific sequence of DNA that is part of a regulatory region that controls transcription of that section of the DNA. The transcribed region might be described as a gene, cistron, or operon.
and
GO:0000987 cis-regulatory region sequence-specific DNA binding: Interacting selectively and non-covalently with a specific upstream regulatory DNA sequence (transcription factor recognition sequence or binding site) located in cis relative to the transcription start site (i.e., on the same strand of DNA) of a gene transcribed by some RNA polymerase. The proximal promoter is in cis with and relatively close to the core promoter.
We dont have any 'trans' regulatory region binding; does that really exist ? ie trans regions exist, but 'trans region binding' implies that a protein links two trans strands; is this really a function?

All children terms should be part of 'cis-regulatory region' DNA binding:
GO:0000984   bacterial-type RNA polymerase transcription regulatory region   sequence-specific DNA binding
GO:0000987 cis-regulatory region   sequence-specific DNA binding
GO:0001046 core promoter   sequence-specific DNA binding
GO:0001161 intronic transcription   regulatory region sequence-specific DNA binding
GO:0001018 mitochondrial promoter   sequence-specific DNA binding
GO:0001019 plastid promoter transcription   regulatory region sequence-specific DNA binding
GO:0032422 purine-rich negative   regulatory element binding
GO:0001163 RNA polymerase I transcription   regulatory region sequence-specific DNA binding
GO:0000977 RNA polymerase II   transcription regulatory region sequence-specific DNA binding
GO:0001016 RNA polymerase III   transcription regulatory region sequence-specific DNA binding
GO:0001147 transcription termination site   sequence-specific DNA binding

Or maybe intronic regions are not considered cis ?

the only reason I can think of to keep both 'GO:0000976 transcription regulatory region sequence-specific DNA binding' and 'GO:0000987 cis-regulatory region sequence-specific DNA binding' is the existence of a phenomenon called transvection, whereby a regulatory region one homolog activates the allele on the other homolog. This is pertinent to organisms that pair the paternal and maternal homologs in the cellular nucleosplasm, such as dipteran insects (!). Transvection may account for quite a lot of compound phenotypes in this animal order. In humans, there have been recurrent reports of 'hubs' that may combine multiple regulatory regions. Whether these include inter-chromosomal contacts remains to be established, I think.
In summary, GO:0000987 requires that the interaction between enhancer/promoters to be on the same chromsome, this is often limited to one topologically associated domain (TAD), or two if the gene is at the boundary of two TADs. GO:0000976 does not stipulate _cis-. Perhaps GO:0000976 should be a "do not annotate term"? Then (Drosophila) cases of transvection should be housed under a 'transcription activation via regulatory DNA _trans_vection' child term of GO:0000976 while all the other annotations of GO:0000976 could migrate to its child GO:0000987 'cis-'?

but this is a process, not a binding ? Or are there proteins that bind both a cis- and a trans- region?

Recent paper on transvection: PMID:33479152
But we agree with @colinlog that transvection is a process and that binding is always in cis relative to a promoter.


Cut out warnings from imports due to numpy ufunc and dtype sizes

Nice jobs with the notebooks- On block 2, if you'd like to get rid of the RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88 warnings, the you can just add at the bottom of the block:
import warnings
warnings.filterwarnings("ignore", message="numpy.dtype size changed")
warnings.filterwarnings("ignore", message="numpy.ufunc size changed")

@AnshulBasia  This a great idea, you should do this.


Fix spheroid location, take 2

Continuation of gh-1537 by @applysomesky.

Looks like all the tests are passing, merging!

Do you have any idea why the tests failed for my last commit?


Feature/provide access to checkboxes validation

Terrible non-DRY handling (and don't even get started on the accessibility) of the property editing contextual menu shortcuts - that's why I'd held off making these fixes/improvements.
However, it's been biting Phil enough and now Adam, that needs must


Add files via upload

add engmet please

@shadyyd First stage of verification passed, official domain is engmet.edu.eg as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@shadyyd Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Enhancement Use update time of hive to avoid cache miss through multi fe nodes.

Proposed changes
Now the update time of hms table is generated by every FE node (Use System.currentTimestamp() separately), so the update time of a hms table may be different between FE nodes, always the same query can not hit the sql-cache if we submit it through the different FE nodes. This pr mainly do following changes to avoid this problem.

Use the transient_lastDdlTime instead of System.currentTimestamp as the schemaUpdateTime of hms tables
Use the eventTime in hms event instead of System.currentTimestamp as the update time when processing hms events

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Expose the version number of the current release in the GUI

https://bitsharestalk.org/index.php/topic,19417.msg249305.html#msg249305

This gets my vote, and a quick win.

Proposed UX


[Optimization] Optimizing Deprecated React Lifecycle

see：
https://legacy.reactjs.org/blog/2018/03/27/update-on-async-rendering.html
There are so many warnings indicating that we should use the new lifecycle instead of the old one, so I think it's necessary to split it into multiple tasks and make the changes one by one.

[ ] src/components/Authorized/PromiseRender.js

[ ] src/components/CountDown/index.js

[ ] src/components/Login/LoginTab.js

[ ] src/components/SideMenu/SideMenu.js

[ ] src/routes/Plugin/Common/Rule.js

[ ] src/routes/Plugin/Common/Selector.js

[ ] src/routes/System/Alert/index.js

[ ] src/routes/System/AppAuth/index.js

[ ] src/routes/System/Dict/index.js

[ ] src/routes/System/Metadata/index.js

[ ] src/routes/System/Plugin/index.js

[ ] src/routes/System/PluginHandle/index.js

[ ] src/routes/Sytem/Resources/index.js

[ ] src/routes/System/Role/index.js

[ ] src/routes/System/User/index.js

[ ] src/routes/System/User/DataPermModal.js

[ ] src/utils/AuthButton.js

[ ] src/utils/AuthRoute.js

Please assign src/routes/Plugin/Common/Rule.js and src/routes/Plugin/Common/Selector.js to me~


'default_app_config' is deprecated in Django 3.2.X

Hello, thank you for your project!
After upgrading to Django 3.2, there's a new deprecation warning, related to default_app_config in __init__.py:
  /Users/.../lib/python3.9/site-packages/django/apps/registry.py:91: RemovedInDjango41Warning: 'pagedown' defines default_app_config = 'pagedown.apps.PagedownConfig'. Django now detects this configuration automatically. You can remove default_app_config.
    app_config = AppConfig.create(entry)

Looks like Django will be removing this option in Django 4.1 — ref.

Added a fix for this:
https://github.com/timmyomahony/django-pagedown/commit/26fdf0452dbedd5c10e0d2cb6d34e66abd83b413
Will be included in the next release.


Fixed issue #1227 add icons on feedback.html

name: Pull_request_TEMPLATE.yml
about: use this template for creation of pull requests
title: '[FIX/Feat] '
labels: ''
assignees: ''

name: Pull Request Template
Description

This pr will add icons on feedback page.

Fixes #1227

Screenshots (if applicable)

before

After

Type of Change

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

Checklist

[ ] I have read the contributing document.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have added necessary tests to ensure my changes are effective.
[x] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Additional Information

why this pr has been closed??

because there were two unsuccessful checks and did not get why they are not check. After so many try to make them done, I create a new pr and closed this one.


sklearn should be scikit-learn

In requirements.txt, we have sklearn but the package has been called scikit-learn for some time. I think you can do a straight swap.

Thanks for the suggestion. I am closing this issue with the latest PR #6.


schema.table does not trigger its callback

There appears to be an issue with knex.schema.table where the callback is not being called. Per the documentation, I've written up a test case:
const knex = require('knex')({})

console.log('accessing table')
knex.schema.table('users', function (table) {
  console.log('inside') // this is never called
})

As a result, the schema building methods are not possible. Here's a demonstration - open your console to see the results.

@timwis You need to call either .toString(), .toSQL(), .then(fn) or .asCallback(callback) to trigger the compiler.
Knex won't start compiling until one of these functions are called, because you're still using the knex API to build up a query/procedure.

Oh, wow.. thanks @wubzz. Do you think it would be helpful for the documentation to mention that? The example for .table says "Outputs" underneath it as if the code listed there actually outputs that.
If so, I'm happy to submit a PR for it.


Run server in IBM Cloud

Get the server running on a free Kubernetes cluster in IBM Cloud. Part of #97.

[ ] push image to IBM Cloud Registry
[ ] define Kubernetes pod for running the server
[ ] port mapping, pass RMI port to the server
[ ] run pod in cluster

Free clusters expire after 30 days. I've got to make the setup repeatable, so I can run a different free cluster for each session with the students.

Registering for a Pay-as-you-go account got me $200 credit within the first 30 days. Should be plenty, with network traffic being the only paid service I'm planning to make use of. 30 days suffice to figure out how to run the server in Cloud, but the student sessions will be afterwards. We'll see.

I'm installing the IBM CLoud developer tools locally for trying out.
Eventually, I'd like to have GitHub workflows (actions) for deploying.

The images I build on my workstation with podman are not visible to docker and vice versa.
Setting up a GitHub workflow to push an image into IBM Cloud Registry will be a priority.
Maybe afterwards, I can get rid of the container-registry plugin and docker on my workstation again.

Here's how to authenticate against the IBM Cloud Container Registry for pushing images:
https://cloud.ibm.com/docs/Registry?topic=Registry-registry_access#registry_access_apikey_auth_other
The trick is to use "iamapikey" as the user name.
I should be able to push directly with podman from my workstation, or promote images from GitHub to IBM Cloud with skopeo in a GitHub workflow.

Pushing from my workstation works on the second attempt. It's "iamapikey", not "ibmapikey" ;-)

I got the pod running, but will need more time to figure out the port mappings.
For the Sockets exercise, I can re-map the server port arbitrarily.
For the Java RMI exercise, the ports in the container must be the same as those outside. That's because the internal port number appears in the stubs that Java RMI returns.

I've made the server ports and hostname configurable through environment variables.
But somehow, I cannot get the port mapping to work - not even for the Socket server port anymore.
Getting tired now... will try another day.

Duh... it works better when I use the external IP address of the worker node.
kubectl describe nodes | grep IP

  InternalIP:  10.x.y.z
  ExternalIP:  184.x.y.z


Parasomic Pixels

Learn coding and so much more

Hello there,
It appears this is a test issue/PR. For future cases, please test on your own repositories.
Testing on public repositories disturbs many developers. Specifically, 8,500 developers are watching this repo.


Version 1.4.3 is broken

Version 1.4.3 gives the following error.
Cannot find module '../dist/index'
Version 1.4.2 works fine with
npx<EMAIL_ADDRESS>list

@aurelien-lbc
Thank you for your reporting!
The release workflow was broken.
Now, v1.4.5 works correctly! 😄


Added Go-Back and Scroll-To-top functionality to Course Description page

Describe the changes you've made
Added Go-back on wishlist page functionality on click of back arrow button in course description page and Added scroll to top button also
Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
After

Checklist

[x] My code follows the code style of this project.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

Ok
On Wed, 31 Mar 2021, 11:11 Abhishek Kumar, @.***> wrote:

@Abhishek-kumar09 commented on this pull request.
In src/layouts/MainLayout/BottomNav.css
https://github.com/codeforcauseorg/edu-client/pull/169#discussion_r604607623
:

.Mui-selected {

font-size: 0.77rem !important;

font-size: 0.7rem !important;
}
}

@media screen and (max-width: 321px) and (min-width: 320px) {
.Mui-selected {

font-size: 0.67rem !important;

font-size: 0.6rem !important;
}
.MuiBottomNavigationAction-label {

font-size: 0.66rem !important;

font-size: 0.6rem !important;

revert this, fixed it in the latest code
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforcauseorg/edu-client/pull/169#pullrequestreview-624908040,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APLHP3SIHSMAD5TURWN6IHLTGKYYZANCNFSM42BUDSKQ
.

Hi @Abhijay007
I have changed the CourseDetail.js location in PR #177 so it would raise conflicts if either one of our PR's gets merged. So to resolve this issue you can move the file inside CourseDetail folder and rename it to index.js

ok, @faisaljamil25 I will do that thanks for letting me know about it !!


Getting closer to feature parity for the toolbar

List of Markdown syntax not yet replicated in toolbar:

heading 2-6 (heading 1 shall not be used so better not to encourage it - Heading 5 and 6 will probably not be used)
blockquote
lists, both ordered and unordered
code, both block (<pre><code>) and inline (<code>)
horizontal rule (I for one use it from time to time but if only HTML input is possible, I won't die, although it would break the principle of feature parity)
anchor links (probably tricker to handle UX-wise, low-priority)

Closing this as Monero forum is a different repository, but @SkepticalHippo is already working on additional buttons for it, so I've tagged him in this message

Created an issue on the correct forum.


Model validation does not happen for value types decorated with Required annotation

Please take a look at issue #468 for detailed information.
public class Person
{
	[Required]
	public int Id { get; set; }

	[Required]
	public string Name { get; set; }
}

Oh this is music to my ears! I wanted this to be resolve for a few years now. I'm happy that 6.0 will solve this. Still I see a little problem on fixing this. All versions of MVC behave like this, and I'm pretty sure most of us are already used to this. What about backcompatibility, how is going to be handled ? I don't think it could have a great impact but I can think of some scenarios. Using it 'wrong' and having [Required] on value types; using entities on your views so [Required] is used for EF and now this will propagated. Not sure if anything else. I guess MVC5 or less will not be affected with this fix.

Commits:
c276ddaa39735a8826b1da27b84dd793865a65e4
f737083c946137dd9a411759c3ca83856a0083a2


tests: re-enable Android e2e tests

Signed-off-by: Mike Schore<EMAIL_ADDRESS>

I think replacing all occurrences of engine.envoyConfiguration!! with engine.envoyConfiguration in the tests will fix the compilation errors.


[Fix-802] Widget-download-element

@yuvalbl tested on all widgets on staging preview, looks good to me.
@daniel-ilan can you double-check me pls?

opened a new one https://github.com/hasadna/anyway-newsflash-infographics/pull/847


Documentation Generation

The command-line slangc tool should support emitting formatted documentation for a module.
Documentation for declarations should be extracted from leading comments using one of the common conventions (e.g., triple-slash /// comment lines). The lexer should produce a unique token type for comments that follow documentation-comment conventions, and the parser should look for these comments when they appear before declarations and combine them into a Modifier attached to the declaration.
The actual documentation comments should probably be written in Markdown (that seems to be the way the wind is blowing, as opposed to more specialized syntax like Doxygen has used historically), which would allow us to pretty much just copy them verbatim into the output for a first pass (and eventually integrate a proper markdown parser if/when we need more advanced features).
There is a lot of prior art here both in Doxygen, and in the documentation-comment systems for Rust and Swift.

We already have some kind of doc gen.


TextureAtlas should re-use texture coordinates for existing subregions

TextureAtlas.addSubRegion allocates a new BoundingRectangle with texture coordinates every time it's called.  If coordinates have already been computed for the given image ID and subRegion, the existing coordinates (at the existing index) should be re-used.

CC #172

Would love to see this worked on during the summer 2017 code sprint!  (Also linking for reference: #2319 )

1 to get this finally fixed.

Thanks for the input!  We'll see if someone is interested at the sprint.


add routes information to events closes #63

@argelius @shprink  This should solve #63. @argelius let me know what you think. I don't like the fact i need to save an extra property this.routesBeforePop, but otherwise we would need to create the same events from the core. The problem is that the pop will be called before the then, thus this.routes is not accurate.

This could work yes but you are right maybe that would be a "core" feature more than a react one. I bet Angular version will need this as well :)

@shprink I spoke with @argelius and we will merge this now. Later on we might do something in the core.

thanks very much!


no cached .wasm files.I have a 9M .wasm file, download it every time.

It uses fetch, and in the browser its cache is fine.
How to cache it? I already set
cacheEnabled
domStorageEnabled
incognito={false}

how did you fixed?


[BUG] - Error starting "Process Data" when trying to calculate GABA edited MEGA PRESS on Siemens

Describe the bug
Loading the TWIX files (GABA edited sequence and non water suppressed reference scan) works perfectly and they are displayed correctly. When I go to "Process Data" I get the following error:
Arrays have incompatible sizes for this operation.
Error in op_eccKlose (line 47)
out.fids=out.fids.exp(1i-inph);
Error in OspreyProcess (line 376)
[raw,raw_ref]                   = op_eccKlose(raw, raw_ref);        % Klose eddy current correction
Error in osp_onProc (line 37)
MRSCont = OspreyProcess(MRSCont);
Related documentation
Error while evaluating UIControl Callback.
To Reproduce
Steps to reproduce the behavior:

Create Job file (enter GABA.dat and GABA_no_water_suppression.dat)
Load Data
Process Data

Expected behavior
Alignment and averaging of the spectra should be done and displayed
Screenshots
If applicable, add screenshots to help explain your problem.
Computer configuration (please complete the following information):

OS: Mac OS Seuoia15
Version Osprey 2.8.0 (downloaded today)
MATLAB version 2024A

Additional context
Thank you for supplying this great piece of software!

Hi @tkhauser,
Thanks for reaching out. Could you take a look at the number of points of the metabolite and the water scan? I am wondering if they have different number of datapoints.
It should be possible to fix this in the code, but I just wanted to be sure that this is the problem first.
Thank you,
Helge

After loading in Osprey, the metabolite scan shows Sz=2080 whereas the water scan shows Sz=2066.
2080 datapoints is also what Gannet gives me although it does not mention the water reference scan.
I am not sure I can set this in the scanning parameters.

Hi @tkhauser,
Sorry for the delayed response. Would you be able to share the data. It seems like an issue with the loader that needs fixing.
Best,
Helge

Hi Helge,
Thanx for looking into this!
I just shared the files via cloud link to your email adress as they exceed the upload limit.
best
Till


LRU forward pass in case of dense linear transition matrix instead of diagonal transition matrix

Hi,
First of all thanks for this great implementation of the LRU presented in  Orvieto et al. 2023.
I am wondering how would you implement the model's forward pass function in case we are using a linear RNN with real and dense transition matrix 'A' (instead of a complex diagonal one that is used in the final LRU version).
Is using parallel_scan will still be possible?
If the answer to my question is yes can you reply here with the implementation to this case?
If the answer is no, how would you implement it in a way it will be more efficient than non linear RNN ( in Orvieto et al. 2023 it is mentioned that moving from non linear RNN to linear RNN should allows us to parallelize training, but if we are not using parallel scan i don't see how the forward pass speed will be different between non linear and linear RNNs).
Thanks,
Tal

Hello,
Unfortunately, using parallel scan will not work anymore.
Prior work to the LRU used non-diagonal recurrence connectivity matrix such as S4. The trick here is to transform the recurrent layer into a convolutional layer, for more detail see e.g. https://srush.github.io/annotated-s4/.
Hope this is useful!
Nicolas


Add simplecov to track test coverage

This is an initial addition of simplecov to Dependabot. Currently no level of coverage is enforced, but a report is generated.
From here, we can at least track coverage, and decide how and at what level we want to enforce code coverage.
Related to #9203

I think currently this will dump a report to stdout for each test run, but since the testsuite is split up heavily it will be hard to get an overall sense of where we stand? It would be nice to think about a way we can track coverage across the entire codebase

It looks like we'd need a final step after all the unit test runs to collate the results. There are some instructions in the simplecov documentation, but I'd need to investigate more on this.

It looks like we'd need a final step after all the unit test runs to collate the results. There are some instructions in the simplecov documentation, but I'd need to investigate more on this.

Relevant article that might help
https://rubyyagi.com/merge-simplecov-circleci/

Also, I was wondering if we could see if there is any integration we could add to include this as a pull request comment with a link to the HTML report as that will be helpful to raise awareness. Can we leverage Github pages for this?

Also, I was wondering if we could see if there is any integration we could add to include this as a pull request comment with a link to the HTML report as that will be helpful to raise awareness. Can we leverage Github pages for this?

I'm not sure about GitHub pages, but I have seen the joshmfrankel/simplecov-check-action and [Codecov supports simplecov] as well. I'll need to look more into it.


Update in Contributing Guideines

Error in Contributing Guidelines  #1328

I have changed some commands and explained few commands which were earlier ambiguous.
Close  #1328

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

Changes are not correct


Consider adding a bpf-entrypoint feature to enable the entrypoint

Problem
As noted at https://github.com/paladin-bladesmith/sol-stake-view-program/pull/5/files#r1699203567, there are some competing ways of defining certain program features and then using them.
In this case, some programs specify a bpf-entrypoint feature, which enables the entrypoint for program builds. Historically, programs have used the no-entrypoint feature to disable the entrypoint, which is an antipattern for crate features, which should typically add functionality rather than remove.
Proposed Solution
Come up with a standard and use it here!
cc @febo @buffalojoec

I think the additive bpf-entrypoint feature is probably a good standard. Optionally we could try to pack a few things into platform tools, like automatically detecting this feature when building with cargo build-sbf and/or allowing some custom crate type (ie. crate-type = ["sbf"]) that could abstract some of the entrypoint boilerplate.
I also think we should consider renaming the feature to sbf-entrypoint. 😅

Hey guys, am I right to say this would be for Solana 2.0+? If so, we already have that information when we generate the template so we can add that flag for Solana 2.0+ only. Wdyt?

There's nothing specific to Solana v2, it's whether we want to create a new convention that will make it easier to share program crates. With that said, this might not be such a big issue when we start generating and publishing client crates.


Push Notifications

Is there any chance of adding support for Browser push notifications?
Using the Notifications API

Can be that the GridEditorBase could be modified and or extended .. I like the link you provided .. I would like to try it - but I am swamped  right now .. if you caught me 4 months ago I would have done it.

You may add it using that doc.

I'm also interested in using the serenity API, but I don't know how to do it.
@welcometoall
You got it? Could you show the way?


updated .gitignore

For Eclipse, IntelliJ and NetBeans
Fixes #
Proposed Changes

Added .gitignore for Eclipse and NetBeans Development
Updated .gitignore for Idea development

🧽 Update or clean up current behavior

@slinkydeveloper and @pierDipi I have made the changes as requested.

/lgtm
/approve

@ohbus Thanks and welcome to Knative Eventing Kafka Broker :tada:


harfbuzz:x64-windows build failure Vcpkg version: 2019.08.27-nohash

I have a run into a problem causing me to be unable to install harfbuzz - which is a prequisite to a number of other packages, like qt-base - that I want to try out.   Seems to occur when cmake tries to extract the harfbuzz tar.gz files....   details below...
Host Environment

OS: Windows 10  x64
Compiler: Microsoft Visual C++ 2019   v16.2.4

To Reproduce
Steps to reproduce the behavior:
./vcpkg install harfbuzz
Failure logs
-Cut and past the appropriate build messages from the console output.

vcpkg install harfbuzz

The following packages will be built and installed:
    harfbuzz[core,ucdn]:x64-windows
Starting package 1/1: harfbuzz:x64-windows
Building package harfbuzz[core,ucdn]:x64-windows...
-- Using cached C:/workspaces/repos/vcpkg/downloads/harfbuzz-harfbuzz-2.5.3.tar.gz
-- Extracting source C:/workspaces/repos/vcpkg/downloads/harfbuzz-harfbuzz-2.5.3.tar.gz
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
Command failed: "C:/Program Files/CMake/bin/cmake.exe" -E tar xjf
C:/workspaces/repos/vcpkg/downloads/harfbuzz-harfbuzz-2.5.3.tar.gz
Working Directory: C:/workspaces/repos/vcpkg/buildtrees/harfbuzz/src/TEMP
Error code: 1
See logs for more information:
C:\workspaces\repos\vcpkg\buildtrees\harfbuzz\extract-err.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_extract_source_archive.cmake:43 (vcpkg_execute_required_process)
scripts/cmake/vcpkg_extract_source_archive_ex.cmake:98 (vcpkg_extract_source_archive)
scripts/cmake/vcpkg_from_github.cmake:114 (vcpkg_extract_source_archive_ex)
ports/harfbuzz/portfile.cmake:3 (vcpkg_from_github)
scripts/ports.cmake:85 (include)
Error: Building package harfbuzz:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: harfbuzz:x64-windows
Vcpkg version: 2019.08.27-nohash
Additionally, attach any relevant sections from the log files above.

-Please attach any additional failure logs mentioned in the console output.

type C:\workspaces\repos\vcpkg\buildtrees\harfbuzz\extract-err.log

CMake Error: Problem with archive_write_header(): Can't create ''
CMake Error: Current file: harfbuzz-2.5.3/README
CMake Error: Problem extracting tar: C:/workspaces/repos/vcpkg/downloads/harfbuzz-harfbuzz-2.5.3.tar.gz

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.
My default triplet is x64-windows, so I also tried with x86-windows, but got the exact same issue.

This issue may be caused by insufficient permissions or by a cmake bug.
Please following the step:

Reinstall this port after re-clone vcpkg on a non-system disk.
Roll back your cmake version to 3.13.4.

@JackBoosY
Thank you very much for the links.  I found the following work-around described in thread: #19102
A Windows setting that causes this problem:
intl.cpl > Administrative > Change system locale...
☑ Beta: Use Unicode UTF-8 for worldwide language support
Unticking it fixes the cmake -E tar error for me, and I was able to get harfbuzz to install correctly.

Happy to close the issue, thanks to the work-around shared by @JackBoosY .  I mean, it would be better if it doesn't need the work-around, but at least I am happy for now.

@zewpo Please do not close this issue until we update cmake to the fixed version.
Thanks.

@zewpo Please do not close this issue until we update cmake to the fixed version.
Thanks.

OK, :) makes sense.  thanks

We have known issue https://github.com/microsoft/vcpkg/issues/8112 to track same issue, so it's safe to close it.


[Crash] RAND_load_file()

I'm using arrow/parquet to write encrypted parquet file. that RAND_load_file() is used in this PR. Actually my crypto.cc is a little different:
https://gist.github.com/thamht4190/b9fab0a7bd98d75eddaaca24382bd310.
Sometimes I got a crash at RAND_load_file. This is the stacktrace from dump file:
trace_rand_load_file.txt
I'm using OpenSSL version 1.1.1b. I got the issue on both MacOS X 10.14 and 10.13.
Can anyone have ideas about this issue? Thanks!

I have a more clear stacktrace now:
trace_rand_load_file_2.txt
Thread 4 (crashed)
 0  libsystem_pthread.dylib!pthread_rwlock_wrlock + 0x0
    rax = 0x0000000000000001   rdx = 0x0000000000000000
    rcx = 0x0000000000000001   rbx = 0x0000000000000000
    rsi = 0x00000001083fc150   rdi = 0x0000000000000000
    rbp = 0x000070000f9f1c00   rsp = 0x000070000f9f1bf8
     r8 = 0x00000000000130a8    r9 = 0x0000000000000000
    r10 = 0x00007fff96afb288   r11 = 0x00007fff96afb280
    r12 = 0x0000000000000000   r13 = 0x000070000f9f1d00
    r14 = 0x0000000000000020   r15 = 0x0000000000000020
    rip = 0x00007fff63effe14
    Found by: given as instruction pointer in context
 1  CortexSync!CRYPTO_THREAD_write_lock + 0x9
    rbx = 0x0000000000000000   rbp = 0x000070000f9f1c00
    rsp = 0x000070000f9f1c00   r12 = 0x0000000000000000
    r13 = 0x000070000f9f1d00   r14 = 0x0000000000000020
    r15 = 0x0000000000000020   rip = 0x0000000108428999
    Found by: call frame info
 2  CortexSync!RAND_get_rand_method + 0x36
    rbx = 0x0000000000000000   rbp = 0x000070000f9f1c20
    rsp = 0x000070000f9f1c10   r12 = 0x0000000000000000
    r13 = 0x000070000f9f1d00   r14 = 0x0000000000000020
    r15 = 0x0000000000000020   rip = 0x00000001083fc3d6
    Found by: call frame info
 3  CortexSync!RAND_add + 0x1b
    rbx = 0x000070000f9f1d00   rbp = 0x000070000f9f1c50
    rsp = 0x000070000f9f1c30   r12 = 0x0000000000000000
    r13 = 0x000070000f9f1d00   r14 = 0x0000000000000020
    r15 = 0x0000000000000020   rip = 0x00000001083fc65b
    Found by: call frame info
 4  CortexSync!RAND_load_file + 0x1c3
    rbx = 0x00007fff96afc030   rbp = 0x000070000f9f2230
    rsp = 0x000070000f9f1c60   r12 = 0x0000000000000000
    r13 = 0x000070000f9f1d00   r14 = 0x0000000000000020
    r15 = 0x0000000000000020   rip = 0x00000001083fd043

Any idea?

Well, its quite odd. The crash suggests that the rand_meth_lock is somehow not initialised or is otherwise invalid. Here is the code that crashes:
https://github.com/openssl/openssl/blob/130b7df2db7d35af75ddf56046afdd1a57a2aea8/crypto/rand/rand_lib.c#L737-L745
But we can see that immediately prior to aquiring the lock we do a "RUN_ONCE" call (which is just pthread_once) and call do_rand_init which initialises the rand_meth_lock. So I can't currently see how that lock could be invalid.

We are running multi threads that may call RAND_load_file() at the same. Is it allowed to do so?

Yes, this should be ok.
OTOH, I note that you are effectively calling RAND_load_file() to seed the DRBG with data from `/dev/urandom'. This should be totally unnecessary. The DRBG will automatically seed itself as required with no explicit action needed. So, while it doesn't explain the crash, one simple solution for you would be to remove the RAND_load_file() calls.

Thanks for your reply. If you need any more information, please let me know. I'll also try to debug on my side.
For RAND_load_file() use case, this is the answer from parquet crypto developer:

we need not just random, but a stronger version called "secure random" to prevent nonce repetition that breaks AES-CTR (and GCM) algorithms.
Loading the /dev/urandom is a standard way to do that, but I'll check for workarounds.
Any suggestion?

@mattcaswell Thanks for your input. What we are looking for, is a C++ (openSSL) version of Java's
https://docs.oracle.com/javase/8/docs/api/java/security/SecureRandom.html
Any suggestions would be appreciated.

we need not just random, but a stronger version called "secure random" to prevent nonce repetition that breaks AES-CTR (and GCM) algorithms.
Loading the /dev/urandom is a standard way to do that, but I'll check for workarounds.

The OpenSSL RAND implementation provides a secure random source. It must be properly seeded in order to work. One place it could get its random seed data from is "/dev/urandom". OpenSSL will do this seeding automatically (obtaining data from "/dev/urandom" if appropriate for the platform) and 99% of the time there is no need to seed manually using RAND_load_file() or similar calls. In very rare circumstances (such as an obscure platform that OpenSSL doesn't know how to or can't seed automatically on) then it may be necessary to manually seed. You'll know if that's the case because the OpenSSL RAND_bytes() call will fail. Doing that seeding from "/dev/urandom" is pointless because OpenSSL will already have tried that.
You almost certainly don't need to worry about any of that. Just call RAND_bytes() and forget about RAND_load_file().

Cool, thanks.

After getting some crash stack traces, I see almost the same stack that it's doing the cleanup step at a main thread:
Thread 0
 0  CortexSync!engine_cleanup_int + 0x0
    rax = 0x00000000ffffffff   rdx = 0x0000000000000307
    rcx = 0x0000000000000001   rbx = 0x0000000000000000
    rsi = 0x0000000000000307   rdi = 0x0000000000000000
    rbp = 0x00007ffee8c2ad00   rsp = 0x00007ffee8c2acd8
     r8 = 0x0000000000000021    r9 = 0x00007fe69ae60fd0
    r10 = 0x0000000000000022   r11 = 0x0000000000000022
    r12 = 0x00007fe69b017200   r13 = 0x0000000000000001
    r14 = 0x000000010894f260   r15 = 0x00000000000001c0
    rip = 0x00000001083b12a0
    Found by: given as instruction pointer in context
 1  CortexSync!OPENSSL_cleanup + 0x13b
    rbx = 0x0000000000000000   rbp = 0x00007ffee8c2ad00
    rsp = 0x00007ffee8c2ace0   r12 = 0x00007fe69b017200
    r13 = 0x0000000000000001   r14 = 0x000000010894f260
    r15 = 0x00000000000001c0   rip = 0x00000001083d7eeb
    Found by: call frame info
 2  libsystem_c.dylib + 0x5cef9
    rbx = 0x0000000000000000   rbp = 0x00007ffee8c2ad50
    rsp = 0x00007ffee8c2ad10   r12 = 0x00007fe69b017200
    r13 = 0x0000000000000001   r14 = 0x000000000000000f
    r15 = 0x00000000000001c0   rip = 0x00007fff63db5ef9
    Found by: call frame info
 3  libsystem_c.dylib + 0x5d1de
    rbp = 0x00007ffee8c2ad70   rsp = 0x00007ffee8c2ad60
    rip = 0x00007fff63db61de
    Found by: previous frame's frame pointer
 4  CortexSync!start + 0x3b
    rbp = 0x00007ffee8c2ad80   rsp = 0x00007ffee8c2ad80
    rip = 0x0000000106fd603b
    Found by: previous frame's frame pointer

I realize that I don't stop the thread where RAND_load_file() is called while the application is killed. But I'm not sure if this can cause crashes at RAND_load_file(). Can you help? @mattcaswell

The cleanup routines assume that there are no other threads doing OpenSSL stuff (if there are why are you trying to cleanup?). Are you explicitly calling OPENSSL_cleanup? Normally there is no reason to do this - it should happen automatically.

No, I don't call OPENSSL_cleanup explicitly. It's called automatically when my application is killed.

I'd suggest stopping all threads explicitly before closing the application and see if that helps.

@mattcaswell
i also encounter similar problem like  @thamht4190 , but i have encounter this problem only one time.
it has never happen again........
i called the func:RAND_bytes.
First, my platform is non-preemptive system.Only one task can be performed at the same time.
but i know  that rand_meth_lock can not be null while RAND_get_rand_method be called in one task.  because Only one task can be performed at the same time in my system .
if OPENSSL_cleanup was executed before the task, the var (rand_inited) will be set to zero(OPENSSL_cleanup() -> rand_cleanup_int() -> CRYPTO_THREAD_lock_free(rand_meth_lock)).
`void rand_cleanup_int(void)
{
const RAND_METHOD *meth = default_RAND_meth;
if (!rand_inited)
    return;

if (meth != NULL && meth->cleanup != NULL)
    meth->cleanup();
RAND_set_rand_method(NULL);
rand_pool_cleanup();

#ifndef OPENSSL_NO_ENGINE
CRYPTO_THREAD_lock_free(rand_engine_lock);
rand_engine_lock = NULL;
#endif
CRYPTO_THREAD_lock_free(rand_meth_lock);
rand_meth_lock = NULL;
CRYPTO_THREAD_lock_free(rand_nonce_lock);
rand_nonce_lock = NULL;
rand_inited = 0;
}
`
then in the func(RAND_get_rand_method) will execute RUN_ONCE to new the lock.
so i cannot understand why the lock can be null ??
is there any probability that the func(OPENSSL_cleanup) and the func (RAND_get_rand_method) was called in one time ?(but i have said that my system is Only one task can be performed at the same time). so i was confused .
is there any probability that the func(OPENSSL_cleanup)  executed fail then the func (RAND_get_rand_method) was called after?
such as
`void rand_cleanup_int(void)
{
const RAND_METHOD *meth = default_RAND_meth;
if (!rand_inited)
    return;

if (meth != NULL && meth->cleanup != NULL)
    meth->cleanup();
RAND_set_rand_method(NULL);
rand_pool_cleanup();

#ifndef OPENSSL_NO_ENGINE
CRYPTO_THREAD_lock_free(rand_engine_lock);
rand_engine_lock = NULL;
#endif
CRYPTO_THREAD_lock_free(rand_meth_lock);
rand_meth_lock = NULL;      ----exit(fail )????   is there any probability(this is my guess.....)
CRYPTO_THREAD_lock_free(rand_nonce_lock);
rand_nonce_lock = NULL;
rand_inited = 0;
}`
can you help?
thanks!

@mattcaswell  i know  you have add a OPENSSL_INIT_NO_ATEXIT
https://github.com/openssl/openssl/commit/8f6a5c56c17aa89b80fef73875beec53aef1f2c8#diff-a88baedd3b3f460db4620f1794f6ef48

@mattcaswell  ,i know  openssl has add a

@mattcaswell ,i know openssl has add OPENSSL_INIT_NO_ATEXIT  avoid atexit .the link is
https://github.com/openssl/openssl/commit/8f6a5c56c17aa89b80fef73875beec53aef1f2c8#diff-a88baedd3b3f460db4620f1794f6ef48

@mattcaswell ,hello，i know openssl has add OPENSSL_INIT_NO_ATEXIT avoid atexit .the link is
8f6a5c5#diff-a88baedd3b3f460db4620f1794f6ef48
but i wonder know that if  i do not OPENSSL_cleanup() while the system is reboot , What will happen?
and how can i stopping all threads explicitly before closing the application ?Do you have any ideas?
thanks~

@mattcaswell ,hello，I know openssl has add OPENSSL_INIT_NO_ATEXIT avoid atexit .the link is
https://github.com/openssl/openssl/commit/8f6a5c56c17aa89b80fef73875beec53aef1f2c8#diff-a88baedd3b3f460db4620f1794f6ef48
but i wonder know that if I do not OPENSSL_cleanup() while the system is reboot , What will happen?
and how can i stopping all threads explicitly before closing the application ?Do you have any ideas?
thanks~

@mattcaswell ,hello，I know openssl has add OPENSSL_INIT_NO_ATEXIT avoid atexit .the link is
8f6a5c5#diff-a88baedd3b3f460db4620f1794f6ef48
but i wonder know that if I do not OPENSSL_cleanup() while the system is reboot , What will happen?
and how can i stopping all threads explicitly before closing the application ?Do you have any ideas?
thanks~

but i wonder know that if I do not OPENSSL_cleanup() while the system is reboot , What will happen?

It really doesn't matter. OPENSSL_cleanup() explicitly frees some global memory and possibly other global resources. Those things will get freed anyway by the OS if the application aborts or for some reason OPENSSL_cleanup() is suppressed. The only reason we need OPENSSL_cleanup() at all is that many people run tools like valgrind to check for memory "leaks" - which they will see if OPENSSL_cleanup() hasn't been called. They're not true leaks in the sense that they don't grow with time - but it doesn't stop people from seeing them and believing them to be leaks.

thank you for your reply~thanks~~~

thank you for your reply~thanks~~~

This question seems to have been answered. Closing.


Fix content links creation/detection for V12

Can you also catch the one in grant item while you're at it? grant-item.hbs might also need the same data-link

Updated


Create release for v0.5 notes with features

After we have a github release, and automated release notes are generated, we want to have a more "human readable" release note form to show in https://developers.masa.ai/docs/masa-subnet/RELEASE_NOTES

Release notes available here.

Quick question here, aren't these human readable already? Or should this be more like marketing kind of readable?

https://github.com/masa-finance/masa-bittensor/releases/tag/v0.5.0

If so, I'd suggest we pass this task to stakeholders to perform the update based on the automated ones.

Quick question here, aren't these human readable already? Or should this be more like marketing kind of readable?

https://github.com/masa-finance/masa-bittensor/releases/tag/v0.5.0

If so, I'd suggest we pass this task to stakeholders to perform the update based on the automated ones.

This might be directly related to roadmap/issues#55, right? Should we close this since the update would be taken care of by the output of roadmap/issues#55?

Closing in favor of https://github.com/masa-finance/roadmap/issues/55


Hi I want pikachu

Hi, can you please add a Pikachu asset?
I think Pikachu makes my coding world full of happiness.

Highly reasonable <3

The Pokemon game sprites might work wonderfully here, but you must spend time making the gifs. Take a look I think it's very doable in a few days


Bumpy mypy versions

Bumping the mypy version required re-ordering some of the overload.
Bumping the flake8 version created new errors, so I used https://github.com/ambv/black to autoformat the *.pyi stub files. (This is also helpful because it's what i used for the autogenerated stubs for #11). Is this the right approach, or should we just manually format these stub files?

I do like the look of black formatted code here. It's not what we normally do for NumPy but I don't see any harm in trying it here. @sproshev any thoughts?

I do like the look of black formatted code here. It's not what we normally do for NumPy but I don't see any harm in trying it here

FYI we've used it in pandas-gbq, it's been great for not worrying about formatting. We've also rolled it out to our smaller / newer internal repos. Would be a mammoth diff for a project like xarray though

Thanks!


Add deploy scripts for business-support-finder

From alphagov-deployment (removed: https://github.gds/gds/alphagov-deployment/pull/1212).
https://trello.com/c/dKfYzDNx

👍


will it work for multivariate time series prediction   both regression and classification

great code thanks
may you clarify :
will it work for multivariate time series prediction   both regression and classification
1
where all values are continues values
weight    height     age   target
1   56        160    34    1.2
2   77        170    54    3.5
3   87        167     43   0.7
4   55        198    72    0.5
5   88        176     32   2.3
2
or  even will it work for multivariate time series where values are mixture of continues  and categorical values
for example 2 dimensions have continues values and 3 dimensions are categorical values
color        weight     gender  height  age  target

1    black       56             m   160    34    yes
2    white       77             f   170    54    no
3    yellow      87             m   167    43    yes
4    white       55             m   198    72    no
5   white       88              f   176    32    yes

There is only one dimension per .wsp file, so you'll need multiple files for multiple dimensions

There is only one dimension per .wsp file, so you'll need multiple files for multiple dimensions

Also whisper not really supports future datapoints IIRC (but maybe it's Graphite limitation, do not remember)

Also whisper not really supports future datapoints IIRC (but maybe it's Graphite limitation, do not remember)

You have to pass a custom now if you want to do that. https://github.com/graphite-project/whisper/blob/master/whisper.py#L667-L696

You have to pass a custom now if you want to do that. https://github.com/graphite-project/whisper/blob/master/whisper.py#L667-L696

For future references - issue above relates probably to https://github.com/openai/whisper and not this project. :)


Can't use chat window

Whenever I use the app currently, and after a clean install, I am unable to use chat. It constantly spams the chat window with the /hideout command. I have tried changing between the executable and the jar, removed all files and started fresh, and looked through the configs and have yet to find a way of using the program and still having the ability of using chat.
I hope this can be resolved soon. There is no attached log because there is no error. I am not using any other hotkey programs like trade macro or anything.

try to rebind /hideout hotkey (settings -> taskbar -> press on hotkey button -> press esc)

There seems to be no hotkey section in any config area within the taskbar. It also seems to be having issues connecting to the server to check for updates.

download latests version from github please, server are down

I am using the latest version updated 19 days ago according to your latestrelease link. I found a spot in the configuration.json file in the appdata folder. I removed it from being Enter by deleting the word. I hope this will work for the time being.

That resolved my issue.


possible testOverlap bug

Congrats on releasing v1! 🎉🙌
But one of my users reported an error with planck beta 16 that seems to be an issue with v1.0.1 as well. Can you guys look into it for me? 🙏🏻
Here are the messages they sent me:

hey @quinton_ashley could you please tell me what this error might mean. I keep getting this error inconsistently from time to time when I play my game. Like everything in the game seems to be working fine but then this pops up randomly and I can't play the game anymore... It wasn't happening until about a few hours ago. I have been trying to figure out what I've been doing wrong for hours but I can't

I would greatly appreciate it if you could help me

You're not doing anything wrong, it seems to be a bug in planck. I'll need to report it to planck's developers. What do you think triggers the error? Can you DM and send the code of the project you're working on?

Hi, here's the link to my project:
https://github.com/dhyan-rai/cs30-major-project/tree/main/rough-sketch
all of the code is in sketch.js and I have used a few other libraries as well. I'm sorry if it's a bit messy since I have recently begun coding and I don't quite know all the conventions yet... This is for my school project and I have been working on it for a few weeks and only a few days back did I see the error popping up. As to what might be causing it, I don't quite know. Nothing I have done in my code seems to be standing out as something that could cause an issue and besides, I don't understand the error all that well.
The error pops up very randomly... sometimes when I run the sketch and sometimes in the middle of running it

Thank you for creating the issue! This seems to be related to #269 (both throw because .getVertex return null in https://github.com/piqnt/planck.js/blob/master/src/collision/Distance.ts#L175 or https://github.com/piqnt/planck.js/blob/master/src/collision/Distance.ts#L178 probably)
Unfortunately I wasn't able to reproduce either (and I won't be able to look into them the next 3 weeks) Could you maybe check if the issue was already present in beta-15? It could be that this is some numerical issue I introduced with #267

hmm okay I'll ask them if they can try it with planck beta-15


Upgrade flux-standard-action to 0.6.1 to prevent "Unknown BABEL option"

redux-promise needs to upgrade flux-standard-action to prevent build errors.  Please refer to the following issue:
https://github.com/acdlite/flux-standard-action/issues/24

Fixed in 0.5.2


forwarding-construct/fc-port with wrong min-elements

The fc-port list in the forwarding-construct grouping is defined with min-elements 2, i.e. must have at least 2 elements.
This is not true for Forwarding Constructs which represent VLANs. They are usually created without member ports which are added afterwards.

For VLAN purposes I would agree that an "empty" VLAN should be represented as "empty" ForwardingConstuct.
A solution could be: by "pruning-and-refactoring" the min-elements as defined in CoreModel.UML could be "corrected".
@openBackhaul: please comment.

Please retest once a new core-model was delivered.

For avoiding warnings during compilation the "min-elements 0;" statement has been removed.
RFC7950: "If no "min-elements" statement is present, it defaults to zero."


add GPT-SoVITS-beta

thks

Yes, definitely, I am looking into it.

The application is too heavy to be added yet. I'd definitely need some help.

Fighting


PHPStan fixes: 27

@dstockto It looks like you reverted a bunch of fixes in OAuthModel. Mind double-checking what happened there?
Also, please incorporate 62b30e935dbb0233fb298c6f45b19ccfe5641075; I tried to push to your fork but it didn't work.

I'll take a look at the other stuff on the oauthmodel later - I am unsure why there were any of those changes in there. I definitely didn't change it manually.

Hi, because of https://github.com/iansltx/joindin-api/pull/17 some of the talk related user_id changes here are most likely not needed

Can't seem to push here, so I'll merge this in and fix on my own branch /shrug


有没有GAIA的交流社区？

Sorry, there is no such platform yet.

您好：
因为我们没有条件去搜集海量不相干数据，也没有大算力平台支持上游模型的训练，请问咱们这个GAIA-det的超网预训练模型开源了吗？我们像基于您上游超网预训练模型测试具体下游检测任务。
致敬！
B.R.
RayChiu
Mobile:157 1032 1221
@.***
发件人： Qing Chang
发送时间： 2022-04-08 14:02
收件人： GAIA-vision/GAIA-det
抄送： raychiu0202; Author
主题： Re: [GAIA-vision/GAIA-det] 有没有GAIA的交流社区？ (Issue #9)
Sorry, there is no such platform yet.
―
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

We already provided the supernet checkpoint trained on COCO and Obj365.

We already provided the supernet checkpoint trained on COCO and Obj365.
Yes, I saw it, but I'm not sure how to run it. Will more detailed documents be provided later? Or I can get your guidance in some way, and then I form a document to share with others

Yes, I saw it, but I'm not sure how to run it. Will more detailed documents be provided later? Or I can get your guidance in some way, and then I form a document to share with others

We already provided the supernet checkpoint trained on COCO and Obj365.

Yes, I saw it, but I'm not sure how to run it. Will more detailed documents be provided later? Or I can get your guidance in some way, and then I form a document to share with others

You can add qq<PHONE_NUMBER>

You can add qq<PHONE_NUMBER>


TechDocs: Use EntityRefLink in header and use relations to reference the owner of the document

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Niiice!  Thanks again @Fox32.


Fixes ip allocation for multi bridge networks

Do not discard errors on ip allocation for gw and bridge
Release addresses on network delete
For pre-provisioned networks, track availability
Add some context on top of ipallocator returned error

Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

@aboch

For pre-provisioned networks, track availability

Why is this required ?
The existing code checks for network availability in the local host by scanning through all the interfaces. Hence there is no chance for an overlap if that is the concern. So, am not quite sure the reason for this particular requirement.

@mavenugo That's correct, there should not be any need to track network availability.
I ended up into doing this so that test framework could pass, given ipAllocator in bridge is a global var initialized at package init, and its db is carried over the several tests.
Now I see its initialization is the real issue: While we still need it to be a package variable (so that the step functions can access it), its initialization should be done in newDriver() call. I tested the new change and they are fine.
Thanks,
I will push the new version soon.

@mavenugo Taken care of your comment. PTAL when you get a chance.
(Because are some glitches in github, CircleCI is not able to pull some tool, and fails)

Thanks @aboch . LGTM

LGTM


Classifier

classifier
This is a framework of sorts for transfer learning using Hugging Face pre-trained models for sequence
classification, e.g., text classification.
The base_class does most of the heavy lifting, however, it must be subclassed to provide a few
model and data loading methods. Implemented are models based on

bert-base-uncased
roberta-base
distilbert-base-uncased

The Hugging Face models can be downloaded from their 'models' site.

If you do have a required model, the underlying library will attempt to
download the required files. These will be cached in ~/.cache.

Configuration
The classifier is driven entirely by a configuration file such as the ones
in the examples/ directory.
The schema for the YAML configuration is shown below.
NB: All the
configuration items must be present. If None is allowed as a value, use the
~ character in the YAML to indicate None. Strict type checking is enforced as well as checking
for obvious errors.

Name
Type(s)
Description

log_id
str
Name of the log file

model_name
str
A valid Hugging Face model name

epochs
int
Number of epochs for model training

batch_size
int
Batch size; should be a power of 2

random_state
int
Sets the random number seed; can be None

checkpoint_path
(str, None)
Where to write the model file.

tensorboard_path
(str, type_none)
Directory for tensorboard's fevents files.

num_labels
int
The number of unique labels

split
float
A number in (0, 1)

warmup_steps
(float, None)
Applied to the scheduler

lr
float
Learning rate

weight_decay
float
See the AdamW documentation

eps
float
Stopping criteria for the optimizer

clip_grad_norm
(float, None)
Limits the gradient to a function of this value

drop_last
bool
If True, drop the last batch if it is incomplete

truncate
bool
If True, truncate to the max length. Recommend False

max_seq_len
(int, None)
If None, this value is computed from the encoded text. Otherwise, this value is used

The schema is in utils/config.py. Changes to the schema are encouraged.
Examples
The examples/ directory contains sample CLI for training the CoLA data set.
General usage is
usage: python example_cola_cli

Trains the `bert-based` models on the CoLA data set

optional arguments:
  -h, --help            show this help message and exit
  -c CONFIG_YAML, --config-yaml CONFIG_YAML
                        path to the `.yml` configuration file
  -d DATA_FILE, --data-file DATA_FILE
                        path the training data
  -m {bert,roberta,roberta-nd,bert-nd,distilbert}, --model-type {bert,roberta,roberta-nd,bert-nd,distilbert}
                        supported model type; default is `roberta'
  -t TRUNC, --truncate TRUNC
                        use rows up to this value; 0 indicates the entire data
                        set

If all goes well, you'll see a summary of the configuration parameters, and the training will begin:
[config.py:184 - log_config()], ------------------------------------------------------
[config.py:199 - log_config()],                class : DistilBertClassifier
[config.py:199 - log_config()],              version :  0.7.4
[config.py:199 - log_config()],               config : sample_distilbert_gc_config.yml
[config.py:199 - log_config()],               log id : distilbert-gc-example-dodd.log
[config.py:199 - log_config()],           model name : distilbert-base-uncased
[config.py:196 - log_config()],               epochs :       1
[config.py:196 - log_config()],           batch size :       8
[config.py:196 - log_config()],         random state :      42
[config.py:193 - log_config()],      checkpoint path :   None
[config.py:193 - log_config()],     tensorboard path :   None
[config.py:196 - log_config()],           num labels :       2
[config.py:202 - log_config()],                split :   9.0E-01
[config.py:196 - log_config()],         warmup steps :       0
[config.py:202 - log_config()],                   lr :   1.0E-05
[config.py:202 - log_config()],         weight decay :   0.0E+00
[config.py:202 - log_config()],                  eps :   1.0E-08
[config.py:202 - log_config()],       clip grad norm :   1.0E+00
[config.py:196 - log_config()],            drop last :       0
[config.py:196 - log_config()],             truncate :       1
[config.py:196 - log_config()],          max seq len :     256
[config.py:199 - log_config()],           model type : distilbert
[config.py:199 - log_config()],               device :   cpu
[config.py:199 - log_config()],        training data : dodd_resp_test.csv
[config.py:196 - log_config()],     training samples :   3,541
[config.py:196 - log_config()],   validation samples :     394
[config.py:199 - log_config()],            optimizer :  AdamW
[config.py:196 - log_config()],          total steps :     443
[config.py:209 - log_config()], ------------------------------------------------------
[classifier.py:305 - train()], into the breach...
[classifier.py:310 - train()],          ==================== Epoch   1 /   1 ====================
[classifier.py:353 - _train_batch()], 	batch     1 /   443 	loss : 0.633	elapsed : 0:00:08

Prediction
The commandline example predict_cli.py reads a model from a checkpoint directory and runs a user-supplied
.csv to effect predictions. Usage is documented in that script.
TensorBoard
If tensorboard_path is a valid directory, events are logged for use in TensorBoard. Tensorboard
is started from the command line. If you've run the example
$ tensorboard --logdir=text_classif/examples/tensorboard/

You should see something very close to
Serving TensorBoard on localhost; to expose to the network, use a proxy or pass --bind_all
TensorBoard 2.4.1 at http://localhost:6006/ (Press CTRL+C to quit)

Point your browser to the TensorBoard URL to visualize various metrics. See the torch recipe here.
Saving and Loading your Trained Models
The configuration file has the entry checkpoint_path and if this is not None, the model is written to
a subdirectory suffixed by _epoch_1, etc., one for each epoch.
The example examples/predict_cli.py shows how to load a specific checkpoint and use it to classify a
set of texts. The raw_text of a document can be converted to a .csv of sentences using utils/raw_text2csv.py.
predict_cli.py will optionally write a .csv with predicted classes, its likelihood as well as all the
columns in the input .csv.
See the documentation in predict_cli.py for usage.
Test Data
The benchmark test data is the Corpus of Linguistic Acceptability (CoLA) and is included in the tests/test_data/cola_public
directory. The full data set has 8,551 sentences labeled as 0 or 1 (not grammatical, grammatical).
See Warstadt et al., (2019) for additional detail.
The directory test_data/responsibility has Gamechanger-specific data that can also be
used for testing purposes.
Training
example_cola_cli.py will train a classification model on the CoLA data.
For CONFIG_YAML, the file sample_distilbert_cola_config.yml has the necessary configuration. The
DATA_FILE should point to /tests/test_data/cola_public/raw/in_domain_train.tsv. Finally, -m distilbert
will train a model using distilbert-base-uncased.
The sample configuration file should be updated if you wish to save a checkpoint or change the
model parameters such as the number of epochs.
Logging and Metrics
Console and/or file logging is supported via utills/initialize_logger().
See examples/example_gc_cli.py for example usage. You'll need to alter the
configuration file to accommodate your paths and names.
At the end of each epoch, a scikit-learn classification report is logged as well
as a formatted confusion matrix.
Subclassing
Since different models
require their own transformers class for tokenizing and sequence classification,
subclassing Classifier allows different models to be trained and tested.
A subclass must implement the following method:

load_model_tokenizer()

The modules roberta_classifier.py, bert_classifier.py, and distilbert_classifier.py are examples of subclasses. These
can be further subclassed for your custom methods.
Example Metrics
[classifier.py:511 - _validate()],

              precision    recall  f1-score   support

           0      0.789     0.418     0.546       268
           1      0.782     0.949     0.857       588

    accuracy                          0.783       856
   macro avg      0.785     0.683     0.702       856
weighted avg      0.784     0.783     0.760       856

[classifier.py:512 - _validate()], confusion matrix

	         pred: 0  pred: 1
true: 0      112      156
true: 1       30      558

[classifier.py:513 - _validate()], 	validation loss : 0.534
[classifier.py:514 - _validate()], 	            MCC : 0.457
[classifier.py:515 - _validate()], 	            AUC : 0.683
[classifier.py:516 - _validate()], 	 accuracy score : 0.783
[classifier.py:517 - _validate()], 	validation time : 0:01:48
[checkpoint_handler.py:30 - write_checkpoint()], saving model with loss : 0.534

License
The MIT License (MIT)
Copyright © 2021 Chris Skiscim, Amina Jackson; Booz Allen Hamilton, Inc.
Permission is hereby granted, free of charge, to any person obtaining a copy of
this software and associated documentation files (the "Software"), to deal in
the Software without restriction, including without limitation the rights to use,
copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the
Software, and to permit persons to whom the Software is furnished to do so,
subject to the following conditions:
The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.
 IN NO EVENT SHALL THE AUTHORS OR
COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE

OK on the (c). It is copyrighted - note that the MIT License states:
The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.
In a former life, an IP lawyer's guidance was to always state 'copyright' otherwise the license cannot be enforced. We should ask about this.

We have had many conversations with legal about this and the resolution was to not put any copyright wording.

Closed due to lack of interest by the reviewers. Functional updates are pending.


Review whether we want retry logic for certain status codes & whether we want to log them

Right now any non-successful status code is a permanent failure - you'll need to reload the whole page to try again. Perhaps building retry logic for stuff like 500 status codes may not be worth the maintenance cost, but we should review.

Code pointers/context:
Currently all failures are considered permanent due to this logic: https://github.com/SatcherInstitute/health-equity-tracker/blob/a5b3bcea954147e3460b21d11dc28517f0f1174a/frontend/src/data/loading/DataManager.ts#L95
Retries could be implemented at the DataFetcher level for short-term retries (like try 3 times in a row over the course of a minute), or at the DataManager level for long-term retries (like this can retry the next time it's loaded since it last failed 3 hours ago)


1.0 Tabs documentation has incorrect initialization

The 1.0 documentation shows the following for initialization:
  var instance = new M.Tabs(options);

Looking at the M.Tabs constructor, it takes two arguments. It should be:
  var instance = new M.Tabs(el, options);

Fixed in 19cfa8d


Why does my integer attribute sometimes save as 65535?

I don't know if this is a issue with tinyxml2 or if I have to look elsewhere.
I noticed in my XML file (created with tinyxml2) that sometimes one of the attributes is set to 65535:
<Item Description="Initial Call">
    <Name Counsel="65535">Happy Junior</Name>
    <Type>#1 Student (Main)</Type>
</Item>

The Counsel attribute is supposed to be either "" or a small number from 1 to 51.
The attribute is set like this in the code:
pStudent->SetAttribute(kCounsel, m_aryWStudentStudyPoints[INDEX(eSchool)][INDEX(StudentAssign::kBibleReading)]); // AJT v16.0.3

The variable m_aryWStudentStudyPoints is declared in the header like this:
CWordArray m_aryWStudentStudyPoints[static_cast<int>(Schools::kCount)];

The constructor sets the sizes of each array:
m_aryWStudentStudyPoints[INDEX(Schools::kMain)].SetSize(INDEX(StudentAssign::kCount));
m_aryWStudentStudyPoints[INDEX(Schools::kClass1)].SetSize(INDEX(StudentAssign::kCount));
m_aryWStudentStudyPoints[INDEX(Schools::kClass2)].SetSize(INDEX(StudentAssign::kCount));

Then later on, there is a Init method that sets each entry to a default:
// Student Main school
SetStudentBibleReader(Schools::kMain, _T(""), kStudyNone);
SetStudentInfo(Schools::kMain, StudentAssign::kItem1, _T(""), _T(""), kStudyNone);
SetStudentInfo(Schools::kMain, StudentAssign::kItem2, _T(""), _T(""), kStudyNone);
SetStudentInfo(Schools::kMain, StudentAssign::kItem3, _T(""), _T(""), kStudyNone);

// Student Class 1
SetStudentBibleReader(Schools::kClass1, _T(""), kStudyNone);
SetStudentInfo(Schools::kClass1, StudentAssign::kItem1, _T(""), _T(""), kStudyNone);
SetStudentInfo(Schools::kClass1, StudentAssign::kItem2, _T(""), _T(""), kStudyNone);
SetStudentInfo(Schools::kClass1, StudentAssign::kItem3, _T(""), _T(""), kStudyNone);

// Student Class 2
SetStudentBibleReader(Schools::kClass2, _T(""), kStudyNone);
SetStudentInfo(Schools::kClass2, StudentAssign::kItem1, _T(""), _T(""), kStudyNone);
SetStudentInfo(Schools::kClass2, StudentAssign::kItem2, _T(""), _T(""), kStudyNone);
SetStudentInfo(Schools::kClass2, StudentAssign::kItem3, _T(""), _T(""), kStudyNone);

Thereafter, the value is maintained as the user picks a different value from a list.
The only other thing to mention is that they may be opening a saved data file. That is using the CArchive::Serialize method:
m_aryStrStudentAssignNames[0].Serialize(ar);
m_aryStrStudentAssignNames[1].Serialize(ar);
m_aryStrStudentAssignNames[2].Serialize(ar);

So, do you know why it is that I managed to end up with a 65535 value in the XML file? Thanks.
MFC
VS2015
Dialog application

Sorry, just too much code to work through there. Closing pending a (much) smaller test case.

You can delete this. The issue was nothing to do with the TinyXml library but an dialog combo that was not initialised and set to -1 (which is 65535).
Please delete the thread.


Aggregate some prometheus stats

I have a problem visualizing some of the envoy's prometheus metrics in grafana.
Currently, there are metrics like:
# TYPE envoy_cluster_outlier_detection_ejections_consecutive_5xx counter
# TYPE envoy_cluster_outlier_detection_ejections_detected_consecutive_5xx counter
# TYPE envoy_cluster_outlier_detection_ejections_detected_consecutive_gateway_failure counter
# TYPE envoy_cluster_outlier_detection_ejections_detected_failure_percentage counter
# TYPE envoy_cluster_outlier_detection_ejections_enforced_consecutive_gateway_failure counter
# TYPE envoy_cluster_outlier_detection_ejections_enforced_total counter
# TYPE envoy_cluster_outlier_detection_ejections_overflow counter
# TYPE envoy_cluster_outlier_detection_ejections_total counter

This is cumbersome to visualize, because we need to create separate queries for each of these metrics.
A more convenient way is to use query with wildcard in metric name (via regexp on __name__ label), but this does not work because of labelsets being the same in all of the metrics (this is the way rate() function works in prometheus).
A better, "prometheus way", would be to merge all of the above metrics into one with extra label, like this:
# TYPE envoy_cluster_outlier_detection counter
envoy_cluster_outlier_detection_ejections{type="consecutive_5xx"}
envoy_cluster_outlier_detection_ejections{type="detected_consecutive_5xx"}
...

This could be also addressed in a compatible way just by adding a label and leaving metrics names as they are:
envoy_cluster_outlier_detection_ejections_consecutive_5xx{type="consecutive_5xx"}
envoy_cluster_outlier_detection_ejections_detected_consecutive_5xx{type="detected_consecutive_5xx"}
...

or even adding whole metric name as a some label everywhere.
PS. There are more such cases, like envoy_cluster_health_check_*.

As a workaround, one can use following configuration snippet:
stats_config:
  stats_tags:
    - regex: "^().+\\.([^\\.]+)"
      tag_name: name

what produces metrics like:
envoy_cluster_outlier_detection_ejections_consecutive_5xx{name="ejections_consecutive_5xx"}
envoy_cluster_outlier_detection_ejections_detected_consecutive_5xx{name="ejections_detected_consecutive_5xx"}

If this is the way it should be done, please close this issue.

cc @ggreenway @jmarantz for comment.

I agree that many prometheus metrics aren't aggregated in the best way. This happens because prometheus isn't the primary way that stats are organized; statsd is the native format. The current aggregations are based on a default set of regular expressions, which sometimes miss things like this.
But I'm worried that changing the default aggregation will introduce compatibility issues.
Long-term, I'd prefer if the internal representation of stats became more structured, instead of starting with the statsd format and trying to extract information from that name to aggregate. But that would be a big project, and I don't know if it would be worthwhile or not.
One option to fix things like this while not being incompatible would be to add a prometheus stats version somewhere to the config, and come up with a better set of regexes that we want to use going forward, but keep the existing ones for users who don't want to change. Or maybe even just have a wiki page of common regex config to add for different, common use-cases.

if we do use a new set of regexes to improve prometheus grouping, let's definitely use re2. I have a start to that here: https://github.com/envoyproxy/envoy/pull/8831

@bartebor I know its the not right place to ask about this but how did you manage to enable the envoy_cluster_outlier_detection_ metrcis, I cant see them on /stats/prometheus but I can see the other envoy_cluster_circuit_breakers metrics?

@reda-ayoub I didn't do anything special, it just works. Have you enabled outlier detection in your cluster configuration?

@reda-ayoub @bartebor i need to enable /see these outlier detection metrices somewhere or in prometheus for visualization. i can see those values(total/active outlier detection) in istio-pilot GET stats but need somewhere to track / visualize them.

@bartebor I'm using envoy through Istio. I installed the last istio release 1.15.1 using Helm on AWS EKS. I'm still not able to see those metrics. I can say that the outlier detection is working because I can see that some pods does not receive any traffic when they trigger the circuit breaker. How did you install/setup envoy?

@reda-ayoub I hava a custom envoy installation with own control plane and configuration, but it should not matter in this case. Note that circuit breaker and outlier detection are separate functions. Make sure you have outlier detection enabled in cluster config. If outlier detection stats are visible in envoy's administration interface (/stats, /stats/prometheus)  like @ankitdiscovery said above they should be also in your telemetry.  It is also possible that some of the metrics are dropped or modified with stats_config in envoy's bootstrap. It may be also due to lack of graphs visualizing those stats; I have no experience with istio, so I can't tell.


Make sure to clone deep actions we receive from remote processes

@chrisnojima Had the original fix to this, but this makes sure that if we have nested objects we don't run into even more subtle bugs.
bugs like: remote process A sends an action with payload {foo: {nested: objData}}. Then remoteProc A is killed and now when we try to access objData we get a nasty, not very helpful, error telling us the object is dead.
@keybase/react-hackers

:+1:


DOCS-1204: Fixes duplicate prereq

Product Version(s):

Issue:

https://tigera.atlassian.net/browse/DOCS-1204
Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

@bmckercher123 PTAL


Bug with yarn version 3: impossible to execute

Hello.
Firstly, thank you for the work on this wrapper.
Today, we have seen a problem at execution.
Here is the stackTrace:
/Users/.../testProject/node_modules/shellcheck/shellcheck-stable/shellcheck:1
����

SyntaxError: Invalid or unexpected token
    at Object.compileFunction (node:vm:352:18)
    at wrapSafe (node:internal/modules/cjs/loader:1033:15)
    at Module._compile (node:internal/modules/cjs/loader:1069:27)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
    at node:internal/main/run_main_module:17:47

After a lot of investigations, we have tested with a new project in these configurations:

npm version 8.5.5: OK
yarn version 1.22.18: OK
yarn version 3.2.2 (in zero install mode): KO

For the moment, we have founded a dirty but functionnal workaround, in the scripts use a path from node_modules:
 "shellcheck": "node_modules/shellcheck/shellcheck-stable/shellcheck *.sh",
Could you fix this bug ?

Can you post the steps to reproduce? Do you get this error when installing or when trying to run shellcheck?

The steps are:

creating new project from scratch with yarn following the install guide  (you can also have an installation of yarn 1 and make an upgrade with yarn set version stable )
create a script file at project root touch testScript.sh
create a script in package.json  "lint:shellcheck": "shellcheck testScript.sh"
run the script via yarn lint:shellcheck
then the previous error is displayed

Are the steps clear enough?
Were you able to reproduce the problem?

Steps are clear. I don't use yarn zero-install myself so I can't say that l'll have time to try and reproduce it in the near future but if you (or anyone) find a fix I'm happy to merge the PR.
If you really want/need my help, what you could do to help me help you is provide a Dockerfile that reproduces the bug. That'd make it easier for me to find and fix the bug Mr. Camus. (If indeed there is a solution that works with Yarn Zero-Install).

Please, see PR #34

This will soon be released in #34.