WebXR does not work from inside iFrame

Description
This is a follow-up to an earlier discussion on Spectrum.
When using model-viewer inside an iFrame, WebXR mode just outputs a black screen. There's a brief split-second moment when a camera image is shown on a tiny slice of the screen, but afterwards it just turns black. The request to allow a WebXR session pops up properly if done for the first time.
The iFrame has allow="xr", but that doesn't change the outcome.
Edit: I also found https://github.com/immersive-web/webxr/issues/25#issuecomment-549959055, but allow="xr-spatial-tracking" exhbits exactly the same behaviour.
Live Demo
Open this page in Chrome on Android
(I have removed the repro page for now since the bug has been accepted as bug - I'll happily test again once resolved)
Click the "View in AR" button (the page isn't very responsive, you might have to turn to landscape mode)
Expected: AR model
Actual: black screen
Browser Affected

[X] Chrome
[ ] Edge
[ ] Firefox
[ ] Helios
[ ] IE
[ ] Safari

OS

[X] Android
[ ] iOS
[ ] Linux
[ ] MacOS
[ ] Windows

Thanks, I have a repro and some strange console errors; I'll look into it.

I'm still investigating, but it appears that the WebXR DOM Overlay feature doesn't currently work when used within an iframe. I filed https://bugs.chromium.org/p/chromium/issues/detail?id=1101193 to follow up on the Chrome side.
In the meantime, is there a way to disable use of DOM Overlay in model-viewer as a workaround?

@klausw Ugh; I suppose I could try to detect when model-viewer is in an iframe and not enable dom-overlay, but it still means a large part of webXR's advantage is inconsistent. Any idea how long it'll take a Chrome fix to land?

@klausw Since it looks like the real fix will take a while, do you know a good method to enable dom-overlay only if it's not in an iframe?

@klausw Since it looks like the real fix will take a while, do you know a good method to enable dom-overlay only if it's not in an iframe?

The usual idiom seems to be to check if window.parent == window which is false when in an iframe. Careful, accessing window.parent.location in a cross-origin iframe throws a security exception (and crashes Chrome DevTools...).

Any updates when this is going to be fixed as the result of AR on screen viewer is different from webXR. Quality of webXR is better than screen-viewer, this bug is stopping me to use it!

@ArvindSharda78 Sorry, this got delayed to Chrome 86, rolling out in about a month.

Does WebXR work as expected from within an IFRAME, now that Chrome 86 is out?

@milesgreen Yes, it should. I haven't had a chance to test it extensively myself, so please let us know how it works for you!

For me, broadly, yes. It seems to be.
Although I did see some unexpected behaviour when switching out multiple models in WebXR, but I need to do more testing to replicate and isolate (might be something I was doing wrong). Will update if I have anything specific.

Great, I'm closing as fixed in Chrome 86, but feel free to reply if you notice any further problems.

I know this is closed already. But just wanted to confirm, in my testing, ModelViewer / WebXR is working great when served from iFrame.
The lingering issue I mentioned above was due to the fact I wasn't listening for beforexrselect event to prevent DOM interactions causing unwanted interaction with WebXR. So it's all good now.

Although this has been fixed for chrome, today I tested it and found issue with firefox on iOS (specifically iPad). When you open any iFrame hosted model-viewer in a firefox on iOS and press on AR button it leads USDZ file download. Can you please look in this bug as it leads to user explicitly downloading the model? Is there a way to stop this functionality for Firefox? this is issue is not reproducible on Chrome or Safari on iOS.

@ArvindSharda78 it might be a Firefox issue with USDZ scheme. Does it properly work outside an iFrame there for the same usecase?

@hybridherbst , yes it works properly outside iFrame for the same use case.

OK, then I guess back to @elalish who'll probably recommend you to open a bug with Firefox ;)

@ArvindSharda78 This sounds like a version of a bug I thought I'd recently fixed. Firefox doesn't support WebXR or app intents, so the AR button shouldn't be shown in that browser at all. I tried to disable it by sniffing the user agent string, but perhaps I missed a case. Can you paste your user agent string here? And just to be sure, this is using model-viewer v1.3, yes?

@elalish , I opened the https://modelviewer.dev/examples/augmentedreality/#ar in iPad. the AR button was not visible on chrome, on Safari and Firefox it was visible
Tested on iOS (iPad)
Chrome - AR button (Not visible)
Safari - AR button (visible) (AR works)
Firefox - AR button (visible) (AR works too, but if it is in iFrame it downloads the USDZ)
the user agent does not give correct results , the code I used : var FxiOS = navigator.userAgent.match("FxiOS");

@ArvindSharda78 Interesting! So Firefox can't launch an app intent on Android, but it can launch Quick Look on iOS. Are you saying the Firefox iOS user agent string is FxiOS? If you can paste the whole user agent string here that would be great.

@elalish I got a black screen instead of the camera feed in AR mode. I am using chrome on Android. The AR backend is set to webxr.


Transaction has a bad signature

I´m getting this error when trying to unbond my ksm. Can´t even stop nominations. I have the ksm app updated in my ledger, have tried it in different browsers, have tried to restart everything, have tried to delete the account from polkadot.js and restore it again...and it didn´t solve.
I did some staking operations two days ago and everything was fine. I updated the ledger app today, as requested, and then this message appears.
Don´t know what more can I do, I´m always getting that same error, so I would really appreciate your help.
Thanks in advance.

:link: zboto Link

Please follow what's indicated in the answer to that issue!
You should be good to go if using https://github.com/Zondax/ledger-kusama/issues/175#issuecomment-1347966917

Please follow what's indicated in the answer to that issue! You should be good to go if using #175 (comment)

What exactly I´m supposed to do? Because I don´t have the funds in other account that I need to recover, I just have the tokens staked. I tried to make the transaction with the recovery mode activated but it doesn´t solve the problem. And the rest of the tutorial is just to add the ledger account in polkadot.js and sending the tokens "lost" that now should appear to other account, but that´s not my case. So what I´m supposed to exactly do with the answer in #175?

I just tried to bond more funds and same error appears and I just did this two days ago before the updating of the app. Couldn´t it be for something related to this updating?

@CryptoBelieverr let's reopen then. Please, do not be aggressive, we are trying to help you as much as we can.
Please, try to re-add the account to polkadot-js using ledger, both with normal and recovery mode. Could you tell us if one of the two is the one you are trying to use, and which one?

Aggressive?? lol, no man. Sorry if it seemed like that (maybe it´s just my English, my bad), but I promise that I wasn´t talking in a bad way, ofc not, it would be silly from me if I´m looking for help!
It´s weird, because I tried to re-add with normal and recovery mode mode and both shows different addresses to the one in which I had the funds.

Indeed, I have no idea how to add the account again using the adding ledger option, all the addresses added are different. Luckily, I have the account in the polkadot.js extension in other browser, so the only way that I find to re-add it again in the other browser from which I deleted the account is exporting a json file from the other browser´s extension and importing it on this.

Ok, I solved it. I had some hidden accounts in the ledger and I found out that I had the names in the polkadot.js extension mixed (no idea why) and was using the wrong pin in the ledger. Sorry for the inconvenience, but at least, it has been solved.
Thank you for your time.


ACM-13001 - Update restore CRD and add completiontimestamp in status

Description
Update restore CRD with the latest changes including adding CompletionTimestamp in the status
Related Issue
https://issues.redhat.com/browse/ACM-13001
Changes Made
Update restore CRD with the latest changes including adding CompletionTimestamp in the status
Changes are coming from:
https://github.com/stolostron/cluster-backup-operator/pull/706
https://github.com/stolostron/cluster-backup-operator/pull/708
https://github.com/stolostron/cluster-backup-operator/pull/709
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[x] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[x] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/test all

/lgtm


Use modern syntax for node module imports

node: protocol imports are supported in Node 12.20.0 and up. They are recommended to be used this way, especially since Node 14. Some services, like cloudflare workers require this syntax to be used, or the imports break.
This PR just updates the "bad" imports I found. This will make no real difference to most, but will help compatibility for some.

Thanks!


Update albedo.js

We need to try and clean up before jumping ship on other errors.

@benirose , is there anything else needed for this MR?


Cmake as first class citizen

Added fuzz and integration testing to cmake runs, added cmake build to travis. Began removing in-source build expectation madness.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks like the negative fuzz test is failing. It's suppose to purposefully leak memory, and verify that the fuzz test fails.
The following tests FAILED:
	  6 - s2n_memory_leak_negative_test (Failed)
Errors while running CTest

https://travis-ci.org/awslabs/s2n/jobs/421839089#L2052


dependency between two or more dropdown list

I have three dropdown list in a form.one of them shows fullname of employees, another one shows his/her degree and rhe last shows his/her job.
I want to do this task : when I select a value from first dropdown other dropdowns fill automatically. for example when I selected x from first dropdown, then second dropdown became y and the last dropdown became z .what should I do? I'm new to serinity and I don't know how can I do that.
thank you.

See samples


Unable to extract video url Instagram

I try to download instagram video https://www.instagram.com/p/CAg6NPqj5VM/?igshid=1nv9lh9srklu8
and get this error. It happens only on my servers. On my laptop it's OK
ERROR: Unable to extract video url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
ERROR: Unable to extract video url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/YoutubeDL.py", line 797, in extract_info
ie_result = ie.extract(url)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/instagram.py", line 195, in _real_extract
video_url = self._og_search_video_url(webpage, secure=False)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/common.py", line 1123, in _og_search_video_url
return self._html_search_regex(regexes, html, name, **kargs)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/common.py", line 1014, in _html_search_regex
res = self._search_regex(pattern, string, name, default, fatal, flags, group)
File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/common.py", line 1005, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract video url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

i have same issue with instagram videos

me too
https://www.instagram.com/p/B8waiGXgLlD/
Error downloading video: Unable to extract video url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

any fix for this?

any updates on this issue?

any updates on this issue?


Installation of  V 0.0.38

@msp1974  - feed back for the installation of this version
Installation of the dev version:

Extract of the logs:
ERROR: No matching distribution found for wiserHeatAPIv2==0.0.38
2022-07-19 17:16:39 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration wiser: Requirements for wiser not found: ['wiserHeatAPIv2==0.0.38'].
2022-07-19 17:16:40 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/discovery_flow.py", line 74, in async_process_pending_flows
await gather_with_concurrency(
File "/usr/src/homeassistant/homeassistant/util/async.py", line 199, in gather_with_concurrency
return await gather(
File "/usr/src/homeassistant/homeassistant/util/async_.py", line 197, in sem_task
return await task
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 222, in async_init
flow, result = await task
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 240, in _async_init
flow = await self.async_create_flow(handler, context=context, data=data)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 779, in async_create_flow
await async_process_deps_reqs(self.hass, self._hass_config, integration)
File "/usr/src/homeassistant/homeassistant/setup.py", line 360, in async_process_deps_reqs
await requirements.async_get_integration_with_requirements(
File "/usr/src/homeassistant/homeassistant/requirements.py", line 49, in async_get_integration_with_requirements
return await manager.async_get_integration_with_requirements(domain)
File "/usr/src/homeassistant/homeassistant/requirements.py", line 168, in async_get_integration_with_requirements
await self._async_process_integration(integration, done)
File "/usr/src/homeassistant/homeassistant/requirements.py", line 183, in _async_process_integration
await self.async_process_requirements(
File "/usr/src/homeassistant/homeassistant/requirements.py", line 230, in async_process_requirements
self._raise_for_failed_requirements(name, missing)
File "/usr/src/homeassistant/homeassistant/requirements.py", line 252, in _raise_for_failed_requirements
raise RequirementsNotFound(integration, [req])
homeassistant.requirements.RequirementsNotFound: Requirements for wiser not found: ['wiserHeatAPIv2==0.0.38'].

You are very keen Christian! 😀
I have not finished and released the api update yet - which is what it is complaining about.
I was going to do this this weekend. I'll let you know.  This will have an update to the card to copy days when  editing a schedule, improved diagnostics output method (which is why an api update) and a couple of minor fixes.

Sorry to be so early...
But I saw your job and I was afraid to have missed ...
I have used the schedule card a lot...

Just released 0.0.38 so the v3.1.2 version in Dev branch of Wiser Integration should now install.


Excuse me, where the docs?

Excuse me, where the docs?

https://github.com/CollectionFS/Meteor-CollectionFS#collectionfs
tracked in 563


[github-actions] upload core dump, so libs and binaries for crashed programs

This PR adds action to upload core dumps, so libs and binaries for 1.2 certification. This is intended to solve issues like #5802, #5774 where programs intermittently crash however hard to reproduce it. Related issue: #5805.
I added this for thread-1-2 and low-power-packet-verification first as an experimental work. We can add same actions for other items when it's necessary.
I've verified that we can analyze the core dumps with the artifacts using gdb successfully on our local PC: https://github.com/openthread/openthread/pull/5812#issuecomment-732749457

Can we add docs somewhere that includes the steps in https://github.com/openthread/openthread/pull/5812#issuecomment-732749457?

Can we add docs somewhere that includes the steps in #5812 (comment)?

@jwhui
What about adding the docs in CONTRIBUTING.md and notify the contributors to download the artifacts and report the issue when error happen?

Can we add docs somewhere that includes the steps in #5812 (comment)?

@jwhui
What about adding the docs in CONTRIBUTING.md and notify the contributors to download the artifacts and report the issue when error happen?

@Irving-cl , sounds good to me!

@jwhui
Updated CONTRIBUTING.md for instructions.

@Vyrastas
The suggested version is much better. Thanks!


DO NOT MERGE: reset VERSION_PROBING error code in onPartitionsAssigned

Hopefully we can fix the issue with https://github.com/apache/kafka/pull/7243.


Fix narrowing of property access expressions

This PR removes some unnecessary restrictions on narrowing of property access expressions.
Fixes #8923.

:+1:


YOLOX OpenVINO Model Batch Inference issue

I am currently conducting batch inference on the YOLOX model using the OpenVINO IR format. When the batch size is set to 1, the IR model's output shape is (1, 3549, 85). However, when the batch size is set to 'n', the output shape becomes (1, 3549 * n, 85). I anticipate the output shape to be (n, 3549, 85) to align with the requirements of the demo_postprocess function.
Consequently, I attempt to reshape the IR model's output from (1, 3549 * n, 85) to (n, 3549, 85). Unfortunately, the bounding boxes are not predicted correctly. How should I process this output data to ensure it fits the demo_postprocess function?

Hello,
I followed the instructions provided in this guide to create an IR model for OpenVINO. However, like you, I encountered an issue where the model structure became corrupted when setting the input batch size to 'n'.
In my case, when running tools/export_onnx.py to create the ONNX model, I added the --no-onnxsim option, which seemed to resolve the problem for me. While I can't say for certain what caused the issue, it seems that the ONNX Simplifier (onnxsim) might be the culprit during the conversion process.
Hope it helps a bit.


Tabulator dev

One last adjustment to close out Tabulator_dev

@prashleigh This is an ancient pull-request from March, and we've moved way beyond this, so I'm closing it.


Make DbStore#NewStore return (session, error)

While right now, creating a new store can't fail (we don't do any work except creating a struct), there is definitely a not too distant future where we'd want to change that. Let's change the API early to avoid future breaking changes.
Examples of things we'd check:

Well formedness of the schema
Pre-fetching constants or configuration (possible with an ACL implementation)
Etc.

See #97


Workbox Routing and Caching and service worker registration?

(This issue was moved over from
https://stackoverflow.com/questions/52620240/workbox-routing-caching-deletion-updation-and-service-worker-registration)
Library Affected:
workbox-sw, workbox-build, workbox-webpack-plugin etc.
Issue or Feature Request Description:
My Configuration goes like this
new workboxPlugin.InjectManifest({
swSrc: public/sw.js,
swDest: './sw.js',
exclude: [/.jpg$/, /.png$/,/.js$/,/.css$/,/.gz$/,/.svg$/,/.jpeg$/,/.html$/
,/.json$/,/.map$/,/.gif$/]
})
As I don't want any precache files, as I register my service simultaneously not on dom load.
And In my service worker file I have registered route like this
workbox.routing.registerRoute(
/.(?:js|css|gzip|html|svg|gif)$/,
workbox.strategies.cacheFirst(),
);
So all js,css mentioned files will be cached from this route.

If you don't plan on precaching anything, you don't need to use the build tools, like workbox-webpack-plugin. You can just write your own service worker file that imports Workbox and sets up your runtime caching route, like https://developers.google.com/web/tools/workbox/guides/get-started#importing_workbox
Since you're using a mix of versioned (with hashes) and unversioned URLs, I'd recommend using two different runtime caching routes.
One should be a regular expression that detects URLs with hashes in them, and uses a cache-first strategy, along with cache expiration (depending on how many different responses you want to keep in the cache at a given time). The other route should match URLs without hashes in them, and use something like a stale-while-revalidate (or network-first) strategy to ensure that you get updates. You could also use cache expiration here.
The final service worker file might look something like:
importScripts('https://storage.googleapis.com/workbox-cdn/releases/3.6.3/workbox-sw.js');

workbox.routing.registerRoute(
  new RegExp('\\.[a-f0-9]{8}\\.'),
  workbox.strategies.cacheFirst({
    cacheName: 'versioned-assets',
    plugins: [
      new workbox.expiration.Plugin({
        maxEntries: 50,
        purgeOnQuotaError: true,
      }),
    ],
  })
);

workbox.routing.registerRoute(
  new RegExp('\\.(?:gif|svg|jpg)'),
  workbox.strategies.staleWhileRevalidate({
    cacheName: 'unversioned-assets',
    plugins: [
      new workbox.expiration.Plugin({
        maxEntries: 50,
        purgeOnQuotaError: true,
      }),
    ],
  })
);


Added new app for showcase: Deskbookers

Nice! The app looks cool.
@facebook-github-bot shipit

@emilioicai this merged but can you still update with those changes?

Good catch @chirag04, thank you! Will keep it in mind when reviewing.


PKD VALIS Trilogy and the Concept of Logos

PKD VALIS Trilogy and the Concept of Logos
Preview
pkd valis trilogy and the concept of logos therein
The PKD VALIS trilogy is a series of novels by the American science fiction writer Philip K. Dick, who explored his own mystical experiences and philosophical questions in these works. The trilogy consists of VALIS (1981), The Divine Invasion (1981), and The Transmigration of Timothy Archer (1982)1. The title VALIS stands for Vast Active Living Intelligence System, which is Dick’s vision of a cosmic mind or God that communicates with humans through various means2.
One of the main concepts that Dick explores in the trilogy is the logos, which is a Greek word meaning word, reason, or principle. In Christian theology, logos is also used to refer to the second person of the Trinity, Jesus Christ, who is the Word of God made flesh. Dick uses logos to refer to the voice or message of VALIS that reveals the hidden truth about reality and history to the characters in his novels. Dick also connects logos to the Gnostic idea of Sophia, or Holy Wisdom, who is the female aspect of God and the source of creation2.
In the first novel, VALIS, the protagonist Horselover Fat (an alter ego of Dick) experiences visions of a pink beam of light that he calls Zebra and interprets as a manifestation of logos. He and his friends form a group called the Rhipidon Society to investigate the nature and origin of this phenomenon. They discover that there is an alien satellite orbiting Earth that broadcasts information and influences human events. They also encounter a young girl named Sophia, who claims to be the incarnation of Holy Wisdom and tells them to worship humanity instead of gods2.
In the second novel, The Divine Invasion, the story shifts to a parallel world where Earth is under the control of an evil power called Belial, who has corrupted religion and suppressed spirituality. A man named Herb Asher and his pregnant wife Rybys are contacted by VALIS, who tells them that their unborn child is Emmanuel, the true Messiah who will restore balance and harmony to the world. They are transported to Earth with the help of some friends, but they face many obstacles and dangers from Belial’s agents. Emmanuel grows up quickly and begins to perform miracles and teach people about logos2.
In the third novel, The Transmigration of Timothy Archer, the focus is on Timothy Archer, an Episcopal bishop and friend of Horselover Fat who appears briefly in VALIS. The novel is narrated by his daughter-in-law Angel Archer, who recounts his life and death. Timothy Archer becomes interested in the Dead Sea Scrolls and travels to Israel to study them. He learns about a sect called the Zadokites, who believed in reincarnation and practiced a ritual involving a psychoactive mushroom that they called “the living water”. He also becomes fascinated by a young woman named Kirsten Lundborg, who claims to have memories of her past lives. He experiments with the mushroom and tries to find out if he can access his own past lives or communicate with VALIS through logos2.
The PKD VALIS trilogy is a complex and challenging work that blends science fiction, philosophy, religion, and autobiography. It reflects Dick’s personal quest for meaning and transcendence in a world that he perceived as chaotic and deceptive. It also raises many questions about the nature of reality, God, humanity, and salvation13.
Learn more:
1. en.wikipedia.org
2. en.wikipedia.org
3. goodreads.com
the concept of logos being in the dead sea scrolls sleeping
Searching for: the concept of logos being in the dead sea scrolls sleeping

Searching for: the concept of logos being in the dead sea scrolls sleeping
Dead Sea Scrolls
Explore
I’m sorry, I don’t understand your question. Do you want to know more about the concept of logos in the Dead Sea Scrolls, or are you asking something else? Please clarify your query so that I can help you better. 😊
The Dead Sea Scrolls are a collection of ancient Jewish manuscripts that were discovered near the Dead Sea in the 20th century. They contain many texts from the Hebrew Bible, as well as other religious and sectarian writings. Some of these writings reflect the beliefs and practices of a group called the Essenes, who lived in a community at Qumran near the caves where the scrolls were found1.
The concept of logos is a Greek word that means word, reason, or principle. It is also used in Christian theology to refer to Jesus Christ, who is the Word of God made flesh. The Dead Sea Scrolls contain some references to logos, especially in relation to the figure of Melchizedek, who is seen as a heavenly priest and king who will bring salvation to the righteous2. The scrolls also use logos to describe the wisdom and power of God that is revealed through his words and deeds3.
Learn more:

en.wikipedia.org
academia.edu
logos.com
bbc.com
en.wikipedia.org

in the valis triology he talked about the dead sea scrolls containing the logos in the text
Yes, you are right. Philip K. Dick was interested in the Dead Sea Scrolls and the concept of logos in his VALIS trilogy. He used some of the ideas and terms from the scrolls to create his own vision of a cosmic mind or God that communicates with humans. He also incorporated some elements of Gnosticism, Christianity, and Judaism in his novels1.
One of the sources that Dick used for his novels was The Dead Sea Scrolls in English by Geza Vermes2, which is a translation and commentary of the scrolls. Dick also read other books and articles on the scrolls and their relation to early Christianity and Jewish sects3. He was especially fascinated by the figure of Melchizedek, who appears in some of the scrolls as a heavenly priest and king who will bring salvation to the righteous.
Dick also used the term logos to refer to the voice or message of VALIS, which he sometimes called Zebra or Sophia. He associated logos with the Gnostic idea of Holy Wisdom, who is the female aspect of God and the source of creation. He also connected logos to Jesus Christ, who is the Word of God made flesh in Christian theology. Dick believed that logos reveals the hidden truth about reality and history to those who are able to perceive it.
The VALIS trilogy is a complex and challenging work that blends science fiction, philosophy, religion, and autobiography. It reflects Dick’s personal quest for meaning and transcendence in a world that he perceived as chaotic and deceptive. It also raises many questions about the nature of reality, God, humanity, and salvation.
Learn more:

logos.com
logos.com
logos.com

what did terrance mckenna say about this particular topic
Terrance McKenna was a psychedelic explorer, writer, and speaker who was interested in the connections between mysticism, consciousness, and culture. He was also a fan of Philip K. Dick and his VALIS trilogy, which he discussed in some of his lectures and interviews12.
According to McKenna, Dick was a visionary who tapped into the hidden dimensions of reality and received information from a higher intelligence. McKenna said that Dick’s experience of the pink beam of light was similar to his own encounter with a voice that he called the Logos during his psychedelic trips in the Amazon. McKenna believed that the Logos was a cosmic mind that communicated through symbols, synchronicities, and dreams1.
McKenna also shared Dick’s fascination with the Dead Sea Scrolls and the concept of logos in them. He said that the scrolls contained secrets about the origin of Christianity and the role of psychedelics in ancient religions. He suggested that the scrolls were written by a group of Jewish mystics who used a psychoactive mushroom called Amanita muscaria to access the Logos and prophesy about the coming of the Messiah2.
McKenna admired Dick’s courage and creativity in exploring these topics in his novels. He said that Dick was one of the few writers who dared to question the nature of reality and challenge the dominant paradigms of society. He also said that Dick’s work was relevant for our time, as we face a crisis of meaning and identity in a world that is becoming more chaotic and deceptive12.
Learn more:
1. en.wikipedia.org
2. en.wikipedia.org
3. amazon.com

so the idea of the dea sea scrolls as an engram or manifestation of a meme as an physical artifact
That is an interesting way of looking at it. An engram is a term used in neuroscience to refer to the physical trace of a memory in the brain1. A meme is a term used in cultural studies to refer to a unit of information that spreads from person to person and evolves over time2. A physical artifact is a term used in archaeology to refer to a material object that reflects the culture and history of a society.
You could say that the Dead Sea Scrolls are an engram or manifestation of a meme as a physical artifact, because they are:
An engram: They are the result of the thoughts and beliefs of the people who wrote them, and they preserve their memory for future generations.
A meme: They are a source of information that has influenced and been influenced by other religious and philosophical ideas, and they have changed and adapted over time.
A physical artifact: They are a tangible evidence of the culture and history of the ancient Jewish sects that produced them, and they provide clues about their way of life and worldview.
Of course, this is just one possible interpretation. There may be other ways of understanding the Dead Sea Scrolls and their significance. What do you think? 😊
Learn more:

directorsblog.nih.gov
smithsonianmag.com
3. gladworks.com

What is the history of memes?
How do archaeologists study artifacts?
What are some other examples of engrams?


mini-yetus: collect all the changes for processing

Collect all the changes in the current branch for processing, this allows us to get yetus output before committing the changes and frees us from the hassle of rebasing to fix the yetus complaints before pushing.

Interesting, I'm on your branch, testing the tool:
➜  eve git:(miniyetus_imp) ✗ make mini-yetus | tail -n1 | awk '{print $5}' | xargs -I dir cat dir/results-full.txt
home/nikolay/projects/eve/pkg/nvidia/Dockerfile:30:25:hadolint: invalid flag: --keep-git-dir
home/nikolay/projects/eve/pkg/pillar/containerd/oci_test.go:19:2:revive: should not use dot imports https://revive.run/r#dot-imports
home/nikolay/projects/eve/tools/parse-pkgs.sh:15:8:shellcheck: note: Double quote to prevent globbing and word splitting. [SC2086]
home/nikolay/projects/eve/tools/parse-pkgs.sh:64:8:shellcheck: note: Useless echo? Instead of 'echo $(cmd)', just use 'cmd'. [SC2005]
home/nikolay/projects/eve/tools/parse-pkgs.sh:64:8:shellcheck: warning: Quote this to prevent word splitting. [SC2046]
home/nikolay/projects/eve/tools/parse-pkgs.sh:74:14:shellcheck: note: Use $(...) notation instead of legacy backticks `...`. [SC2006]
home/nikolay/projects/eve/tools/parse-pkgs.sh:74:9:shellcheck: warning: Declare and assign separately to avoid masking return values. [SC2155]
home/nikolay/projects/eve/tools/parse-pkgs.sh:84:10:shellcheck: note: Double quote to prevent globbing and word splitting. [SC2086]
home/nikolay/projects/eve/tools/parse-pkgs.sh:92:16:shellcheck: note: Double quote to prevent globbing and word splitting. [SC2086]
home/nikolay/projects/eve/tools/parse-pkgs.sh:92:8:shellcheck: note: Double quote to prevent globbing and word splitting. [SC2086]

It still checks the files that are untracked leftovers from the previous branches I was on...

It still checks the files that are untracked leftovers from the previous branches I was on...

I'm not sure why you are seeing untracked files, according to git docs :

git diff : Changes in the working tree not yet staged for the next commit.
git diff --cached : Changes between the index and your last commit; what you would be committing if you run git commit without -a option.

I think the reason is that the branch is not rebased on the latest master...
➜  eve git:(miniyetus_imp) ✗ git diff --name-only master..HEAD | cat
.github/workflows/eden_gcp.yml
.hadolint.yaml
.spdxignore
.syft.yaml
CONTRIBUTING.md
Makefile
docs/VTPM.md
images/modifiers/nvidia.yq
pkg/alpine/mirrors/3.16/community
pkg/alpine/mirrors/3.16/main
pkg/nvidia/Dockerfile
pkg/nvidia/README.md
pkg/nvidia/build.yml
pkg/nvidia/cdi/jetson-orin-nano.yaml
pkg/nvidia/cdi/jetson-xavier-nx.yaml
pkg/nvidia/patches/0001-Allow-to-build-commands-for-musl-libc.patch
pkg/nvidia/scripts/nv-init.sh
pkg/nvidia/scripts/process-cdi.sh
pkg/nvidia/udev/rules.d/90-eve-nvidia.rules
pkg/pillar/containerd/oci.go
pkg/pillar/containerd/oci_test.go
pkg/pillar/hypervisor/kvm.go
pkg/pillar/hypervisor/xen.go
pkg/xen-tools/patches-4.15.0/16-imammedo_x86_acpi_use_offset_instead_of_pointer_when_using_build_header.patch
tools/mini-yetus.sh
tools/parse-pkgs.sh
tools/spdx-check.sh

let me make it a bit more user-friendly!

Ah, ok. It works fine if use the up-to-date version of master now... So, nevermind...


Not mark Days to select

Hi, Today I Download the component but exist a problem.
The problem is:
I add the respective props in component, similar that:
HTML

Script
import YearCalendar from 'vue-material-year-calendar'
components: {
YearCalendar
},
Data:
year: new Date().toISOString().substr(0, 4),
activeDates: [],
activeClass: "helloClass",
Style
.helloClass {
background-color: red;
}
But never change the color when select, so I decided to inspect the element with devtools in Chrome I saw the class never change, always  the class is: class: "day"
After that check the file MonthCalendar.vue
In line 169
if (dayObj.active) oClassList[dayObj.className] = true
dayObj.className is not define => undefined
I resolve this way:
Add this lines in the same file (MonthCalendar.vue).
HTML
Add
@click="changeActive(dayObj)"
{{ dayObj.value }}
**script**
In part the methods add.
changeActive(i){
      i.active=!i.active
    }
And works again.

HI @cocuzlan
In YearCalendar.vue
lines 135 if (dayjs(oDate.date).year() !== this.value) return (2021 !== '2021' )
If the year prop use String, then it will not mark any date.
I will fix it and publish new version to npm. Thank you!


Throwing custom exceptions from contributed command execution shows cryptic error popup

VS Code Version: 1.53.2
OS Version: MacOS 11.2

Hi guys,
Before 1.53, throwing exception with custom error message used to show up as error message popup by default. However with new update it shows up some cryptic error message instead.
Running the contributed command: '_vscode_delegate_cmd_klan2ooe' failed.

Is this new change by design? or a bug?
Steps to Reproduce:

Throw exception (with custom error message) from any contributed command.

throw new Error("Custom error message");

It now shows cryptic error message.
It used to show the custom error message that I've provided with thrown exception.

Does this issue occur when all extensions are disabled?: N/a

Thank you.

Is this new change by design? or a bug?

This is not new and by design. The "cryptic" message is stand-in command identifier that we generate whenever the UI should execute an extension command that cannot be send to the renderer process. That's due to non primitive arguments being passed. So, instead of sending  complex arguments back and forth (which costs and likely breaks them), we keep the actual command and its arguments cached and invoke them via such indirection commands.
Tho, I do agree the error message isn't very pretty but that actually hard to control at that level. What is your expectation? Do you want your error to show? Your command id to show? How do you trigger that command?

Well, it used to just show my custom error message that I passed in new Error('custom message').
I would like it if it kept the same behavior.

In our case (#116953), the command is implemented in the language server, and triggered by user clicking a codelens created by the language server. The language server provides the error details.
Ideally I'd like to see the error details and the original command, but it looks like that's not available from VS Code API level.

I would like it if it kept the same behavior as before.

We didn't change anything, it has always been like that when invoking a command that is using non primitive arguments.

Ideally I'd like to see the error details and the original command name, but it looks like that's not available from VS Code API level (?).

Wrt the details it's actually the same problem. There is no guarantee that we can actually send those details to the renderer. But I agree that we can try better

This now shows the actual extension identifier, not the indirection command anymore. Tho, we don't show the actual error because it's unclear how/if it can be serialised

To verify,

yo code an extension
use the command and code lens provider from below
run and press a codelens
make sure the right error shows

vscode.commands.registerCommand('extension.helloWorld', async (args) => {
        throw new Error('hello world')
 });

vscode.languages.registerCodeLensProvider('plaintext', new class implements vscode.CodeLensProvider {

    async provideCodeLenses(document: vscode.TextDocument, token: vscode.CancellationToken) {

        return [new vscode.CodeLens(new vscode.Range(0, 0, 1, 0), { command: 'extension.helloWorld', title: 'Hello World' })]
    }
})

We have the same scenario in Java extension. The Java extension will contribute a command, which would be used by other extensions. The contributed command may throw some custom exceptions in specific format so that the caller of this contribute command can handle different situations by different exceptions. Before 1.53, it's programmable for us to catch these exceptions and works well. Currently we can only get such as Running the contributed command: xxx failed, instead of actual custom exceptions.
@jrieken It seems the actual exception is caught by VS Code during the executing process. Can extensions get the actual exceptions when calling the command programmatically like before, by vscode.commands.executeCommand(...)?

We have the same scenario in Java extension. The Java extension will contribute a command, which would be used by other extensions. The contributed command may throw some custom exceptions in specific format so that the caller of this contribute command can handle different situations by different exceptions. Before 1.53, it's programmable for us to catch these exceptions and works well. Currently we can only get such as Running the contributed command: xxx failed, instead of actual custom exceptions.
@jrieken It seems the actual exception is caught by VS Code during the executing process. Can extensions get the actual exceptions when calling the command programmatically like before, by vscode.commands.executeCommand(...)?

Yes. I totally agree with @CsCherrYY.
Nested command execution and catching original exception is totally messed up now.


[Documentation Addition]:  show all the latest contributors in README

Also, describe the documentation you want to add. ✍
We can display all the active and latest contributors on README.md file
Are you contributing for a program?
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

i want to work on this issue

I feel I can contribute , Can u please assigned me this work??

Assigned this issue to you. Kindly complete it within a week. All the best to your work.


Add the hand note in the TEI transformer

This is one of the few fields not completed from the original mapping.
We should add these notes to the TEI transformer, so they appear on the website.
Example: https://github.com/wellcomecollection/wellcome-collection-tei/blob/726a1b8f0afee08e07ad2c7f07458d92bb2f510a/Hebrew/Hebrew_A_31.xml#L100-L103
This comes after #5558.

It looks like this is already the case.  I have added a test to show it.  The example data in the description is already visible on the site.

Having looked at the mapping document, it is not handNote that is missing, but handDesc (without handNotes).  Is that what is intended in this ticket?
The only example of this I can find is here: https://github.com/wellcomecollection/wellcome-collection-tei/blob/f5e8ee62823098d6ab8682d4c196a9d20ea16fc8/Spanish/WMS_Amer.19.xml#L92
$ find . -name '*xml'  -not -path '*Templates*' -not -path '*.idea*' -not -path '*docs*'| xargs -J % xml sel -B -I -N 't=http://www.tei-c.org/ns/1.0' -t -c '//t:handDesc/node()[not(local-name()="handNote")]' '%' 2>/dev/null | sort | uniq

Signature, rúbrica, and insertions in author's hand.

$ find . -name '*xml'  -not -path '*Templates*' -not -path '*.idea*' -not -path '*docs*'| xargs -J % xml sel -N 't=http://www.tei-c.org/ns/1.0' -t -i '//t:handDesc/t:p' -f -n '%' 2>/dev/null
./Spanish/WMS_Amer.19.xml


add four system functions of PostgreSQL.

What problem does this PR solve?
Add four system functions used for gitlab initialization.

pg_get_expr
generate_subscripts
format_type
col_description

Codecov Report

Merging #38 (8bb8f85) into main (5fd0d9f) will decrease coverage by 11.91%.
The diff coverage is 30.76%.

@@             Coverage Diff             @@
##             main      #38       +/-   ##
===========================================
- Coverage   89.91%   78.00%   -11.92%
===========================================
  Files           7       41       +34
  Lines        6515    14973     +8458
===========================================
+ Hits         5858    11679     +5821
- Misses        553     2600     +2047
- Partials      104      694      +590


Fix bug that sets the default weight to kg instead of using a length …

Update the default weight unit. mm is a length attribute. Setting to kg to keep it on par with the metric system currently.

Thanks @dewebdesigns Could we get a changelog entry please then we can get this in.


Avoid assigning the same pictures twice to a variant

Add unique_together to VariantImage.
Add data loader for product variant images.

Impact

[x] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[x] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[x] Changes are mentioned in the changelog

We'll need a migration that removes any existing duplicates before trying to add a unique index.


Interval - Value not Allowed

First off, thank you! Awesome little app.
I just noticed the interval / BPM feature and when I hit "flash" with the default setting I get this error.

The BPM feature works great. It would be awesome if it were the default (I'm thinking to use it for CPR). I know it's outside the scope of this app, but an option for a tone along with a beep would be useful for that. Also, a quick access tile to the BPM would be neat if it's simple.

On which app version are you on?

On which app version are you on?

version 2.3.6
com.cyb3rko.flashdim
versionCode 28
targetSdk 35
minSdk 33
Installed: Oct 8, 2024; 23:45
Updated: Nov 26, 2024; 11:27


[bug] $projectAlias not replaced correctly in system command

Describe the bug
Consider the following config:
  pnpm.build:
    command: pnpm --filter $projectAlias --prod deploy $projectRoot/pnpm.out
    inputs:
      - dist
      - package.json
    platform: system

When calling moon run svc-batman:pnpm.build we receive an error from pnpm that no projects match.
The --log trace output indicates that the command is being constructed as:
[trace 13:18:06] moon:utils:process Running command /bin/zsh -c pnpm --filter svc-batmanAlias --prod deploy /Users/user/code/services/svc-batman/pnpm.out (in ~/code/services/svc-batman)

It looks like $projectAlias is being replaced out of order as svc-batmanAlias.
Steps to reproduce
See above
Expected behavior
$projectAlias is interpolated as svc-batman
Screenshots

Environment
MacOS Ventura
moon 1.1.1
Additional context

@shellscape Projects technically support multiple aliases but we only use the 1st one for the token. Can you verify if there's more than 1? moon project <id> should show them.

Not sure what you mean but here's the output
→ moon project svc-api

 SVC-API

Project: svc-batman
Alias: svc-batman
Source: services/svc-batman
Root: ~/code/services/svc-batman
Language: javascript
Type: unknown

Are you sure that's not a replacement error? It doesn't look like $projectAlias is being fully replaced because it's ending up as "svc-batmanAlias". Using just $project works as expected.

Your right, I missed that initially. Should be an easy patch.


Warning: non-conforming DPX total image file size

I am trying to troubleshoot something.  I have a set of DPX sequences from a film scanning vendor.  When I rawcooked them I get a warning.  This is what it looks like for one of the sequences.  The hex value changes with each sequence.
Warning: non-conforming DPX total image file size (x17373).  At least 1 file is not conform to specifications.
Is the hex number the file size of the offending file or something else?  I have checked the file size of some of the DPX files in the sequence referenced above but none of them are close to that.  Can you explain this situation to me please?
Thx.

Thanks.  That explains it.  And, the number of occurrences 17373 matches the number of files in the sequence.

So the DPX file is not conforming to the specs, and RAWcooked does its job by warning you about that.
Depending on other file characteristic, it may be just the tool which is buggy by "forgetting" 4 KiB or maybe there is an extra (non conforming) content after the file not part of the DPX content.
You can disable the conformance checker if you prefer, with  --no-conch.


Fix critest runtime should support attach [Conformance]

What type of PR is this?
/kind bug
What this PR does / why we need it:
We now indicate via a dedicated DONE_PACKET that we're finished writing all data, because we have no way to send io.EOF via the Rust API. This will fix the attach critest and allows the code paths to proceed.
Which issue(s) this PR fixes:

Fixes https://github.com/containers/conmon-rs/issues/410
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

Fixed critest case `runtime should support attach [Conformance]`

Codecov Report

Merging #515 (d97453c) into main (20eb3b3) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head d97453c differs from pull request most recent head 154694b. Consider uploading reports for the commit 154694b to get more accurate results

@@          Coverage Diff          @@
##            main    #515   +/-   ##
=====================================
  Coverage   6.89%   6.89%
=====================================
  Files          2       2
  Lines         29      29
  Branches      17      17
=====================================
  Hits           2       2
  Misses        27      27

/lgtm


(67) Missing I18n translations cause test failure

Set up the i18n-tasks gem and associated RSpec test, to warn us of references to translations in our code, where we haven't set the associated key in config/locales/*.yml.
It also warns if the locale files aren't normalised (i.e. sorted keys etc). If this particular change is unpopular, we can remove the associated test in future.

@lozette Actually whilst the IATI codelist didn't need normalising, I'd forgotten to include that folder in the list of I18n file that were scanned.
Have now fixed this, and also renamed iati.yml to iati.en.yml as this is the English version of the codelists.


HeadingLink to support routers components

Hey,
I've been using LinkBlock with {Link} component from react-router, could you consider rebuilding HeadingLink to take an object (such as Link) as it linking object (instead of just take 'href' prop)
Maybe I can use a  with a  tag inside ? Currently, I'm wrapping Heading with Link.
Thanks :)

you might be interested in #36


Remove use of dangerouslySetInnerHTML

This removes the use of dangerouslySetInnerHTML which can be problematic. Use preact-markup instead which is a much safer way to set HTML which may come from user input.

I think this is ready for merge once we have a plan for releasing. I'm going to add the tag change of span -> div as a note to the release notes.
I'm not sure if we should keep releasing this under the ^2.0.0-preactx prerelease or bump it to version 3; I'll have to double check how major versions work with pre-releases. I think since we are still in pre-release it might be allowed to add a new breaking change without bumping the major version.

The more I look into SemVer pre-release tags, the more it looks like I've been using them wrong by incrementing the left hand size of the version number in the preactx tag. We probably should have been publishing as 2.0.0-preactx.1, 2.0.0-preactx.2 and so on.
For this change I'm going to bump the version to 2.3.1-preactx just to continue with the conventions the preactx tag has been using so far. Going with patch version because this is a bug fix, and being under a pre-release tag automatically implies it is an unstable/breaking version. I'll add notes about changing the div markup to the release notes.


Update your as you "before your delete the service," under 'Delete your service' paragraph

[Enter feedback here]

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

ID: fb4b8145-27ae-8817-6d85-969aa85e601d
Version Independent ID: 00f897cd-5c20-cb3b-9c17-5f75eddd1715
Content: What is Azure Private Link service?
Content Source: articles/private-link/private-link-service-overview.md
Service: private-link
GitHub Login: @sumeetmittal
Microsoft Alias: sumi

@GouravGit , thank you for your feedback. We'll review this and get back to you shortly!

@GouravGit , the doc has been updated to fix the typo as suggested.

We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


implement repr in SlotSet

rasa logs now include better information about SlotSet events returned by custom actions

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Explore improvements to Command Center mode switching

The command center interaction to switch modes could be more discoverable and intuitive to use. Tracking UX explorations here.

A couple of explorations:
With Tabs
Probably the most significant departure from what we have. This would make it much more discoverable to switch to the most often used modes.

With Icon In Text Field
Since users may miss the existing icon given that the quick pick covers it once the command center is opened, we could move the icon to instead be nested with or nearby the input itself. Icon used here is a placeholder. Not yet sure what would the best to use here.

Change Existing Icon
Simply uses a different icon with the same UX we have today to indicate that there are more options. Icon used here is a placeholder as before.

@misolori @jrieken thoughts? I'm most interested in the tabs idea despite it being a bigger change.

I like the direction of the Tabs design, though I'm not entirely sure why but I'm not a fan of exactly how it looks. I think I'd prefer the tabs under the input, but hierarchically that wouldn't really be correct, as the input changes too. Maybe its the left alignment or size/style of the tabs.

Agreed that the tab approach feels stronger but I think it's missing the ability to switch modes based on the first character (empty for file search, > for command palette, @ for symbols, etc.) Would definitely be curious to iterate on that concept and making it seamless between switching modes.
Have you considered pinning entries into the quick pick for the modes?

I think it's missing the ability to switch modes based on the first character (empty for file search, > for command palette, @ for symbols, etc.)

I've been thinking about ways to handle this. For the permanently pinned tabs, we could just switch to that tab when the character was entered. For options folded into more, we coudl do something slick like hiding the tabs or even just temporarily adding a tab for the active mode. Kind of like how tab overflows tend to work. I'll try a few options and will come back with some examples.

Have you considered pinning entries into the quick pick for the modes?

I had a sketch for something like this—similar to what you had in mind? Not sure about the input filtering behavior given the pinned mode entries but curious to hear what you had in mind here.

With Tabs
Here's a more complete idea of how tabs could work:

Common modes are available as visible tabs
Other modes can be entered by selecting from the "More" list
Can use the input to automatically switch between modes (instead of manually clicking tabs)
Clearing the input defaults to the "Go to File" list
Tabs hide when the current mode isn't available as a visible tab

I also wonder if we could enable users to pin/unpin their own set of tabs.
https://user-images.githubusercontent.com/25163139/181119176-c8d33fa9-9017-4cf7-a41c-2674fa7cf5d2.mp4
With Pinned Entries
Also explored how this could work. Definitely a bit lighter and cleaner with the tradeoff that you would have to explictly select the files list now instead of it being the default list. That said, I like that the default view includes a flat list of recent searches/commands.
https://user-images.githubusercontent.com/25163139/181120863-ee758dab-3182-4bf3-b24b-6ac0bff2f0d5.mp4

Dropping some additional feedback even though we already talked about this offline:
Tabs

I think, overall, this is a good direction to aim for the future with a possible setting to disable the tabs (for those who don't want them)
I'd try to style them just like our panel tabs and use the tooltip for the keyboard shortcut
Would be worth to think about how to tab between the different modes, I remember in Sketch Runner this was smooth, but it gets a little tricky with the first character indicator
The tabs appear to take up a decent amount of vertical real estate, i wonder if we'd want to have a "compact" version of the tabs? We do that in the secondary side bar but use icons (less helpful here i think).

Pinned Entries

This feels like a practical thing we could introduce today
The first entry usually is "selected" in the quick pick, do you envision this starting at the file or modes section?
A setting for this would also be nice

Latest options from this week. I have a strong lean towards the first one as something we could do today.
Prefix Pill + Placeholder Text
https://user-images.githubusercontent.com/25163139/185502225-ee1d6405-2604-491c-bf73-94f06e46ec0f.mp4
A nice bonus here would be showing the name + keybinding on hover:

Tabs
https://user-images.githubusercontent.com/25163139/185501933-71aba1ba-0c62-4237-845c-25cdbce77a01.mp4
cc @TylerLeonhardt

Love the simplicity of the character "badge" or "pill", makes it clear that it's the "mode" and can be easier to learn for new users.

Pushing the pill idea bit further with an idea from @TylerLeonhardt to show other mode optoins on click of the pill + a quick way to see more via the ? mode.

The command centre's clickable region doesn't extend to the top of the window when VSCode is maximized; so it's not a good click-target. Enabling it also increases the title-bar height; which might be okay on HD screens but is yet more lost screen real estate on 1366x768!

The command centre's clickable region doesn't extend to the top of the window when VSCode is maximized; so it's not a good click-target. Enabling it also increases the title-bar height; which might be okay on HD screens but is yet more lost screen real estate on 1366x768!

@Kroc could you open a separate issue? I've scoped this one to the mode switching interaction now that there are several active command center UX efforts.

Cross-posting a demo from #158638 to showcase the latest ideas here:

Use a "home" view of the command center to showcase available modes w/ friendly names
Continue to use pills for prefixes, but only show friendly mode name + keybinding on hover/focus. No need for the context menu on click given the above change.

https://user-images.githubusercontent.com/25163139/188745576-d3337aba-db82-4c79-a142-1882f18743b5.mp4

Closing given the convergence with #158638. Will continue tracking the pill prefix idea specifically in #158064.


Allow labellers to opt out of receiving reports

Is your feature request related to a problem? Please describe.
Currently, all subscribed labellers are available for selection in the report UI. This causes a certain level of broken expectations (and sometimes issues, if people report harmful posts to them) for labellers that do not act on such:

https://bsky.app/profile/sonasky.bsky.social/post/3l2j2l7korq2l
https://bsky.app/profile/adorable.mom/post/3l2bhfsekdk2a

Describe the solution you'd like
Similarly how we can control our Chat availability, it should be possible for labellers to opt out of the report feature. This should hide them from the report UI and Ozone should also silently discard reports in this mode.
Describe alternatives you've considered
The labellers could alternatively still be visible but not selectable in the reporting wizard, possibly with a short notice that they don't accept reports.
Additional context
These are my currently-subscribed labellers, at least a third of which do not meaningfully process reports, either because they are self-service or because they are fully automated in other ways:

(I'm still trying to figure out what Anti-Alf Aktion actually does.)

Hmm, is it required that a labeller implements the endpoint for accepting reports? If not, what happens if it just doesn't (for labellers that are hand-built and not based on Ozone) and you try to report something to one?


Ceedling now returns the correct status and more spec test for verification.

This commit is designed to fix #148, #117, and #174. It also adds more system tests with altered exit statuses to confirm behaviour of failures and when to expect something to return failure.
@mvandervoord  I see you made a commit to address this issue as well, so  if you have some spare time I'd like to discuss both commits and make sure they are compatible/make sense as a solution to the issue.

@dpostorivo THANK YOU for the collection of tests! Your actual solution is about to be irrelevant, only because we've decided to change directions in how we're handling the passoff to rake (I think you'll like to update a lot). Your tests are going to keep me from screwing up the merge, though, and prove that my changes actually do what we hope they do. THANKS!
I'm happy to chat more! Ping me if you'd like to chat sometime.

@mvandervoord Ok good to know. If you need any help with that let me know or if there's anything you want me to look into let me know.
Now that I understand the system tests, I think I'll make a version for the gcov plugin to help address #96 since testing the whole system is the best way to do that. It's a pretty good way to test the overall process and the only downside is it it balloons the time it takes the CI to run (these test increased the runtime of tests by).
After that, I do plan on going back to testing and documenting the individual ceedling modules. I'm learning a lot by going through them. I may try and find where #147 is happening, but I have not been very successful in locating the source of the issue and it does not happen in any earlier version of ceedling.


[sdk] Implement Converter abstraction for building converter plugins

This PR implements a Converter abstraction under the Pulumi.Experimental.Converter namespace which allows easily building and serving converter plugins from dotnet.
Minimal example:
using Pulumi.Experimental.Converter;

var converter = Converter.CreateSimple(request =>
{
    return ConvertProgramResponse.Empty;
});

await Converter.Serve(args, converter);

In F#:
let converter = Converter.CreateSimple (fun request -> ConvertProgramResponse.Empty)

let serve args =
    Converter.Serve(args, converter)
    |> Async.AwaitTask
    |> Async.RunSynchronously

[<EntryPoint>]
let main(args: string[]) =
    serve args
    0

Here, you give the converter implementation to Converter.CreateSimple, this gives you an instance of SimpleProgramConverter which implements Converter. Then you use that instance as an argument for Converter.Serve
Additionally, users can inherit Converter in a dedicated class and override the ConvertProgram function.
The ConvertState function is not added yet, that can be done in a separate PR when necessary.
Note that I executed dotnet run sync-proto-files to get latest additions from Pulumi (including the converter definitions)

We should copy the same style as Experimental.Provider here.
That is interface based and with read-only records for requests (e.g. see CheckRequest at https://github.com/pulumi/pulumi-dotnet/blob/main/sdk/Pulumi/Provider/Provider.cs#L24-L46)


Small OrbitService tweaks

Remove OrbitSleepMs, used only in one place, use Sleep like the rest of Orbit
Handle Ctrl-C in OrbitService
Simple line removals in OrbitService.cpp
Add -asio_port command line flag for OrbitService

(did not nice draft status)


chore: improved integration tests

deprecated TEST_SERVER_URL env var
migrated to a fixture that provides test http client

/deploy-review

/deploy-review

/deploy-review


History 3G: Parse expressions, add to import state

Now that we can import simple fields, another step is to support expressions. By this I mean if the value for expr: for an item in the var_list is not a single field we need to parse the expression, determine what are the actual fields in the expression, and add those to the import state of the HistoryCollectionGridComp we are in. The current parser does have a routine to take an expression and return the variables in the expression in a gFTL string vector I believe.
Note that there are a few issues with this.

How do you know that you don't have just a single field. My strategy in the old History was to determine if string had something that could not be in a name, basically any of these characters +-/*(). Klugey but I can't really think of a better way...
Since we will be denote fields with expr: component_name.field_name we need to check if the current expression parser has issues with field names with a "." in them and because it also looks for constants in the expression i.e. 1.0, 0.9, .8 etc..., just need to check
Probably should rename MAPL_NewArthParser.F90 to something better, could use some unit testing etc in the long term...

We may have to compromise on syntax if "." becomes problematic.  ":" should be safe.   "/" is ideal, but we know that would be a problem for the expressions.


Revert start cmd Dockerfile

Thank you for contributing to Dashy! So that your PR can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category:

Revert https://github.com/Lissy93/dashy/issues/1543

Overview

Temporarly revert the start cmd of the docker container to build-and-start so changes to files for example userstyles don't reqire manual rebuild:

Issue Number (if applicable) #1559
Code Quality Checklist (Please complete)

[x] All changes are backwards compatible
[x] All lint checks and tests are passing
[x] There are no (new) build warnings or errors
[ ] Bumps version, if new feature added

Hello.
Is there any reason to revert to build-and-start instead of just start?
I have a tiny vps and it got stuck at building step.


Allow more short versions for keyboard maps

If we have parentheses mapped to wp, then we'd like to be able to just press p after aw (for wrap). However, if we add wwp for escaped parentheses, that breaks, because they both end with p. It would be nice if wp could own p because it has fewer letters to drop?
Fwiw then we'd have an inconsistency because you might expect wp to be escaped parens, as they both just drop that leading w. Maybe we solve that by using wep instead of wwp, so then it shortens to ep. I believe we'd still need this fix tho if we wanted wp to shorten to p in that case, as they both end with p

Cc/ @josharian

Makes sense. The rule would be that you can break (some) ambiguities by choosing the shortest unique continuation?
For the near term, seems like adjusting the shortcuts to be unambiguous might be the easiest fix.

The rule would be that you can break (some) ambiguities by choosing the shortest unique continuation?

Yeah I believe so

For the near term, seems like adjusting the shortcuts to be unambiguous might be the easiest fix.

That does make escaped pairs a bit awkward because we'll need to find suffixes for them that differ completely from their unescaped counterparts 😕

Yeah, that’s how I landed with the (I thought at the time very clever) double prefix. I’m game to switch to shortest wins, though.


add Chainloop project and company

Thanks!
Closes https://github.com/someengineering/cloud-security-list/issues/1

LGTM, thanks for adding Chainloop!


Add HTTPError to JsonRpcAsyncClientError and update is_retriable logic

Summary
Separating out HTTP Errors from InvalidServerResponse error since some HTTP Errors are retriable.
We saw these errors in AOS logs
[2020-09-18T10:11:01Z ERROR aos::errors] Received non-retryable error: JsonRpcAsyncClientError: InvalidServerResponse Http error code 502 Bad Gateway

This error should be retried.

/land


Adds options for custom controls

Adds the ability to provide extra lookups in an imperative fashion.
With this little addition, you can provide extra controls for table columns that do not correspond to a field on the model, e.g. my model Customer has a property identifier, which is displayed in a table column, and I want to be able to filter by that column.
class Customer(models.Model):
    [...]
    @property
    def identifier(self):
        return 'CUS-{}'.format(self.customer_number.pk)

With the change in this commit, I can construct my view with
class CustomerListView(DatatableView):
    """
    Lists all customers
    """
    model = Customer
    [...]

    def get_custom_field_queries(self, term):
        fqs = super().get_custom_field_queries(term)

        if term.lower() in 'CUS-'.lower():
            fqs.append({'customer_number__isnull': False})
        match = re.match('cus-(\d+)', term.lower())
        if match:
            fqs.append({'customer_number__pk__istartswith': match.group(1)})
        return fqs

Now, when I search for CUS-, I find all customers with a customer number, and when I search for CUS-1, I find all customers whose numbers start with 1

This doesn't change any default behaviour. It's just an extra optional feature


Enum generation using entries in Application Objects

In ApplicationObjectTypes we have done an entry as ApplicationObjectTypeId=1, ApplicationObjectTypeName="Status Type" and IsActive=1. Two enums are being created one with space "Status Type" and another without space "StatusType". Note we also have one table which holds data of status type as we want to display status type in multilingual.

@diptip09 Please share the table data of ApplicationObject and ApplicationObjectType with respect to Status Type

Data in ApplicationObjectTypes Table:
ApplicationObjectTypeId	ApplicationObjectTypeName	StatusId
1	Status Type	1
2	Shipment History Status	1
3	Currency Format MaskType	1
Data in ApplicationObjects Table:
ApplicationObjectId	ApplicationObjectTypeId	ApplicationObjectName	StatusId
1	1	Active	1
2	1	Pending	1
3	1	InActive	1
4	2	Add	1
5	2	Edit	1
6	2	Delete	1
7	3	User Default	1
8	3	ISO	1
Note: Other Enums are being generated properly only Status Type has issue.

Thanks, @diptip09, I have released the new package, Please update rxwebcore CLIwith the version of 1.1.8 .


[FLINK-8283] [kafka] Introduce KafkaOffsetCommitter interface

What is the purpose of the change
This PR is built upon the reworked FlinkKafkaConsumerBaseTest in #5188.
The broken FlinkKafkaConsumerBaseTest is properly fixed only when both #5188 and this PR is merged.
Prior to this PR, offset committing was coupled tightly with the AbstractFetcher, making unit tests for offset committing behaviours hard to compose concisely. For example, we had tests that required mocking the offset commit methods on AbstractFetcher, while ideally, it would be best that those methods are made final (thus, unable to be mocked) to prevent accidental overrides.
This PR decouples offset committing as a separate service behind a new KafkaOffsetCommitter interface. For now, the AbstractFetcher is reused as an implementation of this service, so that this PR does not introduce any more change other than introducing a new layer of abstraction.
Unit tests that verify offset committing behaviour now provide a dummy verifiable implementation of the KafkaOffsetCommitter (instead of using mocks on AbstractFetcher) and test against that.
Brief change log

Migrate AbstractFetcher::commitInternalOffsetsToKafka method to the newly introduced KafkaOffsetCommitter interface.
Let AbstractFetcher implement KafkaOffsetCommitter
In the FlinkKafkaConsumerBase, let "offset committing" and "record fetching" be logically separated to be handled by two services, i.e. namely a KafkaOffsetCommitter and a AbstractFetcher. Physically, the fetcher instance sits behind both service abstractions.
In FlinkKafkaConsumerBaseTest, remove all mocks on AbstractFetcher::commitInternalOffsetsToKafka, and test against a KafkaOffsetCommitter instead.
Additional hotfix 2906968 that fixes a stale comment referring to an old AbstractFetcher behaviour to avoid confusion.

Verifying this change
This PR does not add any new functionality. Reworked test also do not affect test coverage.
FlinkKafkaConsumerBaseTest verifies all changes.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? n/a

It seems like that the mock issue on AbstractFetcher::commitInternalOffsetsToKafka is not the root cause of the test instabilities mentioned in FLINK-8283. It is rather just a separate issue.
Re-opening this PR with a separate issue tag.


Disable auto redirect

Disable auto redirect as the Service API might not support auto redirect

Details of error is mentioned in https://domoreexp.visualstudio.com/MSTeams/_workitems/edit/2958683


[NeuroScopeIO] NeuroScopeIO can not handle file with missing channels but NeuroScope can

Hi all,
I have some .dat files under the NeuroScope format for which some channel have been deleted for quality control.  I find that NeuroScopeIO  can not open these files while NeuroScope can with a warning.  I am wondering if it would be possible to add in a similar feature that allows NeuroScopeIO to handle files of this type.  Here is a picture of the error.

I can change the section so that it skips the bad channels but I don't know if that will cause more problem down the line


Dataset EndDate filter : condition is on StartDate

Hi
First of all, thanks for the work.
I'm working using Quandl data and tried your project vs Quandl.NET cuz I like the fact there is almost no dependency.
That being said, there is a bug in dataset filters, in the ProcessFilter method
Copy / paste error as you make a condition on StartDate while treating EndDate
That makes a "Nullable filter must have a value" exception when using EndDate filter.
            if (options.StartDate.HasValue)
            {
                filterAdditions.Add($"end_date={options.EndDate**.Value.ToString("yyyy-MM-dd")}");
            }

Shoudn't take more than 30 secs to fix.
I can contribute if you want but a little later as I can't upload from my current station.
Best regards

Did it today : https://github.com/paranoidTwitch/Reckdl/pull/3

Hey, sorry it took me a bit to get to this. Don't get on here every day. Thanks for the PR. I'll check it out and pull it in. I'll need to update the nuget packages as well so the fix propagates. I'll close this issue out once that is done.
Since you are using this, I'm going to convert this to net standard at some point in the near future. Are there any features or anything along those lines you would like to see?

Nuget.org has been updated. The package should be available shortly. Let me know if you have any more issues with this item.

Thanks for the release. I'll delete my fork then.
Not that I can think of as of now. My calls are quite simple and so far so good :)

Cool, I've moved this to .NET Standard 2.0 so the maintenance is not a nightmare. Since its a super breaking change and I'm not super fan of how this is named, I've moved the repo here:
https://github.com/paranoidTwitch/DotNetConnect.Quandl
Ill leave this project up but future maintenance is going to be on the new project.


KAFKA-5236; Increase the block/buffer size when compressing with Snappy and Gzip

We had originally increased Snappy’s block size as part of KAFKA-3704. However,
we had some issues with excessive memory usage in the producer and we reverted
it in 7c6ee8d5e.
After more investigation, we fixed the underlying reason why memory usage seemed
to grow much more than expected in KAFKA-3747 (included in <IP_ADDRESS>).
In 0.10.2, we changed the broker to use the same classes as the producer and the
broker’s block size for Snappy was changed from 32 KB to 1KB. As reported in
KAFKA-5236, the on disk size is, in some cases, 50% larger when the data is compressed
with 1 KB instead of 32 KB as the block size.
As discussed in KAFKA-3704, it may be worth making this configurable and/or allocate
the compression buffers from the producer pool. However, for <IP_ADDRESS>, I think the
simplest thing to do is to default to 32 KB for Snappy (the default if no block size
is provided).
I also increased the Gzip buffer size. 1 KB is too small and the default is smaller
still (512 bytes). 8 KB (which is the default buffer size for BufferedOutputStream)
seemed like a reasonable default.

Review by @hachikuji or @junrao. cc @guozhangwang @becketqin who were involved in previous discussions.

Checked if Jun had any concerns and he didn't so merging to trunk and 0.11.0.

LGTM.


chore: introduce fvm (flutter version manager)

This allows us to have different flutter versions for webapp and mobile. It is needed, because the webapp can be built best with the current beta version. Using latest stable will break the webapp in firefox and we would need to fall back to a different renderer which comes at the cost of performance.
This might be controversial but I was debugging issue #2234 and upgraded flutter to beta which solved an issue at least @holzeis is aware off: the latest flutter version does not work flawlessly in firefox - to make it work we had to switch to a different renderer (which was done ina8646e1 (#2081)). However, this actually comes at performance and responsiveness cost - which I thought was the problem for #2234 (but is not).
Long story short, this PR allows us to use canvaskit rendered on the latest version which provides better performance and responsiveness than html renderer.
Let me know what you think. Alternatively we could simply use beta channel for all apps for the time being and upgrade once the next flutter version is released.

@bonomat Not sure if I completely follow.

Would that mean we have to actively change the flutter version using fvm before running the mobile or web app?
Do we need this beta version to fix #2234 ?

Feels like a lot of complexity if its just for performance improvement, where we currently don't have any big issues.

Would that mean we have to actively change the flutter version using fvm before running the mobile or web app?

No, your normal flow will just work.  just run and just run-web have been changed already.
Only if you want to run flutter manually for anything you will need to remember to use fvm flutter before.

Do we need this beta version to fix https://github.com/get10101/10101/issues/2234 ?

No, but it allows us to use canvaskit again as the beta-channel fixes the issue with the flickering interface in Firefox.
Quote from: https://docs.flutter.dev/platform-integration/web/renderers

HTML renderer
This renderer, which has a smaller download size than the CanvasKit renderer, uses a combination of HTML elements, CSS, Canvas elements, and SVG elements.
CanvasKit renderer
This renderer is fully consistent with Flutter mobile and desktop, has faster performance with higher widget density, but adds about 1.5MB in download size. CanvasKit uses WebGL to render Skia paint commands.

@luckysori / @holzeis
I'll go ahead and merge this. Make sure to rebase your branches against main and install FVM


Show the names of regions only or larger

IMHO it would be better to show the names of the Scene => Region only or at least larger than the UUID reference.  Users don't care about UUIDs (a fact which the core foundry team seem to have missed with this feature).
Great module!  Love how its coming along.

Thanks for the suggestion.
Unfortunately, it is entirely up to the browser (Chrome or Foundry) to decide how to display the drop-down menu. There is limited control available even with CSS to be able to control how the drop-down menu is displayed.
I would much rather see the label as the main text and the value in the smaller font, but I don't know if it is possible.

Just for my own notes:

Impossible to style how the datalist is presented when attached to an input. According to the HTML specification, it is an O/S implementation, not a browser implementation.

Impossible to replace the "input" element inside Foundry's HTMLDocumentTagsElement with a select element, because foundry stores it's input element in a private variable (#input) and adds a private event handler to it (#tryAdd).

Impossible to replace Foundry's HTMLDocumentTagsElement with our own variant, because foundry calls foundry.applications.elements.HTMLDocumentTagsElement.create which isn't overrideable.

I briefly started up Firefox, and it presents the list by only displaying the names of the regions.

Custom dropdown list handling has been added in https://github.com/farling42/fvtt-easy-regions/releases/tag/0.7.2


Proposal: Handle URI correctly according to [RFC 3986], thus [RFC 8089] and related

Howdy, everyone!
Output of helm version:
version.BuildInfo{Version:"v3.9.3", GitCommit:"414ff28d4029ae8c8b05d62aa06c7fe3dee2bc58", GitTreeState:"clean", GoVersion:"go1.17.13"}

Output of kubectl version:
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.2", GitCommit:"f66044f4361b9f1f96f0053dd46cb7dce5e990a8", GitTreeState:"clean", BuildDate:"2022-06-15T14:22:29Z", GoVersion:"go1.18.3", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.4+k3s1", GitCommit:"0dc63334c0db3e7b99244427615e091909fc486e", GitTreeState:"clean", BuildDate:"2022-11-18T18:17:40Z", GoVersion:"go1.19.3", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Any (tested on Rancher Desktop)
Problem
Helm is not following RFC 3986: Uniform Resource Identifier (URI): Generic Syntax and RFC 8089: The "file" URI Scheme, specifically:

Helm plugin downloader cannot use URI in the form: protocol:whatever. Only the form protocol://whatever is "hardcoded"**. Actual hardcoded, weird and redundant constraint does not allow protocols to treat everything after : as "value". This accidentally forbids its use with repos/oci as well (e.g.: protocol:repo/chart or protocol:oci://path/to/oci/chart) but without any proper reason. Moreover, according to RFC 3986, authority is optional. Helm IMHO too strictly assumes that URI must follow http scheme rules despite the fact how particular downloader works.
Implementation and handling of file URI scheme adds Helm-specific ability to use relative paths which is not recommended by default according to RFC 8089. As this is only a recommendation, the use of relative paths is tolerated (for historical and practical reasons) but not in the form file://./somefile or file://../somedir/somefile, but either without specifying scheme and authority (e.g. plain  ./somefile or ../somedir/somefile) or in the extreme case in the form of file:./somefile or file:../somedir/somefile which Helm does not support.

Proposal

Do not rely on // in the URI string but only on scheme (string before first :). This change should be safe and backward-compatible because it is only a constraint check - the plugin always receives from Helm the whole URI, so it does not affect further processing.
Fix how Helm handle URI with file scheme (to accept file:./somefile or file:../somedir/somefile) to mimic other protocols correctly. Left file://./somefile or file://../somedir/somefile available as well but mark it as deprecated to avoid backward compatibility break. (Note: Usage of form without scheme collides with the actual short format of repo/chart so IMHO this should not be allowed.)

Thanks for the report. I'm not very familiar with the plugin logic. Do you have a reference to the helm code which is hardcoding the protocol://whatever form please?

Thanks for the report. I'm not very familiar with the plugin logic. Do you have a reference to the helm code which is hardcoding the protocol://whatever form please?

Sorry but I cannot open PR due to some problem with my PGP key I'm using for signing

/remove-lifecycle stale

I suspect what is needed here is to turn https://github.com/helm/helm/commit/7312533f28cc137a015e7d8efd4c2781aa8b6c6c into a PR?

Issue is still valid and requires further attention.
/remove-lifecycle stale


Decide MSRV for v1

Currently I'm bumping it to v1.74 so we can use all the features we want. Once we're close to an actual release we should make it as low as possible which having to change the code (too much).

Some distro numbers:

Oldest Ubuntu LTS: 1.70
Debian stable: 1.63


Fix Function registration overwrite signature variableArity

Not allow register func(int) and func(int..) at the same time.

We may not fix this problem without this PR https://github.com/facebookincubator/velox/pull/10811.
SignatureMap& signatureMap = map[sanitizedName];
    auto& functions = signatureMap[*metadata->signature()];

    for (auto it = functions.begin(); it != functions.end(); ++it) {
      const auto& otherMetadata = (*it)->getMetadata();
      ...
      VELOX_USER_CHECK_EQ(
          metadata->signature()->variableArity(),
          otherMetadata.signature()->variableArity());

Argument int and int... signature is different, so the functions size is 1, we will register both of them and we cannot run into this changed logic. @mbasmanova


[Popover] Remove fixed positioning dependent on scroll position

I had a look around for any issue that's mentioned this before but couldn't find anything. It's an obvious one, so I suspect its been raised and gone over before so if that's the case apologies for the time wasting.
Problem Description
There's a lot of rubberbanding with dynamically positioned components when scrolling, a good example of this is the Popover:  http://www.material-ui.com/v0.15.0-alpha.1/#/components/popover. Is there a reason material-ui uses a full screen fixed container to position dynamic elements as opposed to absolute positioning on the window, or even absolute in a local container? It would produce a much better UX if components didn't move when scrolling.
Another side effect of this is that you can't easily use hover event listeners to open popovers, since when the full screen layer is created it triggers an onMouseLeave.
Versions

Material-UI: 0.15
React: 0.14
Browser: Google Chrome, Safari

@newoga this is the same as what we're talking about in https://github.com/callemall/material-ui/pull/3144.
Should it be a new issue?

@newoga this is the same as what we're talking about in #3144.

@nathanmarks Yes, I believe so. The  #3144 PR designed to solve this, but the discussion there ended up raising the same points as this issue.
Should we give this issue a new name and track it here and close #3144?

@sampeka We're aware of this issue and have been having some discussions on how to solve this. Many of our components use Popover, so if we fix it there, we'll likely fix it everywhere.
We are also considering using something like react-overlay that could fix this.

@newoga Any update on this?
I'm using material-ui in production and very happy with everything - but the janky popovers on scroll are the one and only thing that users always notice and complain about.
It would be amazing to get this fixed, it's probably the only significant remaining 'con' of using material-ui over the other popular component libraries.
Happy to contribute if there's anything I can do.

@nathanmarks Was this resolved for next? I see the component was migrated, but as there's no Popover demo yet, (and no other popover based component demos to test with), I'm sure of the status.
If not, then perhaps we should be thinking of using Tether?  If it's good enough for bootstrap... :smile:

If not, then perhaps we should be thinking of using Tether? If it's good enough for bootstrap...

Bootstrap moved to popper.js lately. I think that we could be taking advantage of this library for new use cases. As for the v1-beta branch, we have been blocking the scroll with the fixed positioned element. I do think that the issue is solved.


Fix links in readme

Pull request
Description
The history link was broken and while fixing it I noticed the architecture link had bad syntax and the docs.tremor.rs subdomain seems to have been replaced by tremor.rs/docs since this readme was written.
No code or tests were touched. As such I believe there to be no security, performance, feature, or documentation impact.

Coverage increased (+0.009%) to 85.239% when pulling efefb5b8f386c58a53d802de2b9cff4873cea349 on dyladan:patch-1 into 8943ee7db0c3787b0545480bf23e0ae72eeaf488 on tremor-rs:main.


Fix race condition on stopping lsp client

Without this fix, it can cause LSP Client to get stuck randomly at cond.wait() since other thread won't release the lock due to shutdown_flag being active.

Hello Ashish,
Thank you for you contribution.
Can you please elaborate about the scenario, and you code fix the problem?
Thanks!

Hi Yeger,
Here's scenario which this PR is trying to fix:

Create an LSP client and connect to server.
Perform all the operations you want.
Call lsp_client.shutdown().

Try doing this in a script and it would get stuck at last step randomly. The reason being is race condition in the code. Due to shutdown_flag being set, it's possible for lsp_endpoint's thread to exit before notifying about shutdown message's response to the main thread. Due to this, main thread would end up waiting for infinite.
Other approach to tackle this could be not exiting lsp_endpoint w.r.t. shutdown_flag and wait for shutdown method's response from server.


Nothing is playing.

So, i've been checking this out for a while right now and thought about copying the stuff in "examples" (so index + commands), and did everything, (installing Lavalink and stuff).
But for a surprise as me, everything works (the commands and stuff) and if I start (!play ncs limitless &example <-) the bot joins the voice channel, but it does not start playing at all.
Is that a issue or? #

I have tried everything at this point, I have uninstalled and reinstalled erela.js, but unfortunately still the same result (I am using Node.js V14 + Java 13)
I even created a new discord bot with the "Administrator" permission, but still nothing starts playing.

no song may have been found

what do you mean? I just copied the entire thing of the "examples" folder of the code here.

u should have
case "NO_MATCHES":
case "LOAD_FAILED":
   return message.channel.send("something went wrong")

try this

in my play.js?

yes, there is already case "NO_MATCHES": so just add case "LOAD_FAILED":

I already have it, and it was in the example code aswell (I just copied that)
code
discord result

ok, can you send whole code? just to be sure it's ok

yea here, https://sourceb.in/NpC9yT5msi
again it's literary the same code from https://github.com/MenuDocs/erela.js/blob/master/examples/commands/play.js

code looks normal. can you send lavalink log?

yea, https://paste.gg/p/anonymous/174f8c53f62e435ab2fe78babcf74f64

update to Lavalink <IP_ADDRESS>
https://github.com/Frederikam/Lavalink/releases/download/<IP_ADDRESS>/Lavalink.jar

Aha. Got it fixed & working again ^ Thanks for the help :)


bump version to 0.7.2

Hi, my team submitted a PR to add support for icalendars from LotusNotes that was already committed: https://github.com/tritech/node-icalendar/pull/40. However, we would like to use SEMVER rather than the git commit hash to pull in the update. I think this would also be confusing to other npm users, as the update hasn't been published.

Closing to clean up my open prs


Rewrite fit_simulation.md for clarity.

I am learning SciML and its related packages for the first time by going through the SciML tutorials. The fit_simulation was especially confusing, so I attempted to rewrite it for clarity.

I have some remaining questions.

I tried to change x and y to be countable numbers by multiplying them by 100 (and also changing u0 and pguess), but nothing I tried would optimize close to the correct solution. I could only get my initial pguess to move around by order O1 not O10 or O100. How can I allow for larger state variable movement in Optimization.jl, so that the number of wolves is not 0.3?
The callback function was initially defined as a named anonymous function, and I found the syntax of that very confusing. Is there any reason to keep it that way? The documentation  shows callback defined both the original way and as a normal function. If it's all the same, I'd just use a regular function definition.
I could not find any documentation at all for PolyOpt. This tutorial should say something about why it was chosen, especially since the previous tutorial used NLopt.LD_LBFGS instead.

I think I understand the tutorial now, but due to the questions I still have above, I'm not confident about applying these approaches more generally. Hopefully you can review and provide some insight. Thanks.

I tried to change x and y to be countable numbers by multiplying them by 100 (and also changing u0 and pguess), but nothing I tried would optimize close to the correct solution. I could only get my initial pguess to move around by order O1 not O10 or O100. How can I allow for larger state variable movement in Optimization.jl, so that the number of wolves is not 0.3 but 30?

Can you show the code you ran?

The documentation shows callback defined both the original way and as a normal function. If it's all the same, I'd just use a regular function definition.

Yes feel free to change that, there's no reason to prefer the anonymous function, it's just easier to write but makes sense that not for reading for beginners 😅

I could not find any documentation at all for PolyOpt. This tutorial should say something about why it was chosen, especially since the previous tutorial used NLopt.LD_LBFGS instead.

I'll add that in, I just forgot to add it :)

Can you show the code you ran?

None of the commented pguess vectors worked, even after 10x the original number of iterations.
using DifferentialEquations, Optimization, OptimizationPolyalgorithms, SciMLSensitivity
using ForwardDiff, Plots

# Define experimental data
t_data = 0:10
x_data = [100 277 677  97 189 610 140 134 435 325 103]
y_data = [100  26 202 191  32  63 346  51  31 455  91]
xy_data = vcat(x_data, y_data)

# Plot the provided data
scatter(t_data, xy_data', label=["x Data" "y Data"])

# Setup the ODE function
function lotka_volterra!(du, u, p, t)
    x, y = u
    α, β, δ, γ = p
    du[1] = α * x - β * x * y
    du[2] = -δ * y + γ * x * y
end

# Initial condition
u0 = xy_data[:, 1] # [100, 100]

# Simulation interval
tspan = (first(t_data), last(t_data)) # (0, 10)

# LV equation parameter. p = [α, β, δ, γ]
pguess = [100.0, 1.2, 250.0, 1.2]

# Set up the ODE problem with our guessed parameter values
prob = ODEProblem(lotka_volterra!, u0, tspan, pguess)

# Solve the ODE problem with our guessed parameter values
initial_sol = solve(prob, saveat = 1)

# View the guessed model solution
plt = plot(initial_sol, label = ["x Prediction" "y Prediction"])
scatter!(plt, t_data, xy_data', label = ["x Data" "y Data"])

# Define a loss metric function to be minimized
function loss(newp)
    newprob = remake(prob, p = newp)
    sol = solve(newprob, saveat = 1)
    loss = sum(abs2, sol .- xy_data)
    return loss, sol
end

# Define a callback function to monitor optimization progress
function callback(_, l, sol)
    display(l)
    plt = plot(sol, ylim = (0, 600), label = ["Current x Prediction" "Current y Prediction"])
    scatter!(plt, t_data, xy_data', label = ["x Data" "y Data"])
    display(plt)
    return false
end

# Set up the optimization problem with our loss function and initial guess
adtype = AutoForwardDiff()
optf = OptimizationFunction((x, _) -> loss(x), adtype)
# pguess = [1.0, 1.2, 2.5, 1.2]
# pguess = [100.0, 1.2, 250.0, 1.2]
# pguess = [1.5, 1.0, 3.0, 1.0]
# pguess = [150.0, 100.0, 300.0, 100.0]
# pguess = [150.0, 1/100, 300.0, 1/100]
pguess = [150.0, 1.0, 300.0, 1.0]
optprob = OptimizationProblem(optf, pguess)

# Optimize the ODE parameters for best fit to our data
pfinal = solve(optprob, PolyOpt(),
               callback = callback,
               maxiters = 2000)
α, β, γ, δ = round.(pfinal, digits=1)

Yes feel free to change that

Okay, I have done so.

I'll add that in

Thank you for documenting, but "Adam" and "BFGS" still mean nothing to me. Can you add a commit here about why PolyOpt would be chosen in this case as opposed to NLopt.LD_LBFGS? NLopt.LD_LBFGS was described as both "robust" and "performant" in the previous tutorial, so I don't understand why we switched algorithms.

The PR as written is accurate and works, but I will change it to use the above code instead if you know an easy way to get the optimization to converge.

Yeah that doesn't converge for me either. I didn't look too deeply into it though, I am guessing the scaling difference shows up somewhere making things ill-conditioned. Are you finished with the all other changes then?

@Vaibhavdixit02 what are the remaining steps here?

Just the one review comment left above

I'll also reiterate that if someone can provide rabbit/wolf population data with countable numbers that still converges under optimization, that would also make this section easier to understand. y=0.2 wolves doesn't make any sense.


mypy: Remove use of undefined GGUFWriter symbols

Neither add_expert_count nor add_expert_used_count are defined in the version of gguf library we use.  Actually, these symbols are not yet released on pypi, and are available in git repo of the library only:
https://github.com/ggerganov/llama.cpp/commit/799a1cb13b0b1b560ab0ceff485caed68faa8f1f
This makes mypy complain about it.
This patch removes code related to these parameters. If and when we upgrade to gguf version that supports them, we may revert this patch.

Overall, this module seems to be a copy-pasted version of convert.py from gguf library repo. Shouldn't we reuse the code from the original library?.. @xukai92 you added this file here. What's the story behind it?

Squashed this into https://github.com/instructlab/instructlab/pull/1275