Mythic Metals 0.19.1 broke pickaxes on my server

Since I updated to the newest version this morning, nobody on my server has been able to use a pickaxe. After scanning the logs, I discovered a few lines related to this:
[Worker-Main-1/ERROR]: Couldn't load tag mythicmetals:conduit_blocks as it is missing following references: mythicmetals:aquarium_glass (from fabric)
[Worker-Main-1/ERROR]: Couldn't load tag minecraft:mineable/pickaxe as it is missing following references: #mythicmetals:conduit_blocks (from fabric)
…and also, after a player who made tidesinger armour logged in:
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_leggings removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_helmet removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_chestplate removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_boots removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_leggings removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_helmet removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_boots removed now.
[Server thread/ERROR]: Tried to load unrecognized recipe: mythicmetals:armor/tidesinger_chestplate removed now.
If you need more information, I'd be happy to provide it.

confirm

I believe that @Noaaan knows of the issues and a fix will come soon. As a stop-gap you can use https://modrinth.com/mod/lmft to prevent the entire tag from being thrown out as to temporary fix issues with mining

I am aware of this yes. Minecrafts wonderful tag system, which drives gameplay, has no failsafes for cases like this unfortunately. LMFT is a great tool to use in the meantime, as it loads the tag despite a missing entry.
My plan is to release another fix this evening/tonight if I find the time

The tag issue is fixed as of latest release: https://github.com/Noaaan/MythicMetals/releases/tag/0.19.2%2B1.20.1
The recipe logspam is something I haven't had the chance to reproduce yet. I would appreciate if you could open that into its own issue should it prove a major issue. If not its fine, although I have noted it down and will look into it when I get the chance.
For now I will close this issue to mark that the major pickaxe breakage has been patched.


Viewport=full returning mobile version of page

Seems like some pages are returning mobile content to users when you pass viewport=full as parameter to Splash render.png endpoint.
For example this page: http://groceries.asda.com/asda-webstore/landing/home.shtml#search/chocoloate/1/relevance_desc
Following request to Splash:
localhost:8050/render.png?url=http%3A%2F%2Fgroceries.asda.com%2Fasda-webstore%2Flanding%2Fhome.shtml%23search%2Fchocoloate%2F1%2Frelevance_desc&wait=5&viewport=full

with viewport full returns mobile version of page:

I get proper view only after changing default dimensions of viewport:
localhost:8050/render.png?url=http%3A%2F%2Fgroceries.asda.com%2Fasda-webstore%2Flanding%2Fhome.shtml%23search%2Fchocoloate%2F1%2Frelevance_desc&wait=5&viewport=1366x768

This gives me desktop view

