kitchen docker is unable to ssh into docker when using static ip in the run_options: --net network --ip  "<IP_ADDRESS>"

Context:
I want to be able to use kitchen docker for multi node testing through static ip within a network set by me on the different docker containers / kitchen suites
OS: Mac
kitchen.yml with
run_options: --net mynetwork --ip <IP_ADDRESS>
Test Kitchen version 1.10.2
docker version
Client:
Version:      1.11.2
API version:  1.23
Go version:   go1.5.4
Git commit:   b9f10c9
Built:        Wed Jun  1 21:20:08 2016
OS/Arch:      darwin/amd64
Waiting for SSH service on 'Docker Daemon IP:port' retrying in 3 seconds
Waiting for SSH service on 'Docker Daemon IP:port', retrying in 3 seconds
Waiting for SSH service on 'Docker Daemon IP:port', retrying in 3 seconds
Can anyone help?
Which setting should I give kitchen so that he knows how to login?
Am I doing something wrong?
P.S. Removing run_options kitchen is able to login and converge successfully
Thank you all

I faced the similar problem, you can use https://github.com/test-kitchen/kitchen-docker/pull/203 to solve it

thank you so much @ytsarev . I will give it a try . I was thinking to do something similar but I thought it was something wrong with me. now I know its not
Wouldyou have a working example kitchen.yml you could share that works with your PR?

docker static ip setting
docker network settings
ssh_host ip

Many thanks

hi @ytsarev i used your changes however I get
Waiting for SSH service on <IP_ADDRESS>:32782, retrying in 3 seconds
the port 32782 is not apparently the container port in use
i have changed the port manually to 22
i then get asked for password on my kitchen username 'kitchen kitchen' fails to login manually

@claudio-viola the only thing you can control with ssh_host is the host to connect to. The network connectivity will not be solved automatically. And apparently your are trying to connect from the host to a container private network which is not going to work out of the box, iirc

yes @ytsarev that's exactly what i want to do. Any clue how it is possible to achieve this?
With vagrant it is quite easy and kitchen vagrant is able to connect to the private network.
Trying to achieve the same with kitchen-docker

If you are running basic centos you docker file should look like this
FROM centos:6
RUN yum clean all
RUN yum install -y sudo openssh-server openssh-clients which curl htop
RUN ssh-keygen -t rsa -f /etc/ssh/ssh_host_rsa_key
RUN ssh-keygen -t dsa -f /etc/ssh/ssh_host_dsa_key
RUN mkdir -p /var/run/sshd
RUN useradd -d /home/<%= @username %> -m -s /bin/bash <%= @username %>
RUN echo <%= "#{@username}:#{@password}" %> | chpasswd
RUN echo '<%= @username %> ALL=(ALL) NOPASSWD:ALL' >> /etc/sudoers
RUN mkdir -p /home/<%= @username %>/.ssh
RUN chown -R <%= @username %> /home/<%= @username %>/.ssh
RUN chmod 0700 /home/<%= @username %>/.ssh
RUN touch /home/<%= @username %>/.ssh/authorized_keys
RUN chown <%= @username %> /home/<%= @username %>/.ssh/authorized_keys
RUN chmod 0600 /home/<%= @username %>/.ssh/authorized_keys
RUN curl -L https://www.chef.io/chef/install.sh | bash
RUN echo '<%= IO.read(@public_key).strip %>' >> /home/<%= @username %>/.ssh/authorized_keys
then try kitchen create and kitchen converge

FROM centos:6
RUN yum clean all
RUN yum install -y sudo openssh-server openssh-clients which curl htop
RUN ssh-keygen -t rsa -f /etc/ssh/ssh_host_rsa_key
RUN ssh-keygen -t dsa -f /etc/ssh/ssh_host_dsa_key
RUN mkdir -p /var/run/sshd
RUN useradd -d /home/<%= @username %> -m -s /bin/bash <%= @username %>
RUN echo <%= "#{@username}:#{@password}" %> | chpasswd
RUN echo '<%= @username %> ALL=(ALL) NOPASSWD:ALL' >> /etc/sudoers
RUN mkdir -p /home/<%= @username %>/.ssh
RUN chown -R <%= @username %> /home/<%= @username %>/.ssh
RUN chmod 0700 /home/<%= @username %>/.ssh
RUN touch /home/<%= @username %>/.ssh/authorized_keys
RUN chown <%= @username %> /home/<%= @username %>/.ssh/authorized_keys
RUN chmod 0600 /home/<%= @username %>/.ssh/authorized_keys
RUN curl -L https://www.chef.io/chef/install.sh | bash
RUN echo '<%= IO.read(@public_key).strip %>' >> /home/<%= @username %>/.ssh/authorized_keys

Hi guys,
I'm facing the same issue.
I think it's related to the command used by kitchen to fetch the IP of the container.
If I correctly understand, kitchen use docker inspect --format '{{ .NetworkSettings.IPAddress }}
https://github.com/test-kitchen/kitchen-docker/blob/8743b4561645aea7ef66f67deafc9711e438bd7f/lib/kitchen/driver/docker.rb#L412
The problem is, when you use a network different than the default to run your container, this field is empty.
As you can see in the following exemple:
"NetworkSettings": {
            "Bridge": "",
            "SandboxID": "fca84b497cc9975708e0eeebcf2998cfd52a812565bf95965e9c1302fd396898",
            "HairpinMode": false,
            "LinkLocalIPv6Address": "",
            "LinkLocalIPv6PrefixLen": 0,
            "Ports": {},
            "SandboxKey": "/var/run/docker/netns/fca84b497cc9",
            "SecondaryIPAddresses": null,
            "SecondaryIPv6Addresses": null,
            "EndpointID": "",
            "Gateway": "",
            "GlobalIPv6Address": "",
            "GlobalIPv6PrefixLen": 0,
            "IPAddress": "",
            "IPPrefixLen": 0,
            "IPv6Gateway": "",
            "MacAddress": "",
            "Networks": {
                "va475an4bu3nv5lbmd7r4zemc9prla5v": {
                    "IPAMConfig": null,
                    "Links": null,
                    "Aliases": [
                        "Unit test",
                        "b2897c85bf64"
                    ],
                    "NetworkID": "34cb275a0826995fcfc5cce9653eeda581b402dd8d6c41cfc2842ebaee1b09a5",
                    "EndpointID": "fe36c8e44d8ba7d463cad104746d824f71a374303fa48da29f2fb256458a0e09",
                    "Gateway": "<IP_ADDRESS>",
                    "IPAddress": "<IP_ADDRESS>",
                    "IPPrefixLen": 16,
                    "IPv6Gateway": "",
                    "GlobalIPv6Address": "",
                    "GlobalIPv6PrefixLen": 0,
                    "MacAddress": "02:42:ac:13:00:02",
                    "DriverOpts": null
                }
            }
        }

To get the IP you can do something like this:
docker -H unix:///var/run/docker.sock inspect --format '{{ .NetworkSettings.Networks.va475an4bu3nv5lbmd7r4zemc9prla5v.IPAddress }}' b2897c85bf64
<IP_ADDRESS>

But it's not easy because you need to pass the name of the network on the filter.
One quick and a bit dirty way is to do something like this:
docker -H unix:///var/run/docker.sock inspect  b2897c85bf64 | grep IPAddress | tail -n 1| cut -d "\"" -f 4
<IP_ADDRESS>

But it's not very elegant.
A solution could be to do it with 2 steps:

1 get the network name.
2 get the IP.


Username changes

After persisting the user and their username changes is there an easy way to check the GUID of the user during login so we update the persisted values with the new user name and details?
Example:
User Jane Doe (username: jandoe ) logs in and gets persisted with all her details ( Name, ldapGuid, ldapMemberships, etc... )
She gets married,  Jane Doe is not Jane Johnson ( username: janjon )
In the onLoginSuccess the $user = $event->getUser() see this as a new user since the username was not found.

I got it figured out!


I wanna add Duster as pre-commit hook to my project.  But at the moment, the command (and thus my hook) fails when there are no dirty PHP-files, when I only update the readme for example. So my question is, can we add an extra flag to the lint command, which will show the "no dirty flags" message (maybe as info instead of error in this case), but exits with exit code 0?

I will tag this as a bug because this is the expected behaviour.

Can't wait for the fix. Used Duster only for one night on a private project yet, but I'm sold :wink:

@hylke94 give 1.1.1 a try and let me know if that resolves this issue.

@hylke94 give 1.1.1 a try and let me know if that resolves this issue.

It works! Thank!


Difference between get and getItem

Hi,
Thank you for writing this library.  Its very helpful in my project.
Can you give some information on the difference between get and getItem.  I'm asking because, for some reason, getItem is not working for me (Lockr.getItem(...) - getItem() is not a function)).
Regards,
Akash

There is no getItem function in Lockr. You probably misread our README, which explicitly calls localStorage.getItem.


Medium Blog Link -> 404

Hi there!
The link to Mediun in the README
https://github.com/apex/apex#links
And on the website return a 404
http://apex.run/

oh nice thanks! missed that


Release for Laravel 5.8?

Added in the dev-master, but no separate tag for it.

Hey @rennokki, v3.0.5 has now been tagged

For some kind of reason, it didn't reached Packagist.

Hmm, strange, seems like it's fixed now though.


Installing<EMAIL_ADDRESS>on React 18 project

Describe the bug
Installing<EMAIL_ADDRESS>on React 18 project getting  incorrect dependency warning as below:
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>
And on my github CI, it gets npm err:
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@toast-ui/react-editor
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/runner/.npm/eresolve-report.txt for a full report.

To Reproduce
Steps to reproduce the behavior:

make react 18 project
add the recent version of @toast-ui/react-editor (yarn add @toast-ui/react-editor)
reproduces warning on console

Expected behavior
Install react-editor package without warning on react 18 project
Screenshots

warning on local computer

error on github eslint

Desktop (please complete the following information):

OS: Mac
Browser: Chrome
Version: MacOS Monterey (12.0.1), Chrome 106.0.5249.119 (Recent version)

Additional context
Is there any newer version of react-editor that runs on react 18 project? Or do I just need to ignore the warnings?

@hayeon9826
Thank you for the issue. We will handle to React 18 later.
It is difficult to check quickly because we are currently working on several projects other than TOAST UI.
If you have to do it right now, you can make a wrapper yourself and use it.


Just wanna to talk a joke HaHa

Why your code can not build?
Can you give me a reasonable explanation?

Why your code can not build? Can you give me a reasonable explanation?

Agree with you


