File upload undefined

Hi!
I'm trying to implement the new file uploading feature and I'm having some problems.
It seems like the server doesn't receive the file, just an array with undefined. I tried copy and paste the example code and it doesn't work either, but it works on stackblitz.
This is the output from SuperDebug:
{
  title: "djhfg",
  content: "jkshdfkjsd",
  createdBy: "",
  attachments: [
    {
      name: "favicon.png",
      size: 1571,
      type: "image/png",
      lastModified: 2024-02-02T21:33:38.284Z
    }
  ],
  id: "",
  created: ""
}

... and this is console.log on the server:
{
  id: 'y4djcs',
  valid: false,
  posted: true,
  errors: { attachments: { '0': [Array] } },
  data: {
    title: 'djhfg',
    content: 'jkshdfkjsd',
    createdBy: 'v615zyol29loqu4',
    attachments: [ undefined ],
    id: '',
    created: ''
  }
}

and
if (!form.valid) {
			console.log('Invalid form', form.errors);
			return fail(400, withFiles({ form }));
}

Outputs: "Invalid form { attachments: { '0': [ ‘The file must be a file’ ] } }"
Any suggestions on how to fix this is appreciated.

If it works on Stackblitz, it's hard to say what exactly in your code or environment is not making it work. Sometimes it's as simple as a missing name attribute on the file input field.
Here's a working example of mine anyway, maybe you can figure it out from there: https://stackblitz.com/edit/superforms-2-file-uploads?file=src%2Froutes%2F%2Bpage.svelte,src%2Froutes%2Fschema.ts,package.json

I think it's something with superValidate() that doesn't seem to work for me. If I await request.formData(), the file is there.
+page.server.ts
const schema = z.object({
	image: z
		.custom<File>((f) => f instanceof File, 'Please upload a file.')
		.refine((f) => f.size < 1024 * 1024 * 10, 'Max 10 MB upload size.')
});

export const actions = {
	default: async ({ request }) => {
		const formData = await request.formData();
		const form = await superValidate(formData, zod(schema)); // Doesn't matter if i put in request instead of formData

		console.log((formData.get('image') as File).size); // Works
		console.log(form.data.image.size); // Doesn't work (undefined)
                ...
        }
}

+page.svelte
<script lang="ts">
	import type { PageData } from './$types';
	import Header from '$lib/components/Header.svelte';
	export let data: PageData;
</script>

<Header />

<form method="POST" enctype="multipart/form-data" class="pt-56">
	<input type="file" name="image" />
	<button>Submit</button>
</form>

Solved it!
Needed to change
const form = await superValidate(formData, zod(schema));

to
const form = await superValidate(request, zod(schema), {
			allowFiles: true
});

Does it have something to do with SUPERFORMS_LEGACY option in vite.config.ts? Maybe add it to the documentation.

Right, if you have set SUPERFORMS_LEGACY, no files will be allowed. It's mentioned here, but as always you can never mention something enough, so I'll add it in a couple of more places.


Async updates

#915  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Missing all_candidate_sections_v2.json

Dear Authors,
I think there is a missing file in /src/comp_med_dsum_eval/preprocess/get_relevant_notes.py, line 134:
with open(os.path.join(input_dir, 'all_candidate_sections_v2.json'), 'r') as fh:
        all_candidate_sections_sorted = json.load(fh)

Where can I find this file? or do you plan to share this file in the future?
Thank you in advance!

Hi Hyeonbin - we are sorting this out.  Thanks for calling to our attention.

Hi everyone!
Thanks for making your work available and sharing your code. Unfortunately, the file @hbin0701 pointed out is still missing. It is already needed in the first step of the preprocessing (get_relevant_notes.py), hence making it very hard to reproduce your work.
Thank you!


Space removed between the tasks

Description
Describe the bug
Unnecessary space between the tasks, UI not handled properly here
To Reproduce
Add 1-2 tasks and see the space between
Fixes #130
Type of change
Please delete options that are not relevant.

[/ ] Bug fix (non-breaking change)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ /] My code follows the style guidelines of this project
[ /] I have performed a self-review of my code
[ /] I have commented my code, particularly in hard-to-understand areas

screen-capture (3).webm

Kindly assign the issue and mark it as hacktober
then please check and let me know


BaseURL request problem.

Moya : 8.0.3
Language : Swift 3.1
Let's imagine that you have a link that looks like that:
let urlString = "https://google.com/123/somepath?X-ABC-Asd=123"

When I do my request using Alamofire :
Alamofire.upload(fileURL, to: URL(string: urlString)!, method: .put, headers: nil).response {
    response in
    print(response)
}

it all seems fine(don't look at status code):

but when I do the same request using Moya like that:
enum MyApi {
    case upload
}
extension MyApi : TargetType {
    var baseURL: URL {
        let urlString = "https://google.com/123/somepath?X-ABC-Asd=123"
        return URL(string: urlString)!
    }

    var path: String {
        return ""
    }

    ...
}

provider.request(MyApi.upload)

The created request URL is different that in Alamofire request.

The difference is that before ? character moya inserts the / char.
https://google.com/123/somepath?X-ABC-Asd=123
https://google.com/123/somepath/?X-ABC-Asd=123
Could you help me and tell me what I'm doing wrong?

Hey @k8mil, can you check if it works as expected when you change your baseURL to be just https://google.com and path to return /123/somepath?X-ABC-Asd=123?

@pedrovereza
Thanks for your response.
I've created a sample repo with the test code. Just clone and run it.
https://github.com/k8mil/MoyaBaseURL
Below you can see, screen from Charles with 3 options:
First
request was performed with Alamofire using
URL(string: "https://google.com/123/somepath?X-ABC-Asd=123")!
Second
request was performed using MoyaProvider with
baseURL = "https://google.com/123/somepath?X-ABC-Asd=123"
Third
request was performed using MoyaProvider with
baseURL = "https://google.com and path = "/123/somepath?X-ABC-Asd=123"

And only Alamofire produces that what I want to achieve :)
Important thing is that I'm using Moya in my project and the consuming my backend API with Moya is really great and works perfectly, but I have some case that I have to use the URL(which is not related with backedAPI) that I received from some service and I shouldn't modify that URL. Unfortunately, as you can see in the Second and Thrid example the URL is modified by removing/changing the ? char.

Hey @k8mil thanks for the detailed explanation and examples 👍
I think I have a fix in #1053, care to take a look? 😉

Very interesting use case. Thanks for implementing this @pedrovereza 😄

Thanks @pedrovereza !
I've tested it and it works fine.
So, now I wait till it will be merged with master 👍  :)


Next playlist item doesn't start from beginning

Describe the bug
When double clicking on the playlist item it plays from the position of where current item is currently playing. In other words, it doesn't start from beginning.
To Reproduce
Steps to reproduce the behaviour:

