ShellExecuteEx failed: File not found

Hey there,
thank you for providing this neat soloution!
I have run into a problem I can't figure out. The container is installed, image downloaded and installed successfully, /config binded locally. I've put a test savefile into /config/savefiles as "savefile.sav" and looked at the logs. It seems that something with the savefile is wrong, or my binding is not as local as I hoped it would be:
Checking available space...
Downloading the latest version of the game...
Redirecting stderr to '/root/.steam/logs/stderr.txt'
[  0%] Checking for available updates...
[----] Verifying installation...
Steam Console Client (c) Valve Corporation
-- type 'quit' to exit --
Loading Steam API...OK.
@sSteamCmdForcePlatformType windows
"@sSteamCmdForcePlatformType" = "windows"
Logging in user '${steam_user}' to Steam Public ...
Logged in OK
Waiting for user info...OK
force_install_dir /config/gamefiles
app_update "526870"
Success! App '526870' already up to date.
quit
 * Starting periodic command scheduler cron
   ...done.
Application could not be started, or no application associated with the specified file.
ShellExecuteEx failed: File not found.

As you can see, the savefile seems to be recognized (as there is no error message for a missing savefile, i.e: "Save file cannot be found. You need to generate a new world on your client, and then put it into /config/savefiles/savefile.sav") but not in the right format? Or somtehing? I saved the game as "savefile" in the game itself. I tried it with a different name like "Test_save_1" and renamed it to "savefile" but no dice.
Any idea whats happening here?
Cheers!
EDIT:
I am building the container via docker-compose, if that helps.

Errors out with "ERROR! Failed to install app '526870' (Disk write failure)". I'm working as root, trying to install into a SMB share because the host doesn't have enough disk space available. But I got it to the point described above so I will try to find a solution to this as I know it works. Somehow.
EDIT 3:
After not being ablte to recreate a succesfull installtion on my machine I tried it on a few different machines as well.
A Windows host with docker: same error as in edit 2.
A Ubuntu WSL2 instance: same error as in edit 2.
On the host machine of my Ubuntu system locally, not an SMB share: same error as in edit 2.
I'm dumbfounded. I know at least the installation worked. But I can't get it running again. Really appreciate your input on this.

Hey! Really sorry for not getting back to you sooner; I've just moved house, so everything is a little hectic!
I'm really sorry to hear that it's not quite working out for you right now, despite different configurations. The disk write failure error seems to be permissions-related.
I'm a little unsure as to why you would encounter issues with any of those setups. I'm going to try and reproduce the issue on my end.

Hey, thank you for the response! Hope you can settle down in a little bit.
Don't worry about taking time to answer, this is nothing critical. It's a game after all ;)
I guessed as much that it's a permission problem and confirmed that thorugh some googling. Still no idea how that happens. I'll have a friend of mine take a look in the next few days, maybe he will point out some glaring mistake :D
If you need anything else from my side, let me know.
Cheers.

Just to give you a quick update:
I got the container installed again. But honestly I don't know what I did differnt.
I tried this, without any success. After that failed, I changed the volume back to what it was before (basically what you prepared in the docer-compose.yml) and the installation worked.
Now I am stuck at the error for which I opended this issue in the first place, but I am a step futher, I guess ;)
Cheers.

Another quick update:
I tried what krmedia suggested in the here to no avail.
What kinda permissions does the savefile need for the server to be used? I'm using a mounted Hetzner storagebox for the server data and am fiddeling around with the permissions on the savefile on the directories. Maybe I am missing something here.

So.
I tried a bit more. Still no working server in sight. This is the latest error message I got:
Checking available space...
Downloading the latest version of the game...
Redirecting stderr to '/root/.steam/logs/stderr.txt'
[  0%] Checking for available updates...
[----] Downloading update (48 of 53,901 KB)...
[  0%] Downloading update (3,518 of 53,901 KB)...
[  6%] Downloading update (6,835 of 53,901 KB)...
[ 12%] Downloading update (9,560 of 53,901 KB)...
[ 17%] Downloading update (12,036 of 53,901 KB)...
[ 22%] Downloading update (14,496 of 53,901 KB)...
[ 26%] Downloading update (16,999 of 53,901 KB)...
[ 31%] Downloading update (19,477 of 53,901 KB)...
[ 36%] Downloading update (21,854 of 53,901 KB)...
[ 40%] Downloading update (24,325 of 53,901 KB)...
[ 45%] Downloading update (26,794 of 53,901 KB)...
[ 49%] Downloading update (29,257 of 53,901 KB)...
[ 54%] Downloading update (31,728 of 53,901 KB)...
[ 58%] Downloading update (34,211 of 53,901 KB)...
[ 63%] Downloading update (36,454 of 53,901 KB)...
[ 67%] Downloading update (37,681 of 53,901 KB)...
[ 69%] Downloading update (38,925 of 53,901 KB)...
[ 72%] Downloading update (40,181 of 53,901 KB)...
[ 74%] Downloading update (41,414 of 53,901 KB)...
[ 76%] Downloading update (42,659 of 53,901 KB)...
[ 79%] Downloading update (44,027 of 53,901 KB)...
[ 81%] Downloading update (45,236 of 53,901 KB)...
[ 83%] Downloading update (46,465 of 53,901 KB)...
[ 86%] Downloading update (47,700 of 53,901 KB)...
[ 88%] Downloading update (48,945 of 53,901 KB)...
[ 90%] Downloading update (50,173 of 53,901 KB)...
[ 93%] Downloading update (51,409 of 53,901 KB)...
[ 95%] Downloading update (52,648 of 53,901 KB)...
[ 97%] Downloading update (53,869 of 53,901 KB)...
[ 99%] Downloading update (53,901 of 53,901 KB)...
[100%] Download complete.
[----] Installing update...
[----] Extracting package...
[----] Extracting package...
[----] Extracting package...
[----] Extracting package...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Cleaning up...
[----] Update complete, launching Steamcmd...
Redirecting stderr to '/root/.steam/logs/stderr.txt'
[  0%] Checking for available updates...
[----] Verifying installation...
Steam Console Client (c) Valve Corporation
-- type 'quit' to exit --
Loading Steam API...OK.
@sSteamCmdForcePlatformType windows
"@sSteamCmdForcePlatformType" = "windows"
Logging in user '${STEAM_USER}' to Steam Public ...
Logged in OK
Waiting for user info...OK
force_install_dir /config/gamefiles
app_update "526870"
Success! App '526870' already up to date.
quit
 * Starting periodic command scheduler cron
   ...done.
0012:err:ole:marshal_object couldn't get IPSFactory buffer for interface {00000131-0000-0000-c000-000000000046}
0012:err:ole:marshal_object couldn't get IPSFactory buffer for interface {6d5140c1-7436-11ce-8034-00aa006009fa}
0012:err:ole:StdMarshalImpl_MarshalInterface Failed to create ifstub, hres=0x80004002
0012:err:ole:CoMarshalInterface Failed to marshal the interface {6d5140c1-7436-11ce-8034-00aa006009fa}, 80004002
0012:err:ole:get_local_server_stream Failed: 80004002
000b:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
000b:err:winediag:nodrv_CreateWindow Make sure that your X server is running and that $DISPLAY is set correctly.
0014:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
0014:err:winediag:nodrv_CreateWindow Make sure that your X server is running and that $DISPLAY is set correctly.
0014:err:ole:apartment_createwindowifneeded CreateWindow failed with error 0
0014:err:ole:apartment_createwindowifneeded CreateWindow failed with error 0
0014:err:ole:marshal_object couldn't get IPSFactory buffer for interface {00000131-0000-0000-c000-000000000046}
0014:err:ole:apartment_createwindowifneeded CreateWindow failed with error 14007
0014:err:ole:StdMarshalImpl_MarshalInterface Failed to create ifstub, hres=0x800736b7
0014:err:ole:CoMarshalInterface Failed to marshal the interface {6d5140c1-7436-11ce-8034-00aa006009fa}, 800736b7
0014:err:ole:get_local_server_stream Failed: 800736b7
0010:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
0010:err:winediag:nodrv_CreateWindow Make sure that your X server is running and that $DISPLAY is set correctly.
Could not load wine-gecko. HTML rendering will be disabled.
0028:err:winediag:nodrv_CreateWindow Application tried to create a window, but no driver could be loaded.
0028:err:winediag:nodrv_CreateWindow Make sure that your X server is running and that $DISPLAY is set correctly.
Could not load wine-gecko. HTML rendering will be disabled.
wine: configuration in '/home/satisfactory/.wine' has been updated.
Application could not be started, or no application associated with the specified file.
ShellExecuteEx failed: File not found.

As you can see, the initial problem persists. I can't get the .sav to be recognized (it now sits with 0777 perms in the /config/savefiles dir). But now there seem to be some more problems? I don't know what else to try or where to look at,
If you find the time I'd greatly appreciate your help :)
Cheers.

i´m getting the same error on my Synology DS918+ NAS
Application could not be started, or no application associated with the specified file.

ShellExecuteEx failed: File not found.

@BeardedIntern I ran into a similar error regarding the savefile location. Turns out it´s not working if you place your config folder into the same folder where you extracted the zip. I had to move it it to a similar folder and it is working now.
I was logged in as root for all operations.
My Docker compose yml:
version: '3'
services:
    satisfactory-server:
        container_name: 'satisfactory-server'
        hostname: 'satisfactory-server'
        image: 'wolveix/satisfactory-server:latest'
        environment:
            - STEAMUSER=********
            - STEAMPWD=********
            - STEAMCODE=*******
            - MAXBACKUPS=10
        ports:
            - '7777:7777/udp'
        volumes:
            - '/opt/sf-config/config:/config'
        restart: unless-stopped

Hey @quoten3m0, thank you for the info.
But I am not sure I can follow you. What.zip do you mean? The config dir lives in /media/externalstorage/satisfactory/config. I did not extract anything there or elsewhere for that matter.
My yml looks about the same too.

@BeardedIntern Sorry for getting back to this so late. Were you able to get this working? Or were you still encountering issues?

Sorry for answering so late :D
I wasn't able to get it working and I honestly let it sit for the last few weeks. I didn't poke at it and was just waiting for a response here. I tried it just now and am running in the same issues still.

No worries! Is that with the latest Docker image? If not, please try pulling the latest and see whether you run into the same issue

Yes, it's the latest image. I rebuild the container from scratch so the game itself is up to date too.

Nice. Would you mind posting your host specs that you're trying to host the server from?

I'm running on Ubuntu 20.04.2 LTS 5.4.0-74-generic, 4GB of RAM and 2 CPU cores. The storage is on a separate box which is accessible and gets written to when I update / initialize the container. Had no problems here with permissions so far.
Do you need anything else?

Could you try unmapping the /config/gamefiles directory? So just map the savefiles directory