`

@pawelmhm the image you've posted is 640px wide; with Splash master it should be 1024px. Can you please confirm 1024px doesn't trigger mobile view for you?

ah yes @kmike if I pass 1024x I get desktop view. So maybe this is something in viewport=full? Docs says that it should be have 1024x768.

I'm aware that viewport=full is deprecated, but seems like render_all does not work well

This likely means you're using an older Splash version (or, in another words, docs for a wrong Splash version). 640px width and no render_all option is Splash 1.3.1 behaviour.

ok, so online docs are outdated? https://splash.readthedocs.org/en/latest/api.html#arg-render-all

No, online docs are for splash master, and it seems you're using an older Splash version. You can use https://splash.readthedocs.org/en/1.3.1/api.html to get the docs for v1.3.1.
I agree this is confusing, maybe we should point to stable docs by default.


Minor improvements in the module

What is the expected enhancement?
There are minor improvements to do:

algorithms/init

[x] NNC/NNR/VQR are missing in docstring

datasets

[X] Return np.ndarrays, optionally one-hot encoded (remove all functions not used)

neural_networks/init

[x] OpflowQNN missing in docstring

loss_functions

[x] Should we rename the file loss.py to loss_functions.py?

[x] Should we include the actual loss functions in the init.py to make them importable directly, e.g. "from qml.utils.lossfunctions import CrossEntropyLoss"? (currently there is only the copyright header in the loss_functions init)

[x] Are we using L2LossProbability or should we remove it? Or is it tested somewhere?

[ ] We should extend the docstrings of the loss functions a little bit to include what exactly they are computing.

[x] Did we test the KLDiv somewhere (it doesn't have a gradient implemented)?

[x] Do we need softmax/stable_softmax, doesn't seem to be used anywhere, maybe remove?

Datasets are fixed in #26


Getting an error: the number of rows or columns in the input array doesn't match the size or dimensions of the range.

I have copied this directly from the instructions and it was working, and then it wasn't. As far as I can tell nothing changed. I'm pulling data from a power apps collection and parsing it using JSON, opening the template, copying it into a new excel file, and running the script to put the CSV data (from the JSON) into the new excel file. I'm very new at this, and can't figure out what this error means. It looks to me like the script is getting the dimensions of the data from the CSV data, so why would it not work?
function main(workbook: ExcelScript.Workbook, csv: string) {
/* Convert the CSV data into a 2D array. */
// Trim the trailing new line.
csv = csv.trim();
// Split each line into a row.
let rows = csv.split("\r\n");
let data: string[][] = [];
rows.forEach((value) => {
/*
* For each row, match the comma-separated sections.
* For more information on how to use regular expressions to parse CSV files,
* see this Stack Overflow post: https://stackoverflow.com/a/48806378/9227753
/
let row = value.match(/(?:,|\n|^)("(?:(?:"")[^"])"|[^",\n]*|(?:\n|$))/g);
// Remove the preceding comma.
row.forEach((cell, index) => {
  row[index] = cell.indexOf(",") === 0 ? cell.substr(1) : cell;
});
data.push(row);

});
// Put the data in the worksheet.
let sheet = workbook.getWorksheet("Sheet1");
let range = sheet.getRange("A1:G30");
range.setValues(data);
// Add any formatting or table creation that you want.
}
As I said, this is directly copied from the Microsoft page. Help?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1fdee872-3dfe-61b6-f824-dc8c74625c36
Version Independent ID: c4481c81-ebae-7b3a-5da9-1757fb575188
Content: Convert CSV files to Excel workbooks - Office Scripts
Content Source: docs/resources/samples/convert-csv.md
Product: excel
Technology: scripts
GitHub Login: @o365devx
Microsoft Alias: o365devx

@debresnick Sorry to hear you're hitting this issue.
@AlexJerabek Can you take a look?
Thanks.

Hi @debresnick,
I'm sorry to hear this is no longer working. I'll take a look and see if there's some case that makes this sample fail.
Can you provide a sample csv file for me to test with?
Thanks!

Thank you so much for getting back to me! After a lot of troubleshooting I
determined that I had my csv table missing some double quotes. I got it
running as of late this afternoon.
On Mon, Nov 1, 2021, 4:46 PM Alex Jerabek @.***> wrote:

Hi @debresnick https://github.com/debresnick,
I'm sorry to hear this is no longer working. I'll take a look and see if
there's some case that makes this sample fail.
Can you provide a sample csv file for me to test with?
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OfficeDev/office-scripts-docs/issues/417#issuecomment-956576827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC4Y4NSZ74YF737BZ7GY6RLUJ34EFANCNFSM5G772LKQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ah, that makes sense. The row and column mismatch is probably from a cell with a comma in it being interpreted as two cells.


Sync to hash envelope draft

Synching to https://datatracker.ietf.org/doc/draft-steele-cose-hash-envelope/01/

Should be merged with https://github.com/datatrails/datatrails-docs/pull/870


Installing with npm doesn't work

This looks like an awesome library. Unfortunately I can't get npm to work - npm install results in a folder (@simonwep/selection-js) with no content, other than LICENSE, package.json and README.md
npm install @simonwep/selection-js --save

Thanks  for the issue, it seems like it was broken as I modularized the project. I fixed it (see this commit).

Hi Simon, I gave it a go but NPM is still downloading 0.0.9 - does it take a while to update?

Yep, it's working now. Thanks for that!


Return Video Width and Height

Hello, it would be possible to implement a method that returns the height and width of the video, because video in 4K, resolution of 3840 × 2160 pixels, the smartphone closes the application. So I can check if it will support the video before the crash happens.
Thank you very much.

yes,you can add you logic to ’gplayer.dart/_onGetPlayerData‘ and find 'onVideoSizeChanged'


Autocomplete disabled for

Environment/Code
Ubuntu 15.10, Builds with dart2js, Chrome 49
html
<paper-input autocomplete="given-name">First name</paper-input>
What is expected
When the input form is entered, autofilled with first name stored in Chrome settings.
Actual outcome
The autocomplete is inactivated.
Howto reproduce
Create a new Polymer project, then put the line below in main.html, together with import in main.dart
<paper-input autocomplete="given-name">First name</paper-input>
Observations
Adding two input lines as below to main.html, will give valid autocomplete in the <paper-input> below. This indicates some load/runtime dependency.
<input is="iron-input" autocomplete="email">Email</input>
<input is="iron-input" autocomplete="given-name">First name</input>
<paper-input autocomplete="given-name">First name</paper-input>

Cut from pub.yaml
environment:
  sdk: '>=1.9.0 <2.0.0'

dependencies:
  browser: ^0.10.0
  polymer_elements: ^1.0.0-rc.8
  polymer: ^1.0.0-rc.15
  web_components: ^0.12.0

transformers:
- polymer:
    entry_points: web/index.html
- $dart2js:
    minify: true
    commandLineOptions:
    - --trust-type-annotations
    - --trust-primitives

------------------------------ Workaround -------------------------
Put these two <input> lines below, somewhere in the project. I have chosen index.html, together with an iron-input import in index.dart. Then project wide set all  <paper-input autocomplete="on"> or the choosen autocomplete e.g. <paper-input autocomplete="given-name"> This seems to make the autocomplete work.
<input is="iron-input" autocomplete="email" hidden/>
<input is="iron-input" autocomplete="given-name" hidden/>

Reproduced with polymer_elements: ^1.0.0-rc.9
From pubspec.yaml
environment:
  sdk: '>=1.9.0 <2.0.0'

dependencies:
  browser: ^0.10.0
  polymer_elements: ^1.0.0-rc.9
  polymer: ^1.0.0-rc.17
  web_components: '>=0.11.3 <0.13.0'

The actual behavior of this feature is hard to figure out from the documentation. An answer on StackOverflow help this one out.
http://stackoverflow.com/questions/37445468/paper-input-autocomplete-fails-to-fill
The paper-input element should be wrap with form tags as below. Tested with dart2js builds on Chrome 53.
<form>
  <paper-input autocomplete="given-name">First name</paper-input>
</form>

The issue is closed.


chore: release v1.217.0 [DO NOT MERGE]

fix: osmosis lp withdraw amount display (#3853)
feat(cosmos): replace StakingOpportunities -> EarnOpportunities (#3841)
fix: use opportunity id if available (#3832)
feat: clear cosmos sdk opportunities migration (#3839)
feat(defi): opportunities metadata icons (#3842)
feat: enable osmo staking and swapping flags for all environments (#3852)
fix(defi): cosmos sdk empty opportunity routing (#3840)
feat(DeFi): remove OsmosisManager/Staking (#3848)
Merge branch 'main' into develop
fix: osmosis staking fiat deposit/withdraw errors (#3844)
fix: increase osmosis lp default slippage tolerance (#3846)

closing to recut tomorrow


Can't get it working on iOS

Hi all,
I'm trying to build a single-view web-app using express (not the best framework for my use cases, I know).
Because it is a 'web-app' it would be the best when I open the page in full screen on my phone, there is no overflow for the user to scroll. Currently, the view of the divs takes up the entire height and width of the viewport and fits perfectly, however when you scroll, there is a large, blank white overflow that I want to remove. I want to essentially recreate the demo, but without the need of a separate scroll target, so pretty much no scrolling at all is needed. I have been working on this for like 4 hours now and I can't seem to get it working.
The  BSL demo works fine on my iPhone, and has the desired operation, ability to unlock and lock the view.
The BSL on my server works fine on my desktop, but does not function at all on my phone.
The app consists of the Express backend, with just plain HTML, JS and CSS as the frontend.
The code is below. I am pretty sure it is my stupidity...
'index.html'
...
<!-- Top of the file (in the Head section) -->
<head>
     <script src="http://<IP_ADDRESS>:4000/bodyScrollLock.js"></script>
</head>
...
<!-- Bottom of the file -->
</body>
<script src="http://<IP_ADDRESS>:4000/scrollLocker.js"></script>
...

scrollLocker.js
const targetElement = document.querySelector('scroll-target2');
bodyScrollLock.disableBodyScroll('targetElement');

Cheers,
Will

@laz3rpenguin I'm pretty sure you can't include a <script> element outside the <body> tag, try adding your scripts just before the closing </body> tag


CIS Scan - Report gets deleted from Rancher but a 404 is returned in the browser console.

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Run a CIS scan
Once created, select and delete it.

Result:
The report gets deleted but a 404 is returned when deleting the report in this path:
GET /v3/clusterScans/c-dksfn:cis-1583943634697921385/report
{"baseType":"error","code":"NotFound","message":"clusterscans.management.cattle.io \"cis-1583943634697921385\" not found","status":404,"type":"error"}

Environment information

Rancher version: 2.4 master-head (03/10/2020) commit id: 570461823
Installation option (single install/HA): single

CIS Scan has passed through new release iterations. Please open a new issue if this is still happening


Suggestion for file browser

Hi,
The file browser in OpenSim (all versions) is not user-friendly enough (during opening, loading, and saving). Some functionalities like pasting the path, searching for files, right-clicking on the files for previewing, deleting, copying and etc. are missed when compared to e.g Notepad++ file browser or other windows applications
It would be great if the OpenSim developer team would change the file browser in the new version (4.2).
Regards,
Mohammadreza
Steps to reproduce
Expected result

Actual result

(include screen grabs where appropriate)
Environment and GUI version
Windows 10,
OpenSim 4.2 Beta

Hi, the current file/folder picker design is rather too old. There is neither "Quick access", to quickly find the pinned folders, nor address bar, to paste the directory in which files are located. Sometimes it takes me a while to navigate/find the desired folder. I wonder if it would be possible for OpenSim developers to modify the design of the GUI file/folder picker. Thank you.
@aymanhab @nickbianco


Merge visualfsharp 1

deleted obsolete readme
added missing test AsyncModule.Async.AwaitWaitHandle does not leak memory
merge visualfsharp changes ( splitted on two commit 44f7671 and 023d075 for easier diff, the first is whitespace reformatting only )

This looks good - whitespace only changes, good


OTG Test - TE-3.3: Hierarchical weight resolution

OTG version of TE-3.3: Hierarchical weight resolution
Resolves #732

Pull Request Functional Test Report for #2357 / 1e5cc6e617778032755299f3e6649d0a9cc1dc3b
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TE-3.3: Hierarchical weight resolution

Cisco 8000E

TE-3.3: Hierarchical weight resolution

Cisco XRd

TE-3.3: Hierarchical weight resolution

Juniper cPTX

TE-3.3: Hierarchical weight resolution

Juniper ncPTX

TE-3.3: Hierarchical weight resolution

Nokia SR Linux

TE-3.3: Hierarchical weight resolution

Openconfig Lemming

TE-3.3: Hierarchical weight resolution

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

TE-3.3: Hierarchical weight resolution

Cisco 8808

TE-3.3: Hierarchical weight resolution

Juniper PTX10008

TE-3.3: Hierarchical weight resolution

Nokia 7250 IXR-10e

TE-3.3: Hierarchical weight resolution

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 48.7%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1536

Relevant Lines:
3154

💛  - Coveralls


Shape broadcasting error in norbert filter init_.py while referring v1.0.0

@faroit I was referring to v1.0.0 of this repo as there you implemented numpy version of weiner filter instead of torch but facing broadcasting shape error under softmask=False 'filter*x[...,None]' in weiner filter function.
Can you please help me in this as its very important for me to solve this error ASAP? What changes should I do while preprocessing  audio before feeding into separator function there or something else?
Can you please tell me how can I solve that error in v1.0.0 because for whole setup here we need to refer only that version of open-unmix-pytorch not the latest release of this repo. It would be really appreciable if you take a look at this issue soon. Thanks

Not sure if I understand. Can you post a minimal working example that shoes the error?

@faroit I tried to run v1.0.0 by checking out commit 3f6a42110c4aa77cedd66770e20aecdcecbb348c
Version:
Pytorch 1.10
Python 3.7.4
OS - Windows 10
MUSDB18-HQ dataset I downloaded for evaluation - https://sigsep.github.io/datasets/musdb.html#musdb18-hq-uncompressed-wav
Commands I referred to run eval.py script using pretrained MUSDB18-HQ uncompressed dataset on 'umxhq' model as follows:
git clone https://github.com/sigsep/open-unmix-pytorch.git
git checkout --progress --force 3f6a42110c4aa77cedd66770e20aecdcecbb348c
python eval.py --outdir museval/results --evaldir museval/estimates --model umxhq
After running above command and completion of first iteration I got below error:
`  0%|                                                                                | 0/50 [00:00<?, ?it/s]AUDIO SHAPE (9265664, 2)
D:\Deep_Learning\Anaconda3\lib\site-packages\torch\functional.py:1069: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at  ..\aten\src\ATen\native\TensorShape.cpp:2157.)
return _VF.cartesian_prod(tensors)  # type: ignore[attr-defined]
100%|████████████████████████████████████████████████████████████████████████| 4/4 [18:12<00:00, 273.22s/it]
███████████████████████████| 4/4 [18:12<00:00, 277.32s/it]
Traceback (most recent call last):
File "eval.py", line 157, in
device=device
File "eval.py", line 30, in separate_and_evaluate
device=device
File "D:\audio-sep2\open-unmix-pytorch\test.py", line 165, in separate
use_softmask=softmask)
File "D:\Deep_Learning\Anaconda3\lib\site-packages\norbert_init_.py", line 286, in wiener
y[...,0,:] = v*(np.cos(angle)[...,None])
ValueError: operands could not be broadcast together with shapes (1,9265664,2,4) (9049,2049,1,1)`
Hoping to hear from you soon.
Thanks

@yugaljain1999 you would need to run that version with the respective torch environment, that version had

an older torch version
an older version of musdb and museval
an older version of norbert

@faroit Thanks for you reply but I have used that too as well but encountered same error :(
Is there any other solution? Can you please try it yourself, you will clearly observe it?

In your first comment I see that you were using torch 1.10. That's not the version that was used for open-unmix 1.0.0

That's right @faroit in v1. 0.0 you mentioned pytorch 1.2 torchaudio 0.3 for linux OS.
But still by using that version recently I am getting same error.
That's why please suggest me what should I do now?
If you can install that env in your pc and test then you can observe that error.
Hoping to hear back from you soon :)

it looks to me you are trying to multiply a waveform with something happening in the frequency domain:
(1,9265664,2,4) looks like a waveform, is v really a spectrogram here ?

Yes v is targets spectrogram here
On Sat, 20 Nov, 2021, 2:05 am Antoine Liutkus, @.***>
wrote:

it looks to me you are trying to multiply a waveform with something
happening in the frequency domain:
(1,9265664,2,4) looks like a waveform, is v really a spectrogram here ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sigsep/open-unmix-pytorch/issues/111#issuecomment-974439333,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHY3MAGRQKZNBVEW744L3EDUM2YJTANCNFSM5IKKDXNA
.

@yugaljain1999 i tested v1.0.0 on my machine. It works fine but you have to make sure that you use a local model.pth file or modify the torch hub loader so that uses v1.0.0 from github:
diff --git a/test.py b/test.py
index 00873a8..74902b8 100644
--- a/test.py
+++ b/test.py
@@ -28,7 +28,7 @@ def load_model(target, model_name='umxhq', device='cpu'):
             err = io.StringIO()
             with redirect_stderr(err):
                 return torch.hub.load(
-                    'sigsep/open-unmix-pytorch',
+                    'sigsep/open-unmix-pytorch:v1.0.0',
                     model_name,
                     target=target,
                     device=device,


Missing support for DEFAULT color ?

I defined a whole new color with
const colors = {
    "english-violet": {
        "DEFAULT": "#514663",
        "100": "#ffffff",
        "99": "#eeedef",
        "95": "#dcdae0",
        "90": "#a8a3b1",
        "80": "#857e92",
        "70": "#625973",
        "60": "#514663",
        "50": "#493f59",
        "40": "#41384f",
        "30": "#393145",
        "20": "#312a3b",
        "10": "#18151e",
        "0": "#000"
    }
};

Then added it to the theme:
  plugins: [
    createThemes({
                   light: {
                      'primary': colors["english-violet"],
                      'secondary': 'darkblue',
                      'brand': '#F3F3F3',
                   }
                })
  ],

If I'm using bg-primary with the theme applied, I don't get any color. Other variations works :

                Doesn't work

                Works as intended

                Works also

Without using the theme, the following also work as intended :

                Works as intended

That's really weird, I've been doing more testing and adding more colors to the theme, and some of them works with DEFAULT, some don't.
I tried to remove the hyphen in the name (just in case), no change.
Also tried to use dot notation rather than brackets (I can't see why that would do something, but as it is weird...), no change either.
For what it's worth, VS Code intellisense with official tailwind extension sees the default colors for those which works, and only sees the *-DEFAULT for those that doesn't work.
Example of one that works and properly see the DEFAULT :
    "error": {
        "DEFAULT": "#ba1a1a",
        "0": "#000000",
        "10": "#410002",
        "20": "#690005",
        "25": "#7e0007",
        "30": "#93000a",
        "35": "#a80710",
        "40": "#ba1a1a",
        "50": "#de3730",
        "60": "#ff5449",
        "70": "#ff897d",
        "80": "#ffb4ab",
        "90": "#ffdad6",
        "95": "#ffedea",
        "98": "#fff8f7",
        "99": "#fffbff",
        "100": "#ffffff",
    },

Fixed in 1.2.1

Thanks for the fix ! I'll update ASAP.
(and in case that was not planned, you left a console.log)


Update ts and js quickstart to alpha39

Close #241

hi -- that doc is soon to be obsolete. We have a separate project (with several authors) working on large-scale improvements to the docs, so it's not actually all that useful to submit small fixes like this.
You should see some big improvements very soon.


Add building of rbac resources

Description
Add a helper function to build rbac resources.

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

I assume this PR only extracts code that is already used "in production" into operator-rs

Yes, correct. We already have it in airflow, hbase, spark-k8s etc. I'll replace those with references to this function in subsequent PRs.

bors merge


remove execute permissions on templates

What does this implement/fix?
While working on #1778, I reached a state where the exclude in
https://github.com/nebari-dev/nebari/blob/8bd56f90b84c7686f45459ae8f7cf690721bd60a/.pre-commit-config.yaml#L36-L37
did not apply. With that, pre-commit showed me three files that had execute permissions but no shebang. Given that these are templates and no other template has execute permissions and a shebang, I've just removed the execute permissions here.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

/bot run tests

Seems fine, but it'd be nice to run a k8s test, just to be sure.

/bot run tests

/bot run tests


Rename dev runtime to browser

Description

Testing Instructions

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

#8400  👈
#8399
#8398
#8397
#8396
main

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

Join @timneutkens and the rest of your teammates on  Graphite

Merge activity

Jun 19, 4:21 AM EDT: @timneutkens started a stack merge that includes this pull request via Graphite.


Fix arm64 builds

Move the TARGETPLATFORM before the FROM, otherwise it'll default to the build platform.
Fixes https://github.com/devops-infra/docker-terragrunt/issues/2201

This doesn't really work as intended, it's probably better to drop the TARGETPLATFORM any only use buildx even when not xcompiling. This doesn't work in combination with buildx, see issue https://github.com/docker/buildx/issues/510

Hello. I'll have a look at it when I have the time.
Thanks for taking interest in this!


Expose Configure Snap

To expose Configure Snap, I have added two methods to the EditorPlugin: set_snap_configuration(Dictionary configure_snap) and get_snap_configuration().
Need advice on placing the new methods.
I also need someone else to create a test project for me (if required), as my GPU is not powerful enough to build.
@groud please review whether this is indeed the solution.
[UPDATE] I also need advice on rebase my branch. I have tried git push --force --set-upstream origin expose-snap-settings, but it's not updated to included recent changes.
[UPDATE 2] I've typed git pull --rebase upstream master (As following in the PR Request workflow), but in the process, I have two more commits that I cannot squash. Interactive rebase git rebase -i upstream/master does not update my branch with recent merges, either.
[UPDATE 3] I've also pressed Update with Rebase beforehand, but it gives me two unsquashable commits.

@groud please review whether this is indeed the solution.

Well, no. This only allows retrieving the snap configuration status. It does not exposes a function to snap the point value, which is asked in the proposal.

I've decided to close this PR as I have less of a reason to get snap settings by code. Originally, it's to set snap settings when a TileSet is set, but that would probably be done once before more map are to be made via instancing. The convenience is not really that to be much desired.
If other people wants it, they may look at File changes to see what edits I've made.


Comprehensive testing

Test complete solver with predetermined data sets and known correct solutions.
Implement w/ lazy artifacts?

working on this w/ LazyArtifacts. Test files hosted on dropbox, waiting on @geosamaras for more test cases

also "correct solutions" should be "converged solutions"


Add backflip jump.

https://user-images.githubusercontent.com/10785634/170805302-530d35ac-4741-4b27-b7be-ed071b1e125b.mp4

It looks good, but IMO it's better faster, maybe even 2x, since this is usually meant as a quick dodge.

Alternatively, for this speed of backflip, I think the character would need to jump much higher.

Because jump air time is defined by gravity and jump velocity.y, and I think can't modify gravity.
So then can just decrease jump velocity.y to decrease jump air time.
( Also can't increase jump velocity.y and height, will increase jump air time, which not expected )
How about this version? ( Decreased jump air time from 999ms to 600ms )
https://user-images.githubusercontent.com/10785634/170833668-84b167da-7377-47e8-a9ec-e6168147cbae.mp4

Used existing game.isMovingBackward() to check if backflip instead of check in separated codes.
Found a problem that,
though this PR https://github.com/webaverse/app/pull/2799 used avatar.direction.z instead of keysDirection.z ,
successfully solved the inconsistent velocity issue https://github.com/webaverse/app/issues/2798 .
But compare to instant keysDirection.z, avatar.direction.z has a very short lag due to it's frame comparing and normalizing https://github.com/webaverse/app/blob/0b710da77b85c798e1f34a7c530bba8503017a65/avatars/avatars.js#L1495-L1507
Will cause consecutive aimming jumps appear unexpected backflip.
So I introduced keysDirectionLocal to be used for game.isMovingBackward(), in order to solve the problem of wrong velocity while maintaining instant keyboard input reaction.

https://github.com/webaverse/app/pull/3077#issuecomment-1140291165

The version here looks ok, but it could be smoothed in more. (the animation jumps to hands up immediately)

Added transition at the start of backflip animation.

2- As you can see, the backflip works and looks fine while aiming and walking/running backwards. But while we aim and do naruto run backwards, the direction of avatar is not towards backside, but the back flip jump happens and looks kind of weird. Either we should disable backflip jump for naruto run or we should make sure that direction of avatar while aiming and moving backward is towards the front.

Hello @alisaad673 , fixed this bug, and one more "can't moving when flying" bug.

1-These errors are appearing in the branch, Are they related?

I think they are not related to this PR. Do you have these errors in pure newest z-targeting branch?

These error are not appearing in your other PR https://github.com/webaverse/app/pull/3099.
Can you please check this out? Also I didn't see these in z-targeting too. Maybe try rebasing?
The other bug is fixed.

Hello @alisaad673 , I think the main error is ReferenceError: Cannot access 'cameraManager' before initialization at io-manager.js right?
I can't reproduce it on my local, but fixed a potential import problem which may be the cause.
Is it fixed now?

Yes. This is fixed now.

Hello @alisaad673 and @gonnavis is this ready to be pulled in?

Hello @alisaad673 and @gonnavis is this ready to be pulled in?

Yes, ready for merge or more reviews.

I tested this, it's good. If you z-target and jump backwards you'll backflip nicely. Going to pull it in. The other jump PR will been to be rebased on top of this.


Update airflow to 2.0.0-beta5

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Thank you for the contribution. It was merged directly as 3a6e03d53fb5ca14144517a727a57ff20e2be308. Your contribution is still credited to you.


Basic html tutorial in portuguese

Added the basic html tutorial in portuguese as well as the explanation for each of the tags contained in the basic structure.

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Add new gif example to README

We currently have the fractal example gif in the README. When we release 1.7, it would be great to show an example of an interactive Bokeh visualization too.

On a related note, we should consider gettingrid of the gifs in the showcase notebook - gif support relies on imagemagick and I'm not sure everyone will be able to run that notebook easily.

Happy to do this once Philipp tells me his favorite linked stream example. :-)

This is being addressed by PR #1596.

PR has been merged. Closing.


kpt pkg update doesn't update comments from upstream

kpt pkg update --strategy=resource-merge currently only updates resources only when there are changes to field values. But if there are comments added to the scalar fields in upstream, them are not detected and updated on local. This would significantly affect updating setters added upstream.

https://github.com/kubernetes-sigs/kustomize/pull/2795


Install hangs up

I'm on windows 10 machine. While I tried installing using the command npm install -g headstart it just hangs up like the one attached in the screenshot. Also there was some deprecated warning message.
No progress after this.

After third try it installs completely. But it hangs on cleaning up after running the command headstart init
Need Help.

Hello @jm-kumaresh. I know there are quite a lot of problems with the installation and cleaning process, but that probably because I haven't updated this project in quite some time and most of the packages are outdated.
All the "cleaning up" does is copy the contents of the .tmp folder into its parent folder, so even though its a pain to do, you can overcome this with some manual file moving.
Anyway, an update is long overdue, but I can't make any promises when one will arrive.
Sorry for this inconvenience!

Thanks for the reply flovan. Let me try with your method. BTW your project seems to be more useful and clean apart from other boilerplate. I request you to spend some time on it to update. Because this is more helpful to noob in task runner like me.
Thanks

Another way you can get around the headstart init problems is by copy/pasting a fresh copy of the boilerplate. I noticed you forked that repo so you can use a copy of your local folder repo.
Anyway, I'm glad you like it! I will do my best to make an update since even my colleagues are experiencing problems now, but again, not promises on "when" 🤐

Hey @flovan @jm-kumaresh, I'd like to recommend you to use TARS (https://github.com/tars/tars) It is like headstart, but has more great parts, for example automatic update of tour project!) Just try, and you will fall in love)
I'm sorry for such advertisement content)

Thank you for tuning in @artem-malko! I'm an eat-your-own-dogfood kind of guy, so I'm using my own tools for everything I do, and adapt them to my own purposes, but I'm sure if @jm-kumaresh isn't satisfied with Headstart, that he'll take a look at the other available options, since there are (quite a lot of them)[https://www.staticgen.com] besides TARS.

Hi @flovan @artem-malko thanks for the mention. Let me take a look.
Thanks!

Hi @artem-malko . I have started using Tars. As you said it has more features. But I'm in need of help customising the tasks. I'm struck on it. I tried using Gitter. But I can't understand the language. Could  you help me with that..?

@jm-kumaresh Yes, sure. We can create private chat in gitter.


Bug: Instant Crusade Cards does not allow to start events like upgrading a unique or any event that u choose from few options

latest of both versions :D
but at said in title when i have option on so cards are instant, i cant complete any event that has options to choose,
easy ones to check are what way uniques to upgrade etc

https://youtu.be/Ov2ndmChr_o


ImagesOfColorado update

https://www.reddit.com/message/messages/6ew32a

Hey - I use the subreddit r/sips, a subreddit about a YouTuber. I've noticed that quite often images people post there are instantly posted onto your subreddit by, I assume, your bot
So yeah, that's it really.

added /r/sips to the /r/ImagesOfNetwork subreddit blacklist


Unintuitive behaviour when using mouse to make a visual-mode selection

Context:

This issue assumes that the mouseSelectionStartVisualMode setting enabled, meaning that I expect VSCode selections made via the mouse to also become visual-mode selections in the embedded Neovim.

Given:

If I'm already in visual mode (that is, something is selected via Neovim),
and then I use my mouse to select something elsewhere (this would be a VSCode selection),

Intuitive expectation:

what I previously had selected becomes unselected,
and what I selected with my mouse becomes selected.
(that is, both Neovim and VScode show a selection that begins where my mouse clicked, and ends where I released the mouse button)

Actual behaviour:

my previous cursor position becomes the beginning of a new selection,
and where I dragged my mouse to becomes the end of the new selection.
(that is, both Neovim shows a selection that begins where my cursor was previously, and ends where I released the mouse button)

Motivation:

I believe this is one of the things preventing me from creating a function in my ~/.vimrc which allows me to use the "Expand/Shrink Selection" functionality of VSCode to make NeoVim selections. (I want that, in visual mode, each time I press v, it expands my selection, and each time I press V, it shrinks it.)

Naively, I think this can be fixed by playing with this if condition here:
https://github.com/vscode-neovim/vscode-neovim/blob/f9bcd2555c01ad238d9eca06f1c051c43ee71b15/src/cursor_manager.ts#LL353C37-L353C37.
My reading of that code is that, when a VSCode selection change occurs, only if it is not in visual mode, will the VSCode selection entirely replace the Neovim selection. However, my intuition says that the VSCode selection should always replace the Neovim selection.

Playing around with the local version a bit, I've realized that fixing this would not resolve the "Expand/Shrink Selection" issue, so the "motivation" portion of the issue is not relevant.
I've also learned that this is significant effort, as there is a good reason why that if condition which I mentioned earlier exists. This is because, while your mouse is dragging a selection, you don't want it to replace your selection, but rather extend it.
I think this should be able to be resolved by checking if the new selection is beginning from outside the current neovim selection. If it is, then start a new selection, otherwise extend your current one.
However, I'm not experienced with the neovim API, and don't know how to get this information. I'll leave this issue open and might take another crack at it in the future, but not working on it currently.

Fwiw, this can be solved by changing the code you mentioned. Essentially, whenever the vscode selection is updated, you have to re-send that full selection to nvim. Currently, nvim replies by deleting the vscode selection and showing the fake visual selection.
If you want me to explain in more detail I can. I hope to refactor this code at some point.


Design suggestion - create a single method to normalize all text

The order of the normalizations may matter.  In addition to the individual normalization methods, you could add a method to call all of the normalized methods in the correct order.  This would move the logic from the main execution module to the class for the normalizations.
You should also avoid counting on side effects of one method in other methods.  Since the only method with an input file is lowercase, that method must be called first, which is counting on the side effect of that method call.  A better approach would be to pass in the input value to the constructor for the class or only expose a single methods which performs all of the normalizations.

@goneall Done. I have a return function now in normalize_class file that implements all the normalized methods.


generating so many refresh tokens

Hi, I'm using your library on my app there is something I wanted to ask you:
For every refereshToken mutation I get a new refreshToken and and a new token, the refreshToken is saved to database and the old refreshToken which was used to renew refreshToken and token will become revoked. In my opinion this is so much unnecessarry INSERT and UPDATE for the db. maybe you should not renew refreshToken every time we use refreshToken mutation and just renew it a time very close to expire date(like a day before), in this way you can reduce db operations.

Hi @Glyphack !
The refresh token is not revoked when the RefreshToken mutation is called.
If you want to revoke it, you can call to Revoke mutation or create a listener for refresh_token_rotated signal (you can also delete it).
In my opinion, clients and not the API should decide when to refresh the token.

I did used refresh_token_rotated.
about the last sentence I agree with you but there is something:

For every refereshToken mutation I get a new refreshToken and and a new token
So you don't have any choices other than getting a new refreshToken when you want to get a new Token(accessToken), and every refreshToken is being saved on db.

This is a query to get a new JWT token without creating a new refreashToken.
mutation RefreshToken($refreshToken: String!) {
  refreshToken(refreshToken: $refreshToken) {
    token
    payload
  }
}

Until the last release v0.2.0, a refreshToken was created even if it is not requested in the query.
This problem was fixed PR #77, I hope to create a new release this week.

If anyone stumbles upon this, I found a way to configure the reuse of refreshtokens so that new records are not created every time refreshtoken is requested.
GRAPHQL_JWT = {
    "JWT_VERIFY_EXPIRATION": True,
    "JWT_LONG_RUNNING_REFRESH_TOKEN": True,
    'JWT_REUSE_REFRESH_TOKENS': True, # Eliminates creation of new db records every time refreshtoken is requested.
    'JWT_EXPIRATION_DELTA': timedelta(minutes=60), # Expiry time of token
    'JWT_REFRESH_EXPIRATION_DELTA': timedelta(days=7), # Expiry time of refreshToken
}

Source


[BUG REPORT] Exception: "reflection_pad1d" not implemented for 'Short'

Describe the bug
Been getting errors when I try to use whisper on a audio file converted with Audacity to 16000hz, signed 16 PCM, WAV format:
Exception: "reflection_pad1d" not implemented for 'Short'
To Reproduce
Steps to reproduce the behavior:

Use Audacity to convert a english memo mp3 file into wav with 16000hz, signed 16 PCM, WAV format
Upload to Audio WebUI
Load any whisper model (I used base)
Click Transcribe button
See error

Expected behavior
whisper should be transcribing my audio.
Additional context
I tried it on the OG whisper.cpp and it works normally.

Try uploading the file into the batch section, gradio does some conversions before I get access to the audio, which are kind of unpredictable.

Hmm, I've found that by accident. Guess it's gradio issue. Will close this.


command-not-found for OS X with brew

I have adapted the script at https://github.com/bfontaine/brew-command-not-found to work on zsh to enhance the command-not-found plugin.
What it does:
$ ztelnet
The program 'ztelnet' is currently not installed. You can install it by typing:
  brew install zssh

And when there is no match:
$ asdf
zsh: command not found: asdf

Closed as obsolete as per https://github.com/robbyrussell/oh-my-zsh/pull/4537#issuecomment-152771547.


[Bug] Ender chest stuck with me after death [1.16.5]

I found a tower and opened a (trapped) chest that exploded immediately, killing me in the process. The loot/ender chest got destroyed as well but I still get an icon in my inventory screen that allows me to access it, even though there are no ender chests near me. Sometimes, this icon bugs out and appears in the middle of the inventory screen, right where all the other items are, making it hard to click where I want

.

This is not a bag this because of Ring of the Seven Curses for the mod Enigmatic Legacy. At the begging of the game you have as start item in your inventory The ring an amulet and akashic tome. When you die and you pick up your items from the tombstone the ring will be automatic equipped in the curio slots and the amulet as well. The problem is you can't unequip the ring any more you can't even sleep it's bounded to you until you gain the ring of seven blesses. The ring give that ender chest. You should read what the ring does at the beginning. If your not ready for this challenge start a new wort and destroy the ring right at the begging.

Ahh, good to know! I did notice that I had the ring equipped afterwards and was kind of bummed out about it because I did not want to wear it. I tried to google Enigmatic Legacy on the Ring of Seven Curses but there's not a whole lot of documentation on it so I decided to try a new world just to be safe.
Sorry for wasting your time.


Docker image for v1.2.4 is broken

I'm seeing an error pop up for the v1.2.4 Docker image on Quay:
image="quay.io/bcbio/bcbio-rnaseq:1.2.4-9722272"
docker pull "$image"
docker run -it "$image"
bcbio_nextgen.py --version

Traceback (most recent call last):
  File "/usr/local/bin/bcbio_nextgen.py", line 34, in <module>
    from bcbio.setpath import prepend_bcbiopath
ModuleNotFoundError: No module named 'bcbio'

The image for v.1.2.3 works as expected:
image="quay.io/bcbio/bcbio-rnaseq:1.2.3-2d6a5bb"
docker pull "$image"
docker run -it "$image"

bcbio_nextgen.py --version
# 1.2.3

This issue reproduces for the bcbio-vc image as well.

This particular issue should be fixed now.


[misc] Fixed show locally installed vocabularies if no bioportal key

Also added About vocabulary info html parsing for proper display

@sdumitriu Separated.


Allow clicking paths with brackets in terminal

This PR fixes #118687
As described in the issue, popular frameworks like Next.js and SvelteKit use brackets in a path to denote parameters. VS Code terminal currently does not support the file paths in these widely used frameworks

It looks like the fix might be something a bit more than this and some tests would need to be updated


How to use IBD relation ship PI_HAt value to plot the pedigree plot

@justincy  : Thanks for nice Pedigree plot.
I have several samples with their relationship determined by IBD and we have information with pairs of individuals with IBD PHT scores For example input data:
Sample1, Sample2, 0.25
Sample1, Sample3, 0.125
....
How to use your tool to get the pedigree plot
Say:

Identical twins, and duplicates, are 100% identical by descent (Pihat 1.0)
First-degree relatives are 50% IBD (Pihat 0.5)
Second-degree relatives are 25% IBD (Pihat 0.25)
Third-degree relatives are 12.5% equal IBD (Pihat 0.125)

Could you show me an example of how those values would be used to change the pedigree? Do you want to change the position or the color or something else?

http://www.sciencedirect.com/science/article/pii/S0002929714004273
Table 1 expains the ibd values to relations:
Familial Relationship	IBD
Parental	0
Full-sibling	0.25
Half-sibling, avuncular, and grandparental	0.5
First-cousin, great-grandparental, great-avuncular, and half-avuncular	0.75
Distantly related	varies
Unrelated (includes relationships beyond the third degree)	1

@gpcr Thanks. What is it that you need help with? Are you trying to generate a pedigree based on IBD values?

Yes, how to use IBDs to get the pedigre plots

Sorry but I don't know how to interpret IBD values, plus it's out of the scope of this repository. You linked to a paper about it. You can use that paper to generate data in the format used by the examples: https://github.com/justincy/d3-pedigree-examples/tree/gh-pages/data


Simplify expand

@lnr0626 I'm using Surface.Compiler.compile/3 to build the AST instead of defining it manually. I believe that since there's no expression to be evaluated at runtime, this approach is valid for this case. For other more complex components where we may pass dynamic expressions via props that might be defined in different lines in the source, we'll need to build the AST manually so we can be able to customize (or keep) the meta info.

BTW, I made similar changes in the <#Code> component of https://github.com/surface-ui/surface_site/commit/6004d9dbadd69849b4b9d14d5d8a2157fcd3f241, which looks much cleaner now.

will there be any issues if for example the markdown content includes something wrapped in double curlies? i.e. would it try to evaluate that as an expression?


Fix - 24297 Fix null original type bug

Issue #: /bcgov/entity#24297
Description of changes:

Fix the bug: when it's a firm correction, can't change the legal type, but the change type section is in editing mode

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-edit-dev--pr-601-rlu0ijr2.web.app


Uncaught DOMException when clicking a link for the first time.

I'm using turbo via NPM import and when I click a link for the first time, the page renders perfectly fine. But the following error is shown in the console:

However, on the subsequent navigations no such exception is thrown.
I don't know if this is a major issue since I'm not a JS developer. I just wanted to let you people know about the error.
Thanks!

As this can be related to #104 did you check for duplicate loading of the Turbo library?

Turns out the issue was with browsersync.


Keyboard manager not working.

Microsoft PowerToys version
v0.79.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Keyboard manager not starting no matter what. I tried all troubleshooting methods that I could..
✔ Expected Behavior
To open a window for key mapping.
❌ Actual Behavior
No response after clicking Remap a key.
Other Software
No response

PowerToysReport_2024-04-03-12-32-56.zip
report file

issues not resolved from other threads, I tried other builds
KBMEditor78
KBMEditor79
KBMEditor79revertchords
KBMEditor79RevertCppWinRT
None of these work

I installed Release v0.77.0 it works fine, haven't tried other modules except keyboard manager.


JBPM-4714 Case management test coverage

@ibek I'm getting one failure on the build:
Results :

Failed tests:
  ParallelAsyncJobsTest.testRunBasicAsync:80 [More than 2 async jobs should have been executed]
Expecting:
 <3>
to be less than or equal to:
 <2>

Tests run: 235, Failures: 1, Errors: 0, Skipped: 21

Could you look at this test and fix it? Thanks!

@mrietveld  that's strange, it works fine for me on master:
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 19.487 sec - in org.jbpm.test.regression.ParallelAsyncJobsTest
@livthomas could you look at the test since you've developed it?

I have not developed that test only moved it here from our regression test suite. I know there is a problem and it fails every know and then but I have not been able to fix it so far. Right now, it passes on my side. Anyway, from what I have seen in e-mails and other PRs, the stabilization of tests is in progress and this PR does not modify any files that can influence ParallelAsyncJobsTest so I think it can be merged.


Not sure why this happens when i run it

root@Kali:~/whatsapp-analyzer# python analyzer.py vibes_chat.txt
Traceback (most recent call last):
File "analyzer.py", line 334, in
members = sorted(chat.process(), key=lambda m: sum(m.days))
File "analyzer.py", line 96, in process
self._rmnl()
File "analyzer.py", line 90, in _rmnl
prev = prev[:-1] + ' ' + msg
TypeError: 'NoneType' object has no attribute 'getitem'
root@Kali:~/whatsapp-analyzer#

root@Kali:~/whatsapp-analyzer# python analyzer.py vibes_chat.txt
Traceback (most recent call last):
File "analyzer.py", line 334, in
members = sorted(chat.process(), key=lambda m: sum(m.days))
File "analyzer.py", line 96, in process
self._rmnl()
File "analyzer.py", line 90, in _rmnl
prev = prev[:-1] + ' ' + msg
TypeError: 'NoneType' object has no attribute 'getitem'
root@Kali:~/whatsapp-analyzer#

The error happens while formatting the chat file. I reproduced it by adding a blank line at the start of the file, it should work now.


regression_without_regressed_by: consider unaccessible bugs as invalid

Coverage decreased (-0.02%) to 30.417% when pulling 096771eebca6ecd29e74c7f3d6d639194e158cf7 on calixteman:fix_regressed_by into e14a7606c0bde9149c8e86fb9d8af246d8eb62e4 on mozilla:master.


[Refactor] Hardcode FAQ, keynote speakers and team pages

Overview
FAQs, Keynote speakers, and team pictures and info are all currently pulled from a backend. This increases reads significantly as these are pulled up every time someone loads the page. This is information that will not be changed/updated often, thus making it logical to have it hardcoded instead of being fetched from a backend every time.
What Needs to Change
Refactor code to pull FAQs from a JSON file and keynote speaker/team photos and information locally.
Also, add !change flags for organizer config where applicable. Update documentation accordingly or open an issue for documentation rewriting after opening a PR.

Sponsors may be hardcoded as well


RenderingDevice: Use correct layer count for Cubemaps during boundary checks

Follow-up to #94656.
Removes the code which used to incorrectly multiply the cubemap layer count by 6 for boundary checks.

Thanks!


Duration is ignored when run in replay-file mode

Describe the bug
When we run application in reply-file mode it run continuously and does not stop, even if --duration is set
To Reproduce
loadgen replay-file \
--duration 5 \
--input-file-path in.log \
--output out.log \
--result-log result.log \
--lines-per-second 20000 \
--enable-metrics

Expected behavior
Application should honor duration settings
Desktop:

OS: Ubuntu 22.04.1 LTS

Works as expected, thanks!


Problems in training

Hello, I've been studying your code these days. I am very grateful for you because the code is very helpful to me, but I found some small problems in the training process:

I noticed that you use the code in model.py
https://github.com/JisongXie/StegaStamp_pytorch/blob/184642e61d2fa95bcaf4131e64963202267c875d/model.py#L264-L290
which will make the pixel value of encoded_image exceed the boundary (0,1), which will have the image producing noise when displaying, for example:

Since the image trained by the program itself allows a certain color gamut distortion, I think it is necessary to add an torch.clamp sentence to reduce it to the boundary of (0,1) before being input into the decoded net part, and if you agree, I would like to submit a merge request about this.

The original author's program removes the image loss function from the total loss function in the first 1500 iterations, so as to advance the bit accuracy  to a certain extent, but I find that this increase occurs randomly in your program, which means the bit accuracy will increase in not every training in the first 1500 rounds - they sometimes seem to fluctuate around 0.5. Of course, this is not a big trouble. My current solution is to detect the average value of 10 rounds before and after the 1500th iteration, set a threshold of 0.7, and restart training if it is lower than the threshold. But I am still confused that, have you ever encountered this situation? How do you usually solve it?

@Woodley-Griffith Hi, very grateful for you to find out the problem in my code.

The function torchgeometry.warp_perspective transforms the source image using the specified transformation matrix. It seems some small fluctuation will make the transformed result overflow the range [0, 1]. I agree that it is necessary to add an torch.clamp, which is more reasonable and careful. More merge requests are very welcome, to make the code work better.
I remember that I have never encountered this situation. This is so strange. So, according to what you say, sometimes the training is successful, and sometimes the bit accuracy fluctuate around 0.5, and training is fail? It seems that maybe the initialization of model weight is to blame? I think your solution can solve the problem temporarily, but a more reasonable solution needs to be found.

已收到，非常感谢！

@Woodley-Griffith Nice suggestion and I also agree with what you saying,  is there any update about this issue?
I can not find a commit about this issue.

@sunutf not update and test yet

I guess bit_ACC is not promoted because of different versions. Which version of Pytorch are you using?

@Woodley-Griffith Hi, very grateful for you to find out the problem in my code.
1. The function `torchgeometry.warp_perspective` transforms the source image using the specified transformation matrix. It seems some small fluctuation will make the transformed result overflow the range [0, 1]. I agree that it is necessary to add an `torch.clamp`, which is more reasonable and careful. More merge requests are very welcome, to make the code work better.

2. I remember that I have never encountered this situation. This is so strange. So, according to what you say, sometimes the training is successful, and sometimes the bit accuracy fluctuate around 0.5, and training is fail? It seems that maybe the initialization of model weight is to blame? I think your solution can solve the problem temporarily, but a more reasonable solution needs to be found.

The problem related to accuracy is directly related to the weights initialization. I just removed the kaiming_normal init, and it works well.

@lschirmer Hi there, your reply was very helpful. Just to clarify, did you mean you removed the entire spatial transformer block, like this?
# in class StegaStampDecoder(nn.Module):
self.stn = SpatialTransformerNetwork()
...
transformed_image = self.stn(image)

The original paper seems to emphasize using a spatial transformer: "A spatial transformer network [24] is used to develop robustness against small perspective changes that are introduced while capturing and rectifying the encoded image."

Using a mobile phone to shoot encoded pictures on the screen, the decoding level is much lower than the tensorflow model  by author（tancik），someone has this problem？

I trained multiple models and deployed the selected one.
You can check it out at www.aurastamp.com
You need to wake it up by clicking, which will return false the first time.


Chat - Mentions of phone numbers that don't have accounts are rendered as plain text and not mentions.

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

Action Performed:

Open the app.
Open a chat and create a new chat with a random number like +91 9999 8888 33.
Mention the user in any chat using @ symbol.
Notice that mention is shown as simple text in chat.

Expected Result:
Mentions should be shown as mention whether the phone number has an account or not.
Actual Result:
Phone number mentions are shown as text message if they don't have an account.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.30-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/c559879a-44e9-481a-baf3-21cf6f1510a5
https://github.com/Expensify/App/assets/93399543/238ef14d-088f-4935-8ffa-f5319bdc09c8
Expensify/Expensify Issue URL:
Issue reported by: @Nikhil-Vats
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1687512980767079
View all open jobs on GitHub

I think the second video attached is for a different issue but the first video is correct and shows the issue.
For emails, mentions are rendered correctly even if account doesn't exist (see<EMAIL_ADDRESS>in video) but for phone numbers the same doesn't happen.

This can be closed based on this comment. It is a known issue and will be fixed in next iteration.

Good catch @Nikhil-Vats - closing this!


"Content-focused" excludes many websites you can build with Astro

Folks get confused when they read Why Astro. Can I build my site with Astro even if it isn't content-heavy?
It's not the content-focus that's the limit - it's whether your site makes use of shared client-side state across routes.

Is astro good for only content heavy websites?
Like would astro be a bad choice if I wanted to create something like Google forms

Folks get confused when they read Why Astro. Can I build my site with Astro even if it isn't content-heavy?
It's not the content-focus that's the limit - it's whether your site makes use of shared client-side state across routes.

Is astro good for only content heavy websites?

Like would astro be a bad choice if I wanted to create something like Google forms

As a newbie to Astro,I feel this way too @Jutanium. Maybe there can be a paragraph or a section that explains other projects that this product can build? It would lessen the prespective
tone that the first paragraph has.

I agree this should be done and am happy to make a pr. Maybe say something like

Astro sites can have interactivity, but Astro was not designed to share client-side state across routes.

And then give an example of something that can be done with Astro and something that shouldn't be done. Maybe to use the Google forms example, say Astro can include an interactive multi step form on one page, but if the user closes the page, the form won't load exactly where the user left off with the users entries saved? I don't know if that's the best example though. Any thoughts, anyone?


fix(deprecation): move from deprecated listener to compatible hook system in PHPUnit 8

Closes https://github.com/johnkary/phpunit-speedtrap/issues/60

Hi @johnkary can you have a look please?

@vlaurier I have published a new v3.2.0 that updates the Travis CI matrix. Could you merge master into this branch, resolve conflicts and push again to this PR?
This new implementation will be the basis for SpeedTrap v4.0!

@johnkary @mremi
It should be good that way. Notice I also removed PHPUnit 7 support in composer.json as these changes are not BC compatible.

@johnkary , are you OK with these changes then ?

Any updates @johnkary ?

Any ETA on this merge?  Can I do anything to help test?

I would need this to be merged too.

Thank you, @vlaurier. I will have a look soon and merge toward a v4.0 release.

@johnkary
Also, if you can, please take a look at the ✉ that I sent you in regard to the new event system of phpunit/phpunit!
🤓

@johnkary
Also, if you can, please take a look at the ✉ that I sent you in regard to the new event system of phpunit/phpunit!
🤓

@vlaurier Thank you again for your work implementing the Hooks system. I have built on top of your work and introduced PR #83 that completes the Hooks implementation. This will be a new major release version of SpeedTrap once merged.
Please follow progress of #83 for compatibility with PHPUnit 8.0+.


feat: add sankaku complex support

fix #40

Don't change other plugins, this PR should focus on supporting sankaku complex source


Pin filter input to top of panel

The filter input is pinned to the top, so you can always see what you're filtering!

Closes #40!

Nice!


[refactor]: Remove git info from property of ComponentProfile

Overview
The purpose of this issue is to remove git information from ComponentProfile because issue #6 will add a separate git info which is more detailed and useful in analyzing components. Therefore, we should remove git info from ComponentProfile to ensure better maintainability.
These are 3 places we need to remove vue-scanner.ts
Initial property detail
https://github.com/logicspark/berryjam/blob/07dc984e3f870a9ab48305e2e47b61b706092ea0/src/vue-scanner.ts#L693-L697
Interface file property
https://github.com/logicspark/berryjam/blob/07dc984e3f870a9ab48305e2e47b61b706092ea0/src/types.ts#L61-L65
Initial property component
https://github.com/logicspark/berryjam/blob/07dc984e3f870a9ab48305e2e47b61b706092ea0/src/vue-scanner.ts#L516-L520
There are 2 places to remove from analyze-component-files.spec.ts unit test

https://github.com/logicspark/berryjam/blob/07dc984e3f870a9ab48305e2e47b61b706092ea0/tests/analyze-component-files.spec.ts#L215-L219
https://github.com/logicspark/berryjam/blob/07dc984e3f870a9ab48305e2e47b61b706092ea0/tests/analyze-component-files.spec.ts#L236-L240

Expected Behavior

Remove all of them
Scan to check that there is no error in exporting components.json
Check in components.json that git info is no longer in this JSON file

Has this been fix? if not, can I fix it.

@Nkiriobasi Thanks for the interest. Please go ahead and assign yourself 😁
Note: When making code contribution, please refer to Contributing for more information.


Bump plumbing to get latest fixes 👨🚀

Changes
This should fix issues when the diff is too big.
Should fix #1607
/cc @afrittoli @wlynch
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.

/lgtm
But I don't totally understand why this fixes the errors related to large number of files :question:

@sbwsg see https://github.com/tektoncd/plumbing/pull/144 :angel:

/approve

/test pull-tekton-pipeline-integration-tests


[Pkgconf] pkg-config compatibility feature

Cmake uses the "findPkgConfig module to find pkg-config and define functions to use it to find libraries and create imported targets. pkgconf is a drop in replacement for pkg-config that's easier to build on windows (pkg-config itself can actually build on windows, with msvc just fine, however there are problems with static linking (that are being worked on), and most annoyingly, pkg-config depends on glib which depends on pkg-config, meaning you need to do a bootstrap process to get pkg-config to use the "system" glib.
Anyway, while cmake's FindPkgConfig module will work fine if you have pkgconf on the path and set PKG_CONFIG_EXECUTABLE to "pkgconf" or if you set the full path in PKG_CONFIG_EXECUTABLE, it won't look for pkgconf(.exe) using find_library, only pkg-config(.exe). pkgconf recommends creating a symlink named pkg-config to pkgconf if you're using pkgconf to replace pkg-config. Because symlinks are somewhat fought on windows I just created a copy, pkgconf isn't that big.
I came across this problem when using ffmpeg, I don't want to use the vcpkg specific cmake config file provided by vcpkg, and I'd rather not vendor a find module, so the best way to find it is to use pkg-config, as upstream supports. With this patch my program can just depend on pkgconf[pkg-config-compat] and everything works with no vcpkg specific code at all in my cmake.

Personally I see this PR as invalid.
It is up to upstream CMake to add pkgconf into the find_program call done by FindPkgConf if you ask me.
vcpkg could add a vcpkg-cmake-wrapper for it and do the find_program call with pkgconf before the _find_package call. Adding it as feature for it in pkgconf itself seems wrong.

This is a common thing that happens when pkgconf is built.
From the pkgconf readme:
pkg-config symlink
If you want pkgconf to be used when you invoke pkg-config, you should install a
symlink for this.  We do not do this for you, as we believe it is better for vendors
to make this determination themselves.
$ ln -sf pkgconf /usr/bin/pkg-config

We could do it in a separate package that depends on pkgconf, but that's kinda veering into alternative territory. Now that I think about it this is a bit problematic since vcpkg doesn't support incompatible packages.
Here's fedora's package definition for the same, they do the feature thing, but they have "conflicts" dependencies
https://src.fedoraproject.org/rpms/pkgconf/blob/rawhide/f/pkgconf.spec

on further inspection the fedora folks have pkgconf marked to obsolete pkg-config and don't maintain a pkg-config package anymore.

I think It would be reasonable to create a overlay/custom ports for this.

Since pkgconf is not pkg-config but a replacement for it, so I don’t think it can completely replace pkg-config now unless cmake or linux maintainers switch to support it.

Some ground thoughts:

I would really prefer to use pkgconf on windows internally rather than building pkg-config (with all the complexities and bloat) and rather than downloading binaries from msys (and dealing with mirrors + expiring packages)
Given the significant investment by others in the ecosystem (Fedora), I think it's reasonable to expect it to be a sufficient drop-in replacement for our needs.
From the perspective of separating concerns, it is better to isolate the "choice" of what .pc tool to use to be separate from the definition of pkgconf. I can think of 4 reasonable alternatives off the top of my head:

pkgconf:host
pkg-config:host
pkg-config/pkgconf from the system (APT)
pkg-config from MSYS2 (find_acquire_program)

While it's clear we need to provide control and customization within vcpkg for these things, for the final consumer I think it's reasonable for them to run their own find_program(PKG_CONFIG_EXECUTABLE pkgconf) before calling anything using pkgconfig if they know they want to use pkgconf. Any better solution here requires first firming up a solution to expose host ports to consumers.

For internal use, I think the right path forward here is to introduce a metaport that controls the selection of the pc tool. By default, we will probably want to use pkgconf on Windows and system pkg-config on non-Windows. In order to switch dependencies based on the host, this metaport must be a host-only dependency (simply require HOST_TRIPLET STREQUAL TARGET_TRIPLET).
Then, we will also need to add another port which modifies the behavior of FindPkgConfig. The most straightforward way to do this would be to have a vcpkg-cmake-wrapper for it as a target-triplet port, but this would also affect end consumers. I know affecting the end-user is what your motivating desire is for this PR, but at the moment I'd consider it a weak anti-goal :).
If we took the metaport + findpkgconfig wrapper approach, it would look something like:
ports/tool-pkg-config-select/vcpkg.json
{
    "dependencies": [ { "name": "pkgconf", "platform": "windows" } ]
}

ports/tool-pkg-config-select/portfile.cmake
if(NOT HOST_TRIPLET STREQUAL TARGET_TRIPLET)
    message(FATAL_ERROR "${PORT} must be marked as a host dependency")
endif()
if(VCPKG_TARGET_IS_WINDOWS)
  file(WRITE "${CURRENT_PACKAGES_DIR}/share/tool-pkg-config-select/tool-pkg-config-select.cmake"
    "set(PKG_CONFIG_EXECUTABLE \"${CURRENT_PACKAGES_DIR}/share/tools/pkgconf/pkgconf.exe\"")
else()
  file(WRITE "${CURRENT_PACKAGES_DIR}/share/tool-pkg-config-select/tool-pkg-config-select.cmake"
    "find_program(PKG_CONFIG_EXECUTABLE pkg-config)")
endif()

ports/vcpkg-find-pkg-config/vcpkg.json
{
    "dependencies": [ { "name": "tool-pkg-config-select", "host": true } ]
}

ports/vcpkg-find-pkg-config/portfile.cmake
file(WRITE "${CURRENT_PACKAGES_DIR}/share/pkgconfig/vcpkg-cmake-wrapper.cmake"
    "include(\${CMAKE_CURRENT_LIST_DIR}/../../../${HOST_TRIPLET}/share/tool-pkg-config-select/tool-pkg-config-select.cmake)\n"
    "_find_package(${ARGN})")

I'm sure I've made some mistakes with the sketch above, but hopefully it gets my suggestion across.

why not just get the find module PR accepted upstream and then pull in just the new find module from upstream until the new release of cmake is out?
I don't completely understand how the cmake wrappers work though

why not just get the find module PR accepted upstream and then pull in just the new find module from upstream until the new release of cmake is out?

@barcharcraz, please submit a reqest or a PR to Upstream instead. Is work still being done for this PR?

upstream pr https://gitlab.kitware.com/cmake/cmake/-/merge_requests/6301


Re-prioritise Pantsgiving

This is not a problem per se, but it definitely seems to be prioritising basically everything above pantsgiving as a gnomefolk. Currently only have the 500-turn expansion left (in 450 turns). This /feels/ wrong to me, but I might be off in my calcs.
My calculations are as follows: Great wolf beastly pants give +10 weight, which equates to +~4.5 adventures across 450 turns compared to whatever weight it was prior to the GWBP. +1 stomach can easily beat that?
dont hav pantogram pants that drop anything in this life, so shoudlnt be considering those as I understand it.
Once Ive closeted all my +weight pants, it wears the stainless steel slacks.
Though this might have something to do with what appears to be a -20.94 modifier to Pantsgiving I've just noticed in the maximiser string
Based on the Maximier String and convo with phredd - potentially bad algorithm for picking the pantsgiving food?

I believe the issue here is that dreadsylvanian stew is actually a very bad pantsgiving food to use


Dev

This makes the page object responsible for deriving the correct title based on the h1_title wiki option.
The logic here was removed from the gollum app, where it was being processed in the view.

@bogado thanks for the PR. I think we'd still very much like to have this improvement. However, could you:

reopen it against the gollum-lib-5.x branch?
take out the version updates for the dependencies, unless necessary for this fix (otherwise we'd rather have them as a separate PR)
possibly take out the Macro fixes, which should also be a separate PR (unless it's necessary to make the tests pass)
address the test failures

Ping @bogado

@bogado Are you still interested in working on this? Otherwise I'm closing the PR for now.

Closing due to lack of activity.


List item header does not have rem size

Bug Report
Steps

Open the List example at https://stardust-ui.github.io/react/components/list,
open DevTools in Chrome (or another browser),
manually change the font-size of body from 1rem to another value, e.g. 2rem or 50px.

Expected Result
Nothing should change with the List, it still should look the same:

Actual Result
The header of the list scales with the body font-size, but the rest of the list does not. That happens because the header uses the body styling and not its own rem based styling. Stardust should probably not enforce its consumers to specify 1rem in their body font-size.

Version
0.2.6

Thanks for catching this, will be address in my PR: https://github.com/stardust-ui/react/pull/60


How to make sure that the icons font will not be converted

Sass file:
$var-font: "\f123";

.font-a{
	content: $var-font;
}
.font-b{
	content: "\f123";
}

Ootput:
.font-a{content: "";}
.font-b{content: "\f123";}

How to make sure that the icons font will not be converted. My gulp-sass version is 2.0.4 .
Thkns !

Try the latest beta
npm install node-sass@beta

This is a known issue. It doesn't actually affect font loading assuming you've set the character encoding type
<meta charset="UTF-8">

thanks!


CLA singing BOT has changed

It seems the CLA signing BOT has been changed (and improved).
We need to change our documentation to reflect this. If the labels are no longer used.

See PR https://github.com/PowerShell/xSQLServer/pull/840

Seems all current PR's need to be closed and reopened to get the new CLA status.

Ooooh - I hadn't noticed that.


HTTPRouteHostnameIntersection - Add a wildcard listener test case

What type of PR is this?
/kind cleanup
/kind test
/area conformance
What this PR does / why we need it:
The spec on the Gateway states if hostname is omitted all host names must match. This expands the HTTPRouteHostNameIntersection test to assert this test case
spec ref:
https://github.com/kubernetes-sigs/gateway-api/blob/334604c991f591a24b8fb0623693e845ae84867f/apis/v1/gateway_types.go#L244
Which issue(s) this PR fixes:
N/A
Does this PR introduce a user-facing change?:

NONE

@dprotaso at first glance, this seems like it overlaps with https://github.com/kubernetes-sigs/gateway-api/pull/2669, can you clarify if/how it's different?

That linked test is 'Extended' - we don't have a 'Core' test for the functionality I quote from the text
I was looking for a 'core' test that:

Doesn't specify a hostname in the Gateway
Has unique hostnames names in the HTTPRoute
Requests succeed for hostnames defined in the HTTPRoute
Requests fail for hostnames not defined in the HTTPRoute

adding approvers
/assign @arkodg @mlavacca @sunjayBhatia
adding reviewers
/cc @LiorLieberman @michaelbeaumont @Xunzhuo

Conformance approvers - can I get review please @arkodg  @mlavacca @sunjayBhatia


Comments for gallery items

Gallery items should also be able to comment on.

Show comment count in photo frame in the footer next to the likes info.

Issue resolved now.


We have consul doc for cluster manager. This log shows your YAML is not well formated.

We have consul doc for cluster manager. This log shows your YAML is not well formated.
Originally posted by @wu-sheng in https://github.com/apache/skywalking/issues/4427#issuecomment-592458554
I also have the same exception and it is ok to verify the yaml format.
skywalking version:apache-skywalking-apm-es7-6.6.0

Duplicated, there is nothing we could say.


Test NAMES on invalid/nonexisting channel returns RPL_ENDOFNAMES.

Failure for InspIRCd, patch submitted: https://github.com/inspircd/inspircd/pull/1922

shipit


Bflieck8

updated some views with bootstrap for responsive, updated edit-post to remove the default coming soon image, updated the coming soon image place holder, updated post routes with npm package, still needs work

resolved all merge conflicts all 3 team members were present everyone agreed


Unify naming of Send* functions

The current naming of the Send* functions is not unified, therefore it might be a little confusing when used in other apps.
For example, reading the code in insights-results-aggregator, you can immediately tell what HTTP Code is sent when you use responses.SendInternalServerError or responses.SendCreated, but you can't really tell which code is sent when you use responses.SendResponse and responses.SendError
Acceptance criteria:

SendResponse renamed to SendOk
SendError renamed to SendBadRequest
Created pull requests in other applications with these changes implemented and waiting for this to be merged

@Bee-lee should be close this one please?

@tisnik Sure, it's done, I guess I forgot to close


metrics: Only systemctl restart once per cycle

Rather than doing a systemctl restart on docker and the proxy
for every test invocation, lets just do it the once at the
start before we begin the test cycle.
Fix up the invocation scripts to make the one off call.
Predominanlty because running many frequent restarts on my
ubuntu machine results in systemd disabling docker.service
as it sees it as restarting too often.
Fixes: #619
Signed-off-by: Graham whaley<EMAIL_ADDRESS>

lgtm

lgtm


Questions about evaluation on VITS

It seems the eval for VITS is not performed during training as there is no annotations in json file
# Test set json files do not contain annotations (evaluation must be performed using the COCO evaluation server).  self._do_evaluation = "annotations" in self._ytvis_api.dataset
So is it OK if I offer a json file with annotation in it?

Yes, if you have annotation for the validation set you can evaluate it during training.


the hacks for bookmark does not work

when i used the bookmark noclip.js it opened a webpage and not the hack for noclip help would be  really good

MY GAME GLICHED!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!


[Improvement] it would be better if jsons:getString method can create a string representation if a collection is returned.

In a situation where a json path such as "$..*" is used with jsons:getString() in balllerina 0.8, following error is observed,
org.ballerinalang.util.exceptions.BallerinaException: Failed to get string from json. Error while executing jsonpath: The element matching path: $..* is not a String. 	at org.ballerinalang.nativeimpl.lang.utils.ErrorHandler.handleJsonPathException(ErrorHandler.java:64) 	at org.ballerinalang.nativeimpl.lang.jsons.GetString.execute(GetString.java:94)
It would be better if jsons:getString() method can create a string representation if a collection is returned from evaluating json expression, as this would be the expected behavior according the API doc[1] and what someone comming from a loosly typed programming langauage would expect.
[1] - http://ballerinalang.org/docs/api/0.8/ballerina.lang.jsons.html

jsonpath are no longer supported in ballerina. The existing method toString() returns the string representation of the JSON. hence closing the issue.
sample:
function main (string[] args) {
    json j = {"foo":"bar"};
    string s = jsons:toString(j);
    system:println(s);
}

Output:
{"foo":"bar"}


Can't create nested route with the same first route level name

I would like to call one page for /discover and another page for /discover/detail, but it creates an error.
routes.ts :
{
    path: "/discover",
    component: PageDiscoverHome,
  },
  {
    path: "/discover",
    component: PageDiscoverList,
    children: [
      {
        path: "/detail",
        component: PageDiscoverDetail,
      },
    ],
  },

Error when nested route are called :
Router id 2 > no routes array is set.

Is it normal you define the same path twice on first route level ? It can't work in that way.

In order to complete my answer :
{
    path: "/discover",
    component: PageDiscoverHome,
  }

make /discover available
  {
    path: "/discover",
    component: PageDiscoverList,
    children: [
      {
        path: "/detail",
        component: PageDiscoverDetail,
      },
    ],
  },

make /discover and /discover/detail available.
The possible routes definition I can suggest could be :
{
    path: "/",
    component: PageDiscoverHome,
  },
  {
    path: "/discover",
    component: PageDiscoverList,
    children: [
      {
        path: "/detail",
        component: PageDiscoverDetail,
      },
    ],
  },

In my opinion, router should act like a switch case.
So that it's not a problem to have a nested route base, with the same name.
But in my current project, I don't really need to view urls, so I have another workaround that works.
  {
    path: "/discover-home",
    component: PageDiscoverHome,
  },
  {
    path: "/discover",
    component: PageDiscoverList,
    children: [
      {
        path: "/detail",
        component: PageDiscoverDetail,
      },
    ],
  },


image merge

Merge mysql and redis CloudImage, sometimes SaaS applications need to combine various middleware, if we can merge mutiple CloudImages, user needn't repeat the build.

sealer merge allinone:latest mysql:latest redis:latest

#601


Fixed the UI of ContactUs

Title of the Pull Request
Fixed UI of ContactUs
Description
Fixed size of the input boxes which improved the UI of Contact Us section
Screenshots 📷
Before:

After:

Checklist

[ ] My code follows the code style of this project.
[ ] I have followed the contribution guidelines
[ ] I have performed a self-review of my own code.
[ ] I have ensured my changes don't generate any new warnings or errors.
[ ] I have updated the documentation (if necessary).
[ ] I have resolved all merge conflicts.

Please mention your issue no.


Make sure the latest docstring parser is not used

Description

Short description here *

closes Add ticket reference here
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Failed to merge changes into iso3 due to merge conflict.Please open a pull request for these branches separately.

Merged into branches master in e61c41a799a256a46acce57811b0b9b703058714

Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches. You can safely close this pull request and delete the source branch.

Processing #2980.


The provided matrices are being modified inplace for cosine matrix to matrix distance computation

Hi @talboger
The matrices provided to the function def matrix_to_matrix_distance()  with 'cosine' as the distance metric is getting modified inplace.

good catch, fixed in https://github.com/talboger/fastdist/commit/a7f21b86f81acfbae7bf94c6a4d105ab0744efeb


Error: DNS_ERROR_BAD_PACKET [code 0x0000251e]  A bad packet was received from a DNS server. Potentially the requested address does not exist.

Version: <IP_ADDRESS>4
OS/Distro: Ubuntu desktop 16.04 - unity desktop
Issue/Impact:
Note: replace content with your own
When reporting an issue it's important that we have as much detail as you can provide. The following is a list of commands to check.

systemctl status lwsmd.service
/opt/pbis/bin/lwsm list
/opt/pbis/domainjoin-cli query
pbis status
/opt/pbis/bin/enum-users
attach logs

/opt/pbis/bin/lwsm set-log-target -p lsass -  file /tmp/lsass.log
/opt/pbis/bin/lwsm set-log-level -p lsass - debug
attach log

Output/Error:
Steps to Reproduce:

install command
Domainjoin command
Command that returns issue

Not enough information.
pbis-open will longer receive updates and will be archived. Closing all outstanding issues. Please consider BeyondTrust Active Directory Bridge for continued support.
https://www.beyondtrust.com/privilege-management/active-directory-bridge


Uncaught TypeError: branch.autoAppIndex is not a function

Got this error using the following code:
(function(b,r,a,n,c,h,_,s,d,k){if(!b[n]||!b[n]._q){for(;s<_.length;)c(h,_[s++]);d=r.createElement(a);d.async=1;d.src="https://cdn.branch.io/branch-latest.min.js";k=r.getElementsByTagName(a)[0];k.parentNode.insertBefore(d,k);b[n]=h}})(window,document,"script","branch",function(b,r){b[r]=function(){b._q.push([r,arguments])}},{_q:[],_v:1},"addListener applyCode banner closeBanner creditHistory credits data deepview deepviewCta first getCode init link logout redeem referrals removeListener sendSMS setBranchViewData setIdentity track validateCode".split(" "), 0);

  branch.init('key_live_xxxx');

  branch.autoAppIndex({
    iosAppId:'xxx',
    iosURL:'#/tab/xxx/303',
    androidPackageName:'com.br.xxx',
    androidURL:'#/tab/xxx/303',
    data:{"id":303}
  }, function(err) { console.log(err); });

  branch.deepview(
      {
        'channel': 'mobile_web',
        'feature': 'deepview',
        data : {
          'page_id': 'xxx303',
          '$always_deeplink': true,
          '$deeplink_path': '#/tab/xxx/303/',
          '$fallback_url':'http://www.xxx.com.br/exameple.html',
          '$publicly_indexable':1,
          '$canonical_identifier': 'http://www.xxx.com.br/exameple.html',
          '$desktop_url':'http://www.xxx.com.br/exameple.html',
          '$keywords':['example','branch'],
        }
      },
      {
        'open_app': true,  // If true, Branch attempts to open your app immediately when the page loads. If false, users will need to press a button. Defaults to true
      }
  );

Looking at the documentation (https://github.com/BranchMetrics/web-branch-deep-linking and https://dev.branch.io/features/google-app-indexing/advanced/cordova/) it says that the key is named "iosAppStoreId" however in the example it is shown as "iosAppId", don't think that this error is related but I found this error.
Thanks in advance

---- SOLVED -----
Never mind, solved by including autoAppIndex on the call function:
(function(b,r,a,n,c,h,,s,d,k){if(!b[n]||!b[n].q){for(;s<.length;)c(h,[s++]);d=r.createElement(a);d.async=1;d.src="https://cdn.branch.io/branch-latest.min.js";k=r.getElementsByTagName(a)[0];k.parentNode.insertBefore(d,k);b[n]=h}})(window,document,"script","branch",function(b,r){b[r]=function(){b._q.push([r,arguments])}},{_q:[],_v:1},"addListener applyCode banner closeBanner creditHistory credits data deepview deepviewCta autoAppIndex first getCode init link logout redeem referrals removeListener sendSMS setBranchViewData setIdentity track validateCode".split(" "), 0);
However I still have this doubt about the different key names iosAppId or iosAppStoreId. Also, which ID is this? How can I exactly find it?

@czmarcos sorry for the confusion. The correct parameter is iosAppId and I have updated the documentation over at dev.branch.io with the correct parameter. The github repo's readme file should be updated with the next deploy of the WebSDK.
Also to find your iosAppId, please:

Copy and paste your app's iTunes URL into a text file
Skip to the end of the URL and copy the string of numbers right after id

For example, if we look at the iTunes URL for Branch Monster Factory below:

https://itunes.apple.com/us/app/branch-monster-factory/id**917737838**

The app store id is 917737838.
Hope this helps!

Thanks @rubinsingh!


1D Heat Diffusion Equation: Graph Simulation pyScripts

What's in this PR?

Python script for graph plot of 1D heat equation using FTCS Explicit method, updated approach to plotting the solution

Python script for graph plot of 1D heat equation using Crank-Nicolson Implicit method

Advisory Needed:

Boundary Conditions in Crank-Nicolson set identical to explicit method [SOLVED] - 05/11/2024
BUG: The explicit method solution plot is weird its probably to do with my solution itself [SOLVED] - 05/11/2024

Will be very late in the night before I push my code today, but I will get it done so you can review it tomorrow and get it merged

@alphaville so turns out the bug with my plot had to do with 2 () when calculating the lambdaConstant in code. Its fixed now and the assert code was already added by us during the wednesday meeting, do you want me to add a further error message in the assert? Please advise

@chemardes could you review it when you have some time and comment how readable the code is?

See a couple of comments from my side

@alphaville I have marked the commented code comment thread as resolved and gotten back to you on the others


refactoring project to use cloud init and vanilla terraform for the configs

Is it possible to refactor the project to add an option to use cloud-init or just pure terraform?

Hi @aleon1220 - thanks for your comment. Can you clarify what do you mean? Terraform and cloud-init are both used in the project to kick-start the environment.

Hi @aleon1220 - thanks for your comment. Can you clarify what do you mean? Terraform and cloud-init are both used in the project to kick-start the environment.

Hi @aleon1220 - thanks for your comment. Can you clarify what do you mean? Terraform and cloud-init are both used in the project to kick-start the environment.

yes for example an option that uses 100% terraform provisioning. For the docker-compose part one could use a terraform file provider.
So the main script could offer something like

create infra with terraform only
create infra with terraform + cloud init

i hope it makes more sense.
thanks

Hi @aleon1220 - thanks for your comment. Can you clarify what do you mean? Terraform and cloud-init are both used in the project to kick-start the environment.

yes for example an option that uses 100% terraform provisioning. For the docker-compose part one could use a terraform file provider.
So the main script could offer something like

create infra with terraform only
create infra with terraform + cloud init

i hope it makes more sense.
thanks

I see what you mean. Thing is, provisioners are not encouraged by Hashicorp: in the documentation, they encourage to use either cloud-init or Packer. I wanted to explore Packer a bit further but right now I'm trying to figure out first how to best manage the data persistence aspect of the setup.
If you have any ideas feel free to fork this and let me know how it goes. Thanks for your input!

I see what you mean. Thing is, provisioners are not encouraged by Hashicorp: in the documentation, they encourage to use either cloud-init or Packer. I wanted to explore Packer a bit further but right now I'm trying to figure out first how to best manage the data persistence aspect of the setup.
If you have any ideas feel free to fork this and let me know how it goes. Thanks for your input!

You are right @Vortexmind  Hashicorp discourages the use of provisioners.
Maybe using Terraform Plugin-providers
I will get started with your cool project and if I find improvements I will create the Pull Request PR

You are right @Vortexmind  Hashicorp discourages the use of provisioners.
Maybe using Terraform Plugin-providers
I will get started with your cool project and if I find improvements I will create the Pull Request PR


Wide format input data

Implement function to detect if dataset is wide or long, and make it possible to import wide data.

to test with wow data


Reduce allocations

I ran a allocation test and found some places that produce large numbers of allocations.

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/StyleCop.Analyzers/StyleCop.Analyzers/Helpers/FileHeaderHelpers.cs#L79

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1516ElementsMustBeSeparatedByBlankLine.cs#L217

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1516ElementsMustBeSeparatedByBlankLine.cs#L248

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/64114f1292a6305335429cd89aac6b577fd4f589/StyleCop.Analyzers/StyleCop.Analyzers/ReadabilityRules/SA1121UseBuiltInTypeAlias.cs#L187 (The profiler highlights the return statement, we should investigate whats the problem here)

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/64114f1292a6305335429cd89aac6b577fd4f589/StyleCop.Analyzers/StyleCop.Analyzers/LayoutRules/SA1509OpeningCurlyBracketsMustNotBePrecededByBlankLine.cs#L85 (We probably need a "Dont use TriviaList.AddRange" analyzer)

https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/568898463e62e4ceec124b2dedc389de80da7731/StyleCop.Analyzers/StyleCop.Analyzers/Helpers/DocumentationCommentExtensions.cs#L20

Most of the points I mentioned are fixed now or no longer up to date.