Join a room
Drag some videos in the playlist (in my case I used MKVs but I don't think it matters)
Play one video and skip somewhere middle
Double click on the next video

Expected behaviour
Next video starts from 0:00
Version and platform:

OS: [e.g. Windows 10 64-bit]
Syncplay 1.7.0
MPC-HC (64-bit) 2.0.0

Thanks for reporting this problem. Can you confirm whether or not you get this issue when using mpv/mpv.net?

Hi. Thanks for reply.
I will test it at some point this week and let you know.

@Et0h I can confirm that in mpv.net it has no problem for the issue I have reported. VLC which has red txt showing that tracks time incorrectly also doesn't have this issue. I have also tried MPC-BE and it doesn't have this problem.
It seems to be a problem only with MPC-HC. In my case: MPC-HC (64-bit) 2.0.0

Syncplay's support for MPC-HC was added before the playlist feature exists, and Syncplay's support was for the original MPC-HC which was discontinued in 2017 rather than MPC-HC v2 which is being mainained by clsid2. Syncplay relies on volunteers, and at present nobody is currently volunteering to maintain Syncplay's support for MPC-HC.
As such, you can either try to fix it yourself or move over to mpv which is being actively supported by Syncplay's core developers.

Okay, I had some free time so I tried to fix this issue. I wasn't able to reproduce it but I assume it happens when it takes so long to open a file that the setPosition is cancelled because no file is playing when it tries to reset the position and my computer is just too quick at loading files to see the issue.
@mbalandis Please test the changes at: https://github.com/Syncplay/syncplay/actions/runs/5766082434 (use the Syncplay-1.7.0-Setup.exe or Syncplay_1.7.0_Portable artefacts to get the new build).
Let me know if this change resolves the problem, and also let me know if it accidentally resets the position to 0 if someone with MPC-HC joins a room which is part way through paying a fil.

Hello, I had the same issue on mpv.net on the 1.7.1 version.

Same issue for me, mpv.net and latest syncplay.

@StefenStefan @soredake
I use MPV.net with Syncplay all the time and have never had these issues.
To help figure out and replicate the issues I'd like some more details please:

What version of mpv.net are you using?
How regularly does this happen?
Does it take you to the position of the previously playing file?
How many other people are in the room and what media players do they use?
What file format are you playing?
Are you playing the files from your hard drive?
Does it correct itself or not?
Does this issue only occur with the newer versions of Syncplay and MPV.net or is it a long+running issue?
Are you running on a relatively low spec or high spec computer?
Are you using the public server or a private server? Do you have any latency or connection issues with that server?

@StefenStefan @soredake
I use MPV.net with Syncplay all the time and have never had these issues.
To help figure out and replicate the issues I'd like some more details please:

What version of mpv.net are you using?
How regularly does this happen?
Does it take you to the position of the previously playing file?
How many other people are in the room and what media players do they use?
What file format are you playing?
Are you playing the files from your hard drive?
Does it correct itself or not?
Does this issue only occur with the newer versions of Syncplay and MPV.net or is it a long+running issue?
Are you running on a relatively low spec or high spec computer?
Are you using the public server or a private server? Do you have any latency or connection issues with that server?

What version of mpv.net are you using?

<IP_ADDRESS>

How regularly does this happen?

Sometimes, not always.

Does it take you to the position of the previously playing file?

Yes.

How many other people are in the room and what media players do they use?

One, also using mpv.net (not the latest version, thought).

What file format are you playing?

mkv

Are you playing the files from your hard drive?

Last time this happened I switched from local file to local file.

Does it correct itself or not?

It does not.

Does this issue only occur with the newer versions of Syncplay and/or MPV.net or is it a long running issue? Can you provide an example of a version which does not have this issue?

It's an issue I have for years, although rarely experienced earlier, but now I am forced to switch files manually because of bug in trakt-scrobbler, so I see this problem more often.

Are you running on a relatively low spec or high spec computer?

High spec.

Are you using the public server or a private server? Do you have any latency or connection issues with that server?

This happened on official servers, unofficial server https://syncplay.343.re/ and on self-hosted instance too. No latency issues, I think, but I do have weird problem with my ISP connection where connection are randomly stuck for 20–40 seconds and then aborted.

@Et0h

v1.7.3
Every time when going to the next item in the playlist
Yes
2 people, both using mpv.net (I tried being alone in the room it was the same)
mkv
Yes, SSD
No
I believe it's the same for every version
Ryzen 5 3600x, RX 590, 16GB Ram
Public one - syncplay.pl:8997. No latency issues

@Et0h
I just tested a lot of versions and it appears that the issue occurs from 1.6.5 Beta 1 and up.
Every version above that one has that issue.
I checked 1.6.4a, 1.6.4 and 1.6.3 - These are working fine, they don't have that bug.

@StefenStefan
Thanks to your information I've worked on a potential solution. Please test it and let me know if it fixes it for you: https://github.com/Syncplay/syncplay/actions/runs/9439659160
Technical details of the problem/solution
Thanks to your excellent testing I traced the likely culprit to this commit: https://github.com/Syncplay/syncplay/commit/89ce72d6cbbeebc195e1058acc9d26219af0ad75
I believe that commit was created to fix an issue where Syncplay would erroneously reset when loading a file when you join a room and/or get a file in the playlist which you didn't previously have.
To fix it, I had to dig deep into the mpv logic, and found that when it is loading and tries to get the position it reverts to the global position. Now it returns 0 if it has been recently reset.
f people still experience problems I will have to modify the recentlyReset system to link to whether or not there has been a SyncplayUpdateFile since the last file upload attempt or something, but I'd rather not increase Syncplay's complexity if the current fix is enough.

@StefenStefan Thanks for fix a while ago, I haven't encountered any bugs on latest versions of player so far.

@Et0h
Thank you for the response!
I'm not sure if I did everything correctly, I just downloaded 1.7.4 version and tested it (both portable and the installer one).
Unfortunately I'm still having the same issue.
Not sure if it helps but I tried using VLC and the bug is still present.
Please let me know if there is anything else you need me to test.

@StefenStefan Okay, please now try: https://github.com/Syncplay/syncplay/actions/runs/9453588544 - portable and installer should have same behaviour so you only need to test it once. This reverses a change I made during the relevant period 4 years ago (between 23 June 2019 and 30 May2020).

@Et0h
Unfortunately the bug is still present. Tested with both mpv.net and VLC.

I've made a lot of changes to how playlist advancement and switching works on my test branch. Please let me know if this fixes your issue and/or if it introduces new bugs.
https://github.com/Syncplay/syncplay/actions/runs/9612802702

Still the same, didn't notice any new bugs.

https://github.com/Syncplay/syncplay/commit/78171856143d660e07af90c89f62ca477baedf69
With this commit, it's still not starts from beginning, but syncplay now quickly switches to beginning.

@soredake Progress! How quickly does it switch to the beginning? Before it switches, is it at the position of the previous video before you changed item or from a few seconds in?

How quickly does it switch to the beginning?

Almost instantly, and this happened only once so far.

Before it switches, is it at the position of the previous video before you changed item or from a few seconds in?

IIRC it was on position of the previous video.

Also, is this a change in video from auto advancement at the end of an episode or manually switching?

This time it was manual switch to next item.

Happened again, now it's been auto-switch.

Happened again, now it's been auto-switch.

Could you provide more details about what you are experiencing and the relevant context please.

Next playlist is not started from the beginning, after switching to next item the time was at the end the current file, then after a second syncplay set time to beginning.

This issue is hopefully fixed by #698 which is now merged into the main Syncplay branch: https://github.com/Syncplay/syncplay/actions/runs/11900187832
Please let me know if you continue to have any problems.

I'm so sorry, it still isn't working for me. I just tested earlier versions again (1.6.4a, 1.6.4, and 1.6.3) and unfortunately, they don't actually work. The reason it seemed to "work" for me and the bug was not present on those versions was that Syncplay would crash, leaving mpv.net open independently. I can't believe I didn't notice this during my initial testing. I sincerely apologize.

Thanks for the update @StefenStefan - Any potential fix will need plenty of testing, so it won't be able to make it into Syncplay 1.7.4 but it would be nice to be able to nail down this bug and fix it in a way that doesn't cause more problems than it solves.

If you need any help with testing, I would be happy to assist. Once again, I am really sorry for wasting your time with incorrect information.


Make ImageSlideshowState public and add "goToPage" and "stopAutoPlay"

In my app, I added thumbnails for each image below the slideshow. When the user clicks on the thumbnail I needed a way to change the slideshow to show that image. I also wanted the slideshow to stop auto playing after a thumbnail is clicked. This PR simply exposes ImageSlideshowState as public so that I can access its methods using a GlobalKey<ImageSlideshowState> that is passed as the widget's key. It also adds methods to stop the slideshow and change the page index.
Thanks for creating this package!

@percula
👋
Thanks for the PR!
I will check your PR later.

I merged, Thanks!


Websocket reconnects are shaky

When events like Home Assistant rebooting occur, the reconnection logic can sometimes "get stuck" after a failed reconnect attempt, and refuse to continue retrying.

The current reconnect logic is a bit of a mess. It needs to be gutted, and a more simple "retry every x seconds" mechanism needs to be created

bumping priority
for supervised installs, home assistant will self reboot to upgrade itself. this causes the socket to drop and come back. this presents a regular opportunity for the setup to just stop working while not tied to a specific human interaction

fixed as of 0.2.3


[Backport release-23.05] Add packages

Description of changes
Add animated Firewatch and Lakeside wallpapers from https://github.com/manishprivet/dynamic-gnome-wallpapers under added pkgs/desktops/gnome/artwork/firewatch.nix and pkgs/desktops/gnome/artwork/lakeside.nix
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Also please update the commit messages according to the contribution guide. It will probably be easiest to just do git reset fcc56872ca9c775018b6189face33c06e4936cff and then recreate the commits.

I believe all of my commits in the fork should now be in compliance with the contribution guide.
Thank you for taking the time to review my pull request.

Something weird is still going on:

You are adding more packages and maintainers.
The maintainer commit should come first otherwise the commits adding packages will not be able to find it.
There are multiple init commits, even though only the first two introduce new package.
There should not be any merge requests in the PR.

This could possibly be because I'm still relatively unfamiliar with using git for version control outside of surface-level add, commit, push.
Are you referring to be4d5b7, b4f51d9, and 64c2dce or be4d5b7, b4f51d9, and e47cb26?
Additionally, I believe your referring to 1f5bc0a on that last point, correct?

There is no need to clean fork.
You can use git rebase --interactive master to replay the commits on master, reorder them or edit them.
You can also use git checkout -p HEAD^ to selectively undo parts of previous commit.
Or just use git reset fcc56872ca9c775018b6189face33c06e4936cff to undo the commits and then selectively add just the lines you want with git add -p to recreate the commits.

This should be it.
While there are the two firewatch inits, one just amends indentation.
If you feel it doesn't follow, I can redo it one last time.


Knife Vsphere Clone VM - Windows

Hey All,
Recently our VM Clone for windows VM's has been failing with Hostname Not Specified.
We run the VM Clone command using a customisation script in VMWare for Doing the domain etc. And we pass the Hostname over with --chostname. This is trying to build from a Windows 2012 R2 Template.
It doesn't look like the hostname param is being passed through - no customization_hostname value.
DEBUG: value for config item customization_cpucount: 2
DEBUG: value for config item customization_cpucount: 2
DEBUG: value for config item customization_memory: 4
DEBUG: value for config item customization_memory: 4
DEBUG: value for config item customization_vlan: Vlan_1
DEBUG: value for config item customization_vlan: Vlan_1
DEBUG: value for config item vsphere_dc: Duss
DEBUG: value for config item customization_spec: Chef
DEBUG: value for config item customization_spec: Chef
DEBUG: value for config item customization_dns_ips: x.x.x.x,x.x.x.x
DEBUG: value for config item customization_dns_ips: x.x.x.x,x.x.x.x
DEBUG: value for config item customization_dns_suffixes: Domain.com
DEBUG: value for config item customization_dns_suffixes: Domain.com
DEBUG: value for config item customization_gw: x.x.x.x
DEBUG: value for config item customization_gw: x.x.x.x
DEBUG: value for config item disable_customization: true
DEBUG: value for config item vsphere_dc: Duss

This did work and as far as i'm aware there have been no changes in our chef/vmware environment.
this is my first post on GitHub, and first issue with Chef - If you need any more information please let me know .. and be nice :)
Thanks
Rob

Hey Rob, sorry to hear you're having problems. Did you recently upgrade the gem or install it new?
Thanks for the debug output. Could you also please paste the exact command you're running (redact what you need to)? Some of our options, for better or worse, are contradictory.
Also, paste in the output of gem list knife-vsphere, please.
Thanks,
Sean

Oh, and welcome to Github and the Chef community!

