tokenization breaks sometimes with 2.2.2

caused by this commit: https://github.com/NeekSandhu/onigasm/commit/65fd5b5b1a2475cc9c57e1c9a2e8b45f09d424f4
Originally reported in Theia for PHP grammar: https://github.com/theia-ide/theia/issues/4966

I think I can shed some light on this issue. I have been seeing this problem even with previous versions of onigasm, but always limited to the php grammar (@akosyakov does it fail for you with other grammars too?).
In my case, whenever a built-in PHP function like empty was tokenized, the tokenizer would fail. Diving deeper into the issue, the createOnigScanner function that is passed when initializing the registry gets called with an array [undefined] when it encounters a php built-in function. Passing that array to the OnigScanner would make it fail. Below you can see the code I've used to work around the issue (surprisingly, it doesn't break highlighting at all):
createOnigScanner: (sources) => {

	// FIXME:
	// with php "$var = empty()" sources would be [undefined] which will crash,
	// therefore we sanitize the sources array here. Pretty sure there must be a
	// root cause for this problem, still to find yet
	if (sources[0] === undefined) sources.length = 0;
	return new oniglib.OnigScanner(sources);

},

No, i have not seen it with other grammars. Do you have an example of php files which breaks with 2.2.1? Just want to try it.

I cannot reproduce this issue I described above with 2.2.1, maybe it was fixed in the meantime. However, I'm getting the same error as you when using 2.2.2 on php files (even empty ones). Not sure what's going on.

I read the thread over at Theia issues, and while I do understand the problem, I'd highly appreciate if you guys can give me a filtered and definitive summary of all the observations you guys have made till now.
Because I'm bit confused. @akosyakov mentions how 2.2.1 works fine while @mofux says how problems have been around even in the previous versions?

@NeekSandhu i believe it only affects 2.2.2 and in 2.2.1 it used to work fine

Can you please share the grammar file along with the text that's being tokenized (the one that causes the error). Thanks!

grammar is here: https://github.com/theia-ide/theia/blob/master/packages/textmate-grammars/data/php.tmLanguage.json
code snippet:
<?php
  // Declare the variable 'string' and assign it a value.
  // The
 is the HTML equivalent to a new line.
  $string = 'Hello World!
';

  // You can echo the variable, similar to the way you would echo a string.
  echo $string;

  // You could also use print.
  print $string;

  // Or, if you are familiar with C, printf can be used too.
  printf('%s', $string);
?>

Please try<EMAIL_ADDRESS>with the fix for the issue.


preguide: alpha blockers

Alpha release blockers in preguide:

[x] cmd/preguide: support more efficient development of guides - https://github.com/play-with-go/preguide/issues/78
[x] cmd/preguide: enforce that all target paths are absolute - https://github.com/play-with-go/preguide/issues/69

Both points resolved.


Extra waitForRoadrunner in fetchAndScore

There are actually three wait for roadrunner blocks in fetchAndScore.  The flag we added to skip some of them isn't quite right, and it's still checking more than we want it to.  Here's what fetchAndScore has now:
Check 1 is before the lift raises so the drive motion can complete correctly, and is currently unconditional
Check 2 is before the cone is dropped, and is conditional on the correctPositionAlways option flag.
Check 3 is before the cone is grabbed, and is conditional on the correctPositionAlways option flag.
None of them read the waitForRoadrunner argument, which is meant to indicate the first time the function is called.
Check 1 needs to be conditional upon waitForRoadrunner, so that it only checks the first time when we are finishing driving.
Check 2 is good, and should remain conditional based on the option mode.
Check 3 is redundant because it happens so soon after Check 2, and should be removed.

we need the third wait though so that if there isnt enough time, then it will abort and go to park


Require cycle警告

升级到1.5.0版本后，在ios模拟器上运行时flipper出现这个警告

我是这样使用MMKV的

使用
import * as AppMMKV from "../common/AppMMKV"
const userInfo = AppMMKV.getUserInfo();
return userInfo ? JSON.parse(userInfo).id : null
请问如何消除警告

For anyone who interested in this issue but not able to understand the chinese, i will help translate it.
The first paragraph is saying after he upgrade the package to 1.5.0, there's a flipper warning on ios simulator.
Second paragraph:
Example on how he use MMKV package
Third paragraph:
Example on usage and how can he get rid if the warning

It also happens if you export storage from react component and uses it on some independent file ( in my case in axios config file )
Creating independent file which is exporting the storage fixed the issue with this warning.
"react-native-mmkv": "^2.4.3", // expo dev client


core: test: remove intermediate suite object

The Test object is constructed with a Suite instance, which is
accessible from inside tests using 'this.suite'. However, another object
is created using only certain properties from the input suite object.
Remove the intermediate object so that all properties from the suite
object are exposed in the test code.
Change-type: patch
Signed-off-by: Joseph Kogut<EMAIL_ADDRESS>

@balena-ci rebase


How to change itunes with spotify

I can't use Spotify media control on pock, any advice to change the itunes with spotify?
http://prnt.sc/o5kx6e

Pock uses private Apple’s MediaRemote framework for retrieving media information, so it already works with any media apps.
It also works with Safari if any video/audio is currently playing.
Just drag the ”Now playing” widget into Touch Bar and start playing anything you want! 👌

It wouldn't be possible to set what app will start playing music when tapped if nothing is playing? This defaults to iTunes now despite me never using iTunes. I think that may be what OP wants (it's what I want at least).
Thank you for Pock by the way, it's great!


si-90 -- Support for default data sets

This PR includes:

updates to default data loading behavior to support dynamic reading of static csv file alongside the bundle.html.
conversion of the sample data set in json format into a CSV format

This resolves #90

This PR also includes some package-lock.json updates to fix open vulnerabilities


Error: Upgrade expected but low level API in use

I tried to implement current thread TLS websocket server based on example code:
...
let tls_cx = tokio_tls::TlsAcceptor::from(tls_cx);

let new_service = || {
    service_fn_ok(|req: Request<Body>| {
        let mut res = Response::new(Body::empty());

        // Send a 400 to any request that doesn't have
        // an `Upgrade` header.
        if !req.headers().contains_key(UPGRADE) {
            *res.status_mut() = StatusCode::BAD_REQUEST;
            return res;
        }

        let on_upgrade = req
            .into_body()
            .on_upgrade()
            .map_err(|err| eprintln!("upgrade error: {}", err))
            .and_then(|upgraded| {
                // We have an upgraded connection that we can read and
                // write on directly.
                //
                // Since we completely control this example, we know exactly
                // how many bytes the client will write, so just read exact...
                tokio::io::read_exact(upgraded, vec![0; 7])
                    .and_then(|(upgraded, vec)| {
                        println!("server[foobar] recv: {:?}", &vec);

                        // And now write back the server 'foobar' protocol's
                        // response...
                        tokio::io::write_all(upgraded, b"bar=foo")
                    })
                    .map(|_| println!("server[foobar] sent"))
                    .map_err(|e| eprintln!("server foobar io error: {}", e))
            });

        current_thread::spawn(on_upgrade);

        // Now return a 101 Response saying we agree to the upgrade to some
        // made-up 'foobar' protocol.
        *res.status_mut() = StatusCode::SWITCHING_PROTOCOLS;
        res.headers_mut()
            .insert(UPGRADE, HeaderValue::from_static("foobar"));

        return res;
    })
};

let addr = "<IP_ADDRESS>:12345".parse().unwrap();
let srv = TcpListener::bind(&addr).expect("Error binding local port");

// Use lower lever hyper API to be able to intercept client connection
let http_proto = Http::new();
let http_server = http_proto
    .serve_incoming(
        srv.incoming().and_then(move |socket| {
            tls_cx
                .accept(socket)
                .map_err(|e| io::Error::new(io::ErrorKind::Other, e))
        }),
        new_service,
    )
    .then(|res| match res {
        Ok(conn) => Ok::<_, io::Error>(Some(conn)),
        Err(e) => {
            eprintln!("[1] Error: {}", e);
            Ok(None)
        }
    })
    .for_each(|conn_opt| {
        if let Some(conn) = conn_opt {
            current_thread::spawn(
                conn.and_then(|c| c.map_err(|e| panic!("[2] Hyper error {}", e)))
                    .map_err(|e| eprintln!("[3] Connection error {}", e)),
            );
        }

        Ok(())
    });

println!("Listening on {}", addr);

current_thread::block_on_all(http_server).expect("http server finished with error");

And I got error when on_upgrade future executed: upgrade expected but low level API in use.
How to use high level api to create TLS server?

I'm running into this issue too, but am not able to use the builder pattern like above. Our tcp connection needs to be handled by something else, and sometimes we hand it off to hyper to process web stuff. Is there any more guidance about how to fix this in that case? What about connection is it that would prevent hyper from upgrading it?

@mjibson Perhaps Connection::with_upgrades might work in that case?

If using Http:
let conn = http.serve_connection(socket, svc).with_upgrades();


Deepface verify on LFW- not working on other models and detectors than 'ArcFace' with 'opencv'

Hey,
I would like to compare the metrics for all of the models, detectors and distance metric available on DeepFace framework on LFW dataset. The problem is, that my code is working only on ArcFace and opencv with the default distance metric. The source code is based on this video: https://www.youtube.com/watch?v=9fT-9TFjeDQ.
Attached sample of errors for mtcnn and ssd detectors on Arcface. Also attacked colab link - you can edit it.

I will appreciate your help :)
link: https://colab.research.google.com/drive/1u4AA0HOSG1gezsc2LyG4dYWhCokxW4Dr?usp=sharing

what was your tf version?

It seems that you are using tf 2.5 in the colab. Could you downgrade to 2.2.0? That might be a tf version based trouble.

what was your tf version?
So I changed the version of tf to 2.20, and there is still the same problem. If you want, you can comment and also edit the code in colab.

Oke, it seems a probably bug. I'm going to study it tonight or tomorrow. I'll inform you here.

I just pushed an update to the code repo. You can pull the latest code to have the fixed code. I'm also going to publish it on pip in the next couple of days.


dev-setup: install azure-mgmt-nspkg to avoid errors of no-name-in-module

To avoid errors like:
Run pylint
Modules: azure-cli-core
************* Module azure.cli.core.commands.parameters
E: 23, 4: No name 'mgmt' in module 'azure' (no-name-in-module)
E: 23, 4: Unable to import 'azure.mgmt.resource' (import-error)

Codecov Report

Merging #3191 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3191   +/-   ##
=======================================
  Coverage   70.34%   70.34%
