Problems with Windows PATH variable getting imported

I understand that the Windows PATH variable gets imported into the Linux environment on start-up.
I gather from poking through related issues that this is by design, I guess, with the idea of being able to run Windows executables from the Bash command prompt.
I was quit suprised to find that feature on the road-map. I honestly was not expecting that would be possible at all, and I'm not completely sure I understand how that's going to be useful in practice. I mean, even if they can launch, Windows applications aren't going to be aware of the Linux OS, so it seems a bit odd.
Either way, I find that the import of Windows paths into the Linux environment is causing a lot of practical problems.
For example, node and npm were immediately launchable from the Bash console, but I tried to npm install node-sass, which depends on a global installation of node-gyp, which I have installed globally on on Windows, but these tools are hardware and OS dependent, so it all just sort of blows up in your face.
Another example is composer, which was also immediately launchable from Bash, but I have plugins that cached various file-system paths in flat files with \ windows backslashes in the path, which caused errors. I have various tools installed globally, and I have the global Composer installation folder in my system path - simply importing that to Linux doesn't mean those tools will actually work.
I have lots of scripts and tools set up for my work routine in Windows, and these simply don't work, or get in the way and cause weird/unexpected results in Linux.
I understand that there's a registry work-around which I guess I'll try, but I wanted to open this issue to make you aware that, well, there are issues with this feature. Things that were designed to run under Windows, built for Windows, installed on Windows, etc. often have artifacts like cached data or binaries that don't simply work, even if you can get them to launch under Linux.
I think this feature is going to confuse a lot of Windows users. As others have pointed out, the name "Bash on Windows" is kind of misleading, because the Bash terminal is just a tiny front-end to a very large and complex Linux system behind it - I at least understood and expected this, some Windows users may not, but I never expected a feature like this, and I'm not sure it's useful.
What's more, I spent like two hours drilling through Linux startup scripts to figure out where these path variables were being set in the first place, and didn't find it - I guess they're "hard injected" into the Linux subsystem somehow at startup? I suppose that's why there's a registry key to toggle this feature. I was trying hard to get into "Linux mode", and was expecting to find an initialization like this in e.g. ~/.bashrc which I think is where the PATH variable is normally initialized?
Perhaps a better approach would be to inject the Windows path into a separate variable, e.g. WINDOWS_PATH, rather than injecting directly into PATH, and then initialize PATH in ~/.bashrc where I'd be able to see it and change it?

@mindplay-dk - Thanks for posting.  You bring up many valid concerns, which is why I added the global opt-out registry key in the first place.  We initially flighted this feature in Insider builds and I was fully prepared to pull the ripcord on this feature if a lot of people were having issues with it.  However, my experience was much the opposite and most of the feedback has been overwhelmingly positive.  I know a registry key isn't a very elegant solution and in the future different configuration options like this will be exposed in a more discoverable and user friendly-way.  Alternatively you can also override your $PATH variable inside of your .bashrc file.
Let me try to address some of the specific issues you bring up:

The majority of the issues you are experiencing are likely due to various scripts and tools not correctly handling spaces in the $PATH environment variable.  These are uncommon on *nix but not illegal.  In practice when using the $PATH variable (or components thereof) in a script you should wrap it in quotes.
Could you please give a little bit more detail on how having node installed on Windows and inside WSL "sort of blows up in your face"?
In order to invoke Windows binaries you have to fully qualify them, meaning you must supply the ".exe" extension at the end (even if they are in your $PATH variable).  This makes it very unlikely that Linux tools will accidentally run Windows binaries.
Regarding your "lots of scripts and tools" some specific examples would be helpful, I'd be happy to help you work through any issues you're experiencing and fix any bugs that come up in our implementation.
Unfortunately the $WINDOWS_PATH variable wouldn't work without modifying user-mode Linux binaries to be aware of it.

Lastly I appreciate the feedback!  I'm very proud of my work lighting up interop but know it isn't perfect.   Regarding the feature's usefulness I will defer to our User Voice where this was one of the highest requested features.

I will defer to our User Voice where this was one of the highest requested features.

Okay, if that's what people want, I suppose there's no arguing with that :-)
And I'm not "against" this feature per se, I just find it confusing, perhaps because I think more of UOW as being a tool that enables me to comfortably run Linux tools with closer integration to Windows compared with Docker or an emulator - I guess some people (possibly the majority) may think of it more as better alternative to Git Bash in that it also happens to run Linux binaries.
I guess the latter mode of thinking is what you inspire by naming it "Bash on Windows" :-)

Could you please give a little bit more detail on how having node installed on Windows and inside WSL "sort of blows up in your face"?

Yeah, it kind of comes back to your next comment:

In order to invoke Windows binaries you have to fully qualify them, meaning you must supply the ".exe" extension at the end (even if they are in your $PATH variable). This makes it very unlikely that Linux tools will accidentally run Windows binaries.

True, but most tools these days are not in fact binaries - npm, node-sass, tsc, gulp, composer and phpunit to name a few that I use.
Of these, at least npm, composer (or possibly a Composer plugin) and node-sass failed in some way, and I ended up installing the Linux versions (which take priority in the PATH) anyhow.
The oddest thing about the Node-related issues, was that some node-sass dependencies that had already been installed in node_modules while on Windows couldn't run - and then, even after erasing node_modules to start over, the node-sass package wouldn't install because of platform differences with the installed node-gyp build environment.
Replacing any globally-installed node/gyp/node-sass packages on my Windows drive obviously would be very bad if I ever try to run any of it from a plain Windows shell again, so I ended up creating a Linux install of node/npm/gyp anyway.
But even if I avoid global package installations and install locally in node_modules instead, I'll need separate project-folders for the Windows and Linux versions of every project I'm working on, since any binaries that end up being built to the node_modules folder must have the same hardware/OS architecture builds of their dependencies, so it just doesn't work well, for me, personally, in practice.
Anyhow, overriding the PATH variable in ~/.bashrc is probably the simplest and most correct option, so I will just do that :-)
I guess you can close this issue, though I'll just put one thought out there: maybe importing some paths, such as C:\windows\system32, is okay - but maybe the import needs to be a bit more selective to be really useful? Not sure, I guess that's not really possible, since the path variable is just a variable - we don't really know anything about the paths other than their priority, so there's probably not much we can do...?

since any binaries that end up being built to the node_modules folder must have the same hardware/OS architecture builds of their dependencies, so it just doesn't work well, for me, personally, in practice

@benhillis - It is kind of an existential question WSL will have to face eventually. It isn't just Node.js. Say you are a Java programmer. You have the same kind of problems as @mindplay-dk's quote above. Did you mean Java backed by a bunch of win32 native libraries built with msvc?  Ditto Python, which is what the guy over in #1370 is struggling with presently.
I am sure you have a handle on all this in your head. +1 for the pure awesomeness that is WSL interop. But it is going to be rough sailing with a lot of confusion for while.

@mindplay-dk -- I think you've missed a point about binaries, and as such have not entirely answered @benhillis 's question.  Namely:  In Windows, if it doesn't have a file extension, it ain't gonna run :-)  It may be a .exe file, in which case it will execute directly.  It may be a .js, or .php, or (etc); in all of those cases it will be invoked with the correct interpreter.  Windows will often let you omit the extension when running the program.  But it's still there.
As a result, something like phpunit CAN NEVER be a valid Windows executable because it lacks a file extension entirely.  So, if that's your program's name (as seen on WSL's $PATH), it should be unambiguously the Linux version of that program.
So what you're telling us is that you have a Windows application that is creating scripts that are not valid Windows scripts, and putting them in Windows %PATH%.  That's just weird :-)  To me, the obvious follow-up questions are (1) which app, and what scripts?, and (2) why is it doing that?

In Windows, if it doesn't have a file extension, it ain't gonna run :-)

Well, usually, the way these things are wired on Windows, there's an auto-generated .bat file somewhere - typically right next to "binary", e.g. JS/PHP/phar/rb/etc. script without a file-extension (and with a hashbang) that Windows can't find, but Linux will.
I believe tools such as Composer do that as a work-around, so that end-users don't have to hand-write a .bat file for every package - so e.g. Composer can deploy the Linux "binary" (hashbang script) next to a matching .bat file.
Is it weird that the .bat and the "binary" end up in the same folder? It's not a problem on Windows, because it won't launch the "binary" without the proxy .bat. I guess it deploys them both to the same folder just for simplicity - I guess there was no need to introduce a platform difference and put the binaries in a folder outside of paths, since Windows won't find them either way.
Perhaps one reason they keep it that way, is so that e.g. Git Bash (which I have been using for a long time) will actually skip the .bat and go directly to the "binary", since Git Bash also imports the Windows PATH variable. I'm just making wild guesses here, but it makes sense so me ;-)

aaand, there you are :-)

And here's npm doing a similar thing - it uses this package to do it.
Likely package managers for other scripting languages (ruby, perl, python) are doing similar things?

Hm -- that's a good point, and I think a good use case.
I haven't run into this with Python (which I use a bunch personally), perhaps because it's slightly more common in Python-land to install as a module and invoke as python -m <modulename> <args>; nor with node because I always do local installations for node packages.
(Your point about Windows and Linux local node installations conflicting is a good one, and has been made on other tickets before.  It's particularly bad because even simple .js files, which usually ought to otherwise be platform-independent scripts, don't work because Linux node is incompatible with the newlines used by Windows node.)
It seems to me that Windows can't directly execute these extension-less files, and Linux (meaning, code run within WSL) hasn't tried to mark them as executable.  So perhaps this could be addressed in a future iteration of DrvFs, by not setting the execute flag on files in that case?  Just a thought -- Linux tools will generally ignore files in $PATH if they aren't marked as executable.

I think this bug is giving me Python issues.  Currently trying to set up the ebcli in WSL; pip installs it fine, but when I try to run something like eb logs from within the repo, I get an error like:
-bash: /mnt/c/Users/Fu/AppData/Roaming/Python/Scripts/eb: c:\python27\python.exe: bad interpreter: No such file or directory

It looks like the Windows Python path is beating the one I set in ~/.profile.  Any ideas on how to get around this?