Hey,
Thanks..
*** LOCAL GEMS ***
knife-vsphere (1.2.7)```

Here is the command I am running:

```knife vsphere vm clone "Test.Domain.com" -N Test --cdnsips x.x.x.x,x.x.x.x --cdnssuffix Domain.com --cips x.x.x.x/x —fqdn Test.Domain.com --cdomain Domain.com --cgw x.x.x.x --chostname Test --cvlan Vlan_1 --cspec Chef --cram 4 --ccpu 2 --datastore vsanDatastore --template Windows2012R2Template --dest-folder / --folder Templates --verbose --resource-pool vSAN/Rob -VV```

Some of these options may not apply to the vm clone command, but we will be wrapping the command in another script that bootstraps the node as we do with Linux at present. But this did build windows nodes fine a few weeks ago.

Thanks

Rob

So that line DEBUG: value for config item disable_customization: true means we're skipping over a lot of the customization to do with Windows, and the spec never gets the hostname. But normally you need to pass --disable-customization to do that. Do you maybe have it in a knife.rb or is the script you're using adding it in?

Ah,
My apologies, I was following some debug steps online and added --disable-customization to the last test I did. I do get the issue if I don't have that in the command as well.
DEBUG: value for config item customization_plugin:
DEBUG: value for config item vsphere_dc: Duss
Cloning template Windows2012R2Template to new VM Test.domain.com
ERROR: RbVmomi::Fault: InvalidArgument: A specified parameter was not correct: hostname```

Sorry for confusion there..

Hey no problem.
I'm not seeing where "hostname" is needed in a clone spec (https://www.vmware.com/support/developer/converter-sdk/conv55_apireference/vim.vm.customization.IdentitySettings.html). Any chance that got added to the customization spec on the vsphere server?
Have you always used periods in the hostname knife vsphere vm clone test.domain.com?

The customization spec in VMWare has always had the "Enter a name in the Deploy Wizard" checked. this hasn't changed (as far as I know). But we have also named the machine using the knife vm clone command.
And yes - We have multiple Domains here so fully qualifying the VM name with the domain is very useful.
Thanks

Any ideas on this ?
I want to define the hostname of the machine with the knife vsphere vm clone command... This used to work.. But I can't seem to get it applying at all now.

Sorry nothing immediately comes to mind. It's possible we have a regression -- maybe try installing an older version of the gem or drop some of the hostname arguments to see if there's a conflict.
Other thing you can do is modify the code right before the clone task to print the specs, like here.  pp vmname; pp clone_spec should tell you what's being sent to vSphere which might give us an idea of what the improper field is.
I've been a bit tied up at work lately. My intention is to play more with the Windows cloning as I know we have a few bugs here. Sorry I'm not more responsive :(

hi,
I'm a colleage of Rob and believe I have fixed the issue.
I don't know how 'clean' this fix is, but it's good enough for our implementation.
In file knife-vsphere-1.2.4/lib/chef/knife/vsphere_vm_clone.rb around line number 559 I made this change:
          windows_hostname = RbVmomi::VIM.CustomizationFixedName(
            :name => hostname
          )

          user_data = RbVmomi::VIM.CustomizationUserData(
            fullName: cust_spec.identity.userData.fullName,
            orgName: cust_spec.identity.userData.orgName,
            productId: cust_spec.identity.userData.productId,
            # computerName: cust_spec.identity.userData.computerName
            computerName: windows_hostname
          )

Hey Paul, thanks for that. I'll double check the API docs and make the changes in the gem, unless you wanted to submit a PR yourself. Otherwise I'll give credit in the changelog.

Hey Sean,
feel free to make the changes yourself.
thx for the quick response.
gr, paul


Module version recovery problem

Hi,
If the module version number ends in 2 digits, then only the first digit is returned, and this will distort the version comparison later on.

3130ad72fc497f7c606f30b9699b535fbdd4d48c


⚠ SceneTime has degraded performance

In 5d2567c, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1183 ms

Resolved: SceneTime performance has improved in 125623e after 6 minutes.


fix test

Coverage remained the same at 90.358% when pulling e6fd7c99c4365a6bc967e579ec6f9c37e998a92f on fix_test into 3729fbd5a55078a74046fb55978201c53516d6d5 on develop.


server crashes on startup with just BFAPI and Fabric API installed

with just BFAPI installed, the server starts fine with the following error message
[10:09:44] [main/ERROR]: Adding duplicate key 'ResourceKey[bfapi:events / minecraft:command_registration]' to registry
with BFAPI and Fabric API installed, it fails to start
latest.log
bfapi 1.2.0 and fabric api 0.44.0+1.18

Exactly the same problem.

Just tested it, I got the same result. Thanks for the report and I'll be straight on it

My bad, this is the dumbest issue ever. I literally double registered, the patch will be out soon ;/ (under v1.2.1)

now imma fix mods that depend on it since i renamed an important method 🤦

Done, feel free to get the latest version of the mods <3


Table aliases not are used in WHERE clause

I am working on a gamified learning platform where students in courses can earn experience points and am trying to allow admins of a particular course to view ExperiencePointsRecords of students from that same course, but ran into Coursemology/coursemology2#706.
When a table has to be joined twice, the second instance of the table in aliased is the FROM clause, but the conditions in the WHERE clause does not use the alias.
Models
def Course
    has_many :course_users
end

def User
    has_many :course_users
end

def CourseUser
    belongs_to :user
    belongs_to :course
end

def ExperiencePointsRecord
    belongs_to :course_user
end

Ability definition
read_hash = { course_user: { course: { course_users: { user_id: user.id, role: 'admin' } } } }
can :read, ExperiencePointsRecord, read_hash

Calling accessible_by generates a query like:
...
FROM
    "experience_points_records"
        LEFT OUTER JOIN "course_users"
            ON "course_users" . "id" = "experience_points_records" . "course_user_id"
        LEFT OUTER JOIN "courses"
            ON "courses" . "id" = "course_users" . "course_id"
                AND "courses" . "instance_id" = $1
        LEFT OUTER JOIN "course_users" "course_users_courses"
            ON "course_users_courses" . "course_id" = "courses" . "id"
WHERE
    "experience_points_records" . "course_user_id" = $2
    AND (
        "course_users" . "user_id" = $3
        AND "course_users" . "role" = $4
    )  [["instance_id", 1], ["course_user_id", 4], ["user_id", 9], ["role", "admin"]]

which is incorrect.  course_users should be generated as course_users_courses in the WHERE clause:
WHERE
    "experience_points_records" . "course_user_id" = $2
    AND (
        "course_users_courses" . "user_id" = $3
        AND "course_users_courses" . "role" = $4
    )  [["instance_id", 1], ["course_user_id", 4], ["user_id", 9], ["role", "admin"]]

Is this a bug?

Hi, I'm currently working on this one.

I wrote a failing test for this issue here https://github.com/netguru/cancancan/commit/038d7640af6ce68c4b9e678b72eb020eceda2b3d
I found out that this is not a bug and the problem is actually in ability definition.
Condition is defined like this
{ course_user: { course: { course_users: { user_id: user.id, role: 'admin' } } } }

but should defined be like that instead
{ course_user: { course: { course_users: { user_id: user.id, user: { role: 'admin' } } } } }

Thanks for the update


error: cannot declare variable ‘g_numpyAllocator’ to be of abstract type ‘NumpyAllocator’

I fixed the compiler now I am able to use the command -std=c++11, but I've got that error

Hello hiostreas,
I would like to add something about this error. I think I found the reason : between opencv 2.4.X and opencv 3, the virtual class 'MatAllocator' is not described in the same way, for instance the virtual methods allocate and deallocate are not defined in the same way in both versions of opencv. So if you use opencv 2.4.x, you will probably compile successfully this utility, but I'm quite sure you won't with opencv 3. It took me some hours to find out, I just wanted to share that with future users.
Please tell me if I'm missing something or if I'm totally wrong.
PS : Sorry for my English, my apologies :)
Regards,
Adem

@ademstoce Do you think if there's any way to solve this problem? I've been trying to compile this with python3.

@xgenvn You can use the code from opencv3 repo: https://github.com/opencv/opencv/blob/master/modules/python/src2/cv2.cpp
I think it is going to work if you take the NumpyAllocator and the pyopencv_from method from there.

For opencv3 apparently a lot has changed! Even I was facing the problem. You need to use the latest wrapper functions files from opencv.
Opencv2 wrapper function files
Check out this folder(Opencv3 wrapper function files)
This folder has the wrapper functions that you need to use. This is actually for generating wrapper functions for all modules. But we are concerned only with the cv2.cpp (which has the needed wrapper function definitions) and pycompat.hpp in this folder.
You need to add both these files to your directory. Now from cv2.cpp I needed only the functions that would convert Mat to PyObject and vice-versa. So I deleted the code from line 443 (template<typename _Tp, int m, int n> PyObject* pyopencv_from(const Matx<_Tp, m, n>& matx)) till end
Also for making this file work I had to do the following steps(This may or may not be needed depending on your usage):

Instead of

#include "pyopencv_generated_include.h"
#include "opencv2/core/types_c.h"
#include "opencv2/opencv_modules.hpp"

on lines 12-15, I used
#include <opencv2/opencv.hpp>
#include <opencv2/imgproc.hpp>
#include <opencv2/highgui.hpp>
#include "opencv2/core/core.hpp"
#include "opencv2/core/types_c.h"
#include "opencv2/opencv_modules.hpp"

Removed #define MODULESTR "cv2" on line 8 because I had changed the file name cv2.cpp to something else

Removed lines 87-116 because didn't look much useful for the functions that I was using

Removed lines template<> from lines 393, 426

I do not know the internal logic but apparently instead of having refcount directly in Mat and using it like p->refcount, in opencv3, they have added some UMatData class which will give you this refcount. Hence, this line becomes p->u
And there are many other minute changes as well in the wrapper function definitions as well


Move Window or program to Active Monitor when changing number of monitors or off of focus

Description of the new feature / enhancement
Move off screen window to an active monitor.    Maybe in the right click menu on the program in the Task Bar.
Scenario when this would be used?
I use 3 monitors at my office but when I am at home I have 1 large 4K monitor and if I sleep my computer at work and then open it at home programs will launch where there is no longer a display. To get the screen back visible I have to use the ALT SPACE M and Move the screen back to where I can see it and then manipulate to where I need it.
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/50


Rhf

Description
Migrates item creation and edit modals to react-hook-form.
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

when I first added an item on the website, it didn't appear in the table until I refreshed
I forgot to invalidate the items query after creating an item. I'll fix this.


Import Wallet/SEED shall we add a scan QR code function?

There are many wallets provide Showing seed as QR Code

We should provide a scan QR code function for user to easily import a wallet.

