Category dropdown default

Issue description:
I have a theme feature where the user can set four featured posts based on category, using the dropdown 'choices' => Kirki_Helper::get_terms( 'category' ). I am running a post query based on what is selected is in the dropdown. I don't want to set a default for this field as it's unknown and based on unique categories the user has in their blog.
The dropdowns initially all display a category, but the post query doesn't work until the user selects a different cat in the dropdown. If they want the initially displayed category, they have to first select a different category and then select the initial category again.
This feels like quite bad UX and it'd be nice to either have the option for a placeholder in the dropdown, e.g. 'Select a category'. Is this something that is possible or do I need to approach this in a different way?
Code to reproduce the issue (config + field(s))
$fields[] = array(
	'settings'    => 'feat_cat_1',
	'type'        => 'select',
	'label'       => __( 'Choose a category', 'brio-helper' ),
	'section'     => 'featured_cat_posts',
	'default'     => '',
	'choices'     => Kirki_Helper::get_terms( 'category' ),
);

function shortened for brevity
if (!function_exists('get_cat_post')) {
	function get_cat_post($category) {

		$cat_query = new WP_Query( array(
			'post_type'				=> 'post',
			'showposts'				=> 1,
			'ignore_sticky_posts'	=> true,
			'cat'					=> $category,
			)
		);

		while ( $cat_query->have_posts() ): $cat_query->the_post();

<?php echo get_cat_name($category); ?>

				// some other stuff
		<?php
		endwhile; wp_reset_query();
	}
}

if (!function_exists('brio_featured_cat_post')) {
	function brio_featured_cat_post() {

		$post_cat_1 = get_theme_mod('feat_cat_1');
		$post_cat_2 = get_theme_mod('feat_cat_2');
		$post_cat_3 = get_theme_mod('feat_cat_3');
		$post_cat_4 = get_theme_mod('feat_cat_4');

		?>

			<?php if($post_cat_1) { get_cat_post($post_cat_1); } ?>
			<?php if($post_cat_2) { get_cat_post($post_cat_2); } ?>
			<?php if($post_cat_3) { get_cat_post($post_cat_3); } ?>
			<?php if($post_cat_4) { get_cat_post($post_cat_4); } ?>

	<?php
	}
	add_action('brio_content_before','brio_featured_cat_post',5);
}

I had a similar problem but for the default I just picked an arbitrary value from my values. If you don't know what any of the categories are going to be you could create a function to pick the first one from the list and return it.
The array of defaults is only the keys, not the keys and the values. Like
'default' => array(
'option-1',
'option-2',
),
so maybe you could do :
$defaults = array_keys(Kirki_Helper::get_terms( 'category' ));
$default = $defaults[0];

Adding as a feature request for a placeholder.

Same issue here, with a featured category that ideally should have nothing selected so that it displays the latest posts from all categories if a specific category is not selected.

On output you can already do whatever you want...
You just need to check the value first before getting the posts.
Example:
$args = array(
	'posts_per_page'   => 5,
	'category'         => '',
	'orderby'          => 'date',
	'order'            => 'DESC',
	'post_status'      => 'publish',
);
$selected_category = get_theme_mod( 'my_category_setting' );
if ( $selected_category && null !== get_category( $selected_category ) ) {
	$args['category'] = $selected_category;
}
$posts_array = get_posts( $args );

The above example sets some default arguments.
Then we check if a category is set in the select. If there is a selection and the category exists, we change the category argument to the category selected.
Then, those arguments are passed-on to the get_posts function (or WP_Query or whatever you want to use) and we get the posts.

Fixed, will be included in v3.0.21.
select controls now have a placeholder argument you can use.
See example: https://github.com/aristath/kirki/blob/12c85c3fe0a49e830873f1e983d283c57852d217/example.php#L719-L736

:+1:

Is the placeholder option going to work with the dropdown for sidebars as well (the code that you gave me)?

the placeholder will work for all select controls, regardless of what options those controls contain. So yes, for the sidebars one as well - provided of course you add the placeholder argument.
If you want it empty by default, then set 'default' = > '' too 😉

Sweet 👍

When do you recon 3.0.21 will be out?

-5 minutes

:D, brilliant. Working great with placeholder.


Move file using react-native-fs

I'm using react-native-fs as recommended in the readme, and I'm trying to move the file selected from the document picker to the Document Directory.
However, I get the error below:
Error: “filename.mp3” couldn’t be moved to “Documents” because either the former doesn't exist, or the folder containing the latter doesn't exist.

What am I doing wrong?
FYI, I'm using iOS.
openDocumentPicker() {
  DocumentPicker.show({
    filetype: ['public.audio'],
  },(error,url) => {
    console.log(url);
    this.saveAudio(url);
  });

}

saveAudio(url) {
  const split = url.split('/');
  const name = split.pop();
  const inbox = split.pop();
  const realPath = `${RNFS.TemporaryDirectoryPath}${inbox}/${name}`;
  var destPath = RNFS.DocumentDirectoryPath + '/' + 'name';

  RNFS.moveFile(realPath, destPath)
    .then((success) => {
      console.log('file moved!');
    })
    .catch((err) => {
      console.log("Error: " + err.message);
    });
}

Believe I found the cause. https://github.com/Elyx0/react-native-document-picker/issues/25 and https://github.com/Elyx0/react-native-document-picker/issues/34 explain it. I needed to decode it first.


Use heading for route calculation

Mapbox allows to pass a so called bearings object to the api, which works similarly to our heading parameter.
/13.43,52.51;13.42,52.5;13.43,52.5?bearings=60,45;;45,45
Every bearing contains of {angle, degrees}. Currently our core only allows to use angles and doesn't allow to specify the degrees (as in how big of a degree is allowed). Values can be skipped by passing empty ;;, but the number of bearings must match the number of waypoints.

Here we only need the first value after we know the last few points. For the other entries we could specify pass_through=true.

pass_through=true

Yes, but pass_through is a different story again :). The Mapbox API has a similar parameter called continue_straight. This is similar to pass_through. We can also add continue_straight :).
BTW: [continue_straight] "Defaults to  true for  mapbox/driving and  false for  mapbox/walking and  mapbox/cycling"

Ok. I just meant that we do not need or know heading for the via-points, only for the first point I think. (Except we do some more complicated analysis we could calculate a heading to deliver only on the right side of the road ;) if this is a preference of the driver)

Ok. I just meant that we do not need or know heading for the via-points, only for the first point I think.

Yes sure, but if we do it for one point, we can just do it for all points :). The core supports it and the Mapbox SDK supports it. Implementation wise I don't expect the workload to be different?

heading to deliver only on the right side of the road ;) if this is a preference of the driver

Actually Mapbox has a parameter for this.
approaches

A semicolon-separated list indicating the side of the road from which to approach waypoints in a requested route. Accepts  unrestricted (default, route can arrive at the waypoint from either side of the road) or  curb (route will arrive at the waypoint on the  driving_side of the region). ...


Add Kali Linux as a platform in the role metadata

🗣 Description
Update the role's metadata to list Kali as a platform with the latest release year as the supported version. The comment explaining the relationship between Kali Linux and Debian Bookworm is removed.

💭 Motivation and context
With the addition of Kali as a platform on Ansible Galaxy per https://github.com/ansible/galaxy/issues/3059#issuecomment-1513453476 (and verified against the API) it makes sense to update our role metadata.

🧪 Testing
Automated tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

I see the change logged here under the files changed tab for review changes; however, I don't notice the same via code/meta/main.yml where Kali linux is being made an option. Question is should Kali linux be an option there?

I don't follow what you are saying.  The addition of Kali Linux in meta/main.yml is the only change in this PR, but you seem to be saying that you cannot see it.  I think I am missing something.

Also, labels.yml file the kraken emoji image can be seen as a special character and cause processing error.

The labels.yml file is a UTF-8 file, and hence it can safely contain any valid UTF-8 characters (including emojis).

in the Ansible-lint file 2 commented links on line 2 and line 7 return 404 errors

Good catch, @dv4harr10!  That isn't something that really applies to the changes in this particular PR, though.  Could you create a new issue in this repository for this?

in the Ansible-lint file 2 commented links on line 2 and line 7 return 404 errors

Good catch, @dv4harr10! That isn't something that really applies to the changes in this particular PR, though. Could you create a new issue in this repository for this?

It looks like those actually come from skeleton-generic, so that's the repo the issue should probably be opened in

in the Ansible-lint file 2 commented links on line 2 and line 7 return 404 errors

Good catch, @dv4harr10! That isn't something that really applies to the changes in this particular PR, though. Could you create a new issue in this repository for this?

It looks like those actually come from skeleton-generic, so that's the repo the issue should probably be opened in

Ah, you are correct @jmorrowomni!

in the Ansible-lint file 2 commented links on line 2 and line 7 return 404 errors.  It appears that
https://ansible-lint.readthedocs.io/en/latest/configuring/ should be https://ansible-lint.readthedocs.io/configuring/
and,
https://ansible-lint.readthedocs.io/en/latest/rules/ should be https://ansible-lint.readthedocs.io/rules/

in the Ansible-lint file 2 commented links on line 2 and line 7 return 404 errors. It appears that
https://ansible-lint.readthedocs.io/en/latest/configuring/ should be https://ansible-lint.readthedocs.io/configuring/
and, https://ansible-lint.readthedocs.io/en/latest/rules/ should be https://ansible-lint.readthedocs.io/rules/

Please see cisagov/skeleton-generic#131 and cisagov/skeleton-generic#132.


/v1/bequeath-your-data-and-die :: triggers /v1/relay-server-replacement even /v1/deregister-application is not sent

Note: This is a minor problem occurs only during software upgrade of TAR. This would not affect normal working flow
In callbacks of /v1/bequeath-your-data-and-die, it is expected that if one callback fails, the next callback shall not be triggered.
But in TAR, software upgrade callbacks, PromptForBequeathingDataCausesTransferOfListOfAlreadyGrantedTypeApprovals is partially successful. So, the next callbacks are not expected to happen.
As expected PromptForBequeathingDataCausesRObeingRequestedToInquireForApplicationTypeApprovalsAtNewTAR, PromptForBequeathingDataCausesSubscriptionForDeregistrationNotifications, PromptForBequeathingDataCausesEndingSubscriptionsToOldRelease are not called. But PromptForBequeathingDataCausesRequestForBroadcastingInfoAboutServerReplacement is triggered which is not expected.
This need to be debugged and fixed.

Priority:Medium;
AffectedFlow:Software Upgrade/TAR:/v1/bequeath-your-data-and-die;

Working fine on manual testing


EntryPointNotFoundException: Unable to find an entry point named '{function}' in DLL '{MyC#32Bit}.dll'

Hi,
Got EntryPointNotFoundException while trying to load c# class 32 bit dll  in  a 64bit console application.
Please help to resolve this.
Thanks,

