text:
Note 8 is unclear

"the kernel included Debian backports" - is there a word missing here? should it say "the kernel included in Debian backports"?
There are multiple kernels in Debian backports, which kernel specifically? Any of them? The "cloud" one?

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

ID: ef7e57c1-a0f5-126a-775c-7e99a607197b
Version Independent ID: 22a9dc7b-18c2-6f73-d224-886757a65a7c
Content: Supported Debian virtual machines on Hyper-V
Content Source: WindowsServerDocs/virtualization/hyper-v/Supported-Debian-virtual-machines-on-Hyper-V.md
Service: na
Product: windows-server
Technology: compute-hyper-v
GitHub Login: @shirgall
Microsoft Alias: kathydav

Would also like clarification on this item. We are struggling with Dynamic Memory using a Windows Server 2019 host and Debian Buster guest. Having tried Linux kernel 4.19.0 (shipped with Buster), as well as backported kernels 5.2.0 (cloud and non-cloud), Dynamic Memory is still not working as intended -- memory continues to be allocated by the hypervisor, but not seen/consumed by the guest.
Some clarification around exactly which kernels/configurations are required to functionally enable Dynamic Memory would be GREATLY appreciated, as this statement is currently too ambiguous to be useful.

Looking at the sentence in note 8, I can see that it is open to misinterpretation:

The latest upstream kernel capabilities are only available by using the kernel included Debian backports.

To reduce the amount of misinterpretation without rewriting note 8 by changing the content, I imagine it can be read like this:

The latest upstream kernel capabilities are only available by using the Debian backports included with the kernel.

Looking at the sentence in note 8, I can see that it is open to misinterpretation:

The latest upstream kernel capabilities are only available by using the kernel included Debian backports.

To reduce the amount of misinterpretation without rewriting note 8 by changing the content, I imagine it can be read like this:

The latest upstream kernel capabilities are only available by using the Debian backports included with the kernel.

While I agree this may read a bit better, it still fails to serve the reader. Additional information pertaining to precisely which backports support these features, along with any supplementary configurations would go a long way. Without this information documented somewhere, there's a lot left to interpretation by the reader.

@shirgall and/or @KBDAzure ; Would you be able to share your insight with us in this matter?
Otherwise, would you be able to refer to someone else who may look into this issue?

Is there any additional information on this available?
I am having the same issue, and the same problem understanding the note. I have got the backports repository enabled, but what do I need to install? A newer kernel? The hyperv-daemons?
Getting dynamic memory working for Debian would be great!

@konstruktoid : Please, may I ask for your take on this issue, to see the problem from a different angle?
I would like the opportunity to create a PR and suggest an improvement to the reported confusion around parts of the document.
(By the way, if you would like to open a pull request yourself to suggest the improvement, feel free.)

I would probably rephrase it as:
The latest upstream kernel capabilities are only available by using the kernels available in the Debian backports repository.
@BenLangers, try installing a newer kernel.
(In stable it's 4.19 I believe, and in Backports it's 5.4.8. Compared to 5.8.13 which is latest official stable kernel.)
What's missing in this document is in which kernel version the wanted functionality was added, latest is a pretty vague term.

True, I also subscribe to the fact that "Latest" is a very poor version description. Almost like "No, I won't tell you."
(Try using "latest" in any GH repository where the main content is updated more than once a week ... ) 😉 :trollface:

There you go. PR #4885 is ready for review. Feel free to comment, suggest or discuss the requested change.

authors: BenLangers btb illfated konstruktoid prenetic

repo: MicrosoftDocs/windowsserverdocs


text:
Is 2003 Forst Functional compatibility correct?

The above article states 2016 is 2003 Forest Functional compatible, however the below link contradicts that.
https://docs.microsoft.com/en-us/windows-server/identity/ad-ds/active-directory-functional-levels

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

ID: e32288e6-1a81-01da-51bf-d1564049a28a
Version Independent ID: 7388c199-ee35-63fa-deda-e3f6399de3ee
Content: Upgrade Domain Controllers to Windows Server 2016
Content Source: WindowsServerDocs/identity/ad-ds/deploy/upgrade-domain-controllers.md
Product: windows-server
Technology: identity-adds
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close

source: gharchive/issue

authors: ComputerHabit IngridAtMicrosoft

license_type: permissive

license_source: github-api


text:
Sizing Recommendations for NPS

wanted to see if you could please share the hardware recommendations for the NPS

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

ID: e7b382ac-a815-8623-3d48-f1a67a16eada
Version Independent ID: f1edfa59-688e-344b-7f84-033405028c9f
Content: Network Policy Server Best Practices
Content Source: WindowsServerDocs/networking/technologies/nps/nps-best-practices.md
Product: windows-server
Technology: networking
GitHub Login: @eross-msft
Microsoft Alias: lizross

+1

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.

authors: Akumarmatr alejandrofo robinharwood


text:
Universal Links uses a different AppDelegate callback method

If you're using this class your app is necessarily already running, and if you also use Universal Links (as is advised) then AppDelegate.OpenUrl is not called, it's actually AppDelegate.ContinueUserActivity that gets called. You have to then call Xamarin.Essentials.Platform.OpenUrl from there otherwise your WebAuthenticator.AuthenticateAsync task will never return.

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

ID: 1231a1e1-3734-9021-2cc7-eaf99478afca
Version Independent ID: 01e5fdbe-0b15-4dbe-f8a0-ed40f8593302
Content: Xamarin.Essentials: Web Authenticator - Xamarin
Content Source: docs/essentials/web-authenticator.md
Product: xamarin
Technology: xamarin-essentials
GitHub Login: @Redth
Microsoft Alias: jodick

@redth are you able to comment on this one.

authors: jamesmontemagno kanderson-wellbeats

repo: MicrosoftDocs/xamarin-docs


text:
MS Edge Chromium Roadmap Suggestion

Microsoft not only deliver top rate C/C++ App Ecosystem so do Web Ecosystem in future

MS Edge Chromium webview
MS Edge Chromium's Electron (neutron/proton/particle )

Microsoft can achieve unbeatable long term success if Web Ecosystem is top class support in future. Deliver an API that closely resemble to Electron, to deliver using Native Webview with Microsoft own sandboxing/Custom permission.
Although ChromeOS consider competitor of Windows..Even can consider another layer of integration on ChromeOS and WSL layer.
Working with Chromium closely, Microsoft should think how to sustain the leadership in Desktop  Market so ChromeOS is in control/never surpass in desktop market.
For Mobile Market, supporting Android? but in a way that Microsoft always as leading in Desktop market? It is more reason, Microsoft team should have more power in Chromium Community.

Thank you for the suggestion. This topic is not related to features discussed here, please use the feedback button to submit such feedback.

authors: atanassov meteorsnows

repo: MicrosoftEdge/MSEdgeExplainers


text:
Export sale data

In preparation for the upcoming sale we need to export the data we have on tickets allocation from camps and art.
Exported files should be separate for camps and for art.
the csv file should contain the following columns:

Name of allocating camp/art project
Email of ticket recipient

(@pandamichal anything else?)

I can create a metabase question which show this data, you can export to CSV from metabase

actually.. we already have this question, thanks to Hila (BI expert)
https://production.midburn.org/metabase/question/1

and we have camps data and art data here -
https://production.midburn.org/metabase/question/10
https://production.midburn.org/metabase/question/9