I can't easily unmap the gamefiles from the storagebox to local. With only 40 gigs of storage locally I am quite limited. I will try to make it fit, though I am pretty sure the problem occured on a local storage as well.

Totally understood, just particulary interested given the resolution in #19

After checking the other thread and confirming my hunch that I don't have enough space locally on the server I want to run the game server on I set it up on my local windows machine with docker.
It works.
So, there might actually be some permission issues with the mounted storage.

Thanks for reporting your findings @BeardedIntern. Would you mind providing any additional info regarding your specific storage configuration?

I am using a Hetzner server and storagebox, if that helps.
The storages directories are set up with 770 the files with 777, mounted as root via SMB and is running Ubuntu, I think.
Maybe we can talk this through using a different communication channel so this thread does not get clogged up with our back and forth of determining what's what?

I'm also using a Hetzner server, but not a storagebox as SMB can be a pain in certain scenarios, in my experience. Sure thing, You can reach me as Wolveix#2211 on Discord :)

Identified this as a permissions-related problem. Working on addressing this in an update soon.


Support for Fedora

This change adds installation automation for Fedora.

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

Thanks for this, if you can hit my CLA button I'll go ahead and merge this in.
Much appreciated!

Ah, nevermind I can't read... thanks!


runtime(doc): some improvements to getregion() docs

Mention the default selection behavior
Remove useless sentence
Correct description about space padding

thanks!


remove redundant reference to object instance when calling ext. function

Hey, loving your work on this lib. This is just a minor tweak which I consider to be more idiomatic Kotlin: passing the object instance as a parameter to the function is a bit util-function like, which is helpful when calling it from Java, but from Kotlin we can embrace the fact that it's an extension function to the full extent, and treat it as if it were a member function, hence we can omit this.
However, as this is just a style issue open to interpretation I understand if you'd rather keep it as it is. Hope this helps.

:+1:


Updating vSphere Terraform Build to Support Kubernetes Worker Addition

This is a simple set of fixes to the additional Kubernetes worker node additions to the terraform build.  Cleaned up the additional edge node blocks, and fixed the ip address notation to the terraform vsphere provider formatting.

Thanks @hpreston!


fix(bridge-ui): changed faucet address to correct

Popup message had a broken link, changed to correct one

There is an existing PR open with this change already. Thank you for the contribution though!
https://github.com/taikoxyz/taiko-mono/pull/13098


Importing pyccl fails

Dear all,
I've just pulled the latest changes to CCL and redid the installation of both the C library and the python wrapper. Both installations work. When I then run
import pyccl as ccl
I get
`---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 import pyccl as ccl
2 import numpy as np
3 from astropy.io import fits
4 import astropy.wcs
5 from astropy.wcs import WCS
/Users/Andrina/Documents/WORK/ACTxHSC/code/CCL/pyccl/init.py in ()
39
40 # Cl's and tracers
---> 41 from .cls import angular_cl, NumberCountsTracer, WeakLensingTracer, CMBLensingTracer
42
43 from .lsst_specs import bias_clustering, sigmaz_clustering, \
/Users/Andrina/Documents/WORK/ACTxHSC/code/CCL/pyccl/cls.py in ()
5
6 function_types = {
----> 7     'dndz': lib.trf_nz,
8     'bias': lib.trf_bz,
9     'mag_bias': lib.trf_sz,
AttributeError: 'module' object has no attribute 'trf_nz'`
Would any of you know what is causing this?
Thank you very much for your help!
Cheers

Dear Elisa,
Thank you so much for your super quick reply. I think I figured out what the problem was. I was running python setup.py develop and somehow this does not inherit the latest changes to the C code. Running a normal installation fixed the issues.
Thank you very much for your help!


Problem denormalizing 2nd level children file entities. Issue with ConfigurableEntityNormalizer.

class ObjectA {
	protected $id;
	protected $name;
	/**
	 * @ORM\ManyToMany(targetEntity="AssetFile",indexBy="id")
         * @ORM\JoinTable(
         *     name="object_a_assets",
         *     joinColumns={
         * @ORM\JoinColumn(name="object_a_id",referencedColumnName="id",onDelete="CASCADE")
         *     },
         *     inverseJoinColumns={
         * @ORM\JoinColumn(name="asset_file_id",referencedColumnName="id",onDelete="CASCADE")
         *     }
         * )
         */
	protected $assetFiles;
}

The following class has relation to ORO\Bundle\AttachmentBundle\Entity\File created with the AttachmentExtension in a migration.
class AssetFile {

	protected $id;

	protected $description;
}

Denormalizing data for an object of type ObjectA with ConfigurableEntityNormalizer fails, when it comes to denormalization of the Oro\Bundle\AttachmentBundle\Entity\File.
It is done with FileNormalizer in:
/**
     * {@inheritdoc}
     */
    public function denormalize($data, $class, $format = null, array $context = [])
    {
        $result = null;
        $entity = $this->fileManager->createFileEntity($data);
        if ($entity) {
            $violations = $this->validator->validate(
                $entity->getFile(),
                $context['entityName'],
                $context['fieldName']
            );
            if (!$violations->count()) {
                $result = $entity;
            }
        }

        return $result;
    }

The $context['entityName'] in this case will be ObjectA and validation will fail.
The $context['entityName'] should have value AssetFile.
In the denormalize method of ConfigurableEntityNormalizer at line 80 $context is being updated as follows:
$context = array_merge($context, ['fieldName' => $fieldName]);

so $context['entityName'] is not being updated before denormalizing next $value
$value is instance of Symfony\Component\HttpFoundation\File\File
$entityClass  = Oro\Bundles\AttachmentBundle\Entity\File
$value = $this->serializer->denormalize($value, $entityClass, $format, $context);

@dimonixx , thank you for reporting the issue.
We've created an task in backlog (BAP-14978, I'm providing it here for internal usage) to fix this issue.
Thank you again for your contribution!
Best regards.


Unrecognized arguments: --continuation-token under az storage blob list

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

az storage blob list --container-name <name> --continuation-token <token>

Describe the bug

After I retrieve a continuation token from the first command that list storage blob, I can't use the token since it said ERROR: unrecognized arguments: --continuation-token when I want to get the rest of the result.
To Reproduce

Put --continuation-token after az storage blob list or az storage blob list --help to verify if az support this argument.
Expected behavior

I can get the rest results via --continuation-token as an arguments after az storage blob list similar to this one describe
Environment summary

# MacOS 13.2.1
➤ az --version
azure-cli                         2.46.0

core                              2.46.0
telemetry                          1.0.8

Extensions:
kusto                              0.5.0
aks-preview                      0.5.135
databricks                        0.10.0
interactive                        0.4.6
fzf                                1.0.2

Dependencies:
msal                              1.20.0
azure-mgmt-resource             21.1.0b1

Python location '/usr/local/Cellar/azure-cli/2.46.0/libexec/bin/python'
Extensions directory '/Users/morris_tai/.azure/cliextensions'

Python (Darwin) 3.10.10 (main, Feb 16 2023, 02:55:02) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]

Additional context

I will try powershell AZ module to see if the token works.

Thank you for opening this issue, we will look into it.

Hi @morristai, the parameter for continuation token is --marker. You can check all the parameter description through az storage blob list -h or https://learn.microsoft.com/en-us/cli/azure/storage/blob?view=azure-cli-latest#az-storage-blob-list-optional-parameters

Thanks @evelyn-ys for the quick response, my bad for missing that parameter.


Update apple notes

Resolves #877

To-Do:

[x] I think the link is in another language?

Thanks!


Update http4s-client, http4s-dsl, ... to 0.22.1

Updates

org.http4s:http4s-client
org.http4s:http4s-dsl
org.http4s:http4s-server

from 0.21.25 to 0.22.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

github:http4s/http4s/v0_22?sha=v0.22.0

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-minor, scalafix-migrations

Superseded by #72.


Suggestion: turn list into table

Cool project! :v:
I can see the list becoming very confusing the more people are added. How about turning it into a table (maybe even adding an avatar)?

Oh snap, i actually vibe with this idea. That's a good suggestion, thanks for the feedback! :D Now for the stream of PRs to slow down enough to convert it to a tabular format...

I think the ideal solution would be to also figure out a way to automate the generation of the README, because dealing with the merge conflicts and all that has been a pain.

Good idea!
I would go with a JSON like
[
  {
    "name": "Iheanyi Ekechukwu",
    "title": "Software Engineer",
    "job": "DigitalOcean",
    "website": "http://",
    "profiles": [{ "name": "Twitter", "link": "#" }, {}, {}]
  }
]

so contributors don't have to work with the table directly. Less merge conflicts and all that. Then just create a <table> and replace it in the README?
Sounds like a fun task :)

FWIW, I used to help maintain https://github.com/wbond/package_control_channel and it felt like merge conflicts were always present. Rather than the maintainers dealing with every conflict, we would sometimes ask the people contributing PRs if they could fix merge conflicts in their PR. Or submitting a new PR from the latest master if they didn't know how to fix the conflict. It does put the burden on the person contributing, so it's not without its trade-offs.
However, a HUGE mitigator of merge conflicts is ordering. Alphabetic or lexicographic ordering made it much less likely to have a conflict because less people were trying to update the same line. At some point the single json file was split into different files for each letter of the alphabet which also helped. That's something to keep in mind for the future.

