Save file picker is repeatedly not working

Error message:
Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.
Browser: Chrome
OS: MacOS Monterrey

Exporting image (SVG/PNG) or as excalidraw file (.excalidraw)? Does it happen always, or intermittently? What are the exact steps? What's your Chrome version?
Lastly, what's the Excalidraw version? You can check by opening the context menu by clicking on the canvas → show stats for nerds → click on the version to copy to clipboard → paste here.
Thanks!

@dwelle I'm getting the same behavior but different console error on that URL:
script.mjs:151 DOMException: The request is not allowed by the user agent or the platform in the current context.

Did it start happening only recently (did it work before)? If so, did you make any upgrades (Big Sur → Monterey)?

It did work before, the last time I was using it successfully was in mid-February. When it worked it was working every time, and now that it's stopped it's stopped working every time -- reloading, incognito, different or new files, nothing has made it work.

Ok, this is strange. I assume opening a file doesn't work either?

Actually opening an .excalidraw file does work
On Wed, Apr 27, 2022, at 1:14 PM, David Luzar wrote:

Ok, this is strange. I assume opening a file doesn't work either?
—
Reply to this email directly, view it on GitHub https://github.com/excalidraw/excalidraw/issues/5100#issuecomment-1111261385, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAIPYXTJPZQ4R7MGLVXRI3VHFYWXANCNFSM5UM5EOFQ.
You are receiving this because you commented.Message ID: @.***>

Cannot reproduce either. This is via BrowserStack.

Could you open Excalidraw, and then DevTools and paste in the following line:
document.body.addEventListener('click', () => {try { window.showOpenFilePicker() } catch (err) {console.error(err.name, err.message)}})

This adds a click listener that upon clicking anywhere in the body opens a file open dialog. Does this work?

@tomayac That works, I see an open file picker without any issue.

Thanks for running the test. Additional question: is this happening when the scene is very complex? Here is one theory:

The user clicks the Save button (this is the user gesture, which is valid a period of time Δuser_gesture).
Excalidraw prepares the to-be-saved data, which may take some time Δprep.
The browser tries to show the file save dialog, but if Δprep > Δuser_gesture, the user gesture is no longer good.

In browser-fs-access, the library used by Excalidraw under the hood, we now support a new programming model, where the to-be-saved data may be passed as a Promise<Blob> as well (apart from a pre-prepared Blob and streaming Response).
// `blobOrPromiseBlobOrResponse` is a `Blob`, a `Promise<Blob>`, or a `Response`.
await fileSave(
  blobOrResponseOrPromiseBlob,
  options,
  existingHandle,
  throwIfExistingHandleNotGood
);

Before we explore this, it would be helpful to know if this is the problem. Judging from your video (where apparently the data is available immediately), it's not, but I wanted to be sure.

@tomayac just in case you missed this, @cmheisel was able to reproduce in the demo as well https://github.com/excalidraw/excalidraw/issues/5100#issuecomment-1111176856. Does the image file you're saving there considered big one?

@tomayac - I'm seeing it happen even with a very simple drawing. See attached example.

