Production2021 updates

Updates mainly to the analysis routines. In particular:

Add in a simpler version of the code in analyse-results.py. Apart from simplifying the code, the main difference is to force selection of QC checks in a particular order e.g. pick a range check, then a climatology check etc. This was implemented following feedback from colleagues.
Implement two extra features in db_to_df:

When removing levels flagged by a particular test, allow additional levels around the flagged level to be removed also. This handles the situator where a QC operator or automatic QC test makes slightly different decisions on which levels to flag for the same issue.
Allow removal of levels deeper than a rejected level if it is an XBT. This may be done as once an XBT fails, the rest of the profile may be thought of as suspect.

Fix minor issues due to change in syntax in the latest version of pandas.

Thanks for the review, @BillMills. I've had a go at rewriting db_to_df to make it clearer and have added in testing. One part of it I haven't been able to test is the looping over subsections of the database. I changed this from doing 1000 loops of nprofiles/1000 profiles to doing nprofiles/1000 loops of 1000 profiles. I thought this would be more efficient for smaller databases. I think it is correct, but it is difficult to test if it is working as even the testing on 2 profiles I've added into tests/dbutils_validation.py is quite slow. I've also made the mark_training changes you asked for - I'd like to keep this functionality as I think it is useful to include.

@s-good thanks for these updates, looks great! Per your comments, I extended this in #271 with a simple unit test to at least check that db_to_df doesn't change based on batch size, but these changes are good to go, feel free to merge.


Realtime-chat-app-using-Python

Realtime-chat-app-using-Python
Chat Room has been the most basic step toward creating real-time and live projects. The chat page that we will create will be a simple HTML boilerplate with a simple h1 text with the name of the current user and a link to log out to the user who is just logged in. You may need to comment on the line until we create auth system for this
Hope you will like it
and dont forget to add the Hacktober fest label

Create this issue in amazing Web Projects

Create this issue in amazing Web Projects

Please share the issue link section


moving couch installs to encrypted drive

@nickpell @mkangia
on monoliths (and other servers running couch) the data directory is not on the encrypted drive. This fixes that for new installs. I will migrate old ones.

hooray!

thanks for fixing cal

build failing, not sure if real or transient?

build fail is real

✅


python

Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/requests/models.py", line 972, in json
return complexjson.loads(self.text, **kwargs)
File "/usr/lib/python3.10/json/init.py", line 346, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.10/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

still a bit of a noob when it comes to corrections bro, wouldve fixed it myself but i still have much to learn

guarda have now block Python API. I have Change the API to Atomic Wallet should be Working on newer version


Release/2.16.0

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

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

Project coverage is 69.92%. Comparing base (bf7a862) to head (0ecf62e).
Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2407      +/-   ##
===========================================
- Coverage    69.95%   69.92%   -0.03%
===========================================
  Files          763      763
  Lines        28377    28367      -10
  Branches      4763     4763
===========================================
- Hits         19850    19835      -15
- Misses        7232     7233       +1
- Partials      1295     1299       +4

see 20 files with indirect coverage changes

----
🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Master

Added new content from expansion

Everything except the latest commit should already be in the master branch. If not, I've been lagging behind in my commits.

There is definitely something up, you have merge conflicts. If you could fix those and open the PR again that would be great.


After latest update, it bugged videos making audio to be duplicated

Same here

same

Your topic seems to be duplicate
Your topic seems to be duplicate


Extract traceparent from server calls in Non-Go apps

This PR adds ability for us to track and extract the traceparent from HTTP headers for received messages.
There are couple of things that need to be added:

[ ] Detection to send the received buffer only for server calls
[ ] Integration tests

Codecov Report
Patch coverage: 46.91% and project coverage change: -30.78% :warning:

Comparison is base (55bfb15) 79.78% compared to head (e1c12f0) 49.00%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #302       +/-   ##
===========================================
- Coverage   79.78%   49.00%   -30.78%
===========================================
  Files          42       34        -8
  Lines        3289     2669      -620
===========================================
- Hits         2624     1308     -1316
- Misses        504     1279      +775
+ Partials      161       82       -79

Flag
Coverage Δ

integration-test
?

unittests
49.00% <46.91%> (+1.07%)
:arrow_up:

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

Files Changed
Coverage Δ

pkg/internal/ebpf/common/ringbuf.go
79.77% <40.00%> (-4.57%)
:arrow_down:

pkg/internal/ebpf/httpfltr/httpfltr.go
30.32% <45.83%> (-58.30%)
:arrow_down:

pkg/internal/ebpf/common/common.go
62.50% <66.66%> (ø)

pkg/internal/ebpf/httpfltr/httpfltr_transform.go
100.00% <100.00%> (ø)

... and 28 files with indirect coverage changes

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

Thanks Mario!


bug: data requestor table updates is not logging actions

on the management dashboard, the data requestor table updates is not logging actions.
i made a request to access kalifabugu malaria cohort on qa.restricted, using an account tied to<EMAIL_ADDRESS>i approved the request, giving<EMAIL_ADDRESS>access:

This action was NOT logged in the data requestor table updates

this is still not working.  I just added my gmail account to PERCH Mali on prod and then deleted it and neither action showed up

looks great now!


When the 'inline' attribute is applied to the 'form' component, the 'select' component does not have a menu width.

TuSimple/naive-ui version (版本)
2.34.3
Vue version (Vue 版本)
3.2.47
Browser and its version (浏览器及其版本)
Chrome(111.0.5563.64)
System and its version (系统及其版本)
MacOs(12.4)
Node version (Node 版本)
v18.15.0
Reappearance link (重现链接)
https://codesandbox.io/s/determined-cookies-9wuded?file=/src/Demo.vue
Reappearance steps (重现步骤)
Please check the link.
Expected results (期望的结果)
Same as input, has width.
Actual results (实际的结果)
No menu width
Remarks (补充说明)

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Please support Windows