@iheanyi @morkro Another take on this -
Use (issue/pull request template)[https://github.com/blog/2111-issue-and-pull-request-templates], you can have fixed questions.
Removes the problem of merge conflicts because people are not editing the same file (or any file!)
And the README can be updated via a post-merge hook.
What do you think?
(I'll pick up the implementation if you like)

@siddharthkp Yeah, that would be nice as well. 🤔 I'm here for it.


chore: bump next version to v0.15.0

Next release is going to be v0.15.0, with some exciting changes.

Should also update future_test.go and shepherd those tests into the main suite.

Hm, the daggerverse preview has failed @gerhard :thinking:
Error: response from query: input: daggerverse.deployPreviewWithDaggerMain resolve: call function "DeployPreviewWithDaggerMain": process "/runtime" did not complete successfully: exit code: 2

Stdout:
invoke: GET https://api.github.com/user: 401 Bad credentials []

It definitely shouldn't be triggering on PRs (creds aren't available) - but I'm not quite sure that it should trigger on .changes/.next even on main - I'm just bumping to the next version here. We want it to trigger on the PR right before the release.
We could trigger it on a new markdown file in .changes/? That would be the case we care more about.

Will fix https://github.com/dagger/dagger/pull/9073.

Failure is known failure from https://github.com/dagger/dagger/pull/9094, merging now to get release pieces started :tada:


More documentation annotations in xspec.rnc

For example add more ## documentation before each of the attribute definitions:
                         attribute pending { text }?,
                         attribute focus { text }?

as Oxygen XML Editor will show this documentation when you want to insert each attribute.

Thanks for your input, more documentation is always good!
Attributes pending and focus have exactly the same function as the elements with the same names as described in the documentation.
In xspec.rnc the behaviour of the attributes are documented inside scenario as follows:
scenario =
	## A scenario defines the environment in which a piece of processing takes place.
	##
	## Any parameters defined within a scenario equate to global (stylesheet)
	## parameters.
	##
	## If a scenario has a pending attribute, this has the same semantics as
	## wrapping the scenario in a <pending> element with a label equal to the value
	## of the pending attribute.
	##
	## If any scenario has a focus attribute, any scenarios without a focus
	## attribute will be classed as pending.
	element scenario { common-scenario-attributes, shared?,
		label,
		( matching-scenario |
		  function-scenario |
		  named-scenario ) }

common-scenario-attributes = common-attributes,
                             attribute pending { text }?,
                             attribute focus { text }?

Is it just a matter of repeating this information before common-scenario-attributes? Would that display fine in Oxygen?

The information needs to be set before each attribute definition. I will prepare a pull request.

There are a couple more attributes like "href", "select" which I do not know how to describe, my knowledge of XSpec is not that great. So they should be documented in the same way.

I added documentation for attributes href and select in https://github.com/xspec/xspec/pull/115#issuecomment-292457562.

This is now fixed in #115.


CLI 1.5.0 not installing Webpack? "Cannot find module 'webpack/lib/dependencies/ContextElementDependency'"

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

Angular CLI: 1.5.0
Node: 6.10.3
OS: win32 x64
Angular: 5.0.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, platform-server, router

@angular/cdk: 2.0.0-beta.8
@angular/cli: 1.5.0
@angular/material: 2.0.0-beta.8
@angular-devkit/build-optimizer: 0.0.32
@angular-devkit/core: 0.0.20
@angular-devkit/schematics: 0.0.35
@ngtools/json-schema: 1.1.0
@ngtools/webpack: 1.8.0
@schematics/angular: 0.1.0
typescript: 2.4.2
webpack: error

Repro steps.

Update dependencies of existing project from Angular 4/CLI 1.4 to Angular 5/CLI 1.5
run ng serve.

Also tried deleting node_modules/reinstalling but same error.
The log given by the failure.

Cannot find module 'webpack/lib/dependencies/ContextElementDependency'
Error: Cannot find module 'webpack/lib/dependencies/ContextElementDependency'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (C:\Users\greg.mcgee\Source\Workspaces\WBAC_US\Main\Develop\Angular\wbac-vpw\node_modules\@ngtools\webpack\src\plugin.js:8:34)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)

Desired functionality.

CLI starts build without error.

I had it when used ngc-webpack plugin. Also reinstall packages might help

same problem

Addendum: After installing Webpack manually, the project builds fine.

It seems as though you had a bad npm install. You should not have to install webpack manually. Glad all is working for you.

I removed node_modules and reinstalled again and am still having this issue. I had said in my first post that I had removed/reinstalled modules and it didn't work.

Deleted my package-lock.json file and removed/reinstalled node_modules and it worked. Not sure where the incompatibility was.

delete the package-lock.json did it! Thx


Add version lock styles to rb-canvas

TP: https://rockabox.tpondemand.com/entity/10994
Default state is "Locked Mode":

            Locked Mode

Canvas--edit modifier sets "Edit Mode":

            Edit Mode

No demo for the edit modifier?

I was going to leave it for the Angular implementation as this component is already in use, but you're right, I think I may hardcode a demo, to be fixed up during Angularification.

@chrisvxd I've updated the demo and added a tiny additional commit to prevent the label leaking out when no creative present.

@ohskylab "in review" is for when someone is actively reviewing it! We should probably document them somewhere.


[DOC] Getting started tutorial for Python

Change in the documentation
Create a getting started tutorial for Python. The JavaScript tutorial can be used as a reference. The Python flagd provider can be found here.

Once I can find some downtime I can take a stab at this, I have a few test scripts lying around already. Might not be able to tackle it too soon though since I'll be out most of May. Happy to also review other's work if they are quicker


Removed unused font to avoid loading the font

Issue Reference
This PR addresses the Issue : Fixes #383
Summarize
Loads the font from google fonts even though it is not used. This uses unnecessary bandwidth.

Refer #220 #222. Might need to duplicate.


remove deprecated given methods on AggregateFixtureGiven

with 4.1 some methods were deprecated:
w: axon-addons/axon-jgiven/src/main/kotlin/aggregate/AggregateFixtureGiven.kt: (70, 15): 'andThenTimeAdvancesTo(Instant!): ResultValidator<(raw) Any!>!' is deprecated. Deprecated in Java
w: axon-addons/axon-jgiven/src/main/kotlin/aggregate/AggregateFixtureGiven.kt: (72, 20): 'andThenTimeAdvancesTo(Instant!): ResultValidator<(raw) Any!>!' is deprecated. Deprecated in Java
w: axon-addons/axon-jgiven/src/main/kotlin/aggregate/AggregateFixtureGiven.kt: (77, 15): 'andThenTimeElapses(Duration!): ResultValidator<(raw) Any!>!' is deprecated. Deprecated in Java
w: axon-addons/axon-jgiven/src/main/kotlin/aggregate/AggregateFixtureGiven.kt: (79, 20): 'andThenTimeElapses(Duration!): ResultValidator<(raw) Any!>!' is deprecated. Deprecated in Java

so we should not offer them in the addon anymore

fixed with 4e00c036b41bd6ebd1d26126f8283e371b5eb355


Added static convenience methods for creating non-blocking enumerable…

…s from blocking enumerables and creating empty enumerables.
When satisfying new interfaces containing IAsyncEnumerable references with old blocking enumerable implementations, it's useful to have some basic convenience methods analogous to methods used in System.Threading.Task and System.Linq.Enumerable.  The code is pretty self-explanatory.
Let me know if there is a prefered way to implement these.

Thanks @silijon, I added your change AsyncEnumerable.Empty by myself and added other changes as well. Another helper method already exists: IEnumerableExtensions.ToAsyncEnumerable(). So I'm closing this branch,


Error Report: Warning message/alert shown after logging out of student SSO account

When students logs out of their SSO account, they are seeing the following message/alert:

Issue reported by Vivian
iOS 6.0 (5)

This is related to an issue with how the app is handling pending requests, and ties in to some issues users are reporting with the class card getting stuck on "Updated x hours ago". I'm working on this right now.

Commit cfbb732 should resolve this issue, as well as issues with auto-refreshing.

Verified in v6.0-rc, closing


Use keyof with namespaces

Search Terms

keyof
namespace
Suggestion

I would like to access all keys from a namespace. This was discussed in #16472 but not result there.
I have a list of types in a file with all database types.
Now I want to design a type safe resolver which has a current shape:
import * as Types from './types';

// Types has all the database types, e.g.
// Notification, User ...

export type Resolver<T> = {
  [R in keyof Types]: { // fail
    [P in keyof T]: (parent: T, args: any, ctx: Types.Context, info: Types.GraphQLResolveInfo) => any
  }
};

export const resolver: Resolver<Types.Notification> = {
  Notification: {
    text: (parent, _args, ctx, info) => {
      ...
    }
  }
};

Use Cases

Creation of type safe resolver for graphql.
Examples

Above
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

Did you mean R in keyof typeof Types?

I tried that before, but that returns just one string to me.
Note that almost all exports from types.ts are interfaces.

that it for you. keyof are the keys of a value that exist at runtime. Types (interfaces and type aliases) do not exist at runtime, so they are not "keys". only values have keys.

@mhegazy that's so sad to hear ... I know there was a discussion to have import type syntax, but I guess that line of development is dead.

Hi there,
@mhegazy there is not a way to add this feature to typescript?, that's very useful, in that way we can implement constrained behaviour based on interfaces re-exporting from a single file/module, normally I constraint my data with interfaces and not with classes.


does not work

 is valid HTML

 does render correctly though
Thanks for your project; thought about doing this years ago!

 is valid HTML

It is true that
 is valid HTML, however this is a JSX-based framework, (like React and many others) so all of the rules of JSX applies (like for example, all "tags" must be closed). Parsing HTML is very difficult in itself, and trying to map its concepts to other JSX-based frameworks is very difficult, while the inverse is actually straightforward.