=======================================
  Files         383      383
  Lines       24951    24951
  Branches     3796     3796
=======================================
  Hits        17553    17553
  Misses       6288     6288
  Partials     1110     1110

Continue to review full report at Codecov.

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

/cc @derekbekoe


Morse code decryption

Here we will ask the user the enter a morse code and then decrypt it, for ex the output in the terminal will look like
Enter a Morse code with spaces: .... . .-.. .-.. --- .-- --- .-. .-.. -.. .---- ..--- ...--
Your entered: HELLOWORLD123
I would like to add this in cpp.

We don't need it under our scope


Phpunit Variables are not loading on Test Environment

Laravel Version: 9.x ( latest version )
PHP Version: 8.1
Database Driver & Version: mysql 8

Description:
Phpunit.xml variables are not loaded on abstract TestCase!
After clearing cache, config, etc., I was unable to create a test environment with different DB.
It is not possible for me to get the test  environment variable.
Steps To Reproduce:
There are 2 ways to see the bug:

after changing you phpunit.xml file in your Abstract TestCase add this method and you can see the result:

    protected function setUp(): void
    {
        parent::setUp();

        dump(env('APP_NAME'));
        dd(env('APP_ENV'));
     }

After clearing cache, run test command (php artisan test)

change your bootstrap/app.php:

Create .env.testing file and change your app file:
$app = new Illuminate\Foundation\Application(
    $_ENV['APP_BASE_PATH'] ?? dirname(__DIR__)
);

$app->loadEnvironmentFrom('.env.testing');

dump(env('APP_NAME'));  // still load from .env
dd(env('APP_ENV'));  // still load from .env

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Configure Docker build for bb

Let's refer to the docker image as buoyantio/bb throughout and push it to dockerhub (drop the gcr.io part). I setup a dockerhub repo for you, here: https://hub.docker.com/r/buoyantio/bb/ . We should also configure auto builds for that repo.

Ok, @siggy has enlightened me about automated builds:

We can't add auto build to an existing repository -- we have to create it initially as an auto build repo

Auto builds rely on the docker images being fully buildable from the Dockerfile alone. So the build process can't leverage the docker-build-bb.sh that we have in the repo.

If we want to setup auto build for this repo, then we need to:

Re-create the repo on docker hub (not a problem)

Switch to a multi-stage Dockerfile for building the image (somewhat unfortunate)

For new repositories, I feel like it's worth it to make them auto build, since that simplifies the release process a lot. @pcalcado, @siggy what are your thoughts? More info about auto build here.


Update use-data-gateways-sources-power-bi.md

VNet data gateway is currently only available for P and F SKUs, and not available for EM and A SKUs.
Source: https://learn.microsoft.com/en-us/data-integration/vnet/overview#limitations

@nikkiwaghani
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


[test-case-property-ordering] moved some properties to the last by default

Tell us about your environment

ESLint: master
eslint-plugin-eslint-plugin: 2.0.1

Please show your full configuration:

.eslintrc.js

What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.

//...
        {
            code: "function f() { let a; let a; }",
            parser: looseParserPath,
            parserOptions: { ecmaVersion: 2015 },
            errors: [
                { message: "'a' is already defined.", type: "Identifier" }
            ]
        },
//....

What did you expect to happen?
No errors.
What actually happened? Please include the actual, raw output from ESLint.

According to test-case-property-ordering.md, the default option doesn't seem to include some valid properties.

env
filename
globals
parser
(rules)
settings

Thanks for creating an issue. I agree that this isn't ideal.
Currently, the rule requires that unspecified properties appear at the end of the object. Would it be better to make unspecified properties appear before errors?
If the user provides a custom options object, where errors is at the beginning, I'm not sure how best to handle unspecified properties in that case.

It's fine to move unknown properties to the last. If people are using a custom rule tester, they have to configure that ordering rule for additional properties. This is reasonable.
It's great if the rule recognizes the properties the official rule tester supports, by default :)

I agree that we should change the default. However, this would be a breaking change.


TypeError: failed to execute 'appendChild' on 'Node'

Hi there, it seems that this plugin is throwing an error about once a second:

This is happening when regardless of if the timer is running.

Huh, I'm not getting anything on my computer. Did this change between 0.1.1 and 0.1.2 (the latest version)?

Not sure, I haven't opened dev tools in a while.
I imagine it's because of this:
https://github.com/kzhovn/statusbar-pomo-obsidian/blob/d83cb4a2029efb4c14b807792cd99f5fe6fe4bad/src/main.ts#L52-L53
When this.mode is Mode.NoTimer,  setStatusBarText() returns undefined. Seems like setText expects a value?

I've set it to return the empty string with this update - does that fix it for you?

That did the trick, thanks!


[GRIFFIN-212] Add matchedFraction in streaming accuracy metrics

I'm fine on this change

I don't know what is wrong between apache repo and github repo, I had merged/closed this PR, but it still exists here as OPEN.


win binary pls

subj

It would be nice if there were tagged releases on the GitHub Releases page that came with attached binaries for common platforms (Win32, OSX64, Lin32, Lin64).

https://github.com/cloudinary/fuif/releases

This allows for other tooling to be built around FUIF with less friction as there is a trusted source to get the binaries from and they don't need to be manually built by people that aren't familiar with the build environment/programming languages used by FUIF development. It would also be helpful if the file names for the attachments on the releases followed a specific naming pattern so they could be downloaded programmatically from the GitHub API. Example:

https://github.com/sass/node-sass/releases

It would be premature to build tooling around FUIF since it is only a research prototype at this point. FUIF is going to be part of JPEG XL, and at some point it will have a stable bitstream and a reference implementation around which tooling can be built. But at this point it is too early for that.

@jonsneyers oh, it will be released as "JPEG XL"? It could be nice to mention it on the main page of the repository.

Updated the main page. Thanks for the suggestion!


Payment method preview is clickable for bottom docked modal - reported by @Tushu17

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open app in mobile/ use expensify in desktop web and decrease the screen size
Go to Settings -> Payment
Add paypal account in add payment method
Select Paypal.me
Click on Paypal.me above delete button.

Expected Result:
Text shouldn't be clickable as it doesn't perform any function.
Actual Result:
It is clickable in small screen and in desktop small screen, cursor is shown as pointer over it.
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.41-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/156312087-d0d3650f-6585-4c98-bd78-195dc0f615c4.mp4
https://user-images.githubusercontent.com/43995119/156312271-e277d217-54c5-4b1c-9a23-a7d2383631cb.mp4
Expensify/Expensify Issue URL:
Issue reported by:  @Tushu17
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1645098315409679
View all open jobs on GitHub

Proposal
We need to pass disabled and interactive={false} to MenuItem below -
https://github.com/Expensify/App/blob/5dab0ffc198741cd13b3b79c3f15e256c9a889b1/src/pages/settings/Payments/PaymentsPage/BasePaymentsPage.js#L294-L299
   <MenuItem
       title={this.state.formattedSelectedPaymentMethod.title}
       icon={this.state.formattedSelectedPaymentMethod.icon}
       description={this.state.formattedSelectedPaymentMethod.description}
       wrapperStyle={[styles.pv0, styles.ph0, styles.mb4]}
+      disabled
+      interactive={false}
   />

https://www.upwork.com/jobs/~01a5afd1f614dd5d18

Current assignee @laurenreidexpensify is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

@tgolen sorry I jumped the gun here before confirming it was external. I might have confused the order, but can you please add external label too? Thanks

Current assignee @laurenreidexpensify is eligible for the External assigner, not assigning anyone new.

OK, labeled! The proposal looks fine to me too, so 🟢 on hiring them on Upwork.

Yup, @Tushu17 's proposal looks good.
:ribbon: :eyes: :ribbon:  C+ reviewed

Wow that was quick! :)

📣 @Tushu17 You have been assigned to this job by @laurenreidexpensify!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.42-6 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/8025

If no regressions arise, payment will be issued on 2022-03-22. :confetti_ball:

@Tushu17 sorry I missed this, but as soon as you accept offer I will issue payment in Upwork for $500 for reporting and implementing solution

@laurenreidexpensify ok done, I've accepted the offer.

All C and C+ payments have been issued in Upwork 👍🏽


[Pub/sub] Add match example to API spec

Description
Add matches example to the pub/sub api spec
Issue reference
PR will close: #2923

@hhunter-ms There need to be several more changes here.
Mostly the update here should follow this article https://docs.dapr.io/developing-applications/building-blocks/pubsub/howto-route-messages/

See comments above on routes not route
In the https://docs.dapr.io/reference/api/pubsub_api/#provide-routes-for-dapr-to-deliver-topic-events section after the "In order to deliver topic events, a POST call will be made to user code with the route specified in the subscription response." sentence, say that under routes you can provide rules that match a certain condition to specific path when a message topic is received . I would also mention that you can provide a default route for any rules that do not have a specific match.
I would then add a link to https://docs.dapr.io/developing-applications/building-blocks/pubsub/howto-route-messages
Whilst we are fixing this up, can you also update

https://docs.dapr.io/developing-applications/building-blocks/pubsub/howto-route-messages#programmatic-subscription here from saying "Node" in the tab to "Javascript" and the C# one to .NET, so this is consistent with https://docs.dapr.io/developing-applications/building-blocks/pubsub/subscription-methods/#programmatic-subscriptions
Generally what is the name and order we have been using for languages?


[YUNIKORN-432] Fix broken links in get_involved.md, reporting_issues.md

Fix broken links to get_involved, reporting_issues link.

Looks good, thanks @bmv126


Invalid characters crash fix

See with this project: https://github.com/ClementPadovani/aBike

Hey,
Oh yeah, forgot about the formatting. I’ll get to it

@markohlebar waiting for your go. I’ve added tests


How to prepare dataset for RIR & Musan Augmentation

Hi,
Would like know how to prepare dataset for RIR & Musan Augmentation
I go through the script, and understand that it needs data in .mdb format that should be inside lmdb folder. I have raw audio files, how to prepare data for it?
Also, would like to know, is there any flag in the configuration file, which I can use as a flag to apply augmentation or not.
Thanks

please see https://github.com/wenet-e2e/wekws/blob/main/tools/make_lmdb.py

Just need preparing a wav.scp file and specify the path to save lmdb file. Then run "python make_lmdb.py /path/to/wav.scp /path/to/lmdb_dir" to prepare the lmdb file.
You can turn off augmentation by setting the corresponding probability to 0, or simply do not pass the corresponding lmdb file in run.sh

Thanks


Why does transformer use noam as the learning rate decay scheme?

