[Decode] Fix VC1d small bitstream issue

To fix vc1d small bitstream issue mentioned at https://github.com/Intel-Media-SDK/MediaSDK/issues/2618

@varistar, @dvrogozh could you pls have a look?


chore: update engines versions in package.json

Summary
Type

Chore

Summarise concisely:
What is expected?
Update engines versions in package.json to match our other repositories.

@chambo-e @matthprost I've updated the versions to major.x as discussed.


Implemented `\href' command

Implemented \href command in LaTeX hyperref package.
It seems that LaTeX simply ignores special commands for hyper-linking and processes spacing as usual.
Since I don't make out how to simulate this spacing behaviour in KaTeX, I adopt the following strategy:

If it has the only element, then use its math-type;
if it has more than two elements, and the types of both ends of elements coincides, then use it as a class for the anchor tag;
otherwise, we inject empty span-tag aside both ends of generated anchor tag, both with the same classes as the first and the last ones respectively.

Thanks @konn for working on this.  \href is definitely something we should support.
I don't understand what you mean by your three rules though.  \href should not be setting class, should it?  Isn't that the purpose of \class (see #90)?
Isn't the second argument of \href treated as if it's in \text?  You should be able to use code similar to \text, \raisebox, etc. which switch to text mode.

I don't understand what you mean by your three rules though. \href should not be setting class, should it? Isn't that the purpose of \class (see #90)?

Well, if you don't specify any math-type via class of an anchor tag, you get the different spacing behaviour than latex's default.
Consider the following input:
\documentclass[a4]{article}
\usepackage{hyperref}
\usepackage{amsmath,amssymb}
\begin{document}
$0 \href{http://en.wikipedia.org/wiki/Equals\_sign}{=} 1$.

$0 = 1$.
\end{document}

If you typeset the above by xelatex, lualatex (or latex + dvipdfmx with dvipdfmx argument for hyperref package), you get the following result, with the upper equals-sign linked to Wikipedia:

Here, we see that both \href'ed version of = and plain = result in the same spacing.
On the other hand, if we drop the math-type handling in KaTeX, we get the following result:

So, to get the same spacing adjustment, we have to set a class of anchor tag to mord, mrel or so.

Isn't the second argument of \href treated as if it's in \text? You should be able to use code similar to \text, \raisebox, etc. which switch to text mode.

As above, LaTeX's (more precisely, hyperref's) \href commands accept a math as its second argument, not only text. So we can't treat them as if they're in \text.

Ah, now I understand where you're getting the classes.  (I was worried that they were being specified by the user, but rather they're grabbed from the rendered arguments.)  Now this seems like the right approach to me.  Cool!

@konn Just checking: Is this ready for review again?

@edemaine Yes. I think it’s ready.

I'll have another look this weekend.

@konn sorry about the delay.  Please let me know if you have any questions/concerns about any of the feedback.

@kevinbarabash No problem, thank you for your review!

This could also use some unit tests to verify that \hrefs parse and don't parse in the expected way, see katex-spec.js. Also, it'd be nice to have a snapshot test for the MathML that we're outputting, see mathml-spec.js (running npm test -- -u will update the snapshot file).

OK, I will try to write some unit-tests.

OK, I will try to write some unit-tests.

@konn let me know if you need help with this.

@kevinbarabash Is there any time to look into this?

@konn I've reviewed parseStringGroupWithBalancedBraces, looks good.  I will merge this after the tests pass.  Sorry for taking so long with this.

Thanks!


Fix issue with zsh 5.0.x with local variables and subshells

Tested against 5.0.0 and 5.1. I don't know the root cause of the problem. I should probably checkout zsh-users list or ask on #zsh channel.
Fixes #25

Perfect! Thanks!


[Compute] New command group az vm image terms

Issue #9597
Excited to announce 3 new commands!
az vm image terms accept
az vm image terms cancel
az vm image terms show

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

/azp run Azure.azure-cli (Test Python Wheels)

/azp run Azure.azure-cli

The code looks good. But I prefer to create a new command group called marketplace and put terms there.

/azp

/azp run


Syntax error in Pycord Guide > Interactions > Application commands > Contextual menus > User Commands

Summary
A syntax error in the User Commands code
Reproduction Steps
Hi I am new to the community and I was reviewing the Pycord Guide, but I found a syntax error in the Context Menus section, specifically in the User Commands section, the error is that if you put the guild.ids=[] attribute in the decorator, the User Command does not work, but if you remove it magically works. On the assumption that I have it wrong, I would like you to add an explanation of how to do it right in the Pycord Guide.
Explanatory video
https://github.com/Pycord-Development/guide/assets/92834159/f0c2cc11-29d6-448c-a79c-d418c41e92a6
Expected Results
If this error does not occur, it should run normally.
Actual Results
Gives https error or simply does not give error, but it does not work.
System Information
No response
Checklist

[x] I have searched the open issues for duplicates.

Additional Context
I am Hispanic and my English is very bad, so I used Deepl's translator to express myself, so I apologize if there is something wrong written.

You are supposed to replace ... with the Guild ID of the server you are testing in. (Such as<PHONE_NUMBER>29110855) Will update the guide to make this clearer.

Perfect, thank you very much for making the guide a little clearer.


libtorch

Hi, are there any plans for C++ libtorch interface? Regards,

Currently, there are no plans. Do you have a particular use case for this?

Basicly, C++ development. I want to speed up tensor manipulation/computation/autodiff. There's an option to use https://arrayfire.com and https://github.com/flashlight/flashlight , but I'd like smth more JAX-like. Could you please briefly describe what is needs to be done to prepare C++ interface?

+1 for the c++ interface. My use case is to get per batch Jacobian (and possibly higher order derivatives).
I am not sure why the entire ATen/functorch is out-of-tree. For now, I am going through the annoying route of compiling from source and then copying the ATen/functorch into my project. Writing the actual c++ version of vmap is just 20 lines of code (assuming all arguments are flattened tensors), but mirroring all the stuff like DynamicLayer and TensorWrapper gives me headache.
Well, that was a long way of saying I'd appreciate a native c++ interface.


Left Nav Title is inconsistent and repeated multiple times

The page title repeats feel odd — same for other sections.

Agreed this is odd, suggestions on how to fix?
@gauntface @PaulKinlan

What do you want the solution to be?
Consider this:

I'm happy to just remove the heading, but some has said that.

This looks fine, but I think the whole site should be called Web Fundamentals, and the Web Fundamentals 'section' should be renamed Get(ting) Started (or whatever).
Either that or we have to rename the whole site 'Slash Web' or whatever — but then we lose the brand value of Web Fundamentals.

Just removed the heading.

So does the whole site have a name, since Web Fundamentals is a section of it?

@samdutton - officially, it's slash web. ;)

OK :).
Actually — the written name /web/ is quite cool.


option to not open duplicated file

When I duplicate a file, it's almost always because I want a backup of the file and I do not want to view or edit this duplicate file. It would be great if there were an option to not open a duplicate file.
Thanks for this cool extension! 🐰 🐇

hi @nancym
thanks for the feedback. im glad you like the extension.
it sounds like a common enough scenario to want to duplicate a file without opening it. a core reason for making the extension was to reduce the number of hops to do stuff like this, hence the name!
initially, i wanted to keep the numbers of commands to a minimum and see how I found using them. I wanted to see if i bumped into something i felt I needed but was missing and add them on that basis to keep it lean. i rarely find myself duplicating files, so this is not a pain point me! however, i am happy to add something like this if enough people feel the pain.
there is a case for making alternative commands for the existing commands that work on active files (delete active file ..etc) also. i created a new issue for this #2 .You can give a 👍 to the issue and i will consider adding it if there is more interest, and i have time! atm, my time for this is very limited.


Bump versions for upload to CHaP

@erikd Can you please revert this PR. Version bumps were completely wrong


API Fix & Support for Scheduling

Looks like a capitalization bug in the headers method of the api.py file.  Suddenly, PetSafe expected that the "token" key be capitalized to "Token" or it would give a 404.
Fixed that, and also added some methods for manipulating schedules.
Tried to submit the API fix and the scheduling as separate pull requests, but I'm somewhat new to git and it's not 4AM my time so I stopped caring...
Hope all is well!

Resetting my fork to clean up the commits.  Will issue a new pull request when it's cleaner.


Adds the Wikiann NER corpus for Indonesian

Closes #138 "
The code has been enabled for selecting any language. The languages map LANGUAGES_MAP & LANG_CODES needs to be filled with other languages which are Indonesian.
Checkbox

[x] Confirm that this PR is linked to the dataset issue.
[x] Create the dataloader script nusantara/nusa_datasets/my_dataset/my_dataset.py (please use only lowercase and underscore for dataset naming).
[x] Provide values for the _CITATION, _DATASETNAME, _DESCRIPTION, _HOMEPAGE, _LICENSE, _URLs, _SUPPORTED_TASKS, _SOURCE_VERSION, and _NUSANTARA_VERSION variables.
[x] Implement _info(), _split_generators() and _generate_examples() in dataloader script.
[x] Make sure that the BUILDER_CONFIGS class attribute is a list with at least one NusantaraConfig for the source schema and one for a nusantara schema.
[x] Confirm dataloader script works with datasets.load_dataset function.
[ ] Confirm that your dataloader script passes the test suite run with python -m tests.test_nusantara --path=nusantara/nusa_datasets/my_dataset/my_dataset.py.
[ ] If my dataset is local, I have provided an output of the unit-tests in the PR (please copy paste). This is OPTIONAL for public datasets, as we can test these without access to the data files.

Thanks @holylovenia for the suggestions! Will update this PR shortly!

Thanks @SamuelCahyawijaya! I've fixed the issues you pointed out and added other languages as well. For  "Banyumasan", I've used "bms" as the code. The test case does not specify language so it's throwing an error. Should I default to one of the languages or leave it as it is?

Okay, I've kept the original lang tags then :)


Get unique id of server after server creation

When a server is created it returns what looks like a single server object.  But, using
$result->getUuid()
results in an error - 'call to undefined method getUuid'.
getUuid is a getting defined in model/server.php
Any idea what I could be doing wrong?  I've taken the example from the home page of this project and just added the called to $result->getUuid() so not sure what I'm missing.
Thanks.

Actually, it looks like if the createServer api returns a server response object and you need to use the getServer method first:
$result->getServer()->getUuid()

Glad you figured it out 🙂


--wip-- [skip ci]

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1750 into master will increase coverage by 8.91%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1750      +/-   ##
==========================================
+ Coverage   14.04%   22.96%   +8.91%
==========================================
  Files         281      238      -43
  Lines       56728    15965   -40763
==========================================
- Hits         7968     3666    -4302
+ Misses      47837    12024   -35813
+ Partials      923      275     -648

Impacted Files
Coverage Δ

pkg/httputils/http_utils.go
0% <0%> (-100%)
:arrow_down:

daemon/logger/logmessage.go
0% <0%> (-100%)
:arrow_down:

pkg/httputils/http_error.go
0% <0%> (-100%)
:arrow_down:

apis/server/router.go
0% <0%> (-91.43%)
:arrow_down:

daemon/mgr/container_state.go
0% <0%> (-87.24%)
:arrow_down:

apis/server/volume_bridge.go
0% <0%> (-86.67%)
:arrow_down:

daemon/mgr/container_logger.go
0% <0%> (-84.62%)
:arrow_down:

apis/server/container_bridge.go
0% <0%> (-82.02%)
:arrow_down:

daemon/mgr/volume.go
0% <0%> (-81.25%)
:arrow_down:

ctrd/image.go
0% <0%> (-78.72%)
:arrow_down:

... and 113 more

LGTM

The change is very large. @YaoZengzeng , @Letty5411 and @chuanchang please take a look again. Thanks

Another question is do we need to modify related docs in pouch/doc?


In "Editor", have a crash.

This application will crash when I click the settings button in toolbar(ActionBar).

Fixed in newest version https://play.google.com/store/apps/details?id=org.qpython.qpy3


docs(README.md): deprecating kilt

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area tests

/area proposals

What this PR does / why we need it:
See https://github.com/falcosecurity/evolution/issues/347
/assign @gnosek

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: leogr
Once this PR has been reviewed and has the lgtm label, please assign leodido for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gnosek, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [gnosek]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Werewolf Agility Course Highlight stick (and maybe npc at the end)

Honestly i know it might sound dumb, but when i'm not paying the best of attention on the werewolf course, i often forget to pick up the stick, yeah i know it might be dumb but it does happen. Maybe it would be possible to make it similar to marks of grace, where it highlights it on the floor if you haven't picked it up yet and makes the next obstacle red untill you pick it up?
Also this is something i haven't seen before so i'm not sure about the possibilities, but maybe a reminder in some shape of form that you have to turn the stick in? Maybe it's possible to highlight the npc when you have a certain item, i'm not sure about that but I do believe highlighting the stick would basically be exactly the same as the marks of grace, but maybe i'm wrong.

If you pickup the stick does it appear in your inventory? And if you hand it in, does he take it from inventory?

@KasparRosin Yes, it appears in your inventory, and when you hand it in it dissapears.


export_pre_model.py is not working anymore

I have downloaded the sam hq models and executed this exporter script. This error occurs because the original segement_anything is imported.
If I change the code of the exporter to segement_anything_hq, the script runs without error, but does not export any models.
Could you please upload the exported models to github?
s:\Sources\Master\segment-anything-cpp-wrapper>python export_pre_model.py
Traceback (most recent call last):
File "s:\Sources\Master\segment-anything-cpp-wrapper\export_pre_model.py", line 56, in
sam = SAM.sam_model_registrymodel_type
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\znoopy2k\AppData\Local\Programs\Python\Python311\Lib\site-packages\segment_anything\build_sam.py", line 15, in build_sam_vit_h
return _build_sam(
^^^^^^^^^^^
File "C:\Users\znoopy2k\AppData\Local\Programs\Python\Python311\Lib\site-packages\segment_anything\build_sam.py", line 106, in _build_sam
sam.load_state_dict(state_dict)
File "C:\Users\znoopy2k\AppData\Local\Programs\Python\Python311\Lib\site-packages\torch\nn\modules\module.py", line 2041, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for Sam:
Unexpected key(s) in state_dict: "mask_decoder.hf_token.weight", "mask_decoder.hf_mlp.layers.0.weight", "mask_decoder.hf_mlp.layers.0.bias", "mask_decoder.hf_mlp.layers.1.weight", "mask_decoder.hf_mlp.layers.1.bias", "mask_decoder.hf_mlp.layers.2.weight", "mask_decoder.hf_mlp.layers.2.bias", "mask_decoder.compress_vit_feat.0.weight", "mask_decoder.compress_vit_feat.0.bias", "mask_decoder.compress_vit_feat.1.weight", "mask_decoder.compress_vit_feat.1.bias", "mask_decoder.compress_vit_feat.3.weight", "mask_decoder.compress_vit_feat.3.bias", "mask_decoder.embedding_encoder.0.weight", "mask_decoder.embedding_encoder.0.bias", "mask_decoder.embedding_encoder.1.weight", "mask_decoder.embedding_encoder.1.bias", "mask_decoder.embedding_encoder.3.weight", "mask_decoder.embedding_encoder.3.bias", "mask_decoder.embedding_maskfeature.0.weight", "mask_decoder.embedding_maskfeature.0.bias", "mask_decoder.embedding_maskfeature.1.weight", "mask_decoder.embedding_maskfeature.1.bias", "mask_decoder.embedding_maskfeature.3.weight", "mask_decoder.embedding_maskfeature.3.bias".

The preprocessing model for SAM-HQ is already included in the release 7z file (models/sam_hq_preprocess.onnx, for image size of 1024x720). It can be used directly, for example: ./sam_cpp_test -pre_model="models/sam_hq_preprocess.onnx" -sam_model="models/sam_hq_vit_h.onnx".

Perhaps installing SAM-HQ using standard methods within a virtual environment and then invoking export_pre_model.py could work.

The standard installation method worked. Thank you!


Directional lightmap doesn't work correctly with Desnoiser.

Godot version
v4.1.1.stable.mono.official [bd6af8e0e]
System information
Godot v4.1.1.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated AMD Radeon RX 580 2048SP () - Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz (32 Threads)
Issue description

Directional On, Denoiser Off

Directional On, Denoiser On
The lighting result just looks incorrect while using Directioal and Denoiser simultaneouly. But using each of them alone, the result would be correct. No mater how high quality setting used for LightmapGI and how large lightmap size used, this issue remains.
Steps to reproduce
Create a box mesh on the ground. Add UV2.
Assign a standard gray material to box and floor. No reflection, roughness is 1, metallic is 0.
Setup a simple custom sky or color background as environment lighting.
Bake lightmaps using high or ultra quality preset.
Minimal reproduction project
Box_4.1.zip

It looks like this in master with the new denoiser:

This appears to be correct.

Closing, as this should be fixed as of 4.2.dev6 per the above comment.


The Taiko Team should consider a moderate gas fee for Taiko Blockchain come Mainnet

Describe the feature request
Description of the feature request here.
A feature request for the Taiko Team to consider moderate gas fee on the Taiko Blockchain come Mainnet. Moderate Gas fees on transactions has lot of benefits to both the Blockchain users and the Growth of the Ecosystem.
Here are some advantages of moderate gas fees on Taiko.
Advantages of Low gas ⛽ fee to users on Taiko Blockchain:

Affordability:

Moderate gas fees make transactions and interactions with smart contracts more affordable for users. This is especially crucial for retail users and those engaging in smaller transactions.

Increased Accessibility:

Lower gas fees improve accessibility, enabling a more diverse user base to participate in the ecosystem. It will open up opportunities for users with limited financial resources to engage with Taiko decentralized applications (DApps) and other blockchain services on the Taiko network.

Encourages Regular Usage:

Users are more likely to engage in regular transactions and interactions on the Taiko blockchain when gas fees are moderate. This fosters continuous user engagement, contributing to the overall vibrancy of the ecosystem.

Fosters Innovation:

Lower transaction costs encourage developers to experiment with new ideas and create innovative decentralized applications on the Taiko Ecosystem.

How low gas ⛽ fee on transactions will enhance in  growth of the Taiko Blockchain:

Developer Adoption:

Moderate gas fees will attract developers to build and deploy applications on the Taiko blockchain. Developers are more likely to choose platforms where the cost of deploying and interacting with smart contracts is reasonable, fostering a growing ecosystem.

Community Growth:

A user-friendly ecosystem with moderate gas fees will  attract and a larger community. A growing community leads to more active participation, word-of-mouth marketing, and a positive feedback loop for the Taiko project growth.

Sustainable Growth:

Sustainable growth is achievable when gas fees are moderate. The Taiko Project can scale more effectively as the user and developer base expands without being hindered by high transaction costs.

Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

Reported for spam


[GHSA-76p3-8jx3-jpfq] Prototype pollution in webpack loader-utils

Updates

Affected products

Comments
The prototype pollution vulnerability in webpack loader-utils (CVE-2022-37601) affects versions prior to 1.4.1 and 2.0.3¹². This vulnerability occurs in the parseQuery function within parseQuery.js, where an attacker can manipulate the name variable to modify the prototype of objects¹².
To address this issue, you can:

Upgrade loader-utils to version 3.2.1 or later.
Update your package-lock.json file to ensure all dependencies are using the patched version.

Here's an example of how to update your dependencies:
"dependencies": {
  "loader-utils": ">=3.2.1"
},
"devDependencies": {
  "loader-utils": ">=3.2.1"
}

By upgrading to the patched version, you can mitigate the security risks associated with this vulnerability.
Is there anything else you'd like to know or need help with regarding this update?
¹: GitHub Advisory Database
²: Snyk Vulnerability Database
Source : conversation avec Copilot, 23/09/2024
(1) Prototype pollution in webpack loader-utils · CVE-2022-37601 - GitHub. https://github.com/advisories/GHSA-76p3-8jx3-jpfq.
(2) Prototype Pollution in loader-utils | CVE-2022-37601 | Snyk. https://security.snyk.io/vuln/SNYK-JS-LOADERUTILS-3043105.
(3) CVE-2022-37601: Prototype Pollution in webpack loader-utils - Vulert. https://vulert.com/vuln-db/CVE-2022-37601.
(4) undefined. https://github.com/webpack/loader-utils/releases/tag/v2.0.3.
(5) undefined. https://github.com/webpack/loader-utils/releases/tag/v1.4.1.
(6) undefined. https://dl.acm.org/doi/abs/10.1145/3488932.3497769.
(7) undefined. https://dl.acm.org/doi/pdf/10.1145/3488932.3497769.
(8) undefined. https://lists.debian.org/debian-lts-announce/2022/12/msg00044.html.
(9) undefined. http://users.encs.concordia.ca/~mmannan/publications/JS-vulnerability-aisaccs2022.pdf.
(10) NVD - CVE-2022-37601. https://nvd.nist.gov/vuln/detail/CVE-2022-37601.
(11) CVE-2022-37601 - OpenCVE. https://www.opencve.io/cve/CVE-2022-37601.
(12) Security Bulletin: Decision Optimization in IBM Cloud Pak for Data is .... https://www.ibm.com/support/pages/security-bulletin-decision-optimization-ibm-cloud-pak-data-vulnerable-webpack-loader-utils-vulnerability-cve-2022-37601.
(13) CVE-2022-37601: Improperly Controlled Modification of Object ... - GitLab. https://advisories.gitlab.com/pkg/npm/loader-utils/CVE-2022-37601/.
(14) undefined. https://github.com/webpack/loader-utils/blob/d9f4e23cf411d8556f8bac2d3bf05a6e0103b568/lib/parseQuery.js.
(15) undefined. https://github.com/webpack/loader-utils/issues/212.
(16) undefined. https://github.com/xmldom/xmldom/issues/436.

Hi @aka2024, we are closing this request because is does not suggest a change to the advisory.


Nothing happens when program is run

One day Ace just stopped working. I have MacOS Big Sur installed, and Ace runs and never does anything.

I removed the previous version and installed the newest version and the behavior did not change. There are no error message, so I cannot report any additional detail about what's wrong without a dev telling me what logs to provide.

Hello, would you mind sharing an EPUB with which you are able to reproduce this bug?
I have both Catalina and BigSur MacOS computers, so I will be able to troubleshoot this.
Thank you!
Regards, Daniel

Sure, here's one I just tried.
-Lora
On May 21, 2021, Marisa DeMeglio @.***> wrote:

Hello, would you mind sharing an EPUB with which you are able to
reproduce this bug?
 I have both Catalina and BigSur MacOS computers, so I will be able to
troubleshoot this.
 Thank you!
 Regards, Daniel

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/daisy/ace-gui/issues/137#issuecomment-846064947, or
unsubscribe <https://github.com/notifications/unsubscribe-
auth/ACCWTDBY65UELPHI3BNYI43TOZ77XANCNFSM45HZU67A>.

I am experiencing the same issue. I just switched to a new MacBook Pro 2022 running Monterey 12.6.1, and can no longer validate epubs using ACE by Daisy. I tried running several epubs that had successfully run on my previous MacBook Pro, but they just get stuck running, and never produce a result. I deleted and reinstalled the latest version of ACE by Daisy (1.1.5), but get the same result. I think this is possibly related to Intel vs Apple Silicon?

@Magess and @kvtomes are you able to reproduce this behaviour with Ace App v1.3.2 ?
https://github.com/daisy/ace-gui/releases/tag/v1.3.2

Hi Daniel,
Thanks for asking. Unfortunately, v1.3.2 still won't process any epubs when
running on my Mac. It just keeps spinning indefinitely. I've attached a
couple screenshots in case it helps.
Let me know if you need any other info,
Kris
On Sun, Jul 30, 2023 at 6:54 AM Daniel Weck @.***>
wrote:

@Magess https://github.com/Magess and @kvtomes
https://github.com/kvtomes are you able to reproduce this behaviour
with Ace App v1.3.2 ?
https://github.com/daisy/ace-gui/releases/tag/v1.3.2
—
Reply to this email directly, view it on GitHub
https://github.com/daisy/ace-gui/issues/137#issuecomment-1657119711, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AXNNJTWLVCFKPDPLF7LWG4LXSZDOJANCNFSM45HZU67A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hello, I am afraid I am not able to reproduce the problem you are experiencing. I downloaded Ace App (GUI) 1.3.2 on my MacBook Air M1 (original / early Apple Silicon ARM64 hardware) and the Rosetta emulation of X86/64 instructions from the Ace executable is working fine. I tested with the EPUB linked above in this discussion thread. I am using the latest version of Mac OS Ventura 13.5 (22G74). What version are you running? What hardware are you using? Thank you


add tags

Description
First Time Contributor
I want to add tags to IAM user.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.
Not Tested.
I am trying to figure out how to create docker images for local development testing

/test

/build

@tsusadivyago Is this PR still active? Thanks!

Closing for inactivity


Unexecuted Query Warning

It seems like a common issue could be to create certain types of queries and expect that they'd get executed automatically. For instance:
db.insert('users', { username: 'wbyoung' })

While it may appear that this actually performs the insert, it has not been executed. It either needs an execute or a then on the end of it to actually trigger execution. It would be nice to warn users that they potentially have a small error in their code.

This could be extremely difficult to implement. We can't simply say that all
queries should be executed in the same iteration of the run loop in which they
were created. That completely defeats the point of query objects. Query objects
will frequently be created with the intention of caching them or using them
later to build new queries.
So we'd need to keep track of active queries. That is, queries to which there
are still active pointers. This would be really easy to do if there were some
sort of destructor for JavaScript.
Limitations in JavaScript may make this nearly impossible to implement.

In addition, the immutable aspect of queries means that there are a good number of discarded queries that get generated. For instance:
var query = db.select('users').where({ id: 5 }).limit(5);

The select is created & then discarded when the where is generated. That is then discarded when the limit is generated.
So for what looks like a single query, at least 2 additional queries were generated and discarded. Additionally, internally, many duplicate queries may have been generated while creating each portion of this query.

I still like this idea, and have had it in a personal TODO document for a while now, but am going to close this for the time being. It's currently impossible to implement. If additional features are added to the JavaScript language or new ideas present themselves, I think this would be a great feature.

This has been addressed by the Azul.js Tracker and actually marking queries that are not intended to be used immediately by pinning them.


Object template matching

Closes #6

Coverage remained the same at 100.0% when pulling b81c8f180cbe2c0155cd53f64d845f9b8b068fa2 on 6-ObjectTemplateMatching into c9f4058f7687d0154b2a6f2e7f773f32ff92dd86 on ReleaseCandidate-2.0.0.

Coverage remained the same at 100.0% when pulling b81c8f180cbe2c0155cd53f64d845f9b8b068fa2 on 6-ObjectTemplateMatching into c9f4058f7687d0154b2a6f2e7f773f32ff92dd86 on ReleaseCandidate-2.0.0.


[5.4] [WIP] Namespacing tests

Started with Auth tests, I basically moved the test classes under namespaces that match the structure of the core classes they test.

Curious why namespace the framework tests?

Not sure how useful this is tbh. Will only cause merge conflicts.

This is an experiment to see if it's nicer.

Love this :+1:
Shouldn't we have separate folders for unit/integration tests?

@JosephSilber yes it's part of the plan if it works, trying to tidy up tests a bit so that we have some sort of guidelines on where to put tests related to specific part, etc...

This is an experiment to see if it's nicer.

I do like namespaced tests, but at what cost?

Closing this for now so I can get it out of list until we are ready to revisit this for real. Will re-open then. Thanks


64bit Colour

This is an improvement request.
The FabGL VGA32 hardware "standard" is capable of 6bit colour.  Is it possible to enable this in Eris?

Yes. It's one of those things that is relatively easy to do but I didn't want to force having 6 bit colour on any platform. It's a quasi-palette system (the palette has to be set up before drawing) but just at present uses RGB.  It's a write to $FF10 to set the palette, and the lower byte is 00000bgr, so it would be easy enough to extend this.
The system can theoretically have 256 different colours, but there's only enough RAM in an ESP32 for 4 bit planes, which can be divided into 2 planes, though they don't really exist physically, it's done by colour mapping to work as a background/foreground system for sprite graphics.


Empty amount in some transactions - Send to Self

This happen in the list of transactions.

This is the normal behavior... closing...


CTAs should be dismissable when other CTAs are also visible

Task/Issue URL: https://app.asana.com/0/1157893581871903/1207661039495530/f
Description
HomePanel CTA is not dismissed in RMF is present.
Steps to test this PR
See https://app.asana.com/0/1157893581871903/1207661039495530/f

#4694  👈
develop

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

Join @malmstein and the rest of your teammates on  Graphite

@cmonfortep We shouldn’t rely on currentCtaViewState to see if there’s a visible CTA or not. This is a mutable class that we override. The correct way of doing it is to pass the cta from the click listener itself. Same thing happens with RMF.


Delete dashboard-oauth-config secret when upgrading from 1.18

Description
Fix RHODS-5910 and RHODS-5836
How Has This Been Tested?
Install 1.18.0-9 and check dashboard-oauth-config secret exists:
$ oc get secret dashboard-oauth-config -n redhat-ods-applications
NAME                     TYPE     DATA   AGE
dashboard-oauth-config   Opaque   1      3m45s

Upgrade to 1.19:
$ oc patch catalogsource addon-managed-odh-catalog \
  --type=merge -p '{"spec":{"image":"quay.io/modh/rhods-operator-live-catalog:1.19.0-rhods-5910"}}' -n openshift-marketplace

Wait until the OAuth objects are created and the dashboard is running:
$ oc get secret dashboard-oauth-config -n redhat-ods-applications
NAME                     TYPE     DATA   AGE
dashboard-oauth-config   Opaque   0      78s

$ oc get secret dashboard-oauth-config-generated -n redhat-ods-applications
NAME                               TYPE     DATA   AGE
dashboard-oauth-config-generated   Opaque   1      84s

$ oc get oauthclient dashboard-oauth-client
NAME                     SECRET                             WWW-CHALLENGE   TOKEN-MAX-AGE
dashboard-oauth-client   FF3kKVNlDauyJlZhb52Q77LtuuhlkWBd   false           default

$ oc get pods -l app=rhods-dashboard -n redhat-ods-applications
NAME                               READY   STATUS    RESTARTS   AGE
rhods-dashboard-869489dfcf-76s2g   2/2     Running   0          3m11s
rhods-dashboard-869489dfcf-dl7b5   2/2     Running   0          88s
rhods-dashboard-869489dfcf-mw74h   2/2     Running   0          3m11s
rhods-dashboard-869489dfcf-ngfps   2/2     Running   0          88s
rhods-dashboard-869489dfcf-xl7px   2/2     Running   0          3m11s

/lgtm


Update urls in password-generator.el header

The urls in the password-generator.el header should probably be updated:
https://github.com/vandrlexay/emacs-password-genarator/blob/d754391d11d1d384833eb82fd34e02d2baec36cb/password-generator.el#L6
https://github.com/vandrlexay/emacs-password-genarator/blob/d754391d11d1d384833eb82fd34e02d2baec36cb/password-generator.el#L22
They do redirect to this repository.
But it would be better to point them to the current source.

Fixed. Thank you.


Reset user password

Description
The user password reset process is as follows:

User clicks on “forgot password” on login page
Redirect them to recover password page to enter emails
Backend sends an email for resetting password
User clicks on the included link and fill out a form with new password

Solution

Add forgot password page and reset password page
Enhance mailer to send password reset emails

Testing

[x] Integration testing on Heroku testing environment
[x] Added new unit tests

Checks

[x] Have you removed commented code?
[x] Have you used gofmt to format your code?

Very nice changes. Great work! I was testing it in the testing app, everything works such as reset link to email etc, except my new password didn't get updated in the system. Is it because of the environment?

Just tried in the testing app. The password works as expected now.


docs: Add use examples to the README

* Add use examples on running the image as a Docker container and using mg5_aMC
* Use apt-get clean over apt-get autoclean to remove more unused files

@msneubauer I've merged these instructions in. If you have additional questions just open up an Issue and tag me.


Running LVGL on a separate esp32S3 core

I am creating a UI using squareline(lvgl)
ZX7D00CE01S-8048 has an esp32 s3 with 2 cores
I am relatively new to esp-idf and my main code might get extensive
I am running Wifi,MQTT, an ADC , PWM full gpio
And other tasks
I saw somewhere that it'd be better to run the UI on a separate core as lvgl task handle is sensitive
Since QMSD_ESP32_BSP has lvgl driver embedded
I am not sure how to run the UI on a separate core using RTOS
Please advise

LVGL runs in a separate thread. For more details, refer to qmsd_gui_lvgl_v8.c.

If you need to interact with other peripherals through the UI, there are a couple of ways to do this:

(1) Execute LVGL-related code only within LVGL’s callback functions. In these callbacks, query the peripheral's status and update the display accordingly. The callbacks can be tied to lv_timer timers. Be sure to ensure that the peripheral’s APIs are thread-safe.
(2) If you need to call LVGL-related code from other threads, you must surround the code with qmsd_gui_lock and qmsd_gui_unlock to ensure thread safety for LVGL operations.


Elm 0.19 support?

It appears that the code is not Elm 0.19 compatible.

The code contains an import of Signal - I think this is a thing of Elm's past?
An elm.json file is missing, imports have to be added manually
elm-make in the makefile is now elm make (assuming this is using the default elm compiler)

This issue was intended for the example repo, it may be transferred

servant-to-elm is not the same as servant-elm.
servant-to-elm is compatible with Elm 0.19.


Add the body object to the ClassMapper.fromClass method

I have written a custom InvocableHandlerMethod. Basically, it wraps the response of an RPC call in a in another object. With this, I can catch specific exceptions. So, the wrapper class contains ether the result or an exception. Nevertheless, the wrapper class is in my case a generic class. By the way I use the Jackson2JsonMessageConverter for json serialisation. Unfortunately, the header of the Rabbitmq-Message is set to the class of the response type of the rabbit-listener-method (Somewhere on the HandlerAdapter.invoke(…)). Since I wrap the response to another object, the header is not set correctly. Fortunately, I can give the Jackson2JsonMessageConverter a class ClassMapper so I can change the TypeId-Header.
Unfortunately the ClassMapper  only get the class of the body-object. Since my wrapper-class is generic I cannot determine the typeparamater.
Here is my question: Is it possible to change the ClassMapper-Interface, in a way, that the fromClass-Mathod also get the body-object and not only its class?

Thank you for your response. I am creating a small example so you can see what I am trying to do.
During I put all things together I noticed that this change will not be enough (sorry for that).
I also need the ClassMapper.toClass(...) method to return a JavaType or a ParameterizedTypeReference to express a generic type.
So, in the fromClass Method I will write everything I need for the generic Type in the header and in the toClass Method a want to extract all this information and construct this typ.

Finally, I created an example (and a pull-request).
See here https://github.com/codemaker219/classmapper-example/tree/master

I don't see a reason in all of this since we have a more advance Jackson2JavaTypeMapper strategy which is an extension of the mentioned ClassMapper with all the expected logic.
Closing as Won't Fix


Password sync not working

Describe the Bug
I have a very simple dockerized Directus (latest) running with postgres (based on directus official image). Two roles (admin & editor).
I only export/import not admin users but when trying to connect with an imported user : credentials are wrong, I guess something in the encrypt/decrypt is not working as excepted.
I'm almost using only default config :
// config.json
  directus_users: {
    watch: ["users"],
    query: {
      filter: {
        role: {
          admin_access: { _neq: true },
        },
      },
    },
    onExport: async (item, itemsSrv) => {
      if (item.password && item.password === "**********") {
        const user = await itemsSrv.knex
          .select("password")
          .from("directus_users")
          .where("id", item.id)
          .first();
        if (user) {
          item.password = user.password;
        }
      }

      return item;
    },
    // And then to import the password
    onImport: async (item, itemsSrv) => {
      if (item.password && item.password.startsWith("$argon")) {
        await itemsSrv
          .knex("directus_users")
          .update("password", item.password)
          .where("id", item.id);
        item.password = "**********";
      }

      return item;
    },
  },

// directus_config.js
 directus_roles: {
    watch: ["roles"],
    excludeFields: ["users", "has_role"],
    query: {
      filter: {
        name: { _neq: "Administrator" },
      },
    },
  },

  directus_permissions: {
    watch: ["permissions", "collections", "fields"],
    excludeFields: ["id"],
    groupBy: ["collection"],
    getKey: (o) => `${o.role ?? "public"}-${o.collection}--${o.action}`,
    query: {
      sort: ["role", "collection", "action"],
    },
  },

To Reproduce
Using simple dockerized Directus
# syntax=docker/dockerfile:1.4

####################################################################################################
## Directus install extensions

FROM directus/directus:10.13.1

USER root

RUN corepack enable \
    && corepack prepare<EMAIL_ADDRESS>--activate \
    # Currently required, we'll probably address this in the base image in future release
    && chown node:node /directus

EXPOSE ${PORT}

USER node
RUN pnpm install directus-extension-schema-sync directus-extension-models directus-extension-computed-interface && pnpm config set auto-install-peers true
CMD : \
    && node /directus/cli.js bootstrap \
    && node /directus/cli.js start \
    ;

Import user  (unhashed password is 'password' :
// directus_users.json
[
  {
    "appearance": null,
    "auth_data": null,
    "avatar": null,
    "description": null,
    "email"<EMAIL_ADDRESS>    "email_notifications": true,
    "external_identifier": null,
    "first_name": "Editor",
    "id": "8aba891c-27c3-40a7-b1c2-96228e5ca895",
    "language": null,
    "last_access": null,
    "last_name": null,
    "last_page": null,
    "location": null,
    "password": "$argon2id$v=19$m=65536,t=3,p=4$0baQSe7fW+jFrkJtRU4ekg$pj/wJ+roE+amuseLB6lz9t9Op59YFx3VFOaPcN6ez8U",
    "provider": "default",
    "role": "934eb29a-056a-4480-9c19-4bacce7a910c",
    "status": "active",
    "tags": null,
    "tfa_secret": null,
    "theme_dark": null,
    "theme_dark_overrides": null,
    "theme_light": null,
    "theme_light_overrides": null,
    "title": null,
    "token": null
  }
]

Returns :

Version
2.1.2
Installed Extension Via
Docker file

New (seems unrelated) error occurs on any export :
Error in condensed action: error: select * from "directus_settings" where "id" = $1 and "mv_locked" = $2 and not "mv_hash" = $3 and "mv_ts" < $4 or "mv_ts" is null for update - column "mv_locked" does not exist

Have you installed the extension as per the instructions, seems like you missed the migrations part.

I did but just realized the npx directus schema-sync install --force was needed for every new directus project, thought I only had to run it once and then reuse the schema-sync folder in your directus projects.
Indeed this is fixing the table column not working issue thank you very much but I'm still absolutly unable to transfer user and still getting the wrong credentials error...

After a lot of tries and research I'm definitly stuck with users not being able to be sync, more precisely I'm still unable to connect and new directus instance with imported users as shown above and following the doc.

up ?

What seems to be the issue, is the password incorrect, or does the user not even exist at all?


MCEdit <IP_ADDRESS> crashes upon copying blocks within selection

Just reset my Windows 10 OS with an option to retain data. However, whenever I try to copy within selection, it returns as follows:
Running in portable mode. MCEdit-schematics and mcedit.ini are stored alongside MCEditData
Minecraft_Server.jar storage initialized.
Each server is stored in a subdirectory of D:\Matthew\MCEdit-<IP_ADDRESS> (64-bit)\ServerJarStorage named with the server's version number
Cached servers:  [u'Unknown (45)', u'Unknown (44)', u'Unknown (43)', u'Unknown (42)', u'Unknown (41)', u'Unknown (40)', u'Unknown (39)', u'Unknown (38)', u'Unknown (37)', u'Unknown (36)', u'Unknown (35)', u'Unknown (34)', u'Unknown (33)', u'Unknown (32)', u'Unknown (31)', u'Unknown (30)', u'Unknown (29)', u'Unknown (28)', u'Unknown (27)', u'Unknown (26)', u'Unknown (25)', u'Unknown (24)', u'Unknown (23)', u'Unknown (22)', u'Unknown (21)', u'Unknown (20)', u'Unknown (19)', u'Unknown (18)', u'Unknown (17)', u'Unknown (16)', u'Unknown (15)', u'Unknown (14)', u'Unknown (13)', u'Unknown (12)', u'Unknown (11)', u'Unknown (10)', u'Unknown (9)', u'Unknown (8)', u'Unknown (7)', u'Unknown (6)', u'Unknown (5)', u'Unknown (4)', u'Unknown (3)', u'Unknown (2)', u'Unknown (1)', u'Unknown']
[   ERROR][                   root.py:912]:Error while checking for updates
Traceback (most recent call last):
File "mcedit.py", line 908, in main
File "esky_init_.pyo", line 787, in find_update
File "esky\finder.pyo", line 184, in find_versions
File "esky\finder.pyo", line 164, in open_url
File "urllib2.pyo", line 126, in urlopen
File "urllib2.pyo", line 400, in open
File "urllib2.pyo", line 418, in _open
File "urllib2.pyo", line 378, in _call_chain
File "urllib2.pyo", line 1215, in https_open
File "urllib2.pyo", line 1177, in do_open
URLError: <urlopen error [Errno 1] _ssl.c:504: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure>
Clipping:  _Vector(x=1, y=1, z=1)
[   ERROR][                  root.py:1059]:MCEdit version <IP_ADDRESS>
[   ERROR][                  root.py:1062]:exception: access violation writing 0xFFFFFFFF86A2CAF0
Traceback (most recent call last):
File "mcedit.py", line 1057, in main
File "mcedit.py", line 978, in main
File "albow\root.pyo", line 106, in run
File "albow\root.pyo", line 159, in run_modal
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 730, in gl_draw_all
File "albow\widget.pyo", line 727, in gl_draw_all
File "albow\openglwidgets.pyo", line 23, in gl_draw_self
File "albow\openglwidgets.pyo", line 29, in gl_draw_viewport
File "editortools\thumbview.pyo", line 42, in gl_draw
File "glutils.pyo", line 206, in init
File "OpenGL\latebind.pyo", line 41, in call
File "OpenGL\latebind.pyo", line 45, in call
File "OpenGL\wrapper.pyo", line 574, in wrapperCall
WindowsError: exception: access violation writing 0xFFFFFFFF86A2CAF0
Press RETURN or close this window to dismiss.
I know it is older, but I am far well-versed with that version despite its age. Probably must be something with the GeForce control panels (I make it run GPU for graphics, and customized the FXAA, MFAA, Texture (Trilinear Filtering), as well as the triple buffering where applicable). Any feedbacks are appreciated.

Note that it is in the second hard drive, not the main drive, as it is a 256 GB NVME SSD.

Managed to get the other copy of the <IP_ADDRESS>, and then unzip it and it is now working. Turns out it has nothing to do with GeForce control center presets, as the same as a preceding one became kinda broken after I reset my Windows 10 while retaining the ability to keep documents and the like.


how to estimate the geo-registered information for the custom dataset?

Thanks for the useful work!
I have a question about creating a custom LLFF-like dataset. Based on my understanding, we need to run colmap on the images that we have.  Then, we need to use the VIO algorithm to predict the geo-registered information and group them in a text file (filename, X, Y, Z).
Can you please highlight how I can obtain the geo-registered information using VIO?

I can provide some guidance for this, but I have a couple questions to clarify that you are looking for:
When you say LLFF-like dataset, do you mean single shot localization tests like we did when experimenting on the LLFF dataset or do you intend to run a sequence of images where you want some odometry source to be used for the predict step (like we did for the robot experiment). If you want the first option, then this is easier because all we need to do it input ground truth scale to get metric scaled poses from colmap. If you want the second option then we need to make sure the VIO frame and get's aligned to the colmap frame which takes a couple extra steps.

Thanks a million for replying to my issue. The data was collected with a mobile phone, Huawei Mate 20 Pro, using a custom application implemented in Android Studio to record videos and sequences of inertial data from its internal IMU. Each video consists of a sequence of video frames obtained with a variable sampling rate of around 2 Hz and IMU data recorded at a fixed sampling rate of 100 Hz. This is what the data looks like:
https://ubarcelona-my.sharepoint.com/:u:/g/personal/ahmad_almughrabi_ub_edu/ESvdEvfVWn1OttGNE3F2bckBUKYaxdpU8YeseoyLlqFS3A?e=JrZeIc
I also tried to get a small scene from them to make tracking easier. The images I have are listed in cam0, while I have a set of IMU data (Accelerometer and Gyroscope) captured from Andriod in imu0. I also created a bag file using vio-common https://ubarcelona-my.sharepoint.com/:u:/g/personal/ahmad_almughrabi_ub_edu/EU2sYpJBKBhPmnWXkPkcexoBlIvPjeGHoP5zDfNlOYJUgw?e=S5qP8c


generate new project error.

Error message for my nodejs, I think you left some modules in package.json file
Errror message:
Loading "Gruntfile.js" tasks...ERROR

Error: Cannot find module 'utils'
Warning: Task "webdriver:update" not found. Use --force to continue.

Aborted due to warnings.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs
\node_modules\npm\bin\npm-cli.js" "run-script" "setup-protractor"
npm ERR! node v0.12.4
npm ERR! npm  v2.10.1
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>setup-protractor: grunt webdriver:update
npm ERR! Exit status 3
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>setup-protractor script 'grunt we
bdriver:update'.
npm ERR! This is most likely a problem with the angular-seed package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     grunt webdriver:update
npm ERR! You can get their info via:
npm ERR!     npm owner ls angular-seed
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\git\myapp\frontend\npm-debug.log

See https://github.com/CleverStack/cleverstack-cli/issues/76#issuecomment-121777570


Track not playable (no streams found)

I run into to following error today when playing a song:
May 18 14:55:24 ubuntu mopidy[123169]: ERROR    [YTMusicBackend-3] mopidy_ytmusic No streams found for 8xsL8TKxAao. Falling back to youtube-dl.
May 18 14:55:24 ubuntu mopidy[123169]: WARNING  [Core-15] mopidy.core.tracklist Track is not playable: ytmusic:track:8xsL8TKxAao

Yesterday everything worked fine. I am authenticated and able to view artists etc.
Any ideas?

If I directly link to it: https://www.youtube.com/watch?v=8xsL8TKxAao, I get "No Video Available" and I can't play the track via https://music.youtube.com/watch?v=8xsL8TKxAao, either.  I think the issue is with this particular track.

Thanks for your fast reply.
Both of your suggested direkt links work for me. But it is a general problem as any track won't play. Might this again be an issue with Iris? Weird thing is yesterday everything was fine and I haven't changed my setup. I also tried to reauth.

I tried a fresh mopidy + YTMusic + Iris installation within a docker container:
It worked to play any track before setting up the authentication. After mopidyctl ytmusic setup creating the auth.json and accessing my own premium account I encountered the same error. Feels like google is blocking something but I have no idea how to further debug this...

Sorry, didn't mean to disappear on you like that, got caught up with work/family/life.
It looks like Google has made some more changes and this is likely the issue.  ytmusicapi just released v0.17.1 a few minutes ago (which has some breaking changes and really should have been v0.18,0, but whatever) and that release seems to be related to Google's changes.  I'm going to (sometime soon) upgrade the package to use the new version of ytmusicapi and then spend time fixing what that breaks.  Once I get everything working I'll push the changes and hopefully this will solve your issues.

No worries, I know exactly what you mean :)
I tried do play around with your code but did not manage to run/test in a pip virtualenv. How do you develop/improve this code?
Anyways, thanks for helping. Let me know if I can assist you. Happy to contribute.

Tried your potential fix. It brings me further in the playback.py. The stream is now found, but the decoded URL is forbidden:
May 22 05:26:44 ubuntu mopidy[142495]: INFO     [YTMusicBackend-3] mopidy_ytmusic YTMusic Found AUDIO_QUALITY_HIGH stream with 259394 bitrate for zBqW6yKz8WA
May 22 05:26:44 ubuntu mopidy[142495]: ERROR    [YTMusicBackend-3] mopidy_ytmusic YTMusic found forbidden URL. Updating player URL now.
May 22 05:26:44 ubuntu mopidy[142495]: WARNING  [Core-15] mopidy.core.tracklist Track is not playable: ytmusic:track:zBqW6yKz8WA

Yes, some big changes were made to youtube's backend.  ytmusicapi had to do another release, and I'm updating everthing to use that now.
Should be done soon.

OK, just released 0.2.4 which should hopefully solve this.

Thank you @impliedchaos for your fast commit. It is fixed. You can close the issue.

Im still facing same issue


Update UG - General information

Details

Application details
User Setup
Feature list
FAQ

Completed the issued
Information found here: #18


I cant finetune with this issue

ValueError: You cannot perform fine-tuning on purely quantized models. Please attach trainable adapters on top of the quantized model to correctly perform fine-tuning. Please see: https://huggingface.co/docs/transformers/peft for more details

@HailamAICT I think maybe you forgot to call FastLanguage.get_peft_model :)


qsub: Unknown resource: AMD

Describe the bug
When running with the PBS scripts, I get qsub: Unknown resource: AMD errors.
Expected behavior
I don't think it's necessary to add the AMD specifier to the resource list since other clusters may use different processors, or may not have that resource group listed.
Screenshots
If applicable, add screenshots to help explain your problem.

Hi - please feel free to remove and see if it works. We do not have the
ability to test the PBS version.
On Mon, Mar 23, 2020 at 5:44 PM Aakash Sur<EMAIL_ADDRESS>wrote:

Describe the bug
When running with the PBS scripts, I get qsub: Unknown resource: AMD
errors.
Expected behavior
I don't think it's necessary to add the AMD specifier to the resource list
since other clusters may use different processors, or may not have that
resource group listed.
Screenshots
If applicable, add screenshots to help explain your problem.
[image: image]
https://user-images.githubusercontent.com/7614271/77365900-1f4af100-6d14-11ea-9147-d6858ea3ef84.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/162, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EWYB26ZH6MC3VZ3VGPLRI7J5FANCNFSM4LSGE65Q
.

--
Neva Cherniavsky Durand, Ph.D.
Pronouns: she, her, hers
Assistant Professor, Aiden Lab
www.aidenlab.org


Fix Browsersync watch

Fixes an issue where the default Browsersync instance wasn't watching the dest directory for updates.
Updated files now get injected into the browser or will cause the browser to do a full-page refresh, without needing to edit the config file.

Looks like a fairly simple change. If you're happy to merge this, let's do it! 😄


Conda-forge omniscidb-cpu building for v4.8 failed

Currently the building for v4.8 on conda-forge is failing (https://github.com/conda-forge/omniscidb-cpu-feedstock/pull/11):
In file included from /home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/work/SQLFrontend/omnisql.cpp:64:
/home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/work/SQLFrontend/CommandHistoryFile.h:43:14: error: no member named 'secure_getenv' in the global namespace; did you mean '__secure_getenv'?
    return ::secure_getenv(env_var_name);
           ~~^~~~~~~~~~~~~
             __secure_getenv
/home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeho/bin/../lib/gcc/x86_64-conda_cos6-linux-gnu/7.3.0/../../../../x86_64-conda_cos6-linux-gnu/sysroot/usr/include/stdlib.h:586:14: note: '__secure_getenv' declared here
extern char *__secure_getenv (__const char *__name)
             ^
In file included from /home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/work/SQLFrontend/omnisql.cpp:64:
/home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/work/SQLFrontend/CommandHistoryFile.h:77:11: error: variable 'home_dir' with type 'auto *' has incompatible initializer of type 'auto'
    auto* home_dir = resolveHomeDirectory();
          ^          ~~~~~~~~~~~~~~~~~~~~~~
/home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/work/SQLFrontend/CommandHistoryFile.h:51:53: note: in instantiation of member function 'CommandHistoryFileImpl<DefaultEnvResolver>::resolveCommandFile' requested here
  CommandHistoryFileImpl() : CommandHistoryFileImpl(resolveCommandFile()) {}
                                                    ^
/home/conda/feedstock_root/build_artifacts/omniscidb-cpu_1565570851787/work/SQLFrontend/omnisql.cpp:1078:22: note: in instantiation of member function 'CommandHistoryFileImpl<DefaultEnvResolver>::CommandHistoryFileImpl' requested here
  CommandHistoryFile cmd_file;
                     ^
2 errors generated.
make[2]: *** [SQLFrontend/CMakeFiles/omnisql.dir/build.make:63: SQLFrontend/CMakeFiles/omnisql.dir/omnisql.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1776: SQLFrontend/CMakeFiles/omnisql.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

@pearu had the same problem and fixed that with this patch:
diff --git a/SQLFrontend/CommandHistoryFile.h b/SQLFrontend/CommandHistoryFile.h
index ada604336..68ef07ad6 100644
--- a/SQLFrontend/CommandHistoryFile.h
+++ b/SQLFrontend/CommandHistoryFile.h
@@ -40,7 +40,7 @@ class DefaultEnvResolver {
   auto const* getenv(char const* env_var_name) const { return ::getenv(env_var_name); }
 #else
   auto const* getenv(char const* env_var_name) const {
-    return ::secure_getenv(env_var_name);
+    return ::getenv(env_var_name);
   }
 #endif
 };

@xmnlab Could you try adding #include <cstdlib> to the top of that file (SQLFrontend/CommandHistoryFile.h)?
@ptumati additionally it looks like secure_getenv is a gnu extension, so maybe we should be guarding on that instead of __apple__?

@andrewseidl sure! I can try that :)  thanks!

Adding #include <cstdlib> did not worked.
The following workaround works for omniscdb-cpu v4.8:
export CXXFLAGS="$CXXFLAGS -Dsecure_getenv=getenv"

Note that omniscdb-cpu conda package uses clang as the compiler.


Relax route overlap check, skip for gateway redirect routes

Fixes #779
This was suggested by @slayer in https://github.com/docker/docker/issues/24738#issuecomment-267735948
It seems reasonable to skip these special routes when validating a predefined address pool for the bridge network or for any user-defined network. At the end we already do it for the default route.
This would ease the docker installation process and docker network create for systems which are running a VPN agent.
Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

@aboch Root cause of this issue is checking the conflict with the entries in the routing table. I am not sure why its done this way. Since we assign subnets in the private IP range and rely on NAT for reaching the service from outside we only have to check the subnets of the hosts interfaces.
In fact, the only reason this problem doesn't happen for the default route is because netlink library returns it as Dst: <nil> Src: <nil> Gw: <IP_ADDRESS>. And the reason OpenVPN is installing <IP_ADDRESS>/1 instead of a default route is just a hack/convenience so that users doesn't remove the existing default route.
Unless there is a specific reason we are checking against the routing table I think the fix should be to use the interface subnets for the conflict check.

@sanimej
Yes, that check has always been that way since libnetwork was created.
It already catches the local interfaces, them being directly connected routes.
If your host is connected to a VPN for example, you cannot exclude there is a 172.18.x.x route on the same virtual network.
We cannot assume the default address pools will only be private ranges as we will shortly allow users to configure them.
NAT is not guaranteed to be there for all local scope network, think of MACVLAN driver network.

NAT is not guaranteed to be there for all local scope network, think of MACVLAN driver network.
purpose of macvlan is to plumb to the underlay. So in any deployment of macvlan user has to configure the subnet explicitly.
If your host is connected to a VPN for example, you cannot exclude there is a 172.18.x.x route on the same virtual network.
Its not any different from a default route because you don't know what subnets are reachable through the default route. If any deployment where the private IPs are routable across hosts user has to configure the subnet explicitly. And in cases where private subnets are local to the host (most of the users running into this issue) its sufficient to check only the interfaces on that host.

@sanimej
I see your point, but I am not comfortable in modifying this logic now, after it has been in place for 6+ releases.
Looking again at this PR itself, I agree it is a kind of an hack to embed in the route overlap check a skip for the vpn redirect routes.
Therefore, I'd prefer to close this PR, and leave the current logic unaltered.


Add support for maps in query and path params

Signed-off-by: Maciej Szlosarczyk<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 0414f94765cc9f76e2d586c65fa82d212388f396

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: antonsatin
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

0266a12 Add antonsatin to OWNERS

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

LGTM label has been added.  Git tree hash: d1e2c71a3ad276bd4f7d5a06ab20dbba5cb1c060

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: antonsatin
To complete the pull request process, please ask for approval from maciej-szlosarczyk after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: antonsatin, feliperenan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [feliperenan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[Enhancement]: Add a Zombietest to check parachain activity

Description
Add a zombietest to check that blocks above 10th have candidates' included/backed manifests.
Motivation
Increased coverage.
Additional context
No response
Are you planning to do it yourself in a pull request ?
Yes

Required to check #2131 when it is fixed


Issue while building code

Hi, I am new to hyperledger fabric and while building using go build . I am getting below error
# github.com/hyperledger/fabric/vendor/github.com/tecbot/gorocksdb
exec: "gcc": executable file not found in %PATH%
# github.com/hyperledger/fabric/vendor/github.com/mattn/go-sqlite3
exec: "gcc": executable file not found in %PATH%

What env are you running this on?
If it is ubuntu try installing build-essentials.
Also post go version output.

its windows 7. I guess I need to install some gcc compiler. I will try that and update here.
go version go1.7.4 windows/amd64

After installing tdm-gcc-5.1.0-3.exe and go version 1.6 I am getting below error.
# github.com/hyperledger/fabric/vendor/github.com/tecbot/gorocksdb ..\vendor\github.com\tecbot\gorocksdb\backup.go:4:24: fatal error: rocksdb/c.h: No such file or directory  // #include "rocksdb/c.h"

@dharmjit could you also please confirm that you are on the correct branch of the fabric and have pulled all the latest changes?
$ cd $GOPATH/src/github.com/hyperledger/fabric
$ git status
On branch v0.6

$ git pull https://github.com/hyperledger/fabric.git v0.6
From https://github.com/hyperledger/fabric
 * branch              v0.6       -> FETCH_HEAD
Already up-to-date.

Also, you are only running go build on the learn-chaincode tutorial files, correct?  You are not trying to run the command against the fabric repository?

Ohh I am very sorry. I was running this against fabric repo. Can I also use docker setup for the same? Thanks for helping me out.

No worries!  If you are trying to build the fabric from source, then you'll need to follow instructions from hyperledger for setting up a development environment:
http://hyperledger-fabric.readthedocs.io/en/latest/dev-setup/build.html
For docker, you might want to keep your eye on the IBM Blockchain Dockerhub account:
https://hub.docker.com/r/ibmblockchain/fabric-peer/
There are some published images for fabric v0.6, but none for v1.0 just yet.  There are make commands in the fabric repositories that should build docker images for 1.0, if you need them.  I'd get access to a Linux box before going down this route.  Also, this repository only teaches you how to write chaincode for v0.6, not v1.0.  Here are some examples of v1.0 chaincode directly from the fabric:
https://github.com/hyperledger/fabric/tree/master/examples/chaincode/go


chore: bump deps and go version

Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e


Change date formatting on blog posts

Can we update the date formatting so that it displays:
FR : 8 février 2017
EN : February 8th, 2017

A little bit tricky for ordinal indicator, but it's done ;-)


fix: #100 #102

Fix: #100 #102
resets the code area for each refresh.
Checks if there are any other parameters than the defined, if there are shows the error.
Thanks,
Henil
CC: @certik @Shaikh-Ubaid

Thanks!
@henilp105 I gave you write access. Can you please submit this directly as a branch in the main repository? Our testing infrastructure currently requires that. It will build your PR, share a link, and we can test easily if it works.
It should be possible to setup the testing from forks as well, but we haven't done it yet.

Thanks @certik , I will be closing this and opening a PR from branch on this repo.


(Carbon) crash on startup?

Not an issue, you're missing the TPK file for Carbon. If you built this yourself, you need to build the TPK for Carbon too, which I did not release yet.
Carbon is still WIP anyway so I won't accept issues for it until I release it.

it seems the version you have on GitHub is out of date?


Refactor to run in cadCAD 0.4.18

0.4.18 introduced a configuration class which is was a breaking change with respect to this model given how we pull out the configuration objects for inspection. however it necessitates relatively minor changes to the notebook. i'd like to update v1,v2 (including workshop version), and v3 to run in 0.4.18 if possible. Or if its a more involved update, make sure v3 runs in 0.4.18 and that the local director readmes for earlier models specify which version they use (0.4.17)

i tagged in @tylerdmace for support since he's already been looking at the 0.4.17 --> 0.4.18 change. Hoping you will tell us this is easy.

I thought I had all of these running in 0.4.18. I ran them all yesterday. The main difference was changing to
from cadCAD.configuration import Experiment from appended_configs

I'll double check them all today. I thought you had fixed them, too. Using
the new class should be the only change needed I believe, but again, I'll
verify here today.
On Tue, Aug 11, 2020, 8:25 AM Andrew Clark<EMAIL_ADDRESS>wrote:

I thought I had all of these running in 0.4.18. I ran them all yesterday.
The main difference was changing to
from cadCAD.configuration import Experiment from appended_configs
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BlockScience/Aragon_Conviction_Voting/issues/8#issuecomment-671978639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJN4J6YMZ23FVX3UW65X7DSAFIFNANCNFSM4P2W7DPQ
.

I just reran v3 - https://github.com/BlockScience/Aragon_Conviction_Voting/blob/master/models/v3/Aragon_Conviction_Voting_Model.ipynb - with no issues. I wonder if Z's machine isn't updated to 0.4.18, or has two versions, and juptyer imported 0.4.17 instead of 0.4.18

Yeah, running the old version after the changes have been made to update
the notebook for 0.4.18 would cause similar errors, too. That seems likely
to have happened here.
On Tue, Aug 11, 2020, 8:47 AM Andrew Clark<EMAIL_ADDRESS>wrote:

I just reran v3 -
https://github.com/BlockScience/Aragon_Conviction_Voting/blob/master/models/v3/Aragon_Conviction_Voting_Model.ipynb

with no issues. I wonder if Z's machine isn't updated to 0.4.18, or has
two versions, and juptyer imported 0.4.17 instead of 0.4.18

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BlockScience/Aragon_Conviction_Voting/issues/8#issuecomment-671991077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJN4J6VF7IXGXHGBCAJWF3SAFKWRANCNFSM4P2W7DPQ
.

I just reran both v1 and v2 as well in the master, all good. v2_assignment doesn't run because Z had pulled some code out for the workshop. @tylerdmace :+1:


C# Messaging Extension Action:  "Unable to reach app. Please try again."

Sample information
https://github.com/microsoft/BotBuilder-Samples/tree/main/samples/csharp_dotnetcore/51.teams-messaging-extensions-action
Describe the bug
After following the instructions exactly, I receive the following message when trying any of the commands (create card/submit, fetch roster, etc.)

To Reproduce
Steps to reproduce the behavior:

Follow these instructions
Try to submit a Create Card or Fetch Roster and observe the error message

Expected behavior
Successful card creation or roster fetch
Screenshots

Hi @TracyGH !!
This error typically happens with a appID/password mismatch. Can you confirm that they're correct, and can post what HTTP code ngrok is displaying?

Thanks so much for getting back to me, @jwiley84!
I was able to get the FetchRoster and Share Message actions to function correctly.  However, the Create Card code is returning that 'Unable to reach app.  Please try again.' message.
Below is what I'm seeing as far as local variable values from a breakpoint on line 55 of Bots > TeamsMessagingExtensionsActionBot.cs.   Any thoughts?

Also, I think the Create Card actions are returning a 502 in ngrok, while the other actions are 200.

Hm, I tried it again this morning and now it's working fine.  I did have a firmware update last night and had to restart my machine, not sure if that made a difference.
I'll close this out - thanks again for your help @jwiley84!


[core-client-rest] Support getting request body as a stream

This change implements for REST Clients a method to always get the response body as a stream. This change consists of 2 parts

PipelineRequest to accept a responseAsStream option, this will make core-rest-client return the raw stream.
New client builder getClientWithStream in addition to the existing getClient.

The UX we are trying to accomplish is
const client = FooRestClient("https://foo.org");

// Awaiting get() would load the response body in memory
const result = await client.get("/files/{fileName}", "foo.txt").get();
console.log(result.body);

// Awaiting on asNodeStream would give you the raw stream (without making a double request)
const result = await client.get("/files/{fileName}", "foo.txt").get().asNodeStream();
const stream: NodeJS.ReadableStream = result.body;
// Handle the stream as the user needs to. i.e. write chunks to FS

Notes:
Why keep getClient and getClientWithStream?
tl;dr - To optimize for bundle size.
I thought about just maintaining getClient however there are many SDKs that would never need to handle raw streams., these would be paying (in bundle size currency) for the extra stream handling functionality that wouldn't be used.
From a generator standpoint, we can identify these from the swagger and use the smaller getClient.
What about browsers?
This PR doesn't include browser support. The idea is that browser support comes with WebStreams rather than the current blobs we give our users. For this to happen, we need to support Fetch API in core-rest-pipeline https://github.com/Azure/azure-sdk-for-js/issues/18298 is tracking this work.
Once Fetch API is in place, we'll add an additional method client.path("/foo").get().asWebStream() which would give us the WebStream.
Long term plan is to deprecate asNodeStream once Node supports WebStreams and just keep asWebStream, they are currently experimental in v17

Can you quantify the cost of making the streaming functionality built-in? I don't like the additional public surface area and it doesn't seem particularly intuitive that I would need to use a different client. I would also worry about drift between the client shapes so it eventually becomes problematic to refactor from without stream to with stream when I hit a case where I want it.
Also, what happens when we have another feature we want pay-for-play? More suffixes? I feel like we might want to think about a more generalizable solution to pay-for-play functionality.
So I'm thinking: if the cost isn't THAT great, maybe we can just build it in for now?

I did a quick test and the new client shape with asNodeStream only adds 1kb to the bundle size. I think it is worth merging and just having one public surface

API changes have been detected in @azure-rest/core-client. You can review API changes here

I'll get this as a separate PR once FetchAPI HTTP client is supported in core


Marketplace Module

Be sure to mark this class as deprecated https://github.com/nftlabs/nftlabs-sdk-python/blob/master/thirdweb/modules/market.py#L22


Compendium Talents not Expanding on Player Sheet

Hey folks,
Yesterday, I upgraded to FoundryVTT 0.8.9 and StarWarsFFG 1.6 and I noticed that talents added via a specialization tree were not expanding when clicked on. The edit, information, and send to chat were also not working. This issue didn't apply to talents created manually outside of oggdude.
The bug appears to be from the import-helper.js. I was able to work around it by changing findCompendiumEntityById on line 146:
  static async findCompendiumEntityById(type, id) {
    let entity;

    let packs = Array.from(await game.packs.keys());
    for (let i = 0; i < packs.length; i += 1) {
      let packId = packs[i];
      const pack = await game.packs.get(packId);
      if (pack.documentName === type) {
        await pack.getIndex();
        entity = await pack.index.find((e) => e._id === id);
        if (entity) {
          return await pack.getDocument(entity._id);
        }
      }
    }
  }

It looks like at some point entity's switched from having a public id field to a private one (_id). My work around is almost certainly not the correct way to do this lookup, but it was the most minimal change I could make to get it functioning again.
I think the way this is supposed to work is via something like await game.packs.get("world.oggdudetalents").getDocument("5W5A0X5pRmuleyLm") on the pack. However, maybe this approach was used for backwards compatibility? I'm not sure. Incidentally, I think this code will speed up a lot if it's switched to getEntry since that's doing a map lookup O(1) rather than looping through every compendium item O(n).
I don't think this is !955, but if this is a duplicate, feel free to close this one.

If you have code changes that have worked for you to fix the issue, you can submit a pull request.

My above fix is too much of a hack to file an MR with. I'm confident referencing _id is the wrong way to do this. However, I'll work on a proper fix and file an MR with that.

I filed pull request #985 which I tested locally to confirm it works at least in my environment.
Note: I still experienced a bug expanding some talents. When I checked my specializations compendium, I noticed there were duplicate specializations within it. Some of the specializations seem bugged, presumably from a prior buggy import. If you add a break point in the inspector for findCompendiumEntityById when you expand a talent, the id argument is an empty string in this case. If you still encounter this bug after applying the patch in #985, check your specializations compendium to see if you have duplicate specialization trees. Try each on dummy character sheets to see if all of them suffer the same problem.


Load wrapper

#31
A large number of files has been changed in this PR, although for most of these files the changes are minor. In summary, this PR adds load_cube, load_cubelist and enforce_coordinate_ordering functions and associated unit tests and subsequently proliferates the usage of these functions through the code, as appropriate.
A summary of the edits on a file-by-file basis is below:

lib/improver/utilities/load.py   - Add load_cube and load_cubelist function to wrap iris.load_cube.
lib/improver/utilities/cube_manipulation.py   - Add enforce_coordinate_ordering function.
lib/improver/tests/utilities/test_load.py   - Add tests for the load_cube and load_cubelist functions that have been   added.
lib/improver/tests/utilities/test_cube_manipulation.py   - Add tests for enforce_coordinate_ordering function.
lib/improver/tests/spotdata/spotdata/test_read_input.py   - Remove tests for Load plugin.
lib/improver/tests/generate_ancillaries/test_generate_ancillary.py   - Remove tests on find_standard_ancil function.
.../tests/ensemble_calibration/ensemble_calibration_utilities/test_ensemble_calibration_utilities.py   - Remove tests on ensure_dimension_is_the_zeroth_dimension coordinate.
lib/improver/spotdata/read_input.py   - Remove Load plugin to avoid having multiple methods for loading cubes.
lib/improver/spotdata/ancillaries.py   - Replace use of find_standard_ancil with functions from the load module.
lib/improver/generate_ancillaries/generate_ancillary.py   - Remove find_standard_ancil function to avoid having multiple methods for   loading cubes.
lib/improver/ensemble_copula_coupling/ensemble_copula_coupling_utilities.py   - Replace ensure_dimension_is_the_zeroth_dimension function with   enforce_coordinate_ordering.
lib/improver/ensemble_copula_coupling/ensemble_copula_coupling.py   - Use of the ensure_dimension_is_the_zeroth_dimension function has been   replaced by enforce_coordinate_ordering.
lib/improver/ensemble_calibration/ensemble_calibration_utilities.py   - Removed ensure_dimension_is_the_zeroth_dimension function.
lib/improver/ensemble_calibration/ensemble_calibration.py   - Use of the ensure_dimension_is_the_zeroth_dimension function has been   replaced by enforce_coordinate_ordering.
For all CLIs, usage of iris.load or iris.load_cube has been replaced with usage   of the functions from the load module, either load_cube or load_cubelist.

Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

improver-generate-topography-bands-weights is still using find_standard_ancil, which you have removed, hence travis not liking this at present.


Controllers spec

Hi Reviewer 👋
Here are the changes I implemented to the project.

Added RSpec as a gem to my project.
Created a Request spec file for all my controllers.
Added test for each action:

If the response status was correct
If a correct template was rendered.
If the response body includes the correct placeholder text.

@DammyShittu Thanks man!


Publish on RedissonPubSubStore would block on the eventLoop.

It seems to me that a publish on RedissonPubSubStore would block on the eventLoop, waiting for Redis response. Am I missing something here? Is there a work-around available?

 Can you suggest a solution?

I have given it some thought, but could not come up with a solution that would be universally acceptable. Here are a few approaches that come to my mind, each of which has some pros & cons:
Approach 1:
IMO, there is not much that can be done if the underlying pub-sub system becomes unresponsive.
In the current implementation, an exception would be raised, which would get logged by the ExceptionListener. All further processing for the current packet would get aborted, and in all likelihood there would be a difference in state between the current node, and other nodes.
I am unable to think of a recovery mechanism. In such a case, not ideal, but this might be just slightly better. Just log the error & move on:
public void publish(PubSubType type, PubSubMessage msg) {
    msg.setNodeId(this.nodeId);

    RFuture<Long> f = this.redissonPub.getTopic(type.toString()).publishAsync(msg);

    f.addListener(new FutureListener<Long>() {
        @Override
        public void operationComplete(Future<Long> lf) throws Exception {
            if (!lf.isSuccess()) {
                log.error("Error in publish for type {} and message {}. Cause: {}", type, msg, lf.cause());
            }
        }
    });
}

Approach 2:
Return a Future, and let the caller decide on the recovery mechanism. This is using the google guava library. I guess using Netty's Future family would be more appropriate.
public ListenableFuture<Boolean> publish(PubSubType type, PubSubMessage msg) {
    msg.setNodeId(this.nodeId);

    RFuture<Long> f = this.redissonPub.getTopic(type.toString()).publishAsync(msg);

    SettableFuture<Long> returnFuture = SettableFuture.create();

    f.addListener(new FutureListener<Long>() {
        @Override
        public void operationComplete(Future<Long> lf) throws Exception {
            if (lf.isSuccess()) {
                returnFuture.set(Boolean.TRUE);
            } else {
                returnFuture.setException(lf.cause());
            }
        }
    });

    return returnFuture;
}

Approach 3:
Pass a ChannelPromise to publish method. This would ensure that the subsequent processing after a success/failure setting would most likely happen on the event loop. Not too sure, need to check.
public Promise<Boolean> publish(PubSubType type, PubSubMessage msg, Promise<Boolean> p) {
    ....
    return p;
}


unknown should narrow to {} when non-null

Bug Report
🔎 Search Terms
empty type, unknown, top type, unknown not null
🕗 Version & Regression Information
Starting 3.0 (when unknown was introduced) and until at least 4.6.2

This is the behavior in every version I tried, and I reviewed the FAQ for entries about unknown

⏯ Playground Link
Playground link with relevant code
💻 Code
let myUnknown: unknown = null
let myEmpty: {} = myUnknown ?? 5
if (myUnknown != null) {
    let myEmpty2: {} = myUnknown
}

🙁 Actual behavior
Type 'unknown' is not assignable to type '{}'. (2322)

🙂 Expected behavior
I believe narrowing should be happening. unkown is the top type for everything. {} is the top type for everything non-null non-undefined. Therefore, when an unknown is or'ed (|| or ??) with a non-null, or if checked not-null itself (if (myUnknown != null)), then the result should be narrowed to {}.
Extra context: I'm trying to wrap around JSON.parse to return {} when the result is null. I would have expected that:
JSON.parse(arg) ?? {}

Be of the type {}, but TypeScript is insisting that it's actually unknown.

Duplicate of #32347. Used search terms: narrow unknown in:title