Marketplace README fails because Bob has no OCEAN

Describe the bug
I was going through the READMEs in sequential order. Not using ocean.py github, but installing the library (how users do it). Fresh install of barge.
Everything works fine, up until the marketplace README.
It fails at the step where Bob tries to buy datatokens: tx_result = ocean.fixed_rate_exchange.buy_dt().
The specific error is: web3.exceptions.ContractLogicError: execution reverted: VM Exception while processing transaction: revert ERC20: transfer amount exceeds balance
The problem: Bob doesn't have OCEAN. Further details below.
To fix: barge launching is supposed to put OCEAN into Bob's address. It's clearly not. We've seen this issue before. Looks like something got broken again.
Details
Here's the full call for Bob trying to buy datatokens:
tx_result = ocean.fixed_rate_exchange.buy_dt(
    exchange_id=exchange_id,
    datatoken_amount=ocean.to_wei(20),
    max_base_token_amount=ocean.to_wei(50),
    consume_market_swap_fee_address=ZERO_ADDRESS,
    consume_market_swap_fee_amount=0,
    from_wallet=bob_wallet,
)

Hypothesis: Bob doesn't have enough OCEAN.
 )
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/ocean_lib/models/fixed_rate_exchange.py", line 153, in buy_dt
    return self.send_transaction(
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/ocean_lib/web3_internal/contract_base.py", line 216, in send_transaction
    return contract_function.transact(
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/enforce_typing/decorator.py", line 29, in wrapper
    return func(*args, **kwargs)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/ocean_lib/web3_internal/web3_overrides/contract.py", line 71, in transact
    gas = cf.estimateGas(tx)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/contract.py", line 1039, in estimateGas
    return estimate_gas_for_function(
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/contract.py", line 1620, in estimate_gas_for_function
    return web3.eth.estimate_gas(estimate_transaction, block_identifier)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/eth.py", line 855, in estimate_gas
    return self._estimate_gas(transaction, block_identifier)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/module.py", line 57, in caller
    result = w3.manager.request_blocking(method_str,
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/manager.py", line 198, in request_blocking
    return self.formatted_response(response,
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/manager.py", line 170, in formatted_response
    apply_error_formatters(error_formatters, response)
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/manager.py", line 70, in apply_error_formatters
    formatted_resp = pipe(response, error_formatters)
  File "cytoolz/functoolz.pyx", line 666, in cytoolz.functoolz.pipe
  File "cytoolz/functoolz.pyx", line 641, in cytoolz.functoolz.c_pipe
  File "/home/trentmc/code/play/venv/lib/python3.10/site-packages/web3/_utils/method_formatters.py", line 561, in raise_solidity_error_on_revert
    raise ContractLogicError(f'execution reverted: {response["error"]["message"]}')
web3.exceptions.ContractLogicError: execution reverted: VM Exception while processing transaction: revert ERC20: transfer amount exceeds balance

There are three possible ways to solve:

ensure that Barge is properly minting OCEAN into the first N eth addresses of ganache, and that Bob is among those addresses
In the marketplace README itself, if Alice has OCEAN, then get Alice to send some OCEAN to Bob.
Alice mints a fake token (not OCEAN) then sends some to Bob.

(1) is cleanest, I recommend it, though it has been finicky, we've had this issue before.

Towards solving, using approach of (1), the following may help. It's from the developers.md README. Maybe this envvar needs to be set somewhere before barge is started? Maybe barge doesn't set this anymore?
#needed to mint fake OCEAN for testing with ganache
export FACTORY_DEPLOYER_PRIVATE_KEY=0xc594c6e5def4bab63ac29eed19a134c130388f74f019bc74b8f4389df2837a58

Previously we used mint_fake_OCEAN function at the beginning of every flow, for that we needed to export that env var. I have added back in data-nfts-and-datatokens-flow.md to be more visible. I have provided the fix in #908.

Barge didn't and don't fill the accounts with OCEAN tokens. That address was set for mint_fake_OCEAN() function.

Reopening.
This envvar shouldn't be added before it's needed.
Adding it in the first README - where it's not needed, adds to the complextity of that README. We want to keep the user flow as simple and clean as possible.
Please fix. How:

revert the last change
put the change in the markteplace README. Not sooner.

I'm working on #900 - UX tweaks for READMEs.
If the changes I recommend in the comment above are done, then it will cause conflict with my work on #900.
Therefore I will make those recommended changes as part of #900.
Therefore I can close this issue.
In the future let's always remember UX when we're fixing bugs, ie to avoid introducing unwanted complexity.


Bug report: spo listitem batch add gives errors when columns contain json objects with multiple properties

Priority
(Low) Something is a little off
Description
i have an issue with bulk import for m365 spo:
When i import this CSV:
  Title,FormJSON,Created
  Test 123,"{""test"":""123"",""test2"":""234""}",2024-02-01 12:23:34

with
  m365 spo listitem batch add --filepath <CSV file> -u <sharepoint site> -t <listname> --verbose

it gives me an error that the Created column has an invalid format.
This seems due to the comma inside the JSON string because when I remove the test2 property, it works fine.
Seems the csv importer doesn't handle the quotes correctly in case of double quoted strings.
as a workaround i added this to parseCsvToJson in formatting.js:
        let lines = s?.replace(/""/g,'\xFD').split('\n');

and under if(isNaN(value)..)
        obj[key] = lineMatch[index]?.replace(/\xFD/g,'"');

which seems to solve my issue for now..
Steps to reproduce
as described above
Expected results
Getting a DONE message for correctly imported list of items
Actual results
Writing away 1 items.
{"error":"Creating some items failed with the following errors: \r\n- Line 2: Created - You must specify a valid date within the range of 1/1/1900 and 12/31/8900."}
Diagnostics
No response
CLI for Microsoft 365 version
"v7.4.0-beta.d2ff47d"
nodejs version
v20.9.0
Operating system (environment)
Windows
Shell
PowerShell
cli doctor
No response
Additional Info
No response

Thanks @brugh for reporting this issue. It's good to see you also found a workaround for it in the codebase. Now we'll need to research the effect of applying this regex. Are you interested in working on this?

I noticed we try to parse the CSV contents using our own code. Maybe we shouldn't reinvent the wheel here and use a dependency for it.

That would also be a good approach, instead of diving into all those regex syntaxes, use a package. Just had a quick glance and there appear to be quite a lot available, so we can compare a few different ones.

Awesome to hear that @brugh, I would say that we start with researching which npm package could be interesting to use to replace our custom formatting logic. With a quick glance, papaparse seems to have a lot of potential.
If you need some help with fetching the code locally and getting it running, you can always have a look at our contribution docs: https://pnp.github.io/cli-microsoft365/contribute/contributing-guide. Here you should find a detailed guide with what resources you can use to get the CLI for M365 up and running locally.
Let us know if you need any help.

I would use https://www.npmjs.com/package/csv-parse .. it has millions of weekly downloads, is well maintained and seems very capable. And it doesn't seem too dependent on other libraries. I'm always weary of too many of those ;)

Good find, that is indeed a very dependent package we could use. It also seems to be well-maintained with regular updates which is also a plus. I would say, let's use this package and see what the effects are on different scenarios.

i grabbed the git files locally and changed the parseCsvToJson in utils/formatting.ts to:
  import { parse } from 'csv-parse/sync';
...
  parseCsvToJson(s: string, quoteChar: string = '"', delimiter: string = ','): any {
    return parse(s, {
      quote: quoteChar,
      delimiter: delimiter,
      columns: true,
      skipEmptyLines: true
    });

and ran it with some troublesome csv files. so far it returns the proper objects perfectly with all kinds of quoting involved. Seems like a viable solution to me.

Awesome @brugh! Let's open up a PR for this so we can get it. Just had a quick glance add the docs for csv-parse and it appears that quote and delimiter already default to the char provided. But those are more finale details for the PR process. 😄 Nice find!

Are you still working on this @brugh?

ah, right, good reminder ;)
yes, I'll open a PR


Add support to Mux.com signed policy

Codecov Report

Merging #5 (199b9e7) into main (63fb742) will decrease coverage by 0.68%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
- Coverage   56.93%   56.25%   -0.69%
==========================================
  Files           5        5
  Lines         137      144       +7
==========================================
+ Hits           78       81       +3
- Misses         58       62       +4
  Partials        1        1

Impacted Files
Coverage Δ

api/main.go
0.00% <0.00%> (ø)

api/video.go
77.64% <100.00%> (+0.81%)
:arrow_up:

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 63fb742...199b9e7. Read the comment docs.


Add --print-config current

I would like my vim to be able to automatically detect the indent size based on rustfmt.toml. The easiest way to do that would be to support --print-config current PATH which loads all configuration files normally, as if formatting the file at PATH, and then prints the computed current config to stdout. IDEs with more advanced format-as-you-go features may wish to retrieve other configuration options too.

Closed via #3472.


Ongoing maintenance of this library

What are the ongoing maintenance plans for this library? It looks like the latest release and PR merge was back in January.
There are currently 17 open issues and 11 open pull requests. The demo included with the app doesn't even build (PR #72 fixes this).
I understand that 37signals/Basecamp/HEY has a lot going on. But I work with a bunch of people that rely on this library to power their iOS apps. And it makes for a hard case trying to adopt a library that looks like it might be abandoned soon.
@jayohms, I'd love to help out. If there's any possibility of getting commit access, helping review PRs, or anything else, please let me know! If it helps, I can provide links to blog articles I've written on Turbo Native, past PRs and contributions, my Turbo Native code template, or testimonials from past clients.

Hey @joemasilotti,
Sorry for the slow progress lately. Many of the iOS team members at 37signals are new and haven't had time yet to dig into the library as much as I'd hoped. Our apps depend entirely on turbo-ios, so it's got our full support for the long haul.
I'm aware of your great contributions to the community and I'd love your help! Please email me at jay [at] 37signals.com and we'll figure out what makes the most sense 👍

Thanks Jay! I just sent you an email. Looking forward to hearing back.


Add header authenticator class

I have to consume some weird API's in one of my projects at work. That's why I figured a more general header authenticator class could be of good use. I'm using this one as a custom authenticator in my project right now but I wanted to share it in the case someone else could use it as well.
I couldn't find any dedicated authenticator tests, so I only added the helper method in the AuthenticatesRequests trait to keep it consistent with the rest. If I'm missing anything, please let me know. Feel free to close the MR if this kind of authenticator is not desirable in Saloon.

I tried to reuse this class in other authenticators as well. But I found that it didn't really help with readability:
class TokenAuthenticator extends HeaderAuthenticator implements Authenticator
{

    public function __construct(
        public string $token,
        public string $prefix = 'Bearer'
    ) {
        parent::__construct(
            trim($this->prefix . ' ' . $this->token),
        );
    }
}

Hey @markwalet thank you, this looks like a nice PR!
I was wondering, is this similar to the TokenHeader already provided? I wonder if we could customise that authenticator to have a third $header = 'Authorization' instead maybe?

I mean that's fine by me as well, but if the prefix becomes optional, there's an extra if-statement in that class to remove the extra whitespace when needed. I'm fine with that but you have the chance that it obfuscates the main use-case for the class. Separating them out into multiple classes makes more sense to me personally. Just to give the end-user some more explicit cases to choose from.
But I'm not the maintainer, either way is fine for me. Just let me know what your preference is :)


cant open flycast

Hello, so i tried to run various games in test game offline in fightcade (akatsuki blitskampf, ready2rumble round 1), doesnt boot up,tried deleting emu.cfg didnt do anything.

The issue stems from https://github.com/blueminder/flycast-dojo/commit/90ad0d5570eb04858a3c41f2a69c58c43de1ba89
Flycast Dojo was changed to build against Ubuntu 22.04. It seems as a side effect of that change a chunk of the dynamically linked libraries were updated to versions not currently available in the Freedesktop runtime (most importantly OpenSSL 3.) I don't expect them to be available until Freedesktop runtime 22.08 is released.
Here's ldd output from an old (working) install of Flycast and the new (broken) version:
Old:
    libzip.so.5 => /app/lib/libzip.so.5 (0x00007f0a9f4c3000)
    libssl.so.1.1 => /usr/lib/x86_64-linux-gnu/libssl.so.1.1 (0x00007f0a9f42c000)
    libcrypto.so.1.1 => /usr/lib/x86_64-linux-gnu/libcrypto.so.1.1 (0x00007f0a9f148000)

New:
    libzip.so.4 => not found
    libssl.so.3 => not found
    libcrypto.so.3 => not found

There are two options here:

Build SSL/Crypto libraries manually until Freedesktop 22.08 is available.
Roll back Flycast to ubuntu-latest in the Github Ci until 22.08 is available.

After speaking to blueminder in Discord I'm going to take a quick stab at option #1 tonight, but blueminder and pof are OK to push a Fightcade update with Flycast Dojo built against the older Ubuntu tomorrow.
I'll document here with any progress I make on trying to get the new Flycast to work.

Closing this due to #92

Flycast Dojo 0.4.55

Linux: Revert to Ubuntu 20.04 environment (fixes Flatpak and 20.04 compatibility)

I just installed Fightcade today, but i still have this issue? With libzip.so.4 not being found.

Yep, this is unsurprising. Flycast is going to be broken, I wasn't made aware this update was coming down the pipeline.

@man-of-eel could you please try this test version and let me know if it resolves the issue for you? Won't be able to look at it for a few hours and I'd like to ship this sooner rather than later. https://github.com/flathub/com.fightcade.Fightcade/pull/110#issuecomment-1359998835

Gives me a different error... weird
$ xdg-open fcade://play/flycast/azumanga
$ bwrap: Can't mkdir /app/lib/i386-linux-gnu/GL: Read-only file system
error: ldconfig failed, exit status 256

Could it be because i installed it on a different installation?

Can you try just launching Flycast directly from the Flatpak? flatpak run --command=sh com.fightcade.Fightcade//test should get you a shell, then you can cd into /app/Fightcade/fightcade/flycast and run ./flycast.elf manually to see if it launches.
It's possible having two versions installed simultaneously is causing strange errors, this will test that Flycast itself works as expected (and if it launches I can merge after upgrading com.fightcade.Fightcade.Wine)

flatpak run --command=sh com.fightcade.Fightcade//test gives me the same error, even after uninstalling the stable version.

@man-of-eel could you try the most recent build (here: https://github.com/flathub/com.fightcade.Fightcade/pull/110#issuecomment-1360425294) and let me know if it works?

I just merged the changes thanks to @torao2 doing some testing for me on the Discord. If you continue to have issues related to the Flycast changes feel free to re-open this.


fix - Group chat - Group chat name edited in offline before creating group is not greyed out

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/49163
PROPOSAL: https://github.com/Expensify/App/issues/49163#issuecomment-2349109395
Tests

Go offline
Tap fab -- start chat
Select "add to group" button next to a user and tap next
Tap group name
Rename and save it
Tap create group
Tap header of the group
Verify that the group name menu in the details page is grayed out

[x] Verify that no errors appear in the JS console

Offline tests

Same as above
QA Steps

Same as above

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/ed5dbcc4-865f-4926-abdc-ca64a96151d4

Android: mWeb Chrome

https://github.com/user-attachments/assets/a8324a83-096f-4527-b08f-2120eaf17827

iOS: Native

https://github.com/user-attachments/assets/d5a95cd8-851f-4ff4-b0c6-aaa4c0aaa5f3

iOS: mWeb Safari

https://github.com/user-attachments/assets/732560f7-bb08-4da8-813e-10b51b32ed52

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/7840b775-e73b-4099-8f84-5f0f9d3c2640

MacOS: Desktop

https://github.com/user-attachments/assets/4aefd37b-ac61-441b-b239-87604a8740a4

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


[PennyDreadfulMTG/perf-reports] 500 error at /rotation/ (#12268)

'utf-8' codec can't decode byte 0xc3 in position 5: invalid continuation byte
Reported on decksite by logged_out
Request Method: GET
Path: /rotation/?
Cookies: {'__cfduid': 'd7ae79ba2cdc73a716edb7d9d5391384b1509586117', '_ga': 'GA1.2.901534363.1509754598', '_gid': 'GA1.2.369613643.1514081763', '_gat_gtag_UA_109131120_1': '1'}
Endpoint: rotation
View Args: {}
Person: logged_out
User-Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36
Referrer: https://pennydreadfulmagic.com/
Request Data: {}
UnicodeDecodeError
Stack Trace:
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/home/discord/.local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/home/discord/.local/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "./decksite/cache.py", line 64, in decorated_function
response = make_response(f(*args, **kwargs))
File "./decksite/main.py", line 178, in rotation
view = Rotation()
File "./decksite/views/rotation.py", line 31, in init
self.read_rotation_files()
File "./decksite/views/rotation.py", line 54, in read_rotation_files
name = html.unescape(name.encode('latin-1').decode('utf-8'))

Currently does not repro so closing. We might see it again next season but we'll probably switch to a feed-based legality checker and not see it.


Setting format to 'DD' doesn't disable month view

If I set format option to 'DD', I would expect that the month view would be disabled. However, I am still able to click on the month title and switch views.

This is the correct behavior, as it is still possible to select a day in a different month.
If you want to prevent people from leaving the month, use the min/maxDate to the startOf and endOf the current month (+/- 1 day)

I don't want to prevent people from leaving the month, I just want to disable clicking of the title to show the month view. Users would still be able to get to other months using the left and right arrows.
Basically I'm seeing a formatting issue when I switch to month view so I wanted to disable that view. These images show the issue:
Normal date view

After clicking month title

I know a better solution would be to fix the display bug but in the interim I wondered if there was a way to just disable clicking the title.

There must be some css overwriting the monthview styles. This is not how the picker looks normally. You could disable the code or the link, but the better thing to do would be to fix the styles

I agree, I'll spend some time debugging the styles. Thanks.

FYI, I had set the widgetParent option to the button that I was using to invoke the picker. Removing this option returned all display issues back to normal.

Might I ask how have you set to open the datetimepicker on the button click without widgetParent?
I have a similar structure using Knockout binding handlers and the same thing happens as above

<input type="hidden" data-bind="dateTimePicker: navigation.currentDate, daysOfWeekDisabled: navigation.daysOfWeekDisabled, widgetParent: 'calendar-btn'" />

Another thing, how do you handle when mouse is clicked outside of the calendar?
I tried some workarounds and it works, but it tries

Is mouse up outside of widget parent element to hide it
Is mouse up clicked on the widget parent itself to toggle it
Is mouse up done for specific widget parent, or widgetParent = null/$(body)

The default behavior is correct, but as soon as you set widgetParent to a specific element (like above), it stays open when mouse is clicked outside.

@Bogatinov Honestly at this point I don't recall how I solved the issue and I no longer have access to that codebase to check.


How to use InstantSearch with no UI?

Hello,
I was wondering how Algolia’s InstantSearch operates on non webpage based applications.  My use case is to create a search experience with the Chrome omnibox, for example to search on the MDN.  I only care about the query, which contents comes from the omnibox, and the response, to access and display the candidates in the omnibox.  How to do that?

Hey   @alexherbo2, this can be done using the connectors, you can find more information on that here:
https://www.algolia.com/doc/guides/building-search-ui/widgets/customize-an-existing-widget/js/?language=vue#customize-the-complete-ui-of-the-widgets
https://www.algolia.com/doc/api-reference/widgets/autocomplete/js/#connector
There you will render to your preferred non-dom output

@Haroenv It’s also possible with search and query?

Using the JavaScript API client as you mentioned is what InstantSearch does under the hood. InstantSearch comes handy when you want to build an application and manage the state of the search in your page. From what I understand of your usage, using the API client would be enough.


remove collectives from prerequisites?

the Collective communication episodes probably don't require any prior knowledge of them. Shall we remove it from https://enccs.github.io/intermediate-mpi/#who-is-the-course-for?

@mabraham

Sure!


Bring to head of internal repo

Copybara update to match state of internal repo

FYI @sullivan-sean


[docs] Update site2/docs/functions-quickstart.md

Motivation
While add "Write and run a Go function" section, improve doc on Pulsar Functions: quick start throughout.

Update the the first-person pronoun as the second-person pronoun.
Correct the usage of "Pulsar Functions" and "a Pulsar function". For the usage discussion, see #4507 .
Change the future tense into present tense.
Refine part of structure (heading level).
Refine sentences to be more clear and concise.
Refine some typos.

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? ( docs)

@sijie @wolfstudy Could you please help review it? Thank you.

I've updated the usage of "functions" in this file, if i have any misunderstanding or incorrect usage, feel free to correct it. The principle follows discussion in #4507 :

Pulsar Functions / a Pulsar Function (refer to product/feature specific to Pulsar)
function/functions (refer to common/individual functions)

rerun java8 tests


Editor colors are off since last update

Hello,
I don't know if it's something that broke in the last update, but since it the editor colors at least in the light themes are not conform anymore. I'm back to something that very much looks like the standard logseq light theme.
Anybody experiencing the same thing ?

For the record, I tried to disable all plugins and comment out my own CSS tweaks to be sure this was not coming from this. It did not change anything.

Thanks for the report. Please share,

logseq version
theme version
screenshot of the issue if possible

While at it, please also make sure that the following options are set,

Catppuccin theme variant under Themes
"Light"/"Dark" theme under Settings > General > Theme
"Cancel accent color" under Settings > General > Accent Color

Hello !
Thanks for your answer.

Thanks for the report. Please share,
1. logseq version

0.10.6

2. theme version

0.6.2

3. screenshot of the issue if possible

While at it, please also make sure that the following options are set,
1. Catppuccin theme variant under Themes

2. "Light"/"Dark" theme under Settings > General > Theme

3. "Cancel accent color" under Settings > General > Accent Color

see previous screenshot

Please upgrade Logseq to at least v0.10.7.
Logseq introduced its new design system in v0.10.6 which broke support for nearly all custom themes. After upgrading you should see these two options under accent color, we support both of those.

ok, trying that
Le ven. 26 avr. 2024 à 19:11, Soumik @.***> a écrit :

Please upgrade Logseq to at least v0.10.7.
Logseq introduced its new design system in v0.10.6 which broke support for
nearly all custom themes. After upgrading you should see these two options
under accent color, we support both of those.
Screenshot.2024-04-26.at.10.38.45.PM.png (view on web)
https://github.com/catppuccin/logseq/assets/10350864/39ee8c94-f58d-4d8f-a2d3-c493dce34427
—
Reply to this email directly, view it on GitHub
https://github.com/catppuccin/logseq/issues/42#issuecomment-2079773939,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABSPF3N6GJSEYIHRBGPYTLY7KDDBAVCNFSM6AAAAABG24POHSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZG43TGOJTHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Updated to 0.10.8 (that's what is the unstable of Nixos). It's indeed working now. Thank you very much


[NO SYNC] Added Enemy Health Bar scene

Changes

Added new scene to test enemy health bar mechanic
Used placeholder images for the healthbar
Theoretically, the hp bar follows the camera angle

Testing

To test press space once the scene has loaded to remove health

Illustration

TODO

Add mirror sync

please replace .gitignore with the one on main branch, run git rm --cached git ls-files -i -c --exclude-from=.gitignore, if nothing happens run git rm git ls-files -i -c --exclude-from=.gitignore

I will review this PR but you are issuing a merge to the wrong branch, try development


Add motion_draw_disabled render_mode to Spatial Shaders.

This PR adds a new motion_draw_disabled render_mode that allows materials to skip drawing motion vectors.
This is required for post processing effects to play nicely with TAA. It also can help with the appearance of some translucent materials.
There are currently two methods of doing post processing:

A quad with a vertex shader positioning it infront of the camera.
This works well with the editor camera however with TAA enabled the whole scene appears to shake.
A quad as a child of the players camera.
This solves the shaking issue however the motion vectors are no longer accurate, leading to horrible artifacts.

With this render_mode enabled post processing effects such as this outline get properly antialiased and remain crisp in motion.

TODO: Expose to BaseShader3D

@JFonS any other blockers?

I have now exposed this to BaseMaterial3D.

@EMBYRDEV Could you please rebase against the master instead of merging it into your branch?

@EMBYRDEV Could you please rebase against the master instead of merging it into your branch?

Done!

Is there anything else needing done before looking at getting this merged?

Do you have a test project available to make sure this PR works as intended? This can be a scene with your post-processing shader setup.

@Calinou omg, I put it in my recycle bin last week and never got around to emptying it... lucky. 🤣
Here is a google drive link for the scene in the screenshot.
Both the post processing shader and the smoke on the torches have this render mode enabled. Remove the render mode to see the default behaviour of the engine prior to this PR.

Is there anything else needing done before looking at getting this merged?

Clay needs to give it a review as the rendering lead. I asked and he said that he'd like to take a look.

Rebased :)
@clayjohn any chance this can make it in for 4.2?

Yep, its tagged for "4.2". I just need to give it a review

@EMBYRDEV Are you able to do the changes suggested by @clayjohn?

@EMBYRDEV Are you able to do the changes suggested by @clayjohn?

Happy to look into those changes when I get some free time. Have been incredibly busy the past month or so and currently dealing with post launch support on the game I released last week. No guarantees on precise timing.

Just a note for future reference, after https://github.com/godotengine/godot/pull/81197, we will no longer write motion vectors in the transparent pass. Instead transparent objects will write into the reactivity texture (right now this is just for FSR, but we will extend TAA to support it too). This should allow things to "just work" for post process shaders and other transparents. Once we have https://github.com/godotengine/godot/pull/81197 merged and have had a chance to experiment more, we will need to revisit if motion_draw_disabled is still needed (I suspect it is, but it won't be needed for the original use case anymore, so it is worth revisiting)

@clayjohn is it worth me reworking this PR in the next week or so, or should I wait until #81197 has had some time in the oven?

@clayjohn is it worth me reworking this PR in the next week or so, or should I wait until #81197 has had some time in the oven?

I would save your time! I have a gut feeling this PR is still needed. But I don't have a specific use case in mind other than disabling motion vectors for objects in the transparent pass. So better to wait and see and avoid doing work in case it turns out to be unnecessary.

Just to add some more feedback. I have a lot of problems with motion vectors on transparent materials with TAA.
While using this flag will have the drawback of destroying smaller particles, it is very needed for a lot of effects. TAA often introduces weird artefacts at the edges of transparent particles/quads. Is it planned to also add this flag to the StandardMaterial3D or just just in shaders?

Is it planned to also add this flag to the StandardMaterial3D or just just in shaders?

This PR does expose the setting to StandardMaterial3D too. Just need to get some free time to rebase this PR if @clayjohn believes there is still a use case for this?

Just to add some more feedback. I have a lot of problems with motion vectors on transparent materials with TAA. While using this flag will have the drawback of destroying smaller particles, it is very needed for a lot of effects. TAA often introduces weird artefacts at the edges of transparent particles/quads. Is it planned to also add this flag to the StandardMaterial3D or just just in shaders?

It depends. As of 4.2, we never write motion vectors when using transparent materials. So if your need is just for disabling TAA on transparent materials, then it will be fixed in 4.2 without this PR.
If you are using one of the 4.2 betas, and are still having issues with transparent materials, then this PR won't help you.
If you need to disable motion vectors for another reason, then we can move forward with this PR. Basically this PR is just waiting until someone finds a need for it.

I want to add that this issue with Motion Vectors also applies to more than just post-processing effects. I've been working on a terrain system with polygons that follow the Camera as it moves (in unit steps), the motion vectors used for FSR 2 cause a lot of artifacts when the ground shifts position. I've included videos below to illustrate the issue:
No Upscale + FXAA (baseline, no artifacts):
https://github.com/godotengine/godot/assets/1392440/0254a1d2-8130-4322-97b1-43c24e085935
Upscale (0.5 renderscale) + FSR 2
https://github.com/godotengine/godot/assets/1392440/2f7990ac-7ed6-489f-a4be-cedfb364bb48
As you can see, FSR2 looks great while still.. but in motion these artifacts essentially make it unusable with this terrain technique without the ability to disable motion vectors. So I would love to see this render_mode motion_draw_disabled option added in the future, and would even suggest including an additional motion_draw_custom option in the future. That would allow shaders to output their own screen-space motion vectors explicitly, as any "one size fits all" solution will likely have performance/memory trade-offs and may still be unusable in some situations (such as ray-marched clouds, or other visual effects) where doing motion calculations in the shader is basically the only practical solution.

I want to add that this issue with Motion Vectors also applies to more than just post-processing effects. I've been working on a terrain system with polygons that follow the Camera as it moves (in unit steps), the motion vectors used for FSR 2 cause a lot of artifacts when the ground shifts position.

Can you try this PR out and confirm that it does indeed help with your issue? I'm not convinced that using this shader on a large portion of the screen with an opaque material wont lead to smearing artifacts. Although, with motion vector passthrough, if we ever address https://github.com/godotengine/godot/issues/67332 then the sky might help some of those issues.

Can you try this PR out and confirm that it does indeed help with your issue?
I copied my Godot repo (up-to-date with master) and fetched this PR... however while the new motion_draw_disabled render_mode option is there... FSR 2 itself is not. I'm assuming this PR is from before FSR2 was added into master, but without FSR2 it's hard to test. While TAA does seem to have some subtle artifacts due to motions-vectors... it's not noticeable enough for me to do valid comparison. Certainly no where near the very obvious "fuzzing" FSR2 artifacts.

I tried merging in master.. but there's 5 different conflicts I'm not going to try and resolve myself. If there's an update to this PR with those issues resolved, ping me and I'll try again.

@clayjohn I can confirm that this is still required as of 4.2rc1 With FSR2 enabled the problem isnt as pronounced but if you have FSR disabled and TAA enabled then we do still need this for post processing effects.
I'll take some time soon to rebase this against master to aim for 4.3.

This should now be safe to merge! This paired with the fog_disabled flag solves all the issues with the outline shader 🥰

I have rebased against master and this should be safe to merge now.
As for @clayjohn's suggestion of switching the shader variant instead of using the define, this actually causes a problem as the rest of the pipeline seems to expect the shader to still have the vector in it's output layout.
https://github.com/godotengine/godot/blob/69d9bbeeb37d73dfc6e3e1580a7b1f9d1260cea2/servers/rendering/renderer_rd/shaders/forward_clustered/scene_forward_clustered.glsl#L775-L779
And since we don't check for the disabled flage here it works nicely. If you have any suggestions on how best to handle this then I'll be happy to implement but this seems like the most streamlined solution.

As you can see, FSR2 looks great while still.. but in motion these artifacts essentially make it unusable with this terrain technique without the ability to disable motion vectors. So I would love to see this render_mode motion_draw_disabled option added in the future, and would even suggest including an additional motion_draw_custom option in the future. That would allow shaders to output their own screen-space motion vectors explicitly, as any "one size fits all" solution will likely have performance/memory trade-offs and may still be unusable in some situations (such as ray-marched clouds, or other visual effects) where doing motion calculations in the shader is basically the only practical solution.

I don't think disabling the motion vectors would result in any different result at all in your use case here, it sounds to me like your second suggestion is the one that applies the best: being able to output motion vectors value from the shader. I've suggested as much but it sounds like it'd have to be added by modifying parts of Godot I haven't really looked into yet (shader compiler, the shader template acknowledging the new shader variable, permutations, etc).

motion vectors are not generated for static geometry in the first place and are derived from depth buffer and the camera movement.

Yes I figured, but keep in mind the ground is not actually static, but a mesh which moves along with the camera at distinct unit steps. In between those steps things look fine, it's only for a split second after each "step" (where the ground actually shifts position) where the artifacts occur.
That being said, I agree that ultimately just disabling motion vectors might not fully solve everything here, as other parts (like portions of the grass) will do the same thing as the ground (in terms of following the camera), but also sway with the wind, and so dynamically move each frame. Ultimately having the ability to output correct motion vectors may be the only way to eliminate artifacts (tho for the most part it looks fine until the large snap happens).

Terrain3D has issues with TAA/FSR. Like Phil, our terrain is a clipmap. The meshes follow the camera and the vertices move potentially every frame. I'd like a method to disable motion vectors on the terrain while allowing the user to have them for the rest of their scene. Or a method to clear, overwrite or insert an inverted motion vector so these modes are usable.
Downstream issue
https://github.com/TokisanGames/Terrain3D/issues/302

So I havent tested the new CompositorEffect API but I imagine that removed my usecase for this PR, being PostProcessing shaders that play nicely with AA.
That being said this may still be useful for some particle types where smearing is desired over rippling effects (EG. Small smoke particles & sparks)
As for Terrain3D I have a feeling that enabling this mode would potentially lead to the terrain itself becomign a smeary mess.
@TokisanGames would you be able to give this PR a test to see if this actually solves your issue or if we need a differing approach?

I built this PR rebased off of the 4.2 branch. With TAA enabled on my 3070, motion vectors are mostly calculated correctly, with flickering only in some areas. This PR does disable motion vectors, but it turns my terrain into a blurry mess.
TAA
https://github.com/user-attachments/assets/3c1c3019-8738-4c34-9f00-0d9d211f7434
TAA Motion Vectors
https://github.com/user-attachments/assets/4d1bd372-61d1-4ccd-8bca-f0d1f0cd86c1
TAA Motion vectors disabled
https://github.com/user-attachments/assets/4b9fb8ae-0478-48b8-b8c4-c89e5edfd8e0

Ahh looks liek this isnt the solution to your problem. I think it is still super important to expose the ability to passthrough motion vectors from behind the object as some large billboard sprites like lens flares can cause big problems with TAA but I think in addition to this we need to expose a way to write custom motion vectors from shader.
I'll get this rebased and implement feedback soon so that this can make it in for 4.4 sorry for the delay.


Preparing for ESP-IDF 5 support

The Moddable SDK will move to ESP-IDF 5 in the near future (probably not for the September release, but should be by October). We're currently targeting ESP-IDF v5.1.1.
The target-branch  option to xs-dev's setup " makes it easy to get the Moddable SDK idf-v5 branch to try that out. The ESP32 set-up, however, is still pinned to v4.4.3....
https://github.com/HipsterBrown/xs-dev/blob/b6357ec7e482f3fbf9eb3870872962aae064880a/src/toolbox/setup/esp32.ts#L16
...so the build will fail.
We've discussion having xs-dev get version dependencies from the Moddable SDK. I'm not sure how practical that is to do. But, this is a good test case. ;)
Worst case, we can synchronize on the release of the Moddable SDK that transitions to ESP-IDF 5 an update xs-dev at about the same time.

@phoddie is there an expected timeline for the SDK release to support v5 IDF?
If there's time, I'd like to continue the work on the "platform.json" configuration files support to enable SDK-driven automated setup in xs-dev.
Otherwise, we can add some conditional logic in the ESP32 setup & run code paths to ensure the correct version of the IDF is used based on the Moddable SDK release version.

...is there an expected timeline for the SDK release to support v5 IDF?

The to-do list keeps getting shorter. it looks to be on-track for the October release, so there's some time.

If there's time, I'd like to continue the work on the "platform.json" configuration files support to enable SDK-driven automated setup in xs-dev.

That sound great. If you have something to try, we can put that to the public Moddable repo to try out ahead of a formal release.
FWIW – I'm still not sure how you'll use "platform.json" for this. I'm happy to follow your lead to explore that since xs-dev is the first consumer of the information.

Quick update. IDF v5 support is feature complete and solid. We plan to merge that to our public main branch in the next week or two, so it will have a little time to settle ahead of the October release.

Better late than never on my part 😅 This has been resolved in the latest release v0.28.0. 🚀


UART READ BYTES ERROR (IDFGH-1839)

Development Kit:      [ESP32-DevKitC]

Kit version (DevKitC): [v1]

Module or chip used:  [ESP32-WROOM-32D]

IDF version (run git describe --tags to find it):
v4.1-dev-58-g02c7c3885

Build System:         [Make|CMake]

Compiler version (run xtensa-esp32-elf-gcc --version to find it):
8.2.0

Operating System:     [
LINUX]

Power Supply:         [USB]

Problem Description
I had used UART_NUM_2 event type on esp32 before for reading data of less than 250 bytes and works fine.
But there is a problem in getting exact data of more than 330 bytes and the board started rebooting again and again after reading data.
Expected Behavior
All the data should be copied and the board should not reboot.
Actual Behavior
Problem in getting exact data of more than 330 bytes and the board started rebooting again and again after reading data.
Steps to reproduce

build
2.flash

// It helps if you attach a picture of your setup/wiring here.
Code to reproduce this issue
// the code should be wrapped in the ```cpp tag so that it will be displayed better.
#include "esp_log.h"
#definr BUF_SIZE 1024
uint8_t UART_event_data[350];

void uart_init(void)
{
	uart_config_t uart_config = {
        	.baud_rate = 9600,
        	.data_bits = UART_DATA_8_BITS,
        	.parity = UART_PARITY_DISABLE,
        	.stop_bits = UART_STOP_BITS_1,
        	.flow_ctrl = UART_HW_FLOWCTRL_DISABLE,

    	};
	uart_param_config(UART_NUM_2, &uart_config);
	uart_set_pin(UART_NUM_2, 17, 16, UART_PIN_NO_CHANGE, UART_PIN_NO_CHANGE);
	uart_driver_install(UART_NUM_2, BUF_SIZE * 3, BUF_SIZE * 3, 3072, &uart2_queue, 0);
}
static void uart_2_event_task(void *pvParameters)
{
	if(DEBUG) printf("UART EVENT +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++\n");
	uart_event_t event;
	uint8_t UART_data[2] = {0, 0};
	for(;;)
	{
		uart_flush(UART_NUM_2);
		if(xQueueReceive(uart2_queue, (void * )&event, (portTickType)portMAX_DELAY))
		{
			uint8_t data = 0;
			switch(event.type)
			{
				case UART_DATA:
						uart_read_bytes(UART_NUM_2, &UART_event_data, 334, portMAX_DELAY);
						if(DEBUG) printf("Received data from is ........... \n %s\n", UART_event_data);
						break;

				case UART_FIFO_OVF:
                	   			ESP_LOGI(TAG, "H/W fifo overflow");
                	    			uart_flush_input(UART_NUM_2);
                	   			xQueueReset(uart2_queue);
                	    			break;

				case UART_BUFFER_FULL:
                	   			ESP_LOGI(TAG, "ring buffer full");
                	    			uart_flush_input(UART_NUM_2);
                	   			xQueueReset(uart2_queue);
                	   			break;
				case UART_BREAK:
                	    			ESP_LOGI(TAG, "uart rx break");
                	   			break;

				case UART_PARITY_ERR:
                	    			ESP_LOGI(TAG, "uart parity error");
                	    			break;

                		case UART_FRAME_ERR:
                	    			ESP_LOGI(TAG, "uart frame error");
                	    			break;

				default :
						ESP_LOGI(TAG, "Event raised: %d", event.type);
						break;
			}
			esp_event_loop_delete(uart2_queue);
		}
	}
}

void app_main()
{
uart_init();
xTaskCreate(uart_2_event_task, "uart_2_event_task", 2048, NULL, 6, NULL);

}

// If your code is longer than 30 lines, GIST is preferred.
Debug Logs
Received data from QR Code is ...........
 {
“user_id”:“2",
“entity_id”:“1",
“dev_id”:“1",
“key”:“eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1c2VyX2lkIjoyLCJkYXRldGltZSI6IjIwMTktMDktMDUgMDk6NDI6NDcuNjgwMzE3IiwiZGV2X2lkIjoxLCJlbnRpdHlfaWQiOjF9.xGGpWhBESsqpxnPFflyZ7eQ6pPpXXotadaAAsPVc1Cg”,
“app_id”:“j5l1S4",
“app_time”:“05-09-2019-03-12-47"
}
/home/gohith/WorkForce/esp/esp-idf/components/freertos/queue.c:621 (xQueueTakeMutexRecursive)- assert failed!
abort() was called at PC 0x4008f06f on core 0
0x4008f06f: xQueueTakeMutexRecursive at /home/gohith/WorkForce/esp/esp-idf/components/freertos/queue.c:621 (discriminator 1)

ELF file SHA256: 5d990ea5107040cff6d53d78240913caf67dfb3296d50a590ca5941262b0d91a

Backtrace: 0x4008bf39:0x3ffd00b0 0x4008c2b1:0x3ffd00d0 0x4008f06f:0x3ffd00f0 0x4013d7d5:0x3ffd0110 0x400d5925:0x3ffd0140 0x4008e4b9:0x3ffd0170
0x4008bf39: invoke_abort at /home/gohith/WorkForce/esp/esp-idf/components/esp32/panic.c:157

0x4008c2b1: abort at /home/gohith/WorkForce/esp/esp-idf/components/esp32/panic.c:174

0x4008f06f: xQueueTakeMutexRecursive at /home/gohith/WorkForce/esp/esp-idf/components/freertos/queue.c:621 (discriminator 1)

0x4013d7d5: esp_event_loop_delete at /home/gohith/WorkForce/esp/esp-idf/components/esp_event/esp_event.c:604

0x400d5925: uart_2_event_task at /home/gohith/WorkForce/esp/Work_space/Work/build/../main/kea_beta.c:444

0x4008e4b9: vPortTaskWrapper at /home/gohith/WorkForce/esp/esp-idf/components/freertos/port.c:143

Rebooting...
ets Jun  8 2016 00:22:57

Other items if possible

@Gohith Thanks for reporting this, we will look into it. Thanks.

#definr BUF_SIZE 1024 typo
UART_event_data undefined
uart_flush(UART_NUM_2); why?
esp_event_loop_delete(uart2_queue); what?

Sry uart_event_data was mentioned above, which I forgot to paste (part of
whole code).
And I used uart_flush(UART_NUM_2) for clearing buffer manually and I tried
without that api.
Thanks
On Wed, 11 Sep 2019 at 13:37, negativekelvin<EMAIL_ADDRESS>wrote:

#definr BUF_SIZE 1024 typo
UART_event_data undefined
uart_flush(UART_NUM_2); why?
esp_event_loop_delete(uart2_queue); what?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/4048?email_source=notifications&email_token=AFUEGUUDWKFWV2Z4XNSH5A3QJCRMXA5CNFSM4IVQIMFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6NUTXY#issuecomment-530270687,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUEGUU7GRS7PZKLAVULEL3QJCRMXANCNFSM4IVQIMFA
.

What if you remove  uart_flush(UART_NUM_2); and esp_event_loop_delete(uart2_queue); and increase the stack size?

Getting GURU meditation error.
On Thu, 12 Sep 2019 at 08:55, negativekelvin<EMAIL_ADDRESS>wrote:

What if you remove uart_flush(UART_NUM_2); and
esp_event_loop_delete(uart2_queue); and increase the stack size?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/4048?email_source=notifications&email_token=AFUEGUWIWCOA5525A6SPB2TQJGZBDA5CNFSM4IVQIMFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6QQ2SQ#issuecomment-530648394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUEGUVBI5TEIG3CWKGOR5DQJGZBDANCNFSM4IVQIMFA
.

I got all data from uart and guru meditation error.
On Thu, 12 Sep 2019 at 09:50, Gohith Kanchinatham <
<EMAIL_ADDRESS>wrote:

Getting GURU meditation error.
On Thu, 12 Sep 2019 at 08:55, negativekelvin<EMAIL_ADDRESS>wrote:

What if you remove uart_flush(UART_NUM_2); and
esp_event_loop_delete(uart2_queue); and increase the stack size?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/espressif/esp-idf/issues/4048?email_source=notifications&email_token=AFUEGUWIWCOA5525A6SPB2TQJGZBDA5CNFSM4IVQIMFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6QQ2SQ#issuecomment-530648394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUEGUVBI5TEIG3CWKGOR5DQJGZBDANCNFSM4IVQIMFA
.

Show the log

Received data from QR Code is ...........
{
“user_id”:“2",
“entity_id”:“1",
“dev_id”:“1",
“key”:“eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1c2VyX2lkIjoyLCJkYXRldGltZSI6IjIwMTktMDktMDUgMDk6NDI6NDcuNjgwMzE3IiwiZGV2X2lkIjoxLCJlbnRpdHlfaWQiOjF9.xGGpWhBESsqpxnPFflyZ7eQ6pPpXXotadaAAsPVc1Cg”,
“app_id”:“j5l1S4",
“app_time”:“05-09-2019-03-12-47"
}
Guru Meditation Error: Core  0 panic'ed (LoadProhibited). Exception was unhandled.

Looks like memory corruption, you didn't show all the code so hard to say

Did you delete uart2_queue in esp_event_loop_delete()?  This will be a problem when you call xQueueReceive(uart2_queue,...) again. This should be the cause for the first issue you reported.

Hi, @Gohith
Have you solved this issue?
thanks

I have a similar issue with the UART Rx loosing bytes of data over the when its sent consecutively by a host device with little or no delay between chars.
Unexpected behaviour

the default uart rx isr looses data when the receiving consecutive data over the UART_FULL_THRESH_DEFAULT (120 bytes)
IDF version used: v4.0-beta2-174-g99fb9a3f7-dirty
code is edited, compiled  and debugged using visualGDB version 5.5 ( preview 3, build 3458)
the dev board is a Esp WROVER V4.1
The UART code is based on
https://github.com/espressif/esp-idf/blob/600d542f53ab6540d277fca4716824d168244c8c/examples/peripherals/uart/uart_events/main/uart_events_example_main.c
The uart_event_task() has a high priority.
I tried changing the changing the FIFO full threshold in uart.c  but this did not cure the problem, it still loses data on the FIFO full threshold. Tried setting it to 96 , 64 and it just moved the issue.
when sending 130 bytes, the uart event handler task would get  0x78, 0  this ties in with the UART_FULL_THRESH_DEFAULT (120) but its missing characters.
Sending more data eg 150 was 0x78, 1A when it should have been 0x78,0x1E.
I modified the test file to 632 bytes and I see 0x78,0x78,0x78,0x78,0x78,0x18 when it should have ended 20
Looking at the buffered data the first 120 chars appears ok it, then looses a few in sequence after the first 120 char boundary, then is okay for the rest of the data to the next 120 byte boundary.
A quick inspection of the data at the 120 boundaries sometimes is ok, sometimes is out of sequence
So initially it looks like the ISR always struggles to populate the message que and receive characters in the driver for the first queued data and then randomly looses it for others.
looked in the esp32 forum first for help but no significant contributions yet.
https://www.esp32.com/viewtopic.php?f=12&t=14078

hi, @danpcurran
when the  rx ringbuffer size is small and the UART  working at high baud rate. If the task does not read the data in time, the buffer will be full soon. In this case the data will be lost

@Gohith Thanks for reporting/updates and @danpcurran for sharing updates. Feel free to reopen if the issue still happens.

@negativekelvin @Alvin1Zhang @koobest .
Does anyone knows what is actually causing
UART_FIFO_OVF,          /!< UART FIFO overflow event/
is there any way to solve  this:
xSemaphoreTake( xMutex, portMAX_DELAY );// Reading serial commands from palmtec
uart0_rx(Serial_data1);
xSemaphoreGive( xMutex )

where uart reading is happening

the function uart0_rx is given below

int uart0_rx(char *data)
{
	int length = 0;
	ESP_ERROR_CHECK(uart_get_buffered_data_len(UART_NUM_0,(size_t*)&length));
	length=uart_read_bytes(UART_NUM_0,data,1024, 20 / portTICK_RATE_MS);
	uart_flush_input(UART_NUM_0);
	return length;
}

void uart_init(int BaudRate)
{
    const uart_config_t uart_config = {
    		.baud_rate = BaudRate,
    		.data_bits = UART_DATA_8_BITS,
    		.parity = UART_PARITY_DISABLE,
    		.stop_bits = UART_STOP_BITS_1,
    		.flow_ctrl = UART_HW_FLOWCTRL_DISABLE,
			.source_clk = UART_SCLK_APB,
    };
    uart_driver_install(UART_NUM_0, 2048, 1024, 20, &uart0_queue, 0);
    uart_param_config(UART_NUM_0, &uart_config);
    uart_set_pin(UART_NUM_0, TXD_PIN, RXD_PIN, UART_PIN_NO_CHANGE, UART_PIN_NO_CHANGE);
}

where uart init takes place

does you guys have any suggestion on this as i am getting fifo overflow sometimes only?
the data is actually ~250 bytes with baudrate of 115200


Australian Dollar Option.

You have New Zealand dollars as an option on https://steamdb.info/sales/?min_discount=75&min_rating=80 but any chance you could add the australian dollar region?

SteamDB has said multiple times, that they won't support it until steam recognises it as a separate currency and not just random price inflation (unfortunately)

So is it really just random? or can I base decisions off of one of the other currencies like new zealand?

its up to publishers to decide how much Australia tax is applied afaik, and is wildly inconsistent with some games (like CS:GO) having no price difference, and others being +50%


[AutoPR billing/resource-manager] [Hub Generated] Review request for Microsoft.Billing to add version preview/2018-11-01-preview

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6090

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5564

(message created by the CI based on PR content)


Warnings importing holographic remoting package

Unity 2019.4.10f1
MRTK 2.4
I was still able to complete the tutorial.

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

ID: 55bb143d-e40d-4c26-c310-d4f1777009bb
Version Independent ID: 29327994-e0cc-83d5-025f-ccf5319c38cb
Content: PC Holographic Remoting tutorials - 1. Getting started with PC Holographic Remoting - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/develop/unity/tutorials/mr-learning-pc-holographic-remoting-01.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: jemccull

@polar-kev Does this happen when you import the MRTK.Tutorials.PCHolographicRemoting.unitypackage or at some other stage?

Hi @hferrone, yes, those five warnings are thrown when importing the tutorial assets. Specifically, they are caused by the glb models located at Assets > MRTK.Tutorials.PCHolographicRemoting > Models:

I don't think the issue causing these warnings affects the purpose of this tutorial, so I believe they can be ignored. I also think the warnings will only be shown on first import or after library rebuild.
By the way, I noticed these models are not actively used in the tutorial (only the MarsCuriosityRover model is shown), so we could switch out those five models with models fully supported by MRTK. If we try to find similar models and renamed them to match the current names, we should be able to do it without having to update any tutorial images. Let me know if this is something you want me to look into.


StringBuilder.toString() does not work in GWT with large strings using the LibGDX StringBuilder

Issue details
StringBuilder.toString() does not work in GWT with large strings when using the LibGDX StringBuilder. There is a known bug in the String constructor in GWT when working with large strings which I've worked around in the past in my own code. See here. I'm hitting this problem when creating JSON from a large object.
Reproduction steps/code
The following code will reproduce the issue. This works fine with the Java String Builder
    StringBuilder sb = new StringBuilder();
    for (int i = 1; i < 100000; i++) {
        sb.append("Hello!");
    }
    String s = sb.toString();

Version of LibGDX and/or relevant dependencies
All
Stacktrace
com.google.gwt.core.client.JavaScriptException: (RangeError)
 __gwt$exception: <skipped>: Maximum call stack size exceeded
at Unknown.__valueOf(html-0.js@30:197503)
at Unknown.valueOf_100(html-0.js@10:197712)
at Unknown._String_3(html-0.js@10:197468)
at Unknown.$toString_12(html-0.js@10:63725)
at Unknown.$prettyPrint_2(html-0.js@10:58593)
at Unknown.$prettyPrint_4(html-0.js@10:58679)
at Unknown.$prettyPrint_1(html-0.js@10:46957)
at Unknown.$prettyPrint_0(html-0.js@10:46953)
at Unknown.$prettyPrint(html-0.js@10:46949)
at Unknown.getString_4(html-0.js@25:203579)
at Unknown.$saveUser(html-0.js@43:182979)

Please select the affected platforms

[x] HTML/GWT

Issues with GWT itself should be reported to the GWT project.

I've been told it's fixed in the latest GWT version, but as far as I'm aware there are no plans for LibGDX to be updated to use 2.7 in the near future, so we could do with some kind of workaround for now.

How large is large here? I am not convinced this is a widespread enough issue that hacking around it is actually a worthwhile endeavor.

@MobiDevelop I'll run a test and get back to you on that.

@MobiDevelop 124893 characters triggers the error in my test.

I've been told it's fixed in the latest GWT version

I can confirm that it's fixed in 2.7.

Just got bitten by this yesterday. Using JsArrayString and join as a workaround :(

No longer an issue with the switch to GWT 2.8


[Question] Turn off hdmi then leds go blue

Hello, I found out when I turn off my hdmi device connected to leds hyperion detects whole blue screen and changes color of leds to blue (that's correct because when any hdmi device is turned off blue is probably default screen when no input signal is present). How could I set hyperion to turn off all leds in this case and not blue? Thank you.

For me the threshold doesnt work completely.
Every now and then the LEDs turn on (blue) despite the threshold.
As Im using WS2812b with arduino I use a GPIO Pin of it to measure the Voltange of an USB Port from my TV and wrote 2 lines of code into the Arduino Sketch.
When the TV is turned off, the USB voltage drops low and I stop the LED output from the arduino.
For me that works perfect.
Next step is to write a python script on the raspberry that measures the USB voltage and start and stops the hyperion service depending on the voltage signal :) 👍

problem is we talking about analog signals. When the (blue) video signal gets a distortion, then it is recognized as signal and leds will turn on.
The method how we get the video signal splitter -> converter -> grabber is more a hack than a real solution.
Our way of evaluating the input video signal is more or less reliable and heavily depends on the quality of hardware components.
The safest way to prevent leds from turning on accidently is a switchable power socket for power supply of the leds. This works in almost every setup. BTW there are usb controlled power sockets out there (https://www.amazon.de/EnerGenie-PCW-MS2G-Steckdosenleiste-Master-Slave-Überspannungsschutz/dp/B004FVAFJ6)
connect usb plug with e.g. your TV or pi and the leds power supply with this power socket. The led power will be switched with the power state of usb plug - no arduino tinkering needed.


Add area path option to env variables

ado_area_path and ado_iteration_path as in https://github.com/danhellem/github-actions-issue-to-work-item would be very nice.

Thanks @shauke, now that a couple more folks are requesting it, i will get it done!

I updated master and created a new pre-release version for this. I did not have time to test so please test in production and let me know if you have troubles.

Is changing the documentation (README.md) really enough to add this functionality or are I missing something?
It didn't work as expected either. 😉
And what about the ado_iteration_path? This would make it even more like the Issus Sync Action. 😍

you are missing something. Added the variable and the supporting code.
not going to add iteration path. Sorry

@shauke actually try again. I somehow missed committed the code. Should be all there now

Tested it an now it works as expected. Thanks a lot. 👍
Can I wish for adding the iteration path in a separate GitHub issue or is it a general issue that this will/can never be added for the pull request sync action?

Sorry, I have no plan to add iteration path to the action. If you want to do that, you would need to fork the repo and do it on your own.


Power VS: Fix Service Endpoint validation

The API will currently only allow lowercase names, but all operators match against PascalCase names. This patch adjusts the validation to allow for the expected names. Ratcheting validation will be used for older values, though these shouldn't be in place as no feature currently uses them.

@JoelSpeed I ran some tests on this PR
Scenario 1 - Install a cluster with a value that passes the new validation
I built a release image with this change and deployed a cluster that has a value that passes the new validation (DNSServices below)
Scenario 1 - Proving that the API change works and accepts the new validations (clean install with an image that has my API change)
INFO Install complete!
INFO To access the cluster as the system:admin user when using 'oc', run
INFO     export KUBECONFIG=/home/mjturek/install/mad/auth/kubeconfig
INFO Access the OpenShift web-console here: https://????
INFO Login to the console with user: "kubeadmin", and password: "????"
DEBUG Time elapsed per stage:
DEBUG    Network-infrastructure Provisioning: 8m57s
DEBUG Post-network, pre-machine Provisioning: 15s
DEBUG                   Machine Provisioning: 14m15s
DEBUG       Infrastructure Post-provisioning: 1m23s
DEBUG                     Bootstrap Complete: 19m11s
DEBUG                      Bootstrap Destroy: 7s
DEBUG            Cluster Operators Available: 21m55s
DEBUG               Cluster Operators Stable: 43s
INFO Time elapsed: 1h7m26s

[mjturek@bluemarine1 scripts]$ oc get infrastructure
NAME      AGE
cluster   43m
[mjturek@bluemarine1 scripts]$ oc get infrastructure -o yaml
apiVersion: v1
items:
- apiVersion: config.openshift.io/v1
  kind: Infrastructure
  metadata:
    creationTimestamp: "2024-11-06T16:33:07Z"
    generation: 1
    name: cluster
    resourceVersion: "511"
    uid: 7a282376-9e9a-43be-8aa7-18384adbce20
  spec:
    cloudConfig:
      key: config
      name: cloud-provider-config
    platformSpec:
      powervs:
        serviceEndpoints:
        - name: DNSServices
          url: https://example.com/
      type: PowerVS
  status:
    apiServerInternalURI: https://api-int.rdr-mjturek-capi-mad.powervs-openshift-ipi.cis.ibm.net:6443/
    apiServerURL: https://api.rdr-mjturek-capi-mad.powervs-openshift-ipi.cis.ibm.net:6443/
    controlPlaneTopology: HighlyAvailable
    cpuPartitioning: None
    etcdDiscoveryDomain: ""
    infrastructureName: rdr-mjturek-capi-mad-vmnnz
    infrastructureTopology: HighlyAvailable
    platform: PowerVS
    platformStatus:
      powervs:
        cisInstanceCRN: 'crn:v1:bluemix:public:internet-svcs:global:a/65b64c1f1c29460e8c2e4bbfbd893c2c:60ee139c-1c18-4728-a035-e34533b78a8b::'
        region: mad
        resourceGroup: powervs-ipi-resource-group
        serviceEndpoints:
        - name: DNSServices
          url: https://example.com/
        zone: mad02
      type: PowerVS
kind: List
metadata:
  resourceVersion: ""

Scenario 2 - Upgrade a cluster that has a value that passes the old validation, to a cluster that uses the new validation
I created a 4.18.0-ec.3 cluster that has a value that passes the old validation (see dnsservices below)
NAME      VERSION       AVAILABLE   PROGRESSING   SINCE   STATUS
version   4.18.0-ec.3   True        True          11m     Working towards 4.18.0-0.nightly-ppc64le-2024-11-06-111812: 175 of 944 done (18% complete), waiting on kube-controller-manager, kube-scheduler

Upgrade was successful
history:
    - acceptedRisks: |-
        Target release version="" image="quay.io/mturek/openshift-release:test-endpoint-validation-3" cannot be verified, but continuing anyway because the update was forced: release images that are not accessed via digest cannot be verified
        Forced through blocking failures: Multiple precondition checks failed:
        * Precondition "ClusterVersionRollback" failed because of "LowDesiredVersion": 4.18.0-0.nightly-ppc64le-2024-11-06-111812 is less than the current target 4.18.0-ec.3, and the cluster has no previous Semantic Version
        * Precondition "ClusterVersionRecommendedUpdate" failed because of "UnknownUpdate": RetrievedUpdates=False (VersionNotFound), so the recommended status of updating from 4.18.0-ec.3 to 4.18.0-0.nightly-ppc64le-2024-11-06-111812 is unknown.
      completionTime: "2024-11-06T20:52:33Z"
      image: quay.io/mturek/openshift-release:test-endpoint-validation-3
      startedTime: "2024-11-06T19:36:28Z"
      state: Completed
      verified: false
      version: 4.18.0-0.nightly-ppc64le-2024-11-06-111812
    - completionTime: "2024-11-06T19:28:05Z"
      image: quay.io/openshift-release-dev/ocp-release@sha256:a67abdf194d664a71f2e6695fa5041fa7d3d6168ccbd67806ae72df870c0e602
      startedTime: "2024-11-06T18:46:27Z"
      state: Completed
      verified: false
      version: 4.18.0-ec.3
    observedGeneration: 3
    versionHash: mIxyvKnUz_Q=

Value is still in the cluster infrastructure object
apiVersion: config.openshift.io/v1
kind: Infrastructure
metadata:
  creationTimestamp: "2024-11-06T18:45:53Z"
  generation: 1
  name: cluster
  resourceVersion: "512"
  uid: 00f0bf89-90de-41e8-9231-3acfdb9d0a18
spec:
  cloudConfig:
    key: config
    name: cloud-provider-config
  platformSpec:
    powervs:
      serviceEndpoints:
      - name: dnsservices
        url: https://example.com/
    type: PowerVS
status:
  apiServerInternalURI: https://api-int.rdr-mjturek-capi-mad.powervs-openshift-ipi.cis.ibm.net:6443/
  apiServerURL: https://api.rdr-mjturek-capi-mad.powervs-openshift-ipi.cis.ibm.net:6443/
  controlPlaneTopology: HighlyAvailable
  cpuPartitioning: None
  etcdDiscoveryDomain: ""
  infrastructureName: rdr-mjturek-capi-mad-chgmp
  infrastructureTopology: HighlyAvailable
  platform: PowerVS
  platformStatus:
    powervs:
      cisInstanceCRN: 'crn:v1:bluemix:public:internet-svcs:global:a/65b64c1f1c29460e8c2e4bbfbd893c2c:60ee139c-1c18-4728-a035-e34533b78a8b::'
      region: mad
      resourceGroup: powervs-ipi-resource-group
      serviceEndpoints:
      - name: dnsservices
        url: https://example.com/
      zone: mad02
    type: PowerVS

Updating the value from the old validation to the new validation passes as well
[mjturek@bluemarine1 api]$ oc edit infrastructure
infrastructure.config.openshift.io/cluster edited

[mjturek@bluemarine1 api]$ oc get infrastructure -o yaml
apiVersion: v1
items:
- apiVersion: config.openshift.io/v1
  kind: Infrastructure
  metadata:
    creationTimestamp: "2024-11-06T18:45:53Z"
    generation: 2
    name: cluster
    resourceVersion: "92562"
    uid: 00f0bf89-90de-41e8-9231-3acfdb9d0a18
  spec:
    cloudConfig:
      key: config
      name: cloud-provider-config
    platformSpec:
      powervs:
        serviceEndpoints:
        - name: DNSServices
          url: https://example.com
      type: PowerVS
  status:
    apiServerInternalURI: https://api-int.rdr-mjturek-capi-mad.powervs-openshift-ipi.cis.ibm.net:6443
    apiServerURL: https://api.rdr-mjturek-capi-mad.powervs-openshift-ipi.cis.ibm.net:6443
    controlPlaneTopology: HighlyAvailable
    cpuPartitioning: None
    etcdDiscoveryDomain: ""
    infrastructureName: rdr-mjturek-capi-mad-chgmp
    infrastructureTopology: HighlyAvailable
    platform: PowerVS
    platformStatus:
      powervs:
        cisInstanceCRN: 'crn:v1:bluemix:public:internet-svcs:global:a/65b64c1f1c29460e8c2e4bbfbd893c2c:60ee139c-1c18-4728-a035-e34533b78a8b::'
        region: mad
        resourceGroup: powervs-ipi-resource-group
        serviceEndpoints:
        - name: dnsservices
          url: https://example.com
        zone: mad02
      type: PowerVS
kind: List
metadata:
  resourceVersion: ""

I updated the spec, but It didn't reach the status yet

Joel and I did a few more tests

Editing the infrastructure object without touching the old value succeeded
Editing the infrastructure object, adding an endpoint that matches the new validation but does not touch the old one succeeded. Additionally this test added DNSServices alongside dnsservices. This would not break anything as the new value is the only one that the COs would look for. Example: https://github.com/openshift/cluster-ingress-operator/blob/master/pkg/operator/controller/dns/controller.go#L756-L757


APPID issue

Where can I get APP_ID?

@Muhammad-Ullah fastboard-flutter is a library for Agora Interactive Whiteboard. To get APP_ID, you need a valid agora account, and then enable whiteboard support.


docker-image-not-found not compatible with oci mediatype

This is a follow-on from https://github.com/cypress-io/cypress-docker-images/issues/1093
It is also related to https://github.com/cypress-io/cypress-docker-images/issues/1095

Issue
The npm module docker-image-not-found fails with the error message

RestError [InvalidContentError]: unsupported schema version 2

when a Docker Hub image has been published with an OCI image manifest - see Open Container Initiative (OCI).
It is unable to process
   "schemaVersion": 2,
   "mediaType": "application/vnd.oci.image.index.v1+json",

although it can process
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",

docker-image-not-found is used in the circle.yml workflow.
Logs

See https://github.com/cypress-io/cypress-docker-images/issues/1093

RestError [InvalidContentError]: unsupported schema version 2 in cypress/factory:4.0.1 manifest

Background
According to Docker Hub > Registry > Supported media types

You can push images with the following formats:

OCI image manifest
Docker image manifest version 2, schema 2

Compared to Docker buildx v0.9.1 (released Aug 18, 2022) Docker buildx v0.10.0 (released Jan 10, 2023) introduced SLSA Provenance with support for OCI-compliant multi-platform images.
Comparison with other Docker images
Running the following command to inspect other common images, shows that debian (which Cypress Docker images are based on) is using docker mediaType. There has not been a general clear migration to oci mediaType.
docker manifest inspect

Image
mediaType

node
application/vnd.oci.image.manifest.v1+json

ubuntu
application/vnd.oci.image.manifest.v1+json

fedora
application/vnd.docker.distribution.manifest.v2+json

debian
application/vnd.docker.distribution.manifest.v2+json

cypress/factory
application/vnd.docker.distribution.manifest.v2+json

Suggestion
Force circle.yml to generate docker mediaType to retain compatibility with docker-image-not-found. According to https://github.com/docker/buildx/issues/1509#issuecomment-1378538197 this can be achieved through:

Set --provenance=false

See also issue #1142


Cannot get Teleoperations to work with Gamepad

Hi,
I have managed to obtain GamePad index (=0) but when I execute the following codes in cell 4 of “teleoperation.ipynb” :
—
left_link = traitlets.dlink((controller.axes[0], 'value'), (car, 'steering'), transform=lambda x: -x)
right_link = traitlets.dlink((controller.axes[5], 'value'), (car, 'throttle'), transform=lambda x: -x)
—
resulting in “IndexError: tuple index out of range“ error.  The Gamepad dongle is connected on the remote connecting machine (Mac Catalina or Windows 10) using Google Chrome browser.  There are corresponding updates in axes values with controller interactions (http://html5gamepad.com).
Can someone share some insights how to move forward?  Thanks.

This problem has been resolved with "sudo apt install python3-ipywidgets".  Hope it will be useful to those encountering the same problem.

controller = widgets.Controller(index=1)  --> index=0


Refactorise app_simple

From @dr-rodriguez:

app_simple.py is becoming very large. At some point, you should move plotting/utility functions outside of it (to things like plot.py or utils.py) and leave app_simple.py to solely contain the basics for Flask to handle the different endpoints we serve.

Also update the documentation in certain places (e.g. the creation of the url link in search results)


Updated UA Training config and SK config

Updated UA Training config and SK config

@HuguesTHOMAS Please delete this PR.
I meant to merge this into my fork of the repo, but forgot to change the target branch. This branch is not meant to be in this repo.


chore: delete all secondary API code

Description
Secondary forwarding is now disabled. We can delete the code & the infra.
Risk
N/A
Deploy Plan

deploy
manualy delete the k8s resources (via CLI)

LGTM thx!


Updates and improvements for node-sass v3 (polished)

Use of strict mode. :cop:
Updates and some bug fix for node-sass v3.
Minor changes in formatting and documentation.
Removed unused/unsupported code, etc.
Additional comments.
Documented some undocumented options.
This should fix #22 with a half-decent error output in debug mode.
This should fix #31 ending the css request with status 500.
error function replaced by proposed here.

It includes commit iMoritz@1ea16c1 by @iMoritz fork. :punch:
It needs new tests. :construction:

It looks good to me! :+1:
Would you like to add that missing test you mentioned before this get merged?

I'll review it later today.

@am11, a question about this, I didn't finish understand...
Why error function now throws an error instead of calling next()?
I need understand it to make Travis happy, thanks! :yum:

@bumxu, this should do the trick: https://gist.github.com/am11/66d4b708c15159f94dbc.
Side note: nodeSass.render() returns the Error object and nodeSass.renderSync() throws it.

@am11 Great! :+1:

:tada:
Thanks for your contribution. This one is HUGE! :sunny:


Make some macros indexable #294

You can reject it if you don't like it
This also includes some shorthands for struct creation
Closes #294

Something seems to be wrong with travis?


how to dynamically reload .tern-project file?

Kill the tern/node process. Individual editor plugins could add a command for this, but as of yet, I don't think any of them do.
(Alternatively, it might be a cool feature to have bin/tern 'watch' the project file, and automatically restart when it changes. I'd be happy to review a pull request like that.)


Submission for Encrypting secrets locally

PR Summary

PR Checklist

Do you want this post to be published on a specific date?: Y/N [Specify date]:
[x] I have read the contributors guide and followed the style and process guidelines
[x] PR has a meaningful title
[ ] Includes content related an open issue

see Closing issues using keywords.

[x] This PR is ready to merge and is not Work in Progress

@rod-meaney I made some minor editorial changes. In future, please create a working branch. Don't work in main. It makes it harder to give editorial feedback.


OS Level Exposure Notifications are misleading on iOS

Summary | Résumé
Users are receiving the weekly OS-level exposure notifications that have a recap of the number of potential exposures they have had that week. These notifications are alarming and conflict with messaging in the app. The notifications don’t take into account the EN Framework configuration set by the app.

Steps to reproduce | Étapes à reproduire

Install the app on an iPhone
Expose your phone to a diagnosed phone for 1 minute at a distance of 5 metres.
Observe that the iPhone will not go into an exposed state, as expected
After a few days you will see a weekly, OS level notification that says you have "1 potential exposure".

Environment | Environnement logiciel
iOS

 Submit github issue for Apple OS Notifications

https://twitter.com/ryancbriggs/status/1297176019728728065

Just ran into this myself; got a system-level "your device identified 1 potential exposure this week" notification on iOS, but the COVID Alert app says "No exposure detected".
Confusingly, tapping on the system notification brings you to "Exposure Notifications" in Settings.app.
What should be my (or any user's) interpretation of this? That iOS is naively listing all devices seen and the app is actually filtering by exposure time and distance? iOS also lets me export the exposure checks, which is equally opaque (but appears to show me 4 checks that turned up a match):
{
      "Timestamp" : "2020-09-11 21:31:44 -0400",
      "Files" : [
        {
          "Hash" : "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
          "MatchCount" : 1,
          "KeyCount" : 178,
          "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid",
          "Timestamp" : "2020-09-11 21:31:44 -0400"
        }
      ],
      "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid"
    },
    {
      "Timestamp" : "2020-09-11 21:31:44 -0400",
      "Files" : [
        {
          "Hash" : "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
          "MatchCount" : 0,
          "KeyCount" : 10,
          "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid",
          "Timestamp" : "2020-09-11 21:31:44 -0400"
        }
      ],
      "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid"
    },
    {
      "Timestamp" : "2020-09-11 19:30:56 -0400",
      "Files" : [
        {
          "Hash" : "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
          "MatchCount" : 1,
          "KeyCount" : 175,
          "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid",
          "Timestamp" : "2020-09-11 19:30:56 -0400"
        }
      ],
      "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid"
    },
    {
      "Timestamp" : "2020-09-11 17:30:57 -0400",
      "Files" : [
        {
          "Hash" : "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
          "MatchCount" : 1,
          "KeyCount" : 162,
          "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid",
          "Timestamp" : "2020-09-11 17:30:57 -0400"
        }
      ],
      "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid"
    },
    {
      "Timestamp" : "2020-09-11 13:36:28 -0400",
      "Files" : [
        {
          "Hash" : "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
          "MatchCount" : 1,
          "KeyCount" : 108,
          "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid",
          "Timestamp" : "2020-09-11 13:36:28 -0400"
        }
      ],
      "AppBundleIdentifier" : "ca.gc.hcsc.canada.stopcovid"
    },

Although, confusingly, two separate objects at 2020-09-11 21:31:44 -0400, one of which has a positive MatchCount and one which doesn't.

Wording has been updated as seen here in iOS 14 (again this is at the OS level not via the app)

@lepinsk see https://github.com/cds-snc/covid-alert-app/issues/1163#issuecomment-702741221 for more information about MatchCount.

Closing this as the OS has now updated.


chore(ci): add renovate bot

This PR adds the Renovate[bot] configuration file to automate the dependency tracking and update.
The remaining step is to enable the Renovate GitHub App for this repo.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3874

Relevant Lines:
5791

💛  - Coveralls


fixStream load not have usage priv for workload group

Proposed changes
Issue Number: close #xxx
When requesting execution plans from FE through RPC, the client IP was previously used and changed to the matching user host.
For example, the previous version was' <IP_ADDRESS> ', and the later version after the modification was' 82.157.43%',
FE will directly build userIdentity using the brought IP.
Due to the possibility that only user a @'82.157.43.% 'may exist on FE and not a @'<IP_ADDRESS>', the user may not have permission.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall


Upon opening the attached file, Lapce hogs CPU, then crashes

Lapce Version
0.2.0
System information
Debian 11, Bullseye
Describe the bug
Upon opening  main.rs.gz, Lapce  opens but does not display the file content. Lapce hogs 1 thread CPU and the crashes a few moments later.
Additional information
I suspect is has something to do with the 10,000 items long vector in main.rs.

In VSCode this shows up.

Maybe that line is too long and that's why this error occurrs.
Where I can start to solve this?


considers Newtonsoft.Json to be available on Core, but not on Desktop.

I ran it on the desktop version of the dll in this package:
https://www.nuget.org/packages/Microsoft.AspNet.WebApi.Client/5.2.3
It says it's 100% compatible with .NET Standard and .NET Core - which is great. But it says it's only 89.7% compatible with .NET Framework. Which makes no sense as it's a .NET Framework asset.
It looks like it considers Newtonsoft.Json to be available on Core, but not on Desktop.

I have the same problem.

Closing as API Port was deprecated in favor of binary analysis in .NET Upgrade Assistant.


Add try_repo_update_on_error option to cocoapods action

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
A lot of CocoaPods users from time to time face issue when after updating some pod locally they have CocoaPods error on CI side, because it is needed to go there and run something like bundle exec pod repo update(because CI doesn't know that dependency has some new version). So, I propose to add try_repo_update_on_error option to cocoapods action, which will try to execute pod command one more time with --repo-update in case of error.
Also, yes, I know about already implemented repo_update option. But while it is really worth to run in 5% of cases, it take significant time to wait while it performing every time(and also gives additional load on GitHub servers).
Thanks!

Great idea, I like to fact that this would update the repo only if it needs to be updated (probably one of the most common error cases of the cococapods fastlane action. Would love a confirmation from @orta that this is a good behavior with no side effects. We could even enable that option by default, right?

Yeah, seems fine to me 👍

@KrauseFx yes, maybe we can enable it by default, but some of users already repo_update option enabled(which is not compatible with new try_repo_update_on_error) and we need to think about behavior in this case.
Maybe we can enable try_repo_update_on_error by default, but disable it for users who pass repo_update: true option?
And also: do someone really need repo_update option(which takes significant time) if we have this one? Maybe we can deprecate it?

Ah alright, let's leave it as is then. Is it ready to be reviewed & merged from your side @SiarheiFedartsou?
Thanks for confirming @orta

@KrauseFx yes, I think :)


python3Packages.h3: 3.7.0 -> 3.7.1

Motivation for this change
This change also fixes the build to work with the newer version of h3-py.
Co-authored-by @rmcgibbo and it replaces #111968.
Things done

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

[x] NixOS
[x] macOS
[ ] other Linux distributions

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

closes #111968

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112091 run on x86_64-linux 1

3 packages built:

python37Packages.h3
python38Packages.h3
python39Packages.h3

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112091 run on x86_64-darwin 1

3 packages built:

python37Packages.h3
python38Packages.h3
python39Packages.h3

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112091 run on x86_64-darwin 1

3 packages built:

python37Packages.h3
python38Packages.h3
python39Packages.h3


Relax Flow CLI version parsing requirements & make RegExp more generalized

relates to #509 (but was already solved on the CLI side)

For contributor use:

[x] Targeted PR against master branch
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[x] Updated relevant documentation
[x] Re-reviewed Files changed in the Github PR explorer
[x] Added appropriate labels

I'm going to close this since I think it makes more sense to move to JSON format.


New Coding Style guide

Partially solves for #64.
To be merged after #85.

Now that I am reviewing our "Code Style" section, I'm seeing lots of inconsistencies across our "avoid" and "use" cases. Fixing those in this PR.
By the way, Sphinx tabs turned out to be very useful.

The [blacken-docs] tool is very useful for this. I will add a reference to it in #85.

Fixing some code suggestions in a separate PR. Those didn't appear in batch mode.


Move ResponseCandidateStream to own module

What was wrong?
I've got some work that's moving things around and refactoring.  The diff it generates is ugly because it contains both code changes and moves.
How was it fixed?
Just do the move in this PR which makes the upcoming diff way easier to read.
To-Do

[x] Clean up commit history

Cute Animal Picture

This should be zero functionality change.


when root post deleted, cannot view replies

Describe the bug

when root post of a thread is deleted, users replied to that post cannot view their replies on profile page replies section/tab.
To Reproduce
Steps to reproduce the behavior:

post something (root post).
reply to root post.
delete root post.
try to view reply on profile page replies section or on following feed.

Expected behavior

to view replies on profile page replies section even if root post is deleted.
Screenshots

Details

Platform:
Platform version:
App version:

Additional context

i have used a new account to observe postcount. i followed reproduction steps. counter was 2 before i delete root post. after the deletion counter decreased to 1 but i couldn't view the reply. not shown replies are visible on graysky app.

Not able to reproduce.
Reply to own deleted post: https://bsky.app/profile/dansshadow2.bsky.social/post/3kqvonlrmmk2j
Reply to another account's deleted post: https://bsky.app/profile/dansshadow2.bsky.social/post/3kqvolk3ad42s
Both appear in the Replies tab: https://bsky.app/profile/dansshadow2.bsky.social
Can you provide more info (eg specific URLs)? Thanks.

Ahh interesting. This might be related to what @devinivy was fixing last week. Thanks for confirming.


Organize the requirements fields for API Review meeting

Currently we are asking the user to provide Core concepts, API Documentation, and Hero scenarios.
We should:

Remove API Documentation field as it is the same as asking for the Swagger PR.
Add add a new field alled Architecture diagram link (we have this field already in the informational meetings)
Reorder the fields to Hero scenarios, Architecture diagram, Core concepts

Note: The current documentation doesn't talk about architecture diagram, so it needs to be updated https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/665/Azure-REST-API-Stewardship-Review

@markweitzel could you please update the documentation of the wiki https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/665/Azure-REST-API-Stewardship-Review to reflect the need for the architure diagram?

@ladonnaq to follow up with Mark on documentation

We will first swap documentation wiki pages by Wed 11/16. Then we will update the fields in the wiki

Remove API Documentation field as it is the same as asking for the Swagger PR.

It has been removed as part of version 1.0.0-beta.11

Reorder the fields to Hero scenarios, Architecture diagram, Core concepts

Done as part of 1.0.0-beta.11

Remaining work here:

Add add a new field called Architecture diagram link (we have this field already in the informational meetings)

Add field into calendar invite template
Add field into GH issue template

Update wiki documentation to make sure this is reflected too
Review calendar invite template content and make sure informatio is up to date

It's find as a request but not a requirement (IMHO).


Exception: Could not find metadata in .ipynb-meta, inside .ipynb or external .md file.

Hello, I got these warnings and error when I ran "sudo make html".
WARNING: PLUGIN_PATH setting has been replaced by PLUGIN_PATHS, moving it to the new setting name.
WARNING: Defining PLUGIN_PATHS setting as string has been deprecated (should be a list)
ERROR: Could not process notebook/test.ipynb
  | Exception: Could not find metadata in `.ipynb-meta`, inside `.ipynb` or external `.md` file.
Done: Processed 2 articles, 0 drafts, 0 pages and 0 hidden pages in 0.20 seconds.

Here are my settings and environment

pelican 3.6.3
jupyter 4.0.6
nbconvert 4.1.0
beautifulsoup4 4.4.1

content
    notebook
        test.ipynb
        test.ipynb-meta
plugins
    ipynb
        __init__.py
        ...

In the pelicanconf.py:
...
MARKUP = ('md', 'ipynb')

PLUGIN_PATH = './plugins'
PLUGINS = ['ipynb.markup']

I tried to solve the problem but I couldn't. When I write "PLUGIN_PATHS = ['./plugins']" the warnings are gone but still get that error. Any idea? Thank you.

Sorry, that was my fault. It looks like I didn't describe ipynb meta data in a proper way.

@ghost I have the same problem so if you could tell me what you did wrong I could fix my code.

Ok I found it. In a .ipynb file there are several "metadata" keys. The one you want is at the very bottom at the file.