@OriHoch , number 9 and 10 are how many quotas each community received, right?
Number 1 is who has tickets and which, (correct me if I'm wrong).
Where do we find the allocated quotas per person for the upcoming sale? I don't think we have a question for this yet, but it certainly can be made.

@amotenko name of camp\art and email is enough.
the camps should be downloaded today until 18:00,
Art scheduled for tomorrow 12.3 at 18:00.
Thanks!

@amotenko @pandamichal
full list of camp and art members - https://production.midburn.org/metabase/question/15
need to join with tickets data, it could be done manually in excel, if not I can do it tomorrow

@OriHoch thank you,
but this is not exactly what needed.
If you look at camp_members table, you will see some users with camps associated with the current event id have under addinfo_json {"pre_sale_ticket":"true"}. Those are the guys we need.
If possible with separation to art and camps.
Muchas gracias!

@amotenko https://production.midburn.org/metabase/question/16

the Camp Type column is used to distinguish between art and camps

I added a camp type filter to the question - https://production.midburn.org/metabase/question/16

Danke dir!
I see now the value that metabase has to provide and I think it's a good thing we have, so thank you for that as well.

:+1:

authors: OriHoch amotenko pandamichal

repo: Midburn/spark


text:
[TECH ED] Play Can't Unsee

Link to the coursework
https://cantunsee.space/
Why are we doing this?
To make a professional looking website, you must attend to all the small details. Play this fun game to sharpen your eye for the professional polish that will set your portfolio apart.
Maximum time in hours
.5
How to get help
Play it again!

Kamino cloned this issue to FrenchFry36/Coursework-Planner

Kamino cloned this issue to Ebtesammm/Coursework-Planner

Kamino cloned this issue to kostiantynkovalchuk/Coursework-Planner

Kamino closed and cloned this issue to DelwinMansur/DelwinMansur-Coursework-Planner

repo: Migracode-Barcelona/Module-HTML-CSS


text:
Blank top/title bar

I followed the example code, and I get this:

Am I forgetting something?

Sam, please use this
https://github.com/react-native-simple-router-community/react-native-simple-router/
if
you expect support.
On Tue, Feb 2, 2016 at 12:29 AM Sam notifications@github.com wrote:

Closed #59 https://github.com/MikaelCarpenter/gb-native-router/issues/59
.
—
Reply to this email directly or view it on GitHub
https://github.com/MikaelCarpenter/gb-native-router/issues/59#event-535013695
.

authors: ffxsam sethx

repo: MikaelCarpenter/gb-native-router

license: mit

license_source: bigquery


text:
Issue toNumber() function round up when number is too big (v9.1.1)

I have a issue with toNumber() function.
I try with BigNumber version:
"bignumber.js": "9.1.1",
https://codesandbox.io/s/bignumber-js-forked-sewfsh?file=/src/index.js:0-780
999999999999998400000000000000000000 - 30000000000000000000
Expect Result:  999999999999998370000000000000000000
With toNumber(), same it Rounding Up result:
9.999999999999984e+35
(from 37 -> 40)

This is normal. Your number is way beyond Number.MAX_SAFE_INTEGER.
Use toString instead of toNumber.
Incidentally, JavaScript numbers also exhibit binary floating-point rounding artefacts, which is why we need libraries like bignumber.js in the first place.

@shuckster thanks


text:
Unexpected output for CLI command

Environment
Git SHA-1: [DIRTY]245a3f7d883c516f5f16742cb1ca672872612851
Host OS: Ubuntu 18.04
Using containerization: docker
CPU: Intel(R) Xeon(R) CPU E3-1275 v6 @ 3.80GHz
RAM: 64 GB

Problem
docker exec mina coda advanced  snark-pool-list | jq .
parse error: Invalid numeric literal at line 1, column 6

Reason

Closing this issue as it is stale for a long time.

repo: MinaProtocol/mina


text:
Disable broken tests

This PR removes the broken tests from https://github.com/MinaProtocol/mina/pull/15019. cc @dkijania to fix and re-add them.

!approved-for-mainnet

source: gharchive/pull-request


text:
Move {pickles -> pickles_base}.one_hot_vector

This removes a circular package-to-package dependency between pickles_base and pickles.
Clarification
It's package-to-package circular dependency, not libray-to-library.
Dune handles it well.
But dependency diagram becomes way less readable, also it makes packages incompatible with findlib-based installation approach (because one can't install packages one by one, they depend on one other).
Explain how you tested your changes:

Mina compiles

Checklist:

[x] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[x] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[x] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
[x] Does this close issues? None

!ci-build-me

authors: georgeee rbonichon


text:
Should be scrollable if chart does not fit in the view

A huge thank you for this amazing component. I really love it.
Checklist:

[* ] I updated to the latest version available
[* ] I cleared the cache of my browser

Release with the issue:
v1.9.3
Last working release (if known):
Browser and Operating System:
Windows 10 and Chrome 109 with large monitors -> OK
Android 11 and Chrome with small screen and portrait orientation -> not so good
Description of problem:
Some days ago I added another section so I get energy consumption per room in my house also. That's when it started to become more difficult to see sensors in my mobile.
When sankey charts have 4 or more sections, then the width will grow beyond my mobile phones capabilities in portrait orientation. This makes all other components in that view wider and difficult to look at.
Javascript errors shown in the web inspector (if applicable):
Additional information:
Perhaps not a bug per see, but I believe all components should handle these basic things so filing it as a bug. Maybe related to #86. I have fiddled with the wide option without success.
So better would be that the entire component becomes scrollable when it does not fit in the view.
Also, I'm using the view type grid everywhere.

I am aware of this and am thinking about it but am not sure if scrolling is the solution yet. I've also been thinking about a vertical layout for mobile but that would be a huge change.
For now I just turn my phone horizontally :/

You can now set minimum width per section. If the screen width is less than the sum of min width of all sections. The card will extend beyond the screen and be scrollable

Ah, got it. Thanks, it works.

authors: Floppe MindFreeze

repo: MindFreeze/ha-sankey-chart

license: MIT


text:
importlib_resources deprecated

I think the current version of importlib_resources is deprecated in the repo, the following adjustment works for me, which uses the most updated importlib.resources(I'm using python 3.10):
line 4: import importlib.resources as importlib_resources
line 29: with importlib_resources.as_file(importlib_resources.files("minedojo.tasks.description_files").joinpath(fname)) as p:

import importlib.resources as resources

with resources.path("minedojo.tasks.description_files", fname) as p:
        return str(p)

works for me too

The current version of this repo depends on importlib_resources.path which was deprecated after v5.0.6 (currently importlib_resources is on v6.1.0).
I suggest either:

updating the requirements.txt to reflect the necessary version of importlib_resources
or implement the changes suggested above.

authors: alfaevc astelmach01 karsmars

repo: MineDojo/MineDojo


text:
Dev

Pull Request(PR) 템플릿!

PR하실 때 해당 탬플릿을 사용해 작성해주세요

변경 사항

🚀  에러코드 추가
검색 이름순 오름차순으로 변경

관련 이슈

🔍  이 PR로 해결된 관련 이슈 또는 JIRA 티켓을 나열하세요.

스크린샷 (해당하는 경우):

📷  관련 스크린샷을 삽입하세요.

추가 사항

ℹ  관련된 추가 정보를 추가하세요.

체크리스트

[] ✅  변경 사항을 로컬에서 테스트했습니다.
[] 📝  새로 추가하거나 수정한 코드에 적절한 문서를 추가했습니다.
[] 🧪  코드가 프로젝트의 기존 코드 표준을 충족시키는지 확인했습니다.
[] ✔  기존의 테스트 스위트를 실행하여 모든 테스트가 여전히 통과되는지 확인했습니다.
[] 👀  이 PR에 리뷰어를 할당했습니다.

충돌해결중


text:
TypeError: normalize() argument 2 must be str, not float

I'm creating a PhoneCollection with the drop_dialects and merge_same_language flags both set to False in order to load as many languages as possible.
>>> from phones import PhoneCollection
>>> pc=PhoneCollection(drop_dialects=False,merge_same_language=False)

but I get an exception ...
>>> pc=PhoneCollection(drop_dialects=False,merge_same_language=False)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Python310\lib\site-packages\phones\__init__.py", line 77, in __init__
    ].apply(lambda x: unicodedata.normalize("NFC", x))
  File "C:\Python310\lib\site-packages\pandas\core\series.py", line 4433, in apply
    return SeriesApply(self, func, convert_dtype, args, kwargs).apply()
  File "C:\Python310\lib\site-packages\pandas\core\apply.py", line 1088, in apply
    return self.apply_standard()
  File "C:\Python310\lib\site-packages\pandas\core\apply.py", line 1143, in apply_standard
    mapped = lib.map_infer(
  File "pandas\_libs\lib.pyx", line 2870, in pandas._libs.lib.map_infer
  File "C:\Python310\lib\site-packages\phones\__init__.py", line 77, in <lambda>
    ].apply(lambda x: unicodedata.normalize("NFC", x))
TypeError: normalize() argument 2 must be str, not float

I don't know if you've this in hand in #5,  but for now I can proceed with a function wrapping unicodedata.normalize in an exception handler
i.e.
        def normalize(x):
            try:
                return unicodedata.normalize("NFC", x)
            except:
                return x

        if self.source.allophone_column is not None:
            self.data[self.source.allophone_column] = self.data[
                self.source.allophone_column
            ].apply(lambda x: normalize(x))

Thanks for a really useful library!!

This should not happen anymore in 0.0.3, turns out there were some rogue NaN values. Thanks for pointing this out!

authors: DewiBrynJones MiniXC

repo: MiniXC/phones


text:
Can not update to latest version - version 0.1.5

Hello,
Did you update your library as your quote :
gson: ^0.1.5
Since I get error here :

Because flutter_nct depends on gson ^0.1.5 which doesn't match any versions, version solving failed.
pub get failed (1; Because flutter_nct depends on gson ^0.1.5 which doesn't match any versions, version solving failed.)

p/s : I upgrade from version 0.1.4.

Hello,
The reason for that is, that 0.1.5 is not out already, but i already changed the version in the docs, use 0.1.4, I will publish 0.1.5 soon, it has just one bugfix at the moment and I dont want to publish a new version for every bug, that is the reason.
Thank you for your issue
Minimine

authors: MinimineLP huytower

repo: MinimineLP/dart-gson


text:
delist still shows active listing

I have an NFT that I listed then delisted via the SDK. You can see that token here
Below are code snippets for listing and delisting, although I do notice that compared to the documentation, { wallet } and args are switched in the execute call. I'll switch them but I don't think that should have mattered.
Code snippet for listing:
const handleListThing = async () => {
    const wallet = await selector.wallet();
    const args = {
        nftContractId: thing.node.nft.nft_contract_id,
        price: "100",
        tokenId: thing.node.nft.token_id,
        marketId: "market-v2-beta.mintspace2.testnet",
    };
    await execute(list(args), { wallet: wallet });
};

Code snippet for delisting:
const handleDelistThing = async () => {
    const wallet = await selector.wallet();
    const args = {
        nftContractId: thing.node.nft.nft_contract_id,
        tokenId: thing.node.nft.token_id,
        marketId: "market-v2-beta.mintspace2.testnet",
    };
    await execute(delist(args), { wallet: wallet });
};

After the delist I am still seeing the token in mb_views_active_listings:
{
    "data": {
      "mb_views_active_listings": [
      {
          "approval_id": 4,
          "base_uri": "https://arweave.net",
          "content_flag": null,
          "created_at": "2023-01-04T00:14:54.98842",
          "currency": "near",
          "description": null,
          "extra": null,
          "kind": "simple",
          "listed_by": "everything.testnet",
          "market_id": "market-v2-beta.mintspace2.testnet",
          "media": null,
          "metadata_id": "everything.mintspace2.testnet:bbc881f8c8dad31361911bdf9d80f1a3",
          "minter": "everything.testnet",
          "nft_contract_id": "everything.mintspace2.testnet",
          "price": 100,
          "receipt_id": "FrNoq4v56Wg3ZYJ8e2C6jiU5d9PtNE8NiocCLNkXEp6Z",
          "reference": "3d9f895b-7703-4f67-9d96-568f0b453970",
          "reference_blob": null,
          "title": null,
          "token_id": "6"
      }
    ]
  }
}

If I view more info via nft_listings query, I see that "unlisted_at" is null.
{
  "data": {
    "nft_listings": [
      {
        "accepted_at": null,
        "accepted_offer_id": null,
        "invalidated_at": null,
        "metadata_id": "everything.mintspace2.testnet:bbc881f8c8dad31361911bdf9d80f1a3",
        "price": 100,
        "unlisted_at": null
      }
    ]
  }
}

On the UI however, it seems that the token has been recognized as unlisted since the button available is "List". What query does it use to determine this? I'm assuming something from mb_views_nft_tokens, but unlisted_at is still null here

^ this has been using a tarball generated from commit 7a1be99c4bda4c771b52a3018ed0495a35e66921
Have just pulled in the latest release, will try again with token #7 on everything.mintspace2.testnet contract. These are the results:
List successful, I see it in mb_views_active_listings.
Had to change delist args to have an array of tokens (this should be updated on docs), but delist is successful. I see it removed from active listings. I see unlisted_at populated.
Perfect, thanks!

Thank you for reporting and including all the details. Good it's working now, we'll update the docs accordingly!

authors: cif elliotBraem

repo: Mintbase/mintbase-js


text:
🛑 Ye is down

In 5461949, Ye ($YE) was down:

HTTP code: 500
Response time: 9088 ms

Resolved: Ye is back up in 8bf6884 after 67 days, 4 hours, 58 minutes.

repo: Miouwn/upl


text:
Use nanoseconds in ImageFsInfo Timestamp on Linux

According to CRI-API this should be nanoseconds:
https://github.com/kubernetes/cri-api/blob/release-1.28/pkg/apis/runtime/v1/api.proto#L1554
Additionally, image_window.go has nanoseconds already.

Broken since https://github.com/kubernetes/kubernetes/commit/0d907e015b784a4edeb17c0580bf1e408c966de7 (k8s v1.16)

ping @nwneisen @neersighted

This one rebased just in case

The tests look good now. Wonder if there is something else that I have to do to get this merged.

authors: afbjorklund surik

repo: Mirantis/cri-dockerd


text:
Create Client Authentification

Only authentified clients can access to APIs
OAuth or JWT authentification system
Manage Authorization Token

JWT authentication system
Bearer token
Generate token with URI api/login_check


text:
Create and refactor django tests

Create django tests for:

models
managers
forms
services
views

Hii, Is this issue still availabel ??
Can you assign me ??


text:
AuthResponseMessage did not arrive

I have a perfectly normal authentication script:
https://dotnetfiddle.net/yyqcb4
But OnAuthResponseMessage method is sometimes not called. Sometimes it is called on the first attempt to connect, and sometimes on the tenth. When in HUD i press to "Server + Client" = all okey, but when i created a server and press "Client" this bug is working. I checked the time and in OnStartClient the AuthResponseMessage handler registered is called before the server has sent the message.
This video shows how everything looks on the client side, but the connection every time works on the server:
https://www.veed.io/view/cf9e5d94-8edf-4e60-8ec9-249838b54653?sharingWidget=true&panel=share
Tested on unity editor, windows and android - bug working everywhere.

Unity version: 2021.3.10f1
Mirror branch: 2022.9.15

Mirror update helped.

repo: MirrorNetworking/Mirror


text:
zappa package production fails when using --no_venv

Context
Running zappa package production --no_venv throws TypeError: expected str, bytes or os.PathLike object, not NoneType. This error happens only when running outside of a virtual environment. zappa package production works as expected.
While I believe running zappa outside of a virtual environment using --no_venv is supported (per #835), I understand why this would be a low-priority issue.
Expected Behavior
zappa builds an application package.
Actual Behavior
zappa throws the following error:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/zappa/cli.py", line 2778, in handle
    sys.exit(cli.handle())
  File "/usr/local/lib/python3.8/site-packages/zappa/cli.py", line 512, in handle
    self.dispatch_command(self.command, stage)
  File "/usr/local/lib/python3.8/site-packages/zappa/cli.py", line 551, in dispatch_command
    self.package(self.vargs['output'])
  File "/usr/local/lib/python3.8/site-packages/zappa/cli.py", line 642, in package
    self.create_package(output)
  File "/usr/local/lib/python3.8/site-packages/zappa/cli.py", line 2257, in create_package
    self.zip_path = self.zappa.create_lambda_zip(
  File "/usr/local/lib/python3.8/site-packages/zappa/core.py", line 533, in create_lambda_zip
    split_venv = splitpath(venv)
  File "/usr/local/lib/python3.8/site-packages/zappa/core.py", line 527, in splitpath
    (path, tail) = os.path.split(path)
  File "/usr/local/lib/python3.8/posixpath.py", line 103, in split
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

Possible Fix
If running zappa package outside of a virtual environment is not supported, then the resolution to this issue would be removing support for zappa package --no_venv or reverting #1891.
Another option is adding a conditional check to splitpath() to see whether we are running in a virtual environment before running splitpath(venv).
What appetite do the project maintainers have for supporting zappa use outside of virtual environments?
Your Environment

Zappa version used: 0.51.0
Operating System and Python version: Python 3.8 on Alpine Linux 3.11 (3.8-alpine3.11 Docker image)
The output of pip freeze:

argcomplete==1.11.1
boto3==1.13.6
botocore==1.16.6
certifi==2020.4.5.1
cfn-flip==1.2.3
chardet==3.0.4
click==7.1.2
docutils==0.15.2
durationpy==0.5
future==0.18.2
hjson==3.0.1
idna==2.9
jmespath==0.9.5
kappa==0.6.0
pip-tools==5.1.2
placebo==0.9.0
python-dateutil==2.6.1
python-slugify==4.0.0
PyYAML==5.3.1
requests==2.23.0
s3transfer==0.3.3
six==1.14.0
text-unidecode==1.3
toml==0.10.0
tqdm==4.46.0
troposphere==2.6.1
urllib3==1.25.9
Werkzeug==0.16.1
wsgi-request-logger==0.4.6
zappa==0.51.0

Your zappa_settings.json: the error can be reproduced no matter what's in zappa_settings.json

@woodrad any chance you can try it out with #2119 to check if I haven't missed anything else?

I have a similar issue.  I'm trying to package my app from inside a Docker container.  @jneves I checked out your branch and unfortunately it didn't resolve the issue.

pip install git+https://github.com/Miserlou/Zappa.git@bugfix/gh-2097-fix-deploy-with-no-env
zappa package -s ./zappa_settings.json  --no_venv yields:

Calling package for stage dev..
Oh no! An error occurred! :(

==============

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/zappa/cli.py", line 2778, in handle
    sys.exit(cli.handle())
  File "/usr/local/lib/python3.6/site-packages/zappa/cli.py", line 512, in handle
    self.dispatch_command(self.command, stage)
  File "/usr/local/lib/python3.6/site-packages/zappa/cli.py", line 551, in dispatch_command
    self.package(self.vargs['output'])
  File "/usr/local/lib/python3.6/site-packages/zappa/cli.py", line 642, in package
    self.create_package(output)
  File "/usr/local/lib/python3.6/site-packages/zappa/cli.py", line 2264, in create_package
    disable_progress=self.disable_progress
  File "/usr/local/lib/python3.6/site-packages/zappa/core.py", line 533, in create_lambda_zip
    split_venv = splitpath(venv)
  File "/usr/local/lib/python3.6/site-packages/zappa/core.py", line 527, in splitpath
    (path, tail) = os.path.split(path)
  File "/usr/local/lib/python3.6/posixpath.py", line 107, in split
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

==============

authors: hbrooks jneves woodrad

repo: Miserlou/Zappa


text:
Obscure API and very slow dev

https://vec.io/posts/kraken-high-performance-webrtc-sfu
This is a very disrespectful post. It's ok that you discard SFUs just because they are not written in Go (the only language valid for you) but at least do not write shit about them.
A "SFU just for audio" can be done in 15 minutes, so you have done nothing special.
And BTW:

Unified-Plan is not a "super recent and cool WebRTC feature". It's been there for long years.
Adding video is not "an easy extension". Well, you'll see when you try to implement it.

In short: you should not write shit about others just because you want to promote your own solution on Go.

Sorry if I offend you at any words of that post.
I understand for someone like you, an core engineer of the famous mediasoup, you are able to make an audio only SFU from scratch in under 15 minutes, good job. But I must say that I am clearly not smart enough, even dumb to learn mediasoup and get some demo project running up without using your client library. I know you are smartest to make that excellent work in minutes or hours so that nobody could use it in 15 minutes or days.
Of course Kraken may be slow in production, but quick to develop and get shit fixed.
For the unified plan thing, yes it was introduced years ago, and I assume you should know that it's only stable and default in Chrome for no more that 2 years yet. Of course not cutting edge 😂 Janus doesn't support it though, many engineers not smart as you don't even know it.
I write shit, I don't know Kraken is slower than mediasoup, and I don't believe any beginners would think Kraken API is obsecure than any other SFUs, including your mediasoup.
At least could you please make an working example like the one in Kraken readme, in 15 minutes, without your awesome but too complicated client library?

At least could you please make an working example like the one in Kraken readme, in 15 minutes, without your awesome but too complicated client library?

No, I won't waste my time with people like you. Sorry, nothing personal. It's just that you suck. Bye.

I'm glad your problem fixed, now I will close  the issue if you don't have any other questions.

"Janus doesn't support it"? Do you even do your homework? We've had a branch ready and under test for weeks. I agree with Iñaki that the blog post was highly disrespectful of the work developers like us have been doing FOR YEARS and given away to thousands of people FOR FREE. I won't even deign the "Janus developers are slow" comment the dignity of a response.

As I written in the blog post, "We used Janus in our previous WebRTC application years ago, and it’s our first candidate when we choose SFU for our new project, because we knew its API and flow, and it has very good performance and stability." highly praised Janus. And of course I knew that you have the branch, how could I don't know that? And I even emailed you from the meetecho website to ask for paid service because we do appreciate your work.
But isn't it truth that Janus doesn't have unified plan in your master branch or some stable release? You really want people to use a development branch in a production environment?
Slow is not some bad words, you make things slow and stable. And even @ibc implies Janus is slow Unified-Plan is not a "super recent and cool WebRTC feature". It's been there for long years..
Why are you so mean? I just write my reviews, and I have used and read your code, recommended Janus to people if it fit their needs, I just say that Janus doesn't fit in our specific situation.

You have created an audio only SFU for your super specific use case so any other more flexible and powerful SFU is bad for you. Your design comes with tons of bad assumptions just because your single use case. You should know that instead of writing shit about others.
And I never said that Janus is slow just because it doesn't not officially support a specific feature. Your stuff does not support video and you assume "it's an easy extension". You will figure out what slow means by yourself.

Hi @ibc, you are a good guy, and you have good and delicious soup, and I'm sorry I don't like that taste and I sincerely apologize and you just not suck and bye.

Since no meaningful things are discussed here, I will lock this thread for good to avoid any further abusive words.
Please only report issues of the software here.

authors: cedricfung ibc lminiero

repo: MixinNetwork/kraken


text:
HMR not being called on Windows

My browser is Chrome, stable version 53.0.2785.116 (Official Build) m (64-bit) as of now.
OS: Windows 10, latest version (there were even updates this night... I hate the new auto-auto-updates D: )
I wanted to try out Phenomic, so I simply followed the setup guide.
went to a new empty folder, ran these in order and did nothing else:
npm i phenomic

./node_modules/.bin/phenomic setup

npm install

npm start

My browser page opened, with a stylized list of links to the docs and some posts that are in the boilerplate, so I thought "cool it works"
But then I started editing content/index.md, and I noticed nothing was output to the terminal, and no change was reflected in my browser — the dev console also didn't budge.
In small words: no HMR hot loading on windows 10 with the default config.
It must be an upstream issue, but still, it's cool to know here too.

Update:
It HMR reloads when my project is not in a path containing any type of brackets, or spaces.
It's a path issue somewhere upstream. Most likely chokidar.

Which page are you seeing the posts? Currently, hmr for markdown content only works on pages that contains 1 markdown posts, not the one with mutiple posts, which is filtered with enhanceCollection

I'm not sure I understand the question, but it was the boilerplate's homepage index.html file, at the root of the content tree. Though its layout contained a list of posts, its markdown content was updated correctly once my path was free of spaces or brackets. Editing the homepage index.js layout updated the page too. Didn't try editing the title of one of the posts from its own file.

I see. I guess this is something we can improve. I use Windows 10 too but I never name a file with spaces or brackets.

I have a folder structure where "categories" (like "development", "movies", "programs"...) are named in brackets to appear before any other folder when I sort by name. Never had problems before dealing with CLI tools, because a lot of gui software manage those by simply using the OS's file system management. Here, there are a lot of string manipulation directly between tools like Chokidar or Path, so it's easy to just not know who does what with there paths.

The PR for #943 resolved the issue. Additionally, there was another solution posted by someone else on that same thread. Hot reloads worked fine for me for all md files after the fix. Please consider applying the fixes from the closed item to address this earlier opened issue.
FWIW, the path I was using had no spaces or special characters.

Can anyone confirm which solution works/is the best?

I'm seeing the same problem. Editing text in index.md gives this error in the console on the second change (the first one works):
(btw love phenomic so far)
[HMR] Cannot check for update (Full reload needed)
handleError @ process-update.js?e13e:115
applyCallback @ process-update.js?e13e:45
hotApply @ phenomic.browser.454c1e3….js:528
cb @ process-update.js?e13e:52
hotUpdateDownloaded @ phenomic.browser.454c1e3….js:312
hotAddUpdateChunk @ phenomic.browser.454c1e3….js:284
webpackHotUpdateCallback @ phenomic.browser.454c1e3….js:5
(anonymous) @ 0.454c1e3….hot-update.js:1
process-update.js?e13e:116 [HMR] TypeError: Cannot read property '__dataUrl' of undefined
    at eval (eval at <anonymous> (http://localhost:3333/phenomic.browser.454c1e338f7ff0ca0dbf.js:2838:2), <anonymous>:19:25)

reload in the function performReload is always false, so a full reload is not performed either. A missing arg somewhere?
function performReload() {
    if (reload) {
      if (options.warn) console.warn("[HMR] Reloading page");
      window.location.reload();
    }
  }

What is your OS?

I ran into the same issue, so I looked into it a bit.  The patch works, but does not address the cause of the problem.  It looks like collection filenames are having their slash directions set incorrectly on windows, which causes the find item test in src/client/hot-md.js to fail.
https://github.com/MoOx/phenomic/blob/master/src/client/hot-md.js#L6

The __filename path seems to be set on https://github.com/MoOx/phenomic/blob/master/src/loader/index.js#L42
One simple solution might be to replace "\" with "/" on the relativePath variable.

@micnigh mind doing a PR?

authors: MoOx evenkiel facelesspanda micnigh pcguru thangngoc89

repo: MoOx/phenomic


text:
Index content by arbitrary fields

Just kept the special case for author and authors for most users confort. might change later if we consider it too much.

Awesome!

authors: MoOx bloodyowl


text:
statinamic broken because of babel@6.6 update

https://github.com/istarkov/babel-plugin-webpack-loaders/issues/39
https://phabricator.babeljs.io/T7168
https://phabricator.babeljs.io/T7164

Fixed by babel 6.6.3

That's fast.

repo: MoOx/statinamic


text:
let each view controller has individual navigation bar alpha

Hi Mozhouqi,
我给 navigationbar 的 alpha 方法，进行了 swizzle，是想通过使用原生 api 的方式去更新 bar 的背景视图的 alpha 值，然而这个 alpha 值是全局的，所有就想把 这个值 也存在假的 bar 里，也就这里这个 pr，关于这个 pr 也想听听你的想法。

建议直接修改图片的 alpha 值，改 navigation bar 的 alpha 值不光会影响导航栏的背景色，还会影响导航栏上别的控件。

Hi @karosLi，建议直接修改图片的 alpha 值，改 navigation bar 的 alpha 值不光会影响导航栏的背景色，还会影响导航栏上别的控件。

authors: MoZhouqi karosLi

repo: MoZhouqi/KMNavigationBarTransition


text:
TTS chat feature request

Implement Android native TTS for dyslexic people. Kind of like how Google Bard is doing.

not sure if Android TTS works on all platforms and im not really interested in implementing too much platform specific code. But ill look into it

would be nice, I would love to use it with sherpa-onnx

I can use https://pub.dev/packages/flutter_tts and it will be ok for all platforms (excluding linux maybe) but to be perfectly honest im kind of waiting for there so be a decent open source alternative to something like eleven labs that i can implement

I can use https://pub.dev/packages/flutter_tts and it will be ok for all platforms (excluding linux maybe) but to be perfectly honest im kind of waiting for there so be a decent open source alternative to something like eleven labs that i can implement

I  use this app primarily on Android only. Windows and Linux have better alternatives to this. Maybe you can fork some code from here for Android. https://github.com/jing332/tts-server-android

For anyone interested and so nobody try to take this on by going in a different direction. The current plan is to use piper_tts for text to speech but i haven't finished developing it yet.
https://github.com/Mobile-Artificial-Intelligence/piper_tts

Worth noting that LocalAI supports basically all of the OpenAI API, but locally.  That includes multiple models, text to speech, speech to text, image generation, and image recognition.  It would make sense to use that, as a starting point.  The combination of running a very small model locally on mobile + LocalAI for a more powerful model when remote PC connectivity is available would be great.

sherpa-onnx now has an official Dart client:

https://pub.dev/packages/sherpa_onnx

@davidmigloz Sounds great all integrate that for the 1.3 release

doesnt seem to have out of the box support for flutter TTS yet? only in dart and it needs me to provide my own piper binaries

I haven't tried it myself yet, but yes, it's not "out of the box", you have to manage the TTS models yourself.

@davidmigloz
Managing the models isnt a problem, its managing the binaries. It doesnt appear they have ffigen set up correctly to compile the required C++ code.
This would mean i would need to obtain the correct precompiled binaries for Android x86_64 and arm64, Linux, Windows and now MacOS x86_64 and arm64.
Then i would need to ship these precompiled binaries with maid which is something im not willing to do because its against open source and it would cause issues with f-droid.
I am still slowly working on the TTS question but yeah i need it to compile on build

ahh indeed, I see what you mean

babylon_tts is working now

authors: Ar57m danemadsen davidmigloz lee-b x-legion

repo: Mobile-Artificial-Intelligence/maid


text:
Stop generating static framework targets in Xcode

These targets aren't buildable because of: https://github.com/MobileNativeFoundation/rules_xcodeproj/issues/105
We shouldn't show them in Xcode.

Before
After

Seeing this error if folks have ideas:
cd examples/rules_ios && bazel run //:xcodeproj

Error in fail:
'@//Lib:LibDynamic' is not a top-level target, but was listed in `top_level_targets`. Only list top-level targets (e.g. binaries, apps, tests, or distributable frameworks) in `top_level_targets`. Schemes and `focused_targets`/`unfocused_targets` can refer to dependencies of targets listed in `top_level_targets`, and don't need to be listed in `top_level_targets` themselves.

Looks like not generating the top level static framework causes the underlying buildable library to not be included as top level

Side comment: let's land https://github.com/MobileNativeFoundation/rules_xcodeproj/pull/2512 first, since your stuff is in the same area.

Will rebase and refactor this to work with your changes

Tested generating the Cash App project and seeing a lot nicer scheme selector without unbuildable static framework targets:

@thiagohmcruz Mind giving this a test in Square repo and reporting back when you have time ❤

Seeing some targets like LibDynamic are missing though so will investigate that more

And this is less a problem with rules_ios and more an issue with the fact that apple_framework ends up creating the "static framework" under the hood to wrap everything which is what this removes from Xcode since it isn't supported, correct?

And this is less a problem with rules_ios and more an issue with the fact that apple_framework ends up creating the "static framework" under the hood to wrap everything which is what this removes from Xcode since it isn't supported, correct?

Yeah this is removing from Xcode all things with the .framework.static product type, I thought this would be fine because I assumed it created target like:
foo
  - foo (static framework)
  - foo_swift (swift_library)
  - foo_objc (objc_library)

So looking at the BUILD file it looks like LibDynamic is excluded because it's the top-level static framework (even though its marked link_dynamic = True, probably a rules_ios issue) and it depends on Lib which is still shown in the project so think that makes sense.
Essentially this change will exclude the top level framework.static targets but attempt to keep their library deps included in the project.
Any thoughts @mattrobmattrob @thiagohmcruz around this change?

@brentleyjones Is this something I should look at updating more or do we not want this? I'm kind of on the fence here, if all of these targets are strictly unbuildable currently then hiding them makes sense, but if under certain conditions they work I wouldn't want them to be hidden for folks who've been relying on them.

If they are hidden, then the library(s) that feed into it will still exist.
Can you remind me what happens in the broken experience? Also, how it behaves in the new incremental generation mode? That will inform me on if we should go with this or not. Personally I'm leaning towards taking this change though, since they aren't a supported target type right now.

Can you remind me what happens in the broken experience?

You get this error: https://github.com/MobileNativeFoundation/rules_xcodeproj/issues/1929. The libraries they package should be buildable though.
I'll give the incremental generator a try and report back

With the latest incremental generation mode changes, static framework targets have proper target merging and now build.

Confirmed for our project the merging work and static framework targets build as expected 🎉

authors: brentleyjones luispadron mattrobmattrob


text:
[REQUEST] Make notices more generic

Is your feature request related to a problem? Please describe.
At present, some notices such as RouteColorAndTextInsufficientContrastNotice contain a field contastRatio. These fields are specific and should be wrapped into a field ruleValue containing all rule specific values. This would allow for a more generic treatmentof notices in the future.
Consider the architecture used in geojson  via field properties. See https://geojson.org/

Implemented in https://github.com/MobilityData/gtfs-validator/pull/273

repo: MobilityData/gtfs-validator


text:
Proposal: Support DUPLICATED trips in GTFS-RT

Background
Producers and consumers have identified the need for a lightweight method to take an existing trip defined in static GTFS, and then in real-time say that its pattern (e.g., the stop_times.txt definition) is running at a new service date and/or time [1].
The GTFS-RT spec does not currently define a official way to duplicate trips. The schedule_relationship of ADDED is a potential candidate to convey this type of information. However, the GTFS-RT spec has not clearly defined what exactly ADDED is and how it should be used. As a result, multiple consumers and producers have different interpretations of what an ADDED trip is [2][3].
Proposal
This proposal adds a new trip schedule_relationship of DUPLICATED, which can be used to duplicate an existing trip from GTFS (CSV) but then run that trip pattern at a new time (TripProperties.start_time) and/or service date (TripProperties.start_date). The existing field TripDescriptor.trip_id is used to identify the trip to be duplicated in GTFS trips.txt (CSV), and TripProperties.trip_id defines the new trip_id for the new trip.
This proposal also deprecates the existing trip schedule_relationship of ADDED, which has be used differently by various producers and consumers with varying meaning due to lack of clarify in the current spec [2][3]. ADDED is soft-deprecated, which means that if existing producers and consumers want to continue using it for their own purposes it will remain usable in the .proto file.
This pull request is a subset of the GTFS-ServiceChanges v3.1 spec:
https://bit.ly/gtfs-service-changes-v3_1
A few producers and at least one consumer currently implement this use case via ADDED, so it should be relatively straightforward to change the implementation to using DUPLICATED instead. Therefore, this feature could be adopted prior to the remainder of the ServiceChanges proposal.
Future proposals may add other fields defined in GTFS trips.txt to the TripProperties message  (e.g., route_id, trip_headsign, trip_short_name, block_id, shape_id) so they can be changed in real-time. See the ServiceChanges proposal for details.
[1] https://github.com/google/transit/issues/216
[2] https://github.com/google/transit/issues/106
[3] https://github.com/google/transit/issues/113#issuecomment-436384798

Closing to open a PR on the main google/transit repo.

Proposal opened at https://github.com/google/transit/pull/221.

repo: MobilityData/transit


text:
quant_ptq is not supported

root@597e1d20794c:~/EOD# python -u -m up train --ng=1 --launch=pytorch --config=/root/EOD/configs/quant/det/faster_rcnn/faster_rcnn_r18_FPN_2x_quant_qdrop.yaml --display=10
NOTE! Installing ujson may make loading annotations faster.
No module named 'petrel_client'
init petrel failed
2022-07-14 10:31:18,453-rk0-normalize.py#44:import error No module named 'msbench'; If you need Msbench to prune model,     you should add Msbench to this project. Or just ignore this error.
No module named 'spring_aux'
2022-07-14 10:31:18,758-rk0-spconv_backbone.py#17:import error No module named 'spconv'; If you need spconv, you should install spconv !!!. Or just ignore this error
Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/opt/conda/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/root/EOD/up/__main__.py", line 27, in <module>
    main()
  File "/root/EOD/up/__main__.py", line 21, in main
    args.run(args)
  File "/root/EOD/up/commands/train.py", line 144, in _main
    launch(main, args.num_gpus_per_machine, args.num_machines, args=args, start_method=args.fork_method)
  File "/root/EOD/up/utils/env/launch.py", line 68, in launch
    main_func(*(args,))
  File "/root/EOD/up/commands/train.py", line 134, in main
    runner = RUNNER_REGISTRY.get(runner_cfg['type'])(cfg, **runner_cfg['kwargs'])
  File "/root/EOD/up/utils/general/registry.py", line 81, in get
    assert module_name in self, '{} is not supported, avaiables are:{}'.format(module_name, self)
AssertionError: quant_ptq is not supported, avaiables are:{'base': <class 'up.runner.base_runner.BaseRunner'>, 'kd': <class 'up.tasks.distill.runner.kd_runner.KDRunner'>, 'point': <class 'up.tasks.det_3d.runner.point_runner.PointRunner'>, 'quant': <class 'up.tasks.quant.runner.quant_runner.QuantRunner'>, 'multitask': <class 'up.tasks.multitask.runner.multitask_runner.MultiTaskRunner'>, 'sparse': <class 'up.tasks.sparse.runner.sparse_runner.SparseRunner'>}

Sorry for this bug, the config is not updated, you can replace quant_ptq with quant. @wangshankun

authors: wangshankun yqyao

repo: ModelTC/United-Perception


text:
Budget-Item-Chart

Functionality with chart mentioning percentage of budget item contribution done.
Features developed to see percentage of total budget an item is contributing with, so that we better understand statistics of inflow or outflow items.
Checklist

[x] Feature developed
[x] Created/updated unit tests
[x] Comments in code

Author: Gandhimathi Padmanaban
Email: gmbabu6@gmail.com

Coverage increased (+0.04%) to 77.973% when pulling e5782d7f703255bc5049782e644cafcbf7d9801f on gmbabu6:budget-item-chart into ddf7c8e1ee591036dbf2f6285e5ae412423f4f6e on ModusCreateOrg:master.


text:
FEATURE REQ : Add service catalog product provisioning

Would it be possible to look into adding Service Catalog product provisioning via terraform instead of utilizing direct EC2 provisioning.
resource : https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/servicecatalog_provisioned_product

Hi Navean, i will work in a PoC to have SC added into this pattern. Thanks for your input.

authors: Moglius naveanjp

repo: Moglius/deploy-aws-stuff


text:
QCArchive Overview doc

Description
Adds a brief overview of the QCArchive ecosystem and a short description of how the pieces fit together. Several documentation errors are fixed and the changelog has also been updated.
Status

[x] Ready to go

I also validated all the links worked and the docs all build correctly.

authors: Lnaden dgasmith

repo: MolSSI/QCFractal


text:
Microsoft.VisualStudio.Coverage.Monitor.pdb bug

S_LPROC32 Function 'DuplicateLoggerBuffer' | RVA 0x1C70
S_LOCAL: 'CProfileMonitorControl' -> 'ProfileMonitorControl'
S_DEFRANGE_REGISTER: Register 0x 14A
S_DEFRANGE_REGISTER: Register 0x 14C
S_LOCAL: 'CLoggerBuffer' -> 'SourceBuffer'
S_DEFRANGE_REGISTER: Register 0x 14B
S_DEFRANGE_REGISTER: Register 0x 14E
S_LOCAL: '_GUID' -> 'ExecGuid'
S_DEFRANGE_REGISTER: Register 0x 150
S_DEFRANGE_REGISTER: Register 0x 156
S_LOCAL: '_BUFFERHEADER' -> 'DuplicateBufferHeader'
S_DEFRANGE_REGISTER: Register 0x 151
S_LOCAL: 'CLoggerBuffer' -> 'DuplicateBuffer'
S_DEFRANGE_REGISTER: Register 0x 14D
S_LOCAL: 'PUCHAR' -> 'SourceCoverageData'
S_DEFRANGE_REGISTER: Register 0x 14B
S_LOCAL: 'PUCHAR' -> 'DuplicateCoverageData'
S_DEFRANGE_REGISTER: Register 0x 14A
S_LOCAL: '_BUFFERHEADER' -> 'SourceBufferHeader'
S_DEFRANGE_REGISTER: Register 0x 14B
S_INLINESITE: Parent 0x 5A7C
S_INLINESITE: Parent 0x 5C60
S_LOCAL: 'ULONG' -> 'dwHandle'
S_DEFRANGE_REGISTER: Register 0x 14
https://ufile.io/846wqrra

Can you please provide more information when submitting an issue?
The description says "Microsoft.VisualStudio.Coverage.Monitor.pdb bug", but I don't see what's wrong.
It would be nice if you could describe what you're expecting to see, what's happening, what the actual bug is, etc.

Problem with pointer, unable to run from start to finish

You can download the PDB file and run it once to find out where the error is

Can you reproduce it? The program cannot parse this PDB and will crash,

I did a quick debugging session on this PDB and there are some odd things about it.
It crashes because there is a S_LOCAL which has a type index 0x80100011, yet there are only 0x8795 types in the PDB. Also, this 0x80100011 looks like a bit mask and not a type index to me. Not sure why S_LOCAL would have this value for its type index and I wasn't able to find any hints in the microsoft-pdb repo.
If I handle this out of bounds type index gracefully and return type_index_out_of_range for the type name, then I get the following output for function variables in the function that crashes
S_LPROC32 Function 'DuplicateLoggerBuffer' | RVA 0x1C70
    S_LOCAL: 'CProfileMonitorControl' -> 'ProfileMonitorControl' | Param: True | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 330
    S_DEFRANGE_REGISTER: Register 332
    S_LOCAL: 'CLoggerBuffer' -> 'SourceBuffer' | Param: True | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 331
    S_DEFRANGE_REGISTER: Register 334
    S_LOCAL: '_GUID' -> 'ExecGuid' | Param: True | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 336
    S_DEFRANGE_REGISTER: Register 342
    S_LOCAL: '_BUFFERHEADER' -> 'DuplicateBufferHeader' | Param: False | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 337
    S_LOCAL: 'CLoggerBuffer' -> 'DuplicateBuffer' | Param: False | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 333
    S_LOCAL: 'PUCHAR' -> 'SourceCoverageData' | Param: False | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 331
    S_LOCAL: 'PUCHAR' -> 'DuplicateCoverageData' | Param: False | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 330
    S_LOCAL: '_BUFFERHEADER' -> 'SourceBufferHeader' | Param: False | Optimized Out: False
    S_DEFRANGE_REGISTER: Register 331
    S_INLINESITE: Parent 0x5A7C
        S_INLINESITE: Parent 0x5C60
            S_LOCAL: 'ULONG' -> 'dwHandle' | Param: False | Optimized Out: False
            S_DEFRANGE_REGISTER: Register 20
            S_LOCAL: 'type_index_out_of_range' -> 'pbuf' | Param: False | Optimized Out: False
            S_DEFRANGE_REGISTER: Register 333
            S_DEFRANGE_REGISTER: Register 333
            S_INLINESITE: Parent 0x5C78
                S_CALLEES: Count 1
                S_LOCAL: 'type_index_out_of_range' -> 'pbuf' | Param: False | Optimized Out: False
                S_DEFRANGE_REGISTER: Register 328
                S_DEFRANGE_REGISTER: Register 328
                S_INLINESITE: Parent 0x5CF8
                S_INLINESITE_END:
            S_INLINESITE_END:
            S_INLINESITE: Parent 0x5C78
            S_INLINESITE_END:
        S_INLINESITE_END:
    S_INLINESITE_END:
    S_INLINESITE: Parent 0x5A7C
    S_INLINESITE_END:
    S_INLINESITE: Parent 0x5A7C
    S_INLINESITE_END:
    S_FRAMEPROC: Size 32 | Padding 0 | Padding Offset 0x0 | Callee Registers Size 24
    S_INLINEES: Count 14
    S_LABEL32: '$LN34' | Offset 0xD91
    S_REGREL32: 'ProfileMonitorControl' -> 'CProfileMonitorControl' | Register 335 | Register Offset 0x40
    S_REGREL32: 'SourceBuffer' -> 'CLoggerBuffer' | Register 335 | Register Offset 0x48
    S_REGREL32: 'ExecGuid' -> '_GUID' | Register 335 | Register Offset 0x50
    S_CALLSITEINFO: 'type_index_out_of_range' | Offset 0xC98 | Section 1
    S_CALLSITEINFO: 'type_index_out_of_range' | Offset 0xCBC | Section 1
S_END

As you can see, there are multiple type_index_out_of_range in the output for the function.
Even after this change, it hits asserts due to Unhandled record kind 0x117F with block level 1. I'm unable to find any information on record kind 0x117F.
I then tried running this pdb through llvm-debuginfo-analyzer and it just crashes on the PDB.
I also tried running it throught dia2dump, but it appears it does not print the function variables/scopes.

Output: Microsoft.VisualStudio.Coverage.Monitor-dia2dump.txt.

You can search for the crashing function DuplicateLoggerBuffer in the dia2dump output and see it provides no details about the variables in functions.
A deeper investigation of the PDB be is required to figure out exactly what is going on. Maybe writing a parser with the DIA SDK could be a way forward to help understand what is going on :-)

This is the standard library included in vs2022, while jpeg62.pdb is a library that I compiled using vcpkg. I'm not sure if it's the wrong type. I downloaded some PDB files for system DLLs using x64dbg and also detected a crash. Once this is fixed, I will scan those PDB files again

How many days are there updates?, I would like to scan all the PDBs in the Windows DLL once, and wait for you to update this error

This repo is maintained by contributors in their spare time, so please don't expect people to jump at issues as soon as they are reported.
Furthermore, what are you trying to do, i.e. what information are you after?
Do you need all the function/local variable/register information?
E.g. as stated above, Dia2Dump does not give this information, so running all the other examples for all the PDBs probably runs fine.

I am unlikely to investigate this issue further.  It looks to me like the PDB either has bad data or is using unknown magic values and it will probably require significant effort and/or some trial&error to fully understand the data in this PDB, especially since there is no source available.
As tivolo mentions, the PDB works fine for cases where do you not need function variables and also any use cases where you do not have the source, which I guess would be all use cases outside of Microsoft :-)
So unless there is another PDB with available source that has similar values for S_LOCAL and record kind 0x117F, then I currently don't see much value in fixing this issue.

authors: MolecularMatters icyfox168168 lukekasz tivolo


text:
BUG:in blog section make the search bar in the middle

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
in the blog page the search bar has to be in centre
Add ScreenShots

What browser are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'23 contributor
[X] I want to work on this issue

I would like to resolve this issue please assign me this issue

repo: MonalikaPatnaik/UMatter


text:
Force new on api key description change

Fixes previous PR: https://github.com/Mongey/terraform-provider-confluentcloud/pull/27

Since update isn't implemented, requires force new to be set to true
Tested locally and working

I can manually compile the plugin for now, but is there a process to releasing a new version of the provider?

Thanks @petern-sc !

I can manually compile the plugin for now, but is there a process to releasing a new version of the provider?

Pushing a vX.Y.Z tag to the repo  triggers the release workflow.
I've been periodically creating releases after I run through some manual QA as there's no tests here 😬

Thanks for the info, sounds reasonable @Mongey. I'm happy to leave that to you since I'm assuming I won't have permissions to push the tag, let me know if you need any help 😄.

authors: Mongey petern-sc


text:
Auto Save Slots are not Deletable

Right now, the deletion is using the SaveLabel to target the slot to delete which of course won't allow slots using the AutoSaveLabel to be deleted. There must be a way to identify the kind of slot to delete and use the proper label.

This has been fixed as of 45cffad54419a853c0395c382420d55d49deffd0

repo: Monogatari/Monogatari


text:
Tarea3 Documentación de Merge

Tarea de la documentación del MergeSort sin recursión

Gracias

authors: KOS-mo MontealegreLuis

repo: MontealegreLuis/algoritmos


text:
关于selectPicker 在columns异步耗时操作的时候，在modelValue监听去设置showValue，并不是太合理，会出现showValue为空的情况

Wot Design Uni 版本号
0.2.13
平台
h5, 微信小程序
重现链接
https://wot-design-uni.cn/component/select-picker.html
重现步骤
columns通过异步耗时请求设置的时候。v-model绑定了值，但是其回显的值并没有出现。
原因是监听modelValue的时候，columns还没有返回，而这个时候setShowValue，所以showValue为空了。
建议通过computed计算的方式返回showValue，比较好点。
期望的结果是什么？
columns通过耗时操作设置的时候，依然显示showValue
实际的结果是什么？
耗时操作之后，showValue为空
环境信息
No response
其他补充信息
建议这个问题能快点修复一下，谢谢了

colPicker也是同样的问题

大佬，这个能修复吗。

昨天有在看，不过不太清楚你具体产生问题的场景，提供一下复现demo把，我用来调试。
------------------ 原始邮件 ------------------
发件人:                                                                                "Moonofweisheng/wot-design-uni"                                                                                @.>;
发送时间: 2024年1月24日(星期三) 下午4:12
@.>;
@.***>;
主题: Re: [Moonofweisheng/wot-design-uni] 关于selectPicker 在columns异步耗时操作的时候，在modelValue监听去设置showValue，并不是太合理，会出现showValue为空的情况 (Issue #167)
大佬，这个能修复吗。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

昨天有在看，不过不太清楚你具体产生问题的场景，提供一下复现demo把，我用来调试。
…
------------------ 原始邮件 ------------------ 发件人: "Moonofweisheng/wot-design-uni" @.>; 发送时间: 2024年1月24日(星期三) 下午4:12 @.>; @.>; 主题: Re: [Moonofweisheng/wot-design-uni] 关于selectPicker 在columns异步耗时操作的时候，在modelValue监听去设置showValue，并不是太合理，会出现showValue为空的情况 (Issue #167) 大佬，这个能修复吗。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.>

场景很简单比如，一个表单，在onLoad 中 selectPicker 的columns数据是通过接口或者延时进行添加数据的，延时可以设置长一点。在columns还没有返回的时候，依然是空的时候，就通过v-model绑定selectPicker 的值的时候，他是不回填的。

可以给出一个复现demo出来。附件在当前issue就行。
------------------ 原始邮件 ------------------
发件人:                                                                                "Moonofweisheng/wot-design-uni"                                                                                @.>;
发送时间: 2024年1月24日(星期三) 下午4:47
@.>;
@.@.>;
主题: Re: [Moonofweisheng/wot-design-uni] 关于selectPicker 在columns异步耗时操作的时候，在modelValue监听去设置showValue，并不是太合理，会出现showValue为空的情况 (Issue #167)
昨天有在看，不过不太清楚你具体产生问题的场景，提供一下复现demo把，我用来调试。
…
------------------ 原始邮件 ------------------ 发件人: "Moonofweisheng/wot-design-uni" @.>; 发送时间: 2024年1月24日(星期三) 下午4:12 @.>; @.>; 主题: Re: [Moonofweisheng/wot-design-uni] 关于selectPicker 在columns异步耗时操作的时候，在modelValue监听去设置showValue，并不是太合理，会出现showValue为空的情况 (Issue #167) 大佬，这个能修复吗。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.>
场景很简单比如，一个表单，在onLoad 中 selectPicker 的columns数据是通过接口或者延时进行添加数据的，延时可以设置长一点。在columns还没有返回的时候，依然是空的时候，就通过v-model绑定selectPicker 的值的时候，他是不回填的。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

demo.zip

authors: BeginnerDone Moonofweisheng

repo: Moonofweisheng/wot-design-uni


text:
The macos x86 version prompts for a bad CPU type.

I checked the .github, and found that macos' x86 platform compilation uses macos-lastest.
Currently macos-latest defaults to the arm64, not x86, can you confirm?
Relevant information:
https://github.com/actions/runner-images#available-images
https://github.com/actions/runner-images/issues/9741
https://www.reddit.com/r/github/comments/1cjiat4/is_github_actions_macoslatest_now_only_armbased/

I checked the .github, and found that macos' x86 platform compilation uses macos-lastest. Currently macos-latest defaults to the arm64, not x86, can you confirm?
Relevant information: actions/runner-images#available-images actions/runner-images#9741 reddit.com/r/github/comments/1cjiat4/is_github_actions_macoslatest_now_only_armbased

You are right thank you for letting me know!

There is some odd thing with the x86 runner I can't use it lmao.

Will there be any problems if change the x86 compilation platform to macos-13?

I am unsure I'll have to test it

Thank you for your work.
If that doesn't fix it, I suggest removing the x86 version from the release to prevent misunderstandings.

Thank you for your work.
If that doesn't fix it, I suggest removing the x86 version from the release to prevent misunderstandings.

Yes, I will remove it for now until I get it sorted.

It seems like macos-13-large this is just ridiculous

@0x7a7a can you confirm the x86 build works for you?
https://github.com/MordechaiHadad/bob/actions/runs/10791225725/attempts/1

@0x7a7a can you confirm the x86 build works for you?
MordechaiHadad/bob/actions/runs/10791225725/attempts/1

Sure, I tested it and it works.

repo: MordechaiHadad/bob


text:
Question

About this:
sequenceDiagram
autonumber
Seller->>Mostro: I want to sell Bitcoin
Mostro-->>LN-Node: Give me a hold invoice
LN-Node->>Mostro: Hold invoice
Mostro->>Seller: Please pay this hold invoice
Seller->>LN-Node: Paying through lightning
LN-Node-->>Mostro: Payment received
Mostro-->>Seller: Thanks! talk to buyer now
I'm a little bit confused about why we can not perform p2p communication directly with LN-node, but build on top of the nostr protocol.

I'm a little bit confused about why we can not perform p2p communication directly with LN-node, but build on top of the nostr protocol.
Hi, what does it mean? Can you explain more?

authors: Catrya cymqqqq


text:
UART-PC communication problems with VMs

There is difficulty communicating with the Neblina when going through UART and communicating from a virtual machine.

The problem is not link to Neblina firmware but rather the pyserial package used to communicate with UART over python. Further investigation is required to precisely pinpoint the problem.

authors: acourt souellet-ms

repo: Motsai/neblina-python


text:
Add UTC support

Fixes #177

Adds support to show UTC timestamps
Option to enable/disable UTC timestamps
Open link to moment.js in a new browser window/tab

To check:
In  line 315 event.preventDefault(); was outcommented, see #177 my comment.

Friendly reminder

Friendly reminder

repo: Mottie/GitHub-userscripts


text:
Zip code

Hi there
Most of the coutries shows error because of zip code but 100% sure that this zip code is correct
Any idea please

Hi,
If you can show some examples of the coutry code and how you passing it and also what is the response you are getting would be helpful.
Anyway, there are alot of bugs in their endpoint that i faced and haven't solved yet, So here is a quick solution, you can either dont pass it (if its not required then it will be alright) or you can pass this string "00000", anyway this is not recommended but if you are on dev or test environment this will help you.
Thanks

Thanks for the quick reply
Please check the links below for full request details
https://paste.ofcode.org/sEwZFiGQpGPj3whG5zMyHN
https://paste.ofcode.org/ju4cJ8rv5BE2iCuzDGMhr7

You're right, it's confusing, but anyway, this error is actually from Aramex's side maybe they don't have this city on their end, or maybe they didn't match the zip code with the city.
Anyway here's an advice, you can call the fetchCities method and make sure that this city is supported or not, and ignore the Zipcode in validateAddress method.
I wish I could help you, but there is nothing I can do because this error is not from my end, anyway I will keep this issue open for the community hoping someone could find out the actual reason behind their issue.

Hello,
I am already gettting cities list based on country using fetchCities method also the zipcode is required in validateAddress method , I attached the response without zip code. it's displaying error message "Address - Invalid Zipcode" , so it's look like zip code is required in validateAddress method.
Sample Code :
$validateAddress = Aramex::validateAddress([
'line1' => 'Av. Gaspar de Villarroel E3-83, Quito 170513, Ecuador',
'country_code' => 'EC',
'city' => 'Quito',
]);
dd($validateAddress);

Yes, it is required, but sometimes when you don't pass the zipcode, they accept the request especially for the countries which has areas that don't have a zip code. like UAE for example, this country doesn't have a zip code, so you either pass 00000 as a zip code or you can pass zip code as an empty string (not 100% percent sure), but anyway in all cases, those errors are sent by their end as a response for the APIs calls from the SDK.
I hope this helps you.

Does it work in the localhost ?
when I use rate calculator, I am getting as below
SoapFault SOAP-ERROR: Parsing WSDL: Couldn't load from '<?xml version="1.0" encoding="utf-8"?><wsdl:definitions name="Service_1_0" targetNamespace="http://ws.aramex.net/ShippingAPI/v1/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:wsam="http://www.w3.org/2007/05/addressing/metadata" xmlns:wsx="http://schemas.xmlsoap.org/ws/2004/09/mex" xmlns:wsap="http://schemas.xmlsoap.org/ws/2004/08/addressing/policy" xmlns:msc="http://schemas.microsoft.com/ws/2005/12/wsdl/contract" xmlns:wsp="http://schemas.xmlsoap.org/ws/2004/09/policy" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:wsu="http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-wssecurity-utility-1.0.xsd" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" xmlns:tns="http://ws.aramex.net/ShippingAPI/v1/" xmlns:wsa10="http://www.w3.org/2005/08/addressing" xmlns:wsaw="http://www.w3.org/2006/05/addressing/wsdl" xmlns:wsa="http://schemas.xmlsoap.or

authors: Moustafa22 aammotaibi lbabid


text:
Make EntityRef.copy and related things deeper

When copying a component, also make copies of all EntityRefs that it owns (fields in the component marked @Owns). This ensures that if the original component is removed from its entity, the copy is unaffected. This directly affects ComponentMetadata.copy, but as that's used by EntityRef.copy it's affected too.
I don't know to what extent the existing implementation (shallow copying) is relied on. I had a look at places where these methods were called, and the only place I found that looked like this might cause problems is EntitySetDeltaRecorder, which stores copies of components as they're changed to do some sort of persistence thing with them. I don't really understand what's going on there, so any information on whether this is likely to be a problem would be appreciated. I think that it wouldn't break, but might become less efficient.

It turns out that the usage in EntityDeltaSetRecorder would have been a problem, so I made both the deep and shallow versions available as options. Now the only change to existing behaviour is that EntityRef.copy is deep by default, but that already had a comment that it was incorrect in the case of entities which own other entities.

There is a good chance that in the near future this code will (should) be shared with libraries like TeraNUI (which currently has its own copy of the ClassLibrary-CopyStrategy library), which will have no knowledge of entities. It would be great if we could find a way to do this only via a custom CopyStrategy or some other injectable mechanism that doesn't require us to modify the base ClassLibrary-CopyStrategy library. It's not something we have to do right now, but definitely something we might have to address in the future.

The most sensible rearrangement I can think of that wouldn't require references specifically to EntityRef in these classes would be to put CopyStrategy back the way it was, and put an additional CopyStrategy field in ComponentFieldMetadata (a subclass of FieldMetadata, which already stores a CopyStrategy) that does the deep copy version. It's not clear that changing this to that would be sensible at this stage before it's actually necessary though.

@4Denthusiast That sounds like a decent alternative; if it seems cleaner than the current approach then it might make sense to go for the change right now, but if not we can leave things as they are. It's your decision!

@eviltak I'm actually pretty unsatisfied with the current approach, so I think I will change it.

I've now basically re-written the whole PR so that the deep-copy version is only available on Components, which is the only case where it was useful anyway, and the existing interface isn't changed, just alternative versions of methods are added instead.

repo: MovingBlocks/Terasology


text:
fix: concurrent modification error in update method

Contains
Fixes #2469
How to test
Make a deer fall off a cliff
Outstanding before merging
Should check for null pointers in other areas of the code, I didn't notice any while testing.

Had to edit the two empty lines that I forgot to delete, that's why I closed and reopened this PR.


text:
build(PC): move modules back off the classpath so the sandbox works

Changing facade's dependencies so all the modules still get their dependencies without being on the classpath.
Adds a new moduleCache directory that automatically-resolved jars get put in, kept separate from the modules/ source directory.
Fixes #4375.
How to test
Brief description of how to test / confirm this PR before merging
Outstanding before merging

[x] merge parent #4347. :information_source:  Until then, this diff might help: https://github.com/MovingBlocks/Terasology/compare/chore/tidy-dist-tasks...fix/sandboxed-module-path
[ ] re-integrate serverModules
[ ] build module cache as an output of :modules project instead of defining the task in :facades:PC
[ ] Still have to adjust the wiki doc

Current status: The provideModuleDependencies task is a good proof of concept of how to do the jar-gathering thing without the DependencyResolutionListener hooks. Works nicely under gradle. IntelliJ doesn't realize it's supposed to build the modules at all.
It might be worth looking up how IntelliJ does make its dependency graph out of the gradle projects. I'm assuming it's only checking the main runtimeClasspath or something like that, so when we put the module dependencies in a non-main gradle Configuration it doesn't take them into account.
I guess we can probably come at this from two directions: the gradle side or the IntelliJ side.
From the IntelliJ side, we could edit the run configs so they have to do some additional build-the-modules thing before they start.
From the gradle side, we could include the modules in main runtimeClasspath, but then have the exec tasks use some other classpath without the modules?
I spent a while rummaging around in the intellij-gradle sources. I am far from understanding how it works, but this test suite is informative: https://github.com/JetBrains/intellij-community/blob/211.5361/plugins/gradle/testSources/org/jetbrains/plugins/gradle/importing/GradleDependenciesImportingTest.java
The tests include non-default Configurations and SourceSets, so maybe we don't need to put everything in main after all. In which case it's a matter of how we declare the dependency on that moduleCache, or it's our use of platform. (I don't see any test cases using platform there.)

Going to also link this to #4367 for relevance, since I guess if we have a different module directory that might be a good place to direct the in-game module downloader to in some fashion when running from source, especially if it can sort out dependencies (am realizing I never tried using the in-game downloader to grab something with dependencies - no idea what would happen then)
As for testing - ran this branch with gradlew distForLauncher in a workspace without modules, was then able to run the distributed game fine, very smooth - even if that just tests the dists overall. Added all Omega modules as jars and game still ran, although naturally something crashed because JOML ;-)
Maybe we could get TutorialMinimalEngineDemo some quick love in the near term, only two errors or so in there, and it is my quick goto when moduleland is broken. But me trying to bumble through a fix in the narrow little window I found tonight didn't really go anywhere good. So actually testing from source or breaking the sandbox I haven't been able to do just yet.
(Not that TMED would let me test that without hacking in a simple violation in its module - even Sample needs some sort of world and an inventory to be able to retrieve and trigger the crash blocks ...)

I merged #4347, which makes the diff of this vs develop a more manageable size.

There have been some small changes since Cerv's last review, but I'm confident enough in them to move ahead without a re-review. I'll at least wait for CI to finish. It would be way too embarrassing if one of my "small" changes broke that.
Test Plan

groovyw get CoreSampleGameplay Explosives Sample # note this is not recurse
./gradlew clean extractConfig extractNatives
launch game. One of

for testing gradle:

./gradlew game

for testing IntelliJ:

Click the 🔄 Reload All Gradle Projects button in the Gradle Tool Window. If you don't do this (or restart IntelliJ) after checking out new things to modules/, it might not have recognized them.
Run TerasologyPC configuration

assert game starts
assert Advanced Game Setup screen shows CoreSampleGameplay and Sample modules and all their dependencies, and they are not error-color-red.
activate CoreSampleGameplay and Sample, start a new game
get the gun from Explosives (it's in the second chest CoreSampleGameplay puts in the inventory of new players), make sure it has its shoot-and-destroy-block effects. (There's a possible failure condition where the module loads assets but not code, in which case you can have a gun that doesn't shoot.)
give ReallyCrashGameBlock 1 in the terasology console
place the ReallyCrashGameBlock
assert the game really crashes when you Use it.

Troubleshooting
If things don't go as expected, a few things to check are:

did Sample and Explosives actually build their code?

fdfind -Ig '*.class' modules/

did the modules cache get filled?

ls cachedModules  # you should see .jar files for dependencies here, like CoreRendering and Inventory. There should not be jars for modules we have source for in modules/, or for non-terasology-module things.

For IntelliJ: look in Project Structure / Modules / facades/PC/main / Dependencies and see if our source modules are included in that list, e.g. Terasology.modules.Explosives.main.

authors: Cervator keturn


text:
Added check for insufficient space

Addresses #390
The launcher now checks if there's sufficient space in the download directory before downloading the zip from Jenkins. It also sends a warning message that space is running low if the available space is less than twice the size of the zip file.
As for the warning sign, I haven't added it in first as I'm not sure what would be the best approach:
I was thinking about the download icon turning yellow whenever the space is low, but that would involve setting the style using code so I'm not very sure if it's ok. Or would it be better if something like this appeared in the lower left corner as a sign? For the threshold, I'm thinking about maybe when the available space goes below 200MB?
Let me know what you think 🙂

Can one of the admins please verify this patch?

Took a look into #386 as well and seems it's just a simple issue of the launcher not updating. Seems to have been fixed by adding the update calls.

@GooeyHub: add to whitelist
@iojw: thanks! @skaldarnar mentioned he'd be able to look at this tomorrow. Feel free to poke at other stuff until then :-)
I personally like the idea of the warning icon appearing somewhere, then if you click it (or hover over?) it could give you some details of the warning.
Maybe an ideal approach would be adding a tooltip summary of the warning as you hover over the warning icon, and if applicable clicking the icon would open some relevant UI element.
Over in engine land with MovingBlocks/Terasology#1402 there is a feature request for something similar, but much bigger, on trying to better indicate in the game UI when warnings or errors have been issued in the log. The launcher has the nice log tab right in the UI. Could be neat to have something similar in the game, with a line per "warning" of some sort, and the icon could open that screen.

Hooray Jenkins reported success with all tests good!

Cool! based on @Cervator's comments, I managed to come up with this:

Clicking on it takes you to the Log tab, where you can see a logged warning regarding the low space. Also removed the warning dialog that appears when you click the Download button since the warning sign already accomplishes the same role. I was thinking that this could also be extended to other types of warnings as well in the future.
This is my first time with JavaFX so let me know if there's anything wrong with the FXML and all that. If it's alright, I'll still need to add the image source to the credits page 🙂.

Hooray Jenkins reported success with all tests good!

Merged with minimal changes.

@skaldarnar: Thanks! 😄

Belated "aw man sweet!" and reminder to please do this for the game itself sometime :D

authors: Cervator GooeyHub iojw skaldarnar

repo: MovingBlocks/TerasologyLauncher


text:
Add ability to specify JSONEncoder while using requestJSONEncodable task.

Hi,
I have added a posibility to specify JSONEncoder while using .requestJSONEncodable task type. JSONEncoder allows to set some encoding behaviours like default date format for encoded json. If encoder is not specified it will just user JSONEncoder class.

Codecov Report

Merging #1495 into master will decrease coverage by 0.76%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1495      +/-   ##
==========================================
- Coverage   88.41%   87.65%   -0.77%
==========================================
  Files           5       24      +19
  Lines         164     1053     +889
  Branches        0       96      +96
==========================================
+ Hits          145      923     +778
- Misses         19      128     +109
- Partials        0        2       +2

Impacted Files
Coverage Δ

Sources/Moya/MoyaProvider+Internal.swift
92.73% <100%> (ø)

Sources/RxMoya/MoyaProvider+Rx.swift
93.33% <100%> (+1.66%)
:arrow_up:

Sources/Moya/Endpoint.swift
90.54% <100%> (ø)

Sources/Moya/URLRequest+Encoding.swift
90.9% <100%> (ø)

Sources/ReactiveMoya/MoyaProvider+Reactive.swift
93.47% <100%> (+1.58%)
:arrow_up:

Sources/ReactiveMoya/SignalProducer+Response.swift
90.38% <0%> (-0.53%)
:arrow_down:

Sources/RxMoya/Single+Response.swift
100% <0%> (ø)
:arrow_up:

Sources/Moya/Plugin.swift
75% <0%> (ø)

Sources/Moya/Plugins/AccessTokenPlugin.swift
100% <0%> (ø)

Sources/Moya/Plugins/NetworkLoggerPlugin.swift
91.25% <0%> (ø)

... and 19 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 268b4c6...7798dee. Read the comment docs.

authors: codecov-io piotrbernad

repo: Moya/Moya


text:
Transfers to Moya organization

Little odds and ends. I'm sure we'll be fixing these for a few weeks to come :)

Not a contributor/maintainer but it looks good :ok_hand:

Well, that's fixed. Merge away!

Ha! :bow:
Much appreciated

@esttorhe you forgot to merge!

:bow:

i thought I clicked the button but apparently I did not ¯\_(ツ)_/¯
Fixed that though

authors: ashfurrow esttorhe


text:
mapImage uses implicitly unwrapped optional.

A small change, not breaking. Adds compatibility with RxCocoa's rx_image property on UIImageView.

Cool, this makes sense. :+1:

authors: ashfurrow orta


text:
Reloading UI "breaks" Dynamic Pitch causing it to do nothing

Currently experiencing an issue where reloading the UI or relogging (log out and back in on a character) causes all Dynamic Pitch settings to be ignoring and keeps the character centered on the screen.
Only a full game reboot fixes the issue.

Hi! So to reproduce this, you have to do this?

start the game, log in (everything is fine)
/reload
pitch not working any more

Please enter this into the console while pitch still works and when it does not work any more:
/run print(GetCVar("test_cameraDynamicPitch"), GetCVar("test_cameraDynamicPitchBaseFovPad"), GetCVar("test_cameraDynamicPitchBaseFovPadFlying"), GetCVar("test_cameraDynamicPitchBaseFovPadDownScale"), GetCVar("test_cameraDynamicPitchSmartPivotCutoffDist"))
Is the output the same in both cases?

Hi! So to reproduce this, you have to do this?

start the game, log in (everything is fine)
/reload
pitch not working any more

Correct.

Please enter this into the console while pitch still works and when it does not work any more:
/run print(GetCVar("test_cameraDynamicPitch"), GetCVar("test_cameraDynamicPitchBaseFovPad"), GetCVar("test_cameraDynamicPitchBaseFovPadFlying"), GetCVar("test_cameraDynamicPitchBaseFovPadDownScale"), GetCVar("test_cameraDynamicPitchSmartPivotCutoffDist"))
Is the output the same in both cases?

Yes, output is the same (settings used: 1 0.4 0.5 1 10)

Sorry, but I cannot reproduce this.
Also, if the cvars are set correctly (as confirmed by your print output) then there is nothing else, DynamicCam can do.
One last thing you should try is to disable all Addons (including DynamicCam).
The pitch cvars should still be set afterwards and you should get the warning window that you are using experimental variables. Otherwise, you can always set them via the console command:
/run SetCVar("test_cameraDynamicPitch", 1) SetCVar("test_cameraDynamicPitchBaseFovPad", 0.4) SetCVar("test_cameraDynamicPitchBaseFovPadFlying", 0.5) SetCVar("test_cameraDynamicPitchBaseFovPadDownScale", 1) SetCVar("test_cameraDynamicPitchSmartPivotCutoffDist", 10)
If you still experience your issue then (even with all addons disabled), you could file an in-game bug report. But be polite, we don't want Blizzard to "fix" this by taking away these "test_" cvars. :-)

I just did a great deal of testing. On a starter account I had, there were no issues at all.
I tried cleaning my entire WTF folder and it still did not work on my primary account but worked fine on the starter still.
Could the Shadowlands upgrade itself do something to mess with the camera workings? It is the only thing I can think of that is different between the two accounts.

That's a real mystery. You could try:
/console ActionCam off
(https://wow.gamepedia.com/CVar_ActionCam)
and then again.
/run SetCVar("test_cameraDynamicPitch", 1) SetCVar("test_cameraDynamicPitchBaseFovPad", 0.4) SetCVar("test_cameraDynamicPitchBaseFovPadFlying", 0.5) SetCVar("test_cameraDynamicPitchBaseFovPadDownScale", 1) SetCVar("test_cameraDynamicPitchSmartPivotCutoffDist", 10)
Or you could even do this:
/console cvar_default
(https://wow.gamepedia.com/CVar_cvar_default)
But you will have to go through all your settings again after this.
Or make a backup of your config.wtf file.
But these cvars should be global for all your accounts. So it would be odd if this changes anything.

Ah, I found the issue thanks to your links. I didn't know about how cvars were also server synced so I took a look through all of them that weren't default using another addon.
The issue was my "cameraView" cvar which was apparently set to 0 on my primary account whereas the default is 2. It might have happened a long time ago when I was messing around with the fancier views of DynamicCam and close-ins, which I don't use anymore but I am not sure.
Thanks for the quick feedback and help resolving this =D

Glad that you got it fixed.
But looking at https://wow.gamepedia.com/CVar_cameraView I don't understand how this variable should have had any effect on your issue. Can you reproduce it again, setting cameraView back to 0?

I did indeed reproduce it when I fixed it.

Cause of CVar modification found, MayronUI is changing the CVar unnecessarily upon AFK or entering combat.
Bug has been reported at https://github.com/Mayron/MayronUI-Official/issues/193#issue-804407890

I just tried it myself and it is really like you described it:
Setting cameraView to 0 and reloading breaks pitch until the client is restarted.
Thanks for finding this out! I will put a mechanism into DynamicCam that prevents this variable setting.
What is this variable every used for anyway??
I think this would also be worth a bug report to Blizzard.
Have you already filed one or should I do it?

I just tried it myself and it is really like you described it:
Setting cameraView to 0 and reloading breaks pitch until the client is restarted.
Thanks for finding this out! I will put a mechanism into DynamicCam that prevents this variable setting.
What is this variable ever used for anyway??

From what I could see on wowpedia about the cameraView, it seems to be a mechanism to store different setups and quickly switch between them. 1 is reserved as first person view, 2-5 seems to be user variable (but unused in game client, maybe for addons?). My guess would be that while 0 is not mentioned, it could be the hardcoded view for camera centering when you enable that setting in the client. Pure speculation though.

I think this would also be worth a bug report to Blizzard.
Have you already filed one or should I do it?

I have not filed anything. I was unsure how to word it to begin with and I didn't want Blizzard to mess with the code too much.
I would much rather have a semi-working (now fully working) DynamicCam than no DynamicCam =)

From what I could see on wowpedia about the cameraView, it seems to be a mechanism to store different setups and quickly switch between them. 1 is reserved as first person view, 2-5 seems to be user variable (but unused in game client, maybe for addons?). My guess would be that while 0 is not mentioned, it could be the hardcoded view for camera centering when you enable that setting in the client. Pure speculation though.
The storing and restoring of the views is done with SaveView() and SetView(). But why would one store which was saved last?

Is only cameraView = 0 causing the error or any value other than 2? Have you tried this?

From testing:
0. Centered. If zoom is non-default, zooms in quadratically quickly to roughly halfway, then linearly slowly the rest.
1-5. Pitched (correctly). Zoom as above.
6-7. Centered. Zoom as above.
8. Unknown, most likely centered. Zoom is locked very close (upper body visible only) and camera rotation is limited to front hemisphere only.
9+. Defaults back to 8 on reload.

Hm, I think you are not supposed to set the cameraView using SetCVar() at all.
It is set automatically whenever you use SetView({1,2,3,4,5}) to the number of the view you just set.
The error you discovered seems to occur, whenever cameraView is not {1,2,3,4,5}. Just tried it with 6 and it is the same as with 0.

Hm, I think you are not supposed to set the cameraView using SetCVar() at all.
It is set automatically whenever you use SetView({1,2,3,4,5}) to the number of the view you just set.
Makes sense. Only reason why I did it using SetCVar/console was that it was the way MayronUI did it. It is also the only place in the entire MUI code where they change that CVar, so it is most likely a bug or old workaround for their AFK camera effect.

The error you discovered seems to occur, whenever cameraView is not {1,2,3,4,5}. Just tried it with 6 and it is the same as with 0.
Yup, my list above your reply is the cameraView numbers I tested, so your test matches what I discovered.

In the end it seems that it is a bug combination of a CVar that allows values out of valid range and an addon that changed the number to be out of scope, the combination of which broke ActionCam functionality.

authors: LudiusMaximus NopeDK

repo: Mpstark/DynamicCam