Looks like the same problem I was having. Lots of package & script bins don't have file extensions, so the Win version might get executed if it gets found first. Some people fix it by pre-pending their Linux entries to the PATH var so they always come first, but I think sometimes it's better to remove competing Win paths (especially when they provide identical function.) I added this to my ~/.bashrc for now (it could be improved, but I'm still a newbie.)
Function Source: Removing a directory from PATH
### remove unnecessary Win PATHs
# This can prevent extension-less commands from bleeding into BASH.
# (eg. "ng" would execute the Win bin if "@angular/cli" wasn't installed on Linux.)
#
function path_remove {
  # Delete path by parts so we can never accidentally remove sub paths
  PATH=${PATH//":$1:"/":"} # delete any instances in the middle
  PATH=${PATH/#"$1:"/} # delete any instance at the beginning
  PATH=${PATH/%":$1"/} # delete any instance in the at the end
}

path_remove '/mnt/c/Users/me/AppData/Roaming/npm'
path_remove '/mnt/c/Users/me/AppData/Local/Yarn/bin'
path_remove '/mnt/c/Program Files (x86)/Yarn/bin'
path_remove '/mnt/c/Program Files/Git'
path_remove '/mnt/c/Program Files/Git/cmd'
path_remove '/mnt/c/Program Files/nodejs'
path_remove '/mnt/c/OpenSSL-Win32/bin'
path_remove '/mnt/c/Program Files (x86)/Python27'

@john-osullivan  I noticed that with npm packages I could end up running the wrong version if I simply forgot to install it on WSL too, since it will then use the Windows one. You could try removing Windows' python/ebcli paths from your bash env like I did, coz why would you need access to both at the same time?

I just started looking at the Windows Subsystem for Linux feature now. I had this issue with Node.js and npm.
As stated by @mindplay-dk in the top post, I disabled the Path import feature using the registry key. To clarify for anyone having the same problem. Disable the feature by closing the bash shell, create the DWORD registry key, run the bash shell. If you then type echo $PATH you will see the Windows paths have gone.
I agree with @mindplay-dk, this seems like a backward feature. I think it depends on where you are coming from. I have been working with Linux for many years and do not expect windows binaries in my Linux path. I imagine people who have mostly worked with Windows would expect to be able to run notepad.exe from any command prompt.
This should be an Opt-in rather than an Opt-out feature.

@grantcarthew - I think you nailed it, it depends on where you are coming from.  I hear all the time that having NT paths imported to WSL is peoples favorite feature and they love it.  Conversely, I also hear from others that it breaks their makefiles or other scenarios.  I think the problem is these features are hidden behind some not very well documented registry keys.  I'll create a work item to come up with a better way to set up various WSL configuration options.
Thanks for the feedback.

Just ran into this issue myself where running npm and nodejs were picking up the windows versions - modified the bashrc to fix it - But took me some figuring out before fixing it which lead me here - Just writing to +1 on making it better configurable - as it's not something I was expecting to come across.
Not per say an issue: Other then turning WSL on in windows - thats the only option I know exists for it.

Can confirm that both n-install AND rbenv suffer from this problem.

Can confirm that both n-install AND rbenv suffer from this problem

Suffering the same issues as everyone else while trying to use nodejs. IMO it's not worth polluting PATH in Linux just so we can run Windows apps. Managing PATH manually via bashrc seems like it could get tedious. I heard there's a registry switch for it and will probably use that but I feel strongly there should be an option to turn it on, but have it turned off by default.

So, can you add an alias in ~/.bashrc for this?
I think I have made a mess because I have installed node via Linuxbrew thinking that it would solve my $PATH issues
Is there a solution to this? I'm thinking about doing another install of WSL via the Windows insiders channel, I had it switched on as a beta feature before

Alternatively to the path_remove solution in https://github.com/Microsoft/BashOnWindows/issues/1890#issuecomment-318802876  it seems possible to completely disable appending NT PATH to https://github.com/Microsoft/BashOnWindows/issues/1493#issuecomment-266480323

My machine has gone off for an RMA now so I'll have a fresh and clean install to play with when it comes back 😄

https://github.com/Microsoft/WSL/issues/1890#issuecomment-338719291 works for the installed by lxrun.exe
https://github.com/Microsoft/WSL/issues/2048#issuecomment-324072237 works for the legacy distribution from the Store

I ran into an issue with the Windows path in my bash environment twice.
The second issue was a makefile getting confused on a path that contained embedded parentheses and spaces. While these are legal in Linux filesystems, they are rarely used. The correct fix was in the makefile.
The first issue was more insidious. I had Strawberry Perl on my Windows path. It's built with MinGW. Because Perl allows C interop, Strawberry Perl includes the MinGW C++ stdlib. I also have GCC on my Ubuntu install. GCC just searches the path for a compatible stdlib. When compiling I found that GCC would pick up the MinGW stdlib, which looks like a GCC stdlib, because it was first on the path. The fix was to switch to ActiveState Perl : )
I run one node.js app on WSL (Matt Godbolt's Compiler Explorer) and haven't seen any weird issues with node.

@benhillis IMO this should be disabled by default.
Simply importing the Windows path and expecting meaningful results is a little bit bonkers - even if that works some of the time, for certain things, the results are pretty random, since there's no guarantee that if/when something works, it works for the right reasons.
For example, I just realized Linux has been picking up my Composer global bin-folder and running scripts I installer under Windows with a different version of PHP under Linux - obviously, it has all the wrong dependencies installed, for the wrong version of PHP, and eventually failed hard.
Same would be the case for Node, Python scripts, etc. - just passing paths from the Windows environment to the Linux environment and expecting meaningful results is a little bit naive, isn't it?

@mindplay-dk  Try this to see if it can work for you:
PATH=$(echo "$PATH" | sed -e 's/:\/mnt.*//g') # strip out problematic Windows %PATH% imported var

@mindplay-dk Ah cool.. I'm using Ubuntu 18 Bionic.  I hope Ubuntu will have this option, great to hear.  Thank you.

Just wanted to leave a note here for future devs 👩🚀 -- if you get rid of your Windows path imports via any of the above mentioned methods, it seems to break some functionality like launching a Storybook on a local sever. Just a heads up 👍🏽

I personally have not had any issues at all with this.
My tools installed in wsl will always take precedence over a window one.
eg. composer in wsl will only run the linux version. I have to type composer.exe to run the windows one.

You can also run it from cmd, instead of bash
# Run from bash from WSL2
danon@desktop:~$ npm -h
bash: ./mnt/c/Program Files/node.js/npm: No such file or directory

# Run through cmd
danon@desktop:~$ /mnt/c/Windows/System32/cmd.exe /C npm -h

Usage: npm <command>

where <command> is one of:
    access, adduser, audit, bin, bugs, c, cache, ci, cit,
    clean-install, clean-install-test, completion, config,

It's a little slower, but paths are guranteed to work, since it's actually Windows shell that runs it.

Just add export PATH="$PATH:/usr/bin" to the end of ~/.bashrc, so that usr/bin takes precedence over windows' apps. Probably not a good option.
insignar in the stackoverflow

That was the only method that worked for me

I'm siding 100% with @mindplay-dk on this issue. I just experienced problems with installing a framework on Windows and WSL simultanously, which caused the WSL to mix up the path variables (which are in turn used to install certain programs). This was unexpected behaviour to me and I would have thought that those environments would run more secluded. Also, I don't really see the benefit of this feature other that I can run explorer.exe or code . from the get go. I think it's more intuitive to hook up explorer.exe and other apps inside your .bashrc as an opt-in.

[interop]
appendWindowsPath=false

The name of this setting indicates the problem perhaps it should prepend not append.
Distro should always have its own paths first so nothing breaks.

As was described here by @MartyMcFlyInTheSky, VSCode can be added as alias even when setting appendWindowsPath=false using:
alias code="/mnt/c/users/<user>/appdata/local/programs/'microsoft vs code'/code.exe"
This has a problem, though. VSCode will tell you:
When working on folders in wsl, we recommend to use a WSL window.
In a WSL window, all commands, extensions and the terminal run in the Linux subsystem.
To switch to a WSL window, use Reopen Folder in WSL

But, if you set the alias like that:
alias code="/mnt/c/users/<user>/appdata/local/programs/'microsoft vs code'/bin/code"
The VSCode will be opened "properly" and won't annoy you with the message.
I know the thread is ancient, it's just that I had to do this exact thing myself recently and had to do a bit of googling and reading to find an answer, since having appendWindowsPath=true makes tab-completion incredibly slow, so hopefully my answer helps somebody, too, as the previous answer by @MartyMcFlyInTheSky did for me.


Update READ.ME with correct diff for dependencies

When I first configured react-native-fcm I missed the compile project(':react-native-fcm') line because it's not stated as change in the diff viewer. corrected this

this should be done by react-native link


Add search box to docs

If these are generated with Sphinx, then this should be easy
i.e. the ones at https://prominence-eosc.github.io/docs
Really aids finding stuff!

New layout much easier to find stuff, thanks!
Maybe I missed it, but is there an explanation of what --openmpi actually does?  i.e. I think it launches mpirun udocker some_container some_command ?  For single node mpi jobs (only), there is more than one way to do it, e.g

Use --openmpi --nodes=1;  get mpirun udocker some_container some_command in prominence
Omit --openmpi, use udocker some_container "mpirun some_command" in prominence

It's effectively 2, i.e. mpirun is executed inside the container to ensure the right version of MPI is being used (option 1 depends on MPI on the host).
For multi-node MPI it also makes use of -mca plm_rsh_agent but the ssh command replaced with a script. The end result is that the appropriate commands are executed inside containers on other hosts, without ssh being involved.


upload returns array

(optional) Issue number: #532
Summary of the change: upload function returns array

Hello, is there any prediction to soul solve this problem?

This solves, no refreshing after files are uploaded. Thanks @HalfBottleOfMind !


Cannot see any results in log

I am experiencing difficulty in running the main.sh script to reproduce the results of running this parallel version of Leiden (the results in the paper sound great though!).
With minor modifications to the main.sh script, I added a devcontainer to the repo in a fork here to have a more reproducible environment that I can run experiments with on a laptop (with fewer threads defined) for initial testing.
I downloaded the europe_osm.mtx graph data and when I execute main.sh, I see messages in the log file (see below) but it's not the output I expect. Based on this line of code, I was expecting to see more informative output that describes the discovered partitions. I see similar output when I run on one of the other graph datasets referenced in the paper.
LOG FILE
2024-11-05 21:50:26 OMP_NUM_THREADS=4
2024-11-05 21:50:26 Loading graph /root/Data/europe_osm.mtx ...
2024-11-05 21:50:41 order: 50912018 size: 108109320 [directed] {}
2024-11-05 21:51:35 OMP_NUM_THREADS=4
2024-11-05 21:51:35 Loading graph /root/Data/europe_osm.mtx ...
2024-11-05 21:51:45 order: 50912018 size: 108109320 [directed] {}
2024-11-05 21:52:30 OMP_NUM_THREADS=4
2024-11-05 21:52:30 Loading graph /root/Data/europe_osm.mtx ...
2024-11-05 21:52:44 order: 50912018 size: 108109320 [directed] {}
2024-11-05 21:53:26 OMP_NUM_THREADS=4
2024-11-05 21:53:26 Loading graph /root/Data/europe_osm.mtx ...
2024-11-05 21:53:39 order: 50912018 size: 108109320 [directed] {}
2024-11-05 21:54:31 OMP_NUM_THREADS=4
2024-11-05 21:54:31 Loading graph /root/Data/europe_osm.mtx ...
2024-11-05 21:54:45 order: 50912018 size: 108109320 [directed] {}

Note: I am trying to run the code from the main branch. It's not clear why the code appears to fail silently or if it's running as expected. Can you provide any suggestions on what could be going wrong?

Will close this issue for now. I discovered the program was getting killed due to OOM issues.


Fix PR merging algorithm

The problem is that

The new_body doesn't know the current status.
Diff & merge algorithms can't treat checking status well.

So my solution is:

Scan every pr check status in old_body.
Update it to unchecked.
Then diff and merge as always.
Update check status by results of 1.

Close: https://github.com/motemen/git-pr-release/issues/20

A = <<ERB
- [x] #1001 aaaaaaaaaa
- [x] #1002 cccccccccc
ERB

B = <<ERB
- [ ] #1000 bbbbbbbbbb
- [ ] #1001 aaaaaaaaaa
- [ ] #1002 cccccccccc
ERB
print merge_pr_body A, B

Before
diff: ["!", [0, "- [x] #1001 aaaaaaaaaa"], [0, "- [ ] #1000 bbbbbbbbbb"]]
Found checklist diff; use old one: - [x] #1001 aaaaaaaaaa
diff: ["!", [1, "- [x] #1002 cccccccccc"], [1, "- [ ] #1001 aaaaaaaaaa"]]
Found checklist diff; use old one: - [x] #1002 cccccccccc
diff: ["+", [2, nil], [2, "- [ ] #1002 cccccccccc"]]
Use line as is: - [ ] #1002 cccccccccc
- [x] #1001 aaaaaaaaaa
- [x] #1002 cccccccccc
- [ ] #1002 cccccccccc

After
Found pull-request checkbox #1001 is x.
Found pull-request checkbox #1002 is x.
diff: ["+", [0, "- [ ] #1001 aaaaaaaaaa"], [0, "- [ ] #1000 bbbbbbbbbb"]]
Use line as is: - [ ] #1000 bbbbbbbbbb
diff: ["=", [0, "- [ ] #1001 aaaaaaaaaa"], [1, "- [ ] #1001 aaaaaaaaaa"]]
Use line as is: - [ ] #1001 aaaaaaaaaa
diff: ["=", [1, "- [ ] #1002 cccccccccc"], [2, "- [ ] #1002 cccccccccc"]]
Use line as is: - [ ] #1002 cccccccccc
Update pull-request checkbox #1001 to x.
Update pull-request checkbox #1002 to x.
- [ ] #1000 bbbbbbbbbb
- [x] #1001 aaaaaaaaaa
- [x] #1002 cccccccccc

I think that this solution can be acceptable.
Current problem is that checkbox list items are often lost after after merge. (#20, #21)
This is very critical. and there is a method to convert MR to checkbox, so it is reasonable to support this.
https://github.com/motemen/git-pr-release/blob/v0.6.0/bin/git-pr-release#L22-L24
Pros:

Checkbox items are not missing

Cons:

Checkbox items with no Issue/PR Number will not be merged

these will be duplicates

This is a trivial problem compared to losing items.
What do you think? @motemen

I think this problem is the limit of "2-way merge" algorithm.
It would be better to implement "3-way merge" if it must be strictly.

Last pull-request (past to be)
Last pull-request + edit by human. (as is)
New pull-request (to be)

Apply diff between 1 and  2 into 3. This makes the result correctly.

Overall looks good. Can you pull origin/matter and add some tests?

Okay, I will add some tests basing on #25

@motemen I added tests. Can you review this?

Thanks!


Add pinnings that are consistent with conda-forge

@johanneskoester and @mingwandroid I did not change the jinja variable logic here but I did add the pinnings as closer as I could to conda-forge's pinning script.
Closes #4 and #3

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@johanneskoester merging this b/c the current recipe is not usable with the rest of the stack. Hopefully we will find a proper solution to the pinning bot soon.


Scan Activity Not Opening

I want to start your scan homepage when my "SCAN" button is clicked but button is not working when no preference is passed using putExtra

    upload = (Button) findViewById(R.id.btn_logo);
    upload.setOnClickListener(new ScanButtonClickListener(ScanConstants.OPEN_MEDIA));

This will open media but I want the home page

To open homepage  i.e scanning page,first you have to upload something for scanning .

Can you share the code snippet or how to open the home page?
On Mon, Jul 2, 2018, 12:38 Deepak111992<EMAIL_ADDRESS>wrote:

To open homepage i.e scanning page,first you have to upload something for
scanning .
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jhansireddy/AndroidScannerDemo/issues/76#issuecomment-401691867,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYxQgF8zWEW5xBz0FPUFdKkempa1AP1zks5uCcbXgaJpZM4U6Adb
.

Download the project,you will get everything there .

same i couldnt open the homepage when passing 0 as preference, and i couldnt find anything in the project

When you say Scan Home, do you mean the Image to crop with selection polygon? May be you could have directly started ScanActivity.


Minor cleanup on errors and extension handling

Hopefully you're fine with these changes:
Removed \n inside error messages as formatting should be down to what's displaying, if this is in a gui program forced formatting is just ugh.
Made verbose a static bool, though perhaps that doesnt make sense.

I totally missed the PR and manually applied the changes :facepalm:
Thanks for the suggestions :+1:


Show RFI attachments in Summary page

As an analyst
I want to be able to see the uploads the applicant did in the summary page
So that all the info I need to assess an application is in one place
Description:
Now that the analyst can download the attachments from the application on the summary form, there is a request to also be able to see the RFI attachments in the summary page in the appropriate places.
Acceptance Criteria:
Wireframe

[x] Analyst can see the RFI uploads by the applicant (and analyst) in the appropriate places on the summary page
[x] The RFI uploads are visible as hyperlinks to the documents

RFI replaced a single file upload

Two RFIs for the same single file upload

One RFI for multi-file upload

[x] Below each file, right aligned, display the date & time the file was uploaded by the applicant or the time entered by the analyst
[x] Below a group of files from the same RFI, display the RFI #

Received from RFI-CCBC-000000-0

[x] no spacing between filename & "Received from..." or "Original"
[x] 8px between groups of files
[x] icon is https://fontawesome.com/icons/star?f=classic&s=solid&sz=2xs size: 2xs
[x] icon only appears on most recent if the section is single file upload
[x] icon does not appear on any RFI of multi-file uploads
[x] After an RFI is received, add the word "Original" to the original file(s)
[x] filename and text below ("Original" or "Received from...") are left-aligned
[x] icon is to left of the filename and text below
[ ] indent any rows without a star icon (it may be 14px indent if icon is 12px wide and there's 2px spacing between icon & text

Implementation Details

Hey team! Please add your planning poker estimate with Zenhub @AntBush @marcellmueller @rafasdc

Since we changed that all rfi files are now multi file upload the star icon will never show, might be something to discuss.

hey @marcellmueller in this one do you know why I wouldn't be able to see this Template 1

I can see the other supporting document file but not template 1 in the Summary

@MarsRomer Looking into it!

Looks like it was due to RFI files not being displayed on any field that had the red error since they were missing files. I totally missed this in development since I was testing on fields that already had files 🤦 Anyways there is a fix incoming!

The "RFI" is missing.
In prod it says, "Received from CCBC-000000-0" when it should be "Received from RFI CCBC-000000-0"


Issue with Displaying Games by Category

The homepage displays all posted games correctly, and the search feature works fine. However, when trying to access games by category (e.g., /games/action), the page does not load.
Details:
I have posted 3 games with different categories.
All posts appear on the homepage, but when accessing a category (e.g., /games/action), nothing is displayed.
But thanks for source code

Hello. Thank you for your feedback. Everything works fine for me. Where have you tried in your local machine or in my link which I have given? If you have tried in the link that I have given and directly try to go to a link from the url field, you will get an error, this is caused by the hosting. If you have tried on your local machine, try the link I have given. log in with the account that is the creator and test.

Thanks my bro, i have finish solve this by add indexing in firestore.


Maintainer access to Google analytics

Currently it seems that none of the website maintainers have access to the Google Analytics console
Its not clear whether a new account has been set up and whether we need to be provided access to that - @chalin perhaps you can now give us access ?
cc @mattklein123

I've added you both (@phlax and @mattklein123) as admins using your emails address that I got from OWNERS. You should have received invites. Feel free to add other maintainers as Editors. Let me know when that is done; I may need to bump down your role to Editor some time (I'm still double checking policies on the CNCF side).

great thanks - i didnt receive an email, but i have access now

Just to be clear, I've granted you access to the GA4 property. I still don't have access to the UA property.

just getting up to speed - its a while since i looked at analytics
do you need something from us/someone to setup the UA tracking?

FYI, per CNCF policy, I'm setting your roles to Viewer.

do you need something from us/someone to setup the UA tracking?

The UA site tag should be receiving events too since I've connected to it from the GA4 site tag. What I need is access to the UA property. I've sent out a few requests. In the end, if no one we know still has access to the UA property, we'll just work from the GA4 property. Eventually, the UA property will go away once Google stops supporting UA and drops the data.


Delete/disable blue "i" Message

For "A blue 'i' indicates a past vulnerability that is not expected to affect the installed package", that means this "Message" is useless to me. I hope never see them. Just delete this message, or at least let me disable them from Options setting.

Thank you for the feedback. You are quite correct, most people will get no value from these extraneous messages so for now I have removed them. Perhaps in a future version I will make it configurable, but I would guess it will not be a highly requested feature.
The code fix has been pushed. I will create a new release on the weekend, at which point I will close the issue report.

Release 1.2.5 has been pushed to Visual Studio Gallery
Thanks again for the feedback.


Compilation Fails zig 0.8.0-dev.1141+68e772647 on Windows 10

Hello, I saw in the one other issue currently open that you are sticking with zig 7.1 right now... but thought it would be worthwhile giving you a heads up anyway. The line it is complaining about seems okay to me?

PS F:\github\paulevans\sokol-zig> zig version
0.8.0-dev.1141+68e772647
PS F:\github\paulevans\sokol-zig> zig build
Semantic Analysis... .\src\sokol\gfx.zig:718:5: error: expected function or variable declaration after pub
pub inline fn setup(desc: Desc) void {
    ^
clear...The following command exited with error code 1:
C:\Apps\zig-windows-x86_64-0.8.0-dev.1141+68e772647\zig.exe build-exe F:\github\paulevans\sokol-zig\src\examples\clear.zig F:\github\paulevans\sokol-zig\zig-cache\o\127299955b41b17a241a44ab60483b30\sokol.lib -lc --cache-dir F:\github\paulevans\sokol-zig\zig-cache --global-cache-dir C:\Users\pevans\AppData\Local\zig --name clear --pkg-begin sokol F:\github\paulevans\sokol-zig\src\sokol\sokol.zig --pkg-end --enable-cache
error: the following build command failed with exit code 1:
F:\github\paulevans\sokol-zig\zig-cache\o\bb6c8c71e8d64ef9dad598f452b0d922\build.exe C:\Apps\zig-windows-x86_64-0.8.0-dev.1141+68e772647\zig.exe F:\github\paulevans\sokol-zig F:\github\paulevans\sokol-zig\zig-cache C:\Users\pevans\AppData\Local\zig

So I guess the language changed how you declare inline?
pub fn setup(desc: Desc) callconv(.Inline) void {
    sg_setup(&desc);
}

compiles - it then fails on shutdown.
So this is more of a TODO for 8.0 than a bug!

fwiw zig fmt will automatically upgrade that particular thing to the new syntax.

Thanks, you are correct. That almost worked. Had to change one extra line in debugtext-print.zig to read "Hello '{s}'!" - adding that s.
Here's the diff: https://github.com/floooh/sokol-zig/compare/master...paulevans:zig8.0?expand=1

FWIW I'll try to return to the Zig bindings soon-ish after I've taken care of a couple of pending issues and PRs for the sokol headers. That specific inline thing can be changed here in the generator-script:
https://github.com/floooh/sokol/blob/master/bindgen/gen_zig.py#L436
However, the underlying C-APIs of the sokol-headers have changed a bit (regarding signed vs unsigned parameters) which would also break the current example code. I'll need to do some type-mapping in the bindings-generator, because in Zig I want to change some function parameters to unsigned where the C-API has signed numbers (reason being that it's "safer" to use unsigned parameters in Zig because of Zigs robust overflow checking, while in C it's easy to accidentally produce an underflow, so for C, using "almost always signed" makes more sense).

I have been looking at patching the generator actually already. Although I
didn't realize I needed clang installed to run it. So I have been
on-and-off prodding that clang build on my windows machine - been a while
since I last did that and on a different machine.
Ended up fixing something tiny in fips as well. Just something windows
specific to do with windows store installed python.
Anything I think is useful enough to share I'll fire off some pull requests.
I have been seeing if I could get the sokol-imgui bindings generated for
zig.
Though thanks for the heads up, sounds like that probably won't work out
with recent changes.
So were you planning on just having a shim that cast between signed and
unsigned in the bindings layer then?
I've just started to poke around in zig and I know it can compile c -
though not tried that myself yet - just via other people's projects. The
idea of the extra bindings is to take better advantage of zig's safety
features and make things seem more zig-gy?
On Fri, 19 Feb 2021 at 04:16, Andre Weissflog<EMAIL_ADDRESS>wrote:

FWIW I'll try to return to the Zig bindings soon-ish after I've taken care
of a couple of pending issues and PRs for the sokol headers. That specific
inline thing can be changed here in the generator-script:
https://github.com/floooh/sokol/blob/master/bindgen/gen_zig.py#L436
However, the underlying C-APIs of the sokol-headers have changed a bit
(regarding signed vs unsigned parameters) which would also break the
current example code. I'll need to do some type-mapping in the
bindings-generator, because in Zig I want to change some function
parameters to unsigned where the C-API has signed numbers (reason being
that it's "safer" to use unsigned parameters in Zig because of Zigs robust
overflow checking, while in C it's easy to accidentally produce an
underflow, so for C, using "almost always signed" makes more sense).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/floooh/sokol-zig/issues/7#issuecomment-781978573, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFPRSFFMZPCOYBNH45R3JLS7Y26ZANCNFSM4X3NNLBA
.

I needed clang...

I found the easiest way to install clang on Windows is scoop install llvm. IFIR I haven't tested the code generator on Windows though, mostly used macOS and Linux for this stuff.

is to take better advantage of zig's safety features and make things seem more zig-gy

Mostly to make the API more "zig-gy", but Zig was also an easy first target because it binds really well to C-APIs. The goal is though to support more language bindings with minimal work. Next up is nim support for instance.
I wrote a blog post about this here: https://floooh.github.io/2020/08/23/sokol-bindgen.html


Unable to apply Take() to a navigation property

Hi.  I have an aggregate root "Player" with a collection "ActiveDays" via a "PlayerId" foreign key.  When I try to apply "Take()" to the navigation query to limit the number of results, like so:
Query.Include(player => player.ActiveDays!.Take(limit));
I get the error System.InvalidOperationException : The LINQ expression 'ROW_NUMBER() OVER(PARTITION BY u0.user_id ORDER BY u0.user_id ASC, u0.day_active ASC)' could not be translated. Either rewrite the query in a form that can be translated, or switch to client evaluation explicitly by inserting a call to 'AsEnumerable', 'AsAsyncEnumerable', 'ToList', or 'ToListAsync'. See https://go.microsoft.com/fwlink/?linkid=2101038 for more information.
(Note that I'm mapping the column "user_id" to "PlayerId" as this is a legacy schema that has some inconsistencies.  The ActiveDays table has a composite primary key of user_id and day_active.)
Attempting to add a "AsEnumerable" or similar results in
System.InvalidOperationException : The expression 'player.ActiveDays.AsEnumerable().AsQueryable().Take(__limit_1)' is invalid inside an 'Include' operation, since it does not represent a property access: 't => t.MyProperty'. To target navigations declared on derived types, use casting ('t => ((Derived)t).MyProperty') or the 'as' operator ('t => (t as Derived).MyProperty'). Collection navigation access can be filtered by composing Where, OrderBy(Descending), ThenBy(Descending), Skip or Take operations. For more information on including related data, see http://go.microsoft.com/fwlink/?LinkID=746393.
Is there something I am misunderstanding or is this an issue?
(DB is MySQL ver. 5.7.22,)

The first thing to check is if you can make the query work directly with your dBcontext. Apply your LINQ to dBContext.Players and see if you can get it to work. If not, the problem isn't related to this Specification package.

Hi @mcchessers-sp,
Unfortunately, it has nothing to do with this library. We're passing the expression as it is. I suspect the MySQL provider doesn't know how to translate the filtered include which contains Take.
You may try to test without specifications, working directly with DbContext.

Here I created a full test app using SqlServer provider. It works as expected. Try the same code with MySql provider.
using Ardalis.Specification;
using Ardalis.Specification.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore;

await AppDbContext.SeedAsync();

using var dbContext = new AppDbContext();
var repo = new Repository<Customer>(dbContext);
var spec = new CustomerSpec();

var customer = await repo.FirstOrDefaultAsync(spec);

Console.WriteLine($"We fetched {customer!.Addresses.Count} addresses!");

public class CustomerSpec : Specification<Customer>
{
    public CustomerSpec()
    {
        Query.Include(x => x.Addresses.Take(2));
    }
}
public interface IRepository<T> : IRepositoryBase<T> where T : class
{
}
public class Repository<T> : RepositoryBase<T>, IRepository<T> where T : class
{
    public Repository(AppDbContext dbContext) : base(dbContext)
    {
    }
}
public record Address
{
    public int Id { get; set; }
    public int CustomerId { get; set; }
}
public record Customer
{
    public int Id { get; set; }
    public List<Address> Addresses { get; set; } = new();
}
public class AppDbContext : DbContext
{
    public DbSet<Customer> Customers => Set<Customer>();
    public DbSet<Address> Addresses => Set<Address>();
    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
    {
        var connectionString = "Data Source=(localdb)\\MSSQLLocalDB;Initial Catalog=IncludeTest;Integrated Security=SSPI;ConnectRetryCount=0;";
        optionsBuilder.UseSqlServer(connectionString).LogTo(Console.WriteLine);
    }

    public static async Task SeedAsync()
    {
        using var dbContext = new AppDbContext();
        await dbContext.Database.EnsureDeletedAsync();
        await dbContext.Database.EnsureCreatedAsync();
        var customer = new Customer
        {
            Addresses = new List<Address>()
            {
                new Address(),
                new Address(),
                new Address()
            }
        };
        dbContext.Customers.AddRange(customer);
        await dbContext.SaveChangesAsync();
    }
}

Thank you for the suggestions.  I tried using DbContext outwith the specification:
DbContext.Players.Where(p => p.Id == (int)TestPlayers[0].Id).Include(p => p.ActiveDays.Take(1));
.. and got the same error, which would suggest that perhaps the driver (Pomelo) may be at fault?

Yes, you might open an issue there (if there isn't one already).


Update power.md script section

More efficient bash script, and option to "calibrate" the reading.

Thanks! Closing this as the the firmware/driver for voltage reading has been updated.


Snippets

Some language servers support, or even require, using snippet (and so does compe), we could add a basic snippets plugin. Not sure if this is "defaults" worthy.

Closed in b202db1


I could not see Sign tag from Project Properties in Visual Studio 2022

[Enter feedback here]
Hi there,
I could not see Sign tag from Project Properties in Visual Studio 2022. I also googled and could not find the solution. Could you please advice? Thank you.

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

ID: 769e8747-667e-9f03-5308-93dc3b1663c1
Version Independent ID: df88c409-1613-61b4-ea80-e67fe21c1943
Content: Signing Page, Project Designer - Visual Studio (Windows)
Content Source: docs/ide/reference/signing-page-project-designer.md
Product: visual-studio-windows
Technology: vs-ide-deployment
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@roundmoon88 -- thank you for your feedback and request.  Please consider these resources, if you haven't already:

Visual Studio on Microsoft Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@Mikejo5000 -- please look into this issue.

@roundmoon88 Did you resolve this? Looks like you are using a .NET Core or .NET 5+ project. There is an option to strong name your assemblies under Build > Strong naming.

@Mikejo5000 Hi Mikejo5000. Thank you for your comments. The issue is fixed now. I appreciate you.

@roundmoon88 -- thank you for your successful follow-up!

Hi sorry if this is not the right place to comment on this closed bug. In vs 2019 you could give the signature a strong key name, but I'm not seeing this in vs 2022. I've selected to allow signing under the strong naming area, but no ability to give the key a name.

@dunkhi -- thank you for your follow-up.  The previous comments in this thread cover most of the possibilities.  I as well do not see an option for naming a key in Visual Studio 2022.
If you feed this is a product issue, please add it here:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Use the new Suggest a feature for Visual Studio IDE menu item.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.


Enhancement: footer links now customizable and removable

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
CONTRIBUTING.md also tells you what to expect in the PR process.
Description
Footer links can be customized (edited and/or removed)

📦 Published PR as canary version: 20.45.0--canary.976.05cdc40d9ae9b069f856835d390d301bc52d5e37.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

What is this? key="medium-https://yourwebsite.com/terms-_blank-true-https://yourwebsite.com/imprint-_blank-true-https://yourwebsite.com/privacy-policy-_blank-true-https://yourwebsite.com/glossary-_blank-true-true-© 1999 - 2022 Infineon Technologies AG" It's shown on Storybook.

Suggestion: Since the copyright-text is not a link, then I would suggestion instead of having two different Props, one for the text, and one for showing the text, it might be better to just have one, copyright-text, and if empty, then simply don't show it. Being empty can be construed as not wanting to have it at all. Same can be applied even for the links. If a link, such as Privacy, does not have link set, then it's essentially not needed, and therefore it might be better to just not show it.

What is this? key="medium-https://yourwebsite.com/terms-_blank-true-https://yourwebsite.com/imprint-_blank-true-https://yourwebsite.com/privacy-policy-_blank-true-https://yourwebsite.com/glossary-_blank-true-true-© 1999 - 2022 Infineon Technologies AG" It's shown on Storybook.
Suggestion: Since the copyright-text is not a link, then I would suggestion instead of having two different Props, one for the text, and one for showing the text, it might be better to just have one, copyright-text, and if empty, then simply don't show it. Being empty can be construed as not wanting to have it at all. Same can be applied even for the links. If a link, such as Privacy, does not have link set, then it's essentially not needed, and therefore it might be better to just not show it.

I am not sure if we can conditionally hide and show props on Storybook. Will investigate. And yes, for the copyright-text, you're absolutely right. I will remove it.

2. pyright-text is not a link, then I would suggestion instead of having two different Props, one for the text, and one for showing the text, it might be better to just have one, copyright-text, and if empty, then simply don't

fixed point 1 as well

I am not sure if we can conditionally hide and show props on Storybook. Will investigate. And yes, for the copyright-text, you're absolutely right. I will remove it.

We can. It's already done for some components. Simply replace the this.displayGlossaryLink && condition, with this.glossaryUrl &&, and assign an empty string by default. You could also create a another function to filter the Prop values to make sure it's really empty. In my opinion, if the user does not pass any url value, then they don't want the link to be displayed at all, and therefore we could make the prop value a condition in itself without having to create another Prop for display.

Okay, right! So the same as I have done for the copyright basically. Ill modify the checks and remove the non necessary props


Check if a grid column has value

HI;
I am struggling to check if entire column A in xxGrid is empty or has value.  Thank you

check summary or multiselect plugins/mixins, they must be iterating tru whole data set, so you can take code from there

@ga5tan thank you for the hint;
Using following code, i am tyring to count rows with null "reason" but it is not working.
               var selected = Expenditure.StagingBudgetRow.idProperty;
        var selectedOutput = 0;
        for (var i = 0; i < selected.length; i++) {
            if (Expenditure.StagingBudgetRow.Fields.Reason === null) {
                selectedOutput+=1;
            }
        }
        if (selectedOutput==0) {
            buttons[Q.indexOf(buttons, x => x.cssClass == "synchronize-button")].cssClass += " disabled";
        } else {
            buttons[Q.indexOf(buttons, x => x.cssClass == "synchronize-button")].cssClass += " ";
        }


Add Purebasic

PureBasic is a modern BASIC programming language. The key features of PureBasic are portability (Windows, Linux and OS X supported with the same source code), the production of very fast and optimized native 32-bit or 64-bit executables and, of course, the very simple BASIC language syntax. PureBasic has been created for the beginner and expert alike. We have put a lot of effort into its conception to produce a fast, reliable system and friendly BASIC compiler.

Despite its beginner-friendly syntax, the possibilities are endless with PureBasic's advanced features such as pointers, structures, procedures, dynamically linked lists and much more. Experienced coders will have no problem gaining access to any of the legal OS structures or API objects and PureBasic even allows inline assembly.

https://www.purebasic.com/

Thanks, I'll add it.


Carto panics when failing to stamp a resource with AlwaysHealthy rules

Bug description:
When Cartographer attempts to stamp a resource with an AlwaysHealthy health rule, it may fail to stamp the object. However, it will still evaluate the AlwaysHealthy rule, which, when the resource is reconciled for the first time, will be recognized as having transitioned from Unknown to Healthy (Cartographer assumes the state is always previously Unknown if it has no record of it being stamped out).
This in turn triggers a panic when attempting to record the ResourceHealthyStatusChanged event because there is no guard for the stamped object used in the event message being nil at: https://github.com/vmware-tanzu/cartographer/blob/main/pkg/realizer/realizer.go#L167
Steps to reproduce:
Please provide a set of steps to reproduce the behavior (example):
Apply supply chain with a resource that will fail to stamp (e.g. reference a path that doesn't exist in the template), but has an AlwaysHealthy rule.
Expected behavior:
Definitely: Cartographer does not panic.
Unclear: should a health status transition really have occurred? Is it really correct that we consider the resource healthy even if it's not stamped?
Actual behavior:
Cartographer panics and crashes.
Logs, k8s object dumps:
You'll see a fun panic like this in the logs:
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x184c8b0]

goroutine 721 [running]:
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile.func1()
	/Users/scoward/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:118 +0x2d9
panic({0x2d2cc80, 0x3c58290})
	/usr/local/Cellar/go/1.19/libexec/src/runtime/panic.go:890 +0x267
k8s.io/apimachinery/pkg/apis/meta/v1/unstructured.(*Unstructured).GetAPIVersion(0x0)
	/Users/scoward/go/pkg/mod/k8s.io/apimachinery@v0.25.2/pkg/apis/meta/v1/unstructured/unstructured.go:218 +0x70
k8s.io/apimachinery/pkg/apis/meta/v1/unstructured.(*Unstructured).GroupVersionKind(0x0)
	/Users/scoward/go/pkg/mod/k8s.io/apimachinery@v0.25.2/pkg/apis/meta/v1/unstructured/unstructured.go:426 +0x59
github.com/vmware-tanzu/cartographer/pkg/events.ownerEventRecorder.qualifiedResourceName({{0x30f86a8, 0xc000a9c6c0}, {0x3109710, 0xc0009b17c0}, {0x310ddd8, 0xc000528230}, {{0x310c870, 0xc000f51b30}, 0x0}}, 0x0)
	/Users/scoward/workspace/cartographer/pkg/events/owner_recorder.go:109 +0x90
github.com/vmware-tanzu/cartographer/pkg/events.ownerEventRecorder.ResourceEventf({{0x30f86a8, 0xc000a9c6c0}, {0x3109710, 0xc0009b17c0}, {0x310ddd8, 0xc000528230}, {{0x310c870, 0xc000f51b30}, 0x0}}, {0x2f5744a, ...}, ...)
	/Users/scoward/workspace/cartographer/pkg/events/owner_recorder.go:79 +0xd9
github.com/vmware-tanzu/cartographer/pkg/realizer.(*realizer).Realize(0xc000461e20, {0x310a478, 0xc00139fd70}, {0x30f3c40, 0xc0003ed880}, {0xc000663458, 0x12}, {0xc000184100, 0x7, 0x8}, ...)
	/Users/scoward/workspace/cartographer/pkg/realizer/realizer.go:159 +0xe90
github.com/vmware-tanzu/cartographer/pkg/controllers.(*WorkloadReconciler).Reconcile(0xc000b98000, {0x310a478, 0xc0011601b0}, {{{0xc000d48f90, 0x2a}, {0xc000a1f480, 0x10}}})
	/Users/scoward/workspace/cartographer/pkg/controllers/workload_reconciler.go:148 +0x219c
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile(0xc0007afcc0, {0x310a478, 0xc000f51b00}, {{{0xc000d48f90, 0x2a}, {0xc000a1f480, 0x10}}})
	/Users/scoward/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121 +0x172
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler(0xc0007afcc0, {0x310a478, 0xc000f51b00}, {0x2da3000, 0xc000c4f220})
	/Users/scoward/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320 +0x3d9
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem(0xc0007afcc0, {0x310a3d0, 0xc0009b0b80})
	/Users/scoward/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273 +0x3a5
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2()
	/Users/scoward/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234 +0xdc
created by sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2
	/Users/scoward/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:230 +0x4d9
make: *** [run] Error 2

Versions:

main branch of carto as of 9bf3708f

Deployment info:
n/a
Additional context:
n/a

Is it correct to assume it isHealthy when stamping of the object fails, just because the health status is AlwaysHealthy?
Is that more confusing for users?
What would happen if stamped objects had to exist (barring the incoming passThrough) for AlwaysHealthy to take effect?


Hardcoded skills in autocalc MaxStress

The function searchs for the skill Resilience:
	_calculateMaxStress() { ... 			i => i.type === "skill" && i.name === "Resilience" 		);
I use custom skills in spanish, so Resilience is
      "img": "systems/ac2d20/assets/skills.svg",       "name": "Fortaleza",       "type": "skill",
And produces this error:

You don't need to use custom skills for translation purposes any more as the default ones are automatically translated if necessary.  Custom skills are now purely for if you want to add your own additional skills and if so you use a copy of the base skills to start with.

I dont like the idea of using system english skills for two reasons:
+Skills compendium is not translated itself (appears in english, so for spanish user it keeps appearing Resilience instead of Fortaleza if someone want to search compendia for Fortaleza) - i know this is a little minor detail
+I lose the spanish description of each skill and focus in the skill item, there is no way of adding spanish description and case-use of each skill if I use the i18n

Or going definitively for system skills, removing the skill-compendium setting, you can always draganddrop custom skills whenever needed

This is probably the way forward now everything is translated properly.

@lozanoje I'm probably going ahead with retiring the custom skills for the next release (which will probably only be a few days).
As there's no easy way for me to migrate character skills data, I'm thinking of the following migration steps to run automatically:

Go through each Character and NPC and find any non-standard skills (ie. names don't match exactly), and if found:

Capture the skill name, value and list of tagged Focuses for later
Delete the skill from the character

Add any "missing" skills from the core set

Output a message to chat with the character name and a listing of the skills that were removed by their original name, showing each one's value and a list of tagged focuses (one message per character).

Does that sound OK to you?  That should at least reduce the amount of work required by the GM after updating the system.

@lozanoje I'm probably going ahead with retiring the custom skills for the next release (which will probably only be a few days).
As there's no easy way for me to migrate character skills data, I'm thinking of the following migration steps to run automatically:

Go through each Character and NPC and find any non-standard skills (ie. names don't match exactly), and if found:

Capture the skill name, value and list of tagged Focuses for later
Delete the skill from the character

Add any "missing" skills from the core set

Output a message to chat with the character name and a listing of the skills that were removed by their original name, showing each one's value and a list of tagged focuses (one message per character).

Does that sound OK to you? That should at least reduce the amount of work required by the GM after updating the system.

I made it automatically outside Foundry, used foundry-cli to export all actors and then I programmed the replacement of each skill in spanish back to english in every file and then I packed it again.
Probably best option is: take skill names from world's language file (es.json), find the counterparts in english (en.json) and replace, then list the missing skills
I dont know how many users has their custom skills, probably only spanish users (due to the skill compendium i provided with my translation module)
Fully updated spanish file is at: https://github.com/lozanoje/fvtt-module-ac2d20-es/blob/main/lang/es.json including new tooltip feature, I will try to send you a PR with this updated file

Resolved by the data migration in the next release


Pinned dependency failing to download

Output of helm version: v2.16.1-linux-amd64
Output of kubectl version: 1.14.9
Cloud Provider/Platform (AKS, GKE, Minikube etc.): AWS
https://kubernetes-helm.storage.googleapis.com/helm-v2.16.1-linux-amd64.tar.gz is saying the key does not exist.
v2.16.12 does exist. Is this following SEMVER? I know https://helm.sh/docs/chart_best_practices/conventions/ mentions SEMVER, but it is unclear if this applies to HELM or just the versions of upstream images pulled.

Duplicate of #10005

Closing as related issues: #10005 #10004 #10003 #10011

Thank you and my apologies. We wound up pinning to a newer tagged version, but I also see https://github.com/helm/helm/issues/10005 references a script. I'll see if our platform team can get it into the backlog. I'd probably just tried to alter some old code, but the frustration came from helm deploying Friday and not today without code changes our-side.

but I also see #10005 references a script.

Which script? I don’t see one referenced in the comments.


Statistics explanation screen texts (EXPOSUREAPP-4509)

PR for translations

@janetback, please, kindly review the dots at the end of sentences.

@janetback, please, kindly review the dots at the end of sentences.


Move images to external storage

Helps keep this repo fast and light (and probably will let the images load faster too).
Might take 12–24 hours for these images to reliably show up as the DNS switches over.

Small request: Could you add me to admin the site on CF?

I just sent you an invite. I'm not sure if that'll give you access to R2 though… mind letting me know if you can see/edit the html-green bucket?

I see the site but I don't see the bucket.

Hmm… from some searching around it doesn't look like it's possible. If you have an FTP client like Transmit I could give you an access token and you could use the S3-compatible API?

Sure, let's give that a try. DM me the details.


1.3.8

1.3.8 release

Codecov Report

Merging #76 into master will decrease coverage by 1.97%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
- Coverage   74.19%   72.22%   -1.98%
==========================================
  Files           2        2
  Lines         155      162       +7
==========================================
+ Hits          115      117       +2
- Misses         40       45       +5

Impacted Files
Coverage Δ

pyseleniumjs/e2ejs.py
72.04% <28.57%> (-1.98%)
:arrow_down:

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 24d40f7...cbd10f8. Read the comment docs.


optimizeForSpeed mode sometimes attempts to insert multiple rules as one

Do you want to request a feature or report a bug?
Bug.
What is the current behavior?
In optimizeForSpeed mode, sometimes multiple rules are being incorrectly stuffed into .insertRule() at once, here:
https://github.com/zeit/styled-jsx/blob/343795d3b2fef6d3d085921b0d4cf31b4d9be709/src/lib/stylesheet.js#L129-L139
In production the error is swallowed. This made it exceedingly hard for me to figure out why two seperate Next.js projects were having horrific style issues in production; the SSR page would be styled fine, but on client side route transitions, the styled-jsx styles for many of the components would be simply missing.
Here is a spectrum post from when I first encountered this bug a few weeks ago:
https://spectrum.chat/styled-jsx/general/absolutely-stumped-why-next-js-page-styles-broke~3ad888fa-23fc-4e6c-9948-fffa81a4e956
I found the "fix" at the time was to remove styling from a single component:
https://github.com/jaydenseric/device-agnostic-ui-website/commit/a44fca1f988038d4245054a3b1990eef8d053cbb
Fast forward to now, and I have another project that suddenly started bombing out, here is a Zeit Now deployment:
whimsy-app-hhuy9ally.now.sh
If I put a console.error(error) in here, this is an example error:

Here is a problematic "rule" that was being inserted:
.jsx-3944333314,.jsx-3103593668 ul,.jsx-3103593668 ol{margin-top:1em;margin-bottom:1.5em;padding-left:2em;}.jsx-3944333314:first-child,.jsx-3103593668 ul:first-child,.jsx-3103593668 ol:first-child{margin-top:0;}.jsx-3944333314:last-child,.jsx-3103593668 ul:last-child,.jsx-3103593668 ol:last-child{margin-bottom:0;}.jsx-3944333314>li,.jsx-3103593668 ul > li,.jsx-3103593668 ol > li{margin:0.5em 0;max-width:30em;line-height:var(--daui-line-height);}.jsx-3944333314>li:first-child,.jsx-3103593668 ul > li:first-child,.jsx-3103593668 ul > li:first-child{margin-top:0;}.jsx-3944333314>li:last-child,.jsx-3103593668 ul > li:last-child,.jsx-3103593668 ol > li:last-child{margin-bottom:0;}

It was the result of a few weeks of work without a production deployment, so unfortunately I can't say exactly what change pushed the bug to occur. I haven't lucked out on a miraculous "fix" this time though, so after MANY hours of debugging I'm getting pretty desperate!
If the current behavior is a bug, please provide the steps to reproduce and possibly a minimal demo or testcase in the form of a Next.js app, CodeSandbox URL or similar
I'm sorry I only have complicated, real world examples linked above and I haven't been able to isolate a minimal reproduction.
I have discovered that debugging is a lot easier when running optimizeForSpeed in dev, so traces aren't minified:
{
  "presets": [["next/babel", { "styled-jsx": { "optimizeForSpeed": true } }]]
}

What is the expected behavior?
Styles managed via styled-jsx in a Next.js app render correctly on the client after route transitions.
Environment (include versions)

OS: macOS
Browser: Chrome v78
styled-jsx (version): v3.2.4

Did this work in previous versions?
Not sure.

@jaydenseric hey mate I am sorry that you are experiencing this kind of issue! Sounds super frustrating. Before we even try to debug, can you make sure that both the library and the consumer app are transpiled in the same "mode". The issue could be that your components library is compiled for development mode and when the Next.js app is deployed to prod it uses instead optimizeForSpeed which cannot insert styles compiled for dev mode.

I'll check, but if that is the case, how come none of the styles are broken right now at https://deviceagnosticui.com, or https://jaydenseric.com?

it seems that you are not using optimizeForSpeed there

this article explains how to bundle a library that uses styled-jsx to avoid this issue https://medium.com/@tomaszmularczyk89/guide-to-building-a-react-components-library-with-rollup-and-styled-jsx-694ec66bd2

Thanks for the revelation, although this is a real headache to solve elegantly 😓
Bundling published packages is an antipattern, and you cant have require() inside ESM for a conditional export. ESM for tree-shaking is an important feature for a published a component library, so I don't want to drop it. Publishing invalid ESM is undesirable because eventually, maybe even in January when conditional package exports are shipped by Node.js, I'll be publishing ESM intended to be run natively.
I think I might publish two builds at device-agnostic-ui for development and device-agnostic-ui/production for production, then have Babel/Webpack config to setup in Next.js to rewrite the imports accordingly.

Is optimizeForSpeed even worth it? What exactly are the benefits?
I might just continue to publish device-agnostic-ui components and styles transpilled with it false, and in the Next.js setup instructions explain that the Babel config needs to be configured like this:
{
  "presets": [["next/babel", { "styled-jsx": { "optimizeForSpeed": false } }]]
}

optimizeForSpeed uses the CSSOM apis which are way faster than creating script tags.
As for bundling you have a few options:

Ship the source code and let the host app build it
You can use conditionals, bundlers will treeshake this https://github.com/giuseppeg/style-sheet/blob/aa125093adcef82ca3bd86b58f5f203ef232a4fa/lib/esm/createElement.js (works only if the bundler replaces process.env.NODE_ENV === 'production' with a constant, eg. webpack does that)

won't work in UMD or browser of course

Here is an example of the difference between transpiling published components without and with NODE_ENV=production…
For this component:
https://deviceagnosticui.com/components/Blockquote
Here is without:
export var stylesBlockquote = {
  styles: React.createElement(
    _JSXStyle,
    {
      id: '3946479178'
    },
    '.jsx-3946479178,.' +
      stylesHtml.className +
      ' blockquote{margin-top:1.5em;margin-bottom:1.5em;margin-left:1.5em;max-width:22em;line-height:var(--daui-line-height);font-style:italic;font-weight:300;text-rendering:optimizeLegibility;}.jsx-3946479178:first-child,.' +
      stylesHtml.className +
      ':first-child{margin-top:0;}.jsx-3946479178:last-child,.' +
      stylesHtml.className +
      ':last-child{margin-bottom:0;}'
  ),
  className: 'jsx-3946479178'
}

Here is with:
export var stylesBlockquote = {
  styles: React.createElement(
    _JSXStyle,
    {
      id: '3946479178'
    },
    [
      '.jsx-3946479178,.' +
        stylesHtml.className +
        ' blockquote{margin-top:1.5em;margin-bottom:1.5em;margin-left:1.5em;max-width:22em;line-height:var(--daui-line-height);font-style:italic;font-weight:300;text-rendering:optimizeLegibility;}',
      '.jsx-3946479178:first-child,.' +
        stylesHtml.className +
        ':first-child{margin-top:0;}',
      '.jsx-3946479178:last-child,.' +
        stylesHtml.className +
        ':last-child{margin-bottom:0;}'
    ]
  ),
  className: 'jsx-3946479178'
}

The only difference appears to be the rules are one big string in dev, and an array of a string for each rule for prod.
Could styled-jsx be enhanced to automatically .join() an array of rules encountered in development? That way component libraries could be published only transpilled for production.

Could styled-jsx be enhanced to automatically .join() an array of rules encountered in development?

@jaydenseric it is not possible to mix up optimizedForSpeed styles with non optimizedForSpeed.  With optimizedForSpeed we insert to a single style tag using the CSSOM API, when it is disabled we create multiple style tags.
We can add a warning here though
https://github.com/zeit/styled-jsx/blob/343795d3b2fef6d3d085921b0d4cf31b4d9be709/src/stylesheet-registry.js#L34-L39
like if this._optimizeForSpeed is true but props.children (the styles) are not an array print a warning

Perhaps you misunderstood the suggestion...
It would be impossible to support importing non optimizedForSpeed styles for use in a production build, because it is hard to cut up a style string into an array of rules at runtime - that should always happen build time.
But, I can't see why the other way around can't be supported. You should be able to import optimizedForSpeed styles for use in a development build, because it would be trivial at runtime to detect the styles are an array of rules, and simply array .join() them together into a single string, just like how non optimizedForSpeed styles are normally.
With this behavior, to write a component/styles library all you need to do is publish a single optimizedForSpeed build. This would make it much easier to publish styled-jsx styles and a native ESM build would be possible.

@jaydenseric that is a bit "magic" but it might work, want to put together a PR?

@giuseppeg I got started on a PR a few weeks ago but had to move onto other stuff. Sorry, but I'm now out of time and money to work on OSS, see https://twitter.com/jaydenseric/status/1206800095280128006. I've been working on OSS pro bono full time since August; time to get back into paid work.

@jaydenseric no worries! I get that as I am not paid to maintain any of these OSS projects as well :)


feat: abort collaborative transaction

Closes #269.
Adds the ability to abort a running taker operation.

How to test

Send a collaborative transaction
Try to abort the transaction
Verify that the transaction was aborted and not broadcasted

You might want to watch the logs while testing this feature with npm run regtest:logs:jmwalletd
Question: Currently, the text only says "Are you sure you want to abort the collaborative transaction?"..
Stopping a taker operation might lead to end up wasting a "retry" (when commitment was already sourced).. Should it mention that it might lead to inconveniences if this is used "too much in a row"? How to communicate this best?

Question: Currently, the text only says "Are you sure you want to abort the collaborative transaction?"..
Stopping a taker operation might lead to end up wasting a "retry" (when commitment was already sourced).. Should it mention that it might lead to inconveniences if this is used "too much in a row"? How to communicate this best?

Let's leave it as is and see how many users run into these kinds of issues in the wild.

Notes to self:

"Abort" sounds quite drastic, maybe reword to "Cancel"?
"Collaborative transaction started" toast seems unnecessary now; remove?


Combobox SelectedItem returns null after selecting the same item twice

Describe the bug
In a Combobox, while having its Items property bound to a collection and also having its SelectedItem bound to a string property, selecting the same item twice will update the SelectedItem property's value to null.
To Reproduce
The XAML:
<ComboBox Items="{Binding Drives}" SelectedItem="{Binding SelectedItem, Mode=TwoWay}" SelectedIndex="0">
    <ComboBox.ItemTemplate>
        <DataTemplate>
            <TextBlock>
                <TextBlock.Text>
                    <MultiBinding StringFormat="{}{0}{1}">
                        <Binding Path="Name" FallbackValue="C:\"/>
                        <Binding Path="VolumeLabel" StringFormat="  | {0}" FallbackValue=""/>
                    </MultiBinding>
                </TextBlock.Text>
            </TextBlock>
        </DataTemplate>
    </ComboBox.ItemTemplate>
</ComboBox>

The ViewModel code:
public static List<DriveInfo> Drives => new(DriveInfo.GetDrives());

private string selectedItem;
public string SelectedItem
{
    get => selectedItem;
    set
    {
        if (value != null)
            selectedItem = value;
        else
            System.Console.WriteLine("It was null again.");
     }
}

Expected behavior
value in this case should be the item selected, not null.
Desktop:

OS: Windows 11
Version 0.10.18

I don't get your usecase. If Items is a list of DriveInfo, how should selected item be string? It should be DriveInfo as well I guess.

Well that does seem to work out better. Also gets rid of all the automatic string conversions.


Add “...” in front of highlight if In middle of sentence

@abhayprasanna requested:
Advanced feature - if I were to begin a highlight in the middle of a sentence, so the first letter would be lowercase, it would be cool if it could add [...] or something like that to the beginning of the block to indicate it's an extract of a sentence.

@abhayprasanna how did I do? :-) Is this what you envisioned? I need to clean up the format of the dropdown and how it fits in the window but wanted to confirm this is what you wanted before I push an update!

Looks like some issues with the ... before and after following  links. See this website as example: https://complexity.simplecast.com/episodes/30 and Issue #44 for more details on it and test scenarios.


FCFS algorithm in C

Created first_come_first_serve_algorithm.c file in operating_system folder of C.
updated the C/README.md file for the ame

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
FCFS is a very famous scheduling algorithm that is used in operating systems and I have written the C code for the same
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
#6790

@HarshCasper @ankitaggarwal23 @bhav09 @gat786 @anushkrishnav can anyone please review this pr

Sorry @ankitaggarwal23 I didn't know that we have to link issue to the PR. This is for issue #6790 . So should I make a new issue?

@ankitaggarwal23  could you please review the changes

I'll do the changes

@ankitaggarwal23 can you please review the changes


Use separate pipeline for extraction of features from training samples.

Currently, when in "feature view" (i.e. displaying live data and training samples as features), the features are dynamically calculated from the training samples when navigating between training samples. This uses the live pipeline, which creates glitches in the plot of the features of the live data. We should probably clone the pipeline and use the clone from calculating features of the training samples (and reset it before use).

@nebgnahz this was fixed by https://github.com/damellis/ESP/commit/d99dbf766a4c5bbb04fc6c67b9b2d0e403be03b7 right?

I don't think so. It was planned but not fixed.

Sorry, my mistake! I was confusing this with https://github.com/damellis/ESP/issues/337


Dependency Versioning

Currently Vanna does not require specific version(s) of Python modules, opening up to potential dep mismatch over time – would love to add that in pyproject.toml, are there specific versions the development team tests against?

original need python > 3.9, because some type check need python>3.9. For example, list is not allowed for python 3.8, it must be List.

makes sense for Python version, my point is for other dependencies, i.e. if snowflake driver has a breaking change, or chromadb changes, or OpenAI, Vanna may not be able to integrate anymore, usually packages pin other dependencies to avoid this issue, see Vanna reps vs FastAPI deps below for ref:
usually this tends to manifest as hard to track "it works on my system" issues b/c while you may have already the dependencies, the next install on a different machine may even pull a completely different major version with incompatible method signatures; this is a big concern with respect to production use for me
Vanna:
[project.optional-dependencies]
postgres = ["psycopg2-binary", "db-dtypes"]
bigquery = ["google-cloud-bigquery"]
snowflake = ["snowflake-connector-python"]
duckdb = ["duckdb"]
all = ["psycopg2-binary", "db-dtypes", "google-cloud-bigquery", "snowflake-connector-python","duckdb"]
test = ["tox"]
chromadb = ["chromadb"]
openai = ["openai"]
mistralai = ["mistralai"]
gemini = ["google-generativeai"]
marqo = ["marqo"]

FastAPI
[project.optional-dependencies]
all = [
    "httpx >=0.23.0",
    "jinja2 >=2.11.2",
    "python-multipart >=0.0.7",
    "itsdangerous >=1.1.0",
    "pyyaml >=5.3.1",
    "ujson >=4.0.1,!=4.0.2,!=4.1.0,!=4.2.0,!=4.3.0,!=5.0.0,!=5.1.0",
    "orjson >=3.2.1",
    "email_validator >=2.0.0",
    "uvicorn[standard] >=0.12.0",
    "pydantic-settings >=2.0.0",
    "pydantic-extra-types >=2.0.0",
]

@roshanr10 yeah, one thing we have to pin down is the actual version dependencies. One thing we don't want to do is force upgrades when they aren't necessary. The only true hard version dependency that we have is openai >= 1.0.0
Since we aren't pinning on our end, what you could do is pip freeze the versions of these packages that you currently have installed and add them as requirements directly in your requirements.txt for now


Improvement: AR failed tooltip

At the moment if AR fails the fullscreen window is displayed but that window is not intuitive, so users really don't know what is happening.
A tooltip with a text like 'AR is not available on this device, please make sure you have installed ARCore 1.9 or later' could help.
The same happens with the AR button, but I saw you already have a ticket for that.

ar-status takes care of this now.


Clarify, or link to, info about filenames

The examples have different filenames (pr.yml, pr-push.yml) - presumedly, they have special meaning/side-effects.
It be great if the doc explained when to use which or pointed so some resource that explained those (or just chose the safest default one).

So, turns out the filename is meaningless, right? We searched for uses of "spec-prod" action and found they use a bunch of different filenames.
I think "pr-push.yml" was the most popular. I'm still personally worried that this will confuse people (like me! I was already confused as I thought the name meant something).
I can live with "pr-push.yml" as the default one we use everywhere... but maybe a generic "action.yml" would be better, given how it behaves is defined in the .yml itself, and not related to the filename.

Should add a link to GitHub workflow syntax for details (missed in https://github.com/w3c/spec-prod/pull/69).

We are going with auto-publish.yml everywhere, but we should at least mention that the name is meaningless but we use it (or will use it) at the W3C to know if a spec is auto-published or not.


Add 'originalSchedule' field to API

Since the "schedule" field changes with every run of a job, would it be possible to add an originalSchedule field, which could be used to see what the job schedule looked like when it was first entered into Chronos?

:+1:


docs(website): improve performance

Codecov Report

Merging #35 (9c0509d) into main (075292d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #35   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines          230       230
  Branches        11        11
=========================================
  Hits           230       230

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 075292d...9c0509d. Read the comment docs.

:tada: This PR is included in version 1.1.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[EN DateTimeV2] Date dot separator not recognized

If I enter "13/05/2018" or "13-05-2018", LUIS recognized it successfully as a DateTimeV2.
But if I enter "13.05.2018", then only "2018" is used :
Utterance   13.05.2018
Entities   datetimeV2 --> "2018"
I am working essentially with this date format : "DD.MM.YYYY". It would be great to support also english regional formats, not only "en-us".

Hi @SvartAlfe,
dot-syntax dates like "13.05.2018" are already supported. You can test it with the SimpleConsole project in the repository.
The latest recognizers release is in the process of being deployed to LUIS.
Regarding regional formats in English, multiple formats are supported, but in ambiguous cases, the default fallback is MDY. We plan to enable setting the default fallback, but there is no ETA yet. This is tracked in issue #212.

I tested it in a very simple application, please see the results below. The "." and "-" are not managed the same way. (I am working on eu.luis.ai)


Null reference on new listing

Hey, firstly great site!
I've been playing around with it, and keep hitting a null reference error when creating new listings. I tried putting a simple "!= null statement" around the problem in question, that gets rid of the problem but I get no attached files now.
Screenshot below, if it doesn't show the details enough the error is on the line "foreach (HttpPostedFileBase file in files)" in ListingController

Thanks for reporting. I just made a fix, seems it could be related to the multiple attribute. Could you try get the latest version? And please let me know which browser do you use if it's still not working? Thanks.

That's fixed the image upload problem. I'm still getting the above error on creating a new listing in IE11. I'm NOT getting the error in Firefox 38.0.5. Debugging the two, IEnumerable files is passed in as null in IE11 but as an array with 1 member in Firefox?

i get the error in IE11 too, I updated the fileupload js component and I tested it fixed the issues.

Yep looks good, thanks :)


v3client: wrap watch ctxs with blank ctx

Printing the values in ctx.String() will data race if the value
is mutable and doesn't implement String(), which seems to be common.
Instead, just return a fixed string instead of computing it; v3client
watches don't need as much flexibility for creating separate strings,
so separate ctx strings probably aren't necessary at this point.
Fixes #7811

lgtm


Language localisation

This is not a feature request but rather a question related to language localisation in the app.
I want to add support for my language (Marathi) in this app. I scanned through all the documentation and the forum but couldn't find anything written about the language localisation for this app. I understand that the Arduino IDE has a translation project on transifex and I am looking for something similar for the science journal app on iOS (and on Android too assuming the text strings are the same.)
Can you please help me with this? How can I add a language to the app?
To give an example, my aim is to have the texts shown in the below snaps in Marathi language.

hello!
Can anyone please tell me where the localisation project/files is/are located?

hello!
Can anyone please tell me where the localisation project/files is/are located?


Error Halt add-on with exit code 1

Hello to you all.
I do face following issue:
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
cont-init: info: running /etc/cont-init.d/01-rtsp-to-web
[17:02:01] INFO: Creating configuration
cont-init: info: /etc/cont-init.d/01-rtsp-to-web exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
services-up: info: copying legacy longrun rtsp-to-web (no readiness notification)
s6-rc: info: service legacy-services successfully started
[17:02:04] INFO: Starting discovery service
[17:02:04] INFO: Starting RSTPtoWeb
[16:02:05] WARNING: Halt add-on with exit code 1
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Addon installation is done on Raspberry Pi 3 HAOS
Home Assistant 2022.12.8
Supervisor 2022.12.1
Operating System 9.4
Frontend 20221213.1 - latest

Here some system logs
Logger: homeassistant.components.hassio
Source: components/hassio/websocket_api.py:124
Integration: Home Assistant Supervisor ([documentation](https://www.home-assistant.io/integrations/hassio), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+hassio%22))
First occurred: December 29, 2022 at 12:40:34 PM (6 occurrences)
Last logged: 5:09:38 PM

Failed to to call /addons/3d360630_rtsp-to-webrtc/install - Can't install ghcr.io/allenporter/armv7-rtsp-to-webrtc:1.0.1: 500 Server Error for http+docker://localhost/v1.41/images/create?tag=1.0.1&fromImage=ghcr.io%2Fallenporter%2Farmv7-rtsp-to-webrtc&platform=linux%2Farm%2Fv7: Internal Server Error ("Head "https://ghcr.io/v2/allenporter/armv7-rtsp-to-webrtc/manifests/1.0.1": dial tcp <IP_ADDRESS>:443: i/o timeout")
Failed to to call /addons/3d360630_rtsp-to-webrtc/stats - Container addon_3d360630_rtsp-to-webrtc is not running
Failed to to call /addons/3d360630_rtsp-to-web/stats - Container addon_3d360630_rtsp-to-web is not running

Even restarting whole host device doesn't help, any idea?

just a loose thought came and I figure out something....
well, after port change from default one from 8083 to 1234 it workout!
What does it mean for me? Some diffrent addon/service is using 8083?

This is discussed in #61 though i'm unable to reproduce and am unsure about the root cause or how it's even possible.

Marking as duplicate of  #61 -- welcome your participation there if you learn anything.


What happens to Cells locked with Deposit Lock with less than 400 CKB?

What happens to Cells locked with Deposit Lock with less than 400 CKB?
eg. someone sends 200 CKB to Godwoken Deposit Address. Godwoken isn't going to collect this deposit. I guess the funds are stuck between Layer 1 and Layer 2. Can user unlock his funds?

In Yokai bridge UI, user can cancel their pending deposits by sending a ckb tx.

owner_lock_hash and cancel_timeout in DepositLockArgs provide a way to redeem the token in the case Godwoken ignores this request(e.g., when the network becomes bloated). cancel_timeout specifies a timeout parameter in CKB's since format. When the timeout has reached, the user can create another request to cancel the deposit action, and redeem his/her tokens for other use. owner_lock_hash is used to provide the token owner's identity in case of a cancel deposit action.

See: https://github.com/nervosnetwork/godwoken/blob/master/docs/life_of_a_godwoken_transaction.md#deposit


OTAM Results

Hi, I am trying to reimplement the OTAM results but getting ~2% difference in results(with my own implementation). Are you able to replicate the results, if yes can you share the training arguments like Lr, Lr Decay Step, momentum, dampening, etc?

The otam example script contains arguments which get very similar results to those claimed in the otam paper (possibly even slightly better if I remember correctly), but this isn't an exact reimplementation. I don't use the differentiable min operation for the first row (L250 in model.py), and more training iterations are needed.

In fact, that's what the setting is in scripts/examples.md. So just try running that command.


Update README.md

@MadisonJMyers - changes per your request.
FYI - I assume that Big Data should be capitalized here. @bomeng - can you confirm?

Big Data can be either way, I aw them used capitalized or not frequently.


Add documentation on how to run the app using harbur/captain image

It could be a simple entry on README.md.

The image harbur/captain is not updated regularly and currently not considered official release channel.
At the moment it's not planned to add it as official channel.

@spiddy
do you need help with that?
I will be more than happy to keep the image up-to-date.

If you want to prepare a PR, it will be welcomed.
ᐧ
__
Dimitris Kapanidis
Founder
T/ +34 6780 49 780
Harbur Cloud Solutions SL. - Modernizing Software Delivery
Travessera de Gracia 294 Entlo. 1, 08025, Barcelona, Spain
https://www.harbur.io
NOTICE:  This e-mail and any attachments are confidential and intended
solely for the attention and use of the named addressee(s). If you are not
the intended recipient, be aware that any disclosure, copying, distribution
or use of this e-mail or any part of it, without the prior written consent
of Harbur Cloud Solutions SL. is prohibited and may be unlawful. If you
have received this e-mail in error, please notify immediately by returning
it to the sender and delete this copy from your system.
On Sat, Apr 22, 2017 at 4:24 PM, Mario Luan<EMAIL_ADDRESS>wrote:

@spiddy https://github.com/spiddy
do you need help with that?
I will be more than happy to keep the image up-to-date.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/harbur/captain/issues/71#issuecomment-296376613, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWbJU-pRuIu8QwEPwlbQf834oY4Jiqmks5ryg2QgaJpZM4NAlbg
.


New framework fails - can't update, can't run start

Ionic version:  (check one with "x")
[ ] 1.x
[x ] 2.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Updated framework won't even run basic serve on a simple skeleton.
Have uninstalled and installed ionic
ionic -v
2.2.1
`PS D:\libraries\Programming\ionic\newImageTest> ionic serve
Uh oh! Looks like you're missing a module in your gulpfile:
Cannot find module 'bower'
Do you need to run npm install?
PS D:\libraries\Programming\ionic\newImageTest> ionic -v
2.2.1
PS D:\libraries\Programming\ionic\newImageTest>`
Expected behavior:

Steps to reproduce:

Related code:
insert any relevant code here

Other information:
Doesn't seem to have formed an src folder.
package.json doesn't seem to have created correctly - doesn't have the angular modules or the ionic modules included

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Your system information:

 ordova CLI: 6.5.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.1
Xcode version: Not installed

What commands are you running to start a new project?

ionic start newImageTest --v2

I only ran this as the new CLI has broken my app. I've tried everything including redownloading the working version from bitbucket but I get errors in the build process (just with the fonts being copied)
[16:13:56]  copy: Error copying "D:\libraries\Programming\ionic\webapp\node_modules\ionicons\dist\fonts" to             "D:\libraries\Programming\ionic\webapp\www\assets\fonts" [16:13:56]  copy: Error copying "D:\libraries\Programming\ionic\webapp\node_modules\ionic-angular\fonts" to             "D:\libraries\Programming\ionic\webapp\www\assets\fonts"
I get issues with angularfire2 which I solved by reinstalling angularfire2 and firebase
but then I don't get a server
This all the console shows:#
livereload.js Failed to load resource: net::ERR_ADDRESS_INVALID 2main.js Failed to load resource: the server responded with a status of 404 (Not Found)
Which is why I tried with a starter app to see if it was the new version of the framework.
Thanks
Shane

Can you try executing ionic info from within the project directory?

It's included above but here it is again:
`PS D:\libraries\Programming\ionic\newImageTest> ionic info
Your system information:
ordova CLI: 6.5.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.1
Xcode version: Not installed`
I also tried the tutorial app which does work (with one non-fatal error in the build step)- a watch error after the build dev finished

I've compared the @ionic folder of the node modules in the new tutorial app (working) and my existing app (replaced package.json, deleted node_modules, npm cache clean, npm install) and it's very different. In the new app @ionic contains app-scripts, this doesn't get put in with npm install. The storage folder, subfolders in the new app contain more files than in the rebuilt old app which is giving me an error
Module '"D:/libraries/Programming/ionic/ENTonlyApp/webapp/node_modules/@ionic/storage/es2015/index"' has no exported member 'IonicStorageModule'.
Also noticed that we now need a MenuController for the sidemenu component (which I use) which wasn't needed previously


[move-compiler] allow the inlining origin to be embedded in spec marker

This commit extends UnannotatedExp_::Spec as following:
- Spec(SpecId, BTreeMap<Var, Type>),
+ Spec(SpecId, Option<SpecIdent>, BTreeMap<Var, Type>),

Where SpecIdent marks the origin of the spec block (before inlining)
pub struct SpecIdent {
    pub module: Option<ModuleIdent_>,  // None means a script
    pub function: Symbol,
    pub id: SpecId,
}

This commit also re-assigns SpecId for each UnannotatedExp_::Spec inlined to maintain that each inline spec block has a unique ID. This is needed later in the compilation stage.
This behavior is tested with a spec_inlining.move test case added in the move_check folder. Previously, this test case hits an assertion in the compiler.

Motivation
More on spec inlining
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

CI
New test case

Closing as the changes are enclosed in #879


My csgo suddenly quit

When I am in use, I press F8, csgo gets stuck, and then suddenly exits.

me to this is the new update