Generate an access log from the apiserver

Feature request
Log all requests to the Results apiserver to stdout. This can be accomplished using grpc interceptors from grpc-ecosystem.
Use case

Generate an access log for audit/compliance purposes.
Record request durations and responses for debugging.

/remove-lifecycle stale

Fixed by #275

/closed

/close
I believe this has been fixed (at one point we had a regression, but it was resolved).


Move CMSIS OS APIs under subsys/portability

- portability: move cmsis APIs to subsys/portability

portability: move cmsis api headers
portability: move cmsis api samples/tests

Do we need compat headers for the old locations?

no, those are always included as
#include <cmsis_os2.h>

and
#include <cmsis_os.h>


Support import and export from The Forge

I am a Forge customer (https://forge-vtt.com/) and I think the core data directory is different than a self-hosted FVTT game. That might be what causes this to break when importing/exporting to/from a Forge-hosted world.
Here is an example of a path to an asset I placed in the /adventures/import/ directory.
https://assets.forge-vtt.com/5ea8c7a5da596b7e45b1c898/adventures/import/Baileywiki Mansion LVL1 2100x1300 100ppi NO GRID grass.png

I'm not sure I understand.  You no longer have to put the adventure file in the import folder.  You can select it with the file input and it will upload and import for you.
Are you having trouble exporting or importing?

I am a Forge (hosted) user. When I export a scene from a Forge world using
Adventure Exporter, then try to import that scene to my local instance of
Foundry, I end up having to manually change all of my file references to
point to the imported files.
I want to share my maps with the community and my Patreon, but I can't
expect them to try to figure out that manual mapping. It's also possible
I'm doing something wrong.
On Sat, Sep 19, 2020, 12:24 PM Chris Stadther<EMAIL_ADDRESS>wrote:

I'm not sure I understand. You no longer have to put the adventure file in
the import folder. You can select it with the file input and it will upload
and import for you.
Are you having trouble exporting or importing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cstadther/adventure-import-export/issues/103#issuecomment-695347199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADMLTXS3DS2SWJBPJDJXMMDSGUAOXANCNFSM4RSZ52PQ
.

Ah makes sense.  Just as a workaround maybe until I get a fix written, I originally envisioned content creators using their local version to create the adventures so that users on hosted versions could import easily.

Ah ok. So I should build my maps locally until you've worked it out.
Incidentally, I'm sure KaKaRoTo would be thrilled if you can help export
the original PNGs in a package that doesn't hit his servers everytime
someone loads the shared map. He would probably promote the module to his
community.
On Sat, Sep 19, 2020, 6:08 PM Chris Stadther<EMAIL_ADDRESS>wrote:

Ah makes sense. Just as a workaround maybe until I get a fix written, I
originally envisioned content creators using their local version to create
the adventures so that users on hosted versions could import easily.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cstadther/adventure-import-export/issues/103#issuecomment-695413879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADMLTXQACEXFFIZ5KKT2FXTSGVIW7ANCNFSM4RSZ52PQ
.

I am a Forge user, too.  Forge has a separate "Assets Library" (link) hat the the importer doesn't seem to be able to populate.  The import presents countless errors "something went wrong with your file upload request".  My imported game has no images (obviously not usable).


Add support for balancermember weights

This implements https://tickets.puppetlabs.com/browse/MODULES-2951

Fantastic, thanks for the improvement !


nesting of species geography under genus

localities are not amalgamated under a higher level taxon
If I go to the genus Chalcis, the otu page shows a map distribution of only India ( the one locality entered at the genus level).
It would be much better to draw in the geography from all of the children to show the true distribution of the genus (worldwide). Similarly, the same can be said of each higher level taxon.
... and why not other otu relationships as well such as hosts and associates?

@JohnHeraty thanks very much for logging this, was just pondering how many issues we've discussed that have been burried in our meeting minutes.
Couple of gentle hints about filing an issue:

Identify which Task, or page/section is being referenced.  @mjy  can infer, but TW is big, and there are numerous other developers who may need to understand specifics, so pointing us to what you're seeing helps everyone.
Add a (annotated) screenshot if possible.  A picture is worth many words.  You can use shift-command-4 to draw a window on your screen.  The result is a screenshot added to your desktop.

It would probably make sense to convert dots (if the data comming from specimens) to the country or state shape to display the distribution of a genus or a higher taxon.

a screenshot does not work, so here is the url to the problem.
https://sfg.taxonworks.org/tasks/otus/browse/93578
Chalcis has a huge number of descendant species. Assuming that each of the species has a distribution, none of this is being aggregated to the genus level, which shows up now as just India for the single genus level record that has been input. At a higher group level, the geographic information should be amalgamated from the descendants - however that information is being inputted (shape or dot) for the otus.
As an example, Chalcis canadensis has a North American distribution that is well documented but it is not being pulled to the genus level:
https://sfg.taxonworks.org/tasks/otus/browse?taxon_name_id=468127

@JohnHeraty https://support.apple.com/en-us/HT201361

same as #1755


AD user does not have write access to virtio-fs

Describe the bug
While logged into a VM that uses an active directory login there is no write access to the drive.
To Reproduce
I'm not really sure...
Expected behavior
A clear and concise description of what you expected to happen.
Host:

Debian GNU/Linux bookworm/sid
uname -a-->  Linux ******* 5.18.0-2-amd64 # 1 SMP PREEMPT_DYNAMIC Debian 5.18.5-1 (2022-06-16) x86_64 GNU/Linux
qemu version 7.0.0
libvirt version 8.4.0-1
libvirt XML file

VM:

Windows 10 Enterprise 20H2 19042.1466
Component: virtio-fs
virtio source: virtio-win-0.1.164.iso

win10_virt-manager.zip
Additional context
On a VM I ported from VirtualBox, which uses an active directory corporate login, I do not have write access.
If I log in on this machine as a non-ad user, I have write access.
To evaluate who the guest /thinks/ is the owner of the network drive (Z:\ on my machines) I ran the command:
'C:\Program Files (x86)\WinFsp\bin\fsptool-x64.exe' perm Z:\

When logged in as a normal user I get ...(perm=197609:197121:0775) and as the AD-user machine: ...(perm=0:0:0775)
When logged in as the normal user fsptool id returns (redacted version):
'C:\Program Files (x86)\WinFsp\bin\fsptool-x64.exe' id
User=S-1-5-21-994256757-583842975-1750328797-1001(***\User) (uid=197609)
Owner=S-1-5-21-994256757-583842975-1750328797-1001(***\User) (uid=197609)
Group=S-1-5-21-994256757-583842975-1750328797-513(***\None) (gid=197121)

When logged in as the AD user I get:
'C:\Program Files (x86)\WinFsp\bin\fsptool-x64.exe' id
User=S-1-5-21-***88198(***) (uid=1136774)
User=S-1-5-21-***88198(***) (uid=1136774)
Group=S-1-5-21-***-513() (gid=1049089)

Simply logging into the broken win10 install as a non AD user results in a working Z:\ drive.
Note that when switching from standard user to AD user fsptool perm Z:\ continues to return the uid/gid of the std User, but I do not have write access for this either.

To diagnose the permissions issue I wanted to know what the userid was being communicated between host and guest. During this I found in virtiofs.cpp the function UpdateLocalUidGid.
I copied this code into a simple console application which can be found here
While logged in as the normal user I get a simple 0x303E9 result/
When logged in as the AD user and running the console program  NetUserGetInfo fails with Status == 2221
When logged in as the AD user and running the console program with admin rights I get 2221.
When logged in as the AD user and running the console program as the standard User I still get 2221.
In each of these runs, the same user name (the AD user's name) is returned by WTSQuerySessionInformation.

Hi @Daniel-Trevitz,
Thanks for you investigation of the issue and sample program you wrote.
But, unfortunately, ActiveDirectory is not supported at the moment. It is planned for the future.
So, if you understand how to fix the problem and have a patch you are welcome.
There are some information about how NetUserGetInfo works with ActiveDirectory: https://docs.microsoft.com/en-us/windows/win32/api/lmaccess/nf-lmaccess-netusergetinfo
NetUserGetInfo seems to be inappropriate to get AD user information.
As far as I understand, the problem is that with AD the service must properly setup SECURITY_DESCRIPTOR here:
https://github.com/virtio-win/kvm-guest-drivers-windows/blob/79927c3521c9aed36b7ba10586ee83abd0bd8eb4/viofs/svc/virtiofs.cpp#L1290

@viktor-prutyanov, Well that sure explains it! I just ran a check on the same VM image from virtualbox (pre when I converted the disc image over).
Though it is rather unfortunate that my example program in the other win10 AD enabled VM (same user, same permissions) works correctly - unfortunite as it's the one with visual studio... womp womp. I guess there is some difference between the two ... that I don't really care to investigate.
Are there written plans for the AD support?
Do you know of other parts in the virtio-fs or winfsp that'll need touched?

Just for the records; Err 2221 is NERR_UserNotFound --- "The user name could not be found"

@viktor-prutyanov, Is there any reason to not use WinFSP's FspPosixMapSidToUid? It definitely is a lot more code to run, but it also handles a lot of cases.
NTSTATUS FspToolGetTokenInfo(HANDLE Token,
   TOKEN_INFORMATION_CLASS TokenInformationClass, PVOID *PInfo)
{
   PVOID Info = 0;
   DWORD Size;
   NTSTATUS Result;

   if (GetTokenInformation(Token, TokenInformationClass, 0, 0, &Size))
   {
      Result = STATUS_INVALID_PARAMETER;
      goto exit;
   }

   if (ERROR_INSUFFICIENT_BUFFER != GetLastError())
   {
      Result = FspNtStatusFromWin32(GetLastError());
      goto exit;
   }

   Info = MemAlloc(Size);
   if (0 == Info)
   {
      Result = STATUS_INSUFFICIENT_RESOURCES;
      goto exit;
   }

   if (!GetTokenInformation(Token, TokenInformationClass, Info, Size, &Size))
   {
      Result = FspNtStatusFromWin32(GetLastError());
      goto exit;
   }

   *PInfo = Info;
   Result = STATUS_SUCCESS;

 exit :
   if (!NT_SUCCESS(Result))
      MemFree(Info);

   return Result;
}

int get_id(PSID Sid, UINT32 *Uid)
{
   PWSTR Str = 0;
   NTSTATUS Result;

   if (!ConvertSidToStringSidW(Sid, &Str))
   {
      Result = FspNtStatusFromWin32(GetLastError());
      goto exit;
   }

   Result = FspPosixMapSidToUid(Sid, Uid);
   if (!NT_SUCCESS(Result))
      goto exit;

   Result = STATUS_SUCCESS;

exit:
   LocalFree(Str);

   return Result;
}

int fsp_approach()
{
   NTSTATUS Result;

   HANDLE Token = 0;
   TOKEN_USER *Uinfo = 0;
   TOKEN_PRIMARY_GROUP *Ginfo = 0;

   if (!OpenProcessToken(GetCurrentProcess(), TOKEN_QUERY, &Token))
   {
       Result = FspNtStatusFromWin32(GetLastError());
       return 0;
   }

   std::cout << "Got Token\n";

   Result = FspToolGetTokenInfo(Token, TokenUser, (void**)&Uinfo);
   if (!NT_SUCCESS(Result))
      goto exit;

   std::cout << "Got TokenUser\n";

   Result = FspToolGetTokenInfo(Token, TokenPrimaryGroup, (void**)&Ginfo);
   if (!NT_SUCCESS(Result))
      goto exit;

   std::cout << "Got TokenPrimaryGroup\n";

   UINT32 UID, GID;

   get_id(Uinfo->User.Sid, &UID);
   get_id(Ginfo->PrimaryGroup, &GID);

   std::cout << "FSP UID: " << UID << "\n";
   std::cout << "FSP GID: " << GID << "\n";

exit:
   MemFree(Uinfo);

   if (0 != Token)
      CloseHandle(Token);

}

#791 seems to essentially work. When I start the process the correct UID/GID seems to be used as I now have write access.
However... If I start the process as a service the original problem manifests.

#791 seems to essentially work. When I start the process the correct UID/GID seems to be used as I now have write access.
However... If I start the process as a service the original problem manifests.

Sorry, do you mean virtiofs.exe process or the process that uses virtio-fs mount?
If  virtiofs.exe, that it looks that you need to take a look at impersonation https://docs.microsoft.com/en-us/windows/win32/api/securitybaseapi/nf-securitybaseapi-impersonateloggedonuser

@YanVugenfirer, When I start virtiofs.exe the log entry I added to state the detected UID/GID reads out the same value for both automatically running the service and manually running the service.
However, even though the localGID / localUID are now correct, when I start the service (either via boot or manually restarting) explorer.exe reports that it does not have access.

When I run the service now, I get the message
*** UpdateLocalUidGid: Failed to impersonate logged on user

static VOID UpdateLocalUidGid(VIRTFS *VirtFs)
{
NTSTATUS Result;
HANDLE Token = 0;
TOKEN_USER *Uinfo = NULL;
TOKEN_PRIMARY_GROUP *Ginfo = NULL;

if (!OpenProcessToken(GetCurrentProcess(), TOKEN_QUERY, &Token))
{
    VirtFs->LocalUid = 0;
    VirtFs->LocalGid = 0;
    DBG("Failed to open a process token");
    return;
}

if(!ImpersonateLoggedOnUser(Token))
{
    VirtFs->LocalUid = 0;
    VirtFs->LocalGid = 0;
    DBG("Failed to impersonate logged on user");
    CloseHandle(Token);
    return;
}

Result = FspToolGetTokenInfo(Token, TokenUser, (PVOID*)&Uinfo);
if (!NT_SUCCESS(Result))
   goto exit;

Result = FspToolGetTokenInfo(Token, TokenPrimaryGroup, (PVOID*)&Ginfo);
if (!NT_SUCCESS(Result))
   goto exit;

Result = FspPosixMapSidToUid(Uinfo->User.Sid, &(VirtFs->LocalUid));
Result = FspPosixMapSidToUid(Ginfo->PrimaryGroup, &(VirtFs->LocalGid));

DBG("local UID=%u, local GID=%u", VirtFs->LocalUid, VirtFs->LocalGid);

exit:
SafeHeapFree(Uinfo);
SafeHeapFree(Ginfo);
CloseHandle(Token);
}

I've attached two log files. Same process for both, run once from powershell, and once from the "start" service button in the service manager.
This has an implementation as the above comment, but without impersonate.
Started_as_Service.log
Started_From_Powershell.log

The log file clearly shows on line 3 that the UID and GID are different...  leading credence to needing to do the impersonation - but since that doesn't run...
I also tried to start the service as my AD user, but the service manager says I'm not allowed to do that. What a pain...

Ignore all my previous comments. WTSQueryUserToken did the trick. See PR #792.

Don't think if my issue is related to this one but when copying large game file from Windows guest to a shared drive (virtiofs) with linux host, I get error such as Permission denied to the folder.


acorn -> disposable, remove extra file

It looks like the commit name needs to be changed, build is failing


Design and implement an Organization page

Is your feature request related to a problem? Please describe.
When a user sign a petition, they may want to know who is promoting it, so we need to have a public page for an organization describing it.
Describe the solution you'd like
A simple webpage with the organization name and description with a list of petitions.
An example of what it could look like :

There is a public page for an organization, listing petitions: http://localhost:8000/petition/org/org-slug-name
It is accessible by clicking on the name of the org on the left of your own dashboard if you are part of the organization.
Petitions url also give a hint about the Organization who's created it: http://localhost:8000/petition/org/org-slug-name/petition-slug-title
But I think this URL is for now only used in the org dashboard and not on the org profile, that would need a fix.
Also I'm not sure petition slug name are completely implemented as this is part of #91
But maybe some other graphical hints would be nice to add to the petition template to show the Organization name

I think this is done, right?

Yes, what's left is in #135 (adding logo, description, website etc.)


How to best spring-clean old unused DDEV volumes

When I first started toying around with DDEV I was unaware of ddev remove --remove-data. I've just used ddev remove. That lead to a few relics. If I currently do a ddev list I only have a single site installed and listed (stopped at the moment):
NAME      TYPE     LOCATION          URL(s)                       STATUS
ddevdemo  drupal8  ~/Sites/ddevdemo  http://ddevdemo.ddev.local   stopped
                                     https://ddevdemo.ddev.local  sync active

DDEV ROUTER STATUS: exited
The router is not yet healthy. Your projects may not be accessible.
If it doesn't become healthy try running 'ddev start' on a project to recreate it.

reflected in a docker ps -a:
CONTAINER ID        IMAGE                            COMMAND                  CREATED             STATUS                       PORTS                                                                                NAMES
6826498715e9        drud/ddev-ssh-agent:v1.6.0       "/entry.sh ssh-agent"    3 days ago          Exited (255) 3 minutes ago                                                                                ddev-ssh-agent
c98a11382b9e        drud/ddev-webserver:v1.6.0       "/start.sh"              3 days ago          Exited (0) 3 days ago                                                                                ddev-ddevdemo-web
86b05f6826f7        drud/phpmyadmin:v1.6.0           "/run.sh phpmyadmin"     3 days ago          Exited (0) 3 days ago                                                                                ddev-ddevdemo-dba
67208da9b1c2        drud/ddev-dbserver:v1.6.0-10.2   "/docker-entrypoint.…"   3 days ago          Exited (0) 3 days ago                                                                                ddev-ddevdemo-db
8fc5fcb91595        drud/ddev-router:v1.6.0          "/app/docker-entrypo…"   3 days ago          Exited (255) 3 minutes ago   <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp, <IP_ADDRESS>:8025->8025/tcp, <IP_ADDRESS>:8036->8036/tcp   ddev-router
57b6a08be246        drud/ddev-bgsync:v1.6.0          "bg-sync"                5 days ago          Up 3 minutes (healthy)

but when I do a docker volume ls I get a list of the available and registered volumes which consists not only of the ones related to the ddevdemo project but also of projects only removed with ddev remove and not ddev remove --remove-data:
DRIVER              VOLUME NAME
local               Craft-mariadb
local               ddev-composer-cache
local               ddev-ddevdemo_unisoncatalogvol
local               ddev-ddevdemo_webcachevol
local               ddev-ddevdrupal_unisoncatalogvol
local               ddev-ddevdrupal_webcachevol
local               ddev-gutenspeed_unisoncatalogvol
local               ddev-gutenspeed_webcachevol
local               ddev-router-cert-cache
local               ddev-ssh-agent_dot_ssh
local               ddev-ssh-agent_socket_dir
local               ddevdemo-mariadb
local               ddevdrupal-mariadb
local               gutenspeed-mariadb
local               myd8site-mariadb

What would be the cleanest way for a spring-clean and removing unused volumes? A docker volume prune would be the appropriate choice or even a docker system prune to eliminate even more unused data related to older projects? And in case one of the two would be the right choice would it be reasonable to create a ddev command alias for that like e.g.ddev cleanup or at least leave a remark in the documentation about it (I could write a rough draft for that).

docker volume rm $(docker volume ls -q | grep mariadb)
will kill off the mariadb volumes.
If you want to kill the rest of any volumes created by ddev, docker volume rm $(docker volume ls -q | grep ddev)
Of course you'll want to docker volume ls before you do that.
While spring cleaning, you may want to remove any site folders from ~/.ddev (databases used to be stored there).
And if you have any containers showing that you don't want in docker ps -a, then docker  stop $(docker ps -q) && docker rm -f $(docker ps -aq). Again, do a docker ps -a first so you make sure you know what you're aiming at.

@rfay also thanks for the tips in here! those helped to save a few gigabyte ;))

