Some feeds not showing up

I have a feed that I follow that I haven't seen any updates for a while: http://backcomic.com/rss.xml
If you look at the feed, it's been updated fairly often, and if I look at the feeds view in Stringer, it shows up with the most recent date:

But I haven't actually seen any of the last 15 or so entries. Any idea what's going on?

Hmm, it actually does validate on https://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fbackcomic.com%2Frss.xml

@mockdeep is this still a problem?

@gabrielpoca I haven't seen an update from that feed in quite a while, but it looks like they might have changed the feed url to http://www.backcomic.com/rss.xml, so I'll have to see what happens with that.

@mockdeep is it ok if I closed this for now? I'm trying to clean it up. I'll definitely look into it if it's still an issue.

Yeah, I think this is still an issue. I updated the feed and it still hasn't updated after a day.

@mockdeep I found the problem. The items in that feed have the following structure.
<item>
  <link>http://backcomic.com/164</link>
  <description>

  </description>
  <pubDate>Wed, 25 Oct 2017 00:00:00 -0600</pubDate>
  <guid isPermaLink="true">http://backcomic.com/164</guid>
</item>

Notice that there isn't a title property. That is ok, everything is optional, it only needs to have either a title or a description. The problem is that this feed uses the description field as the content, and I don't think that's a problem stringer should be solving. What do you think?

@gabrielpoca Hmm, so It looks like other sites like xkcd also put the content in the description. Is the difference that they at least have a title attribute as well? If so, I guess the above seems like a case that should be supported by stringer, maybe with just a "(no title)" message of some sort or swap in the feed title.
<item>
  <title>Defensive Profile</title>
  <link>https://xkcd.com/1911/</link>
  <description>
     If I've made you sad, you'd better not tell me, because I am TERRIFIED of that situation and have NO IDEA how to handle it." alt="NO DRAMA ZONE -> If I've made you sad, you'd better not tell me, because I am TERRIFIED of that situation and have NO IDEA how to handle it." />
  </description>
  <pubDate>Fri, 03 Nov 2017 04:00:00 -0000</pubDate>
  <guid>https://xkcd.com/1911/</guid>
</item>

@mockdeep that makes sense, but if I understood correctly the specification, if there isn't a title, then I should use the description, which in the case of your feed would give you a title like "". Or am I missing something?

@gabrielpoca I guess I'd be fine with that. I'd rather anything show up than nothing, so an escaped image tag seems fine to me. It also kind of makes sense to me in terms of what the spec says.

Side note, I've got a couple of other feeds that are showing as red. Should I open a new issue for those?

There's no need, I'll try to look into it this week 👍

Hi @mockdeep I just opened a PR to solve the issue with the first feed, but I cannot find anything wrong with the other ones, they were working just fine. Can you confirm?

Awesome, thanks! I took another look at the other feeds. The ScienceVS one shows as green and as having updated November 16, but I have not seen any actual items show up in my list. Pod Save America shows up red and as having updated on November 20, but again, nothing has showed up in my feed in a few months. The error says it has never worked, which seems weird.

Actually, I just pulled down and deployed the latest commits from Stringer and a bunch of stuff showed up in my feeds, including some of the Pod Save America stuff. However, the links aren't working on the Pod Save America stuff. They just link to /news.

@mockdeep it seems that items on that feed don't have any URL
<item>
--
  | <title>"Don't Hiss. Vote." (LIVE from Richmond)</title>
  | <description>
  | <![CDATA[
The Virginia race, the DNC and Brazile-gate, and the possibility of a shutdown for Dreamers. Lt. Governor Ralph Northam, Justin Fairfax, Attorney General Mark Herring, and Symone Sanders join Jon, Jon, Tommy, and Dan on stage live from Richmond, Virginia.
]]>
  | </description>
  | <itunes:title>"Don't Hiss. Vote." (LIVE from Richmond)</itunes:title>
  | <itunes:episodeType>full</itunes:episodeType>
  | <itunes:summary>The Virginia race, the DNC and Brazile-gate, and the possibility of a shutdown for Dreamers. Lt. Governor Ralph Northam, Justin Fairfax, Attorney General Mark Herring, and Symone Sanders join Jon, Jon, Tommy, and Dan on stage live from Richmond, Virginia.</itunes:summary>
  | <content:encoded>
  | <![CDATA[
The Virginia race, the DNC and Brazile-gate, and the possibility of a shutdown for Dreamers. Lt. Governor Ralph Northam, Justin Fairfax, Attorney General Mark Herring, and Symone Sanders join Jon, Jon, Tommy, and Dan on stage live from Richmond, Virginia.
]]>
  | </content:encoded>
  | <guid isPermaLink="false">gid://art19-episode-locator/V0/59FVIQxWSugLa50r8YB7-H1KaFjsS0f0Ugd8a2JWDjA</guid>
  | <pubDate>Mon, 06 Nov 2017 17:52:42 -0000</pubDate>
  | <itunes:explicit>yes</itunes:explicit>
  | <itunes:image href="https://dfkfj8j276wwv.cloudfront.net/images/0d/28/33/81/0d283381-724c-4caa-abad-b470e950d72d/b81bc65ae9ed5e2c095d68d40fecf3735467132e22415ad4861dd89a2f6527a47f45269a9091f8e98b499f5abc6c0c5423daa6690324ee9a0b9572868319deb7.jpeg" />
  | <itunes:keywords>tommy vietor,donald trump,Jon Lovett,Pod Save America,politics,POTUS,Dan Pfeiffer,jon favreau</itunes:keywords>
  | <itunes:duration>01:34:37</itunes:duration>
  | <enclosure url="https://rss.art19.com/episodes/8ec7727f-0589-4c80-813e-49349155b5a3.mp3" type="audio/mpeg" length="90845936" />
  | </item>

It probably should show a media player with the MP3, but stringer isn't really fit to listen to podcasts. What do you think?

@gabrielpoca hmm, can you take the enclosure url and link to that? With these all I really want to do is download the mp3 and listen to it.
Thanks for all the help, btw.

Another example: https://rss.art19.com/the-daily

Hi @mockdeep solved in this commit. Please let me know if I managed to fixed this time :p

Excellent! I guess it doesn't work for feeds that have already been added, so I'll have to report back after some new ones trickle in. Feel free to close this issue if you like. Thanks so much!

Alright, it works! Thank you!

Cool! :p


No stack when evaluating plugin in SES

I'm trying to add a dependency to a plugin that I'm building, and when I run build I get the error possible html comment syntax rejected around line 30033 I'm pretty sure that this means that there is something in the dependency which isn't able to be run in the ses container, but it would be helpful to have a stack trace so that I know what needs to be fixed. I already made a comment on the realms-shim repository but I also want to confirm that this isn't because of the TODO: mock wallet properly comment.

https://github.com/jvluso/metamask-plugin-beta/tree/transaction-plugin
I don't know what the line is that is having the issue because it is buried somewhere in the web3 dependency.

What's your intent with using web3 in the plugin? web3 has a complex dependency graph, and we have no plans to support its use in plugins. If you require an Ethereum convenience library, we recommend ethers.js. We haven't tested that out ourselves yet, but if you encounter any problems, we will help you out.
I also recommend that you:

Fork https://github.com/MetaMask/mm-plugin/ and move your transaction-plugin to a branch on that fork
Run mm-plugin build, commit the resulting bundle file, push the results

I did some further digging into your issue, which I am recording here for you (if you are interested), and for our future reference:
If you open the bundle file and go to line 30033, you should see something that looks like an HTML comment. You can also search through the bundle file, and you should find strings of the form <!--, which is syntactically valid JavaScript.
However, when I build your plugin on my machine and search through the bundle, I can only find strings of the form <--. Looking at Agoric's RegEx for finding HTML comments in realms-shim, <-- is not going to get picked up. Instead of a SES eval error, I also get a TypeError: undefined is not a constructor when the plugin is evaluated. The lack of a good stack is unfortunately often a problem with SES, but here I'm unsure where the problem is.

My overall goal is to include the Aragon API which uses web3 extensively to build a signer for Aragon DAOs. I know it has its problems, but I don't think I'll really be able to get around it soon.
Deleting the dist while fixing the git config seems to have solved the problem I was having. Hopefully agoric can work on the error message that was being generated.

Got it. We'd be really excited to see an Aragon plugin!
MetaMask has a great relationship with Agoric, and I'll see if we can do anything to improve the SES error stacks. Meanwhile, I'll close this, and feel free to open a new issue if you encounter any other problems.


Some feature Requests

First off, I think the project is amazing. There was just a few feature requests I had I'd love to see:

Allowing us to change the size of the Subject fonts(for like Education, Projects, Work Experience etc.)
Having a simple two column template. With just the different subjects on two columns. Like this, for example:

3. And lastly, allowing us to edit on the page, rather than on the side.
Regardless, thank you again for all the great work!

I really like 3. I'll tinker around and see if it's possible.

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!


Support pm-utils

See discussion in #18
On some system it might be better to use pm-utils. Add as an optional feature.

Much appreciated. :)

But yeah, just going to reiterate, this worked:
https://github.com/rodlie/powerkit/archive/test1.tar.gz
Completely solved my problem.


Update to Obvs5

These changes have now be thanks separately on master and deployed as Obvs.NetMQ.<IP_ADDRESS> to nuget.org. Thanks for the PR though, and sorry for the delay in responding.


Compilation fails if FEAT_PERSISTENT_UNDO is not included in build

Steps to reproduce
When compiling vim (normal option) I have a use case where I do not want the feature persistent_undo. I removed this feature by commenting out the feature in feature.h as I believe this is the only way to not include this feature at compile time.
diff --git a/src/feature.h b/src/feature.h
iindex 14489968f..3879b7a32 100644
--- a/src/feature.h
+++ b/src/feature.h
@@ -1241,9 +1241,9 @@
  * +persistent_undo	'undofile', 'undodir' options, :wundo and :rundo, and
  * implementation.
  */
-#ifdef FEAT_NORMAL
-# define FEAT_PERSISTENT_UNDO
-#endif
+//#ifdef FEAT_NORMAL
+//# define FEAT_PERSISTENT_UNDO
+//#endif

This causes compilation of vim to fail:
| fileio.c: In function 'readfile':
| fileio.c:1308:25: error: 'read_undo_file' undeclared (first use in this function); did you mean 'f_undofile'?
|  1308 |                         read_undo_file = FALSE;
|       |                         ^~~~~~~~~~~~~~
|       |                         f_undofile
| fileio.c:1308:25: note: each undeclared identifier is reported only once for each function it appears in