Yes this would be useful, also export private key as QR and import private key as QR same as metamask. Then you can seamlessly switch between the two which is helpful for developers.

We briefly discussed about this. But I don't think we arrived at a conclusion. There's the concern that this encourages bad security practice because, there's a good chance the user generates the QR code using a website.
I'm fine with doing it. No strong opinions here. @colourful-land ?
But if we decide to do it, let's do it after this sprint, like https://github.com/AlphaWallet/alpha-wallet-android/issues/780

[ ] Decide if we do it. @colourful-land
[ ] If we decide to do it, do it after the sprint (or someone can disagree)

Don't provide export into QR code(this is super bad, it encourage the user to save the seeds in a photo album or cloud without encryption).
Only prove import from scanning QR Codes which are generated from other wallets.

Would argue against it because of security concerns and lack of adoption for such a method with seeds.

Would argue against it because of security concerns and lack of adoption for such a method with seeds.

You against “export to QR” or “ import from QR”
I don’t see any security concerns about “ import from QR.

Don't provide export into QR code(this is super bad, it encourage the user to save the seeds in a photo album or cloud without encryption).
Only prove import from scanning QR Codes which are generated from other wallets.

Bump @colourful-land until we settle on a different arbitrator

 [__] Victor: Import Wallet/SEED shall we add a scan QR code function?

 [700] As a wallet user, I want to export/import my wallet's SEED phrase more readily, so that I'm not frustrated with typing long seed phrases


How to check rego rule for CVE-2021-25741

I checked cluster with new kubescape - it shows that there are no subPath feature, but after grep configs I saw a lot places with subPath - I tried to see a rule in https://{x}.{y}.cyberarmorsoft.com/v1/armoFrameworks?customerGUID=11111111-1111-1111-1111-111111111111&getRules=true&frameworkName=NSA  but did not see anything about this CVE

hi @xawdxawdx
We have moved the rules to an open repo, see the specific rule here: https://github.com/armosec/regolibrary/blob/master/rules/CVE-2021-25741/raw.rego
Do you mind sharing an object (YAML) where you have expected this rule to fail?

You can find here the controls


How to use my own dataset?

If I want to use my own nuscenes format data set for inference and training, what format does it need to meet and what content does it contain?

Did not understand the question, please be more specific. If your dataset has the same format with nuScenes, it should be easy to transfer to your dataset.

What is the LIDAR data used for? There is no mention in the paper that the model uses LIDAR information. Can we replace LIDAR data with ego car coordinate truth values?

Lidar data is used to generate depth GT for dense depth supervision in stage1. You can remove this and lower the learning rate to ensure the training stability.

Thanks.

请问如果我只提供六图环视图像数据，并只需要它根据这些视觉数据进行推理，不需要进行评估测试，该如何做？

把config里eval_mode全部设为False。

那么我需要把我的六图环视图像数据放到哪个目录下呢？假设这些数据没有任何标注。

需要更改nuscenes_infos_val.pkl中的图像路径为你自己的路径，还要更改对应的相机内外参、自车坐标变换等等。


Galaxy Generator error during its final stage with infinite cap

Describe the bug
When the Galaxy Generator reaches its final stage, where the number of Galaxies it can generate is effectively infinite, it errors out with the following error, causing it to not render anymore:
01:19:50.492 TypeError: "mag" is read-only
    roundExpTo ad-notations.esm.js:19
    formatLDecimal ad-notations.esm.js:118
    format format.js:16
    format ui.js:55
    render PelleGalaxyGeneratorPanel.vue:81
    VueJS 14
    update ModernSidebar.vue:23
    dispatch event-hub.js:26
    flushEvents ui.js:143
    promise callback*dispatch ui.js:130
    dispatch event-hub.js:32
    gameLoop game.js:437
    gameLoop intervals.js:56
    setInterval handler*start intervals.js:11
    restart intervals.js:25
    restart intervals.js:53
    postLoadStuff storage.js:569
    loadPlayerObject storage.js:549
    import storage.js:167
    importSave ImportSaveModal.vue:135
    VueJS 3
vue.runtime.esm.js:3065
    VueJS 17
    update ModernSidebar.vue:23
    dispatch event-hub.js:26
    flushEvents ui.js:143
    (Async: promise callback)
    dispatch ui.js:130
    dispatch event-hub.js:32
    gameLoop game.js:437
    gameLoop intervals.js:56
    (Async: setInterval handler)
    start intervals.js:11
    restart intervals.js:25
    restart intervals.js:53
    postLoadStuff storage.js:569
    loadPlayerObject storage.js:549
    import storage.js:167
    importSave ImportSaveModal.vue:135
    VueJS 3

To Reproduce
Unlock the Galaxy Generator, then use it to drain all Rifts. Once the last rift is drained, the generated galaxy cap vanishes, producing the above error.
Expected behavior
The Galaxy Generator should not throw an error.
Screenshots

Savefile
save2.txt

seems to be a bug with formatting code itself, so will have a look after ive sorted be port (also i think the issue here is im trying to format what should infinite galaxies, which is a bug in of itself)

hopefully fixed, not gonna close incase still broken

I can no longer reproduce this error; it appears to be fixed.


As a developer I want to see the AE SOCPT Example to test out the model

Several corresponding "Sprint 9 Examples" files have been uploaded into /workfiles/examples.
These contain the Demog and AE SOCPT examples that were originally defined for PHUSE US Connect, but:

The structure of the Excel file Sprint 9 Examples.xlsx has been updated to reflect the redesigned model (for standalone method definitions).
The AE SOCPT example has been completed:

Records were added to the AnalysisResults sheet, using the new groupValue attribute/slot to specify values for data-driven groupings.
AE_SOC_PT Shell and AE_SOC_PT Display sheets were added.
The following corresponding example files were generated from the information in the Excel file:

Sprint 9 Examples.yaml
Sprint 9 Examples.json
Sprint 9 Examples-LOPA.txt: a textual representation of the list of planned analyses, including details of each referenced analysis.


Установка на FastPanel

Добрый день. Как установить CentpaPress через панель FastPanel, а не скриптом?

CinemaPress*

Я не знаю..
Отправлено из Outlook для Androidhttps://aka.ms/AAb9ysg