yep cool thanks noticed the ddev delete images command already in the merged pull request.

I ran into issues today of not being able to restore a database. Kept getting MySQL ERROR 1114 (HY000) that tables were full. I had been using ddev import-db for to restore this site and database  successfully multiple times over the past few days. Database backup was nearly 700 Meg .sql file.
This is on a MAC and googling for answers clearly noted this was being caused by insufficient disk space but there was still 10GB free on my disk.
@rfay pointed me at the size of the docker image but it was already at the max 64GB
Ran docker system prune (but it did not free up anything)
The command docker volume ls showed something like 50 items for projects that I had created and deleted over the past two years.
I removed all of them using this command: docker volume ls -qf dangling=true | xargs docker volume rm
Success: Database now restores and I have an another 8Gb of free disk space

BTW, docker system prune --volumes would have been the way to kill off the volumes all automatically. And lose all databases everywhere.


Option to use local CDDD installation without server

This is a solution for #4  #7 #8 (and for me who wants to run a big benchmark).
Changes

Modified environment.yml to set up a conda environment with img2mol and CDDD decoder
added local_cddd parameter to the constructor of Img2MolInference class

if local_cddd is set to True, the local CDDD installation is used instead of sending a request to the server (it defaults to False so that the standard behaviour does not change)

The model files are expected to be in the site_packages directory of the conda environment so that img2mol can be imported and used from anywhere