I believe the following patch will solve the issue:
diff --git a/src/fileio.c b/src/fileio.c
index 1fcf0d983..abccfa63d 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -1300,7 +1300,7 @@ retry:
 		    cryptkey = check_for_cryptkey(cryptkey, ptr, &size,
 						  &filesize, newfile, sfname,
 						  &did_ask_for_key);
-# ifdef CRYPT_NOT_INPLACE
+# ifdef defined(CRYPT_NOT_INPLACE) && defined(FEAT_PERSISTENT_UNDO)
 		    if (curbuf->b_cryptstate != NULL
 				 && !crypt_works_inplace(curbuf->b_cryptstate))
 			// reading undo file requires crypt_decode_inplace()
--

Expected behaviour
Successful compilation
Version of Vim
8.2.3752
Environment
Ubuntu 20.04
Logs and stack traces
No response

Steps to reproduce
When compiling vim (normal option) I have a use case where I do not want the feature persistent_undo. I removed this feature by commenting out the feature in feature.h as I believe this is the only way to not include this feature at compile time.
diff --git a/src/feature.h b/src/feature.h
iindex 14489968f..3879b7a32 100644
--- a/src/feature.h
+++ b/src/feature.h
@@ -1241,9 +1241,9 @@
  * +persistent_undo	'undofile', 'undodir' options, :wundo and :rundo, and
  * implementation.
  */
-#ifdef FEAT_NORMAL
-# define FEAT_PERSISTENT_UNDO
-#endif
+//#ifdef FEAT_NORMAL
+//# define FEAT_PERSISTENT_UNDO
+//#endif

This causes compilation of vim to fail:
| fileio.c: In function 'readfile':
| fileio.c:1308:25: error: 'read_undo_file' undeclared (first use in this function); did you mean 'f_undofile'?
|  1308 |                         read_undo_file = FALSE;
|       |                         ^~~~~~~~~~~~~~
|       |                         f_undofile
| fileio.c:1308:25: note: each undeclared identifier is reported only once for each function it appears in

I believe the following patch will solve the issue:
diff --git a/src/fileio.c b/src/fileio.c
index 1fcf0d983..abccfa63d 100644
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -1300,7 +1300,7 @@ retry:
 		    cryptkey = check_for_cryptkey(cryptkey, ptr, &size,
 						  &filesize, newfile, sfname,
 						  &did_ask_for_key);
-# ifdef CRYPT_NOT_INPLACE
+# ifdef defined(CRYPT_NOT_INPLACE) && defined(FEAT_PERSISTENT_UNDO)
 		    if (curbuf->b_cryptstate != NULL
 				 && !crypt_works_inplace(curbuf->b_cryptstate))
 			// reading undo file requires crypt_decode_inplace()
--