Hello @karthit ,
are you trying to load a .NET class library? LegacyWrapper is designed for loading native DLLs.
Kind regards
Franz


Options are stored on the model, this is bad when 2 views use diff options.

Example:
class SubView extends View {
   initialize() {
      // Custom selector.
      Validation.bind(this.model, { selector: 'data-validate' });
   }
}

class MyView extends View {
   initialize() {
      // Standard.
      Validation.bind(this.model);
   }

   onRender() {
     // 2 views sharing the same model.
     let subview = new SubView({model: this.model});
     ....
   }
}

This breaks the parent view, because the last .binds options are the ones that are used in all views bound to the same model.
This is pretty critical as I have a lot of subviews that are basically components.

I just spent hours debugging into the validation code to come to this same conclusion. If 3 different views all call
Validation.bind(this, {
  invalid: this.onInvalid.bind(this)
}

then the model becomes invalid, the same invalid function (whichever was last registered) will be called 3 times, but the correct view will be passed to it as the first parameter.
It really makes it hard to have a Marionette Behavior that reacts to validation issues. If a sub-region of a view has a Validation.bind then it prevents the parent from working correctly unless they share the same options.invalid and options.valid function.

The view bind design is just flawed. I looked into ways to fix it but would increase code complexity.
At the time i gave up and now i think is better to decouple things (do not store bound view to a model property)


Add Synthetic Data Generation Module

Description

Adds a suite of tools for interacting with LLM services. These LLM services are then used to build synthetic data generation tools and example pipelines following the Nemotron 340B Technical Report. The prompt templates used in the report are supplied as defaults throughout the code.
Usage

import nemo_curator as nc
from nemo_curator.synthetic import (
    AsyncNemotronGenerator,
    NemotronGenerator,
)
from openai import OpenAI, AsyncOpenAI

async def demo():
  openai_client = AsyncOpenAI(
      base_url="https://integrate.api.nvidia.com/v1",
      api_key="",
  )
  client = nc.AsyncOpenAIClient(openai_client)
  generator = AsyncNemotronGenerator(client)

  model = "google/gemma-2-27b-it"
  model_kwargs = {
      "top_p": 0.7,
      "max_tokens": 1024,
      "seed": 1234,
  }

  openlines = await generator.run_open_qa_pipeline(
      n_macro_topics=5,
      n_subtopics=3,
      n_openlines=3,
      n_revisions=2,
      model=model,
      base_model_kwargs=model_kwargs,
      conversion_model_kwargs=model_kwargs,
      ignore_conversion_failure=True,
  )

  dialogue = await generator.generate_dialogue(
      openline=openlines[0],
      user_model=model,
      assistant_model=model,
      user_model_kwargs=model_kwargs,
      assistant_model_kwargs=model_kwargs,
  )

  print(dialogue)

Checklist

[x] I am familiar with the Contributing Guide.
[x] New or Existing tests cover these changes.
[ ] The documentation is up to date with these changes.

Need to update docs still, but other than that should be good for a first round.


Notify user that job is running

A user should be aware that a job is running  such that the user does not think previous jobs have completed

This should also make sure that a user  does not prompt the LLM while a job is running. This can be done by temporarily blocking requests


MouseEventArgs and KeyEventArgs are not base types of EventArgs

This page says: "This example demonstrates how delegates can be used with methods that have parameters of a type that are base types of the delegate signature parameter type"
But neither MouseEventArgs or KeyEventArgs are base types of EventArgs, but the opposite is true.

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

ID: f38da104-ca1b-7cc2-15d6-f6486c818e60
Version Independent ID: 98d90b16-a4d0-d532-9fea-c51f6e262288
Content: Using Variance in Delegates (C#)
Content Source: docs/csharp/programming-guide/concepts/covariance-contravariance/using-variance-in-delegates.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

I think the page is correct: in the example, the method in question is:
private void MultiHandler(object sender, System.EventArgs e)

And the delegate signatures are:
public delegate void KeyEventHandler(object sender, KeyEventArgs e);
public delegate void MouseEventHandler(object sender, MouseEventArgs e);

So it's saying that EventArgs is the base type of KeyEventArgs and MouseEventArgs, which is correct.
Maybe the use of plural and singular is confusing? The text talks about "methods" and "delegate signature", but the example has a single method and multiple delegate signatures.


Troposphere and recovery feature

@jesdynf okay sir. This is all reviewed, smoke tested, tweaked slightly (requirements.txt and a few doc changes), and squashed. Please take a look and then once you're happy, I'm sure @danielehrlich will be kind enough to do some QA :)

@danielehrlich okay sir, it's up to you :). Please do some testing on this and then merge it

I'll look at all this tomorrow, good stuff (before GoT of course).
Rain is pretty bad, but our apartment is second floor!!
Sent from my iPhone

On Aug 26, 2017, at 10:57 PM, Matthew Vita<EMAIL_ADDRESS>wrote:
@danielehrlich okay sir, it's up to you :). Please do some testing on this and then merge it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Superceded by #66.


Configuring the ignoreAnnotations parameter is invalid

Describe the bug
esbuild: {
ignoreAnnotations: true
}
This will cause the comments for vue3 to be removed, Affect normal logic.
E.g. /* STABLE / / TEXT */
https://github.com/vitejs/vite/blob/9d7d45e56f98787cd6c7f4245ad5f9562d409997/packages/vite/src/node/optimizer/index.ts#L809
Reproduction

Steps to reproduce
No response
System Info
-

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Is this for optimize in build only? Looks like it's added in https://github.com/vitejs/vite/pull/8358 but I don't have the full context.
It would also be great if you can provide a repro here.

esbuild strips all comments except /* @__PURE__ */ or /* #__PURE__ */. --ignore-annotations only changes the behavior for these pure comments.
repro: https://esbuild.github.io/try/#dAAwLjE5LjMAAC8vIGNvbW1lbnQKY29uc3QgZm9vID0gJ2ZvbycKCi8qIGNvbW1lbnQgKi8KY29uc3QgYmFyID0gJ2JhcicKCmNvbnN0IGJheiA9IC8qIEBfX1BVUkVfXyAqLygoKSA9PiAnJykoKQo
Removing pure comments won't cause problems other than not being tree-shaked.


Fix timeout error in WaitOnApplication

timer.After is a function who returns a new channel, this causes a problem because the timer is resetting in each iteration.
You can see a detailed explain in http://stackoverflow.com/questions/35036653/why-doesnt-this-golang-code-to-select-among-multiple-time-after-channels-work/35036775
Regards

.... strange this one wasn't been caught already! ...
LGTM .. @timoreimann?

@gambol99 maybe WaitOnApplication isn't so widely used. Or, it's just that the Marathon deployments out there are mostly stable and tend to not run into the timeout. :-)
(Coincidentally, I ran into a variation of this problem at work just a week ago.)
@ch3lo, thanks for your PR -- LGTM!

Ah, maybe one thing: Does this hint at a problem with our testing?

I think our test is flawed in the sense that the timeout is chosen smaller than the ticker period, so we end up never iterating through the for loop more than once. Passing in a timeout higher than the ticker period shows that the test (in master) runs infinitely.
My suggestion would be to extract the ticker value into a constant and update the two test cases such that we pick a timeout lower and higher than the ticker value for the happy and unhappy path, respectively (e.g., double and half the timeout).
@gambol99, WDYT?

Hi all, thanks for your prompt response.
The unit test did not fail because we need a timeout greater than the ticker (500ms). So, I wrote a unit test that fails if the deploy takes more time than the specified in the timeout parameter.
What do you think?

Ah yeah, wrapping the method under test in a goroutine is probably a good idea. Could you do the same with the happy path test for consistency and safety reasons?
It's a bit hard to understand why we choose the respective timeouts. What do you think about extracting  the ticker timeout into a constant and referencing it in the tests as suggested in my previous comment?

I think we should also be able to use the configured wait time here: https://github.com/ch3lo/go-marathon/blob/38f690c0e9945069b194e37e4ef41c8068ee1ef8/group.go#L198

Could you please make sure you run gofmt over your PR? At least the new code in application_test.go does not seem to be formatted correctly.

Hey @ch3lo, looking pretty good now. Have one more tinyish usability concern, then I'm happy. :-)

Nice, thank you for your suggestions. Now I'm writing table driven tests in my internal projects and they look more clear.
Cheers

Carried the PR along in PR 202 due to inactivity.
Thanks for your contribution -- appreciated!


⚠ Contemporanea Group has degraded performance

In dc65454, Contemporanea Group ($SITE_CNTMG) experienced degraded performance:

HTTP code: 200
Response time: 5568 ms

Resolved: Contemporanea Group performance has improved in 1586d64.


Use this library in uiviewcontroller

Hi,
How can I use this library in uiviewcontroller not uitableviewcontroller? because I have two button fixed in bottom of screen but with uitableviewcontroller I can't use them.
Thanks,
Vafa

I'm working on that at the moment. There is another issue open with the same problem: https://github.com/enricmacias/ExpandableTableViewController/issues/8


Intellisense suggestions are awful. Unintuitive ordering, poor completions, and tons of irrelevant snippets.

This is literally a dupe of https://github.com/Microsoft/vscode/issues/17796
Not sure why it was closed but it describes the issue very well.
Is there anything the community can do to help with this effort?  I dont mind taking a stab if the contributors feel this is still an issue.

#17796 was closed as a duplicate of #16109 and #23643.
/duplicate of #17796

@weinand both of those are closed as well?
#16109 and #23643
16109 - The last reply is in 2016 and has the comment of Let's give it a try in January
23643 - The last reply from a contributor is discussing something about scoring
Should I just open up another issue since you closed this one? Can we as a community help contribute to this or does everything related to this just get auto closed without discussion?

Please identify specific cases where intellisense could be improved. Intellisense suggestions are awful is not actionable. What is "awful" and how do we measure when are no longer "awful"?

Can we as a community help contribute to this

Yes, you can help by contributing actionable information instead complaining.

Here is an example:

(not sure if it's clear, my cursor is just before the closing ) at the end of the line
It would make sense to have the variables on top, not a snippet. startswith should score higher than fuzzy_match. And extensions (plural) should be before extension since I have tab-completed it before, so it is more recent/frequent.


GTFS2 Broken after upgrade to Homeassistant 2024.5.

Describe the bug
after upgrading to 2024.5 i noticed some strange behavior of the Dashboard Cards which shows mu Bus station next to me.
I tried a restart but HA doesn't come up again.
So i deleted the GTFS2 out of custom components and delete the integration and all entitys.
Tried to Reinstall via HACKS which seems to work.
After tried to add the GTFS2 integration i got the Error below.
Release used
HA: 2024.5
GTFS2: <IP_ADDRESS>
Additional
Logger: homeassistant.setup
Quelle: setup.py:248
Erstmals aufgetreten: 09:54:29 (1 Vorkommnisse)
Zuletzt protokolliert: 09:54:29

Unable to set up dependencies of 'gtfs2'. Setup failed for dependencies: ['zone, person']

i found this breaking change in <IP_ADDRESS> https://github.com/home-assistant/core/pull/114484

Thanks for reporting it and I now see this too.


Remove deprecated classes

Fixnum and Bignum have been deprecated and are part of Integer as of ruby 2.4 https://bugs.ruby-lang.org/issues/12005
Monkey patches redis gem so its compatible with >= 2.1

Thanks for this :)
I only have a couple of comments:

Can you remove the .ruby-version file. I consider it similar to Gemfile.lock, belongs in source control for applications but not for libraries/gems.
Can you ensure the latest 2.2.x, 2.3.x and 2.4.x Ruby versions are listed in .travis.yml and tests pass? I believe it should fine work with 2.3.x, not sure about 2.2.x. If 2.2.x is a major pain to fix though, feel free to leave it out.
Please remove the version bump from this PR. I believe that's better to do outside of feature PRs.

Aside from my nitpicking, this looks good :)


ffuf cannot fuzzing https site that use old TLS/SSL

I tested "ffuf" on "Beep" lab machine on HTB. The lab use old TLSv1. And when:

I use ffuf with https, there is no results for both files and directories
I use ffuf with http, it found files + tons of directories with 302 code
I use ffuf with http, and filter status code 302. it found some files + directories but not enough
I use "wfuzz", "dir", "dirbuster", and "feroxbuster" it can find all the files as well as directories
When I check log file it says "tls: server selected unsupported protocol version 301"
→ Something wrong with ffuf when fuzzing https when server use old TLDs

How to fix these cases?

This is actually a side effect of Go > 1.9, as they deprecated the older TLS versions in the default HTTP client config.
The PR #671 should fix the issue by explicitly defining the minimum allowed version.


Add script for subdomains

Proposed changes
This add script for subdomains.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update docker image version of FIWARE GE.
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[ ] I have signed the CLA
[X] I have updated the change log (CHANGELOG.md)
[X] I send this pull request to the v0.19.0-next branch.
[ ] I have added tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

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