I have changed the instructions for the deposition of the model files accordingly

Easier usage for the end-user::

The model path does not need to be specified anymore (it can, of course, still be handed as an argument to the constructor of Img2MolInference)
The CDDDRequest object does not need to be instantiated by the user (this is taken care of by the constructor of Img2MolInference if local_cddd==False)
The device does not need to be specified manually anymore (it defaults to "cuda:0" if torch.cuda.is_available() else "cpu", but it can also be handed manually to the constructor of Img2MolInference if desired)

Everything has been tested on Windows 11 (WSL) and Linux.

Hello Andreas,
Thanks a lot for your feedback! I am sorry that I am getting back to you so late! I hope everything is OK now. Let me know if you would like me to change anything else.
Have a nice day!
Otto


path conflicts with secondary source in Gemfile

See https://gist.github.com/harto/0dfcee489e248900feb8 for reproduction.
Suppose a Gemfile references one gem from a local path and another from a secondary source, e.g.:
source "https://rubygems.org"
gem "local_gem", path: "local_gem"
gem "other_gem", source: "https://gems.example.com"

If I bundle install, the dependencies are apparently resolved and installed correctly. However, the bundle fails at runtime:
$ bundle exec ruby -e 'puts "hello"'
Could not find gem 'other_gem (>= 0) ruby' in rubygems repository https://gems.example.com/.
Source does not contain any versions of 'other_gem (>= 0) ruby'
Run `bundle install` to install missing gems.

Removing local_gem and running bundle install again appears to fix the problem:
$ bundle exe ruby -e 'puts "hello"'
hello

The local_gem doesn't have any dependencies. Again, see https://gist.github.com/harto/0dfcee489e248900feb8 for a minimal complete example.

Confirmed in RubyGems 1.8.0 and 1.8.2.

It breaks our service: https://github.com/rails-assets/rails-assets

Same here.
I found reverting https://github.com/bundler/bundler/commit/9263e08636d25e2573776b17416e6fabd403026b solves the issue in my environment.

Same here.
I found reverting https://github.com/bundler/bundler/commit/9263e08636d25e2573776b17416e6fabd403026b solves the issue in my environment.

I have a proposed fix at #3427. If fixes the test cases here and in #3417, but if you could try it on your real-world projects too, that would be awesome.

This should be fixed in Bundler 1.8.3 (released today).

I tried Bundler 1.8.3, but the same error still occurs in my environment.

@eagletmt can you create a new issue and include the output from bundle env?

I'll submit a new issue after a bit more investigation.

@TimMoore submitted #3430


email to nick

we should probably figure out #1 before sending this

I added some stuff to the email
@effincleveland: thoughts?
we should probably send this out p soon, but you should have a chance to edit first

sent


Springsandwich not in maven central