Yes, that looks like the right solution.  I'll make a patch with this.
--
BLACK KNIGHT:  Come on you pansy!
[hah] [parry thrust]
[ARTHUR chops the BLACK KNIGHT's right arm off]
ARTHUR:        Victory is mine!  [kneeling]
We thank thee Lord, that in thy merc-
[Black Knight kicks Arthur in the head while he is praying]
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Clients unable to find servers on local machine

On some machines, sometimes when the machine is disconnected from the internet, servers will assume the address <IP_ADDRESS>. For whatever reason, Java's InetAddress.isSiteLocalAddress() does not return true for that method even though that is the official address for localhost.

Fixed in 2.9.1


Check case of integration header in metadata.csv files for metadata validation

What does this PR do?

Check that the provided integration name in metadata.csv files match the normalized version of the display name, including that its lowercase.
Motivation

The system that reads these expects the name to be lowercased.
Additional Notes

I ran this against the integrations in this repo + extras, the only integration impacted is Pihole - https://github.com/DataDog/integrations-extras/blob/master/pihole/metadata.csv
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Here's the related PR to fix extras - https://github.com/DataDog/integrations-extras/pull/721


Convert array elements to proper representation

Fixes https://github.com/prestosql/presto/issues/6048
Extracted from https://github.com/prestosql/presto/pull/307

Ready for review.


Correct ssl_old_test stream handling

The ssl_old_test has not been fully converted to the test framework but it still reuses some test framework utilities. Notably it was creating it's own copy of the global bio_err object directly (which is normally created and owned by the test framework). This causes a problem because ever since commit 2fa9044 access to the bio_err object is controlled by a lock. Since ssl_old_test was circumventing the normal creation and destruction of bio_err, the lock was not being created resulting in a crash under certain error conditions.
We fix this by creating and destroying the bio_err object using the test framework functions designed for that purpose.
Fixes #23184

Fixup pushed. @t8m, @nhorman - please reconfirm

approval reconfirmed

This pull request is ready to merge

Merged to the master and 3.2 branches. Thank you.


[Bug]: Genserver no found

What happened?
When using application, we have Genserver not found crash.
Look on Session timeout probably stop Session.Supervisor
What browsers are you seeing the problem on?
Firefox
Version
No response
Relevant log output
https://sentry.io/organizations/lenra-br/issues/3860388832/?project=5633085&referrer=slack

https://sentry.io/organizations/lenra-br/issues/3860388818/?project=5633085&referrer=slack

https://sentry.io/organizations/lenra-br/issues/3860388832/?project=5633085&referrer=slack

https://sentry.io/organizations/lenra-br/issues/3857703935/?project=5633085&referrer=slack

Should be fixed by: https://github.com/lenra-io/application-runner/pull/303


Question about version pinning

I am currently integrating the client into a web project and I faced some challenges due to the version pinning inside of the file setup.py:
    'requests==2.12.1',
    'python-dateutil==2.6.0'

This is making it tricky to install it together with a set of dependencies which I have already. Would it be an option to relax this and only provide the pinning inside of the file requirements.txt as something like a "known good set" ?

Sounds good!
I'll do this tomorrow and release a patch version.
Cheers

Closing, thanks for merging in the change


Fix extract Frogbot configuration from source branch

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Fix the fallback when can't find frogbot config at the target branch of a pull request.

Does this functionality still exist? Currently, FrogBot is not picking up the file from the source repository. We have a config file in the source repository, but FrogBot is not picking up that file. It always tries to look for a file in the target repository instead of the source repository.


Does track-by="$key" working ?

this.parameters = {}
...
<div v-for="value in parameters" track-by="$key">
        <input name="{{$key}}" v-model="value" />
</div>

Does track-by="$key" working ?
Thanks for your time

If you are iterating over an object, tracking by key is the default behavior so you don't even need that.


Syntax error, unexpected '{' on Request.php  $e->getResponse()->getBody() : '';

Required Information

Operating system: CENTOS 7.6
PHP version: 5.6
PHP Telegram Bot version:
Using MySQL database: yes
MySQL version: 5.7.25
Update Method: Webhook
Self-signed certificate: yes
RAW update (if available):

My bot not reply message, and I check error_log
PHP Parse error:  syntax error, unexpected '{' in  vendor/longman/telegram-bot/src/Request.php on line 343
public static function execute($action, array $data = [])
{
    //Fix so that the keyboard markup is a string, not an object
    if (isset($data['reply_markup'])) {
        $data['reply_markup'] = json_encode($data['reply_markup']);
    }
    $result                  = null;
    $request_params          = self::setUpRequestParams($data);
    $request_params['debug'] = TelegramLog::getDebugLogTempStream();
    try {
        $response = self::$client->post(
            '/bot' . self::$telegram->getApiKey() . '/' . $action,
            $request_params
        );
        $result   = (string) $response->getBody();
        //Logging getUpdates Update
        if ($action === 'getUpdates') {
            TelegramLog::update($result);
        }
    } catch (RequestException $e) {
        $result = ($e->getResponse()) ? (string) $e->getResponse()->getBody() : '';
    } finally {  //====>>> line 34
        //Logging verbose debug output
        TelegramLog::endDebugLogTempStream('Verbose HTTP Request output:' . PHP_EOL . '%s' . PHP_EOL);
    }
    return $result;
}

how to fix it, before it is no problem?

Line 343 in latest release is empty https://github.com/php-telegram-bot/core/blob/0.55.1/src/Request.php#L343
You have damaged installation, clean vendor directory and composer update.

I have changed the request file, it still has an error, the error changes to line 377,,
error :
/vendor/longman/telegram-bot/src/Request.php on line 377
what do you mean
clean vendor directory and composer update? I replace it with the latest vendor directories?

Line 343 in latest release is empty https://github.com/php-telegram-bot/core/blob/0.55.1/src/Request.php#L343
You have damaged installation, clean vendor directory and composer update.

I have changed the request file, it still has an error, the error changes to line 377,,
error :
/vendor/longman/telegram-bot/src/Request.php on line 377
what do you mean
clean vendor directory and composer update? I replace it with the latest vendor directories?

Another thing that comes to my mind is that you might be running a PHP lower that 5.5 which didn't have finally support. Are you sure you're running 5.6? I've seen some webhosts having 5.2 as default...

Another thing that comes to my mind is that you might be running a PHP lower that 5.5 which didn't have finally support - in that case it might try using it as a function but finds no () .
Verify it by running a script which echos phpversion().
Are you sure you're running 5.6? I've seen some webhosts having 5.2 as default...

ohh yess,, i check again phpversion it 5.4, and i change 5.6 it work .. Thank you very much 👍


Specify a minimum requirement for BASH_VERSION

The declare command has been available since very early versions of Bash (pre-4.0), so if you have Bash 4.2 or higher, you definitely have access to the declare builtin.

I copied audit.sh and check_indexes.sh into my project and made audit.sh executable:
curl https://raw.githubusercontent.com/speedshop/ids_must_be_indexed/refs/heads/main/audit.sh -o audit.sh
curl https://raw.githubusercontent.com/speedshop/ids_must_be_indexed/refs/heads/main/check_indexes.sh -o check_indexes.sh
chmod +x ./audit.sh
./audit.sh

When I run it, I get:
./check_indexes.sh: line 247: conditional binary operator expected

If I comment out the line causing this error, I get:
Analyzing schema.rb for missing indexes...

Error: This script requires Bash version 4.2 or later for declare -g support.
Your current Bash version is: 3.2.57(1)-release
Please upgrade your Bash version to run this script.

So, it’s likely not a bug but syntax that only works on Bash 4.2+.
If that’s the case, it might be worth updating the README.md or adding a version check in the script itself like:
# Check Bash version 4.2 or high to have support for 'declare'
minimum_required_version="4.2"
if [ -z "$BASH_VERSION" ] || [[ "${BASH_VERSION%%[^0-9.]*}" < "$minimum_required_version" ]]; then
  echo "Error: This script requires Bash version $minimum_required_version or higher."
  exit 1
fi

After i did a basic brew install bash i got the 5.2 and now it works fine
Have a great day

Yeah I put the check in a bizarre place, I'll change it.


Business Partner Annotations

Description

At the moment, during processing a golden record task only contains the business partner data as a result from cleaning services. However, both for the cleaning services as well as for the sharing member it is beneficial to attach more detail information to how and why the business partner data is processed like that.
Acceptance Criteria
This should include

whether the data point has been approved if it stayed unchanged
generel comments giving more detail explainations
the source from which the data point was taken
the cleaning service which has created the annotation

Additional Information

As data manager of the operating company, I would recommend to NOT realize this functionality because revealing used validation sources would limit freedom of operation of the operating company, reveals potential business secrets to semi-public and competitors and might even (tbc) be in conflict with provider contracts between operating company and third party data providers.

Initiation - Why this issue:
Communication between different cleaning services
Maybe we need to split this topic up and decide for each of them based on a use case for it:

Feedback loop (communication between operator and sharing member, service to service)
Sharing information between cleaning services (communication)

I will create an sig release item for the feedback loop topics and @jupakajo will take the 2. point to the expert group.


memberships fall out

[x] when clicking a new member notification, make sure to refresh supporters list (even if you are on the page / in another creator area).

[ ] show indication to everyone that a chat is in members only mode (i.e. if you're a member or creator, you can't tell).

[x] premium badges don't load here

[x] premium badge not showing on top result or search...

[x] from notifications, clicking a URL which goes to a channel page ends up with "undefined" https://odysee.com/@AussieFighter:828f9bdb59ce634f5f381301b531baa1e30ac077?lc=3bf7[…]a182187b82412aee986a9891e49fecf4c354fa4f2d95cda5&view=undefined

[ ] on memberships page, when you have an active memberships, show the $ tier and total amount somewhere. Also, would there be an easy to way show "all tiers" here?

[x] mention USD bank support on premium only message

[x] upload description box not expanding, no scroll bar:

[ ] https://odysee.com/membershipbeta:cc70596a8fa6ae21add0c9fff3048887bb5ae85d?lc=105126bbb0e3b5c754aa946dc2ecd2f2c2d9763a9d66ca71bdeeda7e359bd2cc


Backup Database

Should be able to have an automated backup of the current DocumentDb database to an Azure storage account.

Azure has backup recovery if you contact support, they will foresee automatic backup/restore in near future I suppose.


Bug 2183198 : [release-4.12] Disables leader election for addons

This commit fixes an with addon leader election. The leader election gets stuck on acquiring lock for the namespace lease. This fix disables the leader election process for addons as it is not required for a single replica deployments.
Main branch PR: https://github.com/red-hat-storage/odf-multicluster-orchestrator/pull/152

/bugzilla refresh

/bugzilla refresh


Bi-Weekly labs / intro fixes

I see Pam already made some paper things using the term "DevOps Open Office". Perhaps I should update the terms in this PR to reflect that name instead?

@ramereth Yeah it might be better to change it to DevOps Open Office since we already printed some posters with that name.


BranchBehindAndAheadDisplay 'Minimal' Setting

System Details

posh-git version/path: 1.0.0 beta2 ~/.local/share/powershell/Modules/posh-git/1.0.0
PowerShell version: 6.1.1
git version 2.19.1
OS: Unix <IP_ADDRESS>

Issue Description
Apologies if this has already been answered elsewhere. I did my best to search for relevant information first, but I couldn't find what I was looking for.
When the BranchBehindAndAheadDisplay setting was first introduced in #256, Minimal would show ↑, ↓, or ↕ as appropriate with no counts. As of commit 4d8e08c, Minimal does not show any arrows, but the branch name is still colored red/green/yellow.
Was this a deliberate change? Obviously the color is sufficient to indicate its remote tracking status, but it seemed odd to me that the arrows would be removed given that it still displays the ≡ symbol when the branch is even with its remote. I couldn't find an issue where this might have been discussed, and it's not mentioned in the changelog for version 0.7.3, which was the next release after that commit.
I only noticed because I was working on an equivalent feature for lyze/posh-git-sh#37 and I wanted to ensure it behaved the same. I like Full myself :smiley:
Thanks for all your work! posh-git is fantastic.

Good catch.  Does this display look better?

Looks great!


Added table of contents, added badges to docs, and general docs cleanup

This is a followup PR to the recent conversion to the Alabster theme to add a couple of features and do some general cleanup.
Closes: https://github.com/crsmithdev/arrow/issues/593

Codecov Report

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

@@          Coverage Diff          @@
##           master   #608   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           7      7
  Lines        1501   1501
  Branches      225    225
=====================================
  Hits         1501   1501

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 24927fa...fdb77c5. Read the comment docs.

@systemcatch I changed the __version__ string extraction method to match what requests does. It helps us to allow for changing the version only in one place, _version.py, rather than in __init__.py and in docs/conf.py. The issue we saw in 0.14.0 should not be present here since we are not importing __init__.py.

Hey @jadchaar, I'm pretty busy until Thursday evening but should have a chance to look at this after then.


cordova add platform ios gives EEXIST: file already exists, mkdir '/path/to/Project.xcworkspace/xcshareddata` with tabris 1.11.0

Problem description
Hi!
I'm trying to upgrade an existing Tabris 1.10.0 app  to 1.11.0. I have changed my tabris version in package.json to 1.11.0. When I attempt to add the cordova ios platform found in the archives at https://tabrisjs.com/api/v1/downloads/1.11.0/ios I get the following:
$ cordova platform add ../../tabris/ios

cordova-custom-config: Installing plugin dependencies...

Adding ios project...
Creating Cordova project for the iOS platform:
        Path: platforms/ios
        Package: com.project.app
        Name: Project
iOS project created with<EMAIL_ADDRESS>config file *.entitlements requested for changes not found at /Users/me/projects/Project/AppNative/platforms/ios/*.entitlements, ignoring
Installing "cc.fovea.cordova.purchase" for ios
Installing "com.ozexpert.devicemeta" for ios
Installing "com.terra.carrierbilling" for ios
Installing "cordova-custom-config" for ios
cordova-custom-config: Installing plugin dependencies...
Installing "cordova-custom-config-project" for ios
Installing "cordova-plugin-android-permissions" for ios
Installing "cordova-plugin-cocoapod-support" for ios
Installing "cordova-plugin-device" for ios
Installing "cordova-plugin-dialogs" for ios
Installing "cordova-plugin-facebook4" for ios
Installing "cordova-plugin-globalization" for ios
Installing "cordova-plugin-inappbrowser" for ios
Installing "cordova-plugin-intercom" for ios
Installing "cordova-plugin-listpicker" for ios
Installing "cordova-plugin-native-info" for ios
Installing "cordova-plugin-segment" for ios
Installing "cordova-plugin-sim" for ios
Installing "cordova-plugin-spinner-dialog" for ios
Installing "cordova-plugin-tune" for ios
Installing "cordova-plugin-x-socialsharing" for ios
Installing "cordova-plugin-x-toast" for ios
Installing "es6-promise-plugin" for ios
Installing "it.almaviva.openurlext" for ios
Installing "veeplay-cordova-plugin" for ios
cordova-custom-config: Skipping auto-restore of config file backup(s)
cordova-custom-config: Applied custom config from config.xml to /Users/me/projects/Project/AppNative/platforms/ios/Project.xcodeproj/project.pbxproj
cordova-custom-config: Applied custom config from config.xml to /Users/me/projects/Project/AppNative/platforms/ios/cordova/build.xcconfig
cordova-custom-config: Applied custom config from config.xml to /Users/me/projects/Project/AppNative/platforms/ios/Project/Project-Info.plist
Searching for new pods
Checking cc.fovea.cordova.purchase for pods.
Checking com.ozexpert.devicemeta for pods.
Checking com.terra.carrierbilling for pods.
Checking cordova-custom-config-project for pods.
Checking cordova-plugin-android-permissions for pods.
Checking cordova-plugin-device for pods.
Checking cordova-plugin-dialogs for pods.
Checking cordova-plugin-facebook4 for pods.
Checking cordova-plugin-globalization for pods.
Checking cordova-plugin-inappbrowser for pods.
Checking cordova-plugin-intercom for pods.
Checking cordova-plugin-listpicker for pods.
Checking cordova-plugin-native-info for pods.
Checking cordova-plugin-segment for pods.
Checking cordova-plugin-sim for pods.
Checking cordova-plugin-spinner-dialog for pods.
Checking cordova-plugin-tune for pods.
Checking cordova-plugin-x-socialsharing for pods.
Checking cordova-plugin-x-toast for pods.
Checking es6-promise-plugin for pods.
Checking it.almaviva.openurlext for pods.                                                                                Checking veeplay-cordova-plugin for pods.
Checking config.xml for pods.
config.xml requires pod: Analytics
Installing pods                                                                                Sit back and relax this could take a while.
Analyzing dependencies

Downloading dependencies
Installing Analytics (3.6.10)
Generating Pods project                                                                                Integrating client project                                                                                Sending stats
                                                                                Pod installation complete! There is 1 dependency from the Podfile and 1 total pod installed.
                                                                                [!] `<XCBuildConfiguration name=`Debug` UUID=`8985ED9D8A31480A81637`>` attempted to initialize an object with an unknown UUID. `59F6FDEEB5474FA889F92` for attribute: `base_configuration_reference`. This can be the result of a merge and  the unknown UUID is being discarded.                                                                                [!] `<XCBuildConfiguration name=`Release` UUID=`BA9D55FEA165497493540`>` attempted to initialize an object with an unknown UUID. `59F6FDEEB5474FA889F92` for attribute: `base_configuration_reference`. This can be the result of a mer
ge and  the unknown UUID is being discarded.

[!] `<XCBuildConfiguration name=`Debug` UUID=`60699C1D63BC4C5680357`>` attempted to initialize an object with an unknown UUID. `59F6FDEEB5474FA889F92` for attribute: `base_configuration_reference`. This can be the result of a merge and  the unknown UUID is being discarded.

[!] `<XCBuildConfiguration name=`Release` UUID=`C401A5933DC04880BE469`>` attempted to initialize an object with an unknown UUID. `59F6FDEEB5474FA889F92` for attribute: `base_configuration_reference`. This can be the result of a merge and  the unknown UUID is being discarded.

[!] The `Project [Debug]` target overrides the `OTHER_LDFLAGS` build setting defined in `Pods/Target Support Files/Pods-Project/Pods-Project.debug.xcconfig'. This can lead to problems with the CocoaPods installation

[!] The `Project [Release]` target overrides the `OTHER_LDFLAGS` build setting defined in `Pods/Target Support Files/Pods-Project/Pods-Project.release.xcconfig'. This can lead to problems with the CocoaPods installation

    - Use the `$(inherited)` flag, or
    - Remove the build settings from the target.
    - Use the `$(inherited)` flag, or
    - Remove the build settings from the target.

Updating ios build to use workspace.
Adding schemes
EEXIST: file already exists, mkdir '/Users/me/projects/Project/AppNative/platforms/ios/Project.xcworkspace/xcshareddata'

Looking through closed issues, I see this issue had the same problem with no response: https://github.com/eclipsesource/tabris-js/issues/1131#issuecomment-283445021
Like the OP, I also had nothing in my platforms directory before I started so not sure why this is happening.
Expected behavior
ios platform is added successfully / without errors
Environment

Tabris.js version: 1.11.0
Device: Mac
OS: mac OS 10.13.6
Tabris Cordova iOS platform version: 4.5.3
Cordova (cli) version: Tested w/ 6.4.0 and 8.1.2<EMAIL_ADDRESS>

Can confirm I'm having this issue with Tabris Cordova iOS version 4.5.4 as well

Are you using the tabris-cli to build your project?
https://www.npmjs.com/package/tabris-cli

@ashwinv11 Please ensure that your project has following structure when building your app locally:
.
├── config.xml
└── www
    ├── app.js
    └── package.json

And execute:
$ cordova -v
6.5.0
$ cat $PATH_TO_CORDOVA_PLATFORM_1.11.0/CordovaLib/VERSION
4.5.4
$ cordova platform add $PATH_TO_CORDOVA_PLATFORM_1.11.0

I couldn't reproduce your problem with that setup.
You stated that your app has been built with Tabris.js iOS platform version 1.11.0, but your Tabris Cordova iOS platform version was 4.5.3 at the same time. As you can see in the cat output, Tabris.js iOS platform 1.11.0 is based on Cordova iOS 4.5.4.

Looks like this was due to a bug with an old version of cordova-plugin-cocoapods-support we were using: https://github.com/blakgeek/cordova-plugin-cocoapods-support/issues/19.
I'll close this, thx for checking @cpetrov


FLUME-3452. Upgrade libthrift version to fix CVE 2020-1938

org.apache.thrift:libthrift:0.14.1 has dependency on tomcat-embed-core : 8.5.46 which is causing CVE 2020-1938.
org.apache.thrift:libthrift:0.14.2 solved the problem for https://mvnrepository.com/artifact/org.apache.thrift/libthrift/0.14.2,
see https://issues.apache.org/jira/browse/FLUME-3452.

Verified the build is still successful with this change.

The unit tests are failing when run with Java 11 as the truststore is using DSA. They work with Java 8.

The unit tests are failing when run with Java 11 as the truststore is using DSA. They work with Java 8. However, after upgrading from version 265 to 362 of Java 8 the tests fail. So the keystores need to be updated.

I am very sorry for the single test failure caused by my submission.
I continue to focus on the tests involved in this change.

I apologize for making it seem like the unit test failures were due to this change. They were not. I just happened to notice that the CI builds were failing in the Thrift tests. When I used Java 11 on my computer I saw the same failures. Then I noticed the CI build was using a newer version of JDK 8 and when I upgraded to that I saw the same errors. I fixed the problem with FLUME-3460.

It's not a problem at all, I wish I could do something with flume.
I did encounter some problems in building the project in the past two days. For details, please refer to this link?
https://github.com/apache/flume/pull/403
or https://issues.apache.org/jira/browse/FLUME-3462
Finally thank you, hope I can do a little work for flume.


Registry tests

This PR will add tests for registry. Also, this required a bit of refactoring which is complete.
registry.py is now python2 compatible
Merge conflicts need to be settled

Hello @nupurgunwant! Thanks for submitting the PR.

In the file tests/test_registry.py, following are the PEP8 issues :

Line 8:5: E303 too many blank lines (2)
Line 9:9: E265 block comment should start with '# '
Line 11:9: E265 block comment should start with '# '
Line 12:9: E265 block comment should start with '# '
Line 16:5: E303 too many blank lines (2)
Line 17:9: E265 block comment should start with '# '
Line 19:9: E265 block comment should start with '# '
Line 20:9: E265 block comment should start with '# '
Line 24:5: E303 too many blank lines (2)
Line 25:9: E265 block comment should start with '# '
Line 27:9: E265 block comment should start with '# '
Line 28:9: E265 block comment should start with '# '
Line 29:62: E502 the backslash is redundant between brackets
Line 30:9: E128 continuation line under-indented for visual indent


Fix data channel not work issue

Need set local description when recevie offer event. copy code from call_sample.dart

#199


How does the NAT server start the proxy?

Network card IP: <IP_ADDRESS>
Public IP: <IP_ADDRESS> (hypothetical)
Port: 443
How to write the proxy startup parameters?
Help me, thank you~

$ ./mtg -b <IP_ADDRESS> -p 443 -4 <IP_ADDRESS> <secret>

@BirkhoffLee yes, it was. If you want to continue with -4, you need to use v1.x, not v2.x


Nightly build install is not working

Hi,
PS C:\Users\mavu> scoop install crystal-nightly
Couldn't find manifest for 'crystal-nightly'.

 scoop install https://github.com/neatorobito/scoop-crystal/raw/main/nightly/crystal-nightly.json
Installing 'crystal-nightly' (1.6.56) [64bit]
The remote server returned an error: (404) Not Found.
URL https://nightly.link/crystal-lang/crystal/actions/artifacts/485523404.zip is not valid

Can't install nightly build with scoop install crystal-nightly or scoop install crystal-preview
if I try the manifest directly, see above error.

It seems like the nightly link hasn't been updated since #26 was merged.
@neatorobito Was that on purpose?

Hi @straight-shoota, @mavu
Yes, I deprecated the nightly builds after the last commit, but I forgot to remove the directory altogether. The API was/is no longer working for getting the nightly artifacts and I also figured people could get the zip if they specifically needed nightlies.
On a separate note, I don’t have the bandwidth to maintain the repo (though it’s mostly self-sustaining now). I’m happy to transfer the rights over to crystal-community if that works for you.


haskellPackages.hfsevents: Fix eval on Darwin hackage2nix config and temporarily

Motivation for this change
Backports the fix to hackage2nix config to not mark hfsevents unbuildable on Darwin. Also quickfixes the package set so we can get darwin-tested evals rolling again on 19.03
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This is being evaluated by ofborg after https://github.com/NixOS/ofborg/pull/339 deployed, so ... good timing.


IRequestPostProcessor executed multiple times

I was originally using IPipelineBehavior to add pre and post hooks to a mediator pipeline but have then swapped out these to use IRequestPreProcessor and IRequestPostProcessor respectfully.
Unfortunately doing so causes my handlers to be executed twice. Contrary to the docs, I'm not registering them explicitly as they appear to be getting registered automatically.
MediatR is registered like so (ASP.NET Core 2.2):
            services.AddMediatR(typeof(PaymentsRegistry).Assembly);
            services.AddTransient(typeof(IPipelineBehavior<,>), typeof(LoggingMediatorBehavior<,>));
            services.AddTransient(typeof(IPipelineBehavior<,>), typeof(SequentialProcessingBehavior<,>));

Example of one my behaviours:
    internal class PaymentRequestedDispatcher : IRequestPreProcessor<AuthorizeCardPayment>
    {
        private readonly IMediator _mediator;
        private readonly ILogger _logger;

        public PaymentRequestedDispatcher(IMediator mediator, ILogger logger)
        {
            _mediator = mediator ?? throw new ArgumentNullException(nameof(mediator));
            _logger = logger ?? throw new ArgumentNullException(nameof(logger));
        }

        public async Task Process(AuthorizeCardPayment command, CancellationToken cancellationToken)
        {
            if (command is null)
                throw new ArgumentNullException(nameof(command));

            _logger.Debug("Executing Payment Requested Dispatcher for Payment {PaymentId}", command.PaymentId.ToString());

            var requested = new Requested(

            );

            await _mediator.Publish(requested, cancellationToken);
        }
    }

From my logs I am seeing all of the pre/post handlers getting executed twice:
2019-10-29 18:21:51 DBG Computing Hash of ApiKey "completed" in 37.5 ms
2019-10-29 18:21:51 DBG Retrieving ApiKeyDetails from Couchbase "completed" in 71.4 ms
2019-10-29 18:21:51 DBG Validating type "PaymentRequest<CardSource>" using "PaymentRequestWithSourceValidator<CardSource, CardSourceValidator>" "completed" in 1.4 ms
2019-10-29 18:21:52 DBG Storing card details in Vault "completed" in 134.3 ms
2019-10-29 18:21:52 DBG Executing Payment Requested Dispatcher for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Inserting GetPayment Document in Couchbase "completed" in 18.6 ms
2019-10-29 18:21:52 DBG Creating new publisher for stream "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Executing Payment Requested Dispatcher for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 INF Found credentials using the AWS SDK's default credential search
2019-10-29 18:21:52 DBG Inserting GetPayment Document in Couchbase "completed" in 5.3 ms
2019-10-29 18:21:52 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Retrieving Merchant from Couchbase "completed" in 26.5 ms
2019-10-29 18:21:52 DBG Retrieving Acquirer service configuration from Couchbase "completed" in 17.3 ms
2019-10-29 18:21:52 DBG Generated acquirer transaction ID 41001 for payment transaction ID 5ea33cbf-98f0-440a-ab23-cef0c774e94a
2019-10-29 18:21:52 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 INF Processing "4" with acquirer: "cko-visa" "completed" in 184.1 ms
2019-10-29 18:21:52 DBG Writing PaymentRecord to Couchbase "completed" in 53.6 ms
2019-10-29 18:21:52 DBG Executing Authorization Result Dispatcher for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Updating GetPayment Document in Couchbase "completed" in 40.6 ms
2019-10-29 18:21:52 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Executing Auto-Capture Handler for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:52 DBG Retrieving URL for SQS queue "default" "completed" in 217.8 ms
2019-10-29 18:21:53 DBG Sending message "CapturePayment" to SQS "completed" in 45.7 ms
2019-10-29 18:21:53 DBG Handling "Enqueue" with Mediator "completed" in 285.3 ms
2019-10-29 18:21:53 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Executing Auto-Void Handler for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Handling "AuthorizeCardPayment" with Mediator "completed" in 831.1 ms
2019-10-29 18:21:53 DBG Executing Authorization Result Dispatcher for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Updating GetPayment Document in Couchbase "completed" in 1.3 ms
2019-10-29 18:21:53 DBG Executing Auto-Capture Handler for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Sending message "CapturePayment" to SQS "completed" in 5.4 ms
2019-10-29 18:21:53 DBG Handling "Enqueue" with Mediator "completed" in 5.9 ms
2019-10-29 18:21:53 DBG Executing Auto-Void Handler for Payment "pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Queueing event to batch for "Payment-pay_p57sma5ssgkevhzcgccpanqxc4"
2019-10-29 18:21:53 DBG Handling "RequestPayment<FullCardSource>" with Mediator "completed" in 1078.5 ms
2019-10-29 18:21:53 DBG Handling "PaymentRequest<CardSource>" with Mediator "completed" in 1177.7 ms
2019-10-29 18:21:53 INF HTTP "POST" "/payments" responded 201 in 2871.4427 ms

Also am I right that the only way to control the order of behaviours is if I use IPipelineBehavior and register them explicitly?

Yes, you have to register pipeline behaviors explicitly anyway, so that you can control the order.
services.AddMediatR adds handlers for you, by scanning assemblies. It also adds pre/post processors. But not behaviors. Here's what it adds for you:
https://github.com/jbogard/MediatR.Extensions.Microsoft.DependencyInjection/blob/master/src/MediatR.Extensions.Microsoft.DependencyInjection/Registration/ServiceRegistrar.cs#L214
And what it scans for you:
https://github.com/jbogard/MediatR.Extensions.Microsoft.DependencyInjection/blob/master/src/MediatR.Extensions.Microsoft.DependencyInjection/Registration/ServiceRegistrar.cs#L214

Yep! Really any of the default services should only be registered once.

I had the same problem, and it was precisely because I registered AddMediatR separately,
this worked.
var assemblies = new System.Reflection.Assembly[]
{
 AppDomain.CurrentDomain.Load("Core"),
 AppDomain.CurrentDomain.Load("Notifications"),
 AppDomain.CurrentDomain.Load("Domain")
});

services.AddMediatR(assemblies);


[Testing] Mock 10% Persistence Module in Utility Tests

Objective
Mock the Persistence Module for all Utility Testing
Origin Document
The Utility Module relies on the Persistence Module in order to execute all of it's testing:
See
func newTestPersistenceModule(_ *testing.M, databaseUrl string) modules.PersistenceModule {

Under utility/test
Goals

Enabling utility tests for persistence business logic that doesn't exist
Remove cross module imports in Utility
Allow complete encapsulation of test suite in Utility
Create a clear and understandable Mock

Deliverable

Code complete implementation
Plenty of comments in the Mock
Updated changelog

Non-goals / Non-deliverables

Little nits that have nothing to do with the issue

General issue deliverables

[ ] Update the appropriate CHANGELOG
[ ] Update any relevant READMEs (local and/or global)
[ ] Update any relevant global documentation & references
[ ] If applicable, update the source code tree explanation
[ ] If applicable, add or update a state, sequence or flowchart diagram using mermaid

[Optional] Testing Methodology

_REPLACE_ME: Make sure to update the testing methodology appropriately_
Task specific tests: make ...
All tests: make test_all
LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md

Creator: @andrewnguyen22

Idea posted in #290 as a starting point for a discussion.

I scrapped the first iteration of the idea (using a passthrough) and submitted a second one (using mocks only). Looking for preliminary feedback to see if it meets the requirements of the ticket before I put more effort into it

See #290 as a reference for future iterations on this work

We have been using the persistence module directly for utility tests, so am closing this out until we refactor the test suite.


Use lists for demes and doi.

See #117 and #123.

Ah, whoops... I just added another commit to this the same moment you approved it @jeromekelleher. Sorry!

Approval stands!


chore: Remove Pickware copyright notice from ManyToOneAssociationFieldResolverTest

1. Why is this change necessary?
As Shopware is open source, including a restrictive copyright notice in one of it's tests should be avoided.
2. What does this change do, exactly?
Removes a copyright notice of Pickware Gmbh, erroneously added in https://github.com/shopware/shopware/pull/3065.
3. Describe each step to reproduce the issue or behaviour.
N/A
4. Please link to the relevant issues (if any).

N/A
5. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfill them.

Thanks!


Branch 1.2 fix spill hang

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #ISSUE-ID)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

just for review && run ci

Run Gluten Clickhouse CI


[Bug] TreeEdgeRenderer can't highlight paint to parent properly

Using the sample from the README, and with  algorithm: BuchheimWalkerAlgorithm(builder, TreeEdgeRenderer(builder)), I am trying to highlight the path from node to topmost parent. Because juncture points in a tree edge are associated with the last-drawn node, the full path can't be highlighted correctly, resulting in this situation:

So I assume you want the color to go from Node 6 to 8 as well right?
Can you share your code


fix(*): fix issue where view isn't reloaded, leaving previous events …

…in the eventsource visible on the calendar

@jakemulley I can confirm it's bug, but adding refreshView in onEventSourceChanged function is not an efficient fix. Because it needs to recalculate the date range and regenerate the cells, but actually you only want to clear the old events attached to the existing cells.
I have committed my fix, basically add the clear old events logic at the beginning of _onDataLoaded function.
Let me know if the fix works. Thanks.


Convert range filter dates to universaltime

All dates in the datetime filter were converted to UTC prior to selecting them from the database. Range filters were not considered however, leading to missing entries.

Thanks

There was a problem with the CI process which meant this wasn't released until today. Thanks again for the fix


Generate random Tensor (and Boolean and Integral) programs

@tomsmeding said:

I wonder if what you need here is generating random programs :p

And indeed I need it. Dimensions only 1, 2 or 3, ranks up to 5, I guess. Maybe 7. Not too high so that the programs don't take forever to vectorize and forever to AD.
Probably just QuickCheck generators? We already have them for orthotope tensors. I guess it's enough to generate Ast programs, which are terms, not Tensor programs, which can desugar and instantiate to terms.
The difficulty I can see at this point is generating correctly ranked and correctly shaped programs. E.g., in tfromList [t, u], t and u should have the same shape.
We could go top-down and first generate a shape and then a constructor that can have that shape, e.g., AstIndex and then, recursively, that would partially determine the shapes of subterms and sometimes we'd have to generate more shapes, within constraints of the outer shape. This sounds complex. We'd surely need unification. Probably a fully fledges shape reconstruction for Ast terms with holes (not the current simplistic shape-checking of concrete Ast terms).
Another top-down methods is to generate a constructor and only then a shape that the constructor can possibly have, etc. Similarly complex.
A bottom-up methods seems hopless: we generate t and u, they have totally different shapes, none of the constructors can take t and u at once, we give up.
Is it know that randomly generating well-typed expressions is as hard as reconstruction of the most general type (or something close) for the expressions?

Here's a promising approach: https://gitlab.haskell.org/ghc/ghc/-/issues/24422


Expense manager customized ngFormioGrid

Hi Everyone,
Please find the ngFormioGrid code that I customized based on to full-fill the requirements in "Expense Manager" application. Kindly review and provide your feedback.
Thanks,
Ravi

Hey Ravi,
It is really hard to tell what was changed here and why. Instead of saying it was customizations for expense manager, can you explain what the customizations were?
Separating the code reformatting into a different commit would help as well.

Hi Randall,
Thanks for reviewing the code.
The ngFormioGrid changes are not actually the expense manager app specific customization. I tried to make code changes in ngFormioGrid in such a way that it should execute as stand alone bower component. I used few functions in "Expense Manager" like "filter Grid Record between two dates" and prepared function in ngFormioGrid which accept two dates (StartDate and EndDate) and return filtered record. I prepared one more function " $scope.exposeGridApi" to expose "GridApi"object. It will simply expose "GridApi" object to use outside of "formio-grid.js". I will do more detailed commenting on code and recommit it for your review further.
Thanks

This pull request kind of scares me with all of the diffs. I will be bringing some of the row selection items into the master (with the work I am doing with a client) so I am thinking we need to re-merge this once I am done with that endeavor and see how we can make the diff not so intimidating.

Travis,
Please let me know if you need me to work on modifications of any part of the ngFormioGrid ? so that it should be flexible enough to merge with your code.
Sent from my ASUS
-------- Original Message --------
From:Travis Tidwell<EMAIL_ADDRESS>Sent:Thu, 24 Mar 2016 02:29:53 +0530
To:formio/ngFormioGrid<EMAIL_ADDRESS>Cc:Ravi Kachhawaha<EMAIL_ADDRESS>Subject:Re: [ngFormioGrid] Expense manager customized ngFormioGrid (#5)

This pull request kind of scares me with all of the diffs. I will be bringing some of the row selection items into the master (with the work I am doing with a client) so I am thinking we need to re-merge this once I am done with that endeavor and see how we can make the diff not so intimidating.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub￼

I would say for now hold off until I commit my changes I have been making. What I would then like is for you to re-do your changes on top of those. I should be done by the end of this week.
Thanks,
Travis.

Sounds good.
Thanks,
Ravi
Sent from my ASUS
-------- Original Message --------
From:Travis Tidwell<EMAIL_ADDRESS>Sent:Thu, 24 Mar 2016 21:00:47 +0530
To:formio/ngFormioGrid<EMAIL_ADDRESS>Cc:Ravi Kachhawaha<EMAIL_ADDRESS>Subject:Re: [ngFormioGrid] Expense manager customized ngFormioGrid (#5)

I would say for now hold off until I commit my changes I have been making. What I would then like is for you to re-do your changes on top of those. I should be done by the end of this week.
Thanks,
Travis.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub￼

This is moved to another pull request now.


feat: implement require macros and fix log docs

Basically want to provide macros to mimic similar behaviour to the rust assert! macros without it adding extra rust details and file information about the assertion.
Built on #492 to avoid conflicts and use the correct method signature, but can switch if we want this and not that PR.
Optional which opinions would be appreciated:

Should the no message call panic() syscall directly? I assumed not because it seems like a micro-optimization that would be hard to debug by default. Pinpointing that it's a require statement might help developers

@matklad would love a quick opinion on this if you think this is a good direction for having this type of assertion available without the Rust internals.

Couple of thoughts:

I wonder if we should just call this assert. The require name doesn't really point to the difference

I personally think that keeping it named require to avoid collision with the std prelude and making it clear that it performs a different function is good rather than having unexpected side effects if someone does/doesn't import the near_sdk::assert, do you have a strong opinion on naming it assert?

You probably don't need _eq/_neq versions -- they exist solely to provide better assertion messages. we don't allow assertion message, assert!(x == y) is enough.

Good point! I'll remove those for now

Would be cool to get some specific numbers on code saving here: I know that fmt is heavy in principle, but I don't have a gut feeling for how much.

So it's hard to tell, since most of fmt is still pulled in for all contracts using the sdk, but roughly can see the contract size changes in #514 (since now we are tying the example builds with docker to the CI checks)

Just in case of ambiguity, I'm assuming your approval as invalid @mikedotexe because I've made changes since your approval.


Initial render of the editor is not visible

wheni use the markdown editor the first time it opens it will not show anything until i set focus to the textare, after that the behavior is as expected.

Yes, I noticed that. Soon I will release a new version to fix it

With the new release, this issue is resolved. Please check and confirm


Running venv Python scripts from deploy.sh - source: not found

Hi, I am using CentOS Linux release 7.6.1810, webhook v2.7.0 from snap.
I'm getting an error trying to call python via venv from a deploy bash script -
[{
    "id": "my_hook",
    "execute-command": "./deploy.sh",
}]

deploy.sh:
#!/bin/sh

source venv/bin/activate
python -c 'print("test")'

# ... some other bash scripting would continue here ...

[webhook] 2020/10/28 18:03:29 [f2db3e] command output: ./deploy.sh: 3: ./deploy.sh: source: not found
Running ./deploy.sh works fine on it's own, it has +x. I'm trying to run a bash script that does a bunch of things, including running a Python script

You should be able to simply run python from the venv path.
#!/bin/sh

/path/to/venv/bin/python -c 'print("test")'

I tried that too, eg, deploy.sh:
#!/bin/sh

./venv/bin/python -c 'print("test")'

Runs fine on its own, but via webhook gives me:
[webhook] 2020/10/28 18:37:14 [d49d98] command output: ./venv/bin/python: error while loading shared libraries: libpython3.6m.so.1.0: cannot open shared object file: No such file or directory

How are you running webhook?  Are you running it as a system service?
Where is libpython3.6m.so.1.0?  What does ldd ./venv/bin/python show?
Also, I'd download a pre-built binary from the Releases page and try that.  I'm suspicious of the snap installation.

Thanks, it works fine using the pre-built binary


High CPU Usage

Hi. Thanks for this project! It's made integrating my USB-UIRT, Broadlink and Harmony Remote integration into Home Assistant much better.
I am having an issue with high CPU usage though.
I am running on Ubuntu 20.04 server through Docker. I got lirc installed and working with my Harmony 650 remote. I got MQTT installed and accessible with Home Assistant. I installed this with docker-compose and it works just like I need it to.
When this container is running, my CPU usage is around 30% and temperature around 50° C. If I stop the container, it goes down to about 16% CPU and 34° C.
I just installed the Glances docker container, and under the docker section, it shows this lirc container at using about 98 % CPU% and in the Tasks section, it shows python3 using 99 - 100.1% CPU%. Stopping the container removes both of these entries from Glances.
Any thoughts on what's causing this?
Here is my docker-compose.yml file for docker-lirc-watcher
version: "3"
services:
  lirc:
    container_name: lirc
    restart: always
    image: pilotak/lirc-watcher
    environment:
      - MQTT_BROKER=<IP_ADDRESS>
    volumes:
      - /var/run/lirc/lircd:/var/run/lirc/lircd

Thanks for this detailed issue. I had the same problem but somehow after an upgrade it got fixed. I will try to send a fix tomorrow

Awesome! Thanks

@travisdneel are you able to build a this image locally? i can send you updated docker-compose.yml should you need.
I pushed an update

First you need to be in the same directory where docker-compose.yml is, then download this repo
cd # location where is docker-compose.yml
git clone https://github.com/pilotak/docker-lirc-watcher

Then you need to alter the compose
version: "3"
services:
  lirc:
    container_name: lirc
    restart: always
--- image: pilotak/lirc-watcher
+++ build: ./docker-lirc-watcher
    environment:
      - MQTT_BROKER=<IP_ADDRESS>
    volumes:
      - /var/run/lirc/lircd:/var/run/lirc/lircd

And finally run
docker-compose up -d --build

Excellent! That worked. Up and running with lower CPU usage and temp! Thanks so much!

perfect i will release it, so please change back the docker-compose file


himportscan upgrade for GHC 9.6

himportscan upgrade for GHC 9.6
missing change 9_4 -> 9_6
add ghc_9_6_2 for CI
add ghc_9_6_2 in .bazelrc
Add stack config for GHC 9.6.2

@mergifyio rebase

@mergifyio rebase

@wavewave I took the liberty of rebasing your PR #60 and adding the missing stack config for GHC 9.6.2 so that CI is now successful. Thank you!


spec: update spec text with proposed semantics

Update the spec text to reflect the new, proposed semantics, including
the use of largestUnit, smallestUnit, three options to hide zero values
as well as rounding durations before formatting.
Fixes: https://github.com/tc39/proposal-intl-duration-format/issues/32

@FrankYFTang the output of the values is not changed here and is not relevant. Regarding the input, I've used the same abstract operations from Temporal so both plural and singular should be accepted (and converted to plural internally). For internal use, plural is still used and that's how it is supposed to be, so no, the table should still contain plurals.
For reference, the records returned have plural fields (https://tc39.es/proposal-temporal/#sec-temporal-topartialduration) and therefore the table needs to be in plural form.

@Ms2ger @FrankYFTang updated everything, PTAL now, things should be much better hopefully...
@Ms2ger

backticks need to become asterisks at some point too

oops, I switched from asterisks to backticks because I'd seen them in other specs and though we were moving from asterisks to backticks...
I just checked and you're right, ECMA 402 uses asterisks. Let me do a search and replace.

@FrankYFTang the output of the values is not changed here and is not relevant. Regarding the input, I've used the same abstract operations from Temporal so both plural and singular should be accepted (and converted to plural internally). For internal use, plural is still used and that's how it is supposed to be, so no, the table should still contain plurals.
For reference, the records returned have plural fields (https://tc39.es/proposal-temporal/#sec-temporal-topartialduration) and therefore the table needs to be in plural form.

I don not understand how could resolvedOptions work if we keep using the plural internally in Table 1
#sec-properties-of-intl-durationformat-instances
[[SmallestUnit]] is one of the String values listed in Table 1 identifying the smallest unit in the input to be formatted.
[[LargestUnit]] is one of the String values listed in Table 1 identifying the largest unit in the input to be formatted.

and #sec-Intl.DurationFormat.prototype.resolvedOptions
4. For each row of Table 2, except the header row, in table order, do
a. Let p be the Property value of the current row.
b. Let v be the value of df's internal slot whose name is the Internal Slot value of the current row.
c. Assert: v is not undefined.
d. Perform ! CreateDataPropertyOrThrow(options, p, v).

and
Table 2: Resolved Options of DurationFormat Instances
...
[[SmallestUnit]] | "smallestUnit"
[[LargestUnit]] | "largestUnit"

So resolvedOptions will output [[SmallestUnit]] directly. And [[SmallestUnit]] is what is listed in Table 1. But Table 1 is coded in Plural. Then how could resolvedOption return them in singular form?

Maybe we need to add an abstract operation to convert Plural form to Singular form and call it from #sec-Intl.DurationFormat.prototype.resolvedOptions and #sec-partitiondurationformatpattern ?

New update PR at #66, closing this.


feat: add openapi extension for auth envvar names

Once we settle on the format, up to add a test in x-fern-openapi-ir-to-fern ? link


Stablize Error API

Reduce or eliminate anyhow dependency without reducing usability, maintainability, or performance. Improving the error handling of the core can-types crate makes the functionality more approachable for a wider range of developers.
Some avenues for this issue:

Custom error structs/enums that do not rely on std.
Beefier error handling using std behind feature gates.
C-like compact error types using singular core types (i.e. integer error values)
Forego error messages in favor of optional return values.

Dependence reduced to Conversion trait. Generic to allow core::Error impl once stabilized.


potential performance improvements for GRIB files

I've been playing around a bit on reading GRIB files, but quickly became hit by the performance impact of the temporary files being created by kerchunk/grib2.py. Thus I tried to find ways around this. As far as I understood up to now, cfgrib requires access to entire files and also requires some file-API while eccodes is happy with in-memory grib messages as well. So I tried to read in grib files using mostly eccodes and circumventing cfgrib where possible, which is orders of magnitude faster than the current method implemented in kerchunk, but sadly, it doesn't do all the magic cfgrib does in assembling proper datasets in all cases. This lack of generality is the reason why I'm not proposing a PR (yet?), but rather seek for further ideas on that topic:

Do others work on this as well?
Do you have ideas on how to do the dataset assembly more generically?

Here's how I'd implement the "decompression", which I belive is relatively generic (but may still be incompatible with what the current kerchunk-grib does):
import eccodes
import numcodecs
from numcodecs.compat import ndarray_copy, ensure_contiguous_ndarray

class RawGribCodec(numcodecs.abc.Codec):
    codec_id = "rawgrib"

    def encode(self, buf):
        return buf

    def decode(self, buf, out=None):
        mid = eccodes.codes_new_from_message(bytes(buf))
        try:
            data = eccodes.codes_get_array(mid, "values")
        finally:
            eccodes.codes_release(mid)

        if hasattr(data, "build_array"):
            data = data.build_array()

        if out is not None:
            return ndarray_copy(data, out)
        else:
            return data

this gist shows how it may be possible to scan GRIB files without the need for temporary files

That's at https://github.com/TomAugspurger/cogrib.

It indeed scans files without first downloading them, and we plan to upstream it here.

That cogrib experiment does need to download the whole file when it's being "kerchunked". User's accessing through fsspec's reference filesystem don't need to download it, and it doesn't need a temporary file.
It'd be nice to avoid the temporary file for scanning too, but one of my desires was to match the output of cfgrib.

cogrib looks very nice :+1:
And yes, the isssue with cfgrib-compatibility is what bothers me most as well in my current attempt (I chose to drop compatibility for speed). I'd really hope we'd be able to figure out a way to do both: no temporary files and cfgrib compatibility.

Actually, can you please enlighten me what "compatibility" means here? I thought cfgrib was a pretty thin wrapper around eccodes.

As far as I understand GRIB (I'm really bad at this), GRIB doesn't know about dimensions and coordinates which are shared between arrays. GRIB files consist of messages (which are chunks + per chunk metadata) and nothing shared by those messages. cfgrib guesses how to assemble those messages to a Dataset based on what it finds among the per-message metadata.

As always with guessing, there are multiple options on how you might want to do this and which kind of conventions are to be followed, so when rolling this guesswork on your own, you might end up with something different.

That matches what I mean by compatibility too. The output of kerchunking a GRIB file should be a list of datasets equal to what you get from cfgrib.open_datasets(file). I'll want to stretch that definition a bit to handle concatenating data from many GRIB files along time, but the basic idea is that I don't want to guess how to assemble messages into datasets.

From working previously on gribs, I do also want to add, that for some files, you cannot use open_datasets without appropriate filters being supplied, because or coordinates mismatch between messages.

Do you mean open_datasets (plural) or open_dataset (singular)? I don't think I've run into files where open_datasets fails, but I haven't tried on too many different types of files.

Yes, single

We've been working a bit more on our gribscan, which is now also available at gribscan/gribscan. It's still very fragile, deliberately doesn't care about being compatible to the output of cfgrib and potentially requires users to implement their own Magician.

Magician?? :)
Do you intend to integrate any of the work into or even replace grib2 in this repo? Do you have any comments on how it compares with @TomAugspurger 's cogrib?
Note that with the latest version of numcodecs, you no longer need to import and register your codec, but can instead include install entrypoints.

:-) yes, we call the customization points a Magician because that's the part where users have to put their guesswork of how to assemble datasets to "magically" stuff the grib messsage together.
That's also the biggest difference to cogrib: We do not try to have a universal tool which makes some dataset out of almost any GRIB. Instead we require customization to make the resulting dataset nicer. That's under the assumption that someone who is involved in creating the inital GRIBs might put some valuable knowledge into it.
The latest version of numcodecs isn't released yet... We've got the entrypoints set up, but they don't yet work 😬

Currently it works for some GRIBs, but is not really stable yet and we need to gain more experience... Thus we thought it might need a little time before we really want it in kerchunk.

@TomAugspurger , I'm sure your thoughts on gribscan would be appreciated, if you have the time to look.

The magician looks quite a lot like what happens in MultiZarrToZarr - if each of the messages of a grib were made into independent datasets, and combined with that, then maybe you wouldn't need your own mages. Sorry, sorcerers, ... er magicians.

Initially we've had a design which built one dataset per grib-file and then put all of them into MultiZarrToZarr. We moved away from that design, because we needed something which looks at the collection of all individual messages. But we didn't come up with the idea to make datasets out of each individual message.


Fixes and more tests

One test I've added is failing with SM. We need to decide if we need that test, and if so what should be the behaviour of the container implementation. (basically this is invalid configuration, not sure if we should have tests like that).
All others are just extension for all factory types.
We need to think where to put "factory" data provider as it is used between traits right now.

These changes are all incorporated in #1.


Adding icons  for different sections on the navbar ,such as contact , about , services etc:-

Adding icons for different sections on the navbar such as for doctors, services etc.

@Anishkagupta04  kindly assign me this issue under gssoc'24

@varshith257 I have already made the PR and it is merged

Can you link the PR here ?

@ #88


OCPBUGS-33154: Incorrect use of go wait groups

Minor change to the wait group to ensure the controller does not hang.
The wg.Add(1) (line 198) should be moved closer to the function it is waiting for (line 226).
Having a function (that could exit) between wg.add() and the function with the matching wg.done() could lead to a state where the controller hangs, waiting for the wg.done() that is never called.

/test e2e-azure-ovn
Unrelated.

/lgtm
Thanks @squirrd!


Automatize tests and quality check

Add TravisCI & CodeClimate (unfortunately ladscape.io unstable) to the project

TravisCI is there already. I just added CodeClimate. Gonna add Coveralls and commit soon.


Normalize line endings before comparing string in tests

PR Summary
Fix tests which fail on comparing string due to differences in line endings.
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@PoshChan please retry static

@adityapatwardhan, successfully started retry of PowerShell-CI-static-analysis

@PoshChan Please rebuild static

@adityapatwardhan, successfully started rebuild of PowerShell-CI-static-analysis

@TravisEz13 Can you have a look


[4108] Fix missing education phase

Context
https://trello.com/c/F69RQGhy/4108-dttp-missing-education-phase
Changes proposed in this pull request

Data migration that fixes missing course_education_phase for Trainees that is not early years related and have an age range

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 DTTP: missing education phase


Remove container runtime related code

What this PR does / why we need it:
Remove container runtime-related code since it will be moved to operating-system-manager. The node- flags have been deprecated since machine-controller can no longer control those configurations since it doesn't generate user data.
I have also removed flags that have been in a deprecated state for more than a year.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind cleanup
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Flags that were used to control user-data for machines have been deprecated. OSM should be used to configure those settings.

Documentation:

TBD

/approve
/lgtm


Ifc Structural analytical model display issue

Describe the bug 📝
I need to display IFC classes relating to Structural FEM model, but it seems that Fragment engine does not currently support them:

IfcStructuralAnalysisModel
IfcStructuralCurveMember
IfcStructuralSurfaceMember
IfcStructuralCurveConnection
IfcStructuralPointConnection
IfcStructuralPointAction
IfcStructuralLoadGroup
IfcStructuralLoadCase
IfcStructuralLoadPlanarForce
IfcStructuralLoadSingleForce
IfcStructuralPlanarAction
IfcStructuralPlanarForce
... and maybe some other structural classes.

Model building_01.ifc (https://github.com/IfcOpenShell/analysis-models/blob/master/ifcFiles/building_01.ifc) in ThatOpenViewer:

The same building_01.ifc model opened in BlenderBIM:

Reproduction ▶
No response
Steps to reproduce 🔢

Download test IFC FEM files from IfcOpenShell repo: https://github.com/IfcOpenShell/analysis-models/tree/master/ifcFiles
Try loading these files with ThatOpen viewer: they seem to load, but no geometry is shown to the user.

System Info 💻
n/a

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[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 repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is not a bug, but a feature request. I'm changing the tag.


Re-designed Footer

I read the issue #53 and decided to work on it.
Since it is my first time contributing to a project.
I will be very glad if you are satified with my work, and give me some advice to work on my short-comings to improve myself.
thank you.

You branch have merge conflicts with main branch!


alias BUG

[√] Read the docs.
[√] 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 https://github.com/vuejs/vue-next instead.

Describe the bug
在对于被使用别名引入的scss文件，在被引入的这个文件中所使用别名引入的静态文件没有被替换为正确的路径。
Reproduction
github链接
System Info

vite version: 2.0.1
Operating System: Mac OS 10.12
Node version: v13.14.0
Package manager (npm/yarn/pnpm) and version: npm/6.14.4

you should append the extension

you should append the extension

I try append the extension, But not yet。
resolve: {
    alias: [
      { find: "@", replacement: path.resolve(__dirname, './src') },
      { find: "~sass", replacement: path.resolve(__dirname, './src/assets/sass') },
      { find: "~assets", replacement: path.resolve(__dirname, './src/assets') }
    ],
    extensions: ['.js', '.vue', '.json', '.scss', '.css']
  }

@import "~sass/app.scss";

@import "~sass/app.scss";

Loading SCSS file is normal,
But I want to use aliases in the SCSS file as well.
Use @import "~sass/app.scss"; is not working,
But use @import "./assets/sass/app" is normal.
I think it's a bug.

using url() to rebase the url can solve this
but I am not sure whether this is a bug with bare @import

I also have this bug.
I use path alias.
<style lang="scss" scoped>
@import<EMAIL_ADDRESS>
</style>

but throw errors
[plugin:vite:css] Can't find stylesheet to import.
  ╷
2 │ @import<EMAIL_ADDRESS>  │         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  ╵

user relative path works.
<style lang="scss" scoped>
@import "../../assets/styles/variables.scss";

</style>

"vite": "^2.0.5"
"vue": "^3.0.7"

Still Can't find stylesheet to import when I update to vite 2.1.3

@xiaohulu would you open a new issue with a minimal reproduction against vite 2.1.3? Thanks!

Thanks for the bug fix, I also came across this earlier.
Now, in sass I can use the tilde aliases in ~/styles/foo.scss  or background: url("~assets/images/foo.gif").
However in a .vue template image file, the tilde doesn't work : eg. <img src="~assets/images/logo-rune.png" ...
Using an at symbol style of alias like @assets/... works in both vue <img<EMAIL_ADDRESS>and scss file: background<EMAIL_ADDRESS>...`
I imagine the ~ tilde style alias doesn't work in .vue file because it's considered to be a node_module right?
So am I correct in saying then, that as best practice, weŕe better off to just always use at symbole style aliases if they work in both cases ?  ( eg. @assets )|
Is there any point to using tilde alias if it doesn't have a consistent behaviour betweenm vue and sass imports?


Reopen editor with context key is based on active not selected editor

Repro:

Open welcome page
Right click tab, notice there is no reopen editor with option
Click another tab
Right click the welcome page, reopen editor with is there

It's interesting as the reopen with is actually being performed on the untitled file even though you have the welcome page selected

Looks like it shows up based on the active editor, not the selected editor:
https://github.com/Microsoft/vscode/blob/97d576cb08586d33e30419c91da05a80acf8cf8f/src/vs/workbench/browser/parts/editor/editor.contribution.ts#L308-L308

@lramos15 when implementing context menu actions for editor tabs, you need to look at the passed in context instead of just going to the active editor. Here is an example how the "Close Others" action knows what the editor is under the mouse:
https://github.com/microsoft/vscode/blob/75aa22e7d99b6353bfc4543218d50d04fbbe04cf/src/vs/workbench/browser/parts/editor/editorCommands.ts#L814-L814
If a method there is not exported but you need it, we can see to export it maybe from editor.ts.

Similar for commands that are shared between explorer and tab:
https://github.com/microsoft/vscode/blob/853f91719012c16d1cbfa6f55678e54700d61c7f/src/vs/workbench/contrib/files/electron-sandbox/fileActions.contribution.ts#L39-L40

@bpasero Thanks for the pointers I have gone ahead and implemented this. The last remaining issue is that the context key for showing the command is based on the active editor and I didn't seem to find an example of one based on the selection. Should we just no-op in that case similar to if you select the last editor tab and click Close to Right

Yeah true, we might not have a context key for that yet. I am fine doing a no-op then.

Do we think this is good to close? There's no context key for selected editor which this is the scenario which I can cover; if a user selects another tab and does reopen with, operate on that if possible. If an editor can't be reopened such as Welcome then No-op. The downside is that if the user has the welcome tab active the enablement is based on the active tab and therefore other tabs would be disabled from being selected and reopend with. This was the same as before and not new behavior.

This seems to work OK, but I would expect the editor to become active when I picked "Reopen with...". Currently the editor seems to stay inactive.

I'm going to disagree just because no other actions like pin, or split make the editor which is acted upon active.

Yeah but this action is specifically saying "Re-Open", isn't that a hint that the user wants to see the contents?

Adding focus isn't an easy implementation right now. File #134131 for underlying problems. Closing this one


Update getPropByPath to support braces and object keys with special characters

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

This PR is related to #22175, #10293, and #17893. Although it's rare, some of us need to lookup props using brace notation and keys that contain special characters. For example when our data looks like this and we need to validate the mountains['Mountain: Mt. Blanca'] property.
{
  mountains: {
    'Mt. Fiji': 1,
    'Mountain: Mt. Blanca': 2
  }
}

This PR updates getPropByPath so it can parse object key paths using brace notation and parse keys containing special characters. It also adds test cases for the getPropByPath function.
I do have an outstanding question related to this code in form-item.vue.
if (path.indexOf(':') !== -1) {
  path = path.replace(/:/, '.');
}

This code will prevent the use of semicolons in object keys. Do you think there is a way to safely remove this? Is there an alternative? This conversions seem like an undocumented feature that the users shouldn't be relying on, but I don't know the code well enough to be certain.

Why hasn't there been any progress


Handle GitHub "Use this template" button

Remome /app dir, move app source to root to be ready for GitHub Use this template button.
Also add package-lock.json to default .gitignore.

It would case problems with boilerplate changelog


Handling multiple requests

Hello, I am working on an add-in that needs to be able to make multiple requests at once.  The idea is:
Get user input from cell -> Package this in a web request -> Our backend sends back a response based on what's sent in the web request.  I have no trouble setting up the request response cycle, but am having some issues with latency, and am wondering if anyone has suggestions so that I can make about 1000-1500 queries at a time.
I thought about making the request, then saving the response content to the excel file, but the problem with this is that the backend data updates on an unpredictable schedule.  Therefore, a user using this saved response could end up with old data in their spreadsheet.
Thanks

1000 requests at a time? Are you developing a DDoS attack app?

No.  Forgive me if that seems a bit unrealistic, it is a specification of the person overseeing the product.  The idea is that a user should be able to get updated data for their stocks from our backend.  My boss has some experience using bloomberg excel api, so I assume that is where he is getting the idea from.
As it is right now a user can make multiple requests, but this slows down once the number of queries increases.  I’m looking for a way to reduce this time.  Thanks

But if you have to submit requests in parallel, this is possible using the async approach.

Appreciate the feedback Sophist.  I was able to run the requests in parallel like you suggested, but it raises another concern.  I'm assuming that in order to get this to work, the client need the WinHTTP Services reference selected.  I want this add in to be ready to go out of the box, and not require the user to have to do configuration.  So now I ask, am I wrong in my assumption?  If not, is there a way for me to turn this reference on with my add-in?
Thanks

I don't know - but the reference is turned on in the VBA project i.e. the Excel spreadsheet or Excel Addin.
So distribute it as one of these and you should be OK.

Got it to work.

Did you make your API support multiple stocks in one call - believe me, in real life over a slow connection populating 1,000 stock details using 1,000 calls will be unbelievably slow. So if 1,000 is your design point, your API should support at least 100 or 200 stocks in one call.


How can I simplify a surface of face with this lib?

Hello.
In the code fit-model.cpp, it uses the default model file sfm_shape_3448.bin to generate the mesh.
But I want to make the surface more simplified, which means less vertices.
How can I generate sfm_shape_3448.bin and some files else depended  to do this?

You could use the topology of the reference.obj. You could reduce the 3448 model to 845 vertices using that topology. If you want to keep the PCA bases orthogonal (or orthonormal), you may want to reorthogonalise after sub-selecting the lower-resolution basis.

Hi, @patrikhuber ,
but how can I generate the *.bin files from the refernce.obj  via eos?
Just like in the example code fit-model.cpp, it needs 3 more files related to the sfm_shape_3448.bin.

You need to write an app (in C++ or with the Python bindings) that does what I outlined above. I'd start with having a look at the documentation for the MorphableModel class and its members.

I'm going to close this, feel free to re-open if you have specific issues with the library.


chore: Release (0.2.6)

:robot: I have created a release beep boop
0.2.6 (2024-05-21)
Bug Fixes

Remove run prefix (13d0a6c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/crate-crypto/peerdas-kzg/releases/tag/v0.2.6 :sunflower:


Validate the parent inference service is ready in trained model controller

What this PR does / why we need it: Prevents the trained model from being deployed if the inference service is not ready
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer: I plan on adding a few other validation checks in controllerafter this PR

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

None

/LGTM

/ok-to-test

/ok-to-test

/LGTM

/lgtm
/approve


Wifi issues with AC 7265

Seems to be related to https://bugzilla.kernel.org/show_bug.cgi?id=200779
Updating the kernel to 4.19 or reverting https://github.com/torvalds/linux/commit/e31f6456c01c76f154e1b25cd54df97809a49edb should fix it.
 Front conversations

Kernel 4.19 will be in when we'll have support for Yocto Warrior and we migrate this repository to Warrior

We can migrate this repo to warrior when https://github.com/balena-os/meta-balena/issues/1458 is done


Some queries on your application

Dear Omkar,
I have a couple of queries on the you application, they are as below

Can you provide install and use instructions? Like which OS, prerequisites etc?
Can your application work without internet after installation?
Are there any android client which you have tested ans works with the application developed?

Regards,
Jagannath S

Dear @JagannathS JagannathS
So far i have gone through this application so answering your questionnaire.

You're right they need to provide installation guide.
But it can be work on any platform because they used javascript.

No online application can work without internet bro.

As far as my testing, I have tested this app in Android, IOS, Desktop as well as web app.

Regards,
Will Meet

when i run this project in eclipse showing the error in pom.xnl file


Undefined array key sezzlepay

Hello everyone.
I am having problems with Sezzle, because it is giving me an error when I give it "save configuration" in the store configuration, when I review the error, it indicates this message:
{"0":"Warning: Undefined array key "sezzlepay" in /var/www/vhosts/dev9.sbmedicaldeveloper.com/httpdocs/vendor/sezzle/sezzlepay/Plugin/Config/Controller/Adminhtml/System/Config/SavePlugin.php on line 113","1":"#1 Sezzle\Sezzlepay\Plugin\Config\Controller\Adminhtml\System\Config\SavePlugin->aroundExecute() called at [vendor/magento/framework/Interception/Interceptor.php:135]\n#2 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->Magento\Framework\Interception\{closure}() called at [vendor/magento/framework/Interception/Interceptor.php:153]\n#3 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->___callPlugins() called at [generated/code/Magento/Config/Controller/Adminhtml/System/Config/Save/Interceptor.php:23]\n#4 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->execute() called at [vendor/magento/framework/App/Action/Action.php:111]\n#5 Magento\Framework\App\Action\Action->dispatch() called at [vendor/magento/module-backend/App/AbstractAction.php:151]\n#6 Magento\Backend\App\AbstractAction->dispatch() called at [vendor/magento/module-config/Controller/Adminhtml/System/AbstractConfig.php:69]\n#7 Magento\Config\Controller\Adminhtml\System\AbstractConfig->dispatch() called at [vendor/magento/framework/Interception/Interceptor.php:58]\n#8 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->___callParent() called at [vendor/magento/framework/Interception/Interceptor.php:138]\n#9 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->Magento\Framework\Interception\{closure}() called at [vendor/magento/module-backend/App/Action/Plugin/Authentication.php:145]\n#10 Magento\Backend\App\Action\Plugin\Authentication->aroundDispatch() called at [vendor/magento/framework/Interception/Interceptor.php:135]\n#11 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->Magento\Framework\Interception\{closure}() called at [vendor/magento/framework/Interception/Interceptor.php:153]\n#12 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->___callPlugins() called at [generated/code/Magento/Config/Controller/Adminhtml/System/Config/Save/Interceptor.php:32]\n#13 Magento\Config\Controller\Adminhtml\System\Config\Save\Interceptor->dispatch() called at [vendor/magento/framework/App/FrontController.php:245]\n#14 Magento\Framework\App\FrontController->getActionResponse() called at [vendor/magento/framework/App/FrontController.php:212]\n#15 Magento\Framework\App\FrontController->processRequest() called at [vendor/magento/framework/App/FrontController.php:147]\n#16 Magento\Framework\App\FrontController->dispatch() called at [vendor/magento/framework/Interception/Interceptor.php:58]\n#17 Magento\Framework\App\FrontController\Interceptor->___callParent() called at [vendor/magento/framework/Interception/Interceptor.php:138]\n#18 Magento\Framework\App\FrontController\Interceptor->Magento\Framework\Interception\{closure}() called at [vendor/magento/framework/Interception/Interceptor.php:153]\n#19 Magento\Framework\App\FrontController\Interceptor->___callPlugins() called at [generated/code/Magento/Framework/App/FrontController/Interceptor.php:23]\n#20 Magento\Framework\App\FrontController\Interceptor->dispatch() called at [vendor/magento/framework/App/Http.php:116]\n#21 Magento\Framework\App\Http->launch() called at [vendor/magento/framework/App/Bootstrap.php:264]\n#22 Magento\Framework\App\Bootstrap->run() called at [pub/index.php:30]\n","url":"/admin_baiq83FNQt2d26W2K6Vuw5AQ6UVqua/admin/system_config/save/key/c8424a6fca5604a6f8bac98cabeb55424e271fce658a2890457c4584c5e07240/section/themecore/","script_name":"/pub/index.php","report_id":"6a808a4b9d2b1113b9c6d01d87c134e8744d2fe859318c16c1b134aec2bd319f"}
I am running magento 2.4.5-p1 and php 8.1

@gustahthorres A new version 7.0.7 has been that contains the fix.