The svg export is "almost" not async (when not embedding it should just be awaiting an empty async function, so it's a microtick away).
And the json export (which doesn't work either) should be completely sync if I'm not mistaken.
@cmheisel to test this, can you run:
document.body.addEventListener('click', async () => {
  try {
    await window.showSaveFilePicker()
  } catch (err) {
    console.error(err.name, err.message);
  }
});

It should allow you to save an empty file on click. Let us know if that works!

@dwelle
Nope, when I click I get the following error:
instrument.ts:129 NotAllowedError The request is not allowed by the user agent or the platform in the current context.

Thank you for confirming!
Though I do wonder, the stack trace is a bit strange (instrument.ts:129). Can you actually change the code I posted above to include the error stack trace in full?
document.body.addEventListener('click', async () => {
  try {
    await window.showSaveFilePicker()
  } catch (err) {
    console.error(err.stack); // orr console.error(error)
  }
});

And then capture a screenshot (or copy the text) of it?
Could it be a browser extension blocking this? Do you have any privacy-related or similar extensions installed?

Better yet, can you open example.com in a guest profile and paste the snippet. This way we make sure no extensions whatsoever are installed.

@tomayac -
With a guest profile on example.com and excalidraw.com I got the save file dialog without issue. Without a guest profile I tried it on excalidraw.com and example.com I got the same error (screenshot attached)

That made me do some digging and I found the culprit for my setup, I think my company (Chrome is managed) has a policy banning File editing! From the site settings screen for Excalidraw:

Mystery solved then. 😅 Closing.

Should we detect and fall back to legacy? As I understand, the "File Editing" permission only covers the new FS API?
Also the OP from @loomsfriendlyghost seems to be a different issue.
@loomsfriendlyghost if you're still facing the issue, can you go through the same steps? Mainly:

go to https://example.com and paste this into the console: (Cmd-opt-I or Ctrl-shift-I on Windows), and click to open file save dialog

document.body.addEventListener('click', async () => {
  try {
    await window.showSaveFilePicker()
  } catch (err) {
    console.error(err.stack); // orr console.error(error)
  }
});

if that works, can you https://googlechromelabs.github.io/browser-fs-access/demo/ and check if saving the image file works?

Thanks

@tomayac should we detect and fall back to legacy? As I understand, the "File Editing" permission only covers the new FS API?

I think it's designed in a way to be non-detectable to avoid it (the managed setting) to become a fingerprinting vector. It will fail at runtime with the error message from above.
I have just released browser-fs-access version 0.28.5, which supports a pattern like in this demo:
import { fileOpen } from 'https://unpkg.com/browser-fs-access';

const FAILURE_NOTE = 'Looks like your browser\'s enterprise configuration prohibits access to the File System Access API. Falling back to the legacy approach to opening files.';

const button = document.querySelector('button');
const pre = document.querySelector('pre');

let defensiveFileOpen = fileOpen;

const onButtonClick = async () => {
  try {
    const file = await defensiveFileOpen();
    pre.textContent += `${file.name}\n`;
    console.log(file);
  } catch (err) {
    if (err.name === 'NotAllowedError') {
      pre.textContent += FAILURE_NOTE;
      const { fileOpenLegacy } = await import('https://unpkg.com/browser-fs-access');
      defensiveFileOpen = fileOpenLegacy;
      return onButtonClick();
    }
  }
};

button.addEventListener('click', onButtonClick);

The confusing part is maybe that it shows the dialog twice.

Save to PNG/SVG doesn't work on MacOS 12.5 / Electron 19:

The request is not allowed by the user agent or the platform in the current context.

Interestingly, "Save to disk" works fine. Is there a major difference between the two?
Excalidraw 0.12.0

@zadam I assume it works on excalidraw.com? Probably something to do with Electron, but hard to say what. Can you post the error stack trace?

It works on excalidraw.com in browser, but not in electron (i.e. I load the excalidraw.com within electron). So yes, it is probably specific to electron. Here's the stacktrace (exlicadraw.com from within electron):

Interesting, Export file picker works in Safari but not Edge (Chrome) on MacOS. Also, that Chrome download image test also fails in Edge, so it is a Chrome/Edge issue.

Any messages in the console? Do you have the exact version numbers of Chrome and the OS at hand? Is it an enterprise policy, as in https://github.com/excalidraw/excalidraw/issues/5100#issuecomment-1113995395?

Thanks @tomayac
Chrome (Edge): Version 105.0.1343.53 (Official build) (arm64) (Mac M1 Mini)
OS: 12.6 (M1)
No Enterprise Policy blocking file editing

Alright. Console messages?

I'm using excalidraw.com but I will try running locally to see console messages

I think you meant Chrome Console not OS console. I installed locally anyway, started, clicked "Save to File" and the Chrome (Edge) console gave this error about FilePicker

Yes, I was talking about the DevTools console indeed. Could you try the steps outlined in https://github.com/excalidraw/excalidraw/issues/5100#issuecomment-1114008081 and report here, please?

FWIW, doesn't reproduce locally, nor on BrowserStack: https://live.browserstack.com/dashboard#os=OS+X&os_version=Monterey&browser=Chrome&browser_version=105.0&zoom_to_fit=true&full_screen=true&url=https%3A%2F%2Fexcalidraw.com%2F&speed=1

example.com with the console code does not present a File Save dialog. Maybe I am doing something wrong.

(At the risk of pointing out the obvious: did you click somewhere in the page?)

Yes, the only hot item is the "More information..."

If this works for everyone else, I can just not use Chrome/Edge for Excelidraw

Can you open a guest profile in Chrome and try again? A guest profile has no extensions whatsoever installed.

Edge InPrivate window did not respond to clicks after adding console code

Extensions can still be configured to run in private/incognito mode. Maybe actually try running in a guest profile. And again, just to make sure: you're clicking the page, not the DevTools area. right!?

Oh, it is Brave. Chrome works out of the box. Version 106.0.5249.103 (Official Build) (arm64)

Guest mode is also not working but the console is telling me this which I will look into:
Tracking Prevention blocked access to storage for https://stats.g.doubleclick.net/j/collect?t=dc&aip=1&_r=3&v=1&_v=j98&tid=UA-387204-13&cid=494702610.1665157883&jid=286577239&gjid=959989043&_gid=590909874.1665157883&_u=SACAAUABAAAAACAAI~&z=1142065368.

Guest mode is also not working but the console is telling me this which I will look into
Tracking Prevention blocked access to storage for https://stats.g.doubleclick.net/j/collect?t=dc&aip=1&_r=3&v=1&_v=j98&tid=UA-387204-13&cid=494702610.1665157883&jid=286577239&gjid=959989043&_gid=590909874.1665157883&_u=SACAAUABAAAAACAAI~&z=1142065368.

Sigh. Yes, Brave has decided to not expose this API: https://github.com/brave/brave-browser/issues/11407. You could have saved us some time by telling us you were using Brave, not Edge or Chrome.

Guest mode is also not working but the console is telling me this which I will look into
Tracking Prevention blocked access to storage for https://stats.g.doubleclick.net/j/collect?t=dc&aip=1&_r=3&v=1&_v=j98&tid=UA-387204-13&cid=494702610.1665157883&jid=286577239&gjid=959989043&_gid=590909874.1665157883&_u=SACAAUABAAAAACAAI~&z=1142065368.

Sigh. Yes, Brave has decided to not expose this API: brave/brave-browser#11407. You could have saved us some time by telling us you were using Brave, not Edge or Chrome.

I mistyped and fixed. Edge, not Brave. Sorry about that.

In this case, it seems like the Edge team needs to take care of this. You can send them feedback:

Fair enough

Installing it as a chrome app first makes it work for me.

if u have this issue,u can try:
app.commandLine.appendSwitch('enable-experimental-web-platform-features');

I'm getting the same issue with error
Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.
Scale 1x, it works, Scale 3x it seems to be "processing" for a couple seconds and then the error window pops up, my guess is that it takes too long to process because of big drawing so chrome doesnt associate the click with the download file event.
A solution might be to basically just show a "Processing bar" or spinning wheel while its processing and when its done display a download button, similar to how other sites that process images do it.
Windows 11, Chrome Version 120.0.6099.72, tried in incognito, guest mode, company profile, private profile, same issue.

Happens to me everytime on https://excalidraw.com/ when trying to export a large schema to PNG

x1 scale => no issue
x2 scale => no issue
x3 scale => Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.

DOMException: Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.
    at Module.y (https://excalidraw.com/assets/file-save-745eba88-h9NxcJsg.js:1:525)
    at pU (https://excalidraw.com/assets/index-pzqFbuE1.js:119:11238)
    at async rm (https://excalidraw.com/assets/index-pzqFbuE1.js:145:101481)
    at async https://excalidraw.com/assets/index-pzqFbuE1.js:191:7327

Happens to me everytime on https://excalidraw.com/ when trying to export a large schema to PNG (using Google Chrome, or the app installed through Google Chrome)

x1 scale => no issue
x2 scale => no issue
x3 scale => Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.

DOMException: Failed to execute 'showSaveFilePicker' on 'Window': Must be handling a user gesture to show a file picker.
    at Module.y (https://excalidraw.com/assets/file-save-745eba88-h9NxcJsg.js:1:525)
    at pU (https://excalidraw.com/assets/index-pzqFbuE1.js:119:11238)
    at async rm (https://excalidraw.com/assets/index-pzqFbuE1.js:145:101481)
    at async https://excalidraw.com/assets/index-pzqFbuE1.js:191:7327

I created a new issue for this a few weeks ago since the bug seems to be different
https://github.com/excalidraw/excalidraw/issues/7456

I just commented on a Chromium bug where someone was running into a seemingly similar problem.


Cumulative dps boost is calculated incorrectly

https://mutaplasmid.space/module/1029125282125/
12.9% dmg modifier, 7.9% rof modifier shows as 21.9% dps boost (possibly due to 1.129×1.079 = 1.218191)
Actual dps boost is, however: 1.129÷(1−0.079) = 1.225841477

I understand how this can be confusing but I think it is shown correctly on the website. Personally I find the way the game represents rate of fire bonuses pretty bad because it doesn't show an increase in the rate of fire but a decrease in the reload time of modules (I see you're a Pyfa dev so I don't think I have to explain this).
What I do on the website is take the value shown in game (7.33% for this module) and show 1÷(1-0.0733) = 7.9%. This value can be directly multiplied by the damage multiplier to get the DPS boost.
Because I think the value shown on the website is correct, I'm going to mark this as "won't fix" for now. But if you think it would be better to show the same value as the game I may change it in the future.

Yes i think having value consistent with ingame is important for integration sake. I have one use-case I'm dealing with on a regular basis:
I am planning to fit some mutated mods to my leshak. To see what kind of stats i will have in the end - I am entering mutated values to Pyfa. However, I am unable to fetch actual rof modifier without doing additional conversion step x = (1−1÷(1+y÷100))×100 where x is the value i want to get and y is value shown on mutaplasmid.space. Pyfa doesn't take this convention out of the blue - it's EVE client which forces it.
There are numerous less obvious cases, e.g. I'm often using calculator to calculate various things - like spoolup time improvement with new damage mod vs old damage mod, w/o entering full stats to pyfa. Also, seeing that RoF value can go beyond limits set in game confused me a lot. For instance, t2 entropic sink rof modifier with decayed mutation can't go beyond value 7.41% "eve rof", but I've seen it as high as 7.9% on mutaplasmid.space. I poked few of my buddies to make ingame screenshot of contract - and value there was different.
So, all-in-all, i think it's not necessarily a bad thing, but some hints showing that it's "real rof", not "eve rof" would definitely help.

This is a good point. If we're talking about compatibility with Pyfa do you think it would be better to show damage multiplier as 1.20x instead of 20% as well?

Probably yes, although this conversion is much more trivial and i don't mind doing it at all.
Regarding user-friendly values - i think that showing final cumulative boost is enough for user to get an impression of how strong this mutation is.

tyvm <3


Transfer into org?

@ortuman: Can you move this repository into the jackal-xmpp organization?

https://github.com/jackal-xmpp

@ortuman: This project is now dead?

@ortuman: This project is now dead?

No it isn't. ATM, I'm totally focused on a project revamp to provide cluster support among other things. I expect it to hace it ready in the next 2-3 months. Thanks for the patience. 🙇♂

No it isn't. ATM, I'm totally focused on a project revamp to provide cluster support among other things. I expect it to hace it ready in the next 2-3 months. Thanks for the patience. 🙇♂

@ortuman: Thanks for your reply.
But why it is not possible to transfer this repo into the organization?

@ortuman: Thanks for your reply.
But why it is not possible to transfer this repo into the organization?

Not a priority now, but will be done when ready.

Not a priority now, but will be done when ready.

For trying to understand better. What's the point of transfering it into .org? I've been thinking about this recently, and since this is a on man project I'd personally would leave it as is (at least for now).

With this logic, please move all from https://github.com/jackal-xmpp into your personal account ;)
Or all into the organization ^^
One person or several people, it is not a problem to have an organization place.
Please change your account status from private to public here: https://github.com/orgs/jackal-xmpp/people.
Your are cited in README, there are your commits, you are in organization, ...

Sorry, but I don't see any advantages in migrating to .org, and yet I do see the drawbacks, such as having to adapt all the code and spending valuable time.


Set up CI

CI is good for many obvious reasons, but it's especially so for people who might struggle to run unit tests on their own machine.

[x] Update unit test documentation, moving it to RtD. See #177.
[x] Explain the architecture of current unit test approach.
[x] Sort issue of running unit test, looking at tmp project and seeing tests pass but not seeing modifications.
[ ] Set up CI for unit tests.

[ ] Document decisions about when and where to run, ie on push, on merge, on forks...

[ ] Ask a collaborator if they are set up in a way that's helpful to their development work.

To fight the post-Djangocon blues, I've started working on this on a branch on my fork.
I've got a GHA workflow with a job strategy matrix setup to test the two providers with tests, Heroku and Platform.sh, across Windows, Linux (Ubuntu), and MacOS with Python 3.7 through 3.11-beta. The full test suite takes around 5 minutes to run through all the different possibilities, which is pretty good!
The only problem is all of the tests are failing.. 😖 I must have missed something in the setup of the project, so I'm gonna keep at it. @ehmatthes - when you get some time, if you want to take a look at the workflow yaml file, maybe you can spot what I've missed?
Sample test run logs

The return code of the platform setup fixture is 1, so something isn't right..
@pytest.fixture(scope='module')
def run_simple_deploy(tmp_project):
    # Call simple_deploy here, so it can target this module's platform.
    cmd = f"sh call_simple_deploy.sh -d {tmp_project} -p heroku"
    cmd_parts = cmd.split()
    result = subprocess.run(cmd_parts)
    breakpoint()

➜ pytest test_heroku_config.py
====================================== test session starts ======================================
platform linux -- Python 3.10.6, pytest-7.0.1, pluggy-1.0.0
rootdir: /home/josh/projects/django-simple-deploy
collected 15 items

test_heroku_config.py
>>>>>>>>>>>>>>>>>>>>>>>>>>>> PDB set_trace (IO-capturing turned off) >>>>>>>>>>>>>>>>>>>>>>>>>>>>
--Return--
> /home/josh/projects/django-simple-deploy/unit_tests/test_heroku_config.py(17)run_simple_deploy(
)->None
-> breakpoint()
(Pdb) result
CompletedProcess(args=['sh', 'call_simple_deploy.sh', '-d', '/tmp/pytest-of-josh/pytest-1036/blog
_project0', '-p', 'heroku'], returncode=1)

Wow, thanks for jumping into that effort! I'm going to work through the tasks in this issue, and then get back to you about CI. To clarify that third task, the unit tests have always been helpful to me. They replicate how a user would run simple_deploy, and when they pass or fail of course the pytest output is helpful. But, it's been even more helpful to be able to go into the tmp directory that pytest creates, and look at exactly how the sample project got modified by the unit test run.
This week was the first time I've looked at the tmp directory project after running unit tests, and seen an unchanged project, despite all tests passing. I need to sort that out, document it better, and I think that will explain the behavior you're seeing.
Also, most of my testing has been on macOS. So Heroku may work on Windows and Linux, because I tested it there. I wouldn't be surprised if Fly.io and Platform.sh both fail on Windows, and maybe Linux.
Here's a couple questions about setting up CI:

If you got CI set up through GA on your fork, can you submit a PR that on approval would start CI on this project?
If someone forks the project to contribute and they push to their own branch, does that trigger CI runs for their own feedback? Or do they not get feedback until they submit a PR?

I finished restructuring the unit tests. Documentation is here.
I'm not planning to touch CI particularly soon, but if you have questions about any of these changes please let me know. The overall structure of the unit tests should be more stable moving forward, and if you have questions about any of the behavior or architecture I can happily answer those questions now. :)

@ehmatthes Looks good! I'll read over the documentation, update my branch accordingly, and open a draft PR for you to look over and we can go from there.

I've opened a draft PR #182, if you would like to take a look at what I have done so far.

I merged the PR about the branch name. After merging I realized more benefit to that change, but I'm not sure you'll see that comment from after merging the PR.


Pass editorState to ToolbarItem Icon

Currently for custom toolbar actions, there is a key to add an icon. it would be nice to enable passing data to that icon so it can edit the editorState instead of jumping to a new input entity.
For example i could render a dropdown for font sizes or a popover with a color picker.
So in a custom toolbar action it would look something like this:
{
    type: "custom",
    icon: props => <CustomIcon editorState={props.editorState}/>,
    action(editorState, onChange) {},
},

Hi @estermer! Still interested in improving this feature? I'll close for now, but you can reopen it if you want to continue 👍


Provide lock free version of hash index

We should use these papers as base http://www.cs.ucf.edu/~dcm/Teaching/COT4810-Spring2011/Literature/SplitOrderedLists.pdf http://www.azulsystems.com/events/javaone_2007/2007_LockFreeHash.pdf
Do not forget that we use JNA and support platforms beyond Oracle JDK which means that we should work with native memory barriers like this one http://trac.mcs.anl.gov/projects/openpa it has very good multiplatform support but robustness should be investigated.

Not relevant any more

Can you give a quick explanation why? Or, maybe just a reference to the change that makes it irrelevant?
Scott

Hi @smolinari
We are working now on new tx model which will be based on locks with page granularity.
But in some cases will allow to hold write-write , read-write, write-read locks by different threads at the same time.
It will allow us to automatically convert all data structures to their mt version with low level of contention between threads.
We tried to base our tx mechanics on CAS model, but in case of high load it requires a lot of retries and as result system SLA becomes unpredictable.
After finishing of page lock based MT model we will concentrate on algorithms which oriented on NUMA architecture, we think it will bring even more performance gains than additional improvements in MT scalability.

Although most of that flew above my head, it sounds good. Thank you for the explanation.
Scott

@smolinari well main idea is that in case of high load, cas or lock-free protocols require many retries and amount of such retries is not predictable and as result client SLA also becomes unpredictable. So we use approach when we use locks but with very low granularity to avoid contention of threads.

What does SLA mean?
Scott

service level agreement, in other words distribution of latency of operations.

I thought that is what you meant, I just wasn't sure how an SLA could relate to multi-threading. But, I guess, if the database is supposed to meet certain specs as a purchase point and those specs are guaranteed, then I guess I can understand it. It isn't really SLA's though. It is more like fulfilling contractual obligations.
Scott


Script doesn't work anymore

I believe this issue happened because Facebook likely changed parts of the format of the dump file (I'm assuming the dump file is the zip that I got when I downloaded my data).

Hey Alex! Thanks for reporting this. If you happen to figure out what changed, feel free to open a PR 😄

I ended up pulling down my Facebook data export again to see what had changed. Unfortunately, it's drastically different than when I first created this script. I've updated the README in case anyone is interested in trying to fix it 👍


Create Barkatul_Mujauddin

Try to add more details and create a folder for each day. More structuring will help you when you look back on all the days and what you learned!

Not the right structure, you need to clone the repo, create a folder using mkdir  then create your each day progress folders and put your learnings.

Thank you so much for your advice
On Tue, 11 Jan, 2022, 1:08 am Saiyam Pathak, @.***>
wrote:

Not the right structure, you need to clone the repo, create a folder using mkdir
then create your each day progress folders and put your learnings.
—
Reply to this email directly, view it on GitHub
https://github.com/saiyam1814/DevOpsRoadmap2022/pull/39#issuecomment-1009277402,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWMMGP3QBTYH4JN2MZRW7YLUVMYTDANCNFSM5LUOQJZA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Add Moes Temp/Humidity/Lux sensor

This PR fixes #276

I might be wrong but to add you device numbers to a existing device you should just edit the driver.compose.json and add your numbers to the readme file. the other files in your pull request should be removed

I might be wrong but to add you device numbers to a existing device you should just edit the driver.compose.json and add your numbers to the readme file. the other files in your pull request should be removed

I missed updating the README, thanks for pointing this out. As for the generated app.json in the root, I would expect it to be in .gitignore if it's not meant to be commited?
I didn't intend for the devcontainer.json to be included, but I don't see a problem with it being there. It enables the use of VSCode DevContainers and/or Github Codespaces. Either you use it, or ignore it (your loss).
I will of course make that update to the PR if that's how the maintainer prefers it.

I am thinking that you want the app.js for creating a release but otherwise it isn't very useful since it needs to be regenerated after merging anyway.

Reverted the app.json commit

_TYZB01_ftdkanlj / TS0222 added to next release.
Not merging since it's already in my local code.


Disconnect clients that stop sending messages

This PR is incomplete, just putting it here to gather feedback and some direction.

Does anyone know how in ligdren to not disconnect clients on the same IP?
Calling Disconnect on a NetConnection is causing all clients to disconnect.

Shouldn't NetConnections be IP + port? That sounds weird.

To solve that I'd investigate how it distinguishes clients in the first place


aria-multiselectable should be supported in role=tablist

Using the latest version of this plugin (2.2.0), I received the error when running eslint:
The attribute aria-multiselectable is not supported by the role tablist
I checked in the ARIA spec and it looks like aria-multiselectable should be allowed in a tablist role.
I think this is simply a matter of adding ARIA-MULTISELECTABLE to the TABLIST entry in https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/src/util/attributes/role.json#L2473-L2497
(If that's the case, happy to submit a PR.)

oversight 😄
thanks for catching - yep that's all you should have to do.

Published in v2.2.1

Thanks for the quick turnaround! Installed 2.2.1 and can confirm the warning is gone.


wrote some tests

Coverage increased (+5.7%) to 42.529% when pulling 461b44bc715ca18602d4d1ac4e565b3130fc56be on enthalpychange:tutorial_wip into d5fd52c818b07dcac4abe7eb852a216c80b6fde2 on keeppythonweird:master.


plasma5: cannot login after update

Issue description
After recent upgrade plasma5 is no longer working. journalctl shows complaints about plasma5 not found.
journalctl-grep-plasma5.txt
Steps to reproduce
nixos-rebuild switch --upgrade
reboot
attempt login
Technical details

System: NixOS: 17.03.1004.b5c83975b4 (Gorilla)
Nix version: nix-env (Nix) 1.11.8
Nixpkgs version: "17.03.1004.b5c83975b4"

I rolled back to my latest working generation

working: 17.03.982.b647a67dfe (Gorilla)
not working: 17.03.1004.b5c83975b4 (Gorilla)

I did some more troubleshooting, and if I switch from slim to sddm it started working again.
sddm working on version: 17.03.1013.bc11395603 (Gorilla)
slim not working on the same version.


Feature Request: exclude comment entry

Feature Requested
Some bibtex managers, like Jabref, use @Comment{} to store information about grouping and other settings. These entries are listed in the picker as empty lines. Is it possible to exclude these comment entries?

Relevant Screenshot
No response
Checklist

[X] The feature would be useful to more users than just me.

Icon added in release 11.7.6 👍


Add argument matcher for expecting a block.

This is @kaiwren's a_block argument matcher with fleshed out sad path specs, and a refactoring of the match logic to allow mixing with other arguments. Review please @kaiwren @ioquatix @benoittgt.

Some further improvements could be made here, do we want to check for blocks in other places? e.g. is expect(obj).to receive(:msg).with(a_block, a_block) valid? And if we don't should we reject the method expectation out of hand?

Maybe we can say this is good enough for a first step? Ready the tests I found it already a good improvement.
Good job

A quick question - IMO ArgumentListMatcher and associated code could use a little refactoring to better express intent, and to include blocks in the args verification in a more consistent way (that's one of the reasons I stopped at a_block). I'm happy to attempt this if it makes sense to do so, and of course, it goes without saying that I'll keep the changes small, and will completely understand if they are rejected. Would that be useful?

You are welcome to have a go at refactoring it if you think you can make it better, please make sure you understand the matcher protocol before changing method names, e.g. it's important and non optional that === is used as the match method, as it's part of our composability and is what lets argument matchers and other matchers be combined.
Thinking about this, theres now ambiguity as to what we mean when, @myronmarston I'm leaning towards adding a_block to match a block "anywhere", and then adding .with_block to mean the block argument in order to be ambiguous, meaning with(...)s arguments will remain "real" argument. Effectively .with_block would just check for the presence of &block, or could check for a specific block?

Procs/Lambdas as an argument and blocks are different things. I suggest consider a_block and a_proc/a_lambda/a_callable?

@JonRowe yep, well noted, I will keep that in mind.
Yep agree that bound and unbound blocks be treated differently. I was unaware of and_yield when I picked up this ticket, but +1 to @JonRowe's point IMO the contracts and use cases for with(a_block) and_yield and with(a_block,  ..., a_block) should be defined before proceeding further to avoid ambiguous apis.

They're not different enough, and can be passed around... Consider:
2.5.1 :001 > def a(&block)
2.5.1 :002?>   puts block.inspect
2.5.1 :003?>   block
2.5.1 :004?> end
 => :a
2.5.1 :005 > a =  a { |a| }
#<Proc:0x00007fd290a01440@(irb):4>
 => nil
2.5.1 :006 > b = a(&proc { |b| })
#<Proc:0x00007fd2909e1c58@(irb):5>
 => nil
2.5.1 :007 > c =  a(&lambda { |c| })
#<Proc:0x00007fd2909d0548@(irb):6 (lambda)>
 => nil
2.5.1 :008 > Proc === a
 => true
2.5.1 :009 > Proc === b
 => true
2.5.1 :010 > Proc === c
 => true

I've done some work to try to tease out ReceivedMessage as a concrete class from Proxy. I'm not too happy with it, but wanted to circle back to check on my direction with the team before moving further in this direction. Do let me know if this makes sense, or if I should move in a different direction. https://github.com/rspec/rspec-mocks/compare/master...kaiwren:refactor-extract-ReceivedMessage-from-Proxy

I'm not sure why this was never merged, migrated to the monorepo as rspec/rspec#114


/dev/pts/N leak

detail https://github.com/kata-containers/agent/pull/880

This issue is being automatically closed as Kata Containers 1.x has now reached EOL (End of Life). This means it is no longer being maintained.

Important:
All users should switch to the latest Kata Containers 2.x release to ensure they are using a maintained release that contains the latest security fixes, performance improvements and new features.

This decision was discussed by the @kata-containers/architecture-committee and has been announced via the Kata Containers mailing list:

http://lists.katacontainers.io/pipermail/kata-dev/2020-November/001601.html
http://lists.katacontainers.io/pipermail/kata-dev/2021-April/001843.html
http://lists.katacontainers.io/pipermail/kata-dev/2021-May/001896.html

If you believe this issue still applies to Kata Containers 2.x, please open an issue against the Kata Containers 2.x repository, pointing to this one, providing details to allow us to migrate it.


Update page to show StartScreen instructions in favor of OnStart

Hello,
As the OnStart functionality will be deprecated/reduced, could you update this page to encourage use of the StartScreen property instead?

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

ID: 10559351-f355-f97f-e7a4-903cd6a804fc
Version Independent ID: 10559351-f355-f97f-e7a4-903cd6a804fc
Content: Launch and Param functions in Power Apps - Power Platform
Content Source: power-platform/power-fx/reference/function-param.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thank you for pointing this out, I'm making an update to this doc.


Canon MakerNote: Allow callable to process tag value

While working on some images I found that the library did not expose the 'CameraTemperature' tag from the ShotInfo MakerNote.
To support this I added a callable and allow that callable to be called with the tag value. I might be useful for other tags too.
Example output from a JPEG image taken on a Canon EOS 2000D:
>>> exif = exifread.process_file(fd, details=True)
>>> exif["MakerNote CameraTemperature"]
(0x0000) Proprietary=55 C @ 0

I don't think the test failures are related to my changes but let me know if they are and happy to amend.


[Popup/Modal] - AJAX Populate

it would be great if you could enable ajax loading in model window,dimmer,popover etc

Seconded that on having dynamic modals using the api to load content from an external file.


Button styles

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: susheelkumar21:x: ga521:x: ram4gitYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: susheelkumar21:x: ga521:x: ram4gitYou have signed the CLA already but the status is still pending? Let us recheck it.


rawkit causes browser to open the rawkit launch page rather than chrome devtools page

Version: 1.3.2
Platform: MINGW64_NT-10.0 WXLPC0HYS9N 2.10.0(0.325/5/3) 2018-02-12 08:49 x86_64 Msys
Chrome Version: Version 66.0.3359.45 (Official Build) beta (64-bit)

whenever I execute npx rawkit app.js, my chrome browser responds by opening up a Rawkit Launch page (https://darcyclarke.github.io/rawkit/?url=chrome-devtools) ...
and not the chrome-devtools:// page that I am expecting. I have the rawkit extension installed and running in my browser so I don't know why it keeps prompting me to "Click to Add Rawkit to Chrome"

Thanks for submitting this issue. I'll look into this today. Potentially try uninstalling and reinstalling the extension; that said, you should never experience that behaviour (there must be an error happening in the extension).

closing this until/unless I see the behaviour again

@ballPointPenguin How did you solve it? It never worked for me, it always redirects to the Rawkit Launch page. rawkit 1.3.2, rawkit chrome extension 1.2.0, Chrome 67.0.3396.87

@tokland I did not solve it. I have not been using rawkit lately. If you are seeing this or a similar problem, you should re-open this ticket or create a new one.


The widget is broken following README

I tried the widget today, and with the code shown in instruction, the widget is broken and cannot be built.
I found out that the chess_piece.dart expected piece.toString() to returns something like "wr", but instead, maybe due to a change in the chess package, it received "Color.WHITEr".
Therefore, I made a local copy of the widget, and then, in utils.dart, getPieceMap function, I changed from:
map[square] = Piece(piece.type.toString(), piece.color.toString()); to map[square] = Piece(piece.type.toString(), piece.color == ch.Color.BLACK ? 'b' : 'w');
And the widget works perfectly afterward!
Thank you for the widget, please check this issue and fix it (more properly, I assume) for others!

Thanks for letting me know. Actually I'm working on writing proper data types. I'm push the change ASAP.
Thanks.

This is fixed with latest release. Thanks


Update share links with code

Added code to generate share links

Codecov Report

Merging #95 (948d608) into master (83fb6da) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #95   +/-   ##
=======================================
  Coverage   76.98%   76.98%
=======================================
  Files          38       38
  Lines         782      782
  Branches      102      102
=======================================
  Hits          602      602
  Misses        168      168
  Partials       12       12

Impacted Files
Coverage Δ

src/drive/share/index.ts
92.50% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


The list content to move as it's being scrolled down.

Hello there, it is currently an issue that when you add more than a few rows to the list, it becomes hard to add stuff as you have to go all the way to the top in order to select your image and move it down. It would be pretty good if the left images box had a size limit, and could be scrolled up and down in order to view other images. It would also be pretty good if the box would move as it's being scrolled up and down.

Sorry for the force pushes!
Don't think I caught what you meant, about the box moving as it's being scrolled?

Exactly, so we don't have to scroll all the way to the top to get an image and move it all the way to the bottom.

I made this small concept so hope it helps!

Alright, so in 411f08a I just made the tier list a big scroll-able box, such that you don't scroll the main page.
In d4ec2e5 I tried making the controls float and the images-box scroll-able instead, but never got the trash can anchored to the bottom right of the screen how I wanted (could probably have put in the images-box corner as depicted).
I currently favor the first approach because it was much simpler, but the issue with each is that they both the flex-grow behavior that lines the images box up with the size of the tier list. This makes sense, are we are decoupling them by the time the tier-list grows off-screen, but I'd prefer to keep the existing behavior until the limits are reached... don't know that that's possible through CSS...

Both 411f08a and d4ec2e5 are actually great candidates for this issue i think, at least in their current forms they're usable. I think if 411f08a had a screenshot mode of some sorts where you could see the whole list at once, it is a perfect solution, making it easier to navigate and make the actual list.

I think making the tier list a scroll box is my preferred solution, but I'm open to whatever works best. Thanks for the help ✌🏻

So ^that's how I'd impliment a toggle for it, just needs an image for the button, and the box of images should still be updated to scroll. This makes the parent viewport static (ie. no element expands the page when overflowing, unless someone puts in a really long title?). Still considering the flex-grow behavior, which I liked and which might still work; Kinda liked the thick scroll bar in the mock up, but that'd be a seperate issue.

they both break the flex-grow behavior that lines the images-box up with the size of the tier list

Still considering the flex-grow behavior, which I liked and which might still work

don't know that that's possible through CSS

lol
-       height: 80vh;
+       max-height: 80vh;

the box of images should still be updated to scroll

Non-issue, works like that already

position: absolute Trash into the lower right so that it floats when overflow is enabled?
Is screenshot-mode a clear enough name for what that button does? (Could just as well be print-mode.)


Keep track of number of subscribers to lists

This commit adds a rake task that will update the "subscriber_count" of subscriber lists ("topics"). It does this by getting the data from GovDelivery. Separately we'll set up a rake task to periodically run the rake task so that the counts are refreshed regularly.
We're adding the counts because we're considering adding something to the content tagger UI that shows how many people are subscribed to a certain taxon. This can help the information architects determine the relative popularity of taxons.
The actual work happens in a sidekiq worker to make sure we're not tying up the rake task executors on Jenkins, and to make sure it's retried when an error occurs. We'll also get data from statsd on the job performance.
https://trello.com/c/YwfSmNQX

@rubenarakelyan correct, the stand-alone changes are in https://github.com/alphagov/email-alert-api/pull/166/commits/7ba543c29de969d8c9e9451c8c5c39aa1c065b8d.


How to use tokio-modbus with dynamic values ?

Hi all,
I'm kind of new to rust development, I trying to implement tokio-modbus to get asynchronous tcp server running alongside of random number generator.
And I'm in trouble to get a randomly generated Vec to dispatch through modbus holding registers.
I implemented this kind of code, but response is always the same and never been recomputed:
#[derive(Clone)]
pub struct MbServer{
    pub(crate) response: Response,
}

impl Service for MbServer {
    type Request = Request;
    type Response = Response;
    type Error = std::io::Error;
    type Future = future::Ready<Result<Self::Response, Self::Error>>;

    fn call(&self, req: Self::Request) -> Self::Future {
        println!("{:?}" , req);
        let res = self.response.clone();
        println!("{:?}", res);
        future::ready(Ok(self.response.clone()))
    }
}

#[tokio::main]
pub async fn main() -> Result<(), Box<dyn Error>> {

    let server = server::tcp::Server::new("<IP_ADDRESS>:502".parse().unwrap());

    let mb_server = MbServer{
        response: Response::ReadInputRegisters(get_datas())
    };

    server.serve(move ||Ok(mb_server.clone())).await.unwrap();

    Ok(())
}

fn get_datas() -> Vec<u16> {
    let mut rng = rand::thread_rng();
    let vals: Vec<u16> = (0..1).map(|_| rng.gen_range(0, 20)).collect();
    vals
}

$ ./modpoll -m tcp -c1 -t3 <IP_ADDRESS>
modpoll 3.10 - FieldTalk(tm) Modbus(R) Master Simulator
Copyright (c) 2002-2021 proconX Pty Ltd
Visit https://www.modbusdriver.com for Modbus libraries and tools.
Protocol configuration: MODBUS/TCP, FC4
Slave configuration...: address = 1, start reference = 1, count = 1
Communication.........: <IP_ADDRESS>, port 502, t/o 1.00 s, poll rate 1000 ms
Data type.............: 16-bit register, input register table
-- Polling slave... (Ctrl-C to stop)
[1]: 12
-- Polling slave... (Ctrl-C to stop)
[1]: 12
-- Polling slave... (Ctrl-C to stop)
[1]: 12
-- Polling slave... (Ctrl-C to stop)
[1]: 12
-- Polling slave... (Ctrl-C to stop)

As you can see the response is always the same and seems the get_datas function was executed only once.
How can I provide dynamic response to the modbus response ?
Thank you for your help

That's obvious. get_datas() is invoked once on server startup and stored. Each response is populated with a copy of that immutable data.

Thanks @uklotzde to point the lack of my competences on rust development, no offense. :smile:
In reality, I've shown you the least advanced code, but working, to let it as simple as possible, but I already tried to make callback function to be called by call function and be able to generate response.
But as I said, I don't have the competences, and I don't know how to make the response dynamic the right way.

This is not a tokio-modbus issue, but a general programming question.
If I would provide you the correct solution you would probably hit the next road block soon. The solution might also depend on your requirements and goals that I am not aware of.

Yes @uklotzde you've right, I will try to find the solution on my own.


Refactor metadata parsing & add table input

Starting a WIP PR, not ready for merge yet. The initial parsing part seems to work well though :)

Close #14


When using the complete data set(about 800K data), the usage of MiB Mem will continue to increase, resulting in OOM. Is there any solution?

When I train with the data set of about 800k objects, the number circled in the graph keeps increasing as the number of training steps increases.
My configs/syncdreamer-train.yaml is the same as provided by the author, except for the data path
https://github.com/liuyuan-pal/SyncDreamer/blob/main/configs/syncdreamer-train.yaml

Hi, the training is based on the pytorch_lightning and it is supposed to manage the resources correctly. You can see that the dataset class https://github.com/liuyuan-pal/SyncDreamer/blob/eb41a0c73748cbb028ac9b007b11f8be70d09e48/ldm/data/sync_dreamer.py#L57
which simply loads data here and is not supposed to cause increasing memory usage. Maybe, you can check whether the memory usage is growing or not when running the dataset solely.

Hi, the training is based on the pytorch_lightning and it is supposed to manage the resources correctly. You can see that the dataset class
https://github.com/liuyuan-pal/SyncDreamer/blob/eb41a0c73748cbb028ac9b007b11f8be70d09e48/ldm/data/sync_dreamer.py#L57
which simply loads data here and is not supposed to cause increasing memory usage. Maybe, you can check whether the memory usage is growing or not when running the dataset solely.

Thank you very much. We think this is a problem with the configured environment. I am checking with docker environment and will give feedback if there is any result.

I have the same problem. Have you found the solution yet? The speed of OOM is proportional to the amount of num work.

I have the same problem. Have you found the solution yet? The speed of OOM is proportional to the amount of num workers.

When I run the code in docker provided by the author, this problem is solved.  Your can try to run with author's docker.

I have the same problem. Have you found the solution yet? The speed of OOM is proportional to the amount of num workers.

When I run the code in docker provided by the author, this problem is solved. Your can try to run with author's docker.

Thank you for your information. I also exactly use the docker env, this may indeed be a docker environment problem.


eval: improve swebench infer error handling and retry

End-user friendly description of the problem this fixes or functionality that this introduces

[X] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Improve error handling and retry for swebench inference

Give a summary of what the PR does, explaining any non-trivial design decisions

Add a shared method assert_and_raise to replace the raw assert statements which will cause automatic retry to fail
Use try and finally to properly cleanup runtime when an evaluation fails in the middle

Link of any specific issues this addresses

LGTM!


[Missing] People Nearby

Blog: https://telegram.org/blog/contacts-local-groups

Support for groups and people nearby was added in TDLib 1.6.0. See Change Log for the full list of changes.


issue with panstart using npm and browserify

I am having an issue detecting the panstart event using hammerjs. I have the following simple example, all the events work and are logged, but not the panstart.
I am using npm and browserify.
var myElement = document.getElementById('myElement');
var Hammer = require("hammerjs");
var mc = new Hammer(myElement);

// listen to events...
mc.on("panend panstart panleft panright", function(ev) {
  console.log(ev.type +" gesture detected.");
});

actually this should be fine now, it was an issue with my use of modules in npm as I had lost reference to the element in the on callback, it all worked fine when I used requires correctly to get a reference to the element.


🆕 Software Suggestion | Whereby

Basic Information
Name: Whereby
Category: Video/Voice Calling
URL: https://whereby.com/
Description
GDPR compliant service. All plans include the "Small" room size (up to 4 participants) in which communication between participants are primarily sent through peer-to-peer connections.
Data Storage & Security FAQ.
Why I am making the suggestion
It seems to be an interesting service for the privacy-focused community and I would like to know what other people in the community think about it.
My connection with the software
None.

[x] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

WhereBy is pretty nice
For up to 4 participants all traffic is routed in a p2p way, meaning no server ever sees it.
Our company has the paid version, which also works really well, however for rooms of more than 4 people their servers are used to route/prioritize/bundle traffic.

WhereBy is pretty nice
For up to 4 participants all traffic is routed in a p2p way, meaning no server ever sees it.
Our company has the paid version, which also works really well, however for rooms of more than 4 people their servers are used to route/prioritize/bundle traffic.
But all the free rooms are 4 people only, so it should be privacy-friendly for all free users.

how does this peer to peer work, is it some form of webrtc?

From their Data Storage & Security FAQ:

In the Free version of the Service, users can only use “Small” room size (up to 4 participants), and this mode is available in all plans. In “Small” room size, communication between participants are primarily sent through peer-to-peer connections, where audio and video streams are sent directly between participants and do not pass through any of our servers, in cases where this is allowed by the network the user is on. Video and audio transmitted in the Service is then sent directly between the participants in a room and is encrypted (DTLS-SRTP) with client-generated encryption keys. In cases where a user is behind a strict firewall or NAT (e.g. on a strict corporate networkm roughly), video and audio need to be relayed via a TURN server, but end-to-end encryption is still maintained.

Me and @LukeSeers  have been working together to uncover the secrets of whereby.com on how they handle their data. This is a small sample of what we have uncovered from our detective work.
This may be a good service for some, however if you are truly interested in privacy, you would read the privacy policy, having done this you will note that the service is not fully compliant with the GDPR, it does not even mention CalOPPA which is a matter of law and should be there as a protection for users all round the world, they collect PII and do not ask for consents on their consent page, and just to add insult to injury they have servers in the USA where the government has regular access to data through the NSA, collecting PII and sharing it without consent is totally against any form of privacy.
They do not have a “Do-Not-Track” policy, as per calOPPA.
We have chosen a couple of quotes from the privacy policy to demonstrate what we want to get across:

We in Whereby are committed to safeguarding the privacy of our users. Our business model is to provide a paid service to users who need additional features on top of the FREE version, and does not rely on widespread collection of general user data. We will only collect and process information that we need to deliver the service to you, and to continue to maintain and develop the service.

Even though this states this at the beginning of the privacy policy more you read the more it starts to fall apart. Which as a result already breaks the trust of privacy users?

Whereby may collect, store and process various kinds of data, with different legal grounds, as listed below. For the categories of data that require your consent, we will actively ask you for consent before collecting any data. You can give and revoke your consents at any time in your Settings page in https://whereby.com.

After looking through their consents page at (https://whereby.com/user/privacy) It doesn't specify data in anyway shape or form, it generalises the whole subject of consent. We only could find that one page, if you find another page that has all the consents, we would love to see it.

We will never store any media sent between participants in a room. Customers who have access to the “Recording” feature will be able to record meetings, and they are then responsible for collecting consents from all participants in the meeting prior to starting the recording. They are also responsible for storing and processing the recording in compliance with regulations after downloading it from Whereby

If they don’t store information, why do they state “after downloading it form Whereby”.

Also it appears to be closed-source, so it's likely a less preferable alternative compared to the open-sourced and unlimited Jitsi Meet (already listed) and Kopano Meet (PR #1980). Note that Kopano Meet uses a fully peer-to-peer model, but with unlimited number of participants (although it's likely not possible to reach a huge number of participants).


cockpit-composer weblate is misconfigured

The visible name of the main branch has been renamed to "main", but the slug is still "master".  Also, all pages currently show an orange ! with the error:

This component is set up incorrectly.
The repository browser URL points to non-existing locations:
https://github.com/osbuild/cockpit-composer/blob/master/build/messages/components/Modal/PendingChanges.json#L0 (404 Client Error: Not Found for url: https://github.com/osbuild/cockpit-composer/blob/main/build/messages/components/Modal/PendingChanges.json#L0)
This can happen if the path within the translation files contains references to a parent directory, or is absolute.

See here, for example:
https://translate.fedoraproject.org/projects/cockpit-composer/master/
The un-renamed slug can be fixed via Manage → Rename, at least.  I'm not sure what's required to fix the other error.

The un-renamed slug can be fixed via Manage → Rename, at least.

I have renamed it.

I'm not sure what's required to fix the other error.

I don't see any error. But links indeed don't work. Example link is https://github.com/osbuild/cockpit-composer/blob/main/build/messages/pages/blueprintEdit/index.json#L0 because in the .pot file locations are generated as build/messages/pages/blueprintEdit/index.json while there is not such location. It should be pages/blueprintEdit/index.js. This is issue when .pot file is being generated.

@marusak Do you have any suggestions on fixing this issue? We extract the translations to a directory when creating the .pot file. This directory won't exist in github.

I don't unfortunately. Either you need to look at the tools you are using if they don't support this in some way or use different approach all together.


Toast on incoming tx

After fully syncing, the dapp should be listening for new notes. When one comes in, we should have a toast that notifies the user that a new note has been detected.
Came up in the weekly sync, that it is currently hard to tell when a user has successfully completed an IBC-in. The toast should help confirm a success.

For this to work, we need a way for the view server to inform notify clients. I don't think this currently exists, we should file an issue on the core repo to add this to the view RPC


Migrate from Ant to Gradle

:tada:

gradle build worked great for me. This can probably be closed.

@petemyron it works but gradle uses ant script. The goal is to rewrite it for gradle.

Done


Client - Cannot set gateway.gateway_api_host via config command

Describe the bug
Cannot set gateway host via config command.
Seems like the gateway.gateway_api_host parameter is not listed in configurable parameters in client source code.
Steps To Reproduce
Here are the steps to reproduce the issue:

Start a hummingbot instance.
Execute config + <TAB>  for completion to pop
Observe that the gateway.gateway_api_host is not listed

Even if you execute config gateway.gateway_api_host command will result in error
Screenshots

Release version
1.12.0
Discord username: klpanagi

hi @klpanagi, upon crosschecking with qa
couldn't reproduce the issue on 1.12.0 and dev-1.13.0

development:

What about gateway.gateway_api_host - yes, currently
can't change using config gateway.gateway_api_host returns invalid key warning message

Yes, I was referring to the gateway_api_host parameter.
I dont know if it has been removed from configurable parameters in the later PRs, but it is really useful to be able to config the host for the gateway, especially now that the gateway source was moved to another repository and people can deploy in virtualized environments (networks in compose, clusters etc).

@klpanagi it should be possible to change that, do you want to create the PR? it should be simple

@klpanagi it should be possible to change that, you can nominate this issue to the poll of March, and I will vote in favor

@klpanagi it should be possible to change that, you can nominate this issue to the poll of March, and I will vote in favor

Thanks @cardosofede , will nominate this to the poll.

This issue is assigned to HB Foundation team

I have scoped the issue and believe it's a P3 bounty, which is currently 10,000 HBOT (https://docs.hummingbot.org/governance/bounties/#bug-bounty-schedule).
@klpanagi Thanks for your interest in working on it - I will assign the bounty to you - please submit a PR to development and we will merge it in.

Thank you @fengtality .

@fengtality  I have created the PR that resolves this issue
https://github.com/hummingbot/hummingbot/pull/6159

#6159 has been approved and merged to development branch. @carlitogetaladajr please arrange for payment of 10,000 HBOT to @klpanagi's ETH wallet address on file. Thanks for your contribution to Hummingbot!


Moved the loadDatabase logic to StatsDB

TL;DR
This solves racing conditions that results not reliable persitatnce for clients (CLI, DApp) for StatsDB
Before this PR we required the client to autoload database from the outside since this was an async process we get into racing conditions when trying to getCollection/addCollection in the constructor of StatsDB
StatsDB now has async init() method that return Promise for loaded database, additionally Config has the statsDBLoaded Promise that reflects the result of async init()
Instead of
const persistenceAdapter = new LokiIndexedAdapter(options.network.id);
    const browserDB = new Loki('timenode-stats.db', {
      adapter: persistenceAdapter,
      autoload: true,
      autoloadCallback: () => {
        // LokiJS stores BN objects as a string.
        // This causes problems when loading from persistent storage.
        // Convert any stored non-BN into BN.
        const stats = browserDB.getCollection('timenode-stats').data;
        stats.forEach(stat => {
          stat.bounty = new BigNumber(stat.bounty);
          stat.cost = new BigNumber(stat.cost);
        });
      },
      autosave: true,
      autosaveInterval: STATS_SAVE_INTERVAL
    });

it's enough to use
const browserDB = new Loki('timenode-stats.db', {
      adapter: persistenceAdapter,
      autosave: true,
      autosaveInterval: STATS_SAVE_INTERVAL
    });

combined with
this.config = new Config({
      providerUrl,
      claiming: options.claiming,
      scanSpread: options.scan,
      logfile: options.logfile,
      logLevel: options.logLevel,
      walletStores: this.keystore,
      password: options.keystorePassword,
      autostart: options.autostart,
      logger,
      economicStrategy: options.economicStrategy,
      statsDb: browserDB
    });

    await this.config.statsDbLoaded;

Pull Request Test Coverage Report for Build 1593

14 of 18 (77.78%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0%) to 78.125%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Stats/StatsDB.ts
12
16
75.0%

Totals

Change from base Build 1591:
-1.0%

Covered Lines:
622

Relevant Lines:
762

💛  - Coveralls


Improve Job-Candidate Matching

Integrate the following changes to make job matching more accurate:

Allow skill by skill matching
Implement AND match instead of OR match

This issue was resolved in v1.5


BUG: Manifest needs var expansion

I'm the developer for q2-itsxpress.  I've been having some users get errors the stem from this file validation module.  The issues happen when small, but not empty, fastq files are being written.  An example of an error  when running qiime in  verbose mode is:
Traceback (most recent call last):
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/plugin/model/file_format.py", line 24, in validate
    self._validate_(level)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/q2_types/per_sample_sequences/_format.py", line 159, in _validate_
    self._check_n_records(record_count_map[level])
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/q2_types/per_sample_sequences/_format.py", line 130, in _check_n_records
    % (i * 4 + 1))
qiime2.plugin.model.base.ValidationError: Missing sequence for record beginning on line 17

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/q2cli/commands.py", line 274, in __call__
    results = action(**arguments)
  File "<decorator-gen-450>", line 2, in trim_pair_output_unmerged
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/sdk/action.py", line 231, in bound_callable
    output_types, provenance)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/sdk/action.py", line 390, in _callable_executor_
    spec.qiime_type, output_view, spec.view_type, prov)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/sdk/result.py", line 244, in _from_view
    result = transformation(view)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/core/transform.py", line 68, in transformation
    self.validate(view)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/core/transform.py", line 143, in validate
    view.validate('min')
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/plugin/model/directory_format.py", line 171, in validate
    getattr(self, field)._validate_members(collected_paths, level)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/plugin/model/directory_format.py", line 101, in _validate_members
    self.format(path, mode='r').validate(level)
  File "/Users/rivers/anaconda3/envs/qiime2-2018.8/lib/python3.5/site-packages/qiime2/plugin/model/file_format.py", line 29, in validate
    ) from e
qiime2.plugin.model.base.ValidationError: /var/folders/py/c71jxp_54712mpc0c84zgty40000gn/T/q2-SingleLanePerSamplePairedEndFastqDirFmt-0c21t4va/385_0_L001_R2_001.fastq.gz is not a(n) FastqGzFormat file:

Missing sequence for record beginning on line 17

When I run the sample through the stand-alone version of ITSxpress it produces a forward and reverse fastq with 52 records.
Is this an error that was addressed by the refactoring you just completed?  I'm happy to test it locally,  but I wasn't sure how to install the latest development version of the whole qiime stack locally.

Hey @arivers!

Is this an error that was addressed by the refactoring you just completed?

Nope. The validation message you shared is actually for the FastqGzFormat, not the manifest formats.

I'm happy to test it locally, but I wasn't sure how to install the latest development version of the whole qiime stack locally.

Have you seen this? https://dev.qiime2.org/latest/quickstart/

Missing sequence for record beginning on line 17

This error is when a sequence is written with no nucelotides:
@seq01
AAAA
+
1111
@seq02

+

@seq03
CCCC
+
2222

Is it possible for you to remove the empty records in itsexpress (or the plugin), prior to returning the output?

Fantastic, this will allow me to solve my validation issue! I was thinking the issue was coming from trying to process blank fastq files not blank fastq records.  Thanks for pointing me to how to set up the dev environment.
I'll ask these questions over in the qiime developer forum next time, after I posted I remembered that's where most of the developer questions are supposed to go.


Peer dependency<EMAIL_ADDRESS>

Now that<EMAIL_ADDRESS>is out, the peer dependency fails to install.
npm ERR! code ERESOLVE
--
15:22:04.041 | npm ERR! ERESOLVE could not resolve
15:22:04.042 | npm ERR!
15:22:04.042 | npm ERR! While resolving<EMAIL_ADDRESS>15:22:04.042 | npm ERR! Found<EMAIL_ADDRESS>15:22:04.042 | npm ERR! node_modules/next
15:22:04.042 | npm ERR<EMAIL_ADDRESS>from the root project
15:22:04.042 | npm ERR!   peer next@">= 12.0.0" from<EMAIL_ADDRESS>15:22:04.042 | npm ERR!   node_modules/next-i18next
15:22:04.043 | npm ERR<EMAIL_ADDRESS>from the root project
15:22:04.043 | npm ERR!
15:22:04.043 | npm ERR! Could not resolve dependency:
15:22:04.043 | npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>15:22:04.043 | npm ERR! node_modules/next-usequerystate
15:22:04.043 | npm ERR<EMAIL_ADDRESS>from the root project
15:22:04.043 | npm ERR!
15:22:04.043 | npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>15:22:04.043 | npm ERR! node_modules/next
15:22:04.043 | npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>15:22:04.043 | npm ERR!   node_modules/next-usequerystate
15:22:04.043 | npm ERR<EMAIL_ADDRESS>from the root project
15:22:04.043 | npm ERR!
15:22:04.043 | npm ERR! Fix the upstream dependency conflict, or retry
15:22:04.043 | npm ERR! this command with --force or --legacy-peer-deps
15:22:04.044 | npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

I published 1.9.1 yesterday which should have the correct peerDeps for Next.js 13.4+ and 14.

Fixed.


why you use 'vue-template-compuler' custom version ?

if you use custom version , in my project , must use lastest version too.
for example :
i use vue as " vue: '2.5.4'" ,you use compiler as  "vue-template-compiler:'^2.5.17'"
I will get an error, vue and vue-template-compiler version is not same,
can please u fix it ? or provide a optimize plan?

u can use
"peerDependencies": {
"vue": "^2.5.2"
},

Fixed in v2.3.4


How to add Read Time feature to the blog posts?

Hey guys,
I knew that already a Read Time feature issue was closed, But that was not solved my issue.
Can anyone tell me how to use it by step-by-step ?
Thank you.

You can access read time data using content.readingTime.text in the /layouts directory.

ckeckout this for reference

Thanks @haramishra, that's how I would do it as well.


Qeustion about DDPG algorithm

For halfcheetah simulation,
No need to break when 'done' becomes true (if done:)?????
For example, when i did simulation with Pendulum-v0 i always put 'break' as 'done' gets 'True'.

When done is true, that means the episode has ended. At this point, you can either stop using the environment, or you can call reset() to start another episode. DDPG does the latter in order to run episodes.


Add test for building docs

Closes #254

Test documentation build check will fail if documentation can't be built.
https://github.com/EIT-ALIVE/eitprocessing/actions/runs/11437789439/job/31818036113?pr=318

Have you tested whether it indeed fails if there's something off?

Yes, see the comment above. Commit c382f4f (9c567ea before rebase) was pushed to this PR with an intentional error and reverted with 4605245. The checks for that commit failed as expected.

Also, this workflow is ignored if the only change is to md files (see line 11). Could this result in a failed documentation build not being picked up if there are documentation only changes? If so, it may be worthwhile moving it to a separate yml.

Good catch. I moved it to a separate action.

Have you tested whether it indeed fails if there's something off?

Yes, see the comment above. Commit c382f4f (9c567ea before rebase) was pushed to this PR with an intentional error and reverted with 4605245. The checks for that commit failed as expected.

My bad, I went straight into the code review and forgot to read the comments.


Change output file format specification

in the network file positions are 0 indexed. Change this and let them start with 1.

in the network file there is a space after the sign "+", whereas in the other lines that start with a "+" there is no space afterwards.

still in the network file replace "NODE" with "HAPLOTYPE", "SEQ" with "SEQUENCE", "POS" with "POSITION" and "LINK"/"CON" with other good names.

in the species list file, there's the line "# [set([0, 2, 3]), set([1, 4, 5, 6, 7, 8])]" ... this line should better be encoded in the network file.

May affect [May affect https://github.com/eeg-ebe/HaplowebMaker/issues/21](url)

Sort the species list file alphabetically


Don't create a tag index for Manifests

Don't create a tag index for Manifests (it was not being used) and remove the
code for creating it.
Closes #938
Signed-off-by: Richard Scothern<EMAIL_ADDRESS>

Looking good.
This obviates the need for blobStore on tagStore, but I'm wondering if removing that should be a part of a different PR.

The tag field should remain. This is the only way to find tag through digest effectively. I use it with notification mechanism to find any update tags as the notification now only notify digest. Is there other way to find tag through digest?
In addition this change will break backward compatible with other tools that rely on that.

@oilbeater This only changes the backend path layout, which is not a supported, stable interface. Tools that rely on backend path layout are at risk of breaking.
This change does not affect the API.

I'm a bit reluctant to remove the tag index, since it has come in handy for me in the past to look at historical manifest versions. I understand this has never been advertised as a feature, and maybe the behavior isn't even intentional, but it seems nice to have. The size of these link files should be negligible, and a garbage collection script that removes the underlying manifests should feel free to ignore them (or clean them up), since they aren't used by the registry.

@aaronlehmann While they are handy, if we don't guarantee them, they aren't useful. As we start adding GC and other pruning features, we won't be able to trust them at all, going forward.

On behalf of @RichardScothern I'm declaring this closed.


Create Block Diagram for Data Flow

As a team member, I need to design and create a block diagram that visually represents the flow of data from the scrape to the sale, supplementing the existing data flow documentation. Each block in the diagram should correspond to a stage in the data flow, referencing the previous documentation, and include privacy and security information.
Acceptance Criteria

Develop a block diagram illustrating the flow of data from scrape to sale.
Each block in the diagram corresponds to a documented stage in the data flow.
Include references to the relevant sections in the previous data flow documentation for each block.
Provide clear descriptions for each block, summarizing its role in the data flow.
Document privacy and security considerations specific to each block.

@sfaria27
email_data_flow.pdf


List of minor issues as they pop up.

Should source code be placed in directory named 'hw2' ?

Yes, it is listed here: https://github.com/uhcs2320/hwTwo#requirements

Oops, I missed it. Sorry!


Run puppeteer with exist chrome instance, throw exception when I have chinese input

Here is the code throw exception, when I specify executablePath with pre-installed chrome path. page.type('#input', '微软') throw exception.
const puppeteer = require('puppeteer');
(async () => {
    const browser = await puppeteer.launch({
        headless: false,
        executablePath: 'C:/Program Files (x86)/Google/Chrome/Application/chrome.exe'
    });
    const page = await browser.newPage();
    await page.goto('http://localhost:8899/');
    // works
    await page.type('#input', 'google');
    // exception
    await page.type('#input', '微软');
})();

Not working :/ in Version 75.0.3770.100 (Official Build) (64-bit), It still opens up new instance of chrome and it has no history whatsoever, I had to open it along with my other chrome tabs.
const puppeteer = require('puppeteer');
const pptr = require('puppeteer-core');
(async () => {

	const browser = await puppeteer.launch({
	  headless: false,
	  executablePath: '/usr/bin/google-chrome'
	});

})();

Here I also tried puppeteer-core , means pptr variable await pptr.launch in above variable, still not working. The issues is not closed!

I found the source of the problem using google-chrome.
https://github.com/GoogleChrome/puppeteer/blob/master/lib/Launcher.js#L359
At the chromium startup, on the stdout (console) displays the
DevTools line listening on ws: //<IP_ADDRESS>: 9222 / devtools / browser / 66993667-8118-4701-98df-ae22d1ba9e4e
At the start of google-chrome this linie is not print out so the puppeter does not connect to the browser


Can't get Termination to work.

After running the command npm install -g node-gyp in Mac terminal and installing the Termination package into Atom and trying to open a new terminal inside Atom nothing happens.
Please help me out!

Ditto - Termination stopped functioning:

@Fred-Barclay we'll take a look over the weekend - let us know if we can help with anything here.

This wont work wit Atom 1.28 you need to upgrade your pty binaries to m57 for compatibility with electron 2.0.3, only thing you can do is compile them or upgrade to node-pty-prebuilt which also has no working windows binaries for electron 2.0.3 for 0.7.5 relase

Hey guys,
Termination 0.6.3 is out which should fix this. Please update and let me know!


Add fire resistance and damage increase effect to Numerian Steel Breastplate

The description doesn’t say that it adds additional damage to your attacks, only that creatures take damage when they strike you?


Git version is not present in ambient new for cargo install --git

When Ambient is installed through cargo install --git, the following code
https://github.com/AmbientRun/Ambient/blob/062370f82e18aab287e3f3983475f1bc59f4c1d9/app/src/cli/new_project.rs#L64-L70
does not work, as the information about the Git version is erased. This was encountered on the Discord: https://discord.com/channels/894505972289134632/1082217873189126154/1089255480049807433

Reprod on windows

Fixed and seems to work on Windows


task_mgr::spawn's shutdown_process_on_error` doesnt work reliably [P:3] [S:0]

Steps to reproduce
Was discovered via https://github.com/neondatabase/neon/issues/3387
Apparently synthetic size calculation task had an error that triggered shut down. Synthetic  size calculation shouldnt lead to pageserver shutdown, and this is fixed in #3392. But shutdown on error should still work even if its triggered erroneously. This is what this issue all about.
Expected result
pageserver restart.
Actual result
pageserver was stuck in semi-alive state when some of the tasks were stopped and some continue running. Postgres protocol listener was shut down so this resulted in connection refused errors during basebackups.
Environment
prod.
Logs, links

https://neondb.slack.com/archives/C033RQ5SPDH/p1674231060776979
Captured stacktraces (many of threads are idle tokio workers)

consider having a timeout for this one

Fixed in https://github.com/neondatabase/neon/pull/6105 -- we now exit(1) if a shutdown_process=true case.


Revise logsumexp in np backend and reduce those redundant tests

Summary
This PR includes the following:

The sp.misc.logsumexp in the numpy backend has been replaced with the sp.special.logsumexp because the former will be deprecated.
The test_logsumexp has been removed because it is redundant to test_elementwise_operations.
The test_logsumexp_optim for Theano has been enabled.

The third item has been reverted.


Reapply ABI generation outside of /dist

This reverts commit https://github.com/safe-global/safe-client-gateway/commit/f99eca7c203bf23b173fd43204c2e707ce1bc411, instead generating ABIs to their own /abis folder as /dist broke build.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 15 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 93.936%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6102

Relevant Lines:
6291

💛  - Coveralls


[SPARK-36242][CORE][3.0] Ensure spill file closed before set success = true in ExternalSorter.spillMemoryIteratorToDisk method

What changes were proposed in this pull request?
The main change of this pr is move writer.close() before success = true to ensure spill file closed before set success = true in ExternalSorter.spillMemoryIteratorToDisk method.
Why are the changes needed?
Avoid setting success = true first and then failure of close spill file
Does this PR introduce any user-facing change?
No
How was this patch tested?

Pass the Jenkins or GitHub Action
Add a new Test case to check The spill file should not exists if writer close fails

Test build #141612 has started for PR 33514 at commit 6196f81.

Test build #141612 has finished for PR 33514 at commit 6196f81.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141612/

Running Scala style checks
========================================================================
[info] Checking Scala style using SBT with these profiles:  -Phadoop-2.7 -Phive-2.3 -Phive-thriftserver -Pspark-ganglia-lgpl -Pmesos -Pkubernetes -Phive -Pyarn -Pkinesis-asl -Phadoop-cloud
Scalastyle checks failed at following occurrences:
[error] SERVER ERROR: Gateway Time-out url=https://dl.bintray.com/typesafe/sbt-plugins/com.alpinenow/junit_xml_listener/0.5.1/jars/junit_xml_listener.jar

It seems that there is a network timeout, not  fails Scala style tests.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46129/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46129/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46129/

retest this please

Test build #141613 has started for PR 33514 at commit 6196f81.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46130/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46130/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46130/

Test build #141613 has finished for PR 33514 at commit 6196f81.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/141613/

Curious why the earlier PR could not have been merged to 3.1/3.0

I'm not sure, but there seems to be no code conflict

Curious why the earlier PR could not have been merged to 3.1/3.0

@mridulm My bad. I didn't merge it since I thought it was an improvement.

My bad. I didn't merge it since I thought it was an improvement.

Sorry, It's my description in jira that makes @Ngone51 misunderstood, before completed the new test case, I just think it's an optimization

Thank you all. Merged to branch-3.0.


Feature: Update ability to shuffle/mix both dragons and statues

Overview
Why

The statue fights are one of the more iconic ones in FF6, fighting them as the build up to KT I think is pretty thematic and really fits the WC philosophy of "vanilla is good when possible"
Based off the previous PR, the statues were being handled closer to Phunbaba 3 in code than Dragons.
A second phase of this I would like to add is to create a "Scale Statue Locations" flag which, when false, will keep the encounter at the statues the vanilla levels (something like 65-ish), and breaking it out in this way helps organize that.

How
Added Flag: --statue-boss-location, [-stloc <original | shuffle | mix>]

original: The statue encounters are that of the vanilla game
shuffle: The statue encounters are shuffled amongst themselves. Some french-vanilla action.
mix: Retain current worlds collide functionality, allowing for Doom, Goddess, Poltrgeist to show up outside of the tower (dragon locations if mixed, standard boss locations)
When no statue location specified, default to mix as to retain backwards combability with flags.

Added Flag --dragon-boss-location, [-drloc <original | shuffle | mix>]

original: The dragon encounters are that of the vanilla game
shuffle: Retain current worlds collide functionality. The dragon encounters are shuffled amongst themselves
mix: Dragons will be available outside of the standard dragon locations (statue locations if mixed, standard boss locations)

This is equivalent to the current "-bmbd" (mix bosses with dragons) flag.
When there is no boss random/shuffle, this is changed to "shuffle"

Testing
Haven't done too much testing outside of checking spoiler logs as I've only really touched the randomization
Ensure when -bbr is enabled every boss in a Boss condition is used

Flags:  -open -sl -oa <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.6.9.7 -ob <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.15 -oc <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.22.9.23 -od <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.30.9.31 -oe <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.36.9.37 -of <IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.6 -bbr -drloc shuffle -stloc shuffle
The flags only have three enemies not required:

Goddess (statue)
White Drgn (dragon)
WrexSoul (normal boss)

I've manually checked about 15 seeds of these flags and it looks like every boss is included except those three

Ensure when neither -bbr or -bbs are enabled that using mix will become shuffle

Flags:  -open -sl -drloc mix -stloc mix
Expected output should have -drloc shuffle -stloc shuffle

--bbs --stloc original
Kefka's Tower                  BehemothSuit
    Atma           -> Chadarnook
    Inferno        -> GhostTrain
    Guardian       -> Ultros 2
    Doom           -> Doom
    Goddess        -> Goddess
    Poltrgeist     -> Poltrgeist

--bbs --stloc shuffle
Kefka's Tower                  Flame Shld
    Atma           -> Number 024
    Inferno        -> TunnelArmr
    Guardian       -> Ultros/Chupon
    Doom           -> Doom
    Goddess        -> Poltrgeist
    Poltrgeist     -> Goddess

--bbs --stloc mix
    Leader         -> Doom
    FlameEater     -> Poltrgeist
    Air Force      -> Goddess

Kefka's Tower                  Red Cap
    Doom           -> Wrexsoul
    Goddess        -> Rizopas
    Poltrgeist     -> Tritoch

--drloc original
    Ice Dragon     -> Ice Dragon
    Storm Drgn     -> Storm Drgn
    Dirt Drgn      -> Dirt Drgn
    Gold Drgn      -> Gold Drgn
    Skull Drgn     -> Skull Drgn
    Blue Drgn      -> Blue Drgn
    Red Dragon     -> Red Dragon
    White Drgn     -> White Drgn

--drloc shuffle
    Ice Dragon     -> Red Dragon
    Storm Drgn     -> Skull Drgn
    Dirt Drgn      -> Ice Dragon
    Gold Drgn      -> White Drgn
    Skull Drgn     -> Storm Drgn
    Blue Drgn      -> Dirt Drgn
    Red Dragon     -> Gold Drgn
    White Drgn     -> Blue Drgn

--drloc mix
    Inferno        -> Ice Dragon
    Dadaluma       -> Storm Drgn
    FlameEater     -> Dirt Drgn
    Umaro          -> Gold Drgn
    Poltrgeist     -> Skull Drgn
    Number 128     -> Blue Drgn
    Dullahan       -> Red Dragon
    TunnelArmr     -> White Drgn

    Ice Dragon     -> Hidon
    Storm Drgn     -> Vargas
    Dirt Drgn      -> Nerapa
    Gold Drgn      -> Rizopas
    Skull Drgn     -> Chadarnook
    Blue Drgn      -> Kefka (Narshe)
    Red Dragon     -> Number 128
    White Drgn     -> Number 024

Menu

Just pushed to this - Added the five missing boss encounters in KT as objective conditions https://github.com/AtmaTek/WorldsCollide/pull/35/commits/89a6157bd80ed413771289e0924a535796ed8cfe

Bug when Boss Battles: Original - Statues become shuffled (dragons probably too)
Should these act independently of that flag since they are kind of their own thing now? Or should original bosses overwrite the stloc, drloc flags?


Create MeanFilter.cpp

What type of PR is this? (check all applicable)

[x] 🚀 Added Name
[ ] ✨ Feature
[ ] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Add Link of GitHub Profile

you checked the Added name then add your name in CONTRIBUTERS.md file


RemoveElement#225

Pull Request Template
What have you Changed(must)
I added new block of code. I a little bit improved and simplify the code. I added exception "ArrayIndexOutOfBoundsException("Length is longer than array length")".
Issue no.(must) - #225
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me if not it will be marked as spam.
[x] My file is in the proper folder
[x] I am following clean code and Documentation.
[x] I have added the title and what program will do first in the file
[x] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

Hi, please check my issue i hope it will be good

Hi, can i delete whole branch and push it again, because i have problem with merge. @SSKale1


Mod users bl permissions setup in junit feature file

Please ignore the commented code .

This PR should be closed as it's been superseded with #408


Eliah Kagan: Watch thegamecracker.com

Eliah Kagan requests the watch of the watch_keyword thegamecracker\.com. See the MS search here and the Stack Exchange search here.

thegamecracker\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


ndm pod crashed twice in 0.3.5

Description

In openshift cluster , installed openebs 0.9 along with ndm 0.3.5.
It is observed that one of the ndm pods is observed to be restarted twice.
It dumped core.

Attached the logs of ndm pod.
Your Environment

kubectl get nodes:

NAME                        STATUS   ROLES          AGE   VERSION
gitlab-master.mayalabs.io   Ready    infra,master   86d   v1.10.0+b81c8f8
gitlab-node1.mayalabs.io    Ready    compute        86d   v1.10.0+b81c8f8
gitlab-node2.mayalabs.io    Ready    compute        86d   v1.10.0+b81c8f8
gitlab-node3.mayalabs.io    Ready    compute        86d   v1.10.0+b81c8f8
gitlab-node4.mayalabs.io    Ready    compute        86d   v1.10.0+b81c8f8
gitlab-node5.mayalabs.io    Ready    compute        86d   v1.10.0+b81c8f8

kubectl get pods -n openebs:

maya-apiserver-9f67c7648-vqjg5                                    1/1     Running   0          36m
openebs-admission-server-666ffddc6f-44x6b                         1/1     Running   0          36m
openebs-localpv-provisioner-d8bb7d58b-85s2q                       1/1     Running   0          36m
openebs-ndm-lwjdl                                                 1/1     Running   0          36m
openebs-ndm-qpbnb                                                 1/1     Running   0          36m
openebs-ndm-slk96                                                 1/1     Running   2          36m
openebs-ndm-xqf8v                                                 1/1     Running   0          36m
openebs-ndm-xwkl6                                                 1/1     Running   0          36m
openebs-provisioner-77d5d6fbb9-sfqfv                              1/1     Running   0          36m
openebs-snapshot-operator-749895ffd-lcszp                         2/2     Running   0          36m

ndm_core.txt
ndm pod logs

Fixed in #276 in NDM repo. The crash occurred because of a memory corruption issue from seachest.


Github Action: Update Content scope and Reference Tests

Task/Issue URL: https://app.asana.com/0/1157893581871903/1204684578899043
Description
Update the workflows for Content scope and reference tests to take advantage of the Github Action we’ve created

PR titles and Asana task names now show the latest version of each library
Tasks are also added to the Release Board / PR section
After the Asana task is created we update the PR description

@malmstein the only thing we need here is to add the version to the Asana task. Is all the refactor this PR does required just to add a version in the asana task?
If it is not required, please remove all the refactoring


GUI User Interface

Create a GUI interface using qt or TK (or something else) that is hopefully cross platform compatible. E.g. both Windows and Mac.

Perhaps using Kivy?
http://kivy.org/
That would be pretty cool.
http://insights.dice.com/2014/11/26/5-top-python-guis-for-2015/


VersionAssembly using PS environment encoding not file encoding

I've ran into an issue where the AssemblyInfo.cs file was encoded with UTF-8 but the build agent was running in UTF-16 LE (for some reason). This meant that when the version was update and written back to the file it was now in UTF-16 LE encoding.
Not a problem for MSBuild to handle properly but SonarQube was falling over as it was expecting UTF-8, which all the other files were in.
A suggested fix is to get the encoding of the file before reading it in and then specify that same encoding when using Out-File. I've got the fix for this already written and will put in the pull request soon containing it.

Fixed and released using @ChrisLGardner  code


[DOCFIX] Add MapR-FS documentation

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11301/
Test PASSed.

@calvinjia thanks for the comments! I updated the PR.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11302/
Test PASSed.

Merged build finished. Test PASSed.

LGTM


Merge for v2.4.2

fix m.cb.pick.plot callback
use the first positional argument passed to m.set_data as data

Codecov Report

Merging #46 (593332f) into master (e0badea) will increase coverage by 2.34%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   81.04%   83.39%   +2.34%
==========================================
  Files           9        9
  Lines        3509     3511       +2
==========================================
+ Hits         2844     2928      +84
+ Misses        665      583      -82

Impacted Files
Coverage Δ

eomaps/callbacks.py
88.52% <100.00%> (+18.68%)
:arrow_up:

eomaps/eomaps.py
83.54% <100.00%> (+0.04%)
:arrow_up:

eomaps/helpers.py
80.76% <0.00%> (+2.46%)
:arrow_up:

eomaps/_cb_container.py
83.87% <0.00%> (+3.27%)
:arrow_up:

Continue to review full report at Codecov.

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


Update tutorial-pipeline-python-sdk.md

Added missing code to generate train.py file.
Note: I am unsure how to test it to make sure it displays the right code it takes from the jupyter notebook. I followed the same pattern as elsewhere.

@jorgeper : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Provide script to enable an Azure Logic App

Provide script to enable an Azure Logic App

Based on the following script:
https://codit.visualstudio.com/Invictus Templates/_git/invictus-templates?path=%2Fdeploy%2FAzure-EnableLogicApps.ps1
This script does require the presence of a "deploy-orderControl.json"-file which acts as configuration to indicate which Logic Apps are to be enabled, in what order.
Documentation on this can be found here:
https://codit.visualstudio.com/Invictus Templates/_wiki/wikis/Invictus Templates/115/EnableDisable-Logic-Apps

Would this be something which can be set up quite easily, since it is similar to #20 ?
Looking into what we can/should include into the next release, to make the scripts that have already been created available.


please specify licensing

Hello,
Can you please specify a LICENSE file at the root of this repo, such as this one for GPLv3 so that users can use your code on other stack exchange sites?
I came across one of your answers on AskUbuntu, and I'd like to use some of your code, modify it, and post it to a similar question on Stack Exchange. Unfortunately, the licensing of the code in question is ambiguous in this repo.
Please explicitly set the license for the content of this repo to the GPL or similar permissive license so that the community can use, adapt, and publish your work on other stack exchange sites.
Thank you,
Michael

@kos0  @aboettger @yar-trach @ByteCommander @bvanrijn  So considering that you guys have contributed to this repository, do any of you have objections or suggestions on how to license this repo ? I typically license my stuff under MIT License, but if you want.

CC0 :)

I like MIT, but anything is fine with me, really. I don't claim ownership of any parts of the code, just contributed some minor fix in one of the scripts ;)

@aboettger  lol,  that'd be probably unfair to force everyone's contribution into public domain

Alright, so unless there's any further objections by Monday, 08/31/2020 ,  I'll go with MIT license for this one

@maltfield
The repository is now licensed under  MIT.  I do appreciate the community interest in my works, and it always is a joy to see when my works are found to be useful.
Feel free to use and distribute the code in this repository, modify as desired, and  most importantly - have fun !


user12986714: Watch healthycounsel.com

user12986714 requests the watch of the watch_keyword healthycounsel\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

healthycounsel\.com has been seen in 0 true positives, 1 false positive, and 0 NAAs.

This domain is already caught by "Pattern-matching website in {}". Given that this is just an exploratory watch, I'd prefer not to double-up on detecting this domain.