Springsandwich is a great library, and adding a dependency to it in a local development environment is fairly simple. However, adding a dependency to Springsandwich in a way which allows for the project to be built via Continuous Integration or Heroku is more complicated. Adding Springsandwich to maven central would simplify both local and CI/Cloud setup.
Instructions for deploying to Maven Central can be found at https://maven.apache.org/guides/mini/guide-central-repository-upload.html
(Also note http://central.sonatype.org/pages/ossrh-guide.html)

A short term solution could be to add details to use jitpack.io. Such as https://jitpack.io/#arikast/springsandwich/-SNAPSHOT

I would love to use this from Maven Central.  That's my main blocker to including it in my project.

Thanks for the suggestion, finally got it on maven central.

com.kastkode
springsandwich
[1.0.2,)


create special option to deal with used barcodes with GU

Eterna allows GU base pairs in the stem, meaning that the 2min_edit is no longer guarantees if you only look at one side of the stem. Add option which would check both sides of the stem instead of just the one.

done 8d52b22


app router support?

does the package support the new next.js app router?

It should work out of the box.

No it is failing, because you are using next/router, but that is not supported in app-router,(https://nextjs.org/docs/messages/next-router-not-mounted)

Lets checkout version1.0.9


Add the CURL run button on the Client page TEST

On the /check-client page, exemplify the CURL command. An improvement would be to add a RUN button that already executed this command on the screen and showed the result.

Hi, I am interested in assisting you but I'm relatively new to the field and do not fully understand your issue. I will of course look it up online but I may need some guidance. However, if you require a helper already familiar with the issue then I wish you all the best.

Hi @hasanosman2020 👋🏻 ! Welcome.!
Let's try to tackle this.! Are you new to mTLS field or NextJS? If you are able to run the project locally, that's a good start. Have you already done that?
The details about mTLS and stuff we can guide you throughtout the issue development.

Hi @tiagostutz. I am new to both mTLS and NextJS. I haven't worked with either before but. have googled them and am confident that I can get to grips with both. I encountered a few issues when trying to run the project but I think that I have managed to debug them. I have taken a screenshot of the project after running yarn dev - is that what is supposed to show on screen?

Thanks
Hasan

Right, I have made the full path here cloning this repo from scratch, running yarn install and yarn run dev from the app folder and it worked. So we have to dig a little deep in your scenario to find out whats happening.

What NodeJS version are you using (run node -v in the same terminal you are running yarn run dev`)
Have you cloned from your fork or from this original repo (we recommend cloning from your fork)
Double check if you ran yarn install at the app folder and if the node_modules folder are there
Those CSS files pointed as missing are indeep in the folder?

Hi @tiagostutz - I believe that I have the project running locally. I did what you suggested - I forked the repo then cloned from my fork and I installed and ran yarn dev from the app folder. Thanks for your help. I have attached a screenshot of running yarn dev.  So I'm good to go.

Hi @hasanosman2020 are you still interested in tackling this issue?

Hello,
Please check it out my PR and see if it solve this problem:
https://github.com/bancodobrasil/mtls-best-friend/pull/13
Thanks


vite: externalize getbindingsproxy from cloudflare preset

TODO

[x] docs
[x] changeset

A couple of minor wording suggestions but nothing blocking 👍


Make self voicing for discussion and calls more usable

Currently the self-voicing from renpy will attempt to read everything every time, so for each new line of dialogue said it will read the discussion name and every single message in the screen. And for calls it will read the name of whoever is calling you + the time of the call every single time.

been a hot minute eh?
i won't merge this as i have some heavy changes in mind that would break this pr. i'm gonna close this and open an issue instead, so that i could tackle this on my own


Render Unto Caeser (Swift & C)

Hi Chris,
Apologies in advance as this is less an issue with your code, and more a general question (that wouldn't fit into a tweet).
I'm currently working my way through your Core Audio book, and recently discovered your "Media Frameworks and Swift: This Is Fine" talk from Forward Swift.
I was pleasantly surprised to discover I had stumbled into the approach you refer to "Render unto Caesar", but was wondering if you could clarify something for me?
Is there any reason I can't write my Audiounit in Swift, with the exception of the render-block which would call into a seperate C function?
Something like:
class MyAudioUnit {
  ...

  func internalRenderBlock() -> AUInternalRenderBlock {
     // render audio via my_external_c_function() here
  }
}

Cheers.

Nevermind.
I found your blog post (http://subfurther.com/blog/2017/04/28/brain-dump-v3-audio-units/), which was a huge help. Thanks!


Validation Layer At Model Level

Validator layer idea ->
Catch fields without defaults not being filled in.
Catch things you thought were numeric being filed in as string
Catch things you thought were integers that are actually decimal numbers
Catching strings that are longer than you made space for in the varchar
fields in the JSON that are not in the schema???
top level (i.e. not a subarray) that is not being filled in… generate a warning?
Cannot pass null to a non-nullable value.

When running from command-line, this is what the validator should output on bad data.

Works perfectly.


The fill for warning/citations goes beyond the rounded ends of the pill

 Raise new issues resulting from using/testing the apps


Fix/111/everything is broken

@midik I don't know if this is ready as you don't have asked for review, but I tested and two times, the whole process (npm i && npm start) get stuck at:
2020-07-23 07:26:18 [info]: 	Gathering sim links...
 ████████████████████████████████████████ 100% | ETA: 0s | 2555/211 | tr / wave-on-a-string
2020-07-23 07:32:22 [info]: 	Getting documents and images...
 ███████████████████████░░░░░░░░░░░░░░░░░  57% | ETA: 2675s | 1504/2638 | wave-on-a-string_gl.html

@kelson42 was there any warnings above?

@midik Nothing

@kelson42 could you please give it another try?


[Design Specs] AlertDialog

@turolopezsanabria currently working on a similar topic (Modal/Dialog). She will continue working on this ticket

you are right, sorry.
Close as duplicated


[1.1.0-preview] Recommendation for NestedModules?

Now that the extension vault will run in a runspace, should I stop including it in nestedmodules in the manifest? My concern is that my module will call my internal version without awareness of the runspace context, and I need to be clear to explicity call microsoft.powershell.secretmanagement\get-secret for example.
To be more clear, I'm asking if SecretManagement should require it to be a nestedmodule, because today it won't work unless it is a nested module and may lead to the ambiguous/wrong module commands being invoked.

Good question.  Yes, SecretManagement does continue to require that the extension vault module implements the vault functions ('Get-Secret', 'Set-Secret', etc.) from within a nested module with the VaultModuleName.Extension name.  So the form of the extension vault module does not change.  This is mentioned in the docs, but maybe it can be made more clear.
This is to hide the extension vault functions from the user.  Extension vault modules can still be imported into a user session, and the vault functions should never be exposed publicly.  As an example, both Microsoft.PowerShell.SecretStore and SecretManagement.KeePass vault modules have public commands and will be loaded into the user session if the commands are invoked.  So the nested module is still needed to hide the vault functions.
It is always a good idea to use fully qualified names in script, just to be sure there is no ambiguity.  But a user should see only one implementation of Get-Secret when running Get-Command Get-Secret.
PS> Get-Command Get-Secret

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Cmdlet          Get-Secret                                         1.1.0      Microsoft.PowerShell.SecretManagement

PS> Get-Module

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Binary     1.1.0      preview    Microsoft.PowerShell.SecretManagem… {Get-Secret, Get-SecretInfo, Get-SecretVault, Reg…
Binary     1.0.2                 Microsoft.PowerShell.SecretStore    {Get-SecretStoreConfiguration, Reset-SecretStore,…

BTW, when testing the v1.10 change for SecretManagement, I updated my copy of SecretManagement.KeePass to work with the hosting change.  This is how I implemented the Unlock-KeePassSecretVault function:
function Unlock-KeePassSecretVault {
<#
    .SYNOPSIS
    Enables the entry of a master password prior to vault activities for unattended scenarios.
    If registering a vault for the first time unattended, be sure to use the -SkipValidate parameter of Register-KeepassSecretVault
    .EXAMPLE
    Get-SecretVault 'MyKeepassVault' | Unlock-KeePassSecretVault -Password $MySecureString
    .EXAMPLE
    Unlock-KeePassSecretVault -Name 'MyKeepassVault' -Password $MySecureString
#>
    param (
        [Parameter(Mandatory)][SecureString]$Password,
        [Parameter(Mandatory,ValueFromPipelineByPropertyName)][String]$Name
    )

    Microsoft.PowerShell.SecretManagement\Unlock-SecretVault -Password $Password -Name $Name
}

Right, that's what I'm working on, and will in fact deprecate this command to just use Unlock-SecretVault directly.


eth_getCode cache

Description
We are caching the contract's bytecode which is acceptable in most cases because the bytecode is immutable and can not be changed. The problem appears when the user calls selfdestruct the the bytecode must be deleted, and if the end-user calls eth_getCode again, the cached response will be returned, which is not accurate and not corresponding to the state.
Steps to reproduce

Deploy a contract with selfdestruct via hardhat example
Get the contract bytecode via ethers.provider.getCode(<contractAddress>) which will call eth_getCode
Call the selfdestruct function
Get the contract bytecode again - the returned value should be 0x because the contract has been deleted

PS: to make sure that selfdestruct is called, you can transfer a hbars to the contract beforehand and check the balances before/after the destruction
Solution

[ ] Remove the cache from eth_getCode
[ ] Add test

Additional context
No response
Hedera network
other
Version
latest
Operating system
Linux

Hmm removing the cache doesn't seem preferable.
Are there other options, can we search the byte code for the SELF DESTRUCT opcode.
May we cache less or don't cache those cases.
Could we combine this logic with an eth_call request to tell us if SELF DESTRUCT has been called?

We can disassemble the bytecode to opcodes and find the selfdestruct, but this will work only for contracts without delegatecall or callcode opcodes in it.
Let's take a look over this case :point_down:
//SPDX-License-Identifier: Unlicense
pragma solidity ^0.8.0;

contract A {
	function destroy(address B) public {
		B.delegatecall(abi.encodeWithSignature("deleteContract()"));
	}
}

contract B {
	function deleteContract() public {
		selfdestruct(payable(msg.sender));
	}
}

We're deploying contracts A and B, the A bytecode doesn't contain selfdestruct. We call A.destroy() which copies the B.deleteContract() opcodes into A and contract A will be deleted. In this case, we can not cache contract A even if it's not containing selfdestruct opcode.
And another example :point_down:
contract A {
    function callB(address _B, address _C, address _D) public {
        _B.call(abi.encodeWithSignature("callC(address,address)", _C, _D));
    }
}

contract B {
    function callC(address _C, address _D) public {
        _C.delegatecall(abi.encodeWithSignature("callD(address)", _D));
    }
}

contract C {
    function callD(address _D) public {
        _D.call(abi.encodeWithSignature("destroy()"));
    }
}

contract D {
    function destroy() public {
        selfdestruct(payable(msg.sender));
    }
}

We can:

cache A and C because they are not containing selfdestruct || delegatecall || callcode opcodes
not cache B because even if it's not containing selfdestruct we cannot be sure that passed _C will not contain selfdestruct or another call
not cache D because can be deleted

So I propose to check the contact's bytecode for opcodes and cache the contract only if it doesn't contain one of selfdestruct || delegatecall || callcode opcodes. If the bytecode contains one of the opcodes, we have to query the mirror node for deleted status every time.

Thanks for the considerations @natanasow
I guess additionally we should also cache if a certain bytecode contains one of those codes that needs an extra mirror node call, that way the relay doesn't unnecessarily do the same logic multiple times for the same contract.


Initial Infected see each other before the infection

About the PR

Initial Infected can see the zombie icons near other Initials.
Why / Balance

I feel like zombies need a slight buff after security/command became exempt from II antag role.
If this is not enough, I can also add buffs to zombified Initial Infected in order to make snowballing a bit easier.
Technical details

Media

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

Breaking changes

Changelog

:cl:

tweak: Initial Infected can now identify each other before the outbreak.

needs testing, apparently you cannot make someone II through admin menu, only make them zombies immediately
if someone can tell how to grant II antag role in dev environment, I would be glad

edit the zombies preset to work with 1 player then forcepreset zombies

Can zombies see the initial infected icon? Because this might lead to zombies avoiding biting initial infected

Can zombies see the initial infected icon? Because this might lead to zombies avoiding biting initial infected

There is no II icon, it is the same zombie icon, so the answer is yes.
however, I think this is alright, because II are already irreversibly zombified. Lorewise zombies can probably feel it.

media?

much needed buff to zombies, after so many bad nerfs

media?

is it needed? It is just the same zombie icon

Converted to draft, since it is currently showing zombies to II, not other II.
Perhaps a separate icon for II is needed?

its pretty easy to see zombies and ii would know more than others

Adding ability to fake zombie icon by washing yourself in zombie blood sounds... awzum... idea for future changes i guess

Closing this for now, since my main laptop is now in repairs.
If this won't get inplemented, I will probably reopen with some other zombie tweaks.

Any word on this? People are itching for zombie buffs but they're just throwing out stat tweaks.

Any word on this? People are itching for zombie buffs but they're just throwing out stat tweaks.

Yeah, my laptop is back, planning to remake this PR today/tomorrow


RuntimeError: Function ViewBackward returned an invalid gradient at index 0 - expected device cpu:0 but got cpu

im trying to run the project on other machine with cpu then this shows up i change config file
parser.add_argument(
        "--cuda",
        action="store_false",
        default=Fales,
        help="by default True, will use GPU to train; " "or else will use CPU;",
    )

still not working
how can i fix it
Average episode rewards is -153.109 	Total timesteps: 195200 	 Percentage complete 3.904
Traceback (most recent call last):
  File "onpolicy/scripts/train_mpe.py", line 315, in <module>
    main(sys.argv[1:])
  File "onpolicy/scripts/train_mpe.py", line 300, in main
    runner.run()
  File "/home/lab688/work/pythonproject/kinformarl/InforMARL-main/onpolicy/runner/shared/graph_mpe_runner.py", line 76, in run
    train_infos = self.train()
  File "/home/lab688/work/pythonproject/kinformarl/InforMARL-main/onpolicy/runner/shared/base_runner.py", line 168, in train
    train_infos = self.trainer.train(self.buffer)
  File "/home/lab688/work/pythonproject/kinformarl/InforMARL-main/onpolicy/algorithms/graph_mappo.py", line 333, in train
    ) = self.ppo_update(sample, update_actor)
  File "/home/lab688/work/pythonproject/kinformarl/InforMARL-main/onpolicy/algorithms/graph_mappo.py", line 246, in ppo_update
    critic_loss.backward()
  File "/home/lab688/work/anaconda3/envs/informarl37/lib/python3.7/site-packages/torch/tensor.py", line 245, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/home/lab688/work/anaconda3/envs/informarl37/lib/python3.7/site-packages/torch/autograd/__init__.py", line 147, in backward
    allow_unreachable=True, accumulate_grad=True)  # allow_unreachable flag
RuntimeError: Function ViewBackward returned an invalid gradient at index 0 - expected device cpu:0 but got cpu

Could you try after fixing the typo "Fales" to "False"?

Gotcha! Let me take a look at this. Can you maybe share the command you are running to execute the code?

here is commend i use :
(my_conda_env)>bash informarl.sh

I think informarl.sh is a deprecated script.  Maybe we can have a chat.

@nsidn98 HI sorry to bother you again
Do you use tensorboard to generate the graph diagram of the architecture in your code? I would like to use this to understand, or where can I add the relevant code to generate the architecture diagram?

ummm are you there?? sir??

ummm are you there?? sir??
@nsidn98

@s1101818 apologies for the late reply! I will go through this during the next week and try to find a fix. We do not use tensorboard but we use wandb. You can visualise the model like this

Thank you!!
I don't know much about add_graph() but I will also try wand, can wandb display a graph of a model?
Anyway, I'll come back here again if I have any other questions.
I appreciate that you made such an interesting model.

I am glad that you found the project interesting! I will get back soon with a fix for this.
Yes, wandb allows you to visualise the graph. You can also use tensorboard instead of wandb if you are more comfortable with using it.

i want to decrease the train step let wandb out put the model graph at windows what number should i change ??
i try set the num_env_steps from 200000 to 200
python -u onpolicy/scripts/train_mpe.py --use_valuenorm --use_popart \
--project_name "informarl" \
--env_name "GraphMPE" \
--algorithm_name "rmappo" \
--seed 0 \
--experiment_name "informarl" \
--scenario_name "navigation_graph" \
--num_agents 3 \
--collision_rew 5 \
--n_training_threads 1 --n_rollout_threads 128 \
--num_mini_batch 1 \
--episode_length 25 \
--num_env_steps 200 \
--ppo_epoch 10 --use_ReLU --gain 0.01 --lr 7e-4 --critic_lr 7e-4 \
--user_name "<INSERT_YOUR_WANDB_NAME>" \
--use_cent_obs "False" \
--graph_feat_type "relative" \
--auto_mini_batch_size --target_mini_batch_size 128
~

Traceback (most recent call last):
  File "onpolicy/scripts/train_mpe.py", line 315, in <module>
    main(sys.argv[1:])
  File "onpolicy/scripts/train_mpe.py", line 289, in main
    runner = Runner(config)
  File "/home/lab688/work/pythonproject/K_Informarl_retry/InforMARL-main/onpolicy/runner/shared/graph_mpe_runner.py", line 24, in __init__
    super(GMPERunner, self).__init__(config)
  File "/home/lab688/work/pythonproject/K_Informarl_retry/InforMARL-main/onpolicy/runner/shared/base_runner.py", line 66, in __init__
    self.save_dir = str(wandb.run.dir)
AttributeError: 'NoneType' object has no attribute 'dir'
new_test20.sh: line 2: --project_name: command not found
new_test20.sh: line 3: --env_name: command not found
new_test20.sh: line 4: --algorithm_name: command not found
new_test20.sh: line 5: --seed: command not found
new_test20.sh: line 6: --experiment_name: command not found
new_test20.sh: line 7: --scenario_name: command not found
new_test20.sh: line 8: --num_agents: command not found
new_test20.sh: line 9: --collision_rew: command not found
new_test20.sh: line 10: --n_training_threads: command not found
new_test20.sh: line 11: --num_mini_batch: command not found
new_test20.sh: line 12: --episode_length: command not found
new_test20.sh: line 13: --num_env_steps: command not found
new_test20.sh: line 14: --ppo_epoch: command not found
new_test20.sh: line 15: --user_name: command not found
new_test20.sh: line 16: --use_cent_obs: command not found
new_test20.sh: line 17: --graph_feat_type: command not found
new_test20.sh: line 18: --auto_mini_batch_size: command not found

it seems not working
how do i do?

Hi, is it possible to run some inference?
I think you just wrote the prompt of training and not the inference.
Thanks

Hi, is it possible to run some inference? I think you just wrote the prompt of training and not the inference. Thanks

Hi, thanks to @nsidn98, I got the test prompt, I will put it below,
Example usage: python onpolicy/scripts/eval_mpe.py  --model_dir='YOUR_PATH_TO_SAVED_WEIGHTS_FOLDER' --render_episodes=1 --world_size=1 --num_agents=<NUM_AGENTS> --num_obstacles=<NUM_OBSTACLES> --seed=0 --num_landmarks=<NUM_AGENTS> --episode_length=25 --use_dones=True
Keep in mind, "--model_dir" refers to the "critic.pt" and "actor.pt" which are in the "\onpolicy\results\GraphMPE\navigation_graph\rmappo\informarl\wandb\run\files" path.
Another thing to say is the output which looks like the image I will attach,

based on the image provided,
the model was executed for 1 episode, got a reward of 268.5, the episode was completed in ~0.27*25 steps (25 is the horizon of the episode) and 2 collisions happened in the episode


Question: Why so many ids?

Concerning the CSS, I would like to understand, why almost everything is handled via ids and why those ids are are cascaded?
To my understanding, an id must only appear once per page, so things like
#amplitude-left #player-left-bottom #control-container #repeat-container #repeat

should never be necessary, because there is only one #repeat on a page, if the CSS is formed according to W3C rules.
Wouldn't it be a lot easier to structure the CSS according to BEM? Using a prefix like, e.g., amp- would make the classes pretty much unique, and specificity could be kept very low, so browsers would render even faster...
Thanks for your response!

I'll definitely be looking into this. I'm not too good at CSS so I'll be researching some BEM stuff. Thanks!

Can I be of any support? Would it be of any help if I shared my version of your player? E.g. via a codepen? Or re-style your examples the BEM way?

If you have an example of a BEM styled player, that'd be great! I have another example I want to use, but want to make sure it's ready to rock and roll before I push it

Cool! Looking forward to it! I'll share mine soon on Codepen!
*Robert M Wildling  **||  **web developer  |  *composer  |  music
engraver  |  3D
Göllnergasse 19/30 || A-1030 Vienna
ph: +43<PHONE_NUMBER>  ||<EMAIL_ADDRESS>https://soundcloud.com/rowild
http://www.amplify.at/projects/wildling/wildling_main.htm
2017-09-28 21:28 GMT+02:00 Dan Pastori<EMAIL_ADDRESS>
If you have an example of a BEM styled player, that'd be great! I have
another example I want to use, but want to make sure it's ready to rock and
roll before I push it
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/521dimensions/amplitudejs/issues/198#issuecomment-332939668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANDK3-lKWx1U3LD6EfikELhuEekqoIkks5sm_NfgaJpZM4PMFEA
.


Inline build is missing slate font data

When building with --inline, there is no sideloaded font for slate.ttf, wf, ... These are referenced back to the source folder. So maybe a) upload the font somewhere and sideload it in the html file, b) use a generic font that is present on all systems (looks ugly) or c) use a already well used font like googles robo or whatever and sideload this font.

Any progress or decision on the font issue when using --inline?

Not as yet. PRs welcome. One option is to use the github repo as a source.

In progress, --fonturl option will be present in the next release. Re: your "looks ugly" comment, note that the slate font only includes the search (magnifying-glass) and warning icons. If you want to specify a different font family, this can be done with the customcss option.

Wow that is awesome. Thx dude. You are the best.

Thanks for the offer of testing - the new option is included in v2.2.1-2

Please chase if not fixed.


Incorporate caching and other improvements into apps

Theo has developed some improvements to the way we're handling data in our apps. These include:

caching - using caching module to not have to generate plots that have already been generated (functools.lru_cache)
s3 mounting using goofys - only download as required
use of iris v2

investigating multiprocessing I have found 2 bugs,
iris v2  - (issue link to be added when created)
bokeh server calbacks  - https://github.com/bokeh/bokeh/issues/4054

No longer relevant to latest implementation of FOREST.


radicale_infcloud out of date

The current version is incompatible with Radicale 3.0.x
Checklist

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name: nix search radicale_infcloud (What is expected here?)

current version: 2017-07-27 (972757bf4c6be8b966ee063e3741ced29ba8169f)
desired version: 2020-08-07 (40f88a5dc0003120d62533ea635f0d94d77f1aca)
Notify maintainers

maintainers: @aneeshusa
Note for maintainers
Please tag this issue in your PR.

They've added a dependency on Radicale which I just removed in my overlay, but there might be a better way:
diff --git a/setup.py b/setup.py
index 0e1bf2a..c3bae30 100755
--- a/setup.py
+++ b/setup.py
@@ -23,4 +23,4 @@ setup(
     platforms="Any",
     packages=["radicale_infcloud"],
     package_data={"radicale_infcloud": web_data},
-    install_requires=["radicale>=3.0.0"])
+    install_requires=[])

Resolved by #191333


Throwaway variable triggers SA1312 (Variable names should begin with lower-case letter)

If the underscore throwaway variable name _ is used I don't think it should trigger this rule, i.e:
        private void EnsureExists()
        {
            var _ = Attributes;
        }

The discard syntax does not require the var keyword:
_ = Attributes;

Duplicate of #3057

facepalm Thanks :)


Rendezvous server doesn't close websockets cleanly

It should send a websockets close handshake with a status code, as per:  https://tools.ietf.org/html/rfc6455#section-7.1.2
The RFC seems to imply that it's optional to do so, but I'd welcome the change, as other websocket libraries assume a connection that TCP closes without the handshake has closed uncleanly, and raise awkward exceptions.

This is on the server side, right? Not the client side? I've observed lingering TCP connections on both the rendezvous and transit servers (just using netstat), but I haven't been able to track them down.
Can you tell if rendezvous_websocket.py is failing to call some necessary Autobahn API, or is there something going wrong inside Autobahn itself?

Yes, server-side. I can't rule out bugs in Autobahn, and I'm no expert in its use, but probably rendezvous_websocket.py needs to call sendClose. This appears to trigger an internal state change in the websocket protocol which (I'm guessing) triggers a TCP close.

Hm. It looks like the server never intentionally closes a connection: it lets the client decide when the conversation has finished. The CLOSE message (client->server) delivers the "mood", and unsubscribes from the mailbox, but doesn't affect the websocket connection. (OPEN/CLOSE refer to mailboxes and mailbox subscriptions, not the underlying connection).
Where are you seeing the awkward exceptions?

Sorry, you said server-side. Could you paste one? I'm not seeing any in the logs of the production rendezvous server. Or are you saying that a client is getting into an awkward state when the TCP connection is dropped without the right websocket code appearing first?

Actually, I think this is completely invalid. The client should probably call sendClose, but it's not a serious bug.


Add a simple issue template for epics

A simple structure:

Summary
References (links)
Alternatives Considered
Acceptance Criteria

Does this epic have any performance impact?
Does this epic have telemetry?

Some ideas from @barklund for adding to such a template:

Is lazy-loading possible?
Does this include new third-party libraries and are they optimal?
Does this introduce new interdependencies between packages?

Does this introduce new interdependencies between packages?

1 on this! There's been so much work to untangle things lately it would be a shame to slide back into it.

I'd also consider asking if there should be a feature flag/experiment associated with the epic.

Does this introduce new interdependencies between packages?

1 on this! There's been so much work to untangle things lately it would be a shame to slide back into it.

FWIW, I think once we've removed the last story-editor dependency from the dashboard package, this becomes mostly irrelevant.


Docker Tag 1.46

Hi Chris,
the "latest" tag exists but 1.46 is missing in docker hub.
10x!

updated


5.7 toolchain from 5. July reports new "function call causes an infinite recursion" warnings for deprecated methods in swift-nio

Describe the bug
The Swift 5.7 toolchain from 5. July for Xcode and Ubuntu produce 7 new warnings:
/src/Sources/NIOCore/EventLoopFuture+Deprecated.swift:19:21: warning: function call causes an infinite recursion
        return self.flatMap(callback)
...

Method definition looks like:
    @inlinable
    @available(*, deprecated, message: "Please don't pass file:line:, there's no point.")
    public func flatMap<NewValue>(file: StaticString = #file, line: UInt = #line, _ callback: @escaping (Value) -> EventLoopFuture<NewValue>) -> EventLoopFuture<NewValue> {
        return self.flatMap(callback)
    }

All methods can be found here: https://github.com/apple/swift-nio/blob/f1d62128c4587a06bee9f5455eef282694b71288/Sources/NIOCore/EventLoopFuture%2BDeprecated.swift#L16-L60
Steps To Reproduce

Clone swift-nio from https://github.com/apple/swift-nio.git
open Package.swift with Xcode and build the package (or just NIOCore)

Or with docker:
git clone https://github.com/apple/swift-nio
cd swift-nio
docker run --rm --volume "$(pwd)/:/src" --workdir "/src/" swiftlang/swift:nightly-5.7-focal swift build

Expected behavior
No warnings should be emitted
Screenshots

Environment (please fill out the following information)

OS: 13.0 Beta (22A5266r)
Xcode Version 14.0 beta 3 (14A5270f) with Swift 5.7 Development Snapshot 2022-07-05 (a)

or Docker with swiftlang/swift:nightly-5.7-focal on arm64
Additional context
Didn't produce these warnings in previous 5.7 snapshots
Other issues new in the 5.7 2022-07-05 snapshot:
https://github.com/apple/swift/issues/59910
https://github.com/apple/swift/issues/59911
https://github.com/apple/swift/issues/59913

@xedin Did something change in overload resolution here? It appears we're picking the default arguments overload, which causes the infinite recursion.

I'm not aware of any ranking changes but I think this is could be related to the recent changes @DougGregor landed for @preconcurrency since all these flatMap* methods have an overload @preconcurrency+@Sendable i.e.:
struct Future<T> {
  func flatMap<NewValue>(_ callback: @escaping (T) -> Future<NewValue>) -> Future<NewValue> {
    fatalError()
  }

  @inlinable
  @preconcurrency
  func flatMap<NewValue>(_ callback: @escaping @Sendable (T) -> Future<NewValue>) -> Future<NewValue> {
    fatalError()
  }
}

extension Future {
  @inlinable
  @available(*, deprecated, message: "")
  func flatMap<NewValue>(file: StaticString = #file, line: UInt = #line, _ callback: @escaping (T) -> Future<NewValue>) -> Future<NewValue> {
    return self.flatMap(callback)
  }
}

The specific example is something like this:
struct Future<T> {
  @inlinable
  @preconcurrency
  func flatMap<NewValue>(_ callback: @escaping @Sendable (T) -> Future<NewValue>) -> Future<NewValue> {
    fatalError()
  }
}

extension Future {
  @inlinable
  @available(*, deprecated, message: "")
  func flatMap<NewValue>(file: StaticString = #file, line: UInt = #line, _ callback: @escaping (T) -> Future<NewValue>) -> Future<NewValue> {
    return self.flatMap(callback)
  }
}

The issue here is that, when calling the @preconcurrency version of flatMap from the deprecated one, there is an argument conversion that adds @Sendable, and that makes the @preconcurrency version worse than the deprecated one. We have ideas to address this.

Fixed in the latest 5.7 toolchain from July 17, 2022


Two Issue i am facing

I am using native base this textinput inside native base content. It raises two issues
(1) When i backspace or clear all it scrolls to top
(2)I have to click two times to get the value.
Can anyone please help on this !!!

Were you able to fix this @AarshApple ??


pull request for e2e test at 11/25/2016 19:56:04

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add another comparison: AdamW on the language modeling

Nice Work! But I wonder how the AdamW performs on the same settings.

Hi,
Sorry for the late reply. In our Table 2 and Figure3, we compare with Adam and its variants on language modeling. Since in all LM experiments, we used weight-decay=0.0, Adam is equivalent to AdamW.


Add overloads for mock functions, fully expand out d.ts file

Here's a potential adjustment to the API and code to allow for overloads with to support a positional import.meta.url.
Making the PR to your branch to not duplicate things; I want to see if CI passes (they do locally).

Darn, I guess CI won't run since this isn't targeting the master branch.

Let me retarget this at master (temporarily) to check CI, but 7651cf6 (#122) is the actual change.

I'm making the assumption that err is not a part of the public API. I would like to make that same argument about opt, if possible, but I wasn't sure either way.

GitHub Actions was erroring so my windows runs got cancelled :(

Verified that this works on my project with "esmock": "jakebailey/esmock#parent-param" and adding the parameter.

This looks great. The adapter function is a great idea and the comments are great. Should we merge it?

I'm afk starting now btw so might not see any response for awhile

If you're happy with it sure. My remaining questions are:

Is opt part of the API, or just used to call esmock internally with extra options? If so, I'd like to drop it from the signature if possible. If it is part of the public API, then I'll fix the d.ts file to be accurate.
Are you happy with the parameter names? The ones that used to be in the d.ts file were more clear to me, but the implementation uses different ones (even in other files). I went for consistency then added a big comment instead.
And a bigger question; if you don't want to transpile esmock itself from TypeScript, would you be open to using inline comments for types and enable type checking? It's hard for me to be confident that what I'm writing is correct; I think types would help, but I don't know what your dev workflow is.

RE: types, I can totally just ignore that for this PR and send a PR showing an example another day, so really, just the first two questions are what I need. 😄

Supporting opts allows esmock to be called with many options at once and it allows future options to be added easily (for me). From my perspective, opts is the main way to control esmock and the "p" and "px" functions are sugar for calling esmock with common predefined opts values.
It would be good for esmock to use the same namings everywhere and it will happen soon, I believe, but until then I'm okay with different names being used.

Cool, thanks. I'll go inspect everything for some opts to document, then.

You're PR re-exports types definitions from inside the "tests" folder so they aren't duplicated (just noticed this) and the type definitions are more specific and better than the previous ones. Great improvements.

I've added the options type.
Unfortunately, though, I'm not sure that I can continue to use esmock in my project; it turns out that ESM loaders aren't compatible with coverage gathering due to various issues (https://github.com/bcoe/c8/issues/325); in my case, I thought that my project had been 100% tested, when in fact there are whole files I forgot to test. Not adding the esmock loader lets me see that in my testing; presumably this is because the mocked version of that module is loaded, the coverage sees that it was used, and so gets 100%, when all that happened was that it was mocked.
There is another (competing?) ESM mocker called mock-import, which can be used with another coverage program by the same author, but while it does work fine enough to mock things (though worse than esmock in that it cannot handle mocking in parallel), it also has issues with coverage.
I'm not sure that this is in scope of esmock at all, unless there's a way to load mock files as completely different paths internally (but correctly set their import.meta.url to where they would be); something I might look into.

CI failure is a flake; my commit only changed the types, so this should me mergable.

That's too bad about the CI coverage. Things are looking pretty good here. If you recommend any links with relevant info, they could be added to the README or wiki here to give people a warning about the coverage issue.
Thanks for hanging out here and making a great contribution.

oh of course you did give a link I'll read through that thanks

I hope I'm not doing anything rude by merging this. I believe you are sharing it for merge so I'm going to hit merge. There are few changes coming over the next week or two and it would be great to get this now.

Yes, I switched this from a draft on purpose! Thanks for merging this.


Fix brew info --json regressions around install status

#14587 unfortunately introduced some notable regressions to brew info --json:

Installed version tracking stopped working, including the outdated boolean (quite popularly used in scripts).
There was an API break where "token" was missing from casks and "name" was missing from formulae.

This PR:

Adds the local install info & the token to the cask output hash.
Largely reverts the changes to Formula#to_hash to instead take from the local state (our Formulary work means this works fine), except for oldname and requirements. In a future 4.0.x release, this will likely go away in favour of reading that data on the Formulary side.
Changes Formula#to_hash_with_variation to take both the local hash and API hash and take the relevant bits from both.

Fixes #14600.

Review period skipped due to critical label.


Deprovision Tenants with Collie

Is your feature request related to a problem? Please describe.
 In my example, this problem goes for AWS, but it might be just as painful for the other platforms as well.
Reach of the feature request
Platform Operators and Project Owners responsible for deleting accounts
Impact of the feature request
Deleting faster could result in a better user experience and even in potential cost savings by deprovisioning unused resources ASAP.
How would a solution look like?
collie tenant delete <identifier>
Not sure how the identifier part would work exactly though. Especially if you have duplicate identifiers across multiple platforms.
(optional) Implementation Hints
Check if the cloud CLIs allow for deletion. It might be that this is not possible.

Before we go on with defining a solution, the first step would be to estimate if this is doable for which cloud platforms. I know from experience that e.g. AWS requires a credit card and makes this quite difficult.

Closed due to lack of community interest - feel free to revive discussion at https://github.com/meshcloud/collie-cli/discussions


[Investigate] scope PDF parsing functionality

This is still an active topic within RAG extraction space
TODO:

Review PDF extraction methods in well reviewed RAG tutorials (over past week)
Break down into what's scope-able for future milestones

@bazooka720 we have a beta release (datafog==2.4.0b3) if you're interested in giving it a spin.  Planning on releasing 2.4.0 tonight.
Main update:

added unstructured.io support via an 'upload_file' function in the DataFog class.
added unstructured[pdf] and unstructured[pptx] packages
unstructured has out-of-the-box support for a variety of formats so I'm writing some tests to confirm support across those but it did pass unit tests for pdf and pptx.

sample usage:

TODO: expose OCR/Image capabilities (which using Unstructured involves passing a 'high_res' param in for setting) - will try to fit that in for tonight's release.
Let me know if you have any questions/feedback and thanks for all the input!


Version bump transform and redbox to solve #1

Oops, didn't see the pull request so I made the change myself. Sorry.

np, as long as it's fixed! ;)


Implement automatic sourcemap upload for Sentry

Please describe the problem that your request should resolve.
The error reports we see in Sentry do not use source maps, which results in a stack trace that is very hard to understand. We should make them more readable
Describe the solution you'd like
Implement automatic source map upload to Sentry - https://docs.sentry.io/platforms/javascript/sourcemaps/uploading/vite
Is it already possible to achieve the same outcome in any other way? If so, how?
No, it is not.

duplicate of #252 🙂


can we support compile in alpine?

so to use it in docker

What kind of problem are you facing?

What kind of problem are you facing?

dependencies like unofficial-curlpp cannot be found or installed by apk or vcpkg.

vcpkg dependency is needed. You would need to install manually,

to

if I use vcpkg install minio-cpp to install minio-cpp, the result would be like:
error: Failed to run "/data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake" to determine the cmake version.
error: /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake failed with exit code: (127).
sh: /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: not found
I think it is because the cmake vcpkg downloaded is not compitable to alpine. I use ldd to check, the result is:
/lib64/ld-linux-x86-64.so.2 (0x7f8bb739f000)
libdl.so.2 => /lib64/ld-linux-x86-64.so.2 (0x7f8bb739f000)
librt.so.1 => /lib64/ld-linux-x86-64.so.2 (0x7f8bb739f000)
libpthread.so.0 => /lib64/ld-linux-x86-64.so.2 (0x7f8bb739f000)
libm.so.6 => /lib64/ld-linux-x86-64.so.2 (0x7f8bb739f000)
libc.so.6 => /lib64/ld-linux-x86-64.so.2 (0x7f8bb739f000)
Error loading shared library ld-linux-x86-64.so.2: No such file or directory (needed by /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake)
Error relocating /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: __strdup: symbol not found
Error relocating /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: __rawmemchr: symbol not found
Error relocating /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: __strftime_l: symbol not found
Error relocating /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: __register_atfork: symbol not found
Error relocating /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: backtrace: symbol not found
Error relocating /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: gnu_get_libc_version: symbol not found
It seems it is a libc problem.

I see  sh: /data/chenhui/code/minio-cpp/vcpkg-master/downloads/tools/cmake-3.25.1-linux/cmake-3.25.1-linux-x86_64/bin/cmake: not found. Do you have supported cmake?


Add requirements and linkage for inter-workload dependencies implementation in agent

Issues: #48

Definition of Done
The PR shall be merged only if all items mentioned in CONTRIBUTING.md have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

[x] All steps in CONTRIBUTING.md have been handled

@krucod3, @lingnoi: After merging the main branch locally I have an unstable stest that fails with different error message (but it constantly fails). I will push the changes to better see what was changed during the merge.

@krucod3, @lingnoi: After merging the main branch locally I have an unstable stest that fails with different error message (but it constantly fails). I will push the changes to better see what was changed during the merge.

I have analyzed the bug yesterday and @krucod3 has made a fix for it in a separate PR #199. The stests are not unstable anymore when this PR is merged on top.


Adapter-static: How to setup the head content of the fallback page

Discussed in https://github.com/sveltejs/kit/discussions/11889

Originally posted by benoit-z February 24, 2024
Hello folks :wave:
I'm building a SPA with sveltekit and static-adapter, works great, but I have some issues with the SEO tags in the  of my pages.
Here's what I'm doing:

I'm making a website with several pages.
I'm building it as an SPA, with some pages prerendered.

I can do that by using the option
fallback: "index.html"
of adapter-static, and on the +page.ts of the pages I want prerendered:
export const prerender = true;
export const ssr = true;

Which works fine.
Now, let's say that I want a specific <title> in the  of my fallback page (which I believe makes sense since this is what the crawlers will see when they try to access any non-prerenderd page). I would need to  put the <title> tag into the app.html file, as it is the file used as a template for the fallback page.
On my prerendered pages, I want a different, more specific <title> though. So I'm using <svelte:head> component in their +page.ts file.
But since the app.html file is also used as a template for my prerendered pages, I end up with duplicates <title> tags in them (the one from app.html and the one from the page's <svelte:head> component.
In summary (app.html) ->
<!DOCTYPE html>

    <meta charset="utf-8" />
    <base href="/" />
    <meta name="viewport" content="width=device-width, initial-scale=1" />

    <title>My fallback title</title>
    %sveltekit.head%

%sveltekit.body%

Is there any way to set the  content of the fallback page other than writing it in app.html?
Or is there any way to use another file than app.html as the template for the fallback page ?
Am I doing this wrong ?
Thanks for your support :raised_hands:

Thanks for the help on #11889 @eltigerchino !
This morning I checked how the same situation was being handled in Nuxt, turns out that when generating the fallback pages, Nuxt is injecting the head tags defined in the nuxt.config.ts file.
I guess the Svelte equivalent of this would be to use the root +layout.ts file when generating the fallback page with adapter-static?

I was wondering if anyone put more thought into this? It would be great if e.g. the root layout could be rendered for the fallback page (maybe opt in just in case).


New source - Serial - Geminated Volume doesn't appear

I know we are gradually adapting TW to all sorts of unorthodox stuff that happens in taxonomy.
Sometimes the periodicals cram two volumes in one. And they are indicated by linking both numbers with a hyphen or an n-dash.
I have tried both as well as slash, but they either cannot be entered or enter but do not appear in the title of the source.
Could we fix this?
In the example below, this journal has a geminated volume 161-162 -- connected by a hyphen.
I cannot reproduce this. I keep getting the source WITHOUT THE VOLUME NUMBER:
Villarreal-Manzanilla, O. & DoNascimiento (2005) Una nueva especie del género Paravima Caporiacco, 1951 (Opiliones, Laniatores, Agoristenidae) del tramo oriental de la Cordillera de La Costa, Estado Miranda, Venezuela. Memoria de la Fundación La Salle de Ciencias Naturales, 101–111. [2004]

Fixed, change will be reflected on the next update

Love you, guys....


Cache queries

When TYTO performs a remote query, it takes several seconds.
While this is fine for occasional queries, it can make automation massively slow.
Can you please add caching for queries so that lookup failures can be resolved quickly after the first one?

Here's my workaround implementation that I'm using locally:
tyto_cache = {}
def tyto_lookup_with_caching(term: str) -> str:
    if term not in tyto_cache:
        try:
            tyto_cache[term] = tyto.SO.get_uri_by_term(term)
        except LookupError as e:
            tyto_cache[term] = e
    if isinstance(tyto_cache[term], LookupError):
        raise tyto_cache[term]
    else:
        return tyto_cache[term]

Closed in #24


Requirements

Hello, sorry for the mess. I am new to ember and ember-cli and especially livereload. My question is - does your library injects only client side, and, therefore, I need to install watchman on the serverside, so that my app would stop throw errors like
WebSocket connection to 'ws://localhost:35729/livereload' failed: Error during WebSocket handshake: Sent non-empty 'Sec-WebSocket-Protocol' header but no response was received

Ok, I've figured it out. It seems like Chrome bug, so I updated it and everything works.


[FEAT] Json constrained support

Motivation
A lot of llm API (Together AI, fireworks, Anyscale...) and other engines (vllm...) support constrained generation with a JSON schema. As outlines is already a dependency of sglang, it is straightforward to extend its usage to directly support json schema in the API.

Modification
Adding json_schema parameter (in sampling params for sglang, as an extra parameter for CompetionRequest/ChatCompletionRequest for openai compatible server)
Adding a new FSMJsonCache for JSON. It inherit FSMCache, so it functions the same way, but in addition it also stores the regex string converted by outlines. This regex string is required by the Jump Forward Cache.
Adding a unit test, and updating sampling params documentation

Checklist

[x] Before submitting a PR for review, make sure it has passed verification in your local development environment: limited env with only 24GB VRAM, some out of memory on the test suite but no functional errors.
[x] Ensure pre-commit pre-commit run --all-files or other linting tools are used to fix potential lint issues.
[x] Confirm that modifications are covered by complete unit tests. If not, please add more unit tests for correctness.
[x] Modify documentation as needed, such as docstrings or example tutorials.

Any idea why the accuracy test is failing? I'm not sure I've done any changes that could impact accuracy, at least when not using a json_schema.

@havetc can you address the comments and rebase?

@havetc can you address the comments and rebase?

Hello! I'm back from a few days on holidays, I'm on it

please fix the lint error

@havetc Thanks for the contribution. It is merged.

Hi @havetc , actually a better method is that we do not need to change anything inside the server.
We can just do the JSON schema -> regex conversion in this function https://github.com/havetc/sglang/blob/5ff25cdf5b1310e83d9e595142b39ae4d7b561e9/python/sglang/srt/sampling/sampling_params.py#L114
Then all changes in python/sglang/srt/managers/tp_worker.py, python/sglang/srt/constrained/fsm_cache.py seems not needed anymore.
Does it make sense? If so, can you simplify your code with another PR?

@merrymercy But as I understand it, that wouldn't cache the Json compilation to regex? As it is a costly operation that can take several second on cpu, it really makes sense to cache it to avoid doing it for each request.

Hi all - thanks for this. I noticed that this doesn't actually expose json_schema to the gen function, which we'd love to be able to use. I just wanted to raise this (not sure if there was another planned PR to introduce that) and also get it onto a branch that we could use internally.
I opened a PR here https://github.com/sgl-project/sglang/pull/1254 (there may be other areas that need updating, I just wanted to hack together something quickly so we could experiment).

@qeternity
Indeed that seems like a miss, I tried to find where to update the code to add a global support for json schema, but as I only ever use sglang as an inference server I didn't think about the gen function.
Your pull request looks good to me (haven't tested though), I think one thing that maybe would be worth updating are the json examples, that uses only regex. (Also, feel free to extend the json unit test with a gen function test if you want to)

I have been testing here for the past few hours, it's all working well. But @merrymercy will have a better view on whether there are other places that also need to be updated to accept json_schema.
Will have a look at docs and tests later if I have time.

@havetc I see. In this case, we can also make a cache on the TokenizerManager to cache the JSON -> regex. In this way, the conversion even overlaps with GPU computation. I just want to make the core part in tp_worker as minimal as possible, so we want to remove this duplication on handling both regex and json schema.


RFC: improve CString construction methods

Amend the methods available to construct a CString to improve composability and follow the conventions emerging elsewhere in the standard library.
Rendered

Cc @alexcrichton @aturon

Where are the benchmarks?

@mahkoh It's not about performance for now, though the changes widen possibilities for optimization in the future. CString::from_iter is to replace the copy-from-slice implementations of IntoBytes, and its performance should be similarly worse off as Vec::from_iter compared to slice.to_vec() (noticeable, but not dramatic; in theory the optimizer should be able to equalize them). If an IntoCString trait is introduced, which is proposed as an open question, that could inherit the performance benefits of IntoBytes implementations.

It's not about performance for now

You might not care about performance but I do.

though the changes widen possibilities for optimization in the future.

The opposite is true. The optimizations possible with a &[u8] are a strict superset of the optimizations possible with an u8 Iterator.

The optimizations possible with a &[u8] are a strict superset of the optimizations possible with an u8 Iterator.

Given that slice::Iter reports accurate length and its implementation of next can in theory be turned into a loop index by the optimizer with invariants optimized away, I don't see principal difference for code that iterates forward and uses size_hint to optimize. In practice, benchmarks I've ran while working on #22681 do show a difference, but it needs a hard look into the generated code to see where it is coming from.

In practice, benchmarks I've ran while working on #22681 do show a difference

The current CString::new performance can easily be improved by an order of magnitude.

but it needs a hard look into the generated code to see where it is coming from.

Go for it.

The current CString::new performance can easily be improved by an order of magnitude.

Then the same can be done for an implementation of IntoCString for slices, if micro-optimizations is the current concern at all for 1.0. I got the vibe that discarding redundant and poorly composable traits and methods has higher priority for now.

@mahkoh I have no dog in this race (yet), but I find the tone of your comments here on this thread to be unnecessarily antagonistic. It is good and reasonable to inquire into the effect that this change has on performance, but we're all on the same team here.

The generic conversion traits did land (and stabilize), meaning that many of the goals here have been resolved. The current constructors still work with Vec<u8>, but this seems an acceptable risk (and there are various ways we could change the internal representation and construction later.) So I'm going to close out this RFC.
Thanks @mzabaluev!


Fix Default Height for Editor Component

Editor component didn't have a minimum height set, so fixing this by passing through a minimum height to EditorComponent. Now, if the scrollable height of the editor is less than the minimum height, we use the minimum height as the height of the component.
Also fixed an issue where the markdown code editor's height was far too high. Now we're calculating the height on the layout() call, which gets called every  time we display the markdown editor.

One more comment - the CI failure is due to an unrelated test failure - Karl's latest fix also has it. So that shouldn't block your checkin (we should get it solved though)