Dip in TS confidence

I'm not sure how much can be done here but I thought I'd start a discussion.
Here's the following TypeScript snippet:
function makeThing(): Thing {
	let size = 0;
	return {
		get size(): number {
		    return size;
		},
		set size(value: string | number | boolean) {
		    let num = Number(value);
		    // Don't allow NaN and stuff.
		    if (!Number.isFinite(num)) {
			size = 0;
			return;
		    }
		    size = num;
		},
	};
}

Which yields the following confidence:
  { languageId: 'ts', confidence: 0.2830791473388672 },
  { languageId: 'rs', confidence: 0.10346205532550812 },
  { languageId: 'js', confidence: 0.09687642008066177 },
  { languageId: 'lua', confidence: 0.060949232429265976 },
  { languageId: 'cs', confidence: 0.052387114614248276 },
  { languageId: 'go', confidence: 0.04457798972725868 }
 // ...

Before #33, the confidence for TS was way over 40%. I'm currently saying "this file is a TS file if the model is at least 20% more confident than the next language" but unfortunately, this fails.
I'm a bit nervous to drop that 20% down any further...
Also interesting that Rust beat out JS...

Just before the merge, I updated the model that was in the PR with a better trained one https://github.com/yoeo/guesslang/pull/33/commits/198352a0027199f29c995afe9db5a66dd9403e99 . Maybe you're using the model that was pushed just before this change.
If that's the case, you can now find the lastest model in the main branch https://github.com/yoeo/guesslang/tree/master/guesslang/data/model
It is not as precise as the 30-languages model, but and it produces better results with your Typescript code snippet:
 ✓ echo $'function makeThing(): Thing {
        let size = 0;
        return {
                get size(): number {
                    return size;
                },
                set size(value: string | number | boolean) {
                    let num = Number(value);
                    // Don\'t allow NaN and stuff.
                    if (!Number.isFinite(num)) {
                        size = 0;
                        return;
                    }
                    size = num;
                },
        };
}' | guesslang -p
Language name       Probability
 TypeScript           32.21%
 JavaScript            9.24%
 Rust                  7.14%
 C#                    5.80%
 C                     4.71%
 Lua                   4.59%

@TylerLeonhardt, just out of curiosity, how do you convert the model to TensorflowJS?
I tried with the current stable version of tensorflowjs, with no special options, and got an error:
tensorflowjs_converter --input_format=tf_saved_model ./guesslang/data/model /tmp/web_model