I'm working on a node binding of Avisynth (which is Windows-only). I'm running the tests on AppVeyor, which is like Travis CI but with Windows instead of Ubuntu. Therefore, I need to upload code coverage from node running on Windows.
I've had an email exchange with Jonathan from your team about this but it's been a few weeks so maybe it has stuck in a spam filter (here's a copy).
At any rate, I have put up a pull request that contains simple fixes that address the problem. Just merging that will add Windows support to codeclimate-test-reporter, and it's small, easy-to-review code, so I'd appreciate if you guys could take a look at it.
Happy New Year and hope you're not too overworked. Your service is pretty good :)

Thanks again for the PR on this. Sorry that it's taking us awhile to review and merge, but this is still on our list.


Update from upstream

Bring in the changes from puppet core for validation and keeping in line with what folks will be running once the provider here is removed.

This needs a fix for handling packages that look like 'www/curl'.  An attempt is made to install the package, even though 'curl' is already present.

This all tests out clean on my systems.  Using both package name and port origin for the resource title results in the correct package being installed.


Dissapear vanished poeples from /team invite

What is the feature request for?
Hi, could you please remove the disappeared people from /team invite? So that they don't appear there when they have vanish
Relevant links
No response
Additional Information
No response
Feature requests are not going to be implemented by booksaw, instead you will have to find another developer to make a pull request to implement the feature.

[X] I understand

(to add on as may as well be addressed at the same time, this also happens with /team info)

This bug has been fixed and the patch will be included in the next update. Thank you for reporting.


Issue with JSON format while GENERATING SESSION

May I know why all of a sudden I am getting the JSON error while generating session

secret_key = "....."
token="...."
cn.generate_session(secret_key, token )
Traceback (most recent call last):
File "F:\Program Files\Python311\dashapp\Lib\site-packages\requests\models.py", line 974, in json
return complexjson.loads(self.text, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\Lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\Lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\Lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "F:\Program Files\Python311\dashapp\Lib\site-packages\breeze_connect\breeze_connect.py", line 797, in api_util
json_data = response.json()
^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\dashapp\Lib\site-packages\requests\models.py", line 978, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "F:\Program Files\Python311\dashapp\Lib\site-packages\breeze_connect\breeze_connect.py", line 858, in generate_session
self.api_util()
File "F:\Program Files\Python311\dashapp\Lib\site-packages\breeze_connect\breeze_connect.py", line 816, in api_util
raise Exception("Invalid JSON format in API response")
Exception: Invalid JSON format in API response

Dear Saravanan,
Thank you for showing interest in Breeze API.
Generate session API is working fine now, please check.
If you still face any issue, please let us know.
[E29CF836]
Thanks & Regards,
Akash Sapkal,
Team Breeze API,
ICICI Securities

From: Saravanan Padmanaban @.>
Sent: 11 October 2024 07:54
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: Subscribed @.***>
Subject: [Idirect-Tech/Breeze-Python-SDK] Issue with JSON format while GENERATING SESSION (Issue #150)
CAUTION: This email originated from outside the organization! Do not click links, open attachments or reply, unless you recognize the sender's email address! Use 'Report suspicious Email' button in the toolbar to report Phishing mails. Beware!
May I know why all of a sudden I am getting the JSON error while generating session
secret_key = "....."
token="...."
cn.generate_session(secret_key, token )
Traceback (most recent call last):
File "F:\Program Files\Python311\dashapp\Lib\site-packages\requests\models.py", line 974, in json
return complexjson.loads(self.text, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\Lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\Lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\Lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "F:\Program Files\Python311\dashapp\Lib\site-packages\breeze_connect\breeze_connect.py", line 797, in api_util
json_data = response.json()
^^^^^^^^^^^^^^^
File "F:\Program Files\Python311\dashapp\Lib\site-packages\requests\models.py", line 978, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "F:\Program Files\Python311\dashapp\Lib\site-packages\breeze_connect\breeze_connect.py", line 858, in generate_session
self.api_util()
File "F:\Program Files\Python311\dashapp\Lib\site-packages\breeze_connect\breeze_connect.py", line 816, in api_util
raise Exception("Invalid JSON format in API response")
Exception: Invalid JSON format in API response
—
Reply to this email directly, view it on GitHubhttps://github.com/Idirect-Tech/Breeze-Python-SDK/issues/150, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3TQZ4Z5AUYWGAQZYUFH6MDZ24ZGXAVCNFSM6AAAAABPX7SBW6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGU4DAMRWGM2DGOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
Print this mail only if absolutely necessary. Save Paper. Save Trees." "The information contained in this e-mail and any attachments to this message are intended for the exclusive use of the intended recipient and may contain proprietary, confidential or legally privileged information. If you are not the intended recipient, please note that you are not authorised to disseminate, distribute or copy this e-mail or any parts of it or act upon/rely on the contents of this e-mail in any manner. Please notify the sender immediately by e-mail and destroy all copies of this e-mail and any attachments. Please also note that ICICI Bank or its subsidiaries and associated companies, (collectively "ICICI Group"), are unable to exercise control or ensure the integrity of/over the contents of the information contained in e-mail transmissions and that any views expressed in this e-mail are not endorsed by/binding on the ICICI Group unless the sender does so expressly with due authority of ICICI Group. Before opening any attachments please check them for viruses and defects and please note that ICICI Group accepts no liability or responsibility for any damage caused by any virus that may be transmitted by this email. Any unauthorized use of this email or any attachment(s) may be treated as infringement of right(s) including right of privacy under the applicable laws of India including Information and Technology Act, 2000. Thank you for your cooperation. Regulatory Disclosures are available on https://www.icicisecurities.com (for institutional business) and www.icicidirect.com (for retail business).

Unable to generate session due to the same exception today. Why is this issue reoccuring?

Dear Ujwal,
Could you please check now and still if you face any issue please let us know?
[E29CF836]
Thanks & Regards,
Hiten Solanki,
Team Breeze API,
ICICI Securities

From: Ujwal Jyot Panda @.>
Sent: Saturday, December 14, 2024 1:25 PM
To: Idirect-Tech/Breeze-Python-SDK @.>
Cc: BREEZEAPI @.>; Comment @.>
Subject: Re: [Idirect-Tech/Breeze-Python-SDK] Issue with JSON format while GENERATING SESSION (Issue #150)
CAUTION: This email originated from outside the organization! Do not click links, open attachments or reply, unless you recognize the sender's email address! Use 'Report suspicious Email' button in the toolbar to report Phishing mails. Beware!
Unable to generate session due to the same exception today. Why is this issue reoccuring?
—
Reply to this email directly, view it on GitHubhttps://github.com/Idirect-Tech/Breeze-Python-SDK/issues/150#issuecomment-2542988395, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3TQZ46PAOA3TCZOVY6XITT2FPP7DAVCNFSM6AAAAABPX7SBW6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBSHE4DQMZZGU.
You are receiving this because you commented.Message ID: @.***>
Print this mail only if absolutely necessary. Save Paper. Save Trees." "The information contained in this e-mail and any attachments to this message are intended for the exclusive use of the intended recipient and may contain proprietary, confidential or legally privileged information. If you are not the intended recipient, please note that you are not authorised to disseminate, distribute or copy this e-mail or any parts of it or act upon/rely on the contents of this e-mail in any manner. Please notify the sender immediately by e-mail and destroy all copies of this e-mail and any attachments. Please also note that ICICI Bank or its subsidiaries and associated companies, (collectively "ICICI Group"), are unable to exercise control or ensure the integrity of/over the contents of the information contained in e-mail transmissions and that any views expressed in this e-mail are not endorsed by/binding on the ICICI Group unless the sender does so expressly with due authority of ICICI Group. Before opening any attachments please check them for viruses and defects and please note that ICICI Group accepts no liability or responsibility for any damage caused by any virus that may be transmitted by this email. Any unauthorized use of this email or any attachment(s) may be treated as infringement of right(s) including right of privacy under the applicable laws of India including Information and Technology Act, 2000. Thank you for your cooperation. Regulatory Disclosures are available on https://www.icicisecurities.com (for institutional business) and www.icicidirect.com (for retail business).


Add information about STARTTLS and SMTPS.

Corresponds to gocd/gocd#723 and gocd/gocd#724.

Should it be SMTPS?

Yes. See: https://github.com/gocd/gocd/issues/724

Should it be SMTPS?

@mdaliejaz, I see what you mean. There was a typo. Fixed.


Add a const fn version of digest update

So for context, I'm working on a library for a special type of hash which involves a CRC32, but the implementation takes every character and makes it lowercase (ascii_lowercase) before it's hashed. Since this is a const fn, I'm not able to create a new allocations for the lowercased version. I've tried const generics, but that requires always knowing the input length at compile time, which is too restrictive. I've found a way to do this, by adding a const fn version of the update function in Digest, which I've implemented in this PR. Here's an example using it:
pub struct Hash40(pub u64);

pub const fn new(string: &str) -> Hash40 {
    let bytes = string.as_bytes();
    let algo = Crc::<u32>::new(&CRC_32_CKSUM);
    let mut digest = algo.digest();
    let mut index = 0;
    while index < string.len() {
        digest = digest.updated(&[bytes[index]]);
        index += 1;
    }
    let crc = digest.finalize() as u64;
    let length_byte = (string.len() as u8 as u64) << 32;
    Hash40(crc | length_byte)
}

This is kind of an undesirable way to calculate the hash anyway, so if we can find some other way to write this, that would be cool too

Is there a reason why we can't make the existing update function const?

Is there a reason why we can't make the existing update function const?

This is because Rust doesn't allow &mut in const fn functions yet. The tracking issue for it is here: https://github.com/rust-lang/rust/issues/57349

Sorry for the delayed response, but wdyt about making Crc::update public instead? Seems like exposing a single low-level API would address a lot of the edge cases without overburdening the high-level APIs.


DTK configuration for spack

Spack (github) is a tool for managing software tools versioning and dependencies.
HPC platforms are going to install Spack by default (@Rombur reference?), it may be a good idea to provide a configuration for it. The issue is that DTK is currently built inside the Trilinos, so it is unclear of the best way to approach this.
It should not be too hard to do, though. For instance, take a look at Trilinos configuration file.

I don't remember where I saw that spack will be used for summit and sierra but here is the reference for trinity http://insidehpc.com/2016/08/spack/

https://groups.google.com/forum/#!topic/spack/DRaOJQUn-TQ

Add spack installation docs similar to Cap docs

The TPLs Spack documentation is in #223.

Added a PR to upstream inclusion of DTK spack build in llnl/spack#4304

PR was merged.


Clarify what monitoring components the user gets when deploying Greenfield Deployment vs the Brownfield Monitoring Component

It might help customer distinguish when to use what?
Another suggestion is to create a list of BOM, that shows everything that this repo contains.

@husamhilal  I would love to learn more on the BOM.
Also, Greenfield and brownfield - I am meeting with Shaun this week to discuss the same. Would you like to participate?
@shaunjacob

Sure, the idea of BOM is to help navigate the AVS ESLZ repo. We have great stuff, but sometimes users get lost in finding resources they are looking for. So, Bill of Material (BOM) (or simple Index page), might be helpful to create and maintain so new users can find what they are looking for.
This week I have a workshop that I'm presenting to one of our partners. But if the time permits and works with my schedule,  I'm happy to participate in the conversations 👍

@husamhilal  on a semi-related note, still in progress but we have a wiki page. You navigate there by hitting the wiki link in the top right of the nav, or using the link below:
https://github.com/Azure/Enterprise-Scale-For-AVS/wiki
I manually moved these pages there but working on automation to keep the pages up to date and in sync.

Thank you @sblair01 . Please suggest once completed

Hello Husam, this is being resolved in the release.

Thanks a lot Prasad, much appreciated 👍


Skipping entity with Id Spam as soon as Player left RFTools dimension.

The fact is when any player is on our main RFtools Dimension TPS is nominal, but as soon as there is no more player in it (disconnected or in other dimension like overwolrd for instance), server console is spamming "Skipping entity with ID" message and TPS is flirting with 0. Does someone encounter same issue ? I can provide any relevant log if asked.
Thx in advance for your support.

What kind of things do you have in that dimension? Are there chunkloaders? Any special machines?

It is a Void dimension which is used as our main dimension where our bases are located. So all kind of machines are there and the chunkload is "powered" by Immibis Dimensional Anchor.

Was it always like this or did this only start recently? And perhaps something has changed that could have triggered this problem (pack update or whatever)? Trying to find something that can be a clue

I'm actually trying to find out which mod could trigger this by deactivate them one by one. But, so far, i didn't find out anything.

Btw McJty can you tell me where i can find some infos to reimport a RFtools Dimension in a brand new overworld? I'm pretty sure my old overworld is a bit corrupted. I would like to generate a new one and import back the dimension where are located all player homes from my SMP server.

Oh sorry for the late reply but didn't notice this question until now. Anyway, here is the answer: https://github.com/McJty/RFTools/wiki/RFTools-Dimension-Export

Hello,
Starting last night I have also started getting this behavior. I am hosting a small dedicated server at home, and when I SSH'd in this morning to check the message "Skipping entity with ID" ocurred about 5 times per second in the logs.
Reading this old ticket I tried joining the server and going to the (RFTools) dimension I build in. The messages ceased. Once I exit the dimension the spam starts again.
The dimension is a void word, with a main floating island (built, not generated) containing a lot of blocks from a lot of different mods. No new mods were added just before the problem started happening, and no config changes that removed an existing (tile)entity were passed for quite a while. The floating island is loaded by a ChickenChunks chunkloader.
The version of RFtools was 3.20. Realizing this was ancient, I updated to 4.21. Unfortunatly, the issue persisted with the exact same behaviour as before. Cool new textures though ;)
I will update here when I discover more. If there is any info I could supply you with to help solving this, please feel free to ask. I have some hobby forge/java experience so feel free to go all technical on me ;)
Attached is a (very messy, apologies) startup log from forge. GitHub does not like .log, so .log.txt it is...
latest.log.txt

Can you reproduce this if you replace the chickenchunks chunkloader with a chunkloader from MFR and setting it up so that the chunk with the curative vat is not loaded?

Just now I tried removing the spotloader in the chunk with the vat, removed my chickenchunks loader and placed a MFR loader with the same radius (and powered it, of course). After using /chunkloaders to confirm it indeed loading the same chunks as my chickenchunks one did, I left the dimension. The spam did not appear. Same for leaving the game completely.
However, removing the MFR one and placing the CC one back without the solving spotloader, also did not bring back the spam. I tried multiple setups that gave the spam before, but unfortunately (or for anything but this research, fortunately) the spam did not return.
tl;dr Could not reproduce with MFR, but original "faulty" ChickenChunks setup also did not reproduce anymore...
If this problem returns in any way I will update here.


Do correct cursor right with storing visual X in CursorRight action

The description was provided by  @vicencb:
Here is a concrete example to reproduce the issue i am referring to:
Given this text file:
1234
ABCD
EFGH
IJKL

Move the cursor to 1 if not already there, then SelectDown, then CursorRight and finally CursorDown.
After this, the cursor is expected to be at E but instead it is after the H.

I've just tested it: the issue is fixed.
As this issue matched the description of #3055 i tried to avoid creating a duplicate.
Are those the same or not?

I've just tested it: the issue is fixed. As this issue matched the description of #3055 i tried to avoid creating a duplicate. Are those the same or not?

No the #3055 is about the fact that when you select with mouse the start and end pointers aren't swapped when it is necessary. See new comment in that issue


Misc improvements

See commits
Sorry that this isn't the cleanest PR: probably should be a couple of separate ones. Also I'm not sure to what extent I'm adhering to your code style (probably not) and if I should add some examples/documentation to the stuff.
Let me know either way and I will fix!

Hey Erik, thanks for the PR! I've looked over it once already, but need some more time to finish reviewing.
I do have some questions in the meantime, though. What is your use-case for needing a different promise library? I'm not married to 'es6-promise' if you needed some functionality of Q or bluebird instead.
Also, do you have some sample code we could use as an example of how to use the new getPartData method? Maybe an example of how you are using it?
I have a few other comments but i'll have to post them later.

We are using Bluebird all over our code base so I didn't want to mix es6-promise and bluebird. Not sure how it's usually accomplished.
Yes I can add an example of getPartData.
Be as blunt as you want about the PR – happy to fix any issues/concerns!

RE: passing in the promise- I think you could still leverage bluebird just fine without needing to alter the promise that this library uses. For example:
'use strict';
var Bluebird = require('bluebird');
var ES6Promise = require('es6-promise').Promise;

function retrieveArray() {
  return ES6Promise.resolve(['foo', 'bar', 'baz']);
}

Bluebird.resolve()
  .then(retrieveArray)
  .spread(function (foo, bar, baz) {
    // es6-promise does not have a .spread method, but this works because of the bluebird chain
    console.log(foo, bar, baz);
    // => foo bar baz
  });

I'd rather not allow passing in a promise library because the Promises/A+ spec does not define that a promise must have the new Promise(function resolve, reject) { ... }); api, which is what is used inside the library. I realize that bluebird also has this API and that's why this works seamlessly for you, but I'd rather avoid any potential issues with someone passing in an incompatible promise.

On your note about Promise – it's fine, I can remove that commit and verify that it works when mixing Promise types. The only thing that would be mildly inconvenient for us is we use a lot of Bluebird's functionality internally so we would have to do some wrapping. No biggie though

lib/helpers/getMessage.js contains a lot of the same logic as getPartData, minus the decoding of the body. What would you think about us making /lib/helpers/getMessage.js do the decoding, and making getPartData use that? That way results from search() as well as results from getPartData() will have the decoded bodies.

thanks for working with me on this! Exposing a sane "fetch" public method is a great addition to the library.

Yeah that makes a lot of sense actually. I can try to make that work

Latest set of commits features everything you mentioned except merging getPartData and getMessage

looking good so far

I merged getPartData and getMessage now.
Maybe getPartData is unnecessary

Let me know if this is ever getting exhausting going back and forth. I have some more feedback, but I'm willing to contribute code to this PR too if you don't wanna have to do everything.

Will fix the remaining things.
I can do some more back and forth, not a problem. Would be nice to merge this PR by EOD though!
I have some other things separately, like putting labels on emails, but will put that in a separate PR

So, I'm not entirely sure that we're doing the decoding right. Correct me if I'm wrong, because you have probably worked with this part of node-imap more than I have. It is my understanding that struct indicates the structure of the "TEXT" body of the message. If the message is a multi-part message, then the part.body will be emitted as a string with boundaries denoting the different parts of the message, i.e. "_----------=_MCPart_1535811677". So in order to decode a single "part" of the body, we would need to break the body at the boundary and feed only that single part to the decoder. However, what we are doing here is just taking a single encoding type and decoding the entire retrieved body as that type, even if it is a multi-part body.
I'm guessing this is an artifact from moving the parsing logic from getPartData to getMessage. In getPartData, you are saying "get the data for only this part of the message" so it makes sense that you wouldn't be worried about multi-part bodies.
I'm thinking that if we don't wanna try to deal with multi-part bodies in getMessage at this time (i.e. it's not worth it to you to spend time on it) then the parsing logic is probably best kept in getPartData for now.
Sorry for reverting my stance on this. Just learned more about the process and how it all works as I was reviewing.

That's more than I knew :) I didn't know anything about IMAP until last week.
Isn't there some JS library to do multipart decoding though?
Either way I'm happy to move the decoding back

If you can move the decoding back, but use getMessage in getPartData, I think I'll call it good enough for now.

Sounds good to me

OK see if this one is cool

LGTM

nice!

i'm cleaning it up a bit then i'll release a new minor version to npm

ok cool – just sent you another PR with two minor additions

v1.4.0 published to npm


QR Code Reader

What's the Problem?
No response
Solution/Idea
Read QR codes from file or screen.
Alternatives
ShareX has a utility for this.
Priorities

Capability
Priority

Read QR codes from file/clipboard
Must

Read QR codes from screen
Should

DevToys Version
No response
Comments
Related #270

+1

How about additionally adding qr code creator?

How about additionally adding qr code creator?

That's tracked in #270. We can merge them if you want, but they seem to be different enough to have in separate issues.

How about additionally adding qr code creator?

That's tracked in #270. We can merge them if you want, but they seem to be different enough to have in separate issues.

I think it's worth merging into the same title of "QR Code Generator/Reader" so that it's easier to keep track.

@xeonmc Okay, we can merge the issues. Do you want to merge into this one or yours?

whichever one had more eyes on I suppose.

Closing in favor of #270 - please add QR code reading in your issue @xeonmc

https://github.com/dlbeer/quirc might be a useful standalone library, and it's licensed under a MIT like license(ISC License)


Support for evalution while training in Seq2Seq Tasks.

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
I want to evaluate each checkpoint with my specified metrics when training the model to choose the best checkpoint for the test dataset evaluation.
I find that the training dataset can be split for validation with --val_size, but it doesn't work in Seq2Seq tasks for the difference between training and evaluation.
So I want to confirm that the present code doesn't support evaluation while training with specified metrics in the Seq2Seq task.
Expected behavior
No response
System Info
No response
Others
No response

We don't support computing scores in training, please evaluate the model after training using the checkpoints
https://github.com/hiyouga/LLaMA-Factory?tab=readme-ov-file#predict


Add check for mismatched markdown specifiers

Pull Request Test Coverage Report for Build 241

11 of 15 (73.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 73.786%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tests/linter_tests/test_linter.py
0
1
0.0%

linters/linter.py
11
14
78.57%

Totals

Change from base Build 238:
-0.01%

Covered Lines:
4861

Relevant Lines:
6588

💛  - Coveralls


what's the size of test set

Table 1 in the paper shows test set as 10k but when we downloaded and checked, it amounted to 20k. Could you please double check the figures?
Thanks.

Hi,
Both numbers are correct. The size of the testset of KPTimes is 20k documents. Inside those 20k documents, 10k are from the NYTimes and 10k are from the JapanTimes.
The goal of including those 2 sources is to evaluate the generalizability of the models using documents similar of the training data, but not from the same distribution.
In your experiments you should report the evaluation using the 20k documents, and if needed for a finer grain show the NYTimes and the JPTimes score.


Can the agent move along the Y-axis?

To make the agent function like a drone, I need it to move along the Y-axis (UP-DOWN direction). I attempted to achieve this using the MoveUp control function registered in default_controls.py, originally designed to raise a sensor with body_action set to False. Now, I've set body_action=True to move the entire agent (including the sensor) along the Y-axis, but the execution doesn't meet expectations—the agent's position remains unchanged. I'm wondering if there are specific mechanisms restricting the agent's movement along the Y-axis and if there are methods to enable movement in this direction.
@registry.register_move_fn(body_action=False)
class MoveUp(SceneNodeControl):
    def __call__(self, scene_node: SceneNode, actuation_spec: ActuationSpec) -> None:
        _move_along(scene_node, actuation_spec.amount, _Y_AXIS)

I simply changed body_action to True, but it didn't work.
Thank you for your answer!

Hey @hjy5311
When using the default movement actions provided with the pointnav task, the Agent's SceneNode (3D transform) is locked to the navmesh. There is a "step_filter" which snaps the Agent to the navmesh after any translation.
You could write custom logic and use a custom action set which does not use the step filter, but you will need to handle the collision detection yourself since the navmesh abstraction doesn't really extend well to flying robots (as they are not restricted to navigable floor regions).
My suggestion is to implement your action set using the collision detection functionality, providing a 3D mesh to represent the robot's body and checking that this body is not intersecting with the environment after each requested discrete motion.
Check out this tutorial (https://github.com/facebookresearch/habitat-sim/blob/main/examples/tutorials/notebooks/ECCV_2020_Interactivity.ipynb) for an example of loading a body, collision detection, velocity control and other useful mechanisms.

Also, I did write a quick and dirty example of a quad-rotor several years ago. Likely the API has shifted significantly, but maybe the reference is still useful: https://github.com/facebookresearch/habitat-sim/commit/c4a4c53345e041677c17b1f502d150666465974e

Thank you very much for your answer. It has been very helpful. I am now considering using an agent to detect if it intersects with the environment. I would like to know if Habitat-sim provides a direct interface at the simulator level for obtaining the bounding boxes or other attributes that reflect the volume size of objects and the scene. This will impact the difficulty of adding collision detection.
By reading the code, I found that there are interfaces available for obtaining the volume information of objects. However, for the scene, I have not yet discovered  code that provides access to the volume attributes related to walls, scene boundaries, etc.
Thank you for your answer.

Hey @hjy5311
You should check out the rigid object tutorial from our tutorial series. It covers topics related to use of physics and collision detection.
To quickly answer your question: yes, Habitat provides bounding box and collision detection routines which you can use to determine if your agent object is in collision with the scene or other objects.
AABB: https://aihabitat.org/docs/habitat-sim/habitat_sim.physics.ManagedRigidObject.html#aabb
Object centric binary contact test: https://aihabitat.org/docs/habitat-sim/habitat_sim.physics.ManagedRigidObject.html#contact_test
Full scene collision detection: https://aihabitat.org/docs/habitat-sim/habitat_sim.simulator.Simulator.html#perform_discrete_collision_detection
Contact point data available after collision detection: https://aihabitat.org/docs/habitat-sim/habitat_sim.simulator.Simulator.html#get_physics_contact_points


disappear with a button a sprite

What is your question?
what I want to say is that I have a mechanism that works that when pressing a button the sprite disappears and an animation is played at the same time and I don't know how the KeyJustPressed works, if someone explains it would help me a lot
also if you do not press the button a in time bf dies

I hope I explained myself well
please I would really appreciate it 😃

This script will print the word HI every time your up keybind is pressed...
function onUpdate()
	if keyJustPressed("up") then
		debugPrint("HI")
	end
end

And this script will print the word HI every time E is pressed...
function onUpdate()
	if keyboardJustPressed("E") then
		debugPrint("HI")
	end
end

THANKS YOUUU 😀

if you know now
then close the issue

I made your script as an event if you need i
mods.zip
t

THANKS YOUUU :grinning:

You're welcomes :D


Update urls to HTTPS

Created with https://github.com/EFForg/https-everywhere/blob/master/utils/rewriter/rewriter.js

Sorry but what is the main point of adding www?

Sorry but what is the main point of adding www?

Consistency. php.net also redirects www.php.net.

Unfortunately, I don't see enough value in such a massive patch so we decline the PR. In case, there will be a user-visible or tools difference in behavior, we will reconsider this!


region parameter

i recommend add region parameter in this react component

@anker-shady sorry for late response, was sick last week. Updated to version v0.7.1 and published to npm with changes included and tested locally for region changes.
PR https://github.com/doppelganger113/amazon-pay-react/pull/3
Closing the ticket.


npm package is a bit of a mess

$ npm install applicationinsights-js
└──<EMAIL_ADDRESS>

It contains everything, including NuGet.exe, PowerShell files etc.

$ tree node_modules/applicationinsights-js/ -L 2
node_modules/applicationinsights-js/
├── API-reference.md
├── bower.json
├── changeSdkVersion.ps1
├── Common.props
├── dist
│   ├── ai.0.js
│   └── ai.js
├── EnlistmentRoot.marker
├── Global.props
├── JavaScript
│   ├── JavaScriptSDK
│   ├── JavaScriptSDK.Tests
│   ├── Microsoft.ApplicationInsights.JavaScript.sln
│   ├── NuGet.JavaScript
│   ├── NuGet.TypeScript
│   └── PostTestScript.cmd
├── LICENSE
├── NuGet.config
├── NuGet.exe
├── NuGet.targets
├── package.json
├── Package.targets
├── PostTestScript.cmd
├── PushNugetPackages.cmd
├── README.md
├── runAllTests.cmd
├── RunTestsInBrowser.ps1
├── ThirdPartyNotices.txt
└── uploadFilesToAzureStorage.ps1
6 directories, 23 files

2. `main` field in `package.json` points to `JavaScriptSDK/AppInsightsModule`. I doubt it's supposed to point to a TypeScript file, but if so it skips the `JavaScript` directory. Preferably it should point to `dist/ai.js` (i.e. unminified file), I think.

3. The one usable thing in here which are the javascript files in `dist` are actually different version (0.22.15).

4. The minified file is `ai.0.js`. That's a bit unconventional, I would expect `ai.min.js`.

Besides, what's the story here? Is it supposed to be an npm package with TypeScript? Or the point of the package is the JavaScript in `dist` directory?

The readme file in "Import as a module and initialize dynamically (download full Application Insights script from CDN)" talks about "modern JS application", however the code example is TypeScript and the `downloadAndSetup` method can only be found in the package in TypeScript files. That was a bit confusing to me. Please, make this scenario available in JS (that is without me compiling the package).

Thanks @ArturDorochowicz! We'll work on improving this early next week, will have another update Monday on this.

With the latest release to npm we addressed issues (2) (published AppInsightsModule.js that is now default entry point) and (3) (updated JS files in dist folder to the latest version - 0.22.18). Won't fix for (1) and (4).

Please reopen if you feel you're still blocked from using AppInsights via npm

Reactivating, looks like AppInsightsModule.js wasn't published correctly, opened PR for this: https://github.com/Microsoft/ApplicationInsights-JS/pull/227

Addressed

I installed the "applicationinsights-js" package now (first of June 2016) and it's still a messy package with a lot of content that really doesn't need to be there.
There are obj/Debug and obj/Release folders laying around, but the biggest issues are the size and the inclusion of multiple .exe files.
chromedriver.exe
IEDriverServer.exe
NuGet.exe
Most of the time I advice against copying and deploying node_modules directly to the server, in the very first beta release of ASP.NET 5, Microsoft had configured and advised for developers to publish bower/node packages directly into the web folder. That is very scary, considering some packages might include files such as .cs (which there are in your package, like IISExpress.cs) and .exe files, and with the wrong IIS configuration, attackers could potentially execute code on the server.
You clearly know about .npmignore to exclude files, as there is multiple of them in the repo, I hope you can go through and remove other files that are really not needed.
When a project already have many NPM packages, which is the nature of npm, it is clearly not welcome to have an extra 13MB package when all that is really needed in the package is a JavaScript file of 206KB. Until it is cleaned up, I have to simply copy the dist/ai.js and manually include it in our projects.

Thanks, we'll fix it!

Just pushed .npmignore file #248, which cleaned it up. Thanks for raising this issue, please feel free to reactivate if there are any concerns.


Reduced gas cost ( -3167gas )

With the code equivalent, executing ecZZ_mulmuladd_S_asm reduced gas cost by 3167 gas:

Gas

Before
207633 gas

After
204466 gas

Testing step:

Open the file: /solidity/tests/WebAuthn_hardhat/test/Webauthn.js

Uncomment line 80: let keyPair = ec.keyFromPrivate( "97ddae0f3a25b92268175400149d65d6887b9cefaf28ea2c078e05cdc15a3c01");

Comment lines 82-83.

npx hardhat test | grep gasused
The purpose of using a fixed key is to perform gas tests with deterministic data.

list:

(uint[2] calldata rs,uint[2] calldata Q) -> (uint256 r,uint256 s,uint256 Qx,uint256 Qy)   save 393 gas

if (rs[0] == 0 || rs[0] >= n || rs[1] == 0||rs[1]>=n) -> iszero(mul(mul(r, s), and(gt(n, r), gt(n, s))))  save 118 gas

Update ecAff_isOnCurve save 176 gas

uint[6] memory -> let pointer := mload(0x40)  save 135 gas

if eq(y2,0) -> if iszero(y2) save 1170 gas
... and more

Additionally,
Why did I submit code in the test path: The current code does not have a code format. In order to make it easier to review the code diffs in the submitted code, I did not format the code. However, for the sake of maintainability, I hope we can unify the style of the formal code (https://github.com/rdubois-crypto/FreshCryptoLib/blob/master/solidity) before making a formal code PR, for example, by using the "solidity.formatter": "forge" plugin. I hope this suggestion can be considered. Thank you.

Those lines seems false:

if iszero(add(scalar_u, scalar_v)), if u+v=0 modulo 2^256 will be incorrect, while not null because addition is performed modulo 2^256.

Please do not modify API, as the function are integrated elsewhere.
Could you use forge environment please ? There are tests vectors not assesed in hardhat (hardhat deprecated now).

got! thanks


FIFO logfile removed on configuration reload

Log files created with  mode ascii_pipe are unlinked upon logging.yaml configuration reload. This means that the named pipe will keep on working as expected only util a process is reading from it. Restarting the reader process breaks logging. The bug has been reproduced against master (1fa443cf).
Setup
Traffic Server was built from master with:
autoreconf -if
./configure --enable-layout=Debian --sysconfdir=/etc/trafficserver --libdir=/usr/lib/trafficserver --libexecdir=/usr/lib/trafficserver/modules
make -j4

The following configuration was used:
# /etc/trafficserver/records.config
CONFIG proxy.config.bin_path STRING /home/ema/dev/trafficserver/src/traffic_server/
CONFIG proxy.config.diags.debug.enabled INT 1
CONFIG proxy.config.diags.debug.tags STRING log-file

# /etc/trafficserver/logging.yaml
formats:
    - name: common
      format: 'test: %<chi> - %<caun> [%<cqtn>] "%<cqtx>" %<pssc> %<pscl>'

logs:
    - filename: fifologs
      format: common
      mode: ascii_pipe

The traffic_manager process was started with:
sudo -u trafficserver ./src/traffic_manager/traffic_manager --nosyslog

Reproducing the issue
Startup with no reader
Upon startup, the fifologs.pipe named pipe is created, but opening it fails (as expected) due to lack of readers:
[Nov 21 09:38:01.876] traffic_server DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/fifologs.pipe, this=0x55ea88a8bfb0
[Nov 21 09:38:01.876] traffic_server DEBUG: <LogFile.cc:175 (open_file)> (log-file) Created named pipe /var/log/trafficserver/fifologs.pipe for logging
[Nov 21 09:38:01.876] traffic_server DEBUG: <LogFile.cc:179 (open_file)> (log-file) attempting to open pipe /var/log/trafficserver/fifologs.pipe
[Nov 21 09:38:01.876] traffic_server DEBUG: <LogFile.cc:182 (open_file)> (log-file) no readers for pipe /var/log/trafficserver/fifologs.pipe
[Nov 21 09:38:01.876] traffic_server DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/error.log, this=0x55ea88a88340
[Nov 21 09:38:01.876] traffic_server DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/scrapfile.log, this=0x55ea88a8ec30

Issuing an HTTP request to traffic_server results in another failed attempt to open the pipe in order to write a log entry for the transaction:
[Nov 21 09:39:06.998] [LOG_PREPROC 0] DEBUG: <LogFile.cc:427 (write_ascii_logbuffer3)> (log-file) entering LogFile::write_ascii_logbuffer3 for /var/log/trafficserver/fifologs.pipe (this=0x55ea88a8bfb0)
[Nov 21 09:39:06.998] [LOG_PREPROC 0] DEBUG: <LogFile.cc:427 (write_ascii_logbuffer3)> (log-file) entering LogFile::write_ascii_logbuffer3 for /var/log/trafficserver/error.log (this=0x55ea88a88340)
[Nov 21 09:39:06.998] [LOG_FLUSH] DEBUG: <LogFile.cc:179 (open_file)> (log-file) attempting to open pipe /var/log/trafficserver/fifologs.pipe
[Nov 21 09:39:06.998] [LOG_FLUSH] DEBUG: <LogFile.cc:182 (open_file)> (log-file) no readers for pipe /var/log/trafficserver/fifologs.pipe

Proper logging upon reader startup
Starting a reader (eg: cat /var/log/trafficserver/fifologs.pipe) and performing another HTTP request:
[Nov 21 09:39:56.139] [LOG_PREPROC 0] DEBUG: <LogFile.cc:427 (write_ascii_logbuffer3)> (log-file) entering LogFile::write_ascii_logbuffer3 for /var/log/trafficserver/fifologs.pipe (this=0x55ea88a8bfb0)
[Nov 21 09:39:56.139] [LOG_PREPROC 0] DEBUG: <LogFile.cc:427 (write_ascii_logbuffer3)> (log-file) entering LogFile::write_ascii_logbuffer3 for /var/log/trafficserver/error.log (this=0x55ea88a88340)
[Nov 21 09:39:56.139] [LOG_FLUSH] DEBUG: <LogFile.cc:179 (open_file)> (log-file) attempting to open pipe /var/log/trafficserver/fifologs.pipe

At this point the pipe is opened successfully, and logs are shown as expected by cat.
lsof /var/log/trafficserver/fifologs.pipe
COMMAND     PID          USER   FD   TYPE DEVICE SIZE/OFF   NODE NAME
cat        4165          root    3r  FIFO  253,1      0t0 917531 /var/log/trafficserver/fifologs.pipe
[TS_MAIN] 31309 trafficserver   43w  FIFO  253,1      0t0 917531 /var/log/trafficserver/fifologs.pipe

Trigger logging.yaml configuration reload and fifo deletion
After adding an empty line to logging.yaml and triggering a configuration reload with pkill -HUP traffic_manager:
[Nov 21 10:05:09.927] [LOG_FLUSH] DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/fifologs.pipe, this=0x1543bc008cc0
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:175 (open_file)> (log-file) Created named pipe /var/log/trafficserver/fifologs.pipe for logging
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:179 (open_file)> (log-file) attempting to open pipe /var/log/trafficserver/fifologs.pipe
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:182 (open_file)> (log-file) no readers for pipe /var/log/trafficserver/fifologs.pipe
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/error.log, this=0x1543bc007880
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:115 (~LogFile)> (log-file) entering LogFile destructor, this=0x1543bc008cc0
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:119 (~LogFile)> (log-file) exiting LogFile destructor, this=0x1543bc008cc0
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:115 (~LogFile)> (log-file) entering LogFile destructor, this=0x1543bc007880
[Nov 21 10:05:09.929] [LOG_FLUSH] DEBUG: <LogFile.cc:119 (~LogFile)> (log-file) exiting LogFile destructor, this=0x1543bc007880

Once this happens, lsof /var/log/trafficserver/fifologs.pipe reports the pipe as not being open by cat and TS_MAIN any longer. The fifo results now deleted:
lsof | grep /var/log/trafficserver/fifologs.pipe
[TS_MAIN]  7855          trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
traffic_s  7855  7869    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_NET    7855  7872    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_NET    7855  7873    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_NET    7855  7874    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_NET    7855  7875    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_NET    7855  7876    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_NET    7855  7877    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[LOG_PREP  7855  7878    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[LOG_FLUS  7855  7879    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7880    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7881    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7882    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7883    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7884    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7885    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7886    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_AIO    7855  7887    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ACCEPT    7855  7888    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ACCEPT    7855  7889    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_TASK   7855  7890    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
[ET_TASK   7855  7891    trafficserver   44w     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)
cat        8018                   root    3r     FIFO              253,1       0t0     917531 /var/log/trafficserver/fifologs.pipe (deleted)

Restarting the reader
Stopping the cat process and starting a new one results in all logs being lost:
[Nov 21 10:08:44.556] [LOG_PREPROC 0] DEBUG: <LogFile.cc:427 (write_ascii_logbuffer3)> (log-file) entering LogFile::write_ascii_logbuffer3 for /var/log/trafficserver/fifologs.pipe (this=0x55f66c78ff30)
[Nov 21 10:08:44.556] [LOG_FLUSH] ERROR: Failed to write log to /var/log/trafficserver/fifologs.pipe: [tried 88, wrote 0, Broken pipe]

Avoiding unlink
diff --git a/proxy/logging/LogObject.cc b/proxy/logging/LogObject.cc
index c745bddb2..0d4ed90a9 100644
--- a/proxy/logging/LogObject.cc
+++ b/proxy/logging/LogObject.cc
@@ -1009,7 +1009,7 @@ LogObjectManager::_solve_filename_conflicts(LogObject *log_object, int maxConfli
             roll_file = false;
           } else {
             if (S_ISFIFO(s.st_mode)) {
-              unlink(filename);
+              //unlink(filename);
               roll_file = false;
             }
           }

With the patch above applied, a logging.yaml reload does not remove the pipe, and the reader (cat in our example) can be restarted at will without breaking logging. Notice how in this case there is no no readers for pipe error:
[Nov 21 10:59:19.579] [LOG_FLUSH] DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/fifologs.pipe, this=0x14dfac008c80
[Nov 21 10:59:19.579] [LOG_FLUSH] DEBUG: <LogFile.cc:179 (open_file)> (log-file) attempting to open pipe /var/log/trafficserver/fifologs.pipe
[Nov 21 10:59:19.579] [LOG_FLUSH] DEBUG: <LogFile.cc:80 (LogFile)> (log-file) exiting LogFile constructor, m_name=/var/log/trafficserver/error.log, this=0x14dfac007840
[Nov 21 10:59:19.580] [LOG_FLUSH] DEBUG: <LogFile.cc:115 (~LogFile)> (log-file) entering LogFile destructor, this=0x14dfac008c80
[Nov 21 10:59:19.580] [LOG_FLUSH] DEBUG: <LogFile.cc:119 (~LogFile)> (log-file) exiting LogFile destructor, this=0x14dfac008c80
[Nov 21 10:59:19.580] [LOG_FLUSH] DEBUG: <LogFile.cc:115 (~LogFile)> (log-file) entering LogFile destructor, this=0x14dfac007840
[Nov 21 10:59:19.580] [LOG_FLUSH] DEBUG: <LogFile.cc:119 (~LogFile)> (log-file) exiting LogFile destructor, this=0x14dfac007840

However, this causes TS_MAIN to open a new file descriptor every time logging.yaml is reloaded. For example, after 3 reloads:
lsof /var/log/trafficserver/fifologs.pipe
COMMAND     PID          USER   FD   TYPE DEVICE SIZE/OFF   NODE NAME
[TS_MAIN] 30483 trafficserver   40w  FIFO  253,1      0t0 925539 /var/log/trafficserver/fifologs.pipe
[TS_MAIN] 30483 trafficserver   44w  FIFO  253,1      0t0 925539 /var/log/trafficserver/fifologs.pipe
[TS_MAIN] 30483 trafficserver   45w  FIFO  253,1      0t0 925539 /var/log/trafficserver/fifologs.pipe
[TS_MAIN] 30483 trafficserver   47w  FIFO  253,1      0t0 925539 /var/log/trafficserver/fifologs.pipe
cat       31587          root    3r  FIFO  253,1      0t0 925539 /var/log/trafficserver/fifologs.pipe

The file descriptor leak shown above should clearly be avoided, though on a busy server it is arguably an improvement over breaking logging altogether, possibly filling up /var with Failed to write log entries.

@wmfgerrit Can you please create a PR for this?

I was able to reproduce the issue on my Fedora 30 server.

Stack trace when reloading the configuration:
#0  LogObjectManager::_solve_filename_conflicts (this=0x7f9270008f58, log_object=0x7f9270015f70, maxConflicts=99) at LogObject.cc:914
#1  0x00000000007847b2 in LogObjectManager::_manage_object (this=0x7f9270008f58, log_object=0x7f9270015f70, is_api_object=false, maxConflicts=99) at LogObject.cc:872
#2  0x000000000076d5c9 in LogObjectManager::manage_object (this=0x7f9270008f58, logObject=0x7f9270015f70, maxConflicts=99) at LogObject.h:367
#3  0x000000000078a615 in YamlLogConfig::loadLogConfig (this=0x7f92bd0b09b8, cfgFilename=0x7f9270011120 "/usr/local/etc/trafficserver/logging.yaml") at YamlLogConfig.cc:102
#4  0x0000000000789f1a in YamlLogConfig::parse (this=0x7f92bd0b09b8, cfgFilename=0x7f9270011120 "/usr/local/etc/trafficserver/logging.yaml") at YamlLogConfig.cc:39
#5  0x000000000076bf7d in LogConfig::evaluate_config (this=0x7f9270008f30) at LogConfig.cc:816
#6  0x000000000076a445 in LogConfig::setup_log_objects (this=0x7f9270008f30) at LogConfig.cc:375
#7  0x0000000000769dfb in LogConfig::init (this=0x7f9270008f30, prev_config=0x7f9270000e50) at LogConfig.cc:278
#8  0x0000000000756cbc in Log::change_configuration () at Log.cc:117
#9  0x0000000000756f9a in Log::periodic_tasks (time_now=1563831090) at Log.cc:221
#10 0x000000000075dc9f in Log::flush_thread_main () at Log.cc:1361
#11 0x000000000075e743 in LoggingFlushContinuation::mainEvent (this=0x15daae0) at Log.cc:285
#12 0x0000000000624949 in Continuation::handleEvent (this=0x15daae0, event=1, data=0x15008e0) at /home/bcall/dev/apache/trafficserver/iocore/eventsystem/I_Continuation.h:190
#13 0x00000000008e4295 in EThread::execute (this=0x7f92bd1b2010) at UnixEThread.cc:322
#14 0x00000000008e2d91 in spawn_thread_internal (a=0x15da440) at Thread.cc:92
#15 0x00007f92c527d5a2 in start_thread () from /lib64/libpthread.so.0
#16 0x00007f92c4e49303 in clone () from /lib64/libc.so.6

re: reproducing this, I can confirm we have also been seeing this @ yahoo!
would it make more sense to reuse existing pipes than remove the unlink?

Fixed in master by #6076


📅 8/12 @ 10:00am PT - Automating gr2m/helpdesk: Issue Forms part III

💁🏻 Automating gr2m/helpdesk: Issue Forms part III
📅 Thursday, August 12, 2021
🕐 10:00am Pacific Time
🎙 no guests
📍 https://www.twitch.tv/gregorcodes
🏷 automation

Subscribe to this issues to get a notification before the show begins and a summary after the show concludes.
Automating gr2m/helpdesk: Issue Forms part III
3rd and final part about the issue form automation. The flow is working now and I'll go through showing how it works.
See the first two episodes

https://github.com/gr2m/helpdesk/issues/34
https://github.com/gr2m/helpdesk/issues/42

Outline

Explain the flow from creating an issue using an issue form, and the parsing of the newly created issue into a form which works with the existing automation around upcoming shows.
Show how I created local tests for the flow, so I don't need to create new issues and wait for actions to run just to check if my code changes work. It allows for much quicker iteration

TODOs
Before the show

[ ]  30 minute announcement tweet
[ ]  30 minute announcement comment

When show begins

[ ]     start of show tweet
[ ]  comment on issue
[ ]  Set twitter profile url

After the show

[ ]  Reset twitter profile after the show
[ ]  recording available tweet

Recording
will be added after the show

Shownotes

Documentation: Creating issue forms
The issue form for creating a new show: .github/ISSUE_TEMPLATE/02_show.yml
The workflow that picks up new issues created using the form: .github/workflows/parse-new-issue.yml

I'm now live on https://twitch.tv/gregorcodes

Recording is now available at https://www.twitch.tv/videos/1116151666


introduce new crypto_key_value property for CONFIG dataset

as we can use env variables we could try to switch onto the new crypto_key_value approach as part of regular CONFIG context.

declare new crypto_key_value=NULL
if it is non empty it should have priority in comparison with crypto_key_path
crypto_key_path should be used only if crypto_key_value is not available

Such approach might organize movement to new security approach on CI with better flexibilities

ticket might be verified using 7.4.22-SNPASHOT

reopened to deliver in better way using separated maven or even gradle project

crypto-key project should be released soon by me (already with com.zebrunner packages)

For QA:
carina version: 8.0.1-SNAPSHOT

verified by @akamarouski


Try to generate audio...

I try to generate the audio files but it fails at the command :
nsynth_generate --checkpoint_path=/home/user/magenta/magenta/models/nsynth/wavenet-ckpt/model.ckpt-200000 --source_path=/home/user/nsynth-pipeline/workdir/embeddings_batched/batch0 --save_path=/home/user/nsynth-pipeline/workdir/audio_output/batch0 --batch_size=512 --alsologtostderr --gpu_number=0
With the output log (it stops running after) :
WARNING:tensorflow:From /usr/local/lib/python2.7/dist-packages/magenta/models/nsynth/wavenet/masked.py:116: init (from tensorflow.python.ops.init_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.initializers.variance_scaling instead with distribution=uniform to get equivalent behavior.
Can't understand why...
NOTE : I have all the files in embeddings_batched and embeddings_input folders, but just empty directories (batch0, ..., batch15) in audio_output

It was just the path to the necessary folders...


TDD Assignment

@ItamarShDev

Nice code but it misses the RED steps with failing tests that lead the implementation

I'll do it again .

check  the other one :+1:


-exclude Application from "winget upgrade --all"

Description of the new feature / enhancement
Can we exclude some app from the "winget upgrade --all" please? Maybe some flag for the apps that can be recognised from the upgrade command? thanks.
Proposed technical implementation details
No response

Hi @giuseppebin
Maybe use pin command is a better way to do this? If you don't want to update specific software, you could use 'winget pin ' command to prevent its updating.

To prevent a package being included in winget upgrade --all you can pin the package as @CoolPlayLin mentioned above.

Duplicate of #476
Duplicate of #3268

Hi @giuseppebin Maybe use pin command is a better way to do this? If you don't want to update specific software, you could use winget pin <package> command to prevent its updating.

oh I'm so sorry... I searched badly for this.
Many thanks.
This command is what i was looking for:
 winget pin add <package>
and the corollary winget unpin <package>
thanks to all.

pinned app aren't excluded
"1 package(s) ont des épingles qui empêchent la mise à niveau. Utilisez la commande « winget pin » pour afficher et modifier les épingles. L’utilisation de l’argument --include-pinned peut afficher plus de résultats."


[BUG] [Urgent] CSS rendered incorrectly.

The Problem
The CSS is not being rendered correctly.
Fix
Make a Folder Called CSS and create a styles.css in it, copy all the content from, index.css, and move it to styles.css, update the references for the same in index.html.
Please close the issue, and refer the issue number in the commit message.

done


move minecraft server to own repository

I really love your work and would like to contribute. Though it would be easier if minecraft server had it's own repository.
Would that be possible?
Thanks in advance.

Thanks. Yeah, that and the elasticsearch directory in the same repo each have a lot of activity/contributions going on independently. I have been wanting to make them repos each, but keep hestitating at considering how to retain contributor designations as such.
It might be sufficient for me to fork that one repo into a new repo, remove all but the minecraft-server , and shift down a directory. Thoughts or suggestions?

hmm github help to the rescue... https://help.github.com/articles/splitting-a-subfolder-out-into-a-new-repository/
Though that only covers the git migration. Should we consider to migrate the issues as well?

was just googling further, and there is some different approaches to moving the issues.
https://github.com/holman/ama/issues/413 refers to this project: https://github.com/google/github-issue-mover. Though I can't seem to find out if you will need to use this tool manually, or if you actually can query the "copy from project", then get a list that is looped across.

Good finds @theWoodman ! I'll work on a combo of the two since I had forgotten issues definitely need to migrate.

https://www.zenhub.com/guides/time-saver-features#move-issue-button We use ZenHub for our projects, really easy to move issues to a new repo when a user puts them in the wrong place.

so, long time no see... and it seems like nothing changed. Would it be possible to have a chat about what the plan could be to move?
I might have some preliminary plan below here:
In terms of keeping the git history with the contributions, it would be the best to fork this repo, move it to a new repo. Then:

git rm on new repo on all the files and folders that isn't needed anymore

git mvon new repo on content from /minecraft_server folder to root folder

run the github issue mover

git rm /minecraft server from old repo

Yeah, thanks for poking this topic. I really would like to isolate the image into its own repo. Yes, that approach would retain all the contributors which is equally important as retaining the open issues.

...so I’ll actively explore this.

...oh, I wonder if Docker Hub will let me shift the git reference?

Doesn't look promising :( The Docker Hub build settings don't provide an edit option for the associated GitHub repo:

Another I might "solve" this is to prune away some of the deprecated images I have in this repo. For example I know use all of the official Elastic images and can deprecated away elasticsearch, logstash, and kibana.

thanks for look into it, and sorry i didn't notice your comment before... Seems like this convo takes as long as pidgin carrieer...
Anyways, you are not the only one that has this problem, so i have tried to look up on the internet what others are doing. Certainly there is no message direct from Docker Inc; though their github feedback repo is a gold mine... if you dig...
https://github.com/docker/hub-feedback/issues/313#issuecomment-265843090
and:
https://success.docker.com/article/how-do-you-rename-a-docker-hub-repository
Anyways, you can't change provider. Though you can delete a dockerhub repo and create a new automated build with the same name. Though that will make you lose all your metadata, and therefore also the user base as this image is the most pulled.
So that is a no-go i guess...
Any other top notch docker images being pulled from the same github repo? in that case, move everything other than your minecraft server, and rearrange it as you also discuss in the last post.

hey @TheLeftMoose could you in the meantime maybe hint a quick and tidy way to pull/fetch the damn minercaft-server directory , i'm almost out nails soon it's gonna be bone i'll be chewing on. got stuck on checkout between --detach and --orphan or a possible three way , i'm not sure who's kinkiest here me itzg or github

It's only been two years :), but I finally worked up the nerve to archive the other directories of this repo, shift the minecraft-server files up to the top level, and rename the repo to effectively make it be its own repo.
Github will handle the redirects, but the new repo URL is https://github.com/itzg/docker-minecraft-server.


Readme linux libgdal dev

Readme :

Add requirement for linux libdgdal-dev
Add Link repo requirement.txt
Add Nav menu

@zifeo you think it's better ?

@Tonow thanks, I was not aware you updated the PR. @ustroetz WDYT?

Thanks! Again, sorry for the slow response. I was on a longer leave.


Add Apache Thrift and others

Hi Ruslan,
Hope you are doing well !
I was trying Hunter today, and it is awesome, and wanted to suggest the following:

Apache Thrift (https://github.com/apache/thrift)
NanoMsg (https://github.com/nanomsg/nanomsg)
Libmill (https://github.com/sustrik/libmill)
Libdill (https://github.com/sustrik/libdill)
Flatbuffers (https://github.com/google/flatbuffers)

Again, Hunter is awesome !
Cheers,
Luccio

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/33048384-add-apache-thrift-and-others?utm_campaign=plugin&utm_content=tracker%2F1032790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1032790&utm_medium=issues&utm_source=github).

I was trying Hunter today, and it is awesome, and wanted to suggest the following

Thanks, I've added:

https://github.com/ruslo/hunter/issues/395
https://github.com/ruslo/hunter/issues/394
https://github.com/ruslo/hunter/issues/393
https://github.com/ruslo/hunter/issues/392
https://github.com/ruslo/hunter/issues/391


Update issue template link to correct repository URL

Motivation
This PR updates the defaulting issue template link in ISSUE_TEMPLATE.md to point correctly to the AleoNet repository. Previously, the link incorrectly pointed to the AleoHQ repository.
Test Plan
To verify this change:

Navigate to the .github/ISSUE_TEMPLATE.md.
Click on the updated link to ensure it correctly redirects to the https://github.com/AleoNet/snarkOS/issues/new/choose page.
Confirm that the page loads the correct repository's issue templates.

No code changes were made, so no additional testing is necessary beyond link verification.
Related PRs
None.

Duplicate: https://github.com/AleoNet/snarkOS/pull/3342/


Bug in advanced search of Customer under sales module

company name field is displayed three times in the search tab

customer display name is not a field included in the customer creation page.so no results are found while searching for a customer in customer display name
!

Advanced search is not working with the existing fields

issue fixed !


Bind for <IP_ADDRESS>:5001 failed: port is already allocated

Describe the bug
When trying to start a docker-compose file with a bound external port at 5001 the an error occurs:
Bind for <IP_ADDRESS>:5001 failed: port is already allocated
This started in 1.4.0 and does not occur in 1.3.0.
To Reproduce

Setup a docker-compose.yaml file with a service bound to port 5001
Attempt to start using docker-compose up
Notice the failure to bind

Expected behavior
The service should start up and bind to port 5001 since nothing is running on it lsof -i :5001 does not show anything running
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.4.0
Commit: 81e06415c67e5b814c35811d403bc5341b4b5250 (v1.4.0)
System info:
macOS: 13.6 (22G120)
CPU: arm64, 8 cores
CPU model: Apple M2
Model: Mac14,2
Memory: 24 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-02-05T01-03-26.997783Z.zip

Screenshots and additional context (optional)
No response

There is no issue to run compose using external port 5001.
cat docker-compose.yml
version: '3'
services:
  web:
    image: nginx
    ports:
     - "5001:80"
% docker compose up
[+] Running 1/0
 ✔ Container 960-web-1  Created                                                                                0.0s
Attaching to web-1
web-1  | /docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
web-1  | /docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
web-1  | /docker-entrypoint.sh: Launching /docker-entrypoint.d/10-listen-on-ipv6-by-default.sh
web-1  | 10-listen-on-ipv6-by-default.sh: info: IPv6 listen already enabled
web-1  | /docker-entrypoint.sh: Sourcing /docker-entrypoint.d/15-local-resolvers.envsh
web-1  | /docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
web-1  | /docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
web-1  | /docker-entrypoint.sh: Configuration complete; ready for start up
web-1  | 2024/02/05 01:56:12 [notice] 1#1: using the "epoll" event method
web-1  | 2024/02/05 01:56:12 [notice] 1#1: nginx/1.25.3
web-1  | 2024/02/05 01:56:12 [notice] 1#1: built by gcc 12.2.0 (Debian 12.2.0-14)
web-1  | 2024/02/05 01:56:12 [notice] 1#1: OS: Linux 6.5.13-orbstack-00121-ge428743e4e98
web-1  | 2024/02/05 01:56:12 [notice] 1#1: getrlimit(RLIMIT_NOFILE): 1048576:1048576
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker processes
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 22
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 23
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 24
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 25
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 26
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 27
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 28
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 29
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 30
web-1  | 2024/02/05 01:56:12 [notice] 1#1: start worker process 31

Please use "sudo lsof -P -i :5001" to check the port.

Thanks, its interesting its actually the same error for any port with this service and starts in 1.4.0+ works just fine in 1.3.0, working on a sample to reproduce it

Got it to reproduce:

Make a file docker-compose.base.yml

version: '3.9'
services:
  web:
    image: nginx
    ports:
     - "5001:80"

Make a file called docker-compose.dev.yml

version: "3.9"
services:
  web:
    extends:
      file: docker-compose.base.yml
      service: web

Make a .env filed with

COMPOSE_FILE=docker-compose.base.yml:docker-compose.dev.yml

Run docker-compose up
See error

❯ docker-compose up
WARN[0000] Found orphan containers ([orbstack-960-web2-1 orbstack-960-redis-1]) for this project. If you removed or renamed this service in your compose file, you can run this command with the --remove-orphans flag to clean it up.
Attaching to web-1
Gracefully stopping... (press Ctrl+C again to force)
Error response from daemon: driver failed programming external connectivity on endpoint orbstack-960-web-1 (bd535683498651f350442a115e08c9f2f4e727d98a88f2116ba908a2c079f8b4): Bind for <IP_ADDRESS>:5001 failed: port is already allocated

Run docker-compose config

❯ docker-compose config
name: orbstack-960
services:
  web:
    image: nginx
    networks:
      default: null
    ports:
      - mode: ingress
        target: 80
        published: "5001"
        protocol: tcp
      - mode: ingress
        target: 80
        published: "5001"
        protocol: tcp
networks:
  default:
    name: orbstack-960_default

BUG: Notice that the ports section is duplicating the ports for the service, it should only be there once.

Downgrade to orbstack 1.3.0
Run docker-compose config

❯ docker-compose config
name: orbstack-960
services:
  web:
    image: nginx
    networks:
      default: null
    ports:
      - mode: ingress
        target: 80
        published: "5001"
        protocol: tcp
networks:
  default:
    name: orbstack-960_default

CORRECT: Notice the ports only have one

Run docker-compose up
CORRECT: It runs successfully

It seems that when extending a dockerfile in versions 1.4.0+ it duplicates the ports.

Fixed for the next version.

Released in v1.4.2.


Add initial description of activitypub federation

Describe how we expect federation to work, including custom types and feature detection behaviour.

resolves https://github.com/manyfold3d/manyfold/issues/1389


Check CI environment variables

fix: https://github.com/rust-lang/cargo/issues/7184
I changed to check the contents of the environment variable.
I run cargo fmt

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

Is there any environment that sets CI but sets it to an empty string or false?

Probably not. However, the fact that there is an issue may be a problem with the current implementation.

If that's the case then I think it's probably not worth it to go ahead and check the value? It arguably just makes the code a bit more confusing and error-prone

surely. So is this closed?

Ok!


⚠ Thumbnails API Endpoint has degraded performance

In 38d59c3, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1376 ms

Resolved: Thumbnails API Endpoint performance has improved in 53a598b after 5 minutes.


Add Image Carousel to Explore Page for Popular Posts

Currently, the Home Page lacks a dynamic element to showcase multiple pieces of content in a visually appealing way.
Proposed Solution:
Integrate a slider component that allows cycling through different content, such as images or featured posts.
Provide configuration options for the slider, such as transition effects, duration, and content selection.

Feature Request
Description:
I would like to propose the addition of a new feature to enhance the user experience on the Explore page of the app. The idea is to introduce an image carousel at the top of the post list, showcasing popular posts. This carousel will provide a visually appealing way to highlight and promote popular content, making it more accessible to users.
Feature Details:

Location: Explore Page
Position: Top of the Post List
Content: Carousel of images representing popular posts
Purpose: Enhance user engagement and highlight trending content

Additional Information:
I believe that this feature would not only improve the overall aesthetics of the Explore page but also contribute to user engagement by surfacing popular posts in a visually appealing manner. The carousel can be designed to dynamically update based on post popularity metrics or user interactions.
Implementation Considerations:

The backend could provide an API endpoint to fetch popular posts.
The carousel can be implemented using [insert preferred library/tool].
Consideration for accessibility and responsiveness should be taken into account during implementation.

Potential Benefits:

Increased user engagement with popular content.
Improved visual appeal of the Explore page.
Enhanced discoverability of trending posts.

Mockup/Illustration (if applicable):

Note:
Feel free to discuss and provide feedback on this proposed feature. Looking forward to the community's thoughts on this addition!


AttributeError: module 'convokit.paired_prediction.util' has no attribute 'display_thread'

Hello, I'm trying to run the hyper graph example on the reddit threads, and when I run the same code in the sample notebook:
for thread in demo_threads:
    print(thread)
    print('---')
    convokit.util.display_thread(threads, thread)
    print()

I get an error saying:
AttributeError: module 'convokit.paired_prediction.util' has no attribute 'display_thread'.
My guess is the method has moved or isn't available anymore: can I please get some help with this?

Ok, I see that this is because I am running the old demo notebook.


Binding a member function that returns a std::map does not compile, while returning a std::vector does

Hello, here I am again!
And this time with a Phd test case,
ThePhd.h.txt
/*
    SPDX-FileCopyrightText: 2023 Ondrej Popp
    SPDX-License-Identifier: GPL-3.0-only OR LicenseRef-KDE-Accepted-GPL
*/

#pragma once

#include <lua.hpp>

#define SOL_ALL_SAFETIES_ON 1
#include <sol/sol.hpp>

#include <string>
#include <vector>
#include <map>

class ThePhd
{
 public:
    ThePhd() = default;

    const std::vector<std::string>& thePhdVector() const
    {
        return m_thePhdVector;
    }

    const std::map<std::string, std::string>& thePhdMap() const
    {
        return m_thePhdMap;
    }

 private:
    std::vector<std::string> m_thePhdVector;

    std::map<std::string, std::string> m_thePhdMap;
};

void bindThePhd(sol::table& t)
{
    t.new_usertype<ThePhd>("ThePhd",
     sol::constructors<ThePhd()>(),

    // this one does compile,
    "thePhdVector", &ThePhd::thePhdVector,

    // while this one does not,
    "thePhdMap", &ThePhd::thePhdMap,

    // but this one does
    "wrapThePhdMap", [](const ThePhd& self)
      {return sol::as_table_t<std::map<std::string, std::string>>(self.thePhdMap());}
   );
}

As commented here above, while trying to bind the Phd, binding thePhdVector does compile, while binding thePhdMap does not. However, when you wrap that inside a sol::as_table_t it does compile, which is a good workaround, but I think the standard binding should work as well for the Phd?
And here is the compiler trace,
thePhdTrace.txt
It appears to be failing around here,
/src/Multimedia/VideoEditors/OpenIO/OpenIo/Cxx/Bridges/Lua/Sol/OpenIo/Tree/ThePhd.h:40:27:   required from here
/usr/local/include/sol/usertype_container.hpp:1191:46: error: unable to deduce ‘auto&’ from ‘i.sol::container_detail::usertype_container_default<std::map<std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char> >, void>::iter::it’
 1191 |                                 auto& it = i.it;
      |                                            ~~^~
/usr/local/include/sol/usertype_container.hpp:1191:39: note:   couldn’t deduce template parameter ‘auto’
 1191 |                                 auto& it = i.it;
      |                                       ^~
/usr/local/include/sol/usertype_container.hpp:1192:47: error: ‘struct sol::container_detail::usertype_container_default<std::map<std::__cxx11::basic_string<char>, std::__cxx11::basic_string<char> >, void>::iter’ has no member named ‘end’
 1192 |                                 auto& end = i.end;
      |                                             ~~^~~

Ok, that was my Phd test case :)
Tx, and have a nice day,
Ondrej

Hello, I have the same problem. Very frustrating

Good morning! Well it's not that bad... You need to write these bindings anyway so it's not too much effort to wrap the map. In addition I have found that std::optional doesn't work either, so you  need to return it from the lambda as well, but without wrapping it inside sol::as_table_t because that does not make sense for a std::optional. There could be more, but I don't remember that right now, I didn't feel like commenting on it all, so I would have to look at the code. In any case, I was able to fix everything that didn't work, generally by returning it from a lambda. So that's not all that bad...
Oh yeah, there is an other thing, from the top of my head, if I remember correctly, optional parameters in a constructor do not work either so you need to expand those. For example, if you have T::T(t1 a, t2 b, t3 c = d) you need to expand that into,
sol::constructors<T(t1 a, t2 b),T(t1 a, t2 b, t3 c)>()  and so on if you have more optional parameters in your constructor.
For the rest, it works like a charm, tx PhD!

@OndrejPopp
you can fix the offending lines like this
				auto& it = i.it();
				auto& end = i.sen();

it should compile. But in my case using pairs doesn't actually run through the std::map anyway.

Ok, tx! But yes, std::map uses pairs so if that doesn't work then you still need to wrap it...

No, with it.end() it won't compile. sen in from sentinel

Ok. Tx.


Reinstalling of VM extensions (when replacing existing disks)

In the section "Post-restore steps" there is the following statement.
Extensions present during the backup configuration are installed, but not enabled. If you see an issue, reinstall the extensions.

If I am creating a new virtual machine, I will need to reinstall the extension, but replacing a disk in an existing virtual machine, it may not be necessary. I have tested with extensions such as AADSSHLogin, AzureMonitorLinuxAgent, etc., and after restoring, they were enabled without reinstallation and worked fine.
If my understanding and result of investigating is correct, please specify in the documentation that in the case of disk replacement, reinstalling of extensions is not required. Thanks.

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

ID: 66081bde-1e22-dd9c-8357-a0485b23b0a7
Version Independent ID: eb2a52a5-d290-84fb-c3fe-2a5d8512669a
Content: Restore VMs by using the Azure portal - Azure Backup
Content Source: articles/backup/backup-azure-arm-restore-vms.md
Service: backup
GitHub Login: @jyothisuri
Microsoft Alias: jsuri

@torumakabe Thank you for posting this question. We will review it and get back to you soon.

@torumakabe - thanks for sharing this information. We've updated the doc with product group's confirmation, the updates will be live soon.

@torumakabe - Closing this issue as the doc has been updated with required changes.


Fuse 3.5.2

New KBFuse 3.5.2 included in installer.

@strib @akalin @keybase/react-hackers

@gabriel Thanks! Is this signed?
I don't think we want to release this until we've worked out whether it breaks Time Machine.  So if merging this is equivalent to putting it in the next build, please no-one merge it yet.

Yeah signed... Yeah we should hold off on merge until we're ok for testing...
I'm doing an app build for this branch, and will post link to that there for testing.

k, please keep the link private if there is significant suspicion that it's a link that will hose Time Machine backups.


Bitmap font loader not working with "fnt"

The bitmap font can't be loaded using "fnt" format with the code from the examples:
app.loader
    .add('TestFont', './TestFont.fnt')
    .load(onAssetsLoaded);

function onAssetsLoaded() {
    const bitmapFontText = new PIXI.BitmapText('bitmap fonts are supported!\nWoo yay!', { font: 'TestFont', align: 'left' });

    bitmapFontText.x = 50;
    bitmapFontText.y = 200;

    app.stage.addChild(bitmapFontText);
}

Expected Behavior
The text will be rendered.
Current Behavior
The font is not available. Instead, an error is thrown:
BitmapText.ts:113 Uncaught Error: Missing BitmapFont "TestFont"
    at new BitmapText (BitmapText.ts:113)
    at onAssetsLoaded (index.js:11)
    at MiniSignal.dispatch (mini-signals.js:93)
    at Loader._onComplete (Loader.js:623)
    at Loader.js:662
    at next (async.js:33)
    at Loader.SpritesheetLoader.use (SpritesheetLoader.ts:37)
    at Loader.js:643
    at async.js:41

Possible Solution
I have no clue.
Steps to Reproduce
Clone https://github.com/Prior99/pixi-bitmapfont-loader-reproduce
yarn
yarn start

Environment

pixi.js version: 5.3.0
Browser & Version: Chrome 85.0.4183.49
OS & Version: Arch Linux

Your font has fnt file extension which is supposed to be xml based format, but instead it is plain text. Renaming it to TestFont.txt should do the trick.

@EneaEntertainment thanks, I will try that.

The workaround is to do this:
PIXI.LoaderResource.setExtensionXhrType('fnt', PIXI.LoaderResource.XHR_RESPONSE_TYPE.TEXT);

Basically, the default for fnt files is to convert this to an XML Document. Because we now support both text and XML-based formats, we should change this to be more accommodating. Adding the line above will change the format but means you can only use either TXT or XML. Your example worked for me when I added this line before calling app.loader.add.
PR is welcome to fix this.

So is *.fnt really meant to be used for XML files only?

Many bitmap-font generation tools (bmGlphy) support fnt files that can be both text or XML. Historically, PixiJS only had XML support. We added text-based recently but didn't adjust the file handling of .fnt to support both.

@bigtimebuddy Thanks for clarifying :) Then I guess this is indeed a bug.


Token storage in separate config folder

Tested on Linux & Windows via this test build (which is set up to target a VPS), how about macOS?


Import SimpleStorage from brownie

Hello there,
I'm trying to import SimpleStorage from brownie but i keep getting the following error:
File "c:\users\capit\appdata\local\programs\python\python39\lib\site-packages\brownie_cli\run.py", line 51, in main
return_value, frame = run(
File "c:\users\capit\appdata\local\programs\python\python39\lib\site-packages\brownie\project\scripts.py", line 53, in run
module = _import_from_path(script)
File "c:\users\capit\appdata\local\programs\python\python39\lib\site-packages\brownie\project\scripts.py", line 149, in _import_from_path
import_cache[import_str] = importlib.import_module(import_str)
File "c:\users\capit\appdata\local\programs\python\python39\lib\importlib_init.py", line 127, in import_module
return _bootstrap.gcd_import(name[level:], package, level)
File "<frozen, line line, in in
File "<frozen, line line, in in
File "<frozen, line line, in in
File "<frozen, line line, in in
File "<frozen, line line, in in
File "<frozen, line line, in in
File ".\scripts\deploy.py", line 1, in
from brownie import accounts, config, SimpleStorage
ImportError: cannot import name 'SimpleStorage' from 'brownie' (c:\users\capit\appdata\local\programs\python\python39\lib\site-packages\brownie_init.py)
I've tried different solutions including unistalling brownie and eth-brownie but i keep getting the same error
here are some pictures to clarify the situation.
Thanks in adavance.

You're supposed to write your contracts in the contracts directory at the root of your brownie project, not in build/contracts.
The latter is used by brownie to store compiler things.
See : https://eth-brownie.readthedocs.io/en/stable/structure.html
Just move your smart contracts into the right directory, compile using brownie compile, and then you should be able to import your SimpleStorage from brownie in your python code !


Not obvious flask defaults to using port 5000

File: compose/gettingstarted.md, CC @londoncalling
I was a bit confused when I encountered port 5000 in step 3. I kept looking back to understand why we were referencing port 5000 and where it was "exposed". Didn't realize flask defaulted to port 5000.
Suggested fix to add a note in step 1 that flask defaults to port 5000 like you noted for redis?

@straticsryan thanks, good point. I'm going to include this fix on a pull request I'm already working on for some other Compose examples. I'll tag you on it when it's closer to being ready.


[V3] Fix up all the tests

Remove V2 tests and update V3 tests. Also remove all Alice and Bob for Alix and Bo.


refactor: replace function definitions with imports from powersimdata

Pull Request doc
Purpose
This is a companion PR to https://github.com/Breakthrough-Energy/PowerSimData/pull/507. Once code is moved from PostREISE to PowerSimData, within PostREISE we import it from PowerSimData. We do this within the old modules to maintain backward compatibility, although eventually we should probably deprecate and remove these in favor of direct imports.
What the code is doing
No code, besides imports.
Testing
Unit tests fail using the current PowerSimData/develop branch, but pass when checked locally using the branch from https://github.com/Breakthrough-Energy/PowerSimData/pull/507
Time estimate
5 minutes.

When would we actually plan to change all the import statements throughout PostREISE to import the functions now located in PowerSimData

When would we actually plan to change all the import statements throughout PostREISE to import the functions now located in PowerSimData

Good question. I don't know if we have any external workflows that depend on PostREISE right now (website maybe?), so I opted for compatibility, but I could change if we want.

When would we actually plan to change all the import statements throughout PostREISE to import the functions now located in PowerSimData

Good question. I don't know if we have any external workflows that depend on PostREISE right now (website maybe?), so I opted for compatibility, but I could change if we want.

Not sure. I don't think the website relies on the analysis/plotting modules. The question is more about release. I guess we need a new release in PowerSimData. Do we want to have a new release in PostREISE too?

When would we actually plan to change all the import statements throughout PostREISE to import the functions now located in PowerSimData

Good question. I don't know if we have any external workflows that depend on PostREISE right now (website maybe?), so I opted for compatibility, but I could change if we want.

Not sure. I don't think the website relies on the analysis/plotting modules. The question is more about release. I guess we need a new release in PowerSimData. Do we want to have a new release in PostREISE too?

I guess to be most correct, we should increment the PowerSimData release to 0.4.3 after https://github.com/Breakthrough-Energy/PowerSimData/pull/507 is merged, then create a new release in PostREISE that requires PowerSimData >= 0.4.3? I guess we can roll that into #308.

There are also a lot of not-time-series-specific functions in postreise.analyze.helpers. Do we want to move these to powersimdata at the same time?

There are also a lot of not-time-series-specific functions in postreise.analyze.helpers. Do we want to move these to powersimdata at the same time?

It seems that the entire module is grid specific and could be moved in powersimdata/input

This has been updated to directly import what used to be postreise.analyze.check and postreise.analyze.helpers from their new locations.

This has been updated to directly import what used to be postreise.analyze.check and postreise.analyze.helpers from their new locations. Tests pass using the PowerSimData branch from Breakthrough-Energy/PowerSimData#512.

Good question. @jon-hagg?

This has been updated to directly import what used to be postreise.analyze.check and postreise.analyze.helpers from their new locations. Tests pass using the PowerSimData branch from Breakthrough-Energy/PowerSimData#512.

Good question. @jon-hagg?

What's the question?

This has been updated to directly import what used to be postreise.analyze.check and postreise.analyze.helpers from their new locations. Tests pass using the PowerSimData branch from Breakthrough-Energy/PowerSimData#512.

Good question. @jon-hagg?

What's the question?

There are a question from @danielolsen. I guess he edited his comment. Nevermind


refactor: remove Borrowable type

This type is only used in few places and saves not enough allocations to
be worth it.

bors r+