Coverage data is based on head (1f6e911) compared to base (514d48a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@              Coverage Diff               @@
##           v0.19.0-next      #222   +/-   ##
==============================================
  Coverage        100.00%   100.00%
==============================================
  Files                 2         2
  Lines              1745      1745
==============================================
  Hits               1745      1745

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

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


[HO] Week of March 22

[ ] Test out hypotheses of why the confounder is not working well:

Try larger number of confounded variables
Try more variables in the “right-hand-side” of the variables that are confounded
Deliverables: Commit #1: list out all the hypotheses being tested
Commit #2: Open an issue with qualms about presentation if deconfounder is actually problematic
Estimate:
Actual:

[ ] Weekly Meeting:

Estimate: 2 hours
Actual:

@hassanobeid1994 feel free to update the task list if you there is anything I missed and add time estimates.


Login Item does not show up anywhere visible

When enabling the "Start when I log in" option, typically we should see a reference to the app in Login Items of System Preferences. However, in HiddenBar's case, this does not happen.
Lingon X fails to see it, as does LaunchControl. Even osascript -e 'tell application "System Events" to get the name of every login item' from a bash shell does not list it. The only tool that I have found which enumerates the loginItem successfully is KnockKnock.
Not sure if this is because of some nonstandard plist in the LauncherApplication.app bundle?

Hi @luckman212, the API for adding the app to login items is now deprecated, so we are using this API for launching the app after login.
You can see more here: https://developer.apple.com/documentation/coreservices/launch_services

But I know that this feature does not work correctly sometimes. So we are trying to figure out the way to fix this problem. Any suggestions or PRs are welcome 👍

out

I want to work on this!

OK dude @L1cardo, feel free to work on this. Every contribution is worthy 🥇

@phucledien can I use cocoapods ?

@L1cardo sr dude, but we want to not use dependencies as much as possible. We just use it, if it really waste us so many efforts for implementing it. And basically, I think this feature does not need to use dependency. :D

OK I will give it a shot

@phucledien What am I going to use is Deprecated in macOS 10.11, but this will work.
Is that OK ?

Hm, I think you shouldn't use the deprecated at all, because I think they are deprecated for some reason. It would be better if you fix our current solution, or point out why it doesn't work sometimes. 🤔

The current method is not “not working”, but just invisible in the system preferences.
The current method is using an “applicationLauncher” to launch Hidden Bar. This can’t be seen in the system preferences
I know a cocoapods library  that can’t let login item visible in system preferences, you could have a look.

@L1cardo  ye, I took a look at it before. But because it using deprecated API so I don't want to use that. Hmm, what is the advantage of being in the login items, you think?

For me:

I can easily see what items automatically launch at login, and I can remove them without asking the permission of the App.
For the app:
it can make the size of App smaller (not having a launcher)
it tells users that the App is not doing some dirty work in the background of your Mac (visible in system preferences)
Old api (like you said, it uses an old api, so it may have problems in the future)

So, what to do is up to you. Or you can have a survey to know what others want.

Ye, I totally agree with ur perspective 👍. But with me, using deprecated API is the thing I want to stay away from. So I think, we should stick with the current method.

Ok  agree with you

So I will close this as there is no official way for doing this. Will reopen when Apple updates their API. Thank you guys 👍


Disable complex tests off main branches

cc @vincentschen Sounding like slicing integration tests will add too much overhead, so let's do this now. Adding complex test instructions to CONTRIBUTING.
Test plan
Manual verification of Travis stages

Integration tests running correctly: https://travis-ci.org/HazyResearch/snorkel/builds/558308494


placeholder get focus by default

Hi,
I use sweetAlert2 with input text , with a placeholder -
but by default, when the alert is pop - the input get focus so the placeholder don't do his job..
When I clicked on the title for example, the placeholder text shown..
Any solution or idea?
my code is:
swal({
title: 'Title?',
type: 'question',
showCancelButton: true,
confirmButtonText: 'Yes',
cancelButtobText: 'No',
input: 'text,
inputPlaceHolder: 'my place holder'
}). then(function(val){
//code..
})
Thanks alot

IE11, I guess?

https://stackoverflow.com/a/16138480/1331425

Yep. I have no another option except IE11 (because I work with closed company network)

I Injected the core.js already


Add gq to format text

Would it be possible to add the gq command (in both normal and visual mode) to format text?

This is a limitation in the codemirror vim implementation, see these issuse:

https://github.com/codemirror/CodeMirror/issues/5529
https://github.com/codemirror/CodeMirror/issues/3330


OP_PAIRCOMMIT

As ToByteVector is not used in other opcodes should we instead go with
uint256 hash = PairCommitHash(vch1, vch2);

popstack(stack);
popstack(stack);
stack.emplace_back(hash.begin(), hash.end());

? or maybe this that returns 24 bytes instead of 32
valtype hash = PairCommitHash(vch1, vch2);

popstack(stack);
popstack(stack);
stack.push_back(hash);

valtype PairCommitHash(Span<const unsigned char> x1, Span<const unsigned char> x2)
{
    // PAD is 0x00, 0x00, 0x00, 0x01 in little endian serializaton
    static const uint32_t PCPAD = 0x01000000u;

    HashWriter ss{HASHER_PAIRCOMMIT};
    ss << x1
       << x2
       << uint32_t(x1.size()) << PCPAD
       << uint32_t(x2.size()) << PCPAD;

    const uint256 hash = ss.GetSHA256();

    return valtype(hash.begin(), hash.end());
}

? or go all the way with passing a reference
valtype hash;

PairCommitHash(vch1, vch2, hash);

popstack(stack);
popstack(stack);
stack.push_back(hash);

void PairCommitHash(Span<const unsigned char> x1, Span<const unsigned char> x2, valtype& ret)
{
    // PAD is 0x00, 0x00, 0x00, 0x01 in little endian serializaton
    static const uint32_t PCPAD = 0x01000000u;

    HashWriter ss{HASHER_PAIRCOMMIT};
    ss << x1
       << x2
       << uint32_t(x1.size()) << PCPAD
       << uint32_t(x2.size()) << PCPAD;

    const uint256 hash = ss.GetSHA256();

    // valtype:clear() does nothing when empty
    ret.clear();
    ret.insert(ret.begin(), hash.begin(), hash.end());
}

?
Asking in regards to readability and efficiency: should probably minimize the number of times the hash is copied / passed around on stack or type converted so long it doesn't hurt code readability.
Interestingly can't get HashWriter to Finalize straight to a buffer.

Simplified implementation based on feedback from Anthony Towns (@ajtowns)

This PR will be kept as a reference for OP_PAIRCOMMIT after it is merged into lnhance-26.x for review only.


Cedar/IPC.c: Add hub release in NewIPC()

Many of you might have noticed that since the implementation of Wireguard, the server cannot exit normally sometimes.
It turns out that the issue is not directly related to Wireguard, but due to a hub reference not released in NewIPC(). It affects all protocols that use IPC.
#1307

Changes proposed in this pull request:

Fix hub release in NewIPC()

Awesome!
BTW, It is not related directly to this PR, CI for FreeBSD+LibreSSL is failing these days. I'll have a look at it.

BTW, It is not related directly to this PR, CI for FreeBSD+LibreSSL is failing these days. I'll have a look at it.

See #1415.

Understood, thanks!


Make sub module of Ecto logic?

Today the repo has an optional dependency on ecto because of Testcontainers.Ecto.
Could we make testcontainers_ecto module ?

Irrelevant due to mix task


Pinnacles Campground not supported

Hi,
I was trying to run camply against pinnacles campground. For some reason I could not get this to work.
I have used the campground ID "234015" from recreation.gov camply could not find it.
camply campsites     --rec-area 234015     --start-date 2023-07-10     --end-date 2023-07-18     --notifications silent     --search-forever
Is my approach wrong?
Thank you!

Hey @kekec, you're close. Pinnacles Campground has campground id 234015 and recreation area id 2893 (Pinnacles National Park, CA). You'd provide a --campground arg instead of a --rec-area arg here:
camply campsites --campground 234015 --start-date 2023-07-10 --end-date 2023-07-18 --search-forever

No worries! Good luck getting a spot in July 🏕


quamash along with asyncio doesn't seem to catch exceptions correctly

Hello,
I have this minimal code to reproduce the problem:
import asyncio
import sys

from PyQt5.QtWidgets import QApplication
from quamash import QEventLoop

def problem():
    app = QApplication(sys.argv)
    loop = QEventLoop(app)
    asyncio.set_event_loop(loop)

    loop.run_until_complete(conn())

async def conn():
    try:
        reader, writer = await asyncio.open_connection('<IP_ADDRESS>', 7777)
    except ConnectionRefusedError:
        print('exception?')

problem()

When the target server is down, it should catch the exception and print the text. But what actually happens is this error:
Event callback failed
Traceback (most recent call last):
  File "D:\Documents\Programming\venv\lib\site-packages\quamash\_windows.py", line 42, in _process_events
    value = callback(transferred, key, ov)
  File "D:\Programy\Python\Python36\lib\asyncio\windows_events.py", line 514, in finish_connect
    ov.getresult()
ConnectionRefusedError: [WinError 1225] The remote computer refused the network connection

I'm at a loss as to what I could do to fix this... when I tried using the default asyncio event loop instead, it worked just fine.
Thanks a lot for any help.
UPDATE: This issue only seems to happen on Windows; e.g. Linux normally prints 'exception?'.

I'll note that in Linux I got the desired exception? printed out.
 ⚙  ~/misc/expe/argoneus   master  python3.6 -m venv venv
~/misc/expe/argoneus   master  venv/bin/pip install quamash pyqt5
Collecting quamash
  Downloading Quamash-0.6.0-py3-none-any.whl
Collecting pyqt5
  Using cached PyQt5-5.10.1-5.10.1-cp35.cp36.cp37.cp38-abi3-manylinux1_x86_64.whl
Collecting sip<4.20,>=4.19.4 (from pyqt5)
  Using cached sip-4.19.8-cp36-cp36m-manylinux1_x86_64.whl
Installing collected packages: quamash, sip, pyqt5
Successfully installed pyqt5-5.10.1 quamash-0.6.0 sip-4.19.8
You are using pip version 9.0.1, however version 9.0.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
 ~/misc/expe/argoneus   master  venv/bin/python t.py
exception?

I get the same result as @argoneuscze in Windows.
PS C:\epc\t\812\testy> py -3.6 -m venv venv
PS C:\epc\t\812\testy> venv\scripts\pip install quamash pyqt5
Collecting quamash
  Downloading Quamash-0.6.0-py3-none-any.whl
Collecting pyqt5
  Using cached PyQt5-5.10.1-5.10.1-cp35.cp36.cp37.cp38-none-win_amd64.whl
Collecting sip<4.20,>=4.19.4 (from pyqt5)
  Using cached sip-4.19.8-cp36-none-win_amd64.whl
Installing collected packages: quamash, sip, pyqt5
Successfully installed pyqt5-5.10.1 quamash-0.6.0 sip-4.19.8
You are using pip version 9.0.1, however version 9.0.3 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' command.
PS C:\epc\t\812\testy> venv\scripts\python t.py
Event callback failed
Traceback (most recent call last):
  File "C:\epc\t\812\testy\venv\lib\site-packages\quamash\_windows.py", line 42, in _process_events
    value = callback(transferred, key, ov)
  File "C:\Program Files\Python36\lib\asyncio\windows_events.py", line 514, in finish_connect
    ov.getresult()
ConnectionRefusedError: [WinError 1225] The remote computer refused the network connection

Note that the program hung and did not return to the prompt.


System test

System test framework developed for every mec services.
Yaml files added directly into this directory for the different services to load at run time (added to AdvantEDGE and then removed)
Only location API currently tested.
Addition of new ones does not require any compile or modification to the main. Only addition of MECservice_test.go is enough to test a MEC service.
code change:
lots of files seem to have changed, but most of them are part of the go gis-client and platform-ctrl-client which were autogenerated. No changes in any of them were done.
rnis-client was changed because autogeneration of enums is not creating compilable code.
procedure to run system test :
deploy dep and core pods
go under test/system
issue go test and all the _test files will execute all the test cases necessary.

Also, please rebaseline to tip of develop branch.


Azure Monitor: Add JavaScript Source Map Support Article

Note to maintainers: Do not merge until actual feature is shipped! I'll remove the tag once everything is ready
/cc @DawgFan

This PR has been moved to the private repo and merged.
#please-close


Serenity/JS 3.0 - Public property cannot be named (TS 4029) in 3.0.0-rc.7 caused by #1099

Hi @jan-molak , I open a new issue for this to keep #1099 clean. I tried to update to 3.0.0-rc.7 to test the changes from this issue. After updating I got typescript error for a lot of my PageElements similar to the issue we had in https://github.com/serenity-js/serenity-js/issues/946:
Public property 'link' of exported class has or is using name 'QuestionStatement' from external module "C:/dev/git/cgm-ch-e2e-common/e2e-serenityjs-wdio/node_modules/@serenity-js/core/lib/screenplay/Question" but cannot be named.ts(4029)

This is again, because I use classes:
import { Question } from '@serenity-js/core';
import { By, PageElement } from '@serenity-js/web';

// when not in a class all is fine
const mainFeature = () =>
    PageElement.located(By.css('.xxx')).describedAs('list of features accessible for current user');

const link = (url: string) =>
    PageElement.located(By.css(`[href="${url}"]`))
        .of(mainFeature())
        .describedAs(`the feature url of ${url}`);

export class FeatureClass {
    mainFeature = () => PageElement.located(By.css('.xxx')).describedAs('list of features accessible for current user');

    // this throws the error when used in a class
    link = (url: string) =>
        PageElement.located(By.css(`[href="${url}"]`))
            .of(this.mainFeature())
            .describedAs(`the feature url of ${url}`);

    // would probably solve the problem
    linkWorking = (url: string): Question<Promise<PageElement>> =>
        PageElement.located(By.css(`[href="${url}"]`))
            .of(this.mainFeature())
            .describedAs(`the feature url of ${url}`);
}

So while importing Question and explicitly name the return type may solve the problem, you found another solution in #946 I think?

Ah, nice catch, thanks!

Okay, so I assume this was not intended and you'll fix it? So I'll wait before changing all my PageElement.of locators and before testing #1099?

Yeah, looks like either ts-node didn't pick it up, or this example is missing from the specs

OK, I can reproduce it in a test. It's actually not the fact that you're using classes, it's that the question gets exported.

3.0.0-rc.8 should address this issue

Had a first look and it seems to work now. Will continue to test #1099, I expect valid results next week. Thanks, Jan!


⚠ Bofrost.es has degraded performance

In d7affec, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1599 ms

Resolved: Bofrost.es performance has improved in 886931a after 6 minutes.


tests+gremlin: re-apply changes

@hunchback I posted some fixes as a diff in my latest comment of https://github.com/skydive-project/skydive/pull/794

run skydive-functional-tests-backend-elasticsearch

Can you please rebase on master and try with:
diff --git a/tests/tests.go b/tests/tests.go
index ed185841..ad097335 100644
--- a/tests/tests.go
+++ b/tests/tests.go
@@ -157,7 +157,7 @@ type Test struct {
 }

 func (c *TestContext) getWholeGraph(t *testing.T, at time.Time) string {
-       gremlin := g.G.Context(at).V()
+       gremlin := g.G.Context(at)

        switch helper.GraphOutputFormat {
        case "ascii":
@@ -165,7 +165,8 @@ func (c *TestContext) getWholeGraph(t *testing.T, at time.Time) string {
                header.Set("Accept", "vnd.graphviz")
                resp, err := c.gh.Request(gremlin, header)
                if err != nil {
-                       t.Fatal(err.Error())
+                       t.Error(err.Error())
+                       return ""
                }

                b, err := ioutil.ReadAll(resp.Body)
@@ -174,18 +175,20 @@ func (c *TestContext) getWholeGraph(t *testing.T, at time.Time) string {
                cmd := exec.Command("graph-easy", "--as_ascii")
                stdin, err := cmd.StdinPipe()
                if err != nil {
-                       t.Fatal(err.Error())
+                       t.Error(err.Error())
+                       return string(b)
                }

                if _, err = stdin.Write(b); err != nil {
-                       t.Fatal(err.Error())
+                       t.Error(err.Error())
+                       return string(b)
                }
                stdin.Write([]byte("\n"))
                stdin.Close()

                output, err := cmd.CombinedOutput()
                if err != nil {
-                       t.Fatal(err.Error())
+                       t.Error(err.Error())
                }

                return "\n" + string(output)
@@ -193,7 +196,7 @@ func (c *TestContext) getWholeGraph(t *testing.T, at time.Time) string {
        default:
                data, err := c.gh.QueryRaw(gremlin)
                if err != nil {
-                       t.Fatal(err.Error())
+                       t.Error(err.Error())
                }

                return string(data)

@lebauce - I have rebased and applied the proposed fix - please re-check.

@hunchback The CI still not pass and you have a conflict file : tests/tests.go

@lebauce can you check why these are failing:
08:20:20.949 ERROR: Step ‘Publish JUnit test result report’ failed: Test reports were found but none of them are new. Did leafNodes run?
08:20:20.949 For example, /var/lib/jenkins/workspace/skydive-functional-tests-backend-elasticsearch/tests.xml is 13 hr old
08:20:20.950
08:20:20.953 Setting status of 8e58ce8e9a415f2fc59ecf9eb18198855ceb5f6b to FAILURE with url https://ci.skydive.network/job/skydive-functional-tests-backend-elasticsearch/2121/ and message: 'Build finished. '
08:20:20.954 Using context: skydive-functional-tests-backend-elasticsearch
08:20:21.332 2018-03-07T09:20:21.266+0100	DEBUG	packet_injector/inject_packet.go:227 InjectPacket.func1	dpdk1: Injecting packet on interface omir-if1
08:20:21.333 2018-03-07T09:20:21.266+0100	WARN	packet_injector/inject_packet.go:231 InjectPacket.func1	dpdk1: Write error: no such device or address
08:20:21.421 2018-03-07T09:20:21.355+0100	DEBUG	ovsdb/ovs_of.go:333 launchContinuousOnSwitch.func1	dpdk1: Terminating command: [ovs-ofctl monitor br-omir watch:]
08:20:21.543 2018-03-07T09:20:21.477+0100	DEBUG	client/client.go:353 (*OnDemandProbeClient).onAPIWatcherEvent	dpdk1: New watcher event delete for 2cbf654c-d050-4b46-71ce-aed3f760d98a
08:20:21.545 --- PASS: TestOvsMirror (2.97s)
08:20:21.545 	tests.go:248: Removing existing captures
08:20:21.545 	tests.go:256: Creating captures
08:20:21.545 	tests.go:267: Executing setup commands
08:20:21.545 	tests.go:277: Checking captures are correctly set up
08:20:21.545 	tests.go:323: Executing settle function
08:20:21.545 	tests.go:341: Executing setup function
08:20:21.545 	tests.go:416: Running checks
08:20:21.545 	tests.go:445: Running tear down commands
08:20:21.545 	tests.go:459: Replaying test with time 2018-03-07 09:20:20.323928063 +0100 CET m=+380.251539473 (Unix:<PHONE_NUMBER>), startTime 2018-03-07 09:20:20.270390783 +0100 CET m=+380.198002175 (Unix:<PHONE_NUMBER>)
08:20:21.546 === RUN   TestK8sContainerNode
08:20:21.546 2018-03-07T09:20:21.480+0100	DEBUG	helper/helper.go:134 ExecCmds	dpdk1: Executing command [kubectl delete deployment skydive-test-container]
08:20:21.641 Finished: ABORTED

@hunchback I canceled the jobs because they were failing with:
-- FAIL: TestPcapInject (0.05s)
	tests.go:248: Removing existing captures
	tests.go:256: Creating captures
	tests.go:267: Executing setup commands
	tests.go:277: Checking captures are correctly set up
	tests.go:323: Executing settle function
	tests.go:341: Executing setup function
	tests.go:416: Running checks
panic: interface conversion: interface {} is int, not int64 [recovered]
	panic: interface conversion: interface {} is int, not int64

@lebauce - I have have fixed using:
case int, int8, int16, int32, int64, uint, uint8, uint16, uint32, uint64:
+			s := fmt.Sprintf("%d", t)
+			if s == "-1" {
+				return q
+			}
+			newQ = newQ.appends(s)

@hunchback
The -1 timestamp is only there to check if the query has to be run against the history or not.
I'm really not a fan of the pattern "ignore an invalid parameter".

Ok, i will remove handling of -1 and update PR
On Wed, Mar 7, 2018, 11:47 Sylvain Baubeau<EMAIL_ADDRESS>wrote:

@hunchback https://github.com/hunchback
The -1 timestamp is only there to check if the query has to be run against
the history or not.
I'm really not a fan of the pattern "ignore an invalid parameter".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/skydive-project/skydive/pull/822#issuecomment-371082823,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACogIwMZHj-TM2ZIx3pfGo6z0KPXvUf_ks5tb6yOgaJpZM4SVPgY
.

@lebauce - all tests have passed :)


Module 1.1 assignment

Given what you have learned about Git, why would you want to use Git in your classroom?
I would like to use Git in my classroom when we start on programming assignments, so students can see each other's progress and what I send for them to do. In addition, I want to give students a working understanding of Git for their future academic and professional work.


Any interest in collect/collectFirst?

Two of my favorite utility functions from the Scala programming language are collect and collectFirst. I think these would make a great addition to the library.

collect -- a map+filter combo where, if the result of the mapping function is undefined, that element is filtered out:

radash.collect([{a: 1}, {b: 2}, {a: 3}], (el) => el.a) // result: [1, 3]

collectFirst -- same as above, but the first defined result is returned immediately:

radash.collectFirst([{a: 1}, {b: 2}, {a: 3}], (el) => el.a) // result: 1

They are essentially more generalized versions of filter and find, where you get the opportunity to transform the value(s) at the same time. collect ends up being a very nice alternative when you don't want to iterate twice but using reduce just feels like too much.
@rayepps would you be interested in a PR for these additions?

Sorry for the long delay @adamhamlin 🙃 These are interesting functions, we do haveselect which is a little similar to collect but I do see the difference.
If you're still interested in making a PR for these I'd be happy to take a look and bring them into radash.

@rayepps I never lost hope, no worries! And yep, I saw select but I think the semantics of collect are a little cleaner/easier to reason about, esp. since it only requires 1 function arg instead of 2. I also expect select would feel a little duplicative if you need to target the same nested property for both mapping and filtering.
But great, I'll go ahead and open a PR soon, thanks

Opened https://github.com/rayepps/radash/pull/327 👍


Fuzzy border around rounded group buttons

See the image above.  There's a strange barely visible fuzzy border around each button but if I use the widget inspector, that isn't part of the group button.

I figured it out, it's the unselected and selected default shadow.  I don't think there should be any shadowing by default.


[Question]: Unable to Delete Nodes from Property Graph Index

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
We are working with the PropertyGraphIndex and trying to delete nodes based on their properties. Below is a summary of the approach we're following:
1. Creating the Property Graph Index:
pg_index_chunk = PropertyGraphIndex(
    nodes=all_nodes,
    property_graph_store=SimplePropertyGraphStore(),
    show_progress=True
)

2. Loading the Saved Property Graph:
pg_index_chunk.storage_context.persist(persist_dir="./pg_storage")
pg_index_chunk = PropertyGraphIndex.from_existing(
    SimplePropertyGraphStore.from_persist_dir("./pg_storage"),
    show_progress=True
)

3. Retrieving and Deleting Nodes: We retrieve nodes based on properties and attempt to delete them:
delete_from_pg = []
nodes = SimplePropertyGraphStore.from_persist_dir("./pg_storage").get(properties={"file_name":'1_pgs_doc.pdf'})
print(nodes)
for i in nodes:
    delete_from_pg.append(i.id)
print(delete_from_pg)
SimplePropertyGraphStore().delete(ids=delete_from_pg)
pg_index_chunk.storage_context.persist(persist_dir="./pg_storage")

Issue:
We are able to retrieve the nodes successfully using the get function, but the delete function doesn't seem to remove the nodes from the property graph. After attempting to delete the nodes and persisting the context again, the nodes still appear to exist in the property graph index.
Question:
How can we successfully delete nodes from the PropertyGraphIndex?
Are we missing any steps in the deletion process or in get process?

@dosu
after executing that code now we are getting some key error for which logs are:
graph_store.delete(ids=delete_from_pg)
  File "/home/likhithrishi/miniconda3/envs/gen/lib/python3.12/site-packages/llama_index/core/graph_stores/simple_labelled.py", line 149, in delete
    triplets = self.get_triplets(
               ^^^^^^^^^^^^^^^^^^
  File "/home/likhithrishi/miniconda3/envs/gen/lib/python3.12/site-packages/llama_index/core/graph_stores/simple_labelled.py", line 70, in get_triplets
    triplets = self.graph.get_triplets()
               ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/likhithrishi/miniconda3/envs/gen/lib/python3.12/site-packages/llama_index/core/graph_stores/types.py", line 152, in get_triplets
    self.nodes[subj],
    ~~~~~~~~~~^^^^^^
KeyError: 'aa3b866f-f990-466f-9906-ddd0c1bf6584'


"AZUGA" is listed in the drop-down menu

Description:
"AZUGA" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "AZUGA" in the search bar.
Expected results
The location is listed in the drop-down menu as "Oraș AZUGA, PRAHOVA (AZUGA)".


How ot use it with VueJS ?

Hi,
Is there a way to use this package with a VueJS Laravel application ?
My form is a VueJS component so I can't use the blade component to generate the hidden input.
How can I do it in VueJS ?
regards

i have the same question, is this possible?


modify pull to make sure no same layers such as 0 layer

when i test function Pull, there is something wrong.
failed to perform "verifyStateHelper"
the reason is that there are two or more 0 layer in a images's layers

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.梁晓雷 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

wrong


Introduce -n/--nostop switch so mutiple sessions can be run

Background
I'm using MicroOS Desktop as my daily driver and so am running toolbox pretty much as my main 'work shell' for all my day to day ops, with a custom image based on the official toolbox image with just a few extras I need (osc, zsh, etc)
My daily workflow has been as follows
Start at least 2 copies of gnome-terminal
Run toolbox -u in each gnome-terminal
sudo zsh in at least one of those toolboxes
The sudo zsh (inside toolbox -u) shell ends up being my osc environment for staging/Factory release management, using different OBS auth than my regular toolbox -u shell
The problem is, if I exit ANY of my toolbox -u sessions, they all die, because the container gets stopped on exit as part of the cleanup trap.
This is really annoying, and so, I wanted a fix
TL;DR / The Fix
This PR introduces a -n / --nostop switch for toolbox which prevents the container from getting cleaned up on exit.
The tmp_user_setup script cleanup still happens, but the container remains running
This means toolbox -u -n can be called repeatedly, at the same time, and all of those sessions will remain valid and work even as they are closed down.
Personally, I'd like this behaviour as the default, but figured I'd start with this change and see how it's accepted.
This has been tested..it's the code I'm actually running myself for my daily work with toolbox.

FWIW, I like this change quite a bit, and I think it's a big improvement in the toolbox experience (and I'm also using toolbox as my main environment since, well, more than a year).
I also would be in favor of making it the default behavior, at least for user (i.e., the -u ones) toolboxes, but it's indeed best to do things in steps.


Google blocked draw when i integrate it with google doc

This app tried to access sensitive info in your Google Account. To keep your account safe, Google blocked this access.

how to resolve it

There is nothing to add from https://github.com/jgraph/drawio/issues/4589 and still nothing we can do about it.


When -Severity is specified, only rules with the specified severity should run

We need to do some preprocessing in the engine before calling each rule. Current workflow is to filter all the returned DiagnosticResult based on the severity level.

Shouldn't that be "only run rules with specified severity (or higher) should run"?

Hi @rkeithhill, our current implementation is to only run the specified ones so if you input ’warning', only warning levels will run (not the error ones). If you want both warning and error, you do -Severity Error, Warning.
Thank you for your suggestion, it makes a lot of sense to include rules with specified severity or higher. If that will be a more useful scenario, should we rename the parameter as "MinimumSeverity"?

Oh I see. Yeah, while -Severity provides lots of flexibility, I think the far more common case would be satisfied by a -MinimumVersion parameter that operates as you suggest it would.

I agree with you, @rkeithhill. If we take this, current plan is to create a new parameter called -MinimumSeverity which has the new behavior. However, we'll keep -Severity to avoid breaking changes.

Fixed in  31de911


Make iostreams component work locally

Either conditionally make it work with the local runtime, or add another implementation that always works locally.

Is this issue still actual ?

Is this issue still actual ?

Yes! The question would be whether a simple mapping hpx::cout --> std::cout would already do the trick. I.e. something like a simple conditional
namespace hpx {
    using cout = std::cout;
}


Add 3.2.2-pshopify11

This promotes 3.2.2-pshopify10-experimental1, which we've been running on Core, to 3.2.2-pshopify11. These are good changes, so we should use them everywhere.
Diff

https://github.com/ruby/ruby/pull/7298
https://github.com/ruby/ruby/pull/8194
https://github.com/ruby/ruby/pull/7740
https://github.com/ruby/ruby/pull/8197

https://github.com/shopify/ruby/compare/v3.2.2-pshopify10...v3.2.2-pshopify11

Oh, he's on vacation.

Oh, he's on vacation.


Added Cosmos Db nested Support for array type of attributes

Why make this change?
Closes #1628
Closes #1624
If document contains Array type of attribute. Filtering using that Array is not possible right now.
What is this change?
As part of this PR, adding support to filter for array type also.
How was this tested?


[add] mirrors for rutracker plugin

Motivation for changes:
Detailed changes:

Checking available domain from list

Config usage if relevant (new plugin or updated schema):

Nothing changed

Are you right. Fixed.


Correct group size calculation in SizeClasses to save memory

Motivation:
In SizeClasses constructor, the group size calculation in short[group * 4][7] sizeClasses is wrong, which will create unnecessary array cells. For example:
If trunk size is 4 MiB (1 << 22), the group size was 19, but it should be 17.
Modification:
Correct group size calculation in SizeClasses.
Result:
Reduce group size in sizeClasses to save memory.

I think there's a same PR https://github.com/netty/netty/pull/13081

I think there's a same PR #13081

Yeah, seems we have the same thought on this.

Thanks!


Tuya on new HA update

Hi, today I updated HA to 2022.10.5 and all my tuya devices connected with official integration were erased. Does anyone have the same issue?

I'm Having a similar issue mine show in home assistant but they are all unavailable so I'm rolling back to the previous version till this has been resolved bc most my stuff is in the tuya platform

Did work for you @fordemon92? I restored my instance but it keeps with the issue

@kevvar no same here but I did go in and click see known issues apparently it sounds like our api key or part of it has ran out and we have to request another 6 months from tuya to get it back and could take them a few days to get us back in but the guide was under the known issues popup like the 4th one down I believe I'm really new to all this so sorry for not being better help

Dont worry budy I'm new in HA too, thanks!

some solution??

some solution??

This work for me #836

some solution??

This worked for me #836 I'm thinking seriously change my tuya devices, doing this things doesn't have any sense


atoma-vllm: Refactor SyncPhysicalTokenBlock to be a simple PhysicalTokenBlock

We might not need to wrap a PhysicalTokenBlock into a Arc<RwLock<PhysicalTokenBlock>>

Deprecated.
See https://github.com/atoma-network/atoma-infer


Fixing Description of GCloud instance Creation

I folllowed all the steps described in README.
gcloud compute instances create ipy-predict \
  --image https://www.googleapis.com/compute/v1/projects/google-containers/global/images/container-vm-v20140522 \
  --zone=us-central1-a --machine-type n1-standard-1 --scopes storage-ro bigquery

This will give an error for not finding the project name.  Then I supply the project name I created earlier on the dashboard, I call it world-cup-1 on my dashboard. Then,
Then I supply the project name using --project=world-cup-1, this time I get
ERROR: (gcloud.compute.instances.create) some requests did not succeed:
 - Project projects/world-cup-1 cannot accept requests to insert while in an inactive billing state.  Billing state may take several minutes to update.

I thought I could evaluate this without enabling billing..
I understand this project is a commercial of sorts for Google Cloud services. But all I am  interested in is seeing the machine learning data and code; can you find a way to share this data as pure CSV files, and show the awesome BigQuery code that created it just as documentation maybe so we can gawk at it seperately?
Maybe a better way is having seperate instructions for executing BQ code on development machines through SDK, so we can at leasty dump the file on disk without the need to creating instances on GC.

It is a bad user experience when you block users because of billing info outdated  , in the same time  this user  is seeing at the top of the page :
You have $300.00 in credit and 271 days left of your free trial


customization to have Map, BigNumber - follow up

Following https://github.com/streamich/json-joy/issues/777
class MyEncoder extends JsonEncoder {
encodeUnknown(value) {
if (typeof value === 'bigint') {
return String(value);
}
return super.encodeUnknown(value);
}
}
I could not find any method named encodeUnknown in your repo : https://github.com/search?q=repo%3Astreamich%2Fjson-joy+encodeUnknown&type=code
Not sure what I am doing wrong and there is no encodeUnknown method to extend  from JsonEncoder imported from
import {JsonEncoder} from<EMAIL_ADDRESS>(I was not able to follow  https://jsonjoy.com/libs/json-joy-js/getting-started to run them)
Any help appreciated, happy Xmass and new year and thx in advance !

I found out this is writeUnknown( to be used


bump protobuf module

There are three more vulnerabilites in dependencies. Bump them here as well?

Actually the problem is the setup-go which installs 1.21.7 while 1.21.8 is available. So we may need to bypass CI checks until setup-go is fixed.

Upstream issue: https://github.com/actions/setup-go/issues/462

I was unable to create a backport for 1.21. @techknowlogick, please send one manually. :tea:
go run ./contrib/backport 29617
...  // fix git conflicts if any
go run ./contrib/backport --continue


improve public page

Forgot to add a screenshot of the download button:

Codecov Report

Merging #2945 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #2945      +/-   ##
==========================================
+ Coverage   95.18%   95.19%   +<.01%
==========================================
  Files          73       73
  Lines       14033    14034       +1
==========================================
+ Hits        13358    13359       +1
  Misses        675      675

Impacted Files
Coverage Δ

qiita_db/util.py
90.73% <100%> (ø)
:arrow_up:

qiita_db/test/test_util.py
99.71% <100%> (ø)
:arrow_up:

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 449a643...a7423e2. Read the comment docs.


Image focal point isn't shown when editing images

Issue Summary
When editing an image that has a focal point it's impossible to see where this focal point is. Your only option is to create a new one.
Steps to Reproduce

Add an image
Add a focal point to image
Navigate away from image library
Edit the image
Try and see where the focal point is

Version

Wagtail 2.0.1

Seems this was actually due to some django form overrides we'd done to make django forms use BEM formatting. Once these were removed it works great again, sorry for the false alarm!

Managed to work this out so it was still possible to use django form overrides, we just needed to include the {{ widget.attrs.class }} tag inside the input's class parameter.
Seems the focal point uses four hidden inputs to store the x/y coordinates with some jquery to populate them, it uses the class to target these inputs instead of the id (bit weird, but hey, it works…).
All sorted in the end!


Update ssh configuration for aws nodegroup

Based on issue and discussions here: https://github.com/kubeflow/kubeflow/issues/3359

This change is

/assign
/ok-to-test
/lgtm
/approve

@googlebot re-check

@mbaijal Please would you sign the CLA as requested by the bot. If you've already signed it, please check that the email address you used to sign the CLA is the same as the one used in this commit. If they're different, please sign the CLA with the email addressed used in this PR.
After signing the CLA, please add a comment on this PR - that will cause the bot to run the CLA check again. :)

Thanks! LGTM!

@mbaijal Please would you verify the CLA as described in earlier comments.

@Jeffwan is this change important to get the AWS setup working properly? If so, would you like to create another PR to apply the fix?
@mbaijal thanks so much for this update. Because of the lack of response, I think it's best to close this PR and push the change through via another PR. I'll come back and close the PR early next week. Please do comment here if you're able to complete the CLA before then.

I'm closing this PR due to lack of response. Please do re-open it if you'd like to continue working on it.
/close

I'm closing this PR due to lack of response. Please do re-open it if you'd like to continue working on it.
/close

Thanks. I can make the changes in separate PRs. For user who customize their cluster, this will be helpful.


Add Animated.ValueXY.extractOffset

Change
The extractOffset method is previously implemented on Animated.Value, along with setOffset and flattenOffset. Both of the latter methods are proxied on Animated.ValueXY, but extractOffset is not. This commit adds the missing method.
Rationale
When creating draggable elements in XY space, in order to compensate for previous gestures' offsets, we currently need to do something like the following at the beginning or end of each gesture:
xy.setOffset(xy.__getValue());
xy.setValue({x: 0, y: 0});

After this commit, the API is equivalent to using a plain Animated.Value:
xy.extractOffset();

Makes sense to me. Thanks for the PR!
cc @janicduplessis
@facebook-github-bot shipit


Mtcnn accuracy problem

Thanks for your work.
I have carefully followed your build instructions and when I run the TensorRT mtcnn face detector on my jetson nano, I am getting low accuracy as like below
code detecting my hand as a face (continuously for some seconds ).
please advise.

I think my TensorRT MTCNN result should be very close to the original Caffe model.
I did not train the original Caffe MTCNN model by myself.  If you'd like to improve its recall/precision, please look into how to re-train or fine tune it.

Hello, Saddam. You can also change values of MINSIZE for faces detected.  By default, it uses 40 minzise in utils/mtcnn. Try changing it to a higher minsize!

@jkjung-avt
thanks for your advice
seems it works fine when I change the minsize from 40 to 100 pixels.
thank you.

Thanks to @julayelei


remove . from branch names

I want to re-request the feature already described in https://github.com/dependabot/feedback/issues/627
we still have the need for branch names which only contain characters allowed in domain names (afaik the only invalid char atm is the .) for use with our deployment pipeline.
as can be seen in https://github.com/dependabot/feedback/issues/627  this is a need several users have.
since the feedback repo was closed I figured we need a new issue in this repo here, so the conversation is not dropped.
https://docs.github.com/en/code-security/supply-chain-security/configuration-options-for-dependency-updates#pull-request-branch-nameseparator is not enough because it still leaves . chars in the dependabot branch names, e.g. dependabot/npm_and_yarn/nouislider-14.6.4

Is there any update on this? We have been battling with this for some time and the dependabot PR's are not useful as the domains that get setup based on the branch name are invalid (TLS issues as well as Github Environments we cant edit them beforehand)

Still waiting for / to be removed from the auto-generated branch names. Or at least to have some configuration on what not to write in the branch name


Resources 0 of 0

I'm new to this tool, but when I cloned the repo, I made sure all dependencies were installed and up to date using an Ubuntu Linux subsystem on windows 10 and executing from vs code.  The az2tf runs for the appropriate subscription and resource group, identifying that it has found both, but continually comes up with no resources.  So the output I get contains an empty azurerm_resource_group__{resource group name}.tf that only has the resource group TF implementation.
When I check the TF State, it also shows only the resource group with no other resources.  I thought this might be a permissions issue but I am an owner on the subscription and am able to run the commands from the various SH files and get back results.
One other point is that I am trying to execute this against Azure Government, and made sure that my Azure CLI context is logged in to Azure Government first.  Like I said it still finds the appropriate subscription and resource group, but is failing to find any of the resources in that resource group.

I doubt this tool will work on Azure Government - that has some differences compared to public Cloud.
I have no access to a Government Cloud setup to test it.
Early on in the process az2tf calls out to resources.azure.com to get a list of things it needs to do - I doubt Azure Government resources appear there, so it is going to run into trouble early on.
I have also never tested this tool on the Linux subsystem - it aught to work in theory.


Fixed XSP profile deployment failure

The XSP profile was failing to deploy any compute nodes.  This turned out to be because DellComputeCount in dell-environment.yaml was not being updated.
This patch makes the deployer always update that count regardless of the profile being deployed.

Fixes: CES-9485


#87440 should be open for discussion

#87440 should be open for discussion because that issue says something about "want to be transparent" and "not censoring".
It is unclear how you decide if any "single person being offended" is a real offended person or just troll or smth else? Would you believe me if I say that I'm offended by letter V?
New acc for obvious reasons
Does this issue occur when all extensions are disabled?: Yes

It's just laughable...
They're as "transparent" and "not censoring" as North Korea.

The person who raised the issue #87268, @Christian-Schiffer (From the medieval Latin name Christianus meaning "a Christian"), deleted his github account. There's a high possibility of this being trolling that got out of hand.

Issue #87440 has been updated an unlocked, opened for discussion.


Regarding the issue of video recording

Which version are you using?
v1.5.0
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[X] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
Description
Describe how to replicate the issue

I configured the recording through an API and found that even after I started recording, there was still no record file generated. I had to preview it before it could be generated. How can I achieve heating in the background recording without pulling the stream,Thanks
Did you attach the server logs?

yes
Did you attach a network dump?

no

This issue cannot be replicated without configuration, server logs and the sequence of API calls. Reopen the issue and provide such data.


Implement SlackTextViewController

Resource:

https://github.com/slackhq/SlackTextViewController

Resolve in pull request: https://github.com/EstefaniaGuardado/exampleXmpp/pull/22


Return short-term token in agent-regstration endpoint.

Fix: https://issues.redhat.com/browse/ACM-12277
In this PR, we change agent-registration to return a short-term bootstrapkubeconfig secret(by default 7 days). And new param in the URL to let user customize the expiration of the bootstrapkubeconfig.

/assign @qiujian16

/assign @zhiweiyin318

@zhujian7 @zhiweiyin318
PR rebased, please take another look, thanks!

/lgtm
/hold
/cc @zhiweiyin318

/unhold


Request: Publish New NPM Version

Hey @MaxArt2501, thanks for this module. For a 4year old package, it's held up pretty well! There's a fix (#2) that would be nice to get a 1.0.1 release of this package, if possible.
I know maintaining old packages can be kind of a drag. I'm happy to help if you'd like.

You know, you're right. This package definitely needs an update. I'll have a look at it these days.
(To be fair, I didn't expect it to be noticed back then when I published it - I just needed something like this for a project of mine.)

You know, you're right. This package definitely needs an update. I'll have a look at it these days.
(To be fair, I didn't expect it to be noticed back then when I published it - I just needed something like this for a project of mine.)


fix(formbutton): properly pass disabled prop

Pass disabled prop to <Form.Button /> without overwriting the context value

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Atom install failed on Windows 10 Techincal Preview

Hello,
I had older version of Atom editor and I wanted to update. Update first failed, so I uninstalled old version of Atom via Control Panel > Uninstall Programs. Then I was trying to do fresh install, but got same problem. I deleted these dirs:
C:\Users\Adomas\AppData\Local\SquirrelTemp
C:\Users\Adomas\AppData\Local\atom\
And tried install again, but got same problem. Here is last log:
Program: Starting Squirrel Updater: --install .
Program: Starting install, writing to C:\Users\Adomas\AppData\Local\SquirrelTemp
CheckForUpdateImpl: Failed to load local releases, starting from scratch: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\Adomas\AppData\Local\atom\packages\RELEASES'.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at Squirrel.Utility.LoadLocalReleases(String localReleaseFile)
   at Squirrel.UpdateManager.CheckForUpdateImpl.<CheckForUpdate>d__28.MoveNext()
CheckForUpdateImpl: Reading RELEASES file from C:\Users\Adomas\AppData\Local\SquirrelTemp
CheckForUpdateImpl: First run or local directory is corrupt, starting from scratch
ApplyReleasesImpl: Writing files to app directory: C:\Users\Adomas\AppData\Local\atom\app-0.186.0
IEnableLogger: Failed to update local releases file: System.IO.IOException: The file exists.

   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.__Error.WinIOError()
   at System.IO.Path.InternalGetTempFileName(Boolean checkHost)
   at Squirrel.ReleaseEntry.BuildReleasesFile(String releasePackagesDir)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<updateLocalReleasesFile>b__ec()
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<updateLocalReleasesFile>d__ed.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<LogIfThrows>d__4f`1.MoveNext()
Program: Starting Squirrel Updater: --install .
Program: Starting install, writing to C:\Users\Adomas\AppData\Local\SquirrelTemp
CheckForUpdateImpl: Failed to load local releases, starting from scratch: System.IO.FileNotFoundException: Could not find file 'C:\Users\Adomas\AppData\Local\atom\packages\RELEASES'.
File name: 'C:\Users\Adomas\AppData\Local\atom\packages\RELEASES'
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
   at Squirrel.Utility.LoadLocalReleases(String localReleaseFile)
   at Squirrel.UpdateManager.CheckForUpdateImpl.<CheckForUpdate>d__28.MoveNext()
CheckForUpdateImpl: Reading RELEASES file from C:\Users\Adomas\AppData\Local\SquirrelTemp
CheckForUpdateImpl: First run or local directory is corrupt, starting from scratch
ApplyReleasesImpl: Found partially applied release folder, killing it: C:\Users\Adomas\AppData\Local\atom\app-0.186.0
ApplyReleasesImpl: Writing files to app directory: C:\Users\Adomas\AppData\Local\atom\app-0.186.0
IEnableLogger: Failed to update local releases file: System.IO.IOException: The file exists.

   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.__Error.WinIOError()
   at System.IO.Path.InternalGetTempFileName(Boolean checkHost)
   at Squirrel.ReleaseEntry.BuildReleasesFile(String releasePackagesDir)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<updateLocalReleasesFile>b__ec()
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.UpdateManager.ApplyReleasesImpl.<updateLocalReleasesFile>d__ed.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Squirrel.Utility.<LogIfThrows>d__4f`1.MoveNext()

Since I deteled old installation I am unable to get Atom working again, it simply does not install anymore. Any ideas how to fix this? Is this because I am using Windows 10 Technical Preview?

Check out your c:/windows/temp, delete any leftover files there too.
Other than that, maybe it's a permissions error? First it can't find the file, then it can't start from scratch... Maybe you don't have write permissions for the folder involved?

This has nothing to do with Win10, @Zireael07 is correct. Delete all your files in %temp% and it'll work again.

Running CCleaner then trying the install again worked for me.

This happened to me after letting Windows 10 auto update me to latest preview.
Some apps (including Microsofts VS 2015 RC) failed after the upgrade, some still worked.
Re-installation of Atom (and a few other apps) failed despite performing the steps above.
It seems root cause is that the upgrade process messes with dir permissions and a lot of dirs (including some stuff in my home dir) become owned by "system installer". Doing anything in those dirs requires permission from system installer.
Recovery process is to reclaim ownership of those dirs and remove rights for "system installer". Its pretty time consuming finding all the stuff it broke and in hindsight I think I would have just re-installed.

I'm using Win 10 and had the same error installing. Mine is a refresh install of Win 10. Clearing the %temp% made the difference. Not sure why.

Verified that cleaning the %temp% dir fixed the issue

I have the same problem. Not I'm using sublime, but miss atom.

cleaning %temp% dir fixed the issue. running win 10 pro (Build 10240)

I tried all these solutions, nothing worked. Also I tried running the program from straight from the download folder, it worked, but there were lots of features missing. Not satisfied, I turned off my antivirus software and it installed like a charm.

Disabled the antivirus.. this fixed it

For me ; "Disabled the antivirus.. this fixed it".
Great ! Tks

Disable antivirus fix this problem.

Yeah!
Disabling antivirus and cleaning %temp% worked

what git we should download to download script in atom

github for windows is always recommended, comes with gui and cmd sysgit but git-scm

After uninstalling McAffee it worked, would be nice to have something that would detect it before installintg.

You don't have to uninstall your antivirus. Just disabled it while you install atom and then it will work. The problem is that atom is trying to modify files and that looks like a suspicious activity for any decent antivirus. Again suspend your antivirus for a bit and then install and presto.

Any ideas what I can do if I cant disable antivirus?

@davidayers Then you'll have to install the zipped version.

If you have McAfee, turn off "Real-Time Scanning" worked for me


Import pdf

For each page x in pdf, add the content of pdf as a new page.
Use images if easier, but later change it to get the actual vectors from the pdf.

getting non-picture information sounds really hard and probably not worth it

May be possible with https://mozilla.github.io/pdf.js/


feat: add EntityField transformation

Why
This is a first attempt at field transformation (right now we just do field validation). Note that this is currently broken due to the following case:

Entity contains LowerCaseStringField
loadManyByFieldEqualingMany('lowercaseField', 'THIS_WILL_BE_LOWERCASED') returns a map from value requested to entities matching that value. The problem is that the value requested is upper-case but the returned map is lower-cased.

How
Do transformation in addition to validation in the field.
Test Plan
Run all tests.

Abandoning, need for this has dwindled.


client: warn when handshake fails due to BADIP

other parts of the code show the meaningful error message too, but
not at the spot where it happened for me.

Thanks, merged


Delete non-reduced fuzz inputs

Can be "reproduced" with:
apt update
apt install wget -y
wget https://raw.githubusercontent.com/bitcoin-core/bitcoin-maintainer-tools/master/delete_nonreduced_fuzz_inputs.sh
bash delete_nonreduced_fuzz_inputs.sh

Can be "reproduced" with:
apt update
apt install wget -y
wget https://raw.githubusercontent.com/bitcoin-core/bitcoin-maintainer-tools/master/delete_nonreduced_fuzz_inputs.sh
bash delete_nonreduced_fuzz_inputs.sh

Github is completely baffled by the size of the diff :see_no_evil:

Github is completely baffled by the size of the diff :see_no_evil:

I ran this twice and the difference was 7000 files, which is expected, because our fuzz targets are not deterministic

I ran this twice and the difference was 7000 files, which is expected, because our fuzz targets are not deterministic

Would be nice if someone could test #44

I ran this and it seemed to achieve something similar to this PR; reducing the seed count by ~80'000:
# master @ 3038a21dd7d6dfb3574401b8c76e48ca87c44769
find . -type f | wc -l
241321

./ delete_nonreduced_fuzz_inputs.sh
# git log
commit f08f63d6121d794b18213f71ead508649055a493 (HEAD -> master)
Author: delete_nonreduced_inputs script<EMAIL_ADDRESS>Date:   Thu Jan 28 08:47:11 2021 +0000

    Reduced inputs for fuzzer,address,undefined,integer

commit 12116b05479ba817f0b9c98e00ae3ddf47f0fbe3
Author: delete_nonreduced_inputs script<EMAIL_ADDRESS>Date:   Thu Jan 28 07:42:43 2021 +0000

    Reduced inputs for fuzzer

commit c9e492d4d9c1281aeb107b027205f895b35d6ec9
Author: delete_nonreduced_inputs script<EMAIL_ADDRESS>Date:   Thu Jan 28 07:26:52 2021 +0000

    Delete fuzz inputs

commit 3038a21dd7d6dfb3574401b8c76e48ca87c44769 (origin/master, origin/HEAD)

# find . -type f | wc -l (my reduced branch)
163311

# find . -type f | wc -l ( this branch @ ae401aa90e5e3fa996b4066cb067f23a26e51a4d)
163242

Would be nice if someone could test #44

I ran this and it seemed to achieve something similar to this PR; reducing the seed count by ~80'000:
# master @ 3038a21dd7d6dfb3574401b8c76e48ca87c44769
find . -type f | wc -l
241321

./ delete_nonreduced_fuzz_inputs.sh
# git log
commit f08f63d6121d794b18213f71ead508649055a493 (HEAD -> master)
Author: delete_nonreduced_inputs script<EMAIL_ADDRESS>Date:   Thu Jan 28 08:47:11 2021 +0000

    Reduced inputs for fuzzer,address,undefined,integer

commit 12116b05479ba817f0b9c98e00ae3ddf47f0fbe3
Author: delete_nonreduced_inputs script<EMAIL_ADDRESS>Date:   Thu Jan 28 07:42:43 2021 +0000

    Reduced inputs for fuzzer

commit c9e492d4d9c1281aeb107b027205f895b35d6ec9
Author: delete_nonreduced_inputs script<EMAIL_ADDRESS>Date:   Thu Jan 28 07:26:52 2021 +0000

    Delete fuzz inputs

commit 3038a21dd7d6dfb3574401b8c76e48ca87c44769 (origin/master, origin/HEAD)

# find . -type f | wc -l (my reduced branch)
163311

# find . -type f | wc -l ( this branch @ ae401aa90e5e3fa996b4066cb067f23a26e51a4d)
163242

Ok, going to merge. If something went wrong, it is easy to revert.

Ok, going to merge. If something went wrong, it is easy to revert.

Post-merge ACK ae401aa90e5e3fa996b4066cb067f23a26e51a4d: nice! :)

Post-merge ACK ae401aa90e5e3fa996b4066cb067f23a26e51a4d: nice! :)


Add test folder to package source distribution

Is your feature request related to a problem? Please describe.
Pyrealm is distributed from PyPI without the tests included. This means that anyone who wants to build Pyrealm themselves doesn't have direct access to the tests to verify their build.
This is relevant to people and orgs (e.g. anaconda etc) who like to build their own packages.
Describe the solution you'd like
Add the tests to the source distribution

That seems sensible - I hadn't really looked at what is included in the source distribution. I explicitly included the pyrealm_build_data package in source, which is excluded from the binary distributions, but it doesn't include anything else needed to build and test the package, so the source distribution doesn't really stand alone.
It's not just tests, really - the docs should probably go in too?

Yes I'd agree documentation would be a useful addition.

@davidorme @MarionBWeinzierl, It was mentioned by one of our reviewers yesterday that they had tried to follow the worked examples in the Pyrealm notes but that the distribution lacks the build data that some of the worked examples use. They managed to figure it out by going direct to GitHub, but we should probably rectify this for v2.0
With your blessing I'll add this issue to the v2.0 milestone

There's a difference between including more of the code in the source distribution (tests and docs probably) and making it easier for users to follow along with the worked examples. I don't know how we do that - could they install the pyrealm_build_data package separately to be able to use that?

I've looked around a bit and I can't see an easy route for the package structure at the moment. I looked at the package extras functionality, thinking that we could do something like pip install pyrealm[data] but I think that only applies to extra package dependencies. You can include package data, but I don't think that is optional and I'd rather not bloat the standard binary install.
Perhaps we need to split pyrealm_build_data out into a standalone data package, but that complicates the docs etc. Or I've missed something else 😄

Could we add a sentence or two to the README/docs that says that for running the tests they have to download the data from the Github repository? That would not change anything about the package but give users the info to run the tests without having to do any digging around in the repository.

Can we create data releases in parallel to package releases? It would be good to be able to point to a zip file on Github or Zenodo. Maybe have some kind of manually triggered CI workflow that bundles the data somewhere?

Can we create data releases in parallel to package releases? It would be good to be able to point to a zip file on Github or Zenodo. Maybe have some kind of manually triggered CI workflow that bundles the data somewhere?

I think the simplest thing is to include the bare minimum data for the worked examples within the docs folder, and leave everything else in build data. Anything needed for the worked examples would then be included in the binaries. We can always trim these down to sensible size (fewer numbers and lower precision?) to they don't burden the package.
How does this sound?
A robust solution for the build data would be zenodo. We'd have to then create fixtures to download the data for each test session which would slow down local test turnaround.

I'm not sure that solves the issue. The binary package does not include the docs folder so users will be no more able to play along with the examples than at the moment. We could include some small data files as package data in the binary but then we constrain our examples in docs to smaller, less illustrative inputs and add size to the package.
I think the current setup of using the build data for regression testing and more illustrative testing works well, but the problem is really just distributing that data for interested users. As @MarionBWeinzierl says, we can just provide links to clone the files, but we could also maintain a separate Zenodo package of the data to make life easy. I don't think that should replace the pyrealm_build_data setup, just provide an easy download for users.

OK, that seems reasonable.

One thing we could easily (?) do is to add a step to our release CI that zips up pyrealm_build_data and adds it as an artifact to the release files. That way we get an updated single downloadable file associated with each release?

Of course, that's already included in the source zip and tar.gz, but it would be more user friendly. Zenodo would be better though.

OK, I'll reformat this issue as a packaging/distro/data meta issue and break out these tings as separate tasks


SvelteKit Auth Helper: Magic link with redirect link is double encoded & breaks functionality

Bug report

[x ] I confirm this is a bug with Supabase, not with my own application.
[ x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I use the sveltekit auth helper in the following form like seen below
<script>
import { Auth } from '@supabase/auth-ui-svelte';
	import { ThemeSupa } from '@supabase/auth-ui-shared';
</script>
<Auth
	supabaseClient={data.supabase}
	view="magic_link"
	redirectTo={`${data.url_origin}/logging-in?redirect=${encodeURIComponent(
		data.url_pathname + data.url_search + '&action=acceptinvite'
	)}`}
	showLinks={false}
	appearance={{
		theme: ThemeSupa,
		style: { input: 'color: #000' }
	}}
	localization={{
		variables: {
			magic_link: { button_label: 'Accept Invite & Signup via Magic Link' }
		}
	}}
/>

The issue is that the link I receive in the email looks like the following:
https://fvuwouathomgsbmfofxb.supabase.co/auth/v1/verify?redirect_to=http%253A%252F%252Flocalhost%253A5173%252Flogging-in%253Fredirect%253D%25252Fflats%25252F63%25252Finvite%25252F%25253Fflatid%25253D0cbdde88-e5f4-4986-9f15-6857ae1322b4%252526userid%25253De62b34d8-7765-47f2-8b80-2603c8716571%252526action%25253Dacceptinvite&token=pkce_7f9d2bb09a71906771d9b41f770a4ff4df78b37496ab28b4a68fc1ce&type=magiclink

When I open this link, I am redirected in by browser straight to (which is wrong)
http://localhost:5173/?code=de072970-fa20-4628-97fa-b85565642d3d
Now when I go on this page https://www.urldecoder.org/ and click decode 1x with the above url, I receive:
https://fvuwouathomgsbmfofxb.supabase.co/auth/v1/verify?redirect_to=http%3A%2F%2Flocalhost%3A5173%2Flogging-in%3Fredirect%3D%252Fflats%252F63%252Finvite%252F%253Fflatid%253D0cbdde88-e5f4-4986-9f15-6857ae1322b4%2526userid%253De62b34d8-7765-47f2-8b80-2603c8716571%2526action%253Dacceptinvite&token=pkce_7f9d2bb09a71906771d9b41f770a4ff4df78b37496ab28b4a68fc1ce&type=magiclink
This url works perfectly fine and redirects correctly!
When I remove encodeURIComponent in my code, the behavior is the same

EDIT: The redirectTo link that is send looks like this:
http://localhost:5173/logging-in?redirect=%2Fflats%2F63%2Finvite%2F%3Fflatid%3D0cbdde88-e5f4-4986-9f15-6857ae1322b4%26userid%3De62b34d8-7765-47f2-8b80-2603c8716571%26action%3Dacceptinvite
Expected behavior
The expected behavior is that the link in the mail essentially should contain the second link and not the first link.
System information

OS: macos
Browser (if applies) chrome
"@supabase/auth-helpers-sveltekit": "^0.10.0",
"@supabase/auth-ui-shared": "^0.1.6",
"@supabase/auth-ui-svelte": "^0.2.2",
"@supabase/supabase-js": "^2.24.0",
sveltekit 1.5.0
Version of Node.js: 18

Solved by support. Was related to a past encoding issue on Supabase`s side


typo and tweak to install instructions

@hosom noticed a typo and updated the install instructions to reflect changes in load.bro files.

Going to go ahead and finalize the pull, no concerns.


fix: autocomplete issue for mobile devices.

Fix for this issue: https://github.com/drac94/react-auth-code-input/issues/45

Ping.

Will this also fix the issue of using the one time code via the quick action above the keyboard in safari safari that pulls it from apple mail and only fills out the first input?

Same issue here, is there an interim fix ?

can we get this merged?

@drac94 review pls


Node2vec Library speed

I saw the result of the elapsed time of openNE(node2vec) are slower than the regular library. 1. Could you describe the result of why it will happen?
2. Can node2vec support multi-gpu

run?

I saw the source code, you just directly use the Node2vec library, so it does not support multi-gpu yet

I also find that in source codes. The title of this project makes me believe that they have supported node2vec with GPU. They may support it soon.

Hi,
Thanks for your discussion! We now support GPU in our new version (https://github.com/thunlp/OpenNE/tree/pytorch) and are currently working on multi-GPU. Please stay tuned!


feature:landing page creation js css

Please accept my changes.

Send me this PR again on branch "knobsobers-dev"


How to enable flakes by default?

Hey I would like to use nix to manage dependencies in a project I work on and I searching for a way to install nix with flakes enabled in a single command. When I ran the installer (macOS 11.2.3) I didn't get experimental features enabled. Is there a flag that would enable them by default?

That's more a question for upstream. There is: --nix-extra-conf-file: Path to nix.conf to prepend when installing /etc/nix/nix.conf

According to #20, flakes are now enabled by default

To clarify a bit further, https://github.com/cachix/install-nix-action (for GitHub Actions) has enabled flakes by default now. If you are using the installer here outside of GitHub Actions or without cachix/install-nix-action, you'll still need to supply the --nix-extra-conf-file option like @zimbatm noted above


fix Wal file should be removed from _wal_path_map when renaming it to tmp directory

Proposed changes
Just like delete wal file, renaming file also need to remove wal from _wal_path_map
Issue Number: close #xxx

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

run buildall