...
E tensorflow/core/grappler/grappler_item_builder.cc:669] Init node head/predictions/class_string_lookup/table_init/LookupTableImportV2 doesn't exist in graph
...
Instructions for updating:
Use `tf.compat.v1.graph_util.extract_sub_graph`
Traceback (most recent call last):
  File ".../bin/tensorflowjs_converter", line 8, in <module>
    sys.exit(pip_main())
  File ".../lib64/python3.9/site-packages/tensorflowjs/converters/converter.py", line 813, in pip_main
    main([' '.join(sys.argv[1:])])
  File ".../lib64/python3.9/site-packages/tensorflowjs/converters/converter.py", line 817, in main
    convert(argv[0].split(' '))
  File ".../lib64/python3.9/site-packages/tensorflowjs/converters/converter.py", line 803, in convert
    _dispatch_converter(input_format, output_format, args, quantization_dtype_map,
  File ".../lib64/python3.9/site-packages/tensorflowjs/converters/converter.py", line 523, in _dispatch_converter
    tf_saved_model_conversion_v2.convert_tf_saved_model(
  File ".../lib64/python3.9/site-packages/tensorflowjs/converters/tf_saved_model_conversion_v2.py", line 683, in convert_tf_saved_model
    optimize_graph(frozen_graph, signature,
  File ".../lib64/python3.9/site-packages/tensorflowjs/converters/tf_saved_model_conversion_v2.py", line 153, in optimize_graph
    raise ValueError('Unsupported Ops in the model before optimization\n' +
ValueError: Unsupported Ops in the model before optimization
OptionalNone, ReadVariableOp, OptionalFromValue

@pyu10055 gave me this pointer in
https://github.com/tensorflow/tfjs/issues/4838#issuecomment-866416464

tensorflowjs_converter --input_format=tf_saved_model --skip_op_check model web_model

Maybe you're using the model that was pushed just before this change.

hmm I grabbed https://github.com/yoeo/guesslang/tree/master/guesslang/data/model this morning actually so I'm fairly certain I have the correct one... I wonder if there's a loss in confidence during the conversion to the tfjs model 🤔

Cool the conversion now works.
I can see that the converter prints messages about various optimisations.
I especially suspect that the int64 to int32 conversion have an impact the model accuracy.

I especially suspect that the int64 to int32 conversion have an impact the model accuracy.

Maybe @pyu10055 has guidance here? Or perhaps @dynamicwebpaige?

The missing ops errors can be ignored, since those are maybe from some of the training functions not used in the inference graph.

@pyu10055 OK, during the training phase I do use I/O functions to read & process the examples and as you spotted these function are not used for inference.


Keiji test branch

Added sport function to display calories and duration information on index.html page.
Added sport function to prepare sports information package to be sent to local storage for use on second page.
Added function to send to local storage  (line 334) how we join recipe info and sport info into local storage is still undecided. Please see this function just as one idea.
Added test function to run all the sport related function in correct sequence. (line 371)
note: sport information still uses temporary calorie variable that needs to be replaced with actual recipe calorie information.

close this to avoid other merge this in accidentally. @meekunn1 please PR again after you updated your branch from main and PR again for your sport part of the code.


Compose plugin causes crash on app startup

Describe the bug
Adding jetbrains compose plugin crashes on iOS devices below iOS 17
org.jetbrains.compose:1.6.10-dev1580
Affected platforms

iOS

Versions

Kotlin version*: 1.9.22
Compose Multiplatform version*: 1.6.2
OS version(s)* (required for Desktop and iOS issues): iOS 15.1
OS architecture (x86 or arm64): arm64
Device (model or simulator for iOS issues): IPad Pro

To Reproduce
Steps and/or the code snippet to reproduce the behavior:

Download a Compose Multiplatform Wizard template
Add this version of the jetbrains compose plugin 1.6.10-dev1580
Sync
Run on affected device

Expected behavior
Not crashing. Display the compose multiplatform template
Additional context
This is the stacktrace in XCode
dyld[558]: Symbol not found: _OBJC_CLASS_$_UITextLoupeSession
 Referenced from: /private/var/containers/Bundle/Application/<UUID>/App.app/App
 Expected in: /System/Library/Frameworks/UIKit.framework/UIKit

It works using this version 2.6.2

@elijah-semyonov
@ellomr3 reported that it reproduces on version: "1.6.10-dev1593"

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


fix: tsconfig 引用路径解析问题

tsconfig 使用 extends 字段引用其他文件的时候，会导致 projectDir变成了一个object，join(projectDir, tsConfig.extends)的时候就会报错

Codecov Report
Base: 88.70% // Head: 88.70% // No change to project coverage :thumbsup:

Coverage data is based on head (e06276e) compared to base (6d70010).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #318   +/-   ##
=======================================
  Coverage   88.70%   88.70%
=======================================
  Files          40       40
  Lines        4135     4135
  Branches     1214     1214
=======================================
  Hits         3668     3668
  Misses        430      430
  Partials       37       37

Impacted Files
Coverage Δ

packages/cli-plugin-build/src/index.ts
97.31% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


DX enhancements

Using Vite with Preact seems like a great way to reduce bundle size and speed up development (HMR 👀)

Will this still work with PurgeCSS? The rollup plugin is archived on GitHub, but was published only 4 months ago... https://v1.purgecss.com/

Preact works fine if I just use preact/compat, but switching off of React Router (not necessary, but just checking) introduced a million subtle bugs, and honestly the whole thing is barely worth it (almost 100kb savings, but considering the pokemon JSON payload is something like 7x that size, I'm kinda eh on the whole thing being worth the hassle of solving these edge cases)

Rollup plugin seems broken, so I'd have to use the CLI https://purgecss.com/CLI.html#installation

It seems the official rollup plugin for PurgeCSS doesn't work correctly https://github.com/FullHuman/purgecss/blob/master/packages/rollup-plugin-purgecss/src/index.ts
I've implemented a very rough plugin that appears to work correctly. I need to clean up the code and put this in its own GitHub repo, open an issue to FullHuman/purgecss, and hopefully get that fixed upstream. In the mean time, I can use my own plugin which seems to work for my purposes.
import { createFilter, FilterPattern } from "@rollup/pluginutils";
import { Plugin } from "vite";
import { PurgeCSS } from "purgecss";

interface PurgePluginOptions {
  include?: FilterPattern;
  exclude?: FilterPattern;
}

export function purgePlugin({
  include,
  exclude,
}: PurgePluginOptions = {}): Plugin {
  const filter = createFilter(include, exclude);
  return {
    name: "purge",
    async generateBundle(options, bundle) {
      console.log(bundle);
      const files = Object.keys(bundle);
      const purge = new PurgeCSS();
      const content = [
        ...files
          .filter((f) => f.endsWith(".js"))
          .filter(filter)
          .map((f) => {
            const thing = bundle[f];
            const raw = thing.type === "chunk" ? thing.code : "";
            return { file: f, extension: "js", raw };
          }),
        ...files
          .filter((f) => f.endsWith(".html"))
          .filter(filter)
          .flatMap((f) => {
            const thing = bundle[f];
            if (thing.type === "asset" && typeof thing.source === "string") {
              return [{ file: f, extension: "html", raw: thing.source }];
            }
            return [];
          }),
      ];
      const css = [
        ...files
          .filter((f) => f.endsWith(".css"))
          .flatMap((f) => {
            const thing = bundle[f];
            if (thing.type === "asset" && typeof thing.source === "string") {
              return [{ file: f, raw: thing.source }];
            }
            return [];
          }),
      ];
      const results = await purge.purge({ content, css });
      // TODO: Since I'm using raw content (which don't have filenames) for
      // everything, I'm relying on PurgeCSS to return result chunks in the same
      // order as the input CSS chunks...
      for (const [i, obj] of Object.entries(css)) {
        const res = results[i];
        const thing = bundle[obj.file];
        if (thing.type === "asset") {
          thing.source = res.css;
        }
      }
    },
  };
}

Fixed this by using PostCSS with PurgeCSS as a plugin! That seems well-maintained


[ci] update test report job dependency

Motivation
Update the test report job's dependency to include the prepare job so that it vars are set.
Test Plan
CI itself

/land

/cancel

Actually it doesn't use prepare. It need an environment: sccache set.

Good catch on the sccache env.
On the prepare step dep, IMO we need it. see the if condition in line 95 ${{ needs.prepare.output.changes-target-branch == 'main' }}

Rex will open a proper fix. Closing this one.


get_recent_posts() raises MissingCookieWarning but we can't pass a valid cookie

Describe the bug
The get_recent_posts() method raises MissingCookieWarning, but we can't pass a valid cookie header to avoid that
To Reproduce
from instascrape import *

instagram_sessionid = "xxx"
headers = {"user-agent": "Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Mobile Safari/537.36 Edg/87.0.664.57",
"cookie": f"sessionid={instagram_sessionid};"}
profile = Profile('https://www.instagram.com/google/')
profile.scrape(headers=headers)
print(profile.posts)
recents = profile.get_recent_posts() #We should pass a cookie here

The code is executed correctly but we get a MissingCookiesWarning: Request header does not contain cookies! It's recommended you pass at least a valid sessionid otherwise Instagram will likely redirect you to their login page. warning
If I try to pass a header cookie:
from instascrape import *

instagram_sessionid = "xxx"
headers = {"user-agent": "Mozilla/5.0 (Linux; Android 6.0; Nexus 5 Build/MRA58N) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Mobile Safari/537.36 Edg/87.0.664.57",
"cookie": f"sessionid={instagram_sessionid};"}
profile = Profile('https://www.instagram.com/google/')
profile.scrape(headers=headers)
print(profile.posts)
recents = profile.get_recent_posts(headers=headers) #This time I try to pass an header cookie

I get a TypeError: get_recent_posts() got an unexpected keyword argument 'headers'
Expected behavior
We should be able to pass a valid cookie to avoid the warning or the warning should not be triggered altogether.

Have the same issue!

Any way around it so far without selenium?

I get the same error and posted about it here https://github.com/chris-greening/instascrape/issues/89#issuecomment-801495835

I fixed it by passing cookies to Selenium before going to the profile. I do this by exporting the cookies from instagram with the chrome extension Cookie-Editor. And then just copy paste it to cookies.json
url = f"https://www.instagram.com/{handle}/"

driver.get(url)  # Needed to fake a login
# Fake login with Cookies
with open("./cookies.json", "r", newline="") as data:  # Open cookies.json
    cookies = json.load(data)
    for cookie in cookies:  # Add cookies to driver
        cookie.pop("sameSite")  # Selenium breaks with sameSite
        driver.add_cookie(cookie)  # Add our authorized cookies

ig_profile = Profile(url)  # Set IG profile
ig_profile.url = url
ig_profile.scrape(headers=headers)  # Scrape IG profile

I spent a lot of time trying this.  Not sure what cookie.pop("sameSite") is doing, so I skilled all that and just ran driver.add_cookie({'name':'sessionid','value':os.environ['INSTAGRAM_SESSIONID']}) which just resulted in the same MissingCookiesWarning.

I've been combing through the code.  Looks like you have to pass your driver to the scrape method as well.  I mention it here https://github.com/chris-greening/instascrape/issues/89#issuecomment-805394041 but I'm still getting the same error even with the driver passed to scrape, which is very weird if you read the code.

Just noting that this issue has made it pretty much impossible for me to use instascrape for my use case.  Due to this issue and https://github.com/chris-greening/instascrape/issues/89 at this point I've abandoned instacrape.

get_recent_post() always returns 24 post no matter the amount,
can I bypass that? like get all the post?

I tried the same thing as he did (adding the cookie manually) but still I'm gettingthe warning. Like what am I doing wrong?
Here's the code I am using:
SESSION_ID = 'my session id'
url = f"https://www.instagram.com/discordbot98/"
webdriver.get(url)
time.sleep(10)
webdriver.add_cookie({'name': 'sessionid', 'value': SESSION_ID})


Download directory "latest/" does not point to the latest files after last update

Hi Jeremy,
after the last update in September https://unmtid-shinyapps.net/download/TIGA/latest/ still points to the files from March.
Just a side note: The link in the Downloads section has an 's' too much and should be https://unmtid-shinyapps.net/download/TIGA/latest/.

Alright, I see. Thanks for clarifying!


Convert oembed to async

Fixes

Fixes #3457 by @sarayourfriend
Description

Converts the oembed endpoint to async. This was fairly straightforward and hopefully serves as a simpler example of the concept for others to work on additional views, if they wanted.
Testing Instructions

Unit tests should pass. Test the oembed endpoint as well and confirm it works the same as in staging.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[N/A] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@krysal Just a heads up, I had to change part of the way #3069 worked because async serializer support does not include to_internal_value, so we can't make the database call in there. The change was just to return only the identifier, and then continue to use aget_object_or_404 with the parsed identifier from the serializer.


Add diagonal streets to our simple city.

Addresses issue #16.
Adds diagonal streets to our SimpleCityBuilder
Screenshots:

@patricioreyna !!!


chore: remove stray files

Remove files introduced in https://github.com/AztecProtocol/aztec-packages/pull/7078.

#7158  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

proof_construction_time_sha256_30_ms (1): 15,465 (+34%)
proof_construction_time_sha256_30_ms (16): 1,378 (-26%)
avm_simulation_time_ms (Token:mint_public): 69.3 (-89%)
avm_simulation_time_ms (Token:assert_minter_and_mint): 226 (+200%)
app_circuit_proving_time_in_ms (AuthRegistry:set_authorized (avm)): 1,652 (+20%)
app_circuit_proving_time_in_ms (FPC:prepare_fee (avm)): 4,886 (-19%)
app_circuit_proving_time_in_ms (FPC:pay_refund (avm)): 8,971 (-19%)
protocol_circuit_proving_time_in_ms (private-kernel-tail-to-public): 85,135 (-16%)
protocol_circuit_proving_time_in_ms (root-parity): 33,633 (-16%)
protocol_circuit_proving_time_in_ms (public-kernel-teardown): 40,373 (-16%)
protocol_circuit_proving_time_in_ms (public-kernel-tail): 165,022 (-16%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_30_ms
:warning: 15,465 (+34%)
3,054 (-2%)
:warning: 1,378 (-26%)
1,416 (-12%)
1,450 (-6%)

proof_construction_time_sha256_100_ms
50,468 (+11%)
13,234 (+9%)
5,673
5,530 (-1%)
5,517

proof_construction_time_poseidon_hash_ms
77.0 (-1%)
34.0
34.0
57.0
87.0

proof_construction_time_poseidon_hash_30_ms
1,721 (+12%)
417 (-1%)
202
224 (-1%)
266

proof_construction_time_poseidon_hash_100_ms
5,737 (-1%)
1,568 (-2%)
726
773 (+2%)
801 (-1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,452
9,464
9,440

l1_rollup_execution_gas
610,273
610,285
610,261

l2_block_processing_time_in_ms
746 (-1%)
1,408 (+1%)
2,678

l2_block_building_time_in_ms
25,690 (+2%)
50,820
101,654 (+2%)

l2_block_rollup_simulation_time_in_ms
25,594 (+2%)
50,522
101,302 (+2%)

l2_block_public_tx_process_time_in_ms
21,953 (+2%)
46,773
97,549 (+2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
6,970 (+1%)
9,838 (+1%)

node_database_size_in_bytes
12,152,912
16,109,648

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
68.0 (-3%)
13,873

GasToken:set_portal
16.7 (+4%)
3,495

Token:constructor
95.4 (+2%)
24,207

FPC:constructor
62.5 (+2%)
13,893

GasToken:mint_public
53.5 (+2%)
10,241

Token:mint_public
:warning: 69.3 (-89%)
19,216

Token:assert_minter_and_mint
:warning: 226 (+200%)
13,034

AuthRegistry:set_authorized
32.6 (+2%)
7,869

FPC:prepare_fee
145 (-19%)
15,187

Token:transfer_public
35.7 (+16%)
31,425

FPC:pay_refund
139 (+10%)
20,080

Benchmarking:increment_balance
2,710 (+2%)
15,465

Token:_increase_public_balance
55.8 (-8%)
15,089

FPC:pay_refund_with_shielded_rebate
147 (-6%)
21,167

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.174 (-3%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.3 (+1%)
16.8 (+1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.599 (+1%)
0.517 (+1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
49.0 (+1%)
75.6
130 (-1%)
243
468 (-1%)

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.501 (+1%)
0.464
0.448 (-1%)
0.442
0.437 (-1%)

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
59.6
111
181 (-1%)
351 (+1%)
688 (-1%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.504
0.498
0.476 (-1%)
0.475 (+1%)
0.473 (-1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
72.7 (+1%)
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.518 (+1%)
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
85,672
670,983

Transaction size based on fee payment method
| Metric |  |
| - |  |


[loadable-address] Move rewriting of yields into the loop that handles struct_extracts, switch_enum, and applies.

yields are like applies in the sense that processing them can produce
struct_extract that need to be rewritten.
rdar://80646212

@swift-ci smoke test


Folder on network drive does not open on restart, gets removed from Recent Folders list

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

VS Code Version: 1.82.0 (system setup), commit 8b617bd08fd9e3fc94d14adb8d358b56e3f72314
OS Version: Windows 10 Enterprise 22H2 Build 19045.3324

Steps to Reproduce:

Have a folder on a network drive
Open folder in VSCode
File > Exit VSCode
Disconnect and reconnect network folder (or reboot computer, or disconnect and reconnect VPN through which network drive is mounted)
Reopen VSCode

Expected behavior:

Window that contains folder should open
folder should be in folder list under File > Open Recent

Actual behavior:

Window does not open
folder is not listed in Recent Folders

Even if the folder is disconnected and hence not available for reopening, it would be helpful if it were still listed as a Recent Folder. This also appears to differ from one networked machine to another.
This is a duplicate of #13699 and #29585 which were both closed in favor of a "collective issue" #13665, which in turn was closed before actually fixing the behavior described in #13699 - Task 2 of #13665 was not completed.
Hence this new issue because I am unable to comment on the closed issues.

/duplicate https://github.com/microsoft/vscode/issues/164491

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Publish 1.4.0 to npm

I can not install v1.4.0 from npm

Hey @kotazi
Published it now, try again 😄
https://www.npmjs.com/package/react-native-social-share


Fix: hpa trait corresponding to cpu parameters when try to use memory

Description of your changes
I fixed build-in hpa trait which is using cpu parameters when try to use memory.
I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
e2e test
Special notes for your reviewer

good catch, thanks


Improving input

It gets too complicated to add the basis_set and the GTH paths in the yml input file. Especially because with hybrids, also the BASIS_ADMM and BASIS_ADMM_MOLOPT are required.
The best way is to add only the path to the folder where all basis type files are stored:
path_basis = /home/v13/cp2k_basis
Then in the program takes care to add all paths in the input. The ADMM basis_set_file_name can also be added in non-hybrid calculations as it wont affect the calculation.

fixed by #151


Remove distributed param

Description

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

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

Comparison is base (34830a7) 80.33% compared to head (c0e4076) 67.74%.
Report is 105 commits behind head on main.

Files
Patch %
Lines

superduperdb/backends/ibis/query.py
73.58%
42 Missing :warning:

superduperdb/backends/sqlalchemy/metadata.py
46.37%
37 Missing :warning:

superduperdb/backends/mongodb/metadata.py
34.09%
29 Missing :warning:

superduperdb/backends/mongodb/artifacts.py
9.09%
20 Missing :warning:

superduperdb/backends/dask/compute.py
48.48%
17 Missing :warning:

superduperdb/backends/local/artifacts.py
68.88%
14 Missing :warning:

superduperdb/backends/mongodb/query.py
83.33%
10 Missing :warning:

superduperdb/base/config.py
78.26%
10 Missing :warning:

superduperdb/backends/base/compute.py
70.37%
8 Missing :warning:

superduperdb/base/build.py
75.00%
7 Missing :warning:

... and 10 more

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1377       +/-   ##
===========================================
- Coverage   80.33%   67.74%   -12.60%
===========================================
  Files          95      102        +7
  Lines        6602     7227      +625
===========================================
- Hits         5304     4896      -408
- Misses       1298     2331     +1033

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


PR: Shift maximum character edge line to match editor characters

Description of Changes
The vertical line indicating the maximum characters per line (edge line) is shifted 3 units to the right so that it aligns with the characters of the editor.
Current behavior:

Shifting line to the right by 3 units:

This is tested also for other font sizes (7-16) and works well, as well as other standard monospace fonts shipping with Windows.
Why shift by 3?
I tested with 3 first based on this being added to the line number margin https://github.com/spyder-ide/spyder/blob/bb2ed9a6c90df30f370f7f67413d31b363cbfe15/spyder/plugins/editor/panels/linenumber.py#L186
And it works quite well, by chance I suppose. I also tried removing the 3 from the line number margin (without shifting edge line), but this has no effect.
An alternative solution which also appears to work quite well is to shift the line relative to the character width. Might be more robust for different (odd) fonts/resolutions. To keep this simple fix very simple I stuck with shifting 3 units as it appears to work well across sizes and fonts. Example shifting 1/3rd of a character width, an appropriate number is probably between 1/3 and 1/2.
char_width = self.editor.fontMetrics().width('9')
x = column * char_width + 0.33 * char_width

Issue(s) Resolved

Fixes #16248
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Reinert Huseby Karlsen

Thanks a lot for your help @rhkarls! Please make your branch to derive from our 5.x branch (instead of master), with the following commands:
git checkout 5.x
git checkout shift-editor-edge-line
git rebase --onto 5.x master shift-editor-edge-line
git push -f origin shift-editor-edge-line

Otherwise looks good to me.

Thanks a lot for your help @rhkarls! Please make your branch to derive from our 5.x branch (instead of master), with the following commands:
git checkout 5.x
git checkout shift-editor-edge-line
git rebase --onto 5.x master shift-editor-edge-line
git push -f origin shift-editor-edge-line

Otherwise looks good to me.

Cheers! Hope its good with the 5.x branch now - sorry I missed to checkout 5.x earlier


Setup Kanban Board

Hello reviewer!
for this milestone, we create a Kanban board to manage the tasks of our "VECS SCHOOL" project.
Group members (4): @vvoo21 / @christianonoh  / @smunoz1988 / @Ellyboi
Link to project: https://github.com/users/christianonoh/projects/6

Hi STUDENT_MENTION
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not have a screenshot of the kanban board.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @smunoz1988 ,
Good job so far!
There are some issues that you still need to work on to prepare your project for the final evaluation, but you are almost there!
To highlight:

All cards have been assigned and pointed.

You are really close to finishing the Microverse program!! Keep it up! 👍👍👍

After implementing the requested changes, please submit another review request. ♻

[ ] According to the project requirements:

At the time of this review, these are the pointings:
Ellyboi 49
christianonoh 49
vvoo21 61
smunoz1988 61
As you can see, there are huge points differences that are greater than the limit which is 5.  (61-49 = 12,  12 > 5)
Therefore:

[ ] Kindly ensure nobody has a difference of 5 points as compared to any other person on the team.  The difference limit is 5.

That way, we will meet this particular requirement.
Tip:  Just reassign some tasks/play with the numbers until the requirement is met.
Cheers and Happy coding!👏👏👏
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the previous reviews unless it is requested otherwise.

@Meltrust  thanks for the review.
We reassign some tasks to match the requirement:
total hours from tasks:

Ellyboi 49
christianonoh 49
vvoo21 46
smunoz1988 46

(+/- 3)

Hi @smunoz1988,
Wow, you did it 🎉

Thank you for implementing the changes requested by the previous reviewer 💪 🥇 ㊗
Well done!
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

Pointings are now equal, according to the requirement ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


image load error handler

hey! first, i really like your plugin, thank you for sharing...
i was wondering, if there is any error handling callback in your options.
i tried replacing the src element of the #imagelightbox by a default error image (with the onLoadStart option), but the img#imagelightbox element will not be created if the loading process of the image src fails, right?
i would like to replace the img src of the lightbox image by a default image, if the src cannot be loaded.

Hi Devbors,
right now there is no errorcallback handling implemented, would be possible though by expanding the error function in line 214 with something. Is your js experience good enough to make that yourself (and send me a pullrequest) or wuld yu wait for me (or someone else) to add it?

@DevbrosDevelopment @rejas I'm looking into implementing Promise, which can be used for this.


only prune the managed resources.

resolve: https://github.com/istio/istio/issues/19815 and https://github.com/istio/istio/issues/19816

make the pruning list be dynamic map instead of static slice
add the created/updated resources to the pruning list
only prune the resource in the dynamic prune list
don't clean the istio CRDs

@morvencao: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator

Nice!

@morvencao: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test test_operator

e2e_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test e2e_operator

test_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test e2e_operator

test_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test test_operator

lint_operator
060766bee1746ff0ea4c319b27ee126e2f002146
link
/test lint_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
72301c1212aa0698f16ac9500120a5c6efe0c4ed
link
/test e2e_operator

lint_operator
060766bee1746ff0ea4c319b27ee126e2f002146
link
/test lint_operator

test_operator
8537f4a3da5e74285a288e582c5b95bf860727a0
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
8537f4a3da5e74285a288e582c5b95bf860727a0
link
/test test_operator

e2e_operator
8537f4a3da5e74285a288e582c5b95bf860727a0
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
8537f4a3da5e74285a288e582c5b95bf860727a0
link
/test e2e_operator

test_operator
d30873a1015e3e00a48e40bb2bcc3688a34d04bd
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator

@morvencao: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
d30873a1015e3e00a48e40bb2bcc3688a34d04bd
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e_operator
d30873a1015e3e00a48e40bb2bcc3688a34d04bd
link
/test e2e_operator

test_operator
bfdb5bc11629218ebbeaaa77e1a6cb20a9ae99c5
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator

@morvencao: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

test_operator
ed0f5d7d78176c1b406711890db33ec601a430d5
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test test_operator

You should verify that this works when the operator is restarted between install and delete.  I suspect this will result in nothing being pruned, as the new instance of the operator won't have performed an install, thus it won't think there are any resource types that need pruning.
@ostromart FYI


Opening multiple files via Windows Explorer does not work as expected

Issue Type: Usability
Steps to reproduce:

Select two files in Windows Explorer

From the context menu choose Open with Code (Insiders)

Expected behavior:

One VS Code instance with the two selected files loaded

Current behavior:

It seems that two instances of VS Code are launched, but only the first wins and opens one of the two files. The second instance shows an error message.

VS Code version: Code - Insiders 1.34.0-insider (d234937aedcbc265d2272ba5861125293239f9c7, 2019-04-05T05:18:43.251Z)
OS version: Windows_NT x64 10.0.17134

Update: It seems that VS Code supports the described scenario most of the times. At the moment the steps above show the expected behavior. I will have to investigate this further...

OK

Though trying hard to reproduce the error, it did not appear any more in the past days.


Regression: Changing the sign column color when there are errors doesn't work any more

See comments here: #825

color

Bloody Webster.


Type flow analysis in constructor not working properly with strictNullChecks

TypeScript Version:  3.2.0-dev.201xxxxx

Search Terms:
strictNullChecks
constructor
Code
(This may actually be two separate issues.)
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.

class X
{
	constructor()
	{
		// Commenting and uncommenting this supposedly no-op line
		// causes a definite assignment error to be reported on the private x field.
		if (this instanceof X) { }
		false && (this.x = null!);

		// Possibly related, but statically verifyable truthy expressions
		// don't cause definite assignment verification to be fulfilled.

		// Causes the definite assignment condition to be fulfilled.
		true && (this.x = null!);

		// *Should* cause the definite assigment condition to be fulfilled,
		// but does not appear to do so.
		"I'm true" && (this.x = null!);
		1 && (this.x = null!);
	}

	private x: X;
}

Associated tsconfig file:
{
	"compilerOptions": {
		"module": "commonjs",
		"target": "es2017",
		"lib": [
			"es7",
			"es2017",
			"es2017.object"
		],
		"inlineSourceMap": true,
		"inlineSources": true,
		"noEmit": true,
		"noImplicitThis": true,
		"strict": true,
		"strictNullChecks": true,
		"rootDir": "./",
		"baseUrl": "./",
		"stripInternal": true
	}
}

Expected behavior:
Uncommenting the if (this instanceof X) { } line should have no effects on the rest of the method. I would also expect expressions whose computed values are statically verifiable to contribute to type flow analysis, rather than just the "true" / "false" literals. Of course there is a very long tail to how far this could be taken, but as it stands, my original intent of using truthy literals as compilation constants (that are replaced in the build process) can't be realized with strictNullChecks:
"__DEBUG__" && this.someFieldOnlyVisibleDuringTests = ...

(Unless of course I just exploit this bug behavior)
Actual behavior:
Covered above.
Playground Link:
(I'm fairly sure strictNullChecks aren't an option in the playground)
Related Issues:

Updates: After more poking around, it appears that any conditional in a constructor that checks for this instanceof WhateverTheCurrentClassIs, regardless of it's contents, basically disables the entire definite assignment checking process.

I'm unable to reproduce this behavior with the current master branch or the playground, even with verbatim copies of your example and tsconfig.json file. I'm not seeing any errors on the private x property, regardless of whether the line with the instanceof check is commented out.

I'm fairly sure strictNullChecks aren't an option in the playground

The are indeed an option. Just click Options, check the box, and click Options again to make the drop down go away.

I appeared to have misreported the issue. The actual error occurs when strictPropertyInitialization is on. It also appears that the case is broader: this instanceof AnythingValid seems to trigger the bizarre behavior.
Here is the relevant playground link:
https://www.typescriptlang.org/play/#src=class Foo
{
	constructor()
	{
		%2F%2Fif (this instanceof Boolean) { }
	}

	readonly y%3A number%3B
	readonly z%3A string%3B
}

Yeah, that looks like an issue in the control flow analyzer.


Error on Android 4.1 Browser and Chrome Desktop 42

Stack trace:
Failed to execute 'insertRule' on 'CSSStyleSheet': Failed to parse the rule '@keyframes react-reveal-956029157852754-1{from {opacity: 0; transform: translate3d(0, -100%, 0);}      to {opacity: 1;transform: none;} }'. at Error (native)
It is only in production.
UPD
The problem in a version of a package in npm:
function animation(e) {     if (!sheet)         return "";     var n = "@keyframes " + (name + counter) + "{" + e + "}", t = effectMap[e];     return t ? "" + name + t : (sheet.insertRule(n, sheet.cssRules.length), effectMap[e] = counter, "" + name + counter++) }

Please, update version in npm repo with https://github.com/rnosov/react-reveal/pull/29 changes.


Doesnt work for 1.21

Mojang changed it so all folders like "functions" and "advancements" now dont have the "s" anymore. Like "function" and "advancement".
But after changing those myself, i have a problem where all enchanted armor and tools are deleted upon death. Not unechanted items.

Pretty sure this line of code is at fault, iirc it should be "enchantments" where it says "enchantment", though not completely certain and not really enough time to look deeper into it.
If that is not the actual error, it’s definitely this line.

Ive finally been able to test it myself but it just stoped working entirely...

Has there been any updates for when 1.21 will be supported?

1.21 rewrite has been released 🎉


"--show-only" shows an error when the template is empty

I get an error when using helm template --show-only templates/test.yaml my-chart and the manifest generated from this is empty, e.g:

Having a template in templates/secret.yaml and values.yaml:

{{- if .Values.secret.enabled }}
apiVersion: v1
kind: Secret
metadata:
  name: my-secret
  labels:
    app.kubernetes.io/name: my-app
    helm.sh/chart: my-chart-1
type: Opaque
data:
  bind-password: {{ (include "postgresql-ha.ldapPassword" .) | b64enc | quote }}
{{- end -}}

secret:
  enabled: false

Executing helm template --show-only templates/secret.yaml my-chart I get:

Error: could not find template templates/secret.yaml in chart

And executing helm template --show-only templates/secret.yaml --set secret.enabled=true my-chart I get:

---
# Source: postgresql-ha/templates/ldap-secrets.yaml
apiVersion: v1
kind: Secret
metadata:
  name: my-secret
  labels:
    app.kubernetes.io/name: my-app
    helm.sh/chart: my-chart-1
type: Opaque
data:
  bind-password: "YWRtaW4="

should it be an error? Actually, the template is there but it is empty, right?
How can I avoid this error message?
Output of helm version:
version.BuildInfo{Version:"v3.0.1", GitCommit:"7c22ef9ce89e0ebeb7125ba2ebf7d421f3e82ffa", GitTreeState:"clean", GoVersion:"go1.13.4"}

Output of kubectl version: not relevant
Cloud Provider/Platform (AKS, GKE, Minikube etc.): not relevant

Worth noting this is different behaviour from Helm 2.

Worth noting this is different behaviour from Helm 2.

This seems intentional. The reason being here is that helm template -x would only render a particular template and show the output. -show-only renders the full chart, then shows the output of said template. The big difference here being that the old syntax required the file to be present in the chart before it was rendered, and the new flag requires the template to exist after the rendering process. There were cases with the old flag where if a particular template relied on another template to render correctly, the user effectively broke the chart author’s assumptions about how the chart is rendered (like with a library chart as a dependency).
In other words, the behaviour here seems correct from a technical standpoint and it is intended to return an error when a template is empty. What‘s unclear to me is what use case the old behaviour is preventing.

Thanks @bacongobbler for the detailed explanation that makes sense. For context we were using the previous behaviour in our tests:
@test "client/ClusterRole: enabled by default" {
  cd `chart_dir`
  local actual=$(helm template \
      -x templates/client-clusterrole.yaml  \
      . | tee /dev/stderr |
      yq 'length > 0' | tee /dev/stderr)
  [ "${actual}" = "true" ]
}

@test "client/ClusterRole: disabled with global.enabled=false" {
  cd `chart_dir`
  local actual=$(helm template \
      -x templates/client-clusterrole.yaml  \
      --set 'global.enabled=false' \
      . | tee /dev/stderr |
      yq 'length > 0' | tee /dev/stderr)
  [ "${actual}" = "false" ]
}

The second test fails with helm 3.
We'll need to change to expecting the "Could not find template" error.
I took a look through the code and the behaviour of stripping out empty templates is pretty deep so I don't think this would be easy to fix, even if that was desired.

closing as stale.

I found this as a workaround to this bug:
Add these lines to the top of the template in question:
---
# This comment is to prevent empty template from erroring on targeted template.

Now helm template --show-only ... will happily print those lines with no error. This is a valid YAML document, though passing it as input to kubectl apply will error, since the document is empty.

I found this as a workaround to this bug:
Add these lines to the top of the template in question:
---
# This comment is to prevent empty template from erroring on targeted template.

Now helm template --show-only ... will happily print those lines with no error. This is a valid YAML document, though passing it as input to kubectl apply will error, since the document is empty.

The default behaviour is extremely user hostile. Adding a yaml comment helps - thank you!


Move Connecting Cloud Storage out of Advanced Usage

Linked issue(s):
None
What change does this PR introduce and why?
Move Connecting Cloud Storage out of Advanced Usage and move Advanced Usage into workflow
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

also https://github.com/kolenaIO/kolena/pull/415 ? If this PR gets green checks first, I'll close mine.

also #415 ? If this PR gets green checks first, I'll close mine.

Realized the structural differences between 2 PRs, will close mine.


Adding obsoletion tag

The number of obsoletion candidates is more than that this time. If you look at the googlesheet:

Tab ROBOT_AddObsSubsetTag to add in release 2024-07 : 346 new obsoletion candidates
Tab ROBOT_AddObsTagForMerge to add in release 2024-07 : 5 new obsoletion candidates
Tab ROBOT_NewTermSplit-ObsTag to add in release 202407 : 1 new obsoletion candidate

WRT the sparql query:
When adding a lot of obsoletion candidate tag, we run into failing on the ROBOT template merging because a comment already exist (for example). To facilitate this, I (with Nico's help) created a sparql query to identify the candidates which will make the ROBOT merging fail.
I thought it would be a good tool to have in the repo for future use.
I removed most of the MONDO:ID present in the original query because it doesn't make sense to keep them (the important and stable part of the query is everything else).
I was in a hurry and did not explain everything related around this sparql query. Sorry. I hope it makes more sense now.

For "Tab ROBOT_AddObsSubsetTag to add in release 2024-07", should there be comments for all rows in the spreadsheet and therefore a comment: Reason of obsoletion... property on the Mondo class?
There are only 339 classes with comment: Reason of obsoletion..., but 352 classes with subset: obsoletion_candidate.

13 candidates already had a comment, so I had to add the obsoletion-related comment manually (otherwise the ROBOT merge would fail because terms have more than one comment).
This is why you see 339 classes with comment: Reason of obsoletion..., but 352 classes with subset: obsoletion_candidate.


[Frontend]  Each question should be its own route

Right now the url for all questions is /questions/923510, we should amend this by adding the question id to the url /questions/923510/1

closed #53


Router.RouteTo includes query params in "path" variable on route

This can lead to route resolution false negatives.
For instance, if you're on a search page and route to the same page with a param (/search?term=test), the route will fail if your test is route.path === '/search';, when this should work.
Workaround is to change to startsWith instead of ===, but this is definitely a bug.  A full page reload doesn't cause the issue, so there's some divergence there.

I think it's line 95 in router.ts.
Also links have href from their element attribute and calling RouteTo directly uses the value from document location.
The combination of these are likely the cause.

Ideally,
Href would include whole url
Path would be pathname only,
Others self evident


scala.xml does not print correctly

@ <a/>
res1: xml.Elem = Elem(
  Elem(
    Elem(
      Elem(
        Elem(
          Elem(
            Elem(
              Elem(
                Elem(
                  Elem(
                    Elem(
                      Elem(
                        Elem(
                          Elem(
                            Elem(
                              Elem(
                                Elem(
                                  Elem(
                                    Elem(
                                      Elem(
                                        Elem(
                                          Elem(
                                            Elem(
                                              Elem(
                                                Elem(
                                                  Elem(
                                                    Elem(
                                                      Elem(
                                                        Elem(
                                                          Elem(
                                                            Elem(
                                                              Elem(
                                                                Elem(
                                                                  Elem(
                                                                    Elem(
                                                                      Elem(
                                                                        Elem(
                                                                          Elem(
                                                                            Elem(
...
@

A workaround like this allows at least seeing the value, although it doesn't respect indentation:
{
  val defaultPprinter = repl.pprinter.live()
  repl.pprinter.bind {
    val default = defaultPprinter()
    default.copy(
      additionalHandlers = default.additionalHandlers orElse {
        case ns: xml.NodeSeq => pprint.Tree.Lazy { ctx =>
          val pretty = new xml.PrettyPrinter(ctx.width, 4).formatNodes(ns)
          if (pretty contains '\n') Iterator("\n", pretty) else Iterator(pretty)
        }
      }
    )
  }
}

On a side note, Ref could use a method which would allow depending on the current 'live' value, like the code above does.

we probably should just forward to .toString in that case.
<0.5.0 had logic to try and detect custom toStrings and use it, >=0.5.0 doesn't (or at least I haven't figured out how to put it in)
Feel free to add your patch to https://github.com/lihaoyi/Ammonite/blob/88cbd8a9f04bf49e83010c82486a596d590008d5/amm/repl/src/main/scala/ammonite/repl/ApiImpls.scala#L92

Fixed in https://github.com/lihaoyi/Ammonite/commit/0bfd399c24cfab28c61381129648b33c994b298f


Need the Ability to give me the best promotions

As a ... User
I need ... see the best deals/ promotions/discounts for my product/products
So that ... I buy products with max discount.
Details and assumptions

If I have certain no. of items in my cart, while checking out, I should get to apply the best promotions on to my cart to get the best discounts.

Acceptance criteria
Given [some initial state]
When [some event]
Then [some measurable outcome]

Closing this issue as it was opened before we had delineated the specific functionality required for our microservice. More specific stories for CRUD / List All / Query / Cancel functionality supersede this more generic story.


build: pass DRIVER_REVISION via buck config

Fixes RUN-3188

Paired with https://github.com/replayio/backend/pull/9415


feat: add an IPFS storage configuration

Implement an IPFS storage configuration.

IPFS's node address configuration was set in config.go, also in service.go.
The ipfs.go which has a Add() and Get method was implemented in storage package.
Add an json_schema.go refers to panacea-coracle function.
Use go-ipfs-api.

Good.
How about adding  test code that run ipfs?

@gyuguen. Ok I'll add it and re-request. For now, I'll draft this branch for run ipfs test.

The ipfs_test.go will be implemented another branch with docker test which is running IPFS.


There is no test, that caching dict is not shared between coroutines

https://github.com/wikibusiness/async_lru/commit/46cf8b832f4826702cd8748f47fbca3d8f9791c9


% with redirections.

Example:
+ year
- Hello. What year were you born?

+ i was born in #
- <@>

+ #
% * what year were you born?{weight=20}
- In <star1>? OK.

I tried that way and it doesn't capture the %.
That way, doing redirections you lose the %. Is there any way to avoid losing the previous one? It's something I did sometimes when I used the AIML version

Maybe you can re-arrange your triggers to avoid the need for this:
+ year
- Hello. What year were you born?

+ i was born in #
- In <star1>? OK.

+ #
% * what year were you born{weight=20}
@ i was born in <star>

You> year
Bot> Hello. What year were you born?
You> 1900
Bot> In 1900? OK.
You> I was born in 1800
Bot> In 1800? OK.
You> year
Bot> Hello. What year were you born?
You> I was born in 1700
Bot> In 1700? OK.

I typed a longer reply to this below:

It looks like this was intended behavior.
In rivescript.brain._getreply():
        # See if there were any %Previous's in this topic, or any topic related
        # to it. This should only be done the first time -- not during a
        # recursive redirection. This is because in a redirection, "lastreply"
        # is still gonna be the same as it was the first time, causing an
        # infinite loop!
        if step == 0:

I tried disabling that check and see if the rivescript-python unit tests failed, but none of the tests actually exercise the bug described in that comment. I traced the git history back and this check was there since the beginning of rivescript-python; so I went and checked the original Perl version's history, and it goes back as far as when I switched from Subversion to Git so don't have much more context on why I programmed it this way.
However, here is an example RiveScript use case that would trigger the bug being protected from:
+ google something for me
- What do you want to search for?

+ *
% what do you want to search for
- Okay, getting that for you: {@google <star>}

+ google *
- Results for "<star>" go here...

You> google Python
Bot> Results for "python" go here...
You> google something for me
Bot> What do you want to search for?
You> javascript
Bot> [ERR: Deep recursion detected]

If a trigger with a %Previous redirected to another trigger that happened to match the same conditions as the original %Previous, an infinite loop occurs; RiveScript is programmed to break from infinite loops after 50 levels of recursion (by the ! global depth = 50 parameter).
With the %Previous check in place as it currently is, this chain of triggers works as expected:
You> google perl
Bot> Results for "perl" go here...
You> google something for me
Bot> What do you want to search for?
You> python
Bot> Okay, getting that for you: Results for "python" go here...

But admittedly for your use case, this behavior isn't adequate.

I'd be willing to accept a pull request if you find a solution to this problem. Simply removing the step == 0 check won't do, as it would cause new bugs in existing bots where the behavior changes. Ideally there should be some way to do a redirect, keeping the "lastreply" and checking the %Previous's, but not re-triggering the same trigger as it did the first time around.

Yes, from the example you've made it worked for me even though that means making a few rules, that's not something I had in mind. But as long as it works, that's fine.
At the moment I can't find another solution to that, except for your example.  Greetings.


Link to FragmentViewBindingDelegate.kt broken

👋
On this page, I clicked on the link for FragmentViewBindingDelegate.kt and received a 404. Was hoping to add it to a side project
Thanks

Updated the docs to https://github.com/airbnb/mavericks/blob/main/view-binding-utils/src/main/java/com/airbnb/mvrx/viewbinding/FragmentViewBindingDelegate.kt


Fix relative build.target-dir configuration

Closes #2700

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+ dcd8b412171851d96f80f55f08b937d10c51b35f

:hourglass: Testing commit dcd8b41 with merge 48fd9de...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


Cannot Get Location

Hello
I installed the clone program last night. So far everything is running great with the exception of finding my location. The weather forecast comes up but I get a "cannot find location" message below.
I have done nothing with the lat/long line of code. My IP address and weather API are inputted
I'm a rookie at coding so I'm sure it's something simple I'm missing.
Thanks!

I am having the same problem. I think the issue is freegeoip.net. I do not know python very well but if you read over the weather class, location_req_url is defined by http://freegeoip.net/json/%s. As far as I know(again, I don't know python), % is acting as a 'getter' for the weather class which is calling the string 'ip' that was defined with your ip address.
So... if I visit the url, sure enough, it returned my coordinates but not the city and state.  For the time being, I just prefer it not be displayed on the mirror.


fix(iflow-koa): Fix koa.use() function signature

Ensure context & next parameters are not optional, and therefore don't have to be checked in the implementation.

Wow, that was fast ! :) Thanks a lot !

Keep PRs coming.
Happy to merge them if they are useful.


Question about license

What is the OSS license for this project?

Materia Version: v0.1.1

Materia Server is under the license Apache2. I'm currently writing the README which will mention it.

done: https://github.com/webshell/materia-server/tree/master#licensing


ypeError: undefined is not an object (evaluating 'this.toastInstance._root')]

Hi, i get ypeError: undefined is not an object (evaluating 'this.toastInstance._root')]
on any toast show:
my method:
_show_toast(type) {
Toast.show({
text: 'Wrong password!',
buttonText: 'Okay',
type: type
})
}

I checked this in NativeBase KitchenSink with latest NativeBase, Toast types worked fine
Share more details, and please follow the issue template guidelines

你的主应用程序组件是否包含在内<Root>？前几天我遇到了这个问题，并为我解决了这个问题。请参阅文档：https：//docs.nativebase.io/Components.html#Toast

you can reference this

thank you very much @StevenT1

Hi, for the record I am seeing this error with latest NativeBase with <Root> as my topmost element. It seems to only occur for android after users have changed to a different Activity from the main app and back (in our case, when we have done an await PermissionsAndroid.request());

Wrapping the component in <Root> will do the trick. <Root> is imported from native-base

It Does. Thanks @saran-shabd

Wrapping the component in  will solve this issue.  is imported from native-base

@jenni-divvito exact same case here. Did you find a work around?

I'm having same issue here. somebody help!

I find this solution in Stackoverflow:
import Root from "native-base";
import Root from native base and wrap the whole container/view inside render's return function
render() {
return (your app-return function code);}


Workshop installation for Windows

I started from scratch and followed the instructions found at https://coderefinery.org/workshops/2019-11-19-stockholm/
Honestly I found it sometimes a bit unclear; difficult to follow (as we always have three cases and windows is always last...) and not always easy to test (commands are usually not named properly (no .exe, etc).
Setup for installation Git-bash has slightly changed (there is now a question about ssh* and the default  is incorrect). It is well explained in the Carpentries setup.
I would recommend:

Ask our attendees to follow Carpentries workshop setup to install git-bash, Python and R: the main advantage is that is it more likely to be up to date (many more workshops and users following and correcting setup instructions).

https://github.com/carpentries/workshop-template (http://carpentries.github.io/workshop-template/).
Then CodeRefinery specific setup:

Install additional python packages (https://coderefinery.github.io/installation/python/). I would also add anaconda-navigator (need to show how to add channels) for installing and managing packages (as it is quite nice for managing mulitple conda environment)
When installing Anaconda with python 3.7, I only had to install:
* sphinx_rtd_theme
* snakemake-minimal
* nbdime
* jupyterlab-git (I don't think jupyterlab github is necessary anymore)

I think jupyterlab-manager is deprecated is not required; management of extension in jupyterlab is available by default in the latest version of jupyterlab.
All jupyterlab extensions are enabled automatically (and can also be controlled in jupyterlab through the extension manager). When something needs to be build etc., a window pops up and ask you to confirm to build the extension. Then everything works fine.
I recommend to switch from Powershell to Git-bash in JupyterLab. This can be done by updating jupyter configuration (C/Users/annefou/.jupyter/jupyter_notebook_config.py):
c.NotebookApp.terminado_settings = {
'shell_command': ['C:\Program Files\Git\bin\bash.exe']
}
This allows to have a good integration of git/github in Jupyterlab (this may be better than having multiple installation of git via conda).
Finally, I recommend to start jupyterlab from anaconda-navigator.

For Visual diff tools, I recommend the usage of p4merge (instead of meld).

git config --global merge.tool p4merge
git config --global mergetool.p4merge.path "C:/Program Files/Perforce/p4merge.exe"
https://www.perforce.com/downloads/visual-merge-tool

Thanks Anne for these observations and recommendations. I have nothing against putting Windows first in the instructions. Now we need to turn these into pull requests.

Yes. That's my next task!

@annefou and @bast : if I understand correctly, the suggestion above is not reflected in the current installation guide, but do you (@annefou ) work on this on a branch or your fork? If not, I will try to implement the suggestion above in the revision work for the upcoming workshop (May 2021).

It is also my understanding that this is not yet reflected in our install guide and this can be an additional route (but then we have 3?).

Yes that was my plan... But I haven't started. When do we need to send setup instructions to attendees?

We will send acceptance notification after the soft deadline of 26th April, so it will be the best if the works are done within this week. Myself and Radovan are responsible for this work (I think I have the main responsibility, though) and I try using as much time as possible on this this week. So probably I will be back to you to ask if there are anything unclear. Is this OK for you?

Is this issue still relevant after the page reorganization?

I believe that it is @annefou that should give a final decision, but in my understanding it is not relevant anymore and this issue can be closed.

yes... Thanks for your great job!


Removing ValueType in QuadTree

Greetings,
I fail to see how I can manage dynamic objects (which change their position, rotation etc).
My current workaround is that on each frame I clear the Quad Tree and reinsert all entities in the scene (which is the obvious bottleneck).
Preferably I would do the reinsertion only for those ValueType objects, that have actually moved etc., like a dirty flag type of thing, so I my question is: is it possible to add a remove/update function for a ValueType object? Or are there better approaches I do not see?
Best regards

Considered - yes, but this seems more like a workaround to be honest.

That's a good point with having a dynamic version, in some cases it might be faster, but you would have to profile.
In this case it would also help to have a remove function, I'll add one.

Added a remove function for the quad tree.


REL: bump version to 2024.7.19a7 for minor release

[NEW] made changelog
Summary by Sourcery
Bump version to 2024.7.19a7 for a minor release, introducing new features, bug fixes, and performance enhancements. Added support for parsing newick format with node statistics, improved coevolution statistics performance, and fixed sequence dotplot handling. Updated documentation with a new tutorial and deprecated outdated coevolution functions.
New Features:

Added support for parsing from newick format with node support statistics in cogent3.

Bug Fixes:

Fixed handling of dotplots for sequences with common gaps, ensuring correct display of aligned paths.

Enhancements:

Introduced convenience class methods for DictArray and DistanceMatrix to simplify creation with numpy arrays and dimension labels.
Rewrote mutual information based coevolution statistics for significant performance improvement using numba.jit and parallel execution capabilities.

Documentation:

Added a new tutorial on using a nonstationary amino-acid model, showcasing cogent3's capabilities.

Chores:

Deprecated old coevolution functions and statistics, marking them for removal by the end of 2024.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.931%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
32368

Relevant Lines:
35209

💛  - Coveralls


Crash when passing --profile

$ ibazel test --profile=myprofile.out  test:golden_test
State: QUERY
Querying for BUILD files...
Error running Bazel exit status 2
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0x40ec1e3]

goroutine 1 [running]:
main.(*IBazel).queryForSourceFiles(0xc42009a000, 0xc420018240, 0x3f, 0x0, 0xc4200cc000, 0xc4200ce000)
	ibazel/ibazel.go:316 +0x1b3
main.(*IBazel).watchFiles(0xc42009a000, 0xc420018240, 0x3f, 0xc420062060)
	ibazel/ibazel.go:345 +0x5a
main.(*IBazel).iteration(0xc42009a000, 0x413b590, 0x4, 0xc420055e18, 0xc42000c060, 0x2, 0x2, 0xc42001c0f0, 0x28)
	ibazel/ibazel.go:204 +0x91e
main.(*IBazel).loop(0xc42009a000, 0x413b590, 0x4, 0xc420055e18, 0xc42000c060, 0x2, 0x2, 0xc420055e20, 0xc420010170)
	ibazel/ibazel.go:176 +0xf6
main.(*IBazel).Test(0xc42009a000, 0xc42000c060, 0x2, 0x2, 0x2, 0x2)
	ibazel/ibazel.go:168 +0x80
main.handle(0xc42009a000, 0x7fff5fbff7ff, 0x4, 0xc420010160, 0x2, 0x2)
	ibazel/main.go:110 +0x22e
main.main()
	ibazel/main.go:99 +0x1bf

@mprobst, is your ibazel up-to-date? Running your command on the tsickle repo, I was able to reproduce this crash from cd96809, but not from HEAD. It appears that @achew22 fixed this in #48 (his diff on line 313 of ibazel.go, which prevents it from attempting to loop over non-existent build targets when the query fails).

Might have been out of date, I'm not sure if I upgraded before or after.


iphone5 10.3.4 crash _flutter.listViews

Steps to Reproduce

Run flutter run -d iPhone -v.
Update the files as follows: ...
flutter_02.log
...
Expected results:
Oops; flutter has exited unexpectedly: "_flutter.listViews: (-32000) Bad state: The client closed with pending request
"_flutter.listViews".".
Actual results:
iphone5 10.3.4

flutter doctor -v
[✓] Flutter (Channel dev, 1.26.0-1.0.pre, on macOS 11.1 20C69 darwin-x64, locale zh-Hans-CN)
    • Flutter version 1.26.0-1.0.pre at /Users/jugg/flutter/flutter
    • Framework revision 63062a6443 (3 周前), 2020-12-13 23:19:13 +0800
    • Engine revision 4797b06652
    • Dart version 2.12.0 (build 2.12.0-141.0.dev)
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/jugg/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 48.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.0

[✓] Connected device (3 available)
    • iPhone (mobile) • 7a16ced0a8ff53346eb6f4766942aa6083fbcea7 • ios            • iOS 10.3.4
    • macOS (desktop) • macos                                    • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)    • chrome                                   • web-javascript • Google Chrome 86.0.4240.183
    ! Error: iPhone is busy: Copying cache files from device. Xcode will continue when iPhone is finished. (code -10)

• No issues found!

Hi @625859257
I have seen a similar crash if the cable is disconnected, please make sure you've your phone properly connected and it's a stable connection.
Can you please provide complete  flutter run -d iPhone -v (logs missing from your flutter_02.log file)  and a complete reproducible minimal code sample
Thank you

Hi @625859257
I have seen a similar crash if the cable is disconnected, please make sure you've your phone properly connected and it's a stable connection.
Can you please provide complete  flutter run -d iPhone -v (logs missing from your flutter_02.log file)  and a complete reproducible minimal code sample
Thank you


Evaluate app and server dependencies for upgrade candidates

This site has been online for going on six years. We upgraded major dependencies a couple of years ago, but it would be prudent to review again and bring anything out of support / with security warnings up to a contemporary version.
To start, let's capture the current version of our major dependencies (Python, Django, Postgres, Solr) and whether that version has reached end of life. Also note whether anything has active Dependabot alerts.
@antidipyramid Can you add this info in a comment here? Here's an example of this exercise from another DataMade project: https://github.com/security-force-monitor/sfm-cms/issues/707#issuecomment-916122866

Here are the major dependencies we're running:
Python 3.6.9: No longer supported. The current version is 3.10. Support for 3.7 will also end soon so we probably want to upgrade to at least 3.8.
Django 2.2.25: No longer supported. The current version is 4.0.4. Django maintains "long term support" for the last feature release in a major update (i.e. x.2). Django 4.2 won't come out until sometime next year so I think it makes sense to shoot for Django 3.2. For any given version, Django recommends that we use the latest supported version of Python (for Django 3.2 that's Python 3.10).
Postgres 9.6.12: No longer supported. The current version is 14.3.
Solr 7.5: No longer supported. The current version is 9.0.0. I think it makes sense to upgrade to Solr 8.8.
We'll want to upgrade Python first as Python 3.6 is not supported by any Django version we'd want to upgrade to.

Support schedules for:

Django: https://endoflife.date/django
Python: https://endoflife.date/python
Postgres: https://endoflife.date/postgresql

Clean up Ara Najarian's stray membership.

@hancush to ensure Metro admins have admin access to upgrade site for event removal, etc.

Done! Now, to deploy in #908.


[Feature Request] Add ability to treat warnings as errors

For model development in federated environments, logs may not be shown to the end users. As a result, some mxnet warning messages are never displayed. People without enough familiarity with mxnet may never identify the actual root cause of issues. This potentially affects many people's productivity.
An example is Storage type fallback detected. In the following example, a sparse operator failure does not harm the model when the problem is small-scale, but it eventually causes memory depletion in larger datasets. To make it worse, the warning message is only displayed once.
We should provide users an option to fail on warning

@mxnet-label-bot add [Feature]


cannot more than 15 addresses

when i try to add more than 15 addressed i get error
[2022-12-20.23:50:36] [Info] 36 addresses loaded (0.0MB)
[2022-12-20.23:50:36] [Info] Allocating bloom filter (0.0MB)
[2022-12-20.23:50:36] [Info] Error: invalid argument

Try to build with this fix, the 16 keys error should be fixed. Tried on CUDA v12.6, but should fix other versions too.
https://github.com/brichard19/BitCrack/pull/419


Improving SQLA query generation

Coverage increased (+0.1%) to 81.077% when pulling ebf1462fd4350fbb6d8b560f846013557734f2db on mistercrunch:literals into d7ea47387f13f13e6586cc8d897bcf25dfb45827 on airbnb:master.


https://nts-web.oscarrc.me produces connection timeout on Cloudflare / HTTP error 522

Alternatively, would it be possible to add build instruction of some kind so that we're not dependent on an externally hosted webapp?
For example, I just dragged and dropped nts-web/index.html into Chromium but all I got seemed to be a blank page.
Thanks in advance.

I will look into that.
You can in fact, build it locally, you just need to clone the repo and in it's folder do
npm install
npm run build // To create a production build or
npm start // To run it locally

I hope this is helpful.


transpiled circuit for Lucy

I recently had the opportunity to submit my first circuit on Oxford-Lucy via Braket, following a prior experiment on IonQ. I'm keen to analyze how the two compare. Could you guide me on how to access the transpiled circuit for Lucy from Braket? Given my original 7-qubit circuit's extensive connectivity and 30 entangling gates, I anticipate the Lucy version might be significantly longer due to the necessary swap operations. How can I retrieve this detailed information from Braket?
Furthermore, I'm looking for the latest specifications of Lucy, including T1, T2, the typical duration of entangling gates, readout fidelity per qubit, and any limitations on concurrent gate execution. Where might I find these details?
Thank you for your assistance.

Thank you Peter, it is very helpful


New widget

👋🏻 There is a widget I would like to use in my app which is currently not present in fluent_ui. The component is https://docs.microsoft.com/en-us/fluent-ui/web-components/components/data-grid. Could you possibly add it? I would really appreciate it!

[ ] DataGrid

This actually isn't something I'd add to the project since it's not related to windows UI.
See other packages like https://pub.dev/packages/syncfusion_flutter_datagrid to get this done


clean goal

There is no option to remove applications added with install-apps. A clean goal could be created to remove alle deployed apps

A typical use case is when a snapshot is updated (mvn -Pliberty in my setup).
My pom profile liberty has
install
I can see that the dropins directory is refreshed with latest snapshot, but the server sees that as a :
[AUDIT   ] CWWKG0018I: The server configuration was not updated. No functional changes were detected.
a mvn clean install -Pliberty could clean all applications before a new install, that would also reflect a change (removal) of dependencies in the pom

Added a clean-goal and published a new 1.2-SNAPSHOT with the latest changes.


Copy of "Fix conformance test flags are exported to other libraries that use the conformance test suite"

Fixes #
Does this PR introduce a user-facing change?:

NONE

I'm not familiar with whether this project aims to guarantee some amount of compatibility for upgrade, but if that's a goal, can this change also include aliases from conformance/utils/suite/features.go to the new location for the content in pkg/features/features.go

I'm not familiar with whether this project aims to guarantee some amount of compatibility for upgrade, but if that's a goal, can this change also include aliases from conformance/utils/suite/features.go to the new location for the content in pkg/features/features.go

Conformance 1.1 already has a ton of breaking changes. We would have to alias every feature, and probably would start forgetting some in the future. IMO this is an acceptable breakage, especially given all implementations will need to adopt to 1.1 anyways.


need to disable button when user has clicked

user is able to keep pressing close position which activates the browser wallet with a new transaction to confirm

I believe this dialog will be changed with new one? It renders issue as invalid

btw, do not add assignees to issues unless it's certain that it should assigned otherwise collaborators will not take these issues as it means that someone already working on it


Branch Android not hitting InitSessionsComplete

I am having an issue where I call the Branch.Init() in the Branch Package version 3.0.13.  I am doing a simple call to the Branch.Init() with my live branchkey and am not receiving a response from branch in the InitSessionsComplete().  I am able to go to a previous commit and access the information that I need.
Here is my Init() code:
void BranchInit()
{
var branchkey = myBranchKey;
if (String.IsNullOrWhiteSpace(branchkey))
{
    TraceLog.Error(TAG, "APP BUILT WITHOUT BRANCH API KEY");
    #if DEBUG
        throw new InvalidOperationException("No Branch Key set, check BuildInfo.xml");
    #else
        return;
    #endif
    }

    //TraceLog was here, confirmed that it got here
    BranchAndroid.Init(this, branchkey, this);

}
Any suggestions on how to resolve this?

Hi @rikkieb34 , sorry for delay,
Look at our example please:
https://github.com/BranchMetrics/xamarin-branch-deep-linking/blob/master/Branch-Xamarin-Testbed.Droid/TestBedApplication.cs
https://github.com/BranchMetrics/xamarin-branch-deep-linking/blob/master/Branch-Xamarin-Testbed.Droid/MainActivity.cs
You should add several info into android manifest from app class and in activity class.

@rikkieb34 @antonargunov
I just upgraded to 3.0.14 from 2.1.4 (It's been a while) and I'm experiencing the same issue on Android. iOS works for me.
Oddly the build deployed from VS works, but the build from automated build (with the same build configuration) does not.

@AntRemo @antonargunov
Sorry its been a while since I posted anything on this, but I am having the same issue as AntRemo.  My Visual Studio debug build works great, however, when I try to use my automated building system it fails.  I added some traces to see if I can spot where it fails, but it only hits my trace to initialize branch.  My traces in InitSessionsComplete() and SessionRequestError() are not getting hit.  Have you had any success with the latest version (3.0.15).

@rikkieb34 try new version please

@antonargunov I am using the latest version.  I have tried setting the launchmode to be single task, I have tried using the intentfilters per the examples given without any success.  Do I need to use the getautoinstance()?  What is the function used for?  I, currently, have the correct branch key and I am able to get the build to work properly in debug builds, however release builds are not working appropriately.  Are there any other suggestions that you could think of that would help?

@rikkieb34 , yes you need to use getautoinstance(), this function is for correct initialization of Branch at the start of the app. Also, as you can see, you need to add into metadata your branch key.

@antonargunov There are multiple branchkeys that are used between the apps that I am building.  Is there a way to use the getautoinstance() in a way that would allow me to use my dynamic branchkeys that are created for my apps.

@antonargunov I ended up getting it to work thank you for your help.

@rikkieb34 , in our example we use
[MetaData("io.branch.sdk.BranchKey", Value = "@string/branch_key")]
you can add into strings.xml several keys (branch_key01, branch_key02, etc.) and switch they as you wish, but you need to use the same key in the method
BranchAndroid.Init (this, GetString(Resource.String.branch_key), appBUO);

@antonargunov I achieved what I needed when calling getautoinstance(this.ApplicationContext) without any MetaData.  Can this cause any issues?  Currently, it seems to be working correctly.

@rikkieb34 check received parameters for clicked link

I hope all is nice


Add a way to save many configurations.

Add a way to save all inputs to a profile that can later be chosen and loaded.

There should be a button to save the existing inputs. User can supply an alphanumeric name to be saved.
There should be a button to load a saved profile. User will select the profile from a dropdown of their saved profiles.

With the incoming changes for storage, this should be simple to KO. That said, I'll work on this next.

https://github.com/Ryan-Florida/Lethal-Company-Guy-in-Chair/pull/2
Just gotta fix up tests but this work is now done.


fix page lists, Hugo v0.57

Hugo v0.57 introduced a breaking change in mainSections.
Using .Site.RegularPages for the home page to resolve the issue.
mainSections is meant for the homepage.

@onedrawingperday thanks for the review!
I've updated the min version to the current latest release.
The theme should work now.


Added DMSwipeCards

Added DMSwipeCards, a tinder style card stack component
Project URL
https://github.com/D-32/DMSwipeCards
Description
Why it should be included to awesome-ios (optional)
It has +100 stars so I think it deserves a place on this list :)
Checklist

[X] Only one project/change is in this pull request
[X] Addition in chronological order (bottom of category)
[X] Supports iOS 9 or later
[X] Supports Swift 3
[X] Has a commit from less than 2 years ago
[X] Has a clear README in English

Thank you for contributing to awesome-ios 😄


[Gateway 2.5] : second test release issue reports

Consolidating issues remaining:

[x] implementation of emergency swap https://github.com/bobanetwork/boba/pull/1097
[x] Discrepancy in balances. Gateway shows no balance https://github.com/bobanetwork/boba/pull/1097
[x] There is no BNB liquidity in the BNB fast bridge so we can not test it
[ ] Modals pop up an inappropriate times and tell you to do things that are against user intent.
[ ] On earn page can't connect to Eth page. Only allows you to connect to BNB
[ ] When switching from Eth to BNB the prompt still advises to switch to Ethereum
[ ] If a user switches to a network - then the gateway should switch the UI for them
[ ] There needs to be a chain select button in the top right corner as per design below.

[ ] wallet connect v2 https://github.com/bobanetwork/boba/issues/688
[ ] Wallet Dialog freezes https://github.com/bobanetwork/boba/issues/1063

awesome!

Change network name from "Boba" to "Boba ETH'

Change "Add token to metamask" to "Add token to wallet"

Adding ETH to the wallet here adds the incorrect contract address (the others are fine).
Please double check all contract addresses on all networks.

Time is wrong. Bridging from L1 to L2 does not take 7 days.

Change "Show Testnet" to "Show Testnets"

Most of this issues has been fixed and push to staging. create another issues to track freshly.
https://github.com/bobanetwork/boba/issues/1140


Something in the hosts list prevents downloading of Chrome Dictionary for Spellcheck.

Hi there, absolutely loving this project and wouldn't do without it.
One minor issue I'm having is identifying which of the entries in this list are blocking communication with the server required to download chrome dictionaries or even enable basic spell check functionality.
Obviously there are numerous google sources that need to be blocked but even if I could just remove the offender temporarily so that I can get spellcheck back that would be great.

Issue can be solved by temporarily renaming C:\Windows\System32\drivers\etc\hosts to C:\Windows\System32\drivers\etc\hosts.backup
Reboot, enable basic spellcheck and download dictionary. Rename your hosts.backup file back to plain old hosts and reboot again.
Basic spellcheck functionality is retained but I do not believe you will be able to use the one that sends your input to google for checking...nor would I want to use it.


Add spec values to ingress

This enables us to inject spec key values to the db console ingress resource.

@honnel it would be great to get your 5 cents here. 🪙

LGTM

Can't say that any of these failure should have to do with this PR - Helm sucks.


ParseNumberPipe({ nullable: true })) not work

ParseNumberPipe({ nullable: true })) not work
I get an error
...
@Get("/")
async readTasks(
@Req() req: NextApiRequest,
@Res() res: NextApiResponse,
@Query("id") id?: ReadTasksQueryRqDTO["id"],
@Query("length", ParseNumberPipe({ nullable: true }))
...

nullable works for values explicitly set to undefined or null which referes to being nonexistent, but in the case of the screenshots you've provided, id=null gets evaluated to a string, as in typeof id is string.
You can log the id and see its type if you want.


Added light and dark theme toggling button

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes # <your_issue_number> #34 #107

Mark the task you have completed ✅

[ ✅] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✅ ] I have performed a self-review of my own code or work.
[✅ ] I have commented my code, particularly in hard-to-understand areas.
[ ✅] My changes generates no new warnings.
[✅ ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ✅] I have added screenshot for review.

## Add your screenshots(Optional) 📸
![Screenshot 2024-06-15 162221](https://github.com/ChromeGaming/Squard-line/assets/107982190/9ae5e1ac-7064-421c-b40e-1e9a3c96cd9c)
![Screenshot 2024-06-15 162238](https://github.com/ChromeGaming/Squard-line/assets/107982190/143ebb6c-a5db-4db1-b141-e3bb687bb308)

Thank you soo much for contributing to our repository 💗

https://github.com/ChromeGaming/Squard-line/assets/107982190/7947163e-75ce-4054-af93-7f42641609c4
@Durgesh4993 I have completed adding toggle.please check it once


Compatibility issues with net-snmpv5.8

I'm working on a project using netsnmp-sys and I noticed that several issues popped up when we updated netsnmp to v5.8. I managed to solve this issue by generating new bindings based on the new libs and updating the current ones. I could move on with a PR but this solution is not backwards compatible and I am not sure if this is something desired. Any thoughts?

@thsioutas Do you have the updates you made in a public repo? I think I am running into the same problem.

@thsioutas Do you have the updates you made in a public repo? I think I am running into the same problem.

No @missouribp, I don't have them in any public repo.