From: Graf @.>
Sent: Tuesday, October 17, 2023 12:47:13 AM
To: CinemaPress/CinemaPress @.>
Cc: Subscribed @.***>
Subject: Re: [CinemaPress/CinemaPress] Установка на FastPanel (Issue #70)
CinemaPress*
—
Reply to this email directly, view it on GitHubhttps://github.com/CinemaPress/CinemaPress/issues/70#issuecomment-1765328940, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APKU2R5Y2N4SSGD7SXQVXO3X7WTODAVCNFSM6AAAAAA6CXC46CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVGMZDQOJUGA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Error when importing ortools.graph.pywrapgraph in python (mach-o, but wrong architecture)

What version of OR-Tools and what language are you using?
Version: ortools==9.2.9972
Language: Python
What operating system (Linux, Windows, ...) and version?
MacOS BigSur (11.6.1) with M1 Chip
What did you do?
Steps to reproduce the behavior:

import ortools
import ortools.graph.pywrapgraph

What did you see?
error when importing ortools.graph.pywrapgraph
 from . import _pywrapgraph ImportError: dlopen(/Users/temporaryadmin/.pyenv/versions/gfs-data-science-dev/lib/python3.10/site-packages/ortools/graph/_pywrapgraph.so, 2): no suitable image found.  Did find:         /Users/temporaryadmin/.pyenv/versions/gfs-data-science-dev/lib/python3.10/site-packages/ortools/graph/_pywrapgraph.so: mach-o, but wrong architecture         /Users/temporaryadmin/.pyenv/versions/3.10.0/envs/gfs-data-science-dev/lib/python3.10/site-packages/ortools/graph/_pywrapgraph.so: mach-o, but wrong architecture
Anything else we should know about your project / environment
Virtual env (python version==3.10.0)

duplicate.

Hi @lperron could you please relate this issue to the one considered as duplicate? As I have tried the solution provided here https://github.com/google/or-tools/issues/2722, but still ran into pywrapgraph related error.


Module naming base

Move the address_type, component_type and invalid_locality_id to a naming_base module to eliminates some dependencies.

The docs step is failing but I made the fixes in #4260


bsdtar: Write error

We have a job using the fs plugin with the following target configuration:
properties:
  shield:
      targets:
        blobstore:
          config:
            base_dir: /var/vcap/store/shared/
          plugin: fs

When we try to run the backup task we get the following error:
Validating TARGET plugin `fs`...
================================
✓ base_dir files in /var/vcap/store/shared/ will be backed up
✓ include all files will be included
✓ exclude no files will be excluded
✓ bsdtar using default /var/vcap/packages/bsdtar/bin/bsdtar
OK

Validating STORE plugin `s3`...
===============================
✓ s3_host <IP_ADDRESS>:443
✓ access_key_id cf-user
✓ secret_access_key
✓ bucket cloudfoundry
✓ prefix backups
✓ signature_version 2
✓ socks5_proxy (no proxy will be used)
✓ skip_ssl_validation yes, SSL will NOT be validated
OK

Running backup task (using bzip2 compression)
=============================================
bsdtar: Write error
Unable to exec '/var/vcap/packages/bsdtar/bin/bsdtar': exit status 1

EXITING 3
TASK FAILED!! shield worker 4 failed to execute the command against the remote agent <IP_ADDRESS>:5444 (Process exited with: 3. Reason was: ())

We have checked that the base_dir (/var/vcap/store/shared/) exists, and no backup file is created at that directory. When we try to run the command manually it works: /var/vcap/packages/bsdtar/bin/bsdtar -c -C /var/vcap/store/shared/ -f - ..
We are using the shield-boshrelease v6.3.6, that ships shield v0.6.6. I've checked if there's any commit at the latest shield github repo that fixes the backup function, but I haven't seen anything.
Any hints on what might happen and how we can debug it?

There was a bug in the version of SHIELD shipped with 6.3.6 BOSH release.  You will want to upgrade to 6.4.0 to get SHIELD 0.7.2, because you want the changes that went into 0.7.0: https://github.com/starkandwayne/shield/releases/tag/v0.7.0

We plan to do that, but I don't want to blindly upgrade without understanding first what's going on. Is there any way we can get a debug output of what is trying to do?

6.3.6 didn't have the -f - args to bsdtar, so it was always trying to write to / read from /dev/st0

Thanks for the explanation. That makes sense. We will upgrade the version and report back when it's done.


docs(README): simplify

Always migrate to latest version, supported by library.

CLI is not gone, it simply moved here: https://github.com/bpmn-io/dmn-migrate-cli


Typo in AbstractDateAssert#isBeforeOrEqualsTo method name?

Summary
AbstractDateAssert#isBeforeOrEqualsTo has a trailing 's' in 'Equals' and it looks like a typo compared to java 8 time assertions (same for AbstractDateAssert#isAfterOrEqualsTo). It is just a style issue and fixing it would be a backwards incompatible change.
Also org.assertj.core.error.ShouldBeBeforeOrEqualsTo and org.assertj.core.error.ShouldBeAfterOrEqualsTo have the same flaw.
Example
assertThat(new Date).isBeforeOrEqualsTo(anotherDate)
assertThat(Instant.now()).isBeforeOrEqualTo(anotherInstant)

Yeah, good catch :man_facepalming: .
The way to go is is deprecate isBeforeOrEqualsTo / isAfterOrEqualsTo and add  isBeforeOrEqualTo / isAfterOrEqualTo. For the other typos a rename refactoring will do.


Question: How to extend/copy array schema but only change the containing type

Hello
I have a question regarding arrays.
Given following simple example
const myObjSchema = z.object({
  name: z.string(),
});

const arrayOfMyObj = z.array(myObjSchema).min(1).max(2);

What i wanna do is kind of changing the containing type of the array like this.
const arrayOfMyObjExt = arrayOfMyObj.changeType(myObjSchema.extend({ foo:z.string()}));

but i want to keep the constraints of min and max of the original array.
Why do i need this?
I want to extend and existing schema with some OpenAPI documentation and i do not have access to the original
schema definitions. I want to keep as much from the original schema as possible to not run out of sync.
For z.object i can use extends but there is nothing like this for z.array.
Any hints?
Thank you so much for help.

Maybe you can try something like this:
const arrayOfMyObj = z.array(myObjSchema).min(1).max(2);

const newSchema = arrayOfMyObj.and(
  z.array(
    z.object({
      foo: z.string(),
    })
  )
);

Transforms original schema into an Intersection type.


Reload user when his name was changed

Hi,
When a user change his name how can't his opponent to know that name was changed and load him.
Currently I load all users with theirs ids when I log in with
- (BFTask QB_GENERIC(NSArray QB_GENERIC(QBUUser *) *) *)getUsersWithIDs:(NSArray QB_GENERIC(NSNumber *) *)usersIDs;
The problem that this method only load users which is not already cached BUT all my users are cached so it doesn't reload them.
Thanks.

Hi @mikeRozen
We will implement such functionality in a near future (something like foreceReload flag). For now you can either clear your cache and load users again or implement your own method, that will always load users from server and cache them even though they were already cached.

Thanks.

@mikeRozen  Added in v 0.4.2.


Add Papa Bakary Camara into owner list

Add Papa Bakary Camara into owner list

Fixed in https://github.com/RedHatInsights/insights-results-smart-proxy/pull/424

Fixed in https://github.com/RedHatInsights/insights-results-smart-proxy/pull/424


 height not recalculated on window resize while the card is being open

Expected Behavior
When changing the width of the window, the height of the expandable card should adapt to fit its content
Current Behavior
The height of the card stays static on resize and so if the card's child height changes, the content gets hidden
Steps to Reproduce
Expand the card and then resize the window.
Context (Environment)
using orbit 0.17.1

Solved in #689


Allow additional text following formatter pragma comments

I'm not sure if we should do this, but I was curious what it would look like to implement.
Closes https://github.com/astral-sh/ruff/issues/8874

Can you double check if we the new implementation is flexible enough to cover Black's preview style https://github.com/astral-sh/ruff/issues/8892

While I agree we should allow pragma comments to follow other comments e.g. # foo # fmt: skip or #foo; fmt: skip, I don't think this implementation will cover those cases and we'll need to do something a little more involved. Additionally, I don't think the semicolon separated format is supported by our linter pragmas e.g. # foo; noqa and we should probably implement support for that in both at once?
Additionally, the syntax proposed here is actually different than Black's preview style. Black does not allow a reason unless it is separated by a ;  or # :
# valid
foo   = 1  # fmt: skip; reason
foo   = 1  # noqa; fmt: skip; reason
foo   = 1  # fmt: skip; noqa
foo   = 1  # fmt: skip # reason
foo   = 1  # fmt: skip # reason # noqa
foo   = 1  # reason # fmt: skip

# invalid
foo   = 1  # fmt: skip - reason; noqa
foo   = 1  # fmt: skip - reason
foo   = 1  # fmt: skip reason

I'd be okay with requiring a ;  separator, I guess.

I don't see why we should gate this behind a preview flag

Agreed, unless we are unsure it is the right syntax to support.

In light of #8892 I think the goal here should be to determine if we want to allow trailing text after pragma directives. I think this question can be considered separately from multiple pragma directives e.g.
foo   =  1  # noqa - some reason; fmt: skip - other reason

This seems like a valid use-case?

I think I like this overall. It might be nice to start by sticking to Black's syntax here. It does look more conservative right? In that we could expand it something more liberal like you have here later?

I think this question can be considered separately from multiple pragma directives e.g.

My understanding here is that if this PR is merged as-is, then a comment like # noqa - some reason; fmt: skip - other reason would today be allowed and be seen as just a noqa directive, right? But if you wanted to support that second fmt: skip directive, it would potentially change the meaning of what was once just free-form text to a pragma directive. I think a simple restriction on what's allowed in this PR would mitigate that concern? (If it's worth mitigating at all. It could be a non-issue.)

I think it's correct to allow arbitrary contents after the # fmt: on etc. It's also consistent with how we treat # noqa in the linter.


Add mill-scalafix and mill-git to the doc

Not sure if you want to add these but they seem core enough to me to have them in the doc.

Yes, we want to collect (all) thirdparty plugins here, at least a pointer to them. Can you please insert the new modules in alphabetical order?

Done

Thank you!


Fixes #6108 Updated version checking statements in Ansible scripts

Using version-comparison feature of Ansible to compare the versions.
https://docs.ansible.com/ansible/latest/user_guide/playbooks_tests.html#version-comparison
Verified that it is working on both Ansible 2.4 and Ansible 2.5

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bingoarun
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ijsnellf
Assign the PR to them by writing /assign @ijsnellf in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @bingoarun. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bingoarun, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test

@bingoarun: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bingoarun: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bingoarun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bingoarun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test e2e-bookInfo

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bingoarun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bingoarun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bingoarun: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test e2e-bookInfo

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
99f0b8fc66ff28153b5f9d64b73b7410a2856b40
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

FWIW: I've had a PR fixing this issue waiting for review/merge for over 3 weeks :)
You need to also fix the regex.. See: PR #5710
(I want to know how you got this reviewed and merged so quick ;-)


Docextract improvements

Originally
We can now check for existing tickets for a given recid. Which means we can always create tickets for extracted references by making sure there is not existing ticket.

Given this is very INSPIRE specific and the overall legacy code base freeze I am closing this issue.


[BUG] No downloads available

Describe the bug
I downloaded hydralauncher-2.0.2-setup.exe from releases and installed launcher (I have never installed Hydra on my PC before). After this I opened launcher and created new account. After that, "No downloads available" is written for all games.
Steps to Reproduce

Download hydralauncher-2.0.2-setup.exe and run it
Create new account

Expected behavior
It doesn't look like there aren't downloads for absolutely all games.
Screenshots

Operating System
Windows 10
Hydra Version
2.0.2 "Leviticus"
Additional Information
No response

you gotta add the sources yourself using this
https://hydralinks.cloud/


change file structure for node related files

I've noticed that there 4 different files starting with node the html5_parser module. This seems like it could be grouped into a folder called node or nodes.
I would suggest something like this:
html5_parser/
├─ .../
├─ node/
│  ├─ data/
│  │  ├─ comment.rs
│  │  ├─ document.rs
│  │  ├─ element.rs
│  │  ├─ mod.rs
│  │  ├─ text.rs
│  ├─ arena.rs
│  ├─ mod.rs

If you want I could create a pr for this?

I introduced these files in one of my PRs. Im good with this structure, although note that we moved away from the mod.rs convention (see #102). I can’t remember if I faced a challenge with this or if I just didn’t try.
If you can make it work, then sounds good to me

Oh, I was only paying attention to the node_data files, this is for all node related files. Either way, if we can support a better structure with the newer convention we’re using, then that’ll probably be good

so something like this?
html5_parser/
├─ .../
├─ node/
│  ├─ data/
│  │  ├─ comment.rs
│  │  ├─ document.rs
│  │  ├─ element.rs
│  │  ├─ text.rs
│  ├─ arena.rs
│  ├─ data.rs
├─node.rs

Yeah, something like that would likely work. Although since this is also changing the structure of all the node files, I'd also request review from @jaytaph && @emwalker in your PR


python312Packages.jsonargparse: 4.28.0 -> 4.29.0

Diff: https://github.com/omni-us/jsonargparse/compare/refs/tags/v4.28.0...v4.29.0
Changelog: https://github.com/omni-us/jsonargparse/blob/4.29.0/CHANGELOG.rst
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 314376 run on x86_64-linux 1

4 packages built:

python311Packages.jsonargparse
python311Packages.jsonargparse.dist
python312Packages.jsonargparse
python312Packages.jsonargparse.dist


fix: new fullscreen settings for many-screens setups

adds some index options to specify which monitor should show fullscreen windows

remove the confusing "swap displays" toggle

the 2 new options can be Enums too, thus a bit tricky to populate comfortably (advise if needed)

should we re-address (basically toggle it off and on) the screen when setting is updated in order to update immediately ? or should this be a user action ?

<3

it (should) look like this 🌞 :

(vlc for ref) :

Hi @ylmrx could you rebase the PR to the latest dev commit?

there you go @pixtur !
I set it to hide the mainmenubar and the timeline when in focus mode too.

My focus mode fix was breaking, but i figured the confusing imgui.menuitem strange behavior...
We can now have an "checkmarked" menu entry , which still take its bool return into account by removing "ref", now the if gets correctly executed, and the focus mode works as intended from both the key shortcut, and the menu entry (which also get its correct status)

Thanks for your contribution!


Sage Feature Toggle - Add secondary link

Description
This PR adds a secondary link option to the Sage Feature Toggle.
This work is being requested as a part of COMM-671.
Screenshots

Before
After

Test notes

Steps for testing

Please see https://github.com/Kajabi/kajabi-products/pull/17612 for testing information.

@joedakroub it would be great to update the props file to match your settings and possibly also show another example in preview of the additional option you've added.


Build failure

Two problems:
The first problem: ldlibs missing
d source/plugin/wiimote; make -j3 --jobserver-auth=3,4 plug.a
make[1]: warning: -j3 forced in submake: resetting jobserver mode.
make[1]: Entering directory '/home/hwkiller/Builds/AUR/moltengamepad/src/MoltenGamepad-95b0d472770360bacbc4286fce13e0c1511ebbdb/source/plugin/wiimote'
mv -f .d/source/core/devices/generic/generic_file.Td .d/source/core/devices/generic/generic_file.d
cd source/plugin/steamcontroller; make -j3 --jobserver-auth=3,4 plug.a
make[1]: 'plug.a' is up to date.
make[1]: Leaving directory '/home/hwkiller/Builds/AUR/moltengamepad/src/MoltenGamepad-95b0d472770360bacbc4286fce13e0c1511ebbdb/source/plugin/wiimote'
cd source/plugin/joycon; make -j3 --jobserver-auth=3,4 plug.a
make[1]: warning: -j3 forced in submake: resetting jobserver mode.
make[1]: Entering directory '/home/hwkiller/Builds/AUR/moltengamepad/src/MoltenGamepad-95b0d472770360bacbc4286fce13e0c1511ebbdb/source/plugin/steamcontroller'
make[1]: warning: -j3 forced in submake: resetting jobserver mode.
make[1]: Entering directory '/home/hwkiller/Builds/AUR/moltengamepad/src/MoltenGamepad-95b0d472770360bacbc4286fce13e0c1511ebbdb/source/plugin/joycon'
make[1]: 'plug.a' is up to date.
make[1]: Leaving directory '/home/hwkiller/Builds/AUR/moltengamepad/src/MoltenGamepad-95b0d472770360bacbc4286fce13e0c1511ebbdb/source/plugin/steamcontroller'
make[1]: 'plug.a' is up to date.
make[1]: Leaving directory '/home/hwkiller/Builds/AUR/moltengamepad/src/MoltenGamepad-95b0d472770360bacbc4286fce13e0c1511ebbdb/source/plugin/joycon'
mv -f .d/source/core/protocols/socket/socket_out.Td .d/source/core/protocols/socket/socket_out.d
ar rcs source/core/mg_core.a source/core/commandline.o source/core/config_parse.o source/core/main.o source/core/moltengamepad.o source/core/options.o source/core/output_slot.o source/core/parser.o source/core/plugin_loader.o source/core/profile.o source/core/slot_manager.o source/core/udev.o source/core/uinput.o source/core/commands/assign.o source/core/commands/clear.o source/core/commands/load.o source/core/commands/move.o source/core/commands/print.o source/core/commands/save.o source/core/commands/set.o source/core/devices/device_manager.o source/core/devices/input_source.o source/core/eventlists/eventlist.o source/core/eventlists/gamepad_event_list.o source/core/protocols/debug_messages.o source/core/protocols/message_stream.o source/core/protocols/ostream_protocol.o source/core/virtual_devices/virtual_device.o source/core/virtual_devices/virtual_gamepad.o source/core/virtual_devices/virtual_kbm.o source/core/devices/generic/generic_device.o source/core/devices/generic/generic_file.o source/core/devices/generic/generic_manager.o source/core/devices/generic/generic_parse.o source/core/event_translators/axis/axis2axis.o source/core/event_translators/axis/axis2btns.o source/core/event_translators/axis/axis2rel.o source/core/event_translators/button/btn2axis.o source/core/event_translators/button/btn2btn.o source/core/event_translators/button/btn2rel.o source/core/event_translators/general/multitrans.o source/core/event_translators/general/redirect_trans.o source/core/event_translators/group/calibratable.o source/core/event_translators/group/exclusive_chord.o source/core/event_translators/group/simple_chord.o source/core/event_translators/group/stick_dpad.o source/core/event_translators/group/thumbstick.o source/core/event_translators/group/wiigyromouse.o source/core/protocols/socket/socket_in.o source/core/protocols/socket/socket_out.o
cat: source/plugin/wiimote/ldlibs: No such file or directory

This stems from source/plugin/wiimote/Makefile::$(shell cat ldlibs)
ldlibs does not exist.
After I just added a blank ldlibs, another error emerged.
Second problem: Undefined references
/bin/ld: source/core/mg_core.a(eventlist.o): warning: relocation against `key_list' in read-only section `.text'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `get_key_name(int)':
eventlist.cpp:(.text+0xb3): undefined reference to `key_list'
/bin/ld: eventlist.cpp:(.text+0xbf): undefined reference to `key_list'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `get_axis_name(int)':
eventlist.cpp:(.text+0x113): undefined reference to `axis_list'
/bin/ld: eventlist.cpp:(.text+0x11f): undefined reference to `axis_list'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `get_rel_name(int)':
eventlist.cpp:(.text+0x143): undefined reference to `rel_list'
/bin/ld: eventlist.cpp:(.text+0x14f): undefined reference to `rel_list'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `get_key_id(char const*)':
eventlist.cpp:(.text+0x1c3): undefined reference to `key_list'
/bin/ld: eventlist.cpp:(.text+0x1ca): undefined reference to `key_list'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `get_axis_id(char const*)':
eventlist.cpp:(.text+0x263): undefined reference to `axis_list'
/bin/ld: eventlist.cpp:(.text+0x26a): undefined reference to `axis_list'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `get_rel_id(char const*)':
eventlist.cpp:(.text+0x2bc): undefined reference to `rel_list'
/bin/ld: eventlist.cpp:(.text+0x2c8): undefined reference to `rel_list'
/bin/ld: source/core/mg_core.a(eventlist.o): in function `lookup_event(char const*)':
eventlist.cpp:(.text+0x363): undefined reference to `key_list'
/bin/ld: eventlist.cpp:(.text+0x36a): undefined reference to `key_list'
/bin/ld: eventlist.cpp:(.text+0x3f3): undefined reference to `axis_list'
/bin/ld: eventlist.cpp:(.text+0x3fa): undefined reference to `axis_list'
/bin/ld: eventlist.cpp:(.text+0x42c): undefined reference to `rel_list'
/bin/ld: eventlist.cpp:(.text+0x433): undefined reference to `rel_list'
/bin/ld: warning: creating DT_TEXTREL in a PIE
collect2: error: ld returned 1 exit status
make: *** [Makefile:77: moltengamepad] Error 1

[System] Arch linux 64

About ldlibs: that error should be nonfatal, and the build completed for me on a fresh clone. It is not ideal, and a blank file would do the correct thing, so lets go ahead and put those files for plugins that do not need to link any additional libraries.
As for the undefined references: it looks like something failed in producing key_list.o and the other _lists, or otherwise are not available during linking. Might there be an earlier error? Those key lists do tend to depend on what key codes you have defined in your kernel headers, but I would imagine the versions in the repo would suffice for Arch 64. I am on Arch 64, and the repo built fine on my end from a fresh clone.
Can you try make clean; make eventlists;  make?

Ah, that was it. make eventlists wasn't called. Thank you.
I may make a new AUR package for this, because it is currently missing some deps and makedeps; it was also last updated 2 years ago.
I just have one question about it - Is there a reason for trying to disable all other controllers and disabling uaccess? SC-controller and steam both use uniput and uaccess. Is it possible for me to just use a simplified udev rule where uaccess is used to manage the evdev nodes, and otherwise to create the permissions needed for the wii controllers and such? The reason is, that I have other controllers too (x360, steam controller) that I don't necessarily want to manage via MG. Ideally, MG would just handle the wiimotes as configured, and not touch other controllers.
We can close this issue thread and talk elsewhere if you would like. If I make an updated AUR package, I would like a way to ask you questions if possible.


"You must authenticate"

Hi!
When I use the "Download plugin data" function, CheckDLC always gives this error message after a few minutes:
"Use saved data for Steam [25.10.2023 10:24]. You must authenticate with "Authenticate" button."
When I click on this message, it takes me to Steam library integration page, and it says "User is authenticated." I tried authenticating a few more times just to make sure, but the error message keeps appearing every time I use the function. My Steam profile is public.
Because of this error, but some games' DLCs are not recognized. I need to click on CheckDLC->Refresh Data for those games, and only then I can see the "View game DLC" button.

Same authentification error with Steam

Same authentification error with Steam
Thanks

Can you try with the last version?

No problems now, thanks.

That's #82, I'd assume everyone gets that currently


[vue ui] Run tasks in vue ui, it adds --mode flag even I already have one in script command.

Version
3.0.0-beta.16
Reproduction link
https://codesandbox.io/s/yp18qom2kx
Steps to reproduce
1. Already have a --mode flag in script command.
// package.json
{
  "scripts": {
    "build:dev": "vue-cli-service build --mode build-dev"
  },
  // ...
}

2. Run the task in ui will get this:
$ vue-cli-service build --mode build-dev --mode development --dest dist --target app --dashboard

It still adds a --mode flag even the command has one.
What is expected?
Use the correct mode value from command --mode.
What is actually happening?
Have two --mode in the command.

Shouldn't each Task check for existing flags before pushing new flags to the args array?
For this particular case, the implementation of onBeforeRun could check if each of the flags it's pushing already exists before trying to push them.

What if you want to override an argument with the UI?

I think having a checkbox on the UI to decide if we want to override arguments or not could be a good option. What do you think @Akryum?
Or maybe overriding arguments with the UI should be the correct behavior?

I think putting checkboxes everywhere would really clutter the UX.

@Akryum  I already install the rc5.
In the UI tasks, it can remove the duplicate arguments now, but how could I use my own --mode value ?
I have some custom environment value in the .env.build-dev file.

I've added (unset) options in the mode selects.


Added 'next-inferno' to community plugins

This PR contains similar to next-preact plugin for page rendering with very fast Inferno library. Now as community plugin 👌
Also I set community plugin in alphabetical order.

Looks great, thank you @queses!


update deprecated summary functions

Updates deprecated summary functions throwing errors for release of tensorflow 1.0

Thanks.  TensorFlow's 'Update Script' needs improvement.


DateTime Parser with granularity

Coverage decreased (-0.2%) to 53.076% when pulling 4574477ed1c3534c8bfd38c86ddc6ae159c08492 on compae:feature/date-Field-options into 6b3638780c02e90bfdb635176cc10090a023fce3 on Stratio:master.

:shipit:

A job will handle this PR ASAP on Jenkins

Rebase && testing in progress


Add getpeereid function

This is a wrapper around getsockopt() for getting the uid/gid of a remote Unix domain socket peer. It was added in FreeBSD 4.6 and present in all modern BSDs I checked (including Mac OS X).

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 4e6dd24 has been approved by alexcrichton

:hourglass: Testing commit 4e6dd24 with merge 89c6c6c...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 89c6c6c83eb0914207ed29ba5e40dd535165cdc6 to master...


Enable media streaming

Should address  https://github.com/alanmcgovern/monotorrent/issues/257 well enough for now

@OneFingerCodingWarrior I've (hopefully) simplified things and am in a place where I'm comfortable shipping this in the next release!
I would love it if you could give this new API a shot and let me know if it works well for you. If you have any suggestions to make it easier to work with let me know! If it fails miserably then I really want to know what went wrong :p

@ilyalatt @uponatime2019 Can you take a read my previous comment on this PR. If would be great if you could test things and let me know how you get on. I'd love to release this as long as there are no show-stopper issues! I won't make breaking changes to this API until i release a 'MonoTorrent 2.0.0' :)

Well , I will try the new API soon or later.
In fact I also implemented a simple http media streaming function using wcf, but because my code style (well ,you have met it ^_^) and dependency of WCF conflict with MonoTorrent, and unfamiliar with git, so I didn't push the changes. But  I am pretty happy to switch to your new API, and thanks for all the work you have done,

In fact I also implemented a simple http media streaming function using wcf

My hope is that anyone who wants to expose the data in a custom service/whatever can simply call StreamProvider.CreateStream and then read any data they need from that! They will be able to seek that stream and read data from it and MonoTorrent will magically ensure that data is prioritised/downloaded!

I went ahead and merged this! I'll test it over the next few days before issuing a release with this! There is a small chance I'll make some tweaks between now and then, but i'd say it's unlikely :)


SSL Certficates as PEM strings with Schema Registry config

Hi,
Currently we are connecting to Kafka and Schema registry (for Avro) by passing Certificate locations for SSL authentication and we are working with .Net applications.
We are planning to use certificates from KeyVault by passing certificates as PEM strings, we observe that Confluent.Kafka (2.2.0) have the option for Kafka configuration to pass certificates as PEM strings.( https://docs.confluent.io/platform/current/clients/confluent-kafka-dotnet/_site/api/Confluent.Kafka.ClientConfig.html#Confluent_Kafka_ClientConfig_SslCaPem).
But we are not seeing any option to pass these certificates as string to Schema registry configuration.
Is there any option to pass these certificates as strings to the Schema registry OR
Is there any option other than saving certificates on local and passing that location to Schema registry config ?

I am also interested in this


The process to use this tool on a private cloud is unclear

Description
The readme should clarify how to use this tool for private cloud instances.
Currently, the readme has this to say:
The authenticator of the CLI defaults to the default Auth0 cloud auth0.auth0.com. This can be customized for personalized cloud offerings by setting the following env variables:

	AUTH0_AUDIENCE - The audience of the Auth0 Management API (System API) to use.
	AUTH0_CLIENT_ID - Client ID  of an application configured with the Device Code grant type.
	AUTH0_DEVICE_CODE_ENDPOINT - Device Authorization URL
	AUTH0_OAUTH_TOKEN_ENDPOINT - OAuth Token URL

When I set these 4 env variables and run auth0 login, I am redirected to the finder app (I am running on a Mac) instead of the web browser. It is possible this is because I have the wrong values set and it is unable to start the device code process. Alternatively, there could be a bug with auth0-cli that will always redirect to finder if AUTH0_CLIENT_ID is set. See the conversation here: https://github.com/auth0/auth0-cli/issues/354
If I have the wrong values set, then it might be a good idea to clarify in the readme where these values can be found.
Alternative
Through trial and error, I did manage to get the CLI working by running these commands:
export AUTH0_CLI_CLIENT_DOMAIN=TENANT.PRIVATE_INSTANCE.auth0.com
export AUTH0_CLI_CLIENT_ID=CLIENT_ID_OF_M2M_APPLICATION_IN_TENANT
export AUTH0_CLI_CLIENT_SECRET=SECRET_OF_M2M_APPLICATION_IN_TENANT
auth0 config init

This created ~.config/auth0/config.json and I was then able to run additional auth0 commands!
If this is a valid method of setting this up, then I would suggest adding it to the readme.

Hi @andrewclaus, thanks for raising this.
The process documented in the README is the one intended for getting the interactive login to work (auth0 login) with a private cloud instance. As you mention, some users are getting an error when the AUTH0_CLIENT_ID is set, and we'll be looking into this.
The other process is for getting the CLI to work in non-interactive environments like a CI job, but is a valid workaround for the previous issue. We could document the non-interactive process in the README as well.
I'll leave this issue open to track that.

@andrewclaus thanks for opening this ticket and remaining patient. We agreed that we could provide a lot more guidance for authenticating to a private cloud tenant. Which is why we updated the README to include a warning (see below) about private cloud users needing to authenticate via client credentials; authenticating as a user with device flow will not work.
I consider this issue to be complete so I'm going to close, however, if there is any additional feedback or improvements that could be made here, please feel free to open a new issue. Thanks for your patience!


Add smkn7-smr.sch.id

Official school website: https://smkn7-smr.sch.id

@ExLog I am sorry for a delay with processing your request.
Pull request merged. Thank you!


Index doc

Write an index documentation file.

I decided to move the index document into the README.md. It was quite small and it makes more sense to have it available for reading immediately.


Provide a way to control debugger stop focus changes

VS Code 1.10.2
Windows 10 Anniversary

In the latest update of the PowerShell extension (0.11.0) we now use an integrated terminal instead of the Debug Console for interactive execution of commands.  PowerShell allows the user to type commands in the console to initiate debugger operations like "stepInto", "stepOut" and "continue".  When the user types such a command and presses "Enter", the focus jumps back up to the editor pane when the debugger stops at the next line or breakpoint.
I would really appreciate a way to tell the debugger client UI to not change the focus automatically so that the user can keep typing commands into the console without having to manually set focus back there.
Thanks!

/cc @isidorn @weinand

First I checked does it make sense in general to pass focus to the editor when stepping and I think it does. As a reference I also checked what Chrome debugger is doing and they also always pass focus to the editor.
However I understand your scenario and we could tackle it by either:

introducing a vscode settings e.g 'debug.autoFocusEditor' - the problem with this approach is if this setting will be useful to the users in general (I think not)
Introduce a field in the DAP stopped event which would specify if we should autoFocus editor - also not a fan of, since we rarely specify how the UX should behave in the protocol

I am fine with doing option 1 if the need for this is great or if at least somebody else will also profit.
@daviwil @weinand thoughts?

I agree that option 2 isn't ideal since the protocol should focus on the details of the debugging session, not the UX.  I don't think the debug.autoFocusEditor is ideal either because it would prevent me from turning that on by default for all PowerShell extension users (unless I automatically add it to their user settings, which I'd prefer not to do).
What about a third option, allowing this to be configured by the debugger contribution in my package.json?  I know there are no UX settings right now for the debugger contributions, but this seems like the most appropriate place to start adding them.

Since none of the sollutions appear particularly appealing to me I would wait for us to get at least one more use case for this before implementing something.
But let's also wait and see @weinand thoughts

This is also happening when I hit F8 to run the "current line" in the PowerShell Extension. The Integrated Terminal is capturing focus, so I have to click back into my code in order to continue working.

Closing because we are not planning to implement this within the next 12 months.


Handle push notifications while Live Bidding Interface is up

no pushes being generated as of yet so no QA has been done

done for now. Individual updates will be tackled with new issues/epics


iterator.js uses generator functions

iterator.js has generator function, which breaks the uses of this library on IE11

any news about this prob?

bump


User information in header lost on screen refresh

.controller('MainCtrl', function($scope, $rootScope, $state, $location,  CoreService, User, gettextCatalog, AppAuth) {
AppAuth.ensureHasCurrentUser(function(user)
{
  $scope.currentUser = user;
});

Needs to be changed to
.controller('MainCtrl', function($scope, $rootScope, $state, $location,  CoreService, User, gettextCatalog) {
$scope.currentUser = User.getCurrent();

Thanks @palmerjc!
It looks like it's a regression cause by this commit https://github.com/beeman/loopback-angular-admin/commit/7497639eaf9028988c587dd928214e156313ae06
@dthib could you maybe find time to fix this?

Make a PR to solve this based on @palmerjc  work.

That's exactly what I changed because it was not working with social authentication :)
I guess you are working with normal login?
Best,

If you dont use social networks still works with this code.
Try to reload, ones you get logged in.

This issue is cause by /client/app/modules/users/services/appauth.service.js
The ensureHasCurrentUser Method
$cookies.access_token is always false.
change all "$cookies.access_token" to "$cookies.accessToken" solves this issue

Thanks for the contributions guys.
This should be fixed in master, @palmerjc could you please verify this?
@dthib can you please verify if the social login now works on master too?

Thanks for the fix! OK I will check that during the week.
Best,
Thibaut
Le 9 août 2015 04:00, "Bram Borggreve"<EMAIL_ADDRESS>a écrit :

Thanks for the contributions guys.
This should be fixed in master, @palmerjc https://github.com/palmerjc
could you please verify this?
@dthib https://github.com/dthib can you please verify if the social
login now works on master too?
—
Reply to this email directly or view it on GitHub
https://github.com/beeman/loopback-angular-admin/issues/96#issuecomment-129089186
.

Hi again,
I have tested it and I don't think it's better:

The social login it's not working anymore since User.getCurrent if still called and undefined is returned for social connexion (any idea of the reason for this behavior?)
If i put back ensureHasCurrentUser, the connexion is  not working anymore because of the change   access_token to accessToken

If you have any idea...
Best,
Thibaut

Hi
The problem still persists.
Social logins not working ( using ensureHasCurrentUser() and also User.getCurrent() ) with and without accessToken changes.
My observation is

access token is available as "access_token" only.
the http call to "auth/current" from ensureHasCurrentUser is looking for client web server path ( http://localhost:9000/auth/current ) whereas its available at server ( http://localhost:3000/auth/current ). Hence the current user data is not available, hence the page refresh is always redirecting to login page even post login ( And so do the social login redirect ).

Appreciate if there is a quick fix available for this.
Thanks

I think this bug should be fixed by merge https://github.com/beeman/loopback-angular-admin/pull/135.

Should be fixed. If not, please reopen the issue with some steps to reproduce. Thanks!


⚠ Bofrost.es has degraded performance

In 8ec7624, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 3386 ms

Resolved: Bofrost.es performance has improved in 9b38d13.


Colorguess_game

PR Description 📜
"Color Guess" is a simple game that tests the player's ability to recognize colors based on their RGB values. The game presents the player with a set of color options, one of which matches the displayed RGB values. The player needs to select the correct color to win the game.
Fixes #962

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @prachisingh1008 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit please review my PR request

follow proper guideline and create a PR again !!

Thank you  @prachisingh1008 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Make max-readahead configurable to enhance read performance

Currently, readAheadMax is a non-configurable parameter for fuse users, set by default to conf.Chunk.BlockSize * 8, typically 32MB. This isn't sufficient for low-concurrency read requests. I intend to make max-readahead a user-configurable parameter.
After testing, I set max-readahead to 512MB and observed that under single-threaded, with uncached local disk reads, the throughput increased from 230MB/s to 2.2GB/s, showing a 10x improvement.

Thanks for the PR. Maybe also document it (in docs) too?


cli prints warning about wrong kernel versions

Problem description
Tools are throwing a warning about an unusual kernel version.
Expected behaviour
No error message if the kernel has not changed
Actual behaviour
Warning:
detected an unusual remote kernel file update for source 'https://downloads.vorteil.io/kernels' on kernel '20.10.5'

Resolved with 9d48d47c76768bbc105a1e80d5a5bf9fe18eb509

Resolved with 9d48d47c76768bbc105a1e80d5a5bf9fe18eb509


fix: removes react-native prop from package.json

Description

removes react-native prop from package.json

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


pretty-print JSON in Vaultfile

Pretty-print JSON in Vaultfile, that will help users that want to "cat" the Vaultfile and also when running "diff", "patch", manually reviewing it, etc.

Good idea.


vendor: update govmm and agent to support memory update

vendor: update govmm and agent to support memory update
Fixes #671

To support kata-runtime update memory, need separate CPU/memory in agent grpc handler
and handler query-memory-devices QMP commands in govmm.

1.Implement query-memory-devices QMP commands(govmm).
2.Add "CpuOnly" field to onlineCPUMem grpc request(agent).

govmm Shortlog:
b16291c qemu/qmp: support query-memory-devices qmp command.

agent Shortlog:
74a5364 build: fix make proto error
b1c2ad8 agent: add support for online memory and cpu separately.

Signed-off-by: Zichang Lin<EMAIL_ADDRESS>Signed-off-by: Clare Chen<EMAIL_ADDRESS>

Related to #624

lgtm but build failing. From Travis:
../virtcontainers/qemu.go:862:53: not enough arguments in call to q.qmpMonitorCh.qmp.ExecuteNetPCIDeviceAdd
	have ("context".Context, string, string, string, string, string)
	want ("context".Context, string, string, string, string, string, int)

CI is failed. I think there are some problems in CI environment, unrelated to my change.

We have a issue track this failure. https://github.com/kata-containers/tests/issues/680

This is duplicate of https://github.com/kata-containers/runtime/pull/649

@WeiZhang555 This PR need govmm and agent update. And the govmm update is newly that #649 . Two choose:

Merge #649 first, then I make a rebase on this PR.
Close #649 , just merge this PR.

PLZ rebase this. #649 is just going to be merged~

yep - #649 is now merged so this needs a rebase.

Don't merge,  #624 need more agent.pb update.

If so, I think this can be a commit of PR #624

I agree with @caoruidong.

Combine this into #624 . Close.


Typos in Readme file

Hi,
I observed few typos in readme file that can be improved, let me know if I can work on that?

Go for it! ☺

Hi @origranot ,
I have fixed the typos in the readme file, have a look at it https://github.com/origranot/url-shortener/pull/153


Added VQE and PSI4toBroombridge samples

Implemented a VQE as described by Peruzzo, et. al. Added conversion code from PSI4 to Broombridge. Made with love in Seattle. :)
Main VQE Code: Christopher Kang<EMAIL_ADDRESS>Ansatz preparation + classical parameter selection: Rob Minneker<EMAIL_ADDRESS>PSI4 to Broombridge conversion: Julian Roth<EMAIL_ADDRESS>

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

@christopherkang I'd love to move forward with this sample, would you like me to close this PR and reopen on a new branch or did you want to have another look?

Hey @guenp! The VQE code here may be obsolete given the VQE code here: https://github.com/microsoft/QuantumLibraries/blob/main/Chemistry/src/Runtime/JordanWigner/JordanWignerVQE.qs What do you think?
As for the PSI4toBroombridge code, do any comparables exist?
(Apologies for my hazy memory! This was a group project from my freshman year)


Zero price at listings ?

I'm not sure if it's legit or if it's the same bug as #3533 ?
Seems I've bought NFT for 0 BSX
https://beta.kodadot.xyz/bsx/gallery/3060885173-9

0 BSX at listing
https://beta.kodadot.xyz/bsx/gallery/3060885173-7?page=1

I'm not sure if it's legit or if it's the same bug as #3533 ?

I think it's the same. @kkukelka said it should be fixed by snek rather than front-end 👀

Maybe some types have changed on their side, yeah.

@yangwao it's the same "bug". not actually a bug, but the prices of the NFTs come back in the wrong format. our format function cannot work with current value as it expects 12 decimals

our format function cannot work with current value as it expects 12 decimals

Does it seem there are 12 decimals?
src https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Fbasilisk-kodadot.hydration.cloud#/settings/metadata

our format function cannot work with current value as it expects 12 decimals

Does it seem there are 12 decimals? src https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Fbasilisk-kodadot.hydration.cloud#/settings/metadata

the problem is we get different BSX values at different parts in the app. Like for example profileStatsById returns me meta:<PHONE_NUMBER>0000000 which is then formatted (by our helper function) correctly to 10,000 BXS. But when fetching like a GalleryItem with nftById it return me price: 10000 which is already formatted in a way
maybe @vikiival has an idea

snek indexer always keeps price in the original format ->
so if you get price:  10000 it means that either someone:

[ ] put price 0,00000001 (10k / 10 ** 12)
[ ] put price 10k and we have bug in the component

our format function cannot work with current value as expects 12 decimals

Let me make it correct -> our format function rounds to two decimal places so 0.001 is 0

For those kind situation, maybe we can convert 0.000001 -> 0.051

👋

hi @vikiival
if I LISTing at 10,000 BSX, What I should pass to transaction function would be 10,000 * (10**12)?


feat(ui): export modal for all the entities

Describe your changes:
#10021

Worked on centralizing export functionality
Integrated export functionality in teams page

https://github.com/open-metadata/OpenMetadata/assets/71748675/03b0f56c-8f00-4e56-ab73-1cc58710d94c

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Hello, @Sachin-chaurasiya comments have been addressed!

Hello, @Sachin-chaurasiya comments have been addressed!

Can we change the component file name as well?


Community steps

i approved but didnt merge so you have a chance to respond to the roletypeid comment. i'm fine with it as is if you think it's best to keep it this way, so feel free to merge.


Update version in readme example

Noticed the version is outdated.

cargo-release can help you take care of this, if desired :)


Release of semantic parser

Congrats on the work so far! Will the semantic parser specified in your blogpost be made available?

Please see the Neuro-Symbolic Toolkit for related open source packages. I believe you are referring to the AMR-to-Logic component in particular.


Fix ruamel.yaml dependency

The current dependency on ruamel.yaml causes an infinite loop when resolving dependencies using Poetry. ruyaml is a drop-in replacement.

Hi @Fuyukai,
Are you actually using the sphinx_toolbox.pre_commit extension? That's the only part that requires ruamel.yaml.
If you're not, I can make ruamel.yaml an optional dependency, so it will only be installed if required, and then you won't see the endless loop.

Hi @Fuyukai,
Are you actually using the sphinx_toolbox.pre_commit extension? That's the only part that requires ruamel.yaml. If you're not, I can make ruamel.yaml an optional dependency, so it will only be installed if required, and then you won't see the endless loop.

Hi @domdfcoding
I just came upon this. I also have an issue with ruamel.yaml, but the problem comes from the fact that in the requirements.txt it is listed as ruamel-yaml. This causes an issue with Poetry as you can see in this ticket:
https://github.com/python-poetry/poetry/issues/3200
Can we change the requirement to be ruamel.yaml and not ruamel-yaml?

Opened up a new PR at https://github.com/sphinx-toolbox/sphinx-toolbox/pull/76