What's the benefit of "noam" in machine translation compared to exponential or square root decay?
Also, why is it that in the base mode of the transformer. gradient clipping is set to 0?

I am also confused about why they multiply the weight decay with a very large number 5000. It is so weird.

All the multiplications are performed because T2T uses normalized values: we try to make the learning rate of 0.1 work with various optimizers (normally Adam would use 0.002 or so) and we try to make weight-decay per-parameter (people usually tune it per-model, but then whenever you change hidden_size you need to change that too, and a number of other things and so on). I can see that the normalized choices are sometimes confusing when compared to the literature, but it's very hard to make different models and hparam_sets work ok without some form of normalization.

is there any pointer to noam decay? i was googling but cannot find how exactly is noam decay done.

For now, the best pointer is Section 5.3 in the "Attention Is All You Need" paper (https://arxiv.org/abs/1706.03762). The equation is there already, but we'll try to add more details and intuition in a later version of the paper.

Why is this called "noam" schedule? The following paper does not name that equation as such.

For now, the best pointer is Section 5.3 in the "Attention Is All You Need" paper (https://arxiv.org/abs/1706.03762). The equation is there already, but we'll try to add more details and intuition in a later version of the paper.

Noam Shazeer is the second author of the paper.

Did not notice that. Thanks for the quick response!

Training Tips for the Transformer Model https://ufal.mff.cuni.cz/pbml/110/art-popel-bojar.pdf


Create brust.txt

add brust.txt

@brust8023 This domain seem to not belong to a university, nor to a college. I have to close the request without merging. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request.
Shall you decide to apply again, please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof which shows that the school recognizes the domain which you are submitting as an official email domain for the students.


What are units for definition of maximum allowed walking limit?

In file build_transfer_file.py in function extract_graph() you  read a network using OSMnx package as:
G = ox.graph_from_place(query, network_type='drive')
According to description of this package length of edges of this network is measured in spatial units, not in time units.
In function initialize() you request from an user to define maximum allowed walking limit in seconds and get  value WALKING_LIMIT.
Then in function find_transfer_len you find distance from each stop using graph G in function
nx.single_source_dijkstra_path_length(G, source_info[1], cutoff=WALKING_LIMIT * 2, weight='length')
And this is the place that I don' t understand because length of edges of this network is measured in spatial units but for cutoff you use time units. Can you explain this?

Dear Ekleiman,
In my experience, the link lengths in OSM are typically measured in meters. I based my calculations on an assumed walking speed of 1 meter per second, making the cutoff (in seconds) directly correlated with length. Given the age of this issue, I'll be closing it now. If you have additional questions or need further clarification, please feel free to open a new issue. I assure you of a prompt response.


Remove GitHub actions badge.

Godot workflow is long removed in master, and is only found in the 3.2-android-app-bundle branch currently. As of now Github actions has no way to show passing if all checks are passed but only to show the badge of each workflow individually. We can make a feature request later on but for now, we should remove the Godot CI badge as it reflects completely something else.

Thanks!


fix: fix branch link 'master' to 'main'

I find TypeScript repo main branch changed 'master' to 'main'. but the links in the translation didn't change

@microsoft-github-policy-service agree


Sorted key path improvements

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[ ] New extensions are written in Swift 5.0.
[ ] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[ ] I have added tests for new extensions, and they passed.
[ ] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[ ] All extensions are declared as public.
[ ] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 587 tests, with 0 failures (0 unexpected) in 10.535 (11.968) seconds

:book:
tvOS: Executed 562 tests, with 0 failures (0 unexpected) in 2.274 (2.822) seconds

:book:
macOS: Executed 421 tests, with 0 failures (0 unexpected) in 0.992 (1.273) seconds

  Generated by :no_entry_sign: Danger

You've deprecated functions that allow you to just call sort(by: someKeyPath) but the replacements don't allow for the same - you must supply a comparison function.

Maybe we can, but the problem in allowing call it like that, is since we only deprecate the old one calling like that sort(by: someKeyPath), the compiler would have two overloads to choose from the old sort(by:ascending) and sort(by:with:) which can lead to ambiguous error or it chooses the deprecated one.

Maybe we can, but the problem in allowing call it like that, is since we only deprecate the old one calling like that sort(by: someKeyPath), the compiler would have two overloads to choose from the old sort(by:ascending) and sort(by:with:) which can lead to ambiguous error or it chooses the deprecated one.

If there are overloads then delete the old ones that we don't need anymore, no?

If there are overloads then delete the old ones that we don't need anymore, no?

Maybe, since it would not be a breaking change :)

You've removed the ascending parameter, which means that you're introducing a breaking change.

I just removed the default value so that method can't be called like sort(by: ) anymore, which is not a breaking change because those will be now calling the sort(by:) with the default comparator overload.


Add options "undef", "replace" form CSV#open

I want to use CSV#open with undef: :repace, replace: "*" options form wrong character.

Orphan.


Throw syntaxError with loc property if such error was encountered

Fixes #115

looks good?

Don't get why ClassProperty still uniterable, but LGTM.

@zxqfox cause of this - https://github.com/cst/cst/pull/116/files#diff-18f06f03761fb72832a33d01d66e130dR14 Ж)

@markelog Omg, I need glasses :neckbeard:


Refactor files.js into FilesController.

Nicer structure, get rid of globals, follow-up PR from #315.

While you're at it, any opposition to moving all the handlers or controllers into a sub directory, starting with this one?

Yup, no opposition, good idea, moving to src/Controllers.

Updated with ES6 import style and updated the comment.


2.0 docs

See release plan in iterative/dvc/issues/5367.

Finalize pending 1.x-related updates (extracted from #1255) - #2074

[x] Review all .dvc file examples throughout docs (in YAML code blocks) — note we require YAML 1.2 (se iterative/dvc/pull/4415 and #1764) - mostly done #1608.
[x] ^ Also push/pull examples
[x] Review DVC cache info everywhere, especially in the internal files/dirs guide.
[x] plots refs per #1414
[x] other refs? per #1824

Would be nice to backport those to v1 when done, if possible

[x] "Parametrized" pipelines (dvc.yaml changes) - @jorgeorpinel + @skshetry (https://github.com/iterative/dvc/wiki/Parametrization) - #2052
[x] New/changed Get Started

New example project and story plan (including new pipelines and experiments) - @dberenbaum
Rewrite https://dvc.org/doc/start

[x] "DVC Experiments" (sample project: https://github.com/dberenbaum/dvc-exp-mnist)

Doc run-cache? (#1289) — #2137
Prepare docs structure conceptually for regular (persistent) and new (transient) experiments.
Experiment Management guide (#816) — #2146
Document dvc exp refs — #1926 #2199, others
Ideas: https://replicate.ai/docs/tutorial#experiments-and-checkpoints

[x] Live metrics - @pared?

Write README in that repo
Add refs to https://dvc.org/doc/api-reference

dvc stage (https://github.com/iterative/dvc.org/issues/2134#issuecomment-771214046)

[ ] Archived v1 docs - #2006 / #1167 + https://github.com/iterative/dvc/pull/5315
[ ] Update Katacoda scenarios - @iesahin ?
[x] config/remote {} --repo (#2064) - @efiop + @jorgeorpinel

@jorgeorpinel we can skip API references for Dvclive, after iterative/dvc#5466 this method will be removed.

Sounds good, thanks.

Glad to report that most of the mayor stuff is done or has mergeable PRs! Changing priority here.


add --cidr example and explaination

Using the default 32 never worked (Ubuntu bare metal on a to-be control plane)

Thanks for this! Unfortunately due to a number of requests I've had to refactor the subnet code so this change is unfortunately out of date.


Add "shared_domain" variable

For Cloud Foundry providers it would be nice, if we could use a specific shared domain that we offer, in the docs. For example the page https://docs.cloudfoundry.org/devguide/deploy-apps/routes-domains.html#domains-dns would be easier to comprehend for new users if it included a specific example (e.g. "cfapps.io" for PWS or "scapp.io" for Swisscom) instead of using the generic ".shared-domain.com". I would therefore vote to create a new variable shared_domain in config.yml and use it everywhere instead of examples like "shared-domain.com". The Cloud Foundry docs could still set that variable to "shared-domain.com" to stay generic.
If you guys agree, I can make a PR.

I just noticed that we already have this variable as app_domain. That resolves this issue.

Thanks @mastertinner !


ImageNet-ResNet50-Preact.npz file tensorflow branch

Hi, I'm trying to run the tensorflow_final branch, but I can't find the file ImageNet-ResNet50-Preact.npz, in the other branch I found a version for pytorch, but I can't find the version for tensorflow. Please, if someone can tell me how to find it, I would appreciate it very much, thanks

I heavily discourage using the TensorFlow branch but the link is here https://github.com/vqdang/hover_net/tree/tensorflow-final/src#training


Search doesn't work for certain results

Description
I'm using the bitsearch.to site and plugin and certain result wont find anything. "My Neighbor Totoro" for example, but if I go to the site and search it shows 100s of results
Steps to reproduce
Enable the bitsearch.to plugin
Enter "My Neighbor Totoro" as the search term
See no results
See many results on the site https://bitsearch.to/search?q=My+Neighbor+Totoro
Logs
qbittorrent.log
Versions
QBittorrent: 4.6.3
Plugin: 1.0
OS: Windows 11 10.0.22621.3374

Hi, I'll take a look, if it's failing with certain search terms, it's probably related to some parsing issue.

Version 1.1 has been pushed, and this issue it's fixed.


Installing then uninstalling isort breaks python.sortImports

Type: Bug

Behaviour
Expected vs. Actual
Installing and then uninstalling another extension should not silently break a feature of the python extension
Steps to reproduce:

Install isort
reload. I'm guessing the python extension detects that the user has "migrated"
Uninstall isort, reload
Efforts to sort imports will now have no effect
Uninstall and reinstall the python extension
Sorting imports works again.

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.9.15
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Pylance

There is no relevant python output or errors.

User Settings

envFile: "<placeholder>"

languageServer: "Pylance"

linting
• mypyArgs: "<placeholder>"
• mypyEnabled: true
• mypyPath: "<placeholder>"
• pylintArgs: "<placeholder>"

sortImports
• args: "<placeholder>"
• path: "<placeholder>"

formatting
• provider: "black"
• blackArgs: "<placeholder>"

terminal
• activateEnvironment: false

Extension version: 2022.16.1
VS Code version: Code 1.72.2 (d045a5eda657f4d7b676dedbfa7aab8207f8a075, 2022-10-12T22:16:26.920Z)
OS version: Linux x64 5.15.0-50-generic
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i7-12700K (20 x 3600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 1, 1

Memory (System)
62.60GB (35.56GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

@micimize We are migrating users to ma-python.isort extension. Are you using pre-release version of the python extension? If yes, then python extension ships with the isort extension.  When you say install isort do you mean the isort module or the ms-python.isort extension?

@micimize Can you share your settings for the sortImports?

I just tried to reproduce this on v2022.16.1 and could not, so maybe there was some other issue.
For completeness, what I thought happened was this:

I saw it mentioned in another issue here that y'all were planning to migrate to ms-python.isort, so I installed and configured it.
It didn't work with isort 4.3.21 and I couldn't figure out the config change in v5 needed to get the same behavior, so I gave up and uninstalled it.
I switched my config back to python.sortImports, but wasn't able to sort files.
Eventually I reinstalled the python extension, and it started working again

My guess was that some internal flag had been saved marking me as having migrated to ms-python.isort, but maybe it was something else entirely.
These are my settings:
  "python.sortImports.path": "${workspaceFolder}/.venv/bin/isort",
  "python.sortImports.args": [
    "--line-width=119",
    "--case-sensitive",
    "--skip-glob=node_modules,mypydir",
    "--force-single-line-imports",
    "--section-default=FIRSTPARTY",
    "--indent=\"    \"",
    "--project=tests"
  ],

I've encountered the same situation.
At some point, I uninstalled the ms-python.isort extension.
After then, neither Python Refactor: Sort Imports or Organize imports works, even if I reinstalled the isort extension.
After reinstall the python extension (I suppose this may also reinstall the isort extension), both commands work fine again!. That's weird. Shouldn't only one isort functionality work at a time?
BTW, I don't have python.sortImports.path or python.sortImports.args in any of my settings.json files.

@roytseng-tw We are transitioning to use the isort extension. So, temporarily both commands will work. The Python Refactor: Sort Imports command internally just triggers Organize Imports.


Add a Decider for generational tuning

This PR will be broken down into smaller chunks at a later date
Add metrics into the YoungGenRca to detect undersized young
generations
Add a YoungGenDecisionPolicy to suggest modification actions when an
issue is detected
Add a GenerationalSizingDecider to make generation tuning decisions
Modify the NodeConfigCollector so that NewRatio can be determined
dynamically on the ES Cluster
Add a PersistableSlidingWindow class to allow large sliding windows to
be persisted and loaded from disk to avoid losing data on node restarts

Fixes #:
Description of changes:
Tests:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This PR has been broken up into various pieces. The decider piece is contained in #460


Transaction section improvements

As discussed on last meeting, here i'm summarize things to do on transactions section:

[x] [list tab] Currently we're showing only last 5 transactions. We need to filter by dates, not by limit
[x] [list tab] Date picker selection actually does not affects transactions list
[x] [categories tab] "income" and "Expenses" filter does not change view
[x] [categories tab] Graph should take colors by category
[ ] [categories tab] Subcategories shows IDs instead of values when opened
[x] [categories tab] Subcategories has overflowed text when opened/closed (Fixed on #116)
[x] [accounts tab] Should be entirely done from scratch
[x] [all tabs] Collapsible header shows "settembre 2022" when collapsed, regardless on actual month selected (Fixed on #102)

I think this could be closed, everything done in my latest merge.

I'll close this one and open a specific issue for the remaining task (Subcategories shows IDs instead of values when opened)


feat: static kiosk link

Creates a kiosk page that can be shared at a static URL, and is compatible with Apple's "Add to Home Screen" functionality.
Primary purpose is to support kiosks on iPads.

Approving. Though since we aren't doing the public access we could technically just make the existing one static at a URL

Eventually i'd like to make kiosks optionally public, since the people using/operating the kiosk are very often not developers, and there's no benefit to me verifying at a kiosk on my own device, since i just prevent myself from verifying at the intended kiosk
so i'll leave this as-is for now and we can make the changes to allow for public kiosks later


Use 'Main' as the development branch on digital-platform

Admin and Apply use 'Main' as the development branch, while Digital Platform still uses 'develop'. This can cause confusion and error in development so should be aligned so that all repos use 'main' as the development branch.

It seems that can be done on github UI, then needs to change our local/remote branch name on our own git.
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-branches-in-your-repository/renaming-a-branch#updating-a-local-clone-after-a-branch-name-changes

Thanks @KoWeiJAC yes, we just need to co-ordinate doing this as it’s a bit disruptive so needs to be done when everyone is between tasks on digital-platform:
e.g. all devs will need to check in work and detach from the digital-platform repo. Then the branch is renamed. Then all devs will take a fresh clone of the repo. Simples!


Signs for SREDevOps.org

              SREDevOps.org
              Company
              SRE, DevOps and Platform Engineering Community

              Nicolas Georger
              Individual
              SRE, DevOps and Platform Engineering Community

@ngeorger your PR has some conflicts that need to be resolved before we can merge it.
Also, Can you please put the company in the sections of the companies - see here?

Added in https://github.com/opentffoundation/manifesto/pull/1031.


Cairo runner new unit tests

This PR add test to ecdsa.rs file

Codecov Report

Merging #179 (efa4582) into cairo-runner-unit-tests (25d634a) will increase coverage by 1.93%.
The diff coverage is n/a.

@@                     Coverage Diff                     @@
##           cairo-runner-unit-tests     #179      +/-   ##
===========================================================
+ Coverage                    91.13%   93.07%   +1.93%
===========================================================
  Files                           13       13
  Lines                          722      722
===========================================================
+ Hits                           658      672      +14
+ Misses                          64       50      -14

Impacted Files
Coverage Δ

src/cairo_runner.rs
93.28% <ø> (ø)

src/ecdsa.rs
92.85% <ø> (+78.57%)
:arrow_up:

src/lib.rs
75.00% <ø> (+75.00%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update install-harness-self-managed-enterprise-edition-using-helm-ga.md

custom changes
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


'No Response' in Apple Home after a certain Time

What issue do you have? Please be as thorough and explicit as possible.
After adding the Devices to Apple Home, they work for around 18h and then stopp. The only way to get them to work again is to remove the Bridge vom Home and add it back again. After the next cycle of around 18h they stopp again.
In Homebridge itself they continue to be shown and work as intended, even when Apple Home says it's not getting a response.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
Homebridge UI-X

Which version of Homebridge/HOOBS do you have?
1.6.0

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Node 18.13.0

Which version of this plugin (homebridge-ewelink) do you have? Has the issue started since upgrading from a previous version?
10.4.0

Which eWeLink devices do you have that are causing issues? Please include product models if applicable.
1 NS Panel Pro, 3 M5 Multiswitches and 1 R5 Remote

Please paste any relevant logs below.
[2/3/2023, 8:07:56 AM] [eWeLink] Fetching devices from eWeLink home/id [Zuhause] [625eafbb9590610009936e5c].
[2/3/2023, 8:08:00 AM] [eWeLink] LAN monitoring started.
[2/3/2023, 8:08:00 AM] [eWeLink] [Panel NS] initialising with options {"logging":"standard","offset":0}.
[2/3/2023, 8:08:00 AM] [eWeLink] [Panel NS] initialised and LAN mode unavailable as not discovered/supported [133:NSPanel].
[2/3/2023, 8:08:00 AM] [eWeLink] [Remote] has been added to Homebridge.
[2/3/2023, 8:08:00 AM] [eWeLink] [Remote] initialising with options {"logging":"standard"}.
[2/3/2023, 8:08:00 AM] [eWeLink] [Remote] initialised and LAN mode unavailable as not discovered/supported [174:NON-OTA-GL(174)].
[2/3/2023, 8:08:00 AM] [eWeLink] [Sleep3 M5] initialising with options {"inchChannels":"","inUsePowerThreshold":0,"logging":"standard","showAs":"default"}.
[2/3/2023, 8:08:00 AM] [eWeLink] [Sleep3 M5] initialised and LAN mode unavailable as not discovered/supported [161:M5-2C].
[2/3/2023, 8:08:00 AM] [eWeLink] [Sleep1 M5] initialising with options {"inchChannels":"","inUsePowerThreshold":0,"logging":"standard","showAs":"default"}.
[2/3/2023, 8:08:00 AM] [eWeLink] [Sleep1 M5] initialised and LAN mode unavailable as not discovered/supported [161:M5-2C].
[2/3/2023, 8:08:00 AM] [eWeLink] [Bath M5] initialising with options {"inchChannels":"","inUsePowerThreshold":0,"logging":"standard","showAs":"default"}.
[2/3/2023, 8:08:00 AM] [eWeLink] [Bath M5] initialised and LAN mode unavailable as not discovered/supported [161:M5-2C].
[2/3/2023, 8:08:02 AM] [eWeLink] ✓ Setup complete. Don't forget to ☆ this plugin on GitHub if you're finding it useful!

Hi,
Closing this just as been a little overwhelmed with issues recently.
If this is still an issue or a feature you would like implemented then please do re-open this ticket!
Thanks for your understanding!


feat: added custom settings section

This PR allows users to add custom settings to the talker settings page.
These custom settings are not managed or stored by talker but are just displayed on the settings page. The settings can be grouped into groups and each group can be enabled or disabled.
I understand if you decide not to merge this because it is a bit out of scope for collecting logs, but it is super helpful for me, as I prefer to have just one place to all debug related things, I'm using it in multiple apps and thought I'll share it.
Thank you for developing and maintaining this great package :)

Is there anything I can do to be able to merge this?

@Frezyx would you like me to fix conflicts so this PR can be merged?


Doc for not up to date for registerNotificationReceivedBackground

I noticed some inconsistencies between the website documentation and the typescript code.
When looking at the doc (https://github.com/wix/react-native-notifications/blob/master/website/docs/api/general-events.md), it says
Notifications.events().registerNotificationReceivedBackground((notification: Notification, completion: (response: NotificationCompletion) => void) => {
  console.log(JSON.stringify(notification.payload));

  // Calling completion on iOS with `alert: true` will present the native iOS inApp notification.
  completion({alert: true, sound: true, badge: false});
});

However in typescript  response is of type NotificationBackgroundFetchResult. Which one is the correct one ?

up


Fix empty callbacks in fit

Fix for #187.
If verbose=0 then callbacks.callbacks is empty and accessing the last element fails.
The question is: Should the fit method really return no history when verbose=0?

Closing, you fixed it! :)

The expected behavior would to always return history, even with in non-verbose mode. Verbosity controls real-time logging.


Provide several customization

fixes #52

@rwinch I think (and hope :) the PR is now ready to be reviewed - I refactored some of the existing unit tests and tried to cover all the mentioned customization features and bugs

Is it going to be merged?


Fixed bug on dynamic_key? checking

The nested fields index starts with 0.

Hi there!
I have a bug with ActionForm and this is exactly the fix :)
Can you merge this please?
Thanks!

Action Form is not for production use. You're doing so at your own risk 😁


[Access Request] page 609 "IC Partner Card" make EnableInboxDetails protected

Hi,
Could you please make variable EnableInboxDetails protected, so we could utilize it within our extension.
Best Regards,
Gintautas

duplicate of #13089

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Voting updates across Console and Token UI not updating dynamically

Currently my votes or change of votes doesn't get transcended to both platforms. So basically if I vote or change my vote on Console, I need to refresh Token UI to see the change and vice versa. Ideally the vote / change of vote should be captured dynamically across both the platforms.

This to be covered in https://github.com/vegaprotocol/token-frontend/issues/1228


OLS-1239: Update text of the 3 privacy warnings

/approve
/lgtm
/retest

Override failing test not related to this change:
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-console"


Out of date safety note on #[ffi_function]?

This is a complicated topic, so correct me if I'm wrong here, but #[ffi_function] states:
/// # Safety
///
/// ⚠ You _must_ ensure that methods exported with `#[ffi_function]` will never panic. We highly encourage you
/// to wrap all your code in panic guards. This is a standard Rust FFI concern and has nothing to do with Interoptopus.
/// Failure to follow this advice will probably lead to undefined behavior down the road. The author has been there and does not recommend it.

I believe this is no longer the case now that c_unwind was stabilized and the default behavior for extern "C" was changed in Rust 1.81. See https://github.com/rust-lang/rust/pull/116088
It still may be a good idea to use panic guards for functionality's sake, but I believe it's no longer UB as any extern "C" function that hits a panic will abort rather than let the unwind cross the FFI boundary now.

People using older compilers might still run into that issue, but with the next release(s) we could bump our MSRV to 1.81 and remove some auto-generated panic guards to increase performance a bit.


Allow checking if current component is page

What problem does this feature solve?
During work on application we faced a need to use mixin for pages only. Right now we have two options:

check page inside mixnin
const isPage = this.$options.name && this.$options.name.toLowerCase().endsWith('page'), but this is very fragile
manually include required mixin everytime we create new page

What does the proposed changes look like?
As an example it would be nice to have this.$options.isPage boolean

This feature request is available on Nuxt community (#c8680)

I would suggest to use directly a PageComponent that you create with the mixin inside and use it into your pages directly:
import PageComponent from '~/components/Page.vue'

export default PageComponent.extend({
  // asyncData
  // ...
})

This will avoid to have any kind of global Mixin that has an impact on performances.
What is your mixin exactly and what are you trying to achieve with it that you can't do with another way?
This way we will be able to find the best solution :-)

@Atinux so, do I need to extend each page I have in my project?

@Atinux what I am trying to achieve is to add schema.org markup to each page using head () function in mixin

I think an option to inject a mixin only into page components would be great, because it allows to keep .vue pages without script blocks.

this.$vnode?.data.nuxtChild

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. I'm closing this issue as it's marked as a Nuxt 2 related enhancement and it's not a critical issue.
That doesn't mean it might not be relevant for Nuxt 3. If it is, please feel free to open a new issue (or just comment, and I can reopen it). 🙏
Thank you for your understanding and for your contribution to Nuxt! 🎉


Clean build gives Failed to start dhcp6_pd_daemon.service: Unit otbr-agent.service not found

Describe the bug A clear and concise description of what the bug is.
First time clean build fails
I believe recent change to script/_dhcpv6_pd_ref causes the issue
commit 094d0ccf2f95aca07ff5a1a9d5fafde5a02a3c3e
To Reproduce Information to reproduce the behavior, including:
From a clean system (e.g RPI)  which has never built otbr
Connect Wi-Fi interface using raspi-config.
sudo apt-get update
sudo apt-get install git
git clone https://github.com/openthread/ot-br-posix
cd ot-br-posix
./script/bootstrap
INFRA_IF_NAME=wlan0 ./script/setup
fails (see below)

Git commit id
IEEE 802.15.4 hardware platform
Build steps
Network topology

Expected behavior A clear and concise description of what you expected to happen.
Build completes
Console/log output If applicable, add console/log output to help explain your problem.
[Install]
WantedBy=multi-user.target
have systemctl
command -v systemctl
sudo systemctl daemon-reload
sudo systemctl enable dhcp6_pd_daemon.service
Created symlink /etc/systemd/system/multi-user.target.wants/dhcp6_pd_daemon.service → /etc/systemd/system/dhcp6_pd_daemon.service.
sudo systemctl start dhcp6_pd_daemon.service
Failed to start dhcp6_pd_daemon.service: Unit otbr-agent.service not found.
Additional context Add any other context about the problem here.
On a clean system otbr-agent does not exist at this stage.

Hi @pac8086 , the issue is because I changed the default Raspi config incorrectly to include what should only be enabled in reference device. I will let you know once fix is merged.

Same issue here, is there a workaround like checking out an older commit?


Quest details

Since Darkzo dropped the ball here, I'm gonna hijack; is there any intent on fixing the new quest log functionality? The quests only come up in KCV if you hunt through "all" to find them and I actually found it more beneficial in 4.2.4 to open up the "active" quest tab in game to put them in KCV's quest area each time I went to do something.
Also, how do translations work? The new quests seem to be untranslated, dunno if that's supposed to happen or not.

There is no way to “fix” it due to the way the game sends quest information.
Translations for new quests will be out a bit later.

Thanks.

The loss of quest loading from tabs other than "全　All" is exclusive to the latest release, I tried to replicate it on 4.2.4 and the quests work just fine. Grabacr did something to mess it up?

The way KCV works is that every time it receives a page of quests with a different-to-before content, it empties its current stored pages. Subsets of quests trigger this clean-up, but do not contain all of the active quests.


Unlock connector buttons should be disabled if the connector status is ‘Available’

Unlock connector buttons should be disabled if the connector status is ‘Available’
Avoid generic message ‘Request has failed’ but rather ‘Cannot unlock the connector’ (to be checked everywhere)

@fcoutellier
Just deployed the version 1.3.5 Beta on both stores


Merge latest updates from jaraco/skeleton

This PR pulls in the latest updates from https://github.com/jaraco/skeleton, adjusted to handle a few merge conflicts.
Notably, the update adds the use of towncrier for generating a changelog, which we are not yet set up to take advantage of. I'll leave it for a future PR to either do that setup, or decide that we don't want to use it and remove the associated configuration file.

I think this should be good to go now. I tested the merge locally and it seems to proceed without conflicts.


Instant game crash

Game crashes within a few seconds of opening. Most recent version of CET, game version 1.61 on Steam.
CET log has this:
[info] [Options] Cyber Engine Tweaks is starting...
[error] [Initialize] 5D
[error] [Initialize] 30
[error] [Initialize] A1
[error] [Initialize] 30
[error] [Initialize] F
[error] [Initialize] 7F
[error] [Initialize] 5E
[error] [Initialize] 4C
[error] [Initialize] A9
[error] [Initialize] BA
[error] [Initialize] D8
[error] [Initialize] 43
[error] [Initialize] D9
[error] [Initialize] AE
[error] [Initialize] 85
[error] [Initialize] 50

No it does not, ask for support on discord.


3.3.13-beta start up problems on Linux

snoggetorp% ./PabloDraw
An error occurred.  Please report this information to https://github.com/cwensley/pablodraw/issues:
System.NullReferenceException: Arg_NullReferenceException
at GLib.GType.op_Explicit(Type)
at GLib.Object.ClassInitializer.AddProperties(IntPtr)
at GLib.Object.ClassInitializer.ClassInit(IntPtr)
(PabloDraw:57431): GLib-GObject-CRITICAL **: 19:58:49.316: g_object_new_is_valid_property: object class '__gtksharp_5_Eto_GtkSharp_Forms_GtkShrinkableVBox' has no property named 'homogeneous'
(PabloDraw:57431): GLib-GObject-WARNING **: 19:58:49.321: ../glib/gobject/gsignal.c:2517: signal 'destroy' is invalid for instance '0x556a6c3f8a00' of type '__gtksharp_5_Eto_GtkSharp_Forms_GtkShrinkableVBox'
(PabloDraw:57431): Gtk-CRITICAL **: 19:58:49.323: gtk_widget_set_visible: assertion 'GTK_IS_WIDGET (widget)' failed
(PabloDraw:57431): Gtk-CRITICAL **: 19:58:49.325: gtk_box_pack: assertion 'GTK_IS_WIDGET (child)' failed
(PabloDraw:57431): Gtk-CRITICAL **: 19:58:49.341: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
(PabloDraw:57431): GLib-GObject-CRITICAL **: 19:58:49.346: g_object_new_is_valid_property: object class '__gtksharp_6_Eto_GtkSharp_Forms_EtoVBox' has no property named 'homogeneous'
(PabloDraw:57431): GLib-GObject-WARNING **: 19:58:49.346: ../glib/gobject/gsignal.c:2517: signal 'destroy' is invalid for instance '0x556a6c3f9ea0' of type '__gtksharp_6_Eto_GtkSharp_Forms_EtoVBox'
(PabloDraw:57431): Gtk-CRITICAL **: 19:58:49.346: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
(PabloDraw:57431): Gtk-CRITICAL **: 19:58:49.346: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
Unhandled exception. System.NullReferenceException: Arg_NullReferenceException
at GLib.GType.op_Explicit(Type)
at GLib.Object.ClassInitializer.AddProperties(IntPtr)
at GLib.Object.ClassInitializer.ClassInit(IntPtr)
zsh: IOT instruction (core dumped)  ./PabloDraw

Hey @ericolito, thanks for reporting the issue! What disto/version are you using?

I have similar, but assume its a dependancy issue? Maybe something to do w/ GLib:
An error occurred.  Please report this information to https://github.com/cwensley/pablodraw/issues:
System.NullReferenceException: Arg_NullReferenceException
at GLib.GType.op_Explicit(Type)
at GLib.Object.ClassInitializer.AddProperties(IntPtr)
at GLib.Object.ClassInitializer.ClassInit(IntPtr)
(PabloDraw:16810): GLib-GObject-CRITICAL **: 13:34:04.322: g_object_new_is_valid_property: object class '__gtksharp_5_Eto_GtkSharp_Forms_GtkShrinkableVBox' has no property named 'homogeneous'
(PabloDraw:16810): GLib-GObject-WARNING **: 13:34:04.324: ../../../gobject/gsignal.c:2510: signal 'destroy' is invalid for instance '0x556a937273a0' of type '__gtksharp_5_Eto_GtkSharp_Forms_GtkShrinkableVBox'
(PabloDraw:16810): Gtk-CRITICAL **: 13:34:04.324: gtk_widget_set_visible: assertion 'GTK_IS_WIDGET (widget)' failed
(PabloDraw:16810): Gtk-CRITICAL **: 13:34:04.325: gtk_box_pack: assertion 'GTK_IS_WIDGET (child)' failed
(PabloDraw:16810): Gtk-CRITICAL **: 13:34:04.332: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
(PabloDraw:16810): GLib-GObject-CRITICAL **: 13:34:04.334: g_object_new_is_valid_property: object class '__gtksharp_6_Eto_GtkSharp_Forms_EtoVBox' has no property named 'homogeneous'
(PabloDraw:16810): GLib-GObject-WARNING **: 13:34:04.334: ../../../gobject/gsignal.c:2510: signal 'destroy' is invalid for instance '0x556a9372c320' of type '__gtksharp_6_Eto_GtkSharp_Forms_EtoVBox'
(PabloDraw:16810): Gtk-CRITICAL **: 13:34:04.334: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
(PabloDraw:16810): Gtk-CRITICAL **: 13:34:04.334: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
Unhandled exception. System.NullReferenceException: Arg_NullReferenceException
at GLib.GType.op_Explicit(Type)
at GLib.Object.ClassInitializer.AddProperties(IntPtr)
at GLib.Object.ClassInitializer.ClassInit(IntPtr)
Aborted (core dumped)

Hey @ericolito, thanks for reporting the issue! What disto/version are you using?

Arch Linux. I tried downgrading the "gtk-sharp-2" package as the stable version of Pablodraw for Linux only seems to work with gtk-sharp-2 2.2.12 and below, but this didn't help when it comes to the beta version.

Interestingly I have the same problem - Ubuntu 22.04 - but if I manually build using the included instructions and run the resulting debug binary, everything works as expected.
However, if I package everything into a standalone release build (self contained, no dll or other files), I get the same failure. Not sure if this helps but hopefully it can at least diagnose what the issue is not.

Is there any solution for that? This issue is closed without any fix I think.
Got the same problem :(

i have isuee on debian 12
ragnarok@ragnarok:/tmp$ ./PabloDraw
An error occurred.  Please report this information to https://github.com/cwensley/pablodraw/issues:
System.NullReferenceException: Arg_NullReferenceException
   at GLib.GType.op_Explicit(Type)
   at GLib.Object.ClassInitializer.AddProperties(IntPtr)
   at GLib.Object.ClassInitializer.ClassInit(IntPtr)

(PabloDraw:105967): GLib-GObject-CRITICAL **: 17:32:17.933: g_object_new_is_valid_property: object class '__gtksharp_5_Eto_GtkSharp_Forms_GtkShrinkableVBox' has no property named 'homogeneous'

(PabloDraw:105967): GLib-GObject-WARNING **: 17:32:17.936: ../../../gobject/gsignal.c:2517: signal 'destroy' is invalid for instance '0x5594671c1f60' of type '__gtksharp_5_Eto_GtkSharp_Forms_GtkShrinkableVBox'

(PabloDraw:105967): Gtk-CRITICAL **: 17:32:17.937: gtk_widget_set_visible: assertion 'GTK_IS_WIDGET (widget)' failed

(PabloDraw:105967): Gtk-CRITICAL **: 17:32:17.939: gtk_box_pack: assertion 'GTK_IS_WIDGET (child)' failed

(PabloDraw:105967): Gtk-CRITICAL **: 17:32:17.949: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed

(PabloDraw:105967): GLib-GObject-CRITICAL **: 17:32:17.952: g_object_new_is_valid_property: object class '__gtksharp_6_Eto_GtkSharp_Forms_EtoVBox' has no property named 'homogeneous'

(PabloDraw:105967): GLib-GObject-WARNING **: 17:32:17.952: ../../../gobject/gsignal.c:2517: signal 'destroy' is invalid for instance '0x5594671c2b20' of type '__gtksharp_6_Eto_GtkSharp_Forms_EtoVBox'

(PabloDraw:105967): Gtk-CRITICAL **: 17:32:17.952: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed

(PabloDraw:105967): Gtk-CRITICAL **: 17:32:17.953: gtk_box_pack: assertion 'GTK_IS_BOX (box)' failed
Unhandled exception. System.NullReferenceException: Arg_NullReferenceException
   at GLib.GType.op_Explicit(Type)
   at GLib.Object.ClassInitializer.AddProperties(IntPtr)
   at GLib.Object.ClassInitializer.ClassInit(IntPtr)
Abortado


Added CLI library

What I did:

Added the CLI11 Library and just wired in the existing options into it

Signed-off-by: Kishore Ganesh<EMAIL_ADDRESS>

@melvinw I've changed it to std::optional and have changed the rest as per your suggestions. However, I think we won't be able to use auto [tv, tz]  since tv and tz are already declared, so I've assigned it to temporaries and reassigned them (Is there a better way to do this?).

Ah right, I forgot structured bindings don't work that way. You should be able to do std::tie(tv, tz) = ... I think

Yes, that worked!


fix: 🐛 run date validation on date picker window close

Date validation was only running on blur of the text element, meaning if
you only used the window, it didn't validate until you focused and
blured the text input or tried to submit. Used setTouched to remedy
this.
✅ Closes: #127
before: https://www.loom.com/share/b8946d66630342f396d11444bac681d9
after: https://www.loom.com/share/e4c1ecd7f8994d25ba53ba5e1518ae8b
There are still some issues here with both the datetimepicker and calendaronlydatepicker components. This did not fix them, I think their issues might be more related to some yup and moment clash but didn't have time to ferret those out in this ticket. I'll try to make an issue demoing both of those issues tomorrow.

:tada: This PR is included in version 4.2.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The examples in the slo init help file miss out the slo part of the command

Describe the bug
If you run reliably slo init -h command the examples in the help are worded:
$ reliably init:
  this method interactively creates a manifest file, asking you questions
  on the command line and adding your answers to the manifest file.

$ realibly init -f <path>:
  this method works the same as reliably init, but allows you to specify
  the location of the file. This is useful if you use a multi-repo approach
  to source control.

To Reproduce
As above
Expected behavior
The output should be:
$ reliably slo init:
  this method interactively creates a manifest file, asking you questions
  on the command line and adding your answers to the manifest file.

$ realibly slo init -f <path>:
  this method works the same as reliably init, but allows you to specify
  the location of the file. This is useful if you use a multi-repo approach
  to source control.

**Additional context**
N/A

Fixed by #273


Select views show a big empty space at the end of the list

I don't know if this was already reported, but select views are right now showing a big empty space at the end of the list:

cc: @piensaenpixel @xavijam

Could you take a look @piensaenpixel please?

sure

uhm i cant reproduce :( that's production. But i'm checking.

Could you try to reproduce it with these instructions:  https://github.com/CartoDB/cartodb/issues/9247

Na, dont worry I've just found the problem.


Catch the error for not filling Keywords in DOI form

Description
Saving DOI form without filling required Keywords field should show error in the form and not being redirected to error page 400.
Related issues
Fixes https://github.com/CSCfi/metadata-submitter-frontend/issues/790
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Changes Made

Modify FormTagField to catch error correctly when they Keyword field is not filled.

Testing

[ ] Unit Tests
[ ] Integration Tests
[x] Tests do not apply
[ ] Needs testing (start an issue or do a follow up PR about it)

fixes #790

Keywords cannot be empty anymore, which is great! The only thing which I wonder is the style issue. Keywords required field is looking different from other required fields.  It is implemented differently but is there a possibility that warning texts would look the same?

Yes, It's good that you are pointing out the differences in showing error styles.
The fact is that in DOI form,Keywords field is a text field (string type) while other required fields are nested items (array type). The errors are shown based on field's type and that's why they appears differently. This happens not only in DOI form but also in other forms.
We can modify them to look the same way (in another PR) but before that I think it's better to consult @ainoc for the design :)


[local_auth] Update the pubspec.

Missed pubspec update in https://github.com/flutter/plugins/pull/2489.

Closing as obsolete.


Add aria-label to buttons in Sidebar

First iteration: set the aria-label where also data-tip is used, using
the same text.

Set the aria-label on the three default buttons in the bottom of the
sidebar
Set the aria-label on the TabItem component for the services.

Todo:

Think about better texts to aid the user instead of just re-using the
labels

Motivation and Context
Based on the slack notification in #general @Lanie Molinar, first try improve WIA-ARIA compliance
How Has This Been Tested?
Not yet... Was not succesfull to run Franz in dev mode on Windows.

Closing this in favor of #1166.
Thanks @mroeling for your initial push.


Import StreamProgress over 100%

Hello,
I am trying to import planet.pbf (55GB) using the OSMVecto example. According to log output, the import is at over 1800%:
[INFO ] 2020-12-17 08:51:28.203 [main] StreamProgress - Progress: 1850.24%

I'm running out of disk space since the import seems to require a lot (currently ~1.3TB and growing) and it's difficult to estimate the disk requirements for the full import with >100% progress indicator.

Thanks a lot for reporting this, there is clearly a problem with the progress logger that need to be corrected. Until now, Baremaps has been developed and tested at a country level (switzerland) with PFB files from geofabrick (which is my most pressing need). I scheduled some planet wise tests with the OSM exports in January, so you are clearly ahead of me and your input is very valuable ;)
Could you tell me a little be more about your setup? Are you running it in docker or is it a bare metal server? Also, if you have time to dig in this issue, your inputs, propositions and patches are more than welcome.

Also, do you know at which stage of the import you are? The first stage being the creation of an LMDB cache for the nodes and ways. The second stage being the creation of the geometries and the import in the database. This should appear in the logs.

I'm using the OSMVecto example which runs the import.sh in docker. I'm running 12c/24t on bare metal with 128GB RAM and an NVME SSD. I had to cancel the import at the moment because of disk space concerns.
The import stage was
baremaps import \
        --database "jdbc:postgresql://localhost:5432/osmvecto?allowMultiQueries=true&user=osmvecto&password=osmvecto" \
        --file "${osm_file}"

Unfortunately I already removed the container/volume and related data to regain disk space, but I will re-run the import overnight to get more insight into the logs.

Nice, I would love to have this kind of setup at hand ;) I managed to get a machine with 12CPUs and 64GB of RAM but the disk space, but the disk space is not sufficient for a planet import. My plan is to perform the tests on an AWS Aurora/Fargate setup, but I'm worried to explode my side project budget with a single run. I will keep you informed and, please, do not hesitate to report additional issues.

I ran the planet.osm.pbf again and the issue with StreamProgress appears with both first and second stage.
I also ran a country extract to estimate disk requirements for the whole planet and a 603MB PBF expanded into a 34GB Postgres DB. Extrapolating this to the whole planet, a 55GB PBF would result in a 3.2TB DB and unfortunately I don't have that kind of space at the moment.

Two more data points: the 2.2MB pbf for Liechtenstein produces a 102MB database and the 336MB pbf for Switzerland produces a 12GB database. I guess that the tags column can get pretty massive as it is not compressed with a dictionary. According to the pricing page for AWS Aurora Serverless, the storage rate is $0.10 per GB-month, which remains relatively cheap to host a 3 to 4 TB database.

It may be interesting to test the zson extension that compresses JSONB. I guess that a lot of the tags are redundant.
https://github.com/postgrespro/zson

It may be interesting to test the zson extension that compresses JSONB. I guess that a lot of the tags are redundant.
https://github.com/postgrespro/zson


Standarize open ai llm response_format based on models passed

Should check for the model passed is compatible with the response_format
Only compatible models/response_format should be passed to openai SDK
https://github.com/bolna-ai/bolna/blob/96004ecd14d94ce4bc79170d1d5a516298ffe0fc/agents/llms/openai_llm.py#L23-L32
https://github.com/bolna-ai/bolna/blob/96004ecd14d94ce4bc79170d1d5a516298ffe0fc/agents/llms/openai_llm.py#L53-L58

I think this bug has been resolved.

Hi @sachanayush47 , yes this was resolved as part of https://github.com/bolna-ai/bolna/pull/42.


Error when running makemigrations

When I run manage.py makemigrations, I get the following error:
Traceback (most recent call last):
  File "./manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/core/management/__init__.py", line 345, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/core/management/base.py", line 348, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/core/management/base.py", line 399, in execute
    output = self.handle(*args, **options)
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/core/management/commands/makemigrations.py", line 65, in handle
    loader = MigrationLoader(None, ignore_no_migrations=True)
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/db/migrations/loader.py", line 49, in __init__
    self.build_graph()
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/db/migrations/loader.py", line 306, in build_graph
    _reraise_missing_dependency(migration, parent, e)
  File "/Users/frederikcreemers/.virtualenvs/employebla/lib/python2.7/site-packages/django/db/migrations/loader.py", line 276, in _reraise_missing_dependency
    raise exc
django.db.migrations.exceptions.NodeNotFoundError: Migration common.0001_initial dependencies reference nonexistent parent node (u'auth', u'0008_alter_user_username_max_length')

I've swapped out Django's auth.User model for my own, not sure if that could be causing the problem.

Sorry, looking at this more closely, this looks like an error that's not specific to this project.


Update Composer dependencies (2019-04-14-00-03)

Loading composer repositories with package information
Updating dependencies
Writing lock file
Generating optimized autoload files
> DrupalProject\composer\ScriptHandler::createRequiredFiles

Visual regression test passed!
View the visual regression test report


dio ^5.0.0 and network_logger any, version solving failed

Running "flutter pub get" in Flutter...
Resolving dependencies...
Because network_logger >=1.0.0 depends on dio ^4.0.0 and network_logger <1.0.0 depends on dio ^3.0.10, every version of network_logger requires dio ^3.0.10 or ^4.0.0.
So, because mindari depends on both dio ^5.0.0 and network_logger any, version solving failed.
pub get failed
command: "/Users/karthick/developement/flutter/bin/cache/dart-sdk/bin/dart __deprecated_pub --directory . get --example"
pub env: {
"FLUTTER_ROOT": "/Users/karthick/developement/flutter",
"PUB_ENVIRONMENT": "vscode.dart-code:flutter_cli:get",
"PUB_CACHE": "/Users/karthick/.pub-cache",
}

Fixed. Please run pub get to download the latest version.


Possibility of saving solution at irregularly spaced time intervals

Currently, SaveSolutionCallback accepts only either a fixed interval or a fixed dt parameter (i.e. outputting either at uniformly spaced timesteps or at uniformly spaced times).
From my experience it would be nice to have the option to pass in a list of output times which may be non-uniformly spaced (for example, in some reacting flow computations, a logarithmic timescale is really helpful as a lot of kinetics can slow down exponentially over time).
If there is interest in having such a feature (probably as an additional keyword in the SaveSolutionCallback initialization?) I could try my hand at implementing it.

This would be nice! You can probably use
https://docs.sciml.ai/DiffEqCallbacks/stable/timed_callbacks/#DiffEqCallbacks.PresetTimeCallback
for this feature.


CLI generate: Add option to continue watching in case of error

Problem
If you run prisma generate --watch, edit the schema and a validation error occurs, then the generate command exits with code 1. Eg:
[prisma:generate:watch] Schema parsing
[prisma:generate:watch] error: Error validating field `journalIssue` in model `UserDocument`: The relati
[prisma:generate:watch] on field `journalIssue` on Model `UserDocument` is missing an opposite relation
[prisma:generate:watch] field on the model `JournalIssue`. Either run `prisma format` or add it manually
[prisma:generate:watch]     at Socket.Writable.write (node:internal/streams/writable:334:10)
[prisma:generate:watch]     at GeneratorProcess2.sendMessage (D:\Programming\JabRefOnline\node_modules\prisma\build\index.js:32830:24)
[prisma:generate:watch]     at D:\Programming\JabRefOnline\node_modules\prisma\build\index.js:32874:14
[prisma:generate:watch]     at new Promise (<anonymous>)
error Command failed with exit code 1.
[prisma:generate:watch] yarn run prisma:generate:watch exited with code 1

This may be inconvenient since after fixing the error you have to restart the generation watch.
Suggested solution
Add a cli option which disables exit in case of errors, i.e. something like prisma generate --watch --continueOnError

Add a cli option which disables exit in case of errors

I'd argue that this should be the default behavior and instead, if there's demand for it, add an option to --exitOnError.

My point of view is, it would be better to not exit by default with --watch when iterating on my schema like suggested here.
For example, I do a typo and correct it, then the flow would be interrupted by relaunching the command after an error.
Though I could not reproduce the flow here as it seems that prisma generate --watch doesn't exit on error anymore.
@Janpot could you try and see what's the behavior for you now?

Can't reproduce anymore neither. Found the GitHub issue after I prisma generate --watch did not recover after fixing an error and immediately assumed I was hitting this problem. Trying to reproduce now it seems like it recovers correctly for me as well.
Going through my log output again, looks like the process didn't exit, but rather the file watcher missed an update or something.

Ok then I think the best here will be that I close this issue since it's already the current behavior.
If you see a bug or a possible improvement, it will be greatly appreciated if you could create a new issue 🙇🏼♂


Getting warning: Cannot find the PR id when attempting to use reporting action

When attempting to integrate this into a github action that takes in aggregated coverage reports I am getting the warning message mentioned in the title. I've looked around to see if anyone else has had a similar issue and I am thinking that actions/toolkit #289 might have exhibited similar behaviour.
It appears that the fix on that project was to check several different potential sources for the issue number.
It's also entirely possible that I'm not using this action as it was originally intended.

Here is the portion of my github action that is supposed to be generating the report:
      - name: Download coverage reports
        uses: actions/download-artifact@v2
        with:
          path: coverage/ # contains several directories (i.e. coverage/{group_one,group_two,...}/.resultset.json)

      - name: Collate reports
        run: ./github_scripts/coverage_report # generates a collated report which includes coverage/.resultset.json and coverage/.last_run.json

      - name: Create tarball of reports
        run: tar -czvf rspec-coverage-report.tar.gz coverage/

      - name: Upload collated coverage report
        uses: actions/upload-artifact@v2
        with:
          path: rspec-coverage-report.tar.gz

      - name: Generate Github coverage comment
        uses: aki77/simplecov-report-action@v1
        with:
          token: ${{ secrets.GITHUB_TOKEN }}

Output from github actions page

I was trying to use this on Push events which is why it wasn't working. I was able to come up with a solution that makes it work with that workflow instead.

@csaunders-humi what was the solution?

I used different github actions and ran simplecov myself.

It used find-pull-request-action to figure out what PR the change was for, then used create-or-update-comment to append a message.


tdm-container-kind attribute type needs to update in UML diagram

tdm-container-kind is simple string type in yang and current device implementation for this attribute is also as string but in UML diagram it is represented as tdm-container-kind list.

DMIP call on 15th of July analysed the UML and the YANG and could not find anything to be corrected in relation to the description of this issue.
It has been decided to immediately close this issue without further activity.


github page for numalogic

We need to have GitHub pages as we have in numaflow.

Hi @vigith
I have open a PR for this here issue-30

we are already working on this https://numalogic.numaproj.io/


update: return list of obsolete branches

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#228  👈
#227
#226
#225
#224
#223
#221
#220
#219
#218 : 1 other dependent PR (#222 )
master

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

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Aug 10, 5:36 PM EDT: @rphmeier started a stack merge that includes this pull request via Graphite.


Add silent option

Hi,
Could you review this. I added silent option.

Coverage remained the same at 69.186% when pulling a681f5e17d21426120d759f42cf38fa7b7e82e1d on sugarshin:patch-1 into bf4816504d193dbb51ed14fc414b2bcf49cc88f0 on Nickersoft:master.

Hey! Thanks for the pull request! It seems as though this PR is breaking one of the Travis tests. Please look into fixing it and I'd be happy to merge it once the issue is resolved :)

Coverage remained the same at 69.186% when pulling f7cde91ab16376278fa4f89853d63559ffb62b22 on sugarshin:patch-1 into bf4816504d193dbb51ed14fc414b2bcf49cc88f0 on Nickersoft:master.

I fixed test. Because this tests for only Firefox.

Thanks!

Thanks. please bump version, also publish to npm.

@Nickersoft ping

@sugarshin New version was just published (v0.0.13)!

Thanks!


How to use ktor with Compose Web?

I have been able to get the starter project working and now i want to play around with connecting with a backend using ktor client.  I added the implementation and the project builds but when i add import io.ktor i get an error that io is not found.
Am i doing something wrong?
Thank you.

I would suggest you to go through https://play.kotlinlang.org/hands-on/Full Stack Web App with Kotlin Multiplatform/01_Introduction tutorial to create pair of client(JS)-server(ktor) apps and after this enhance client with Compose for Web support.

I was referring to ktor on the client.  The problem with that tutorial is there is an issue with using kotlinx.serialization.  I was only able to get it working using a specific version that is mentioned in an issue where the person had a the same problem.  Otherwise it wont work.

@rossedwardsus  Maybe my setup will help you
in sourceSets
 val commonMain by getting {
            dependencies {
                implementation("io.ktor:ktor-client-core:1.6.7")
                implementation("io.ktor:ktor-client-serialization:1.6.7")
                implementation("org.jetbrains.kotlinx:kotlinx-serialization-json:1.3.2")
            }
        }

val jsMain by getting {
            dependencies {
                implementation(compose.web.core)
                implementation(compose.runtime)
                implementation("io.ktor:ktor-client-js:1.6.7")
                implementation("io.ktor:ktor-client-json-js:1.6.7")
            }
        }

in plugins section:
kotlin("plugin.serialization") version "1.6.10"

and httpclient object
val httpClient = HttpClient {
    install(JsonFeature) {
        serializer = KotlinxSerializer(json = kotlinx.serialization.json.Json {
            isLenient = true
            ignoreUnknownKeys = true
        })

        accept(ContentType.Application.Json)
    }
}

@suslovsergey Thanks.  Are you also including compose in the plugin section?

@rossedwardsus yeah -
    kotlin("multiplatform")
    id("org.jetbrains.compose")

Hello.  What versions are you using?

@rossedwardsus
kotlin.version=1.6.10
compose.version=1.0.1

Right.  The problems seems to have been with a previous version of compose.  Which is the one i tried initially.

@rossedwardsus I am glad to help

@rossedwardsus are there any questions left here?

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


Please add Eugene, Oregon!

Mayor Lucy Vinis<EMAIL_ADDRESS>Emily Semple<EMAIL_ADDRESS>Betty Taylor, Ward 2<EMAIL_ADDRESS>Alan Zelenka, Ward 3<EMAIL_ADDRESS>Jennifer Yeh, Ward 4<EMAIL_ADDRESS>Mike Clark, Ward 5<EMAIL_ADDRESS>Greg Evans, Ward 6<EMAIL_ADDRESS>Claire Syrett, Ward 7<EMAIL_ADDRESS>Chris Pryor, Ward 8<EMAIL_ADDRESS>

DUE TO HIGH VOLUME EMPTY ISSUES, OR ISSUES MISSING THE FOLLOWING INFORMATION WILL BE IMMEDIATELY CLOSED. THANK YOU!

A subject message for the email
A list of emails (at least one) for the government officials you’d like to receive the message
A detailed message concerning the budgeting decisions that are at stake in your community
Please read more about our email style guidelines here.

If this information is copied from an external source please provide a link to the original source.
See our current links for examples at https://defund12.org.
Thank you!!


Allow sources to have multiple source types

This will require merging of some sources, some view changes (e.g. source page) and importer updating for some sources.

per @acoffman: estimate 1 day of work for backend changes, ui effort ~1 week.

@acoffman to create overarching UI issue.

TODOs not tackled in E#405:

Foundation One currently has two importers but the interaction source is no longer in the database and the source tsv could not be located. I think we just need to delete the interaction importer for this source.
ChEMBL. As part of #464 we are removing CheMBL drug claims but we are keeping ChEMBL as a drug source. We also import ChEMBL interactions so those two sources could be merged.
Sources with multiple interaction types currently appear on the main source page multiple times (once for each type). We should think about how to handle this more gracefully.

First bullet point is addressed by #536. Second bullet point should be addressed there as well, but hasn't yet. Third bullet point will be part of v5 and is tracked by #556


Initial Cake Build Script

Opening this for discussion later.

Hi @gep13, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

thanks!


Merge master back into develop after 1.15 release

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[x] Automated Testing

[x] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Should be tested already, it's deployed to production!

Build finished.

Build finished.


Fixed indention error in SaslBot()

Commented out ldap import so the app will run.  Didn't delete it so it can be uncommented when SaslBot() is implemented and ldap is added to requirements.txt.
brief description of changes

Hello @colpanick! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file honeybot/main.py:

Line 10:1: E265 block comment should start with '# '

Sorry for the late notice, but the project was moved. Please see the readme
Sent with GitHawk


why no support vue.js

I really want to use the wired.js component in the vue.js project. Can I support it?

Actually looking for something. I am going to try using https://github.com/vuejs/vue-web-component-wrapper to make them work this weekend. I will report how it goes.
That whole wrapper library is based on this approach https://alligator.io/vuejs/vue-integrate-web-components/ but fixes few rough edges.

The simplest way to get started with vue, is to just use the pre-packaged version
<script<EMAIL_ADDRESS>
Here's an example of using components in vue with v-model binding:
https://jsfiddle.net/shihn/8b9svwro/
Hope it helps

Sorry for not responding to you in a timely manner, the error when I use this component in the project is as follows:

I did not find any solution,can you help me?

Sorry to bother you for so long and the problem has been solved. Thank you

@fanxinjie Glad it worked out. What was the problem? Maybe I can improve the documentation

@pshihn Sorry, due to the exam leads only to see your message today.
The error has always been because the first component that I used to reference on demand failed. I converted to reference directly in the index.html file through the CDN. At this time, I forgot to remove the previous on-demand reference registration assembly code.
Another problem is that referencing components on demand fails.


updated index.mdx

Fixed broken link in index.mdx.
Replaced old URL with new working URL https://github.com/rishabkumar7/the-devops-guide#readme.

Hi @mohdaquib171 thank for contributing, we need the link to be the webpage and not the markdown file in GitHub, I have update it in index.mdx. Should be fixed now.
Thanks for your contribution.


Unable to login, authorization redirects to bsky.social's landing page

Typing in my handle (maxfz.xyz) and clicking login sends me to the authorization process like expected, but clicking accept redirects me to https://bsky.social/about instead of back to picosky. Going back to picosky after that doesn't show me as logged in.

Tried to log in with my DID, results in the same behavior

no idea how this could be happening, i havent changed anything
can you confirm to me you are logging through bsky.social and it originated from psky.social

https://github.com/user-attachments/assets/cfec8acb-e9fc-42e7-a108-7bf48d748f80

ngl i have no idea how this could be happening
ill look into it but i cant reproduce it on any of my accounts

is it possible to have request logs with "persist logs" turned on so it doesn't wipe on page navigation

Do you have any privacy-related browser extensions installed?

@notjuliet:
https://github.com/user-attachments/assets/8319945d-fc52-493e-b6cd-37f46f1c6ddc
@DavidBuchanan314:
i do have uBlock Origin and Privacy Badger, i'll try disabling those

i'm pretty sure this is a firefox issue? managed to "fix" by signing in on chrome and copying the app's localStorage over. after looking at the requests, it seems having a redirect with a url in the query portion (e.g. ?iss=https://bsky.social for oauth) causes that url to take precedence somehow?

found the issue in one of my addons, "Skip Redirect", and fixed it by adding a whitelist rule for ://psky.social


compile fails error: 'countSatnav' was not declared in this scope

sketch\VanBusRx.cpp: In member function 'void TVanPacketRxQueue::_AdvanceHead()':
sketch\VanBusRx.cpp:1262:34: error: 'countSatnav' was not declared in this scope
1262 |     if (_head->IsSatnavPacket()) countSatnav++;  // TODO - remove
|                                  ^~~~~~~~~~~

Fixed with commit dfce8b2 . Thanks for noticing :smile:


"results.txt" has shell color characters in it

Example:
Tool: truffleHog
OrgorUser: terraform RepoName: fluentd-sumologic
^[[92mReason: Generic Password^[[0m
^[[92mDate: 2018-03-28 21:58:30^[[0m
^[[92mHash: e05234bc366d0ab0f5174f1085d84c032dd8b8a6^[[0m
^[[92mFilepath: aaa/bbb.txt^[[0m
^[[92mBranch: origin/master^[[0m
^[[92mCommit: Updated s3 key for remote state
^[[0m
^[[93m+  password = "admin"^[[0m

This is because of trufflehogs output. That contains these characters and I am simply taking the output and lumping them together in this wrapper. Id recommend opening the results file in a shell to see the colored output. I dont think I can provide any support wrt the trufflehogs output itself unfortunately.


Duplicate project?

Is there any difference between this and svelte-sonner?

hey 👋
honestly, I came to know about svelte-sonner after I finished porting.
while the end implementation does not differ from the previously mentioned repo because both are ports of the original, but I try to write it more svelte like syntax internally.


Feature/switch reasonml textmate grammar to sane reasonml grammar

This PR switches the grammar currently being used to the sane reasonml grammar
The reason being (excerpt from the new grammar's repo)

This is based on the grammar included in OCaml and Reason IDE, which unfortunately is hand-tuned in a rather haphazard way to be "optimized" for a few of the original author's preferred themes, to the detriment of everything else. This fork attempts to provide a more coherent grammar that properly distinguishes different tokens so that they can be properly selected for highlighting. In effect, this should provide a better default for most themes (except, of course, the select few the original grammar was hand-tuned for) and moves the repsonsibility of optimizing highlighting for specific languages from the grammar to each theme, where it should be to remain maintainable (and quite frankly just plain sane).

Having tested it out locally this seems to provide more regular and consistent highlighting, open to discussion though if any reason users disagree 👍

Sounds good to me, @Akin909 ! Thank you! I had a few issues with the previous highlighting for Reason too 👍


Document and describe types of deployments Azavea commonly engages in

Different types of projects are naturally going to be better or worse for management through k8s. For example (and looking to the clear cut cases at the extremes), it is likely unwise to manage a static application with k8s whereas a process with heavy asynchronous workflows and the need for a distributed set of resources (zookeeper, Kafka, etc) is more likely to benefit from deployment through k8s. The first step is to describe the deployment-scenarios we actually engage in.
So: what types of deployment do we do at Azavea?

I've put me thoughts in this Google Doc: https://docs.google.com/document/d/18FRR6xiFEOks_l2qP63dtgrVFh2ZPKxqFgCr_gGY4ic/edit?usp=sharing. It's far from complete, hope I can elicit additional feedback and ideas to incorporate.

@rajadain Can we close this out now, even if the document isn't complete?  It's increasingly clear that k8s is not going to provide a one-size-fits-all solution for our common deployments.  At least not yet.  We can revisit in the future.

Agreed. Closing for now.