Why JSX? (from Solid's documentation):

[...] JSX is a great DSL made for compilation. It has clear syntax, supports TypeScript, works with Babel, supports other tooling like Code Syntax Highlighting and Prettier. It was only pragmatic to use a tool that basically gives you that all for free. As a compiled solution it provides great DX. Why struggle with custom Syntax DSLs when you can use one so widely supported?

More from the JSX's Draft Spec:

JSX is an XML-like syntax extension to ECMAScript without any defined semantics. It's NOT intended to be implemented by engines or browsers. It's NOT a proposal to incorporate JSX into the ECMAScript spec itself. It's intended to be used by various preprocessors (transpilers) to transform these tokens into standard ECMAScript.

I hope this gives you an idea on why we use JSX (tooling, developer experience, widely supported, familar, made for transpilation, etc) instead of HTML.


Gauge doesn't appear to support template variables in data source

I tried using "{{my_var}}" as a data source. I get this error:
Expression {{my_var}}: Parse error at char 2: unexpected left brace '{'

This seems to work for Graphs, so I think this might be something that's only broken for Gauges.

@stuartnelson3


How do I create custom dataset class?

Could you make your issue more clear?

I want to train my own dataset (which are xml and jpg format, Pascal VOC Format) .The explanation on the Custom Dataset Tutorial is kinda complex and I do not get it how to replace my own dataset. If someone had already done this step, could you please explain me the easy way? I have tried also COCO dataset format , but it does not work out.  @FateScript

@ImSuMyatNoe
Just modify the following two lines accroding to your own setting.
reference code


using new hardware_reset api causing application to stuck

Required Info

Camera Model
T265 & D400

Firmware Version
<IP_ADDRESS>6

Operating System & Version
Linux (Ubuntu 18)

Kernel Version (Linux Only)
4.15.0-54-generic

Platform
PC

SDK Version
2.32.1

Language
C++

Segment
Robot

Issue Description
Hey all,
my system is using your new api extension including the hardware_reset method.
my system is multi cam system, same process, but different threads are handling both D435/415 and T265.
The system needs to work with more then one of a kind, currently 2 * T265 and 1 * D4X5.
We are now testing the very much needed hardware_reset method.
When working with single device everything seems to work pretty well, we noticed some problems when working with more then one device.
In case the hardware_reset is been called while other thread is trying to call pipe.start() or in case  that more then one thread is calling the hardware_reset simultaneously, we notice our application is getting segmentation fault.
are you familiar with the issue?

???

Hi @galikrosentalik
Similar to your other github thread.
Read through this thread:  https://github.com/IntelRealSense/librealsense/issues/5614
Also a PR that can also help:  https://github.com/IntelRealSense/librealsense/pull/5645

Hi @galikrosentalik
Similar to your other github thread.
Read through this thread:  https://github.com/IntelRealSense/librealsense/issues/5614
Also a PR that can also help:  https://github.com/IntelRealSense/librealsense/pull/5645
Let us know if this helps.
*If we don’t hear from you in 7 days, this issue will be closed.

I read the other threads comment.
I did not understand how this issue is related to mine, do you think the last pull request approved by dorodnic suppose to solve my as well?

Thank you for highlighting this issue with hardware_reset while using T265 along with other RealSense products. We have moved our focus to our next generation of products and consequently, we will not be addressing this issue with the T265.


First pass at rectifying the UTF-8 and value classes

@IanCa @happy5214 @monique2208 The conversion to support UTF-8 is more complex than I had thought and requirements about allowed characters appear throughout the spec.  I am sure that I didn't address them all.  Can each of you review the changes that I am proposing and let me know of problems? I'd like to get this support released for the language schema, but it is more complicated than I thought. Thx

Yes... what would you suggest for naming.
On Thu, Mar 21, 2024 at 4:41 PM IanCa @.***> wrote:

@.**** commented on this pull request.
In docs/source/02_Terminology.md
https://github.com/hed-standard/hed-specification/pull/571#discussion_r1534717694
:

A contiguous portion of the data recording during which some aspect of the experiment
is fixed or noted.
+
+## 2.2 Character sets and restrictions
+
+Starting with HED standard schema versions 8.3.0 and above, HED will allow UTF-8 characters in various settings.
+The types of characters referred to in this specification are:
+
+| Name | Description |
+| ---- | ----------- |
+| ascii | utf-8 codes 0 to 127 (single byte) |
+| non-ascii | utf-8 codes greater than 128 (multi-byte) |
So below 32, the only relevant characters I see are \t, \r, and \n. Do we
want to just remove the ascii designation and instead specify which < 32
characters are allowed manually when relevant?(as newline already does)
—
Reply to this email directly, view it on GitHub
https://github.com/hed-standard/hed-specification/pull/571#discussion_r1534717694,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJCJOX4ERDRLXWITB3JZRDYZNHXJAVCNFSM6AAAAABE6BAIE6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNJTGQ4DANJSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Ctrl+X not working in nano

OS version and name: elementary OS Loki
Hyper.app version: @https://github.com/zeit/hyper/commit/e31f975f788f3ced0175c5cffd203690d99c350c
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/WoodyWoodsta/722486e90139eba0138aa0df82140296
Relevent information from devtools (CMD+SHIFT+I on Mac OS, CTRL+SHIFT+I elsewhere): N/A
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue

As of https://github.com/zeit/hyper/commit/be286c0d5aaa14528112213c9be3201384d2dcfa, Ctrl+X does not work in nano, since it is being caught by hyper for the cut command. I'm sure this may be one of many conflicting keyboard shortcuts. Perhaps a way to temporarily bypass the hyper accelerators would help for these cases.

For the reference, https://github.com/zeit/hyper/commit/a8b17e6322622adb4a5e86911f006225fb8f6eac still works with macOS using CTRL + X to quit the nano

I noticed that "CTRL+SHIFT+x" actually work for nano, in replace of "CTRL+x".

Gnome Terminal has copy/paste keybindings set as "SHIFT+CTRL+X" and "SHIFT+CTRL+V" instead of CTRL+X/CTRL+V. On further inspection, every every commonly used short-cut in Gnome Terminal is prefixed with "SHIFT" to avoid conflicts with terminal applications that use the same keystrokes. Maybe that's the solution/convention Hyper should adopt for Linux?

For reference, this is still an issue on the 1.0.0 release..

FWIW and because nobody mentioned it, the exact same problem exists on Windows as it also uses CTRL+* command bindings unlike OS X.
The best way forward I suppose would be to prefix Hyper-key bindings with CTRL+SHIFT rather than simple CTRL, or to add bindings configurability in the user config, so that users can remove all the unneeded shortcuts and/or rebind conflicting ones.

I know that some of the key-bindings already use Shift and so it might be a little tricky. I was happy with the Ctrl+Super (or Win-key) as it was before. Perhaps this is grounds for a plugin that can switch between common sets of key-bindings found in other popular terminals.

@WoodyWoodsta has a good point, though I would almost see the re-assignment of such highly-used shortcuts as a high-priority compared to supporting custom keybindings.
Ctrl+x is assigned to Cut, but the question is do more people use the Cut-shortcut than those who use nano and want to exit in the default (and displayed) way? What's more confounding is that Ctrl+k is assigned to "Clear" in Hyper, but this would act as Cut in nano... 😞

Hy, just started to play with this app. A quick question regarding this subjec, is there a way to change the shortcuts in hyper? It will be great if we could customize these shortcuts.

Hi, I've put in a pull request that hopefully addresses this issue here: https://github.com/zeit/hyper/pull/1191 . It would make the default mod key into Ctrl+Shift on Windows and Linux so that there are no longer conflicts with CLI apps using Ctrl shortcuts.
It does not make the mod key configurable in hyper.js, but I think there's another patch in the works for that.
Feedback is helpful! The pull request is surely not perfect.

+1 this is an odd experience - for instance Control + W should cut back one word, but keeps closing my terminal. What's the best way to fix the bindings?

Chiming in to agree that something should be done to preserve the ctrl+w shortcut.

@taylorthurlow #1509 Already address this.

@ppot Ah didn't think to check that. Glad to hear, thanks!

@taylorthurlow Nothing for!

close in #1876

Thanks a lot @ppot! :)


Tweak SelectAcceleratorCode

This slightly changes SelectAcceleratorCode to make it more legible,
slightly more reliable and slightly faster. The reliability comes from
moving the procedure for annotating functions in the HC reachable set
with 'always_inline' into runOnModule from doFinalization. This is
because the latter offers no guarantees beyond that it will run last,
thus making it possible for other passes to be interposed and possibly
fail. The speed increase comes by way of observing that the only viable
root nodes for the HC reachable set are the kernels i.e. functions which
have teh AMDGPU_KERNEL calling convention. Obtaining the function name
is less convoluted as well.

LGTM.
I think in the future we should follow LLVM coding style in clang/llvm code, e.g. by  using capitalized variable names. Otherwise we will get trouble upstreaming them.

@yxsamliu thank you for the comments. I've tried to update the PR accordingly, if you'd care to take a look. In general I'd like to strive for closeness to Clang / LLVM style as much as possible so that we can try to upstream sooner as opposed to later, so that observation was more than welcome.

I think it is fine to add it to lib/Target/AMDGPU.


Update robots.txt.ejs

Fixes #2709

@bram-atmire Revisiting this. I was curious to validate whether Googlebot supported this syntax. I used google/robotstxt and was happy to see that Google's robots.txt parser does indeed recognize that it is not allowed to crawl those entity search facets:
$ ./robots robots.txt 'Googlebot' 'https://repository.edu/entities/person/808206c2-7e1a-4aeb-8950-ad37f67d6bf7?f.author=Prasanna,%20B.M.,equals&spc.page=1'
user-agent 'Googlebot' with URI 'https://repository.edu/entities/person/808206c2-7e1a-4aeb-8950-ad37f67d6bf7?f.author=Prasanna,%20B.M.,equals&spc.page=1': DISALLOWED


ARC-102 postlogin redirect last fix

This is the last fix needed for the redirect after multiple tests with staging.

Similar to my nitpick the other day: unused var - events

Where do you see this?


Render :emoji: codes where not natively supported

GitHub displays emoji codes (e.g. : :bug:) as emoji icons (e.g. : :bug:), almost everywhere :

repository description
commit message
commit comment
issue/PR message & comment

But not in :

issue/PR title
action title

The list is probably incomplete, but you get the idea.
Would Refined GitHub be suitable to fix it ?
This is mostly interesting when using the gitmoji project.
Thanks !

Thanks for the detailed request.
I'm not sure it's worth supporting emoji codes when one could just use the real emojis, but… I don't feel strongly about it.

Well, some git software and terminal emulators doesn't support or badly displays emojis, either because of encoding issues, or because of font issues, that's why I (and some gitmoji users) use emoji codes instead, which, for instance, are fully supported in Gitea (a self-hosted alternative to GitHub, like GitLab but 100% FLOSS).
Thanks

Is there something I can do to make it happen ? Thanks.

There’s a feature called “format-discussion-titles” I think which you can extend with this. As for GitHub Actions, I’m afraid that it might slow down the page when the log is long.

Oh, I really wanted to try doing it, but I just discovered you're using TypeScript and I really hate that thing, I'm truly sorry :cry:

https://github.blog/changelog/2021-02-11-additional-emoji-support/

@yakov116 I noticed, but the 2 example links are unchanged.

Emojis are the future. Not worth supporting the past here, sorry.

There are interfaces that renders unicode emojis badly especially in terminal emulators and native apps, that's why emoji codes are preferred.

I know, those should be fixed instead.

Agreed, but it's much more work than making GitHub render emoji codes. Additionally, emoji codes features cross-platform autocompletion, while autocompleting unicode emojis usually require specific software on each operating system.


[eeoc] Update for new site

Closes #247.

Seems to run fine with --archive
As mentioned in #247, the new site does not contain all of the reports of the old site. I don't know what the process would be for adding the new site if the old site contains more data, so I left the original eeoc.py in for now.

TODO:

