Type information lost when using opaque subtype constraint for another opaque type

flow code
bytes.js
export type BytesRange = 1 | 2 | 3 | 4 | 5 | 6 | 7 | 8 | 9 | 10 | 11 | 12 | 13 | 14 | 15 | 16 |
    17 | 18 | 19 | 20 | 21 | 22 | 23 | 24 | 25 | 26 | 27 | 28 | 29 | 30 | 31 | 32

export opaque type Bytes : string = string

export opaque type BytesN<T: BytesRange> : Bytes = Bytes

address.js
...

export const bytesToAddressTruncate = (bts: Bytes): Address => {
    ...
};

expected result
It should be possible to pass a BytesN<BytesRange> Into a function taking Bytes, such as bytesToAddressTruncate above.
actual result
Values are considered strings, ie the subtype's subtype - similar to how you couldn't pass them through functions earlier (https://github.com/facebook/flow/issues/4640).
Error: src/abi/Event.js:95
 95:             address = bytesToAddressTruncate(b);
                                                  ^ string. This type is incompatible with the expected param type of
 46: export const bytesToAddressTruncate = (bts: Bytes): Address => {
                                                 ^^^^^ opaque type `Bytes`. See: src/types/address.js:46

spec
OS
Windows 10, 64 bits
Node.js
>node --version
v8.2.1

flow
>.\node_modules\.bin\flow.cmd version
Flow, a static type checker for JavaScript, version 0.53.1

Can you show me how you are constructing the value that gets passed in as (bfs: Bytes)?
I think that type error is telling you that you're passing in a string rather than a Bytes or a BytesN. Constructing instances of those opaque types can only happen via functions defined in the same module as the opaque type, so just calling bytesToAddressTruncate("some literal string") is expected to not be able to match the opaque type.

It's done in a function in bytes file, though now that I investigate more, a cast right above the call is succeessful, so this is not the problem. The problem seems to be in the bytesToAddressTruncate function itself, or perhaps one of the functions it calls in turn. It does not seem to be a problem with opaque types, I just did not understand the error report. Closing.

yep. i got this error even after ensuring that the function was never called from anywhere else:
Error: src/types/address.js:49
 49: export const bytesToAddressTruncate = (bts: Bytes): Address => {
                                                 ^^^^^ string. This type is incompatible with
 49: export const bytesToAddressTruncate = (bts: Bytes): Address => {
                                                 ^^^^^ opaque type `Bytes`

the return type annotation in a nested function had been left out.

This is a bit bizarre.
bytes.js
export opaque type Bytes : string = string

address.js
export const test1 = (bts: Bytes): number => {
    if (bts.length < 40) {
        return 1;
    } else if (bts.length === 40) {
        return 2;
    } else {
        return 3;
    }
};

This produces an error:
Error: src/types/address.js:63
 63: export const test1 = (bts: Bytes): number => {
                                ^^^^^ string. This type is incompatible with
 63: export const test1 = (bts: Bytes): number => {
                                ^^^^^ opaque type `Bytes`

But if i change the second condition to use == instead of ===, i.e. else if (bts.length == 40), it starts working.

I have another example of weird behavior as well now. There may actually be something here.
numbers.js
export opaque type JSInt: number = number
export opaque type JSUint: JSInt = JSInt

export const testU = (n?: JSUint): string => {
    if (n) {
        return "set";
    }
    return "unset";
};

export const test = (n?: JSInt): string => {
    if (n) {
        return "set";
    }
    return "unset";
};

Error: src/types/bytes.js:277
277: export const testU = (n?: JSUint): string => {
                               ^^^^^^ number. This type is incompatible with
277: export const testU = (n?: JSUint): string => {
                               ^^^^^^ opaque type `JSUint`

Error: src/types/bytes.js:284
284: export const test = (n?: JSInt): string => {
                              ^^^^^ number. This type is incompatible with
284: export const test = (n?: JSInt): string => {
                              ^^^^^ opaque type `JSInt`

Both of those fail, even though JSInt is just a number and not depending on another opaque type, so it ought to work. If I put them in the same file as the type definitions they will work though. It seem to have to do with certain operations, like === and in this case checking existence. When testU is changed to this, it will pass:
export const testU = (n: JSUint): string => {
    return n.toString();
};

Finally, note that none of these functions are actually called. They fail simply by existing.

Ah, ok, these are some very interesting cases. I poked at it a bit and agree with you that it seems like opaque types, when they are supposed to be subtypes of number/string, generate type errors for certain operations like === or checking existence where they are trying to use their super type's implementation of those.

This example shows getting an error trying to pass my opaque type to a function in a for..of, but if I move the call out of the loop it's fine. brokenReassignment errors just trying to move the value to a new variable.
Flow, a static type checker for JavaScript, version 0.53.1
export opaque type OpaqueString: string = string

import type { OpaqueString } from './types'

export function otherFunc (id: OpaqueString) {
  return []
}

function broken (id: OpaqueString) {
  //                 ^^^^^^^^^^^^ string. This type is incompatible with
  //                 ^^^^^^^^^^^^ opaque type `NetworkAccountId`

  for (const c of otherFunc(id)) { }
}

function brokenReassignment (id: OpaqueString) {
  const foo: OpaqueString = id
  //         ^^^^^^^^^^^^ string. This type is incompatible with
  //         ^^^^^^^^^^^^ opaque type `NetworkAccountId`

  for (const c of otherFunc(foo)) { }
}

function workaround (id: OpaqueString) {
  const flow4674 = otherFunc(id)
  for (const c of flow4674) { }
}


Links opening in new tabs

Related Issue

Issue No. #185

Changes you made :

The links of social media in footer are now opening in new tabs.

Describe the changes :

This is required for better user experience on the website.

Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Output Screenshots :

@AvadhootK Good 👍
But plz make the chages in all pages
You have made it in only index.html

Sure I'll do that.

@AvadhootK I appreciate your work
But in the mean time one of our conributor has changed the footer, along with footer he has added the feature which u mentioned.
So actually your issue was resolved.
I apologize for the same.
Plz look for other issues or create new issues.
Thank you 👍


chore: Adjust label for proper case sensitivity

Description
In this commit, the label "closed" has been updated to "Closed" to adherence to proper labeling conventions within the automation system.

Ready to merge


pPicBuf capacity is getting full due to duplicate frame in pRefPic when shorterm frame number is matching with new Pic

While Decoding an erroneous/Currupted bitstream, Observed that mb_skip_run is going out of range and causing abrut frame end and as a result when the next data comes, it is of same number . In function AddShortTermToList, we are overwriting a pic in pShortRefList without unsettin it and as a result we are lossing a Pic slot and happed twice in this bitstream so lost 2 slots. iCapicity of 5 is hit now and not able to get FreePic from than.
Also I feel this scenario is possible in burst losses/very small frame number roll over.
Calling the function "SetUnRef(pRefPic->pShortRefList[LIST_0][iPos])" before overwriting it fixed the issue.
Please correct/suggest me what i did was okay or not ?.
if (pRefPic->uiShortRefCount[LIST_0] > 0) {
// Check the duplicate frame_num in short ref list
for (int32_t iPos = 0; iPos < pRefPic->uiShortRefCount[LIST_0]; iPos++) {
if (!pRefPic->pShortRefList[LIST_0][iPos]) {
return ERR_INFO_INVALID_PTR;
}
if (pPic->iFrameNum == pRefPic->pShortRefList[LIST_0][iPos]->iFrameNum) {
// Replace the previous ref pic with the new one with the same frame_num
//  pRefPic->pShortRefList[LIST_0][iPos]->bUsedAsRef = false;
SetUnRef(pRefPic->pShortRefList[LIST_0][iPos]);
pRefPic->pShortRefList[LIST_0][iPos] = pPic;
//ME2392
//  pPic->bUsedAsRef = false;
return ERR_INFO_DUPLICATE_FRAME_NUM;
}
}

In my opinion， your solution is correct. if we dont use SetUnRef, the frame buffer will left in reflist (this buffer will error util whole decoder end) and will result reference list error.

True Fermay. Thanks for the update.


Segnet for Network.GetNetwork()

Got a drop down of available networks in our application that saves as a string, segnet missing from the GetNetwork() method :)
https://github.com/MetacoSA/NBitcoin/blob/b74dbc79337dc84bb606db55e9aba33ebf03a6e3/NBitcoin/Network.cs#L802

good catch thanks, it is merged!


Error in website doc

Bug, feature request, or proposal:
i followed the theming guide of https://material.angular.io/guide/theming
i add the following line in styles.css:
@import '~@angular/material/core/theming/all-theme';
@angular/cli gives me webpack error:
Module not found: Error: Can't resolve '@angular/material/core/theming/all-theme' in 'C:\Users\me\WebstormProjects\myproject\ src'  @ ./~/css-loader?{"sourceMap":false}!./~/postcss-loader!./src/styles.css 4:10-123  @ ./src/styles.css  @ multi ./src/styles.css webpack: Failed to compile.
i guess the documentation is not uptodate ?
SOLUTIN WAS to rename styles file:
mv styles.css styles.scss
then adjusting angular-cli.json
VERSION:
"@angular/cli": "1.0.0-beta.31",
"@angular/material": "^2.0.0-beta.1",

The CLI is expecting that import to be .css
As it is, you are trying to import .scss into .css without preprocessing it hence the error. This isn't a documentation issue, it's a configuration issue. IMO if you want to leverage the theming engine in material, just use scss in your project, particularly if you are using the CLI.

Given that this seems specific to your build setup, I'm going to close this.

Changed the cli to use scss. A thing  witch is not hinted the documentation
Le 11 févr. 2017 07:51, "Joshua Wiens"<EMAIL_ADDRESS>a écrit :

The CLI is expecting that import to be .css
As it is, you are trying to import .scss into .css without preprocessing
it hence the error. This isn't a documentation issue, it's a configuration
issue. IMO if you want to leverage the theming engine in material, just use
scss in your project, particularly if you are using the CLI.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/issues/3054#issuecomment-279126157,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD09Ojl5BgsFm7z8jYbmPLUeU0m7W8EXks5rbVphgaJpZM4L-EDg
.


Where is the downloaded dataset?

I have completely run the script: python generated_case_task4_2024. py, but in the directory DESED_task master/data/dcase/dataset/audio/train/, the strong_label_real, unlabel_in_domain, and weak are all empty.
When I run the script: Python train_pretrained.py, the result is as follows:
Resampling syntax folder to 16 kHz
Resampling synth_val_folder to 16 kHz
Resampling real_masestro_train_folder to 16 kHz
Resampling real_masestro_val_folder to 16 kHz
Resampling strong_folder to 16 kHz
0it [00:00,? It/s]
Resampling weak_folder to 16 kHz
0it [00:00,? It/s]
Resampling unlabeled_folder to 16 kHz
0it [00:00,? It/s]
Resampling test_folder to 16 kHz
0it [00:00,? It/s]
Global seed set to 42
Traceback (most recent call last):
File "/home/dl/data/DESED_task master/recipes/dcase2024_task4_baseline/train_pretrained. py", line 621, in
Single_run（
File "/home/dl/data/DESED_task master/recipes/dcase2024_task4_baseline/train_pretrained. py", line 413, in single_run
Samplers=[torch. utils. data. RandomSampler (x) for x in tottrain_data]
File "/home/dl/data/DESED_task master/recipes/dcase2024_task4_baseline/train_pretrained. py", line 413, in
Samplers=[torch. utils. data. RandomSampler (x) for x in tottrain_data]
File "/home/dl/anaconda3/envs/dcase2024/lib/python3.8/site packages/torch/utils/data/sampler. py", line 143, in __ init__
Raise Value Error (f "num samples should be a positive integer value, but got num samples={self. num samples}")
Value Error: num samples should be a positive integer value, but got num samples=0
I don't know what to do, do I still need to add: only strong only real only synth only maestro after Python generated_case_task4_2024. py and run it again? Or should I do something else to solve this problem?

Hi Kai,
did you also generated the embeddings ?
On Fri, May 3, 2024, 10:37 AM Kai Qi @.***> wrote:

1.png (view on web)
https://github.com/DCASE-REPO/DESED_task/assets/115334799/9828f817-bbac-4084-a75c-f9d86a0617f2
—
Reply to this email directly, view it on GitHub
https://github.com/DCASE-REPO/DESED_task/issues/95#issuecomment-2093153049,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEO36OMFEA2V37AW52MLRQ3ZAOOKLAVCNFSM6AAAAABHEAICCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJTGE2TGMBUHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hello,
Do you mean to run this script: Python extraction_embeddings. py -- output_dir/ Embeddings“

I ran this script after running Python generated_case_task4_2024. py
However, I am currently running Python generated_case_task4_2024. py -- only strong/only real/only syntax/only maestro. I don't know if this will be effective.
After running Python generated_case_task4_2024. py -- only strong, the following prompt is given:
The file..// Data/dcase/dataset/metadata/train/audioset_strong. tsv already exists
Time of the program: 0.001397848129272461 s said that there is already a tsv file (where to download the corresponding wav file QAQ)
Just finished running Python generated_case_task4_2024. py -- only real gave the following prompt:
Please check your missing files: missing files, you can relaunch 'download.audiosed_sets' to try to include them, then, send them missing files to
Time of the program: 86329.15910863876 seconds
The dcase dataset has been saved in the following path:..// Data/dcase/dataset
Generatedcase_task4_2024. py:217: UserWarning: You have missing files
Please try to redownload dded_real again:
Import Desed
Desed. download_studioset_data ('../../data/dcase/dataset ', n_jobs=3, chunk_size=10)
Please send your missing files to the task organizers to get your missing files

Your download is successful you have to contact romain serizel via the link in website.
Please follow the instructions there

好的，我以为在这联系您也可以

，谢谢您的耐心回复！祝好！


add options for soft-deletion of channels and packages

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (1a42ed2) 83.57% compared to head (a4086e4) 83.58%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

quetz/_version.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #681   +/-   ##
=======================================
  Coverage   83.57%   83.58%
=======================================
  Files          79       79
  Lines        6236     6239    +3
=======================================
+ Hits         5212     5215    +3
  Misses       1024     1024

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Draft: Fairfood

Closes #2
Feedback welcome @twothreenine
Todo

[x] logon to fairfood.org
[x] fetch and parse rss
[ ] convert to foodsoft_article
[ ] fetch origin, manufacturer
[ ] price_net, ... conversion
[ ] Test & Accept @twothreenine

Great! I couldn't find the time to test this for now, but go on if you want to. Or let me merge it if you want me to continue. We'll need it mid-March so it's not very urgent, should be doable now. Please also add requirements in the Readme.

I updated README and imports.
I think it is best if you continue, it is not clear for me, how you added the articles in the first place.
If you like, we could split the work (e.g.: I work on a methode to parse the origin, and so furth...)

it is not clear for me, how you added the articles in the first place

Odd but good to know, I thought the simple example was very clear, or did you overlook that?

I think, I don't have write access to your repo, thus I can't merge :) (which is no problem for me)

Since it was in "draft" status I couldn't merge it and I thought you had to change the status first, but I found out how to do that now :)


SDSTOR-11060 : range_put bug due to lack of available space in a node

Before update the leaf node, a check is required to make sure there is enough available size for new value

Codecov Report

Merging #134 (56803d9) into master (14fe362) will decrease coverage by 6.46%.
The diff coverage is 19.35%.

:exclamation: Current head 56803d9 differs from pull request most recent head 9eeef62. Consider uploading reports for the commit 9eeef62 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
- Coverage   44.18%   37.73%   -6.46%
==========================================
  Files          83       74       -9
  Lines        9004     7543    -1461
  Branches     1151      997     -154
==========================================
- Hits         3978     2846    -1132
+ Misses       4642     4375     -267
+ Partials      384      322      -62

Files Changed
Coverage Δ

...clude/homestore/btree/detail/btree_mutate_impl.ipp
52.06% <19.35%> (-5.17%)
:arrow_down:

... and 13 files with indirect coverage changes


Any plan to support ensemble formation?

This is a feature request.
When we are in the state of hyperparameter tuning, we usually form ensembles to further boost its performance. Do you plan to implement (semi-) automatic way to form ensembles?

Thank you for your feature request.
We've understood that some users including Kagglers need features to help form ensembles using optimization results, and we'd like to implement such features in the future.
For the meanwhile, please obtain trials using study.trials or study.trials_dataframe() and find trials suitable for ensemble models manually. In addition to that, this FAQ page may be useful to save/restore the models trained in trials.
If you already have your implementation, we will welcome your pull request.

Thank you for the reference. I will look into that.


[1.12.2] Feature Suggestion: Changeable Drop Rates

I've been looking through the configs and the JSON, and have seen that there's a lay coefficient, but I'm not sure if that's just a multiplier on the drops when it is time for them to drop, or if it is a timer multiplier. As a pack dev I would like to change the timer of when a chicken drops its items in a much easier fashion if possible.

Sorry @DrDarkfire  I thought I had already responded to this. Yes its a multiplier, the time is based off the tier of the chicken, and the coeffiecnt multiplier is a multiplyer on that time. Its does this to be dynamic, and would require some rewrites to do this. Dont think it would be feasible atm.


[Question] Is there an API for getting a list of QnAMaker services for a subscription?

I checked in the list of QnAMaker APIs(https://westus.dev.cognitive.microsoft.com/docs/services/58994a073d9e04097c7ba6fe/operations/58994a073d9e041ad42d9baa).
There is no API for getting list of QnAMaker services.

Correct - this information is not currently available via API.

is there any plan to include such API in near future

The correct place to ask that question would be on this repo


Check ycsb proportion explicitly against none.

When I set ycsb_readproportion to 0.0, it ended up not setting the readproportion on the underlying ycsb command. This is because we are using if FLAGS.ycsb_readproportion to check if the flag is set, but the flag could be set explicitly to 0.0 and still fail the check.

@googlebot I fixed it.


callback of method Illuminate\Support\Collection<int,Illuminate\Database\Eloquent\Model>::each() expects callable

Larastan Version: #.#.#
--level used:

Description
When using a scope from a builder and then using ->each() it drops an error.
Parameter #1 $callback of method Illuminate\Support\Collection<int,Illuminate\Database\Eloquent\Model>::each() expects callable(Illuminate\Database\Eloquent\Model, int): bool|void, Closure(App\Models\Transaction): mixed given.
Laravel code where the issue was found
        $monthlyRobuxEarnings = Transaction::whereSupplierWithTrashed($supplier)
            ->selectRaw('DATE(`created_at`) as date, SUM(`robux_amount`) as total_robux_amount')
            ->groupBy('date')
            ->orderBy('date')
            ->whereBetween('created_at', [now()->startOfMonth()->startOfDay(), now()->endOfMonth()->endOfDay()])
            ->get()
            ->each(static function (Transaction $transaction) {
                $date = Carbon::parse($transaction['date']);
                $transaction['formatted_date'] = $date->copy()->format('M d');
                $transaction['date'] = $date->day;
            });

In one of the chained methods the specific model (Transation) is lost.
Try removing the method calls one by one and retest.

In one of the chained methods the specific model (Transation) is lost.
Try removing the method calls one by one and retest.

@szepeviktor, I've edited the issue, you better read it again.
The issue happens because of the builder.

@szepeviktor, I've edited the issue, you better read it again.
The issue happens because of the builder.

@AdirYad You need docblocks to describe your custom builder correctly. You can read it here.

@AdirYad You need docblocks to describe your custom builder correctly. You can read it here.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3970

Questions ?
Please refer to the Backport tool documentation


Update the User Guide after 2101 feedback

Let's:

Update the user interface preview
Add a short introduction explaining how the app can help HR managers
Add 'how to use this guide' section
Modify language to suit HR managers and have a more friendly tone
* removed jargon
Add glossary
Move the command summary to the top
Add the new job-related commands to the summary and features section
Features: Standardised the full stops
Add caution labels to irreversible commands
Add more FAQs

Codecov Report

Merging #86 (69ddab7) into master (8329e07) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #86   +/-   ##
=========================================
  Coverage     71.01%   71.01%
  Complexity      492      492
=========================================
  Files            84       84
  Lines          1856     1856
  Branches        181      181
=========================================
  Hits           1318     1318
  Misses          485      485
  Partials         53       53

Continue to review full report at Codecov.

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


⚠ ClamAV has degraded performance

In 26c3abc, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in e690170 after 6 minutes.


StackOverflowException with HashMap

I think this might be related to https://github.com/mockito/mockito/issues/818 as maybe it was ported from the culprit over there.
I'm using 1.7.8 and actually in my case I'm not mocking a HashMap, but returning one from an every block
 val blob : CloudBlockBlob = mockk() // from azure-storage-java
 val metadata : HashMap<String?,String?> = hashMapOf()
 every { blob.metadata } returns metadata

The StackOverflowException occurs when blob calls metadata. I don't know why HashMap is getting mocked though.
Honestly I don't know what is happening but that mockito bug looked too similar to ignore
Exception in thread "main" java.lang.StackOverflowError
at java.lang.Class.getDeclaredMethod(Class.java:2127)
at java.util.HashMap.get(HashMap.java:556)
at sun.reflect.Reflection.filterMethods(Reflection.java:291)
at java.lang.Class.privateGetDeclaredMethods(Class.java:2701)

Thanks for reporting it. Indeed class transformation is very similar to Mockito.

The fix is here, it is a little bit ugly, but should be OK. To make it really nice, a ByteBuddy should be changed.

Thanks, I'm using a workaround now, I'll test after the next release when I have some down time

Version 1.7.10 released, please check if it fixes your case and close the ticket.

That fixed it thanks!


MDX support

MDX support

Stack created with Sapling. Best reviewed with ReviewStack.

#5561
-> #4701

I believe this isn't working because I use a separate typecheck and compilation step. Since I have a js_library dep for my JS files, it's got nothing to type check against and so it fails. Not 100% sure how to resolve this yet, but it seems like I would need to feed my JS files into a ts_project with allowJsFiles: true. The problem is that Bazel would be unhappy that there would be an overlapping sourceMap input -- maybe I can resolve this by having like source-map-2.map


Add CMake option to link MSVC runtime library statically

Because of dependencies it is hard to get it consistently right. Closing.


Add Default region in the options

Hi
I would like to be able to set a default region (maybe just a string) in the options so i can tunnel on eu region.

+1

It would seem this could be similar to #20 ... in that, if the extension allowed you to arbitrarily specify additional ngrok command line parameters, you could accomplish both with a single PR.

It should be noted you can also set a default region in your ngrok configuration file.
See here: https://lornajane.net/posts/2018/using-config-files-with-ngrok
Simply add region: eu to your ngrok.yml config file.


(Fixed) Move period inside ending quotations in hanging-your-two-fact…

…or-authentication-method#adding-a-totp-app
Fixes #34225
Why:
To maintain consistency with style guide
Closes:
34225
What's being changed (if available, include any code snippets, screenshots, or gifs):
Moved the period inside the ending quotations at

On GitHub, type the code into the field under "Verify the code from the app".

Check off the following:

[X] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[X] For content changes, I have completed the self-review checklist.

@iamAyushChamoli Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏


Add link to module-level documentation

With this PR, the side panel will show a link straight do doc.deno.land if the module has a top-level mod.ts. If not, the link is hidden.
This implicitly assumes that mod.ts is a module’s canonical entry point and I don’t think that’s ever been decided, but it’s better than nothing for now.

@surma Run deno fmt to fix formatting.


Is there any chance to access the pert-trained model?

Hey there!
Is there any chance to access the pert-trained model?
Roi

Hi, yes I will post the pretrained models along with the rest of the code soon.

Thanks! Waiitng

Hi! Links to the pretrained models are now available along with the full code. Sorry for the delay!


Make sure alt text is used everywhere

No images are currently being used in the site so leaving this for some other time.

This is covered by Lighthouse/Pa11y tests, closing


How do you make the "Game Over " animation?

How do you make the "Game Over " animation?

@zmxv Any details about how you make this ?

I applied a "transform" animation to an overlay. I'll explain in detail with source code when I get to write the next tutorial.

Thanks!
Looking forward to it.


Loading upstream tarball doesn't respect package.proxy array

Describe the bug
Having multiple uplinks point to the same registry, the package.proxy array gets ignored.
Instead we'll walk through the uplinks and select the last one in the list that can serve the URL we've found for our upstream tarball.

To Reproduce

storage: ./storage

uplinks:
  cachednpm:
    url: https://registry.npmjs.org/
    cache: true
  npmjs:
    url: https://registry.npmjs.org/
    cache: false

packages:
  '**':
    access: $all
    publish:
    proxy: cachednpm

Run with above config (listening on localhost)
Try requesting<EMAIL_ADDRESS>Notice that<EMAIL_ADDRESS>does not exist

The uplink chosen here was npmjs, not cachednpm.
Expected behavior

Making the upstream request should respect the packages.proxy array.
In this example, selecting cachednpm for it's uplink, causing it to cache the tarball.
Environment information
Verdaccio version: 4.4.1
System:
    OS: Linux 5.0 Ubuntu 18.04.3 LTS (Bionic Beaver)
Binaries:
    Node: 12.13.1 - /usr/local/bin/node
    Yarn: 1.21.1 - /usr/local/bin/yarn
    npm: 6.13.4 - /usr/local/bin/npm

Additional context

I believe the culprit here is:
https://github.com/verdaccio/verdaccio/blob/32aabca6412afdd19a2603205d661f00a667088f/src/lib/storage.ts#L207-L211
The isUplinkValid check will be true for both uplinks. So the last one in the array will be selected.
Perhaps it should be using hasProxyTo from src/lib/config-utils.ts?

In the config above, every package (denoted by '**' are requested from cachednpm uplink, so it will be retrieved from cache.

In the config above, every package (denoted by '**' are requested from cachednpm uplink, so it will be retrieved from cache.

That's what I would expect. But it doesn't :smile:

Its the for loop - it doesn't exit at the first resolution, like you mentioned @Beanow - maybe the "workaround" is to have the one you really want to be last in the collection?

Try setting a specific package (like 'react') to be taken from cachednpm and other without cache. Then, paste results and check if the storage contains cached package.

I've hacked in a log (in build/lib/storage.js), just after the for-loop I linked.
console.log('DECIDED ON UPLINK:', uplink.upname);

This outputs DECIDED ON UPLINK: npmjs
Notice I've not added npmjs as proxy for any package at all.

Did you mean something like this @sergiohgz?
uplinks:
  cachednpm:
    url: https://registry.npmjs.org/
    cache: true
  npmjs:
    url: https://registry.npmjs.org/
    cache: false

packages:
  'statuses':
    access: $all
    proxy: cachednpm
  '**':
    access: $all
    proxy: npmjs

Request<EMAIL_ADDRESS>Request http://localhost:4873/statuses/-/statuses-1.5.0.tgz

$ tree ./storage
./storage
├── statuses
│   └── package.json
└── @verdaccio
    └── streams
        └── package.json

I cannot reproduce the issue, let me investigate deeper, it's working as expected.

@nothingismagick

Its the for loop - it doesn't exit at the first resolution, like you mentioned @Beanow - maybe the "workaround" is to have the one you really want to be last in the array?

It will always select the last one with a matching URL, so it won't let you use different settings for different packages that way.

Well, I understand your thoughts, and that's correct, the loop is not intended to stop if it find several urls that matches the uplink selected, so it will get latest uplink. I'm going to make a PR with the fix soon :wink:
On the other hand, we didn't evaluate the need for setting the same URL for different uplink definitions. Why would you want to do that? (pure curiosity :smiley:)

Well, as the example shows, the uplink definition includes some configuration like cache: boolean.
I would like to control those for different packages.
For example, some should cache while others shouldn't.
The particular use-case I'm thinking of, is to provide a redundant host for our own packages when you're hitting a firewall for the major registries. To preserve disk-space it should proxy (but not cache) '**', but should cache our project's packages.

-        if (self.uplinks[uplinkId].isUplinkValid(file.url)) {
+        if (hasProxyTo(name, uplinkId, self.config.packages)) {

This change works for me. Though should probably add a unit test to prove.


Prettier format codebase

Prettier format codebase https://prettier.io/

fixed in https://github.com/visjs-community/visjs-network/pull/11


Fatal  error with pulsar client and Java 17(open jdk 17.0.2_8-7-1937)

Describe the bug
We are deploying our consumer app using pulsar client 2.9.1(tried with 2.10 version too) with java version 17(open jdk 17.0.2_8-7-1937) and we are getting fatal error during init of PulsarClient. The app is working with Java 11 and Java 15.
To Reproduce
Steps to reproduce the behavior:
Error occurs in this statement -  PulsarClient.builder().serviceUrl(broker).connectionsPerBroker(4).ioThreads(Runtime.getRuntime().availableProcessors()).build();
Expected behavior
Pulsar client init and running fine
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]

@vamsiprasanth-narvar could you please kindly provide the core dump file and error report file?

Closed as stale.
Please open a new issue if it's still relevant to master version and latest JDK 17 patch versions.


Implement initial Kotlin generator

This is the preliminary implementation of the Kotlin generator. It produces the Kotlin cluster API in a manner compatible with the existing Java Cluster API. This enables us to experiment with using TLV-based generic Im APIs.

It would be nice to have a Super Class or implemented interface for each Cluster Class.

This is not the final version, current cluster APIs does not have any command function to share, we also don't need the chipClusterPtr and devicePtr if we switch to use generic IM API. Will add Super Class later if there is something we could share among all Cluster APIs


[DOCU-1476] HTTP log plugin custom header code block

https://konghq.atlassian.net/browse/DOCU-1476 correction as noted by Fel and others in slack
Direct review link:
https://deploy-preview-2839--kongdocs.netlify.app/hub/kong-inc/http-log/#custom-headers
Reviewers: also not sure why there are double angle brackets around the token placeholder? removed a set.

Thanks @javierguerragiraldez  and @HCloward for the reviews!


Clickbox for ape atoll agility course to big

I noticed that the 4th obstacle of the course, the skull slope, isnt correctly highlighted with the green clickbox.  The clickbox is to big, so if you click to high/low in the clickbox, you dont go up the slope.
Its a minor bugg/glitch but can be annoying.

Looks like the object (ID 15483) clickbox dips below the ground. There are multiple distinct accessible clickboxes, one for each of the skulls on the ground. However, all of these are encapsulated in one game object.
Highlighting these likely isn't possible to fix.
Here's a video demonstrating it. All these clickboxes are considered one object together:
https://user-images.githubusercontent.com/1868974/103426849-75de3900-4b8a-11eb-8a6c-29dc9fd289a4.mp4

I haven't noticed that! Thanks for looking into it.

fixed in fdd84f929de8199f50514f80d98f97dbbd6b20d6


Multi-window backtesting for TimeSeriesPredictor

Currently, if no tuning_data is passed to autogluon.timeseries.TimeSeriesPredictor.fit, we use the last prediction_length timesteps of train_data as hold-out validation set. This PR enables new train / val splitting strategies for timeseries datasets, such as multi-window backtesting.

New abstract class autogluon.timeseries.splitter.AbstractTimeSeriesSplitter that encapsulates the train / val split logic in its split method.
Two currently supported strategies:

LastWindowSplitter - reserve last prediction_length steps (default, same as before)
SlidingWindowSplitter - reserve multiple windows of length prediction_length (potentially overlapping) for validation

autogluon.timeseries.TimeSeriesPredictor now accepts an optional argument splitter.
tuning_data.index is now different from train_data.index (a suffix _[{start}:{end}] is appended to show how the original series was split). We do this because a single item in train_data may now correspond to multiple items in tuning_data.  This shouldn't affect users since tuning_data is not exposed in the public API.

Updates:

autogluon.timeseries.dataset.ts_dataframe.TimeSeriesDataFrame now makes sure that the provided item_id index either has all string or all integer types.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks @canerturkmen! I've addressed the comments and additionally added a check to TimeSeriesDataFrame that ensures that the item_id index contains only integer or only string values. This should prevent us from getting duplicate item_ids in the validation set.


feat: more graceful handling of API call didn't return a message message.

When the model fails to generate a message for send_message, we should re-try or give a better errror:
Letta.letta.server.server - ERROR - Error in server._step: API call didn't return a message: id='message-9fdf48af-b625-46f0-8b41-68bc3131d7f5' choices=[] created=datetime.datetime(2024, 9, 26, 6, 40, 38, 163866, tzinfo=datetime.timezone.utc) model='gpt-4o-2024-05-13' system_fingerprint='fp_e375328146' object='chat.completion' usage=UsageStatistics(completion_tokens=48, prompt_tokens=3699, total_tokens=3747)

@sarahwooders Thank you for capturing this.  I'm happy to provide context, and have some suggestions:

Since this error seems to be caused by LLMs failing to format function calls correctly, you could include a message to that effect in the error message so folks can see that possibility in the log.
Since some LLMs are known by the community to be compatible (or not) with Letta, you could update the documentation to say so, and list known-good LLMs so folks don't just try newer models like gpt-4o that have generally good reputations but don't work well with Letta.
@swooders. There's a gotcha: just because the config file says gpt-4 doesn't mean Agents will use it.  For example, I had already created an agent using gpt-4o, then switched to gpt-4 in the config.  I then tried the previously created Agent, thinking it would use the setting from the config file.  It did not.  The Agent overrode the config and continued to use gpt-4o.  I only got it working after I updated both the config AND created a new agent which also (redundantly) specifies gpt-4.

Suggestion: I wish it was more clear in the UX that Agents override the model from the config.  In fact, nowhere on the Agents dashboard or Agent Chat UIs does it mention what model is being used.  Expanding an Agent in the dashboard only shows Agent Persona and Human Persona.  I had to notice in the logs that gpt-4o was still being used in order to figure out the config was being overridden by the agent.  Also, it would be helpful if Agents could be edited and deleted in the ADE.  I've had to just create new ones repeatedly to get to a stable working state, and now there's cruft I can't get rid of.  I'm not even sure where Agents are stored, and I don't see any config files for them in ~/.letta/agents/

regarding this message sometimes I got the impression that this error / warning came even though a return message was actually returned (because I see the returned message in the stack trace)
most likely something can be improved here at least in give some better feedback to the user, if not more.

This should be mitigated as part of this PR -  when choices are empty, that usually indicates a flake on the LLM provider side. I added a retry mechanism here that helps.

@mattzh72  Thank you for the update


Add py.typed to distribution

See: https://www.python.org/dev/peps/pep-0561/
For mypy users to get type checking when coding against this library, a py.typed file should be vendored with the distribution.
For Poetry, see: https://blog.whtsky.me/tech/2021/dont-forget-py.typed-for-your-typed-python-package/

This project is typed. That should be obvious without boilerplate.
I am not willing to declare that this project is typed according to mypy, because, well, generally speaking it's not (not validated against it, anyway), and also I pretty much think mypy is doing everything wrong and I am very sad that good alternatives to it can't become popular.


Port forward for clients

Hi, there!
Thank you for using SoftEther.
Prerequisites

[ Yes] Can you reproduce? Yes
[ Yes ] Are you running the latest version of SoftEtherVPN? Yes
All operating systems
Processor: All

Description
[Description of the bug]
Can you please add port forwarding both to Softether VPN server and Softether OpenVPN server config, so that when I connect to either I can port forward on my machine as on host, THANK YOU IF IN ADVANCE! please report forward my message to developers.
Expected behavior:
Opening ports / forwarding ports on client from softether OpenVPN and Softether default servers
Actual behavior:
Impossible to open ports on client side
Steps to reproduce

Set whole Softether folder to read only
In NAT configuration of the hub enter your host default gateway to replicate on client
Wait till you lose connection or do you just loose remote access connection.

Close due to inactivity


When open a alert, close previous alerts

Before open a alert, check if another alert is already opened
and close it. Also, add a hideAlert() method to the activity class,
and use it to avoid duplicate code.

Thanks


Path too long.

Hi.
I'm trying to use this package and I get the error
The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.
at ChildProcess.exithandler (child_process.js:203:12)
at emitTwo (events.js:87:13)
at ChildProcess.emit (events.js:172:7)
at maybeClose (internal/child_process.js:818:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)

npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "wininstaller"
I Updated to NPM3 that it's supposed to fix this problem but I'm still getting this error.

Make sure you delete node_modules after you update to npm3. Then, do you still get this? What are the dependencies you're trying to install?

Hi.
I deleted node_modules folder after upgrading to npm3 and I did a new npm install. Everything correct in this step but still getting same error when trying to create the installer.
My dependencies are the following.
"devDependencies": {
"angular-mocks": "^1.4.8",
"bourbon": "^4.2.5",
"bourbon-neat": "^1.7.2",
"browser-sync": "^2.11.0",
"browserify": "^11.1.0",
"browserify-istanbul": "^0.2.1",
"browserify-ngannotate": "^2.0.0",
"browserify-shim": "^3.8.11",
"bulk-require": "^0.2.1",
"bulkify": "^1.1.1",
"canonical-path": "0.0.2",
"conventional-changelog": "^1.0.1",
"del": "^2.0.2",
"dgeni": "^0.4.2",
"electron-installer-squirrel-windows": "^1.3.0",
"electron-packager": "^5.1.1",
"electron-prebuilt": "^0.35.2",
"electron-windows-installer": "^1.2.5",
"envify": "^3.4.0",
"gulp": "^3.9.0",
"gulp-angular-gettext": "^2.1.0",
"gulp-angular-templatecache": "^1.8.0",
"gulp-autoprefixer": "^3.1.0",
"gulp-changed": "^1.3.0",
"gulp-concat": "^2.6.0",
"gulp-connect": "^2.2.0",
"gulp-if": "^2.0.0",
"gulp-imagemin": "^2.4.0",
"gulp-jshint": "^2.0.0",
"gulp-livereload": "^3.8.0",
"gulp-notify": "^2.2.0",
"gulp-sass": "^2.0.4",
"gulp-sourcemaps": "^1.5.2",
"gulp-streamify": "^1.0.2",
"gulp-uglify": "^1.5.1",
"gulp-util": "^3.0.6",
"highlight.js": "^8.9.1",
"isparta": "^4.0.0",
"jasmine-core": "^2.4.1",
"jasmine-spec-reporter": "^2.4.0",
"jshint": "^2.8.0",
"jshint-stylish": "^2.1.0",
"karma": "^0.13.15",
"karma-browserify": "^4.4.2",
"karma-chrome-launcher": "^0.2.2",
"karma-jasmine": "^0.3.6",
"karma-ng-html2js-preprocessor": "^0.2.0",
"protractor": "^3.0.0",
"run-sequence": "^1.1.5",
"vinyl-buffer": "^1.0.0",
"vinyl-source-stream": "^1.1.0",
"watchify": "^3.4.0"
},
"dependencies": {
"angular": "1.4.4",
"angular-gettext": "^2.1.2",
"angular-hammer": "^2.1.10",
"angular-ui-router": "^0.2.15",
"jquery": "^2.2.0",
"lodash": "^3.10.1",
"moment": "^2.10.6"
}

I close this issue due to it was an error with an internal file that wasn't correctly erased. Thank you for your attention.


UpdateManifest msbuild task needs more info

This talks about updating selected properties in a manifest

What kind of manifest? win32 assembly manifests? appx manifests? something else?
How do you select what properties to update and how do you update them? Can I use this to do e.g. variable substitution in an assembly manifest e.g. to stamp out the build architecture into the resulting manifest?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9ef97556-2e3c-b2ce-dcb2-6d0d09547c7d
Version Independent ID: f00e5930-43c2-45dd-0543-7c554f0871de
Content: UpdateManifest Task - Visual Studio
Content Source: docs/msbuild/updatemanifest-task.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@asklar -- Alexander, thank you for your questions.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@ghogen -- Gordon, please look into this issue.

@asklar The task is used for ClickOnce deployment. It's called from Microsoft.Common.CurrentVersion.targets in the _DeploymentSignClickOnceDeployment target.


Added Turkish Translation

Turkish translation completed and checked for any errors.

Deployed! Thanks for the contribution - feel free to let me know if you encounter any issues or have any more updates to add.

Thank you!
We have a minor typo where while all location have Title Case names one of them is in regular sentence case (the one for Circus Tent) - probably not worth the effort of fixing right now...

The title/regular case fixes have now been implemented and deployed. Thanks again!


enable contextcheck linter

Codecov Report

Merging #3 (b8e1ea3) into main (fc27e9f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage   39.14%   39.14%
=======================================
  Files          42       42
  Lines        3806     3806
=======================================
  Hits         1490     1490
  Misses       2048     2048
  Partials      268      268

Flag
Coverage Δ

2.0.5
39.14% <ø> (ø)

2.1.4
38.96% <ø> (ø)

2.2.2
?

2.3.3
38.96% <ø> (ø)

2.4.0
38.96% <ø> (ø)

community
39.14% <ø> (ø)

integration
39.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Home directory expansion unsupported in git.ignoredRepositories

VSCode Version: 1.33.1
OS Version: Arch Linux

Steps to Reproduce:

Add "git.ignoredRepositories": [ "/home/carlos" ] to your settings, replacing "carlos" by your username, of course. Here /home/carlos is a git repo (with many untracked files, that's why I want it to be ignored)
Restart and check that the repo is ignored. That should work.
Now replace "/home/carlos" above by simply "~".
Restart and check that the repo is NOT ignored.

Now, some settings in vscode do support basic shell expansion. I would say that "~" should be supported in any case, since it makes configuration relative to the user home and thus portable to other profiles.
Does this issue occur when all extensions are disabled?: Yes

I think in what matters to me this could be seen as an instance of https://github.com/microsoft/vscode/issues/2809. I mean, I want this to be a general mechanism, not an ad hoc solution for this particular setting. So I'm closing this.


Updated Runner's static translation

What is the context of this PR?
Update Runner's static content file as the translation has been updated
Trello
How to review
Deploy the branch to your environment and execution any schemas to check the translation works properly
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Update Runner's static content file (1 SP)


Add ClinVar star rating and review status

Closes #124. Built on top of #146 and includes all changes from it. More details: https://github.com/opentargets/platform/issues/1137.
The ClinVar star rating (from 0 to 4) and the review status (one of a few possible values) are now extracted from ClinVar and added to the evidence strings. Because of how evidence this represented (see https://github.com/opentargets/platform/issues/1137#issuecomment-663633967 and https://github.com/opentargets/platform/issues/1137#issuecomment-683728359), the values are only output for germline mutations. Further discussion with OT about this pending.
Also submitted a PR to improve the OT JSON schema, details: https://github.com/opentargets/json_schema/pull/96
Also also, fixed a memory leak & removed unnecessary code complexity in the evidence string comparison protocol.

For the record, additional actions done here:

Update the code to support somatic_mutation evidence strings
Update the JSON schema PR (opentargets/json_schema#96)
Do final compatibility checks, merge the PR, and send upstream notifications


Why is the opacity of the DateInput field set to 0?

Perhaps not an issue in the application by definition, but this still feels like the best place to report something that does not make sense to me.
I would like to know more as to why the CSS opacity value of the <input> for the DateInput is set to 0? It feels semantically wrong because an opacity of 0 implies full transparency, ie the element is invisible, ie I should not see it on the page. But that is indeed not the case. It is quite visible on the page.
This also does not play well with the Capybara acceptance test framework. By default, their fill_in method looks only for <input> fields that are visible. Setting the opacity to 0 makes Capybara correctly believe that this field should not be filled in. I had to use a workaround of a visible: false option in that DSL to trick Capybara, but I think that this hack should not be necessary.
Thank you for your feedback.

Poking due to lack of activity. @ljharb ?

Hey @ecbrodie, the inputs are in need of a bit of a refactor. The original
designs we were working with wanted to devalue the inputs in favor of the
calendar interaction. As a result the library doesn't show a cursor in the
inputs, even when you can type. This is something I am looking into fixing,
but unfortunately, has not been high on my priority. I hope to get to it in
the coming weeks (months?).
On Mon, Jul 24, 2017, 8:54 PM Evan Brodie<EMAIL_ADDRESS>wrote:

Poking due to lack of activity. @ljharb https://github.com/ljharb ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/airbnb/react-dates/issues/329#issuecomment-317621656,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUdtanfbBAC2JleDoDAsKFewE0M68qaks5sRWcMgaJpZM4MBFJk
.

I think this has been addressed now that we are using real life inputs! :)


[bug] numbers to be compared

...that is the result of no unit testing 😅 currently we are comparing prices fully in string instead of numbers, so the trends will be faulty (e.g.: '50'=>'1750' ===> ▼ [??] )
Make them number while checking the value:
let prevPrice = '50'
let currentPrice = '1750'
let prevPriceInt = Number(prevPrice)
let currentPriceInt = Number(currentPrice)
let trend
    if (prevPriceInt < currentPriceInt) {
      trend = '▲'
    } else if (prevPriceInt > currentPriceInt) {
      trend = '▼'
    } else if (prevPriceInt === 'n/a' || currentPriceInt === 'n/a') {
      trend = ''
    } else {
      trend = ''
    }
console.log(trend)

FYI: n/a will go in else branch in all cases as it will have NaN value
let prevPrice = 'n/a'
let currentPrice = '1750'
let prevPriceInt = Number(prevPrice)
let currentPriceInt = Number(currentPrice)
let trend
    if (prevPriceInt < currentPriceInt) {
      trend = '▲'
    } else if (prevPriceInt > currentPriceInt) {
      trend = '▼'
    } else if (prevPriceInt === 'n/a' || currentPriceInt === 'n/a') {
      trend = 'TEST: is it catched?'
    } else {
      trend = ''
    }
console.log(trend)
VM1419:15
undefined
console.log(prevPriceInt)
VM1559:1 NaN

parseInt() is a better method to convert the prices to number as a faulty price string cleaning (e.g. '1950,-') would result in NaN as well with Number(), while parseInt() will convert it to a parse-able number (e.g.: '1950,-' => 1950)


Divergence detection

from https://github.com/rst-tu-dortmund/teb_local_planner/pull/259

not needed for now, it was a bit unstable in testing


New check: dot of soft dotted characters should disappear

What needs to be checked?
The dot of soft dotted characters should disappear when an accent is placed on them.
For example j́ (U+006A LATIN SMALL LETTER J, U+0301 COMBINING ACUTE) should look like ȷ́ (U+0237 LATIN SMALL LETTER DOTLESS J, U+0301 COMBINING ACUTE).
Detailed description of the problem
Several Unicode characters have the Soft_Dotted propery as described in https://www.unicode.org/reports/tr44/#Soft_Dotted

Soft_Dotted | Characters with a "soft dot", like i or j. An accent placed on these characters causes the dot to disappear. An explicit dot above can be added where required, such as in Lithuanian. See Section 7.1, Latin in [Unicode].

See "Diacritics on i and j" in Section 7.1, "Latin" in The Unicode Standard.

Diacritics on i and j. A dotted (normal) i or j followed by some common nonspacing
marks above loses the dot in rendering. Thus, in the word naïve, the ï could be spelled with
i + diaeresis. A dotted-i is not equivalent to a Turkish dotless-i + overdot, nor are other cases
of accented dotted-i equivalent to accented dotless-i (for example, i +  ̈  ≠ ı +  ̈ ). The same
pattern is used for j. Dotless-j is used in the Landsmålsalfabet, where it does not have a case
pair.
To express the forms sometimes used in the Baltic (where the dot is retained under a top
accent in dictionaries), use i + overdot + accent (see Figure 7-2).
All characters that use their dot in this manner have the Soft_Dotted property in Unicode.

For a list of soft dotted characters see https://util.unicode.org/UnicodeJsps/list-unicodeset.jsp?a=[:Soft_Dotted=Yes:] which currently shows "iⅈ𝐢𝑖𝒊𝒾𝓲𝔦𝕚𝖎𝗂𝗶𝘪𝙞𝚒ⁱᵢįị ḭ ɨᶤ 𝼚 ᶖ jⅉ𝐣𝑗𝒋𝒿𝓳𝔧𝕛𝖏𝗃𝗷𝘫𝙟𝚓ʲⱼ ɉ ʝᶨ ϳ і ј". These are Latin, Greek and Cyrillic letters, modifier letters and mathematical symbols.
Optional fix
The ccmp feature should have lookups that substitute soft dotted glyphs by their dotless equivalents when followed by marks with at least one mark above. In some cases additional glyphs are or could be needed when full decomposition is not adequate, like istroke.dotless or optionally iogonek.dotless if /idotless/ogonek is not adequate.
For example:
@COMBINING_TOP_MARKS = [gravecomb acutecomb circumflexcomb tildecomb macroncomb diaeresiscomb brevecomb dotaccentcomb caroncomb ringcomb]; # etc.

feature ccmp {
    lookup ccmp_soft_dotted {
        lookupflag UseMarkFilteringSet @COMBINING_TOP_MARKS;
        sub [i j]' @COMBINING_TOP_MARKS by [idotless jdotless];
        sub [i-cy je-cy]' @COMBINING_TOP_MARKS by [idotless jdotless];
        # for iogonek, idotbelow, itildebelow either decompose or substitute for dotless glyphs
        sub iogonek' @COMBINING_TOP_MARKS by idotless ogonekcomb; # or "by iogonek.dotless"
        sub idotbelow' @COMBINING_TOP_MARKS by idotless dotbelowcomb; # or "by idotbelow.dotless"
        sub itildebelow' @COMBINING_TOP_MARKS by idotless tildebelowcomb; # or "by itildebelow.dotless"
        # for istroke, jstroke, imod or others substitute for dotless glyphs
        sub istroke' @COMBINING_TOP_MARKS by istroke.dotless;
        # etc.
    } ccmp_soft_dotted;
} ccmp;

Resources and exact process needed to replicate
The characters with the Unicode Soft Dotted property are defined in https://www.unicode.org/Public/UCD/latest/ucd/PropList.txt and are listed on https://util.unicode.org/UnicodeJsps/list-unicodeset.jsp?a=[:Soft_Dotted=Yes:]
The combining marks that should cause the dot to disappear are mostly in the Combining Diacritical Marks block, but also in the Combining Diacritical Marks Extended, Combining Diacritical Marks for Symbols, Cyrillic, Cyrillic Extended-A, Cyrillic Extended-B, Cyrillic Extended-D.
The sequences that are currently known to be used in orthographies are i̇́ i̇̀ i̋ i̍ i᷆ i᷇ i̓ i̊ i̇̃ i̐ ɨ́ ɨ̀ ɨ̂ ɨ̋ ɨ̏ ɨ̌ ɨ̄ ɨ̃ ɨ̈ ɨ̧́ ɨ̧̀ ɨ̧̂ ɨ̧̌ ɨ̱́ ɨ̱̀ ɨ̱̈ į́ į̇́ į̀ į̂ į̄ į̄́ į̄̀ į̄̂ į̄̌ į̃ į̇̃ į̌ ị́ ị̀ ị̂ ị̄ ị̃ ḭ́ ḭ̀ ḭ̄ j́ j̀ j̄ j̑ j̃ j̇̃ j̈ і́.
Expected Profile
Given this is a Unicode compliance check it should probably be in the universal profile.

[ ] Vendor-specific: Google Fonts
[ ] Vendor-specific: Adobe Fonts
[ ] OpenType (requirements imposed by the OpenType specification)
[x] Universal (broadly accepted best practices on the type design community)
[ ] Other:

Expected Result
Which log result level should it have:

[x] 🔥 FAIL
The check should FAIL when a font displays the dot of soft dotted characters used in a sequence used in a language orthography.
For example:

ị /idotbelow (LATIN SMALL LETTER I) is used in orthographies like Igbo (20 to 30 million speakers) where ị́ /idotbelow/acutecomb (U+0069 LATIN SMALL LETTER I, U+0301 COMBINING ACUTE) is used,
j /j (U+006A LATIN SMALL LETTER J) is used in orthographies like Dutch (24 million speakers) where j́ /j/acutecomb (U+006A LATIN SMALL LETTER J, U+0301 COMBINING ACUTE) is used.
į /iogonek (U+012F LATIN SMALL LETTER I WITH OGONEK) is used in orthographies like Navajo (160,000 speakers) where į́ /iogonek/acutecomb (U+012F LATIN SMALL LETTER I WITH OGONEK, U+0301 COMBINING ACUTE) is used.

[x] ⚠ WARN
The check should WARN when a font displays the dot of soft dotted characters used in sequences not know to be used in an orthography.
For example:

ⁱ (U+2071 SUPERSCRIPT LATIN SMALL LETTER I) is not used, as far as we know, in orthographies in combination with a combining mark above. Its combinations with mark above may be used in more specialized phonetic or scientific notation.
𝐢 (U+1D422 MATHEMATICAL BOLD SMALL I) is not used in orthographies, its combinations with mark above may be used in specialized scientific notation.

Severity assessment
4, fonts that would FAIL are unsable in the languages they apparently aim to cover.
Edit:

2023-05-20: simplified example ccmp lookup fix with UseMarkFilteringSet and comments.

@MariannaPaszkowska #4140 does change it to WARN among other things.


Taylorwaggoner flexkube

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
can not read a block mapping entry; a multiline key may not be an implicit key at line 2541, column 17:
                logo: flexkube.svg
                    ^

Deploy preview for landscape failed.
Built with commit c7d38b9ee07ea7510a4b78d686ccd7503445871d
https://app.netlify.com/sites/landscape/deploys/5e3de78483835300081d421e

LGTM


Make advanced_functionality/Command Chain Syntax more configurable

Currently, advanced_functionality is a great feature (see https://github.com/discordrb/discordrb/wiki/Commands#command-chain-syntax). However, sometimes a user may only want to use one feature, or a feature specific to a command only. This is what I am requesting here.
Only enable certain features globally
After advanced_functionality: true in the bot definition, you should be able to add:
advanced_functionality_features: []

Where [] is an array of features that can be enabled.
Only enable per command
To do this, the syntax in Discordrb::Commands::CommandBot.command() could be extended to look like:
bot.command(:command_name, advanced_functionality: true)

This would be an optional arg auto set to false that would enable all advanced functionality for the specific command.
Only enable certain features per-command
This is supposed to be the above, except also except a features array similar to the first example. This imo is the most important!

As discussed in the channel, it is currently not practical for us to further tweak the CommandBot API without making the code harder to understand, or the API more complicated to support random use cases. Since it cannot be easily extended by users, or well maintained by developers, it will eventually have a proper review and reimplementation that will hopefully result in something sufficiently powerful.
Such a solution will crop up sometime during the v4 cycle, hopefully landing on something decisive in v5 that will be better all around.

As discussed in the channel, it is currently not practical for us to further tweak the CommandBot API without making the code harder to understand, or the API more complicated to support random use cases. Since it cannot be easily extended by users, or well maintained by developers, it will eventually have a proper review and reimplementation that will hopefully result in something sufficiently powerful.

I see. Will a feature similar to advanced_functionality be available when the new version pops up though?

It will probably look nothing like the current command bot, so that concept will likely not exist.
The goals of providing a "command bot" primarily revolve around the parser, to which the library will deliver to the (single) most common use case, and as mentioned, have some surface so it can be extended by users.

As discussed in the channel, it is currently not practical for us to further tweak the CommandBot API without making the code harder to understand, or the API more complicated to support random use cases.

True, a use case like this seems oddly specific and doesn't look like it belongs in something that's meant for common use cases:
!user > bold The user name is:
**The user name is:** meew0

Indeed that isn't. However the quotes feature is, @FluorescentGreen5

Hmm, maybe shell splitting shouldn't be considered advanced functionality in the CommandBot revamp. Although, shell splitting can be easily done manually for now.
irb(main):001:0> require 'shellwords'
=> true
irb(main):002:0> '.command "something and" "something else"'.shellsplit
=> [".command", "something and", "something else"]```

I know :+1:
Should be inbuilt though


Document integrations with Datadog, splunk etc. for Metaflow logs & metrics

Context in http://slack.outerbounds.co

Original thread on gitter.

sure if you want to start a gdoc, that will be great!


Unable to enable cluster mode

Describe the Bug
I am trying to switch an existing cluster with cluster_mode_enabled = false (default) to cluster_mode_enabled = true.
As part of this change, I have also made the following changes:

remove cluster_size
set automatic_failover_enabled = true
set cluster_mode_num_node_groups = 1
set cluster_mode_replicas_per_node_group = 1

Proposed plan:

Apply error:

module.shared-redis.aws_elasticache_parameter_group.default[0]: Modifying... [id=<redacted>-uw2-shared]
╷
│ Error: modifying ElastiCache Parameter Group: InvalidParameterValue: The parameter cluster-enabled cannot be modified.
│       status code: 400, request id: <redacted>
│
│   with module.shared-redis.aws_elasticache_parameter_group.default[0],
│   on .terraform/modules/shared-redis/main.tf line 91, in resource "aws_elasticache_parameter_group" "default":
│   91: resource "aws_elasticache_parameter_group" "default" {
│
╵

According to AWS, this parameter cannot be modified:

Expected Behavior
Successfully enable cluster mode on Redis
Steps to Reproduce
See above
Screenshots
No response
Environment

OSX Sonoma 14.2
Terraform v1.6.5 on darwin_arm64
AWS provider version 5.30.0
Module version 0.53.0
Redis engine version 7.0

Additional Context
No response

@rymancl it looks like AWS disallows changing of this parameter. I suspect the only way to do this is to delete the cluster and restore from a snapshot with cluster mode enabled.

@rymancl or at least you can not manually change it. Maybe if you add another instance to the "cluster" then AWS will change that parameter to true behind the scenes.

@rymancl it looks like AWS disallows changing of this parameter. I suspect the only way to do this is to delete the cluster and restore from a snapshot with cluster mode enabled.

That's true, but as of redis 7.0 AWS allows you to do in place moves from cluster mode disabled to cluster mode enabled.
Docs here

Beginning with Redis 7, ElastiCache for Redis supports switching between Redis (cluster mode disabled) and Redis (cluster mode enabled).

The bug I believe is that the module is trying to set that parameter when it shouldn't be. I assume AWS will set it for you.

Ah I see. Try a targeted apply (assuming these are dev/sandbox resources) to module.shared-redis.aws_elasticache_replication_group.default[0] then see if AWS changes that parameter. Terraform should pick it up before the next plan and won't attempt to change it again.
Maybe the ideal solution would be for this module to not set it at all and assume that AWS manages it, depending on the outcome of the above.


Mangohud not working on bazzite-nvidia

I previously had Mangohud working when I manually layered Goverlay and Mangohud with rpm-ostree. I saw there was an update for the nvidia image that added the bazzite version of mangohud, so I removed the layered mangohud and goverlay, and updated 'rpm-ostree' and 'just update'.
Now mangohud seems to be broken or incomplete. If I run 'mangohud vkcube' I get the following error in the terminal with no mangohud display.
ERROR: ld.so: object '/usr/$LIB/mangohud/libMangoHud_opengl.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
Selected GPU 0: NVIDIA GeForce GTX 1080 Ti, type: DiscreteGpu

If I try to layer mangohud again with rpm-ostree I get the following.
error: "mangohud" is already provided by: mangohud-100.bazzite.0.0.git.747.40ec1d49-1.fc38.i686 mangohud-100.bazzite.0.0.git.747.40ec1d49-1.fc38.x86_64. Use --allow-inactive to explicitly require it.

Any help would be appreciated!

This should now be fixed, was an issue with our MangoHud package missing some .so files

Awesome! Thanks for the fix!


better pane flow on small vertical viewports

fixes trello ticket

Coverage remained the same at 87.713% when pulling c28f3183e061f66e850867fd0549b16af25ffcac on better-panes into 7dc7cb058874e4dd14baaac1d3730042d62aa5b4 on master.


error whil openning the built

raceback (most recent call last):
File "main-o.py", line 5, in
File "", line 2, in
File "", line 1, in
ModuleNotFoundError: No module named 'pyaes'

What are you doing that causes this error?

What are you doing that causes this error?

Ijust open up the built

This is wierd. Can you tell me the output of where python && python --version in command prompt.
Also try to redownload the grabber from github (in case any file is missing)

Sorry idont know how to do that

Check on google "How to open command prompt"

Also like I said, try to redownload the files.

Python 3.10.9 (tags/v3.10.9:1dd9be6, Dec  6 2022, 20:01:21) [MSC v.1934 64 bit (AMD64)] on win32

Write the full command where python && python --version

C:\Users\vianz\AppData\Local\Programs\Python\Python310\python.exe
C:\Users\vianz\AppData\Local\Microsoft\WindowsApps\python.exe
Python 3.10.9

If you tried to redownload the files but you are getting still the same problem, then there is something wrong with your system. You can use this online VM and build the grabber there. Don't forget to install python in this VM from the official site.
Make sure to turn off VMPROTECT in case you are going to test it on the same VM.

If you tried to redownload the files but you are getting still the same problem, then there is something wrong with your system. You can use this online VM and build the grabber there. Don't forget to install python in this VM from the official site.
Make sure to turn off VMPROTECT in case you are going to test it on the same VM.

Ok thank you


keyhelp integration

Is it possible to integrate it into boxbilling keyhelp yes there is an api available
Link https://app.swaggerhub.com/apis-docs/keyhelp/api/1.1.1

@Radio-Quergedacht pull request are always welcome

Not actionable. Feel free to contribute about this by creating pull requests.


Snapshotter dm verity vhd

@kevpar @katiewasnothere @ambarve @dcantah, this still depends on the hcsshim PR, but your feedback would be great

@anmaxvl Relies on this correct: https://github.com/microsoft/hcsshim/pull/985?

@anmaxvl Relies on this correct: microsoft/hcsshim#985?

yeah


Move test data into polyflame.samples

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.58%. Comparing base (19ec188) to head (05528f4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   74.51%   74.58%   +0.07%
==========================================
  Files           9        9
  Lines         361      362       +1
==========================================
+ Hits          269      270       +1
  Misses         92       92

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Dependabot created_at returning None

Describe the bug
I've found that running policy-as-code with a remediation time set in my config under dependabot, will throw the following error:
Error: unsupported operand type(s) for +: 'NoneType' and 'datetime.timedelta'
Upon investigation, it appears that line 185 in checks.py is always returning None in my case. Possibly relating to the ghastoolkit library? I've checked the API myself and all of the dependabot alerts have a created_at date set. It looks like the toolkit is calling the createdAt() in dependencyalert.py, which should return the time, but fails to do so:
https://github.com/GeekMasher/ghastoolkit/blob/main/src/ghastoolkit/supplychain/dependencyalert.py#L26
I believe the line actually throwing the error is line 278 in policy.py. But as said above, I think the actual issue is coming from the ghastoolkit failing to return the dependabot created_at value appropriately.
To Reproduce
Steps to reproduce the behavior:

workflow:

name: Fully Compliant

on:
  push:
    branches: [ main, ghas-pac ]
  pull_request:
    branches: [ main ]
  workflow_dispatch:

jobs:
  compliance:
    runs-on: ubuntu-latest

    steps:
    - name: Advance Security Compliance Action
      uses<EMAIL_ADDRESS>      with:
        # Set the severity levels which to set the threshold. All previous
        # severities are included so selecting 'error' also selects 'critical' and
        # 'high' along with 'error'.
        severity: error

        # Repository owner/name.
        # This can be setup using a separate repository to the one being analysed
        #  for security compliance
        repository: <repo>

        # The owner/repo of where the policy is stored
        # This generally is primarily used for a single repository that defines
        #  your organizations security policy
        policy: <repo>

        # The path where the policy lives. This might be in either the current
        #  repository or in a remote repository.
        policy-path: .github/policy/ghas-policy.yml

        # The branch you want to target using `policy` argument
        policy-branch: ghas-pac

        # GitHub Personal Access Token to access the GitHub API.
        # Secret Scanning and Dependabot do not allow their resources to be
        #  exposed to Actions so this might need to be set using a token that has
        #  the ability to access the resources
        token: ${{ secrets.POLICY_PAT }}

        # [optional] GitHub reference, this is passed in by Actions at runtime
        ref: refs/heads/main

        # [optional] What course of action to take upon discovering security issues that pass
        # threshold?
        action: break

policy:

name: High Threat Model Policy

codescanning:
  level: high
  remediate:
    critical: 5
    high: 30
    medium: 60

# Dependency Security Alters
dependabot:
  level: high
  remediate:
    critical: 5
    high: 30
    medium: 60

secretscanning:
  level: all

# Dependency Licensing Alerts
licensing:
  conditions:
    ids:
      - GPL-*
      - LGPL-*
      - AGPL-*

  warnings:
    # Warning is the dependency isn't known
    ids:
      - Other
      - NA

Expected behavior
Dependabot checks to finish without error, by returning the created_at date and allowing the alert time to be calculated.
Screenshots

build
GitHub Actions Ubuntu-latest

Nice catch! Waiting for v0.4.3 of ghastoolkit before I update the dependencies

https://github.com/advanced-security/policy-as-code/releases/tag/v2.3.2

@pritchyspritch Please try out and tell me if you have any issues

@GeekMasher I have tried and it seems a new bug has been introduced by a different change in the same release (I think), it appears your github actions workflow flagged it but didn't fail so it may have been missed: https://github.com/advanced-security/policy-as-code/actions/runs/5426075051
Exception: Failed to load GraphQL query :: GetDependencyInfo

@pritchyspritch This indicts that the GetDepdencyInfo graphql query isn't present in a location that the Action can find it. To solve this I think I will hardcode those queries into the ghastoolkit because they are heavily used. I will share the PR here when ready

This PR should fix the issue once merged
https://github.com/advanced-security/policy-as-code/pull/28


S2U-21 Fix whitespaces being removed from assessment title

When titleDecoration was an empty string, all white-spaces were removed from the title.

I re-tested and looks like it's not working, I'm rebuilding the QA server again to re-test.


Update README.md

[skip ci]

[skip ci]

test

retest this please

retest this please

retest this please

retest this please

retest this please


Migrate from Hickory to Elementary

We currently test our annotation processors with Hickory, an outdated library that hasn't been maintained in over a decade. This PR switches us to Elementary, a modern and well-maintained annotation processor testing library. Note that Elementary requires both Java 11 and JUnit 5, both of which we now support.

well-maintained

Actively perhaps, but well? https://github.com/Pante/elementary/issues/153#issuecomment-1280182769 should just not happen.


iOS - Create a new Tiki Sdk on wallet change

The example app should work as a real implementation. It should no hold a list of SDKs. Instead, it should hold the list of addresses and build the SDK with each address as needed.
Acceptance criteria:
1 - The app model holds just one instance of TIKI SDK
2 - The TIKI SDK is build on every change in the address


QueenParseTreeVisitor.java: Finish implementing unann ref...

The puzzle -1677088966 originating from #49 has to be resolved:
https://github.com/jvmqueen/queen-of-java/blob/9dee3c1d6dc924e06704a1430dec22234dff4e06/src/main/java/org/queenlang/transpiler/QueenParseTreeVisitor.java#L1018-L1018
"Finish implementing unann reference types with unann array types.".
The puzzle was created by amihaiemil at 2023-01-06 11:44:57 +0200.
Estimation is 60 minutes.
If you have any technical questions, don't ask me, I won't be able to help. Open new issues instead.

@amihaiemil this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-01-16T09:50:34.473996.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

Puzzle disappeared from the code, that's why I closed this ticket.

@amihaiemil thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Pointer Masking Example with N=16

Feedback by Ved Shanbhogue:

Would it be better if the example was representative of the only N
values (N=16) supported by the extension?

Also the current example in the spec is wrong. The hex doesn't match the binary:
0xAAFFFFFF12345678
NVBITS[1010101] VBITS[11111111111111111111111110001...000]

Bit 56 is set, so this is 0xABFFFFFF12345678.

Addressed in 92a88a2.


Question about number of LAION-Aesthetic samples

Hi Romain,
Thanks for the tool. It is amazing.
I see https://github.com/LAION-AI/laion-datasets/blob/main/laion-aesthetic.md mentions "use laion-aesthetic which contains 120M aesthetic samples (score > 7)". Among them, the english version has 51M.
There is also "625K image-text pairs with predicted aesthetics scores of 6.5 or higher" on https://laion.ai/blog/laion-aesthetics/, also from "subsets of the LAION 5B samples with English captions".
It is a bit confusing since 51M >> 625K, despite the 51M one has an even higher score threshold.
I wonder what do I miss here and what is the difference between these two versions of laion-aesthetic?
Thanks for your time and efforts. Your response is appreciated.
Best
Dongxu

Hi, the one mentioned in the example here is aesthetic V1 while the second
one you mention is V2. Details are explained in that blogpost
On Tue, Nov 22, 2022, 15:52 Dongxu @.***> wrote:

Hi Romain,
Thanks for the tool. It is amazing.
I see
https://github.com/LAION-AI/laion-datasets/blob/main/laion-aesthetic.md
mentions "use laion-aesthetic which contains 120M aesthetic samples (*score

7*)". Among them, the english version has 51M.

There is also "625K image-text pairs with predicted aesthetics scores of
6.5 or higher" on https://laion.ai/blog/laion-aesthetics/, also from
"subsets of the LAION 5B samples with English captions".
It is a bit confusing since 51M >> 625K, despite the 51M one has an even
higher score threshold.
I wonder what do I miss here and what is the difference between these two
versions.
Thanks for your time and efforts. Your response is appreciated.
Best
Dongxu
—
Reply to this email directly, view it on GitHub
https://github.com/rom1504/img2dataset/issues/225, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437SRXLMNDYYUXKUKQFTWJTMZ7ANCNFSM6AAAAAASH5FSDQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

That's clear. Thanks for the clarification.
Best


update documentation

What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
What this PR does / why we need it:

update documentation

/assign @leogr
Which issue(s) this PR fixes:

Fixes https://github.com/falcosecurity/charts/issues/543
Special notes for your reviewer:
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 16ee50f441dc1f5d6d8047451865f01516b82dc2

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [cpanato,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


fix(readme): Fix Deno Deploy access token page

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed UI of Velocity meta data

Issue Reference
Fixed UI issue
https://user-images.githubusercontent.com/106804569/226616800-ab8d2777-191e-4e70-977a-ef1728758594.png

@atul-webkul
please mention the issue link

@devansh-webkul
This pull request is related to admin responsiveness. so as we discussed, I am closing this pull request.


how to save response audio to a file?

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

I'm trying to save response audio to a file, I added these code to SpeechSynthesizer, inside executePreHandleAfterValidation funtion. The file is saved no problem, However, when I open the result.mp3 file in a player, the voice is shattered, as twice as fast compare to normal speak. Can't really hear it properly, it only last 4 seconds, it is very different to what Alexa spoke, I wonder if this is a format/decode issue? If I want to save exactly what Alexa said audio to a file, where is the best place to do that?
I shared result.mp3 on google drive https://drive.google.com/file/d/1wsnOl8ChO1RZJahscPrWnzJCK5cWAg0l/view?usp=sharing
This is the code I add at SpeechSynthesizer after line 456
std::ofstream out("result.mp3");
if (out.is_open()) {
char buf[1024];
avsCommon::avs::attachment::AttachmentReader::ReadStatus status;
while (true) {
auto len = speakInfo->attachmentReader->read(buf, sizeof(buf), &status);
if (status == avsCommon::avs::attachment::AttachmentReader::ReadStatus::OK) {
out.write(buf, len);
ACSDK_DEBUG9(LX("executePreHandleAfterValidation")
.d("ReadStatus", "OK"));
} else if (status == avsCommon::avs::attachment::AttachmentReader::ReadStatus::CLOSED) {
ACSDK_DEBUG9(LX("executePreHandleAfterValidation")
.d("ReadStatus", "CLOSED"));
break;
}
}
speakInfo->attachmentReader->close();
ACSDK_DEBUG9(LX("executePreHandleAfterValidation")
.d("output .mp3 file", "finished"));
}
What is the expected behavior?

What behavior are you observing?

Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

Tell us what hardware you're using:

[x] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[x] Other - tell us more: Windows 10

Hi!
I would suggest you to play around GStreamer's plugin "filesink" that can be used in MediaPlayer.
https://gstreamer.freedesktop.org/data/doc/gstreamer/head/gstreamer-plugins/html/gstreamer-plugins-filesink.html
MediaPlayer code:
MediaPlayer/GStreamerMediaPlayer/src/MediaPlayer.cpp
If you want to save speech only you can make sure that your modified MediaPlayer is used there:
https://github.com/alexa/avs-device-sdk/blob/master/SampleApp/src/SampleApplication.cpp#L366

thanks for your reply, I found something here https://github.com/alexa/avs-device-sdk/issues/14 , but I tried the code it doesn't seems working, is there anything similar I can do instead of use filesink?

@mavamazon thanks for your reply, a quick question is there a way to clone a AttachmentReader?

Hi @gunnicus,
There is no way to clone the AttachmentReader. You can have multiple readers, which should be enough for you to read the stream and redirect it to a file.

Hi, thanks for your reply. Can you please give me a hint or a sample how can I add another reader, which class should I look at. Great thanks.


np.load should take one more argument

https://github.com/davidsandberg/facenet/blob/096ed770f163957c1e56efa7feeb194773920f6e/src/align/detect_face.py#L85
this line should be:
data_dict = np.load(data_path, allow_pickle=True, encoding='latin1').item()
when the version of numpy higher than v1.16

I second this as I upgraded my numpy and facenet stopped working.  I hand-modified the detect_face.py to fix this.   For all users though we need this fix done in this github; Facenet should be present-day compatible, not past-compatible.   To make everyone happy, put a test on numpy.version and to do both versions.   (with a warning that numpy < 1.15 support is deprecated).   I'd be happy to do and test this if there's a need for another support developer on facenet, just let me know.


Apparent duplication of spoken content after the first submission

Hi Andrew,
As suggested, here is a copy of my posting in the discussion on the blog page.
In my recent testing I have noticed what appears to be a bug. I am not sure if this is in the aws-lex-audio.js file or in Lex.
If the problem  is in the aws-lex-audio.js file - ie. that it does not appear to be initializing the recorder buffer in each cycle, it is a guess based on my seeing that Lex is passing uninitialized slot and transcript text to my Lambda function on subsequent tests.
So for example: I might say "give me the summary". "give" is (correctly) put in a slot named actionVerb. In the first cycle the Lambda function gets:
transcript = "give me the summary"
actionVerb = "give"
If I click on the (mic) image and repeat "give me the summary", the Lambda function shows:
transcript = "give me the summary give me the summary"
actionVerb = "give give"
Now this may be a Lex problem, so please let me know if my guess - that the recorder buffer is not being initialized each time - is wrong. I looked at the "startRecording" function and it looks ok.
I did originally take the code from github, and made only one addition for sessionAttributes (plus some console.logs).
I have just compared aws-lex-audio.js on github with what I have and this shows as:
`
231c231
< var DEFAULT_USER_ID = 'LexTester';

var DEFAULT_USER_ID = 'userId';
353,354d346
< console.log('Sending:');
< console.log(state.lexConfig);
357d348
< //console.log(data);
402d392
< lexConfig.sessionAttributes = lexConfig.hasOwnProperty('sessionAttributes') ? lexConfig.sessionAttributes : {};
`

When I restart the client (in chrome), it does not show duplicates.
OS: macosx 10.13.4
Browser: chrome
Any help is appreciated.
Colin Goldberg

Thanks for posting this here. If you change your Intent to "Return parameters to client" do you see the same behavior?

Closing due to lack of activity. Please re-open if this is still an issue.

Sorry, I missed your question of 15 days ago. Setting it to "Return parameters to client", I am not sure if I am looking at the best place to detect possible duplication. I had the javascript console open, and the response the second time was the same as the first time - so it does not seem as if there was duplication.
Should I look anywhere else to detect this?
Colin

I have just tested again, and I think I have to retract my last conclusion.
I created a new intent, and have been testing it with my copy of your code on github. "Return parameters to client" is the (unchanged) setting.
I have evidence of 'duplication' (or 'not initializing the buffer...') - see below.
My utterance was "set the context to geography". My steps were:

click the link, click "Allow"
say "set the context to geography"
it shows "Passive"
click the link, click "Allow"
say "set the context to geography"

The javascript console shows:

In case the pasted image is not clear, the second (ie. last) entry shows:
inputTranscript: "set the context to geography set the context to geography"
Regards
Colin Goldberg

Okay, interesting. It seems that the conversation did not advance after you clicked "allow" the first time. Can you help me answer the following questions?

When you say clicked "allow" do you mean the Chrome popup which asks if you want to allow access to the mic (I assume so but can you confirm)?
You are using Chrome as well correct (you also mentioned this above but I want to confirm).

Assuming the above two answers, the behavior you should see is that Chrome prompts you for access to the mic only once. Once you say "yes" it should not ask you again (it saves your preference) and the conversation state should transition to Listening. Subsequent page loads should automatically grant you access to the mic without a popup. So, I'm curious why you are being prompted twice (or, more than once for that matter)... are you always prompted or only after clearing your preferences?

Testing this again...

opened the page

it shows "Passive"

clicked the 'mic', then the Chrome 'allow' option

spoke "set the context to geography"

console output:

it shows "Passive"

speaking does not activate anything

clicked the 'mic', then the Chrome 'allow' option

spoke "set the context to geography"

console output:

Yes, I am using Chrome. I hope the above sequence shows that Chrome is asking me to allow access when it shows "Passive" - ie. each time. I was not clearing anything during the test.
Also, it is silent for each response - no audio output accompanying the receipt of the responses above.

Interesting, I played around with Chrome and the only way I was able to get it to prompt me every time was to open the file locally (rather than serve it locally). Presumable Chrome does not store preferences for local file paths. In any case, I cannot reproduce the issue you're seeing--when prompted by Chrome and after allowing access the example code transitions to listening. Perhaps we can get together on a conference call to debug? Is it okay if I contact you at your github account email address to setup a call?

I have been testing using a local file - maybe that's the answer - or part of it.
I will test it from my website to make sure - give me a little time as I am refactoring the site. I will be happy to communicate by email shortly to set up a call if necessary.

Sounds good!
On Wed, May 23, 2018, 6:42 AM Colin Goldberg<EMAIL_ADDRESS>wrote:

I have been testing using a local file - maybe that's the answer - or part
of it.
I will test it from my website to make sure - give me a little time as I
am refactoring the site. I will be happy to communicate by email shortly to
set up a call if necessary.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lex-browser-audio-capture/issues/7#issuecomment-391351028,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOWVFRx7tgfmPao4OPbHRMIif5iMIIzks5t1Wc8gaJpZM4TuCJn
.

If you want to schedule a time for next week, please let me know your preferences (day/time/zone) - email is ok.

Was able to reproduce. The buffer was not cleared if there was not audio response. Made a change to ensure the buffer is cleared after every interaction--even if there's no response from Lex.


Improving simulations.py input options

Context for changes
Example usage and tests
Users can now specify  weight_opts for any noise class as in the following example format:
python flamingpy/simulations.py -noise "passive" -distance 3 -ec "primal" -boundaries "open" -delta 0.05 -pswap 0.1 -errprob 0.1 -trials 50 -decoder "MWPM" -weight_opts "{'method': 'blueprint', 'prob_precomputed': True}"
Performance results justifying changes

[x] Not Applicable

Workflow actions and tests
Expected benefits and drawbacks
Related Github issues

[x] Not Applicable

Checklist and integration statements

[ ] My Python and C++ codes follow this project's coding and commenting styles as indicated by existing files. Precisely, the changes conform to given black, docformatter and pylint configurations.
[ ] I have performed a self-review of these changes, checked my code (including for codefactor compliance), and corrected misspellings to the best of my capacity. I have synced this branch with others as required.
[ ] I have added context for corresponding changes in documentation and README.md as needed.
[ ] I have added new workflow CI tests for corresponding changes, ensuring codecoverage is 95% or better, and these pass locally for me.
[ ] I have updated CHANGELOG.md following the template. I recognize that the developers may revisit CHANGELOG.md and the versioning, and create a Special Release including my changes.

@ilan-tz, please check above to see if this was what we wanted here. If so, I can finalize this to be merged.


Bug: Icon Hover over Projects

What happened?
Hover Effect for github is not properly according to actual github, we need to fixed this
Add screenshots

What browsers are you seeing the problem on?
No response
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@Harshu467 please avoid to create unnecessary issues. Now you've created 3 issues for icon hover problem. So, try to create a single issue for same problem.
fix this issue under #1829
Thanks for understanding.


Customers & invoices

Add functions to create, update, delete customers and to create invoice drafts in e-conomic.

Thanks


Set default text in textview

how i can set default text in RichEdtiroView like as
textviewText.text = "this is a Data"

how i can set default text in RichEdtiroView like as
textviewText.text = "this is a Data"


void(0) is not a function with dotnet new Angular project

I made multiple attempts to use the package in a project created via "dotnet new angular." My boss tried too. All of them ran into the "void(0) is not a function" error. We used mostly Chrome, but I also tried one branch in Edge and saw the same problem. I've published all of our attempts on github at the url below. Each branch is another failed attempt.
https://github.com/verybadcat/AngularTreeComponent
Here is package.json from one of the branches:
{
  "name": "Angular",
  "private": true,
  "version": "0.0.0",
  "scripts": {
    "test": "karma start ClientApp/test/karma.conf.js"
  },
  "dependencies": {
    "@angular/animations": "4.2.5",
    "@angular/common": "4.2.5",
    "@angular/compiler": "4.2.5",
    "@angular/compiler-cli": "4.2.5",
    "@angular/core": "4.2.5",
    "@angular/forms": "4.2.5",
    "@angular/http": "4.2.5",
    "@angular/platform-browser": "4.2.5",
    "@angular/platform-browser-dynamic": "4.2.5",
    "@angular/platform-server": "4.2.5",
    "@angular/router": "4.2.5",
    "@ngtools/webpack": "1.5.0",
    "@types/webpack-env": "1.13.0",
    "angular-tree-component": "^4.1.0",
    "angular2-template-loader": "0.6.2",
    "aspnet-prerendering": "^3.0.1",
    "aspnet-webpack": "^2.0.1",
    "awesome-typescript-loader": "3.2.1",
    "bootstrap": "3.3.7",
    "css": "2.2.1",
    "css-loader": "0.28.4",
    "es6-shim": "0.35.3",
    "event-source-polyfill": "0.0.9",
    "expose-loader": "0.7.3",
    "extract-text-webpack-plugin": "2.1.2",
    "file-loader": "0.11.2",
    "html-loader": "0.4.5",
    "isomorphic-fetch": "2.2.1",
    "jquery": "3.2.1",
    "json-loader": "0.5.4",
    "preboot": "4.5.2",
    "raw-loader": "0.5.1",
    "reflect-metadata": "0.1.10",
    "rxjs": "5.4.2",
    "style-loader": "0.18.2",
    "to-string-loader": "1.1.5",
    "typescript": "2.4.1",
    "url-loader": "0.5.9",
    "webpack": "2.5.1",
    "webpack-hot-middleware": "2.18.2",
    "webpack-merge": "4.1.0",
    "zone.js": "0.8.12"
  },
  "devDependencies": {
    "@types/chai": "4.0.1",
    "@types/jasmine": "2.5.53",
    "chai": "4.0.2",
    "jasmine-core": "2.6.4",
    "karma": "1.7.0",
    "karma-chai": "0.1.0",
    "karma-chrome-launcher": "2.2.0",
    "karma-cli": "1.0.1",
    "karma-jasmine": "1.1.0",
    "karma-webpack": "2.0.3"
  }
}

Also worth noting -- a couple of times, we did see the tree, very briefly. Then the error happened and it disappeared.

Now cross-posted to SO, here: https://stackoverflow.com/questions/45884280/use-angular-tree-component-in-dotnet-new-angular-project


S3 Public Access Block

Hi @vishjain - did you intend this PR to be smaller? I think it needs a rebase and maybe some cleaning. What parts are the main ones?

This happened by mistake, please ignore.