Handle reports that have multiple PDFs (eg, https://oig.eeoc.gov//reports/audit/2016-001-aoig)
Manually add timestamps

Awesome, thanks for doing this! I'll take a look at it. I'm going to add a commit to rename eeoc-new to eeoc, as we don't need to keep the old scraper around.
Regarding the old reports no longer online, we have copies of the PDFs and associated metadata around still. I'll find which reports are now missing, clean them up, and add them over at the unitedstates/reports repository.
Reports with multiple files are a longstanding hairy issue. (see #112) In cases such as that example, we usually pick the most important/substantive file, dropping transmittal letters or memoranda. The EPA OIG scraper is a good example of that approach.
Thanks again for tackling this!

Thanks, @divergentdave, for rounding out the PR!

Looks good, merging and deploying!


Broken links op 0.10 master - Issue # 119

Updated links for pipeline.yaml, apply_manifest_task.yaml and update_deployment_task.yaml

Updated verbiage for image type resource

Hi @vijaymateti thank you for your interest in Pipelines and contributing. However we have deprecated this repository.
The latest documentation on OpenShift Pipelines is now available on: https://docs.openshift.com/container-platform/latest/pipelines/understanding-openshift-pipelines.html
Considering this, can we close this issue?


DUPLO-14936, DUPLO-14934

Overview

Added example of usage for EFS lifecycles policies
Removed transision_to_archive lifecycle policy

Summary of changes
This PR does the following:

...
...

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[ ] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-14934 TF Document changes required for  duplocloud_aws_efs_lifecycle_policy resource

Task linked: DUPLO-14936 TF:efs_lifecycle_policy: getting error "One or more LifecyclePolicy objects specified are malformed" for transition_to_archive lifecycle policy

/update_changelog


cmd/go: temporary directories left behind on windows-amd64-2012 builder

The logs for the windows-amd64-2012 builder (and only that builder, as far as I can tell — CC @alexbrainman) show many flakes at the end of the cmd/go tests, of the form:
2019/03/12 20:59:55 unexpected files left in tmpdir: [go-build421361794 go-build697015714]

(https://build.golang.org/log/177e49c04af866fec137d316334a80a1a5137376)
The test otherwise seems to pass.
That log line comes from here (CC @ianlancetaylor):
https://github.com/golang/go/blob/1650f1ba0b964a06a242c3318e85b3b46f010614/src/cmd/go/go_test.go#L285
The fact that the temporary directories start with the prefix go-build suggests that they come from here:
https://github.com/golang/go/blob/820ad17303e42665fbe9d38d79f07ed218e86302/src/cmd/go/internal/work/action.go#L227
I'm not sure what to make of the fact that those flakes are specific to the one builder. Perhaps there is a kernel quirk on that specific version of Windows that causes it not to clean up completely on exit?
(CC @jayconrod)

The logs for the windows-amd64-2012 builder (and only that builder

Strange. I cannot explain why windows-amd64-2012 builder is different from others.

The fact that the temporary directories start with the prefix go-build suggests that they come from here:

I did not debug this failure. But, generally, you cannot delete directory, if some process have open files in that directory. Same if some process current directory is set to directory being deleted or any directory under.
I suppose someone needs to reproduce this, and then somehow debug this. Once we know how it happens, we might understand better why only windows-amd64-2012 have the problem.
Alex

I spent some time debugging this with gomote and logging in cmd/go TestMan & temporary hooks/logs in io/ioutil.TempDir.
It seems that whatever is making those directories that don't get deleted do NOT come from the same process that runs the test (running the TestMain). I see tons of temp dirs get created from the cmd/go (TestMain) process, but not the ones that cause failures, so I suspect it's a child process (where my logging hook isn't installed) that is creating the directory.
Time for more logging, I guess.

I spent some time debugging this with gomote

I also would not be surprised that this bug is not Go related.
Maybe antivirus is running (or some other Windows program, for example, something that indexes files on disk) and locking our executables at random times. If windows-amd64-2012 computer is configured in a slightly different way than others, that would explain why we see this problem on windows-amd64-2012 only.
Alex

I'm getting tired of builder flakes. Time to find this and either fix it or work around it.

I'm getting tired of builder flakes. Time to find this and either fix it or work around it.

I did some exercise https://github.com/golang/go/issues/25965#issuecomment-482037476 maybe related to this issue. And I can see vastly different results between my Windows 7 and Windows 10 computers. Maybe similar effect can explain what we see with windows-amd64-2012 builder.
Alex

With Bryan's latest debugging CL, they log Access is denied now:
https://build.golang.org/log/47129350051d5b9c0600f9421c2680d7610bd4cd
ok  	unicode/utf8	0.035s
ok  	cmd/addr2line	1.594s
ok  	cmd/api	0.063s
ok  	cmd/asm/internal/asm	0.383s
ok  	cmd/asm/internal/lex	0.023s
ok  	cmd/compile	0.026s
go: failed to remove work dir: remove C:\Users\gopher\AppData\Local\Temp\1\go-build328544146\b001\compile.test.exe: Access is denied.
ok  	cmd/compile/internal/gc	13.031s
ok  	cmd/compile/internal/ssa	0.380s
ok  	cmd/compile/internal/syntax	0.039s
ok  	cmd/compile/internal/test	0.060s [no tests to run]
ok  	cmd/compile/internal/types	0.043s
ok  	cmd/cover	2.564s
ok  	cmd/doc	0.109s
ok  	cmd/fix	7.416s
go test proxy starting
go proxy_test: invalid module path encoding "example.com/invalidpath/v1": malformed module path "example.com/invalidpath/v1": invalid version
go test proxy running at GOPROXY=http://<IP_ADDRESS>:50896/mod
go proxy: no archive golang.org/x/text/language 14c0d48
go proxy: no archive golang.org/x/text/language 14c0d48
go proxy: no archive golang.org/x/text/language 14c0d48
go proxy: no archive golang.org/x/text/foo 14c0d48
go proxy: no archive golang.org/x 14c0d48
go proxy: no archive golang.org 14c0d48
go proxy: no archive rsc.io v1.5.0
go proxy: no archive example.com/split/subpkg v1.0.0
go proxy: no archive this.domain.is.invalid/somemodule v1.0.0
go proxy: no archive this.domain.is.invalid/somemodule v1.0.0
PASS
2019/04/11 22:08:38 unexpected files left in tmpdir: [go-build125570794 go-build372752938 go-build463765546 go-build666810130]
FAIL	cmd/go	181.572s
2019/04/11 22:08:39 Failed: exit status 1

So perhaps we should check for the Access is denied error code specifically and retry (a few times or for a short while) before we give up and return with the directory leaked.
We probably don't want to stall forever if the user has a really slow antivirus installed or something, but a wait of ~500ms or so seems reasonable to me. (Any opinions from the other folks following?)

Retrying for a bit on Access is denied SGTM. Another option might be to only do it if we're in a test, but that might be surprising.

Retrying for a bit on Access is denied SGTM.

I second that.

wait of ~500ms or so seems reasonable to me

I agree. My experiment https://github.com/golang/go/issues/25965#issuecomment-482037476 has -sleep parameter that puts sleep between process exiting and executable deleting, and having -sleep=500ms makes my test pass.
Unrelated, but should we also remove
defer time.Sleep(5 * time.Millisecond)

from windows version of (Process).wait now that it serves no purpose (see #25965 for details)? I find that code is mostly not needed on Windows 10. That code is only needed if you are trying to delete executable immediately after its execution on older Windows version.
Alex

but should we also remove
defer time.Sleep(5 * time.Millisecond)

Forget that suggestion - I am getting exited easily now days. We still have plenty of tests that build executable in a temp directory, run the executable and then delete the directory. These all will fail without time.Sleep. Probably many other developers have tests like that too.
Alex

So far so good. Will keep an eye on the builders today to verify that this is resolved.

Five oks in a row. I'm calling it fixed. 🙂

greplogs -l -e 'unexpected files left in tmpdir: .*go-build' --since=2022-01-01
2022-08-16T01:54:23-5e7697b/windows-amd64-2012
2022-08-08T17:29:15-84be091/windows-amd64-2012
2022-01-28T22:21:43-25b4b86/openbsd-386-70
😩


Query :: force client evaluation on First()

We are currently translating First and FirstOrDefault calls into SQL whenever we can, however we should consider forcing client eval on First, so that we can mimic L2O behavior of throwing if there is more than one element in the collection. We should continue translating FirstOrDefault

First, so that we can mimic L2O behavior of throwing if there is more than one element in the collection.

Do you mean throwing if there is no elements in the collection?

right :)

@maumar can you share some details of what happens now (i.e. wrong results, poor exception message, etc.)

it behaves just like FirstOrDefault, i.e. it never throws.

Clearing for re-triage. This could be a perf regression/breaking change for people who are already using it.

EF Team Triage: we will switch to client eval. FirstOrDefault() is the only one of these operators that we can translate to SQL correctly.

Fixed in d698b57b561a7c27ff513b131937ad2d8c39de89


Make database instance volatile

Make the database INSTANCE variable volatile to ensure thread safety.

I'll comment here given that https://codelabs.developers.google.com/codelabs/android-room-with-a-view/#12 suggests filing bugs on GitHub but "Issues" isn't available for this project.
The code lab currently has code that destroys all the data at every start of the app and doesn't say anything about how to create the initial data only when the database gets created the first time. It would be nice if it would at least provide a link for that use-case especially given that most people likely don't want to delete their data every time the app starts.

done in #4 - including with the comment.


Failed to create cache file

Getting below error with 0.1.12 but works fine with 0.1.11.  If I create .ruff_cache/0.1.12 then it proceeds without error.
pre-commit 3.6.0
ruff 0.1.13
Amazon Linux 2
Python 3.11
Error running pre-commit run -a
ruff.....................................................................Failed
- hook id: ruff
- exit code: 2

ruff failed
  Cause: Failed to create cache file '/home/me/src/repo/.ruff_cache/0.1.12/6018100345838758466'
  Cause: No such file or directory (os error 2)

My hook:
  - repo: https://github.com/astral-sh/ruff-pre-commit
    rev: v0.1.12
    hooks:
      - id: ruff
        args:
          - --line-length
          - "140"
          - --ignore
          - "E501"

Duplicate of https://github.com/astral-sh/ruff/issues/9489, I believe the fix is coming soon in 0.1.13 :)

Should be fixed now via v0.1.13 -- please let me know if not!

Hi @charliermarsh
Thank's for your efforts to maintain this great tool
We are using the latest version 0.3.5 in our pre-commit-config.yaml file :
repos:
  - repo: https://github.com/pre-commit/pre-commit-hooks
    rev: v4.4.0
    hooks:
      - id: check-merge-conflict
      - id: check-added-large-files
      - id: check-ast
      - id: check-symlinks
      - id: check-yaml
      - id: trailing-whitespace
      - id: check-json
      - id: debug-statements
      - id: pretty-format-json
        args: ["--autofix", "--allow-missing-credentials"]
        exclude: Pipfile.lock
  - repo: https://github.com/charliermarsh/ruff-pre-commit
    rev: v0.3.5
    hooks:
      - id: ruff
        args:
          - --fix
      # Run the formatter.
      - id: ruff-format
  - repo: https://github.com/psf/black
    rev: 23.3.0
    hooks:
      - id: black
        args: ["--target-version", "py311"]
  - repo: https://github.com/asottile/pyupgrade
    rev: v3.7.0
    hooks:
      - id: pyupgrade
        args: [--py310-plus]

And running the lint in github actions workflow with this command docker compose run -u root --rm app bash -c "git config --global --add safe.directory /app && pre-commit run --all-files"
But we are getting this error :
ruff.....................................................................Failed
- hook id: ruff
- exit code: 2

error: Failed to initialize cache at /app/.ruff_cache: Permission denied (os error 13)
ruff failed
  Cause: Failed to create temporary file
  Cause: No such file or directory (os error 2) at path "/app/.ruff_cache/0.3.5/.tmpx87Ap3"

ruff-format..............................................................Failed
- hook id: ruff-format
- exit code: 2

error: Failed to initialize cache at /app/.ruff_cache: Permission denied (os error 13)
ruff failed
  Cause: Failed to create temporary file
  Cause: No such file or directory (os error 2) at path "/app/.ruff_cache/0.3.5/.tmpHIXT8j"

We tried to update the Dockerfile :
FROM python:3.11-slim-buster

# Prevent __pycache__ folder and files
ENV PYTHONDONTWRITEBYTECODE 1
ENV PYTHONUNBUFFERED 1

RUN set -ex \
  && apt-get update \
  && apt-get install -qq -y --no-install-recommends \
  make \
  curl \
  gcc \
  git \
  jq \
  gettext \
  procps \
  python3-dev \
  sudo \
  postgresql-client \
  libssl-dev \
  swig \
  && pip install pipenv \
  && rm -rf /var/lib/apt/lists/* /usr/share/doc /usr/share/man \
  && apt-get clean

WORKDIR /app

# Add Tini
ENV TINI_VERSION v0.19.0
ADD https://github.com/krallin/tini/releases/download/${TINI_VERSION}/tini /tini
RUN chmod +x /tini

# Create appuser consumer for the application
RUN groupadd -g 9898 appuser && useradd --create-home -r -u 9898 -g appuser appuser -s /bin/bash
RUN usermod -aG sudo appuser
RUN mkdir /public_assets
RUN chown -R appuser:appuser /public_assets
RUN chown -R appuser:appuser /app
RUN chown -R appuser:appuser /etc/environment
RUN echo "appuser ALL=(ALL) NOPASSWD:ALL" >> /etc/sudoers

# Celery beat
RUN mkdir -p /home/celery/var/run
RUN chown -R appuser:appuser /home/celery/var/run && chmod 755 /home/celery/var/run

# Ruff
RUN mkdir -p /app/.ruff_cache
RUN chown -R appuser:appuser /app/.ruff_cache && chmod 755 /app/.ruff_cache

COPY Pipfile Pipfile.lock ./
RUN pipenv install --system --deploy --dev --verbose

ADD . ./

RUN chmod +x /app/docker-entrypoint.sh

USER appuser
EXPOSE 8000

ENTRYPOINT ["/tini", "-g", "--", "/app/docker-entrypoint.sh"]

But it didn't work
Note that we are using pre-commit = "==3.7.0"
Thank you

@lemaadi thanks for the detailed write-up. I think it would be useful to create a new issue instead as otherwise it might get missed because the original issue is closed.


Broken SPA routing for assets

Describe the bug
This has nothing to do with the CLI, it is an issue that occurs when I deploy my app to production on the Netlify platform. I don't know where else to put this, so I put it here.
Your docs about SPA routing (and all the default answers in your forums and tons of blog posts) state this as the simple solution for SPA routing:
/*    /index.html   200

After all I have read, this should work - but it doesn't.
I'm deploying a Vue3 app. After the build step, I have the following files in the dist directory (from which the app is being deployed):
dist/assets/index.3d0d314e.css
dist/assets/index.8eab93c6.js
dist/favicon.ico
dist/index.html

The following happens: If I enter the SPA via its root, the assets are being loaded correctly. If I enter it via /some/path, it tries to load the assets from /assets/some/path.
I tried to set the above redirect rule via netlify.toml (which I tried to put in the project root as well as in the dist directory), I tried it via a _redirects file in the dist directory. I even tried to do it GitHub Pages style with a custom 404.html file that is a copy of index.html. I also tried it with Node 18.x and 16.x. None of those work. The only way my SPA is working correctly is if the user enters it from its root.
Am I doing something wrong here or is this a bug?
Steps to reproduce
Build Vue3 app with vue-router
Test it locally --> routing works fine
Push it to GitHub repo
Create site on Netlify
Link it to the above GitHub repo
Configure automatic deploys
Push changes to repo
See how build and deployment is successful
Open site in browser
Browse around from https://example.netlify.app --> works fine
Enter via https://example.netlify.app/some/route --> doesn't load assets
Configuration
[[redirects]]
  from = "/*"
  to = "/index.html"
  status = 200

Environment
NodeJS
NodeJS 18.x (I also tried with 16.x)
VueJS
"vue": "^3.2.38",
"vue-router": "^4.1.5"
Netlify Build Settings
Base Directory: client
Build command: npm run build (which executes the Vue3 default build routine run-p type-check build-only)
Publish Directory: client/dist
Builds: Activate builds

I'm not sure how Netlify handles Vue builds behind the scenes, but all the docs mention vue-cli (1, 2), but vue-cli is not used in Vue3 anymore in favor of create-vue (3).

This has nothing to do with the CLI, it is an issue that occurs when I deploy my app to production on the Netlify platform. I don't know where else to put this, so I put it here.

The appropriate place is Netlify Forums: https://answers.netlify.com/

Closing as the correct place to post this is indeed https://answers.netlify.com/ :)


Adds radiation damage to nuclear bombs

About the PR
Makes the nuclear bomb explosion also deal radiation damage.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase
No visible changes.

Changelog

:cl:

add: The nuclear bomb now deals radiation damage in addition to the normal explosion damage.

I feel as it should also use the RadiationSystem to radiate the affected areas

I feel as it should also use the RadiationSystem to radiate the affected areas

Does radiation even work off grid.

I feel as it should also use the RadiationSystem to radiate the affected areas

Does radiation even work off grid.

It does. You could spawn a big radiation source, with insane radiation intensity to feel nuke explosion zone.
Though I'm not sure why would you do that if round will end in one minute and everything in radius supposed to be dead anyway.

It seems like sloth emo and me are all of the opinion that this is unnecessary and will just contribute to worse nuke/explosion performance, so closing this.


Assignment two

UofT DSI SQL - Assignment 2 Submission - Jason Pereira
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Design a logical model for a book store, define SCD architecture, used Coalesce, windows functions using row_number, dense_rank, count(), Partition by, INSRT(), TRIM(), REGEXP, UNION, Cross/Inner Join, Create tables, drop table, insert into table, current_timestamp, delete from table, update table, etc.
What did you learn from the changes you have made?
I understood how to apply all the functions taught in the course to the use case in the assignment which has helped me grasp the meaning of each one,
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
I tried my best to use the teaching from Thomas but to accomplish the asks in the assignment but due to the complexity of the assignment, I had to do significant research on some functions to assist me in my logic to finish the assignment.
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
Yes! This was the hardest part of the course and assignments so far. Mainly because the assignment is too demanding on applying several topics and for someone like me with no deep SQL background it took way TOO MUCH time to think and trial and error to get this done. I can feel the true exhaustion this week, In my case, the questions coming from other students from the working sessions tend to confuse and delay my thinking even more so besides juggling other things happening in my life to attend the support sessions, I find it to be not the best use of my time so my approach is to revisit the videos and do the research myself until I get it done, I understand it is needed for the students to grasp the topics but for this one I feel we should have been given at least 2 weeks to complete. I really hope the assignments don't get harder than this as the course progresses. :)
How were these changes tested?
All code was applied and tested in SQLite against the farmers market DB to ensure the results were as expected.
A reference to a related issue in your repository (if applicable)
Checklist

[ X] I can confirm that my changes are working as intended

Thanks!


feat: Display the reason for application pod restarts on the Devtron UI

Description
Readme icon inpod restart
Fixes https://github.com/devtron-labs/devtron/issues/3792
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] Does this PR require documentation updates?
[ ] I've updated documentation as required by this PR.
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas

Merged in demo/resource-browser


Allow register definition twice (but with another runtime arguments)

We can improve TyphoonAssembly by allowing this:
- (id)objectWithaArg:(id)arg
{
    return [TyphoonDefinition withClass:[Object class] configuration:^(TyphoonDefinition *definition) {
       [definition injectProperty:@selector(value) with:arg];
    }];
}

- (id)objectWithPredefinedArgument
{
    return [self objectWithArg:@"SomeRuntimeArgument"];
}

👍 I'd also like to see this, it'd make reusing definitions and DRYing up assemblies easier.

@chorakm Yes.. but currently it's not implemented yet. Hope we'll get the time to implement that soon.

Oops. Mistakenly assigned to mowens.

Looks good. Will it also work where the argument comes in as a parameter, like this:
- (BaseViewController *)baseViewControllerWith:(UIView *)view andViewModel:(ViewModel *)viewModel {
    return [TyphoonDefinition withClass:[MLBaseViewController class] configuration:^(TyphoonDefinition *definition) {
    [definition useInitializer:@selector(initWithView:viewModel:) parameters:^(TyphoonMethod *initializer) {
        [initializer injectParameterWith:view];
        [initializer injectParameterWith:viewModel];
    }];
}];

- (SignUpViewController *)signUpViewControllerWithViewModel:(ViewModel *)viewModel {
  [self baseViewControllerWith:[self signUpView:viewModel] andViewModel:viewModel]
}

This scenario was asked for a few times: http://stackoverflow.com/q/28330683/404201

Tested on current project:
Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'No component matching id 'TyphoonDefinition-Config_development.plist'.'


Python: don't drop trailing comma from single-element tuple on join

By default, TreeSJ drops the trailing comma when joining tuples, so:
(
  1,
)

becomes (1) instead of (1, ). This is a problem in Python if the tuple has only a single member, as the two forms mean different things: a parenthesized number vs. a tuple.
This can be easy to miss if the join happens as part of a nested collapse. Would be great if TreeSJ could automatically preserve the comma in this particular case when dealing with Python.

It makes sense, thanks!

Thanks for the quick fix. Tested and works perfectly here.
Would there be an easy way to preserve the comma only with single-element tuples? With more elements, the trailing comma is still syntactically correct, but some linters will complain.
If not, certainly better to always keep the comma than to drop it.

At the moment this is not possible, but I will think about how to implement it in the future


swarmctl: add a validation for 'swarmctl version'

Signed-off-by: Kei Ohmura<EMAIL_ADDRESS>

Codecov Report

Merging #1934 into master will increase coverage by 0.71%.

@@            Coverage Diff             @@
##           master    #1934      +/-   ##
==========================================
+ Coverage   54.01%   54.73%   +0.71%
==========================================
  Files         108      108
  Lines       18535    18535
==========================================
+ Hits        10012    10145     +133
+ Misses       7303     7155     -148
- Partials     1220     1235      +15

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 bf90af2...22696f6. Read the comment docs.

What commits should I do?

LGTM


Add documentation for development environment configuration

Problem

The documentation that exists only describes the bare minimum for a working development environment. It does not cover the steps required for setting up, e.g., a Python development environment that works with jedi or pylance to get auto-complete suggestions in editors such as Vim or VSCode.
Description

There are two options for configuring this. Both have trade-offs as far as ease of use goes:

Use the Python interpreter inside the docker container. This can take two forms. Either you can use something like https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-containers or IntelliJ's own version which is built in. Or you can mount the Python installation from the container to the local machine and point your editor to that executable. The latter only works if the Python executable in the container will run on your local machine (i.e., you run Linux and happen to have a similar enough execution environment to the Python base image). You could use a local interpreter but mount the lib directory into the virtual environment to avoid needing to download dependencies twice (in the second option below), but it's bound to be flaky. The first option is almost certainly the most reliable, but requires proprietary editors/editor extensions and is therefore not free. It is probably the "easiest" though, especially for VSCode users if we supplied a devcontainers.json file in our monorepo. PyCharm/IntelliJ may have a similar auto-configuration tool. This option would not work for vim users, as far as I can tell, as Jedi doesn't support remote interpreters.
Run pipenv install --dev inside each project directory and configure your editor to use the appropriate interpreter depending on the directory you're working in. This can be done in VSCode (both free and non-free versions) using multi-directory workspaces. I'm sure PyCharm/IntelliJ and vim-jedi have similar capabilities. This is almost certainly the most reliable option, but it does duplicate dependency installation (once on the local and once more inside the container).

Either way, one of these is required to get completions working locally. My preference is the second because it doesn't require any special tools aside from the ones we already require. However, it does warrant some special documentation. For example, the API depends on psycopg2, which requires pg_config to exist locally. On macOS and Linux you'll need to install something like Arch's postgresql-libs package or just install Postgres in its entirety (it is usually distributed with the userland libraries).

It is probably the "easiest" though, especially for VSCode users if we supplied a devcontainers.json file in our monorepo.

We do supply a devcontainers.json file so this might be even more feasible compared to the other option.
Personally, I exclusively use the Docker container for API and ingestion server development, but still install all Python dependencies locally for autocomplete and highlighting in IDEs.

Personally, I exclusively use the Docker container for API and ingestion server development, but still install all Python dependencies locally for autocomplete and highlighting in IDEs.

That's what I do as well for Openverse. On a project I worked on long ago I used the remote interpreter in PyCharm to great success and I was disappointed that VSCode makes it so much more complicated and hides it behind a proprietary plugin that cannot be installed on non-MS distributions of VSCode. vim-jedi doesn't appear to support it at all :confused:

Just confirming that on macOS, you need to setup Brew and then use it to get postgresql (entirely) to have pg_config.


Show both Output and Debug Console at the same time

Hi,
Is it possible to show both Output and Debug Console at the same time?
If not, could you please add such a feature?
It might also be interesting, if we could show several output channels at the same time too.

this goes in line with my feedback on https://github.com/Microsoft/vscode/issues/2806

I'm relaying an idea by @WebFreak001: Put tabs on the window so you can choose between Output, Debug Console, etc.

Closing, we currently do not plan to work on this.


Pods cannot connect to internet when behind a proxy, have to manually export http_proxy

Summary

Pods cannot connect to the internet by default when behind a proxy. Have to set 'export http_proxy=http://xxxx:xxx' manually within pods for them to work.
What Should Happen Instead?

Pods should be able to connect to the internet via proxy, when the environment variable HTTP_PROXY is set for '/etc/environment/' or 'var/snap/microk8s/current/args/containerd-env' that should have also set the pods environment.
Reproduction Steps

Have a proxy setup
Use default microk8s containerd
Launch a curl pod, curl google.com
if now set 'export http_proxy=http://xxxx.xxx', things should work.

Introspection Report

Can you suggest a fix?

Are you interested in contributing with a fix?

Hi @matthewygf
This is in fact working as intended. The containerd environment variables are not expected and must not be part of the environment of any running workloads.
If you need an http_proxy to access the network, you must also configure it per pod, since there is no way for them to get this from the host.
One could possible attempt to automate this by using mutating webhooks, but that is not a setup that is supported out of the box.

Hmmm, got it, thanks for the reply.


Update manifest.json

I changed the colour to the TU colour
(issue #43 )

Sure but this is only part of the issue:joy:

Closing this one as it's a very empty PR

It does partly fix the issue though, right?

What did you want to change then?

Yes but there is more to do, let's do a PR with all of this fixed (and if you pick up an issue, @Selawen, please do the entire one, as changing one string is 0 effort for us as well, we could do that when doing this issue anyway:smile:)

I understand what went wrong now, I didn't realise there were 3 different issues packed into 1

Ah okay great, no offense of course, just want to have everything in 1 place, and I don't like too many commits (though right now there are way too many in the master as well imo)

The best practice is actually to commit often. We can always squash commits later if we would want to.
Reference: Git Best Practices

Yes, but I disagree with committing such a small change, which doesn't need any kind of checkpoint or something (which is why you would commit often). Then we have by far the easiest and smallest part of 1 issue solved. So I'd reckon that if a contributor wants to do a PR, they should have fixed the entire issue, so we don't merge 1/3rd of the issue


Unable to reproduce the results with the provided model weights and code on LEVIER dataset.

To Reproduce the error
Steps to reproduce the behavior:

added the given weights in the checkpoints/elgcnet_levir/  as best_checkpoint.pt
run eval_cd.py

Error received
  File "/content/elgcnet/eval_cd.py", line 55, in main
    model.eval_models(checkpoint_name=args.checkpoint_name)
  File "/content/elgcnet/models/evaluator.py", line 178, in eval_models
    self._load_checkpoint(checkpoint_name)
  File "/content/elgcnet/models/evaluator.py", line 66, in _load_checkpoint
    checkpoint = torch.load(os.path.join(self.checkpoint_dir, checkpoint_name), map_location='cpu')
  File "/usr/local/lib/python3.10/dist-packages/torch/serialization.py", line 713, in load
    return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
  File "/usr/local/lib/python3.10/dist-packages/torch/serialization.py", line 920, in _legacy_load
    magic_number = pickle_module.load(f, **pickle_load_args)
_pickle.UnpicklingError: invalid load key, '\x27'```

Make sure that you are using the correct file path.
Its a common practice to look at the state dict and its keys before loading the model to avoid the error due to key mismatch problem.

Go to evaluator.py and look at the _load_checkpoint function. Update your function to correctly load the model weights.
def _load_checkpoint(self, checkpoint_name='best_ckpt.pt'):

        if os.path.exists(os.path.join(self.checkpoint_dir, checkpoint_name)):
            self.logger.write('loading last checkpoint...\n')
            # load the entire checkpoint
            checkpoint = torch.load(os.path.join(self.checkpoint_dir, checkpoint_name), map_location='cpu')

            if isinstance(self.net_G, torch.nn.DataParallel):
                msg = self.net_G.module.load_state_dict(checkpoint) #['model_G_state_dict'])
            else:
                msg = self.net_G.load_state_dict(checkpoint) #['model_G_state_dict'])
            print(msg)

            self.net_G.to(self.device)
            self.logger.write('\n')

Hope it works!

i tried doing the mentioned changed but still received the same error as before. Please look into it.

There might be problem with with your downloaded files or the libraries you are using.
I am not getting any error. Here is the log
checkpoint_name: elgcnet_levir_ckpt.pt checkpoint_dir: ./elgcnet_levir
<All keys matched successfully>

Begin evaluation...
Is_training: False. [1,2048],  running_mf1: 0.96269
Is_training: False. [101,2048],  running_mf1: 0.50000
Is_training: False. [201,2048],  running_mf1: 0.97121
Is_training: False. [301,2048],  running_mf1: 0.99011
Is_training: False. [401,2048],  running_mf1: 0.50000
Is_training: False. [501,2048],  running_mf1: 0.50000
Is_training: False. [601,2048],  running_mf1: 0.50000
Is_training: False. [701,2048],  running_mf1: 0.88616
Is_training: False. [801,2048],  running_mf1: 0.92438
Is_training: False. [901,2048],  running_mf1: 0.89096
Is_training: False. [1001,2048],  running_mf1: 0.97063
Is_training: False. [1101,2048],  running_mf1: 0.92529
Is_training: False. [1201,2048],  running_mf1: 0.49982
Is_training: False. [1301,2048],  running_mf1: 0.88886
Is_training: False. [1401,2048],  running_mf1: 0.50000
Is_training: False. [1501,2048],  running_mf1: 0.97076
Is_training: False. [1601,2048],  running_mf1: 0.97004
Is_training: False. [1701,2048],  running_mf1: 0.49924
Is_training: False. [1801,2048],  running_mf1: 0.50000
Is_training: False. [1901,2048],  running_mf1: 0.49871
Is_training: False. [2001,2048],  running_mf1: 0.98605
acc: 0.99118 miou: 0.91450 mf1: 0.95368 iou_0: 0.99076 iou_1: 0.83825 F1_0: 0.99536 F1_1: 0.91201

You can get help from here.

I went through the link you provided. I speculate that The .pt file that i have received after training is of 127 MB  while the .pt file uploaded is mere 44MB , i believe there's some mismatch somewhere.


Zap Tutorials link is dead

Pressing the "Zap Tutorial" button in app goes to to a dead link

Description
In the welcome page of the app, pressing the "zap tutorials" button leads to the dead link _https://ln-zap.github.io/zap-tutorials/zap-desktop-getting-started_

Expected Behavior
Have the link go to available documentation, or else remove it until a suitable tutorial is in place
Possible Fixes

Link to available documentation such as the github readme
https://github.com/LN-Zap/zap-desktop/blob/master/README.md
Link to youtube tutorial (outdated version)
https://www.youtube.com/watch?v=Z2XDt7iKE7A&list=PLMj6UA3-f3cQq_TjCXAAeP1mmRnjgnDrI
Add scrolling tips to users in text form
Remove the link

Steps to Reproduce

2)

Context
As a new user this bug makes onboarding more difficult. While it does not affect functionally I think it is important to provide first time users with as fluid a experience as possible.
Even removing the link would make the experience a bit nicer.

Your Environment

Zap version: Zap 0.6.2-beta
Operating System and version: linux appimage
Web-browser: Firefox and Brave

@agebben this has been fixed here: https://github.com/LN-Zap/zap-desktop/commit/1f480b6cc77f4f46641154722d3f4c2c66212b1a


CMD+I has different behavior if Inline Chat is open in the file somewhere

Testing #199158

in an editor, hit cmd+i -> inline chat opens DON'T CLOSE INLINE CHAT
click somewhere else in the file
hit cmd+i

For me, completions open. I was expecting:

Inline chat to move down to my cursor
A new Inline chat to start here

For a large file, I might forget that I started inline chat in an off screen portion of my file...

Hi thanks for writing about this issue. As you mentioned, I think when the inline chat is already open we do not open it again. I guess we could change the context key here. Thoughts cc @jrieken ?


Create migration to gamemaker

I'm not sure how plausible this would be, as it uses it's own proprietary GML language (similar to JavaScript, but not really), and porting assets like game scenes would be impossible due to how GMS' works internally (no concept of prefabs, etc)
GameMaker Studio support isn't currently planned or considered, especially as it's not very requested.


[W-11070475] Recurring Donations Mobile

Critical Changes
Adapt design to be mobile responsive
Changes
rename the component and refactor to adapt to the proposed design

W-11070475


In v2.1.0, 2.1.1 shows a field error in configuration of cybersource payment method

In v2.1.0, 2.1.1 shows a field error in configuration of cybersource payment method
How to Reproduce:
Step 1: Install this module
Step 2: Click on Configure -> Payments Methd in this otpion go to the bottom and here shows CyberSource Payment option and shows a option but here in some field not added for add a data for configuration:
Not showing these fields properly:

Profile Id
Access Key
Secret Key

Note: This issue shows on v2.1.0, v2.1.1 release but with v2.0.0 version shows works fine
Possible Solution:
Should be shows a proper field here so that we can continue this payment after configuration like this, checks in v2.0.0:

This issue has been Fixed.

I have checked this issue, now this issue has been resolved with v2.1.0 and also v2.1.1:
V2.1.0:

V2.1.1:

So, I am closing this issue.


Added missing spring enums for generic_6dof_joint

Seems the enums around the spring properties hadn't been bound so they weren't available in GDScript. Just adding them here.
I've only just started playing with joints so don't feel confident enough yet to add some meaningful descriptions.

Thanks!

Cherry-picked for 3.2.2.


Unable to build mediapipe for older Android versions

Please make sure that this is a build/installation issue and also refer to the troubleshooting documentation before raising any issues.
System information (Please provide as much relevant information as possible)

OS Platform and Distribution (e.g. Linux Ubuntu 16.04, Android 11, iOS 14.4): Ubuntu 20.04
Compiler version (e.g. gcc/g++ 8 /Apple clang version 12.0.0):
Programming Language and version ( e.g. C++ 14, Python 3.6, Java ): Java
Installed using virtualenv? pip? Conda? (if python): No
MediaPipe version:
Bazel version: 4.2.1
XCode and Tulsi versions (if iOS):
Android SDK and NDK versions (if android): SDK 28.0.3
Android AAR ( if android):
OpenCV version (if running on desktop):

Describe the problem:
I am trying to build mediapipe example apps for older Android versions. But it throws error saying Bazel requires Android SDK 30 or greater. I understand the bazel and SDK versions are not compatible. Which version should I use to build Mediapipe with Android SDK build-tools 28?
Provide the exact sequence of commands / steps that you executed before running into the problem:
Complete Logs:
Include Complete Log information or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached:
Bazel requires Android build tools version 30.0.0 or newer, 28.0.3 was provided

Hi @AbinayaKumar25,You can try our Gradle Android API solutions https://google.github.io/mediapipe/getting_started/android_solutions.html#mediapipe-android-solutions

I tried to build aar file but I ended up with the error as stated above. Bazel requires android build tools 30 or newer. I have a hard constraint that I can’t update my build tools. Any suggestions?

What is the error are you seeing while building the code.

Build failed. Bazel requires android build tools 30 or newer. My Bazel version is 4.2.1

This are Prerequisites  https://google.github.io/mediapipe/getting_started/android.html#prerequisite, But still you can build. and run applications with android:minSdkVersion="21"

Does it mean I cannot run applications with Android buildtools version 28 or 29?

@sgowroji  Could you confirm whether mediapipe cannot be built with Android build tools version 29 or below?

@sgowroji could you confirm the above question?

Hi @AbinayaKumar25,
Could you please go through the above comment and let us know the status of the issue. Thank you!


doc_comment::doctest!() is never called

It seems that the test feature requirement for the doctest!() macro currently disables the validation of the tests cases inside the README.md file.
https://github.com/Geal/nom/blob/f40e4a74bd25abea2d2de306476c143dd0e9a181/src/lib.rs#L425-L426
I assume that the reason here is rustdoc not using the test flag (https://github.com/rust-lang/rust/pull/61199). So this will not work until https://github.com/rust-lang/rust/pull/61351 and https://github.com/rust-lang/rust/pull/62213 are merged and we change the l.425 into #[cfg(doctest)].

thanks for the report, I moved to cfg(doctest) in d4d79d0


Add autoplay feature, resolve issue52

Co-authored-by: Coco Lin<EMAIL_ADDRESS>Co-authored-by: Lyu Ji<EMAIL_ADDRESS>This pull request solve the issue52. We added the feature of checkbox to allow users choose whether to autoplay the video, with the default set to be true.
This is now working on safari, but not on Chrome, so we will create a new issue about how to make this feature working on all browsers.

Sending you a screenshot. It looks like adding this caused the controls to get a bit messy. I can't use the volume slider and the play button is nowhere to be found.


Test multiple SDKs in Travis.

I believe this will fix CI on master.

I'm merging it, even now Travis won't work until recent RN changes and test fixes have been merged.


Image Missing from README

Added Image to README

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This file modifies a pre-existing file.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

20251e85d270a2fcdbd0275321d44c48fcd2c702

This is pointless.


Doc builds are badly broken

Describe the bug
It is currently impossible to complete building the docs.
To Reproduce
Steps to reproduce the behavior:

make docs
Experience the shame of failure
Profit

This is due to the metaprogramming done to dynamically build the help string for search commands. The solution is to skip auto-documenting them and instead write up a guide to using search.


Create sharif.txt

<EMAIL_ADDRESS>This is how the domain looks like.
Thanks.

the university domain is http://www.sharif.ir/home - please somehow confirm that the provided domain also belongs to the university and is used for emails. Thank you!

How can I do so?
Would showing the login page of the E-mail system with this domain be
enough?
That's the university portal address; the E-mail domains are (the ones that
I know of):
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Greetings!
On Jul 21, 2015 9:55 PM, "Elena Ganenkova"<EMAIL_ADDRESS>wrote:

the university domain is http://www.sharif.ir/home - please somehow
confirm that the provided domain also belongs to the university and is used
for emails. Thank you!
—
Reply to this email directly or view it on GitHub.

@BehnazHosseini yes, the login page should work fine - you can just send me a screen shot of it. Thank you!

[image: Inline image 1]
This is it.
On Fri, Jul 24, 2015 at 5:14 PM, Elena Ganenkova<EMAIL_ADDRESS>wrote:

@BehnazHosseini https://github.com/BehnazHosseini yes, the login page
should work fine - you can just send me a screen shot of it. Thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/swot/pull/1169#issuecomment-124507986.

@BehnazHosseini the image didn't attach unfortunately. Please send it to me to<EMAIL_ADDRESS>Thank you!

@BehnazHosseini thank you - i've got your email and added the domain

You're very welcome. :)
On Aug 3, 2015 4:12 PM, "Elena Ganenkova"<EMAIL_ADDRESS>wrote:

@BehnazHosseini https://github.com/BehnazHosseini thank you - i've got
your email and added the domain
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/swot/pull/1169#issuecomment-127205519.

also please add another subdomain from sharif, mech.sharif.ir

Hi awatme
This pull-request is closed, I don't know if they will see your request.
I think the best thing you can do is open a new one for mech.sharif.ir (with the correct file created under /lib/domains/ir/mech/sharif.txt - I think) and send them proof
I only saw your message because I have it in my notifications =)


HTML Credits

Other web mapping platforms like Leaflet and OpenLayers define attributions as HTML strings.  This changes our Credit to do the same.  HTML allows for much more flexibility than only exposing options for setting the text, image source and URL link.  For example, will allow users to create credits like
"My source is from some website with some license", something that is frequently needed for proper attribution, but not currently supported with our Credit.
Summary of changes:

Modified Credit to accept an html string instead of an option object
Added xxs.js library for sanitizing the HTML before creating a DOM element from it
Since Credit no longer differentiates between 'text' credits and 'image' credits, modified CreditDisplay to add all credits to the same credit container
To keep the Cesium credit first, created a separate CreditDisplay._cesiumCreditContainer to add the Cesium credit

@hpinkos, thanks for the pull request! Maintainers, we have a signed CLA from @hpinkos, so you can review this at any time.
:warning: I noticed that a file in one of our ThirdParty folders (ThirdParty/, Source/ThirdParty/) has been added or modified. Please verify that it has a section in LICENSE.md and that its license information is up to date with this new version. Once you do, please confirm by commenting on this pull request.

I am a bot who helps you make Cesium awesome! Contributions to my configuration are welcome.
:earth_africa: :earth_americas: :earth_asia:

@mramato can you review when you have some time? thanks!

The Bing key warning message can now be improved since we can have inline links and other styling.

Looks like there is some missing test coverage, particular in the deprecated code.
It's probably also a good idea to expose createDomNode as a private function for testing it directly.
It also looks like a data attribution link appears in the bottom of the tests at some point and doesn't go away.

Thanks @hpinkos!  This will be a big improvement.  That's it for my initial review, but I actually didn't review Credit.js or CreditDisplay.js too closely yet, once all my initial feedback is addressed I'll give this another look.
Thanks again!

@mramato ready for another look.  I addressed all of your comments.

@mramato I think this is ready now

Changed span to div
Automatically add target="_blank" to all a tags

One last bug.  Switching to Sentinel-2 (or any of the 3 ion imagery assets in the BaseLayerPicker) does not show the correct credit.  The ion logo isn't there and the other attribution gets shown on screen instead of in the lightbox.

I also tweaked the Bing/Mapbox signup credits to make them more obvious.

@mramato fixed

Whoops, broke one of the tests.  One sec

@mramato ready

Sorry, found one more issue.  The ion Sandcastle examples are triggering deprecation warnings: http://localhost:8080/Apps/Sandcastle/index.html?src=Blue Marble.html&label=ion Assets

@mramato that will be fixed when we change the attributions being sent from the ion server.  I'm wasn't sure if it's something we should work around here.

Makes sense, since we're putting that change in by tomorrow, I'll just hold off on merging this (but otherwise looks ready).

@hpinkos one more thing, loading ion demos, such as http://localhost:8080/Apps/Sandcastle/index.html?src=Sentinel-2.html&label=ion Assets results in the logo being push up above Data attribution instead of next to it (even though there are no other credits)

@mramato that's a problem with the credit being sent from the ion server.  I submitted a fix for it.  I guess we should wait to merge this until that goes through.

@mramato  this is ready now

Thanks!

Kind of a minor thing, so feel free to disregard.
The Bing key message that is printed to the console includes the html tags, but would look better without them.


HTML

HTML
USER STORY
ACCEPTANCE CRITERIA
Given a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

As per section 3.1.6 for prework. All HTML content has been created and pushed to the main branch.


openapi: Fix principals array type

We do not accept heterogeneous arrays containing both user ids and email addresses.
This also happens to disallow an empty array, which is fine since the principals parameter should be omitted if the default to the calling user is desired.

Merged, thanks for the bug fix @andersk!


Relax typing for

There are lot of values which are not covered by the existing typing (e.g. "og:title","og:url","og:image" and so on) for meta-property attributes. I would suggest to declare the typing as string.

Actually the property attribute does not exist on the current typing for the <meta> tag at all. For some reason MDN doesn't list that attribute although it seems that it is indeed a part of the HTML5 standard. Anyhow, I think adding this attribute with string type should be fine.